diff --git a/setup/stock_whole_kit_constraint/odoo/addons/stock_whole_kit_constraint b/setup/stock_whole_kit_constraint/odoo/addons/stock_whole_kit_constraint new file mode 120000 index 000000000..5e49291ba --- /dev/null +++ b/setup/stock_whole_kit_constraint/odoo/addons/stock_whole_kit_constraint @@ -0,0 +1 @@ +../../../../stock_whole_kit_constraint \ No newline at end of file diff --git a/setup/stock_whole_kit_constraint/setup.py b/setup/stock_whole_kit_constraint/setup.py new file mode 100644 index 000000000..28c57bb64 --- /dev/null +++ b/setup/stock_whole_kit_constraint/setup.py @@ -0,0 +1,6 @@ +import setuptools + +setuptools.setup( + setup_requires=['setuptools-odoo'], + odoo_addon=True, +) diff --git a/stock_whole_kit_constraint/README.rst b/stock_whole_kit_constraint/README.rst new file mode 100644 index 000000000..739c896a4 --- /dev/null +++ b/stock_whole_kit_constraint/README.rst @@ -0,0 +1,98 @@ +========================== +Stock whole kit constraint +========================== + +.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png + :target: https://odoo-community.org/page/development-status + :alt: Beta +.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png + :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html + :alt: License: AGPL-3 +.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fmanufacture-lightgray.png?logo=github + :target: https://github.com/OCA/manufacture/tree/13.0/stock_whole_kit_constraint + :alt: OCA/manufacture +.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png + :target: https://translation.odoo-community.org/projects/manufacture-13-0/manufacture-13-0-stock_whole_kit_constraint + :alt: Translate me on Weblate +.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png + :target: https://runbot.odoo-community.org/runbot/129/13.0 + :alt: Try me on Runbot + +|badge1| |badge2| |badge3| |badge4| |badge5| + +This module allows to configure a product that has a BoM of type kit to disallow partial +deliveries so that the components can't be partially delivered. + +**Table of contents** + +.. contents:: + :local: + +Configuration +============= + +To allow/disallow the partial delivery of kits: + +#. Go to the kit product template or variant and then to the *Inventory* tab, + *Logistics* group. +#. The "Allow Partial Kit" check controls this. If marked, it will allow it. +#. By default, the check is not marked. + +Usage +===== + +To use this module, you need to: + +#. Make a delivery picking with a kit product. +#. Try to deliver it partially. +#. An error will raise. + +If you want to deliver other items in the picking you can do so and leave the +whole kit components units pending in a backorder. + +Bug Tracker +=========== + +Bugs are tracked on `GitHub Issues `_. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed +`feedback `_. + +Do not contact contributors directly about support or help with technical issues. + +Credits +======= + +Authors +~~~~~~~ + +* Tecnativa + +Contributors +~~~~~~~~~~~~ + +* Tecnativa __ + + * David Vidal + +Maintainers +~~~~~~~~~~~ + +This module is maintained by the OCA. + +.. image:: https://odoo-community.org/logo.png + :alt: Odoo Community Association + :target: https://odoo-community.org + +OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use. + +This module is part of the `OCA/manufacture `_ project on GitHub. + +You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/stock_whole_kit_constraint/__init__.py b/stock_whole_kit_constraint/__init__.py new file mode 100644 index 000000000..0650744f6 --- /dev/null +++ b/stock_whole_kit_constraint/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/stock_whole_kit_constraint/__manifest__.py b/stock_whole_kit_constraint/__manifest__.py new file mode 100644 index 000000000..88e8d4b93 --- /dev/null +++ b/stock_whole_kit_constraint/__manifest__.py @@ -0,0 +1,14 @@ +# Copyright 2021 Tecnativa - David Vidal +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). +{ + "name": "Stock whole kit constraint", + "summary": "Avoid to deliver a kit partially", + "version": "13.0.1.0.0", + "category": "Stock", + "website": "https://github.com/OCA/manufacture", + "author": "Tecnativa, Odoo Community Association (OCA)", + "license": "AGPL-3", + "installable": True, + "depends": ["mrp"], + "data": ["views/product_template_views.xml"], +} diff --git a/stock_whole_kit_constraint/i18n/es.po b/stock_whole_kit_constraint/i18n/es.po new file mode 100644 index 000000000..8e5a1b473 --- /dev/null +++ b/stock_whole_kit_constraint/i18n/es.po @@ -0,0 +1,52 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * stock_whole_kit_constraint +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 12.0\n" +"Report-Msgid-Bugs-To: \n" +"POT-Creation-Date: 2021-03-19 10:39+0000\n" +"PO-Revision-Date: 2021-03-19 11:41+0100\n" +"Last-Translator: <>\n" +"Language-Team: \n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: 8bit\n" +"Plural-Forms: \n" +"Language: es\n" +"X-Generator: Poedit 2.3\n" + +#. module: stock_whole_kit_constraint +#: model:ir.model.fields,field_description:stock_whole_kit_constraint.field_product_product__allow_partial_kit_delivery +#: model:ir.model.fields,field_description:stock_whole_kit_constraint.field_product_template__allow_partial_kit_delivery +#: model:ir.model.fields,field_description:stock_whole_kit_constraint.field_stock_move__allow_partial_kit_delivery +msgid "Allow Partial Kit Delivery" +msgstr "Permitir entrega parcial de kit" + +#. module: stock_whole_kit_constraint +#: model:ir.model.fields,help:stock_whole_kit_constraint.field_product_product__allow_partial_kit_delivery +#: model:ir.model.fields,help:stock_whole_kit_constraint.field_product_template__allow_partial_kit_delivery +msgid "If not set when a kit product components are delivered, it won't be allowed to do it partially." +msgstr "No no está activo, cuando haya una entrega con un kit, no se permitirá la entrega parcial de sus componentes." + +#. module: stock_whole_kit_constraint +#: model:ir.model,name:stock_whole_kit_constraint.model_product_template +msgid "Product Template" +msgstr "Plantilla de producto" + +#. module: stock_whole_kit_constraint +#: model:ir.model,name:stock_whole_kit_constraint.model_stock_move +msgid "Stock Move" +msgstr "Movimiento de existencias" + +#. module: stock_whole_kit_constraint +#: model:ir.model,name:stock_whole_kit_constraint.model_stock_picking +msgid "Transfer" +msgstr "Albarán" + +#. module: stock_whole_kit_constraint +#: code:addons/stock_whole_kit_constraint/models/stock_picking.py:23 +#, python-format +msgid "You can't make a partial delivery of components of the %s kit" +msgstr "No puede realizar una entrega parcial de los componentes del kit %s" diff --git a/stock_whole_kit_constraint/i18n/stock_whole_kit_constraint.pot b/stock_whole_kit_constraint/i18n/stock_whole_kit_constraint.pot new file mode 100644 index 000000000..5c414e923 --- /dev/null +++ b/stock_whole_kit_constraint/i18n/stock_whole_kit_constraint.pot @@ -0,0 +1,49 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * stock_whole_kit_constraint +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 12.0\n" +"Report-Msgid-Bugs-To: \n" +"Last-Translator: <>\n" +"Language-Team: \n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: \n" + +#. module: stock_whole_kit_constraint +#: model:ir.model.fields,field_description:stock_whole_kit_constraint.field_product_product__allow_partial_kit_delivery +#: model:ir.model.fields,field_description:stock_whole_kit_constraint.field_product_template__allow_partial_kit_delivery +#: model:ir.model.fields,field_description:stock_whole_kit_constraint.field_stock_move__allow_partial_kit_delivery +msgid "Allow Partial Kit Delivery" +msgstr "" + +#. module: stock_whole_kit_constraint +#: model:ir.model.fields,help:stock_whole_kit_constraint.field_product_product__allow_partial_kit_delivery +#: model:ir.model.fields,help:stock_whole_kit_constraint.field_product_template__allow_partial_kit_delivery +msgid "If not set, and this product is delivered with a BoM of type kit, partial deliveries of the components won't be allowed." +msgstr "" + +#. module: stock_whole_kit_constraint +#: model:ir.model,name:stock_whole_kit_constraint.model_product_template +msgid "Product Template" +msgstr "" + +#. module: stock_whole_kit_constraint +#: model:ir.model,name:stock_whole_kit_constraint.model_stock_move +msgid "Stock Move" +msgstr "" + +#. module: stock_whole_kit_constraint +#: model:ir.model,name:stock_whole_kit_constraint.model_stock_picking +msgid "Transfer" +msgstr "" + +#. module: stock_whole_kit_constraint +#: code:addons/stock_whole_kit_constraint/models/stock_picking.py:23 +#, python-format +msgid "You can't make a partial delivery of components of the %s kit" +msgstr "" + diff --git a/stock_whole_kit_constraint/models/__init__.py b/stock_whole_kit_constraint/models/__init__.py new file mode 100644 index 000000000..7e2c2a891 --- /dev/null +++ b/stock_whole_kit_constraint/models/__init__.py @@ -0,0 +1,3 @@ +from . import product_template +from . import stock_move +from . import stock_picking diff --git a/stock_whole_kit_constraint/models/product_template.py b/stock_whole_kit_constraint/models/product_template.py new file mode 100644 index 000000000..5540989f5 --- /dev/null +++ b/stock_whole_kit_constraint/models/product_template.py @@ -0,0 +1,13 @@ +# Copyright 2021 Tecnativa - David Vidal +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). +from odoo import fields, models + + +class ProductTemplate(models.Model): + _inherit = "product.template" + + allow_partial_kit_delivery = fields.Boolean( + default=True, + help="If not set, and this product is delivered with a BoM of type " + "kit, partial deliveries of the components won't be allowed.", + ) diff --git a/stock_whole_kit_constraint/models/stock_move.py b/stock_whole_kit_constraint/models/stock_move.py new file mode 100644 index 000000000..aaaa4249a --- /dev/null +++ b/stock_whole_kit_constraint/models/stock_move.py @@ -0,0 +1,51 @@ +# Copyright 2021 Tecnativa - David Vidal +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). +from odoo import api, fields, models + + +class StockMove(models.Model): + _inherit = "stock.move" + + allow_partial_kit_delivery = fields.Boolean( + compute="_compute_allow_partial_kit_delivery", compute_sudo=True, + ) + + @api.depends("product_id.product_tmpl_id.allow_partial_kit_delivery", "state") + def _compute_allow_partial_kit_delivery(self): + """Take it from the product only if it's a kit""" + self.write({"allow_partial_kit_delivery": True}) + for move in self.filtered( + lambda x: x.product_id and x.state not in ["done", "cancel"] + ): + # If it isn't a kit it will always be True + if not move.bom_line_id or move.bom_line_id.bom_id.type != "phantom": + move.allow_partial_kit_delivery = True + continue + move.allow_partial_kit_delivery = ( + move.bom_line_id.bom_id.product_tmpl_id.allow_partial_kit_delivery + ) + + def _check_backorder_moves(self): + """Check if there are partial deliveries on any set of moves. The + computing is done in the same way the main picking method does it """ + quantity_todo = {} + quantity_done = {} + for move in self: + quantity_todo.setdefault(move.product_id.id, 0) + quantity_done.setdefault(move.product_id.id, 0) + quantity_todo[move.product_id.id] += move.product_uom_qty + quantity_done[move.product_id.id] += move.quantity_done + for ops in self.mapped("move_line_ids").filtered( + lambda x: x.package_id and not x.product_id and not x.move_id + ): + for quant in ops.package_id.quant_ids: + quantity_done.setdefault(quant.product_id.id, 0) + quantity_done[quant.product_id.id] += quant.qty + for pack in self.mapped("move_line_ids").filtered( + lambda x: x.product_id and not x.move_id + ): + quantity_done.setdefault(pack.product_id.id, 0) + quantity_done[pack.product_id.id] += pack.product_uom_id._compute_quantity( + pack.qty_done, pack.product_id.uom_id + ) + return any(quantity_done[x] < quantity_todo.get(x, 0) for x in quantity_done) diff --git a/stock_whole_kit_constraint/models/stock_picking.py b/stock_whole_kit_constraint/models/stock_picking.py new file mode 100644 index 000000000..a756094ce --- /dev/null +++ b/stock_whole_kit_constraint/models/stock_picking.py @@ -0,0 +1,29 @@ +# Copyright 2021 Tecnativa - David Vidal +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). +from odoo import _, models +from odoo.exceptions import ValidationError + + +class StockPicking(models.Model): + _inherit = "stock.picking" + + def _check_backorder(self): + """On the moment of the picking validation, we'll check wether there + are kits that can't be partially delivered or not""" + moves = self.mapped("move_lines").filtered( + lambda x: not x.allow_partial_kit_delivery and x.bom_line_id + ) + boms = moves.mapped("bom_line_id.bom_id") + for bom in boms: + bom_moves = moves.filtered(lambda x: x.bom_line_id.bom_id == bom) + # We can put it in backorder if the whole kit goes + if not sum(bom_moves.mapped("quantity_done")): + continue + if bom_moves._check_backorder_moves(): + raise ValidationError( + _( + "You can't make a partial delivery of components of the " + "%s kit" % bom.product_tmpl_id.display_name + ) + ) + return super()._check_backorder() diff --git a/stock_whole_kit_constraint/readme/CONFIGURE.rst b/stock_whole_kit_constraint/readme/CONFIGURE.rst new file mode 100644 index 000000000..f3185dfd2 --- /dev/null +++ b/stock_whole_kit_constraint/readme/CONFIGURE.rst @@ -0,0 +1,6 @@ +To allow/disallow the partial delivery of kits: + +#. Go to the kit product template or variant and then to the *Inventory* tab, + *Logistics* group. +#. The "Allow Partial Kit" check controls this. If marked, it will allow it. +#. By default, the check is not marked. diff --git a/stock_whole_kit_constraint/readme/CONTRIBUTORS.rst b/stock_whole_kit_constraint/readme/CONTRIBUTORS.rst new file mode 100644 index 000000000..61589bc27 --- /dev/null +++ b/stock_whole_kit_constraint/readme/CONTRIBUTORS.rst @@ -0,0 +1,3 @@ +* Tecnativa __ + + * David Vidal diff --git a/stock_whole_kit_constraint/readme/DESCRIPTION.rst b/stock_whole_kit_constraint/readme/DESCRIPTION.rst new file mode 100644 index 000000000..cf1aba01d --- /dev/null +++ b/stock_whole_kit_constraint/readme/DESCRIPTION.rst @@ -0,0 +1,2 @@ +This module allows to configure a product that has a BoM of type kit to disallow partial +deliveries so that the components can't be partially delivered. diff --git a/stock_whole_kit_constraint/readme/USAGE.rst b/stock_whole_kit_constraint/readme/USAGE.rst new file mode 100644 index 000000000..a5bbc3d41 --- /dev/null +++ b/stock_whole_kit_constraint/readme/USAGE.rst @@ -0,0 +1,8 @@ +To use this module, you need to: + +#. Make a delivery picking with a kit product. +#. Try to deliver it partially. +#. An error will raise. + +If you want to deliver other items in the picking you can do so and leave the +whole kit components units pending in a backorder. diff --git a/stock_whole_kit_constraint/static/description/icon.png b/stock_whole_kit_constraint/static/description/icon.png new file mode 100644 index 000000000..757fd5906 Binary files /dev/null and b/stock_whole_kit_constraint/static/description/icon.png differ diff --git a/stock_whole_kit_constraint/static/description/index.html b/stock_whole_kit_constraint/static/description/index.html new file mode 100644 index 000000000..0e4615447 --- /dev/null +++ b/stock_whole_kit_constraint/static/description/index.html @@ -0,0 +1,446 @@ + + + + + + +Stock whole kit constraint + + + +
+

