A fix

attachment uploadsize.patch ignored as obsolete

Patch looks okay at a glance. I'd have to test it and switch branches to commit it, though.

ste.coda wrote:

A second fix

I've corrected the Patch by adding two trim functions before ini_get (line 26-27 of the patch), removing an useless parenthesis on line 29 and removing a double semicolon on line 39.

Attached:

Applied in r50706. Thanks.

This has regressed somewhere in the meantime; the current code in SpecialUpload.php only checks upload_max_filesize.

Note that UploadFromFile::verifyUpload() does check both upload_max_filesize and post_max_size, but it doesn't seem to trigger properly -- I see no error message instead of seeing a verification failure error.

The limits output for the special page UI and the API should check both settings and return the minimum (actually the max file size would be a bit _smaller_ than the max post size, since a little space will be used for other form variables, but that's a start!)

sumanah wrote:

Adding "reviewed" since Stefano's patch was reviewed and, indeed, committed. cc'ing NeilK since he was recently talking/asking about size limits in uploading.