[IMP] report_partner_address: remove sync address_details from parent

Author: AungKoKoLin1997

report_partner_address/README.rst

@@ -51,6 +51,9 @@ Details.
 
 If Address Details is left blank, the standard address format is used.
 
+Note: Address Details does not sync to child partners like Odoo’s
+standard address fields.
+
 Bug Tracker
 ===========
 

report_partner_address/models/res_partner.py

@@ -1,52 +1,18 @@
 # Copyright 2025 Quartile (https://www.quartile.co)
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo import api, fields, models
+from odoo import fields, models
 
 
 class ResPartner(models.Model):
     _inherit = "res.partner"
 
     address_details = fields.Text(
-        compute="_compute_address_details",
-        store=True,
-        readonly=False,
         translate=True,
-        recursive=True,
         help="Custom address format used in reports and portal pages. "
         "When set, this replaces the standard address format for external display.",
     )
 
-    @api.depends("parent_id.address_details", "type")
-    def _compute_address_details(self):
-        for partner in self:
-            if partner.parent_id and partner.type == "contact":
-                partner.address_details = partner.parent_id.address_details
-                # No translation sync here to avoid unnecessary DB operations
-
-    @api.model
-    def _address_fields(self):
-        """For address_details to be synced from parent."""
-        return super()._address_fields() + ["address_details"]
-
-    def _fields_sync(self, values):
-        """Copy address_details translations from parent for contacts."""
-        result = super()._fields_sync(values)
-        parent = self.parent_id
-        if parent and self.type == "contact" and parent.address_details:
-            # Copy JSONB translations directly to avoid recursion
-            self.env.cr.execute(
-                """
-                UPDATE res_partner
-                SET address_details = p.address_details
-                FROM res_partner p
-                WHERE res_partner.id = %(partner_id)s
-                AND p.id = %(parent_id)s
-                """,
-                {"partner_id": self.id, "parent_id": parent.id},
-            )
-        return result
-
     def _prepare_display_address(self, without_company=False):
         self.ensure_one()
         address_format, args = super()._prepare_display_address(

report_partner_address/readme/USAGE.md

@@ -4,3 +4,5 @@
 The address in reports and portal pages will use the value from Address Details.
 
 If Address Details is left blank, the standard address format is used.
+
+Note: Address Details does not sync to child partners like Odoo’s standard address fields.

report_partner_address/static/description/index.html

@@ -397,6 +397,8 @@ <h1><a class="toc-backref" href="#toc-entry-1">Usage</a></h1>
 <p>The address in reports and portal pages will use the value from Address
 Details.</p>
 <p>If Address Details is left blank, the standard address format is used.</p>
+<p>Note: Address Details does not sync to child partners like Odoo’s
+standard address fields.</p>
 </div>
 <div class="section" id="bug-tracker">
 <h1><a class="toc-backref" href="#toc-entry-2">Bug Tracker</a></h1>

report_partner_address/tests/test_report_partner_address.py

@@ -55,27 +55,24 @@ def test_address_details_empty_uses_standard_format(self):
         self.assertIn("123 Main St", res)
         self.assertIn("Springfield", res)
 
-    def test_contact_inherits_address_details_from_parent(self):
+    def test_contact_does_not_inherit_address_details(self):
         contact = self.env["res.partner"].create(
             {"name": "Contact Person", "type": "contact", "parent_id": self.partner.id}
         )
-        self.assertEqual(
-            contact.address_details,
-            "Test Company\n1-2-3 Jingumae, Shibuya-ku\nTokyo 150-0001\nJapan",
+        res = contact.with_context(lang="ja_JP")._display_address()
+        # Should use standard format of parent address since address_details is empty
+        self.assertEqual(res, "150-0001\n Shibuya-ku, Tokyo\n1-2-3 Jingumae\n\n日本")
+        # Contact address_details should be empty (do not inherit from the parent)
+        self.assertFalse(contact.address_details)
+        contact.with_context(lang="ja_JP").write(
+            {"address_details": "〒150-0001\n東京都渋谷区神宮前1-2\nテスト株式会社"}
         )
-        self.assertEqual(
-            contact.with_context(lang="ja_JP").address_details,
-            "〒150-0001\n東京都渋谷区神宮前1-2-3\nテスト株式会社",
+        contact.address_details = (
+            "Test Company\n1-2 Jingumae, Shibuya-ku\nTokyo 150-0001\nJapan"
         )
-
-    def test_non_contact_does_not_inherit_address_details(self):
-        delivery_address = self.env["res.partner"].create(
-            {
-                "name": "Delivery Address",
-                "type": "delivery",
-                "parent_id": self.partner.id,
-                "street": "456 Delivery St",
-            }
+        res = contact.with_context(lang="en_US")._display_address()
+        self.assertEqual(
+            res, "Test Company\n1-2 Jingumae, Shibuya-ku\nTokyo 150-0001\nJapan"
         )
-        # Non-contact address should NOT inherit parent's address_details
-        self.assertFalse(delivery_address.address_details)
+        res = contact.with_context(lang="ja_JP")._display_address()
+        self.assertEqual(res, "〒150-0001\n東京都渋谷区神宮前1-2\nテスト株式会社")

report_partner_address/views/res_partner_views.xml

@@ -5,10 +5,7 @@
         <field name="inherit_id" ref="base.view_partner_form" />
         <field name="arch" type="xml">
             <xpath expr="//div[hasclass('o_address_format')]" position="after">
-                <field
-                    name="address_details"
-                    readonly="type =='contact' and parent_id != False"
-                />
+                <field name="address_details" />
             </xpath>
         </field>
     </record>