Account for change to Transifex plural forms

Transifex has added a "many" form to es, fr, and it.
sadiqkhoja · May 30, 2022 · 70a60ee · 70a60ee
1 parent 0bd99bb
commit 70a60ee
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/bin/util/transifex.js b/bin/util/transifex.js
@@ -14,10 +14,10 @@ const locales = {
   en: {},
   cs: {},
   de: {},
-  es: {},
-  fr: { pluralCategories: ['one', 'other'] },
+  es: { pluralCategories: ['one', 'many', 'other'] },
+  fr: {},
   id: {},
-  it: {},
+  it: { pluralCategories: ['one', 'many', 'other'] },
   ja: { warnVariableSeparator: false },
   sw: {}
 };
@@ -30,7 +30,9 @@ const sourceLocale = 'en';
   for (const [locale, options] of Object.entries(locales)) {
     const normalized = { ...defaults, ...options };
 
-    if (normalized.pluralCategories == null) {
+    if (normalized.pluralCategories != null) {
+      normalized.pluralCategories = [...normalized.pluralCategories].sort();
+    } else {
       const pluralRules = new Intl.PluralRules([locale]);
       normalized.pluralCategories = pluralRules.resolvedOptions().pluralCategories;
     }

diff --git a/src/i18n.js b/src/i18n.js
@@ -65,9 +65,14 @@ const pluralizationRules = {
     */
     return 2;
   },
+  // es, fr, and it have three plural forms in Transifex: one, many, and other.
+  // However, we never use "many".
+  es: (count) => (count === 1 ? 0 : 2),
   id: noPlural,
   ja: noPlural
 };
+pluralizationRules.fr = pluralizationRules.es;
+pluralizationRules.it = pluralizationRules.es;