Techognito/starter-kit
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

cockpit-po-plugin: Fix crash with plural forms

Browse source 
Declare the `pos` variable. Fixes regression introduced in commit
7f6ef51c12.

Add plural forms to German translations to exercise this code path.
This commit is contained in:
Martin Pitt 2021-01-07 08:11:22 +01:00 • committed by Martin Pitt
parent 7f6ef51c12
commit e418cba057
2 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
po/de.po
View file

@ -12,6 +12,7 @@ msgstr ""
"MIME-Version: 1.0\n" "MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n" "Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n" "Content-Transfer-Encoding: 8bit\n"
"Plural-Forms: nplurals=2; plural=n != 1\n"
#: src/index.html:20 #: src/index.html:20
msgid "Cockpit Starter Kit" msgid "Cockpit Starter Kit"

2
src/lib/cockpit-po-plugin.js
View file

@ -59,7 +59,7 @@ module.exports = class {
// We know the brace in is the location to insert our function // We know the brace in is the location to insert our function
if (plurals) { if (plurals) {
pos = output.indexOf('{', 1); const pos = output.indexOf('{', 1);
output = output.substr(0, pos + 1) + "'plural-forms':" + String(plurals) + "," + output.substr(pos + 1); output = output.substr(0, pos + 1) + "'plural-forms':" + String(plurals) + "," + output.substr(pos + 1);
} }