In version 6.x-2.5 I can't UNSELECT the checkboxes Multiple and Listbox on the Form Component "Select"

(Re)Create the bug.

1) Create a From
2) Create on that form a Component "Select"
3) Fill in at least two options like:
1| Option 1
2| Option 2
4) Save the form
5) Click on Edit toe edit the form. Now you see that Multiple and Listbox are selected.
6) Unselect the checkboxes Multiple and Listbox
7) Save the form
8) Edit the form --> checkboxes Multiple and Listbox are still selected.

I downgraded to module WebFrom 6.x-2.4 that doesn't have this bug.

Regards,
Neorg

CommentFileSizeAuthor
#1 webform_select_uncheck.patch3.49 KBquicksketch

Comments

quicksketch’s picture

Status: Active » Needs review
StatusFileSize
new3.49 KB

Thanks for the report. It seems that to PHP, "0" == "Y". :P

This patch should correct the problem by using the more strict equality operator ===. I've already applied the patch, but if you could confirm it fixes the problem I'd appreciate the extra testing.

sunshinee’s picture

Wow. I've bugged you so much lately (unnecessarily), I wasn't about to post with this one. :) The patch works. Thanks for the quick response.

Joy

quicksketch’s picture

Status: Needs review » Fixed

Awesome. Please don't be discouraged from posting issues! Especially when they come with such complete instructions for reproducing a bug. :)

...just, search first. ;)

Thanks again. I've put out a 2.6 version that will be available in a few minutes.

easp’s picture

Version: 6.x-2.5 » 6.x-2.6
Status: Fixed » Active

I am running webform 6.x-2.6.

I have a multi-page form with a few checkboxes on the second page.

If I submit page 2 of the form with the checkboxes unchecked it submits the form correctly.
If I submit page 2 with the checkboxes unchecked and the page does not validate, all the checkboxes get checked with the validation error.

This problem appears only on page 2 or higher pages. It does not happen on page 1 of the form.

quicksketch’s picture

Status: Active » Fixed

easp, that's a different problem from the one described here (this is for the configuration of select lists, not filling out the resulting form). Please open a separate issue.

Status: Fixed » Closed (fixed)

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