X-Git-Url: https://projects.mako.cc/source/selectricity-live/blobdiff_plain/823d530ba46c1fb3518c4deef0081738503837eb..008cc97a2f0d84b4de46f2125133afe6c7f620c7:/app/models/quick_vote.rb diff --git a/app/models/quick_vote.rb b/app/models/quick_vote.rb index 494f7ac..e0bab76 100644 --- a/app/models/quick_vote.rb +++ b/app/models/quick_vote.rb @@ -1,8 +1,9 @@ class QuickVote < Election + before_validation :build_candidate_names after_validation :create_candidates validates_uniqueness_of :name - validates_presence_of :name - attr_accessor :raw_candidates + + attr_accessor :candidate_names attr_accessor :reviewed def initialize(params={}) @@ -14,16 +15,12 @@ class QuickVote < Election Time.now + 30.days - 1.second end - def enddate - super(30) - end - def validate - if not @raw_candidates or @raw_candidates.length < 2 + if @candidate_names.length < 2 errors.add(nil, "You must list at least two candidates.") end - - @raw_candidates.each do |c| + + @candidate_names.each do |c| unless c.instance_of? String errors.add(nil, "Candidates must be strings") next @@ -33,9 +30,11 @@ class QuickVote < Election errors.add(nil, "Candidate name must not be empty") next end - end if @raw_candidates - - errors.add(nil, "Candidates must all be unique") if @raw_candidates and @raw_candidates.uniq! + end if @candidate_names + + if @candidate_names and @candidate_names.uniq! + errors.add(nil, "Candidates must all be unique") + end if name =~ /[^A-Za-z0-9]/ errors.add(:name, "must only include numbers and letters.") @@ -47,15 +46,10 @@ class QuickVote < Election if name =~ /^(create|index|confirm|change|results)$/ errors.add(:name, " is a reserved word.") end - + if enddate < startdate - errors.add(nil, "QuickVotes can't end before they start!") + errors.add(nil, "QuickVotes can't end before they start") end - - end - - def candidatelist=(candlist) - @raw_candidates = candlist end def name @@ -66,9 +60,21 @@ class QuickVote < Election reviewed.to_i == 1 end + def build_candidate_names + @candidate_names ||= [] + if @candidate_names.empty? and not candidates.empty? + @candidate_names = candidates.collect {|c| c.name} + end + end + def create_candidates return unless errors.empty? - @raw_candidates.each do |name| + + # delete the candidates + candidates.each {|c| c.destroy} + + # create the new list based on the names + @candidate_names.each do |name| candidate = Candidate.new({:name => name}) self.candidates << candidate end