[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