Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0037156 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 09. Financial management | major | always | 2017-10-24 13:45 | 2018-01-08 18:19 | |||
| Reporter | ngarcia | View Status | public | |||||
| Assigned To | AtulOpenbravo | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR18Q1 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 28d8fe1517d7 | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | dmiguelez | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0037156: Cannot unpost a financial account transaction reconciled with a previous period's date: c_period_id is null in fact_acct | |||||||
| Description | Cannot unpost a financial account transaction reconciled with a previous period's date: c_period_id is null in fact_acct | |||||||
| Steps To Reproduce | As group admin role: Create a financial account transaction with today's date in a financial account with matching algorithm not defined (in order to manually reconcile it) Process it and post it Launch Reconcile process and change statement date to a month before Remove the Hide Transactions after statement date option and select previously created transaction Set an ending balance and reconcile Try to unpost the financial account (selecting the option of Delete accounting entry) and check the following error message is shown: "Period Closed. It is not possible to unpost a document in a closed period" | |||||||
| Proposed Solution | The problem is in TransactionsDao.getPeriod function, it does not return any period and null is set in TransactionsDao.updateAccountingDate function. The date filters are wrong. Moreover, the TransactionsDao.getPeriod is not filtering the result by org or calendar. Check if DALUtils.getPeriodByDate function could be used in this case | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0100271) AtulOpenbravo (developer) 2017-11-06 08:34 |
Test Plan - Login as International Group Admin role. - Navigate to Financial Account: Bank Account 1, set Matching Algorithm as blank. - Create a transaction in "Bank Account 1" with Type= BP Withdrawal, Trans Date = Acct Date = today's date, GLItem: Salaries, Withdrawal Amount = 100, - Process it and post it. - Launch Reconcile process and change statement date to a month before = todays date - 30 - Remove the Hide Transactions after statement date option and select previously created transaction - Set an ending balance = (starting balance - 100) for eg: Starting Bal= 4473292.33, Ending Balance = 4473192.33 - Click on reconcile. - Realize that previous financial transaction account date is updated with statement date. - Search for the above transactionand try to unpost it. (selecting the option of Delete accounting entry) - Realize document is unposted successfully. |
|
(0100277) hgbot (developer) 2017-11-06 09:16 |
Repository: erp/devel/pi Changeset: 28d8fe1517d7212b0ac707cfad8f555c510002ae Author: Atul Gaware <atul.gaware <at> openbravo.com> Date: Mon Nov 06 11:45:05 2017 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/28d8fe1517d7212b0ac707cfad8f555c510002ae [^] Fixes Issue 37156, 37219:Cannot unpost a financial account transaction reconciled with a previous period's date: c_period_id is null in fact_acct ** Use AccDefUtility.getCurrentPeriod to get the perid for financial account transaction date. ** Removed public getPeriod method. --- M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/TransactionsDao.java --- |
|
(0100282) dmiguelez (developer) 2017-11-06 10:37 |
Code Review + Testing Ok |
|
(0101374) hudsonbot (developer) 2018-01-03 12:38 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/c81e0d3cbab5 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2017-10-24 13:45 | ngarcia | New Issue | |
| 2017-10-24 13:45 | ngarcia | Assigned To | => Triage Finance |
| 2017-10-24 13:45 | ngarcia | Modules | => Core |
| 2017-10-24 13:45 | ngarcia | Resolution time | => 1510614000 |
| 2017-10-24 13:45 | ngarcia | Triggers an Emergency Pack | => No |
| 2017-10-24 13:56 | ngarcia | Issue Monitored: networkb | |
| 2017-10-24 15:59 | ngarcia | Proposed Solution updated | |
| 2017-10-24 19:30 | AtulOpenbravo | Assigned To | Triage Finance => AtulOpenbravo |
| 2017-10-24 19:30 | AtulOpenbravo | Status | new => scheduled |
| 2017-10-25 00:19 | eugeni | Issue Monitored: eugeni | |
| 2017-11-03 13:30 | dmiguelez | Relationship added | related to 0037219 |
| 2017-11-06 08:34 | AtulOpenbravo | Note Added: 0100271 | |
| 2017-11-06 09:16 | hgbot | Checkin | |
| 2017-11-06 09:16 | hgbot | Note Added: 0100277 | |
| 2017-11-06 09:16 | hgbot | Status | scheduled => resolved |
| 2017-11-06 09:16 | hgbot | Resolution | open => fixed |
| 2017-11-06 09:16 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/28d8fe1517d7212b0ac707cfad8f555c510002ae [^] |
| 2017-11-06 10:37 | dmiguelez | Review Assigned To | => dmiguelez |
| 2017-11-06 10:37 | dmiguelez | Note Added: 0100282 | |
| 2017-11-06 10:37 | dmiguelez | Status | resolved => closed |
| 2018-01-03 12:38 | hudsonbot | Checkin | |
| 2018-01-03 12:38 | hudsonbot | Note Added: 0101374 | |
| 2018-01-08 18:19 | dmiguelez | Fixed in Version | => 3.0PR18Q1 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |