[IMP] estate: change property state upon creation of offer and add properties assigned to salesman in the users form

Author: aurb9

estate/__manifest__.py

@@ -9,6 +9,7 @@
         'views/estate_property_type_views.xml',
         'views/estate_property_views.xml',
         'views/estate_menus.xml',
+        'views/res_users_views.xml',
     ],
     'installable': True,
     'application': True,

estate/models/__init__.py

@@ -1 +1 @@
-from . import property, property_offer, property_tag, property_type
+from . import property, property_offer, property_tag, property_type, res_users

estate/models/property.py

@@ -8,7 +8,7 @@
 class Property(models.Model):
     _name = 'estate.property'
     _description = "Estate property"
-    _order = "id desc"
+    _order = 'id desc'
 
     name = fields.Char(string="Title", required=True)
     description = fields.Text()
@@ -73,13 +73,6 @@ def _onchange_garden(self):
         self.garden_area = 10 if self.garden else 0
         self.garden_orientation = 'north' if self.garden else ''
 
-    @api.onchange('offer_ids')
-    def _onchange_state(self):
-        if not self.offer_ids:
-            return
-
-        self.state = 'offer_received'
-
     @api.constrains('selling_price', 'expected_price')
     def _check_selling_price(self):
         for record in self:
@@ -107,3 +100,11 @@ def action_set_cancelled(self):
 
         self.state = 'cancelled'
         return True
+
+    @api.ondelete(at_uninstall=False)
+    def _unlink_except_new_or_cancelled(self):
+        for record in self:
+            if record.state not in ['new', 'cancelled']:
+                raise UserError(
+                    self.env._("Only new and cancelled properties can be deleted.")
+                )

estate/models/property_offer.py

@@ -1,6 +1,7 @@
 from dateutil.relativedelta import relativedelta
 
 from odoo import api, fields, models
+from odoo.exceptions import UserError
 
 
 class PropertyOffer(models.Model):
@@ -46,3 +47,15 @@ def action_confirm(self):
     def action_refuse(self):
         self.status = 'refused'
         return True
+
+    @api.model_create_multi
+    def create(self, vals):
+        for record in vals:
+            property = self.env['estate.property'].browse(record['property_id'])
+            min_price = min(property.offer_ids.mapped('price')) if property.offer_ids else 0.0
+            if record['price'] < min_price:
+                raise UserError(self.env._("The offer must be higher than %d.", min_price))
+
+            property.state = 'offer_received'
+
+        return super(PropertyOffer, self).create(vals)

estate/models/res_users.py

@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+
+class ResUsers(models.Model):
+    _inherit = 'res.users'
+
+    property_ids = fields.One2many('estate.property', 'user_id', string="Properties", domain="[('active', '=', 'True')]")

estate/views/res_users_views.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="res_users_view_form_test" model="ir.ui.view">
+        <field name="name">res.users.view.form.inherit.estate</field>
+        <field name="model">res.users</field>
+        <field name="inherit_id" ref="base.view_users_form"/>
+        <field name="arch" type="xml">
+            <notebook position="inside">
+                <page name="Properties">
+                    <field name="property_ids" readonly="True"/>
+                </page>
+            </notebook>
+        </field>
+    </record>
+</odoo>