Currently format_size() returns bytes for all cases < 1024. So if size is 1 it will return "1 bytes". This patch fixes it to return "1 byte".

CommentFileSizeAuthor
#3 format_size_0.patch1.01 KBChrisKennedy
format_size.patch567 bytesChrisKennedy

Comments

ChrisKennedy’s picture

ChrisKennedy commented

Hmm my < was filtered as html. Here it is again:

Currently format_size() returns bytes for all cases where size < 1024. So if size == 1 it returns "1 bytes". This patch causes it to return "1 byte".

dries’s picture

dries commented
Status: Needs review » Needs work

That only works for English, but not for certain other languages. I'm afraid this isn't the proper fix. We have to let the locale module decide this because that is where we keep per-language plural/singular rules.

ChrisKennedy’s picture

ChrisKennedy commented
Status: Needs work » Needs review
StatusFileSize
new format_size_0.patch1.01 KB

Good point - how about this?

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Needs review » Fixed

Committed to HEAD.

Anonymous’s picture

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