ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Geometry.Collection does not call parent destructor, so bounds are not reset, r=fredj (closes #2688)

Browse Source 
git-svn-id: http://svn.openlayers.org/trunk/openlayers@10405 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf
This commit is contained in:
bartvde
2010-06-16 14:25:15 +00:00
parent eb4d5a2447
commit cca1395fc6
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/OpenLayers/Geometry/Collection.js
+1
View File
@@ -63,6 +63,7 @@ OpenLayers.Geometry.Collection = OpenLayers.Class(OpenLayers.Geometry, {
destroy: function () { destroy: function () {
this.components.length = 0; this.components.length = 0;
this.components = null; this.components = null;
OpenLayers.Geometry.prototype.destroy.apply(this, arguments);
}, },
/** /**
tests/Geometry/Collection.html
+6 -4
View File
@@ -252,17 +252,19 @@
} }
function test_Collection_destroy(t) { function test_Collection_destroy(t) {
t.plan( 1 ); t.plan( 3 );
coll = new OpenLayers.Geometry.Collection(); coll = new OpenLayers.Geometry.Collection();
coll.components = {}; coll.addComponents(new OpenLayers.Geometry.Point(0,0));
coll.addComponents(new OpenLayers.Geometry.Point(10,10));
coll.getBounds();
coll.destroy(); coll.destroy();
t.ok(coll.components == null, "components array cleared"); t.ok(coll.components == null, "components array cleared");
t.ok(coll.getBounds() == null, "bounds is cleared");
t.ok(coll.id == null, "id is cleared");
} }
</script> </script>
</head> </head>
<body> <body>