]> projects.mako.cc - selectricity-live/commitdiff
Streamlined and made more consistent the code that checks for custom skin uploads...
author<jlsharps@mit.edu> <>
Wed, 4 Jun 2008 23:28:40 +0000 (19:28 -0400)
committer<jlsharps@mit.edu> <>
Wed, 4 Jun 2008 23:28:40 +0000 (19:28 -0400)
app/controllers/election_controller.rb

index 0b529b6f4630d76d8a7f6920c36bf24c15684d7c..68d59cc520fd1905474a12dea3991894682151af 100644 (file)
@@ -46,7 +46,8 @@ class ElectionController < ApplicationController
     @election.anonymous = 1
     @election.startdate = Time.now
     
-    unless params[:top_bar][:uploaded_data].to_s.empty?
+    holder = create_theme_hash
+    unless holder.values.all? {|v| v.has_value?("")}
       token_generator = UniqueTokenGenerator.new( 16 )
       @election.embed_custom_string = token_generator.token
       add_theme(@election.embed_custom_string)
@@ -60,6 +61,15 @@ class ElectionController < ApplicationController
     end
   end
   
+  def create_theme_hash
+    target = Hash.new
+    params.each do |k,v|
+      target[k] = v if k=="top_bar" or k=="default_image" or k=="bg1" \
+                    or k=="bg2" or k=="bottom_bar"
+    end
+    return target
+  end
+  
   # TODO add filter to verify that the person working on or looking at
   # something is the owner
   def edit_general_information
@@ -69,7 +79,9 @@ class ElectionController < ApplicationController
   def update_general_information
     @election = Election.find(params[:id])
     
-    unless (params[:top_bar][:uploaded_data].to_s.empty? and params[:default_image][:uploaded_data].to_s.empty? and params[:bg1][:uploaded_data].to_s.empty? and params[:bg2][:uploaded_data].to_s.empty? and params[:bottom_bar][:uploaded_data].to_s.empty?)
+    
+    holder = create_theme_hash
+    unless holder.values.all? {|v| v.has_value?("")}
       unless @election.embed_custom_string
         token_generator = UniqueTokenGenerator.new( 16 )
         @election.embed_custom_string = token_generator.token
@@ -86,6 +98,7 @@ class ElectionController < ApplicationController
     end
   end
   
+  #Takes care of uploading custom images -- unnecessarily long, how can I compress?
   def add_theme(prefix)
     unless params[:top_bar][:uploaded_data].to_s.empty?
       previous = SkinPicture.find(:first,

Benjamin Mako Hill || Want to submit a patch?