Ticket #1621: patch-1621-r7538-A0.diff

tests/Renderer/Elements.html

@@ -308 +308 @@
-    }
-    
-    function test_Elements_erasegeometry(t) {
-4
+5
-        
-        var el = document.createElement('div');
-        document.body.appendChild(el);
@@ -358 +358 @@
-        
-        OpenLayers.Renderer.Elements.prototype.eraseGeometry(geometry);
-        t.ok(el.parentNode != document.body, "geometry components correctly removed when passed a multipolygon");
+        
+        var el = document.createElement('div');
+        document.body.appendChild(el);
+        el.id = 'bar';
+        var geometry = {
+            CLASS_NAME: "OpenLayers.Geometry.Collection",
+            components: [{
+                id: 'bar'
+            }]
+        };
+        
+        OpenLayers.Renderer.Elements.prototype.eraseGeometry(geometry);
+        t.ok(el.parentNode != document.body, "geometry components correctly removed when passed a collection");
-    }    
-
-  </script>

lib/OpenLayers/Renderer/Elements.js

@@ -332 +332 @@
-    eraseGeometry: function(geometry) {
-        if ((geometry.CLASS_NAME == "OpenLayers.Geometry.MultiPoint") ||
-            (geometry.CLASS_NAME == "OpenLayers.Geometry.MultiLineString") ||
-
+
+
-            for (var i = 0; i < geometry.components.length; i++) {
-                this.eraseGeometry(geometry.components[i]);
-            }