[IMP] estate: implement chapter 10 & 11

Author: Mathilde411

Diff for: estate/models/property.py

@@ -1,12 +1,22 @@
 from dateutil.relativedelta import relativedelta
 
 from odoo import models, fields, api
-from odoo.exceptions import UserError
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools import float_is_zero, float_compare
 
 
 class Property(models.Model):
     _name = 'estate.property'
     _description = 'Property'
+    _sql_constraints = [
+        ('check_expected_price', 'CHECK(expected_price > 0)', 'The Expected Price must be positive.'),
+        ('check_selling_price', 'CHECK(selling_price >= 0)', 'The Selling Price must be positive.'),
+        ('check_bedrooms', 'CHECK(bedrooms >= 0)', 'The number of bedrooms must be positive.'),
+        ('check_living_area', 'CHECK(living_area > 0)', 'The living area must be positive.'),
+        ('check_facades', 'CHECK(facades > 0)', 'The number of facades must be positive.'),
+        ('check_name_unique', 'UNIQUE(name)', 'The Property name must be unique.')
+    ]
+    _order = 'id desc'
 
     name = fields.Char(string='Title', required=True)
     description = fields.Text()
@@ -65,6 +75,14 @@ def _onchange_garden(self):
             self.garden_orientation = None
             self.garden_area = 0
 
+    @api.constrains('selling_price', 'expected_price')
+    def _check_date_end(self):
+        for record in self:
+            if not float_is_zero(record.selling_price, precision_digits=2) and float_compare(record.selling_price,
+                                                                                             0.9 * record.expected_price,
+                                                                                             precision_digits=2) >= 0:
+                raise ValidationError("The selling price must not be below 90% of the expected price.")
+
     def action_sold(self):
         for record in self:
             if record.state in ['sold', 'canceled']:

Diff for: estate/models/property_offer.py

@@ -6,6 +6,11 @@
 class PropertyOffer(models.Model):
     _name = 'estate.property.offer'
     _description = 'Property Offer'
