Ruby on Rails: named scope with lambda and an array

Question

named_scope :all_public, lambda { |users| 
       { :conditions =>  ["visibility = ? || (visibility = ? && user_id = ?)", Shared::PUBLIC, Shared::PRIVATE, users] }
     }

That works nice for one user, but is there a way to modify it to work where users is an array of user ids?

When I learned SQL, I used the symbols instead of the words. =\ but I guess MySQL can do both. — NullVoxPopuli
– NullVoxPopuli, Commented Aug 6, 2010 at 20:21

bjg · Accepted Answer · 2010-08-06 18:57:00Z

1

Something like this and then just pass a single element array for the single ID case

named_scope :all_public, lambda { |users| 
       { :conditions =>  ["visibility = ? OR (visibility = ? AND user_id IN (?))", Shared::PUBLIC, Shared::PRIVATE, users.join(',')] }
     }

answered Aug 6, 2010 at 18:57

bjg

7,4771 gold badge27 silver badges21 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

mark Over a year ago

I think you'd need to collect the id's for that to work: users.collect(&:id).join(',')

bjg Over a year ago

@mark Yes you'd be correct if the argument was an array of User objects but I understood from the question that it would be an array of IDs. Either way, I agree, the caller must do the collect either before or inside the named scope

mark Over a year ago

You're right, if it were to have worked for a single user then it would also have needed an id. Naming convention is to blame. :)

NullVoxPopuli Over a year ago

shhh. naming some stuff is hard. (not here, but like named scope with two parameters where the visibility can change and so can the user_ids object) =p

NullVoxPopuli Over a year ago

actually this isn't what I want. =( posting more specific question soon.

Collectives™ on Stack Overflow

Ruby on Rails: named scope with lambda and an array

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related