Ticket #90: 90.patch

tests/lib/GeoExt/widgets/tree/LayerContainer.html

@@ -54 +54 @@
-            t.ok(node.firstChild.layer === layer, "child layer is correct");
-
-            node.destroy();
-//
+
-            
-        }
-        
@@ -117 +117 @@
-            t.eq(root.childNodes[3].layer.name, "c", "[c, a, b, d] first layer drawn at bottom of root");
-            
-            root.destroy();
-//
+
-            
-        }
-        

lib/GeoExt/widgets/tree/LayerContainer.js

@@ -100 +100 @@
-        }
-    },
-
-    /** private: method[onDestroy]
-     */
-    onDestroy: function() {
-        if(this.layerStore) {
-            this.layerStore.un("add", this.onStoreAdd, this);
-            this.layerStore.un("remove", this.onStoreRemove, this);
-        }
-        GeoExt.tree.LayerContainer.superclass.onDestroy.apply(this, arguments);
-    },
-    
-    /** private: method[recordIndexToNodeIndex]
-     *  :param index: ``Number`` The record index in the layer store.
-     *  :return: ``Number`` The appropriate child node index for the record.
@@ -220 +210 @@
-            this.layerStore.insert(newRecordIndex, [record]);
-            delete this._reordering;
-        }
+    },
+
+    /** private: method[destroy]
+     */
+    destroy: function() {
+        if(this.layerStore) {
+            this.layerStore.un("add", this.onStoreAdd, this);
+            this.layerStore.un("remove", this.onStoreRemove, this);
+        }
+        GeoExt.tree.LayerContainer.superclass.destroy.apply(this, arguments);
-    }
-    
-});
-
-/**

lib/GeoExt/widgets/tree/LayerNode.js

@@ -64 +64 @@
-        node.visibilityChanging = false;
-    },
-    
-onD
+d
-     */
-onD
+d
-        delete this.radio;
-onD
+d
-    }
-});
-
@@ -225 +225 @@
-     *  state
-     */
-    addVisibilityEventHandlers: function() {
-
-
-
-
-
-
+
+
+
+
-        this.on({
-
-
-
-
-
-
+
-            scope: this
-        });
-    },
-    
+    /** private: method[onLayerVisiilityChanged
+     *  handler for visibilitychanged events on the layer
+     */
+    onLayerVisibilityChanged: function() {
+        if(!this.visibilityChanging &&
+                this.attributes.checked != this.layer.getVisibility()) {
+            this.getUI().toggleCheck(this.layer.getVisibility());
+        }
+    },
+    
+    /** private: method[onCheckChange]
+     *  :param node: ``GeoExt.tree.LayerNode``
+     *  :param checked: ``Boolean``
+     *  handler for checkchange events 
+     */
+    onCheckChange: function(node, checked) {
+        if (checked && this.layer.isBaseLayer && this.layer.map) {
+            this.layer.map.setBaseLayer(this.layer);
+        }
+        this.layer.setVisibility(checked);
+    },
+    
-    /** private: method[addStoreEventHandlers]
-     *  Adds handlers that make sure the node disappeares when the layer is
-     *  removed from the store, and appears when it is re-added.
-     */
-    addStoreEventHandlers: function() {
-        this.layerStore.on({
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
-            scope: this
-        });
-    },
-    
+    /** private: method[onStoreAdd]
+     *  :param store: ``Ext.data.Store``
+     *  :param records: ``Array(Ext.data.Record)``
+     *  :param index: ``Nmber``
+     *  handler for add events on the store 
+     */
+    onStoreAdd: function(store, records, index) {
+        var l;
+        for(var i=0; i<records.length; ++i) {
+            l = records[i].get("layer");
+            if(this.layer == l) {
+                this.getUI().show();
+            } else if (this.layer == l.name) {
+                // layer is a string, which means the node has not yet
+                // been rendered because the layer was not found. But
+                // now we have the layer and can render.
+                this.render(bulkRender);
+                return;
+            }
+        }
+    },
+    
+    /** private: method[onStoreRemove]
+     *  :param store: ``Ext.data.Store``
+     *  :param record: ``Ext.data.Record``
+     *  :param index: ``Nmber``
+     *  handler for remove events on the store 
+     */
+    onStoreRemove: function(store, record, index) {
+        if(this.layer == record.get("layer")) {
+            this.getUI().hide();
+        }
+    },
+    
-    /** private: method[addChildNodes]
-     *  Calls the add method of a node type configured as ``childNodeType``
-     *  to add children.
@@ -282 +313 @@
-        } else if(typeof this.childNodeType.add === "function") {
-            this.childNodeType.add(this);
-        }
+    },
+    
+    /** private: method[destroy]
+     */
+    destroy: function() {
+        this.layer.events.un({
+            "visibilitychanged": this.onLayerVisibilityChanged,
+            scope: this
+        });
+        delete this.layer;
+        this.layerStore.un("add", this.onStoreAdd, this);
+        this.layerStore.un("remove", this.onStoreRemove, this);
+        delete this.layerStore;
+        this.un("checkchange", this.onCheckChange, this);
+
+        GeoExt.tree.LayerNode.superclass.destroy.call(this);
-    }
-});
-