hot-glue 0.6.17 → 0.6.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a6abeaf2b7456e5f554928c58bb5deae7fb01a0c664ad0d717572f20c6185537
-  data.tar.gz: da58696031065cf910116374477ce9f8f3b388bef0a2ad756289953b70a31d8b
+  metadata.gz: ddd7d67de32f74ce4e79fabcaaa216e9e5cfbbd84c4bb6d9ab53b6ba2dd57f00
+  data.tar.gz: 5e4e51cce50f7c7e3579bb09c452c5c0b02b4331870a22c9bd249281dab8cd7a
 SHA512:
-  metadata.gz: 626a8d361bef7d74109406ba5beae46b0daf4db297c2f95fcd92431bc1b6f9ea97d224acb8ef898bb0281b8cb3bc369d5507f6956751da137c4e5ecb4e3a93d5
-  data.tar.gz: 3a646740022a5c699051ad99c400178f1abdafc911a9c1e40c400b52a16458f3181e98ed73e0e6db8fc64fdaa8829b93f8432efcf4026873dbb2236708c2308e
+  metadata.gz: e250a3b6598e3e83312bad3b27d7da0f200563ab7e89a544f23aa46913eb1f23ea796854dcdf7d2939291ec8b8ce92fe245c629658982b4f47a71e20a19e1c5f
+  data.tar.gz: '08b99eba601f426decbcd5e2fe5daf6c9b121c1285e847d2f7b827da14f1c16a30bb20e2813770a68b20a623fe85584e1e9fe6784c86f42eb5d97c375715421d'

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    hot-glue (0.6.17)
+    hot-glue (0.6.18)
       ffaker (~> 2.16)
       kaminari (~> 1.2)
       rails (> 5.1)

data/README.md

@@ -680,15 +680,49 @@ For the `$` modifier only, if the field name ends with `_cents`, the modifier wi
 
 ### `--alt-foreign-key-lookup=`
 
-Use for a join table to specify that a field should be looked up by a different field
+Use for a join table to specify that a field should be looked up by a different field. For example, when you want to lookup a user by a (complete) email address.
 
+In these contexts, the lookup must be exact match to the text entered (no partial matches supported). 
 
-`./bin/rails generate hot_glue:scaffold AccountUser --alt-foreign-key-lookup=user_id{email}`
+Use `+` to map to the `find_or_create_by` method. (Without `+` it will use `find_by`.)
+
+This is accomlished using a little magic of a lookup field called `__lookup_X_Y` passed in the form parameters.
+
+The lookup field is used to look up the associated record, then deleted from the params.
+
+#### When used in `--gd` mode
+The lookup will use the base class and have access to all records.
+`user = User.find_or_create_by!(email: params[:__lookup_email])`
+`modified_params.tap { |hs| hs.delete(:__lookup_target_email)}`
+`modified_params.merge!(user: user)`
+
+Example 1
+`./bin/rails generate hot_glue:scaffold AccountUser --gd --alt-foreign-key-lookup=user_id{email}`
 
 Here we are specifying that the `user_id` field should be looked up by the `email` field on the User table. 
 If no existing user exists, we create one because we are using the `find_or_create_by!` method.
 
+### When used with a Hawk
+
+A hawk applied to the same field will be enforced within this mechanism.
+
+Example #2
+`bin/rails generate hot_glue:scaffold Appointment --gd --alt-foreign-key-lookup='user_id{email}' --hawk='user_id{current_user.family}'`
+
+
+Whether or not in Gd mode, the hawk is  enforced by scoping the find to the hawk's scope
+```
+user = current_user.family.users.find_by(email: appointment_params[:__lookup_user_email] )
+modified_params.tap { |hs| hs.delete(:__lookup_user_email)}
+@appointment = Appointment.new(modified_params.merge(user: user))
+```
+
+
+
+### With Factory
+
 Use with a factory pattern like this one:
+
 ``` 
 class AccountUserFactory
   attr_accessor :account_user
@@ -712,9 +746,17 @@ end
 this works with a factory creation syntax like so:
 
 ```
