[IMP] estate: add some protection on create/delete operation using inheritance and add a page in User to see Properties linked to salesperson

Author: clom-odoo

estate/__manifest__.py

@@ -9,6 +9,7 @@
         'views/estate_property_tag_views.xml',
         'views/estate_property_views.xml',
         'views/estate_menus.xml',
+        'views/inherited_user.xml',
         'security/ir.model.access.csv',
     ],
     'application': True

estate/models/__init__.py

@@ -2,3 +2,4 @@
 from . import estate_property_tag
 from . import estate_property_offer
 from . import estate_properties
+from . import inherited_user

estate/models/estate_properties.py

@@ -92,3 +92,8 @@ def _check_minimum_sell_price(self):
         for a_property in self:
             if float_compare(a_property.selling_price, a_property.expected_price * 0.9, 2) == -1:
                 raise ValidationError("The seling prce can not be lower than 90% of the expected price")
+
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_new_or_cancelled(self):
+        if any(record.state not in ["new", "cancelled"] for record in self):
+            raise UserError("Property should be New or Cancelled in order to be deleted")

estate/models/estate_property_offer.py

@@ -56,3 +56,12 @@ def action_cancel(self):
                 raise UserError("Offer already accepted")
             offer.status = "refused"
         return True
+
+    @api.model
+    def create(self, vals):
+        property_id = self.env["estate_property"].browse(vals["property_id"])
+        if property_id.state == "new":
+            property_id.state = "offer received"
+        if property_id.best_price > vals["price"]:
+            raise UserError("The price of a new offer can't be below the price of an already existing offer.")
+        return super().create(vals)

estate/models/inherited_user.py

@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+
+class InheritedUser(models.Model):
+    _inherit = "res.users"
+
+    property_ids = fields.One2many("estate_property", inverse_name="salesperson")

estate/views/inherited_user.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="inherited_user_estate_view_form" model="ir.ui.view">
+    <field name="name">inherited.base.view_users_form</field>
+    <field name="model">res.users</field>
+    <field name="inherit_id" ref="base.view_users_form"/>
+    <field name="arch" type="xml">
+        <xpath expr="//page" position="after">
+            <page string="Properties">
+                <field name="property_ids" />
+            </page>
+        </xpath>
+    </field>
+</record>
+</odoo>