[OpenLayers-Dev] Text Layer and loadstart

[Bart van den Eijnden (OSGIS)]

Hi Erik,

I haven't been using the Text layer that much lately to be honest, but 
this changeset looks to have fixed that issue.

Though if I read the code correctly, it seems that loadstart is 
triggered twice? First in moveTo and then in loadText? This was fixed 
for GeoRSS [1], but maybe not for Text.

[1] http://trac.openlayers.org/ticket/1580

Best regards,
Bart

Erik Uzureau wrote:
> Bart, Chris,
>
> I just looked in the code in trunk and this no longer seems to be the case....
> Looks like this changeset got it:
> http://trac.openlayers.org/changeset/5828#file2
>
> Can you confirm?
>
> Erik
>
> On Wed, Dec 19, 2007 at 9:54 AM, Christopher Schmidt
> <crschmidt at metacarta.com> wrote:
>   
>> On Wed, Dec 19, 2007 at 09:51:25AM +0100, Bart van den Eijnden (OSGIS) wrote:
>>     
>>> Hi list,
>>>
>>> with the current code it seems impossible to listen to a loadstart event on
>>> the Text Layer, since it already fires this event in the constructor.
>>>
>>> Or am I missing something?
>>>       
>> Nope. That's a bug. The load should not actually be started in
>> initialize: instead, it should be started in moveTo, if it's not yet
>> loaded. Layer.GML does this, Layer.GeoRSS and Layer.Text do not. A patch
>> for this would be great. Please open a ticket -- if you plan to work on
>> it, or work with someone else on it, pleaes put it in 2.6 and I'll
>> review when it's completed.
>>
>> Regards,
>> --
>> Christopher Schmidt
>> MetaCarta
>> _______________________________________________
>> Dev mailing list
>> Dev at openlayers.org
>> http://openlayers.org/mailman/listinfo/dev
>>
>>     
>
>
>   


-- 
Bart van den Eijnden
OSGIS, Open Source GIS
bartvde at osgis.nl
http://www.osgis.nl