---factory-creation='factory = AccountUserFactory.new(params: account_user_params, account: account)
+rails generate hot_glue:scaffold AccountUser --alt-foreign-key-lookup=user_id{email} --factory-creation='factory = AccountUserFactory.new(params: account_user_params, account: account)
 ```
-*See the `--factory-creation` section. 
+
+In this example, the lookup is *not performed inside of the create action*, because it is assumed you will do so yourself inside of your factory.
+
+As you see in the example above, params is passed to your factory and it is the `account_user_params` fromm the controller. 
+
+#### Warning:
+When building a non-Gd controller with a `--alt-foreign-key-lookup`, if you don't also hawk the same field you will get an error. 
+
+To fix, either hawk the field or use with a factory creation pattern. This is because the associated object is on your graph but Hot Glue doesn't know how to securly load it without knowing its relationship to the current user. Use the `--hawk` mechanism to specify that relationship, and the lookup mechanism will integrate nicely. 
 
 
 
@@ -1902,7 +1944,24 @@ These automatic pickups for partials are detected at build time. This means that
 
 # VERSION HISTORY
 
-#### 2025-05-0097 - v0.6.17
+#### 2025-05-18  v0.6.18
+• Significant additions to `--alt-foreign-key-lookups` which can now work:
+- on its on, without needing a factory
+- with a factory
+- with our without the hawk
+- in Gd mode or, if not, use with `--factory-creation` or use with `--hawk`
+
+See notes above for details.
+
+• Adds Integer as serachable field
+
+_ foo and bar are integers _ 
+
+`rails g hot_glue:scaffold Thing --include=foo,bar --search=set --search-fields=foo,bar`
+
+
+
+#### 2025-05-09 - v0.6.17
 
 
 • Adds Stimulus JS & `--stimmify` or `--stimmify=xyz`
@@ -1935,6 +1994,7 @@ https://jasonfleetwoodboldt.com/courses/rails-7-crash-course/rails-7-stimulus-js
 • Adds `--hidden` option
 Pass a list of fields, like include or show-only. This will make the field hidden on the form *but still updated via its submission*
 
+• Show only fields associations are now safe-nil using `&` to not crash if the association doesn't exist
 
 
 

data/app/helpers/hot_glue/controller_helper.rb

@@ -143,6 +143,27 @@ module HotGlue
       end
     end
 
+    def integer_query_constructor(match, search)
+      if match.blank? || search.blank?
+        nil
+      else
+        case match
+        when '='
+          "= #{search.to_i}"
+        when '≥'
+          ">= #{search.to_i}"
+        when '>'
+          "> #{search.to_i}"
+        when '≤'
+          "<= #{search.to_i}"
+        when '<'
+          "< #{search.to_i}"
+        else
+          nil
+        end
+      end
+    end
+
     def enum_constructor(field_name, value, **args)
       return nil if value.blank?
       ["#{field_name} = ?", value]

data/lib/generators/hot_glue/fields/association_field.rb

@@ -82,25 +82,23 @@ class AssociationField < Field
     display_column = HotGlue.derrive_reference_name(assoc_class_name)
 
 
-    "<%= #{singular}.#{assoc_name}.#{display_column} %>"
+    "<%= #{singular}.#{assoc_name}&.#{display_column} %>"
   end
 
   def form_field_output
     assoc_name = name.to_s.gsub("_id","")
+
+
     assoc = eval("#{class_name}.reflect_on_association(:#{assoc_name})")
-    if alt_lookup.keys.include?(name.to_sym)
-      alt = alt_lookup[:lookup_as]
+
+    if alt_lookup.keys.include?(name)
       assoc_name = name.to_s.gsub("_id","")
       assoc = eval("#{class_name}.reflect_on_association(:#{assoc_name})")
 
-      alt = alt_lookup[:lookup_as]
+      lookup_field = alt_lookup[name][:lookup_as]
+      assoc = alt_lookup[name][:assoc].downcase
       parts = name.split('_')
-      "<%= f.text_field :__lookup_#{alt}, value: @#{singular}.#{assoc_name}.try(:#{alt}), placeholder: \"search by #{alt}\" " + (stimmify ? ", 'data-#{@stimmify}-target': '#{camelcase_name}' " : "")  + "%>"
-
-      # if modify_as
-      #   modified_display_output
-      # else
-      # end
+      "<%= f.text_field :__lookup_#{assoc}_#{lookup_field}, value: @#{singular}.#{assoc_name}&.#{lookup_field}, placeholder: \"#{lookup_field}\" " + (stimmify ? ", 'data-#{@stimmify}-target': '#{camelcase_name}' " : "")  + "%>"
     elsif modify_as && modify_as[:typeahead]
       search_url  = "#{namespace ? namespace + "_" : ""}" +
         modify_as[:nested].join("_") + ( modify_as[:nested].any? ? "_" : "") +
@@ -188,11 +186,9 @@ class AssociationField < Field
 
     if assoc_class
       display_column =  HotGlue.derrive_reference_name(assoc_class.to_s)
-
       "<%= #{singular}.#{assoc}.try(:#{display_column}) || '<span class=\"content \">MISSING</span>'.html_safe %>"
     else
       "<%= #{singular}.#{assoc}.try(:to_label) || '<span class=\"content \">MISSING</span>'.html_safe %>"
-
     end
 
   end
@@ -274,5 +270,33 @@ class AssociationField < Field
     end
   end
 
+  def prelookup_syntax
+    field =  @alt_lookup[name.to_s]
+    if field[:with_create]
+
+      method_name = "find_or_create_by"
 
+    else
+      method_name = "find_by"
+    end
+    field_name = field[:assoc].downcase.gsub("_id","")
+    assoc_class = field[:assoc].classify
+
+    assoc = plural
+
+    ## TODO: add the hawk here
+    res = +""
+    if @hawk_keys[name.to_sym]
+      res << "#{field_name} = #{@hawk_keys[name.to_sym][:bind_to].first}.#{method_name}(#{field[:lookup_as]}: #{singular}_params[:__lookup_#{field[:assoc].downcase}_#{field[:lookup_as]}] )"
+    elsif @god
+      assoc_name = field[:assoc]
+      res << "#{field_name} = #{assoc_class}.#{method_name}(#{field[:lookup_as]}: #{singular}_params[:__lookup_#{field[:assoc].downcase}_#{field[:lookup_as]}] )"
+    else
+      raise "Field #{field_name} is an alt lookup in a non-Gd context which is a security vulnerability"
+    end
+
+    res << "\n    modified_params.tap { |hs| hs.delete(:__lookup_#{field[:assoc].downcase}_#{field[:lookup_as]})}"
+    return res
+
+  end
 end

data/lib/generators/hot_glue/fields/field.rb

@@ -6,7 +6,7 @@ class Field
                 :self_auth,
                 :singular_class,  :singular, :sql_type, :ownership_field,
                 :update_show_only, :namespace, :pundit, :plural,
-                :stimmify, :hidden, :attachment_data
+                :stimmify, :hidden, :attachment_data, :god
 
 
   def initialize(
@@ -35,6 +35,7 @@ class Field
     @stimmify = scaffold.stimmify
     @hidden = scaffold.hidden
     @attachment_data = scaffold.attachments[name.to_sym]
+    @god = scaffold.god
 
 
     # TODO: remove knowledge of subclasses from Field
@@ -225,4 +226,6 @@ class Field
     false
   end
 
+  def prelookup_syntax; nil; end
+
 end

data/lib/generators/hot_glue/fields/integer_field.rb

@@ -29,6 +29,26 @@ class IntegerField < Field
   end
 
   def search_field_output
-    raise "Integer search not implemented"
+    "      <div>" +
+    "\n        <%= f.select 'q[0][#{name}_match]', options_for_select([['', ''], ['=', '='], " +
+    "\n         ['≥', '≥'], ['>', '>'], " +
+    "\n         ['≤', '≤'], ['<', '<']], @q[\'0\']['#{name}_match'] ), {} ," +
+    "\n         { class: 'form-control match' } %>"+
+    "\n        <%= f.text_field  'q[0][#{name}_search]',  {value: @q[\'0\'][:#{name}_search], autocomplete: 'off', size: 4, class: 'form-control', type: 'number'} %>" +
+    "\n      </div>"
   end
+
+
+  def where_query_statement
+    ".where(\"#{name} \#{#{name}_query }\")"
+  end
+
+  def load_all_query_statement
+    "#{name}_query = integer_query_constructor(@q['0'][:#{name}_match], @q['0'][:#{name}_search])"
+  end
+
+  def code_to_reset_match_if_search_is_blank
+    "    @q['0'][:#{name}_match] = '' if @q['0'][:#{name}_search] == ''"
+  end
+
 end

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -20,7 +20,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
                 :big_edit, :button_icons, :bootstrap_column_width,
                 :columns,
                 :default_boolean_display,
-                :display_as, :downnest_children, :downnest_object, :hawk_keys, :layout_object,
+                :display_as, :downnest_children, :downnest_object, :god,  :hawk_keys, :layout_object,
                 :modify_as,
                 :nest_with, :path, :plural, :sample_file_path, :show_only_data, :singular,
                 :singular_class, :smart_layout, :stacked_downnesting,
@@ -84,7 +84,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   # determines if labels appear within the rows of the VIEWABLE list (does NOT affect the list heading)
   class_option :inline_list_labels, default: nil # default is set below
-  class_option :factory_creation, default: ''
+  class_option :factory_creation, default: nil
   class_option :alt_foreign_key_lookup, default: '' #
   class_option :attachments, default: ''
   class_option :stacked_downnesting, default: false
@@ -437,7 +437,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @related_sets = {}
     related_set_input.each do |setting|
       name = setting.to_sym
-      byebug
       association_ids_method = eval("#{singular_class}.reflect_on_association(:#{setting.to_sym})").class_name.underscore + "_ids"
       class_name = eval("#{singular_class}.reflect_on_association(:#{setting.to_sym})").class_name
 
@@ -480,7 +479,10 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       end
     end
 
-    @factory_creation = options['factory_creation'].gsub(";", "\n")
+    unless options['factory_creation'].nil?
+      @factory_creation = options['factory_creation'].gsub(";", "\n")
+    end
+
     identify_object_owner
     setup_fields
 
@@ -506,6 +508,9 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       setting =~ /(.*){(.*)}/
       key, lookup_as = $1, $2
 
+      if !eval("#{class_name}.reflect_on_association(:#{key.to_s.gsub("_id","")})")
+        raise "couldn't find association for #{key} in the object #{class_name}"
+      end
       assoc = eval("#{class_name}.reflect_on_association(:#{key.to_s.gsub("_id","")}).class_name")
 
       data = {lookup_as: lookup_as.gsub("+",""),
@@ -514,7 +519,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @alt_lookups[key] = data
     end
 
-    puts "------ ALT LOOKUPS for #{@alt_lookups}"
+
 
     # @update_alt_lookups = @alt_lookups.collect{|key, value|
     #   @update_show_only.include?(key) ?
@@ -578,6 +583,18 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       end
     end
 
+
+    puts "------ ALT LOOKUPS for #{@alt_lookups}"
+    @alt_lookups.each do |key, value|
+      if !@columns_map[key.to_sym].is_a?(AssociationField)
+        raise "You specified an alt-lookup for #{key} but that field is not an association field"
+      elsif !@columns_map[key.to_sym]
+        raise "You specified an alt-lookup for #{key} but that field does not exist in the list of columns"
+      elsif !@god && !@hawk_keys.include?(key.to_sym)
+        raise "You specified an alt-lookup for #{key} in non-Gd mode but this would leave the lookup unprotected. To fix, use with --hawk or with --factory-creation "
+      end
+    end
+
     # search
     @search = options['search']
     if @search == 'set'
@@ -677,7 +694,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     if options["hawk"]
       options['hawk'].split(",").each do |hawk_entry|
         # format is: abc_id[thing]
-
         if hawk_entry.include?("{")
           hawk_entry =~ /(.*){(.*)}/
           key, hawk_to = $1, $2
@@ -687,6 +703,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
         end
 
         hawk_scope = key.gsub("_id", "").pluralize
+
         if eval(singular_class + ".reflect_on_association(:#{key.gsub('_id', '')})").nil?
           raise "Could not find `#{key.gsub('_id', '')}` association; add this to the #{singular_class} class: \nbelongs_to :#{key.gsub('_id', '')} "
         end
