hot-glue 0.5.14 → 0.5.16

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -1,4 +1,3 @@
-
 require 'rails/generators/erb/scaffold/scaffold_generator'
 require_relative './default_config_loader'
 require 'ffaker'
@@ -12,23 +11,16 @@ require_relative './layout_strategy/bootstrap'
 require_relative './layout_strategy/hot_glue'
 require_relative './layout_strategy/tailwind'
 
-
-
 class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   include DefaultConfigLoader
   hook_for :form_builder, :as => :scaffold
 
   source_root File.expand_path('templates', __dir__)
-  attr_accessor :alt_lookups, :attachments, :auth,
-                :big_edit, :bootstrap_column_width,
-                :columns,
-                :downnest_children, :downnest_object,
-                :button_icons,
-                :hawk_keys, :layout_object,
-                :nest_with,
-                :path,  :plural,
-                :sample_file_path,  :singular, :singular_class, :smart_layout,
-                :stacked_downnesting,  :update_show_only
+  attr_accessor :alt_lookups, :attachments, :auth, :big_edit, :button_icons, :bootstrap_column_width, :columns,
+                :downnest_children, :downnest_object, :hawk_keys, :layout_object, :modify,
+                :nest_with, :path, :plural, :sample_file_path, :show_only_data, :singular,
+                :singular_class, :smart_layout, :stacked_downnesting, :update_show_only, :ownership_field,
+                :layout_strategy, :form_placeholder_labels, :form_labels_position
 
   class_option :singular, type: :string, default: nil
   class_option :plural, type: :string, default: nil
@@ -76,7 +68,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   # determines if the labels show up BEFORE or AFTER on the NEW/EDIT (form)
   class_option :form_labels_position, type: :string, default: 'after' #  choices are before, after, omit
-  class_option :form_placeholder_labels, type: :boolean,  default: false # puts the field names into the placeholder labels
+  class_option :form_placeholder_labels, type: :boolean, default: false # puts the field names into the placeholder labels
 
   # determines if labels appear within the rows of the VIEWABLE list (does NOT affect the list heading)
   class_option :inline_list_labels, default: 'omit' # choices are before, after, omit
@@ -84,8 +76,9 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   class_option :alt_foreign_key_lookup, default: '' #
   class_option :attachments, default: ''
   class_option :stacked_downnesting, default: false
-  class_option :bootstrap_column_width, default: nil #must be nil to detect if user has not passed
+  class_option :bootstrap_column_width, default: nil # must be nil to detect if user has not passed
   class_option :button_icons, default: nil
+  class_option :modify, default: {}
 
   def initialize(*meta_args)
     super
@@ -101,17 +94,16 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @meta_args = meta_args
 
     if options['specs_only'] && options['no_specs']
-      raise(HotGlue::Error,  "*** Oops: You seem to have specified both the --specs-only flag and --no-specs flags. this doesn't make any sense, so I am aborting. Aborting.")
+      raise(HotGlue::Error, "*** Oops: You seem to have specified both the --specs-only flag and --no-specs flags. this doesn't make any sense, so I am aborting. Aborting.")
     end
 
     if !options['exclude'].empty? && !options['include'].empty?
-      exit_message =  "*** Oops: You seem to have specified both --include and --exclude. Please use one or the other. Aborting."
+      exit_message = "*** Oops: You seem to have specified both --include and --exclude. Please use one or the other. Aborting."
       puts exit_message
 
       raise(HotGlue::Error, exit_message)
     end
 
-
     if @stimulus_syntax.nil?
       @stimulus_syntax = true
     end
@@ -121,7 +113,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       raise(HotGlue::Error, message)
     end
 
-    @markup =  get_default_from_config(key: :markup)
+    @markup = get_default_from_config(key: :markup)
     @sample_file_path = get_default_from_config(key: :sample_file_path)
     @bootstrap_column_width ||= get_default_from_config(key: :bootstrap_column_width) || 2
 
@@ -156,13 +148,13 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
     @plural = options['plural'] || @singular.pluralize # respects what you set in inflections.rb, to override, use plural option
     @namespace = options['namespace'] || nil
-    use_controller_name =  plural.titleize.gsub(" ", "")
-    @controller_build_name = (( @namespace.titleize.gsub(" ","") + "::" if @namespace) || "") + use_controller_name + "Controller"
+    use_controller_name = plural.titleize.gsub(" ", "")
+    @controller_build_name = ((@namespace.titleize.gsub(" ", "") + "::" if @namespace) || "") + use_controller_name + "Controller"
     @controller_build_folder = use_controller_name.underscore
     @controller_build_folder_singular = singular
 
     @auth = options['auth'] || "current_user"
