brick 1.0.210 → 1.0.211

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 803d4790408a0b789ff8086b5b81346581071b260cbc528039fd8cfa8894b7ac
-  data.tar.gz: 4503e1b9feba62d33e83f3fadc25a34b70bf9c661dc750187a3c391e3b24d5e8
+  metadata.gz: 7950119f788fe1cac3fde68806f2b4a1145da0cf17ff0d47363b102be9c69933
+  data.tar.gz: 33bb7d89a0744890d4450afde332dbd0f0635a8d3acd0beb4107cf4e44ff9718
 SHA512:
-  metadata.gz: 1fe28c2a9ca30ef663f28fe79e65e1b2787686c6fce6d5738d9b6addd6070437411d5210e1ba641b28eae5215fb37756f3faa80f246d823d5d553c9db990a427
-  data.tar.gz: daa9bb88c21467075717062c1ede0ab38af4207bf96d9a2495ce84f99850a4d0cb09f4c1c95017cb3862ef915fab85036397375d43518f7bee2d7bd6f1255aed
+  metadata.gz: 959c6ada7e03de34f566ad6059a4431ea66d2be9f12764de3edb3760074a70f3e91354e9052bc3c1a74eed4d5d7a274a78cf490195815b736b02f282ffe03d47
+  data.tar.gz: a87abaf8c878fecda93fc6751bae171e5194b6b63d933f83ba331191aee701859d3022754f68af3d730c25add0cc0a8174246fd18913040bed75ec700ba2b543

data/lib/brick/extensions.rb

@@ -92,11 +92,49 @@ module ActiveRecord
         reflect_on_association(assoc).foreign_type || "#{assoc}_type"
       end
 
-      def _brick_all_fields
-        rtans = if respond_to?(:rich_text_association_names)
-                  rich_text_association_names&.map { |rtan| rtan.to_s.start_with?('rich_text_') ? rtan[10..-1] : rtan }
-                end
-        columns_hash.keys.map(&:to_sym) + (rtans || [])
+      def _brick_all_fields(skip_id = nil)
+        col_names = columns_hash.keys
+        # If it's a composite primary key then allow all the values through
+        # TODO: Should disallow any autoincrement / SERIAL columns
+        if skip_id && (pk_as_array = _pk_as_array).length == 1
+          col_names -= _pk_as_array
+        end
+        hoa, hma, rtans = _activestorage_actiontext_fields
+        col_names.map(&:to_sym) + hoa + hma.map { |as| { as => [] } } + rtans.values
+      end
+
+      # Return three lists of fields for this model --
+      # has_one_attached, has_many_attached, and has_rich_text
+      def _activestorage_actiontext_fields
+        fields = [[], [], {}]
+        if !(self <= ActiveStorage::Blob) && respond_to?(:generated_association_methods) # ActiveStorage
+          generated_association_methods.instance_methods.each do |method_sym|
+            method_str = method_sym.to_s
+            fields[0] << method_str[0..-13].to_sym if method_str.end_with?('_attachment=') # has_one_attached
+            fields[1] << method_str[0..-14].to_sym if method_str.end_with?('_attachments=') # has_many_attached
+          end
+        end
+        if respond_to?(:rich_text_association_names) # ActionText
+          rich_text_association_names&.each do |rtan| # has_rich_text
+            rtan_str = rtan.to_s
+            fields[2][rtan] = rtan_str.start_with?('rich_text_') ? rtan_str[10..-1].to_sym : rtan
+          end
+        end
+        fields
+      end
+
+      def _active_storage_name(col_name)
+        if Object.const_defined?('ActiveStorage') && (self <= ActiveStorage::Attachment || self <= ActiveStorage::Blob)
+          if (col_str = col_name.to_s).end_with?('_attachments')
+            col_str[0..-13]
+          elsif col_str.end_with?('_blobs')
+            col_str[0..-7]
+          end
+        end
+      end
+
+      def _pk_as_array
+        self.primary_key.is_a?(Array) ? self.primary_key : [self.primary_key]
       end
 
       def _br_quoted_name(name)
@@ -147,8 +185,8 @@ module ActiveRecord
           skip_columns = _brick_get_fks + (::Brick.config.metadata_columns || []) + [primary_key]
           dsl = if (descrip_col = columns.find { |c| [:boolean, :binary, :xml].exclude?(c.type) && skip_columns.exclude?(c.name) })
                   "[#{descrip_col.name}]"
