[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