Workaround for mapnik-vector-tile bug

mapbox/mapnik-vector-tile#90 mapbox/mapbox-gl-js#1019
mapbox · Jun 15, 2015 · a615643 · a615643
1 parent 3263ad9
commit a615643
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/lib/vectortilefeature.js b/lib/vectortilefeature.js
@@ -71,7 +71,11 @@ VectorTileFeature.prototype.loadGeometry = function() {
             line.push(new Point(x, y));
 
         } else if (cmd === 7) {
-            line.push(line[0].clone()); // closePolygon
+
+            // Workaround for https://github.com/mapbox/mapnik-vector-tile/issues/90
+            if (line) {
+                line.push(line[0].clone()); // closePolygon
+            }
 
         } else {
             throw new Error('unknown command ' + cmd);

diff --git a/test/fixtures/12-1143-1497.vector.pbf b/test/fixtures/12-1143-1497.vector.pbf
diff --git a/test/parse.test.js b/test/parse.test.js
@@ -147,3 +147,10 @@ test('https://github.com/mapbox/vector-tile-js/issues/15', function(t) {
     t.ok(tile.layers["stuttgart-rails"].feature(0));
     t.end();
 });
+
+test('https://github.com/mapbox/mapbox-gl-js/issues/1019', function(t) {
+    var data = fs.readFileSync(__dirname + '/fixtures/12-1143-1497.vector.pbf');
+    var tile = new VectorTile(new Protobuf(data));
+    t.ok(tile.layers["water"].feature(1).loadGeometry());
+    t.end();
+});