[OpenLayers-Commits] r2957 - trunk/openlayers/lib/OpenLayers

[commits at openlayers.org]

Author: sderle
Date: 2007-04-01 17:56:20 -0400 (Sun, 01 Apr 2007)
New Revision: 2957

Modified:
   trunk/openlayers/lib/OpenLayers/Map.js
Log:
Clean up layers and controls a little more carefully in map.destroy(). Fixes #596.


Modified: trunk/openlayers/lib/OpenLayers/Map.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Map.js	2007-04-01 21:37:17 UTC (rev 2956)
+++ trunk/openlayers/lib/OpenLayers/Map.js	2007-04-01 21:56:20 UTC (rev 2957)
@@ -243,7 +243,7 @@
         this.unloadDestroy = null;
 
         if (this.layers != null) {
-            for(var i=0; i< this.layers.length; i++) {
+            for (var i = this.layers.length - 1; i>=0; --i) {
                 //pass 'false' to destroy so that map wont try to set a new 
                 // baselayer after each baselayer is removed
                 this.layers[i].destroy(false);
@@ -251,7 +251,7 @@
             this.layers = null;
         }
         if (this.controls != null) {
-            for(var i=0; i< this.controls.length; i++) {
+            for (var i = this.controls.length - 1; i>=0; --i) {
                 this.controls[i].destroy();
             } 
             this.controls = null;