[IMP] Real Estate: Added some constraints to the price

Author: Astremy

estate/models/estate_property.py

@@ -92,7 +92,7 @@ def _compute_property_livable(self):
     def _compute_best_offer(self):
         for property in self:
             property.best_offer = max([0, *property.offer_ids.mapped("translated_price")])
-    
+
     def action_set_as_cancelled(self):
         for property in self:
             if property.stage in ["sold", "cancelled"]:
@@ -126,3 +126,13 @@ def action_set_as_sold(self):
         'CHECK(total_area >= 0)',
         'The total area can\'t be negative.',
     )
+
+    _check_expected_price = models.Constraint(
+        'CHECK(expecting_price > 0)',
+        'The expected price has to be stricly positive'
+    )
+
+    _check_selling_price = models.Constraint(
+        'CHECK(selling_price >= 0)',
+        'The selling price has to be stricly positive'
+    )

estate/models/estate_property_offer.py

@@ -10,7 +10,6 @@ class PropertyOffer(models.Model):
     property_id = fields.Many2one("estate.property", string="Property", required=True)
     partner_id = fields.Many2one("res.partner", string="Partner", index=True, required=True)
 
-
     # Beginning of the deadline part
     def _current_date(self):
         return fields.Date.today()
@@ -35,7 +34,6 @@ def _inverse_validity(self):
 
     # End of the deadline part
 
-
     # Beginning of the currency part
     currency_id = fields.Many2one("res.currency", "Currency")
     property_currency_id = fields.Many2one("res.currency", "Partner Currency", compute="_compute_property_currency_id")
@@ -63,7 +61,6 @@ def _compute_property_currency_id(self):
 
     # End of the currency part
 
-
     # Beginning of the state / validation part
     status = fields.Selection([
         ("accepted", "Accepted"),
@@ -82,8 +79,12 @@ def action_confirm(self):
     def action_refuse(self):
         for offer in self:
             offer.status = "refused"
-    
-    # End of the state / validation part
 
+    # End of the state / validation part
 
     sequence = fields.Integer("Sequence", default=0)
+
+    _check_price = models.Constraint(
+        'CHECK(price > 0)',
+        'The price has to be stricly positive'
+    )

estate/models/estate_tag.py

@@ -38,3 +38,8 @@ def _default_color(self):
     color = fields.Integer(
         string='Color Index', default=lambda self: self._default_color(),
         help='Tag color. No color means no display in kanban or front-end, to distinguish internal tags from public categorization tags.')
+
+    _unique_name = models.Constraint(
+        'UNIQUE(name)',
+        'The tag name has to be unique',
+    )