Changeset 7650

Timestamp:

07/31/08 21:27:35 (4 months ago)

Author:

tschaub

Message:

Adding strategy and protocol base classes for better vector behavior. This just sets the common API for strategy and protocol. In general, a vector layer can have many strategies. A setLayer method gets called on each of these layers. When a layer is added to a map, any strategies on that layer get activated. When the layer is destroyed, any strategies get destroyed (this could be changed to deactivate for symmetry). A vector layer may also have a protocol. A protocol is typically constructed with a format. The layer doesn't need to know about the format. The protocol doesn't need to know about the layer. Strategies coordinate feature management for a layer. Specific strategies and protocols to follow. r=crschmidt (closes #1646)

Files:

• trunk/openlayers/lib/OpenLayers.js (modified) (1 diff)
• trunk/openlayers/lib/OpenLayers/Layer/Vector.js (modified) (4 diffs)
• trunk/openlayers/lib/OpenLayers/Protocol.js (added)
• trunk/openlayers/lib/OpenLayers/Strategy.js (added)
• trunk/openlayers/tests/Protocol.html (added)
• trunk/openlayers/tests/Strategy.html (added)
• trunk/openlayers/tests/list-tests.html (modified) (2 diffs)

trunk/openlayers/lib/OpenLayers.js

@@ -181 +181 @@
-            "OpenLayers/Renderer/VML.js",
-            "OpenLayers/Layer/Vector.js",
+            "OpenLayers/Strategy.js",
+            "OpenLayers/Protocol.js",
-            "OpenLayers/Layer/PointTrack.js",
-            "OpenLayers/Layer/GML.js",

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

@@ -126 +126 @@
-     */
-    styleMap: null,
-
+
+
+
+
+
+
+
+
+
+
+
+
+
-    /**
-     * Property: renderers
@@ -195 +207 @@
-        this.features = [];
-        this.selectedFeatures = [];
+        
+        // Allow for custom layer behavior
+        if(this.strategies){
+            for(var i=0, len=this.strategies.length; i<len; i++) {
+                this.strategies[i].setLayer(this);
+            }
+        }
+
-    },
-
@@ -202 +222 @@
-     */
-    destroy: function() {
+        if (this.stategies) {
+            for(var i=0, len=this.strategies.length; i<len; i++) {
+                this.strategies[i].destroy();
+            }
+            this.strategies = null;
+        }
+        if (this.protocol) {
+            this.protocol.destroy();
+            this.protocol = null;
+        }
-        this.destroyFeatures();
-        this.features = null;
@@ -258 +288 @@
-            this.renderer.map = this.map;
-            this.renderer.setSize(this.map.getSize());
+        }
+        if(this.strategies) {
+            for(var i=0, len=this.strategies.length; i<len; i++) {
+                this.strategies[i].activate();
+            }
-        }
-    },

trunk/openlayers/tests/list-tests.html

@@ -111 +111 @@
-    <li>Popup/FramedCloud.html</li>
-    <li>Projection.html</li>
+    <li>Protocol.html</li>
-    <li>Renderer.html</li>
-    <li>Renderer/Elements.html</li>
@@ -118 +119 @@
-    <li>Request/XMLHttpRequest.html</li>
-    <li>Rule.html</li>
+    <li>Strategy.html</li>
-    <li>Style.html</li>
-    <li>StyleMap.html</li>