Apologies to anyone who was holding their breath on this one; it's taken me this long to look into the issue again. What I found was that GeoAPI still wasn't updating with just a 'cd geotools; mvn install' so I grabbed GeoAPI from svn and built a patch against that. I have updated the ticket with a patch for GT2.5.x that includes stubs for some other factory methods added in the interim. I don't currently have a checkout of trunk, but will verify my changes apply there and update the ticket if needed sometime soon.
thanks. -- David Winslow OpenGeo - http://opengeo.org/ On Wed, 2009-03-25 at 00:48 +1100, Jody Garnett wrote: > Doh; good catch. It is super late for me now so I am going to have to > catch this one tomorrow. > Jody > > On Wed, Mar 25, 2009 at 12:46 AM, David Winslow <[email protected]> wrote: > > On Tue, 2009-03-24 at 07:36 +1100, Jody Garnett wrote: > >> Okay David the code is ready for you to create a patch (both 2.5.x and > >> trunk). I got slammed by the refractions repository going down so I > >> was unable to commit everything until now. > >> > >> Jody > > > > I don't see a matchCase parameter in FilterFactory2.like() on GeoAPI > > 2.2-SNAPSHOT. Should I just put a cast to LikeFilterImpl in the parser > > so I can turn on matchCase? > > > > -- > > David Winslow > > OpenGeo - http://opengeo.org/ > > > >> > >> On Sat, Mar 21, 2009 at 2:21 AM, David Winslow <[email protected]> > >> wrote: > >> > I'd like to see this in the next GeoServer release, so yes, I need this > >> > patch applied to GeoAPI 2.2-SNAPSHOT and GeoTools 2.5.x. I haven't even > >> > tried applying it to gt trunk, but of course I'll be happy to resolve > >> > any conflicts that pop up. > >> > > >> > -- > >> > David Winslow > >> > OpenGeo - http://opengeo.org/ > >> > > >> > On Fri, 2009-03-20 at 17:46 +1100, Jody Garnett wrote: > >> >> Created a GEOAPI issue here: http://jira.codehaus.org/browse/GEO-158 > >> >> > >> >> Question for you; do you need this patch applied to geoAPI > >> >> 2.2-SNAPSHOT (ie for GeoTools 2.5.x?) > >> >> Jody > >> >> > >> >> On Thu, Mar 19, 2009 at 1:03 PM, Jody Garnett <[email protected]> > >> >> wrote: > >> >> > That it would; attach to the jira issue when ready. > >> >> > > >> >> > Andrea I noticed that there is a "patch attached" field I can turn on > >> >> > in Jira these days; would that be helpful to identify issues that are > >> >> > so important a developer has made a patch for them? > >> >> > > >> >> > Jody > >> >> > > >> >> > On Thu, Mar 19, 2009 at 12:14 PM, David Winslow > >> >> > <[email protected]> wrote: > >> >> >> matchCase support would definitely take care of my use case. I can > >> >> >> put > >> >> >> together a patch for GeoTools if that helps. > >> >> >> > >> >> >> -- > >> >> >> David Winslow > >> >> >> OpenGeo - http://opengeo.org/ > >> >> >> > >> >> >> On Thu, 2009-03-19 at 10:27 +1100, Jody Garnett wrote: > >> >> >>> I was going to say that there is ad difference between what the code > >> >> >>> can do and what the SLD format will allow. > >> >> >>> > >> >> >>> That for exploring the options Andrea; adding the matchCase option > >> >> >>> to > >> >> >>> GeoAPI is a good plan - shall we do it. > >> >> >>> > >> >> >>> Jody > >> >> >>> > >> >> >>> On Thu, Mar 19, 2009 at 1:56 AM, Andrea Aime <[email protected]> > >> >> >>> wrote: > >> >> >>> > David Winslow ha scritto: > >> >> >>> >> Hi all, > >> >> >>> >> > >> >> >>> >> I was trying to do a case-insensitive search against a text > >> >> >>> >> field by > >> >> >>> >> making use of the newly added strToLower and strToUpper filter > >> >> >>> >> functions, only to find that GeoTools' parser expects a > >> >> >>> >> PropertyName and > >> >> >>> >> Literal as the contents of the PropertyIsLike, even though the > >> >> >>> >> filter > >> >> >>> >> object itself supports an arbitrary expression for the > >> >> >>> >> "property" field. > >> >> >>> >> > >> >> >>> >> I created a ticket with a patch that makes it act as I would > >> >> >>> >> expect: > >> >> >>> >> http://jira.codehaus.org/browse/GEOT-2396 > >> >> >>> >> > >> >> >>> >> I'm happy to expand on the patch if it misses any corner cases, > >> >> >>> >> etc. but > >> >> >>> >> would like some feedback on whether this is the right direction > >> >> >>> >> to take > >> >> >>> >> on this problem. > >> >> >>> > > >> >> >>> > Did not see your mail on gt2-devel but I already provided some > >> >> >>> > feedback > >> >> >>> > in the jira itself > >> >> >>> > > >> >> >>> > Cheers > >> >> >>> > Andrea > >> >> >>> > > >> >> >>> > -- > >> >> >>> > Andrea Aime > >> >> >>> > OpenGeo - http://opengeo.org > >> >> >>> > Expert service straight from the developers. > >> >> >>> > > >> >> >>> > ------------------------------------------------------------------------------ > >> >> >>> > Apps built with the Adobe(R) Flex(R) framework and Flex > >> >> >>> > Builder(TM) are > >> >> >>> > powering Web 2.0 with engaging, cross-platform capabilities. > >> >> >>> > Quickly and > >> >> >>> > easily build your RIAs with Flex Builder, the Eclipse(TM)based > >> >> >>> > development > >> >> >>> > software that enables intelligent coding and step-through > >> >> >>> > debugging. > >> >> >>> > Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com > >> >> >>> > _______________________________________________ > >> >> >>> > Geotools-devel mailing list > >> >> >>> > [email protected] > >> >> >>> > https://lists.sourceforge.net/lists/listinfo/geotools-devel > >> >> >>> > > >> >> >> > >> >> >> > >> >> > > >> > > >> > > >> > >> ------------------------------------------------------------------------------ > >> Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are > >> powering Web 2.0 with engaging, cross-platform capabilities. Quickly and > >> easily build your RIAs with Flex Builder, the Eclipse(TM)based development > >> software that enables intelligent coding and step-through debugging. > >> Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com > >> _______________________________________________ > >> Geotools-devel mailing list > >> [email protected] > >> https://lists.sourceforge.net/lists/listinfo/geotools-devel > > > > > > ------------------------------------------------------------------------------ > Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are > powering Web 2.0 with engaging, cross-platform capabilities. Quickly and > easily build your RIAs with Flex Builder, the Eclipse(TM)based development > software that enables intelligent coding and step-through debugging. > Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com > _______________________________________________ > Geotools-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/geotools-devel ------------------------------------------------------------------------------ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
