[Bug 1824005] Re: installing / refreshing snap impossible to translate

[Launchpad Bug Tracker]

This bug was fixed in the package ubuntu-release-upgrader - 1:19.10.6

---------------
ubuntu-release-upgrader (1:19.10.6) eoan; urgency=medium

  * DistUpgradeQuirks.py: modify string regarding refreshing and installing
    snaps so that it is translatable. (LP: #1824005)

 -- Brian Murray <brian at ubuntu.com>  Fri, 14 Jun 2019 10:33:21 -0700

** Changed in: ubuntu-release-upgrader (Ubuntu Eoan)
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to ubuntu-release-upgrader in
Ubuntu.
https://bugs.launchpad.net/bugs/1824005

Title:
  installing / refreshing snap impossible to translate

Status in Release Upgrader:
  New
Status in Ubuntu Translations:
  Triaged
Status in ubuntu-release-upgrader package in Ubuntu:
  Fix Released
Status in ubuntu-release-upgrader source package in Eoan:
  Fix Released

Bug description:
  Hi,

  In Disco, for ubuntu-release-upgrader package, there is currently an odd string:
  %sing snap %s

  Manfred Hampl found out:
  "that "%s" is a placeholder that is filled with a string parameter, and "ing" is constant text.

  Digging into the source https://git.launchpad.net/ubuntu/+source
  /ubuntu-release-upgrader/tree/DistUpgrade/DistUpgradeQuirks.py#n459
  (not sure whether it is the right version), it seems to be another
  example of bad tricks that make translations impossible.

  As far as I can see there are two branches, and depending on the status of the snap, it is either installing or refreshing that snap.
  The value of "command" is either "install" or "refresh", and the string %sing results either in "installing" or in "refreshing".

  With the way that this was done, I do not see any possibility that the
  words "install" and "refresh" could be translated to a different
  language.

  The only reasonable solution that I can suggest is a split into two messages, e.g.
  …
              if re.search("^installed: ", …
                  …
                  command = 'refresh'
                  self._view.updateStatus(_("refreshing snap %s" % snap))
              else:
                  command = 'install'
                  self._view.updateStatus(_("installing snap %s" % snap))
              try:
                  self._view.processEvents()
  …
  This would make the strings easily translatable."

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-release-upgrader/+bug/1824005/+subscriptions