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

Commit b91eabc

Browse files
committed
Merge pull request #260 from realtymaps/1.X
fix(leafletData unresolveMap): argument name renamed scopeId to mapId
2 parents ffcfa2f + 5132ee1 commit b91eabc

File tree

10 files changed

+41
-82
lines changed

10 files changed

+41
-82
lines changed

.jshintrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
"eqeqeq": true,
88
"immed": true,
99
"indent": 4,
10-
"latedef": true,
10+
"latedef": false,
1111
"newcap": true,
1212
"noarg": true,
1313
"regexp": true,

bower.json

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,14 @@
3434
"Leaflet.utfgrid": "danzel/Leaflet.utfgrid",
3535
"Leaflet.awesome-markers": "*",
3636
"leaflet-providers": "~1.1.5",
37-
"leaflet.vector-markers": "~0.0.3",
37+
"leaflet.vector-markers": "0.0.4",
3838
"webgl-heatmap-leaflet": "*",
39-
"leaflet-plugins": "*",
40-
"esri-leaflet": "*",
41-
"proj4": "*",
39+
"leaflet-plugins": "1.X",
40+
"esri-leaflet": "1.X",
41+
"proj4": "2.X",
4242
"font-awesome": "~4.3.x",
43-
"proj4leaflet": "*",
44-
"Leaflet.MakiMarkers": "*",
43+
"proj4leaflet": "0.X",
44+
"Leaflet.MakiMarkers": "2.X",
4545
"Leaflet.heat": "https://github.com/Leaflet/Leaflet.heat/archive/gh-pages.tar.gz",
4646
"Leaflet.ExtraMarkers": "https://github.com/coryasilva/Leaflet.ExtraMarkers/archive/v1.0.1.tar.gz",
4747
"Leaflet.fullscreen": "http://github.com/Leaflet/Leaflet.fullscreen/archive/v0.0.4.tar.gz",

dist/ui-leaflet.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*!
2-
* ui-leaflet 1.0.0 2015-11-03
2+
* ui-leaflet 1.0.0 2016-06-07
33
* ui-leaflet - An AngularJS directive to easily interact with Leaflet maps
44
* git: https://github.com/angular-ui/ui-leaflet
55
*/
@@ -456,8 +456,8 @@ angular.module('ui-leaflet').service('leafletData', ["leafletLogger", "$q", "lea
456456
_private[itemName] = {};
457457
});
458458

