[IMP] Estate: Adding inheritance

[FIX] super()

Author: vicire1

estate/__manifest__.py

@@ -5,21 +5,14 @@
     'author': "Victor Decleire",
     'application': True,
     'license': "LGPL-3",
-    # 'description': """
-    # Description text
-    # """,
-    # # data files always loaded at installation
     'data': [
         'security/ir.model.access.csv',
 
         'views/estate_property_offer_views.xml',
+        'views/inherited_model_views.xml',
         'views/estate_property_views.xml',
         'views/estate_property_tag_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_menus.xml',
     ],
-    # # data files containing optionally loaded demonstration data
-    # 'demo': [
-    #     'demo/demo_data.xml',
-    # ],
 }

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 inherited_model

estate/models/estate_property.py

@@ -35,7 +35,7 @@ class EstateProperty(models.Model):
         selection=[('new', 'New'), ('received', 'Offer Received'), ('accepted', 'Offer Accepted'), ('sold', 'Sold'), ('cancelled', 'Cancelled')],
         default='new', copy=False, required=True)
     seller_id = fields.Many2one('res.users', default=lambda self: self.env.user)
-    buyer_id = fields.Many2one('res.partner')
+    buyer_id = fields.Many2one('res.partner', readonly=True)
     tag_ids = fields.Many2many('estate.property.tag', string="Tags")
     type_id = fields.Many2one('estate.property.type', string="Type")
     offer_ids = fields.One2many('estate.property.offer', 'property_id')
@@ -82,5 +82,10 @@ def action_set_cancelled(self):
     @api.constrains('selling_price', 'expected_price')
     def _check_selling_price(self):
         self.ensure_one()
-        if len(self.offer_ids) > 0 and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
+        if self.buyer_id and float_compare(self.expected_price, 0.9 * self.selling_price, 0) > 0:
             raise ValidationError("The selling price cannot be lower than 90% of the expected price.")
+
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_not_new_or_cancelled(self):
+        if any(record.state in ['new', 'cancelled'] for record in self):
+            raise UserError("Can't delete a property which has a state of new or cancelled!")

estate/models/estate_property_offer.py

@@ -1,5 +1,7 @@
 from odoo import fields, models, api
 from odoo.tools.date_utils import add
+from odoo.exceptions import ValidationError
+from odoo.tools.float_utils import float_compare
 
 
 class EstatePropertyOffer(models.Model):
@@ -40,3 +42,13 @@ def action_refuse_offer(self):
         for record in self:
             record.status = 'refused'
         return 1
+
+    @api.model
+    def create(self, vals):
+        for val in vals:
+            created_property = self.env['estate.property'].browse(val['property_id'])
+            if any(float_compare(val['price'], offer.price, 0) < 0 for offer in created_property.offer_ids):
+                raise ValidationError("A bigger offer already exists")
+            if created_property.state == 'new':
+                created_property.state = 'received'
+        return super().create(vals)

estate/models/inherited_model.py

@@ -0,0 +1,7 @@
+from odoo import fields, models
+
+
+class InheritedModel(models.Model):
+    _inherit = "res.users"
+
+    property_ids = fields.One2many("estate.property", "seller_id", domain="['|', ('state', '=', 'new'), ('state', '=', 'received')]")

estate/views/inherited_model_views.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <record id="res_users_view_form" model="ir.ui.view">
+        <field name="name">res.users.view.form.inherit</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='page_security']" position="after">
+                <page string="Real Estate Properties">
+                    <field name="property_ids"/>
+                </page>
+            </xpath>
+        </field>
+    </record>
+</odoo>