Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.If you go to node/add/forum when no forums exist yet, and attempt to submit the form, you get:
# An illegal choice has been detected. Please contact the site administrator.
# warning: strtr() [function.strtr]: The second argument is not an array. in /Users/webchick/Sites/head/modules/syslog/syslog.module on line 99.
You ought to not be able to get to this page anyway when no forums exist :P, but I think that's a separate issue from the fact that FAPI doesn't deal well with this condition.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | empty_required_options-172636-5.patch | 1.05 KB | chx |
| #4 | empty_required_options-172636-4.patch | 1.04 KB | chx |
| #2 | form_watchdog-172636-2.patch | 845 bytes | chx |
Comments
Comment #1
jjgronk CreditAttribution: jjgronk commented"An illegal choice has been detected. Please contact the site administrator."
Comment #2
chx CreditAttribution: chx commentedFixes the watchdog call. Forum (actually taxonomy) patch forthcoming.
Comment #3
Dries CreditAttribution: Dries commentedI committed the first patch; waiting for the next one. Thanks.
Comment #4
chx CreditAttribution: chx commentedThis is a broader issue than forum.
Comment #5
chx CreditAttribution: chx commentederror was setting the wrong element.
Comment #6
catchpatch fixes the issue, but introduces notices instead:
# notice: Undefined property: stdClass::$taxonomy in drupal6/modules/forum/forum.module on line 219.
# notice: Undefined index: #name in drupal6/includes/form.inc on line 1262.
# notice: Undefined index: #id in drupal6/includes/form.inc on line 1262.
Comment #7
dpearcefl CreditAttribution: dpearcefl commentedIs this still a problem in current D6?
Comment #8
dpearcefl CreditAttribution: dpearcefl commented