Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 12 years ago

Closed 12 years ago

#30908 closed defect (bug) (fixed)

Remove tailing (white)space from size_format()

Reported by:	tillkruess	Owned by:	pento
Priority:	normal	Milestone:	4.2
Component:	General	Version:	2.5
Severity:	normal	Keywords:
Cc:		Focuses:

Description

size_format() returns an unnecessary tailing space if the unite is B.

Attachments (1)

30908.diff (1.3 KB ) - added by tillkruess 12 years ago.

Download all attachments as: .zip

Change History (4)

@tillkruess
12 years ago

Attachment 30908.diff added

#1 @tillkruess
12 years ago

Attached patch removed that single space. Patch also includes PHPdoc and WP coding standards updates.

#2 @pento
12 years ago

Milestone Awaiting Review → 4.2
Owner set to pento
Status new → assigned
Version trunk → 2.5

#3 @pento
12 years ago

Resolution → fixed
Status assigned → closed

size_format() incorrectly included a trailing space for B values: less than 1024 bytes.

Also add a unit test to check for this, so we don't do it again.

Fixes #30908.

Props tillkruess.

Note: See TracTickets for help on using tickets.

Download in other formats: