[ADD] estate_account: generate invoice for property sold through inheritance

Added logic to create a customer invoice when a property is marked as Sold
Invoice includes 6% commission and ₹100 administrative fees
Display property in user which buy by them
only can delete property if it is in new or cancelled state

[FIX] estate_account: fixed check style and tutorial changes

 fixed check style and tutorial changes in estate_account and estate_property

Author: rodh-odoo

estate_account/__init__.py

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

estate_account/__manifest__.py

@@ -0,0 +1,9 @@
+{
+    'name': 'Estate Accounting',
+    'version': '1.0',
+    'depends': ['real_estate', 'account'],
+    'category': 'Real Estate',
+    'license': 'LGPL-3',
+    'installable': True,
+    'application': False,
+}

estate_account/models/__init__.py

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

estate_account/models/estate_property.py

@@ -0,0 +1,40 @@
+from odoo import models, Command
+from odoo.exceptions import UserError
+
+
+class EstateProperty(models.Model):
+    _inherit = 'estate.property'
+
+    def action_sold(self):
+        res = super().action_sold()
+
+        journal = self.env['account.journal'].search([('type', '=', 'sale')], limit=1)
+        if not journal:
+            raise UserError("No sale journal found. Please configure at least one sale journal.")
+
+        for record in self:
+            if not record.buyer_id:
+                raise UserError("Please set a Buyer before generating an invoice.")
+            if not record.selling_price:
+                raise UserError("Please set a Selling Price before generating an invoice.")
+
+            invoice_vals = {
+                "partner_id": record.buyer_id.id,
+                "move_type": "out_invoice",
+                "journal_id": journal.id,
+                "invoice_line_ids": [
+                    Command.create({
+                        "name": "6% Commission",
+                        "quantity": 1,
+                        "price_unit": 0.06 * record.selling_price,
+                    }),
+                    Command.create({
+                        "name": "Administrative Fees",
+                        "quantity": 1,
+                        "price_unit": 100.0,
+                    }),
+                ]
+            }
+
+            self.env["account.move"].create(invoice_vals)
+        return res

real_estate/__manifest__.py

@@ -13,10 +13,10 @@
     'data': [
     'security/ir.model.access.csv',
     'view/estate_property_views.xml',
-    
+    'view/estate_property_offer_views.xml',
     'view/estate_property_type_views.xml',
     'view/estate_property_tag_views.xml',
-    'view/estate_property_offer_views.xml',
     'view/estate_menus.xml',
+    'view/estate_res_users_view.xml'
     ],
 }

real_estate/models/__init__.py

@@ -2,3 +2,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
+from . import estate_res_users

real_estate/models/estate_property.py

@@ -19,7 +19,6 @@ class EstateProperty(models.Model):
     ('check_expected_price_positive', 'CHECK(expected_price > 0)', 'The expected price must be strictly positive.'),
     ('check_selling_price_positive', 'CHECK(selling_price >= 0)', 'The selling price must be positive.')
 ]
-
     bedrooms = fields.Integer(default=2)
     living_area = fields.Float(string="Living Area")
     facades = fields.Integer(string="Facades")
@@ -59,7 +58,8 @@ class EstateProperty(models.Model):
     @api.constrains('selling_price', 'expected_price')
     def _check_selling_price(self):
         for record in self:
-            if float_is_zero(record.selling_price, precision_digits=2):
+            if (float_is_zero(record.selling_price, precision_digits=2)
+            or record.state not in ['offer_accepted', 'sold']):
                 continue
             min_acceptable_price = 0.9 * record.expected_price
             if float_compare(record.selling_price, min_acceptable_price, precision_digits=2) < 0:
@@ -98,3 +98,9 @@ def action_sold(self):
                 raise UserError("Cancelled properties cannot be sold.")
             record.state = 'sold'
         return True
+
+    @api.ondelete(at_uninstall=False)
+    def _check_deletion(self):
+        for record in self:
+            if record.state not in ['new', 'cancelled']:
+                raise UserError("You can only delete a property if its state is 'New' or 'Cancelled'.")

real_estate/models/estate_property_offer.py

@@ -13,7 +13,7 @@ class EstatePropertyOffer(models.Model):
         ('new', 'New'),
         ('accepted', 'Accepted'),
         ('refused', 'Refused')
-    ], string="State",default='new', required=True)
+    ], string="State", default='new', required=True)
     _sql_constraints = [
     ('check_price', 'CHECK(price > 0)', 'The offer price must be strictly positive.'),
 ]
@@ -27,17 +27,19 @@ class EstatePropertyOffer(models.Model):
     )
     validity = fields.Integer(default=7)
     date_deadline = fields.Date(compute="_compute_date_deadline", inverse="_inverse_date_deadline", store=True)
+
     @api.depends('create_date', 'validity')
     def _compute_date_deadline(self):
         for record in self:
             create_date = record.create_date or fields.Datetime.now()
             record.date_deadline = create_date.date() + timedelta(days=record.validity)
+
     def _inverse_date_deadline(self):
         for record in self:
             create_date = record.create_date or fields.Datetime.now()
             delta = record.date_deadline - create_date.date()
             record.validity = delta.days
-        
+
     def action_accept(self):
         for offer in self:
             if offer.property_id.buyer_id:
@@ -49,7 +51,20 @@ def action_accept(self):
                 'state': 'offer_accepted'
             })
         return True
+
     def action_refuse(self):
         for offer in self:
             offer.state = 'refused'
         return True
+
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            price = vals.get('price')
+            property_obj = self.env['estate.property'].browse(vals.get('property_id'))
+            existing_offers = property_obj.offer_ids.filtered(lambda o: o.price >= price)
+            if existing_offers:
+                raise ValidationError("You cannot create an offer with a lower or equal price than existing offers.")
+            property_obj.state = 'offer_received'
+
+        return super().create(vals_list)

real_estate/models/estate_property_type.py

@@ -21,6 +21,7 @@ class EstatePropertyType(models.Model):
         string="Offers Count",
         compute="_compute_offer_count"
     )
+
     @api.depends("offer_ids")
     def _compute_offer_count(self):
         for record in self:

real_estate/models/estate_res_users.py

@@ -0,0 +1,12 @@
+from odoo import models, fields
+
+
+class ResUsers(models.Model):
+    _inherit = 'res.users'
+
+    property_ids = fields.One2many(
+        'estate.property',
+        'salesperson_id',
+        string="Properties",
+        domain=[('state', '!=', 'cancelled')]
+    )