Get rails ActiveRecord instead of Array of objects for multiple queries

Question

I have a table that stores queries that return a list of users. I then have a method "get_public" to a "Banana" model that execute multiple queries using logic AND between them.

So, when I do

Banana.find(x).get_public I receive an Array of users (the ones suitable to that banana object).

The get_public method is like this:

def get_public
  pb = []
  banana_queries.each do |q|
    pb << User.find_by_sql(q.query)
  end
  pb.inject(:'&')
end

But, would be great if I could get ActiveRecord::Relation instead. I want to do something like this after: Banana.find(x).get_public.where(...) Any way to modify get_public and achieve this?

Have you tried including ActiveRecord::QueryMethods in your class to get relation query methods? — Bala
– Bala, Commented Aug 28, 2013 at 16:29
What is pb.inject(:'&') supposed to be doing in this case and have you tested it's behavior? — Peter Alfvin
– Peter Alfvin, Commented Aug 28, 2013 at 16:37

Community · Accepted Answer · 2017-05-23 12:26:01Z

1

I am not sure I correcly undestood the problem, but I will try to help anyway.

As especified here

where returns an ActiveRecord::Relation
find (and its related dynamic methods) returns a single model object

So I suggest divide your queries into: 'joins' and 'where' fields. Your new code should look like something like this:

 pb << User.joins(q.query_joins).where(q.query_where)

Also find methods will are deprecated in rails 4, so using where is recommended.

Hope I haven't missed the point too much :-)

edited May 23, 2017 at 12:26

CommunityBot

11 silver badge

answered Aug 28, 2013 at 16:33

Pablo R

647 bronze badges

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

Collectives™ on Stack Overflow

Get rails ActiveRecord instead of Array of objects for multiple queries

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related