複数形の定義における不一致

Moin · 2024 年 7 月 21 日午後 3:39

pluralized (count) 翻訳のオーバーライドに関する問題からの議論を続けます:

ハンガリー語だけがこの影響を受けているわけではないと思います。#contribute:bug で問題が指摘されていたためヘブライ語を確認したところ、さらに多くの言語が影響を受けている可能性があると考え、さらに調査を始めました。

packages/plurals/cardinals.js

main


      
          const a = (n) => n == 1 ? 'one' : 'other';
          const b = (n) => (n == 0 || n == 1) ? 'one' : 'other';
          const c = (n) => n >= 0 && n <= 1 ? 'one' : 'other';
          const d = (n) => {
            const s = String(n).split('.'), v0 = !s[1];
            return n == 1 && v0 ? 'one' : 'other';
          };
          const e = (n) => 'other';
          const f = (n, c) => {
            const s = String(n).split('.'), i = s[0], v0 = !s[1], i1000000 = i.slice(-6);
            return n == 1 && v0 ? 'one'
              : !c && i != 0 && i1000000 == 0 && v0 || c > 5 ? 'many'
              : 'other';
          };
          const g = (n) => n == 1 ? 'one'
              : n == 2 ? 'two'
              : 'other';
          
          export const af = a;
          export const ak = b;

This file has been truncated. show original

github.com/discourse/discourse

config/locales/client.tr_TR.yml

main


      
            separator: "."
            delimiter: ","
          human:
            storage_units:
              format: "%n %u"

もう1つ見つかった時点で、さらに多くの言語が影響を受けていると予想されます。

david · 2024 年 7 月 22 日午後 12:10

@Moinさん、ありがとうございます！

この変更this changeを加え、問題の拡大を防ぎました。これにより、コンパイル失敗の影響を限定し、サイト全体に影響を与えるのではなく、_MF文字列のみに影響するようにします。

根本原因の調査を継続し、解決策が得られ次第、こちらで更新します。

loic · 2024 年 7 月 25 日午後 12:32

MessageFormat コンパイラを緩和するパッチをマージしました。これにより、メッセージ内の無効な複数キーが許可されるようになります。

github.com/discourse/discourse

FIX: Allow invalid plural keys in MF translations

main ← loic-handle-bad-mf-pluralization-rules

opened 04:08PM - 24 Jul 24 UTC

Flink

+40 -8

We can get translations with invalid plural keys from Crowdin or from custom ove…rrides. Currently, this will raise an error and the locales won’t be outputted at all. This PR addresses this issue by using the new `strict: false` option of our `messageformat-wrapper` gem, allowing to generate locales even if there are invalid plural keys present.

これにより、ほとんどの問題が解決されるはずです。他のパッチもすぐに続く予定です。

loic · 2024 年 7 月 26 日午前 10:27

別のパッチがマージされました。

github.com/discourse/discourse

FIX: Validate MF strings when adding overrides

main ← loic-validate-translation-overrides

opened 03:00PM - 25 Jul 24 UTC

Flink

+103 -21

Currently, when adding translation overrides, values aren’t validated for MF str…ings. This results in being able to add invalid plural keys or even strings containing invalid syntax. This PR addresses this issue by compiling the string when saving an override if the key is detected as an MF one. If there’s an error from the compiler, it’s added to the model errors, which in turn is displayed to the user in the admin UI, helping them to understand what went wrong. ---- ![image](https://github.com/user-attachments/assets/7315f643-b764-411b-b962-92dac0cd0b20) ![image](https://github.com/user-attachments/assets/bb22c536-5bda-4f5f-8be6-6171b4427245)

これにより、管理UIで無効なMFオーバーライドが作成されるのを防ぎます。

loic · 2024 年 7 月 29 日午後 1:46

このPRをマージしました:

Ruby の複数形ルールは、JS のものと最新の状態になったはずです

j.jaffeux · 2024 年 8 月 3 日午後 1:46

このトピックは、最後の返信から5日後に自動的に閉じられました。新しい返信は許可されていません。

トピック		返信	表示
Issue with pluralized (count) translation override Bug translation , fixed	9	313	2024 年 7 月 22 日
Bad MF translation string in hebrew client locale causes fatal site error Bug	36	462	2024 年 7 月 22 日
Overridden translation sticking after being pluralized Bug translation	3	364	2024 年 8 月 2 日
Error message I don't find : "Error: No 'other' form found in pluralFormatPattern 0" Support	13	1036	2020 年 4 月 24 日
Can't override pluralized string if it is missing in the locale file Bug translation	5	1069	2019 年 5 月 13 日

関連トピック