-    @auth_identifier = options['auth_identifier'] || (! @god && @auth.gsub("current_", "")) || nil
+    @auth_identifier = options['auth_identifier'] || (!@god && @auth.gsub("current_", "")) || nil
 
     if options['nest']
       raise HotGlue::Error, "STOP: the flag --nest has been replaced with --nested; please re-run using the --nested flag"
@@ -179,20 +171,48 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @include_fields = []
 
       # semicolon to denote layout columns; commas separate fields
-      @include_fields += options['include'].split(":").collect{|x|x.split(",")}.flatten.collect(&:to_sym)
+      @include_fields += options['include'].split(":").collect { |x| x.split(",") }.flatten.collect(&:to_sym)
     end
 
-    @show_only = []
-    if !options['show_only'].empty?
-      @show_only += options['show_only'].split(",").collect(&:to_sym)
+    # @show_only_data = {}
+    # if !options['show_only'].empty?
+    #   show_only_input = options['show_only'].split(",")
+    #   show_only_input.each do |setting|
+    #     if setting.include?("[")
+    #       setting =~ /(.*)\[(.*)\]/
+    #       key, lookup_as = $1, $2
+    #       @show_only_data[key.to_sym] = {cast: $2 }
+    #     else
+    #       @show_only_data[setting.to_sym] = {cast: nil}
+    #     end
+    #   end
+    # end
+
+    @show_only = options['show_only'].split(",").collect(&:to_sym)
+    if @show_only.any?
+      puts "show only field #{@show_only}}"
     end
 
+    @modify = {}
+    if !options['modify'].empty?
+      modify_input = options['modify'].split(",")
+      modify_input.each do |setting|
+        setting =~ /(.*){(.*)}/
+        key, lookup_as = $1, $2
+
+        if ["$"].include?($2)
+          @modify[key.to_sym] =  {cast: $2}
+        elsif $2.include?("|")
+          binary = $2.split("|")
+          @modify[key.to_sym] =  {binary: {truthy: binary[0], falsy: binary[1]}}
+        end
+      end
+    end
     @update_show_only = []
     if !options['update_show_only'].empty?
       @update_show_only += options['update_show_only'].split(",").collect(&:to_sym)
     end
 
-
     # syntax should be xyz_id{xyz_email},abc_id{abc_email}
     # instead of a drop-down for the foreign entity, a text field will be presented
     # You must ALSO use a factory that contains a parameter of the same name as the 'value' (for example, `xyz_email`)
@@ -202,48 +222,40 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @alt_foreign_key_lookup = alt_lookups_entry.each do |setting|
       setting =~ /(.*){(.*)}/
       key, lookup_as = $1, $2
-      assoc = eval("#{class_name}.reflect_on_association(:#{key.to_s.gsub("_id","")}).class_name")
+      assoc = eval("#{class_name}.reflect_on_association(:#{key.to_s.gsub("_id", "")}).class_name")
 
-      data = {lookup_as: lookup_as.gsub("+",""),
-              assoc: assoc,
-              with_create: lookup_as.include?("+")}
+      data = { lookup_as: lookup_as.gsub("+", ""),
+               assoc: assoc,
+               with_create: lookup_as.include?("+") }
       @alt_lookups[key] = data
     end
 
     puts "------ ALT LOOKUPS for #{@alt_lookups}"
 
-    @update_alt_lookups = @alt_lookups.collect{|key, value|
+    @update_alt_lookups = @alt_lookups.collect { |key, value|
       @update_show_only.include?(key) ?
-        {  key: value }
-        : nil}.compact
+        { key: value }
+        : nil }.compact
 
-    @label = options['label'] || ( eval("#{class_name}.class_variable_defined?(:@@table_label_singular)") ? eval("#{class_name}.class_variable_get(:@@table_label_singular)") :  singular.gsub("_", " ").titleize )
-    @list_label_heading =  options['list_label_heading'] || ( eval("#{class_name}.class_variable_defined?(:@@table_label_plural)") ? eval("#{class_name}.class_variable_get(:@@table_label_plural)") : plural.gsub("_", " ").upcase )
+    @label = options['label'] || (eval("#{class_name}.class_variable_defined?(:@@table_label_singular)") ? eval("#{class_name}.class_variable_get(:@@table_label_singular)") : singular.gsub("_", " ").titleize)
+    @list_label_heading = options['list_label_heading'] || (eval("#{class_name}.class_variable_defined?(:@@table_label_plural)") ? eval("#{class_name}.class_variable_get(:@@table_label_plural)") : plural.gsub("_", " ").upcase)
 
