FIX: Allow setting an array custom field to a singleton value (discou…

…rse#24636)

Also, validation happens per item in an array field.

app/models/concerns/has_custom_fields.rb

@@ -20,14 +20,7 @@ def append_field(target, key, value)
     def validate(obj, name, value)
       return if value.nil?
 
-      size =
-        if Array === type || (type != :json && Array === value)
-          value.map { |v| serialize(v).bytesize }.max || 0
-        else
-          serialize(value).bytesize
-        end
-
-      if size > max_length
+      if serialize(value).bytesize > max_length
         obj.errors.add(
           :base,
           I18n.t("custom_fields.validations.max_value_length", max_value_length: max_length),
@@ -281,7 +274,7 @@ def save_custom_fields(force = false)
           field_type = descriptor.type
 
           if Array === field_type || (field_type != :json && Array === value)
-            value = value || []
+            value = Array(value || [])
             value.compact!
             sub_type = field_type[0]
 

spec/lib/concern/has_custom_fields_spec.rb

@@ -134,6 +134,13 @@ class CustomFieldsTestItemCustomField < ActiveRecord::Base
       expect(db_item.custom_fields).to eq("a" => "b", "c" => "d")
     end
 
+    it "handles assigning singleton values to array fields" do
+      CustomFieldsTestItem.register_custom_field_type "array", [:integer]
+      test_item = CustomFieldsTestItem.new
+      test_item.custom_fields = { "array" => "1" }
+      test_item.save
+    end
+
     it "handles arrays properly" do
       CustomFieldsTestItem.register_custom_field_type "array", [:integer]
       test_item = CustomFieldsTestItem.new