On Apr 11, 5:18 pm, Shawn <[email protected]> wrote: > Also, forking in filter_expr makes this work for both ds.filter and > ds.where. > > Shawn
Both filter, where, and having call _filter, so overriding it there fixes those cases. There are a few places where filter_expr is called directly by other code, but I'm not sure if it makes sense to try to fix those cases. I did use your idea of checking for empty?, as it is a tad faster. Thanks for that. I just pushed the commit to github: http://github.com/jeremyevans/sequel/commit/27154a5f53b29bf2075da680989d70c3cf0ac89c Jeremy -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/sequel-talk?hl=en.
