From 476055ae2f82ebd91634d2ec9a3c49fd8fad52fe Mon Sep 17 00:00:00 2001 From: AlexPForgeFlow Date: Thu, 21 Nov 2024 11:01:10 +0100 Subject: [PATCH] [16.0][FIX] product_supplierinfo_stock_picking_type: rename on change picking type method --- .../models/purchase_order.py | 4 +++- .../tests/test_product_supplierinfo_stock_picking_type.py | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/product_supplierinfo_stock_picking_type/models/purchase_order.py b/product_supplierinfo_stock_picking_type/models/purchase_order.py index 45f758a5d1b..49a3fb6d652 100644 --- a/product_supplierinfo_stock_picking_type/models/purchase_order.py +++ b/product_supplierinfo_stock_picking_type/models/purchase_order.py @@ -7,6 +7,8 @@ class PurchaseOrder(models.Model): _inherit = "purchase.order" @api.onchange("picking_type_id") - def onchange_picking_type_id(self): + def onchange_picking_type_id_onchange_product(self): + # Method name is to avoid conflicts with mrp_subcontracting_dropshipping + # that have an onchange method named onchange_picking_type_id for line in self.order_line: line.onchange_product_id() diff --git a/product_supplierinfo_stock_picking_type/tests/test_product_supplierinfo_stock_picking_type.py b/product_supplierinfo_stock_picking_type/tests/test_product_supplierinfo_stock_picking_type.py index c08695c56bc..6203c00c408 100644 --- a/product_supplierinfo_stock_picking_type/tests/test_product_supplierinfo_stock_picking_type.py +++ b/product_supplierinfo_stock_picking_type/tests/test_product_supplierinfo_stock_picking_type.py @@ -102,5 +102,5 @@ def test_product_picking_type_c(self): po = self._create_purchase_order(self.picking_in_c) self.assertEqual(po.order_line.price_unit, 20) po.picking_type_id = self.picking_in_a - po.onchange_picking_type_id() + po.onchange_picking_type_id_onchange_product() self.assertEqual(po.order_line.price_unit, 5)