A SYLEAM Module - Warehouse Management System

[wms] move.check_assign() doesn't update UoS

Bug #841654 reported by Dukai Gábor
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
A SYLEAM Module - Warehouse Management System
Confirmed
Medium
Christophe CHAUVET
A SYLEAM Module - Warehouse Management System beta4 "b4"

Bug Description

Hi!
This bug is also present in the stock module but it doesn't get exposed without using many locations. move.check_assign() only updates the UoM but not the UoS. Because picking.action_invoice_create() uses the UoS qty, this bug results in wrong invoiced quantities if check_assign split the moves.

Related branches

lp:~gdukai/wms/fix-check_assign_uos
Ready for review for merging into lp:wms
OpenERP WMS Commiter: Pending requested
Diff: 46 lines (+23/-3)
1 file modified
wms/object/move.py (+23/-3)
Revision history for this message
Dukai Gábor (gdukai) wrote :

Fix is in the related branch. IMHO calling move.onchange_quantity() is better than directly writing the UoS qty.

Revision history for this message
Christophe CHAUVET (christophe-chauvet) wrote :

Hi Dukai

Thanks for the fix, that's a good solution tu use on_change instead of force UOS

Can you for merging this branch ?

Regards,

Changed in wms:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → beta4
assignee: nobody → Christophe Chauvet - http://www.syleam.fr/ (christophe-chauvet)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.