[ WP6.7 ] 日本語の翻訳が効いていなかったのを修正しました

[mtdkei]

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

#2269

どういう変更をしたか？

具体的には、PHP翻訳ファイルとJavaScript用JSONファイルの読み込みにおいて、特定のファイルを直接指定するように変更してみました。

スクリーンショットまたは動画

変更前 Before

編集画面

設定画面

変更後 After

編集画面

設定画面

実装者の確認事項

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

• 複数の意図の変更 （ 機能の不具合修正 + 別の機能追加など ） を含んでいないか？
• Files changed (変更ファイル)の内容は目視で確認したか？
• readme.txt に変更内容は書いたか？
• 本当にちゃんと確認をしたか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。

• 書けそうなテストは書いたか？
  →テスト箇所が分からなかったのでスキップしました。わかり次第追加します。

変更内容について何を確認したか、どういう方法で確認をしたかなど

• 編集画面のブロック一覧で日本語に翻訳されていることを確認しました。
• 編集画面の/で日本語入力した時に日本語でブロックが出てくることを確認しました。
• ダッシュボードから 設定 > VK Blocks の設定画面で日本語になっていることを確認しました。
• WP6.7と6.6で確認しました。

レビュワーに回す前の確認事項

• 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー確認方法・確認内容など

ただ、このような方法でいいのかも含め様々な角度からご確認いただけたらと思います。

---

レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

• プルしたか？
• ビルドしたか？
• ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
• npm install したか？
• composer install したか？
• キャッシュをクリアして確認したか？

[mthaichi]

@mtdkei ありがとうございます！
テストが通っていないので、6.7未満と以上の時とで読み込み方を変えないといけないかもです。
inc/vk-blocks/vk-blocks-function.php でも同じような処理がありますが、こちらも同様な処理が必要かもです。

[mthaichi]

@mtdkei ありがとうございます！ テストが通っていないので、6.7未満と以上の時とで読み込み方を変えないといけないかもです。 inc/vk-blocks/vk-blocks-function.php でも同じような処理がありますが、こちらも同様な処理が必要かもです。

と書きましたが、言語が英語なのに日本語が表示されてしまっているので、問題があるということですね。
アーカイブリストだけ、この方法でやると翻訳されてしまいます。サーバーサイドレンダリングしている場合に発生するのかと思いきや、そうでもないようで、アーカイブリストだけが英語版なのに日本語に翻訳されてしまいます。

すみませんが、この点調査いただければと。（私も時間があれば調べてみます）

[mtdkei]

@mthaichi
ありがとうございます。確かに確認が抜けてました。
サイトが日本語環境の時だけ日本語で表示されるようにしました。日本語以外の環境下では英語が表示されるようにしました。

[mthaichi]

@mtdkei すみません。なんかおかしなことになってしまったので #2316 を作り直しました。

[mthaichi]

@mtdkei これは急ぎの修正なのでApproveしたのですが、アーカイブリストだけが、英語環境で日本語になってしまう原因はわかりましたか？ 応急処置的な修正で根本原因に触れてないように見えましたので、一応確認です。

原因を、後で構わないので、極力はっきりさせておきたいところですね。

[mtdkei]

@mthaichi
そうでした！こちらの見解を書きそびれていました。
こちらのいくつかの環境では、ブロックはほとんどが日本語で表示されていました。英語で表示されているものはこちら に記載しているように、ブロック名で __() 関数を使っている箇所だけでした。
そのため、サイトの言語設定を環境設定で判定する形に変更し、この現象に対処してみました。

[mtdkei]

#2316 をマージしましたのでこちらはCloseします。ありがとうございました。