Changeset 7601

Timestamp:

07/29/08 23:41:22 (4 months ago)

Author:

tschaub

Message:

Adding a featureadded event to the draw feature control event types. First tests ever for the draw feature control. Thanks sbenthall for the patch. r=me (closes #1508)

Files:

• trunk/openlayers/lib/OpenLayers/Control/DrawFeature.js (modified) (3 diffs)
• trunk/openlayers/tests/Control/DrawFeature.html (added)
• trunk/openlayers/tests/list-tests.html (modified) (1 diff)

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

@@ -31 +31 @@
-    
-    /**
+     * Constant: EVENT_TYPES
+     *
+     * Supported event types:
+     *  - *featureadded* Triggered when a feature is added
+     */
+    EVENT_TYPES: ["featureadded"],
+    
+    /**
-     * APIProperty: featureAdded
-     * {Function} Called after each feature is added
@@ -51 +59 @@
-     */
-    initialize: function(layer, handler, options) {
+        
+        // concatenate events specific to vector with those from the base
+        this.EVENT_TYPES =
+            OpenLayers.Control.DrawFeature.prototype.EVENT_TYPES.concat(
+            OpenLayers.Control.prototype.EVENT_TYPES
+        );
+        
-        OpenLayers.Control.prototype.initialize.apply(this, [options]);
-        this.callbacks = OpenLayers.Util.extend({done: this.drawFeature},
@@ -65 +80 @@
-        this.layer.addFeatures([feature]);
-        this.featureAdded(feature);
+        this.events.triggerEvent("featureadded",{feature : feature});
-    },
-

trunk/openlayers/tests/list-tests.html

@@ -91 +91 @@
-    <li>Control/DragFeature.html</li>
-    <li>Control/DragPan.html</li>
+    <li>Control/DrawFeature.html</li>
-    <li>Control/LayerSwitcher.html</li>
-    <li>Control/ModifyFeature.html</li>