Skip to content
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

Support layer group filtering #5250

Merged
merged 2 commits into from
Jan 9, 2020
Merged

Support layer group filtering #5250

merged 2 commits into from
Jan 9, 2020

Conversation

adube
Copy link
Contributor

@adube adube commented Oct 24, 2019

The main goal of this patch is to introduce the support of filtering a WMS layer that uses a group as layer name.

To accomplish this, the following change was made.

The creation of attributes within an OGC data source object is made by first checking if there are attributes set for the layers used by the data source in the inner ogcLayers_ property. Then, they are combined together, i.e. only the attributes that are commonly shared in all layers are kept.

From there, the rest works as usual.

Known issue

There is an issue with this approach. To resume: it is currently not supported by MapServer to give the name of the group in the LAYERS parameter while giving only one filter in the FILTER property. Therefore, this patch breaks the MapServer WMS layers in those cases.

More about this issue in GSGMF-1133.

The above issue has been fixed in MapServer: MapServer/MapServer@ba27152

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adube adube force-pushed the gsgmf-1133-layer-group-filters branch 5 times, most recently from fa7934d to 6eaa347 Compare October 30, 2019 14:59
@sbrunner sbrunner added this to the 2.5 milestone Nov 4, 2019
@adube adube force-pushed the gsgmf-1133-layer-group-filters branch from 6eaa347 to e7d8a88 Compare December 23, 2019 13:28
@adube
Copy link
Contributor Author

adube commented Dec 23, 2019

The patch on MapServer has been merge. Please, write a comment here once the GMF demo server has its MapServer version up-to-date.

@adube adube force-pushed the gsgmf-1133-layer-group-filters branch from e7d8a88 to 712c74f Compare January 7, 2020 15:12
@adube adube changed the title [WIP] Support layer group filtering Support layer group filtering Jan 7, 2020
@adube adube requested review from sbrunner and fredj January 7, 2020 16:44
@adube
Copy link
Contributor Author

adube commented Jan 7, 2020

@sbrunner or @fredj : Ready for review.

@sbrunner
Copy link
Member

sbrunner commented Jan 7, 2020

Any reason why we don't have the geom filter?

@sbrunner
Copy link
Member

sbrunner commented Jan 7, 2020

When we click on get data we should only have the features from the layer Hotel (trigged with the GetData metadata...)

@adube
Copy link
Contributor Author

adube commented Jan 7, 2020

When we click on get data we should only have the features from the layer Hotel (trigged with the GetData metadata...)

Here's a snippet from the JSON returned by the themes service:

{
  "maxResolutionHint": 999999999,
  "minResolutionHint": 0,
  "name": "hotel_label",
  "queryable": true
},

To my understanding, the label layer returns features because it is queryable.

If that's an issue, then it should be logged as a separate task in JIRA, as it is not related to this PR.

@adube
Copy link
Contributor Author

adube commented Jan 7, 2020

Any reason why we don't have the geom filter?

That's an issue intruduced in this PR. I'll look into it.

@adube adube changed the title Support layer group filtering [WIP] Support layer group filtering Jan 7, 2020
@adube
Copy link
Contributor Author

adube commented Jan 7, 2020

Any reason why we don't have the geom filter?

That's an issue intruduced in this PR. I'll look into it.

The issue has been fixed. Ready for review.

@adube adube changed the title [WIP] Support layer group filtering Support layer group filtering Jan 7, 2020
@sbrunner
Copy link
Member

sbrunner commented Jan 8, 2020

If that's an issue, then it should be logged as a separate task in JIRA, as it is not related to this PR.

=> https://jira.camptocamp.com/browse/GSGMF-1171

@sbrunner
Copy link
Member

sbrunner commented Jan 8, 2020

The issue has been fixed. Ready for review.

Confirmed, thanks :-)

@adube
Copy link
Contributor Author

adube commented Jan 8, 2020

@sbrunner or @fredj Just a reminder: this PR is ready for review.

@adube adube merged commit 0485c76 into master Jan 9, 2020
@adube adube deleted the gsgmf-1133-layer-group-filters branch January 9, 2020 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants