uploading in Multisite over quota results in "Http Error"

[dd32]

When in Multisite mode, If you attempt to upload a file that will take you over your usage quota (For example, a 30MB file with the default 10MB limit), the upload will start, and finish with a "HTTP Error" error message.

Expected result: Either the upload would be prevented if the file will take it over the limit, or a error message explaining that they've used up their quota should show.

[dd32]

uploading smaller files does of course succeed. I see there are some strings that are supposed to convey this error message already, unsure of why they're not showing up.

[uglyrobot]

You can't check filesize before uploading with the browser uploader. I think it's possible with the flash uploader but it would be a major rewrite.

[dd32]

Its possible with the flash uploader, it specifically has a parameter to specify the maximum filesize uploadable, At present, this is set to the hosts maximum upload size.

This ticket has a 2 pronged attack,

1. The error message shown is non-descriptive, it should mention the fact that they dont have enough quota
2. It shouldnt allow files greater than min(server_limit, user_limit) from being uploaded.

[wpmuguru]

(In [14420]) limit upload size to site's available limit, see #12853

[wpmuguru]

(In [14421]) use filter vs action in 14420, see #12853

[nacin]

(In [14437]) @since for some new ms functions re: upload file size limits. see #12853.

[nacin]

We need to still handle the error message aspect, if possible.

[wpmuguru]

(In [14682]) show upload limit in media upload UI, see #12853

[wpmuguru]

That could do with some CSS or <h#> tags.

[wpmuguru]

(In [14693]) boldly go where no man has gone before, see #12853

[ocean90]

We get also a HTTP Error when check_admin_referer() fails and when the file size is higher then the allowed file size (ms-options.php).

Both because of the wp_die().

[wpmuguru]

(In [14694]) observe fileupload_maxk in media uploader, see #12853

[ocean90]

[14694] works fine. You need also patch from #12225. Then it looks like this ​http://grab.by/4pf1

[ocean90]

In 12853.patch I want to show you my idea. I introduced ASYNC_UPLOAD. If ASYNC_UPLOAD is true, don't use wp_die() and only return $file. Then let wp_handle_upload() handle the error message.

[wpmuguru]

(In [14716]) only die on async upload error, see #12853

[wpmuguru]

(In [14717]) die in non-async upload in [14716], see #12853

[wpmuguru]

If there is still an issue, someone can re-open.

[dd32]

[14693] - That is a bit too bold IMO, That is something thats better suited to a small paragraph text below the upload buttons, I've added ui-feedback in hope someone will come up with some better wording and/or location..

[dd32]

(In [14776]) We're not as bold as some people make us out to be. Move the maximum upload size from a Heading to a paragraph text. Fix screen formatting on Multisite when over quota. see #12853

[dd32]

Changed it from a header to a paragraph text. Does anyone object to that, have better wording, or a better location for it?

[johnonolan]

Nope, I think it works better as paragraph text!