ol-mapbox-style - npm Package Compare versions

Comparing version 2.2.0 to 2.2.1

dist/olms.js

@@ -265,4 +265,17 @@ (function(_g){(function(f){var r=(typeof require==='function'?require:function(name){return {"_":null,"ol/proj":ol.proj,"ol/tilegrid":ol.tilegrid,"ol/map":ol.Map,"ol/format/geojson":ol.format.GeoJSON,"ol/format/mvt":ol.format.MVT,"ol/layer/vector":ol.layer.Vector,"ol/layer/vectortile":ol.layer.VectorTile,"ol/source/vector":ol.source.Vector,"ol/source/vectortile":ol.source.VectorTile,"ol/style/style":ol.style.Style,"ol/style/fill":ol.style.Fill,"ol/style/stroke":ol.style.Stroke,"ol/style/circle":ol.style.Circle,"ol/style/icon":ol.style.Icon,"ol/style/text":ol.style.Text}[name];});if (typeof exports==='object'&&typeof module!=='undefined'){module.exports=f(r)}else if(typeof define==='function'&&define.amd){define(["_","ol/proj","ol/tilegrid","ol/map","ol/format/geojson","ol/format/mvt","ol/layer/vector","ol/layer/vectortile","ol/source/vector","ol/source/vectortile","ol/style/style","ol/style/fill","ol/style/stroke","ol/style/circle","ol/style/icon","ol/style/text"],f.bind(_g,r))}else{f(r)}})(function(require,define,module,exports){var _m=(function e(t,n,r){function s(o,u){if(!n[o]){if(!t[o]){var a=typeof require=="function"&&require;if(!u&&a)return a(o,!0);if(i)return i(o,!0);var f=new Error("Cannot find module '"+o+"'");throw f.code="MODULE_NOT_FOUND",f}var l=n[o]={exports:{}};t[o][0].call(l.exports,function(e){var n=t[o][1][e];return s(n?n:e)},l,l.exports,e,t,n,r)}return n[o].exports}var i=typeof require=="function"&&require;for(var o=0;o<r.length;o++)s(r[o]);return s})({1:[function(require,module,exports){
   }
+
   var glLayers = glStyle.layers;
   var layerIds = [];
+
+  function finalizeLayer(layer) {
+    if (layerIds.length > 0) {
+      map.addLayer(layer);
+      applyStyle(layer, glStyle, layerIds).then(function () {
+        layer.setVisible(true);
+      }, function (e) {
+        throw e;
+      });
+    }
+  }
+
   var glLayer, glSource, glSourceId, id, layer, mapid, url;
@@ -274,6 +287,8 @@ for (var i = 0, ii = glLayers.length; i < ii; ++i) {
     } else {
-      layerIds.push(glLayer.id);
       id = glLayer.source || getSourceIdByRef(glLayers, glLayer.ref);
       if (id != glSourceId) {
+        finalizeLayer(layer);
+        layerIds = [];
         glSource = glStyle.sources[id];
+
         if (glSource.type == 'vector') {
@@ -311,23 +326,8 @@ url = glSource.url;
         }
-        if (glSourceId) {
-          map.addLayer(layer);
-          applyStyle(layer, glStyle, layerIds).then(function () {
-            layer.setVisible(true);
-          }, function (e) {
-            throw e;
-          });
-          layerIds = [];
-        }
         glSourceId = id;
       }
+      layerIds.push(glLayer.id);
     }
   }
-  if (layerIds.length > 0) {
-    map.addLayer(layer);
-    applyStyle(layer, glStyle, layerIds).then(function () {
-      layer.setVisible(true);
-    }, function (e) {
-      throw e;
-    });
-  }
+  finalizeLayer(layer);
 }
@@ -334,0 +334,0 @@

index.js

@@ -221,4 +221,17 @@ /*
   }
+
   var glLayers = glStyle.layers;
   var layerIds = [];
+
+  function finalizeLayer(layer) {
+    if (layerIds.length > 0) {
+      map.addLayer(layer);
+      applyStyle(layer, glStyle, layerIds).then(function() {
+        layer.setVisible(true);
+      }, function(e) {
+        throw e;
+      });
+    }
+  }
+
   var glLayer, glSource, glSourceId, id, layer, mapid, url;
@@ -230,6 +243,8 @@ for (var i = 0, ii = glLayers.length; i < ii; ++i) {
     } else {
-      layerIds.push(glLayer.id);
       id = glLayer.source || getSourceIdByRef(glLayers, glLayer.ref);
       if (id != glSourceId) {
+        finalizeLayer(layer);
+        layerIds = [];
         glSource = glStyle.sources[id];
+
         if (glSource.type == 'vector') {
@@ -268,23 +283,8 @@ url = glSource.url;
         }
-        if (glSourceId) {
-          map.addLayer(layer);
-          applyStyle(layer, glStyle, layerIds).then(function() {
-            layer.setVisible(true);
-          }, function(e) {
-            throw e;
-          });
-          layerIds = [];
-        }
         glSourceId = id;
       }
+      layerIds.push(glLayer.id);
     }
   }
-  if (layerIds.length > 0) {
-    map.addLayer(layer);
-    applyStyle(layer, glStyle, layerIds).then(function() {
-      layer.setVisible(true);
-    }, function(e) {
-      throw e;
-    });
-  }
+  finalizeLayer(layer);
 }
@@ -291,0 +291,0 @@

package.json

 {
   "name": "ol-mapbox-style",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "Use Mapbox style objects with OpenLayers",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

732587

decreased by-0.03%

Lines of code

3439

decreased by-0.23%

No dependency changes