scaffold_plus 1.8.0 → 1.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5b36d5e911b460ba3b91084f076a0c9905aab19d
-  data.tar.gz: fdcefa09351b7cfac5a43e460ea62f38c5626bb8
+  metadata.gz: 784dd7153e90d8309d4fff348da8592ea4a95e7d
+  data.tar.gz: c177a2e5941f52aecf4e141089f93da22b212de0
 SHA512:
-  metadata.gz: 3a7a18ef9e70632da0d2bdf1c3698ee20a0579f867c5e7a59d752a8021638e2a42e4622bba1848237be31a754f1671f9950bb590ffa09f5b27fe538b260c6e13
-  data.tar.gz: 5f14a2d39d0e5c38d0d94adf7c34bb6f6c58f596ea3d5fe308a18fcd2b9d0b1fd5fe5dae91ebb91541f982bd858812b15ebbcbfa581f4988a07933922cf16f7c
+  metadata.gz: fc2de205ec80b1a11dfc489ed615f6cb40b3a62c0b54831749b8ffe5314fcbba5984c12617489614789b308a2ee759383e9a71562651cde39ff03a8ac1ad854d
+  data.tar.gz: 6a74fa0d56e659817da2fbb5b5fbf882bb51ba5ac9b0e76368ac23730ddef82cdbd20f4e50b5bde30b4e0251afa759bf7b486fc04d9c7c03297a7ef6d00dd05c

data/README.md

@@ -27,6 +27,13 @@ Or install it yourself as:
 
 This helper creates migrations for some 'ALTER TABLE' statements.
 
+### Add support for an enum field (requires Rails 4.1 and the enum_help Gem)
+    rails generate scaffold_plus:enum
+
+This helper adds parent#has_many and child#belongs_to to the models
+and updates the mass assignment whitelist in the controller.
+It can also add a migration for the parent_id and a counter.
+
 ### Add regular one-to-many association (has_many / belongs_to)
     rails generate scaffold_plus:has_many
 

data/lib/generators/scaffold_plus/enum/enum_generator.rb

