[IMP] real_estate: add action methods and models constraints

Author: Oussema Heni

estate/models/estate_property.py

@@ -1,11 +1,23 @@
 from odoo import fields, models, api
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools.float_utils import float_compare, float_is_zero
 from datetime import timedelta
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Estate property"
 
+    _check_expected_price = models.Constraint(
+        "CHECK(expected_price > 0)",
+        "The expected price must be strictly positive.",
+    )
+
+    _check_selling_price = models.Constraint(
+        "CHECK(selling_price >= 0)",
+        "The selling price must be positive.",
+    )
+
     name = fields.Char('Name', required=True, default='My new house')
     description = fields.Text('Description')
     active = fields.Boolean(default=True)
@@ -115,3 +127,34 @@ def _inverse_date_deadline(self):
                 record.validity = (
                     record.date_deadline - record.create_date.date()
                 ).days
+
+    def action_cancel(self):
+        for record in self:
+            if record.state == "sold":
+                raise UserError("A sold property cannot be cancelled.")
+            record.state = "cancelled"
+        return True
+
+    def action_sold(self):
+        for record in self:
+            if record.state == "cancelled":
+                raise UserError("A cancelled property cannot be sold.")
+            record.state = "sold"
+        return True
+
+    @api.constrains("selling_price", "expected_price")
+    def _check_selling_price(self):
+        for record in self:
+            if float_is_zero(record.selling_price, precision_rounding=0.01):
+                continue
+
+            min_price = record.expected_price * 0.9
+
+            if float_compare(
+                record.selling_price,
+                min_price,
+                precision_rounding=0.01
+            ) < 0:
+                raise ValidationError(
+                    "The selling price cannot be lower than 90% of the expected price."
+                )

estate/models/estate_property_offer.py

@@ -6,7 +6,13 @@ class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Estate Property Offer"
 
+    _check_offer_price = models.Constraint(
+        "CHECK(price > 0)",
+        "The offer price must be strictly positive.",
+    )
+
     price = fields.Float()
+
     status = fields.Selection(
         [
             ('accepted', 'Accepted'),
@@ -61,3 +67,22 @@ def _inverse_date_deadline(self):
                 record.validity = (
                     record.date_deadline - record.create_date.date()
                 ).days
+
+    def action_accept(self):
+        for record in self:
+            other_offers = record.property_id.offer_ids - record
+            other_offers.write({"status": "refused"})
+
+            record.status = "accepted"
+
+            record.property_id.write({
+                "buyer_id": record.partner_id.id,
+                "selling_price": record.price,
+                "state": "offer_accepted",
+            })
+        return True
+
+    def action_refuse(self):
+        for record in self:
+            record.status = "refused"
+        return True

estate/models/estate_property_tag.py

@@ -5,4 +5,9 @@ class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Estate Property Tag"
 
+    _unique_tag_name = models.Constraint(
+        "UNIQUE(name)",
+        "The property tag name must be unique.",
+    )
+
     name = fields.Char(required=True)

estate/models/estate_property_type.py

@@ -5,4 +5,9 @@ class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Estate property type"
 
+    _unique_type_name = models.Constraint(
+        "UNIQUE(name)",
+        "The property type name must be unique.",
+    )
+
     name = fields.Char('Name', required=True)

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
 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,estate.property.offer,model_estate_property_offer,,1,1,1,1
+access_estate_property_offer,estate.property.offer,model_estate_property_offer,base.group_user,1,1,1,1

estate/views/estate_property_offer_views.xml

@@ -1,34 +1,65 @@
 <?xml version="1.0"?>
 <odoo>
-    <record id="view_estate_property_offer_tree" model="ir.ui.view">
-        <field name="name">estate.property.offer.tree</field>
-        <field name="model">estate.property.offer</field>
-        <field name="arch" type="xml">
-            <list>
-                <field name="price"/>
-                <field name="partner_id"/>
-                <field name="status"/>
-                <field name="validity"/>
-                <field name="date_deadline"/>
-            </list>
-        </field>
-    </record>
+  <record id="view_estate_property_offer_tree" model="ir.ui.view">
+    <field name="name">estate.property.offer.tree</field>
+    <field name="model">estate.property.offer</field>
+    <field name="arch" type="xml">
+      <list>
+        <field name="price" />
+        <field name="partner_id" />
+        <field name="status" />
+        <field name="validity" />
+        <button
+          title="accept-btn"
+          name="action_accept"
+          type="object"
+          icon="fa-check"
+        />
+        <button
+          title="refuse-btn"
+          name="action_refuse"
+          type="object"
+          icon="fa-times"
+        />
+        <field name="date_deadline" />
+      </list>
+    </field>
+  </record>
 
-    <record id="view_estate_property_offer_form" model="ir.ui.view">
-        <field name="name">estate.property.offer.form</field>
-        <field name="model">estate.property.offer</field>
-        <field name="arch" type="xml">
-            <form>
-                <sheet>
-                    <group>
-                        <field name="price"/>
-                        <field name="partner_id"/>
-                        <field name="validity"/>
-                        <field name="date_deadline"/>
-                        <field name="status"/>
-                    </group>
-                </sheet>
-            </form>
-        </field>
-    </record>
+  <record id="view_estate_property_offer_form" model="ir.ui.view">
+    <field name="name">estate.property.offer.form</field>
+    <field name="model">estate.property.offer</field>
+    <field name="arch" type="xml">
+      <form>
+        <header>
+          <button
+            name="action_accept"
+            type="object"
+            string="Accept"
+            icon="fa-check"
+            invisible="status == 'accepted'"
+            title="accept-form-btn"
+          />
+
+          <button
+            name="action_refuse"
+            type="object"
+            string="Refuse"
+            icon="fa-times"
+            invisible="status == 'refused'"
+            title="reject-form-btn"
+          />
+        </header>
+        <sheet>
+          <group>
+            <field name="price" />
+            <field name="partner_id" />
+            <field name="validity" />
+            <field name="date_deadline" />
+            <field name="status" />
+          </group>
+        </sheet>
+      </form>
+    </field>
+  </record>
 </odoo>

estate/views/estate_property_views.xml

@@ -29,6 +29,28 @@
     <field name="model">estate.property</field>
     <field name="arch" type="xml">
       <form string="Estate Property">
+        <header>
+          <button
+            name="action_sold"
+            type="object"
+            string="Sold"
+            class="btn-primary"
+            invisible="state in ('sold', 'cancelled')"
+          />
+
+          <button
+            name="action_cancel"
+            type="object"
+            string="Cancel"
+            invisible="state in ('sold', 'cancelled')"
+          />
+
+          <field
+            name="state"
+            widget="statusbar"
+            statusbar_visible="new,offer_received,offer_accepted,sold,cancelled"
+          />
+        </header>
         <sheet>
           <div class="oe_title mb32">
             <h1>