-    @new_button_label = options['new_button_label'] || ( eval("#{class_name}.class_variable_defined?(:@@table_label_singular)") ? "New " + eval("#{class_name}.class_variable_get(:@@table_label_singular)") : "New " + singular.gsub("_", " ").titleize )
+    @new_button_label = options['new_button_label'] || (eval("#{class_name}.class_variable_defined?(:@@table_label_singular)") ? "New " + eval("#{class_name}.class_variable_get(:@@table_label_singular)") : "New " + singular.gsub("_", " ").titleize)
     @new_form_heading = options['new_form_heading'] || "New #{@label}"
 
-
-
     setup_hawk_keys
     @form_placeholder_labels = options['form_placeholder_labels'] # true or false
-    @inline_list_labels = options['inline_list_labels']  || 'omit' # 'before','after','omit'
-
+    @inline_list_labels = options['inline_list_labels'] || 'omit' # 'before','after','omit'
 
     @form_labels_position = options['form_labels_position']
-    if !['before','after','omit'].include?(@form_labels_position)
+    if !['before', 'after', 'omit'].include?(@form_labels_position)
       raise HotGlue::Error, "You passed '#{@form_labels_position}' as the setting for --form-labels-position but the only allowed options are before, after (default), and omit"
     end
 
-    if !['before','after','omit'].include?(@inline_list_labels)
+    if !['before', 'after', 'omit'].include?(@inline_list_labels)
       raise HotGlue::Error, "You passed '#{@inline_list_labels}' as the setting for --inline-list-labels but the only allowed options are before, after, and omit (default)"
     end
 
-
-
-
-
-
     @god = options['god'] || options['gd'] || false
     @specs_only = options['specs_only'] || false
 
@@ -260,9 +272,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @no_list_heading = options['no_list_heading'] || false
     @stacked_downnesting = options['stacked_downnesting']
 
-
-
-
     @display_list_after_update = options['display_list_after_update'] || false
     @smart_layout = options['smart_layout']
 
@@ -276,7 +285,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @downnest_children = [] # TODO: defactor @downnest_children in favor of downnest_object
     @downnest_object = {}
     if @downnest
-      @downnest_children = @downnest.split(",").map{|child| child.gsub("+","")}
+      @downnest_children = @downnest.split(",").map { |child| child.gsub("+", "") }
       @downnest_object = HotGlue.construct_downnest_object(@downnest)
     end
 
@@ -299,7 +308,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @magic_buttons = options['magic_buttons'].split(',')
     end
 
-
     @small_buttons = options['small_buttons'] || false
 
     @build_update_action = !@no_edit || !@magic_buttons.empty?
@@ -310,7 +318,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @ujs_syntax = !defined?(Turbo::Engine)
     end
 
-
     # NEST CHAIN
     # new syntax
     # @nested_set = [
@@ -321,10 +328,10 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     # }]
     @nested_set = []
 
