rostra 0.1.15 → 0.1.16

data/app/controllers/rostra/base/answers_controller.rb

@@ -6,7 +6,6 @@ module Rostra
 
       def vote
         rostra_user.vote_on(@answer, params[:vote_direction])
-
         respond_to do |format|
           format.html { redirect_to question_path(@question, anchor: "rostra_answer_#{@answer.id}") }
           format.js

data/app/controllers/rostra/base/questions_controller.rb

@@ -42,7 +42,7 @@ module Rostra
       end
 
       def show
-        @answers = @question.answers
+        @answers = @question.answers.order('votes_count desc')
         impressionist(@question) # increments page views
       end
 

data/app/models/vote.rb

@@ -14,4 +14,13 @@ class Vote < ActiveRecord::Base
   # Comment out the line below to allow multiple votes per user.
   validates_uniqueness_of :voteable_id, :scope => [:voteable_type, :voter_type, :voter_id]
 
+  after_create :update_counter_cache
+  after_destroy :update_counter_cache
+
+  private
+
+  def update_counter_cache
+    voteable.update_attribute(:votes_count, voteable.plusminus)
+  end
+
 end

data/app/views/rostra/answers/vote.js.erb

@@ -1,2 +1,2 @@
 var selector = "#rostra_answer_<%= @answer.id%> .vote_wrapper";
-$(selector).replaceWith('<%= escape_javascript(render(:partial => 'rostra/shared/votes', locals: {resource: @answer})) %>');
+$(selector).replaceWith('<%= escape_javascript(render(partial: 'rostra/shared/votes', locals: {resource: @answer})) %>');

data/app/views/rostra/questions/index.html.erb

@@ -12,8 +12,8 @@
 
             <div class="stats">
               <div class="vote_count" title="Number of times this question was voted up">
-                <span class="count"><%= question.plusminus %></span>
-                <span class="type"><%= question.plusminus.abs == 1 ? 'vote' : 'votes' %></span>
+                <span class="count"><%= question.votes_count %></span>
+                <span class="type"><%= question.votes_count.abs == 1 ? 'vote' : 'votes' %></span>
               </div>
               <div class="answer_count" title="Number of times this question was answered">
                 <span class="count"><%= question.answer_count %></span>

data/app/views/rostra/questions/vote.js.erb

@@ -1,2 +1,2 @@
-var selector = "#rostra_question_<%= @question.id%> .vote_wrapper";
+var selector = "#rostra_question_<%= @question.id %> .vote_wrapper";
 $(selector).replaceWith('<%= escape_javascript(render partial: 'rostra/shared/votes', locals: { resource: @question }) %>');

data/app/views/rostra/shared/_votes.html.erb

@@ -1,6 +1,6 @@
 <div class="vote_wrapper">
   <%= link_to_vote(:up, resource) %>
-  <div class="count"><%= resource.plusminus %></div>
+  <div class="count"><%= resource.votes_count %></div>
   <%= link_to_vote(:down, resource) %>
 </div>
 

data/db/migrate/20111106162542_add_votes_count_to_rostra_answers.rb

@@ -0,0 +1,5 @@
+class AddVotesCountToRostraAnswers < ActiveRecord::Migration
+  def change
+    add_column :rostra_answers, :votes_count, :integer, default: 0
+  end
+end

data/db/migrate/20111106164824_add_votes_count_to_rostra_questions.rb

@@ -0,0 +1,5 @@
+class AddVotesCountToRostraQuestions < ActiveRecord::Migration
+  def change
+    add_column :rostra_questions, :votes_count, :integer, default: 0
+  end
+end

data/lib/rostra/version.rb

@@ -1,3 +1,3 @@
 module Rostra
-  VERSION = "0.1.15"
+  VERSION = "0.1.16"
 end

data/lib/rostra.rb

@@ -61,6 +61,7 @@ module Rostra
         else
           vote(resource, {:exclusive => true, :direction => direction})
         end
+        resource.reload
       end
 
       # Check if the user is following a given question

metadata

@@ -2,7 +2,7 @@
 name: rostra
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.1.15
+  version: 0.1.16
 platform: ruby
 authors: 
 - Cory Schires
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-11-02 00:00:00 Z
+date: 2011-11-06 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rails
@@ -556,6 +556,8 @@ files:
 - db/migrate/20111019162723_create_rostra_question_followings.rb
 - db/migrate/20111024223022_add_send_email_notifications_to_question_followings.rb
 - db/migrate/20111029214642_add_impressions_count_to_questions.rb
+- db/migrate/20111106162542_add_votes_count_to_rostra_answers.rb
+- db/migrate/20111106164824_add_votes_count_to_rostra_questions.rb
 - lib/generators/rostra/install_generator.rb
 - lib/generators/rostra/templates/app/helpers/rostra/application_helper.rb
 - lib/generators/rostra/templates/config/initializers/rostra.rb