From 13ab34c4e630d82025ac6063d6e368a80272582d Mon Sep 17 00:00:00 2001
From: xunbu <xunbu3@qq.com>
Date: Wed, 15 Oct 2025 13:31:40 +0800
Subject: [PATCH] =?UTF-8?q?=E4=BC=98=E5=8C=96docxv3.0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../ai_translator/docx_translator.py          | 35 +++++++++++--------
 1 file changed, 20 insertions(+), 15 deletions(-)

diff --git a/docutranslate/translator/ai_translator/docx_translator.py b/docutranslate/translator/ai_translator/docx_translator.py
index 0b18ca8..8d53305 100644
--- a/docutranslate/translator/ai_translator/docx_translator.py
+++ b/docutranslate/translator/ai_translator/docx_translator.py
@@ -30,22 +30,23 @@ def is_image_run(run: Run) -> bool:
     return '<w:drawing' in xml or '<w:pict' in xml
 
 
+# ==================== MODIFICATION START ====================
+#  对 is_formatting_only_run 函数进行了修改
+#  旧的实现无法识别仅包含颜色等 rPr 属性的空 Run，导致其与后续文本 Run 错误合并。
+#  新的实现通过一个更简单的标准来判断：只要一个 Run 的文本内容在去除空白后为空，
+#  它就被认为是纯格式化的，从而解决了交叉引用文本消失的问题。
+# ==========================================================
 def is_formatting_only_run(run: Run) -> bool:
     """
-    检查一个 Run 是否主要用于格式化，例如一个空的粗体/斜体/下划线 Run。
+    检查一个 Run 是否仅用于格式化，不包含应被翻译的实质性文本。
+    这包括：
+    - 完全没有文本的 Run (即使它带有颜色等格式)。
+    - 只包含空格、制表符等空白字符的 Run。
     """
-    text = run.text
-    if not text.strip():
-        # Handles empty runs with formatting
-        if run.underline or run.bold or run.italic or run.font.strike or run.font.subscript or run.font.superscript:
-            return True
-        # Handles runs that are just whitespace but have formatting that might be visually significant
-        if text and run.underline:
-            return True
-        # A simple tab run is also considered formatting-only for our purpose
-        if text == '\t':
-            return True
-    return False
+    return not run.text.strip()
+
+
+# ===================== MODIFICATION END =====================
 
 
 # ---------------- 配置类 ----------------
@@ -85,8 +86,8 @@ class DocxTranslator(AiTranslator):
     RECURSIVE_CONTAINER_TAGS = {
         qn('w:smartTag'), qn('w:sdtContent'), qn('w:hyperlink'),
     }
-    # [v5.0] 定义不应翻译其结果的域指令, [v5.3] 增加了 'REF'
-    SKIPPABLE_FIELD_INSTRUCTIONS = {'PAGEREF', 'SEQ', 'PAGE', 'NUMPAGES', 'DATE', 'TIME', 'SECTION', 'REF'}
+    # [v5.0] 定义不应翻译其结果的域指令
+    SKIPPABLE_FIELD_INSTRUCTIONS = {'PAGEREF', 'SEQ', 'PAGE', 'NUMPAGES', 'DATE', 'TIME', 'SECTION'}
 
     def __init__(self, config: DocxTranslatorConfig):
         super().__init__(config=config)
@@ -263,7 +264,9 @@ class DocxTranslator(AiTranslator):
 
             first_real_run_index = -1
             for i, run in enumerate(runs):
+                # 确保run仍然在文档树中
                 if run.element.getparent() is not None:
+                    # 找到第一个可以写入文本的run
                     run.text = final_text
                     first_real_run_index = i
                     break
@@ -272,6 +275,7 @@ class DocxTranslator(AiTranslator):
                 self.logger.warning(f"无法应用翻译 '{final_text}'，因为找不到有效的run。")
                 return
 
+            # 从第一个有效的run之后开始，删除所有多余的run
             for i in range(first_real_run_index + 1, len(runs)):
                 run = runs[i]
                 parent_element = run.element.getparent()
@@ -279,6 +283,7 @@ class DocxTranslator(AiTranslator):
                     try:
                         parent_element.remove(run.element)
                     except ValueError:
+                        # 如果元素已经被其他操作移除，这里会抛出ValueError，可以安全地忽略
                         self.logger.debug(f"尝试删除一个不存在的run元素。这通常是安全的。")
                         pass