Changeset 1585

Timestamp:

10/05/06 02:20:47 (2 years ago)

Author:

euzuro

Message:

fix for #191 - graceful dealing with setCenter() on lonlat out of maxexteng (on first setcenter() call). added test

Files:

• trunk/openlayers/lib/OpenLayers/Map.js (modified) (1 diff)
• trunk/openlayers/tests/test_Map.html (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Map.js

@@ -577 +577 @@
-    */
-    setCenter: function (lonlat, zoom, dragging) {
+        
+        if (!this.center && !this.isValidLonLat(lonlat)) {
+            lonlat = this.maxExtent.getCenterLonLat();
+        }
+        
-        var zoomChanged = (this.isValidZoomLevel(zoom)) && 
-                          (zoom != this.getZoom());

trunk/openlayers/tests/test_Map.html

@@ -248 +248 @@
-    }
-    
+    function test_088_Map_setCenter(t) {
+        t.plan(1);
+        map = new OpenLayers.Map($('map'));
+        var baseLayer = new OpenLayers.Layer.WMS("Test Layer", 
+            "http://octo.metacarta.com/cgi-bin/mapserv?",
+            {map: "/mapdata/vmap_wms.map", layers: "basic"},
+            {maxResolution: 'auto', maxExtent: new OpenLayers.Bounds(-10,-10,10,10)});
+        map.addLayer(baseLayer);
+        var ll = new OpenLayers.LonLat(-100,-150);
+        map.setCenter(ll, 0);
+        t.ok(map.getCenter().equals(new OpenLayers.LonLat(0,0)), "safely sets out-of-bounds lonlat");
+    }
+
-    
-    function test_99_Map_destroy (t) {