Ticket #2156: 2156.patch

tests/Map.html

@@ -1297 +1297 @@
-        map.destroy();
-        
-    }
+    
+    function test_panTo(t) {
+        
+        t.plan(2);
+        
+        var map = new OpenLayers.Map("map");
+        map.addLayer(
+            new OpenLayers.Layer(null, {isBaseLayer: true})
+        );
+        map.setCenter(new OpenLayers.LonLat(0, 0), 0);
+        
+        map.panTo(new OpenLayers.LonLat(1, 0));
+        t.eq(map.panTween.playing, true, "the map pan tween is playing before destroy");
+        
+        map.destroy();
+        t.ok(!map.panTween || !map.panTween.playing, "the map pan tween is not playing after destroy");
+        
+    }
-
-  </script>
-</head>

lib/OpenLayers/Map.js

@@ -632 +632 @@
-        if (!this.unloadDestroy) {
-            return false;
-        }
+        
+        // make sure panning doesn't continue after destruction
+        if(this.panTween && this.panTween.playing) {
+            this.panTween.stop();
+        }
-
-        // map has been destroyed. dont do it again!
-        OpenLayers.Event.stopObserving(window, 'unload', this.unloadDestroy);