To reprodude: Create a poll with two choices. Edit it, erase the two choices and add two new below. Then the "You must fill in at least two choices" error is fired.

CommentFileSizeAuthor
#3 advpoll_validate_fix.patch630 bytesChrisKennedy

Comments

ChrisKennedy’s picture

ChrisKennedy commented

I tried those steps for a binary poll (after applying patch at http://drupal.org/node/199097) and didn't get that error - it added the new choices and deleted the removed choices. Are you sure about this bug?

anders.fajerson’s picture

anders.fajerson commented

The trick is to leave the two privously added choices there, but manually emptied (do not use the dynamic remove button). When you sumbit the second time you should have four choices in the form, but only the last two filled in.

ChrisKennedy’s picture

ChrisKennedy commented
Status: Active » Needs review
StatusFileSize
new advpoll_validate_fix.patch630 bytes

Here ya go.

anders.fajerson’s picture

anders.fajerson commented
Status: Needs review » Reviewed & tested by the community

Nooo! $_POST!! But should be fine :) Thanks.

ChrisKennedy’s picture

ChrisKennedy commented
Status: Reviewed & tested by the community » Fixed

http://drupal.org/cvs?commit=91560

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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