[FIX] price_security_sale_margin: enhance view cache key handling and update invisible fields logic

Author: lef-adhoc

price_security_sale_margin/models/sale_order.py

@@ -32,5 +32,7 @@ def _get_view(self, view_id=None, view_type="form", **options):
                     node.set("invisible", "1")
                     modifiers = json.loads(node.get("modifiers") or "{}")
                     modifiers["invisible"] = True
+                    node.set("column_invisible", "1")
+                    modifiers["column_invisible"] = True
                     node.set("modifiers", json.dumps(modifiers))
         return arch, view

price_security_sale_margin/models/sale_order_line.py

@@ -21,7 +21,7 @@ def _get_view_cache_key(self, view_id=None, view_type="form", **options):
     @api.model
     def _get_view(self, view_id=None, view_type="form", **options):
         arch, view = super()._get_view(view_id, view_type, **options)
-        if view_type == "tree":
+        if view_type == "list":
             if self.env.user.has_group("price_security.group_only_view"):
                 readonly_fields = (
                     arch.xpath("//field[@name='purchase_price']")