Defect #33085
closed
Unable to update the values of a custom field for enumerations when multiple values option is enabled
Added by Thomas Löber almost 5 years ago.
Updated almost 5 years ago.
Description
Hi,
When I add a custom field to an enumeration and the custom field has the "Multiple values" option set, it is not possible to change the custom value when editing the enumeration.
This is because with multiple values the POST parameter is an array, but that is not permitted in the EnumerationsController.
Here is a patch to fix this issue:
def enumeration_params
# can't require enumeration on #new action
- cf_ids = @enumeration.available_custom_fields.map{|c| c.id.to_s}
+ cf_ids = @enumeration.available_custom_fields.map{|c| c.multiple? ? {c.id.to_s => []} : c.id.to_s}
params.permit(:enumeration => [:name, :active, :is_default, :position, :custom_field_values => cf_ids])[:enumeration]
end
end
Best regards,
Thomas
Files
I have confirmed that values for a custom field of time tracking activities cannot be saved if "Multiple values" option is enabled.
Thomas Löber wrote:
Here is a patch to fix this issue:
[...]
I have confirmed that the code you wrote in descriptipn solves this problem.
To permit an array value of strong_parameters, it is necessary to set an empty array, so {c.id.to_s => []} is set when c.multiple? is true.
The attached patch adds a test to the code written by Thomas Löber.
- Target version set to Candidate for next minor release
- Target version changed from Candidate for next minor release to 4.0.7
Setting the target version to 4.0.7.
- Subject changed from Cannot use custom fields with multiple values in enumerations to Unable to update the values of a custom field for enumerations when multiple values option is enabled
- Status changed from Confirmed to Resolved
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patches. Thank you for providing the fix.
- Status changed from Resolved to Closed
Also available in: Atom
PDF