On admin/settings/uploads the maximum file upload size is displayed at the bottom of the page. The script assumes it to be in MB when it's actually in bytes. Attached patch uses format_size() to display the number correctly.

CommentFileSizeAuthor
#3 file_upload_maxsize_0.patch947 bytesChrisKennedy
#1 file-upload-maxsize.png10.1 KBChrisKennedy
file_upload_maxsize.patch927 bytesChrisKennedy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented
FileSize
file-upload-maxsize.png10.1 KB

Here's an example screenshot - using the patch will display the correct value of 2 MB.

Dries’s picture

Dries CreditAttribution: Dries commented

Good job, Chris. The fix looks good. However, personally, I think that message is not positioned properly. Why is it below the fieldset?

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented
FileSize
file_upload_maxsize_0.patch947 bytes

You're right. On my test site I have no extra roles defined so it doesn't look strange. With extra roles it clearly needs to be within the fieldset.

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented
Status: Needs review » Reviewed & tested by the community
Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)