From 62dcc193fe2ea3d51f8253f9fc8ea9b7e7539dc9 Mon Sep 17 00:00:00 2001 From: William Henrotin Date: Tue, 13 Apr 2021 08:33:17 +0000 Subject: [PATCH] [FIX] stock_account: get unit_cost in sudo mode Commit ad5ac19218b4c529c651da951acdb93e1ac1227d adds a sudo() in the test condition but not inside the condition. Close #69144 closes odoo/odoo#69197 X-original-commit: 7cd23a2cd464ccdfcb028b4f2c948654634b973c Signed-off-by: William Henrotin --- addons/stock_account/models/stock_move.py | 2 +- addons/stock_account/tests/test_stockvaluation.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/stock_account/models/stock_move.py b/addons/stock_account/models/stock_move.py index cfad1fd9066..1a00a2af9f4 100644 --- a/addons/stock_account/models/stock_move.py +++ b/addons/stock_account/models/stock_move.py @@ -31,7 +31,7 @@ class StockMove(models.Model): price_unit = self.price_unit # If the move is a return, use the original move's price unit. if self.origin_returned_move_id and self.origin_returned_move_id.sudo().stock_valuation_layer_ids: - price_unit = self.origin_returned_move_id.stock_valuation_layer_ids[-1].unit_cost + price_unit = self.origin_returned_move_id.sudo().stock_valuation_layer_ids[-1].unit_cost return not self.company_id.currency_id.is_zero(price_unit) and price_unit or self.product_id.standard_price @api.model diff --git a/addons/stock_account/tests/test_stockvaluation.py b/addons/stock_account/tests/test_stockvaluation.py index c8e28db0242..44ec162d48d 100644 --- a/addons/stock_account/tests/test_stockvaluation.py +++ b/addons/stock_account/tests/test_stockvaluation.py @@ -953,7 +953,7 @@ class TestStockValuation(TransactionCase): stock_return_picking_action = stock_return_picking.create_returns() return_pick = self.env['stock.picking'].browse(stock_return_picking_action['res_id']) return_pick.move_lines[0].move_line_ids[0].qty_done = 1.0 - return_pick._action_done() + return_pick.with_user(self.inventory_user)._action_done() self.assertEqual(self.product1.standard_price, 16)