Ticket #106: 106.3.patch

File 106.3.patch, 2.3 kB (added by tschaub, 1 year ago)

patch against trunk/geoext

  • tests/lib/GeoExt/data/LayerStore.html

    old new  
    8484            t.eq(mapPanel.layers.getCount(),1,"Adding layers to MapPanel's LayerStore does not create duplicate layers");  
    8585        } 
    8686 
     87        function test_store_to_map(t) { 
     88 
     89            t.plan(8); 
     90             
     91            var map = new OpenLayers.Map("mappanel"); 
     92            var layers = [new OpenLayers.Layer.Vector("a"), 
     93                          new OpenLayers.Layer.Vector("b"), 
     94                          new OpenLayers.Layer.Vector("c")]; 
     95 
     96            var store = new GeoExt.data.LayerStore({ 
     97                map: map, 
     98                layers: layers 
     99            }); 
     100 
     101            t.eq(store.getCount(), 3, "three layers in store"); 
     102            t.eq(map.layers.length, 3, "three layers on map"); 
     103 
     104            t.eq(store.getAt(0).get("layer").name, "a", "first layer correct in store"); 
     105            t.eq(map.layers[0].name, "a", "first layer correct on map"); 
     106 
     107            t.eq(store.getAt(1).get("layer").name, "b", "second layer correct in store"); 
     108            t.eq(map.layers[1].name, "b", "second layer correct on map"); 
     109 
     110            t.eq(store.getAt(2).get("layer").name, "c", "third layer correct in store"); 
     111            t.eq(map.layers[2].name, "c", "third layer correct on map"); 
     112 
     113            map.destroy(); 
     114        } 
     115 
    87116        function test_load_clear(t) { 
    88117            t.plan(2); 
    89118             
  • lib/GeoExt/data/LayerStore.js

    old new  
    135135        var layers = map.layers.slice(0); 
    136136 
    137137        if(initDir & GeoExt.data.LayerStore.STORE_TO_MAP) { 
    138             var records = this.getRange(); 
    139             for(var i=records.length - 1; i>=0; i--) { 
    140                 this.map.addLayer(records[i].get("layer")); 
    141             } 
     138            this.each(function(record) { 
     139                this.map.addLayer(record.get("layer")); 
     140            }, this); 
    142141        } 
    143142        if(initDir & GeoExt.data.LayerStore.MAP_TO_STORE) { 
    144143            this.loadData(layers, true);