[Users] WFSCapabilitiesReader - default layer strategies
Alexandre Dube
adube at mapgears.com
Wed Mar 23 15:56:50 CET 2011
I would definitively like it. Thanks a lot for your help, Matt.
Alexandre
On 11-03-23 10:39 AM, Matt Priour wrote:
> Looks pretty good to me. I can write unit tests later this week if you
> would like.
> Matt Priour
>
> *From:* Alexandre Dube <mailto:adube at mapgears.com>
> *Sent:* Wednesday, March 23, 2011 8:15 AM
> *To:* users at geoext.org <mailto:users at geoext.org>
> *Cc:* Eric Lemoine <mailto:eric.lemoine at camptocamp.com> ; Matt Priour
> <mailto:mpriour at kestrelcomputer.com>
> *Subject:* Re: [Users] WFSCapabilitiesReader - default layer strategies
>
> Matt,
>
> http://trac.geoext.org/ticket/412#comment:3
>
> See the patch with the according changes your proposed. It looks
> cleaner to me. What do you think ?
>
> Alexandre
>
>
> On 11-03-23 07:57 AM, Alexandre Dube wrote:
>> Matt,
>>
>> You suggestion makes sense to me. I'll make the according changes
>> this morning and report back.
>>
>> Thanks a lot for your review,
>>
>> Alexandre
>>
>>
>> On 11-03-22 04:48 PM, Matt Priour wrote:
>>> Looking at the ticket & patch, I think that code is always going to
>>> be kind of ugly when you allow people to mix strings and objects in
>>> configuration like that.
>>> I think specifying either a pure string
>>> ex: "BBOX"
>>> or a pure object
>>> ex: {type:"BBOX",ratio:1.25}
>>> would be easier to deal with and would better match with the other
>>> configuration object conventions used in ExtJS, OL, & GeoExt
>>> However people could mix the 2 different configuration specification
>>> types in the strategies array:
>>> ex:
>>> strategies : [ "Save", {type:"BBOX",ratio:1.25} ]
>>> Thanks for getting this started.
>>> Matt Priour
>>> Kestrel Computer Consulting
>>>
>>> *From:* Alexandre Dube <mailto:adube at mapgears.com>
>>> *Sent:* Tuesday, March 22, 2011 3:02 PM
>>> *To:* Matt Priour <mailto:mpriour at kestrelcomputer.com>
>>> *Cc:* Eric Lemoine <mailto:eric.lemoine at camptocamp.com> ; GeoEXT
>>> Users <mailto:users at geoext.org>
>>> *Subject:* Re: [Users] WFSCapabilitiesReader - default layer strategies
>>>
>>> Hi,
>>>
>>> An update about this thread. I created a ticket :
>>> http://trac.geoext.org/ticket/412
>>>
>>> It contains a patch that fixes the issue. I don't really like how
>>> I made it (it could have been better, I think) and it's missing test
>>> units. Could someone give me a comment/hint about it, please ?
>>>
>>> Kind regards,
>>>
>>> Alexandre
>>>
>>>
>>> On 11-02-15 09:27 AM, Matt Priour wrote:
>>>> +1 on the option to use the short strategy name string with an
>>>> options object.
>>>> We could implement this immediately in GeoExt when creating layers
>>>> from the capabilities store and the layerOptions, if there is not
>>>> equal support for that syntax directly in OL.
>>>> I've also come across this issue and ended up solving it by
>>>> listening to 'add' event on the layer store and then modifying
>>>> options on the layers accordingly. I would much prefer the ability
>>>> to set the defaults in the layerOptions rather than having to
>>>> modify them after creation.
>>>> Matt Priour
>>>> Kestrel Computer Consulting
>>>>
>>>> *From:* Eric Lemoine <mailto:eric.lemoine at camptocamp.com>
>>>> *Sent:* Tuesday, February 15, 2011 7:51 AM
>>>> *To:* Alexandre Dube <mailto:adube at mapgears.com>
>>>> *Cc:* GeoEXT Users <mailto:users at geoext.org>
>>>> *Subject:* Re: [Users] WFSCapabilitiesReader - default layer strategies
>>>>
>>>> > What if we started supporting this :
>>>> > var layer = new OpenLayers.Layer.Vector("layer", {
>>>> > protocol: ...,
>>>> > strategies: ["OpenLayers.Strategy.BBOX"]
>>>> > });
>>>> >
>>>> > and this too :
>>>> > var layer = new OpenLayers.Layer.Vector("layer", {
>>>> > protocol: ...,
>>>> > strategies: [{"OpenLayers.Strategy.BBOX": {ratio: 1}}]
>>>> > });
>>>>
>>>> That was my suggestion. And I think {"BBOX"} and {"BBOX": {ratio: 1}}
>>>> could also be supported.
>>>>
>>>>
>>>>
>>>> --
>>>> Eric Lemoine
>>>>
>>>> Camptocamp France SAS
>>>> Savoie Technolac, BP 352
>>>> 73377 Le Bourget du Lac, Cedex
>>>>
>>>> Tel : 00 33 4 79 44 44 96
>>>> Mail : eric.lemoine at camptocamp.com <mailto:eric.lemoine at camptocamp.com>
>>>> http://www.camptocamp.com
>>>> _______________________________________________
>>>> Users mailing list
>>>> Users at geoext.org <mailto:Users at geoext.org>
>>>> http://www.geoext.org/cgi-bin/mailman/listinfo/users
>>>
>>>
>>> --
>>> Alexandre Dubé
>>> Mapgears
>>> www.mapgears.com
>>
>>
>> --
>> Alexandre Dubé
>> Mapgears
>> www.mapgears.com
>>
>>
>> _______________________________________________
>> Users mailing list
>> Users at geoext.org
>> http://www.geoext.org/cgi-bin/mailman/listinfo/users
>
>
> --
> Alexandre Dubé
> Mapgears
> www.mapgears.com
--
Alexandre Dubé
Mapgears
www.mapgears.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.geoext.org/pipermail/users/attachments/20110323/87ef65df/attachment-0001.htm
More information about the Users
mailing list