Ticket #3044 (closed feature: fixed)

Opened 2 years ago

Last modified 19 months ago

use ResourceURL

Reported by: fredj Owned by:
Priority: minor Milestone: 2.12 Release
Component: Layer.WMTS Version: 2.10
Keywords: Cc:
State: Complete

Description

If present, use the ResourceURL of the layer to build the url of the tiles. Only applies if the request encoding is REST.

Depends on #2986

ref: OGC 07-057r7 chapter 10

Attachments

3044.0.patch Download (7.4 KB) - added by fredj 2 years ago.
first working draft
3044.1.patch Download (7.4 KB) - added by fredj 2 years ago.
rename resourceURL to resourceUrl, "dimension" still don't work
openlayers-3044.patch Download (7.7 KB) - added by ahocevar 19 months ago.
slightly different approach; with dimension handling

Change History

Changed 2 years ago by fredj

first working draft

Changed 2 years ago by fredj

rename resourceURL to resourceUrl, "dimension" still don't work

  Changed 2 years ago by fredj

  • state set to Review

ready to review

  Changed 2 years ago by fredj

  • state changed from Review to Needs More Work

dimension not managed yet

  Changed 19 months ago by ahocevar

  • state changed from Needs More Work to Review

@fredj: I uploaded a new patch which handles the dimensions. I also don't see a need to introduce a new property (resourceUrl). To be more in line with the XYZ layer, we can use the url property and check if it is a template url or a fixed one.

My patch also passes the template from the capabilities to a layer created with Format.WMTSCapabilities::createLayer.

Tests pass in Safari5.1. Thanks for any review.

Changed 19 months ago by ahocevar

slightly different approach; with dimension handling

follow-up: ↓ 5   Changed 19 months ago by erilem

Andreas' patch looks good to me. I'll just test it in a WMTS-based app of ours.

in reply to: ↑ 4   Changed 19 months ago by erilem

  • state changed from Review to Commit

Replying to erilem:

Andreas' patch looks good to me. I'll just test it in a WMTS-based app of ours.

The lights are green for me. Thanks FredJ and Andreas for the work on that.

  Changed 19 months ago by ahocevar

  • status changed from new to closed
  • state changed from Commit to Complete
  • resolution set to fixed
Note: See TracTickets for help on using tickets.