[wp-trac] [WordPress Trac] #19001: Media uploader (plupload) - exceeding max file size doesn't currently give feedback/error

WordPress Trac wp-trac at lists.automattic.com
Wed Oct 19 21:07:46 UTC 2011


#19001: Media uploader (plupload) - exceeding max file size doesn't currently give
feedback/error
-------------------------+------------------
 Reporter:  WPsites      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.3
Component:  Media        |     Version:  3.3
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------
Changes (by kurtpayne):

 * cc: kpayne@… (removed)
 * keywords:   => has-patch


Comment:

 When you add one file that's in error, the `Error` event fired and the
 `FilesAdded` event was never being fired.  [[attachment:19001.1.patch]]
 adds a check to ensure that the file is in the queue list so that an error
 message can be displayed next to the file name.

 When you upload multiple files, the `Error` event was still being fired,
 but when the upload the `FilesAdded` event was still being called after
 each upload was done (regardless of error state).  This is why there were
 duplicates in the file list in [[attachment:19001.patch]].  In the
 [[attachment:19001.1.patch]], I added a check in the `fileQueued()`
 function to make sure duplicates weren't being created.

 When uploading a file that is too large, the error message is:

 > You may only upload 1 file.

 This may be confusing for some users.  It is only one file, it just
 happens to be larger than the allowed upload size.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19001#comment:3>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list