Merge lp:~dobey/ubuntuone-client/ngettext-plurals into lp:ubuntuone-client

Proposed by dobey
Status: Merged
Approved by: Elliot Murphy
Approved revision: 248
Merged at revision: not available
Proposed branch: lp:~dobey/ubuntuone-client/ngettext-plurals
Merge into: lp:ubuntuone-client
Diff against target: 22 lines
1 file modified
bin/ubuntuone-client-applet (+4/-1)
To merge this branch: bzr merge lp:~dobey/ubuntuone-client/ngettext-plurals
Reviewer Review Type Date Requested Status
Elliot Murphy (community) Approve
Stuart Langridge (community) Approve
Review via email: mp+13217@code.launchpad.net

Commit message

Use ngettext for the files updated notification

To post a comment you must log in.
Revision history for this message
Stuart Langridge (sil) wrote :

Looks fine to me.

review: Approve
Revision history for this message
Elliot Murphy (statik) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/ubuntuone-client-applet'
--- bin/ubuntuone-client-applet 2009-10-09 17:57:11 +0000
+++ bin/ubuntuone-client-applet 2009-10-12 13:20:23 +0000
@@ -50,6 +50,7 @@
50DBusGMainLoop(set_as_default=True)50DBusGMainLoop(set_as_default=True)
5151
52_ = gettext.gettext52_ = gettext.gettext
53P_ = gettext.ngettext
5354
54APPLET_BUS_NAME = "com.ubuntuone.ClientApplet"55APPLET_BUS_NAME = "com.ubuntuone.ClientApplet"
55APPLET_CONFIG_NAME = APPLET_BUS_NAME + ".Config"56APPLET_CONFIG_NAME = APPLET_BUS_NAME + ".Config"
@@ -574,7 +575,9 @@
574 with self.__lock:575 with self.__lock:
575 n = pynotify.Notification(576 n = pynotify.Notification(
576 _("Updating Finished"),577 _("Updating Finished"),
577 _("Ubuntu One finished updating %(total)d files.") %578 P_("Ubuntu One finished updating %(total)d file.",
579 "Ubuntu One finished updating %{total}d files.",
580 self.__total) %
578 { 'total' : self.__total })581 { 'total' : self.__total })
579 self.__total = 0582 self.__total = 0
580 self.__updating = 0583 self.__updating = 0

Subscribers

People subscribed via source and target branches