@@ -836,6 +853,8 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
                            :confirmation_token, :confirmed_at,
                            :confirmation_sent_at, :unconfirmed_email
 
+
+      # TODO: this should exclude any nested parents
       @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) }
@@ -844,7 +863,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       @columns = @the_object.columns.map(&:name).map(&:to_sym).reject { |field| !@include_fields.include?(field) }
     end
 
-    @columns = @columns -@nested_set.collect { |set| (set[:singular] + "_id").to_sym  }
+    @columns = @columns - @nested_set.collect { |set| (set[:singular] + "_id").to_sym  }
 
     if @attachments.any?
       puts "Adding attachments-as-columns: #{@attachments}"
@@ -888,8 +907,21 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def creation_syntax
-    if @factory_creation == ''
-      "@#{singular } = #{ class_name }.new(modified_params)"
+    if @factory_creation.nil? && ! @alt_lookups.any?
+      (      @hawk_keys.any? ?   "modified_params = hawk_params({#{ hawk_to_ruby }}, modified_params)\n    " : "")  + "@#{singular } = #{ class_name }.new(modified_params)"
+    elsif @factory_creation.nil? && @alt_lookups.any?
+
+      prelookup_syntax = @alt_lookups.collect{|lookup, data|
+        col = @columns_map[lookup.to_sym]
+        col.prelookup_syntax
+      }.join("\n")
+
+      prelookup_syntax + "\n     @#{singular } = #{ class_name }.new(modified_params" +
+        (@alt_lookups.any? ? (".merge(" + @alt_lookups.collect{|lookup,field|
+          field_name = lookup.gsub("_id","")
+          "#{field_name}: #{field_name}"
+        }.join(",") + ")" ) : "") + ")"
+
     else
       res = +"begin
       #{@factory_creation}
