From 5b34e7e7e6b90b63c8ef29f98a89cd8b10ea5af7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9my=20Voet=20=28ryv=29?= Date: Fri, 10 Sep 2021 15:23:41 +0000 Subject: [PATCH] [IMP] mrp_subcontracting: improve uninstall hook MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Clean/remove subcontracting operation type when we uninstall subcontracting. task-2633558 closes odoo/odoo#76612 X-original-commit: 3452e821feca3622457eeb919bca2f0821214a71 Signed-off-by: Arnold Moyaux Signed-off-by: Rémy Voet --- addons/mrp_subcontracting/__init__.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/addons/mrp_subcontracting/__init__.py b/addons/mrp_subcontracting/__init__.py index 99d4d43ad32..ae586949745 100644 --- a/addons/mrp_subcontracting/__init__.py +++ b/addons/mrp_subcontracting/__init__.py @@ -1,10 +1,10 @@ # -*- coding: utf-8 -*- -from odoo import SUPERUSER_ID, api - from . import models -from . import wizard from . import report +from . import wizard + +from odoo import SUPERUSER_ID, api def uninstall_hook(cr, registry): @@ -12,10 +12,13 @@ def uninstall_hook(cr, registry): warehouses = env["stock.warehouse"].search([]) subcontracting_routes = warehouses.mapped("subcontracting_route_id") warehouses.write({"subcontracting_route_id": False}) + operations_type_to_remove = (warehouses.subcontracting_resupply_type_id | warehouses.subcontracting_type_id) + operations_type_to_remove.active = False # Fail unlink means that the route is used somewhere (e.g. route_id on stock.rule). In this case # we don't try to do anything. try: with env.cr.savepoint(): subcontracting_routes.unlink() + operations_type_to_remove.unlink() except: pass