If this is going to be a fix that causes any regressions we should try and get it in quick with a major release coming up. Reading https://osgeo-org.atlassian.net/browse/GEOT-5398 the single issues reports three bugs.
You should modify the code to be consistent - base don your description you are looking at inconsistencies with how property names are handled - probably to account for property names that have a space in the the name. One thing that confuses me in your bug report is the reference to double quotes. *org.geotools.filter.text.commons.ExpressionToTextvisit(final PropertyName expression, final Object extraData) must treat the property name "id" as a special case and output it with double quote*s. So you are looking at capturing a property name like FIELD NAME as "FIELD NAME" (or is it [FIELD NAME]). -- Jody Garnett On 4 April 2016 at 10:27, Dipl.-Ing. B. Strauss <servi...@strauss.eng.br> wrote: > Hi all, > > my name is Burkhard Strauss and I am using geotools since 2007. > > The bug I reported (GEOT-5398) was fixed already a year ago. But there is > one issue left: The deprecated ECQL Syntax "ID IN(...)" is still accepted > by the parser. Thus a property named ID must be delimited by > ExpressionToText.visit(PropertyName > expression, Object extraData). > > On the other hand the java-doc of ECQLIDPredicateTest.deprecatedSyntax() > says > > deprecated syntax (TODO it should be unsupported in the next version) > > > Should I modify the parser or the ExpressionToText filter visitor? > > Regards > Burkhard Strauss > > > > > ------------------------------------------------------------------------------ > > _______________________________________________ > GeoTools-Devel mailing list > GeoTools-Devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel > >
------------------------------------------------------------------------------
_______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