@@ -0,0 +1,56 @@
+require 'rails/generators/active_record'
+
+module ScaffoldPlus
+  module Generators
+    class EnumGenerator < ActiveRecord::Generators::Base
+      desc "Add an enum field to a resource (requires enum_help gem)"
+      argument :name, type: :string,
+               desc: "The object that will have the enum field"
+      argument :column, type: :string,
+               desc: "The column to be used as enum field"
+      argument :values, type: :array, banner: "VALUE [...]",
+               desc: "Values (counting from 0, first is default)"
+      class_option :migration, type: :boolean, default: false,
+               desc: 'Create a migration for added attributes'
+      class_option :index, type: :boolean, default: true,
+               desc: 'Add an index to the migration'
+      class_option :permit, type: :boolean, default: false,
+               desc: 'Allow mass assignment for added attributes'
+      class_option :before, type: :boolean, default: false,
+               desc: 'Add a line before generated text in model'
+      class_option :after, type: :boolean, default: false,
+               desc: 'Add a line after generated text in model'
+      source_root File.expand_path('../templates', __FILE__)
+
+      def add_migration
+        return unless options.migration?
+        migration_template "enum_migration.rb", "db/migrate/#{migration_name}.rb"
+      end
+
+      def update_model
+        inject_into_class "app/models/#{name}.rb", class_name do
+          list = values.map{|v| ":#{v}"}.join(', ')
+          text = options.before? ? "\n" : ""
+          text << "  enum #{column}: [ #{list} ]\n"
+          text << "\n" if options.after?
+          text
+        end
+      end
+
+      def update_controller
+        return unless options.permit?
+        text = ":#{column}"
+        file = "app/controllers/#{table_name}_controller.rb"
+        gsub_file file, /(permit\(.*)\)/, "\\1, #{text})"
+        # Special case: no previous permit
+        gsub_file file, /^(\s*params)\[:#{name}\]$/, "\\1.require(:#{name}).permit(#{text})"
+      end
+
+      protected
+
+      def migration_name
+        "add_ancestry_to_#{table_name}"
+      end
+    end
+  end
+end

data/lib/generators/scaffold_plus/enum/templates/enum_migration.rb

@@ -0,0 +1,8 @@
+class <%= migration_name.camelize %> < ActiveRecord::Migration
+  def change
+    add_column :<%= table_name %>, :<%= column %>, :integer, default: 0
+<%- if options.index? -%>
+    add_index :<%= table_name %>, :<%= column %>
+<%- end -%>
+  end
+end

data/lib/generators/scaffold_plus/has_many/has_many_generator.rb

@@ -10,6 +10,8 @@ module ScaffoldPlus
                desc: "The child resources that belongs_to the NAME object"
       class_option :dependent, type: :string, banner: 'ACTION',
                desc: 'Can be destroy, delete, or restrict'
+      class_option :permit, type: :boolean, default: false,
+               desc: 'Allow mass assignment for added attributes'
       class_option :nested, type: :array, banner: 'attribute [...]',
                desc: 'Add accepts_nested_attributes_for (incl. whitelisting)'
       class_option :route, type: :boolean, default: false,
@@ -79,7 +81,7 @@ module ScaffoldPlus
         end
       end
 
-      def add_to_permit
+      def update_parent_controller
         return if options.nested.blank?
         list = options.nested.map{|n| ":#{n}"}.join(', ')
         text = "#{children}_attributes: [ #{list} ]"
@@ -89,7 +91,16 @@ module ScaffoldPlus
         gsub_file file, /^(\s*params)\[:#{name}\]$/, "\\1.require(:#{name}).permit(#{text})"
       end
 
-      def update_child_controller_and_view
+      def update_child_controller
+        return unless options.permit?
+        text = ":#{name}_id"
+        file = "app/controllers/#{children}_controller.rb"
+        gsub_file file, /(permit\(.*)\)/, "\\1, #{text})"
+        # Special case: no previous permit
+        gsub_file file, /^(\s*params)\[:#{name}\]$/, "\\1.require(:#{name}).permit(#{text})"
+      end
+
+      def update_nested_resource
         return unless options.route?
         child = children.singularize
         file = "app/controllers/#{children}_controller.rb"

data/lib/generators/scaffold_plus/has_one/has_one_generator.rb

@@ -10,8 +10,12 @@ module ScaffoldPlus
                desc: "The child resource that belongs_to the NAME object"
       class_option :dependent, type: :string, banner: 'ACTION',
                desc: 'Can be destroy, delete, or restrict'
+      class_option :permit, type: :boolean, default: false,
+               desc: 'Allow mass assignment for added attributes'
       class_option :nested, type: :array, banner: 'attribute [...]',
                desc: 'Add accepts_nested_attributes_for (incl. whitelisting)'
+      class_option :route, type: :boolean, default: false,
+               desc: 'Add a nested route and update CHILD controller'
       class_option :foreign_key, type: :string,
                desc: 'Set the name of the foreign key directly'
       class_option :inverse, type: :boolean, default: false,
@@ -31,6 +35,16 @@ module ScaffoldPlus
         migration_template 'child_migration.rb', "db/migrate/#{migration_name}.rb"
       end
 
+      def add_to_route
+        return unless options.route?
+        gsub_file "config/routes.rb", /^  resources :#{table_name} do$/ do |match|
+          match << "\n    resources :#{child.pluralize}"
+        end
+        gsub_file "config/routes.rb", /^  resources :#{table_name}$/ do |match|
+          match << " do\n    resources :#{child.pluralize}\n  end"
+        end
+      end
+
       def add_to_models
         inject_into_class "app/models/#{name}.rb", class_name do
           text = before_array.include?(name) ? "\n" : ""
@@ -58,16 +72,42 @@ module ScaffoldPlus
         end
       end
 
-      def add_to_permit
-        return unless options.nested
+      def update_parent_controller
+        return if options.nested.blank?
         list = options.nested.map{|n| ":#{n}"}.join(', ')
-        text = ":#{child}, #{child}_attributes: [ #{list} ]"
+        text = "#{child}_attributes: [ #{list} ]"
         file = "app/controllers/#{table_name}_controller.rb"
         gsub_file file, /(permit\(.*)\)/, "\\1, #{text})"
         # Special case: no previous permit
         gsub_file file, /^(\s*params)\[:#{name}\]$/, "\\1.require(:#{name}).permit(#{text})"
       end
 
+      def update_child_controller
+        return unless options.permit?
+        text = ":#{name}_id"
+        file = "app/controllers/#{child.pluralize}_controller.rb"
+        gsub_file file, /(permit\(.*)\)/, "\\1, #{text})"
+        # Special case: no previous permit
+        gsub_file file, /^(\s*params)\[:#{name}\]$/, "\\1.require(:#{name}).permit(#{text})"
+      end
+
+      def update_nested_resource
+        return unless options.route?
+        children = child.pluralize
+        file = "app/controllers/#{children}_controller.rb"
+        gsub_file file, /GET .#{children}.new$/ do |match|
+          match = "GET /#{table_name}/:id/#{children}/new"
+        end
+        gsub_file file, /^    @#{child} = #{child.camelize}.new$/ do |match|
+          match = "    @#{name} = #{class_name}.find(params[:#{name}_id])\n" +
+                  "    @#{child} = @#{name}.#{children}.build"
+        end
+        file = "app/views/#{children}/_form.html.erb"
+        gsub_file file, /form_for\(@#{child}/ do |match|
+          match = "form_for([@#{name}, @#{child}]"
+        end
+      end
+
       protected
 
       def before_array

data/lib/scaffold_plus/version.rb

@@ -1,3 +1,3 @@
 module ScaffoldPlus
-  VERSION = "1.8.0"
+  VERSION = "1.9.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scaffold_plus
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.9.0
 platform: ruby
 authors:
 - Volker Wiegand
@@ -84,6 +84,8 @@ files:
 - lib/generators/scaffold_plus/autofocus/autofocus_generator.rb
 - lib/generators/scaffold_plus/collection/collection_generator.rb
 - lib/generators/scaffold_plus/collection/templates/view.html.erb
+- lib/generators/scaffold_plus/enum/enum_generator.rb
+- lib/generators/scaffold_plus/enum/templates/enum_migration.rb
 - lib/generators/scaffold_plus/force_ssl/force_ssl_generator.rb
 - lib/generators/scaffold_plus/friendly_id/friendly_id_generator.rb
 - lib/generators/scaffold_plus/friendly_id/templates/friendly_id_migration.rb