Rails: find_by_contents with conditions; syntax help please

Question

Looking to limit a find based on conditions in a Rails 2.0.2.

Find statement:

@employees = Employee.find_by_contents(params[:keywords].to_s, :include => [ :categories, :revisions, :approvals, :archives, :related_documents ])

Need to add a condition to limit find

:conditions=>["archived = '0'"]

Though this doesn't work

@employees = Employee.find_by_contents(params[:keywords].to_s, :include => [ :categories, :revisions, :approvals, :archives, :related_documents ], :conditions=>["archived = '0'"])

Anyone know what the syntax should be?

what type of the `archived` field? Really `Strig`? Or `Boolean`? — fl00r, Feb 04 '11 at 22:07
No errors, has just ignored the conditions statement. boolean. — Jeffrey, Feb 07 '11 at 18:49
So why do you using :conditions=>["archived = '0'"] condition??? rather :conditions=>["archived = ?", false] — fl00r, Feb 07 '11 at 22:12

score 0 · Answer 1 · answered Feb 04 '11 at 21:17

If you pass conditions an array, I think it needs to be for a parameterized condition. Try either/both of the following:

    @employees = Employee.find_by_contents(params[:keywords].to_s, 
                  :include => [ :categories, :revisions, :approvals, 
                                :archives, :related_documents ], 
                  :conditions=> "archived = '0'")

or

@employees = Employee.find_by_contents(params[:keywords].to_s,                           
                  :include => [ :categories, :revisions, :approvals, 
                                :archives, :related_documents ],
                  :conditions=>["archived = ?",'0'])

Neither of them were successful at limiting the results. – Jeffrey Feb 04 '11 at 21:40 — Jeffrey, Feb 04 '11 at 21:40

fl00r · Answer 2 · 2011-02-07T22:13:18.123

0

Firstly, what error do you have?

And try this

@employees = Employee.find_by_contents(params[:keywords].to_s, :include => [ :categories, :revisions, :approvals, :archives, :related_documents ], :conditions=>["employees.archived = ?", false])

edited Feb 07 '11 at 22:13

answered Feb 04 '11 at 21:55

fl00r

82,987
33
217
237

No error. It just ignores the conditions statement as if it wasn't there. – Jeffrey Feb 07 '11 at 16:33
see update. use `:conditions=>["employees.archived = ?", false]` – fl00r Feb 07 '11 at 22:13

score 0 · Accepted Answer · answered Feb 07 '11 at 18:48

0

Was having quite a time with adding a condition to ferret find_by, so I added a post command to filter if it's archived. If you think of a better way still, let me know. Thanks.

@employees = Employee.find_by_contents(params[:keywords].to_s, 
              :include => [ :categories, :revisions, :approvals, 
                            :archives, :related_documents ])
@employees = @employees.find_all {|p| !p.archived}

answered Feb 07 '11 at 18:48

Jeffrey

4,098
11
42
66

bad solution, because you can use only AR – fl00r Feb 07 '11 at 22:13

Rails: find_by_contents with conditions; syntax help please

3 Answers3