Started Creation of "advanced" quickvote tab. Also started to create a quickvote...

[selectricity] / app / models / quick_vote.rb

diff --git a/app/models/quick_vote.rb b/app/models/quick_vote.rb
index d6ab55b124dd2daa328b7452c80880975b553a31..c04b0d7a3fe3a3539012ba45c50f4cc5632f2687 100644 (file)
--- a/app/models/quick_vote.rb
+++ b/app/models/quick_vote.rb
@@ -1,51 +1,83 @@
 class QuickVote < Election
   after_validation :create_candidates
   validates_uniqueness_of :name
 class QuickVote < Election
   after_validation :create_candidates
   validates_uniqueness_of :name

+  validates_presence_of :name

   attr_accessor :raw_candidates
   attr_accessor :reviewed
   attr_accessor :raw_candidates
   attr_accessor :reviewed

-
-  def validate
-    if @raw_candidates.length < 2
-      errors.add("You must list at least two candidates.")
-    end
-
-    if name =~ /[^A-Za-z0-9]/
-      errors.add("The name must only include numbers and letters.")
-    end
-  end

   
   def initialize(params={})
     super
   
   def initialize(params={})
     super

-    self.enddate =  DateTime.now + 30
+    self.startdate = Time.now

     self.active = 1
     self.active = 1

-    self.anonymous = 1
+    self.anonymous = 1 unless self.anonymous
+    self.enddate = read_attribute( :enddate ) || \
+                   Time.now + 30.days - 1.second
+  end
+  
+  def enddate
+    super(30)

   end
   end

+  
+  def validate
+    if not @raw_candidates or @raw_candidates.length < 2
+      errors.add(nil, "You must list at least two candidates.")
+    end
+    
+    @raw_candidates.each do |c|
+      unless c.instance_of? String
+        errors.add(nil, "Candidates must be strings")
+        next
+      end
+      c.strip!
+      if c.length == 0
+        errors.add(nil, "Candidate name must not be empty")
+        next
+      end
+    end if @raw_candidates

 
 

-  def candidatelist=(candstring='')
-    @raw_candidates = candstring.split(';').collect {|cand| cand.strip }
+    errors.add(nil, "Candidates must all be unique") if @raw_candidates and @raw_candidates.uniq!
+
+    if name =~ /[^A-Za-z0-9]/
+      errors.add(:name, "must only include numbers and letters.")
+    end
+    if name =~ /^[0-9]+$/
+      errors.add(:name, "must not be a number")
+    end
+    
+    if name =~ /^(create|index|confirm|change|results)$/
+      errors.add(:name, " is a reserved word.")
+    end

   end
 
   end
 

-  def candidatelist
-    @raw_candidates.join("; ")
+  def candidatelist=(candlist)
+    @raw_candidates = candlist

   end
 
   def name
   end
 
   def name

-    read_attribute( :name ).downcase()
+    read_attribute( :name ).downcase() if read_attribute( :name )

   end
 
   def reviewed?
   end
 
   def reviewed?

-    if reviewed.to_i == 1
-      return true
-    else
-      false
-    end
+    reviewed.to_i == 1

   end
 
   def create_candidates
   end
 
   def create_candidates

+    return unless errors.empty?

     @raw_candidates.each do |name|
       candidate = Candidate.new({:name => name})
     @raw_candidates.each do |name|
       candidate = Candidate.new({:name => name})

-      candidate.save

       self.candidates << candidate
     end
   end
       self.candidates << candidate
     end
   end

+
+  ### Convert a shortname or id into a QuickVote
+  def self.ident_to_quickvote(ident)
+    return nil unless ident
+    if ident.match(/^\d+$/)
+      quickvote = QuickVote.find(ident)
+    else
+      quickvote = QuickVote.find(:all, :conditions => ["name = ?", ident])[0]
+    end
+
+    return quickvote
+  end

 end
 end