[IMP] estate: fix some linting problem

Author: raibr

estate/models/estate_property.py

@@ -9,6 +9,7 @@ class EstateProperty(models.Model):
     _name = 'estate.property'
     _description = "Real Estate Property"
     _order = 'id desc'
+
     _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.")
 
@@ -100,7 +101,8 @@ def action_cancel(self):
             property.state = 'cancelled'
         return True
 
-    def unlink(self):
+    @api.ondelete(at_uninstall=True)
+    def delete(self):
         for property in self:
             if property.state not in ('new', 'cancelled'):
                 raise UserError(self.env._("Only new and cancelled properties can be deleted."))

estate/models/estate_property_offer.py

@@ -10,8 +10,9 @@ class EstatePropertyOffer(models.Model):
     _description = "Real Estate Property Offer"
     _order = 'price desc'
 
-    price = fields.Float(required=True, string="Price")
     _check_price = models.Constraint('Check(price > 0)', "The offer price must be strictly positive.")
+
+    price = fields.Float(required=True, string="Price")
     status = fields.Selection(selection=[('accepted', "Accepted"), ('refused', "Refused")], copy=False, string="Status")
     partner_id = fields.Many2one(comodel_name='res.partner', string="Partner", required=True)
     property_id = fields.Many2one(comodel_name='estate.property', string="Property", required=True, ondelete='cascade')

estate/models/estate_property_type.py

@@ -7,6 +7,7 @@ class EstatePropertyType(models.Model):
     _name = 'estate.property.type'
     _description = "Real Estate Property Type"
     _order = 'name'
+
     _check_unique_name = models.Constraint('UNIQUE(name)', "The type name must be unique.")
 
     name = fields.Char(required=True, string="Name")

estate/models/res_users.py

@@ -9,5 +9,5 @@ class ResUsers(models.Model):
     property_ids = fields.One2many(
         comodel_name='estate.property',
         inverse_name='salesperson_id',
-        string="Properties",
+        string="Estate Properties",
     )

estate/views/estate_property_type_views.xml

@@ -20,8 +20,9 @@
             <form string="Property Type">
                 <sheet>
                     <div class="oe_button_box" name="button_box">
-                        <button name="%(estate_property_offer_action)d" type="action" class="oe_stat_button" icon="fa-money">
-                            <field name="offer_count" widget="statinfo" string="Offers"/>
+                        <button name="%(estate_property_offer_action)d" type="action"
+                            class="oe_stat_button" icon="fa-money">
+                            <field name="offer_count" widget="statinfo" string="Offers" />
                         </button>
                     </div>
                     <group>