[4.2] stock/wizard return->false locations on packing list, but right on stock move

[André Kurz (Hasa.ch)]

When using the wizard, "Return packing" from a packing list, the new return packing list which is created has the same Location and Destination Location as the original. But the return Location should be the old Final Location and the return Destination Location should be the old Location.

The problem is only in the packing list, this is ok for the return stock move.

I added this code to fix this :

--- /home/andre/Desktop/tinyerp-server/addons/stock/wizard/wizard_return_old.py 2008-12-19 16:03:50.000000000 +0100
+++ /home/andre/Desktop/tinyerp-server/addons/stock/wizard/wizard_return.py 2008-12-19 15:26:54.000000000 +0100
@@ -84,7 +84,8 @@
                                new_type='internal'
                        new_picking=pick_obj.copy(cr, uid, pick.id, {'name':'%s (return)' % pick.name,
                                        'move_lines':[], 'state':'draft', 'type':new_type, 'loc_move_id':False,
- 'date':date_cur, 'invoice_state':data['form']['invoice_state']})
+ 'date':date_cur, 'invoice_state':data['form']['invoice_state'],
+ 'location_dest_id':pick.location_id.id or None,'location_id':pick.loc_move_id.id or None})
                if pick.loc_move_id:
                        new_location=pick.loc_move_id.id
                else:

Version 4.2
~openerp/openobject-addons/4.2 : viewing / for revision 43

[Fabien (Open ERP) (fp-tinyerp)]

These values are for information purpose only. This is not a bug.
We changed in the trunk, so that these fields are not anymore on the packing but only on moves.