[IMP] estate: Server101 Finished Chapter 11

Author: VincentJanssen-Code

estate/models/estate_property.py

@@ -1,5 +1,5 @@
 from dateutil.relativedelta import relativedelta
-from odoo import fields, models, api, exceptions
+from odoo import fields, models, api
 from odoo.tools.float_utils import float_compare, float_is_zero
 from odoo.exceptions import ValidationError, UserError
 
@@ -9,8 +9,8 @@ class EstateProperty(models.Model):
     _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"),
@@ -79,7 +79,7 @@ def _compute_best_price(self):
     @api.onchange("garden")
     def _on_change_garden(self):
         self.garden_area = 10 if self.garden else 0
-        self.garden_orientation = 'North' if self.garden else ''
+        self.garden_orientation = 'north' if self.garden else ''
 
     @api.constrains('selling_price', 'expected_price')
     def _constrain_prices(self):

estate/models/estate_property_offer.py

@@ -17,15 +17,18 @@ class EstatePropertyOffer(models.Model):
 
     partner_id = fields.Many2one('res.partner', required=True)
     property_id = fields.Many2one('estate.property', required=True)
+    property_type_id = fields.Many2one(related="property_id.property_type_id", store=True)
     validity = fields.Integer(string="Validity Duration", default=7)
     date_deadline = fields.Date(string="Deadline", compute="_compute_date_deadline", inverse="_inverse_date_deadline")
 
+
     def accept_offer(self):
         for record in self:
             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
+                record.property_id.state = 'offer_accepted'
         return True
 
     def refused_offer(self):

estate/models/estate_property_tag.py

@@ -9,3 +9,4 @@ class EstatePropertyTag(models.Model):
     _unique_tag = models.UniqueIndex("(name)", "Tag name must be unique in database")
 
     name = fields.Char(required=True)
+    color = fields.Integer()

estate/models/estate_property_type.py

@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import fields, models, api
 
 
 class EstatePropertyType(models.Model):
@@ -11,3 +11,10 @@ class EstatePropertyType(models.Model):
     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")
+    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)

estate/views/estate_property_offer_views.xml

@@ -1,6 +1,13 @@
 <?xml version="1.0"?>
 <odoo>
 
+    <record id="estate_property_offer_model_action" model="ir.actions.act_window">
+        <field name ="name">Property Offer</field>
+        <field name="res_model">estate.property.offer</field>
+        <field name="view_mode">list,form</field>
+        <field name="domain">[('property_type_id', '=', active_id)]</field>
+    </record>
+
     <record id="estate_property_offer_model_form" model="ir.ui.view">
         <field name ="name">estate.property.offer.form</field>
         <field name="model">estate.property.offer</field>
@@ -23,14 +30,14 @@
         <field name="name">estate.property.offer.list</field>
         <field name="model">estate.property.offer</field>
         <field name="arch" type="xml">
-            <list string="Property offer list">
+            <list string="Property offer list" editable="bottom" decoration-success="status == 'accepted'" decoration-danger="status == 'refused'">
                 <field name="price"/>
                 <field name ="partner_id"/>
                 <field name="date_deadline"/>
                 <field name="validity"/>
-                <button name="accept_offer" type="object" string="Accept" icon="fa-check"/>
-                <button name="refused_offer" type="object" string="Refuse" icon="fa-times"/>
-                <field name ="status"/>
+                <field name="property_type_id"/>
+                <button name="accept_offer" type="object" string="Accept" icon="fa-check" invisible="status == 'accepted' or status == 'refused'"/>
+                <button name="refused_offer" type="object" string="Refuse" icon="fa-times" invisible="status == 'accepted' or status == 'refused'"/>            
             </list>
         </field>
     </record>

estate/views/estate_property_tag_views.xml

@@ -20,5 +20,15 @@
             </form>
         </field>
     </record>
+    <record id="estate_property_tag_model_list" model="ir.ui.view">
+        <field name ="name">estate.property.tag.list</field>
+        <field name="model">estate.property.tag</field>
+        <field name="arch" type="xml">
+            <list string="Property tag list" editable="bottom"> 
+                <field name="name"/>                   
+            </list>
+        </field>
+    </record>
+
 
 </odoo>

estate/views/estate_property_type_views.xml

@@ -13,6 +13,9 @@
         <field name="arch" type="xml">
             <form string="Property type Form">
                 <sheet>
+                    <div class="oe_stat_button">
+                        <button name="estate.estate_property_offer_model_action" string="Offers"  type="action"/>
+                    </div>
                     <group>
                         <h1 class="mb32">
                             <field name="name"/>

estate/views/estate_property_views.xml

@@ -5,30 +5,32 @@
         <field name ="name">Property</field>
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form</field>
+        <field name="context">{'search_default_available':True}</field>
     </record>
 
     <record id="estate_property_view_list" model="ir.ui.view">
         <field name="name">estate.property.list</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
-            <list string="Property list">
+            <list string="Property list" decoration-bf="state == 'offer_accepted'" decoration-success="state == 'offer_accepted' or state == 'offer_recieved'" decoration-muted="state == 'sold'">
                 <field name="name"/>
                 <field name ="postcode"/>
                 <field name ="bedrooms"/>
                 <field name ="expected_price"/>
                 <field name ="selling_price"/>
-                <field name ="date_availability"/>
+                <field name ="date_availability" optional="True"/>
             </list>
         </field>
     </record>
+
     <record id="estate_property_model_form" model="ir.ui.view">
         <field name ="name">estate.property.form</field>
         <field name="model">estate.property</field>
         <field name="arch" type="xml">
             <form string="Property list">
                 <header>
-                    <button name="sell_property" type="object" string="Sell"/>
-                    <button name="cancel_property" type="object" string="Cancel"/>
+                    <button name="sell_property" type="object" string="Sell" invisible="state == 'sold' or state == 'cancelled'"/>
+                    <button name="cancel_property" type="object" string="Cancel" invisible="state == 'sold' or state == 'cancelled'"/>
                     <field name="state" widget="statusbar" statusbar_visible="new,offer_received,offer_accepted,sold,cancelled"/>
                 </header>
                 <sheet>
@@ -38,7 +40,7 @@
                         </h1>
                     </group>
                     <group>
-                        <field name="tag_ids" widget="many2many_tags"/>
+                        <field name="tag_ids" widget="many2many_tags" options="{'color_field': 'color'}"/>
 
                     </group>
                     <group>
@@ -62,13 +64,13 @@
                                 <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>
                         <page string="Offers">
-                            <field name="offer_ids"/>
+                            <field name="offer_ids" readonly=" state == 'sold' or state =='cancelled' or state == 'offer_accepted'"/>
                         </page>
                         <page string="Other Info">
                             <group>
@@ -92,7 +94,7 @@
                 <field name="postcode"/>
                 <field name="expected_price"/>
                 <field name="bedrooms"/>
-                <field name="living_area"/>
+                <field name="living_area" filter_domain="[('living_area','>=',int(self))]"/>
                 <field name="facades"/>
                 <filter string="Available" name="available" domain="['|',('state','=','Offer_Received'),('state','=','New')]"/>
                 <filter string="Postcodes" name="postcodes" context="{'group_by':'postcode'}"/>