Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Map Filter search by resource layer feature #10827
Map Filter search by resource layer feature #10827
Changes from 23 commits
11d1b5c
bc4bf3a
a99651f
1eebc61
074c511
1d01240
31ef333
cf05766
54281be
0a2f425
5ab58d6
1b1a573
2922ce5
ea387b5
f075998
1c4d836
f5181b1
2a9266d
4095957
3c9878f
805693e
dfb65e3
cb3140d
f9fac10
37a5d45
4dc361a
57d6742
56b61de
dcbe236
6afe8ca
f8e4f48
eab6138
9dda454
a6de407
24a7299
1674617
820a911
387752c
89e64d1
ae5b220
2fb80c9
bcab5d0
91b1e98
8868cbd
fcbfcd8
cecdc2c
ddc814c
6b67f99
53918a1
e13e233
f30d648
e1ea2f3
f8b0ba5
f18555c
52aad84
9782f7f
5f6717d
ede08dd
ae1adfb
aa2fba0
84a16ac
5672e47
43a3e4f
dc23575
d85c4d2
dea8266
3d84a0f
0b735eb
43348ae
75acb85
54d79d5
d0da3dd
9b67cbc
a90dfaa
98e92dd
3665ef0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be stored in
this.filter.feature_collection
? Storing the feature in a different variable is causing an exception on line 545:if (this.filter.feature_collection().features.length > 0) {
:Uncaught (in promise) TypeError: can't access property "length", mapQuery.features is undefined
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the popup feature lacks a resourceid, its geometry does get stored in
this.filter.feature_collection
here.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a change in my latest commits so that
restoreState
will work as expected, drawing a buffered polygon on the mapThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm finding that if a feature geometry filter is applied, the search is refreshed, and then the
Map Filter Enabled
search criteria is removed, the filter isn't removed from the URL and the filter remains applied.Adding another
Filter by Feature
then shows a Request Failed banner, and continues to add geometries to the map filter.