Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0032840 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Modules] Multi-Dimensional Profit And Loss | major | always | 2016-05-03 14:17 | 2016-05-20 13:28 | |||
| Reporter | ngarcia | View Status | public | |||||
| Assigned To | AtulOpenbravo | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | 768b92293823 | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Regression date | ||||||||
| Regression introduced by commit | ||||||||
| Regression level | ||||||||
| Review Assigned To | aferraz | |||||||
| Support ticket | ||||||||
| OBNetwork customer | OBPS | |||||||
| Regression introduced in release | ||||||||
| Summary | 0032840: Generate Profit and Loss Fact Acct process creates records in obdpl_fact_acct_pl although there are no accounts defined in cube | |||||||
| Description | Generate Profit and Loss Fact Acct process creates records in obdpl_fact_acct_pl although there are no accounts defined in cube. The problem happens when the accountsFilter parameter is empty in the populatePAndLFactAcctTable function of the GenerateProfitAndLossFactAcctTable class. The account filter is not applied in the insert function then. | |||||||
| Steps To Reproduce | As system admin role: Generate the Cube from 'DPL - P&L Template' As group admin role: Do not configure the Profit and Loss Accounts in 'Cube Definition - ... - Cube Report Dimension Element' tab Schedule the Generate Profit and Loss Fact Acct background process Go to OB Analytics window, select the DPL cube and add Account dimension in Columns and debitAmount measure in Rows Check values are shown and they should not (the report should be empty as only the values for the accounts defined in the Cube Report Dimension Element should be shown) | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||
|
|||||||||||||||
 Relationships |
|
Notes |
|
|
(0086500) AtulOpenbravo (viewer) 2016-05-17 15:42 edited on: 2016-05-17 15:45 |
Test Plan A - As system admin role: Generate the Cube from 'DPL - P&L Template' - As group admin role: - Do not configure the Profit and Loss Accounts in 'Cube Definition - ... - Cube Report Dimension Element' tab - Schedule the Generate Profit and Loss Fact Acct background process - Check through database there are no records in the obanaly_cube_rep_dim_elem table. This is correct as we don't define them: select count(*) from obanaly_cube_rep_dim_elem No records found. - Check through database there are no record in the obdpl_fact_acct_pl table. select count(*) from obdpl_fact_acct_pl No records found. Test Plan B - As system admin role: Generate the Cube from 'DPL - P&L Template' - As group admin role: - Configure the Profit and Loss Accounts in - Cube Definition = "DPL - P&L - F&B International Group US_A_US Dollar" - Cube Report = "Multi-Dimensional Profit And Loss Process" - Cube Report Dimension = "account - " - Cube Report Dimension Elements - Add 4005 Revenue, Show as Member and Children - Add 4990 Total Revnue, Show as Member - Schedule the Generate Profit and Loss Fact Acct background process. - Check through database there are record in the obdpl_fact_acct_pl table. select * from obdpl_fact_acct_pl - Check following query select distinct account_id from obdpl_fact_acct_pl 2 records found for Revenue and Total Revenue Elements. |
|
(0086611) hgbot (developer) 2016-05-20 13:26 |
Repository: erp/pmods/org.openbravo.financial.dimensionalpandlreport Changeset: 768b92293823bcddeb7ececb7fbe1fd723632c24 Author: Atul Gaware <atul.gaware <at> openbravo.com> Date: Tue May 17 17:58:16 2016 +0530 URL: http://code.openbravo.com/erp/pmods/org.openbravo.financial.dimensionalpandlreport/rev/768b92293823bcddeb7ececb7fbe1fd723632c24 [^] Fixes issue 32840: Generate PL FactAcct process creates unnecessary records Generate Profit and Loss Fact Acct process creates records in obdpl_fact_acct_pl although there are no accounts defined in cube Check accountsfilter is empty or not prior to insert query --- M src/org/openbravo/financial/dimensionalpandlreport/GenerateProfitAndLossFactAcctTable.java --- |
|
(0086612) hgbot (developer) 2016-05-20 13:26 |
Repository: erp/pmods/org.openbravo.financial.dimensionalpandlreport Changeset: efc982cba79773e810025b518fbc0647870129ca Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Fri May 20 13:12:26 2016 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.financial.dimensionalpandlreport/rev/efc982cba79773e810025b518fbc0647870129ca [^] Related to issue 32840: Code review improvements --- M src/org/openbravo/financial/dimensionalpandlreport/GenerateProfitAndLossFactAcctTable.java --- |
|
(0086613) hgbot (developer) 2016-05-20 13:26 |
Repository: erp/pmods/org.openbravo.financial.dimensionalpandlreport Changeset: a48080568eb81e3f851b9f575fe9d384cf6845c3 Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Fri May 20 13:24:16 2016 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.financial.dimensionalpandlreport/rev/a48080568eb81e3f851b9f575fe9d384cf6845c3 [^] Related to issue 32840: Update module version --- M src-db/database/sourcedata/AD_MODULE.xml --- |
|
(0086614) aferraz (viewer) 2016-05-20 13:28 |
Code review + Testing OK |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2016-05-03 14:17 | ngarcia | New Issue | |
| 2016-05-03 14:17 | ngarcia | Assigned To | => Triage Finance |
| 2016-05-03 14:17 | ngarcia | Resolution time | => 1465509600 |
| 2016-05-03 14:17 | ngarcia | OBNetwork customer | => Yes |
| 2016-05-03 14:17 | ngarcia | Relationship added | related to 0032008 |
| 2016-05-09 11:08 | aferraz | Assigned To | Triage Finance => AtulOpenbravo |
| 2016-05-13 15:20 | AtulOpenbravo | Assigned To | AtulOpenbravo => ngarcia |
| 2016-05-13 15:20 | AtulOpenbravo | Status | new => feedback |
| 2016-05-13 16:59 | ngarcia | Status | feedback => new |
| 2016-05-13 16:59 | ngarcia | Assigned To | ngarcia => AtulOpenbravo |
| 2016-05-13 17:00 | ngarcia | Steps to Reproduce Updated | View Revisions |
| 2016-05-17 11:32 | ngarcia | Issue Monitored: networkb | |
| 2016-05-17 15:42 | AtulOpenbravo | Status | new => scheduled |
| 2016-05-17 15:42 | AtulOpenbravo | Note Added: 0086500 | |
| 2016-05-17 15:45 | AtulOpenbravo | Note Edited: 0086500 | View Revisions |
| 2016-05-20 13:26 | hgbot | Checkin | |
| 2016-05-20 13:26 | hgbot | Note Added: 0086611 | |
| 2016-05-20 13:26 | hgbot | Status | scheduled => resolved |
| 2016-05-20 13:26 | hgbot | Resolution | open => fixed |
| 2016-05-20 13:26 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.financial.dimensionalpandlreport/rev/768b92293823bcddeb7ececb7fbe1fd723632c24 [^] |
| 2016-05-20 13:26 | hgbot | Checkin | |
| 2016-05-20 13:26 | hgbot | Note Added: 0086612 | |
| 2016-05-20 13:26 | hgbot | Checkin | |
| 2016-05-20 13:26 | hgbot | Note Added: 0086613 | |
| 2016-05-20 13:28 | aferraz | Review Assigned To | => aferraz |
| 2016-05-20 13:28 | aferraz | Note Added: 0086614 | |
| 2016-05-20 13:28 | aferraz | Status | resolved => closed |
| 2016-05-23 11:27 | aferraz | Relationship added | related to 0033020 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |