I think you asked on the devel list - something along the lines of "this
has been available on master for a month" any objects to back porting it to
11.x :)
Since it has been a month, and it is not right before the 11.2 release we
should be good.
So for reference this fixes a few ECQL encoding / decoding issues (but not
all of them). In addition to "fixing" the representation of FeatureId in an
IN filter (GEOT-4783 <https://jira.codehaus.org/browse/GEOT-4783>):
IN (river.1,river.2) should not be used, the correct representation is IN
('river.1','river.2')
This change recognises the OR encoding of an IN statement used to check
attributes along with a few other special cases.
When it recognises a special case the ECQL notation will be used rather
than the plain CQL OR representation
Example: if you are just testing numbers you can use x IN (1,2,3)
The is an alternative to: x = 1 OR x = 2 OR x =3
I also messed with brackets a bit so they are only used in the event of
nested logical expressions, the alternative would be (x = 1) OR (x=2) OR
(x=3)
This work is available on a single commit:
https://github.com/geotools/geotools/commit/649853a304a4779ed86121151193ea0fe049f367
Comment on April 30th: "Resolved on master, will back port to stable if it
lasts the month"
--
Jody
Jody Garnett
On Wed, Jun 25, 2014 at 9:35 AM, Andrew Hulbert <[email protected]> wrote:
> Looks like it was not backported yet when comparing this file
> (specifically line 116) on 11.x branch versus master:
>
>
> https://github.com/geotools/geotools/blob/11.x/modules/library/cql/src/main/java/org/geotools/filter/text/ecql/FilterToECQL.java
>
> That's one still causing a ClassCastException.
>
> Let me know what the process is to get it backported.
>
> Thanks for taking a look.
>
> Andrew
>
>
> On 06/24/2014 06:38 PM, Jody Garnett wrote:
>
> I cannot remember if I back ported it or not - however I do not know of
> any reason why it cannot be taken across (the changes were pretty limited).
>
> jody
>
> Jody Garnett
>
>
> On Tue, Jun 24, 2014 at 10:24 PM, Andrew Hulbert <[email protected]>
> wrote:
>
>> Jody,
>>
>> Finally getting back around to this...
>>
>> Just wondering if its possible to get a backport of
>> https://github.com/geotools/geotools/pull/431 to the 11.x branch?
>>
>> I might be have time in a few weeks to reformat it and test it against
>> 11.x but figured I'd ask first!
>>
>> Thanks,
>>
>> Andrew
>>
>>
>> ------------------------------------------------------------------------------
>> Open source business process management suite built on Java and Eclipse
>> Turn processes into business applications with Bonita BPM Community
>> Edition
>> Quickly connect people, data, and systems into organized workflows
>> Winner of BOSSIE, CODIE, OW2 and Gartner awards
>> http://p.sf.net/sfu/Bonitasoft
>> _______________________________________________
>> GeoTools-Devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>
>
>
------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
