How to duplicate the error:

-Create a simple module with a "checkboxes" element.
-Check off a couple items.
-Save
-Edit same node
-Deselect all previously checked items
-User sees "illegal choice" error

To the best of my ability to determin the cause, it seems to happen because the $_POST['edit']['form_title'] does not exist but $node->form_title does.

CommentFileSizeAuthor
#5 illegal_choices_errors.png183.72 KBBèr Kessels

Comments

chx’s picture

chx commented
Status: Active » Closed (duplicate)

Let me list what you do not read:

  • bug queue
  • patch queue
  • development list

The error is reported for a gazillion of times. Matt Westgate and Chris Johnson helped me crafting the patch last night (and let's not forget Adrian, hunmonk, m3avrck for brainstorming it). I mailed the devel list this morning asking for testing.

Anyways, thanks for the report :)

morbus iff’s picture

morbus iff
they/them
commented

This is probably my fault - I came in late to one of his discussions and didn't get the full details before I said "Make an Issue outta it."

Steve Dondley’s picture

Steve Dondley commented

Well, I don't think it should be a requirement to follow the development list to submit a bug.

I did get lazy and not do a search in the bug/patch queue. I usually do. But I find it kind of hard to pinpoint duplicate bugs/patches with it. Maybe I don't how to use it very efficiently? At any rate, it's easier for me to report a bug and let someone else to determine whether or not it's a dupe. You have confirmed this. :) Though I'm sorry if it created more work for you. :(

php_coder2114’s picture

php_coder2114 commented

None of the patches work.

Bèr Kessels’s picture

Bèr Kessels commented
StatusFileSize
new illegal_choices_errors.png183.72 KB

This bug also appears in the following case:
1. Create nodes
2. Create new vocabulary w. terms (multilpe select, not required!)
3. Go edit the node. the select is empty!
4. Store the node.

See attached screendumpt for the PHP errors and the form errors.

I would expect form aPI to:
* select <none> as the default in case of a NULL value
* default to <none&gt w/o an error if nothing was selected.