The returned "default_value" from Options Element should be either a string or an array depending on the state of of the "multiple" checkbox. It looks like the default value returned is actually based on the *original* #multiple property, not on the user-specified value of the multiple checkbox. This patch makes it so that the returned "default_value" is based on the checkbox, rather than on the original state of the element.

CommentFileSizeAuthor
options_element_multiple_value.patch1.26 KBquicksketch

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

David_Rothstein’s picture

David_Rothstein commented

Related issue: #1009508: Elements with the "multiple" but not "multiple_toggle" property (e.g. checkboxes) don't have their default values saved

I think it turns out there are some times you want it based on the original #multiple property, and some times you don't.