]> projects.mako.cc - selectricity-live/commitdiff
Refactor the ident_to_quickvote method as a classmethod of the model
authorJohn Dong <jdong@mit.edu>
Tue, 14 Aug 2007 20:50:49 +0000 (16:50 -0400)
committerJohn Dong <jdong@mit.edu>
Tue, 14 Aug 2007 20:50:49 +0000 (16:50 -0400)
app/controllers/quickvote_controller.rb
app/models/quick_vote.rb

index acbf012ff5cb3df503371f59f4b8d71c466e567f..83a6cc5fc659e5bcfabcc75a1d6e31baa7af5474 100644 (file)
@@ -49,7 +49,7 @@ class QuickvoteController < ApplicationController
   #############################################################
 
   def index
-    @election = ident_to_quickvote(params[:ident])
+    @election = QuickVote.ident_to_quickvote(params[:ident])
     
     # if the person has specified an election, we show them the voting
     # page. otherwise, we redirect back to main the page
@@ -87,7 +87,7 @@ class QuickvoteController < ApplicationController
 
   def confirm
     # we need the election to verify that we have the right voter
-    election = ident_to_quickvote(params[:ident])
+    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 = ?", 
@@ -142,21 +142,9 @@ class QuickvoteController < ApplicationController
   ###############################################################
 
   def results
-    @election = ident_to_quickvote(params[:ident])
+    @election = QuickVote.ident_to_quickvote(params[:ident])
     @election.results
     @candidates = {}
     @election.candidates.each {|c| @candidates[c.id] = c}
   end
-
-  private
-  def ident_to_quickvote(ident)
-    if ident.match(/^\d+$/)
-      quickvote = QuickVote.find(ident)
-    else
-      quickvote = QuickVote.find_all(["name = ?", ident])[0]
-    end
-
-    return quickvote
-  end
-
 end
index 6259784bfbaf1bc847ad8e67b6fad04a2722cf45..5f566c6eb1e6575e1f8f6a7845e67ba21d963c53 100644 (file)
@@ -74,4 +74,15 @@ class QuickVote < Election
 
 
   end
+
+  ### Convert a shortname or id into a QuickVote
+  def self.ident_to_quickvote(ident)
+    if ident.match(/^\d+$/)
+      quickvote = QuickVote.find(ident)
+    else
+      quickvote = QuickVote.find_all(["name = ?", ident])[0]
+    end
+
+    return quickvote
+  end
 end

Benjamin Mako Hill || Want to submit a patch?