[IMP] estate: Create New module estate_account to create Property invoice

Will add a new module that depends on the estate and account module

It overrides the sold action to create an invoice for the property sold

Author: shsak-odoo

estate/models/estate_property.py

@@ -3,16 +3,15 @@
 
 
 class EstateProperty(models.Model):
-    _name = 'estate_property'
-    _description = 'Estate Property details'
-    _order = 'id desc'
+    _name = "estate_property"
+    _description = "Estate Property details"
+    _order = "id desc"
 
     name = fields.Char(required=True)
     description = fields.Text()
     postcode = fields.Char()
     date_availability = fields.Date(
-        default=lambda self: fields.Date.add(fields.Date.today(), months=3),
-        copy=False
+        default=lambda self: fields.Date.add(fields.Date.today(), months=3), copy=False
     )
     expected_price = fields.Float(required=True)
     selling_price = fields.Float(readonly=True, copy=False)
@@ -23,88 +22,97 @@ class EstateProperty(models.Model):
     garden = fields.Boolean()
     garden_area = fields.Integer()
     garden_orientation = fields.Selection(
-        string='Garden Orientation',
+        string="Garden Orientation",
         selection=[
-            ('north', 'North'),
-            ('south', 'South'),
-            ('east', 'East'),
-            ('west', 'West')
-        ]
+            ("north", "North"),
+            ("south", "South"),
+            ("east", "East"),
+            ("west", "West"),
+        ],
     )
     state = fields.Selection(
-        string='Status',
+        string="Status",
         selection=[
-            ('new', 'New'),
-            ('offer_received', 'Offer Received'),
-            ('offer_accepted', 'Offer Accepted'),
-            ('sold', 'Sold'),
-            ('cancelled', 'Cancelled')
+            ("new", "New"),
+            ("offer_received", "Offer Received"),
+            ("offer_accepted", "Offer Accepted"),
+            ("sold", "Sold"),
+            ("cancelled", "Cancelled"),
         ],
-        default='new',
+        default="new",
         required=True,
-        copy=False
+        copy=False,
     )
     active = fields.Boolean(default=True)
-    property_type_id = fields.Many2one('estate.property.type', string='Type')
-    buyer = fields.Many2one('res.partner', copy=False)
-    salesperson = fields.Many2one('res.users', string='Salesman', default=lambda self: self.env.user)
-    tag_ids = fields.Many2many('estate.property.tag', string='Tag')
-    offer_ids = fields.One2many('estate.property.offer', 'property_id')
-    total_area = fields.Integer(compute='_compute_total_area')
-    best_price = fields.Float(compute='_compute_best_price')
+    property_type_id = fields.Many2one("estate.property.type", string="Type")
+    buyer = fields.Many2one("res.partner", copy=False)
+    salesperson = fields.Many2one(
+        "res.users", string="Salesman", default=lambda self: self.env.user
+    )
+    tag_ids = fields.Many2many("estate.property.tag", string="Tag")
+    offer_ids = fields.One2many("estate.property.offer", "property_id")
+    total_area = fields.Integer(compute="_compute_total_area")
+    best_price = fields.Float(compute="_compute_best_price")
 
     _check_expected_price = models.Constraint(
-        'CHECK(expected_price > 0)',
-        'The Expected price cannot be 0 or less then 0'
+        "CHECK(expected_price > 0)", "The Expected price cannot be 0 or less then 0"
     )
 
     _check_selling_price = models.Constraint(
-        'CHECK(selling_price >= 0)',
-        'The Selling price cannot be less then 0'
+        "CHECK(selling_price >= 0)", "The Selling price cannot be less then 0"
     )
 
-    @api.constrains('selling_price', 'buyer', 'expected_price')
+    @api.constrains("selling_price", "buyer", "expected_price")
     def _check_selling_price_90p(self):
         for record in self:
             if record.selling_price == 0:
                 return False
-            if float_compare((record.selling_price / record.expected_price) * 100, 90, precision_digits=2) < 0:
-                raise exceptions.ValidationError(_('Selling Price should be 90% or more of expected price.'))
+            if (
+                float_compare(
+                    (record.selling_price / record.expected_price) * 100,
+                    90,
+                    precision_digits=2,
+                )
+                < 0
+            ):
+                raise exceptions.ValidationError(
+                    _("Selling Price should be 90% or more of expected price.")
+                )
 
