[#MBS-5084] beta: Don't zero-pad minutes - MusicBrainz JIRA

MusicBrainz Server

beta: Don't zero-pad minutes

Created: 03/Aug/12 11:38 AM Updated: 17/Aug/12 09:45 AM Resolved: 14/Aug/12 01:30 AM

Component/s:

None

Affects Version/s:

Beta

Fix Version/s:

Beta

Issue Links:

Relates

This issue is related to:

~~MBS-4382~~ Display total medium/release duration

Description

« Hide

On musicbrainz.org, minutes are not zero-padded. On beta.musicbrainz.org they are, e.g. compare http://musicbrainz.org/release/83a93c47-3305-4910-a9d6-211210923d81 and http://beta.musicbrainz.org/release/83a93c47-3305-4910-a9d6-211210923d81

I'm not sure where this came in or why, but IMO it's completely unnecessary (98% of the track times in the database are under 10 minutes and if we want things to line up, it would be more effective to right-align them) and unexpected (e.g. neither my music player nor iTunes nor Amazon zero-pad minutes).

Sort Order:

[ Permalink | « Hide ]

patate12 added a comment - 03/Aug/12 11:53 AM

It's not zero-padded any more (or I can't see it).

[ Permalink | « Hide ]

nikki added a comment - 03/Aug/12 11:58 AM

It appears to be caused by ~~MBS-4382~~ which was just removed from beta because of ~~MBS-5068~~

[ Permalink | « Hide ]

Ian McEwen added a comment - 14/Aug/12 01:30 AM

Closing since the branch that caused it was pulled out of beta.

[ Permalink | « Hide ]

Kuno Woudt added a comment - 17/Aug/12 07:37 AM - edited

Ian, probably better to change the "affects version" and/or "fix version". The issue does need to be fixed, preferably before I ship the branch to beta again

[ Permalink | « Hide ]

Ian McEwen added a comment - 17/Aug/12 08:19 AM

There isn't an affects or fix version that applies; as far as I'm concerned this is just part of that ticket at this point. So I guess I could have closed it as duplicate instead, but the difference seems silly.

[ Permalink | « Hide ]

Kuno Woudt added a comment - 17/Aug/12 09:45 AM

fixed in commit 7910461780ae3327d08201bc8c80502b6ea4d4f3 (in the mbs-4382 branch).