From: John Dong Date: Fri, 17 Aug 2007 21:31:45 +0000 (-0400) Subject: Fixed all triggerable DeprecationWarnings. BE ON LOOKOUT FOR REGRESSIONS. X-Git-Url: https://projects.mako.cc/source/selectricity/commitdiff_plain/9086b07aefc0560832373bb4ba30888d8d43f0d6?ds=sidebyside Fixed all triggerable DeprecationWarnings. BE ON LOOKOUT FOR REGRESSIONS. --- diff --git a/app/controllers/application.rb b/app/controllers/application.rb index 8c6f8ae..9b48030 100644 --- a/app/controllers/application.rb +++ b/app/controllers/application.rb @@ -4,6 +4,6 @@ class ApplicationController < ActionController::Base include AuthenticatedSystem helper :user - model :user + require_dependency "user" end diff --git a/app/controllers/quickvote_controller.rb b/app/controllers/quickvote_controller.rb index e327ed6..8b853eb 100644 --- a/app/controllers/quickvote_controller.rb +++ b/app/controllers/quickvote_controller.rb @@ -59,7 +59,7 @@ class QuickvoteController < ApplicationController # look to see that the voter has been created and has voted in # this election, and has confirmed their vote - @voter = QuickVoter.find_all(["session_id = ? and election_id = ?", + @voter = QuickVoter.find(:all, :conditions => ["session_id = ? and election_id = ?", session.session_id, @election.id])[0] # if the voter has not voted we destroy them @@ -92,7 +92,7 @@ class QuickvoteController < ApplicationController election = QuickVote.ident_to_quickvote(params[:ident]) # find out who the voter is for this election - @voter = QuickVoter.find_all(["session_id = ? and election_id = ?", + @voter = QuickVoter.find(:all, :conditions => ["session_id = ? and election_id = ?", session.session_id, election.id])[0] if not @voter @@ -122,7 +122,7 @@ class QuickvoteController < ApplicationController end def change - voter = QuickVoter.find_all(["session_id = ?", session.session_id])[0] + voter = QuickVoter.find(:all, :conditions => ["session_id = ?", session.session_id])[0] voter.destroy redirect_to quickvote_url( :ident => params[:ident] ) end diff --git a/app/controllers/voter_controller.rb b/app/controllers/voter_controller.rb index 779e83f..e94f4a7 100644 --- a/app/controllers/voter_controller.rb +++ b/app/controllers/voter_controller.rb @@ -7,7 +7,7 @@ class VoterController < ApplicationController def index password = params[:id] password = params[:vote][:password] if params[:vote] - if @voter = FullVoter.find_all( [ "password = ?", password ] )[0] + if @voter = FullVoter.find(:all, :conditions => [ "password = ?", password ] )[0] render :action => 'fullvote' end end @@ -41,7 +41,7 @@ class VoterController < ApplicationController private def authenticate password = params[:id] - @voter = FullVoter.find_all( [ "password = ?", password ] )[0] + @voter = FullVoter.find(:all, :conditions => [ "password = ?", password ] )[0] end end diff --git a/app/models/election.rb b/app/models/election.rb index cf7bba5..3baeef0 100644 --- a/app/models/election.rb +++ b/app/models/election.rb @@ -57,7 +57,7 @@ class Election < ActiveRecord::Base end def quickvote? - type == 'QuickVote' + self.class == 'QuickVote' end def active? diff --git a/app/models/full_voter.rb b/app/models/full_voter.rb index 04071a0..980fb5c 100644 --- a/app/models/full_voter.rb +++ b/app/models/full_voter.rb @@ -5,7 +5,7 @@ class FullVoter < Voter def create_password token_generator = UniqueTokenGenerator.new( 16 ) until password and not password.empty? \ - and Voter.find_all( [ "password = ?", password ]).empty? + and Voter.find(:all, :conditions => [ "password = ?", password ]).empty? self.password = token_generator.token end end diff --git a/app/models/quick_vote.rb b/app/models/quick_vote.rb index ad72717..fef4ef8 100644 --- a/app/models/quick_vote.rb +++ b/app/models/quick_vote.rb @@ -88,7 +88,7 @@ class QuickVote < Election if ident.match(/^\d+$/) quickvote = QuickVote.find(ident) else - quickvote = QuickVote.find_all(["name = ?", ident])[0] + quickvote = QuickVote.find(:all, :conditions => ["name = ?", ident])[0] end return quickvote diff --git a/app/models/selectricity_service.rb b/app/models/selectricity_service.rb index 9e276ef..6d338de 100644 --- a/app/models/selectricity_service.rb +++ b/app/models/selectricity_service.rb @@ -7,10 +7,10 @@ class SelectricityService < ActionWebService::Base if election candidates=election.candidates.collect { |c| c.id } vote_list[0].each do |vote| - raise ArgumentError.new "Invalid Candidate ID #{vote}" unless candidates.index(vote) + raise ArgumentError.new("Invalid Candidate ID #{vote}") unless candidates.index(vote) end - raise ArgumentError.new "You must rank all candidates" unless candidates.length <= vote_list[0].length - raise ArgumentError.new "Please rank each candidate only once" if vote_list[0].uniq! + raise ArgumentError.new("You must rank all candidates") unless candidates.length <= vote_list[0].length + raise ArgumentError.new("Please rank each candidate only once") if vote_list[0].uniq! voter = QuickVoter.new voter.election = election voter.ipaddress = "XMLRPC Request" @@ -22,7 +22,7 @@ class SelectricityService < ActionWebService::Base voter.vote.confirm! voter.save! else - raise ArgumentError.new "Cannot find election #{election_name}" + raise ArgumentError.new("Cannot find election #{election_name}") end end def quickvote_candidate_ids_to_names(shortname, id_list) @@ -86,14 +86,14 @@ class SelectricityService < ActionWebService::Base end def list_quickvotes() all=Array.new - QuickVote.find_all.each do |election| + QuickVote.find(:all).each do |election| all << get_quickvote(election.name) end return all end def get_quickvote(shortname) return ElectionStruct.new unless election=QuickVote.ident_to_quickvote(shortname) - return ElectionStruct.new (:id => election.id, :name => election.name, :description => election.description, :candidate_ids => election.candidates.collect {|c| c.id }, :candidate_names => election.candidates.collect {|c| c.name } ) + return ElectionStruct.new(:id => election.id, :name => election.name, :description => election.description, :candidate_ids => election.candidates.collect {|c| c.id }, :candidate_names => election.candidates.collect {|c| c.name } ) end def create_quickvote(election) qv=QuickVote.new(:name => election.name, :description => election.description) diff --git a/app/models/token.rb b/app/models/token.rb index 2bab513..4ed9598 100644 --- a/app/models/token.rb +++ b/app/models/token.rb @@ -5,7 +5,7 @@ class Token < ActiveRecord::Base super token_generator = UniqueTokenGenerator.new( 16 ) - until not token.empty? and Token.find_all( [ "token = ?", token ]).empty? + until not token.empty? and Token.find(:all, :conditions => [ "token = ?", token ]).empty? self.token = token_generator.token end