hot-glue 0.6.8 → 0.6.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 86900d536cda31c5759bd3603aff4e32a7c141bd015f4c3910c1ac2fbd67c083
-  data.tar.gz: 4dfeabc8f4a4f65a626cf6510b11a546d6cf860735a6cb582b6811ac25bdb1a2
+  metadata.gz: 76c514956d7ff1717545fdb8f6d548e2cfe05b270b5309c91e6b58cc12a42700
+  data.tar.gz: 2cfc8788264527a2eb559054aad0069345c735c24bbc1a195b6a8174facc08da
 SHA512:
-  metadata.gz: 40f77d7d6552441122511e1fbd144ff9090fac458840333d9d4062c443466e9fe12932fb2748e40a9799562b71fa028b53910f34da77eae281bb18b8055e0cd5
-  data.tar.gz: 281870175e5d080e79f0afc9d190e5f6047660cea7cc16470fe3feec2cdd18da67ef99e0d686fa2c9378d45a77cf4b0b9a041635df3aeee57c84796a972f69fe
+  metadata.gz: 0cf0fd5f2a72c21f99fb79873e35e2c8ec62f6a3cd3fe24b879d6997d3b3d994ceb0482cca5c114e9fd2607e2c12c367d6611da493b68f19ac976c77ecd9c71f
+  data.tar.gz: 1f70c724db1fbdee83d0825b2aa06150b5de5edec9041c82342baef973a93050c0229c449eee4082556d47839a063eb5d52dd038da36430a47a1f2499a0cb809

data/README.md

@@ -914,7 +914,7 @@ This means you can be a somewhat lazy about your bang methods, but keep in mind
 
 Finally, you can raise an ActiveRecord error which will also get passed to the user in the flash alert area.
 
