OpenLayers OpenLayers

Changeset 7387

Show
Ignore:
Timestamp:
06/19/08 18:08:10 (2 months ago)
Author:
crschmidt
Message:

"GeoRSS layer triggers loadstart event twice", from Bart. Includes tests.
(Closes #1580)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js

    r7339 r7387  
    122122        OpenLayers.Layer.Markers.prototype.moveTo.apply(this, arguments); 
    123123        if(this.visibility && !this.loaded){ 
    124             this.events.triggerEvent("loadstart"); 
    125124            this.loadRSS(); 
    126125        } 

  • trunk/openlayers/tests/Layer/GeoRSS.html

    r7015 r7387  
    7575 
    7676        });; 
     77    } 
     78    function test_Layer_GeoRSS_load_events (t) { 
     79        t.plan( 1 );     
     80        layer = new OpenLayers.Layer.GeoRSS('Test Layer', georss_txt); 
     81        var map = new OpenLayers.Map('map'); 
     82        var baseLayer = new OpenLayers.Layer.WMS("Test Layer",  
     83            "http://octo.metacarta.com/cgi-bin/mapserv?", 
     84            {map: "/mapdata/vmap_wms.map", layers: "basic"}); 
     85        map.addLayer(baseLayer); 
     86        map.addLayer(layer); 
     87        layer.events.register("loadstart", t, function() { this.ok(true, "loadstart event triggered once (#1580)") }); 
     88        map.setCenter(new OpenLayers.LonLat(0,0),0); 
    7789    } 
    7890    function test_Layer_GeoRSS_events (t) {