Passing nil in array for conditions appears to be broken #220

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

gmile commented Mar 12, 2011

I've noticed that passing nil inside an array in conditions generates wrong SQL. It appears that Arel is to blame. I'll pushing a patch for Arel also, if you confirm this.

Here are some failing tests.

Owner

dhh commented Apr 27, 2011

@tenderlove, can you look at this?

Owner

tenderlove commented Apr 29, 2011

Merged in, so I'm closing.

tenderlove closed this Apr 29, 2011

gmile changed the title from Passing nil in array for conditions appears broken to Passing nil in array for conditions appears to be broken Jun 12, 2014

gmile referenced this pull request in rails/arel Mar 2, 2015

Merged

Passing nil in array generates improper SQL #40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment