Re: [Geotools-devel] ECQL parser doesn't handle time zones properly

[Justin D. <jde...@op...>]

Hey Dustin,

Thanks for the heads up. If you could whip up a patch or a pull request
(ideally with a test case) that would be great.

-Justin

On Thu, Jul 12, 2012 at 7:49 PM, Dustin Parker <dp...@fo...>wrote:

> Huh, the CS-W 2.0.2 spec doesn't mention time zones in their grammar, just
> 'Z'. Anyway, this code:
>
> private String extractTime(String cqlDateTime) {
> // splits dateTime to get the Time and time zone in the second array
> element
> String strUpper = cqlDateTime.toUpperCase();
> String[] dateTime = strUpper.split("T");
> assert dateTime.length >= 2 : "date and time is required by the sintax
> rule";
> // splits the time and time zone
> String[] time = dateTime[1].split("[+|-|Z]");
> return time[0]; // the time
> } Doesn't work because the regex is wrong; it should be "[-+Z]".
>
> Thanks,
>
> --
> Dustin Parker - Forward Slope, Inc.
> Cell: 619 277 2591
>
> VGhlIDFzdCBydWxlIG9mIG9wdGltaX**phdGlvbiBpcyAi
> ZG9uJ3Qgb3B0aW1pemUuIiBUaGUgMm**5kIHJ1bGUgb2Yg
> b3B0aW1pemF0aW9uIGlzICJvcHRpbW**l6ZSBsYXRlci4i
>
>
>
>
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> GeoTools-Devel mailing list
> Geo...@li...
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>


-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.