Comments on: Implementing Search in Ruby on Rails

By: Hans

Hans — Thu, 18 Mar 2010 09:50:41 +0000

@darren: Why do think, that it is not necessary to split up ‘query’ and check against each word of it?

By: erwin

erwin — Sat, 09 Jan 2010 15:43:18 +0000

how you did implement your view?

By: Darren

Darren — Sat, 25 Oct 2008 23:50:45 +0000

That search method in your model could be quite a bit simpler than that I believe. mySQL is pretty smart so the downcase isn’t needed. You also don’t need to split up a string and check against each word.

def self.search(query)
unless query.to_s.strip.empty?
find(:all, :conditions => ['title like ? or description like ?', "%#{query}%", "%#{query}%"], :order => ‘created_at desc’, :limit=>25)
end
end

By: Zaf

Zaf — Thu, 25 Sep 2008 07:42:42 +0000

undefined local variable or method `query’ for #:0xb75a1a4c>

this pops up.. how can i fix it?

By: Jose

Jose — Wed, 20 Feb 2008 13:08:54 +0000

Jao, you can use this code for pagination: http://snippets.dzone.com/posts/show/389

In this example you could write:

def songs
@query = @params["query"]
@song_pages, @songs = paginate_collection(Song.search(@query), :per_page => 10)
end

By: Rory

Rory — Fri, 14 Sep 2007 01:29:59 +0000

The built-in Rails bind variable facility will handle that, P.

By: P

Thu, 13 Sep 2007 22:06:55 +0000

Doesn’t this leave your database open to SQL injection? What’s to stop the user from entering something like “; drop table songs ;” or something like that?

By: Jim K.

Jim K. — Thu, 12 Apr 2007 14:57:55 +0000

Great tutorial. Thanks a bunch.

By: Matt

Matt — Tue, 20 Feb 2007 20:32:46 +0000

Good stuff Rory. Thanks for the tip!

By: jao

jao — Mon, 27 Mar 2006 19:09:00 +0000

first i must apologize for my lack of experience.
What i need to know is how would you use pagination within this method?

thank you.

jao