Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ja]BCD クエリーのエラー #7849

Open
potappo opened this issue Aug 20, 2022 · 4 comments
Open

[ja]BCD クエリーのエラー #7849

potappo opened this issue Aug 20, 2022 · 4 comments
Labels
l10n-ja Issues related to Japanese content.

Comments

@potappo
Copy link
Collaborator

potappo commented Aug 20, 2022

日本語版ページにおいて、BCD クエリ―のエラーが散見されます。
原因は以下になるかと思います。

  • 英語版でクエリ―が変わっている場合
  • 英語版ではもうBCDテーブルがない場合

まとめて抽出して修正する方法で、確実に実行できる方法は、すぐには思いつかないですが、issue として立てておきます。

@potappo potappo added the l10n-ja Issues related to Japanese content. label Aug 20, 2022
@debiru
Copy link
Contributor

debiru commented Aug 21, 2022

bad-bcd-queries を抽出しました。

修正すべき正しい query の値は抽出できていません(手動で英語版を確認する必要がある)。

@debiru
Copy link
Contributor

debiru commented Aug 22, 2022

日本語版の Bad BCD queries を修正しようとしたら、英語版の方でも Bad BCD queries になっていたというケースがいくつかありました。

修正可能なページは、以下に限られるかもしれません。

@debiru
Copy link
Contributor

debiru commented Aug 22, 2022

それと、日本語版しかページが存在しないいわゆるゴミページに Bad BCD queries が散見されるようです。これらはページ自体を削除してしまうのが適切な可能性があります。

@debiru
Copy link
Contributor

debiru commented Aug 23, 2022

日本語版で Bad BCD queries を修正することについて、少し事情が変わりました。

#7901 (comment) の通り、日本語版では browser-compat メタデータ項目は不要とのことです。

つまり、日本語版の Bad BCD queries を抽出することに意味はなくなり、すべきことは「browser-compat を削除する」かつ「Compat マクロの引数を削除する」ということになります。

一括置換すればほとんど解決しますが、注意すべきことがあります。

  • browser-compat: -A -B のように、リストで書かれているページがある可能性があります。
  • {{Compat("A")}} {{Compat("B")}} のように、複数のマクロを記述しているページがある可能性があります。
  • {{Compat("api.AbortController")}} のように」と説明文中でマクロ表記が書かれていることがあります。(参照)
  • {{Compat("A", 10)}} のように数値を引数に取っているマクロがあります。

これらの存在を先にチェックしてから作業する必要があります。

一括置換するとしたら、上記を考慮した上で次のようなコマンドを実行することになるかと思います。

grep -r 'browser-compat' . -l | xargs perl -i -pe "BEGIN{undef $/;} s@^browser-compat: [^\n]+\n@@smg"
grep -r -i '{{Compat(' . -l | xargs perl -i -pe 'BEGIN{undef $/;} s@{{Compat\("[^"]+"\)}}@{{Compat}}@smg'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ja Issues related to Japanese content.
Projects
None yet
Development

No branches or pull requests

2 participants