Stock whole kit constraint

+ + +

Beta License: AGPL-3 OCA/manufacture Translate me on Weblate Try me on Runbot

+

This module allows to configure a product that has a BoM of type kit to disallow partial +deliveries so that the components can’t be partially delivered.

+

Table of contents

+ +
+

Configuration

+

To allow/disallow the partial delivery of kits:

+
    +
  1. Go to the kit product template or variant and then to the Inventory tab, +Logistics group.
    2. +
  2. The “Allow Partial Kit” check controls this. If marked, it will allow it.
    3. +
  3. By default, the check is not marked.
    4. +
+
+
+

Usage

+

To use this module, you need to:

+
    +
  1. Make a delivery picking with a kit product.
    2. +
  2. Try to deliver it partially.
    3. +
  3. An error will raise.
    4. +
+

If you want to deliver other items in the picking you can do so and leave the +whole kit components units pending in a backorder.

+
+
+

Bug Tracker

+

Bugs are tracked on GitHub Issues. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed +feedback.

+

Do not contact contributors directly about support or help with technical issues.

+
+
+

Credits

+
+

Authors

+
    +
  • Tecnativa
    • +
+
+
+

Contributors

+ +
+
+

Maintainers

+

This module is maintained by the OCA.

+Odoo Community Association +

OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use.

+

This module is part of the OCA/manufacture project on GitHub.

+

You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

+
+
+
+ + diff --git a/stock_whole_kit_constraint/tests/__init__.py b/stock_whole_kit_constraint/tests/__init__.py new file mode 100644 index 000000000..5eb9fbc7c --- /dev/null +++ b/stock_whole_kit_constraint/tests/__init__.py @@ -0,0 +1 @@ +from . import test_stock_whole_kit_constraint diff --git a/stock_whole_kit_constraint/tests/test_stock_whole_kit_constraint.py b/stock_whole_kit_constraint/tests/test_stock_whole_kit_constraint.py new file mode 100644 index 000000000..71a892fe8 --- /dev/null +++ b/stock_whole_kit_constraint/tests/test_stock_whole_kit_constraint.py @@ -0,0 +1,145 @@ +# Copyright 2021 Tecnativa - David Vidal +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). +from odoo.exceptions import ValidationError +from odoo.tests import Form, common + + +class TestStockWholeKitConstraint(common.SavepointCase): + @classmethod + def setUpClass(cls): + super().setUpClass() + cls.customer = cls.env["res.partner"].create({"name": "Mr. Odoo"}) + # Kit 1 that can be partially delivered + cls.product_kit_1 = cls.env["product.product"].create( + {"name": "Product Kit 1", "type": "consu"} + ) + cls.component_1_kit_1 = cls.env["product.product"].create( + {"name": "Component 1 Kit 1", "type": "product"} + ) + cls.component_2_kit_1 = cls.env["product.product"].create( + {"name": "Component 2 Kit 1", "type": "product"} + ) + bom_form = Form(cls.env["mrp.bom"]) + bom_form.product_tmpl_id = cls.product_kit_1.product_tmpl_id + bom_form.product_id = cls.product_kit_1 + bom_form.type = "phantom" + with bom_form.bom_line_ids.new() as line: + line.product_id = cls.component_1_kit_1 + with bom_form.bom_line_ids.new() as line: + line.product_id = cls.component_2_kit_1 + cls.bom_kit_1 = bom_form.save() + # Kit 2 - disallow partial deliveries + cls.product_kit_2 = cls.env["product.product"].create( + { + "name": "Product Kit 2", + "type": "consu", + "allow_partial_kit_delivery": False, + } + ) + cls.component_1_kit_2 = cls.env["product.product"].create( + {"name": "Component 1 Kit 2", "type": "product"} + ) + cls.component_2_kit_2 = cls.env["product.product"].create( + {"name": "Component 2 Kit 2", "type": "product"} + ) + bom_form = Form(cls.env["mrp.bom"]) + bom_form.product_tmpl_id = cls.product_kit_2.product_tmpl_id + bom_form.product_id = cls.product_kit_2 + bom_form.type = "phantom" + with bom_form.bom_line_ids.new() as line: + line.product_id = cls.component_1_kit_2 + with bom_form.bom_line_ids.new() as line: + line.product_id = cls.component_2_kit_2 + cls.bom_kit_2 = bom_form.save() + # Manufactured product as control + cls.product_mrp = cls.env["product.product"].create( + { + "name": "Product Kit 2", + "type": "consu", + # Force the setting in a manufactured product. + # It should not affect it + "allow_partial_kit_delivery": False, + } + ) + bom_form = Form(cls.env["mrp.bom"]) + bom_form.product_tmpl_id = cls.product_mrp.product_tmpl_id + bom_form.product_id = cls.product_mrp + bom_form.type = "normal" + with bom_form.bom_line_ids.new() as line: + line.product_id = cls.component_1_kit_2 + cls.bom_mrp = bom_form.save() + # Not a kit product as control + cls.regular_product = cls.env["product.product"].create( + { + "name": "Regular test product", + "type": "product", + # Force the setting in a regular product. It should not affect it + "allow_partial_kit_delivery": False, + } + ) + # Delivery picking + picking_form = Form(cls.env["stock.picking"]) + picking_form.picking_type_id = cls.env.ref("stock.picking_type_out") + picking_form.partner_id = cls.customer + with picking_form.move_ids_without_package.new() as move: + move.product_id = cls.product_kit_1 + move.product_uom_qty = 3.0 + with picking_form.move_ids_without_package.new() as move: + move.product_id = cls.product_kit_2 + move.product_uom_qty = 3.0 + with picking_form.move_ids_without_package.new() as move: + move.product_id = cls.product_mrp + move.product_uom_qty = 3.0 + with picking_form.move_ids_without_package.new() as move: + move.product_id = cls.regular_product + move.product_uom_qty = 3.0 + cls.customer_picking = picking_form.save() + cls.customer_picking.action_confirm() + + def test_01_all_partially_done_but_the_disallow_partial_kit(self): + """No quantity is done for the kit disallowed and only partially for the + others so the backorder wizard raises.""" + moves_allowed = self.customer_picking.move_lines.filtered( + lambda x: x.bom_line_id.bom_id != self.bom_kit_2 + ) + moves_allowed.write({"quantity_done": 1}) + response = self.customer_picking.button_validate() + self.assertEqual("stock.backorder.confirmation", response.get("res_model")) + + def test_02_all_done_but_partial_disallow_partial_kit(self): + """We try to deliver partially the disallowed kit""" + moves_disallowed = self.customer_picking.move_lines.filtered( + lambda x: x.bom_line_id.bom_id == self.bom_kit_2 + ) + moves_disallowed.write({"quantity_done": 1}) + with self.assertRaises(ValidationError): + self.customer_picking.button_validate() + # We can split the picking if the whole kit components are delivered + moves_disallowed.write({"quantity_done": 3}) + # We've got a backorder on the rest of the lines + response = self.customer_picking.button_validate() + self.assertEqual("stock.backorder.confirmation", response.get("res_model")) + + def test_03_all_done(self): + """Deliver the whole picking normally""" + self.customer_picking.move_lines.write({"quantity_done": 3}) + self.customer_picking.button_validate() + self.assertEqual("done", self.customer_picking.state) + + def test_04_manual_move_lines(self): + """If a user adds manual operations, we should consider it as well""" + # We need to enable detaild operations to test this case + self.customer_picking.picking_type_id.show_operations = True + picking_form = Form(self.customer_picking) + for product in (self.bom_kit_1 + self.bom_kit_2).mapped( + "bom_line_ids.product_id" + ): + with picking_form.move_line_ids_without_package.new() as line: + line.product_id = product + line.qty_done = 3 + picking_form.save() + self.customer_picking.move_lines.filtered( + lambda x: x.product_id in (self.product_mrp, self.regular_product) + ).write({"quantity_done": 3}) + self.customer_picking.button_validate() + self.assertEqual("done", self.customer_picking.state) diff --git a/stock_whole_kit_constraint/views/product_template_views.xml b/stock_whole_kit_constraint/views/product_template_views.xml new file mode 100644 index 000000000..a423c002c --- /dev/null +++ b/stock_whole_kit_constraint/views/product_template_views.xml @@ -0,0 +1,31 @@ + + + + + product.template + + + + + + + + + + product.product + + + + + + + + + +