vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
8 stars 1 forks source link

[ Other ] Addressing PHP notices related to translations in WordPress 6.7 #1130

Closed kurudrive closed 3 weeks ago

kurudrive commented 3 weeks ago

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

WordPress 6.7 で

 Notice: Function _load_textdomain_just_in_time was called <strong>incorrectly</strong>. Translation loading for the <code>vk-all-in-one-expansion-unit</code> domain was triggered too early. This is usually an indicator for some code in the plugin or theme running too early. Translations should be loaded at the <code>init</code> action or later. Please see <a href="https://developer.wordpress.org/advanced-administration/debug/debug-wordpress/">Debugging in WordPress</a> for more information. (This message was added in version 6.7.0.) in /Users/kurudrive/Local Sites/wpnext/app/public/wp-includes/functions.php on line 6114

が発生する

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/issues/1137#event-15249962168

どういう変更をしたか?

パッケージの読み込みファイルで翻訳関数を使っている部分がひっかかっていた。 翻訳関数のあるファイルは init で読み込むように変更 しかし、順番の変更に伴い、HTML要素への属性補完の wp_filter_content_tags フィルターが効かなくなってしまうので、wp_filter_content_tags() はフィルターではなく直接かけるように変更

ソースコードについて

デザイン・UI

プログラムの変更の場合

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

その他

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

確認URL

( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )

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

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


レビュワー向け

確認して変更が反映されていない場合の確認事項