[OpenLayers-Dev] Text Layer and loadstart

[Erik Uzureau]

good looking out, bart. I've opened a ticket
http://trac.openlayers.org/ticket/1696

if you feel like making the patch, i'll approve it. otherwise, i can
make it when i get some freetime
erik

On Fri, Aug 22, 2008 at 11:42 AM, Bart van den Eijnden (OSGIS)
<bartvde at osgis.nl> wrote:
> 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
>
>