Refactoring migration to make it more readable.

gerwinbrunner · Jun 5, 2009 · 16be16d · 16be16d
1 parent 327f259
commit 16be16d
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/db/migrate/20090605160611_update_content_of_unmodified_default_templates.rb b/db/migrate/20090605160611_update_content_of_unmodified_default_templates.rb
@@ -1,19 +1,22 @@
 class UpdateContentOfUnmodifiedDefaultTemplates < ActiveRecord::Migration
-  TEMPLATES = %w( invoice packing_slip variable_reference )
+  TEMPLATES    = %w( invoice packing_slip variable_reference )
 
   NEW_CONTENTS = TEMPLATES.inject({}) do |memo, template|
                    memo[template] = File.read("#{RAILS_ROOT}/db/printing/#{template}.liquid")
                    memo
                  end
-
-  MD5 = {"invoice"            => "3cbe75be4e5559a0643581bb48c43222", 
-         "packing_slip"       => "84054e0b1b3659a969c0d3b984aff8e7", 
-         "variable_reference" => "caa1eb749f63a296ebddf9019ba188c8" }
+
+  ORIGINAL_MD5 = { "invoice"            => ["3cbe75be4e5559a0643581bb48c43222", "cb44126c6c299701fef22058670ae34f"],
+                   "packing_slip"       => ["84054e0b1b3659a969c0d3b984aff8e7", "cbaba8dfbcc4b64955b7ed12c708a4ea"], 
+                   "variable_reference" => ["caa1eb749f63a296ebddf9019ba188c8", "328811bd3ee651c26f1e1c46bc8850eb", "b1775d4c8c17320f45a3d40601562f02"]
+                 }
 
   def self.up
     transaction do
       TEMPLATES.each do |template|
-        execute "UPDATE print_templates SET body = #{NEW_CONTENTS[template].inspect} WHERE MD5(body) IN (#{MD5[template].join(', ')})"
+        # Hashes of old default template contents (save to overwrite)        
+        hashes = ORIGINAL_MD5[template].collect{ |md5| "'#{md5}'"}.join(",")
+        execute "UPDATE print_templates SET body = #{NEW_CONTENTS[template].inspect} WHERE MD5(body) IN (#{hashes})"
       end
     end    
   end