@@ -954,10 +986,11 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
       template "system_spec.rb.erb", dest_file
     end
-
-    if File.exist?("#{filepath_prefix}app/views#{namespace_with_dash}/_errors.#{@markup}")
-      File.delete("#{filepath_prefix}app/views#{namespace_with_dash}/_errors.#{@markup}")
-    end
+    # if !File.exist?("#{filepath_prefix}app/views#{namespace_with_dash}/_errors.#{@markup}")
+    #   # File.delete("#{filepath_prefix}app/views#{namespace_with_dash}/_errors.#{@markup}")
+    #
+    #   template "_errors.erb", File.join("#{filepath_prefix}app/views#{namespace_with_dash}", "_errors.#{@markup}")
+    # end
   end
 
   def spec_foreign_association_merge_hash

data/lib/generators/hot_glue/templates/controller.rb.erb

@@ -27,10 +27,12 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     def <%= @nested_set[0][:singular] %><% if @god
        next_object = nil
        collect_objects = @nested_set.reverse.collect {|x|
-           if eval("#{next_object || class_name}.reflect_on_association(:#{x[:singular]})").nil?
-             raise "***** Unable to find the association `#{x[:singular]}` on the class #{next_object || class_name} ..... you probably want to add `belongs_to :#{x}` to the #{next_object || class_name} object?"
+           if eval("#{next_object || class_name}.reflect_on_association(:#{x[:singular]})").nil? #&& eval("! #{next_object || class_name}.instance_methods.include?(:#{x[:singular]})")
+             raise "***** Unable to find the association `#{x[:singular]}` on the class #{next_object || class_name} ..... you probably want to add `belongs_to :#{x[:singular]}` to the #{next_object || class_name} object?"
            end
+           # if  eval("#{next_object || class_name}.reflect_on_association(:#{x[:singular]})")
            next_object = eval("#{next_object || class_name}.reflect_on_association(:#{x[:singular]})").class_name
+           # end
        }
        root_object = collect_objects.last
      else
@@ -110,11 +112,10 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     modified_params = modified_params.merge(<%= @object_owner_sym %>: <%= @object_owner_eval %>) <% elsif @object_owner_optional && any_nested? %>
     modified_params = modified_params.merge(<%= @object_owner_name %> ? {<%= @object_owner_sym %>: <%= @object_owner_eval %>} : {}) <% end %>
 
-      <% if @hawk_keys.any? %>
-    modified_params = hawk_params({<%= hawk_to_ruby %>}, modified_params)<% end %>
     <%= controller_attachment_orig_filename_pickup_syntax %>
     <%= creation_syntax %>
-    <% if @pundit %><% @related_sets.each do |key, related_set| %>
+
+      <% if @pundit %><% @related_sets.each do |key, related_set| %>
     check_<%= related_set[:association_ids_method].to_s %>_permissions(modified_params, :create)<% end %><% end %>
       <% if @pundit && !@pundit_policy_override %>
     authorize @<%= singular %><% elsif @pundit && @pundit_policy_override %>
@@ -195,8 +196,20 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     modified_params = modified_params.merge(<%= @object_owner_name %> ? {<%= @object_owner_sym %>: <%= @object_owner_eval %>} : {}) <% end %>
     <% if @pundit %><% @related_sets.each do |key, related_set| %>
     check_<%= related_set[:association_ids_method].to_s %>_permissions(modified_params, :update)<% end %><% end %>
+      <% if (@alt_lookups.any?) %>
+    <%=  @alt_lookups.collect{|lookup, data|
+       @columns_map[lookup.to_sym].prelookup_syntax unless @update_show_only.include?(lookup.to_sym)
+    }.join("\n") %>
+      <% elsif @factory_creation %>
 
