Nice to meet you.
My name is Hideaki Hagiwara.
I live in japan.
I think it is better to improve logic of MMovementLineConfirm.ProcessLine() method.
Because if Scrapped Quantity field in Move confirmation Document is entered quantity, iDempiere create 'Physical inventory Document' at 'warehouse of Locator to' for adjust Scrapped Quantity. So, value of Movement Quantity field in 'Inventory Move Document' should involve ConfirmedQty and ScrappedQty. And I think usually material of Scrapped Quantity have been moved from 'Locator form' to 'Locator to' in fact.
Please refer to Attachment.
Thank you for reading.
And Sorry for my weird English.