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.

Sign up for GitHub

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

Jump to bottom

3.3 bugfix featureinfo queryable #519

Merged
merged 5 commits into from
Apr 24, 2015
Merged

3.3 bugfix featureinfo queryable #519

merged 5 commits into from
Apr 24, 2015

Conversation

stephanr
Copy link
Member

This Pull Requests fixes the the issue #517 by

  • correcting the loading of the layer options (correct the enabled=false)
  • enhancement of handling of the three states of the feature info radius
    • < 0 ==> default (as specified in service endpoint)
    • = 0 ==> disabled

    • 0 ==> user specified radius

  • porting the 3.4 layer analiesis method to a single helper class to make a Layer in WMS/GetCapabilties only queryable=0 if all layers in the sub structure are not queryable

@stephanr stephanr added bug error issue and bug (fix) in progress labels Mar 20, 2015
@stephanr stephanr self-assigned this Mar 20, 2015
@stephanr stephanr added this to the 3.3.15 milestone Mar 20, 2015
@stephanr
Copy link
Member Author

The builds failed because of problems with the deegree3-demo/deegree3-testing wich are referenced by the tests, so #513 has to be resolved before this test ca be rerun

@stephanr
Copy link
Member Author

The first Travis Job passed succesfull [1], the other one got killed at half the time of first.
[1] https://travis-ci.org/deegree/deegree3/jobs/55931279

@stephanr stephanr mentioned this pull request Apr 24, 2015
Closed
@tfr42 tfr42 added ready and removed in progress labels Apr 24, 2015
@MrSnyder
Copy link
Contributor

Corresponding 3.4 ticket: #518.

@tfr42
Copy link
Member

tfr42 commented Apr 24, 2015

Thank you for your contribution. Your pull requests has been accepted by the TMC.
This pull request will be part of the 3.3.15 release.

MrSnyder added a commit that referenced this pull request Apr 24, 2015
@MrSnyder
Merge pull request #519 from gritGmbH/3.3-bugfix-featureinfo-queryable
3a69672 
3.3 bugfix featureinfo queryable
@MrSnyder MrSnyder merged commit 3a69672 into deegree:3.3-master Apr 24, 2015
@MrSnyder MrSnyder removed the ready label Apr 24, 2015
@stephanr stephanr mentioned this pull request Apr 27, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@stephanr stephanr

Labels
bug error issue and bug (fix)
Projects
None yet
Milestone
3.3.15
Development

Successfully merging this pull request may close these issues.
3 participants
@stephanr @MrSnyder @tfr42