Changeset 5900

Timestamp:

01/25/08 19:00:37 (10 months ago)

Author:

euzuro

Message:

arg parser triggers load of baselayer too early. thanks for the patch, kleptog. i've taken it upon myself to run the damn tests. all pass. (Closes #1290)

Files:

• trunk/openlayers/lib/OpenLayers/Control/ArgParser.js (modified) (2 diffs)

trunk/openlayers/lib/OpenLayers/Control/ArgParser.js

@@ -87 +87 @@
-
-            var args = OpenLayers.Util.getParameters();
+            // Be careful to set layer first, to not trigger unnecessary layer loads
+            if (args.layers) {
+                this.layers = args.layers;
+    
+                // when we add a new layer, set its visibility 
+                this.map.events.register('addlayer', this, 
+                                         this.configureLayers);
+                this.configureLayers();
+            }
-            if (args.lat && args.lon) {
-                this.center = new OpenLayers.LonLat(parseFloat(args.lon),
@@ -98 +107 @@
-                                         this.setCenter);
-                this.setCenter();
-            }
-    
-            if (args.layers) {
-                this.layers = args.layers;
-    
-                // when we add a new layer, set its visibility 
-                this.map.events.register('addlayer', this, 
-                                         this.configureLayers);
-                this.configureLayers();
-            }
-        }