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

In TileOutOfRange-exceptions the "locator"-value has no value #153

Merged
merged 1 commit into from Feb 1, 2017

Conversation

Projects
None yet
4 participants
@tbonfort
Member

tbonfort commented Sep 16, 2016

According to the WMTS standard (I have looked in OGC 07-057r7), the "locator"-value of an exception with exceptionCode=TileOutOfRange should contain the name of the parameter which is out of range (see Table 23 in http://portal.opengeospatial.org/files/?artifact_id=35326).

image
Screenshot of Table 23. The for grey exceptions are from the general OWS, the TileOutOfRange-exception is specific to WMTS

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Sep 16, 2016

Member

please check this PR

Member

tbonfort commented Sep 16, 2016

please check this PR

@GregersP

This comment has been minimized.

Show comment
Hide comment
@GregersP

GregersP Sep 19, 2016

Thanks!
Will check it out :-)

GregersP commented Sep 19, 2016

Thanks!
Will check it out :-)

@Daniel-Madsen

This comment has been minimized.

Show comment
Hide comment
@Daniel-Madsen

Daniel-Madsen Feb 1, 2017

Is it possible to have this fix merged into 1-4 branch or the main branch? I did some testing on this, and it seems to solve the problem :-)

Daniel-Madsen commented Feb 1, 2017

Is it possible to have this fix merged into 1-4 branch or the main branch? I did some testing on this, and it seems to solve the problem :-)

@tbonfort tbonfort merged commit 5dacd41 into mapserver:branch-1-4 Feb 1, 2017

@tbonfort tbonfort deleted the tbonfort:issues/153-tileoutofrange-exception branch Feb 1, 2017

@Icemand

This comment has been minimized.

Show comment
Hide comment
@Icemand

Icemand Feb 20, 2017

Are there any plan for next official release where this would be a part of?

Icemand commented Feb 20, 2017

Are there any plan for next official release where this would be a part of?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment