Issue with pluralized (count) translation override

Lhc_fl · July 16, 2024, 2:37am

I found the cause of the issue. Discourse’s I18n relies on the plural definition of an external library:

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse-i18n/src/index.js#L7

But in this library, we can see that Hungarian is defined as using type a-class plurals, which distinguish between one and other.

github.com/eemeli/make-plural

packages/plurals/cardinals.js

main


      
          },
          
          hsb: (n) => {
            const s = String(n).split('.'), i = s[0], f = s[1] || '', v0 = !s[1], i100 = i.slice(-2), f100 = f.slice(-2);
            return v0 && i100 == 1 || f100 == 1 ? 'one'
              : v0 && i100 == 2 || f100 == 2 ? 'two'
              : v0 && (i100 == 3 || i100 == 4) || (f100 == 3 || f100 == 4) ? 'few'
              : 'other';
          },
          
          hu: a,
          
          hy: (n) => n >= 0 && n < 2 ? 'one' : 'other',
          
          ia: d,
          
          id: e,
          
          ig: e,
          
          ii: e,

I don’t understand Hungarian at all, maybe someone could help point out whether their plural definition is wrong or ours

Topic		Replies	Views
Inconsistency in plural definition Bug fixed	5	147	August 3, 2024
Handling plural forms in activity pub localization strings Translations activitypub	11	113	November 4, 2025
Overridden translation sticking after being pluralized Bug translation	4	316	August 2, 2024
Always use %{count} variable when translating pluralized strings Translations	7	2839	May 20, 2019
Can't override pluralized string if it is missing in the locale file Bug translation	6	1017	May 15, 2019

Issue with pluralized (count) translation override

Related topics