Merge pull request #134 from probins/splitnoml
Removing MultiLineString dependency from the Split control. Thanks @probins for the contribution.
This commit is contained in:
@@ -277,8 +277,7 @@ OpenLayers.Control.Split = OpenLayers.Class(OpenLayers.Control, {
|
|||||||
afterFeatureModified: function(event) {
|
afterFeatureModified: function(event) {
|
||||||
if(event.modified) {
|
if(event.modified) {
|
||||||
var feature = event.feature;
|
var feature = event.feature;
|
||||||
if(feature.geometry instanceof OpenLayers.Geometry.LineString ||
|
if (typeof feature.geometry.split === "function") {
|
||||||
feature.geometry instanceof OpenLayers.Geometry.MultiLineString) {
|
|
||||||
this.feature = event.feature;
|
this.feature = event.feature;
|
||||||
this.considerSplit(event.feature);
|
this.considerSplit(event.feature);
|
||||||
}
|
}
|
||||||
@@ -313,17 +312,20 @@ OpenLayers.Control.Split = OpenLayers.Class(OpenLayers.Control, {
|
|||||||
* {Boolean} The target is eligible for splitting.
|
* {Boolean} The target is eligible for splitting.
|
||||||
*/
|
*/
|
||||||
isEligible: function(target) {
|
isEligible: function(target) {
|
||||||
return (
|
if (!target.geometry) {
|
||||||
target.state !== OpenLayers.State.DELETE
|
return false;
|
||||||
) && (
|
} else {
|
||||||
target.geometry instanceof OpenLayers.Geometry.LineString ||
|
return (
|
||||||
target.geometry instanceof OpenLayers.Geometry.MultiLineString
|
target.state !== OpenLayers.State.DELETE
|
||||||
) && (
|
) && (
|
||||||
this.feature !== target
|
typeof target.geometry.split === "function"
|
||||||
) && (
|
) && (
|
||||||
!this.targetFilter ||
|
this.feature !== target
|
||||||
this.targetFilter.evaluate(target.attributes)
|
) && (
|
||||||
);
|
!this.targetFilter ||
|
||||||
|
this.targetFilter.evaluate(target.attributes)
|
||||||
|
);
|
||||||
|
}
|
||||||
},
|
},
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
Reference in New Issue
Block a user