Changeset 7616

Timestamp:

07/30/08 18:03:40 (4 months ago)

Author:

tschaub

Message:

Adding a beforefeatureselected event to the vector layer events. If a listener returns false, the feature is not selected. The modify feature control now listens for beforefeatureselected and triggers beforefeaturemodified. If a listener returns false, feature modification never begins. r=elemione,ahocevar (closes #1427)

Files:

• trunk/openlayers/examples/modify-feature.html (modified) (2 diffs)
• trunk/openlayers/lib/OpenLayers/Control/ModifyFeature.js (modified) (4 diffs)
• trunk/openlayers/lib/OpenLayers/Control/SelectFeature.js (modified) (2 diffs)
• trunk/openlayers/lib/OpenLayers/Layer/Vector.js (modified) (1 diff)
• trunk/openlayers/tests/Control/ModifyFeature.html (modified) (5 diffs)

trunk/openlayers/examples/modify-feature.html

@@ -30 +30 @@
-            map.addControl(new OpenLayers.Control.MousePosition());
-            
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
-            controls = {
-                point: new OpenLayers.Control.DrawFeature(vectors,
@@ -54 +48 @@
-                            OpenLayers.Handler.RegularPolygon,
-                            {handlerOptions: {sides: 5}}),
-
-
+
-            };
-            

trunk/openlayers/lib/OpenLayers/Control/ModifyFeature.js

@@ -202 +202 @@
-        );
-        this.layer.events.on({
+            "beforefeatureselected": this.beforeSelectFeature,
-            "featureselected": this.selectFeature,
-            "featureunselected": this.unselectFeature,
@@ -240 +241 @@
-    destroy: function() {
-        this.layer.events.un({
+            "beforefeatureselected": this.beforeSelectFeature,
-            "featureselected": this.selectFeature,
-            "featureunselected": this.unselectFeature,
@@ -288 +290 @@
-        return deactivated;
-    },
+    
+    /**
+     * Method: beforeSelectFeature
+     * Called before a feature is selected.
+     *
+     * Parameters:
+     * object - {Object} Object with a feature property referencing the
+     *     selected feature.
+     */
+    beforeSelectFeature: function(object) {
+        return this.layer.events.triggerEvent(
+            "beforefeaturemodified", {feature: object.feature}
+        );
+    },
-
-    /**
@@ -302 +318 @@
-        this.dragControl.activate();
-        this.onModificationStart(this.feature);
-        this.layer.events.triggerEvent("beforefeaturemodified", 
-                                       {feature: this.feature});
-    },
-

trunk/openlayers/lib/OpenLayers/Control/SelectFeature.js

@@ -309 +309 @@
-     */
-    select: function(feature) {
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
-    },
-
@@ -324 +329 @@
-     *
-     * Parameters:
- 
+
-     */
-    unselect: function(feature) {

trunk/openlayers/lib/OpenLayers/Layer/Vector.js

@@ -73 +73 @@
-    EVENT_TYPES: ["beforefeatureadded", "featureadded", "featuresadded",
-                  "beforefeatureremoved", "featureremoved", "featuresremoved",
-
+beforefeatureselected", "
-                  "beforefeaturemodified", "featuremodified", "afterfeaturemodified"],
-

trunk/openlayers/tests/Control/ModifyFeature.html

@@ -165 +165 @@
-
-    function test_selectFeature(t) {
-
-
+
+
+
+
+
-        var control = new OpenLayers.Control.ModifyFeature(layer);
-        control.vertices = [];
-        control.virtualVertices = [];
-
-
-
+
+
+
+
-        control.dragControl.activate = function() { t.ok(true, "drag Control activated"); }
-        control.onModificationStart = function(feature)  { t.eq(feature.id, fakeFeature.id, "On Modification Start called with correct feature."); } 
-        
-        
-        // Start of testing
-        
-        control.collectVertices = function() { t.fail("Collect vertices called when geom is a point"); }
-{'id':'myFakeFeature','geometry':{'CLASS_NAME':'OpenLayers.Geometry.Point'}}
+new OpenLayers.Feature.Vector(new OpenLayers.Geometry.Point(0, 0))
-        
-        // Points don't call collectVertices
@@ -196 +199 @@
-            t.ok(features == 'a' || features == 'd', "features passed correctly"); 
-        }
-
+
+
+
+
+
+
+
-        
-        // OnSelect calls collectVertices and passes features to layer 
@@ -360 +369 @@
-
-    function test_onModificationStart(t) {
-2
+1
-        var map = new OpenLayers.Map("map");
-        var layer = new OpenLayers.Layer.Vector();
@@ -367 +376 @@
-        map.addControl(control);
-        control.activate();
-        
-        // test that beforefeaturemodified is triggered
-        layer.events.on({"beforefeaturemodified": function(event) {
-            t.eq(event.feature.id, testFeature.id,
-                 "beforefeaturemodified is triggered with correct feature");
-        }});
-        
-        // make sure onModificationStart is called on feature selection
@@ -456 +459 @@
-        map.destroy();
-    }
+    
+    function test_events(t) {
+        t.plan(2);
+        var map = new OpenLayers.Map("map");
+        var layer = new OpenLayers.Layer.Vector();
+        map.addLayer(layer);
+        var control = new OpenLayers.Control.ModifyFeature(layer);
+        map.addControl(control);
+        control.activate();
+        
+        // make sure onModificationStart is called on feature selection
+        var testFeature = new OpenLayers.Feature.Vector(
+            new OpenLayers.Geometry.Point(Math.random(), Math.random())
+        );
+        
+        // test that beforefeatureselected is triggered
+        function handle_beforefeatureselected(event) {
+            t.ok(event.feature == testFeature, "beforefeatureselected called with the correct feature");
+        }
+        layer.events.on({
+            "beforefeatureselected": handle_beforefeatureselected
+        });
+        layer.events.triggerEvent("beforefeatureselected", {
+            feature: testFeature
+        });
+        layer.events.un({
+            "beforefeatureselected": handle_beforefeatureselected
+        });
+        
+        // test that beforefeatureselected is triggered
+        function handle_featureselected(event) {
+            t.ok(event.feature == testFeature, "featureselected called with the correct feature");
+        }
+        layer.events.on({
+            "featureselected": handle_featureselected
+        });
+        layer.events.triggerEvent("featureselected", {
+            feature: testFeature
+        });
+        layer.events.un({
+            "featureselected": handle_featureselected
+        });
+
+        map.destroy();
+    }
+
-
-