ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Return if a vertex was removed in ol.interaction.Modify#removeVertex_

Browse Source
This commit is contained in:
Frederic Junod
2016-07-04 17:07:40 +02:00
parent b5da32340e
commit 460ea4e809
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/ol/interaction/modifyinteraction.js
View File

@@ -909,8 +909,9 @@ ol.interaction.Modify.prototype.removePoint = function() {
ol.interaction.Modify.prototype.removeVertex_ = function() { ol.interaction.Modify.prototype.removeVertex_ = function() {
var dragSegments = this.dragSegments_; var dragSegments = this.dragSegments_;
var segmentsByFeature = {}; var segmentsByFeature = {};
var deleted = false;
var component, coordinates, dragSegment, geometry, i, index, left; var component, coordinates, dragSegment, geometry, i, index, left;
var newIndex, right, segmentData, uid, deleted; var newIndex, right, segmentData, uid;
for (i = dragSegments.length - 1; i >= 0; --i) { for (i = dragSegments.length - 1; i >= 0; --i) {
dragSegment = dragSegments[i]; dragSegment = dragSegments[i];
segmentData = dragSegment[0]; segmentData = dragSegment[0];
@@ -1016,7 +1017,7 @@ ol.interaction.Modify.prototype.removeVertex_ = function() {
} }
} }
return true; return deleted;
}; };

3
test/spec/ol/interaction/modifyinteraction.test.js
View File

@@ -280,8 +280,9 @@ describe('ol.interaction.Modify', function() {
simulateEvent('pointerdown', 40, 0, false, 0); simulateEvent('pointerdown', 40, 0, false, 0);
simulateEvent('pointerup', 40, 0, false, 0); simulateEvent('pointerup', 40, 0, false, 0);
modify.removePoint(); var removed = modify.removePoint();
expect(removed).to.be(true);
expect(first.getGeometry().getRevision()).to.equal(firstRevision + 1); expect(first.getGeometry().getRevision()).to.equal(firstRevision + 1);
expect(first.getGeometry().getCoordinates()).to.have.length(4); expect(first.getGeometry().getCoordinates()).to.have.length(4);
expect(first.getGeometry().getCoordinates()[3][0]).to.equal(40); expect(first.getGeometry().getCoordinates()[3][0]).to.equal(40);