Code review comment for lp:~numerigraphe-team/openobject-addons/7.0-sale-line-multiple-moves

Revision history for this message
Raphael Collet (OpenERP) (rco-openerp) wrote :

I am not convinced of the usefulness of this proposal (but that's just my opinion).
However, the subsequent handling of moves looks wrong to me. As you wrote, a procurement can have only one move attached. Ignoring all moves except the last is not correct: you should create one procurement per move, shouldn't you?

I will reject this proposal, as it hasn't focussed much attention.

Cheers,
Raphael

review: Disapprove

« Back to merge proposal