Changeset 3691577

Timestamp:

09/11/2026 02:35:30 PM (6 weeks ago)

Author:

hippogo50

Message:

Version 0.9.4: fix minor bugs related to language handling

Location:

transer-translate/trunk

Files:

• readme.txt (modified) (3 diffs)
• transer-translate.php (modified) (4 diffs)

transer-translate/trunk/readme.txt

@@ -5 +5 @@
 Tested up to: 7.0
 Requires PHP: 7.4
-Stable tag: 0.9.3
+Stable tag: 0.9.4
 License: GPLv2 or later
 License URI: https://www.gnu.org/licenses/gpl-2.0.html
@@ -116 +116 @@
 == Changelog ==
 
+= 0.9.4 =
+* Minor bug fixes.
+* 軽微な不具合を修正しました。
+
 = 0.9.3 =
 * Fixed an issue where visiting a URL with a language prefix that wasn't one of your contracted languages (e.g. `/fr/` when only English is contracted) could show a "page not found" error instead of the original page.
@@ -161 +165 @@
 == Upgrade Notice ==
 
+= 0.9.4 =
+Minor bug fixes. / 軽微な不具合を修正しました。
+
 = 0.9.3 =
 Fixes a "page not found" error that could occur with certain language-prefixed URLs. / 特定の言語プレフィックス付きURLで、ページが見つからないエラーになる不具合を修正しました。

transer-translate/trunk/transer-translate.php

@@ -4 +4 @@
  * Plugin URI:        https://www.transer.io/
  * Description:       Translates the whole page server-side and adds multilingual SEO tags and a language switcher automatically. Translations can be corrected directly on the translated page.
- * Version:           0.9.3
+ * Version:           0.9.4
  * Requires at least: 5.8
  * Requires PHP:      7.4
@@ -18 +18 @@
 }
 
-define('TRANSER_TRANSLATE_VERSION', '0.9.0');
+define('TRANSER_TRANSLATE_VERSION', '0.9.3');
 define('TRANSER_TRANSLATE_OPTION_KEY', 'transer_translate_settings');
 define('TRANSER_TRANSLATE_COOKIE_NAME', 'transer_lang');
 
 /**
- * 管理画面の表示言語を、WordPressの言語設定に合わせる。
+ * 【重要】管理画面の表示言語は、WordPressが自動的に、Text Domain・
+ * Domain Pathヘッダー（このファイル冒頭）だけを見て処理する想定だが、
+ * WordPress 6.7以降の「just-in-time」翻訳読み込みの仕組みには、
+ * プラグイン自身のlanguages/フォルダに同梱した.moファイルが、
+ * 正しく読み込まれない既知の不具合が、複数、報告されている
+ * （WordPress.org公式サポートフォーラムでも、同種の報告が多数あり、
+ * 実際にDescriptionの翻訳が反映されない不具合として確認された）。
  *
  * languages/ に同梱しているのは ja・zh_CN・zh_TW・ko の4言語。
  * それ以外のロケールでは翻訳ファイルが見つからないため、
  * ソース中の英語（msgid）がそのまま表示される。
+ *
+ * 対策として、WordPressの自動読み込みが効かなかった場合に備え、
+ * load_textdomain() を使って、同梱の.moファイルを直接、明示的に
+ * 読み込むフォールバックを用意する（自動読み込みが正しく機能した
+ * 場合は、この処理は何もしない）。
  */
-add_action('init', function () {
-    load_plugin_textdomain(
-        'transer-translate',
-        false,
-        dirname(plugin_basename(__FILE__)) . '/languages'
-    );
-});
+add_action('plugins_loaded', function () {
+    if (is_textdomain_loaded('transer-translate')) {
+        return; // 既に読み込み済み（自動読み込みが正しく機能した）なら、何もしない
+    }
+    $locale = determine_locale();
+    $mofile = plugin_dir_path(__FILE__) . 'languages/transer-translate-' . $locale . '.mo';
+    if (file_exists($mofile)) {
+        load_textdomain('transer-translate', $mofile);
+    }
+}, 1);
 
 function transer_translate_get_option($key, $default = '') {
@@ -720 +734 @@
     $contract_langs = transer_translate_get_option('contract_langs', array());
 
-    // 【重要】strip_lang_prefix()と同じ考え方。「全言語翻訳オプション」が
-    // 有効な場合、契約言語一覧だけでなく、対応する全言語コードも、
-    // 翻訳を許可する対象に含める（実際に依頼された対応）。
-    $all_langs_enabled = transer_translate_get_option('all_langs_enabled', false);
+    // 【重要・仕様変更】以前は transer_translate_get_option('all_langs_enabled', false)
+    // で、WordPress側に保存された（＝設定画面で「変更を保存」を押した時点の）値を
+    // 見ていたが、これだと、transer.io側（TransDataデータベースのdictionary.alllang）
+    // で全言語オプションを切り替えても、この設定画面で「変更を保存」を押すまで、
+    // 反映されないという課題があった（実際に指摘・確認された）。
+    //
+    // 実際に翻訳して良いかどうかの最終判定は、常にtranslate.service側
+    // （app.py）が、その場でTransDataデータベースを直接参照して行っており
+    // （WordPress側の自己申告は一切信用しない設計）、courseの安全性・
+    // 課金の正確性は、この設定の値に関わらず、常に確保されている。
+    // そのため、ここでは常にtrueとして扱い、「保存を忘れる」という
+    // 運用上のリスク自体を無くす（実際に依頼された対応）。
+    $all_langs_enabled = true;
     if ($all_langs_enabled) {
         $all_lang_codes = transer_translate_get_option('all_lang_codes', array());
@@ -939 +962 @@
         return;
     }
-    
-    
-    if (transer_translate_should_process()) {
-        return;
-    }
+
+    /*
+     * 【重要・不具合修正】以前は、transer_translate_should_process()が
+     * trueを返す（＝翻訳される）場合はここで早期returnし、翻訳される
+     * ページ側（buffer_callback内）でだけlangbox.jsを注入する、という
+     * 住み分けにしていた。
+     *
+     * しかし、should_process()はWordPress側に保存された、契約言語・
+     * 全言語オプションの設定を見て判定するため、ダッシュボード側
+     * （transer.io、実際にはTransDataデータベース）で設定を変更した
+     * 直後は、WordPress側の設定を「保存」し直すまで、値が古いまま
+     * 残ってしまう。この結果、「実際には翻訳されない設定に変わって
+     * いるのに、should_process()は"翻訳される"と誤って判定し続け、
+     * ここでの早期returnにより、原文ページに言語ボックス自体が
+     * 一切表示されなくなる」という不具合が、実際に確認された。
+     *
+     * langbox.js自体は、既にサーバーから都度リアルタイムに取得する
+     * elarray・worldlangplus（全言語翻訳オプションの実際の状態）を
+     * 見て、表示内容を正しく判断できる（実際に確認・修正済み）ため、
+     * PHP側でのこの早期return（＝<script>タグ自体を出すか出さないかの
+     * 事前判定）は不要である。常に無条件でlangbox.jsを注入し、
+     * 「表示するかどうか・どの言語を表示するか」の実際の判断は、
+     * langbox.js自身に委ねる。
+     *
+     * 【重複注入について】翻訳されるページでは、buffer_callback内でも
+     * 同じlangbox.jsが注入されるため、1ページに<script>タグが2回
+     * 含まれることがあるが、langbox.js側に重複初期化防止のガードを
+     * 追加済みのため、実害はない（軽微な追加リクエストが1回増えるのみ）。
+     */
 
     $api_key = transer_translate_get_option('api_key', '');