confirm_form needs to be polite with $form['#attributes']
cha0s - October 10, 2008 - 19:15
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | forms system |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Description
confirm_form() shouldn't assume that it is the only one on the form to set the '#attributes' element. Please merge the array instead of assigning it.
#1
bump...
#2
Why not suggesting a patch? It will have to be against the 7.x code base (HEAD tag in CVS).
Please read http://drupal.org/patch/create for more information.
#3
Sure thing. After playing I realized we can just concat the string... which leads me to my next issue:
$form['#attributes']['class'] should be an array. What would I file that under, a feature request..?
#4
@cha0s, let's open a task, if you are willing to take the lead on this.
#5
The last submitted patch failed testing.