Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0014825 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Modules] [Openbravo ERP] 09. Financial management | minor | always | 2010-10-08 07:27 | 2014-04-01 21:46 | |||
Reporter | eintelau | View Status | public | |||||
Assigned To | eduardo_Argal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | fb8df49a0cc3 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Linux 32 bit | Database | PostgreSQL | Java version | 1.6 | |||
OS Version | 5.2 | Database version | 8.3 | Ant version | 1.7 | |||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Regression date | ||||||||
Regression introduced by commit | ||||||||
Regression level | ||||||||
Review Assigned To | ||||||||
Support ticket | ||||||||
OBNetwork customer | No | |||||||
Regression introduced in release | ||||||||
Summary | 0014825: Credit Memo journal entry is incorrect when APRM module enabled | |||||||
Description | When 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 Reproduce | 1) Enable APRM module 2) Create a AP credit memo 3) Complete & post the credit memo 4) Check the resulting journal entries. | |||||||
Proposed Solution | Negate the payment amount before using it to create a payables/receivables journal line. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0031784) hgbot (developer) 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 (viewer) 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 (viewer) 2010-10-19 16:54 |
Changeset has been backed out in order to review properly the fix before |
(0031988) adrianromero (viewer) 2010-10-19 17:27 |
Backout changeset https://code.openbravo.com/erp/devel/pi/rev/12ab1003e0c3 [^] |
(0032215) hgbot (developer) 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 (viewer) 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 (viewer) 2014-04-01 21:46 |
Closed as marked as Resolved for too long |
![]() |
|||
Date Modified | Username | Field | Change |
2010-10-08 07:27 | eintelau | New Issue | |
2010-10-08 07:27 | eintelau | Assigned To | => dalsasua |
2010-10-08 07:57 | dalsasua | Assigned To | dalsasua => eduardo_Argal |
2010-10-13 05:25 | hgbot | Checkin | |
2010-10-13 05:25 | hgbot | Note Added: 0031784 | |
2010-10-13 05:25 | hgbot | Status | new => resolved |
2010-10-13 05:25 | hgbot | Resolution | open => fixed |
2010-10-13 05:25 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/fc2b40560e15dfb49c821dc520a952d5e182d8f1 [^] |
2010-10-13 10:07 | hudsonbot | Checkin | |
2010-10-13 10:07 | hudsonbot | Note Added: 0031792 | |
2010-10-19 16:54 | adrianromero | Note Added: 0031985 | |
2010-10-19 16:54 | adrianromero | Status | resolved => new |
2010-10-19 16:54 | adrianromero | Resolution | fixed => open |
2010-10-19 17:27 | adrianromero | Note Added: 0031988 | |
2010-10-26 11:58 | hgbot | Checkin | |
2010-10-26 11:58 | hgbot | Note Added: 0032215 | |
2010-10-26 11:58 | hgbot | Status | new => resolved |
2010-10-26 11:58 | hgbot | Resolution | open => fixed |
2010-10-26 11:58 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/fc2b40560e15dfb49c821dc520a952d5e182d8f1 [^] => http://code.openbravo.com/erp/devel/pi/rev/fb8df49a0cc3f9a314dfd2ce43ff047782a4703c [^] |
2010-11-05 12:44 | hudsonbot | Checkin | |
2010-11-05 12:44 | hudsonbot | Note Added: 0032415 | |
2010-11-11 11:56 | psarobe | Project | Openbravo ERP => Modules |
2014-04-01 21:46 | plujan | Note Added: 0065713 | |
2014-04-01 21:46 | plujan | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |