[Users] WFSCapabilitiesReader - default layer strategies

[Alexandre Dube]

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.geoext.org/pipermail/users/attachments/20110323/73c7a5f2/attachment.htm