brick 1.0.149 → 1.0.150

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 91e6b3d87c423a90e6e6cf62a07235e185373d9bb60990579a13ccea83c6a645
-  data.tar.gz: 8670e080c676eb7c5be3bb62b7ce4e7a310e8d30c98c3d189d513bd9e06a2af3
+  metadata.gz: 7914cc56d8cd8919835028d5ad9b05a50a2b17baf32c9699d1f78bbaf9417e0d
+  data.tar.gz: a7f945e136fe8401c54e8fd24a4d92a5cef24b22bb68d2caceac28d809ff8f07
 SHA512:
-  metadata.gz: 58ff4070f313c577b027f2c83babb99c2aeecaf0cfd05edbed89ed6e5a32485a9bdc8849e303d9eb16fa6f5824beb66e4dbf282843bc4e2b1a00c7ec13e0616a
-  data.tar.gz: 20cd0a6b43f267a653923a86ded57b041cbe49adbd1e239236faac18415eab0e8515490e0828dcc67f4d7e4f809efae9ff7249a1ab5d9a9626a8a24dced8d2d9
+  metadata.gz: fa3ed344e3d4beed83cf9a6fd7059d78032497f8a6c51c825fd50d163e61bd8029afe3dd82e055631d8258b0c9cf54f7d176e064c47fe9fb5a70cc25cdb67faf
+  data.tar.gz: e43ca92a155c805d4dc5be0a2548d9612bc0b1a92cb2e65722ca4c55c81dbef9b476493d6764eece75489e9caf60e2a1046ca960d9a58cafe060aeae11db954a

data/lib/brick/extensions.rb

@@ -2238,12 +2238,7 @@ class Object
 
           if is_need_params
             code << "  def #{params_name}\n"
-            permits = model.columns_hash.keys.map(&:to_sym)
-            permits_txt = permits.map(&:inspect) +
-                          model.reflect_on_all_associations.select { |assoc| assoc.macro == :has_many && assoc.options[:through] }.map do |assoc|
-                            permits << { "#{assoc.name.to_s.singularize}_ids".to_sym => [] }
-                            "#{assoc.name.to_s.singularize}_ids: []"
-                          end
+            permits_txt = self._brick_find_permits(model, (permits = model.columns_hash.keys.map(&:to_sym)))
             code << "    params.require(:#{require_name = model.name.underscore.tr('/', '_')
                              }).permit(#{permits_txt.join(', ')})\n"
             code << "  end\n"
@@ -2259,6 +2254,98 @@ class Object
       [built_controller, code]
     end
 
+    def _brick_find_permits(model, current_permits)
+      model.reflect_on_all_associations.select { |assoc| assoc.macro == :has_many }.each_with_object([]) do |assoc, s|
+        if assoc.options[:through]
+          current_permits << { "#{assoc.name.to_s.singularize}_ids".to_sym => [] }
+          s << "#{assoc.name.to_s.singularize}_ids: []"
+        elsif assoc.active_record.instance_methods.include?(:"#{assoc.name}_attributes=")
+          # Support nested attributes which use the friendly_id gem
+          self._brick_nested_friendly_id if Object.const_defined?('FriendlyId') &&
+                                            assoc.klass.instance_variable_get(:@friendly_id_config)
+          new_attrib_text = self._brick_find_permits(assoc.klass, (new_permits = assoc.klass.columns_hash.keys.map(&:to_sym)))
+          new_permits << :_destroy
+          current_permits << { "#{assoc.name}_attributes".to_sym => new_permits }
+          s << "#{assoc.name}_attributes: #{new_attrib_text}"
+        end
+      end
+      current_permits
+    end
+
+    def _brick_nested_friendly_id
+      unless @_brick_nested_friendly_id
+        ::ActiveRecord::Base.class_exec do
+          if private_instance_methods.include?(:assign_nested_attributes_for_collection_association)
+            alias _brick_anafca assign_nested_attributes_for_collection_association
+            def assign_nested_attributes_for_collection_association(association_name, attributes_collection)
+              association = association(association_name)
+              slug_column = association.klass.instance_variable_get(:@friendly_id_config)&.slug_column
+              return _brick_anafca unless slug_column
+
+              # Here is the FriendlyId version of #assign_nested_attributes_for_collection_association
+              options = nested_attributes_options[association_name]
+              attributes_collection = attributes_collection.to_h if attributes_collection.respond_to?(:permitted?)
+
+              unless attributes_collection.is_a?(Hash) || attributes_collection.is_a?(Array)
+                raise ArgumentError, "Hash or Array expected for attribute `#{association_name}`, got #{attributes_collection.class.name} (#{attributes_collection.inspect})"
+              end
+
+              check_record_limit!(options[:limit], attributes_collection)
+
+              if attributes_collection.is_a? Hash
+                keys = attributes_collection.keys
+                attributes_collection = if keys.include?("id") || keys.include?(:id)
+                                          [attributes_collection]
+                                        else
+                                          attributes_collection.values
+                                        end
+              end
+              existing_records = if association.loaded?
+                                   association.target
+                                 else
+                                   attribute_ids = attributes_collection.filter_map { |a| a["id"] || a[:id] }
+                                   if attribute_ids.empty?
+                                     []
+                                   else # Implement the same logic as "friendly" scope
+                                     association.scope.where(association.klass.primary_key => attribute_ids)
+                                                      .or(association.scope.where(slug_column => attribute_ids))
+                                   end
+                                 end
+              attributes_collection.each do |attributes|
+                attributes = attributes.to_h if attributes.respond_to?(:permitted?)
+                attributes = attributes.with_indifferent_access
+                if attributes["id"].blank? && attributes[slug_column].blank?
+                  unless reject_new_record?(association_name, attributes)
+                    association.reader.build(attributes.except(*::ActiveRecord::Base::UNASSIGNABLE_KEYS))
+                  end
+                elsif (attr_id_str = attributes["id"].to_s) && 
+                      (existing_record = existing_records.detect { |record| record.id.to_s == attr_id_str ||
+                                                                            record.send(slug_column).to_s == attr_id_str })
+                  unless call_reject_if(association_name, attributes)
+                    # Make sure we are operating on the actual object which is in the association's
+                    # proxy_target array (either by finding it, or adding it if not found)
+                    # Take into account that the proxy_target may have changed due to callbacks
+                    target_record = association.target.detect { |record| record.id.to_s == attr_id_str ||
+                                                                          record.send(slug_column).to_s == attr_id_str }
+                    if target_record
+                      existing_record = target_record
+                    else
+                      association.add_to_target(existing_record, skip_callbacks: true)
+                    end
+
+                    assign_to_or_mark_for_destruction(existing_record, attributes, options[:allow_destroy])
+                  end
+                else
+                  raise_nested_attributes_record_not_found!(association_name, attributes["id"])
+                end
+              end
+            end # anafca
+          end
+        end if ::ActiveRecord::QueryMethods.instance_methods.include?(:or)
+        @_brick_nested_friendly_id = true
+      end
+    end
+
     def _brick_get_hm_assoc_name(relation, hm_assoc, source = nil)
       assoc_name, needs_class = if (relation[:hm_counts][hm_assoc[:inverse_table]]&.> 1) &&
                                    hm_assoc[:alternate_name] != (source || name.underscore)

data/lib/brick/version_number.rb

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

metadata

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