OpenLayers OpenLayers

Ticket #577: layerswitcherMem.2.patch

File layerswitcherMem.2.patch, 5.8 kB (added by euzuro, 2 years ago)

as per previous comment, i have refitted this patch so that it does things the proper way, which means cleaning up after itself -- not just letting things slide until the unloadCache() is called. Also, this improvement to the patch fixes a problem in the updateMap() function where we were still relying on the layer to be tagged off of the inputElement (the circular ref we are here deleting)

  • lib/OpenLayers/Control/LayerSwitcher.js

    old new  
    2525    baseLayersDiv: null, 
    2626 
    2727    /** @type Array */ 
    28     baseLayerInputs: null, 
     28    baseLayers: null, 
    2929     
    3030     
    3131    /** @type DOMElement */ 
    …… 
    3535    dataLayersDiv: null, 
    3636 
    3737    /** @type Array */ 
    38     dataLayerInputs: null, 
     38    dataLayers: null, 
    3939 
    4040 
    4141    /** @type DOMElement */ 
    …… 
    8585        return this.div; 
    8686    }, 
    8787 
     88    /** user specifies either "base" or "data". we then clear all the  
     89     *    corresponding listeners, the div, and reinitialize a new array. 
     90     *  
     91     * @private 
     92     *  
     93     * @param {String} layersType Ei 
     94     */ 
     95    clearLayersArray: function(layersType) { 
     96        var layers = this[layersType + "Layers"]; 
     97        if (layers) { 
     98            for(var i=0; i < layers.length; i++) { 
     99                var layer = layers[i]; 
     100                OpenLayers.Event.stopObservingElement(layer.inputElem); 
     101                OpenLayers.Event.stopObservingElement(layer.labelSpan); 
     102            } 
     103        } 
     104        this[layersType + "LayersDiv"].innerHTML = ""; 
     105        this[layersType + "Layers"] = new Array(); 
     106    }, 
     107 
     108 
    88109    /** Goes through and takes the current state of the Map and rebuilds the 
    89110     *   control to display that state. Groups base layers into a radio-button 
    90111     *   group and lists each data layer with a checkbox. 
    …… 
    95116    redraw: function() { 
    96117 
    97118        //clear out previous layers  
    98         this.baseLayersDiv.innerHTML = ""; 
    99         this.baseLayerInputs = new Array(); 
     119        this.clearLayersArray("base"); 
     120        this.clearLayersArray("data"); 
    100121         
    101         this.dataLayersDiv.innerHTML = ""; 
    102         this.dataLayerInputs = new Array(); 
    103          
    104122        var containsOverlays = false; 
    105123         
    106124        var layers = this.map.layers.slice(); 
    …… 
    128146                inputElem.value = layer.name; 
    129147                inputElem.checked = checked; 
    130148                inputElem.defaultChecked = checked; 
    131                 inputElem.layer = layer; 
    132                 inputElem.control = this; 
    133149 
    134150                if (!baseLayer && !layer.inRange) { 
    135151                    inputElem.disabled = true; 
    136152                } 
     153                var context = { 
     154                    'inputElem': inputElem, 
     155                    'layer': layer, 
     156                    'layerSwitcher': this 
     157                } 
    137158                OpenLayers.Event.observe(inputElem, "mouseup",  
    138                               this.onInputClick.bindAsEventListener(inputElem)); 
     159                              this.onInputClick.bindAsEventListener(context)); 
    139160                 
    140161                // create span 
    141162                var labelSpan = document.createElement("span"); 
    …… 
    143164                    labelSpan.style.color = "gray"; 
    144165                } 
    145166                labelSpan.innerHTML = layer.name; 
    146                 labelSpan.style.verticalAlign = (baseLayer) ? "bottom" : "baseline"; 
     167                labelSpan.style.verticalAlign = (baseLayer) ? "bottom"  
     168                                                            : "baseline"; 
    147169                OpenLayers.Event.observe(labelSpan, "click",  
    148                               this.onInputClick.bindAsEventListener(inputElem)); 
     170                              this.onInputClick.bindAsEventListener(context)); 
    149171                // create line break 
    150172                var br = document.createElement("br"); 
    151173     
    152174                 
    153                 var groupArray = (baseLayer) ? this.baseLayerInputs 
    154                                              : this.dataLayerInputs; 
     175                var groupArray = (baseLayer) ? this.baseLayers 
     176                                             : this.dataLayers; 
    155177                groupArray.push(inputElem); 
    156178                                                      
    157179     
    …… 
    173195     *  
    174196     * @private 
    175197     *  
     198     * @context  
     199     *      {DOMElement} inputElem 
     200     *      {OpenLayers.Control.LayerSwitcher} layerSwitcher 
     201     *      {OpenLayers.Layer} layer 
     202     *  
    176203     * @param {Event} e 
    177204     */ 
    178205 
    179206    onInputClick: function(e) { 
    180         if (!this.disabled) { 
    181             if (this.type == "radio") { 
    182                 this.checked = true; 
     207 
     208        if (!this.inputElem.disabled) { 
     209            if (this.inputElem.type == "radio") { 
     210                this.inputElem.checked = true; 
    183211                this.layer.map.setBaseLayer(this.layer, true); 
    184212                this.layer.map.events.triggerEvent("changebaselayer"); 
    185213            } else { 
    186                 this.checked = !this.checked; 
    187                 this.control.updateMap(); 
     214                this.inputElem.checked = !this.checked; 
     215                this.layerSwitcher.updateMap(); 
    188216            } 
    189217        } 
    190218        OpenLayers.Event.stop(e); 
    …… 
    211239    updateMap: function() { 
    212240 
    213241        // set the newly selected base layer         
    214         for(var i=0; i < this.baseLayerInputs.length; i++) { 
    215             var input = this.baseLayerInputs[i];    
    216             if (input.checked) { 
    217                 this.map.setBaseLayer(input.layer, false); 
     242        for(var i=0; i < this.baseLayers.length; i++) { 
     243            var layerEntry = this.baseLayers[i]; 
     244            if (layerEntry.input.checked) { 
     245                this.map.setBaseLayer(layerEntry.layer, false); 
    218246            } 
    219247        } 
    220248 
    221249        // set the correct visibilities for the overlays 
    222         for(var i=0; i < this.dataLayerInputs.length; i++) { 
    223             var input = this.dataLayerInputs[i];    
    224             input.layer.setVisibility(input.checked, true); 
     250        for(var i=0; i < this.dataLayers.length; i++) { 
     251            var layerEntry = this.dataLayers[i];    
     252            layerEntry.layer.setVisibility(layerEntry.input.checked, true); 
    225253        } 
    226254 
    227255    },