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
Wms 1 3 compliance #3812
Wms 1 3 compliance #3812
Conversation
…thin the WIDTH/HEIGHT parameters
…owed to be empty (also if a layer has only one point)
…ifferent to WIDTH/HEIGHT of requested image
…uests in the capabilities document is a URL prefix
👍 |
Do you want to backport to 2.14 version ? |
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.
Just some comments, thats looks good to me :-)
@@ -77,14 +80,21 @@ void QgsConfigParserUtils::appendCrsElementToLayer( QDomElement& layerElement, c | |||
layerElement.insertAfter( crsElement, precedingElement ); | |||
} | |||
|
|||
void QgsConfigParserUtils::appendLayerBoundingBoxes( QDomElement& layerElem, QDomDocument& doc, const QgsRectangle& layerExtent, | |||
void QgsConfigParserUtils::appendLayerBoundingBoxes( QDomElement& layerElem, QDomDocument& doc, const QgsRectangle& lExtent, |
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.
Looks wrong ...
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.
Maybe I need glasses, could you be more specific what is wrong?
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 miss the line QgsRectangle layerExtent = lExtent;
than I suspect an unwanted change :-)
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.
Ah, I see. Thanks anyway.
@rldhont: Yes, a backport to 2.14 makes sense. |
…t for WMS 1.3.0 compliance
…mage width/height
The OGC test suite for WMS 1.3.0 has been run against QGIS server. This PR fixes the issues where the server does not exactly behave as in the specification. With these changes, QGIS server is an OGC compliant WMS 1.3. server.