Feb 27 [UPDT] : Updated 'so_bom_selection'

so_bom_selection/__manifest__.py

@@ -21,7 +21,7 @@
 ################################################################################
 {
     'name': "Bill Of Material from Sale Order Line",
-    'version': '18.0.1.0.0',
+    'version': '18.0.1.0.1',
     'category': 'Manufacturing',
     'summary': 'Select the BOM in sale order line and generate the '
                'manufacturing order of components',

so_bom_selection/doc/RELEASE_NOTES.md

@@ -4,3 +4,8 @@
 #### Version 18.0.1.0.0
 #### ADD
 - Initial commit for Bill Of Material from Sale Order Line
+
+####  20.02.2025
+#### Version 18.0.1.0.1
+#### UPT
+- Update in the Confirm button action and Write function for Bill Of Material from Sale Order Line to fix the issue of creating two MOs

so_bom_selection/models/sale_order.py

@@ -29,35 +29,45 @@ class SaleOrder(models.Model):
     def action_confirm(self):
         """ Create manufacturing order of components in selected BOM """
         for rec in self.order_line:
-            if rec.bom_id and rec.bom_id.type != 'phantom' and rec.bom_id.type != 'subcontract':
+            if rec.bom_id and rec.bom_id in rec.product_template_id.bom_ids:
-                mo = self.env["mrp.production"].create(
+                # Store the original sequence order
-                    {
+                original_sequences = {bom.id: bom.sequence for bom in rec.product_template_id.bom_ids}
-                        "product_id": rec.product_id.id,
+                # Set the selected BOM's sequence to 0
-                        "product_uom_id": rec.product_id.uom_id.id,
+                rec.bom_id.sequence = 0
-                        "origin": self.name,
+                # Get all other BOMs and update their sequence accordingly
-                        'company_id': self.env.user.company_id.id,
+                other_boms = rec.product_template_id.bom_ids - rec.bom_id
-                        "product_qty": rec.product_uom_qty,
+                sequence_counter = 1  # Start sequence from 1 for others
-                        "qty_to_produce": rec.product_uom_qty,
+                for bom in other_boms.sorted('sequence'):
-                        "bom_id": rec.bom_id.id,
+                    bom.sequence = sequence_counter
-                        "sale_line_id": rec.id,
+                    sequence_counter += 1
-                    }
+        result = super().action_confirm()
-                )
+        # Restore original sequences after the confirmation
-                moves_raw_values = mo._get_moves_raw_values()
+        for rec in self.order_line:
-                list_move_raw = []
+            if rec.bom_id and rec.bom_id in rec.product_template_id.bom_ids:
-                for move_raw_values in moves_raw_values:
+                for bom in rec.product_template_id.bom_ids:
-                    list_move_raw += [(0, 0, move_raw_values)]
+                    if bom.id in original_sequences:
-        return super().action_confirm()
+                        bom.sequence = original_sequences[bom.id]
+            manufacturing_order = self.env["mrp.production"].search(
+                [('origin', '=', self.name),
+                 ('state', '=', 'confirmed')])
+            if manufacturing_order:
+                for mo in manufacturing_order:
+                    print(mo.product_qty)
+                    mo.update({'qty_to_produce': mo.product_qty})
+                # mo.update({'state':'draft'})
+        return result
 
     def write(self, values):
-        """Super write method to change the manufacturing quantity
+        """Super write method to change the qty_to_produce in the MO
-        based on sale order quantity"""
+            based on sale order quantity"""
         res = super().write(values)
         for order_line in self.order_line:
             if order_line.product_uom_qty:
-                mo = self.env["mrp.production"].search(
+                manufacturing_order = self.env["mrp.production"].search(
-                    [('sale_line_id', '=', order_line.id),
+                    [('origin', '=', self.name),
-                     ('state', '=', 'draft')])
+                     ('state', '=', 'confirmed')])
-                if mo:
+                if manufacturing_order:
-                    mo.write({'product_qty': order_line.product_uom_qty})
+                    for mo in manufacturing_order:
-                    mo.write({'qty_to_produce': order_line.product_uom_qty})
+                        mo.write({'qty_to_produce': mo.product_qty})
         return res
+

so_bom_selection/static/description/index.html

@@ -462,7 +462,8 @@
                                 <div class="p-md-5 p-3 position-relative">
                                     <div class="row">
                                         <div class="col-md-12">
-                                            <h1 style="font-weight:bold; font-size:calc(1.1rem + 1vw); line-height:120%; text-align:center; text-transform:capitalize; font-size: 40px;
+                                            <h1 style="
+                                            font-weight:bold; font-size:calc(1.1rem + 1vw); line-height:120%; text-align:center; text-transform:capitalize; font-size: 40px;
                               font-weight: 700;">
                                       <span style="color:#121212; font-size:calc(1.1rem + 1vw)"> Manufacturing Order created for components with source as corresponding Sale Order
                                       </span>