[FIX] fix style

Author: hazei-odoo

estate/__init__.py

@@ -1,4 +1 @@
-# -*- coding: utf-8 -*-
-
-
 from . import models

estate/__manifest__.py

@@ -14,7 +14,7 @@
         'views/estate_property_type.xml',
         'views/estate_property_views.xml',
         'views/res_users_views.xml',
-        'views/menu_views.xml',  
+        'views/menu_views.xml',
     ],
     'application': True,
     'license': 'LGPL-3',

estate/models/estate_property.py

@@ -15,10 +15,10 @@ class EstateProperty(models.Model):
     _order = "id desc"
 
     _check_expected_price = models.Constraint(
-         'CHECK(expected_price >= 0)', 'The expected price must be strictly positive.')
+        '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.')
+        'CHECK(selling_price > 0)', 'The selling price must be positive.')
 
     # ---------------------------------------- Fields Declaration ---------------------------------
 
@@ -66,12 +66,12 @@ class EstateProperty(models.Model):
     @api.depends('living_area', 'garden_area')
     def _compute_total_area(self):
         for property in self:
-             property.total_area = property.living_area + property.garden_area
+            property.total_area = property.living_area + property.garden_area
 
     @api.depends('offer_ids.price')
     def _compute_best_price(self):
         for record in self:
-                record.best_price = max(record.offer_ids.mapped('price')) if record.offer_ids else 0
+            record.best_price = max(record.offer_ids.mapped('price')) if record.offer_ids else 0
 
     # ----------------------------------- Constrains and Onchanges --------------------------------
 
@@ -83,7 +83,6 @@ def _check_selling_price_expected_price(self):
             if float_compare(record.selling_price, 0.9 * record.expected_price, precision_digits=2) == -1:
                 raise UserError("The selling price must be at least 90% of the expected price.")
 
-
     @api.onchange('garden')
     def _onchange_garden(self):
         if self.garden:
@@ -102,7 +101,6 @@ def _onchange_offers(self):
 
             if not has_offers and not any(offer.status == 'accepted' for offer in record.offer_ids) and record.state in ('offer_received',):
                 record.state = 'offer_receive'
-
     # ------------------------------------------ CRUD Methods -------------------------------------
 
     @api.ondelete(at_uninstall=False)
@@ -113,7 +111,6 @@ def _if_new_or_canceled(self):
     # ---------------------------------------- Action Methods -------------------------------------
 
     def action_set_sold(self):
-        print(self.state)
         if self.state == "cancelled":
             raise UserError("A cancelled property can not be sold")
         self.state = "sold"

estate/models/estate_property_offer.py

@@ -16,12 +16,7 @@ class EstatePropertyOffer(models.Model):
     # --------------------------------------- Fields Declaration ----------------------------------
 
     price = fields.Float(string='price', required=True)
-    status = fields.Selection(selection=[('accepted', 'Accepted'),
-        ('refused', 'Refused')],
-        string="Status",
-        copy=False,
-        # default='accepted',
-    )
+    status = fields.Selection(selection=[('accepted', 'Accepted'), ('refused', 'Refused')], string="Status", copy=False)
     validity = fields.Integer(string='Validity(days)', default=7)
     partner_id = fields.Many2one('res.partner', string='Partner', required=True)
     property_id = fields.Many2one('estate.property', required=True)
@@ -43,32 +38,35 @@ def create(self, vals_list):
         for vals in vals_list:
             if vals.get("property_id") and vals.get("price"):
                 prop = self.env["estate.property"].browse(vals["property_id"])
-                
+
                 if float_compare(vals["price"], prop.expected_price * 0.9, precision_rounding=0.01) < 0:
-                    raise UserError("The offer price must be at least 90%% of the expected price.")
+                    raise UserError(
+                        "The offer price must be at least 90%% of the expected price.")
 
                 if prop.offer_ids:
                     max_offer = max(prop.mapped("offer_ids.price"))
                     if float_compare(vals["price"], max_offer, precision_rounding=0.01) <= 0:
-                        raise UserError("The offer must be higher than %.2f" % max_offer)
+                        raise UserError(
+                            "The offer must be higher than %.2f" % max_offer)
 
         new_offers = super().create(vals_list)
 
         for offer in new_offers:
             offer.property_id.state = "offer_received"
-        
+
         return new_offers
 
     # ---------------------------------------- Action Methods -------------------------------------
 
     def action_accept(self):
-        print("Accepting offer...")
         for offer in self:
             if any(prop_offer.status == 'accepted' for prop_offer in offer.property_id.offer_ids):
-                raise UserError("An offer has already been accepted for this property.")
+                raise UserError(
+                    "An offer has already been accepted for this property.")
+
             if offer.status == 'refused':
                 raise UserError("A refused offer cannot be accepted.")
-        else:
+
             offer.status = 'accepted'
             offer.property_id.write({
                 'state': 'offer_accepted',
@@ -78,7 +76,7 @@ def action_accept(self):
         return True
 
     def action_refuse(self):
-            if any(offer.status == 'accepted' for offer in self):           
-                raise UserError("An accepted offer cannot be refused.")
-            self.status = 'refused'
-            return True
+        if any(offer.status == 'accepted' for offer in self):
+            raise UserError("An accepted offer cannot be refused.")
+        self.status = 'refused'
+        return True

estate/models/res_users.py

@@ -7,4 +7,4 @@ class ResUsers(models.Model):
 
     property_ids = fields.One2many(
         "estate.property", "salesman_id", string="Available Properties", domain=[("state", "in", ["new", "offer_received"])]
-    )
+    )

estate_account/__manifest__.py

@@ -2,8 +2,8 @@
     'name': "Tutorial Real Estate Accounting",
     'description': 'model links the estate and account modules',
     'version': '1.0',
-    'depends': ['estate','account',],
-    'data':[],
+    'depends': ['estate', 'account'],
+    'data': [],
     'auto_install': True,
     'author': 'Hazei',
     'license': 'LGPL-3',

estate_account/models/estate_property.py

@@ -1,19 +1,17 @@
 from odoo import models, Command
-from odoo.exceptions import UserError
 import logging
 
 _logger = logging.getLogger(__name__)
 
+
 class EstateProperty(models.Model):
-    
-    # ---------------------------------------- Private Attributes ---------------------------------
 
+    # ---------------------------------------- Private Attributes ---------------------------------
     _inherit = "estate.property"
 
     # ---------------------------------------- Action Methods -------------------------------------
 
     def action_set_sold(self):
-        print("top")
         res = super().action_set_sold()
         journal = self.env["account.journal"].search([("type", "=", "sale")], limit=1)
         for prop in self:
@@ -36,5 +34,4 @@ def action_set_sold(self):
                     ],
                 }
             )
-        print("bottom")
         return res