[IMP] real_estate: Enforce business logic and modular invoicing integration

- Enforced business rules for property deletion and offer validation
- Linked available properties to users via domain and UI integration
- Introduced estate_account as a modular extension for invoicing
  with commission and fixed administrative charges

Author: metu-odoo

estate_account/__init__.py

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

estate_account/__manifest__.py

@@ -0,0 +1,11 @@
+{
+    "name": "Estate Account",
+    "version": "1.0",
+    "depends": ["real_estate", "account"],
+    "author": "",
+    "category": "Accounts for Estate",
+    "data": [],
+    "license": "LGPL-3",
+    "installable": True,
+    "application": True,
+}

estate_account/models/__init__.py

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

estate_account/models/estate_property.py

@@ -0,0 +1,48 @@
+from odoo import Command, models
+
+
+class Property(models.Model):
+    _inherit = "estate.property"
+
+    def action_sold(self):
+        for property in self:
+            journal = self.env["account.journal"].search(
+                [("type", "=", "sale")], limit=1
+            )
+
+            commission = property.selling_price * 0.06
+            discount = property.selling_price * 0.05
+            invoice_lines = [
+                Command.create(
+                    {
+                        "name": "6% Commission",
+                        "quantity": 1,
+                        "price_unit": commission,
+                    }
+                ),
+                Command.create(
+                    {
+                        "name": "Administrative Fees",
+                        "quantity": 1,
+                        "price_unit": 100.00,
+                    }
+                ),
+                Command.create(
+                    {
+                        "name": "Discount",
+                        "quantity": 1,
+                        "price_unit": -discount,
+                    }
+                ),
+            ]
+
+            invoice_vals = {
+                "partner_id": property.buyer_id.id,
+                "move_type": "out_invoice",
+                "journal_id": journal.id,
+                "invoice_line_ids": invoice_lines,
+            }
+
+            self.env["account.move"].create(invoice_vals)
+
+        return super().action_sold()

real_estate/__manifest__.py

@@ -11,6 +11,7 @@
         "views/estate_property_type_views.xml",
         "views/estate_property_tag_views.xml",
         "views/estate_property_menus.xml",
+        "views/res_users_views.xml",
     ],
     "license": "LGPL-3",
     "installable": True,

real_estate/models/__init__.py

@@ -2,3 +2,4 @@
 from . import estate_property_offers
 from . import estate_property_tags
 from . import estate_property_types
+from . import res_users

real_estate/models/estate_property.py

@@ -88,6 +88,9 @@ def _onchange_garden(self):
 
     def action_sold(self):
         for record in self:
+            if not record.state == "offer_accepted":
+                raise UserError("Accept an offer first")
+
             if record.state == "cancelled":
                 raise UserError("You cannot mark a cancelled property as sold.")
             record.state = "sold"
@@ -117,3 +120,11 @@ def _check_selling_price(self):
                 raise ValidationError(
                     "Selling price must be at least 90% of the expected price."
                 )
+
+    @api.ondelete(at_uninstall=False)
+    def _check_deletion_state(self):
+        for rec in self:
+            if rec.state not in ["new", "cancelled"]:
+                raise UserError(
+                    "Only properties in 'New' or 'Cancelled' state can be deleted."
+                )

real_estate/models/estate_property_offers.py

@@ -67,3 +67,18 @@ def action_refuse_offer(self):
             "Offer price must be strictly positive.",
         )
     ]
+
+    @api.model_create_multi
+    def create(self, vals):
+        for val in vals:
+            property_id = self.env["estate.property"].browse(val["property_id"])
+
+            if property_id.state == "new":
+                property_id.state = "offer_received"
+
+            if val["price"] <= property_id.best_offer:
+                raise UserError(
+                    f"The offer must be higher than {property_id.best_offer}"
+                )
+
+        return super().create(vals)

real_estate/models/res_users.py

@@ -0,0 +1,11 @@
+from odoo import fields, models
+
+
+class ResUsers(models.Model):
+    _inherit = "res.users"
+
+    property_ids = fields.One2many(
+        comodel_name="estate.property",
+        inverse_name="seller_id",
+        domain="[('state', 'in', ('new','offer_received'))]",
+    )

real_estate/views/estate_property_menus.xml

@@ -1,15 +1,18 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <odoo>
     <!-- Root menu -->
-    <menuitem id="estate_menu_root" name="Real Estate"/>
+    <menuitem id="estate_menu_root" name="Real Estate" />
 
     <!-- First level submenu under root -->
-    <menuitem id="estate_menu_advertisements" name="Advertisements" parent="estate_menu_root"/>
-    <menuitem id="estate_menu_settings" name="Settings" parent="estate_menu_root"/>
+    <menuitem id="estate_menu_advertisements" name="Advertisements" parent="estate_menu_root" />
+    <menuitem id="estate_menu_settings" name="Settings" parent="estate_menu_root" />
 
     <!-- Action menu under first level -->
-    <menuitem id="estate_property_menu_action" parent="estate_menu_advertisements" action="real_estate_main_action"/>
-    <menuitem id="estate_property_type_menu_action" parent="estate_menu_settings" action="estate_property_type_action"/>
-    <menuitem id="estate_property_tag_menu_action" parent="estate_menu_settings" action="estate_property_tag_action"/>
+    <menuitem id="estate_property_menu_action" parent="estate_menu_advertisements"
+        action="real_estate_main_action" />
+    <menuitem id="estate_property_type_menu_action" parent="estate_menu_settings"
+        action="estate_property_type_action" />
+    <menuitem id="estate_property_tag_menu_action" parent="estate_menu_settings"
+        action="estate_property_tag_action" />
 
 </odoo>