Control/Split isEligible() should check feature has a geometry

This commit is contained in:
Peter Robins
2012-01-09 11:13:36 +00:00
parent 128791be2c
commit 1b031bcd26

View File

@@ -312,16 +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 {
typeof target.geometry.split === "function" return (
) && ( target.state !== OpenLayers.State.DELETE
this.feature !== target ) && (
) && ( typeof target.geometry.split === "function"
!this.targetFilter || ) && (
this.targetFilter.evaluate(target.attributes) this.feature !== target
); ) && (
!this.targetFilter ||
this.targetFilter.evaluate(target.attributes)
);
}
}, },
/** /**