-    @api.depends('living_area', 'garden_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
 
-    @api.depends('offer_ids.price')
+    @api.depends("offer_ids.price")
     def _compute_best_price(self):
         for record in self:
-            record.best_price = max(record.offer_ids.mapped('price') or [0])
+            record.best_price = max(record.offer_ids.mapped("price") or [0])
 
-    @api.onchange('garden')
+    @api.onchange("garden")
     def _onchange_garden(self):
         if self.garden:
-            self.garden_orientation = 'north'
+            self.garden_orientation = "north"
             self.garden_area = 10
         else:
             self.garden_orientation = None
             self.garden_area = 0
 
     def action_cancel_property(self):
-        if self.filtered(lambda x: x.state == 'sold'):
-            raise exceptions.UserError(_('A sold property cannot be cancelled'))
+        if self.filtered(lambda x: x.state == "sold"):
+            raise exceptions.UserError(_("A sold property cannot be cancelled"))
 
-        self.state = 'cancelled'
-        return True
+        self.state = "cancelled"
 
     def action_property_sold(self):
-        if self.filtered(lambda x: x.state == 'cancelled'):
-            raise exceptions.UserError(_('A cancelled property cannot be sold'))
+        if self.filtered(lambda x: x.state == "cancelled"):
+            raise exceptions.UserError(_("A cancelled property cannot be sold"))
 
-        self.state = 'sold'
-        return True
+        self.state = "sold"
 
     @api.ondelete(at_uninstall=False)
     def _unlink_if_new_or_sold(self):
-        if self.filtered(lambda x: x.state not in ('new', 'cancelled')):
-            raise exceptions.UserError(_('Cannot delete property which is not yer sold or cancelled'))
+        if self.filtered(lambda x: x.state not in ("new", "cancelled")):
+            raise exceptions.UserError(
+                _("Cannot delete property which is new or cancelled")
+            )

estate/views/estate_property_views.xml

@@ -15,8 +15,12 @@
                     <h1>
                         <field name="name" nolabel="True" />
                     </h1>
-                    <field name="tag_ids" widget="many2many_tags" class="mb-5"
-                        options="{'color_field': 'color'}" />
+                    <group>
+                        <group>
+                            <field name="tag_ids" widget="many2many_tags"
+                                options="{'color_field': 'color'}" string="Tags" />
+                        </group>
+                    </group>
                     <group>
                         <group>
                             <field name="property_type_id" string="Property Type"
@@ -106,6 +110,6 @@
         <field name="name">Properties</field>
         <field name="res_model">estate_property</field>
         <field name="context">{'search_default_available': True}</field>
-        <field name="view_mode">list,form</field>
+        <field name="view_mode">list,form,kanban</field>
     </record>
-</odoo>
+</odoo>

estate_account/__init__.py

@@ -0,0 +1 @@
+from . import models

estate_account/__manifest__.py

@@ -0,0 +1,12 @@
+{
+    "name": "Estate Accounting",
+    "version": "1.0",
+    "depends": ["estate", "account"],
+    "author": "Shivam Saksham(shsak)",
+    "category": "Sales",
+    "description": """
+    A Real Estate Invoicing Module
+    """,
+    "license": "LGPL-3",
+    "data": [],
+}

estate_account/models/__init__.py

@@ -0,0 +1 @@
+from . import estate_property

estate_account/models/estate_property.py

@@ -0,0 +1,30 @@
+from odoo import models, Command
+
+
+class EstateProperty(models.Model):
+    _inherit = "estate_property"
+
+    def action_property_sold(self):
+        self.env["account.move"].create(
+            {
+                "move_type": "out_invoice",
+                "partner_id": self.buyer.id,
+                "invoice_line_ids": [
+                    Command.create(
+                        {
+                            "name": self.name,
+                            "quantity": 1,
+                            "price_unit": self.selling_price * 0.6,
+                        }
+                    ),
+                    Command.create(
+                        {
+                            "name": "Administrative Fees",
+                            "quantity": 1,
+                            "price_unit": 100,
+                        }
+                    ),
+                ],
+            }
+        )
+        return super().action_property_sold()