peteonrails-vote_fu 0.0.10 → 0.0.11

data/CHANGELOG.markdown

@@ -1,3 +1,7 @@
+2009-02-11
+==========
+* Merge in xlash's bugfix for PostgreSQL and his has\_karma patch for multi-model support. 
+
 2008-12-02
 ==========
 * Merge in maddox's README typo fix and his ActiveSupport.Dependency patch

data/lib/has_karma.rb

@@ -6,6 +6,9 @@ module PeteOnRails
 
       def self.included(base)
         base.extend ClassMethods
+        class << base
+          attr_accessor :karmatic_objects
+        end
       end
 
       module ClassMethods
@@ -15,9 +18,13 @@ module PeteOnRails
               #{voteable_type.to_s.classify}
             end
           RUBY
-          
           include PeteOnRails::VoteFu::Karma::InstanceMethods
           extend  PeteOnRails::VoteFu::Karma::SingletonMethods
+          if self.karmatic_objects.nil?
+            self.karmatic_objects = [eval(voteable_type.to_s.classify)]
+          else
+            self.karmatic_objects.push(eval(voteable_type.to_s.classify))
+          end
         end
       end
       
@@ -35,13 +42,22 @@ module PeteOnRails
       # This module contains instance methods
       module InstanceMethods
         def karma(options = {})
+          #FIXME cannot have 2 models imapcting the karma simultaneously
           # count the total number of votes on all of the voteable objects that are related to this object
-          self.karma_voteable.sum(:vote, options_for_karma(options))          
+          #2009-01-30 GuillaumeNM The following line is not SQLite3 compatible, because boolean are stored as 'f' or 't', not '1', or '0'
+          #self.karma_voteable.sum(:vote, options_for_karma(options))
+          #self.karma_voteable.find(:all, options_for_karma(options)).length
+          karma_value = 0
+          self.class.karmatic_objects.each do |object|
+            karma_value += object.find(:all, options_for_karma(object, options)).length
+          end
+          return karma_value
         end
         
-        def options_for_karma (options = {})
-            conditions = ["u.id = ?" , self[:id] ]
-            joins = ["inner join votes v on #{karma_voteable.table_name}.id = v.voteable_id", "inner join #{self.class.table_name} u on u.id = #{karma_voteable.name.tableize}.#{self.class.name.foreign_key}"]            
+        def options_for_karma (object, options = {})
+            #GuillaumeNM : 2009-01-30 Adding condition for SQLite3
+            conditions = ["u.id = ? AND vote = ?" , self[:id] , true]
+            joins = ["inner join votes v on #{object.table_name}.id = v.voteable_id", "inner join #{self.class.table_name} u on u.id = #{object.name.tableize}.#{self.class.name.foreign_key}"]
             { :joins => joins.join(" "), :conditions => conditions }.update(options)          
         end
         

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: peteonrails-vote_fu
 version: !ruby/object:Gem::Version 
-  version: 0.0.10
+  version: 0.0.11
 platform: ruby
 authors: 
 - Peter Jackson
@@ -12,7 +12,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-01-07 00:00:00 -08:00
+date: 2009-02-11 00:00:00 -08:00
 default_executable: 
 dependencies: []