On Aug 22, 2:02 pm, Jeremy Evans <[email protected]> wrote:
> Seehttp://pastie.org/2413439for the patch.  I'm not sure if I want
> to commit this to Sequel because of the potential backwards
> compatibility issues.  I'd like to get opinions from the community
> before I make my decision.

This commit doesn't handle the following case correctly:

DB.literal(:x + 1 +
'a')
=> "((x + 1) || 'a')"

It should be raising an error.  It's not difficult to fix that, but
due to the potential backwards compatibility issues and the fact that
nobody chimed in that they'd like this feature, I'll probably shelve
the idea.

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.

Reply via email to