Nov 3 : [FIX] Bug Fixed 'product_combo_pack'

product_combo_pack/models/product_template.py

@@ -93,7 +93,6 @@ class ProductPack(models.Model):
         """Update the list price of the product with the pack price."""
         self.list_price = self.pack_price
 
-
     def get_quantity(self):
         """Calculate the pack quantity based on the availability of
         pack products."""
@@ -126,13 +125,12 @@ class ProductPack(models.Model):
             if not location_id:
                 raise UserError(_(
                     'You need to select the location to update the pack quantity...!'))
-        new_quantity = self.env['stock.quant'].with_context(inventory_mode=True).sudo().create(
+        self.env['stock.quant'].with_context(inventory_mode=True).sudo().create(
             {
                 'product_id': product_id,
                 'location_id': location_id,
                 'inventory_quantity': self.pack_quantity,
             })
-        new_quantity.action_apply_inventory()
 
     @api.onchange('pack_location_id')
     def change_quantity_based_on_location(self):

product_combo_pack/models/sale_order.py

@@ -21,27 +21,35 @@
 #
 ################################################################################
 from odoo import api, fields, models
+from collections import defaultdict
 
 
 class SalePack(models.Model):
     """Model for extending the sale order to include a selection of packs."""
     _inherit = 'sale.order'
 
-    product_pack_id = fields.Many2one('product.product', string='Select Pack',
-                                      domain=[('is_pack', '=', True)],
-                                      help='The selected pack product for'
-                                           ' the sale order.')
+    product_pack_id = fields.Many2many('product.product', string='Select Pack',
+                                       domain=[('is_pack', '=', True)],
+                                       help='The selected pack product for'
+                                            ' the sale order.')
 
     @api.onchange('product_pack_id')
     def onchange_product_pack_id(self):
         """Perform actions when the selected pack product changes."""
         if self.product_pack_id:
-            self.order_line = [(0, 0, {
-                'product_id': self.product_pack_id.id,
-                'name': self.product_pack_id.name,
-                'product_uom_qty': 1,
-                'price_unit': self.product_pack_id.list_price,
-            })]
+            new_order_lines = []
+            for rec in self.product_pack_id:
+                product_already_added = any(
+                    line.product_id.id == rec._origin.id for line in
+                    self.order_line)
+                if not product_already_added:
+                    new_order_lines.append((0, 0, {
+                        'product_id': rec.id,
+                        'name': rec.name,
+                        'product_uom_qty': 1,
+                        'price_unit': rec.pack_price,
+                    }))
+                    self.order_line = new_order_lines
         elif not self.product_pack_id:
             self.order_line = [(5, 0, 0)]
 

product_combo_pack/views/product_form_views.xml

@@ -47,9 +47,7 @@
                                 <field name="pack_price"/>
                                 <button name="update_price_product"
                                         type="object"
-
-                                        string="Update Price ">
-                                </button>
+                                        string="Update Price"/>
                             </div>
                         </group>
                         <group string="Pack Quantity"

product_combo_pack/views/sale_order_views.xml

@@ -8,7 +8,7 @@
         <field name="inherit_id" ref="sale.view_order_form"/>
         <field name="arch" type="xml">
             <field name="sale_order_template_id" position="before">
-                <field name="product_pack_id" options="{'no_create': True}"
+                <field name="product_pack_id" options="{'no_create': True}" widget="many2many_tags"
                 />
             </field>
         </field>