Openbravo Issue Tracking System - Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0014825Modules[Openbravo ERP] 09. Financial managementpublic2010-10-08 07:272014-04-01 21:46
Reportereintelau 
Assigned Toeduardo_Argal 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS Version5.2
Product Version 
Target VersionFixed in Version 
Merge Request Status
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Support ticket
OBNetwork customerNo
Regression introduced in release
Summary0014825: Credit Memo journal entry is incorrect when APRM module enabled
DescriptionWhen a credit memo (AP or AR) is posted using the APRM module, the generated journal is incorrect and doesn't balance.

For example, if you have AP Credit Memo for $400, the expected journal is:
                DR CR
Payables 400
Expense 400

What you actually get is:
                 DR CR
Payables 400
Expense 400
Suspense Balance 800

The error is due to the fact that the credit memo records it line amounts/totals as positive value, but the payment plan has a negative amount. When the document is posted, the negative payment amount is used to create a payables DR. However because the amount is negative (& negative amounts are disabled), the DR is turned into a CR.

If you look at the logic for the same process without the APRM module installed you can see that the debt payment amount is negated before being used to create the journal entry.

Steps To Reproduce1) Enable APRM module
2) Create a AP credit memo
3) Complete & post the credit memo
4) Check the resulting journal entries.
Proposed SolutionNegate the payment amount before using it to create a payables/receivables journal line.
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-10-08 07:27eintelauNew Issue
2010-10-08 07:27eintelauAssigned To => dalsasua
2010-10-08 07:57dalsasuaAssigned Todalsasua => eduardo_Argal
2010-10-13 05:25hgbotCheckin
2010-10-13 05:25hgbotNote Added: 0031784
2010-10-13 05:25hgbotStatusnew => resolved
2010-10-13 05:25hgbotResolutionopen => fixed
2010-10-13 05:25hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/fc2b40560e15dfb49c821dc520a952d5e182d8f1 [^]
2010-10-13 10:07hudsonbotCheckin
2010-10-13 10:07hudsonbotNote Added: 0031792
2010-10-19 16:54adrianromeroNote Added: 0031985
2010-10-19 16:54adrianromeroStatusresolved => new
2010-10-19 16:54adrianromeroResolutionfixed => open
2010-10-19 17:27adrianromeroNote Added: 0031988
2010-10-26 11:58hgbotCheckin
2010-10-26 11:58hgbotNote Added: 0032215
2010-10-26 11:58hgbotStatusnew => resolved
2010-10-26 11:58hgbotResolutionopen => fixed
2010-10-26 11:58hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/fc2b40560e15dfb49c821dc520a952d5e182d8f1 [^] => http://code.openbravo.com/erp/devel/pi/rev/fb8df49a0cc3f9a314dfd2ce43ff047782a4703c [^]
2010-11-05 12:44hudsonbotCheckin
2010-11-05 12:44hudsonbotNote Added: 0032415
2010-11-11 11:56psarobeProjectOpenbravo ERP => Modules
2014-04-01 21:46plujanNote Added: 0065713
2014-04-01 21:46plujanStatusresolved => closed

Notes
(0031784)
hgbot   
2010-10-13 05:25   
Repository: erp/devel/pi
Changeset: fc2b40560e15dfb49c821dc520a952d5e182d8f1
Author: Ben Sommerville <ben.sommerville <at> eintel.com.au>
Date: Wed Oct 13 14:23:12 2010 +1100
URL: http://code.openbravo.com/erp/devel/pi/rev/fc2b40560e15dfb49c821dc520a952d5e182d8f1 [^]

Fixes issue 14825: Negate payment amounts for credit memos when posting journal with APRM installed

---
M src/org/openbravo/erpCommon/ad_forms/DocInvoice.java
---
(0031792)
hudsonbot   
2010-10-13 10:07   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/fc2b40560e15 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/8e8cd8bfe8c7 [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.18519.obx [^]
(0031985)
adrianromero   
2010-10-19 16:54   
Changeset has been backed out in order to review properly the fix before
(0031988)
adrianromero   
2010-10-19 17:27   
Backout changeset
https://code.openbravo.com/erp/devel/pi/rev/12ab1003e0c3 [^]
(0032215)
hgbot   
2010-10-26 11:58   
Repository: erp/devel/pi
Changeset: fb8df49a0cc3f9a314dfd2ce43ff047782a4703c
Author: Eduardo Argal Guibert <eduardo.argal <at> openbravo.com>
Date: Tue Oct 26 11:58:14 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/fb8df49a0cc3f9a314dfd2ce43ff047782a4703c [^]

Fixes bug 0014825, 0014931

---
M src/org/openbravo/erpCommon/ad_forms/DocInvoice.java
---
(0032415)
hudsonbot   
2010-11-05 12:44   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/fb8df49a0cc3 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/f2cf138fa03c [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.18797.obx [^]
(0065713)
plujan   
2014-04-01 21:46   
Closed as marked as Resolved for too long