Changeset 7938

Timestamp:

09/03/08 10:15:26 (3 months ago)

Author:

elemoine

Message:

apply patch attached to #1697

Files:

• sandbox/camptocamp/unhcr/lib/OpenLayers/Layer/Vector.js (modified) (2 diffs)
• sandbox/camptocamp/unhcr/tests/Layer/Vector.html (modified) (2 diffs)

sandbox/camptocamp/unhcr/lib/OpenLayers/Layer/Vector.js

@@ -482 +482 @@
-     */
-    removeFeatures: function(features, options) {
+        if(!features || features.length === 0) {
+            return;
+        }
-        if (!(features instanceof Array)) {
-            features = [features];
-        }
-        if (features.length <= 0) {
-            return;
-        }
-
@@ -556 +556 @@
-            features = this.features;
-        }
-
-
-
+
+
+
+
+
-        }
-    },

sandbox/camptocamp/unhcr/tests/Layer/Vector.html

@@ -258 +258 @@
-
-    function test_Layer_Vector_destroy (t) {
-4
+5
-
-        var options = {protocol: new OpenLayers.Protocol(), 
@@ -272 +272 @@
-        t.eq(layer.protocol, null, "layer.protocol is null after destroy");
-        t.eq(layer.strategies, null, "layer.strategies is null after destroy");
+        
+        // test that we can call layer.destroy a second time without trouble
+        try {
+            layer.destroy();
+            layer.destroy();
+            t.ok(true, "layer.destroy called twice without any issues");
+        } catch(err) {
+            t.fail("calling layer.destroy twice triggers exception: " + err + " in " + err.fileName + " line " + err.lineNumber);
+        }
+        
-    }
-