[Users] WFSCapabilitiesReader - default layer strategies

[Matt Priour]

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 
Sent: Tuesday, March 22, 2011 3:02 PM
To: Matt Priour 
Cc: Eric Lemoine ; GeoEXT Users 
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 
  Sent: Tuesday, February 15, 2011 7:51 AM
  To: Alexandre Dube 
  Cc: GeoEXT Users 
  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
  http://www.camptocamp.com
  _______________________________________________
  Users mailing list
  Users at geoext.org
  http://www.geoext.org/cgi-bin/mailman/listinfo/users




-- 
Alexandre Dubé
Mapgears
www.mapgears.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.geoext.org/pipermail/users/attachments/20110322/ef208c56/attachment-0001.htm