[IMP] picking sans limit du nombre de so par bac

Author: Michael Michot

email_template/__manifest__.py

@@ -8,7 +8,7 @@
     "version": "14.0.1.0.0",
     "author": "Pharmasimple",
     "license": "AGPL-3",
-    "website": "https://www.pharmasimple.com",
+    "website": "https://github.com/akretion/phs-addons",
     "depends": [
         "mail",
     ],

phs_stock/__manifest__.py

@@ -10,12 +10,14 @@
     "website": "https://github.com/akretion/phs-addons",
     "depends": [
         # "stock",
+        "product",
         "stock_picking_batch",
         "shopfloor",
     ],
     "data": [
         "wizard/batch_force_create_view.xml",
         "views/res_partner_views.xml",
+        "views/product_category.xml",
         "views/stock_picking_batch.xml",
         "views/stock_picking_views.xml",
         "security/ir.model.access.csv",

phs_stock/models/__init__.py

@@ -1,3 +1,4 @@
+from . import product_category
 from . import stock_picking_batch
 from . import res_partner
 from . import product_template

phs_stock/models/product_category.py

@@ -0,0 +1,11 @@
+from odoo import fields, models
+
+
+class ProductCategory(models.Model):
+    _inherit = "product.category"
+
+    no_order_box_limit = fields.Boolean(
+        help="""This falg determine if during the picking process,
+        products ca be put in the same box,
+        no matter the number of order limit of the batch rule"""
+    )

phs_stock/models/stock_picking_batch.py

@@ -183,6 +183,7 @@ def write(self, values):
         if (
             len(self) == 1
             and "location_dest_id" in values
+            and not self.product_id.categ_id.no_order_box_limit
             and not self._check_if_so_allready_in_box(values["location_dest_id"])
         ):
             last_scanned_box = self._check_if_so_in_other_box(

phs_stock/tests/test_stock_picking_batch.py

@@ -1,3 +1,4 @@
+from odoo.exceptions import UserError
 from odoo.tests.common import TransactionCase
 from odoo.tools.safe_eval import safe_eval
 
@@ -31,6 +32,13 @@ def setUp(self):
                 "picking_type_id": self.type_pick_out_id.id,
             }
         )
+        self.output_loc = self.env.ref("stock.stock_location_output")
+        self.bac1 = self.env["stock.location"].create(
+            {"name": "BAC-0001", "location_id": self.output_loc.id}
+        )
+        self.bac2 = self.env["stock.location"].create(
+            {"name": "BAC-0002", "location_id": self.output_loc.id}
+        )
 
     def test_no_rule_no_batch(self):
         # No batch creation without rules
@@ -92,3 +100,35 @@ def test_cancel_batch(self):
         batch_ids[0].action_cancel()
         for move_line in move_lines:
             self.assertFalse(move_line.picking_id.batch_id)
+
+    def test_category_no_limit(self):
+        """Product from catagory with "no order limit in box" settings
+        can be put all in the same box,
+        no matter the box attributed to the others products from the same order
+        and no matter the number of order already in the destination box"""
+
+        self.picking_batch.search([]).mapped(lambda r: r.action_cancel())
+        batch = self.rule.batch_creation(True)[0]
+        batch[0].picking_ids.move_line_ids.mapped("product_id.categ_id").write(
+            {"no_order_box_limit": True}
+        )
+        try:
+            for line in batch[0].picking_ids.move_line_ids:
+                line.write({"location_dest_id": self.bac1.id})
+        except Exception as e:
+            self.fail("test_category_no_limit raise unwanted error:{}".format(e))
+
+    def test_category_with_limit(self):
+        """Product from catagory with "no order limit in box" settings
+        can be put all in the same box,
+        no matter the box attributed to the others products from the same order
+        and no matter the number of order already in the destination box"""
+
+        self.picking_batch.search([]).mapped(lambda r: r.action_cancel())
+        batch = self.rule.batch_creation(True)[0]
+        batch[0].picking_ids.move_line_ids.mapped("product_id.categ_id").write(
+            {"no_order_box_limit": False}
+        )
+        with self.assertRaises(UserError):
+            for line in batch[0].picking_ids.move_line_ids:
+                line.write({"location_dest_id": self.bac1.id})

phs_stock/views/product_category.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo>
+<!-- product.category inherit form view -->
+<record id="product_category_form_view" model="ir.ui.view">
+    <field name="name">product.category.inherit.view.form</field>
+    <field name="model">product.category</field>
+    <field name="inherit_id" ref="product.product_category_form_view" />
+    <field name="arch" type="xml">
+        <field name="parent_id" position="after">
+            <field name="no_order_box_limit" />
+        </field>
+    </field>
+</record>
+</odoo>

setup/email_template/odoo/addons/email_template

@@ -0,0 +1 @@
+../../../../email_template

setup/email_template/setup.py

@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)