hot-glue 0.6.17 → 0.6.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a6abeaf2b7456e5f554928c58bb5deae7fb01a0c664ad0d717572f20c6185537
-  data.tar.gz: da58696031065cf910116374477ce9f8f3b388bef0a2ad756289953b70a31d8b
+  metadata.gz: 9e7085c942cab4dc377c75180d546b57f6ca76bdb3c96f3b2a837d1fdd887b0f
+  data.tar.gz: 04fd50131e53977f25c8817c7fd7bd33e34cb0aa784c995394c94302491b1426
 SHA512:
-  metadata.gz: 626a8d361bef7d74109406ba5beae46b0daf4db297c2f95fcd92431bc1b6f9ea97d224acb8ef898bb0281b8cb3bc369d5507f6956751da137c4e5ecb4e3a93d5
-  data.tar.gz: 3a646740022a5c699051ad99c400178f1abdafc911a9c1e40c400b52a16458f3181e98ed73e0e6db8fc64fdaa8829b93f8432efcf4026873dbb2236708c2308e
+  metadata.gz: fa984faeba62c9303c5ae0c64b32143e054289380ffdfecf2f0d457c7b2c2d69912b69901d78ee6fff75f1e6997ed19e86a4cc00a8c048341fbf2505e687a984
+  data.tar.gz: 052122bbbc7d15c134653efc4da2b74df8400447eccae13d395627bd5d79b2a616f612c97ac995f07b563a8589c9609406320562616f7efe637b94d7289a5e8a

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. 
 
 
 
@@ -1401,6 +1443,16 @@ If you leave the 2nd parameter blank when using the 3rd parameter, it will defau
 
 `--attachments='avatar{thumbnail||direct}'`
 
+#### `--attachments` Long form syntax with 4 parameters
+
+The final (4th) parameter should be `dropzone` to enable dropzone support for this attachment.
+
+```
+bin/rails generate hot_glue:scaffold Video --auth='current_user' --auth-identifier='user' --namespace='account_dashboard' --nested='account' --smart-layout  --attachments='video_file{thumbnail||direct|dropzone}'
+```
+
+See below for dropzone details.
+
 
 #### For S3 Setup
 
@@ -1902,7 +1954,38 @@ These automatic pickups for partials are detected at build time. This means that
 
 # VERSION HISTORY
 
-#### 2025-05-0097 - v0.6.17
+#### 2025-06-10 v0.6.19
+
+• Fixes magic button output behavior to correctly show the string returned by the bang menthod
+• Fixes internal syntax of modify_as for modified fields; note default is now 'Yes' and 'No' for radio buttons
+• fixes destroy behavior with explicit .destroy! and ActiveRecord::RecordNotDestroyed; documents previously undocumented 4th parameter for attachment input
+
+
+#### 2025-05-18  v0.6.18
+• Significant additions to `--alt-foreign-key-lookup` which can now work:
+- on its own, without needing a factory (`--factory-creation`)
+- with a factory
+- with our without the hawk
+- in Gd mode 
+- if not in Gd mode, use with `--factory-creation` or use with the `--hawk`
+
+See notes above for details.
+
+example apps
+https://github.com/hot-glue-for-rails/HGHumanSpaMay25
+https://github.com/hot-glue-for-rails/HGAltLookups11
+
+
+• Adds Integer as searchable field
+
+_ foo and bar are integers _ 
+
+`rails g hot_glue:scaffold Thing --include=foo,bar --search=set --search-fields=foo,bar`
+
+• On show only association fields, view output is now safe-nil using `&` to not crash if the association doesn't exist.
+
+
+#### 2025-05-09 - v0.6.17
 
 
 • Adds Stimulus JS & `--stimmify` or `--stimmify=xyz`
@@ -1935,6 +2018,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? ? "_" : "") +
@@ -150,7 +148,7 @@ class AssociationField < Field
       if @stimmify
         col_target = HotGlue.to_camel_case(name.to_s.gsub("_", " "))
         data_attr = ", data: {'#{@stimmify}-target': '#{col_target}'} "
-      els
+      else
         data_attr = ""
       end
 
@@ -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/boolean_field.rb

@@ -23,9 +23,9 @@ class BooleanField < Field
 
   def radio_button_display
     "  <%= f.radio_button(:#{name}, '0', checked: #{singular}.#{name}  ? '' : 'checked', class: '#{@layout_strategy.form_checkbox_input_class}') %>\n" +
