active_scaffold 4.0.1 → 4.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18ae38a98f317aedf4c54ade9d6cf3090b9de8e7042104dbe8493b10f173603a
-  data.tar.gz: 63548a6d7f130d6771818f8387be0b548e74bcd1ed7dc856733b3f72c2a96fda
+  metadata.gz: 1411d7317097c53427854d287da2fe123d38498ab8bbff98308798c6da5ffc9b
+  data.tar.gz: 46a02c5b035126602809edc855e0b6edb71f532c5bd66e91a5347a596895743e
 SHA512:
-  metadata.gz: e1380da8c13475de3a825f828bb3d9109a52690f45d49cabd1be5bcbb138a2f36d985a676b9637e8a992a6ee551dc0f204b9103916e17845ee3990673219ab2c
-  data.tar.gz: 1861237d6327330311899eb90f0f4e8baee3cf81287f6336fb2c0634ab86073c41fa3838b7c2f923a11093e43ae07d074ac0a1a68bdc8df0ed1b23f91356bee2
+  metadata.gz: 250738344e423f2a7509c299f7fcc813a7d36b557d245b3f983ff6bcf6c122729482f2b469625e78d2d9d4252a6d74976ebce6b8cd532ff7ca3633f391171cc3
+  data.tar.gz: 3bda9ffc65eeda39907a9a216ea7f6f8af6ddc799182f7a4d867a2790f9c53d5ba7ec0297326eb1bb19974734e49926bc09ac5561114030c226b986d87cd41f6

data/CHANGELOG.rdoc

@@ -1,3 +1,8 @@
+= 4.0.2
+- Fix using update_columns in field search with search UI submitting a hash, such as date fields
+- Support disabling update_columsns on field_search
+
+= 4.0.1
 - Add :text_editor show_ui, to avoid escaping html code
 - Fix action link on association column named record
 
@@ -11,6 +16,10 @@
 - Thread safety enabled by default, can't be disabled, deprecate ActiveScaffold.threadsafe!
 - Support subgroups on field search
 
+= 3.7.12
+- Fix using update_columns in field search with search UI submitting a hash, such as date fields
+- Support disabling update_columsns on field_search
+
 = 3.7.11.1
 - Fix action link on association column named record
 

data/lib/active_scaffold/actions/core.rb

@@ -115,7 +115,11 @@ module ActiveScaffold::Actions
       copy_attributes(saved_record, record) if saved_record
       apply_constraints_to_record(record) unless scope
       create_association_with_parent record, check_match: true if nested?
-      update_record_from_params(record, columns, attributes || {}, true)
+      if @form_action == :field_search
+        update_columns_from_params(record, columns, attributes || {}, :read, avoid_changes: true, search_attributes: true)
+      else
+        update_record_from_params(record, columns, attributes || {}, true)
+      end
     end
 
     def updated_record_with_column(column, value, scope)

data/lib/active_scaffold/attribute_params.rb

@@ -55,13 +55,17 @@ module ActiveScaffold
     #
     # This is a secure way to apply params to a record, because it's based on a loop over the columns
     # set. The columns set will not yield unauthorized columns, and it will not yield unregistered columns.
-    def update_record_from_params(parent_record, columns, attributes, avoid_changes = false)
+    def update_record_from_params(parent_record, columns, attributes, avoid_changes = false, search_attributes: false)
       crud_type = parent_record.new_record? ? :create : :update
       return parent_record unless parent_record.authorized_for?(crud_type: crud_type)
 
-      multi_parameter_attrs = multi_parameter_attributes(attributes)
       assign_locking_column(parent_record, attributes)
+      update_columns_from_params(parent_record, columns, attributes, crud_type, avoid_changes: avoid_changes, search_attributes: search_attributes)
+      parent_record
+    end
 
+    def update_columns_from_params(parent_record, columns, attributes, crud_type, avoid_changes: false, search_attributes: false)
+      multi_parameter_attrs = multi_parameter_attributes(attributes)
       columns.each_column(for: parent_record, crud_type: crud_type, flatten: true) do |column|
         # Set any passthrough parameters that may be associated with this column (ie, file column "keep" and "temp" attributes)
         assign_column_params(parent_record, column, attributes)
@@ -69,16 +73,16 @@ module ActiveScaffold
         if multi_parameter_attrs.key? column.name.to_s
           parent_record.send(:assign_multiparameter_attributes, multi_parameter_attrs[column.name.to_s])
         elsif attributes.key? column.name
+          next if search_attributes && params_hash?(attributes[column.name])
+
           update_column_from_params(parent_record, column, attributes[column.name], avoid_changes)
         end
       rescue StandardError => e
         message = "on the ActiveScaffold column = :#{column.name} for #{parent_record.inspect} " \
                   "(value from params #{attributes[column.name].inspect})"
         Rails.logger.error "#{e.class.name}: #{e.message} -- #{message}"
-        raise
+        raise e.class, "#{e.message} -- #{message}"
       end
-
-      parent_record
     end
 
     def assign_column_params(parent_record, column, attributes)

data/lib/active_scaffold/config/field_search.rb

@@ -37,6 +37,9 @@ module ActiveScaffold::Config
 
     cattr_accessor :reset_form, instance_accessor: false
 
+    # whether refresh columns defined in update_columns when a column is changed, as create and update forms do
+    cattr_accessor :update_columns
+
     # instance-level configuration
     # ----------------------------
 
@@ -85,8 +88,11 @@ module ActiveScaffold::Config
 
     attr_accessor :reset_form
 
+    # whether refresh columns defined in update_columns when a column is changed, as create and update forms do
+    attr_accessor :update_columns
+
     UserSettings.class_eval do
-      user_attr :optional_columns, :group_options, :grouped_columns, :human_conditions, :floating_footer
+      user_attr :optional_columns, :group_options, :grouped_columns, :human_conditions, :floating_footer, :update_columns
     end
   end
 end

data/lib/active_scaffold/helpers/search_column_helpers.rb

@@ -6,8 +6,10 @@ module ActiveScaffold
       # It does not do any rendering. It only decides which method is responsible for rendering.
       def active_scaffold_search_for(column, options = nil)
         options ||= active_scaffold_search_options(column)
-        search_columns = active_scaffold_config.field_search.columns.visible_columns_names
-        options = update_columns_options(column, nil, options, form_columns: search_columns, url_params: {form_action: :field_search})
+        if active_scaffold_config.field_search.update_columns
+          search_columns = active_scaffold_config.field_search.columns.visible_columns_names
+          options = update_columns_options(column, nil, options, form_columns: search_columns, url_params: {form_action: :field_search})
+        end
         record = options[:object]
         if column.delegated_association
           record = record.send(column.delegated_association.name) || column.active_record_class.new

data/lib/active_scaffold/version.rb

@@ -2,7 +2,7 @@ module ActiveScaffold
   module Version
     MAJOR = 4
     MINOR = 0
-    PATCH = 1
+    PATCH = 2
     FIX = nil
 
     STRING = [MAJOR, MINOR, PATCH, FIX].compact.join('.')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_scaffold
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 4.0.2
 platform: ruby
 authors:
 - Many, see README
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-02-11 00:00:00.000000000 Z
+date: 2025-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails