[MIG] web_portal_properties: Migration to 18.0

Author: etobella

web_portal_properties/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Web Portal Properties",
     "summary": """Add a new field on properties to show them on portal""",
-    "version": "17.0.1.0.0",
+    "version": "18.0.1.0.0",
     "license": "AGPL-3",
     "author": "Dixmit,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/web",
@@ -16,5 +16,8 @@
             "web_portal_properties/static/src/**/*.esm.js",
             "web_portal_properties/static/src/**/*.xml",
         ],
+        "web.assets_unit_tests": [
+            "web_portal_properties/static/tests/**/*.test.js",
+        ],
     },
 }

web_portal_properties/static/src/components/portal_properties/portal_properties.esm.js

@@ -1,8 +1,7 @@
-/** @odoo-module **/
 import {PortalPropertyDefinition} from "./portal_properties_definition.esm";
 import {PropertiesField} from "@web/views/fields/properties/properties_field";
 import {_t} from "@web/core/l10n/translation";
-import {archParseBoolean} from "@web/views/utils";
+import {exprToBoolean} from "@web/core/utils/strings";
 import {registry} from "@web/core/registry";
 import {usePopover} from "@web/core/popover/popover_hook";
 
@@ -30,11 +29,12 @@ export const portalPropertiesField = {
     component: PortalPropertiesField,
     displayName: _t("Properties"),
     supportedTypes: ["properties"],
+    additionalClasses: ["o_field_properties"],
     extractProps({attrs}, dynamicInfo) {
         return {
             context: dynamicInfo.context,
             columns: parseInt(attrs.columns || "1", 10),
-            showAddButton: archParseBoolean(attrs.showAddButton),
+            showAddButton: exprToBoolean(attrs.showAddButton),
         };
     },
 };

web_portal_properties/static/src/components/portal_properties/portal_properties_definition.xml

@@ -9,13 +9,13 @@
             <div class="o_field_property_definition_portal d-contents mb-3 mb-sm-0">
                 <label
                     t-att-for="getUniqueDomID('portal')"
-                    class="o_form_label align-self-center text-900"
+                    class="o_form_label align-self-center text-900 o_property_display_in_portal"
                 >
                     Display in Portal
                     <sup
-                        class="text-info"
-                        title="Whether or not this Property Field is displayed in the Portal views"
-                    >?</sup>
+                    class="text-info"
+                    title="Whether or not this Property Field is displayed in the Portal views"
+                >?</sup>
                 </label>
                 <CheckBox
                     id="getUniqueDomID('portal')"

web_portal_properties/static/tests/properties.esm.test.js

@@ -0,0 +1,84 @@
+import {animationFrame, click, expect, runAllTimers, test} from "@odoo/hoot";
+import {
+    defineModels,
+    fields,
+    models,
+    mountView,
+    onRpc,
+    toggleActionMenu,
+} from "@web/../tests/web_test_helpers";
+import {defineMailModels} from "@mail/../tests/mail_test_helpers";
+
+class Partner extends models.Model {
+    display_name = fields.Char();
+    properties = fields.Properties({
+        string: "Properties",
+        searchable: false,
+        definition_record: "company_id",
+        definition_record_field: "definitions",
+    });
+    company_id = fields.Many2one({
+        string: "Company",
+        relation: "properties.definition.model",
+    });
+    _records = [
+        {
+            id: 1,
+            display_name: "first partner",
+            properties: {
+                property_1: "char value",
+                property_2: "b",
+            },
+            company_id: 37,
+        },
+    ];
+}
+
+class PropertiesDefinitionModel extends models.Model {
+    _name = "properties.definition.model";
+    name = fields.Char({string: "Name"});
+    definitions = fields.PropertiesDefinition();
+    _records = [
+        {
+            id: 37,
+            name: "Company 1",
+            definitions: [],
+        },
+    ];
+}
+
+defineModels([Partner, PropertiesDefinitionModel]);
+defineMailModels();
+test("Portal Properties: check field", async () => {
+    onRpc("has_access", () => true);
+    await mountView({
+        type: "form",
+        resModel: "partner",
+        resId: 1,
+        arch: `
+                <form>
+                    <sheet>
+                        <group>
+                            <field name="company_id"/>
+                            <field name="properties" widget="portal_properties"/>
+                        </group>
+                    </sheet>
+                </form>`,
+        actionMenus: {},
+    });
+    expect(".o_field_properties").toHaveCount(1);
+    await toggleActionMenu();
+    await animationFrame();
+    expect(".o-dropdown--menu span:contains(Add Properties)").toHaveCount(1, {
+        message: "The add button must be in the cog menu",
+    });
+    await click(".o-dropdown--menu span .fa-cogs");
+    await runAllTimers();
+    await animationFrame();
+    expect(".o_property_field_popover").toHaveCount(1, {
+        message: "Should have opened the definition popover",
+    });
+    expect(".o_property_field_popover .o_property_display_in_portal").toHaveCount(1, {
+        message: "The display in portal checkbox should be visible",
+    });
+});

web_portal_properties/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_ui

web_portal_properties/tests/test_ui.py

@@ -0,0 +1,17 @@
+# Copyright 2026 Dixmit
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+import odoo
+
+from odoo.addons.web.tests.test_js import WebSuite
+
+
+@odoo.tests.tagged("post_install", "-at_install")
+class TestWebPortalProperties(WebSuite):
+    """Test Web Portal Properties"""
+
+    def get_hoot_filters(self):
+        self._test_params = [("+", "@web_portal_properties")]
+        return super().get_hoot_filters()
+
+    def test_automation_oca(self):
+        self.test_unit_desktop()

web_portal_properties/views/portal_templates.xml

@@ -8,7 +8,8 @@
         <div class="row" t-if="properties">
             <t t-foreach="properties" t-as="property">
                 <div class="col-6" t-if="property.get('view_in_portal')">
-                    <strong t-esc="property['string']" /><t>: </t>
+                    <strong t-esc="property['string']" />
+                    <t>: </t>
                     <t t-if="property['value']">
                         <span
                             t-esc="property['value'][1]"