cartodb/Leaflet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

This should be oldLatLng, even if latlng is wrong, let's have this be right. refs #2412 #1423

Browse Source
This commit is contained in:
danzel 2014-01-29 14:51:55 +13:00
parent 78be0ad0ec
commit 8cc7404f53
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
spec/suites/layer/marker/MarkerSpec.js
View File

@ -109,7 +109,7 @@ describe("Marker", function () {
marker.setLatLng(afterLatLng); marker.setLatLng(afterLatLng);
expect(eventArgs).to.not.be(null); expect(eventArgs).to.not.be(null);
expect(eventArgs.oldLatlng).to.be(beforeLatLng); expect(eventArgs.oldLatLng).to.be(beforeLatLng);
expect(eventArgs.latlng).to.be(afterLatLng); expect(eventArgs.latlng).to.be(afterLatLng);
expect(marker.getLatLng()).to.be(afterLatLng); expect(marker.getLatLng()).to.be(afterLatLng);
}); });

4
src/layer/marker/Marker.js
View File

@ -55,10 +55,10 @@ L.Marker = L.Layer.extend({
}, },
setLatLng: function (latlng) { setLatLng: function (latlng) {
var oldLatlng = this._latlng; var oldLatLng = this._latlng;
this._latlng = L.latLng(latlng); this._latlng = L.latLng(latlng);
this.update(); this.update();
return this.fire('move', { oldLatlng: oldLatlng, latlng: this._latlng }); return this.fire('move', { oldLatLng: oldLatLng, latlng: this._latlng });
}, },
setZIndexOffset: function (offset) { setZIndexOffset: function (offset) {