[IMP] estate: added constrains and made changes in all views

Added the following constraints to their corresponding models:
A property expected price must be strictly positive
A property selling price must be positive
An offer price must be strictly positive
A property tag name and property type name must be unique

Added a python constraint so that the selling price cannot be lower than
90% of the expected price.

Used the statusbar widget in order to display the state of the estate.property

Added ordering to each model as per requirement.

Added manual ordering to property type field.

Added widget options to color the tag.

Added conditional display of buttons Using the invisible attribute

Made property tag and offers list view editable

Made the field date_availability on the estate.property list view optional and
hidden by default

Added conditional decoration to property and offer list view.

Added availability to the default search filter

Added a stat button to the property type to view offers for each property.

Added kanban view to the estate property

Author: kdes-odoo

estate/__manifest__.py

@@ -14,10 +14,11 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
+        'views/estate_property_offer_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
-        'views/estate_property_offer_views.xml',
+        'views/estate_res_user_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 estate_res_user

estate/models/estate_property.py

@@ -7,9 +7,7 @@ class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Details of all the properties will be stored over here."
     _order = "id desc"
-
-
-    name = fields.Char('Name', required=True,default='Unknown Property')
+    name = fields.Char('Name', required=True, default='Unknown Property')
     description = fields.Text('Description')
     postcode = fields.Char('Postcode')
     date_availability = fields.Date(
@@ -41,18 +39,6 @@ class EstateProperty(models.Model):
     offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers")
     total_area = fields.Float(compute="_compute_total_area", string="Total Area", readonly=True)
     best_price = fields.Float(compute="_compute_best_price", string="Best Price", readonly=True)
-    state = fields.Selection(
-        string="State",
-        selection=[
-            ("new", "New"),
-            ("offer_received", "Offer Received"),
-            ("offer_accepted", "Offer Accepted"),
-            ("sold", "Sold"),
-            ("canceled", "Canceled"),
-        ],
-        default="new",
-        copy=False,
-    )
 
     _sql_constraints = [
         ('check_expected_price', 'CHECK(expected_price > 0)', 'The expected price must strictly Positive.'),
@@ -82,6 +68,7 @@ def _onchange_garden(self):
     def action_sold(self):
         if self.state != "canceled":
             self.state = "sold"
+            print("Log by KDES", flush=True)
         else:
             raise UserError("Canceled properties can't be sold")
         return True
@@ -103,3 +90,7 @@ def _check_price(self):
             if float_utils.float_compare(record.selling_price, record.expected_price * 0.9, precision_rounding=3) == -1:
                 raise ValidationError(_('The selling cannot be lower than 90% of the expected price.'))
 
+    @api.ondelete(at_uninstall=False)
+    def _unlink_if_state_check(self):
+        if any(record.state not in ('new', 'canceled') for record in self):
+            raise UserError(_("You cannot delete a property that is not in the 'New' or 'Canceled' state."))

estate/models/estate_property_offer.py

@@ -1,4 +1,4 @@
-from odoo import api, fields, models, api
+from odoo import api, fields, models, _
 from odoo.exceptions import UserError
 
 
@@ -23,7 +23,7 @@ class EstatePropertyOffer(models.Model):
     _sql_constraints = [
         ("check_price", "CHECK(price > 0)", "The price must be strictly positive.")
         ]
-    
+
     @api.depends("validity")
     def _compute_date_deadline(self):
         for record in self:
@@ -39,15 +39,6 @@ def _inverse_date_deadline(self):
         for record in self:
             record.validity = (record.date_deadline - record.create_date.date()).days
 
-    def action_accept_offer(self):
-        self.ensure_one()
-        self.status = "accepted"
-        self.property_id.selling_price = self.price
-        self.property_id.buyer_id = self.partner_id
-        self.property_id.state = "offer_accepted"
-        return True
-
-
     def action_accept_offer(self):
         if self.property_id.state in {'accepted', 'sold'}:
             raise UserError(_('An offer as already been accepted.'))
@@ -63,3 +54,12 @@ def action_refuse_offer(self):
         self.ensure_one()
         self.status = "refused"
         return True
+
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            property = self.env["estate.property"].browse(vals["property_id"])
+            property.state = "offer_received"
+            if property.best_price > vals["price"]:
+                raise UserError("The offer must be higher than the existing offer")
+        return super().create(vals_list)

estate/models/estate_property_tag.py

@@ -11,4 +11,4 @@ class EstatePropertyTag(models.Model):
 
     _sql_constraints = [
         ('check_name', 'UNIQUE(name)', 'A tag must be unique.'),
-    ]
+    ]

estate/models/estate_property_type.py

@@ -1,12 +1,11 @@
 from odoo import models, fields, api
 
+
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Types of properties available in the estate module."
     _order = "name"
 
-
-
     name = fields.Char('Name', required=True)
     property_ids = fields.One2many("estate.property", "property_type_id", string="Property")
     sequence = fields.Integer("Sequence", default=1, help="Used to order types. Lower is better.")
@@ -20,4 +19,4 @@ class EstatePropertyType(models.Model):
     @api.depends('offer_ids')
     def compute_offer_count(self):
         for record in self:
-            record.offer_count = len(record.offer_ids)
+            record.offer_count = len(record.offer_ids)

estate/models/estate_res_user.py

@@ -0,0 +1,8 @@
+from odoo import fields, models
+
+
+class User(models.Model):
+    _inherit = "res.users"
+
+    property_ids = fields.One2many("estate.property", inverse_name="salesperson_id")
+    domain = ["|", ("state", "=", "new"), ("state", "=", "offer_received")]

estate/views/estate_menus.xml

@@ -26,4 +26,4 @@
                 name="Properties Tags"
                 parent="menu_estate_settings"
                 action="action_estate_property_tag"/> 
-</odoo>
+</odoo>

estate/views/estate_property_offer_views.xml

@@ -41,4 +41,4 @@
             </form>
         </field>
     </record>
-</odoo>
+</odoo>

estate/views/estate_property_tag_views.xml

@@ -29,4 +29,4 @@
             </form>
         </field>
     </record>
-</odoo>
+</odoo>