Changeset 3087

Timestamp:

04/19/07 16:00:15 (2 years ago)

Author:

crschmidt

Message:

Fix ka-Map's clone method, to fix bug with Overview Map reported by Stephen
Woodbridge. (Thx Stephen.)

Files:

• trunk/openlayers/lib/OpenLayers/Layer/KaMap.js (modified) (1 diff)
• trunk/openlayers/tests/Layer/test_KaMap.html (modified) (2 diffs)

trunk/openlayers/lib/OpenLayers/Layer/KaMap.js

@@ -146 +146 @@
-
-    },
+    
+    /**
+     * @param {Object} obj
+     * 
+     * @returns An exact clone of this OpenLayers.Layer.Grid
+     * @type OpenLayers.Layer.Grid
+     */
+    clone: function (obj) {
+        
+        if (obj == null) {
+            obj = new OpenLayers.Layer.KaMap(this.name,
+                                            this.url,
+                                            this.params,
+                                            this.options);
+        }
+
+        //get all additions from superclasses
+        obj = OpenLayers.Layer.Grid.prototype.clone.apply(this, [obj]);
+
+        // copy/set any non-init, non-simple values here
+        if (this.tileSize != null) {
+            obj.tileSize = this.tileSize.clone();
+        }
+        
+        // we do not want to copy reference to grid, so we make a new array
+        obj.grid = new Array();
+
+        return obj;
+    },    
+
-
-    /** @final @type String */

trunk/openlayers/tests/Layer/test_KaMap.html

@@ -148 +148 @@
-
-    function test_10_Layer_KaMap_clone(t) {
-4
+5
-        
-        var options = {tileSize: new OpenLayers.Size(500,50)};
@@ -169 +169 @@
-
-        t.eq( clone.alpha, layer.alpha, "alpha copied correctly");
+        
+        t.eq( clone.CLASS_NAME, "OpenLayers.Layer.KaMap", "Clone is a ka-map layer");
-
-        layer.grid = null;