Thanks (and sorry for the huge patch).
I will try to describe the the transformation in more detail:
basically the problem is that since all the function implementations implement
both geoapi and geotoos function interfaces, they're not synchronized in
order to behave correctly whether one or the other is used
1- a new constructor to set the function name, since some functions, though
extending AbstractFunctionImpl, were returning null in getName
function_x()
super("X");
2- new method setParameters and List getParameters
This change implies that setArgs delegates in setParamenters to set the
instance variable that maintains the parameters (or arguments) in a
ArrayList; as a bunch of functions were storing the arguments by themselves
but getArgs/getParameters and setArgs/setParameters were inconsistent
3- specific setting method to update parameters like setExpression was changed
to taking into account the new parameters allocation (ArrayList)
4- new method evaluate() that implements the function evaluation delegating
in the old getValue(), as much of them did not implement evaluate.
5- Additionaly, a new test was added to assure that the new and the old
interface go well: org.geotools.filter.FunctionExpressionImplTest
If you need any more, do not hesitate to contact me.
Thanks a lots
On Friday 18 May 2007 10:55, you wrote:
> Mauricio,
> the patches are huge, and I guess they are doing some regular
> transformation on the filters.
> I would be helpful to know that the transformation is without
> scrolling down 480kb of patch :)
> Cheers
> Andrea
--
Mauricio Pazos
www.axios.es
tel-:+34 94 441 63 84
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
