search

jasonmk / datastax_rails

A Ruby-on-Rails interface to Datastax Enterprise. Replaces the majority of ActiveRecord functionality.
MIT License
23 stars 9 forks source link

downcase_query to omit timestamps #22

Closed jmphilbrick closed 10 years ago

jmphilbrick commented 10 years ago

the "T" and "Z" characters in the date/time format are getting downcased. now prevents date/time fields from getting downcased

Don't love the way I removed the date from the downcasing, feels a little clunky but does the trick.

jmphilbrick commented 10 years ago

good catch on the bug, had a bad push where it didn't pick my latest changes.

upcasing the date afterwards makes a whole lot more sense and is alot cleaner, good call.

jasonmk commented 10 years ago

Implemented the date regex version and checked into both 1.2 and 2.0. Thanks for the catch.