handle addData in geojson fixes #33

geoman-io · May 31, 2016 · 2a70d8a · 2a70d8a
1 parent d20f42b
commit 2a70d8a
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 2 deletions.
diff --git a/demo/demo.js b/demo/demo.js
@@ -71,7 +71,8 @@ var geoJsonData = {
     ]
 };
 var geoJsonButton = document.getElementById('test-geojson');
-var geoJsonLayer = L.geoJson(geoJsonData).addTo(map3);
+var geoJsonLayer = L.geoJson().addTo(map3);
+geoJsonLayer.addData(geoJsonData);
 geoJsonLayer.pm.toggleEdit();
 var bounds = geoJsonLayer.getBounds();
 map3.fitBounds(bounds);

diff --git a/dist/leaflet.pm.js b/dist/leaflet.pm.js
@@ -597,9 +597,21 @@ L.PM.Edit.LayerGroup = L.Class.extend({
         this._layerGroup = layerGroup;
         this._layers = layerGroup.getLayers();
 
+        // listen to the edit event of the layers in this group
         for(var i=0; i<this._layers.length; i++) {
             this._layers[i].on('pm:edit', this._fireEdit, this);
         }
+
+        // if a new layer is added to the group, reinitialize
+        this._layerGroup.on('layeradd', function(e) {
+            self.initialize(layerGroup);
+
+            // if editing was already enabled for this group, enable it again
+            // so the new layers are enabled
+            if(e.target.pm.enabled()) {
+                self.enable();
+            }
+        });
     },
     _fireEdit: function() {
         this._layerGroup.fireEvent('pm:edit');

diff --git a/dist/leaflet.pm.min.js b/dist/leaflet.pm.min.js
diff --git a/src/js/L.PM.LayerGroup.js b/src/js/L.PM.LayerGroup.js
@@ -5,9 +5,21 @@ L.PM.Edit.LayerGroup = L.Class.extend({
         this._layerGroup = layerGroup;
         this._layers = layerGroup.getLayers();
 
+        // listen to the edit event of the layers in this group
         for(var i=0; i<this._layers.length; i++) {
             this._layers[i].on('pm:edit', this._fireEdit, this);
         }
+
+        // if a new layer is added to the group, reinitialize
+        this._layerGroup.on('layeradd', function(e) {
+            self.initialize(layerGroup);
+
+            // if editing was already enabled for this group, enable it again
+            // so the new layers are enabled
+            if(e.target.pm.enabled()) {
+                self.enable();
+            }
+        });
     },
     _fireEdit: function() {
         this._layerGroup.fireEvent('pm:edit');