Project

General

Profile

Actions

Bug #17482

closed

Incorrect concatentation of environment import strings to form sentences

Added by Dominic Cleal about 8 years ago. Updated almost 8 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Category:
Internationalization
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Some of the newly extracted strings in #15779 are incorrectly concatenated to form sentences. They must be extracted and translated as one - individual words can't be translated and added together, it will not work in some languages.

https://github.com/theforeman/foreman/commit/eb371ca33dcdd7a4f71d3cda0cfcad701cd3ae7e#diff-e9a92ed45620b6f0de59ab2e0bd3c791R18
https://github.com/theforeman/foreman/commit/eb371ca33dcdd7a4f71d3cda0cfcad701cd3ae7e#diff-e9a92ed45620b6f0de59ab2e0bd3c791R41

It's been separated into the form:

_("Update") + " " + _("environment") + name + " (5 " + _("classes") + ")" 

This should be:

_("Update environment %{environment} (%{count} classes)")

and should also use n_() to handle plural/non-plural forms.

The string "Update on background" is also added, which should be "Update in the background" or possibly, "Update in background".


Related issues 1 (0 open1 closed)

Related to Foreman - Feature #15779: Make report, puppet classes and environments importing asynchronous using foreman-tasksClosedMarek Hulán07/22/2016Actions
Actions #1

Updated by Dominic Cleal about 8 years ago

  • Related to Feature #15779: Make report, puppet classes and environments importing asynchronous using foreman-tasks added
Actions #2

Updated by Dominic Cleal about 8 years ago

  • Description updated (diff)
Actions #3

Updated by Marek Hulán about 8 years ago

  • Assignee set to Marek Hulán
  • Target version set to 1.4.4
Actions #4

Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4054 added
Actions #5

Updated by Marek Hulán about 8 years ago

  • Target version changed from 1.4.4 to 1.10.1
Actions #6

Updated by Dominic Cleal about 8 years ago

  • Translation missing: en.field_release changed from 189 to 209
Actions #7

Updated by Marek Hulán almost 8 years ago

  • Target version changed from 1.10.1 to 1.10.2
Actions #8

Updated by Marek Hulán almost 8 years ago

  • Target version changed from 1.10.2 to 1.11.2
Actions #9

Updated by Marek Hulán almost 8 years ago

  • Target version changed from 1.11.2 to 1.11.4
Actions #10

Updated by Marek Hulán almost 8 years ago

  • Target version changed from 1.11.4 to 1.12.1
Actions #11

Updated by Dominic Cleal almost 8 years ago

  • Status changed from Ready For Testing to Rejected
  • Translation missing: en.field_release deleted (209)

#15779 reverted, removing the strings.

Actions

Also available in: Atom PDF