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

Attachments

2

Activity

萩原秀明 
August 7, 2016 at 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.

Carlos Ruiz 
August 6, 2016 at 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

Fixed

Details

Assignee

Reporter

Labels

Components

Affects versions

Priority

Created June 2, 2016 at 11:40 AM
Updated June 6, 2018 at 6:38 PM
Resolved August 10, 2016 at 4:04 PM