Openbravo Issue Tracking System - Modules | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0043380 | Modules | Remittance | public | 2020-03-02 11:44 | 2020-03-13 09:15 |
Reporter | santi_lozano | ||||
Assigned To | AtulOpenbravo | ||||
Priority | high | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | OS | 5 | OS Version | ||
Product Version | |||||
Target Version | Fixed in Version | ||||
Merge Request Status | |||||
Regression date | |||||
Regression introduced by commit | |||||
Regression level | |||||
Review Assigned To | dmiguelez | ||||
Support ticket | 14826 | ||||
OBNetwork customer | OBPS | ||||
Regression introduced in release | |||||
Summary | 0043380: Remittance of a payment out using credit gets unbalance and posts on Suspense Balancing account | ||||
Description | If you try to create a remittance for a payment that is using credit, the post is using a suspense balancing account. It happens because it seems that there is no code to add the payment line related to the credit, then it manages it as an unbalance. Notice that the payment out can be posted correctly | ||||
Steps To Reproduce | :::: REPRODUCE :::: * Download an Openbravo tip instance locally * Go to module management and install modules: “Remittances infrastructure” and “SEPA Credit Transfer Customer-to-Bank” (rebuild the application) * With role F&B International Group Admin go to window Enterprise module management, select both installed modules, then click OK * Go to General Ledger Configuration window, select F&B International Group US/A/US Dollar, go to Active Tables tab. Set tables REM_Remittance, REM_RemittanceLine_Cancel and REM_RemittanceLine_Return as active * Go to window Remittance Type, select SEPA Credit Transfer Customer-to-Bank (pain.001.001.003), go to Accounting tab. Create a new one: Organization = *, General Ledger = F&B International Group US/A/US Dollar, Settlement Account = 8052, Sent Account = 4920 * Go to Financial Account window, select Bank – Account 1, go to Payment Method tab. Add payment method SEPA Credit Transfer Customer-to-Bank with Payment Out Allowed * Go to Purchase Order, create a new one: Organization = F&B US East Coast, Business Partner = Be Soft Drinker, Inc., Payment Method = SEPA Credit Transfer Customer-to-Bank. On Lines tab add one with: Product = Cola, Ordered Quantity = 20 (Total Net Amount should be 11.00 as Net Unit Price is 0.55) * Book the Purchase Order * Go to Payment Out window, create a new one: Organization = F&B US East Coast, Business Partner = Be Soft Drinker, Inc., Generated Credit = 5. Click on button Add Details, then on Done (without selecting anything, Description should be updated “Amount left as credit: 5”) * Go back to the Purchase Order, refresh the window. Click on button Add Payment, select the credit line created on Credit To Use section, Action Regarding Document = Process Made Payment(s), then click on Done (a message will show the Document No. of the new payment that has been created) * Go to Remittance window, create a new one: Organization = F&B US East Coast, Remittance Type = SEPA Credit Transfer Customer-to-Bank (pain.001.001.003), Financial Account = Bank – Account 1 – USD, Name = Test 001 * Click on button Select Payments, then on Done * Click on button Process, then OK * Click on Post button. A window with the posting has been opened (Journal Entries Report), notice that the Debit is using the Suspense Balancing account (account 9999 configured at window General Ledger Configuration, tab General Accounts) * Go to window Settle / Protest Remittances, select the created one, add Accounting Date for today, click on Settle. Go to Remittance window tab Settled, click on button Post. Notice that here is also using the Suspense Balancing Accounting :::: DEBUG :::: 1) Once here you could stop eclipse and run it again with the following breakpoints AcctServer.java line 1168 m_fact[index] = createFact(m_as[index], conn, con, vars); AcctServer.java line 1206 m_fact[index].balanceSource(conn); 2) You can run Eclipse on debug mode, go to Remittance window, tab Settled, click on Unpost, select Delete accounting entry then OK 3) Now click on Post again, click on F8 to go to the second breakpoint, click on F5 to enter to the second breakpoint method. Here (Fact.java) on line 466, 471, 476, adds a new line (variables: this > m_lines) with the Suspense Balancing account, as the previous lines were not balanced because the credit line was missing 4) Click on F8 to end it. Repeat process 2 to go to the first breakpoint. Step Into it now (F5), here (DOCREMRemittanceCancel.java) the lines (variables: fact > m_lines) are added, on line 174, 257, notice that the credit line is not added (there is no code for it) then it happens the unbalance seen before (step 3) 5) More comments on this: 5.1) notice that if there exists multicurrency the line that adds the Suspense Balance account is on DOCREMRemittanceCancel.java (from line 270) 5.2) notice that we are working with settled remittances here, if we were working with other types the java class could be DOCREMRemittance.java or other :::: MORE :::: * you can see that the payment out can be posted correctly * on Financial Account window, select Bank – Account 1, tab Payment Method and the SEPA Credit Transfer Customer-to-Bank, set Upon Payment Use = In Transit Payment Account * go to the Payment Out window and post it | ||||
Proposed Solution | |||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2020-03-02 11:44 | santi_lozano | New Issue | |||
2020-03-02 11:44 | santi_lozano | Assigned To | => Triage Finance | ||
2020-03-02 11:44 | santi_lozano | Resolution time | => 1584918000 | ||
2020-03-02 11:44 | santi_lozano | Support ticket | => 14826 | ||
2020-03-02 11:44 | santi_lozano | OBNetwork customer | => OBPS | ||
2020-03-02 11:46 | santi_lozano | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=20484#r20484 | ||
2020-03-02 11:49 | santi_lozano | Note Added: 0118234 | |||
2020-03-02 12:37 | Practics | Issue Monitored: Practics | |||
2020-03-02 13:26 | dmiguelez | Assigned To | Triage Finance => AtulOpenbravo | ||
2020-03-05 06:40 | AtulOpenbravo | Status | new => scheduled | ||
2020-03-11 17:03 | AtulOpenbravo | Note Added: 0118524 | |||
2020-03-11 17:04 | AtulOpenbravo | Note Edited: 0118524 | bug_revision_view_page.php?bugnote_id=0118524#r20543 | ||
2020-03-12 16:08 | dmiguelez | Note Edited: 0118524 | bug_revision_view_page.php?bugnote_id=0118524#r20550 | ||
2020-03-13 09:09 | hgbot | Checkin | |||
2020-03-13 09:09 | hgbot | Note Added: 0118573 | |||
2020-03-13 09:09 | hgbot | Checkin | |||
2020-03-13 09:09 | hgbot | Note Added: 0118574 | |||
2020-03-13 09:09 | hgbot | Checkin | |||
2020-03-13 09:09 | hgbot | Note Added: 0118575 | |||
2020-03-13 09:09 | hgbot | Checkin | |||
2020-03-13 09:09 | hgbot | Note Added: 0118576 | |||
2020-03-13 09:09 | hgbot | Status | scheduled => resolved | ||
2020-03-13 09:09 | hgbot | Resolution | open => fixed | ||
2020-03-13 09:09 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/mods/org.openbravo.module.remittance/rev/c4ade9832d949a7f69edd8e2c9a1e13481a70efa [^] | ||
2020-03-13 09:09 | hgbot | Checkin | |||
2020-03-13 09:09 | hgbot | Note Added: 0118577 | |||
2020-03-13 09:15 | dmiguelez | Review Assigned To | => dmiguelez | ||
2020-03-13 09:15 | dmiguelez | Note Added: 0118578 | |||
2020-03-13 09:15 | dmiguelez | Status | resolved => closed |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|