Merge lp:~dpm/ubuntuone-client/plurals-fix into lp:~dobey/ubuntuone-client/pot-build-fix

Proposed by David Planella
Status: Rejected
Rejected by: dobey
Proposed branch: lp:~dpm/ubuntuone-client/plurals-fix
Merge into: lp:~dobey/ubuntuone-client/pot-build-fix
Diff against target: 30 lines
1 file modified
bin/ubuntuone-client-applet (+3/-3)
To merge this branch: bzr merge lp:~dpm/ubuntuone-client/plurals-fix
Reviewer Review Type Date Requested Status
Guillermo Gonzalez (community) Approve
Tim Cole (community) Approve
Review via email: mp+13660@code.launchpad.net
To post a comment you must log in.
Revision history for this message
David Planella (dpm) wrote :

Fix for bug 456578, after conversation with dobey.

Revision history for this message
Tim Cole (tcole) wrote :

This looks strangely familiar, but looks fine.

review: Approve
Revision history for this message
Guillermo Gonzalez (verterok) wrote :

looks good.

review: Approve

Unmerged revisions

264. By David Planella

Rephrased sentence to make it not necessary to use plurals for translation

263. By David Planella

Fix for extraction of ngettext strings (LP #455527)

Additionally marked another string to use plurals

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-14 18:23:35 +0000
+++ bin/ubuntuone-client-applet 2009-10-20 19:10:21 +0000
@@ -50,7 +50,7 @@
50DBusGMainLoop(set_as_default=True)50DBusGMainLoop(set_as_default=True)
5151
52_ = gettext.gettext52_ = gettext.gettext
53P_ = gettext.ngettext53ngettext = gettext.ngettext
5454
55APPLET_BUS_NAME = "com.ubuntuone.ClientApplet"55APPLET_BUS_NAME = "com.ubuntuone.ClientApplet"
56APPLET_CONFIG_NAME = APPLET_BUS_NAME + ".Config"56APPLET_CONFIG_NAME = APPLET_BUS_NAME + ".Config"
@@ -518,7 +518,7 @@
518 def __update_transfer_status(self, done=False):518 def __update_transfer_status(self, done=False):
519 """Update the status display."""519 """Update the status display."""
520 with self.__lock:520 with self.__lock:
521 text = _("Updating %(transfers)d of %(total)d files...") % (521 text = _("Updating file %(transfers)d of %(total)d...", % (
522 { 'transfers' : self.__updating,522 { 'transfers' : self.__updating,
523 'total' : self.__total})523 'total' : self.__total})
524 label = self.__litems["status"].get_child()524 label = self.__litems["status"].get_child()
@@ -582,7 +582,7 @@
582 with self.__lock:582 with self.__lock:
583 n = pynotify.Notification(583 n = pynotify.Notification(
584 _("Updating Finished"),584 _("Updating Finished"),
585 P_("Ubuntu One finished updating %(total)d file.",585 ngettext("Ubuntu One finished updating %(total)d file.",
586 "Ubuntu One finished updating %(total)d files.",586 "Ubuntu One finished updating %(total)d files.",
587 self.__total) % { 'total' : self.__total })587 self.__total) % { 'total' : self.__total })
588 self.__total = 0588 self.__total = 0

Subscribers

People subscribed via source and target branches

to all changes: