From 5739a98e37ee6451bada528d35acbdc2e54654d9 Mon Sep 17 00:00:00 2001
From: "Moises Lopez - https://www.vauxoo.com/" <moylop260@vauxoo.com>
Date: Sun, 25 Jan 2026 07:58:11 -0600
Subject: [PATCH 1/4] [REF] xml-template-prettier-incompatible: Add new cases
 detected

Continue from https://github.com/OCA/odoo-pre-commit-hooks/pull/150

Related to https://github.com/OCA/odoo-pre-commit-hooks/issues/149
---
 README.md                                     | 14 ++---
 .../checks_odoo_module_xml.py                 | 62 ++++++++++++-------
 test_repo/test_module/website_templates.xml   | 18 ++++++
 tests/test_checks.py                          |  2 +-
 4 files changed, 67 insertions(+), 29 deletions(-)

diff --git a/README.md b/README.md
index 7f3bb8dc..46262a09 100644
--- a/README.md
+++ b/README.md
@@ -456,7 +456,7 @@ options:
  * xml-deprecated-qweb-directive
 
     - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L7 Deprecated QWeb directive `t-esc-options`. Use `t-options` instead
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L37 Deprecated QWeb directive `t-field-options`. Use `t-options` instead
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L55 Deprecated QWeb directive `t-field-options`. Use `t-options` instead
 
  * xml-deprecated-qweb-directive-15
 
@@ -523,14 +523,14 @@ options:
 
  * xml-not-valid-char-link
 
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L59 The resource in in src/href contains a not valid character
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L61 The resource in in src/href contains a not valid character
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L77 The resource in in src/href contains a not valid character
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L79 The resource in in src/href contains a not valid character
 
  * xml-oe-structure-missing-id
 
     - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L9 Consider removing the class `oe_structure` or adding a proper id to the tag. The id must contain `oe_structure`
     - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L13 Consider removing the class `oe_structure` or adding a proper id to the tag. The id must contain `oe_structure`
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L41 Consider removing the class `oe_structure` or adding a proper id to the tag. The id must contain `oe_structure`
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L59 Consider removing the class `oe_structure` or adding a proper id to the tag. The id must contain `oe_structure`
 
  * xml-record-missing-id
 
@@ -550,9 +550,9 @@ options:
 
  * xml-template-prettier-incompatible
 
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L22 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...`
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L24 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...`
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L26 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...`
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L22 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent)
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L24 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent)
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L26 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent)
 
  * xml-view-dangerous-replace-low-priority
 
diff --git a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
index 4cbf2b9f..e16ff05e 100644
--- a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
+++ b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
@@ -633,27 +633,47 @@ def verify_template_prettier_incompatible(self, template, manifest_data):
         """There are text tags incompatible with prettier xml autofix
         More info https://github.com/OCA/odoo-pre-commit-hooks/issues/149"""
         target_attrs = {"t-out", "t-esc", "t-raw"}
-        for node_textarea in template.xpath(".//textarea"):
-            if len(node_textarea_child := node_textarea.getchildren()) != 1:
-                continue
-            node_textarea_child = node_textarea_child[0]
-            has_text = node_textarea.text and node_textarea.text.strip()
-            has_tail = node_textarea_child.tail and node_textarea_child.tail.strip()
-            found_attr = set(node_textarea_child.attrib) & target_attrs
-            if node_textarea_child.tag != "t" or has_text or has_tail or not found_attr:
-                continue
-            found_attr = found_attr.pop()
-            self.register_error(
-                code="xml-template-prettier-incompatible",
-                message=(
-                    f"Node `<{node_textarea.tag} ...><{node_textarea_child.tag} {found_attr}=...` "
-                    "incompatible for Prettier XML auto-fix. To prevent unexpected text insertion "
-                    f"prefer `<{node_textarea.tag} {found_attr}=...`"
-                ),
-                filepath=manifest_data["filename_short"],
-                line=node_textarea_child.sourceline,
-            )
-            # TODO: Autofix using the same node
+        # Tags that wrap inline content and are affected by prettier formatting
+        inline_wrapper_tags = {"textarea", "b", "strong", "i", "em", "span", "a", "u", "s", "small", "mark"}
+
+        for wrapper_tag in inline_wrapper_tags:
+            for node_wrapper in template.xpath(f".//{wrapper_tag}"):
+                children = node_wrapper.getchildren()
+                if len(children) != 1:
+                    continue
+
+                node_wrapper_child = children[0]
+
+                # Check if child has template attributes
+                found_attr = set(node_wrapper_child.attrib) & target_attrs
+                if not found_attr:
+                    continue
+
+                # Only check for t or span tags with template attributes
+                if node_wrapper_child.tag not in ("t", "span"):
+                    continue
+
+                # Check if wrapper has ONLY whitespace (or nothing) before the child
+                wrapper_text = node_wrapper.text or ""
+                has_meaningful_text = wrapper_text.strip()
+
+                # The problem occurs when:
+                # 1. There's a template attribute on the child
+                # 2. The wrapper has no meaningful text before the child (only whitespace/newlines)
+                # This means prettier will reformat and add unwanted newlines
+                if not has_meaningful_text:
+                    found_attr = found_attr.pop()
+                    self.register_error(
+                        code="xml-template-prettier-incompatible",
+                        message=(
+                            f"Node `<{node_wrapper.tag} ...><{node_wrapper_child.tag} {found_attr}=...` "
+                            "incompatible for Prettier XML auto-fix. To prevent unexpected text insertion "
+                            f"prefer `<{node_wrapper.tag} {found_attr}=...>` (move attribute to parent)"
+                        ),
+                        filepath=manifest_data["filename_short"],
+                        line=node_wrapper_child.sourceline,
+                    )
+                    # TODO: Autofix using the same node
 
     def has_escaped_double_quotes(self, filename) -> bool:
         """Check if filename contains escaped double quotes " -> &quot;
diff --git a/test_repo/test_module/website_templates.xml b/test_repo/test_module/website_templates.xml
index f118dbda..55592ee0 100644
--- a/test_repo/test_module/website_templates.xml
+++ b/test_repo/test_module/website_templates.xml
@@ -28,6 +28,24 @@
         <textarea name="fixed" t-out="campaign.description" />
         <!--prettier compatible-->
         <textarea name="mixed">   Ver: <t t-out="campaign.url" /></textarea>
+        <!--prettier compatible-->
+        <div>
+            <div class="page">
+                Dear
+                <b>
+                <span t-out="partner_name or ''" />
+            </b>:<br />
+                By this means, you are informed that the budget movement of
+                <span t-out="move_type" />
+                with the number:
+                <b>
+                <span t-out="move_name" />
+            </b>
+                    has been liquidated and transferred
+                to the available balance of the related budgetary item.<br />
+                Regards,
+            </div>
+        </div>
     </template>
 
     <!-- Deprecated QWeb directive "t-field-options". -->
diff --git a/tests/test_checks.py b/tests/test_checks.py
index 65a413a1..523af715 100644
--- a/tests/test_checks.py
+++ b/tests/test_checks.py
@@ -44,7 +44,7 @@
     "xml-not-valid-char-link": 2,
     "xml-redundant-module-name": 3,
     "xml-syntax-error": 2,
-    "xml-template-prettier-incompatible": 3,
+    "xml-template-prettier-incompatible": 5,
     "xml-view-dangerous-replace-low-priority": 7,
     "xml-xpath-translatable-item": 4,
     "xml-oe-structure-missing-id": 6,

From d8a7de83bec656413911aefa2fb5a02234ae27a2 Mon Sep 17 00:00:00 2001
From: "Moises Lopez - https://www.vauxoo.com/" <moylop260@vauxoo.com>
Date: Sun, 25 Jan 2026 08:26:19 -0600
Subject: [PATCH 2/4] improve-msg

---
 README.md                                          | 6 +++---
 src/oca_pre_commit_hooks/checks_odoo_module_xml.py | 4 +++-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index 46262a09..31f457eb 100644
--- a/README.md
+++ b/README.md
@@ -550,9 +550,9 @@ options:
 
  * xml-template-prettier-incompatible
 
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L22 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent)
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L24 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent)
-    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L26 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent)
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L22 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent) or using 'style' attribute instead of tag e.g. <tag style="font-weight: bold">Black Text... instead of <b><tag...
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L24 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent) or using 'style' attribute instead of tag e.g. <tag style="font-weight: bold">Black Text... instead of <b><tag...
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L26 Node `<textarea ...><t t-out=...` incompatible for Prettier XML auto-fix. To prevent unexpected text insertion prefer `<textarea t-out=...>` (move attribute to parent) or using 'style' attribute instead of tag e.g. <tag style="font-weight: bold">Black Text... instead of <b><tag...
 
  * xml-view-dangerous-replace-low-priority
 
diff --git a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
index e16ff05e..88f654ab 100644
--- a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
+++ b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
@@ -668,7 +668,9 @@ def verify_template_prettier_incompatible(self, template, manifest_data):
                         message=(
                             f"Node `<{node_wrapper.tag} ...><{node_wrapper_child.tag} {found_attr}=...` "
                             "incompatible for Prettier XML auto-fix. To prevent unexpected text insertion "
-                            f"prefer `<{node_wrapper.tag} {found_attr}=...>` (move attribute to parent)"
+                            f"prefer `<{node_wrapper.tag} {found_attr}=...>` (move attribute to parent) or "
+                            "using 'style' attribute instead of tag "
+                            "e.g. <tag style=\"font-weight: bold\">Black Text... instead of <b><tag..."
                         ),
                         filepath=manifest_data["filename_short"],
                         line=node_wrapper_child.sourceline,

From e88721152607a03e9cbc923451aa235ebb8abf5b Mon Sep 17 00:00:00 2001
From: "Moises Lopez - https://www.vauxoo.com/" <moylop260@vauxoo.com>
Date: Sun, 25 Jan 2026 11:08:48 -0600
Subject: [PATCH 3/4] [ADD] xml-superfluous-attributeless: Add new check to
 remove superfluous "span" tag

---
 README.md                                     |  6 ++++++
 .../checks_odoo_module_xml.py                 | 21 +++++++++++++++++++
 tests/test_checks.py                          |  1 +
 3 files changed, 28 insertions(+)

diff --git a/README.md b/README.md
index 31f457eb..084dd3ef 100644
--- a/README.md
+++ b/README.md
@@ -543,6 +543,12 @@ options:
     - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/broken_module/model_view_odoo2.xml#L158 Redundant module name `<menuitem id="broken_module.menu_root"` Use `<menuitem id="menu_root"` instead
     - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/broken_module/model_view_odoo2.xml#L165 Redundant module name `<menuitem id="broken_module.menu_root2"` Use `<menuitem id="menu_root2"` instead
 
+ * xml-superfluous-attributeless
+
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/broken_module/template1.xml#L30 Remove superfluous attributeless `<span` Serve no purpose and cause formatting inconsistencies with Prettier
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/broken_module/template1.xml#L34 Remove superfluous attributeless `<span` Serve no purpose and cause formatting inconsistencies with Prettier
+    - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/test_module/website_templates.xml#L60 Remove superfluous attributeless `<span` Serve no purpose and cause formatting inconsistencies with Prettier
+
  * xml-syntax-error
 
     - https://github.com/OCA/odoo-pre-commit-hooks/blob/v0.2.20/test_repo/broken_module/file_no_exist.xml#L1 [Errno 2] No such file or directory: ''
diff --git a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
index 88f654ab..bac187ab 100644
--- a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
+++ b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
@@ -787,6 +787,7 @@ def check_xml_double_quotes_py(self):
         "xml-dangerous-qweb-replace-low-priority",
         "xml-duplicate-template-id",
         "xml-id-position-first",
+        "xml-superfluous-attributeless",
         "xml-template-prettier-incompatible",
     )
     def check_xml_templates(self):
@@ -800,6 +801,7 @@ def check_xml_templates(self):
         Indentify nodes incompatible with Prettier XML auto-fix generating possible unexpected text insertion
         """
         template_ids: Dict[str, List[FileElementPair]] = defaultdict(list)
+        pattern = re.compile(rb"<span>(.*?)</span>")
         for manifest_data in self.manifest_datas:
             for template in self.xpath_template(manifest_data["node"]):
                 if self.is_message_enabled(
@@ -811,6 +813,25 @@ def check_xml_templates(self):
                     if not template_id:  # pragma: no cover
                         continue
                     template_ids[template_id].append(FileElementPair(manifest_data["filename_short"], template))
+                if self.is_message_enabled("xml-superfluous-attributeless", manifest_data["disabled_checks"]):
+                    for node_attrless in template.xpath(".//span[not(@*)]"):
+                        self.register_error(
+                            code="xml-superfluous-attributeless",
+                            message=f"Remove superfluous attributeless `<{node_attrless.tag}`",
+                            info=f"Serve no purpose and cause formatting inconsistencies with Prettier",
+                            filepath=manifest_data["filename_short"],
+                            line=node_attrless.sourceline,
+                        )
+                        if self.autofix:
+                            node_content = node_xml.NodeContent(manifest_data["filename"], node_attrless)
+                            new_content_node = pattern.sub(rb"\1", node_content.content_node, count=1)
+                            if new_content_node != node_content.content_node:
+                                # Modify the record attrib to propagate the change to other checks
+                                node_content.content_node = new_content_node
+                                utils.perform_fix(manifest_data["filename"], bytes(node_content))
+                                # TODO: check if the "for" affects updating the nodes
+                                self.update_node(manifest_data)  # update sourceline after delete a node
+
                 if self.is_message_enabled("xml-template-prettier-incompatible", manifest_data["disabled_checks"]):
                     self.verify_template_prettier_incompatible(template, manifest_data)
 
diff --git a/tests/test_checks.py b/tests/test_checks.py
index 523af715..edf8ae80 100644
--- a/tests/test_checks.py
+++ b/tests/test_checks.py
@@ -56,6 +56,7 @@
     "xml-deprecated-oe-chatter": 1,
     "xml-field-bool-without-eval": 2,
     "xml-field-numeric-without-eval": 7,
+    "xml-superfluous-attributeless": 5,
 }
 
 

From 7ba3aabc7f6d5301adaab4ffb83237a31e4891b7 Mon Sep 17 00:00:00 2001
From: "Moises Lopez - https://www.vauxoo.com/" <moylop260@vauxoo.com>
Date: Sun, 25 Jan 2026 11:09:17 -0600
Subject: [PATCH 4/4] fixup

---
 src/oca_pre_commit_hooks/checks_odoo_module_xml.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
index bac187ab..ed4e507d 100644
--- a/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
+++ b/src/oca_pre_commit_hooks/checks_odoo_module_xml.py
@@ -670,7 +670,7 @@ def verify_template_prettier_incompatible(self, template, manifest_data):
                             "incompatible for Prettier XML auto-fix. To prevent unexpected text insertion "
                             f"prefer `<{node_wrapper.tag} {found_attr}=...>` (move attribute to parent) or "
                             "using 'style' attribute instead of tag "
-                            "e.g. <tag style=\"font-weight: bold\">Black Text... instead of <b><tag..."
+                            'e.g. <tag style="font-weight: bold">Black Text... instead of <b><tag...'
                         ),
                         filepath=manifest_data["filename_short"],
                         line=node_wrapper_child.sourceline,