brick 1.0.170 → 1.0.172

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f802431e56f45276546a47b94f3571a6b23a2d0701c34b1f27964fd7ecf5b8f
-  data.tar.gz: 3fd88c0f0626bc9abf08e9317187baf3e755adb5840ac9b56b18c04ddc2577c7
+  metadata.gz: 8a38be99462ba1e409c48be433e3ab289b971872c6a3988e28159f52cfdc4645
+  data.tar.gz: 4af53a2029649d13b978f9ab263ce2ead02818afd807751621cf6fd38ac1b7dc
 SHA512:
-  metadata.gz: 3d5e4731d7956222b2c0ab28f19c5ac60addcea87b556a0765dd28a998c2364148bb348a57a9b3541096da71c6d9e2a4440cf6a038fb40d0b2d5c42aadf01ce5
-  data.tar.gz: c955e2d400887fe6e8ff3869bb6ca2e5a996898187e47d16764676b36e6a68a9ece28cf5593e993760c960cb25ba9c479904a2e327e824dd485e592df0d72d79
+  metadata.gz: 9890e01ab3124bf1256f094569abd92c92712d440df8f3ccf9f06cf6ec57fb29bf4afa3e7acc7344253330ae80cbb17983aa4b6b508f2748b13b8926553dadd1
+  data.tar.gz: 280c4a94f788fe459f83ea18fa858152020d952eadb0bc5cd9aee645e03faf65d3dca5edd045328366d7f67a62879341043657396c6bf35ab0e282850cd889fe

data/lib/brick/extensions.rb

@@ -1676,7 +1676,7 @@ class Object
                               # options[:source] = other.first[:inverse][:assoc_name].to_sym
                               #  And also has been:
                               # hm.first[:inverse][:assoc_name].to_sym
-                              options[:source] = hm.last.to_sym
+                              options[:source] = hm[1].to_sym
                             else
                               through = hm.first.fetch(:alternate_chosen_name, hm.first[:alternate_name])
                             end
@@ -1692,11 +1692,11 @@ class Object
               # Was:
               # options[:class_name] = hm.first[:inverse_table].singularize.camelize
               # options[:foreign_key] = hm.first[:fk].to_sym
-              far_assoc = relations[hm.first[:inverse_table]][:fks].find { |_k, v| v[:assoc_name] == hm.last }
+              far_assoc = relations[hm.first[:inverse_table]][:fks].find { |_k, v| v[:assoc_name] == hm[1] }
               options[:class_name] = far_assoc.last[:inverse_table].singularize.camelize
               options[:foreign_key] = far_assoc.last[:fk].to_sym
             end
-            options[:source] ||= hm.last.to_sym unless hmt_name.singularize == hm.last
+            options[:source] ||= hm[1].to_sym unless hmt_name.singularize == hm[1]
             code << "  has_many :#{hmt_name}#{options.map { |opt| ", #{opt.first}: #{opt.last.inspect}" }.join}\n"
             new_model_class.send(:has_many, hmt_name.to_sym, **options)
           end
@@ -1805,11 +1805,11 @@ class Object
         end
 
         # Add any relevant user-requested HMTs
-        Brick.config.hmts.each do |hmt|
+        Brick.config.hmts&.each do |hmt|
           # Make sure this HMT lines up with the current HM
           next unless hmt.first == table_name && hmt[1] == inverse_table &&
                       # And has not already been auto-created
-                      !(y = hmts.fetch(hmt[2], nil)&.find { |x| x.first[:assoc_name] == hmt[1] })
+                      !(hmts.fetch(hmt[2], nil)&.any? { |existing_hmt| existing_hmt.first[:assoc_name] == hmt[1] })
 
           # Good so far -- now see if we have appropriate HM -> BT/HM associations by which we can create this user-requested HMT
           if (hm_assoc = relation[:fks].find { |_k, v| !v[:is_bt] && v[:assoc_name] == hmt[1] }.last) &&
@@ -1885,7 +1885,7 @@ class Object
                 cspd.select! { |val| val == "'self'" }
                 cspd << style_value
               else
-                cspd << "'sha256-VbHigzrnU2KiWIjHStrKhnGxWRv25WXQNdKr5qhevD8='"
+                cspd << "'sha256-W2ts9oOhJ+mkUTLCn4yS7VZSGkeqfdUogX8MEjE0TSw='"
               end
               cspd << 'https://cdn.jsdelivr.net'
             end

data/lib/brick/frameworks/rails/engine.rb

@@ -1669,8 +1669,8 @@ end
     brick_form_for(obj, options, #{model_name}, bts, #{pk.inspect}) %>
 
 #{unless args.first == 'new'
-  # Was:  confirm_are_you_sure = ActionView.version < ::Gem::Version.new('7.0') ? "data: { confirm: 'Delete #\{model_name} -- Are you sure?' }" : "form: { data: { turbo_confirm: 'Delete #\{model_name} -- Are you sure?' } }"
-  confirm_are_you_sure = "data: { confirm: 'Delete #\{model_name} -- Are you sure?' }"
+  # Was:  confirm_are_you_sure = ActionView.version < ::Gem::Version.new('7.0') ? "data: { confirm: \"Delete #\{model_name} -- Are you sure?\" }" : "form: { data: { turbo_confirm: \"Delete #\{model_name} -- Are you sure?\" } }"
+  confirm_are_you_sure = "data: { confirm: \"Delete #\{model_name} -- Are you sure?\" }"
   ret = +"<%= button_to(\"Delete #\{@#{obj_name}.brick_descrip}\", send(\"#\{#{model_name}._brick_index(:singular)}_path\".to_sym, @#{obj_name}), { method: 'delete', class: 'danger', #{confirm_are_you_sure} }) %>"
   hms_headers.each_with_object(ret) do |hm, s|
     # %%% Would be able to remove this when multiple foreign keys to same destination becomes bulletproof

data/lib/brick/version_number.rb

@@ -5,7 +5,7 @@ module Brick
   module VERSION
     MAJOR = 1
     MINOR = 0
-    TINY = 170
+    TINY = 172
 
     # PRE is nil unless it's a pre-release (beta, RC, etc.)
     PRE = nil

data/lib/brick.rb

@@ -278,7 +278,8 @@ module Brick
         if (through = hmt.last.options[:through])
           # ::Brick.relations[hmt.last.through_reflection.table_name]
           skip_hms[through] = nil if hms[through] && model.is_brick? &&
-                                     hmt.last.klass != hmt.last.active_record # Don't pull HMs for HMTs that point back to the same table
+                                     hmt.last.klass != hmt.last.active_record && # Don't pull HMs for HMTs that point back to the same table
+                                     hmt[1].source_reflection.belongs_to? # Don't pull HMs when the HMT is built from HM -> HM
           # End up with a hash of HMT names pointing to join-table associations
           model._br_associatives[hmt.first] = hms[through] # || hms["#{(opt = hmt.last.options)[:through].to_s.singularize}_#{opt[:source].to_s.pluralize}".to_sym]
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: brick
 version: !ruby/object:Gem::Version
-  version: 1.0.170
+  version: 1.0.172
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-31 00:00:00.000000000 Z
+date: 2023-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord