[Gvsig_desarrolladores] Filter Extension problem with doubles
Nacho Uve
nachouve en gmail.com
Mar Dic 28 19:09:23 CET 2010
Yes!! You are right!! Thank you, Puga!!
Maybe the best choice is the second one ("pass a different actionCommand
string").
I'll wait for core-developers opinion before fix my patch.
Regards,
Nacho Varela
2010/12/28 Francisco Puga <fpuga en cartolab.es>
> Hi,
>
> I just review the patch and i found a trouble with it. I wrote my
> comment in the bugtracker but also i paste it here:
>
> There is a problem with the provided patch. If you use the filter over
> a layer (from the FilterExtension itself or from other extension), the
> attribute datasource of the class is set, so the second time that you
> try to use it from the FilterExtension it will use the previous
> datasource and not the new.
>
> I think that there are three different approach to handle this:
> * Set datasource to null after calling the method doExecute()
> * Pass a different actionCommand string to the execute method if it's
> used from a external extension
> * Maybe, refactor to extract the logic from the class extension
> itself, and instantiated the new class each time is needed.
>
> El día 28 de diciembre de 2010 13:19, Nacho Uve <nachouve en gmail.com>
> escribió:
> > Although not fully related to the previous mail, I would like report on
> the
> > new ticket[1] that I sent to Tracker about FiltroExtension.
> >
> > That's a trivial patch to add a new method setDataSource() on
> > FiltroExtension. That change do NOT modify the current behaviour of the
> > tool, but it allows to use it on other contexts (p.e with NavTable[2]).
> >
> > Please, take a look to the patch and tell me if it is possible to apply
> > it... and of course, any idea about "filter decimal bug"?
> >
> > Thank you!
> >
> > Regards,
> > Nacho Varela
> >
> > [1]:
> >
> https://forge.osor.eu/tracker/index.php?func=detail&aid=14912&group_id=89&atid=790
> > [2]:
> >
> https://forge.osor.eu/tracker/index.php?func=detail&aid=14908&group_id=122&atid=462
> _______________________________________________
> gvSIG_desarrolladores mailing list
> gvSIG_desarrolladores en listserv.gva.es
> http://listserv.gva.es/cgi-bin/mailman/listinfo/gvsig_desarrolladores
>
------------ próxima parte ------------
Se ha borrado un adjunto en formato HTML...
URL: http://listserv.gva.es/pipermail/gvsig_desarrolladores/attachments/20101228/e3d571ac/attachment.htm
Más información sobre la lista de distribución gvSIG_desarrolladores