-For more information see [Example 5 in the Tutorial](https://school.jfbcodes.com/8188)
+For more information see [Example 6 in the Tutorial](https://school.jfbcodes.com/8188)
 
 
 ### `--downnest`
@@ -934,6 +934,24 @@ The 'Abcs' portal will display as 5 bootstrap columns instead of the typical 4.
 This puts the downnested portals on top of one another (stacked top to bottom) instead of side-by-side (left to right). This is useful if you have a lot of downnested portals and you want to keep the page from getting too wide.
 
 
+
+### `--record-scope=`
+
+Record scope allows you to apply a model based scope for the controller being generated.
+This is applied on top of all other scopes, searches, and modifiers applied to the built controller.
+
+`bin/rails :generate hot_glue:scaffold Order --record-scope='.is_open'`
+
+Be sure to use single quotes (`'`) and don't forget the dot (`.`) before your scope(s).
+
+Make sure your Order model has a scope `is_open`, like so:
+
+```
+scope :is_open, -> {where(state == 'open')}
+```
+
+Now all records displayed through the generated controller 
+
 ## FLAGS (Options with no values)
 These options (flags) also uses `--` syntax but do not take any values. Everything is assumed (default) to be false unless specified.
 
@@ -999,10 +1017,17 @@ Omits list views.
 
 ### `--big-edit`
 
-If you do not want inline editing of your list items but instead want to fall back to full page style behavior for your edit views, use `--big-edit`. Turbo still handles the page interactions, but the user is taken to a full-screen edit page instead of an edit-in-place interaction.
+If you do not want inline editing of your list items but instead want to fallback to full-page style behavior for your edit views, use `--big-edit`. 
+
+The user will be taken to a full-screen edit page instead of an edit-in-place interaction.
 
 When using `--big-edit`, any downnested portals will be displayed on the edit page instead of on the list page. 
 
+Big edit makes all edit and magic button operations happen using `'data-turbo': false`, fully reloading the page and submitting HTML requests instead of TURBO_STREAM requests. 
+
+Likewise, the controller's `update` action always redirects instead of using Turbo.
+
+
 ### `--display-list-after-update`
 
 After an update-in-place normally only the edit view is swapped out for the show view of the record you just edited.
@@ -1662,6 +1687,26 @@ These automatic pickups for partials are detected at buildtime. This means that
 
 # VERSION HISTORY
 
+#### 2024-12-14 v0.6.9
+
+• `--record-scope`
+Record scope allows you to apply a model based scope for the controller being generated.
+This is applied on top of all other scopes, searches, and modifiers applied to the built controller.
+
+`bin/rails :generate hot_glue:scaffold Order --record-scope='.is_open'`
+
+(You can chain multiple scopes.)
+
+Be sure to use single quote marks (`'`) and don't forget the dot (`.`) before your scope(s).
+
+Make sure your Order model has a scope `is_open`
+
+• `--big-edit` now always uses non-turbo form submits, for update + magic buttons
+• refactor of modified datetime feature to prefer current user as set by the --auth setting (will not work in @gd mode). future implemenation will further refine
+• when using big edit, `update.turbo_stream.erb` is no longer written
+• removes Pundit policy_scope() around new operations
+• refactors to how parent objects from a nested controller pass these variables to lower-level partials; this implementation hard-cards the nested set as locals and also builds a `nested_for` key (string)
+
 
 #### 2024-12-05 - v0.6.8
 • fixes in modify_date_inputs_on_params for current_user_object

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

@@ -40,7 +40,7 @@ class DateTimeField < Field
   end
 
   def form_field_output
-    "<%= datetime_field_localized(f, :#{name}, #{singular}.#{name}, label: '#{ name.to_s.humanize }' ) %>"
+    "<%= f.datetime_field( :#{name}, value: #{singular}.#{name} && #{singular}.#{name}.change(sec: 0), class: '#{@layout_strategy.form_input_class}'  ) %>"
   end
 
   def viewable_output

data/lib/generators/hot_glue/layout_strategy/base.rb

@@ -38,6 +38,10 @@ module LayoutStrategy
       ""
     end
 
+    def form_input_class
+      ""
+    end
+
     def search_closing
       ""
     end

data/lib/generators/hot_glue/layout_strategy/bootstrap.rb

@@ -69,6 +69,10 @@ class LayoutStrategy::Bootstrap < LayoutStrategy::Base
     "form-check-label"
   end
 
+  def form_input_class
+    "form-control"
+  end
+
   def search_opening
     '<div class="row"><div class="col-md-12 card"><div class="card-body">'
   end

data/lib/generators/hot_glue/markup_templates/erb.rb

@@ -62,9 +62,14 @@ module  HotGlue
       text.lines.collect{|line| add_spaces + line}.join("")
     end
 
-    def magic_button_output(path:, singular:, magic_buttons:, small_buttons: )
+    def magic_button_output(path:,
+                            singular:,
+                            big_edit: ,
+                            magic_buttons:, small_buttons: )
       magic_buttons.collect{ |button_name|
-        "<%= form_with model: #{singular}, url: #{path}, html: {style: 'display: inline', data: {\"turbo-confirm\": 'Are you sure you want to #{button_name} this #{singular}?'}} do |f| %>" +
+        "<%= form_with model: #{singular}, url: #{path}, html: {style: 'display: inline', data: {\"turbo-confirm\": 'Are you sure you want to #{button_name} this #{singular}?'  " +
+          (big_edit ? ", \"turbo\": false" : "") +
+           "}} do |f| %>" +
           "<%= f.hidden_field :__#{button_name}, value: \"__#{button_name}\" %>" +
           "<%= f.submit '#{button_name.titleize}'.html_safe, disabled: (#{singular}.respond_to?(:#{button_name}able?) && ! #{singular}.#{button_name}able? ), class: '#{singular}-button #{@layout_strategy.button_applied_classes} #{@layout_strategy.magic_button_classes}' %>" +
         "<% end %>"

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -23,9 +23,12 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
                 :display_as, :downnest_children, :downnest_object, :hawk_keys, :layout_object,
                 :modify_as,
                 :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, :pundit,
-                :self_auth, :namespace_value, :related_sets, :search_clear_button, :search_autosearch
+                :singular_class, :smart_layout, :stacked_downnesting,
+                :update_show_only, :ownership_field,
+                :layout_strategy, :form_placeholder_labels,
+                :form_labels_position, :pundit,
+                :self_auth, :namespace_value, :record_scope, :related_sets,
+                :search_clear_button, :search_autosearch
   # important: using an attr_accessor called :namespace indirectly causes a conflict with Rails class_name method
   # so we use namespace_value instead
 
@@ -97,6 +100,8 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   class_option :code_after_create, default: nil
   class_option :code_before_update, default: nil
   class_option :code_after_update, default: nil
+  class_option :record_scope, default: nil
+
 
   class_option :search, default: nil # set or predicate
 
@@ -335,6 +340,7 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     @new_in_modal = options['new_in_modal'] || false
 
     @smart_layout = options['smart_layout']
+    @record_scope = options['record_scope']
 
     @pundit = options['pundit']
     if @pundit.nil?
@@ -1195,10 +1201,16 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
   end
 
   def current_user_object
-    default_current_user = options['auth'] || "current_user"
-    if eval("defined?(#{default_current_user})")
-      default_current_user
-    else
+
+    # TODO: in god mode, there's no way for the time input
+    # to know who the current user under this design
+    # for timeinput = user_centered , we need to know who the current user is
+    # so we can set the time zone to the user's time zone
+    #
+    if options['auth']
+      options['auth']
+    elsif @god
+      # do we use current_user here
       "nil"
     end
   end
@@ -1209,14 +1221,15 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
   def magic_button_output
     @template_builder.magic_button_output(
-      path: HotGlue.optionalized_ternary(namespace: @namespace,
-                                         target: @singular,
-                                         nested_set: @nested_set,
-                                         with_params: true,
-                                         put_form: true),
-      singular: singular,
-      magic_buttons: @magic_buttons,
-      small_buttons: @small_buttons
+      path: HotGlue.optionalized_ternary( namespace: @namespace,
+                                          target: @singular,
+                                          nested_set: @nested_set,
+                                          with_params: true,
+                                          put_form: true),
+                                          big_edit: @big_edit,
+                                          singular: singular,
+                                          magic_buttons: @magic_buttons,
+                                          small_buttons: @small_buttons
     )
   end
 
@@ -1245,7 +1258,6 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
       formats.each do |format|
         source_filename = cc_filename_with_extensions("#{@markup}/#{view}", "#{@markup}")
         dest_filename = cc_filename_with_extensions("#{view}", "#{@markup}")
-        # byebug
         dest_filepath = File.join("#{filepath_prefix}app/views#{namespace_with_dash}",
                                   @controller_build_folder, dest_filename)
 
@@ -1361,7 +1373,10 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
 
     unless @no_edit
       res << 'edit'
-      res << 'update'
+
+      unless @big_edit
+        res << 'update'
+      end
     end
 
     res
@@ -1527,10 +1542,12 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
     end
 
     if pundit
-      res << "    @#{ plural_name } = policy_scope(#{ object_scope }).page(params[:page])#{ n_plus_one_includes }#{ ".per(per)" if @paginate_per_page_selector }"
+      res << "    @#{ plural_name } = policy_scope(#{ object_scope })#{record_scope}.page(params[:page])#{ n_plus_one_includes }#{ ".per(per)" if @paginate_per_page_selector }"
     else
       if !@self_auth
-        res << spaces(4) + "@#{ plural_name } = #{ object_scope.gsub("@",'') }#{ n_plus_one_includes }.page(params[:page])#{ ".per(per)" if @paginate_per_page_selector }"
+
+        res << spaces(4) + "@#{ plural_name } = #{ object_scope.gsub("@",'') }#{ n_plus_one_includes }#{record_scope}"
+
         if @search_fields
           res << @search_fields.collect{ |field|
             wqs = @columns_map[field.to_sym].where_query_statement
@@ -1539,12 +1556,14 @@ class HotGlue::ScaffoldGenerator < Erb::Generators::ScaffoldGenerator
             end
           }.compact.join
         end
+        res << ".page(params[:page])#{ '.per(per)' if @paginate_per_page_selector }"
+
       elsif @nested_set[0] && @nested_set[0][:optional]
-        res << "@#{ plural_name } = #{ class_name }.all"
+        res << "@#{ plural_name } = #{ class_name }.#{record_scope}.all"
       else
-        res << "@#{ plural_name } = #{ class_name }.where(id: #{ auth_object.gsub("@",'') }.id)#{ n_plus_one_includes }"
+        res << "@#{ plural_name } = #{ class_name }.#{record_scope}.where(id: #{ auth_object.gsub("@",'') }.id)#{ n_plus_one_includes }"
 
-        res << ".page(params[:page])#{ ".per(per)" if @paginate_per_page_selector }"
+        res << "#{record_scope}.page(params[:page])#{ ".per(per)" if @paginate_per_page_selector }"
       end
     end
     res << "\n"

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

@@ -60,20 +60,20 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
   <% if !@self_auth %>
   def load_<%= singular_name %>
     <% if @nested_set[0] && @nested_set[0][:optional] %>if params.include?(:<%= @nested_set.last[:singular] %>_id)
-      @<%= singular_name %> = <%= object_scope.gsub("@",'') %>.find(params[:id])
-    else <% end %>@<%= singular_name %> = <%= object_scope %>.find(params[:id])<% if @nested_set[0] && @nested_set[0][:optional]  %>
+      @<%= singular_name %> = <%= object_scope.gsub("@",'') %><%= @record_scope %>.find(params[:id])
+    else <% end %>@<%= singular_name %> = <%= object_scope %><%= @record_scope %>.find(params[:id])<% if @nested_set[0] && @nested_set[0][:optional]  %>
     end<% end %>
   end
   <% else %>
   def load_<%= singular_name %>
-    @<%= singular_name %> = (<%= auth_object.gsub("@",'') %><%= " if params.include?(:#{@nested_set[0][:singular]}_id)" if @nested_set.any? && @nested_set[0][:optional] %>)<% if @nested_set.any? && @nested_set[0][:optional] %> || <%= class_name %>.find(params[:id])<% end %>
+    @<%= singular_name %> = (<%= auth_object.gsub("@",'') %><%= " if params.include?(:#{@nested_set[0][:singular]}_id)" if @nested_set.any? && @nested_set[0][:optional] %>)<% if @nested_set.any? && @nested_set[0][:optional] %> || <%= class_name %>.find(params[:id])<% end %><%= @record_scope %>
   end<% end %>
   <% if @paginate_per_page_selector %>def per
     params[:per] || 10
   end<% end %>
   <% unless @no_list %>
   def load_all_<%= plural %>
-    <%= load_all_code %>
+  <%= load_all_code %>
   end
 
   def index<% if @search == "set" %>
@@ -88,9 +88,9 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
   end<% end %>
 
 <% if create_action %>  def new<% if @object_owner_sym %>
-    @<%= singular_name %> = <% if @pundit %>policy_scope(<% end %><%= class_name %><% if @pundit %>)<% end %>.new<% if eval("#{class_name}.reflect_on_association(:#{@object_owner_sym})").class == ActiveRecord::Reflection::BelongsToReflection  %>(<%= @object_owner_sym %>: <%= @object_owner_eval %>)<% end %><% elsif @object_owner_optional && any_nested? %>
-    @<%= singular_name %> = <% if @pundit %>policy_scope(<% end %><%= class_name  %><% if @pundit %>)<% end %>.new({}.merge(<%= @nested_set.last[:singular] %> ? {<%= @object_owner_sym %>: <%= @object_owner_eval %>} : {}))<% else %>
-    @<%= singular_name %> = <% if @pundit %>policy_scope(<% end %><%= class_name  %><% if @pundit %>)<% end %>.new(<% if any_nested? %><%= @object_owner_sym %>: <%= @object_owner_eval %><% end %>)<% end %>
+    @<%= singular_name %> = <%= class_name %>.new<% if eval("#{class_name}.reflect_on_association(:#{@object_owner_sym})").class == ActiveRecord::Reflection::BelongsToReflection  %>(<%= @object_owner_sym %>: <%= @object_owner_eval %>)<% end %><% elsif @object_owner_optional && any_nested? %>
+    @<%= singular_name %> = <%= class_name  %>.new({}.merge(<%= @nested_set.last[:singular] %> ? {<%= @object_owner_sym %>: <%= @object_owner_eval %>} : {}))<% else %>
+    @<%= singular_name %> = <%= class_name  %>.new(<% if any_nested? %><%= @object_owner_sym %>: <%= @object_owner_eval %><% end %>)<% end %>
     <% if @pundit %>authorize @<%= singular_name %><% end %><% if @pundit %>
     @action = 'new'
   rescue Pundit::NotAuthorizedError
@@ -101,7 +101,7 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
 
   def create
     flash[:notice] = +''
-    modified_params = modify_date_inputs_on_params(<%= singular_name %>_params.dup, <%= current_user_object %>, <%= datetime_fields_list %>)<% if @object_owner_sym &&  eval("#{class_name}.reflect_on_association(:#{@object_owner_sym})").class == ActiveRecord::Reflection::BelongsToReflection  %>
+    modified_params = modify_date_inputs_on_params(<%= singular_name %>_params.dup, <%= current_user_object %>, <%= datetime_fields_list %>)<% if @object_owner_sym && eval("#{class_name}.reflect_on_association(:#{@object_owner_sym})").class == ActiveRecord::Reflection::BelongsToReflection  %>
     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 %>
 
@@ -123,6 +123,7 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
                                  nested_set: @nested_set,
                                  modifier: 'edit_',
                                  with_params: true,
+                                 instance_last_item: true,
                                  put_form: true).gsub("(#{singular}", "(@#{singular}") %><% end %>
     else
       flash[:alert] = "Oops, your <%= singular_name %> could not be created. #{@hawk_alarm}"
@@ -138,11 +139,13 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
 <% unless @no_edit %>
   def show<% if @pundit %>
     authorize @<%= singular %><% end %>
-    redirect_to <%=   HotGlue.optionalized_ternary(namespace: @namespace,
+    redirect_to <%=  HotGlue.optionalized_ternary(namespace: @namespace,
                                  target:  @singular,
+                                 top_level: false,
                                  nested_set: @nested_set,
                                  modifier: 'edit_',
                                  with_params: true,
+                                 instance_last_item: true,
                                  put_form: true).gsub("(#{singular}", "(@#{singular}") %>
   end
 
@@ -165,7 +168,7 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
     end
     <% end %>
 
-    modified_params = modify_date_inputs_on_params(<% if @update_show_only %>update_<% end %><%= singular_name %>_params.dup<%= controller_update_params_tap_away_magic_buttons  %>, <%= current_user_object %>, <%= datetime_fields_list %>) <% if @object_owner_sym &&  eval("#{class_name}.reflect_on_association(:#{@object_owner_sym})").class == ActiveRecord::Reflection::BelongsToReflection  %>
+    modified_params = modify_date_inputs_on_params(<% if @update_show_only %>update_<% end %><%= singular_name %>_params.dup<%= controller_update_params_tap_away_magic_buttons  %>, <%= current_user_object %>, <%= datetime_fields_list %>)<% if @object_owner_sym &&  eval("#{class_name}.reflect_on_association(:#{@object_owner_sym})").class == ActiveRecord::Reflection::BelongsToReflection  %>
     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 @pundit %><% @related_sets.each do |key, related_set| %>
@@ -186,7 +189,9 @@ class <%= controller_class_name %> < <%= controller_descends_from %>
       <% if @display_list_after_update %>    load_all_<%= plural %><% end %>
       flash[:notice] << "Saved #{@<%= singular %>.<%= display_class %>}"
       flash[:alert] = @hawk_alarm if @hawk_alarm
-      <% unless @big_edit %>render :update, status: :unprocessable_entity<% else  %>redirect_to <%= @namespace %>_<%= @plural %>_path<% end %>
+      <% unless @big_edit %>render :update, status: :unprocessable_entity<% else  %>
+      redirect_to <%= path_helper_plural(false) %>
+     <% end %>
     else
       flash[:alert] = "<%= singular_name.titlecase %> could not be saved. #{@hawk_alarm}"
       @action = 'edit'

data/lib/generators/hot_glue/templates/erb/_edit.erb

@@ -4,7 +4,7 @@
   <\%= render(partial: "<%= namespace_with_trailing_dash %>errors", locals: {resource: <%= singular %> }) %>
   <\% end %>
   <h2>Editing <\%= <%= singular %>.<%= display_class %> %></h2>
-  <\%= form_with model: <%= singular %>, url: <%= form_path_edit_helper %> do |f| %>
+  <\%= form_with model: <%= singular %>, url: <%= form_path_edit_helper %><%= ", html: {'data-turbo': false}" if @big_edit %> do |f| %>
     <\%=  render partial: "<%= namespace_with_trailing_dash + @controller_build_folder + "/" %>form", locals: {:<%=  singular  %> => <%= singular %>, f: f}<%= @nested_set.collect{|arg| ".merge(#{arg[:singular]} ? {#{arg[:singular]}: #{arg[:singular]}} : {})" }.join %> \%>
   <% if @edit_within_form_partial %><\%= render partial: "edit_within_form", locals: {f: f, <%=  singular  %>: <%= singular %>}<%= @nested_set.collect{|arg| ".merge(#{arg[:singular]} ? {#{arg[:singular]}: #{arg[:singular]}} : {})" }.join %> %><% end %>
   <\% end %>

data/lib/generators/hot_glue/templates/erb/_line.erb

@@ -1,8 +1,7 @@
 <% if @turbo_streams %><\%= turbo_stream_from <%= singular %>  %>
 <% end %><\%= turbo_frame_tag "<%= @namespace %>__#{ dom_id(<%= singular %>) }" do  %>
   <div class='<%= @layout_strategy.row_classes %>' data-id='<\%= <%= singular  %>.id %>' data-edit='false'>
-    <\%= render partial: '<%= show_path_partial %>', locals: { <%= singular %>: <%= singular %> }<% @nested_set.each do |nest_arg| %>.merge(defined?(<%= nest_arg[:singular] %>) ? {<%= nest_arg[:singular] %>: <%= nest_arg[:singular] %>, nested_for: "<%= nest_arg[:singular] %>-#{<%= nest_arg[:singular] %>.id}"} : {})<% end %> %>
-
+    <\%= render partial: '<%= show_path_partial %>', locals: { <%= singular %>: <%= singular %><% if @nested_set.any? %>, <%= @nested_set.collect{|nest_arg|  "#{nest_arg[:singular]}: #{ nest_arg[:singular]  }"}.join(", ") %>, nested_for: "<%= @nested_set.collect{|nest_arg| nest_arg[:singular] + "-\#{#{nest_arg[:singular]}.id}"}.join("__") %>"<% end %> } %>
   </div>
 <\% end %>
 

data/lib/generators/hot_glue/templates/erb/_show.erb

@@ -25,14 +25,15 @@
 <% end %>
 
 
-<% if @list_after_each_row_partial %><\%= render partial: "list_after_each_row", locals: {<%= @singular %>: <%= @singular %>} %> <% end %>
+<% if @list_after_each_row_partial %><\%= render partial: "<%= namespace_with_trailing_dash %><%= @controller_build_folder %>/list_after_each_row", locals: {<%= @singular %>: <%= @singular %><% if @nested_set.any? %>, <%= @nested_set.collect{|x| x[:singular] + ": " + x[:singular]  }.join(", ") %><% end %>} %>
+<% end %>
 
 <%= @layout_strategy.button_style %>
 <div class=" scaffold-line-buttons <%= @layout_strategy.column_classes_for_button_column %>" <%= @layout_strategy.button_style %>>
   <%= magic_button_output %>
 
   <% if destroy_action %>
-    <\%= form_with url: <%= delete_path_helper %>, html: {data: {'<%= @ujs_syntax ? 'confirm' : 'turbo-confirm' %>': "Are you sure you want to delete #{ <%= @singular + "." + display_class %> }?"}, style: "display: inline-block;"}, method: :delete do |f| %>
+    <\%= form_with url: <%= delete_path_helper %>, html: {data: {'<%= @ujs_syntax ? 'confirm' : 'turbo-confirm' %>': "Are you sure you want to delete #{ <%= @singular + "." + display_class %> }?" }, style: "display: inline-block;"}, method: :delete do |f| %>
       <\%= f.submit "Delete".html_safe,  class: "delete-<%= singular %>-button btn btn-primary btn-sm" %>
     <\% end %>
   <% end %>

data/lib/generators/hot_glue/templates/erb/edit.erb

@@ -4,7 +4,7 @@
     <div class="col-md-12">
       <\%= link_to "<% if @button_icons == 'font-awesome' %><i class='fa fa-arrow-circle-left  2x'></i><% end %> Back to list".html_safe, <%= path_helper_plural(true) %> %>
 <% end %>
-<\%= render partial: "edit", locals: {<%= singular %>: @<%= singular %><%= @nested_set.collect{|arg| ", #{arg[:singular]}: @#{arg[:singular]}" }.join(", ") %>}  %>
+<\%= render partial: "edit", locals: {<%= singular %>: @<%= singular %><%= @nested_set.any? ? () : "" %>}  %>
 <% if @big_edit %>
     </div>
   </div>
@@ -21,13 +21,7 @@
           <% downnest_object_name = eval("#{downnest_class}.table_name") %>
           <% downnest_style = @layout_strategy.downnest_style %>
 
-          <\%= render partial: "<%= namespace_with_trailing_dash %><%= downnest_object_name %>/list", locals: {
-          <%= @singular %>: @<%= @singular %>,
-          <%= downnest_object_name %>: @<%= @singular %>.<%= downnest %>
-          }
-          .merge({nested_for: "<% if @nested_set.any? %>#{nested_for + "__" if defined?(nested_for)}<% end %><%= @singular %>-#{@<%= @singular %>.id}"})
-          <%= @nested_set.collect{|arg| ".merge(defined?(@#{arg[:singular]}) ? {#{arg[:singular]}: @#{arg[:singular]}} : {} )"}.join("\n") %>
-          \%>
+          <\%= render partial: "<%= namespace_with_trailing_dash %><%= downnest_object_name %>/list", locals: {<%= @singular %>: @<%= @singular %>,  <%= downnest_object_name %>: @<%= @singular %>.<%= downnest %><% if @nested_set.any? %>, <%= @nested_set.collect{|x| "#{x[:singular]}: @#{x[:singular]}"}.join(", ") %>, nested_for: "<%= @nested_set.collect{|x| "#{x[:singular]}-" + "\#{" + "@#{x[:singular]}.id}"}.join("__") %>__<%= singular %>-#{@<%= @singular %>.id}" <% end %> } \%>
 
           </div>
         </div>

data/lib/generators/hot_glue/templates/erb/edit.turbo_stream.erb

@@ -1,3 +1,3 @@
-<\%= turbo_stream.replace "<%= @namespace %>__#{dom_id(<%= singular %>)}" do %>
+<\%= turbo_stream.replace "<%= @namespace %>__#{dom_id(@<%= singular %>)}" do %>
   <\%= render 'edit', locals: { <%= singular %>: @<%= singular %><%= @nested_set.collect{|arg| ", #{arg[:singular]}: @#{arg[:singular]} " }.join("") %> } %>
 <\% end %>

data/lib/generators/hot_glue/templates/typeahead_views/typeahead_controller.js

@@ -24,7 +24,6 @@ export default class extends Controller {
       return
     }
     this.previousQuery = this.query
-
     const url = new URL(this.urlValue)
     url.searchParams.append("query", this.query)
     url.searchParams.append("typeahead_identifier", typeaheadIdentifier)

data/lib/hot-glue.rb

@@ -25,19 +25,22 @@ module HotGlue
                                 modifier: "",
                                 with_params: false,
                                 top_level: false,
-                                put_form: false)
+                                put_form: false,
+                                instance_last_item: false )
     instance_sym = top_level ? "@" : ""
 
+
     if nested_set.nil? || nested_set.empty?
       return modifier + "#{(namespace + '_') if namespace}#{target}_path" + (("(#{instance_sym}#{target})" if put_form) || "")
     elsif nested_set[0][:optional] == false
 
-      return modifier + ((namespace + "_" if namespace) || "") + nested_set.collect{|x|
+      res = modifier + ((namespace + "_" if namespace) || "") + nested_set.collect{|x|
         x[:singular] + "_"
       }.join() + target + "_path" + (("(#{nested_set.collect{
         |x| instance_sym + x[:singular] }.join(",")
-      }#{ put_form ? ',' + instance_sym + target : '' })") || "")
+      }#{ put_form ? ',' + (instance_last_item ? "@" : instance_sym) + target : '' })") || "")
 
+      res
 
     else
       # copy the first item, make a ternery in this cycle, and recursively move to both the

data/lib/hotglue/version.rb

@@ -1,5 +1,5 @@
 module HotGlue
   class Version
-    CURRENT = '0.6.8'
+    CURRENT = '0.6.9'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hot-glue
 version: !ruby/object:Gem::Version
-  version: 0.6.8
+  version: 0.6.9
 platform: ruby
 authors:
 - Jason Fleetwood-Boldt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-04 00:00:00.000000000 Z
+date: 2024-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails