[IMP] estate: adding CRUD rules for properties and offers + adding properties list in user form

aizu-odoo · aizu-odoo · commit 5403072797c4 · 2024-11-25T14:22:11.000+01:00
diff --git a/estate/__manifest__.py b/estate/__manifest__.py
@@ -10,5 +10,7 @@
         'views/estate_property_tag_views.xml',
         'views/estate_property_views.xml',
         'views/estate_menu_views.xml',
-    ]
+        'views/res_users_views.xml',
+    ],
+    'license': 'OEEL-1',
 }
diff --git a/estate/models/__init__.py b/estate/models/__init__.py
@@ -2,3 +2,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
+from . import res_users
diff --git a/estate/models/estate_property.py b/estate/models/estate_property.py
@@ -1,4 +1,4 @@
-from odoo import api, fields, models
+from odoo import api, fields, models, _
 from odoo.exceptions import UserError, ValidationError
 from odoo.tools.float_utils import float_compare, float_is_zero
 from dateutil.relativedelta import relativedelta
@@ -45,7 +45,7 @@ class EstateProperty(models.Model):
         default='new'
     )
 
-    property_type_id = fields.Many2one('estate.property.type', string="Property Type", options="{'no_create':True}")
+    property_type_id = fields.Many2one('estate.property.type', string="Property Type")
     buyer_id = fields.Many2one('res.partner', string="Buyer", copy=False)
     salesperson_id = fields.Many2one('res.users', string="Sales Person", default=lambda self: self.env.user)
     property_tag_ids = fields.Many2many('estate.property.tag')
@@ -65,6 +65,7 @@ def _check_selling_price(self):
             if not float_is_zero(record.selling_price, precision_digits=2) and float_compare(record.selling_price, record.expected_price * 0.9, precision_digits=2) <= 0:
                 raise ValidationError("The selling price cannot be lower than 90 percent of the expected price.")
 
+
     @api.depends('living_area', 'garden_area')
     def _compute_tot_area(self):
         for record in self:
@@ -75,6 +76,13 @@ def _compute_best_price(self):
         for record in self:
             record.best_price = max(record.property_offer_ids.mapped('price'), default=0)
 
+
+    @api.ondelete(at_uninstall=False)
+    def _prevent_property_deletion(self):
+        for record in self:
+            if record.state not in ('new', 'cancelled'):
+                raise UserError(_("Only new or cancelled properties can be deleted."))
+
     @api.onchange('garden')
     def _onchange_garden(self):
         if self.garden:
@@ -89,14 +97,13 @@ def action_set_sold(self):
             if record.state != 'cancelled':
                 record.state = 'sold'
             else:
-                raise UserError("Sold properties cannot be cancelled.")
+                raise UserError(_("Sold properties cannot be cancelled."))
         return True
 
     def action_set_cancelled(self):
         for record in self:
             if record.state != 'sold':
                 record.state = 'cancelled'
             else:
-                raise UserError("Sold properties cannot be cancelled.")
+                raise UserError(_("Sold properties cannot be cancelled."))
         return True
-
diff --git a/estate/models/estate_property_offer.py b/estate/models/estate_property_offer.py
@@ -1,4 +1,5 @@
-from odoo import api, fields, models
+from odoo import api, fields, models, _
+from odoo.exceptions import UserError
 from dateutil.relativedelta import relativedelta
 
 
@@ -27,6 +28,14 @@ def _compute_date_deadline(self):
             start_date = record.create_date or fields.Date.today()
             record.date_deadline = start_date + relativedelta(days=record.validity)
 
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            if self.price < self.env['estate.property'].browse(vals['property_id']).best_price:
+                raise UserError(_("You cannot put in an offer that is lower than the current best price."))
+            self.env['estate.property'].browse(vals['property_id']).state = 'offer_received'
+        return super(EstatePropertyOffer, self).create(vals_list)
+
     def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - record.create_date.date()).days
diff --git a/estate/models/estate_property_type.py b/estate/models/estate_property_type.py
@@ -7,17 +7,17 @@ class EstatePropertyType(models.Model):
     _order = 'sequence'
 
     name = fields.Char(required=True)
-    sequence = fields.Integer('Sequence')
+    sequence = fields.Integer()
     property_ids = fields.One2many('estate.property', 'property_type_id')
     offer_ids = fields.One2many('estate.property.offer', 'property_type_id')
-    offer_count = fields.Integer(compute="_compute_count_offers")
+    offer_count = fields.Integer(compute="_compute_offer_count")
 
     _sql_constraints = [
         ('unique_property_type_name_check', 'UNIQUE(name)',
          'Type name should be unique.'),
     ]
 
     @api.depends('offer_ids')
-    def _compute_count_offers(self):
+    def _compute_offer_count(self):
         for record in self:
             record.offer_count = len(record.offer_ids)
diff --git a/estate/models/res_users.py b/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', 'salesperson_id', domain=[('state', '!=', 'sold')])
diff --git a/estate/security/ir.model.access.csv b/estate/security/ir.model.access.csv
@@ -2,4 +2,4 @@ id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
 estate.access_estate_property,access_estate_property,estate.model_estate_property,base.group_user,1,1,1,1
 estate.access_estate_property_type,access_estate_property_type,estate.model_estate_property_type,base.group_user,1,1,1,1
 estate.access_estate_property_tag,access_estate_property_tag,estate.model_estate_property_tag,base.group_user,1,1,1,1
-estate.access_estate_property_offer,access_estate_property_offer,estate.model_estate_property_offer,base.group_user,1,1,1,1
+estate.access_estate_property_offer,access_estate_property_offer,estate.model_estate_property_offer,base.group_user,1,1,1,1
diff --git a/estate/views/estate_property_type_views.xml b/estate/views/estate_property_type_views.xml
@@ -21,12 +21,13 @@
                     <sheet>
                         <div name="button_box" class="oe_button_box">
                             <button
-                                    type="action"
-                                    name="estate.estate_property_offer_action"
-                                    class="oe_stat_button"
-                                    icon="fa-money"
-                                    string="Offers"
-                                />
+                                type="action"
+                                name="estate.estate_property_offer_action"
+                                class="oe_stat_button"
+                                icon="fa-money"
+                            >
+                                <field name="offer_count" string="Offers" widget="statinfo"/>
+                            </button>
                         </div>
                         <h1>
                             <field name="name"/>
diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
@@ -8,8 +8,8 @@
             <field name="arch" type="xml">
                 <list
                     string="Properties"
-                    decoration-success="state == 'offer_received'"
-                    decoration-primary="state == 'offer_accepted'"
+                    decoration-success="state in ('offer_received', 'offer_accepted')"
+                    decoration-bf="state == 'offer_accepted'"
                     decoration-muted="state == 'sold'">
                     <field name="name"/>
                     <field name="property_type_id"/>
@@ -58,7 +58,7 @@
                         />
                         <group>
                             <group>
-                                <field name="property_type_id"/>
+                                <field name="property_type_id" options="{'no_create':True}"/>
                                 <field name="postcode"/>
                                 <field name="date_availability"/>
                             </group>
@@ -86,8 +86,7 @@
                                 <field 
                                     name="property_offer_ids"
                                     readonly="state in ('offer_accepted','sold','cancelled')"
-                                >
-                                </field>
+                                />
                             </page>
                             <page string="Other Info">
                                 <group>
diff --git a/estate/views/res_users_views.xml b/estate/views/res_users_views.xml
@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<odoo>
+    <data>
+
+        <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="//page[@name='account_security']" position="after">
+                     <page string="Real Estate Properties">
+                        <field name="property_ids"/>
+                    </page>
+                </xpath>
+            </field>
+        </record>
+
+    </data>
+</odoo>

Original file line number	Diff line number	Diff line change
`@@ -10,5 +10,7 @@`
`10`	`10`	`'views/estate_property_tag_views.xml',`
`11`	`11`	`'views/estate_property_views.xml',`
`12`	`12`	`'views/estate_menu_views.xml',`
`13`		`- ]`
	`13`	`+ 'views/res_users_views.xml',`
	`14`	`+ ],`
	`15`	`+ 'license': 'OEEL-1',`
`14`	`16`	`}`