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

Fix for #527 - 3.3: Queryable not configurable #209

Closed

Conversation

diegomigliavacca
Copy link
Contributor

http://tracker.deegree.org/deegree-services/ticket/527

It's possible to set a layer as queryable or not queryable setting the attribute "enabled" on the "l:FeatureInfo" element. This configuration was not working properly.

@MrSnyder MrSnyder closed this Apr 29, 2014
@MrSnyder MrSnyder reopened this Apr 29, 2014
@copierrj
Copy link
Member

copierrj commented May 7, 2014

Your patch doesn't compile. Could you fix it?

@tfr42 tfr42 modified the milestones: 3.3.9, 3.3.10 May 7, 2014
@tfr42
Copy link
Member

tfr42 commented Jun 26, 2014

Thank you for your contribution. Unfortunately, it does not compile. It cannot be considered for inclusion, before this has been fixed. Sorry.
Additional information:

@tfr42 tfr42 modified the milestones: 3.3.11, 3.3.10 Jun 26, 2014
@tfr42 tfr42 modified the milestones: 3.3.12, 3.3.11 Jul 10, 2014
@tfr42 tfr42 added the bug label Jul 10, 2014
@tfr42 tfr42 modified the milestone: 3.3.12 Jul 10, 2014
@diegomigliavacca
Copy link
Contributor Author

Now it compiles. But there are "deegree-compliance-tests" errors.

@tfr42
Copy link
Member

tfr42 commented Jul 24, 2014

Can you please post those errors here. Thanks.

@tfr42 tfr42 removed this from the 3.3.12 milestone Jul 24, 2014
@tfr42 tfr42 added this to the 3.3.12 milestone Sep 11, 2014
@tfr42 tfr42 modified the milestones: 3.3.14, 3.3.12 Nov 13, 2014
@tfr42
Copy link
Member

tfr42 commented Feb 6, 2015

Thank you for your contribution. Unfortunately, it does not compile / has test errors. It cannot be considered for inclusion, before this has been fixed. Sorry.
Additional information:

And please post those deegree-compliance-tests errors here.

@tfr42 tfr42 modified the milestones: 3.3.15, 3.3.14 Feb 6, 2015
@tfr42 tfr42 added the contributions welcome asking for contribution (time and money sponsor) label Mar 4, 2015
@stephanr
Copy link
Member

I think this one should be superseded by #518 which fixed the issue on a wider scope.

@tfr42 tfr42 added the invalid invalid label Apr 24, 2015
@tfr42 tfr42 removed contributions welcome asking for contribution (time and money sponsor) tests failing unit or Integration tests fail labels Apr 24, 2015
@tfr42
Copy link
Member

tfr42 commented Apr 24, 2015

Dear Diego,
please review the changes made in PR #518 if they address the same issue. We will wait for your feedback until the next TMC meeting before we will close this PR.

@tfr42 tfr42 modified the milestones: 3.3.16, 3.3.15 Apr 24, 2015
@diegomigliavacca
Copy link
Contributor Author

yes, the first part of the PR #518 fixes this issue as well.

@diegomigliavacca diegomigliavacca deleted the bug#527 branch April 24, 2015 16:21
@tfr42 tfr42 removed this from the 3.3.16 milestone Oct 1, 2015
@tfr42 tfr42 added duplicate and removed bug error issue and bug (fix) invalid invalid needs discussion requires discussion with contributor labels Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants