From 39c6c9d595de9a154ef1afacb0bf21249c031dab Mon Sep 17 00:00:00 2001
From: times-odoo <times@odoo.com>
Date: Wed, 3 Jun 2026 14:51:54 +0530
Subject: [PATCH 1/5] [ADD] sale_kit_management: introduce dynamic sale kit
 product management and order configurator

Implemented a product kit architecture allowing users to bundle sub-products,
dynamically compute bundle pricing, and explode kit selections into sale orders.

Rationale:
Sales teams need a flexible mechanism to sell multi-component product bundles
(kits) without manually tracking constituent items or distorting unit pricing
structures. This module addresses that by introducing a kit layout toggle on
products, establishing automatic cost roll-ups based on child line assignments,
and integrating an interactive configuration wizard inside the sales quotation
interface to generate dependent order lines dynamically.

Technical choices:
- Created 'product.subproduct.line' and 'sale.order.kit.config.line' models
  to store static templates and active configuration states respectively.
- Overrode CRUD operations on subproduct lines to automatically recompute bundle
  prices using aggregate methods ('_compute_kit_list_price').
- Inherited 'sale.order.line' to establish tree cascading links ('kit_parent_line_id')
  and overrode 'write' and 'unlink' to mirror quantity scale changes or handle
  recursive removal adjustments automatically.
- Designed a 'sale.order.kit' configuration wizard using 'default_get' to preload
  kit architectures and inject exploded subproduct items as discrete, protected,
  muted rows contextually attached beneath a section header block.
- Applied multi-level list restrictions inside view templates using column-invisible
  and read-only flags to block explicit manual overrides of derived child row values.

Task Reference: 6268133
---
 sale_kit_management/__init__.py               |   2 +
 sale_kit_management/__manifest__.py           |  16 ++
 sale_kit_management/models/__init__.py        |   4 +
 .../models/product_subproduct_line.py         |  29 ++++
 .../models/product_template.py                |  25 ++++
 .../models/sale_order_kit_config_line.py      |  11 ++
 sale_kit_management/models/sale_order_line.py |  64 ++++++++
 .../security/ir.model.access.csv              |   6 +
 sale_kit_management/views/product_views.xml   |  32 ++++
 .../views/sale_order_views.xml                |  63 ++++++++
 sale_kit_management/wizard/__init__.py        |   1 +
 sale_kit_management/wizard/sale_order_kit.py  | 138 ++++++++++++++++++
 .../wizard/sale_order_kit_views.xml           |  49 +++++++
 13 files changed, 440 insertions(+)
 create mode 100644 sale_kit_management/__init__.py
 create mode 100644 sale_kit_management/__manifest__.py
 create mode 100644 sale_kit_management/models/__init__.py
 create mode 100644 sale_kit_management/models/product_subproduct_line.py
 create mode 100644 sale_kit_management/models/product_template.py
 create mode 100644 sale_kit_management/models/sale_order_kit_config_line.py
 create mode 100644 sale_kit_management/models/sale_order_line.py
 create mode 100644 sale_kit_management/security/ir.model.access.csv
 create mode 100644 sale_kit_management/views/product_views.xml
 create mode 100644 sale_kit_management/views/sale_order_views.xml
 create mode 100644 sale_kit_management/wizard/__init__.py
 create mode 100644 sale_kit_management/wizard/sale_order_kit.py
 create mode 100644 sale_kit_management/wizard/sale_order_kit_views.xml

diff --git a/sale_kit_management/__init__.py b/sale_kit_management/__init__.py
new file mode 100644
index 00000000000..9b4296142f4
--- /dev/null
+++ b/sale_kit_management/__init__.py
@@ -0,0 +1,2 @@
+from . import models
+from . import wizard
diff --git a/sale_kit_management/__manifest__.py b/sale_kit_management/__manifest__.py
new file mode 100644
index 00000000000..88797770afe
--- /dev/null
+++ b/sale_kit_management/__manifest__.py
@@ -0,0 +1,16 @@
+{
+    'name': 'Sale Kit Management',
+    'version': "1.0",
+    'category': "Sales/Sales",
+    'description': "Creates a Kit Option in Products and Sales Order",
+    'depends': ['sale_management'],
+    'data': [
+        'security/ir.model.access.csv',
+        'wizard/sale_order_kit_views.xml',
+        'views/product_views.xml',
+        'views/sale_order_views.xml',
+    ],
+    'installable': True,
+    'author': "times",
+    'license': "LGPL-3",
+}
diff --git a/sale_kit_management/models/__init__.py b/sale_kit_management/models/__init__.py
new file mode 100644
index 00000000000..28136695f4f
--- /dev/null
+++ b/sale_kit_management/models/__init__.py
@@ -0,0 +1,4 @@
+from . import product_template
+from . import product_subproduct_line
+from . import sale_order_line
+from . import sale_order_kit_config_line
diff --git a/sale_kit_management/models/product_subproduct_line.py b/sale_kit_management/models/product_subproduct_line.py
new file mode 100644
index 00000000000..9e84a4255a0
--- /dev/null
+++ b/sale_kit_management/models/product_subproduct_line.py
@@ -0,0 +1,29 @@
+from odoo import api, fields, models
+
+
+class ProductSubproductLine(models.Model):
+    _name = 'product.subproduct.line'
+    _description = "Subproducts"
+
+    product_tmpl_id = fields.Many2one('product.template', ondelete='cascade')
+    product_id = fields.Many2one('product.product', string="Product")
+    product_uom_qty = fields.Float(related='product_id.qty_available', string="Quantity on Hand")
+    kit_unit_qty = fields.Float(string="Kit Unit Qty")
+    price_unit = fields.Float(related='product_id.lst_price', string="Unit Price")
+
+    @api.model_create_multi
+    def create(self, vals_list):
+        lines = super().create(vals_list)
+        lines.mapped('product_tmpl_id')._compute_kit_list_price()
+        return lines
+
+    def write(self, vals):
+        res = super().write(vals)
+        self.mapped('product_tmpl_id')._compute_kit_list_price()
+        return res
+
+    def unlink(self):
+        templates = self.mapped('product_tmpl_id')
+        res = super().unlink()
+        templates._compute_kit_list_price()
+        return res
diff --git a/sale_kit_management/models/product_template.py b/sale_kit_management/models/product_template.py
new file mode 100644
index 00000000000..d2a2fa95f39
--- /dev/null
+++ b/sale_kit_management/models/product_template.py
@@ -0,0 +1,25 @@
+from odoo import api, fields, models
+
+
+class ProductTemplate(models.Model):
+    _inherit = 'product.template'
+
+    is_kit = fields.Boolean()
+    subproduct_ids = fields.One2many('product.subproduct.line', 'product_tmpl_id')
+
+    @api.depends('is_kit', 'subproduct_ids.price_unit', 'subproduct_ids.kit_unit_qty')
+    def _compute_kit_list_price(self):
+        for product in self:
+            if product.is_kit and product.subproduct_ids:
+                product.list_price = sum(
+                    sub.price_unit * sub.kit_unit_qty
+                    for sub in product.subproduct_ids
+                )
+
+    @api.onchange('subproduct_ids')
+    def _onchange_subproduct_ids(self):
+        if self.is_kit and self.subproduct_ids:
+            self.list_price = sum(
+                sub.price_unit * sub.kit_unit_qty
+                for sub in self.subproduct_ids
+            )
diff --git a/sale_kit_management/models/sale_order_kit_config_line.py b/sale_kit_management/models/sale_order_kit_config_line.py
new file mode 100644
index 00000000000..c778e67bef9
--- /dev/null
+++ b/sale_kit_management/models/sale_order_kit_config_line.py
@@ -0,0 +1,11 @@
+from odoo import fields, models
+
+
+class SaleOrderKitConfigLine(models.Model):
+    _name = 'sale.order.kit.config.line'
+    _description = "Kit Configuration Line"
+
+    sale_order_line_id = fields.Many2one('sale.order.line', string="Kit Parent Line", ondelete='cascade', required=True, index=True,)
+    product_id = fields.Many2one('product.product', string="Product", required=True)
+    kit_unit_qty = fields.Float(string="Kit Unit Qty", default=1.0)
+    price_unit = fields.Float(string="Unit Price")
diff --git a/sale_kit_management/models/sale_order_line.py b/sale_kit_management/models/sale_order_line.py
new file mode 100644
index 00000000000..48f7109769a
--- /dev/null
+++ b/sale_kit_management/models/sale_order_line.py
@@ -0,0 +1,64 @@
+from odoo import _, api, fields, models
+
+
+class SaleOrderLine(models.Model):
+    _inherit = 'sale.order.line'
+
+    is_kit = fields.Boolean(related='product_id.product_tmpl_id.is_kit')
+    is_kit_subproduct = fields.Boolean(string="Is Kit Subproduct", default=False)
+    kit_unit_qty = fields.Float(string="Kit Unit Qty")
+    kit_parent_line_id = fields.Many2one('sale.order.line', string="Kit Parent Line", ondelete='cascade', index=True,)
+    has_kit_subproducts = fields.Boolean(compute='_compute_has_kit_subproducts')
+    kit_config_line_ids = fields.One2many('sale.order.kit.config.line', 'sale_order_line_id', string="Kit Configuration")
+
+    @api.depends('order_id.order_line.kit_parent_line_id')
+    def _compute_has_kit_subproducts(self):
+        for line in self:
+            if line.is_kit and line.id:
+                line.has_kit_subproducts = any(
+                    l.kit_parent_line_id.id == line.id
+                    for l in line.order_id.order_line
+                    if l.is_kit_subproduct
+                )
+            else:
+                line.has_kit_subproducts = False
+
+    def _compute_kit_price(self):
+        for line in self:
+            if line.is_kit and line.kit_config_line_ids:
+                line.price_unit = sum(
+                    cl.price_unit * cl.product_uom_qty
+                    for cl in line.kit_config_line_ids
+                )
+
+    def write(self, vals):
+        result = super().write(vals)
+        if 'product_uom_qty' in vals:
+            for line in self.filtered(lambda l: l.is_kit and l.id):
+                child_lines = self.env['sale.order.line'].search([
+                    ('kit_parent_line_id', '=', line.id),
+                    ('is_kit_subproduct', '=', True),
+                ])
+                for child in child_lines:
+                    if child.kit_unit_qty:
+                        child.product_uom_qty = line.product_uom_qty * child.kit_unit_qty
+        return result
+
+    def unlink(self):
+        child_lines = self.env['sale.order.line'].search([
+            ('kit_parent_line_id', 'in', self.ids)
+        ])
+        if child_lines:
+            child_lines.unlink()
+        return super().unlink()
+
+    def action_open_kit_configurator(self):
+        self.ensure_one()
+        return {
+            'name': _("Kit"),
+            'type': 'ir.actions.act_window',
+            'res_model': 'sale.order.kit',
+            'view_mode': 'form',
+            'target': 'new',
+            'context': {'default_sale_order_line_id': self.id},
+        }
diff --git a/sale_kit_management/security/ir.model.access.csv b/sale_kit_management/security/ir.model.access.csv
new file mode 100644
index 00000000000..9f4e0029266
--- /dev/null
+++ b/sale_kit_management/security/ir.model.access.csv
@@ -0,0 +1,6 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_product_subproduct_line,access_product_subproduct_line,model_product_subproduct_line,sales_team.group_sale_salesman,1,1,1,1
+access_sale_order_kit,access_sale_order_kit,model_sale_order_kit,sales_team.group_sale_salesman,1,1,1,1
+access_sale_order_kit_line,access_sale_order_kit_line,model_sale_order_kit_line,sales_team.group_sale_salesman,1,1,1,1
+access_sale_order_kit_config_line,access_sale_order_kit_config_line,model_sale_order_kit_config_line,sales_team.group_sale_salesman,1,1,1,1
+
diff --git a/sale_kit_management/views/product_views.xml b/sale_kit_management/views/product_views.xml
new file mode 100644
index 00000000000..3a97936e4db
--- /dev/null
+++ b/sale_kit_management/views/product_views.xml
@@ -0,0 +1,32 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <record id="product_form_kit_view" model="ir.ui.view">
+        <field name="name">product.template.form.kit</field>
+        <field name="model">product.template</field>
+        <field name="inherit_id" ref="product.product_template_form_view" />
+        <field name="arch" type="xml">
+
+            <xpath expr="//span[@name='sale_option']" position="after">
+                <span name="kit_option" class="d-inline-flex">
+                    <field name="is_kit" string="Is Kit" />
+                    <label string="Kit" for="is_kit" />
+                </span>
+            </xpath>
+
+            <xpath expr="//notebook/page[@name='general_information']" position="after">
+                <page name="subproducts" string="Subproducts" invisible="not is_kit">
+                    <field name="subproduct_ids">
+                        <list editable="bottom">
+                            <field name="product_id" string="Product" />
+                            <field name="product_uom_qty" string="On Hand Qty" />
+                            <field name="kit_unit_qty" string="Kit Unit Qty"
+                                decoration-info="product_id" />
+                            <field name="price_unit" string="Unit Price" />
+                        </list>
+                    </field>
+                </page>
+            </xpath>
+
+        </field>
+    </record>
+</odoo>
\ No newline at end of file
diff --git a/sale_kit_management/views/sale_order_views.xml b/sale_kit_management/views/sale_order_views.xml
new file mode 100644
index 00000000000..77fb0c77d24
--- /dev/null
+++ b/sale_kit_management/views/sale_order_views.xml
@@ -0,0 +1,63 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <record id="sale_order_form_kit" model="ir.ui.view">
+        <field name="name">sale.order.form.kit</field>
+        <field name="model">sale.order</field>
+        <field name="inherit_id" ref="sale.view_order_form" />
+        <field name="arch" type="xml">
+
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_id' and @optional='hide']"
+                position="after">
+                <button name="action_open_kit_configurator"
+                    type="object"
+                    icon="fa-cubes"
+                    invisible="not is_kit or has_kit_subproducts" width="25px" />
+            </xpath>
+
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_id']"
+                position="before">
+                <field name="is_kit_subproduct" column_invisible="1" />
+                <field name="has_kit_subproducts" column_invisible="1" />
+            </xpath>
+
+            <xpath expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list"
+                position="attributes">
+                <attribute name="decoration-muted">is_kit_subproduct</attribute>
+            </xpath>
+
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_id' and @optional='hide']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct or not product_updatable</attribute>
+            </xpath>
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_uom_qty']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct</attribute>
+            </xpath>
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_uom_id']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct</attribute>
+            </xpath>
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='price_unit']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct</attribute>
+            </xpath>
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='tax_ids']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct</attribute>
+            </xpath>
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='discount']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct</attribute>
+            </xpath>
+
+        </field>
+    </record>
+</odoo>
\ No newline at end of file
diff --git a/sale_kit_management/wizard/__init__.py b/sale_kit_management/wizard/__init__.py
new file mode 100644
index 00000000000..28ae5190a0f
--- /dev/null
+++ b/sale_kit_management/wizard/__init__.py
@@ -0,0 +1 @@
+from . import sale_order_kit
diff --git a/sale_kit_management/wizard/sale_order_kit.py b/sale_kit_management/wizard/sale_order_kit.py
new file mode 100644
index 00000000000..5f91188ae50
--- /dev/null
+++ b/sale_kit_management/wizard/sale_order_kit.py
@@ -0,0 +1,138 @@
+from odoo import _, api, fields, models
+
+
+class SaleOrderKitLine(models.TransientModel):
+    _name = 'sale.order.kit.line'
+    _description = "Kit Wizard Line"
+
+    wizard_id = fields.Many2one('sale.order.kit', ondelete='cascade')
+    product_id = fields.Many2one('product.product', string="Product", required=True)
+    kit_unit_qty = fields.Float(string="Kit Unit Qty", default=1.0)
+    price_unit = fields.Float(string="Price")
+    amount = fields.Float(string="Amount", compute='_compute_amount', store=True)
+
+    @api.depends('kit_unit_qty', 'price_unit')
+    def _compute_amount(self):
+        for line in self:
+            line.amount = line.kit_unit_qty * line.price_unit
+
+
+class SaleOrderKit(models.TransientModel):
+    _name = 'sale.order.kit'
+    _description = "Kit Configurator Wizard"
+
+    sale_order_line_id = fields.Many2one('sale.order.line', required=True)
+    parent_product = fields.Many2one(
+        'product.product',
+        string="Product",
+        compute='_compute_parent_product',
+    )
+    kit_line_ids = fields.One2many('sale.order.kit.line', 'wizard_id', string="Sub Products")
+    kit_total_price = fields.Float(
+        string="Kit Price",
+        compute='_compute_kit_total_price',
+    )
+
+    @api.depends('sale_order_line_id.product_id')
+    def _compute_parent_product(self):
+        for rec in self:
+            rec.parent_product = rec.sale_order_line_id.product_id
+
+    @api.depends('kit_line_ids.price_unit', 'kit_line_ids.kit_unit_qty')
+    def _compute_kit_total_price(self):
+        for rec in self:
+            rec.kit_total_price = sum(
+                line.price_unit * line.kit_unit_qty
+                for line in rec.kit_line_ids
+            )
+
+    @api.model
+    def default_get(self, fields):
+        res = super().default_get(fields)
+        sol_id = res.get('sale_order_line_id') or self.env.context.get('default_sale_order_line_id')
+        if sol_id:
+            sol = self.env['sale.order.line'].browse(sol_id)
+
+            if sol.kit_config_line_ids:
+                lines = [(0, 0, {
+                    'product_id': line.product_id.id,
+                    'kit_unit_qty': line.kit_unit_qty,
+                    'price_unit': line.price_unit,
+                }) for line in sol.kit_config_line_ids]
+
+            else:
+                product_tmpl = sol.product_id.product_tmpl_id
+                lines = [(0, 0, {
+                    'product_id': sub.product_id.id,
+                    'kit_unit_qty': sub.kit_unit_qty,
+                    'price_unit': sub.price_unit,
+                }) for sub in product_tmpl.subproduct_ids]
+
+            res['kit_line_ids'] = lines
+        return res
+
+    def action_confirm(self):
+        self.ensure_one()
+        sol = self.sale_order_line_id
+
+        sol.kit_config_line_ids.unlink()
+        config_vals = [{
+            'sale_order_line_id': sol.id,
+            'product_id': line.product_id.id,
+            'kit_unit_qty': line.kit_unit_qty,
+            'price_unit': line.price_unit,
+        } for line in self.kit_line_ids]
+        self.env['sale.order.kit.config.line'].create(config_vals)
+
+        sol.price_unit = sum(
+            line.price_unit * line.kit_unit_qty
+            for line in self.kit_line_ids
+        )
+
+        return {'type': 'ir.actions.act_window_close'}
+
+    def action_add_to_sol(self):
+        self.ensure_one()
+        sol = self.sale_order_line_id
+        order = sol.order_id
+        product_name = sol.product_id.display_name
+
+        sol.kit_config_line_ids.unlink()
+        config_vals = [{
+            'sale_order_line_id': sol.id,
+            'product_id': line.product_id.id,
+            'kit_unit_qty': line.kit_unit_qty,
+            'price_unit': line.price_unit,
+        } for line in self.kit_line_ids]
+        self.env['sale.order.kit.config.line'].create(config_vals)
+
+        sol.price_unit = sum(
+            line.price_unit * line.kit_unit_qty
+            for line in self.kit_line_ids
+        )
+
+        order.order_line.filtered(
+            lambda l: l.kit_parent_line_id.id == sol.id
+        ).unlink()
+
+        self.env['sale.order.line'].create({
+            'order_id': order.id,
+            'display_type': 'line_section',
+            'name': f"Subproducts of {product_name}",
+            'sequence': sol.sequence + 1,
+            'kit_parent_line_id': sol.id,
+        })
+
+        for line in self.kit_line_ids:
+            self.env['sale.order.line'].create({
+                'order_id': order.id,
+                'product_id': line.product_id.id,
+                'product_uom_qty': line.kit_unit_qty * sol.product_uom_qty,
+                'price_unit': line.price_unit,
+                'sequence': sol.sequence + 2,
+                'is_kit_subproduct': True,
+                'kit_parent_line_id': sol.id,
+                'kit_unit_qty': line.kit_unit_qty,
+            })
+
+        return {'type': 'ir.actions.act_window_close'}
diff --git a/sale_kit_management/wizard/sale_order_kit_views.xml b/sale_kit_management/wizard/sale_order_kit_views.xml
new file mode 100644
index 00000000000..e97e0ef32ec
--- /dev/null
+++ b/sale_kit_management/wizard/sale_order_kit_views.xml
@@ -0,0 +1,49 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<odoo>
+    <record id="sale_order_kit_form_wizard" model="ir.ui.view">
+        <field name="name">sale.order.kit.form</field>
+        <field name="model">sale.order.kit</field>
+        <field name="arch" type="xml">
+            <form string="Kit Configurator">
+                <sheet>
+
+                    <group>
+                        <group>
+                            <label string="Kit Product" class="o_form_label me-2"
+                                for="parent_product" />
+                            <field name="parent_product" readonly="1" nolabel="1" />
+                        </group>
+                        <group>
+                            <field name="kit_total_price" widget="monetary" readonly="1" />
+                        </group>
+                    </group>
+
+                    <separator string="Sub Products" />
+                    <field name="kit_line_ids" nolabel="1">
+                        <list editable="bottom">
+                            <field name="product_id" string="Product" />
+                            <field name="kit_unit_qty" string="Kit Unit Qty"
+                                decoration-info="product_id" />
+                            <field name="price_unit" string="Price" />
+                            <field name="amount" string="Amount" readonly="1" />
+                        </list>
+                    </field>
+
+
+                </sheet>
+                <footer>
+                    <button string="Add to Order Line"
+                        name="action_add_to_sol"
+                        type="object"
+                        class="btn btn-primary"
+                        data-hotkey="q" />
+                    <button string="Confirm"
+                        name="action_confirm"
+                        type="object"
+                        class="btn btn-secondary"
+                        data-hotkey="x" />
+                </footer>
+            </form>
+        </field>
+    </record>
+</odoo>

From a192aaf7cf48bdd9b30c0b463b2d998bff2757c1 Mon Sep 17 00:00:00 2001
From: times-odoo <times@odoo.com>
Date: Wed, 3 Jun 2026 17:18:44 +0530
Subject: [PATCH 2/5] [IMP] sale_kit_management: refactor kit price computation
 and handle subproduct line unlinking edge cases

Optimized the kit pricing lifecycle and added recursive recalculation logic
when individual kit subproduct lines are unlinked from a sales order.

Rationale:
The pricing engine needed to be more robust against manual user interventions.
If a user unlinks a child subproduct line directly from the sales order view,
the parent kit line must dynamically lower its 'price_unit' to match the
remaining components. Additionally, if all exploded components are removed,
the system now purges the orphaned section header and resets the parent's unit
price back to its standard standalone catalog fallback value ('lst_price').

Technical choices:
- Converted 'price_unit' in 'product.subproduct.line' to a computed, stored field
  to allow optional manual overrides during initial bundle definitions.
- Renamed and streamlined '_compute_kit_list_price' into a dedicated context
  recalculation method '_recompute_kit_list_price' to decouple it from strict
  api.depends evaluation traps.
- Rewrote 'unlink' in 'sale.order.line' to cleanly capture unlinked child rows,
  re-sum remaining sibling records using aggregate methods, and clear detached
  'line_section' rows when the kit's subproduct array goes blank.
- Streamlined 'default_get' inside 'sale.order.kit' by consolidating the logic
  to rely exclusively on 'default_sale_order_line_id' from the context.
- Cleaned up trailing spaces and redundant configuration line re-creations inside
  the wizard's 'action_add_to_sol' invocation thread.
---
 .../models/product_subproduct_line.py         | 21 ++++++++++----
 .../models/product_template.py                | 11 ++++----
 .../models/sale_order_kit_config_line.py      |  2 +-
 sale_kit_management/models/sale_order_line.py | 28 +++++++++++++++++--
 sale_kit_management/wizard/sale_order_kit.py  | 15 ++--------
 5 files changed, 49 insertions(+), 28 deletions(-)

diff --git a/sale_kit_management/models/product_subproduct_line.py b/sale_kit_management/models/product_subproduct_line.py
index 9e84a4255a0..c937e00f23a 100644
--- a/sale_kit_management/models/product_subproduct_line.py
+++ b/sale_kit_management/models/product_subproduct_line.py
@@ -6,24 +6,33 @@ class ProductSubproductLine(models.Model):
     _description = "Subproducts"
 
     product_tmpl_id = fields.Many2one('product.template', ondelete='cascade')
-    product_id = fields.Many2one('product.product', string="Product")
+    product_id = fields.Many2one('product.product', string="Product", required=True)
     product_uom_qty = fields.Float(related='product_id.qty_available', string="Quantity on Hand")
-    kit_unit_qty = fields.Float(string="Kit Unit Qty")
-    price_unit = fields.Float(related='product_id.lst_price', string="Unit Price")
+    kit_unit_qty = fields.Float(string="Kit Unit Qty", default=1.0)
+    price_unit = fields.Float(string="Unit Price", compute='_compute_price_unit', store=True, readonly=False)
+
+    @api.depends('product_id')
+    def _compute_price_unit(self):
+        for line in self:
+            if line.product_id:
+                line.price_unit = line.product_id.lst_price
+            else:
+                line.price_unit = 0.0
 
     @api.model_create_multi
     def create(self, vals_list):
         lines = super().create(vals_list)
-        lines.mapped('product_tmpl_id')._compute_kit_list_price()
+        lines.mapped('product_tmpl_id')._recompute_kit_list_price()
         return lines
 
     def write(self, vals):
         res = super().write(vals)
-        self.mapped('product_tmpl_id')._compute_kit_list_price()
+        if 'price_unit' in vals or 'kit_unit_qty' in vals or 'product_id' in vals:
+            self.mapped('product_tmpl_id')._recompute_kit_list_price()
         return res
 
     def unlink(self):
         templates = self.mapped('product_tmpl_id')
         res = super().unlink()
-        templates._compute_kit_list_price()
+        templates._recompute_kit_list_price()
         return res
diff --git a/sale_kit_management/models/product_template.py b/sale_kit_management/models/product_template.py
index d2a2fa95f39..7804b92e147 100644
--- a/sale_kit_management/models/product_template.py
+++ b/sale_kit_management/models/product_template.py
@@ -7,13 +7,12 @@ class ProductTemplate(models.Model):
     is_kit = fields.Boolean()
     subproduct_ids = fields.One2many('product.subproduct.line', 'product_tmpl_id')
 
-    @api.depends('is_kit', 'subproduct_ids.price_unit', 'subproduct_ids.kit_unit_qty')
-    def _compute_kit_list_price(self):
-        for product in self:
-            if product.is_kit and product.subproduct_ids:
-                product.list_price = sum(
+    def _recompute_kit_list_price(self):
+        for tmpl in self:
+            if tmpl.is_kit and tmpl.subproduct_ids:
+                tmpl.list_price = sum(
                     sub.price_unit * sub.kit_unit_qty
-                    for sub in product.subproduct_ids
+                    for sub in tmpl.subproduct_ids
                 )
 
     @api.onchange('subproduct_ids')
diff --git a/sale_kit_management/models/sale_order_kit_config_line.py b/sale_kit_management/models/sale_order_kit_config_line.py
index c778e67bef9..9d7e74271df 100644
--- a/sale_kit_management/models/sale_order_kit_config_line.py
+++ b/sale_kit_management/models/sale_order_kit_config_line.py
@@ -5,7 +5,7 @@ class SaleOrderKitConfigLine(models.Model):
     _name = 'sale.order.kit.config.line'
     _description = "Kit Configuration Line"
 
-    sale_order_line_id = fields.Many2one('sale.order.line', string="Kit Parent Line", ondelete='cascade', required=True, index=True,)
+    sale_order_line_id = fields.Many2one('sale.order.line', string="Kit Parent Line", ondelete='cascade', required=True, index=True)
     product_id = fields.Many2one('product.product', string="Product", required=True)
     kit_unit_qty = fields.Float(string="Kit Unit Qty", default=1.0)
     price_unit = fields.Float(string="Unit Price")
diff --git a/sale_kit_management/models/sale_order_line.py b/sale_kit_management/models/sale_order_line.py
index 48f7109769a..336c5bca66e 100644
--- a/sale_kit_management/models/sale_order_line.py
+++ b/sale_kit_management/models/sale_order_line.py
@@ -7,7 +7,7 @@ class SaleOrderLine(models.Model):
     is_kit = fields.Boolean(related='product_id.product_tmpl_id.is_kit')
     is_kit_subproduct = fields.Boolean(string="Is Kit Subproduct", default=False)
     kit_unit_qty = fields.Float(string="Kit Unit Qty")
-    kit_parent_line_id = fields.Many2one('sale.order.line', string="Kit Parent Line", ondelete='cascade', index=True,)
+    kit_parent_line_id = fields.Many2one('sale.order.line', string="Kit Parent Line", ondelete='cascade', index=True)
     has_kit_subproducts = fields.Boolean(compute='_compute_has_kit_subproducts')
     kit_config_line_ids = fields.One2many('sale.order.kit.config.line', 'sale_order_line_id', string="Kit Configuration")
 
@@ -45,12 +45,36 @@ def write(self, vals):
         return result
 
     def unlink(self):
+        subproduct_lines = self.filtered(lambda l: l.is_kit_subproduct)
+        parent_lines = subproduct_lines.mapped('kit_parent_line_id')
+
         child_lines = self.env['sale.order.line'].search([
             ('kit_parent_line_id', 'in', self.ids)
         ])
         if child_lines:
             child_lines.unlink()
-        return super().unlink()
+
+        result = super().unlink()
+
+        for parent in parent_lines.exists():
+            remaining = self.env['sale.order.line'].search([
+                ('kit_parent_line_id', '=', parent.id),
+                ('is_kit_subproduct', '=', True),
+            ])
+            section_lines = self.env['sale.order.line'].search([
+                ('display_type', '=', 'line_section'),
+                ('kit_parent_line_id', '=', parent.id),
+            ])
+            if remaining:
+                parent.price_unit = sum(
+                    child.price_unit * child.kit_unit_qty
+                    for child in remaining
+                )
+            else:
+                if section_lines:
+                    section_lines.unlink()
+                parent.price_unit = parent.product_id.lst_price
+        return result
 
     def action_open_kit_configurator(self):
         self.ensure_one()
diff --git a/sale_kit_management/wizard/sale_order_kit.py b/sale_kit_management/wizard/sale_order_kit.py
index 5f91188ae50..4deae25f12e 100644
--- a/sale_kit_management/wizard/sale_order_kit.py
+++ b/sale_kit_management/wizard/sale_order_kit.py
@@ -1,4 +1,4 @@
-from odoo import _, api, fields, models
+from odoo import api, fields, models
 
 
 class SaleOrderKitLine(models.TransientModel):
@@ -49,7 +49,7 @@ def _compute_kit_total_price(self):
     @api.model
     def default_get(self, fields):
         res = super().default_get(fields)
-        sol_id = res.get('sale_order_line_id') or self.env.context.get('default_sale_order_line_id')
+        sol_id = self.env.context.get('default_sale_order_line_id')
         if sol_id:
             sol = self.env['sale.order.line'].browse(sol_id)
 
@@ -98,23 +98,12 @@ def action_add_to_sol(self):
         product_name = sol.product_id.display_name
 
         sol.kit_config_line_ids.unlink()
-        config_vals = [{
-            'sale_order_line_id': sol.id,
-            'product_id': line.product_id.id,
-            'kit_unit_qty': line.kit_unit_qty,
-            'price_unit': line.price_unit,
-        } for line in self.kit_line_ids]
-        self.env['sale.order.kit.config.line'].create(config_vals)
 
         sol.price_unit = sum(
             line.price_unit * line.kit_unit_qty
             for line in self.kit_line_ids
         )
 
-        order.order_line.filtered(
-            lambda l: l.kit_parent_line_id.id == sol.id
-        ).unlink()
-
         self.env['sale.order.line'].create({
             'order_id': order.id,
             'display_type': 'line_section',

From c86a3ed13e638abbf1141f811e4fba2b4645e331 Mon Sep 17 00:00:00 2001
From: times-odoo <times@odoo.com>
Date: Thu, 4 Jun 2026 10:14:29 +0530
Subject: [PATCH 3/5] [IMP] sale_kit_management: implement ondelete hooks and
 enforce dedicated sequence bases for kit explosion

Refactored the kit breakdown lifecycle using modern Odoo deletion hooks and
introduced a isolated sequencing threshold to safeguard line visual hierarchy.

Rationale:
Relying entirely on overriding the standard 'unlink' model method is fragile
because backend system cleanups or automated actions can pass pre-filtered or
mutated recordsets, leading to execution gaps. Migrating this cleanup logic to
'@api.ondelete' methods ensures consistent performance. Additionally, exploding
subproducts immediately using arbitrary incremental logic ('sequence + 1')
frequently breaks if a user later rearranges regular line items. Isolating all
exploded components above a high-value sequence base index prevents collision errors.

Technical choices:
- Replaced standard 'unlink' method overrides in 'ProductSubproductLine' and
  'SaleOrderLine' with optimized '@api.ondelete(at_uninstall=False)' hook decorators.
- Introduced a computed, stored 'amount' field to tracking line totals in the backend.
- Declared '_KIT_SUBPRODUCT_SEQ_BASE' to safely gate explode rows away from core lines.
- Updated the wizard creation payload to insulate sequence positioning dynamically using
  the current count of existing exploded kit section layouts.
- Overrode 'create' in 'SaleOrderLine' to dynamically scale down any regular, non-kit
  item values targeting indices past the dedicated subproduct barrier.
---
 .../models/product_subproduct_line.py         | 21 ++++++++---
 sale_kit_management/models/sale_order_line.py | 36 ++++++++++++-------
 sale_kit_management/views/product_views.xml   |  1 +
 sale_kit_management/wizard/sale_order_kit.py  | 15 +++++---
 4 files changed, 51 insertions(+), 22 deletions(-)

diff --git a/sale_kit_management/models/product_subproduct_line.py b/sale_kit_management/models/product_subproduct_line.py
index c937e00f23a..b86924f1b72 100644
--- a/sale_kit_management/models/product_subproduct_line.py
+++ b/sale_kit_management/models/product_subproduct_line.py
@@ -10,6 +10,12 @@ class ProductSubproductLine(models.Model):
     product_uom_qty = fields.Float(related='product_id.qty_available', string="Quantity on Hand")
     kit_unit_qty = fields.Float(string="Kit Unit Qty", default=1.0)
     price_unit = fields.Float(string="Unit Price", compute='_compute_price_unit', store=True, readonly=False)
+    amount = fields.Float(string="Amount", compute='_compute_amount', store=True)
+
+    @api.depends('kit_unit_qty', 'price_unit')
+    def _compute_amount(self):
+        for line in self:
+            line.amount = line.kit_unit_qty * line.price_unit
 
     @api.depends('product_id')
     def _compute_price_unit(self):
@@ -31,8 +37,13 @@ def write(self, vals):
             self.mapped('product_tmpl_id')._recompute_kit_list_price()
         return res
 
-    def unlink(self):
-        templates = self.mapped('product_tmpl_id')
-        res = super().unlink()
-        templates._recompute_kit_list_price()
-        return res
+    @api.ondelete(at_uninstall=False)
+    def _ondelete_recompute_kit_price(self):
+        for tmpl in self.mapped('product_tmpl_id'):
+            if tmpl.is_kit:
+                remaining = tmpl.subproduct_ids - self
+                if remaining:
+                    tmpl.list_price = sum(
+                        sub.price_unit * sub.kit_unit_qty
+                        for sub in remaining
+                    )
diff --git a/sale_kit_management/models/sale_order_line.py b/sale_kit_management/models/sale_order_line.py
index 336c5bca66e..35996d8bbc5 100644
--- a/sale_kit_management/models/sale_order_line.py
+++ b/sale_kit_management/models/sale_order_line.py
@@ -4,6 +4,8 @@
 class SaleOrderLine(models.Model):
     _inherit = 'sale.order.line'
 
+    _KIT_SUBPRODUCT_SEQ_BASE = 9990
+
     is_kit = fields.Boolean(related='product_id.product_tmpl_id.is_kit')
     is_kit_subproduct = fields.Boolean(string="Is Kit Subproduct", default=False)
     kit_unit_qty = fields.Float(string="Kit Unit Qty")
@@ -11,6 +13,16 @@ class SaleOrderLine(models.Model):
     has_kit_subproducts = fields.Boolean(compute='_compute_has_kit_subproducts')
     kit_config_line_ids = fields.One2many('sale.order.kit.config.line', 'sale_order_line_id', string="Kit Configuration")
 
+    @api.model_create_multi
+    def create(self, vals_list):
+        for vals in vals_list:
+            if vals.get('is_kit_subproduct') or vals.get('kit_parent_line_id'):
+                continue
+            seq = vals.get('sequence', 0)
+            if seq >= self._KIT_SUBPRODUCT_SEQ_BASE:
+                vals['sequence'] = self._KIT_SUBPRODUCT_SEQ_BASE - 1
+        return super().create(vals_list)
+
     @api.depends('order_id.order_line.kit_parent_line_id')
     def _compute_has_kit_subproducts(self):
         for line in self:
@@ -44,37 +56,35 @@ def write(self, vals):
                         child.product_uom_qty = line.product_uom_qty * child.kit_unit_qty
         return result
 
-    def unlink(self):
-        subproduct_lines = self.filtered(lambda l: l.is_kit_subproduct)
-        parent_lines = subproduct_lines.mapped('kit_parent_line_id')
-
+    @api.ondelete(at_uninstall=False)
+    def _ondelete_handle_kit(self):
         child_lines = self.env['sale.order.line'].search([
-            ('kit_parent_line_id', 'in', self.ids)
+            ('kit_parent_line_id', 'in', self.ids),
         ])
         if child_lines:
             child_lines.unlink()
 
-        result = super().unlink()
-
-        for parent in parent_lines.exists():
+        subproduct_lines = self.filtered(lambda l: l.is_kit_subproduct)
+        for parent in subproduct_lines.mapped('kit_parent_line_id'):
+            if parent in self:
+                continue
             remaining = self.env['sale.order.line'].search([
                 ('kit_parent_line_id', '=', parent.id),
                 ('is_kit_subproduct', '=', True),
             ])
-            section_lines = self.env['sale.order.line'].search([
-                ('display_type', '=', 'line_section'),
-                ('kit_parent_line_id', '=', parent.id),
-            ])
             if remaining:
                 parent.price_unit = sum(
                     child.price_unit * child.kit_unit_qty
                     for child in remaining
                 )
             else:
+                section_lines = self.env['sale.order.line'].search([
+                    ('display_type', '=', 'line_section'),
+                    ('kit_parent_line_id', '=', parent.id),
+                ])
                 if section_lines:
                     section_lines.unlink()
                 parent.price_unit = parent.product_id.lst_price
-        return result
 
     def action_open_kit_configurator(self):
         self.ensure_one()
diff --git a/sale_kit_management/views/product_views.xml b/sale_kit_management/views/product_views.xml
index 3a97936e4db..1619169f9b2 100644
--- a/sale_kit_management/views/product_views.xml
+++ b/sale_kit_management/views/product_views.xml
@@ -22,6 +22,7 @@
                             <field name="kit_unit_qty" string="Kit Unit Qty"
                                 decoration-info="product_id" />
                             <field name="price_unit" string="Unit Price" />
+                            <field name="amount" string="Total Amount" />
                         </list>
                     </field>
                 </page>
diff --git a/sale_kit_management/wizard/sale_order_kit.py b/sale_kit_management/wizard/sale_order_kit.py
index 4deae25f12e..92715b4772e 100644
--- a/sale_kit_management/wizard/sale_order_kit.py
+++ b/sale_kit_management/wizard/sale_order_kit.py
@@ -21,6 +21,8 @@ class SaleOrderKit(models.TransientModel):
     _name = 'sale.order.kit'
     _description = "Kit Configurator Wizard"
 
+    _KIT_SUBPRODUCT_SEQ_BASE = 9990
+
     sale_order_line_id = fields.Many2one('sale.order.line', required=True)
     parent_product = fields.Many2one(
         'product.product',
@@ -104,21 +106,26 @@ def action_add_to_sol(self):
             for line in self.kit_line_ids
         )
 
+        existing_kit_sections = order.order_line.filtered(
+            lambda l: l.display_type == 'line_section' and l.kit_parent_line_id
+        )
+        base_seq = self._KIT_SUBPRODUCT_SEQ_BASE + len(existing_kit_sections) * 10
+
         self.env['sale.order.line'].create({
             'order_id': order.id,
             'display_type': 'line_section',
             'name': f"Subproducts of {product_name}",
-            'sequence': sol.sequence + 1,
+            'sequence': base_seq,
             'kit_parent_line_id': sol.id,
         })
 
-        for line in self.kit_line_ids:
+        for idx, line in enumerate(self.kit_line_ids, start=1):
             self.env['sale.order.line'].create({
                 'order_id': order.id,
                 'product_id': line.product_id.id,
                 'product_uom_qty': line.kit_unit_qty * sol.product_uom_qty,
-                'price_unit': line.price_unit,
-                'sequence': sol.sequence + 2,
+                'price_unit': 0,
+                'sequence': base_seq + idx,
                 'is_kit_subproduct': True,
                 'kit_parent_line_id': sol.id,
                 'kit_unit_qty': line.kit_unit_qty,

From 178255583d385728e936d753bc6d7a766f00aeb9 Mon Sep 17 00:00:00 2001
From: times-odoo <times@odoo.com>
Date: Thu, 4 Jun 2026 12:18:50 +0530
Subject: [PATCH 4/5] [FIX] sale_kit_management: prevent direct deletion of
 individual kit subproduct lines and fix child row pricing

Restructured the kit line deletion safeguards by forcing contextual removal
dependencies and initializing zero-value unit pricing for exploded items.

Rationale:
Allowing users to selectively delete individual child subproduct lines
directly from the sale order lines list view compromises data integrity and
desynchronizes the parent kit's computed pricing. To guarantee business flow
consistency, the system now validates that subproduct lines can only be
removed if their corresponding parent kit product or their entire structural
enclosing section layout is targeted in the same batch deletion. Additionally,
ensuring that exploded child rows have their 'price_unit' forced to zero blocks
duplicate pricing visibility issues inside client quotes.

Technical choices:
- Integrated a strict batch lookup inside the '@api.ondelete' method of
  'SaleOrderLine' to catch standalone subproduct row removals.
- Added a validation trigger that throws a 'UserError' message when independent
  child subproduct unlinking patterns are detected.
- Modified 'write' operations to force child component unit values to zero
  upon initial quantity matrix redistribution calculations.
- Enforced a safety boundary condition ('id not in self.ids') during batch
  child lookup workflows to prevent recursive search traps.
- Injected conditional 'readonly' visibility control over the
  'product_template_id' list field attribute inside inherited layout records.
---
 sale_kit_management/models/sale_order_line.py | 40 +++++++++----------
 .../views/sale_order_views.xml                |  5 +++
 2 files changed, 23 insertions(+), 22 deletions(-)

diff --git a/sale_kit_management/models/sale_order_line.py b/sale_kit_management/models/sale_order_line.py
index 35996d8bbc5..ac02a0dba7a 100644
--- a/sale_kit_management/models/sale_order_line.py
+++ b/sale_kit_management/models/sale_order_line.py
@@ -1,4 +1,5 @@
 from odoo import _, api, fields, models
+from odoo.exceptions import UserError
 
 
 class SaleOrderLine(models.Model):
@@ -54,38 +55,33 @@ def write(self, vals):
                 for child in child_lines:
                     if child.kit_unit_qty:
                         child.product_uom_qty = line.product_uom_qty * child.kit_unit_qty
+                        child.price_unit = 0
         return result
 
     @api.ondelete(at_uninstall=False)
     def _ondelete_handle_kit(self):
+        for line in self.filtered(lambda l: l.is_kit_subproduct):
+            if line.kit_parent_line_id in self:
+                continue
+            section_in_batch = any(
+                l.display_type == 'line_section'
+                and l.kit_parent_line_id.id == line.kit_parent_line_id.id
+                for l in self
+            )
+            if section_in_batch:
+                continue
+            raise UserError(_(
+                "You cannot delete subproduct lines directly. "
+                "Delete the parent kit product or delete the entire subproducts section."
+            ))
+
         child_lines = self.env['sale.order.line'].search([
             ('kit_parent_line_id', 'in', self.ids),
+            ('id', 'not in', self.ids),
         ])
         if child_lines:
             child_lines.unlink()
 
-        subproduct_lines = self.filtered(lambda l: l.is_kit_subproduct)
-        for parent in subproduct_lines.mapped('kit_parent_line_id'):
-            if parent in self:
-                continue
-            remaining = self.env['sale.order.line'].search([
-                ('kit_parent_line_id', '=', parent.id),
-                ('is_kit_subproduct', '=', True),
-            ])
-            if remaining:
-                parent.price_unit = sum(
-                    child.price_unit * child.kit_unit_qty
-                    for child in remaining
-                )
-            else:
-                section_lines = self.env['sale.order.line'].search([
-                    ('display_type', '=', 'line_section'),
-                    ('kit_parent_line_id', '=', parent.id),
-                ])
-                if section_lines:
-                    section_lines.unlink()
-                parent.price_unit = parent.product_id.lst_price
-
     def action_open_kit_configurator(self):
         self.ensure_one()
         return {
diff --git a/sale_kit_management/views/sale_order_views.xml b/sale_kit_management/views/sale_order_views.xml
index 77fb0c77d24..fb6916b7e6e 100644
--- a/sale_kit_management/views/sale_order_views.xml
+++ b/sale_kit_management/views/sale_order_views.xml
@@ -32,6 +32,11 @@
                 position="attributes">
                 <attribute name="readonly">is_kit_subproduct or not product_updatable</attribute>
             </xpath>
+            <xpath
+                expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_template_id']"
+                position="attributes">
+                <attribute name="readonly">is_kit_subproduct or (id and not product_updatable)</attribute>
+            </xpath>
             <xpath
                 expr="//notebook/page[@name='order_lines']/field[@name='order_line']/list/field[@name='product_uom_qty']"
                 position="attributes">

From e6b6243474f89d3e5c47e5403a052af523aaf922 Mon Sep 17 00:00:00 2001
From: times-odoo <times@odoo.com>
Date: Thu, 4 Jun 2026 12:31:53 +0530
Subject: [PATCH 5/5] [IMP] sale_kit_management: allow kit reconfiguration and
 ensure child lines clean rewrite

Updated the kit configurator layout and wizard actions to support multiple
reconfigurations of an already exploded sales order kit.

Rationale:
Previously, the kit configuration button vanished once sub-products were
exploded onto the sales order lines. This prevented users from correcting
or modifying a kit component bundle after adding it. By keeping the button
visible for all kit templates and adding an explicit cleanup search block
prior to line generation, the wizard now smoothly purges stale child records
and cleanly rewrites both the runtime configuration values and exploded lines.

Technical choices:
- Adjusted the XML list view button xpath, dropping 'has_kit_subproducts'
  from the 'invisible' restriction attribute.
- Re-introduced the baseline population mapping routine for the
  'sale.order.kit.config.line' context table during final submission.
- Added a pre-generation sweep query against 'sale.order.line' to isolate
  and drop any stale exploded subproduct elements previously linked to the parent.
---
 sale_kit_management/__manifest__.py           |  2 +-
 sale_kit_management/models/sale_order_line.py |  2 +-
 .../views/sale_order_views.xml                |  2 +-
 sale_kit_management/wizard/sale_order_kit.py  | 31 +++++++++++++++++--
 .../wizard/sale_order_kit_views.xml           |  4 ++-
 5 files changed, 35 insertions(+), 6 deletions(-)

diff --git a/sale_kit_management/__manifest__.py b/sale_kit_management/__manifest__.py
index 88797770afe..37ba4179aea 100644
--- a/sale_kit_management/__manifest__.py
+++ b/sale_kit_management/__manifest__.py
@@ -3,7 +3,7 @@
     'version': "1.0",
     'category': "Sales/Sales",
     'description': "Creates a Kit Option in Products and Sales Order",
-    'depends': ['sale_management'],
+    'depends': ['sale_management', 'stock'],
     'data': [
         'security/ir.model.access.csv',
         'wizard/sale_order_kit_views.xml',
diff --git a/sale_kit_management/models/sale_order_line.py b/sale_kit_management/models/sale_order_line.py
index ac02a0dba7a..4f0e74385ff 100644
--- a/sale_kit_management/models/sale_order_line.py
+++ b/sale_kit_management/models/sale_order_line.py
@@ -5,7 +5,7 @@
 class SaleOrderLine(models.Model):
     _inherit = 'sale.order.line'
 
-    _KIT_SUBPRODUCT_SEQ_BASE = 9990
+    _KIT_SUBPRODUCT_SEQ_BASE = 200
 
     is_kit = fields.Boolean(related='product_id.product_tmpl_id.is_kit')
     is_kit_subproduct = fields.Boolean(string="Is Kit Subproduct", default=False)
diff --git a/sale_kit_management/views/sale_order_views.xml b/sale_kit_management/views/sale_order_views.xml
index fb6916b7e6e..5c2d0ac5ccf 100644
--- a/sale_kit_management/views/sale_order_views.xml
+++ b/sale_kit_management/views/sale_order_views.xml
@@ -12,7 +12,7 @@
                 <button name="action_open_kit_configurator"
                     type="object"
                     icon="fa-cubes"
-                    invisible="not is_kit or has_kit_subproducts" width="25px" />
+                    invisible="not is_kit" width="25px" />
             </xpath>
 
             <xpath
diff --git a/sale_kit_management/wizard/sale_order_kit.py b/sale_kit_management/wizard/sale_order_kit.py
index 92715b4772e..adcbdf2806e 100644
--- a/sale_kit_management/wizard/sale_order_kit.py
+++ b/sale_kit_management/wizard/sale_order_kit.py
@@ -21,7 +21,7 @@ class SaleOrderKit(models.TransientModel):
     _name = 'sale.order.kit'
     _description = "Kit Configurator Wizard"
 
-    _KIT_SUBPRODUCT_SEQ_BASE = 9990
+    _KIT_SUBPRODUCT_SEQ_BASE = 200
 
     sale_order_line_id = fields.Many2one('sale.order.line', required=True)
     parent_product = fields.Many2one(
@@ -34,6 +34,7 @@ class SaleOrderKit(models.TransientModel):
         string="Kit Price",
         compute='_compute_kit_total_price',
     )
+    has_kit_subproducts = fields.Boolean(compute='_compute_has_kit_subproducts')
 
     @api.depends('sale_order_line_id.product_id')
     def _compute_parent_product(self):
@@ -48,6 +49,11 @@ def _compute_kit_total_price(self):
                 for line in rec.kit_line_ids
             )
 
+    @api.depends('sale_order_line_id.has_kit_subproducts')
+    def _compute_has_kit_subproducts(self):
+        for rec in self:
+            rec.has_kit_subproducts = rec.sale_order_line_id.has_kit_subproducts
+
     @api.model
     def default_get(self, fields):
         res = super().default_get(fields)
@@ -100,6 +106,13 @@ def action_add_to_sol(self):
         product_name = sol.product_id.display_name
 
         sol.kit_config_line_ids.unlink()
+        config_vals = [{
+            'sale_order_line_id': sol.id,
+            'product_id': line.product_id.id,
+            'kit_unit_qty': line.kit_unit_qty,
+            'price_unit': line.price_unit,
+        } for line in self.kit_line_ids]
+        self.env['sale.order.kit.config.line'].create(config_vals)
 
         sol.price_unit = sum(
             line.price_unit * line.kit_unit_qty
@@ -109,7 +122,21 @@ def action_add_to_sol(self):
         existing_kit_sections = order.order_line.filtered(
             lambda l: l.display_type == 'line_section' and l.kit_parent_line_id
         )
-        base_seq = self._KIT_SUBPRODUCT_SEQ_BASE + len(existing_kit_sections) * 10
+        max_sequence = max(seq.sequence
+                           for seq in existing_kit_sections
+                           ) if existing_kit_sections else 0
+
+        base_seq = self._KIT_SUBPRODUCT_SEQ_BASE + max_sequence * 2
+
+        child_lines = self.env['sale.order.line'].search([
+            ('kit_parent_line_id', 'in', sol.id),
+            ('id', 'not in', sol.id),
+        ])
+        if child_lines:
+            for line in child_lines:
+                if line.display_type == 'line_section':
+                    base_seq = line.sequence
+            child_lines.unlink()
 
         self.env['sale.order.line'].create({
             'order_id': order.id,
diff --git a/sale_kit_management/wizard/sale_order_kit_views.xml b/sale_kit_management/wizard/sale_order_kit_views.xml
index e97e0ef32ec..4905f3157d0 100644
--- a/sale_kit_management/wizard/sale_order_kit_views.xml
+++ b/sale_kit_management/wizard/sale_order_kit_views.xml
@@ -15,6 +15,7 @@
                         </group>
                         <group>
                             <field name="kit_total_price" widget="monetary" readonly="1" />
+                            <field name="has_kit_subproducts" column_invisible="1" />
                         </group>
                     </group>
 
@@ -41,9 +42,10 @@
                         name="action_confirm"
                         type="object"
                         class="btn btn-secondary"
+                        invisible="has_kit_subproducts"
                         data-hotkey="x" />
                 </footer>
             </form>
         </field>
     </record>
-</odoo>
+</odoo>
\ No newline at end of file