From 31e3a97577f45f53dd142d7cd201174875824363 Mon Sep 17 00:00:00 2001
From: Yannick Tivisse <yti@odoo.com>
Date: Mon, 13 Nov 2017 15:56:09 +0100
Subject: [PATCH] [FIX] stock: Fix strange ORM bidouille on picking creation
Purpose
=======
This bidouille on the ORM command seems to be from another age where the onchange on o2m list were not working well.
The issue is that _convert_to_write could return commands like (6, 0, []).
In that case the last tuple element is not dictionary, but a list.
Specification
=============
Fix that brol.
---
addons/stock/models/stock_picking.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/addons/stock/models/stock_picking.py b/addons/stock/models/stock_picking.py
index 5a65fe1703ea..b967f7c5b055 100644
--- a/addons/stock/models/stock_picking.py
+++ b/addons/stock/models/stock_picking.py
@@ -480,7 +480,7 @@ class Picking(models.Model):
# As it is a create the format will be a list of (0, 0, dict)
if vals.get('move_lines') and vals.get('location_id') and vals.get('location_dest_id'):
for move in vals['move_lines']:
- if len(move) == 3:
+ if len(move) == 3 and move[0] == 0:
move[2]['location_id'] = vals['location_id']
move[2]['location_dest_id'] = vals['location_dest_id']
res = super(Picking, self).create(vals)
--
GitLab