fix migration generator for greater-than-one-word model names e.g. En…

…codingProfile

generators/revisable_migration/revisable_migration_generator.rb

@@ -1,9 +1,21 @@
 class RevisableMigrationGenerator < Rails::Generator::NamedBase
   def manifest
     record do |m|
-      revisable_columns = [["revisable_original_id", "integer"], ["revisable_branched_from_id", "integer"], ["revisable_number", "integer"], ["revisable_name", "string"], ["revisable_type", "string"], ["revisable_current_at", "datetime"], ["revisable_revised_at", "datetime"], ["revisable_deleted_at", "datetime"], ["revisable_is_current", "boolean"]]
+      revisable_columns = [
+        ["revisable_original_id",      "integer"],
+        ["revisable_branched_from_id", "integer"],
+        ["revisable_number",           "integer"],
+        ["revisable_name",             "string"],
+        ["revisable_type",             "string"],
+        ["revisable_current_at",       "datetime"],
+        ["revisable_revised_at",       "datetime"],
+        ["revisable_deleted_at",       "datetime"],
+        ["revisable_is_current",       "boolean"]
+      ]
 
-      m.migration_template 'migration.rb', 'db/migrate', :migration_file_name => "make_#{class_name.downcase}_revisable", :assigns => {:cols => revisable_columns}
+      m.migration_template 'migration.rb', 'db/migrate',
+        :migration_file_name => "make_#{class_name.underscore.pluralize}_revisable",
+        :assigns => { :cols => revisable_columns }
     end
   end
 end

generators/revisable_migration/templates/migration.rb

@@ -1,13 +1,14 @@
-class Make<%= class_name.underscore.camelize %>Revisable < ActiveRecord::Migration
+<% table_name = class_name.underscore.pluralize -%>
+class Make<%= class_name.underscore.pluralize.camelize %>Revisable < ActiveRecord::Migration
   def self.up
-    <% cols.each do |c| -%>
-    add_column :<%= class_name.downcase.pluralize %>, :<%= c.first %>, :<%= c.last %>
+    <% cols.each do |column_name,column_type| -%>
+    add_column :<%= table_name %>, :<%= column_name %>, :<%= column_type %>
     <% end -%>
   end
 
   def self.down
-    <% cols.each do |c| -%>
-    remove_column :<%= class_name.downcase.pluralize %>, :<%= c.first %>
+    <% cols.each do |column_name,_| -%>
+    remove_column :<%= table_name %>, :<%= column_name %>
     <% end -%>
   end
 end