fix TypeError thrown when Map.querySourceFeatures doesn't include a p…

…arameters option
mapbox · Nov 4, 2016 · d99504e · d99504e
1 parent 358cabe
commit d99504e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/js/source/tile.js b/js/source/tile.js
@@ -174,7 +174,7 @@ class Tile {
 
         if (!layer) return;
 
-        const filter = featureFilter(params.filter);
+        const filter = featureFilter(params && params.filter);
         const coord = { z: this.coord.z, x: this.coord.x, y: this.coord.y };
 
         for (let i = 0; i < layer.length; i++) {

diff --git a/test/js/source/tile.test.js b/test/js/source/tile.test.js
@@ -34,6 +34,9 @@ test('querySourceFeatures', (t) => {
         geojsonWrapper.name = '_geojsonTileLayer';
         tile.rawTileData = vtpbf({ layers: { '_geojsonTileLayer': geojsonWrapper }});
 
+        result = [];
+        tile.querySourceFeatures(result);
+        t.equal(result.length, 1);
         result = [];
         tile.querySourceFeatures(result, {});
         t.equal(result.length, 1);