As indicated on your other bug 796320 I think this one needs a workaround right now, as it is a source of mistakes that the user cannot easily detect or fix manually.
Your patch is not correct however, because the procurement module does not depend on the 'sale' module, and should thus not depend on 'sale.order.line' etc. The suggested patch in Rohan's merge proposal was not correct either, namely because it only corrected the 'picked rate' and not the 'shipped' flag, so only partially covered the issue.
I suggest to apply a mix of both as the workaround: we need to correct the workflow test (and thus check_move_done()), but this needs to be done within the sale module, not the procurement module. (I'm reopening the bug so we do that)
Hi Patrick,
As indicated on your other bug 796320 I think this one needs a workaround right now, as it is a source of mistakes that the user cannot easily detect or fix manually.
Your patch is not correct however, because the procurement module does not depend on the 'sale' module, and should thus not depend on 'sale.order.line' etc. The suggested patch in Rohan's merge proposal was not correct either, namely because it only corrected the 'picked rate' and not the 'shipped' flag, so only partially covered the issue.
I suggest to apply a mix of both as the workaround: we need to correct the workflow test (and thus check_move_done()), but this needs to be done within the sale module, not the procurement module. (I'm reopening the bug so we do that)
Thanks for your well-documented bug reports!