Hey- Ivan Grcic wrote: > OK... i found it..inside Layer.Vector...now the question is: should i > put something like this: > > if (OpenLayers.Util.indexOf(this.features, feature) != -1){ > > inside drawFeature: function(feature, style) {
Nope. Making drawFeature that safe at the expense of iterating through every feature with every draw is a mistake. > > or inside every other function inside Layer.Vector that uses > drawFeature function? > > When I put it here: > > drawFeature: function(feature, style) { > --> if (OpenLayers.Util.indexOf(this.features, feature) != -1){ > .. > .. > } > } > > it works pretty fine for me :) Eric? > > On Mon, Nov 3, 2008 at 9:53 AM, Ivan Grcic <[EMAIL PROTECTED]> wrote: >> Yup, its not Cluster Strategy... >> http://trac.openlayers.org/ticket/1806#comment:2 >> >> On Mon, Nov 3, 2008 at 9:11 AM, Eric Lemoine <[EMAIL PROTECTED]> wrote: >>> Hi. >>> >>> Yes, this looks like a bug in OpenLayers. >>> >>> I'm under the impression that this isn't a bug in the cluster strategy >>> but either in Layer.Vector or in Control.SelectFeature. Indeed, I >>> think SelectFeature calls layer.drawFeature on feature >>> selection/unselection without checking whether the feature is in the >>> layer. And i'm saying this may be a bug in Layer.Vector because it may >>> be that drawFeature should be responsible for checking whether the >>> feature is in the layer before actually drawing it. >>> >>> This is just coming from the top of my head so i may be wrong, but i'm >>> feeling this is a good track to follow :) >>> >>> Eric >>> >>> 2008/11/2, Ivan Grcic <[EMAIL PROTECTED]>: >>>> Hi list, >>>> >>>> i discovered a bug while developing my OL application, in cluster >>>> strategy class. >>>> >>>> Bug occurs when you hover over one feature that represent a cluster >>>> (or externalGraphic or like in cluster example - vector circle >>>> http://openlayers.org/dev/examples/strategy-cluster.html) >>>> >>>> When you zoom in/out while that feature-cluster is selected (by >>>> quickly double-clicking or easier using scroll on ur mouse), on next >>>> zoom that cluster is rendered (stays on screen) together with newly >>>> created clusters! And clearly it shouldnt be there. The old >>>> ("phantom") cluster is not selectable, as he doesnt exist on the new >>>> zoom level..he just stays there on screen...annoying me :) >>>> >>>> As you zoom in again and again, phantom clusters always render, always >>>> at the same position in the map div... >>>> >>>> Please confirm that this is a bug, im going crazy here with this one :| >>>> >>>> -- >>>> Ivan Grcic >>>> _______________________________________________ >>>> Dev mailing list >>>> Dev@openlayers.org >>>> http://openlayers.org/mailman/listinfo/dev >>>> >> >> >> -- >> Ivan Grcic >> > > > -- Tim Schaub OpenGeo - http://opengeo.org Expert service straight from the developers. _______________________________________________ Dev mailing list Dev@openlayers.org http://openlayers.org/mailman/listinfo/dev