-                elsif (pk_parts = self.primary_key.is_a?(Array) ? self.primary_key : [self.primary_key])
-                  "#{name} ##{pk_parts.map { |pk_part| "[#{pk_part}]" }.join(', ')}"
+                else
+                  "#{name} ##{_pk_as_array.map { |pk_part| "[#{pk_part}]" }.join(', ')}"
                 end
           ::Brick.config.model_descrips[name] = dsl
         end
@@ -841,6 +879,8 @@ module ActiveRecord
                          end
                          through_sources.push(src_ref) unless src_ref.belongs_to?
                          from_clause = +"#{_br_quoted_name(through_sources.first.table_name)} br_t0"
+                         # ActiveStorage will not get the correct count unless we do some extra filtering later
+                         tbl_nm = 'br_t0' if Object.const_defined?('ActiveStorage') && through_sources.first.klass <= ActiveStorage::Attachment
                          fk_col = through_sources.shift.foreign_key
 
                          idx = 0
@@ -934,10 +974,14 @@ module ActiveRecord
           tbl_nm = hm.macro == :has_and_belongs_to_many ? hm.join_table : hm.table_name
           hm_table_name = _br_quoted_name(tbl_nm)
         end
+        # ActiveStorage has_many_attached needs a bit more filtering
+        if (k_str = hm.klass._active_storage_name(k))
+          where_ct_clause = "WHERE #{_br_quoted_name("#{tbl_nm}.name")} = '#{k_str}' "
+        end
         group_bys = ::Brick.is_oracle || is_mssql ? hm_selects : (1..hm_selects.length).to_a
         join_clause = "LEFT OUTER
 JOIN (SELECT #{hm_selects.map { |s| _br_quoted_name("#{'br_t0.' if from_clause}#{s}") }.join(', ')}, COUNT(#{'DISTINCT ' if hm.options[:through]}#{_br_quoted_name(count_column)
-          }) AS c_t_ FROM #{from_clause || hm_table_name} GROUP BY #{group_bys.join(', ')}) #{_br_quoted_name(tbl_alias)}"
+          }) AS c_t_ FROM #{from_clause || hm_table_name} #{where_ct_clause}GROUP BY #{group_bys.join(', ')}) #{_br_quoted_name(tbl_alias)}"
         self.joins_values |= ["#{join_clause} ON #{on_clause.join(' AND ')}"] # Same as:  joins!(...)
       end unless cust_col_override
       while (n = nix.pop)
@@ -2369,6 +2413,8 @@ class Object
             end
           end
 
+          params_name_sym = (params_name = "#{singular_table_name}_params").to_sym
+
           # By default, views get marked as read-only
           # unless model.readonly # (relation = relations[model.table_name]).key?(:isView)
           code << "  def new\n"
@@ -2376,7 +2422,11 @@ class Object
           code << "  end\n"
           self.define_method :new do
             _schema, @_is_show_schema_list = ::Brick.set_db_schema(params)
-            new_params = model.attribute_names.each_with_object({}) do |a, s|
+            new_params = begin
+                           send(params_name_sym)
+                         rescue
+                         end
+            new_params ||= model.attribute_names.each_with_object({}) do |a, s|
               if (val = params["__#{a}"])
                 # val = case new_obj.class.column_for_attribute(a).type
                 #       when :datetime, :date, :time, :timestamp
@@ -2397,8 +2447,6 @@ class Object
             add_csp_hash
           end
 
-          params_name_sym = (params_name = "#{singular_table_name}_params").to_sym
-
           code << "  def create\n"
           code << "    @#{singular_table_name} = #{model.name}.create(#{params_name})\n"
           code << "  end\n"
@@ -2415,8 +2463,7 @@ class Object
               end
               render json: { result: ::Brick.unexclude_column(table_name, col) }
             else
-              @_lookup_context.instance_variable_set("@#{singular_table_name}".to_sym,
-                                                     (created_obj = model.send(:create, send(params_name_sym))))
+              created_obj = model.send(:create, send(params_name_sym))
               @_lookup_context.instance_variable_set(:@_brick_model, model)
               if created_obj.errors.empty?
                 index
@@ -2493,7 +2540,16 @@ class Object
               if (upd_hash ||= upd_params).fetch(model.inheritance_column, nil)&.strip == ''
                 upd_hash[model.inheritance_column] = nil
               end
-              obj.send(:update, upd_hash || upd_params)
+              # Do not clear out a has_many_attached field if it already has an entry and nothing is supplied
+              hoa, hma, rtans = model._activestorage_actiontext_fields
+              all_params = params[singular_table_name]
+              hma.each do |hma_field|
+                if upd_hash.fetch(hma_field) == [''] && # No new attachments...
+                   all_params&.fetch("_brick_attached_#{hma_field}", nil) # ...and there is something existing
+                  upd_hash.delete(hma_field)
+                end
+              end
+              obj.send(:update, upd_hash)
               if obj.errors.any? # Surface errors to the user in a flash message
                 flash.now.alert = (obj.errors.errors.map { |err| "<b>#{err.attribute}</b> #{err.message}" }.join(', '))
               end
@@ -2543,7 +2599,7 @@ class Object
 
           if is_need_params
             code << "  def #{params_name}\n"
-            permits_txt = model._brick_find_permits(model, permits = model._brick_all_fields)
+            permits_txt = model._brick_find_permits(model, permits = model._brick_all_fields(true))
             code << "    params.require(:#{require_name = model.name.underscore.tr('/', '_')
                              }).permit(#{permits_txt.map(&:inspect).join(', ')})\n"
             code << "  end\n"

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

@@ -641,6 +641,11 @@ window.addEventListener(\"popstate\", linkSchemas);
                 type_col = hm_assoc.inverse_of&.foreign_type || hm_assoc.type
                 keys << [type_col, poly_type]
               end
+              # ActiveStorage has_one_attached and has_many_attached needs additional filtering on the name
+              if (as_name = hm_assoc.klass&._active_storage_name(hm_assoc.name)) # ActiveStorage HMT
+                prefix = 'attachments.' if hm_assoc.through_reflection&.klass&.<= ActiveStorage::Attachment
+                keys << ["#{prefix}name", as_name]
+              end
               keys.to_h
             end
 
@@ -1506,7 +1511,7 @@ end
 
     if (pk = hm.first.klass.primary_key)
       hm_singular_name = (hm_name = hm.first.name.to_s).singularize.underscore
-      obj_br_pk = (pk.is_a?(Array) ? pk : [pk]).each_with_object([]) { |pk_part, s| s << "br_#{hm_singular_name}.#{pk_part}" }.join(', ')
+      obj_br_pk = hm.first.klass._pk_as_array.map { |pk_part| "br_#{hm_singular_name}.#{pk_part}" }.join(', ')
       poly_fix = if (poly_type = (hm.first.options[:as] && hm.first.type))
                    "
                      # Let's fix an unexpected \"feature\" of AR -- when going through a polymorphic has_many

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

@@ -110,6 +110,41 @@ module Brick::Rails::FormBuilder
                    ::Brick::Rails.display_binary(val)
                  end
         end
+      when :file, :files
+        if attached = begin
+                        self.object.send(method)
+                      rescue
+                      end
+          # Show any existing image(s)
+          existing = []
+          got_one = nil
+          (attached.respond_to?(:attachments) ? attached.attachments : [attached]).each do |attachment|
+            next unless (blob = attachment.blob)
+
+            existing << blob.key
+            out << "#{blob.filename}<br>"
+            url = begin
+                    self.object.send(method)&.url
+                  rescue StandardError => e
+                    # Another possible option:
+                    # Rails.application.routes.url_helpers.rails_blob_path(attachment, only_path: true)
+                    Rails.application.routes.url_helpers.rails_storage_proxy_path(attachment, only_path: true)
+                  end
+            out << if url
+                     "<img src=\"#{url}\" title=\"#{val}\">"
+                   else # Convert the raw binary to a Base64 image
+                     ::Brick::Rails.display_binary(attachment.download, 500_000)
+                   end
+            got_one = true
+            out << '<br>'
+          end
+          out << 'Update: ' if got_one
+        end
+        out << self.hidden_field("_brick_attached_#{method}", value: existing.join(',')) unless existing.blank?
+        # Render a "Choose File(s)" input element
+        args = [method.to_sym]
+        args << { multiple: true } if col&.type == :files
+        out << self.file_field(*args)
       when :primary_key
         is_revert = false
       when :json, :jsonb

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

@@ -445,19 +445,25 @@ function onImagesLoaded(event) {
           obj.send("#{model.brick_foreign_type(v.first)}=", v[1].first&.first&.name)
         end
       end if obj.new_record?
-      rtans = model.rich_text_association_names if model.respond_to?(:rich_text_association_names)
-      (model.column_names + (rtans || [])).each do |k|
+      hoa, hma, rtans = model._activestorage_actiontext_fields
+      (model.column_names + hoa + hma + rtans.keys).each do |k|
         pk_pos = (pk.index(k)&.+ 1)
         next if (pk_pos && pk.length == 1 && !bts.key?(k)) ||
                 ::Brick.config.metadata_columns.include?(k)
 
         col = model.columns_hash[k]
-        if !col && rtans&.include?(k)
-          k = k[10..-1] if k.start_with?('rich_text_')
-          col = (rt_col ||= ActiveRecord::ConnectionAdapters::Column.new(
-                              '', nil, ActiveRecord::ConnectionAdapters::SqlTypeMetadata.new(sql_type: 'varchar', type: :text)
-                            )
-                )
+        if !col
+          kwargs = if hoa.include?(k) # has_one_attached
+                     { sql_type: 'binary', type: :file }
+                   elsif hma.include?(k) # has_many_attached
+                     { sql_type: 'binary', type: :files }
+                   elsif rtans&.key?(k) # has_rich_text
+                    k = rtans[k]
+                    { sql_type: 'varchar', type: :text }
+                   end
+          col = (ActiveRecord::ConnectionAdapters::Column.new(
+                  '', nil, ActiveRecord::ConnectionAdapters::SqlTypeMetadata.new(**kwargs)
+                )) if kwargs
         end
         val = obj.attributes[k]
         out << "

data/lib/brick/version_number.rb

@@ -5,7 +5,7 @@ module Brick
   module VERSION
     MAJOR = 1
     MINOR = 0
-    TINY = 210
+    TINY = 211
 
     # 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.210
+  version: 1.0.211
 platform: ruby
 authors:
 - Lorin Thwaits
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-03-22 00:00:00.000000000 Z
+date: 2024-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord