Cannot reverse MatchInv-Doc When CLIENT_ACCOUNTING_DISABLED

Description

Hi,all.

I am Hideaki Hagiwara in Japan.

I think same problem IDEMPIERE-2347.

I found that MMatchInv class have reverse() method and use IsPosted() like a MMatchPO.reverse() method

And MatchInvReverse and MatchPOReverse that are process classes use isPosted() to check before perform reverse() method.

But I consider that this problem is not typo bug.

Order of posting is important for some Cost Accounting method.

I made patch.

Please check it.

Environment

None

Activity

Show:
Carlos Ruiz
August 6, 2016, 8:30 PM

, I think instead of reading isPosted the processes must look for isProcessed flag. Attaching a patch IDEMPIERE-3120_CR.patch - can you please review / comment / test?

Regards,

Carlos Ruiz

萩原秀明
August 7, 2016, 1:03 PM

Hi, Calros.

Thank you for your reply and your patch.
I confirmed your patch.

I think It is OK!
I think that it is most important in this ticket to be able to reverse.

So, I agree with your patch.
Thank you for asking for my opinion.

Assignee

Carlos Ruiz

Reporter

萩原秀明

Labels

Tested By

None

Components

Affects versions

Priority

Major
Configure