[CLN] estate: code cleanup to match Ruff

Author: VincentJanssen-Code

estate/__init__.py

@@ -1 +1 @@
-from . import models
+from . import models

estate/__manifest__.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 {
     'name':"estate",
     'description':"test",
@@ -16,4 +15,4 @@
         'views/estate_menu_views.xml',
         'security/ir.model.access.csv']
 
-}
+}

estate/models/estate_property.py

@@ -1,72 +1,70 @@
 from dateutil.relativedelta import relativedelta
 from odoo import fields, models,api,exceptions
-from odoo.tools.float_utils import float_compare, float_is_zero;
-from odoo.exceptions import ValidationError;
+from odoo.tools.float_utils import float_compare, float_is_zero
+from odoo.exceptions import ValidationError
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Property"
     _order = "id desc"
 
-    _positif_expected_price = models.Constraint("CHECK (expected_price > 0)","A price can't be negatif");
-    _positif_selling_price = models.Constraint("CHECK (selling_price > 0)","A price can't be negatif");
+    _positif_expected_price = models.Constraint("CHECK (expected_price > 0)", "A price can't be negatif")
+    _positif_selling_price = models.Constraint("CHECK (selling_price > 0)", "A price can't be negatif")
 
-    state = fields.Selection(selection = [("New","New"), ("Offer_Received","Offer Received") ,("Offer_Accepted","Offer Accepted"), ("Sold","Sold"), ("Cancelled","Cancelled")])
-    active = fields.Boolean('Active',default=True)
-    name = fields.Char(required=True,default="Unkown")
+    state = fields.Selection(selection=[("New", "New"), ("Offer_Received", "Offer Received"), ("Offer_Accepted", "Offer Accepted"), ("Sold", "Sold"), ("Cancelled", "Cancelled")])
+    active = fields.Boolean('Active', default=True)
+    name = fields.Char(required=True, default="Unkown")
     description = fields.Text()
     postcode = fields.Char()
-    date_availability = fields.Date("Available From",copy=False,default= fields.Datetime.today() + relativedelta(months=3))
-    last_seen= fields.Date("Last Seen", default=fields.Datetime.now)
+    date_availability = fields.Date("Available From", copy=False, default=fields.Datetime.today() + relativedelta(months=3))
+    last_seen = fields.Date("Last Seen", default=fields.Datetime.now)
     expected_price = fields.Float(required=True)
-    selling_price = fields.Float(readonly=True,copy=False)
-    best_price = fields.Float(string="Best Price",compute="_get_best_price")
+    selling_price = fields.Float(readonly=True, copy=False)
+    best_price = fields.Float(string="Best Price", compute="_get_best_price")
     bedrooms = fields.Integer(default=2)
     living_area = fields.Integer()
     facades = fields.Integer()
     garage = fields.Boolean()
     garden = fields.Boolean()
     garden_area = fields.Integer()
-    garden_orientation = fields.Selection(selection=[('North','North'),('South','South'),('East','East'),('West','West')])
-    total_area = fields.Float(compute="_get_total_area",string="Total Area");
-    property_type_id = fields.Many2one("estate.property.type",string="Type")
-    buyer_id = fields.Many2one("res.partner",string="Buyer")
-    seller_id = fields.Many2one("res.users",default=lambda self : self.env.user,string="Seller")
-    tag_ids = fields.Many2many("estate.property.tag",string="Tags")
-    offer_ids = fields.One2many("estate.property.offer","property_id",string="Offers")
+    garden_orientation = fields.Selection(selection=[('North', 'North'), ('South', 'South'), ('East', 'East'), ('West', 'West')])
+    total_area = fields.Float(compute="_get_total_area", string="Total Area")
+    property_type_id = fields.Many2one("estate.property.type", string="Type")
+    buyer_id = fields.Many2one("res.partner", string="Buyer")
+    seller_id = fields.Many2one("res.users", default=lambda self : self.env.user,string="Seller")
+    tag_ids = fields.Many2many("estate.property.tag", string="Tags")
+    offer_ids = fields.One2many("estate.property.offer", "property_id",string="Offers")
 
     def sell_property(self):
         for record in self:
-            if(record.state == "Cancelled"):
+            if (record.state == "Cancelled"):
                 raise exceptions.UserError("Can't sell cancelled property.")
             record.state = "Sold"
             return True
-        
+
     def cancel_property(self):
         for record in self:
-            if(record.state == "Sold"):
+            if (record.state == "Sold"):
                 raise exceptions.UserError("Can't cancel sold property.")
             record.state = "Cancelled"
             return True
 
-    @api.depends('living_area','garden_area')
+    @api.depends('living_area', 'garden_area')
     def _get_total_area(self):
         for record in self:
