-
Problem report
-
Resolution: Fixed
-
Major
-
5.0.0alpha1
-
All
-
Sprint 61 (Feb 2020), Sprint 62 (Mar 2020)
-
1
There is a statement added in 5.0.0alpha1 and the latest trunk, which is specified only as %s even when multiple arguments are required.
When translating, We sometimes want to change the order, so please make sure the order is clear, such as %1$s and %2$s. It is also specified in the guidelines.
Current state:
**
#: include/classes/core/CModuleManager.php:228 #, c-format msgid "%s - thrown by module located at %s." #: include/classes/core/ZBase.php:451 #, c-format msgid "Action class %s must extend %s class." #: include/classes/core/ZBase.php:445 #, c-format msgid "Class %s not found for action %s." #: include/classes/core/CModuleManager.php:155 #, c-format msgid "Identical ID (%s) is used by modules located at %s." #: include/classes/core/CModuleManager.php:162 #, c-format msgid "Identical namespace (%s) is used by modules located at %s." #: include/classes/core/CModuleManager.php:222 #, c-format msgid "Module.php class must extend %s for module located at %s."
In Guidelines:
If more than one variable is inserted, variable placeholders MUST have order number.
https://www.zabbix.com/documentation/guidelines/coding/php#translatable_strings
