improve logic of MMovementLineConfirm.ProcessLine() method.

Description

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.

Environment

None

Activity

Show:
Heng Sin Low
July 29, 2013, 9:49 AM

Hi Hideaki Hagiwara,

I have comes to a different conclusion after analysis of the issue highlighted. I believe the confirmed qty should include scrapped qty and the error is in the calculation of difference qty. Difference qty should be Target Qty - Confirmed Qty instead of Target Qty - Confirmed Qty - Scrapped Qty. This is to better differentiate between stock that is lost during transport ( difference ) and stock that is damaged but have reach its destination ( scrapped ).

Regards,
Low

萩原秀明
July 29, 2013, 12:57 PM

Hi Heng Sin.

Thank you for the reply.
I understood your explanation.
I completely agree with you.

I confirmed that committed trekglobal/iDempiere repository.

I am looking forward to committing iDempiere/iDempiere repository.

Thank you very much
And please contact me if there is anything that I can help you with.

Assignee

Heng Sin Low

Reporter

萩原秀明

Labels

Tested By

None

Components

Priority

Minor
Configure