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

bug/#1416 - additional zoom check before tile download #1417

Merged
merged 1 commit into from Oct 9, 2019

Conversation

@monsieurtanuki
Copy link
Collaborator

commented Oct 9, 2019

New class:

  • SampleUnreachableOnlineTiles: demo checking if the zoom restriction for tiles is correctly applied, to be found under "More Samples / Tileproviders / Zoom Restricted Online Tiles (14-15)"

Impacted classes:

  • MapTileApproximater: called new method TileLoader.loadTileIfReachable instead of TileLoader.loadTile
  • MapTileModuleProviderBase: new method isTileReachable; new method TileLoader.loadTileIfReachable, to be called instead of direct calls to TileLoader.loadTile
  • MapTilePreCache: called new method TileLoader.loadTileIfReachable instead of TileLoader.loadTile
  • SampleFactory: added new demo SampleUnreachableOnlineTiles
New class:
* `SampleUnreachableOnlineTiles`: demo checking if the zoom restriction for tiles is correctly applied, to be found under "More Samples / Tileproviders / Zoom Restricted Online Tiles (14-15)"

Impacted classes:
* `MapTileApproximater`: called new method `TileLoader.loadTileIfReachable` instead of `TileLoader.loadTile`
* `MapTileModuleProviderBase`: new method `isTileReachable`; new method `TileLoader.loadTileIfReachable`, to be called instead of direct calls to `TileLoader.loadTile`
* `MapTilePreCache`: called new method `TileLoader.loadTileIfReachable` instead of `TileLoader.loadTile`
* `SampleFactory`: added new demo `SampleUnreachableOnlineTiles`
@monsieurtanuki monsieurtanuki merged commit 2aab5a9 into master Oct 9, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.