-    if ! @nested.nil?
+    if !@nested.nil?
       @nested_set = @nested.split("/").collect { |arg|
         is_optional = arg.start_with?("~")
-        arg.gsub!("~","")
+        arg.gsub!("~", "")
         {
           singular: arg,
           plural: arg.pluralize,
@@ -342,8 +349,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @object_owner_optional = false
       @object_owner_name = @auth.gsub("current_", "").to_s
 
-
-    elsif @auth && ! @self_auth && @nested_set.none? && !@auth.include?(".")
+    elsif @auth && !@self_auth && @nested_set.none? && !@auth.include?(".")
       @object_owner_sym = @auth.gsub("current_", "").to_sym
       @object_owner_eval = @auth
       @object_owner_optional = false
@@ -364,12 +370,11 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       end
     end
 
-
     @factory_creation = options['factory_creation'].gsub(";", "\n")
     identify_object_owner
     setup_fields
 
-    if  (@columns - @show_only - (@ownership_field ?  [@ownership_field.to_sym] : [])).empty?
+    if (@columns - @show_only - (@ownership_field ? [@ownership_field.to_sym] : [])).empty?
       @no_field_form = true
     end
 
@@ -379,7 +384,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @associations = []
     @columns_map = {}
     @columns.each do |col|
-      if !(@the_object.columns_hash.keys.include?(col.to_s) ||  @attachments.keys.include?(col))
+      if !(@the_object.columns_hash.keys.include?(col.to_s) || @attachments.keys.include?(col))
         raise "couldn't find #{col} in either field list or attachments list"
       end
 
@@ -388,7 +393,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       end
 
       if @the_object.columns_hash.keys.include?(col.to_s)
-        type =  @the_object.columns_hash[col.to_s].type
+        type = @the_object.columns_hash[col.to_s].type
       elsif @attachments.keys.include?(col)
         type = :attachment
       end
@@ -402,10 +407,8 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @columns_map[col] = this_column_object.field
     end
 
-
-
     # create the template object
-    if  @markup == "erb"
+    if @markup == "erb"
       @template_builder = HotGlue::ErbTemplate.new(
         layout_strategy: @layout_strategy,
         magic_buttons: @magic_buttons,
@@ -423,26 +426,22 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
         attachments: @attachments,
         columns_map: @columns_map
       )
-    elsif  @markup == "slim"
-      raise(HotGlue::Error,  "SLIM IS NOT IMPLEMENTED")
-    elsif  @markup == "haml"
-      raise(HotGlue::Error,  "HAML IS NOT IMPLEMENTED")
+    elsif @markup == "slim"
+      raise(HotGlue::Error, "SLIM IS NOT IMPLEMENTED")
+    elsif @markup == "haml"
+      raise(HotGlue::Error, "HAML IS NOT IMPLEMENTED")
     end
 
     builder = HotGlue::Layout::Builder.new(generator: self,
-                                         include_setting: options['include'],
-                                         buttons_width: buttons_width )
+                                           include_setting: options['include'],
+                                           buttons_width: buttons_width)
     @layout_object = builder.construct
 
-
-
     @menu_file_exists = true if @nested_set.none? && File.exist?("#{Rails.root}/app/views/#{namespace_with_trailing_dash}_menu.#{@markup}")
 
     @turbo_streams = !!options['with_turbo_streams']
   end
 
-
-
   def setup_hawk_keys
     @hawk_keys = {}
 
@@ -459,9 +458,9 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
         end
 
         hawk_scope = key.gsub("_id", "").pluralize
-        optional = eval(singular_class + ".reflect_on_association(:#{key.gsub('_id','')})").options[:optional]
+        optional = eval(singular_class + ".reflect_on_association(:#{key.gsub('_id', '')})").options[:optional]
 
-        @hawk_keys[key.to_sym] = {bind_to: [hawk_to], optional: optional}
+        @hawk_keys[key.to_sym] = { bind_to: [hawk_to], optional: optional }
         use_shorthand = !options["hawk"].include?("{")
 
         if use_shorthand # only include the hawk scope if using the shorthand
@@ -474,7 +473,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     end
   end
 
-
   def setup_attachments
     @attachments = {}
 
@@ -539,7 +537,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
           direct_upload = nil
           field_for_original_filename = nil
-          dropzone  = nil
+          dropzone = nil
         end
 
         if thumbnail && !eval("#{singular_class}.reflect_on_attachment(:#{key}).variants.include?(:#{thumbnail})")
@@ -550,11 +548,10 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 "
         end
 
-
-        @attachments[key.to_sym] = {thumbnail: thumbnail,
-                                    field_for_original_filename: field_for_original_filename,
-                                    direct_upload: direct_upload,
-                                    dropzone: dropzone}
+        @attachments[key.to_sym] = { thumbnail: thumbnail,
+                                     field_for_original_filename: field_for_original_filename,
+                                     direct_upload: direct_upload,
+                                     dropzone: dropzone }
       end
 
       puts "ATTACHMENTS: #{@attachments}"
@@ -562,15 +559,15 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def identify_object_owner
-    auth_assoc = @auth && @auth.gsub("current_","")
+    auth_assoc = @auth && @auth.gsub("current_", "")
 
-    if @object_owner_sym && ! @self_auth
+    if @object_owner_sym && !@self_auth
       auth_assoc_field = auth_assoc + "_id" unless @god
       assoc = eval("#{singular_class}.reflect_on_association(:#{@object_owner_sym})")
 
       if assoc
         @ownership_field = assoc.name.to_s + "_id"
-      elsif ! @nested_set.any?
+      elsif !@nested_set.any?
         exit_message = "*** Oops: It looks like is no association `#{@object_owner_sym}` from the object #{@singular_class}. If your user is called something else, pass with flag auth=current_X where X is the model for your users as lowercase. Also, be sure to implement current_X as a method on your controller. (If you really don't want to implement a current_X on your controller and want me to check some other method for your current user, see the section in the docs for auth_identifier.) To make a controller that can read all records, specify with --god."
         raise(HotGlue::Error, exit_message)
 
@@ -583,7 +580,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
         raise(HotGlue::Error, exit_message)
       end
-    elsif  @object_owner_sym && ! @object_owner_eval.include?(".")
+    elsif @object_owner_sym && !@object_owner_eval.include?(".")
       @ownership_field = @object_owner_name + "_id"
     end
   end
@@ -596,18 +593,16 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
                            :confirmation_token, :confirmed_at,
                            :confirmation_sent_at, :unconfirmed_email
 
-      @exclude_fields.push( @ownership_field.to_sym ) if ! @ownership_field.nil?
-
-
-      @columns = @the_object.columns.map(&:name).map(&:to_sym).reject{|field| @exclude_fields.include?(field) }
+      @exclude_fields.push(@ownership_field.to_sym) if !@ownership_field.nil?
 
+      @columns = @the_object.columns.map(&:name).map(&:to_sym).reject { |field| @exclude_fields.include?(field) }
 
     else
-      @columns = @the_object.columns.map(&:name).map(&:to_sym).reject{|field| !@include_fields.include?(field) }
+      @columns = @the_object.columns.map(&:name).map(&:to_sym).reject { |field| !@include_fields.include?(field) }
     end
 
     if @attachments.any?
-      puts "adding attachments-as-columns: #{@attachments}"
+      puts "Adding attachments-as-columns: #{@attachments}"
       @attachments.keys.each do |attachment|
         @columns << attachment if !@columns.include?(attachment)
       end
@@ -616,21 +611,16 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     end
   end
 
-
-
-
-
-
   def check_if_sample_file_is_present
     if sample_file_path.nil?
       puts "you have no sample file path set in config/hot_glue.yml"
       settings = File.read("config/hot_glue.yml")
       @sample_file_path = "spec/files/computer_code.jpg"
       added_setting = ":sample_file_path: #{sample_file_path}"
-      File.open("config/hot_glue.yml", "w") { |f| f.write settings + "\n" +   added_setting }
+      File.open("config/hot_glue.yml", "w") { |f| f.write settings + "\n" + added_setting }
 
       puts "adding `#{added_setting}` to config/hot_glue.yml"
-    elsif ! File.exist?(sample_file_path)
+    elsif !File.exist?(sample_file_path)
       puts "NO SAMPLE FILE FOUND: adding sample file at #{sample_file_path}"
       template "computer_code.jpg", File.join("#{filepath_prefix}spec/files/", "computer_code.jpg")
     end
@@ -639,11 +629,11 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def fields_filtered_for_email_lookups
-    @columns.reject{|c| @alt_lookups.keys.include?(c) } + @alt_lookups.values.map{|v| ("__lookup_#{v[:lookup_as]}").to_sym}
+    @columns.reject { |c| @alt_lookups.keys.include?(c) } + @alt_lookups.values.map { |v| ("__lookup_#{v[:lookup_as]}").to_sym }
   end
 
   def creation_syntax
-    merge_with = @alt_lookups.collect{ |key, data|
+    merge_with = @alt_lookups.collect { |key, data|
       "#{data[:assoc].downcase}: #{data[:assoc].downcase}_factory.#{data[:assoc].downcase}"
     }.join(", ")
 
@@ -668,7 +658,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def filepath_prefix
-    'spec/dummy/' if Rails.env.test?
+    'spec/dummy/' if $INTERNAL_SPECS
   end
 
   def copy_controller_and_spec_files
@@ -687,14 +677,14 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     unless @no_specs
       dest_file = File.join("#{filepath_prefix}spec/features#{namespace_with_dash}", "#{plural}_behavior_spec.rb")
 
-      if  File.exist?(dest_file)
+      if File.exist?(dest_file)
         existing_file = File.open(dest_file)
         existing_content = existing_file.read
         if existing_content =~ /\#HOTGLUE-SAVESTART/
-          if  existing_content !~ /\#HOTGLUE-END/
+          if existing_content !~ /\#HOTGLUE-END/
             raise "Your file at #{dest_file} contains a #HOTGLUE-SAVESTART marker without #HOTGLUE-END"
           end
-          @existing_content =  existing_content[(existing_content =~ /\#HOTGLUE-SAVESTART/) .. (existing_content =~ /\#HOTGLUE-END/)-1]
+          @existing_content = existing_content[(existing_content =~ /\#HOTGLUE-SAVESTART/)..(existing_content =~ /\#HOTGLUE-END/) - 1]
           @existing_content << "#HOTGLUE-END"
 
         end
@@ -703,7 +693,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
         @existing_content = "  #HOTGLUE-SAVESTART\n  #HOTGLUE-END"
       end
 
-
       template "system_spec.rb.erb", dest_file
     end
 
@@ -717,7 +706,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def testing_name
-    singular_class.gsub("::","_").underscore
+    singular_class.gsub("::", "_").underscore
   end
 
   def spec_related_column_lets
@@ -736,7 +725,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def columns_spec_with_sample_data
-    @columns_map.map  { |col, col_object|
+    @columns_map.map { |col, col_object|
       unless col_object.is_a?(AssociationField)
         random_data = col_object.spec_random_data
         col.to_s + ": '" + random_data.to_s + "'"
@@ -745,14 +734,14 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def regenerate_me_code
-    "rails generate hot_glue:scaffold #{ @meta_args[0][0] } #{@meta_args[1].collect{|x| x.gsub(/\s*=\s*([\S\s]+)/, '=\'\1\'')}.join(" ")}"
+    "rails generate hot_glue:scaffold #{ @meta_args[0][0] } #{@meta_args[1].collect { |x| x.gsub(/\s*=\s*([\S\s]+)/, '=\'\1\'') }.join(" ")}"
   end
 
   def object_parent_mapping_as_argument_for_specs
 
     if @self_auth
       ""
-    elsif @nested_set.any? && ! @nested_set.last[:optional]
+    elsif @nested_set.any? && !@nested_set.last[:optional]
       ", " + @nested_set.last[:singular] + ": " + @nested_set.last[:singular]
     elsif @auth && !@god
       ", #{@auth_identifier}: #{@auth}"
@@ -773,7 +762,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def objest_nest_params_by_id_for_specs
-    @nested_set.map{|arg|
+    @nested_set.map { |arg|
       "#{arg[:singular]}_id: #{arg[:singular]}.id"
     }.join(",\n          ")
   end
@@ -791,8 +780,8 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def capybara_make_updates(which_partial = :create)
-    @columns_map.map { | col, col_obj|
-      show_only_list = which_partial == :create ? @show_only : (@update_show_only+@show_only)
+    @columns_map.map { |col, col_obj|
+      show_only_list = which_partial == :create ? @show_only : (@update_show_only + @show_only)
 
       if show_only_list.include?(col)
         "      page.should have_no_selector(:css, \"[name='#{testing_name}[#{ col.to_s }]'\")"
@@ -804,7 +793,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   def path_helper_args
     if @nested_set.any? && @nested
-      [(@nested_set).collect{|a| "#{a[:singular]}"} , singular].join(",")
+      [(@nested_set).collect { |a| "#{a[:singular]}" }, singular].join(",")
     else
       singular
     end
@@ -812,9 +801,9 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   def path_helper_singular
     if @nested
-      "#{@namespace+"_" if @namespace}#{(@nested_set.collect{|x| x[:singular]}.join("_") + "_" if @nested_set.any?)}#{@controller_build_folder_singular}_path"
+      "#{@namespace + "_" if @namespace}#{(@nested_set.collect { |x| x[:singular] }.join("_") + "_" if @nested_set.any?)}#{@controller_build_folder_singular}_path"
     else
-      "#{@namespace+"_" if @namespace}#{@controller_build_folder_singular}_path"
+      "#{@namespace + "_" if @namespace}#{@controller_build_folder_singular}_path"
     end
   end
 
@@ -841,7 +830,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
                                  top_level: true)
   end
 
-
   def delete_path_helper
     HotGlue.optionalized_ternary(namespace: @namespace,
                                  target: @singular,
@@ -868,15 +856,15 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def line_path_partial
-    "#{@namespace+"/" if @namespace}#{@controller_build_folder}/line"
+    "#{@namespace + "/" if @namespace}#{@controller_build_folder}/line"
   end
 
   def show_path_partial
-    "#{@namespace+"/" if @namespace}#{@controller_build_folder}/show"
+    "#{@namespace + "/" if @namespace}#{@controller_build_folder}/show"
   end
 
   def list_path_partial
-    "#{@namespace+"/" if @namespace}#{@controller_build_folder}/list"
+    "#{@namespace + "/" if @namespace}#{@controller_build_folder}/list"
   end
 
   def new_path_name
@@ -889,11 +877,11 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   def nested_assignments
     return "" if @nested_set.none?
-    @nested_set.map{|a| "#{a}: #{a}"}.join(", ") #metaprgramming into Ruby hash
+    @nested_set.map { |a| "#{a}: #{a}" }.join(", ") # metaprgramming into Ruby hash
   end
 
   def nested_assignments_top_level # this is by accessing the instance variable-- only use at top level
-    @nested_set.map{|a| "#{a[:singular]}"}.join(", ") #metaprgramming into Ruby hash
+    @nested_set.map { |a| "#{a[:singular]}" }.join(", ") # metaprgramming into Ruby hash
   end
 
   def nest_assignments_operator(top_level = false, leading_comma = false)
@@ -909,11 +897,11 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def nested_objects_arity
-    @nested_set.map{|a| "@#{a[:singular]}"}.join(", ")
+    @nested_set.map { |a| "@#{a[:singular]}" }.join(", ")
   end
 
   def nested_arity_for_path
-    [@nested_set[0..-1].collect{|a| "@#{a[:singular]}"}].join(", ") #metaprgramming into arity for the Rails path helper
+    [@nested_set[0..-1].collect { |a| "@#{a[:singular]}" }].join(", ") # metaprgramming into arity for the Rails path helper
   end
 
   def object_scope
@@ -959,7 +947,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def no_devise_installed
-    !Gem::Specification.sort_by{ |g| [g.name.downcase, g.version] }.group_by{ |g| g.name }['devise']
+    !Gem::Specification.sort_by { |g| [g.name.downcase, g.version] }.group_by { |g| g.name }['devise']
   end
 
   def magic_button_output
@@ -990,27 +978,24 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
         source_filename = cc_filename_with_extensions("#{@markup}/#{view}", "#{@markup}")
         dest_filename = cc_filename_with_extensions("#{view}", "#{@markup}")
 
-
         dest_filepath = File.join("#{filepath_prefix}app/views#{namespace_with_dash}",
                                   @controller_build_folder, dest_filename)
 
-
         template source_filename, dest_filepath
-        gsub_file dest_filepath,  '\%', '%'
+        gsub_file dest_filepath, '\%', '%'
 
       end
     end
 
     turbo_stream_views.each do |view|
       formats.each do |format|
-        source_filename = cc_filename_with_extensions( "#{@markup}/#{view}.turbo_stream.#{@markup}")
+        source_filename = cc_filename_with_extensions("#{@markup}/#{view}.turbo_stream.#{@markup}")
         dest_filename = cc_filename_with_extensions("#{view}", "turbo_stream.#{@markup}")
         dest_filepath = File.join("#{filepath_prefix}app/views#{namespace_with_dash}",
                                   @controller_build_folder, dest_filename)
 
-
         template source_filename, dest_filepath
-        gsub_file dest_filepath,  '\%', '%'
+        gsub_file dest_filepath, '\%', '%'
 
       end
     end
@@ -1023,7 +1008,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       dest_filename = cc_filename_with_extensions("#{singular_class.underscore}", "rb")
       dest_filepath = File.join("#{filepath_prefix}app/models", dest_filename)
 
-
       puts "appending turbo callbacks to #{dest_filepath}"
 
       text = File.read(dest_filepath)
@@ -1035,7 +1019,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       append_text << "  after_update_commit lambda { broadcast_replace_to self, target: \"#{@namespace}__\#{dom_id(self)}\", partial: \"#{@namespace}/#{@plural}/line\" }\n  after_destroy_commit lambda { broadcast_remove_to self, target: \"#{@namespace}__\#{dom_id(self)}\"}\n"
 
       replace = text.gsub(/class #{singular_class} < ApplicationRecord/, append_text)
-      File.open(dest_filepath, "w") {|file| file.puts replace}
+      File.open(dest_filepath, "w") { |file| file.puts replace }
     end
   end
 
@@ -1052,7 +1036,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def all_views
-    res =  %w(index  _line _list _show)
+    res = %w(index  _line _list _show)
 
     unless @no_create
       res += %w(new _new_form _new_button)
@@ -1062,7 +1046,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       res += %w{edit _edit}
     end
 
-    if !( @no_edit && @no_create)
+    if !(@no_edit && @no_create)
       res << '_form'
     end
     res
@@ -1114,7 +1098,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   def all_line_fields
     @template_builder.all_line_fields(
       col_identifier: @layout_strategy.column_classes_for_line_fields,
-      perc_width: @layout_strategy.each_col,     #undefined method `each_col'
+      perc_width: @layout_strategy.each_col, # undefined method `each_col'
       layout_strategy: @layout_strategy,
       layout_object: @layout_object
     )
@@ -1173,7 +1157,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def controller_magic_button_update_actions
-    @magic_buttons.collect{ |magic_button|
+    @magic_buttons.collect { |magic_button|
       "    if #{singular}_params[:#{magic_button}]
       begin
         res = @#{singular}.#{magic_button}!
@@ -1188,15 +1172,14 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     }.join("\n") + "\n"
   end
 
-
   def controller_update_params_tap_away_magic_buttons
-    @magic_buttons.collect{ |magic_button|
+    @magic_buttons.collect { |magic_button|
       ".tap{ |ary| ary.delete('__#{magic_button}') }"
     }.join("")
   end
 
   def controller_update_params_tap_away_alt_lookups
-    @alt_lookups.collect{ |key, data|
+    @alt_lookups.collect { |key, data|
       ".tap{ |ary| ary.delete('__lookup_#{data[:lookup_as]}') }"
     }.join("")
   end
@@ -1211,7 +1194,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   def n_plus_one_includes
     if @associations.any? || @attachments.any?
-      ".includes(" + (@associations.map{|x| x} + @attachments.collect{|k,v| "#{k}_attachment"}).map{|x| ":#{x.to_s}"}.join(", ") + ")"
+      ".includes(" + (@associations.map { |x| x } + @attachments.collect { |k, v| "#{k}_attachment" }).map { |x| ":#{x.to_s}" }.join(", ") + ")"
     else
       ""
     end
@@ -1223,7 +1206,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     if @nested_set.none?
       "\"\""
     else
-      @nested_set.collect{|arg|
+      @nested_set.collect { |arg|
         "(((\"__#{arg[:singular]}-\#{" + "@" + arg[:singular] + ".id}\") if @" + arg[:singular] + ") || \"\")"
       }.join(" + ")
     end
@@ -1235,7 +1218,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     if @nested_set.none?
       ""
     else
-      ", \n    nested_for: \"" + @nested_set.collect{|a| "#{a[:singular]}-" + '#{' + instance_symbol + a[:singular] + ".id}"}.join("__") + "\""
+      ", \n    nested_for: \"" + @nested_set.collect { |a| "#{a[:singular]}-" + '#{' + instance_symbol + a[:singular] + ".id}" }.join("__") + "\""
     end
   end
 
@@ -1246,32 +1229,31 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def hawk_to_ruby
-    res = @hawk_keys.collect{ |k,v|
+    res = @hawk_keys.collect { |k, v|
       "#{k.to_s}: [#{v[:bind_to].join(".")}]"
     }.join(", ")
     res
   end
 
   def controller_attachment_orig_filename_pickup_syntax
-    @attachments.collect{ |key, attachment|  "\n" + "    modified_params[:#{ attachment[:field_for_original_filename] }] = #{singular}_params['#{ key }'].original_filename" if attachment[:field_for_original_filename] }.compact.join("\n")
+    @attachments.collect { |key, attachment| "\n" + "    modified_params[:#{ attachment[:field_for_original_filename] }] = #{singular}_params['#{ key }'].original_filename" if attachment[:field_for_original_filename] }.compact.join("\n")
   end
 
   def any_datetime_fields?
-    (@columns - @attachments.keys.collect(&:to_sym)).collect{|col| eval("#{singular_class}.columns_hash['#{col}']").type}.include?(:datetime)
+    (@columns - @attachments.keys.collect(&:to_sym)).collect { |col| eval("#{singular_class}.columns_hash['#{col}']").type }.include?(:datetime)
   end
 
   def post_action_parental_updates
     if @nested_set.any?
-      "\n" + @nested_set.collect{ |data|
+      "\n" + @nested_set.collect { |data|
         parent = data[:singular]
         "@#{singular}.#{parent}.reload"
       }.join("\n")
     end
   end
 
-
   def turbo_parental_updates
-    @nested_set.collect{| data|
+    @nested_set.collect { |data|
       "<%= turbo_stream.replace \"#{@namespace + '__' if @namespace}\#{dom_id(@#{data[:singular]})}\" do %>
     <%= render partial: \"#{@namespace}/#{data[:plural]}/edit\", locals: {#{data[:singular]}: @#{singular}.#{data[:singular]}.reload} %>
   <% end %>"