"1 songs played today" is not grammatically correct

Bug #1351421 reported by James Hunt
This bug report is a duplicate of:  Bug #1363386: Use plural forms where appropriate. Edit Remove
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu Music App
Confirmed
Low
Andrew Hayzen
libusermetrics (Ubuntu)
Confirmed
Low
Unassigned

Bug Description

If I play a single song in the music app, the welcome shows "1 songs played today". This is not grammatically correct.

Here's the relevant chunk from the music-app.qml:

Metric {
    id: songsMetric
    name: "music-metrics"
    format: "<b>%1</b> " + i18n.tr("songs played today")
    emptyFormat: i18n.tr("No songs played today")
    domain: "com.ubuntu.music"
}

FWICS, it looks like the problem is due to a limitation of libusermetrics. If so, two possible solutions come to mind:

1) Extending libusermetrics to allow the specification of a "singleFormat:" which in this case would be:

    singleFormat: "<b>%1</b> " + i18n.tr("song played today") // It's now "song", not "songs"

2) Having some JS construct the format dynamically based on the value somehow? The problem there being that every app that wanted to get this right would need to implement the JS.

Related branches

Revision history for this message
Andrew Hayzen (ahayzen) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better.
I can confirm that when using the package music-app on ubuntu-touch using the steps provided by James that I can reproduce the bug. Therefore I have set this bug to confirmed.

I have see this with other apps and *thought* this was working for us at one point, however I have just noticed that the photos is now stated in the follow format.
"photos taken today: <b>%1</b>"

Which eliminates the issue.

Changed in music-app:
status: New → Confirmed
importance: Undecided → Low
Andrew Hayzen (ahayzen)
Changed in music-app:
assignee: nobody → Andrew Hayzen (andrew-hayzen)
Revision history for this message
James Hunt (jamesodhunt) wrote :

Hi Andrew,

Yes, that's the simplest fix of course. However, care is going to need to be taken to ensure all consumers of libusermetrics apply such a format.

I suggest we atleast review all the core apps since although this issue does not affect the functionality of the apps...

 - the issue is very prominent (you see the welcome screen whenever you switch on the screen)
 - using the original format creates a bad impression imho (atleast when the phone is set to language=English).

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in libusermetrics (Ubuntu):
status: New → Confirmed
Changed in libusermetrics (Ubuntu):
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.