-    <% if @hawk_keys.any? %>    modified_params = hawk_params({<%= hawk_to_ruby %>}, modified_params)<% end %>
+      <% end %>
+      <% if (@alt_lookups.keys.collect(&:to_sym) - @update_show_only).any? %>
+    modified_params.merge!(<%=  @alt_lookups.collect{|lookup,field|
+    field_name = lookup.gsub("_id","")
+      "#{field_name}: #{field_name}" unless @update_show_only.include?(lookup.to_sym)
+    }.join(",") %>)
+    <% end %>
+   <% if @hawk_keys.any? %> modified_params = hawk_params({<%= hawk_to_ruby %>}, modified_params)<% end %>
     <%= controller_attachment_orig_filename_pickup_syntax %>
       <% if @pundit && !@pundit_policy_override %>
       authorize @<%= singular_name %>
@@ -221,6 +234,11 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
      <% end %>
     else
       flash[:alert] = "<%= singular_name.titlecase %> could not be saved. #{@hawk_alarm}"
+      <%= @alt_lookups.collect{ |k,v|
+      assoc = k.gsub("_id","")
+      "@#{singular }.#{k} = #{class_name}.find(@#{singular }.id).person.id if @#{singular }.errors.include?(:#{assoc})"
+
+      }.join("\n") %>
       @action = 'edit'
       <% unless @big_edit %>render :update<% else %>render :edit<% end %>, status: :unprocessable_entity
     end<% if @pundit %>
@@ -261,12 +279,12 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
   end<% end %><% end %>
 
   def <%=singular_name%>_params
-    params.require(:<%= testing_name %>).permit(<%= ((fields_filtered_for_strong_params - @show_only ) + @magic_buttons.collect{|x| "__#{x}"}).collect{|sym| ":#{sym}"}.join(", ") %><%= ", " + @related_sets.collect{|key, rs| "#{rs[:association_ids_method]}: []"}.join(", ") if @related_sets.any? %><%= ", " +  @alt_lookups.collect{|k,v|  ":__lookup_#{v[:lookup_as]}"  }.join(", ") if @alt_lookups.any? %>)
+    params.require(:<%= testing_name %>).permit(<%= ((fields_filtered_for_strong_params - @show_only ) + @magic_buttons.collect{|x| "__#{x}"}).collect{|sym| ":#{sym}"}.join(", ") %><%= ", " + @related_sets.collect{|key, rs| "#{rs[:association_ids_method]}: []"}.join(", ") if @related_sets.any? %><%= ", " +  @alt_lookups.collect{|k,v|  ":__lookup_#{v[:assoc].downcase}_#{v[:lookup_as]}"  }.join(", ") if @alt_lookups.any? %>)
   end<% if @update_show_only %>
 
   <% unless @no_edit %>
   def update_<%=singular_name%>_params
-    params.require(:<%= testing_name %>).permit(<%= ((fields_filtered_for_strong_params - @update_show_only)  + @magic_buttons.collect{|x| "__#{x}"}).collect{|sym| ":#{sym}"}.join(", ") %><%= ", " + @related_sets.collect{|key, rs| "#{rs[:association_ids_method]}: []"}.join(", ") if @related_sets.any? %>)
+    params.require(:<%= testing_name %>).permit(<%= ((fields_filtered_for_strong_params - @update_show_only)  + @magic_buttons.collect{|x| "__#{x}"}).collect{|sym| ":#{sym}"}.join(", ") %><%= ", " + @related_sets.collect{|key, rs| "#{rs[:association_ids_method]}: []"}.join(", ") if @related_sets.any? %><%=  ", " + @alt_lookups.collect{|k,v|  ":__lookup_#{v[:assoc].downcase}_#{v[:lookup_as]}"  }.join(", ") if @alt_lookups.any?  %>)
   end<% end %>
   <% end %>
 

data/lib/hotglue/version.rb

@@ -1,5 +1,5 @@
 module HotGlue
   class Version
-    CURRENT = '0.6.17'
+    CURRENT = '0.6.18'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hot-glue
 version: !ruby/object:Gem::Version
-  version: 0.6.17
+  version: 0.6.18
 platform: ruby
 authors:
 - Jason Fleetwood-Boldt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-08 00:00:00.000000000 Z
+date: 2025-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails