Re: [OpenLayers-Dev] Cluster strategy should not process data when layer is currently disabled (visibility=false)

2008-11-14 Thread Eric Lemoine
On Thu, Nov 13, 2008 at 10:12 PM, Eric Lemoine [EMAIL PROTECTED] wrote: Alexandre, Layers do not have the zoomend event type, so I don't think your patch is valid. Try with moveend and a check in the listener to handle zoom change only. Eric Alexandre, Could you apply the patches attached

Re: [OpenLayers-Dev] Cluster strategy should not process data when layer is currently disabled (visibility=false)

2008-11-14 Thread Alexandre Dube
Eric Lemoine wrote: Alexandre, Could you apply the patches attached to #1831 et #1834 and report back? Thanks, http://trac.openlayers.org/ticket/1831 http://trac.openlayers.org/ticket/1834 -- Eric Problem solved for me. No events ( like clustering) are triggered by the layer on

Re: [OpenLayers-Dev] Cluster strategy should not process data when layer is currently disabled (visibility=false)

2008-11-13 Thread Alexandre Dube
I get the same behavior as Ivan's on my application which has very similar resolution settings on map an vector layer. Because a layer's features are not destroyed when it goes out of range (inRange) or becomes invisible (visibility), the cluster strategy still clusters those existing

Re: [OpenLayers-Dev] Cluster strategy should not process data when layer is currently disabled (visibility=false)

2008-11-13 Thread Ivan Grcic
Hi Alex, i made a little hack to it...inside Strategy.Cluster.js inside cluster function I added one more check: (OpenLayers.Util.indexOf(this.layer.resolutions, resolution) != -1) it doesnt do the clustering if current resolution is not contained inside layers array of resolutions. Index:

Re: [OpenLayers-Dev] Cluster strategy should not process data when layer is currently disabled (visibility=false)

2008-11-13 Thread Eric Lemoine
You guys are good to find bugs :-) I think that, instead of listening to the map's zoomend events, the cluster strategy should listen to its layer's moveend events and check in the listener that this is actually a zoom change before doing anything. The moveend listener receives an object with a

Re: [OpenLayers-Dev] Cluster strategy should not process data when layer is currently disabled (visibility=false)

2008-11-13 Thread Tim Schaub
Hey- Eric Lemoine wrote: Alexandre, Layers do not have the zoomend event type, so I don't think your patch is valid. Try with moveend and a check in the listener to handle zoom change only. Eric And note that moveend is not triggered for base layers. This is the issue I was referring