+    _sql_constraints = [
+        ('check_offer_price', 'CHECK(price > 0)', 'The Offer Price must be positive.'),
+        ('check_validity', 'CHECK(validity >= 0)', 'The Offer Validity must be positive.'),
+    ]
+    _order = 'price desc'
 
     state = fields.Selection([
         ('received', 'Received'),
@@ -19,6 +24,7 @@ class PropertyOffer(models.Model):
     validity = fields.Integer(string='Validity (days)', default=7)
     deadline_date = fields.Date(string='Deadline Date', compute='_compute_deadline_date',
                                 inverse='_inverse_deadline_date')
+    property_type_id = fields.Many2one('estate.property.type', related='property_id.property_type_id')
 
     @api.depends('validity', 'deadline_date', 'create_date')
     def _compute_deadline_date(self):

Diff for: estate/models/property_tag.py

@@ -4,5 +4,10 @@
 class PropertyTag(models.Model):
     _name = 'estate.property.tag'
     _description = 'Property Tag'
+    _sql_constraints = [
+        ('check_name_unique', 'UNIQUE(name)', 'The Tag name must be unique.')
+    ]
+    _order = 'name asc'
 
     name = fields.Char(string='Tag', required=True)
+    color = fields.Integer(string='Color')

Diff for: estate/models/property_type.py

@@ -1,8 +1,21 @@
-from odoo import models, fields
+from odoo import models, fields, api
 
 
 class PropertyType(models.Model):
     _name = 'estate.property.type'
     _description = 'Property Type'
+    _sql_constraints = [
+        ('check_name_unique', 'UNIQUE(name)', 'The type name must be unique.')
+    ]
+    _order = 'sequence asc, name asc'
 
     name = fields.Char(string='Title', required=True)
+    sequence = fields.Integer('Sequence', default=1)
+    property_ids = fields.One2many('estate.property', 'property_type_id')
+    offer_ids = fields.One2many('estate.property.offer', 'property_type_id')
+    offer_count = fields.Integer(compute='_compute_offer_count')
+
+    @api.depends('offer_ids')
+    def _compute_offer_count(self):
+        for record in self:
+            record.offer_count = len(record.offer_ids)

Diff for: estate/views/estate_property_offer_views.xml

@@ -4,7 +4,7 @@
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list string="Property offers">
+            <list string="Property offers" editable="bottom">
                 <field name="price" optional="show"/>
                 <field name="partner_id"/>
                 <field name="deadline_date" optional="show"/>
@@ -22,29 +22,10 @@
         </field>
     </record>
 
-    <record id="estate_property_offer_view_form" model="ir.ui.view">
-        <field name="name">estate.property.offer.form</field>
-        <field name="model">estate.property.offer</field>
-        <field name="arch" type="xml">
-            <form string="Property offers">
-                <header>
-                    <button name="action_accept" invisible="state in ['accepted', 'refused']" string="Accept"
-                            type="object" class="btn-primary"/>
-                    <button name="action_refuse" invisible="state in ['accepted', 'refused']" string="Refuse"
-                            type="object"/>
-                    <button name="action_reset" invisible="state not in ['accepted', 'refused']" string="Reset"
-                            type="object"/>
-                </header>
-                <sheet>
-                    <group>
-                        <field name="price"/>
-                        <field name="partner_id"/>
-                        <field name="deadline_date"/>
-                        <field name="validity"/>
-                        <field name="state"/>
-                    </group>
-                </sheet>
-            </form>
-        </field>
+    <record id="estate_property_offer_action" model="ir.actions.act_window">
+        <field name="name">Property Types</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+        <field name="view_mode">list</field>
     </record>
 </odoo>

Diff for: estate/views/estate_property_tag_views.xml

@@ -4,28 +4,12 @@
         <field name="name">estate.property.tag.list</field>
         <field name="model">estate.property.tag</field>
         <field name="arch" type="xml">
-            <list string="Property tags">
+            <list string="Property tags" editable="bottom">
                 <field name="name"/>
             </list>
         </field>
     </record>
 
-    <record id="estate_property_tag_view_form" model="ir.ui.view">
-        <field name="name">estate.property.tag.form</field>
-        <field name="model">estate.property.tag</field>
-        <field name="arch" type="xml">
-            <form string="Property tags">
-                <sheet>
-                    <div class="oe_title">
-                        <h1>
-                            <field name="name"/>
-                        </h1>
-                    </div>
-                </sheet>
-            </form>
-        </field>
-    </record>
-
     <record id="estate_property_tag_view_filter" model="ir.ui.view">
         <field name="name">estate.property.tag.list.select</field>
         <field name="model">estate.property.tag</field>
@@ -39,6 +23,6 @@
     <record id="estate_property_tag_action" model="ir.actions.act_window">
         <field name="name">Property tags</field>
         <field name="res_model">estate.property.tag</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list</field>
     </record>
 </odoo>

Diff for: estate/views/estate_property_type_views.xml

@@ -5,6 +5,7 @@
         <field name="model">estate.property.type</field>
         <field name="arch" type="xml">
             <list string="Property Types">
+                <field name="sequence" widget="handle"/>
                 <field name="name"/>
             </list>
         </field>
@@ -16,11 +17,37 @@
         <field name="arch" type="xml">
             <form string="Property Types">
                 <sheet>
+                    <div class="oe_button_box" name="button_box">
+                        <button class="oe_stat_button" type="action" name="estate.estate_property_offer_action"
+                                icon="fa-envelope">
+                            <field string="Offers" name="offer_count" widget="statinfo"/>
+                        </button>
+                    </div>
                     <div class="oe_title">
                         <h1>
                             <field name="name"/>
                         </h1>
                     </div>
+                    <notebook>
+                        <page string="Properties">
+                            <field name="property_ids">
+                                <list>
+                                    <field name="name"/>
+                                    <field name="postcode" optional="hide"/>
+                                    <field name="bedrooms" optional="hide"/>
+                                    <field name="living_area" optional="hide"/>
+                                    <field name="expected_price" optional="show"/>
+                                    <field name="selling_price" optional="hide"/>
+                                    <field name="date_availability" optional="hide"/>
+                                    <field name="state" widget="badge"
+                                           decoration-info="state == 'offer_received'"
+                                           decoration-success="state in ['sold', 'offer_accepted']"
+                                           decoration-danger="state == 'canceled'"
+                                           optional="show"/>
+                                </list>
+                            </field>
+                        </page>
+                    </notebook>
                 </sheet>
             </form>
         </field>

Diff for: estate/views/estate_property_views.xml

@@ -8,6 +8,7 @@
                 <field name="name"/>
                 <field name="property_type_id" optional="show"/>
                 <field name="postcode" optional="show"/>
+                <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}" optional="show"/>
                 <field name="bedrooms" optional="hide"/>
                 <field name="living_area" optional="hide"/>
                 <field name="expected_price" optional="show"/>
@@ -43,11 +44,11 @@
                         </h1>
                     </div>
                     <div class="mb-3">
-                        <field name="tag_ids" widget="many2many_tags"/>
+                        <field name="tag_ids" widget="form.many2many_tags" options="{'color_field': 'color'}"/>
                     </div>
                     <group>
                         <group>
-                            <field name="property_type_id"/>
+                            <field name="property_type_id" can_create="False" can_write="False"/>
                             <field name="postcode"/>
                             <field name="date_availability"/>
                         </group>
@@ -66,8 +67,8 @@
                                 <field name="facades"/>
                                 <field name="garage"/>
                                 <field name="garden"/>
-                                <field name="garden_area"/>
-                                <field name="garden_orientation"/>
+                                <field name="garden_area" invisible="not garden"/>
+                                <field name="garden_orientation" invisible="not garden"/>
                                 <field name="total_area"/>
                             </group>
                         </page>
@@ -94,10 +95,10 @@
             <search string="Search Property">
                 <field name="name"/>
                 <field name="postcode"/>
-                <field name="expected_price"/>
-                <field name="bedrooms"/>
-                <field name="living_area"/>
-                <field name="facades"/>
+                <field name="expected_price" filter_domain="[('living_area', '>=', self)]"/>
+                <field name="bedrooms" filter_domain="[('living_area', '>=', self)]"/>
+                <field name="living_area" filter_domain="[('living_area', '>=', self)]"/>
+                <field name="facades" filter_domain="[('living_area', '>=', self)]"/>
 
                 <filter name="available" string="Available"
                         domain="[('date_availability', '&lt;=', context_today().strftime('%Y-%m-%d'))]"/>
@@ -115,5 +116,6 @@
         <field name="name">Properties</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available': True}</field>
     </record>
 </odoo>