-      "  <%= f.label(:#{name}, value: '#{modify_binary? && modify_as[name.to_sym][:binary][:falsy] || 'No'}', for: '#{singular}_#{name}_0') %>\n" +
+      "  <%= f.label(:#{name}, value: '#{modify_binary? && modify_as[:binary][:falsy] || 'No'}', for: '#{singular}_#{name}_0') %>\n" +
       " <br /> <%= f.radio_button(:#{name}, '1',  checked: #{singular}.#{name}  ? 'checked' : '' , class: '#{@layout_strategy.form_checkbox_input_class}') %>\n" +
-      "  <%= f.label(:#{name}, value: '#{modify_binary? && modify_as[name.to_sym][:binary][:truthy] || 'Yes'}', for: '#{singular}_#{name}_1') %>\n"
+      "  <%= f.label(:#{name}, value: '#{modify_binary? && modify_as[:binary][:truthy] || 'Yes'}', for: '#{singular}_#{name}_1') %>\n"
   end
 
   def checkbox_display
@@ -59,9 +59,9 @@ class BooleanField < Field
       "<% if #{singular}.#{name}.nil? %>
         <span class=''>MISSING</span>
     <% elsif #{singular}.#{name} %>
-      #{modify_as[name.to_sym][:binary][:truthy]}
+      #{modify_as[:binary][:truthy]}
     <% else %>
-      #{modify_as[name.to_sym][:binary][:falsy]}
+      #{modify_as[:binary][:falsy]}
     <% end %>"
     else
       "<% if #{singular}.#{name}.nil? %>
@@ -75,11 +75,11 @@ class BooleanField < Field
   end
 
   def truthy_value
-    modify_as[name.to_sym][:binary][:truthy] || 'Yes'
+    modify_as[:binary][:truthy] || 'Yes'
   end
 
   def falsy_value
-    modify_as[name.to_sym][:binary][:falsy] || 'No'
+    modify_as[:binary][:falsy] || 'No'
   end
 
   def label_class

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(
@@ -25,7 +25,7 @@ class Field
     @form_placeholder_labels = scaffold.form_placeholder_labels
     @ownership_field = scaffold.ownership_field
     @form_labels_position = scaffold.form_labels_position
-    @modify_as = scaffold.modify_as
+    @modify_as = scaffold.modify_as[name.to_sym] # note whenever used as field, don't relookup the key
     @display_as = scaffold.display_as
     @pundit = scaffold.pundit
     @plural = scaffold.plural
@@ -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
@@ -112,7 +113,7 @@ class Field
   end
 
   def viewable_output
-    if modify_as[:modify]
+    if modify_as
       modified_display_output(show_only: true)
     else
       field_view_output
@@ -158,7 +159,6 @@ class Field
     #                end
     #   res = "<span class='badge <%= #{badge_code} %>'>" + res + "</span>"
     # end
-    # byebug
     res
   end
 
@@ -225,4 +225,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 %>
@@ -185,8 +186,13 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     flash[:alert] = nil
     <% @magic_buttons.each do |button| %>
     if <%= singular_name %>_params[:__<%= button %>]
-      @<%= singular_name %>.<%= button %>!
-      flash[:notice] << "<% singular %> <%= button.titlecase %>."
+
+      if res = @<%= singular_name %>.<%= button %>!
+        flash[:notice] << "<% singular %> <%= button.titlecase %>"
+        flash[:notice] << " #{res}" if res.is_a?(String)
+      else
+        flash[:alert] = " <%= button.titlecase %> failed."
+      end
     end
     <% end %>
 
@@ -195,8 +201,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 +239,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 %>
@@ -236,9 +259,9 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     skip_authorization
     raise Pundit::NotAuthorizedError if ! <%= @pundit_policy_override %>.destroy?<% end %>
     begin
-      @<%=singular_name%>.destroy
+      @<%=singular_name%>.destroy!
       flash[:notice] = '<%= singular_name.titlecase %> successfully deleted'
-    rescue ActiveRecordError => e
+    rescue ActiveRecord::RecordNotDestroyed => e
       flash[:alert] = '<%= singular_name.titlecase %> could not be deleted'
     end
     <%= post_action_parental_updates.join("\n    ") %>
@@ -261,12 +284,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.19'
   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.19
 platform: ruby
 authors:
 - Jason Fleetwood-Boldt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-08 00:00:00.000000000 Z
+date: 2025-06-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails