Sitewide Search

Gemfile

@@ -23,6 +23,7 @@ gem "rest-client", '~> 1.6.3' # For University-Web
 
 gem "twitter",  :require => false
 gem "whenever", :require => false
+gem 'texticle', '~> 2.0', :require => 'texticle/rails'
 
 gem 'newman', '~> 0.1.1'
 

Gemfile.lock

@@ -192,6 +192,8 @@ GEM
       tilt (~> 1.1, != 1.3.0)
     test_notifier (1.0.0)
       notifier
+    texticle (2.0.3)
+      activerecord (~> 3.0)
     therubyracer (0.9.9)
       libv8 (~> 3.3.10)
     thor (0.14.6)
@@ -245,6 +247,7 @@ DEPENDENCIES
   sass-rails (~> 3.1.5.rc.2)
   slugger (= 0.3.0)
   test_notifier (~> 1.0.0)
+  texticle (~> 2.0)
   therubyracer
   twitter
   uglifier (>= 1.0.3)

app/controllers/search_controller.rb

@@ -0,0 +1,9 @@
+class SearchController < ApplicationController
+  respond_to :html
+
+  def index
+    @articles = Article.search_with_index(params[:query]).paginate(:page => params[:page])
+    @articles = @articles.public_only unless signed_in?
+    @articles = ArticleDecorator.decorate(@articles)
+  end
+end

app/models/article.rb

@@ -11,8 +11,9 @@ class Article < ActiveRecord::Base
     opt.validates_length_of :title, :minimum => 3
   end
 
-  scope :newest,      order("sticky desc, created_at desc")
-  scope :public_only, where(:public => true)
+  scope :newest,            order("sticky desc, created_at desc")
+  scope :public_only,       where(:public => true)
+  scope :search_with_index, lambda { |query| search_by_title_or_body(query, query) } 
 
   def to_param
     slug

app/views/articles/index.html.haml

@@ -5,6 +5,6 @@
 
 = render :partial => "header"
 
-#articles= render :partial => "article", :collection => @articles
+#articles= render :partial => "shared/article", :collection => @articles
 
 = will_paginate @articles

app/views/articles/show.html.haml

@@ -2,10 +2,10 @@
 
 = render :partial => "header"
 
-= render :partial => "article", :locals => { :article => @article }
+= render :partial => "shared/article", :locals => { :article => @article }
 
 - if @article.editable_by?(current_user)
   .control-bar
     = link_to 'Edit', edit_article_path(@article), :class => "clean-gray"
     = link_to 'Destroy', @article, :confirm => 'Are you sure?',
-      :method => :delete, :class => "clean-gray warn"
+      :method => :delete, :class => "clean-gray warn"

app/views/search/index.html.haml

@@ -0,0 +1,5 @@
+- content_for(:title) { "Search results" }
+
+#articles= render :partial => "shared/article", :collection => @articles
+
+= will_paginate @articles

app/views/articles/_article.html.haml → app/views/shared/_article.html.haml

@@ -7,6 +7,6 @@
 
   .article_content= article.body
 
-  = render 'user_info', :article => article, :profile_size => "100"
+  = render 'shared/user_info', :article => article, :profile_size => "100"
 
 = article.bottom(@articles, articles_path)

app/views/shared/_navigation.html.haml

@@ -10,6 +10,9 @@
       = link_to "Unicorns", page_path("unicorns")
     - else
       = link_to "About", about_path
+    %form{:id => "search", :action => "/search", :method => "GET"}
+      %input{:type => "text", :name => "query"}
+      %input#submit{:type => "submit"}
     %span#account
       - if current_user
         = link_to current_user.name, person_path(current_user.github)

config/routes.rb

@@ -26,4 +26,6 @@
 
   match '/about',     to: 'pages#show', id: 'about', as: 'about'
   match '/pages/:id', to: 'pages#show', as: 'page'
+
+  match '/search' => 'search#index'
 end

db/migrate/20120219213020_add_search_index_to_articles.rb

@@ -0,0 +1,14 @@
+class AddSearchIndexToArticles < ActiveRecord::Migration
+  def up
+    execute "
+      create index articles_title_idx on articles using gin(to_tsvector('english', title));
+      create index articles_body_idx on articles using gin(to_tsvector('english', body));"
+  end
+
+  def down
+    execute "
+      drop index articles_title_idx;
+      drop index articles_body_idx;
+      "
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20120217002323) do
+ActiveRecord::Schema.define(:version => 20120219213020) do
 
   create_table "activities", :force => true do |t|
     t.integer  "author_id"

test/integration/search_test.rb

@@ -0,0 +1,22 @@
+require 'test_helper'
+
+class SearchTest < ActionDispatch::IntegrationTest
+  def setup
+    @user = FactoryGirl.create(:user)
+    FactoryGirl.create(:article, title: "Article 1 - interesting", author: @user)
+    FactoryGirl.create(:article, title: "Article 2 - meh", author: @user)
+  end
+
+  test "articles can be searched" do
+    visit articles_path
+
+    within("#search") do
+      fill_in "query", :with => "interesting"
+      click_button "submit" 
+    end
+
+    assert_content("Article 1")
+    assert_no_content("Article 2")
+  end
+end
+

test/unit/article_test.rb

@@ -59,4 +59,4 @@ def setup
 
     assert_equal(1, Article.public_only.count)
   end
-end
+end

test/unit/search_test.rb

@@ -0,0 +1,26 @@
+require 'test_helper'
+
+class SearchTest < ActiveSupport::TestCase
+  def setup
+    @user = FactoryGirl.create(:user)
+  end
+
+  test "title can be searched" do
+    FactoryGirl.create(:article, title: "Article 1 - interesting", author: @user)
+    FactoryGirl.create(:article, title: "Article 2", author: @user)
+
+    titles = Article.search_with_index("interesting").map { |a| a.title }
+
+    assert_equal(["Article 1 - interesting"], titles)
+  end
+
+  test "body can be searched" do
+    FactoryGirl.create(:article, title: "Article 1", body: "interesting", author: @user)
+    FactoryGirl.create(:article, title: "Article 2", body: "boring", author: @user)
+
+    titles = Article.search_with_index("interesting").map { |a| a.title }
+
+    assert_equal(["Article 1"], titles)
+  end
+end
+