[IMG] estate: Chapter 12: Inheritance

Author: youness benbraitit (yoben)

estate/__manifest__.py

@@ -18,6 +18,7 @@
         'views/estate_list.xml',
         'views/estate_view_form.xml',
         'views/estate_view_search.xml',
+        'views/estate_res_users_views.xml',
         'views/estate_menus.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 res_users

estate/models/estate_property.py

@@ -68,20 +68,6 @@ class EstateProperty(models.Model):
         "Best Offer", compute="_compute_best_price", readonly=True
     )
 
-    # 💡 REPLACED deprecated models.Constraint with _sql_constraints
-    _sql_constraints = [
-        (
-            'check_expected_price',
-            'CHECK(expected_price > 0)',
-            'The expected price must be strictly positive.',
-        ),
-        (
-            'check_selling_price',
-            'CHECK(selling_price >= 0)',
-            'The selling price must be positive or zero.',
-        ),
-    ]
-
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
         for rec in self:
@@ -135,3 +121,9 @@ def _check_selling_price_constraint(self):
     _check_selling_price = models.Constraint(
         'CHECK(selling_price >= 0)', 'The selling price must be positive or zero.'
     )
+
+    @api.ondelete(at_uninstall=False)
+    def _ondelete_check_state(self):
+        for prop in self:
+            if prop.state not in ('new', 'cancelled'):
+                raise UserError('Only properties in New or Cancelled state can be deleted.')

estate/models/estate_property_offer.py

@@ -1,4 +1,6 @@
 from odoo import api, fields, models
+from odoo.exceptions import ValidationError
+from odoo.tools.float_utils import float_compare
 from datetime import timedelta
 
 
@@ -57,3 +59,28 @@ def action_refuse(self):
         for offer in self:
             offer.state = 'refused'
         return True
+
+    @api.model
+    def create(self, vals_list):
+        if isinstance(vals_list, dict):
+            vals_list = [vals_list]
+
+        for vals in vals_list:
+            prop_id = vals.get('property_id')
+            price = vals.get('price')
+            if prop_id and price is not None:
+                prop = self.env['estate.property'].browse(prop_id)
+                existing_prices = prop.offer_ids.mapped('price')
+                if existing_prices:
+                    best = max(existing_prices)
+                    if float_compare(price, best, precision_digits=2) < 0:
+                        raise ValidationError(
+                            'You cannot create an offer lower than an existing offer.'
+                        )
+
+        offers = super(EstatePropertyOffer, self).create(vals_list)
+
+        for offer in offers:
+            if offer.property_id:
+                offer.property_id.state = 'received'
+        return offers

estate/models/res_users.py

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

estate/views/estate_res_users_views.xml

@@ -0,0 +1,22 @@
+<odoo>
+  <data>
+    <record id="view_users_form_inherit_estate_properties" model="ir.ui.view">
+      <field name="name">res.users.form.inherit.estate.properties</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="Properties">
+            <field name="property_ids">
+              <tree editable="bottom">
+                <field name="name"/>
+                <field name="state"/>
+                <field name="expected_price"/>
+              </tree>
+            </field>
+          </page>
+        </xpath>
+      </field>
+    </record>
+  </data>
+</odoo>