Closed (fixed)
Project:
FileField
Version:
6.x-3.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Jul 2009 at 21:47 UTC
Updated:
8 Aug 2009 at 04:20 UTC
Jump to comment: Most recent file
The filefield_widget function sets the form encoding type incorrectly, erasing any other attributes that may exist on the form. It currently does $form['#attributes'] = array('enctype' => 'multipart/form-data');. What it should do is $form['#attributes']['enctype'] = 'multipart/form-data';, possibly checking first to make sure that the #attributes element exists and is an array.
In particular, the current mechanism will erase any custom classes that may have been added to the form previously.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | filefield_enctype_set.patch | 809 bytes | quicksketch |
Comments
Comment #1
quicksketchOddly we were doing it correctly in 2 places but not in the third. This patch corrects the problem and has been applied to CVS. It will be in the 3.2 version.
Comment #2
quicksketch