-            record.total_area = record.garden_area + record.living_area;
-
+            record.total_area = record.garden_area + record.living_area
 
     @api.depends('offer_ids')
     def _get_best_price(self):
         for record in self:
-            max:int = 0
-
-            if(len(record.offer_ids)==0):
+            max: int = 0
+            if (len(record.offer_ids) == 0):
                 record.best_price = 0
                 continue
-        
+
             for offer in record.offer_ids:
-                if(offer.price >= max):
+                if (offer.price >= max):
                     max = offer.price
                 record.best_price = max
 
@@ -75,10 +73,11 @@ def _garden_pre_fill(self):
         self.garden_area = 10 if self.garden else 0
         self.garden_orientation = 'North' if self.garden else ''
 
-    @api.constrains('selling_price','expected_price')
+    @api.constrains('selling_price', 'expected_price')
     def _check_prices(self):
-            for record in self:
-                if float_is_zero(record.selling_price,2):
-                    continue;
-                if(float_compare(record.selling_price,record.expected_price*.8,2) == -1):
-                    raise ValidationError(f"Selling price is too low {record.selling_price}");
+        for record in self:
+            if float_is_zero(record.selling_price, 2):
+                continue
+            if (float_compare(record.selling_price, record.expected_price * 0.8, 2) == -1):
+                raise ValidationError(f"Selling price is too low {record.selling_price}")
+

estate/models/estate_property_offer.py

@@ -11,37 +11,33 @@ class EstatePropertyOffer(models.Model):
     _postif_price = models.Constraint("CHECK (price > 0)", "A price can't be negatif")
 
     price = fields.Float(string="Price")
-    status = fields.Selection(copy=False,selection = [("Accepted","Accepted"),("Refused","Refused")])
-    partner_id = fields.Many2one('res.partner',required=True)
+    status = fields.Selection(copy=False, selection = [("Accepted", "Accepted"), ("Refused", "Refused")])
+    partner_id = fields.Many2one('res.partner', required=True)
     property_id = fields.Many2one('estate.property', required=True)
-    validity = fields.Integer(string="Validity Duration",default=7)
-    date_deadline = fields.Date(string="Deadline",compute="_get_date_deadline",inverse="_set_date_deadline")
+    validity = fields.Integer(string="Validity Duration", default=7)
+    date_deadline = fields.Date(string="Deadline", compute="_get_date_deadline", inverse="_set_date_deadline")
 
     def accept_offer(self):
         for record in self:
-            if(record.property_id.selling_price == 0):
-                record.status = "Accepted";
-                print(type(record.property_id.seller_id))
-                print(type(record.partner_id))
-                record.property_id.buyer_id = record.partner_id;
-                record.property_id.selling_price = record.price;
-        
-        return True;
-    
+            if (record.property_id.selling_price == 0):
+                record.status = "Accepted"
+                record.property_id.buyer_id = record.partner_id
+                record.property_id.selling_price = record.price
+        return True
+
     def refused_offer(self):
         for record in self:
-            record.status = "Refused";
-        return True;
-
+            record.status = "Refused"
+        return True
 
     @api.depends("validity")
     def _get_date_deadline(self):
         for record in self:
-            if(isinstance(record.create_date,bool)):
+            if (isinstance(record.create_date,bool)):
                 record.date_deadline = fields.Datetime.now() + relativedelta(days=record.validity)
                 return
             record.date_deadline = record.create_date + relativedelta(days=record.validity)
 
     def _set_date_deadline(self):
         for record in self:
-            record.validity = (datetime.combine(record.date_deadline,time()) - record.create_date).days
+            record.validity = (datetime.combine(record.date_deadline, time()) - record.create_date).days

estate/models/estate_property_tag.py

@@ -6,6 +6,6 @@ class EstatePropertyTag(models.Model):
     _description = "Property Tag"
     _order = "name asc"
 
-    _unique_tag = models.UniqueIndex("(name)","Tag name must be unique in database")
+    _unique_tag = models.UniqueIndex("(name)", "Tag name must be unique in database")
 
-    name = fields.Char(required=True)
+    name = fields.Char(required=True)

estate/models/estate_property_type.py

@@ -6,10 +6,8 @@ class EstatePropertyType(models.Model):
     _description = "Property Type"
     _order = "sequence, name asc"
 
-    _unique_type = models.UniqueIndex("(name)","Property type name must be unique in database")
+    _unique_type = models.UniqueIndex("(name)", "Property type name must be unique in database")
 
     name = fields.Char(required=True)
-    property_ids = fields.One2many("estate.property","property_type_id")
-    sequence = fields.Integer('Sequence',default=1,help="use to order the list inside the type view")
-
-   
+    property_ids = fields.One2many("estate.property", "property_type_id")
+    sequence = fields.Integer('Sequence', default=1, help="use to order the list inside the type view")