If there are duplicate keys entered for the options of a select component, the options will be reset when the component is saved. With Webform without options_element, it raises a form validation error giving the user the chance to fix the keys.

CommentFileSizeAuthor
#2 options_element_1490292_duplicate_key.patch637 bytesliam morland

Comments

quicksketch’s picture

quicksketch
he/him
commented

Thanks, yeah Options Element should prevent such collisions.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Active » Needs review
StatusFileSize
new options_element_1490292_duplicate_key.patch637 bytes

Patch attached.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

Is there something I can do to help move this issue forward?

quicksketch’s picture

quicksketch
he/him
commented

This patch looks pretty good to me. If you can solicit someone to give it a review that would help cut down on my QA time.

kolafson’s picture

kolafson commented
Status: Needs review » Reviewed & tested by the community

Tested it on a clean install and it worked fine. The options are no longer cleared upon submit.

quicksketch’s picture

quicksketch
he/him
commented
Status: Reviewed & tested by the community » Fixed

Works great on D6 too. Committed to both branches.

Status: Fixed » Closed (fixed)

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