[FIX] chapter7-estate: Fixed action fields

Author: haman-odoo

estate/__manifest__.py

@@ -16,5 +16,5 @@
     'category': 'Sales/Real Estate',
     'installable': True,
     'auto_install': False,
-    'description': """Description text""",
+    'description': """Real estate management tutorial module with properties, offers, types and tags.""",
 }

estate/models/__init__.py

@@ -1,4 +1,4 @@
-from . import estate_property  # noqa: F401
+from . import estate_property
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer

estate/models/estate_property.py

@@ -1,41 +1,37 @@
-from odoo import models, fields
 from dateutil.relativedelta import relativedelta
 from datetime import date
+from odoo import models, fields, api
 
 
 class EstateProperty(models.Model):
     _name = "estate.property"
     _description = "Real Estate Property"
 
-    name = fields.Char(string="Property Title", required=True)
-    description = fields.Text(string="Description")
-    postcode = fields.Char(string="Postcode")
+    name = fields.Char(required=True)
+    description = fields.Text()
+    postcode = fields.Char()
     date_availability = fields.Date(
-        string="Available From",
         default=lambda sself: date.today() + relativedelta(months=3),
         copy=False
     )
     expected_price = fields.Float(string="Expected Price", required=True)
     selling_price = fields.Float(
-        string="Selling Price",
         readonly=True,
         copy=False
     )
-
-    bedrooms = fields.Integer(string="Bedrooms", default=2)
-    living_area = fields.Integer(string="Living Area (sqm)")
-    facades = fields.Integer(string="Number of Facades")
-    garage = fields.Boolean(string="Has Garage")
-    garden = fields.Boolean(string="Has Garden")
-    garden_area = fields.Integer(string="Garden Area (sqm)")
+    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'),
         ],
-        string="Garden Orientation"
     )
     active = fields.Boolean(default=True)
     state = fields.Selection(
@@ -46,31 +42,33 @@ class EstateProperty(models.Model):
         ('sold', 'Sold'),
         ('cancelled', 'Cancelled'),
     ],
-    string="Status",
     required=True,
     copy=False,
     default='new',
     )
     property_type_id = fields.Many2one(
         "estate.property.type",
-        string="Property Type",
     )
     salesperson_id = fields.Many2one(
         "res.users",
-        string="Salesman",
         default=lambda self: self.env.user,
     )
     buyer_id = fields.Many2one(
         "res.partner",
-        string="Buyer",
         copy=False,
     )
     tag_ids = fields.Many2many(
         "estate.property.tag",
-        string="Tags",
     )
     offer_ids = fields.One2many(
         "estate.property.offer",
         "property_id",
-        string="Offers",
     )
+    total_area = fields.Float(
+        compute="_compute_total_area"
+    )
+
+    @api.depends('living_area', 'garden_area')
+    def _compute_total_area(self):
+        for record in self:
+            record.total_area = record.living_area + record.garden_area

estate/models/estate_property_offer.py

@@ -4,26 +4,19 @@
 class EstatePropertyOffer(models.Model):
     _name = "estate.property.offer"
     _description = "Real Estate Property Offer"
-
     price = fields.Float(string="Price")
-
     status = fields.Selection(
         [
             ("accepted", "Accepted"),
             ("refused", "Refused"),
         ],
-        string="Status",
-        copy=False,        # do not copy when duplicating the offer
+        copy=False,
     )
-
     partner_id = fields.Many2one(
         "res.partner",
-        string="Partner",
         required=True,
     )
-
     property_id = fields.Many2one(
         "estate.property",
-        string="Property",
         required=True,
     )

estate/models/estate_property_tag.py

@@ -4,5 +4,4 @@
 class EstatePropertyTag(models.Model):
     _name = "estate.property.tag"
     _description = "Real Estate Property Tag"
-
-    name = fields.Char(string="Name", required=True)
+    name = fields.Char(required=True)

estate/models/estate_property_type.py

@@ -4,5 +4,4 @@
 class EstatePropertyType(models.Model):
     _name = "estate.property.type"
     _description = "Real Estate Property Type"
-
-    name = fields.Char(string="Name", required=True)
+    name = fields.Char(required=True)

estate/views/estate_menus.xml

@@ -1,14 +1,15 @@
-<!-- /home/odoo/odoo19/tutorials/estate/views/estate_menus.xml -->
 <odoo>
     <menuitem id="menu_estate_root" name="Real Estate"/>
     <menuitem id="menu_estate_advertisements" name="Advertisements" parent="menu_estate_root" sequence="10"/>
     <menuitem id="menu_estate_properties" name="Properties" parent="menu_estate_advertisements" action="action_estate_property" sequence="10"/>
-    <!-- Settings / Property Types -->
+
     <menuitem id="menu_estate_settings" name="Settings" parent="menu_estate_root" sequence="20"/>
+
     <menuitem id="menu_estate_property_types" name="Property Types"
               parent="menu_estate_settings"
               action="action_estate_property_type"
               sequence="10"/>
+
     <menuitem id="menu_estate_property_tags" name="Property Tags"
               parent="menu_estate_settings"
               action="action_estate_property_tag"/>

estate/views/estate_property_offer_views.xml

@@ -1,6 +1,4 @@
-<!-- /home/odoo/odoo19/tutorials/estate/views/estate_property_offer_views.xml -->
 <odoo>
-    <!-- Offers: List view -->
     <record id="view_estate_property_offer_list" model="ir.ui.view">
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
@@ -13,7 +11,6 @@
         </field>
     </record>
 
-    <!-- Offers: Form view -->
     <record id="view_estate_property_offer_form" model="ir.ui.view">
         <field name="name">estate.property.offer.form</field>
         <field name="model">estate.property.offer</field>

estate/views/estate_property_tag_views.xml

@@ -1,6 +1,4 @@
-<!-- /home/odoo/odoo19/tutorials/estate/views/estate_property_tag_views.xml -->
 <odoo>
-    <!-- Action for property tags -->
     <record id="action_estate_property_tag" model="ir.actions.act_window">
         <field name="name">Property Tags</field>
         <field name="res_model">estate.property.tag</field>

estate/views/estate_property_type_views.xml

@@ -1,6 +1,4 @@
-<!-- /home/odoo/odoo19/tutorials/estate/views/estate_property_type_views.xml -->
 <odoo>
-    <!-- Action for property types -->
     <record id="action_estate_property_type" model="ir.actions.act_window">
         <field name="name">Property Types</field>
         <field name="res_model">estate.property.type</field>