Handle empty source tiles and queue them properly

oneman · Sep 24, 2019 · 818bdc4 · 818bdc4
1 parent 3992aed
commit 818bdc4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/src/ol/TileQueue.js b/src/ol/TileQueue.js
@@ -112,10 +112,12 @@ class TileQueue extends PriorityQueue {
       if (state === TileState.ABORT) {
         abortedTiles = true;
       } else if (state === TileState.IDLE && !(tileKey in this.tilesLoadingKeys_)) {
-        this.tilesLoadingKeys_[tileKey] = true;
-        ++this.tilesLoading_;
-        ++newLoads;
         tile.load();
+        if (tile.getState() === TileState.LOADING) {
+          this.tilesLoadingKeys_[tileKey] = true;
+          ++this.tilesLoading_;
+          ++newLoads;
+        }
       }
     }
     if (newLoads === 0 && abortedTiles) {

diff --git a/src/ol/source/VectorTile.js b/src/ol/source/VectorTile.js
@@ -236,15 +236,17 @@ class VectorTile extends UrlTile {
               empty = false;
               sourceTile.addEventListener(EventType.CHANGE, this.handleTileChange.bind(this));
               sourceTile.load();
+            } else {
+              sourceTile = null;
             }
           } else {
             empty = false;
           }
           covered = false;
-          if (!sourceTile) {
+          if (sourceTile === undefined) {
             return;
           }
-          if (sourceTile.getState() !== TileState.EMPTY && tile.getState() === TileState.IDLE) {
+          if (sourceTile !== null && tile.getState() === TileState.IDLE) {
             tile.loadingSourceTiles++;
             const key = listen(sourceTile, EventType.CHANGE, function() {
               const state = sourceTile.getState();