Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
forms system
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jan 2008 at 01:09 UTC
Updated:
11 Feb 2008 at 12:11 UTC
The image button value handler originally used the variable name $array for some of its internal work. We changed it to use $post later, but one instance snuck through. When multiple image buttons are present in a form using #TREE = TRUE, notices are thrown because of the bogus variable. This patch just fixes the error, changing the variable name.
Thanks, E_ALL!
| Comment | File | Size | Author |
|---|---|---|---|
| image_button_fix.patch | 643 bytes | eaton |
Comments
Comment #1
eaton commentedTook a closer look, and realized this will prevent proper submission of forms using image buttons if #tree is used. Bumping it up to critical.
Comment #2
pasqualleTested with form in css injector module.
The E_notice is gone after the patch.
the bug is evident, the fix is good.
Comment #3
gábor hojtsyThanks, committed.
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.