Skip to content
This repository was archived by the owner on Sep 20, 2019. It is now read-only.

Commit d431344

Browse files
committed
Destroying scope should only remove its own defaults
1 parent b551246 commit d431344

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

src/directives/leaflet.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ angular.module('ui-leaflet', ['nemLogging']).directive('leaflet',
135135
});
136136

137137
scope.$on('$destroy', function () {
138-
leafletMapDefaults.reset();
138+
leafletMapDefaults.reset(attrs.id);
139139
map.remove();
140140
leafletData.unresolveMap(attrs.id);
141141
});

src/services/leafletMapDefaults.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,11 @@ angular.module('ui-leaflet').factory('leafletMapDefaults', function ($q, leaflet
4848

4949
// Get the _defaults dictionary, and override the properties defined by the user
5050
return {
51-
reset: function () {
52-
defaults = {};
51+
reset: function (scopeId) {
52+
if (!isDefined(scopeId)){
53+
scopeId = 'main';
54+
}
55+
delete defaults[scopeId];
5356
},
5457
getDefaults: function (scopeId) {
5558
var mapId = obtainEffectiveMapId(defaults, scopeId);

0 commit comments

Comments
 (0)