@@ -21,6 +21,8 @@ # last_login_ip :string(255) # stylesheet :string(255) # old_password :string(20) +# karma :integer(4) default(20), not null +# nb_votes :integer(4) default(0), not null # created_at :datetime # updated_at :datetime # @@ -37,13 +39,13 @@ class Account < ActiveRecord::Base validates_presence_of :login, :message => "Veuillez choisir un pseudo" validates_presence_of :email, :message => "Veuillez entrer votre adresse email" - validates_uniqueness_of :email, :case_sensitive => false, :message => "Cette adresse email est déjà utilisée pour un compte LinuxFr.org" ### Authentication ### acts_as_authentic do |config| config.validates_length_of_login_field_options :within => 3..30, :message => "Le login doit faire au moins 3 caractères, et pas plus de 30 caractères" config.validates_uniqueness_of_login_field_options :case_sensitive => true, :message => "Ce login est déjà utilisé, veuillez en choisir un autre" + config.validates_uniqueness_of_email_field_options :case_sensitive => true, :message => "Cette adresse email est déjà utilisée pour un compte LinuxFr.org" config.perishable_token_valid_for 24.hours end @@ -97,6 +99,25 @@ class Account < ActiveRecord::Base user.reactivate! end +### Karma ### + + def daily_karma + self.karma -= Vote.count(:conditions => ["nodes.user_id = ? AND (votes.created_at BETWEEN ? AND ?) AND vote = 0", + self.id, DateTime.yesterday.beginning_of_day, DateTime.yesterday.end_of_day], + :joins => :node).to_i + self.karma += Vote.count(:conditions => ["nodes.user_id = ? AND (votes.created_at BETWEEN ? AND ?) AND vote = 1", + self.id, DateTime.yesterday.beginning_of_day, DateTime.yesterday.end_of_day], + :joins => :node).to_i + self.karma -= Relevance.count(:conditions => ["comments.user_id = ? AND (relevances.created_at BETWEEN ? AND ?) AND vote = 0", + self.id, DateTime.yesterday.beginning_of_day, DateTime.yesterday.end_of_day], + :joins => :comment).to_i + self.karma += Relevance.count(:conditions => ["comments.user_id = ? AND (relevances.created_at BETWEEN ? AND ?) AND vote = 1", + self.id, DateTime.yesterday.beginning_of_day, DateTime.yesterday.end_of_day], + :joins => :comment).to_i + self.nb_votes = 3 + karma / 10 + save + end + ### Presentation ### def email_address app/models/account.rb @@ -100,6 +100,12 @@ class Comment < ActiveRecord::Base ### Calculations ### + before_create :default_score + + def default_score + self.score = Math.log10(user.account.karma).to_i - 1 + end + def nb_answers self.class.published.descendants(materialized_path).count end @@ -129,6 +135,7 @@ class Comment < ActiveRecord::Base def votable_by?(user) user && !deleted? && self.user != user && (Time.now - created_at) < 3.months && + user.account.nb_votes > 0 && !user.relevances.exists?(:comment_id => id) end app/models/comment.rb @@ -37,6 +37,7 @@ class Content < ActiveRecord::Base def votable_by?(user) user && !deleted? && self.user != user && (Time.now - created_at) < 3.months && + user.account.nb_votes > 0 && !user.votes.exists?(:node_id => node.id) end app/models/content.rb @@ -24,12 +24,14 @@ class Relevance < ActiveRecord::Base # An user can vote for a comment... def self.for(user, comment) + Account.decrement_counter(:nb_votes, user.account.id) user.relevances.create(:comment_id => comment.id, :vote => true) Comment.increment_counter(:score, comment.id) end # ...or he can vote against it def self.against(user, comment) + Account.decrement_counter(:nb_votes, user.account.id) user.relevances.create(:comment_id => comment.id, :vote => false) Comment.decrement_counter(:score, comment.id) end app/models/relevance.rb @@ -27,6 +27,7 @@ class Vote < ActiveRecord::Base # An user can vote for a node... def self.for(user, node) + Account.decrement_counter(:nb_votes, user.account.id) cancel(user, node) user.votes.create(:node_id => node.id, :vote => true) Node.increment_counter(:score, node.id) @@ -35,6 +36,7 @@ class Vote < ActiveRecord::Base # ...or he can vote against it def self.against(user, node) + Account.decrement_counter(:nb_votes, user.account.id) cancel(user, node) user.votes.create(:node_id => node.id, :vote => false) Node.decrement_counter(:score, node.id) app/models/vote.rb @@ -48,6 +48,7 @@ = image_tag 'icones/lock-insecure.png', :alt => "Accès en clair (http)", :class => "access" - if current_user %h1= current_user.name + %p Il vous reste #{current_user.account.nb_votes} avis %p= link_to "Proposer un contenu", '/proposer_un_contenu' %p= link_to "Mon tableau de bord", dashboard_path %p= link_to "Modifier mes préférences", edit_account_path app/views/layouts/application.html.haml @@ -7,6 +7,7 @@ class CreateVotes < ActiveRecord::Migration t.datetime :created_at end add_index :votes, [:node_id, :user_id], :unique => true + add_index :votes, [:created_at, :vote, :node_id] end def self.down db/migrate/20090130001540_create_votes.rb @@ -7,6 +7,7 @@ class CreateRelevances < ActiveRecord::Migration t.datetime :created_at end add_index :relevances, [:comment_id, :user_id], :unique => true + add_index :relevances, [:created_at, :vote, :comment_id] end def self.down db/migrate/20090205000452_create_relevances.rb @@ -17,6 +17,8 @@ class CreateAccounts < ActiveRecord::Migration t.datetime :last_login_at t.string :current_login_ip t.string :last_login_ip + t.integer :karma, :null => false, :default => 20 + t.integer :nb_votes, :null => false, :default => 0 t.string :stylesheet t.string :old_password, :limit => 20 t.timestamps @@ -29,6 +31,11 @@ class CreateAccounts < ActiveRecord::Migration execute "ALTER TABLE `accounts` MODIFY COLUMN `perishable_token` VARCHAR(255) BINARY CHARACTER SET latin1 COLLATE latin1_bin NOT NULL" add_index :accounts, :user_id + add_index :accounts, :login + add_index :accounts, :email + add_index :accounts, :persistence_token + add_index :accounts, :single_access_token + add_index :accounts, :perishable_token end def self.down db/migrate/20090505233940_create_accounts.rb @@ -1,6 +1,7 @@ namespace :linuxfr do desc "Daily crontab" task :daily => [ + :daily_karma, :flush_poll_ips, :delete_old_passive_accounts, :delete_old_boards, @@ -9,6 +10,11 @@ namespace :linuxfr do 'friendly_id:remove_old_slugs' ] + desc "New day => update karma and give new votes" + task :daily_karma => :environment do + Account.active.find_each {|a| a.daily_karma } + end + desc "New day => the users can revote on polls" task :flush_poll_ips => :environment do PollIp.delete_all lib/tasks/linuxfr.rake @@ -23,6 +23,8 @@ # old_password :string(20) # created_at :datetime # updated_at :datetime +# karma :integer(4) default(20), not null +# nb_votes :integer(4) default(0), not null # # Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html test/fixtures/accounts.yml @@ -23,6 +23,8 @@ # old_password :string(20) # created_at :datetime # updated_at :datetime +# karma :integer(4) default(20), not null +# nb_votes :integer(4) default(0), not null # require 'test_helper' test/unit/account_test.rb 30f8362ae71e83c28d36f9dafef26fb215a6258c Bruno Michel bmichel@menfin.info http://github.com/nono/linuxfr.org/commit/649632b27a53395a9fb492c4bfcdf43c44c42c2d 649632b27a53395a9fb492c4bfcdf43c44c42c2d 2009-11-06T16:44:16-08:00 2009-11-06T16:44:16-08:00 Users karma and nb votes 3cf36ac8c00d04504869204e7690e9d448a1988a Bruno Michel bmichel@menfin.info