459-
this.unresolveMap = function (scopeId) {
460-
var id = leafletHelpers.obtainEffectiveMapId(_private.map, scopeId);
459+
this.unresolveMap = function (mapId) {
460+
var id = leafletHelpers.obtainEffectiveMapId(_private.map, mapId);
461461
_privateItems.forEach(function (itemName) {
462462
_private[itemName][id] = undefined;
463463
});
@@ -466,14 +466,14 @@ angular.module('ui-leaflet').service('leafletData', ["leafletLogger", "$q", "lea
466466
//int repetitive stuff (get and sets)
467467
_privateItems.forEach(function (itemName) {
468468
var name = upperFirst(itemName);
469-
self['set' + name] = function (lObject, scopeId) {
470-
var defer = getUnresolvedDefer(_private[itemName], scopeId);
469+
self['set' + name] = function (lObject, mapId) {
470+
var defer = getUnresolvedDefer(_private[itemName], mapId);
471471
defer.resolve(lObject);
472-
setResolvedDefer(_private[itemName], scopeId);
472+
setResolvedDefer(_private[itemName], mapId);
473473
};
474474

475-
self['get' + name] = function (scopeId) {
476-
var defer = getDefer(_private[itemName], scopeId);
475+
self['get' + name] = function (mapId) {
476+
var defer = getDefer(_private[itemName], mapId);
477477
return defer.promise;
478478
};
479479
});

dist/ui-leaflet.min.js

Lines changed: 0 additions & 39 deletions
This file was deleted.

dist/ui-leaflet.min.no-header.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/ui-leaflet_dev_mapped.js

Lines changed: 8 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/ui-leaflet_dev_mapped.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,28 +32,27 @@
3232
"grunt-contrib-watch": "^0.6.1",
3333
"grunt-conventional-changelog": "^1.2.2",
3434
"grunt-graphviz": "^0.1.1",
35-
"grunt-karma": "^0.10.1",
35+
"grunt-karma": "2.X",
3636
"grunt-karma-coveralls": "^2.5.3",
3737
"grunt-ng-annotate": "^1.0.1",
3838
"grunt-open": "^0.2.3",
3939
"grunt-protractor-runner": "^2.0.0",
4040
"grunt-shell": "^1.1.2",
4141
"grunt-shell-spawn": "^0.3.2",
42-
"jasmine-core": "^2.2.0",
4342
"jsdom": ">= 0.1.23 < 4.0.0",
44-
"karma": "0.12.32",
43+
"karma": "0.X",
4544
"karma-chrome-launcher": "^0.1.8",
46-
"karma-coffee-preprocessor": "^0.3.0",
45+
"karma-coffee-preprocessor": "1.X",
4746
"karma-coverage": "^0.5.3",
4847
"karma-firefox-launcher": "^0.1.4",
4948
"karma-html2js-preprocessor": "^0.1.0",
50-
"karma-jasmine": "^0.3.6",
49+
"karma-jasmine": "1.X",
5150
"karma-mocha-reporter": "1.0.2",
52-
"karma-phantomjs-launcher": "^0.1.4",
51+
"karma-phantomjs-launcher": "1.X",
5352
"karma-requirejs": "^0.2.2",
5453
"karma-script-launcher": "^0.1.0",
5554
"load-grunt-config": "^0.17.0",
56-
"lodash": "^3.7.0",
55+
"lodash": "3.X.0",
5756
"matchdep": "^1.0.0",
5857
"mkdirp": "^0.5.0",
5958
"protractor": "^2.0.0",

src/services/leafletData.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ angular.module('ui-leaflet').service('leafletData', function (leafletLogger, $q,
2727
_private[itemName] = {};
2828
});
2929

30-
this.unresolveMap = function (scopeId) {
31-
var id = leafletHelpers.obtainEffectiveMapId(_private.map, scopeId);
30+
this.unresolveMap = function (mapId) {
31+
var id = leafletHelpers.obtainEffectiveMapId(_private.map, mapId);
3232
_privateItems.forEach(function (itemName) {
3333
_private[itemName][id] = undefined;
3434
});
@@ -37,14 +37,14 @@ angular.module('ui-leaflet').service('leafletData', function (leafletLogger, $q,
3737
//int repetitive stuff (get and sets)
3838
_privateItems.forEach(function (itemName) {
3939
var name = upperFirst(itemName);
40-
self['set' + name] = function (lObject, scopeId) {
41-
var defer = getUnresolvedDefer(_private[itemName], scopeId);
40+
self['set' + name] = function (lObject, mapId) {
41+
var defer = getUnresolvedDefer(_private[itemName], mapId);
4242
defer.resolve(lObject);
43-
setResolvedDefer(_private[itemName], scopeId);
43+
setResolvedDefer(_private[itemName], mapId);
4444
};
4545

46-
self['get' + name] = function (scopeId) {
47-
var defer = getDefer(_private[itemName], scopeId);
46+
self['get' + name] = function (mapId) {
47+
var defer = getDefer(_private[itemName], mapId);
4848
return defer.promise;
4949
};
5050
});

test/karma-unit.conf.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ module.exports = function(karma) {
77
basePath: '../',
88

99
preprocessors: {
10-
'test/**/**/*.coffee': ['coffee'],
10+
'test/unit/**/*.coffee': ['coffee'],
1111
},
1212
coffeePreprocessor: {
1313
options: {
@@ -28,13 +28,12 @@ module.exports = function(karma) {
2828
'bower_components/leaflet.vector-markers/dist/Leaflet.vector-markers.js',
2929
mainLib,
3030
'test/unit/bootstrap.coffee',
31-
'test/unit/*.js',
3231
'test/unit/**/*.js',
3332
'test/unit/**/*.coffee',
3433
'bower_components/Leaflet.PolylineDecorator/leaflet.polylineDecorator.js',
3534
//do not include those specs for jasmine html runner by karma kama_jasmine_runner.html
36-
{pattern:'**/**/**/*.coffee', included: false},
37-
{pattern: '**/*.js.map', included: false}
35+
{pattern:'test/**/**/*.coffee', included: false},
36+
{pattern: 'dist/**/*.js.map', included: false}
3837
],
3938
// Frameworks
4039
frameworks: ["jasmine"],

0 commit comments

Comments
 (0)