[IMP] estate: Implement salesperson management with properties association

Author: raibr

estate/__manifest__.py

@@ -21,6 +21,7 @@
         "views/estate_property_offer_views.xml",
         "views/estate_property_type_views.xml",
         "views/estate_property_tag_views.xml",
+        "views/res_users_views.xml",
         "views/estate_menus.xml",
     ],
     "application": True,

estate/models/__init__.py

@@ -4,3 +4,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
+from . import res_users

estate/models/estate_property.py

@@ -2,7 +2,7 @@
 
 from odoo import api, fields, models
 from odoo.exceptions import UserError, ValidationError
-from odoo.tools.float_utils import float_compare, float_is_zero
+from odoo.tools import float_compare, float_is_zero
 
 
 class EstateProperty(models.Model):
@@ -46,6 +46,7 @@ class EstateProperty(models.Model):
         string="Status",
     )
     buyer_id = fields.Many2one(comodel_name='res.partner', string="Buyer")
+    salesperson_id = fields.Many2one(comodel_name='res.users', string="Salesperson", default=lambda self: self.env.user)
     property_type_id = fields.Many2one(comodel_name='estate.property.type', string="Property Type")
     tag_ids = fields.Many2many(comodel_name='estate.property.tag', string="Tags")
     offer_ids = fields.One2many(
@@ -98,3 +99,9 @@ def action_cancel(self):
                 raise UserError(self.env._("Sold property cannot be cancelled."))
             property.state = 'cancelled'
         return True
+
+    def unlink(self):
+        for property in self:
+            if property.state not in ('new', 'cancelled'):
+                raise UserError(self.env._("Only new and cancelled properties can be deleted."))
+        return super().unlink()

estate/models/estate_property_offer.py

@@ -14,7 +14,7 @@ class EstatePropertyOffer(models.Model):
     _check_price = models.Constraint('Check(price > 0)', "The offer price must be strictly positive.")
     status = fields.Selection(selection=[('accepted', "Accepted"), ('refused', "Refused")], copy=False, string="Status")
     partner_id = fields.Many2one(comodel_name='res.partner', string="Partner", required=True)
-    property_id = fields.Many2one(comodel_name='estate.property', string="Property", required=True)
+    property_id = fields.Many2one(comodel_name='estate.property', string="Property", required=True, ondelete='cascade')
     validity = fields.Integer(default=7, string="Validity (days)")
     date_deadline = fields.Date(
         compute='_compute_date_deadline',
@@ -23,13 +23,24 @@ class EstatePropertyOffer(models.Model):
         string="Deadline",
     )
 
-    # self.property_id.status = 'offer_received' when offer is created
-    @api.model
-    def create(self, vals):
-        offer = super().create(vals)
-        if offer.property_id.state == 'new':
-            offer.property_id.state = 'offer_received'
-        return offer
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            property_id = vals.get('property_id')
+            price = vals.get('price')
+
+            if property_id and price:
+                property = self.env['estate.property'].browse(property_id)
+                if property.offer_ids:
+                    max_offer = max(property.offer_ids.mapped('price'))
+                    if price < max_offer:
+                        raise UserError(self.env._("The offer amount must be higher than %.2f") % max_offer)
+
+        offers = super().create(vals_list)
+        for offer in offers:
+            if offer.property_id.state == 'new':
+                offer.property_id.state = 'offer_received'
+        return offers
 
     @api.depends('validity')
     def _compute_date_deadline(self):

estate/models/res_users.py

@@ -0,0 +1,13 @@
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from odoo import fields, models
+
+
+class ResUsers(models.Model):
+    _inherit = 'res.users'
+
+    property_ids = fields.One2many(
+        comodel_name='estate.property',
+        inverse_name='salesperson_id',
+        string="Properties",
+    )

estate/security/ir.model.access.csv

@@ -2,4 +2,5 @@ id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 access_estate_property,access_estate_property,model_estate_property,base.group_user,1,1,1,1
 access_estate_property_type,access_estate_property_type,model_estate_property_type,base.group_user,1,1,1,1
 access_estate_property_tag,access_estate_property_tag,model_estate_property_tag,base.group_user,1,1,1,1
-access_estate_property_offer,access_estate_property_offer,model_estate_property_offer,base.group_user,1,1,1,1
+access_estate_property_offer,access_estate_property_offer,model_estate_property_offer,base.group_user,1,1,1,1
+access_res_users_estate_salesperson,access_res_users_estate_salesperson,model_res_users,base.group_user,1,1,1,1

estate/views/estate_menus.xml

@@ -28,8 +28,13 @@
               parent="estate_settings_menu" 
               action="estate_property_tag_action"/>
 
-      <menuitem id="estate_property_offer_menu_action" 
+    <menuitem id="estate_property_offer_menu_action" 
               name="Property Offers" 
               parent="estate_settings_menu" 
               action="estate_property_offer_action"/>
+    
+    <menuitem id="estate_salesperson_menu_action" 
+              name="Salespersons" 
+              parent="estate_settings_menu" 
+              action="estate_salesperson_action"/>
 </odoo>

estate/views/estate_property_offer_views.xml

@@ -47,7 +47,5 @@
         <field name="name">Property Offers</field>
         <field name="res_model">estate.property.offer</field>
         <field name="view_mode">list,form</field>
-        <field name="domain">[('property_id.property_type_id', '=', active_id)]</field>
-        <field name="domain">[('property_id.property_type_id', '=', active_id)]</field>
     </record>
 </odoo>

estate/views/estate_property_views.xml

@@ -54,6 +54,7 @@
                             <field name="expected_price" />
                             <field name="best_price" />
                             <field name="selling_price" />
+                            <field name="salesperson_id" />
                             <field name="buyer_id" />
                         </group>
                     </group>

estate/views/res_users_views.xml

@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <record id="res_users_view_form_inherit_estate" 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">
+            <xpath expr="//notebook" position="inside">
+                <page string="Real Estate Properties">
+                    <field name="property_ids">
+                        <list string="Properties">
+                            <field name="name"/>
+                            <field name="postcode"/>
+                            <field name="expected_price"/>
+                            <field name="state"/>
+                        </list>
+                    </field>
+                </page>
+            </xpath>
+        </field>
+    </record>
+
+    <!-- Action -->
+    <record id="estate_salesperson_action" model="ir.actions.act_window">
+        <field name="name">Salespersons</field>
+        <field name="res_model">res.users</field>
+        <field name="view_mode">list,form</field>
+    </record>
+</odoo>