[IMP] Product Kit : Added a new product type

In this commit we have introduced following changes:

- Updated unlink method to delete all subproducts if main product is deleted.
- Updated both sales and invoice reports to print subproducts or not based on a
  boolean field 'print_kit'.

Author: vkas-odoo

Diff for: product_kit/__init__.py

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

Diff for: product_kit/__manifest__.py

@@ -9,12 +9,15 @@
     'depends': [
         'base',
         'stock',
-        'sale_management'
+        'sale_management',
+        'account'
     ],
-    'data' : [
+    'data': [
         'views/product_template_views.xml',
         'views/sale_order_views.xml',
         'views/kit_products_wizard_views.xml',
         'security/ir.model.access.csv',
+        'views/sale_order_report.xml',
+        'views/account_move_views.xml',
     ]
-}
+}

Diff for: product_kit/models/__init__.py

@@ -1,4 +1,6 @@
 from . import product_template
 from . import sale_order_line
 from . import kit_products_wizard_line
-from . import kit_products_wizard
+from . import kit_products_wizard
+from . import sale_order
+from . import account_move

Diff for: product_kit/models/account_move.py

@@ -0,0 +1,14 @@
+from odoo import models, fields
+
+
+class AccountMove(models.Model):
+    _inherit = "account.move"
+
+    sale_id = fields.Many2one('sale.order', string="Sale Order", compute='_compute_sale_id', store=True, readonly=True)
+    print_kit = fields.Boolean(related="sale_id.print_kit", readonly=True)
+    from_wizard = fields.Boolean(related="line_ids.sale_line_ids.from_wizard", store=True, readonly=True)
+
+    def _compute_sale_id(self):
+        for move in self:
+            sale_orders = move.line_ids.mapped("sale_line_ids.order_id")
+            move.sale_id = sale_orders and sale_orders[0] or False

Diff for: product_kit/models/kit_products_wizard.py

@@ -1,5 +1,6 @@
 from odoo import models, fields, api
 
+
 class KitProductsWizard(models.TransientModel):
     _name = "kit.products.wizard"
     _description = "Kit Products Wizard"
@@ -33,13 +34,13 @@ def default_get(self, fields_list):
                     kit_product_vals.append((0, 0, {
                         "product_id": line.product_id.id,
                         "product_qty": line.product_uom_qty,
-                        "price_unit": line.product_id.list_price, 
+                        "price_unit": line.product_id.list_price,
                     }))
             else:
                 for kit_product in product_template.kit_product_ids:
                     kit_product_vals.append((0, 0, {
                         "product_id": kit_product.id,
-                        "product_qty": 1,  
+                        "product_qty": 1,
                         "price_unit": kit_product.list_price,
                     }))
 
@@ -48,40 +49,52 @@ def default_get(self, fields_list):
         return res
 
     def action_generate_order_lines(self):
-        """Creates sale order lines for each kit product in the wizard"""
+        """Updates or creates sale order lines for each kit product in the wizard"""
         self.ensure_one()
 
         main_product_line = self.env["sale.order.line"].search([
             ("order_id", "=", self.sale_order_id.id),
-            ("product_id", "in", self.product_template_id.product_variant_ids.ids), 
+            ("product_id", "in", self.product_template_id.product_variant_ids.ids),
         ], limit=1)
-        # breakpoint()
-        original_price = main_product_line.price_unit if main_product_line else self.product_template_id.list_price
 
-        total_price = original_price 
+        original_price = self.product_template_id.list_price
+        total_price = original_price
 
         for kit_product in self.kit_product_ids:
+            existing_line = self.env["sale.order.line"].search([
+                ("order_id", "=", self.sale_order_id.id),
+                ("product_id", "=", kit_product.product_id.id),
+                ("from_wizard", "=", True),
+            ], limit=1)
 
-            sub_total = kit_product.product_qty * kit_product.price_unit
+            sub_total = kit_product.product_qty * kit_product.product_id.list_price
             total_price += sub_total
-            self.env["sale.order.line"].create({
-                "name": kit_product.product_id.name,  
-                "order_id": self.sale_order_id.id,
-                "product_id": kit_product.product_id.id,
-                "product_uom_qty": kit_product.product_qty,
-                "price_unit": 0.00,
-                "from_wizard": True,
-            })
+
+            if existing_line:
+                existing_line.write({
+                    "product_uom_qty": kit_product.product_qty,
+                    "price_unit": 0.00,  # Ensure sub-products remain zero-priced
+                })
+            else:
+                self.env["sale.order.line"].create({
+                    "name": kit_product.product_id.name,
+                    "order_id": self.sale_order_id.id,
+                    "product_id": kit_product.product_id.id,
+                    "product_uom_qty": kit_product.product_qty,
+                    "price_unit": 0.00,
+                    "from_wizard": True,
+                })
 
         if main_product_line:
             main_product_line.price_unit = total_price * main_product_line.product_uom_qty
         else:
             self.env["sale.order.line"].create({
                 "order_id": self.sale_order_id.id,
                 "product_id": self.product_template_id.product_variant_id.id,
-                "product_uom_qty": 1,  
-                "price_unit": total_price,  
+                "product_uom_qty": 1,
+                "price_unit": total_price,
                 "name": self.product_template_id.name,
-            })  
+            })
 
         return {"type": "ir.actions.act_window_close"}
+

Diff for: product_kit/models/kit_products_wizard_line.py

@@ -1,5 +1,6 @@
 from odoo import models, fields
 
+
 class KitProductsWizardLine(models.TransientModel):
     _name = "kit.products.wizard.line"
     _description = "Kit Products Wizard Line"
@@ -8,5 +9,3 @@ class KitProductsWizardLine(models.TransientModel):
     product_id = fields.Many2one("product.product", string="Product")
     product_qty = fields.Float(string="Quantity")
     price_unit = fields.Float(string="Price")
-
-

Diff for: product_kit/models/product_template.py

@@ -1,10 +1,11 @@
 from odoo import models, fields
 
+
 class ProductTemplate(models.Model):
     _inherit = 'product.template'
 
     is_kit = fields.Boolean(string="Is Kit")
     kit_product_ids = fields.Many2many(
-        'product.product', 
+        'product.product',
         string="Sub Products"
-    )
+    )

Diff for: product_kit/models/sale_order.py

@@ -0,0 +1,7 @@
+from odoo import models, fields
+
+
+class SaleOrder(models.Model):
+    _inherit = "sale.order"
+
+    print_kit = fields.Boolean(string="Print in report", default=True)

Diff for: product_kit/models/sale_order_line.py

@@ -7,7 +7,6 @@ class SaleOrderLine(models.Model):
     is_kit = fields.Boolean(related="product_template_id.is_kit", store=True)
     from_wizard = fields.Boolean(string="Created from Wizard", default=False)
 
-
     def action_open_kit_wizard(self):
         """Opens the wizard with Sale Order ID in context"""
         return {
@@ -17,7 +16,21 @@ def action_open_kit_wizard(self):
             "view_mode": "form",
             "target": "new",
             "context": {
-                "default_sale_order_id": self.order_id.id,  
+                "default_sale_order_id": self.order_id.id,
                 "default_product_template_id": self.product_template_id.id,
             }
         }
+
+    def unlink(self):
+        """Delete only the related wizard-generated sale order lines when a main product line is deleted."""
+        main_product_lines = self.filtered(lambda line: not line.from_wizard)
+
+        for main_line in main_product_lines:
+            kit_lines = self.search([
+                ("order_id", "=", main_line.order_id.id),
+                ("from_wizard", "=", True),
+                ("product_id", "in", main_line.product_id.kit_product_ids.ids),
+            ])
+            kit_lines.unlink()
+
+        return super().unlink()

Diff for: product_kit/views/account_move_views.xml

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <template id="report_invoice_document" inherit_id="account.report_invoice_document">
+        <xpath expr="//tbody/t[@t-foreach='lines']"  position="attributes">
+            <attribute name="t-if">line.move_id.sale_id.print_kit or not line.sale_line_ids.from_wizard</attribute>
+        </xpath>
+    </template>
+</odoo>
+<!-- //table[@name='invoice_line_table']//tbody[@hasclass('invoice_tbody')] -->

Diff for: product_kit/views/sale_order_report.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+    <template id="report_saleorder_document_inherit" inherit_id="sale.sale_order_portal_content">
+        <xpath expr="//table[@id='sales_order_table']//tbody[@class='sale_tbody']//t[@t-foreach='lines_to_report']" position="attributes">
+            <attribute name="t-if">
+                (sale_order.print_kit and line.from_wizard) or (not line.from_wizard)
+            </attribute>
+        </xpath>
+    </template>
+    <template id="report_saleorder_document" inherit_id="sale.report_saleorder_document">
+        <xpath expr="//tbody/t[@t-foreach='lines_to_report']" position="attributes">
+            <attribute name="t-if">doc.print_kit or not line.from_wizard</attribute>
+        </xpath>
+    </template>
+</odoo>

Diff for: product_kit/views/sale_order_views.xml

@@ -5,6 +5,9 @@
         <field name="model">sale.order</field>
         <field name="inherit_id" ref="sale.view_order_form"/>
         <field name="arch" type="xml">
+            <xpath expr="//field[@name='payment_term_id']" position="after">
+                <field name="print_kit"/>
+            </xpath>
             <xpath expr="//field[@name='order_line']/list//field[@name='product_template_id']" position="after">
                 <button name="action_open_kit_wizard" type="object" string="+" class="oe_highlight"
                     invisible = "not is_kit or state in ['sale']"/>