Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0039859Openbravo ERP09. Financial managementpublic2018-12-27 16:302019-03-28 14:55
ReporterLeyre 
Assigned ToAugustoMauch 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR19Q2 
Merge Request Status
Review Assigned ToSandrahuguet
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2018-05-18
Regression introduced in release3.0PR18Q3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/3404760330bcfea180a7869095eefbabf5e760e0 [^]
Triggers an Emergency PackNo
Summary0039859: Reconciliation reports not generated
DescriptionWhen the printing of the reconciliation reports is launched, the popup remains frozen and no report is generated.
Steps To ReproduceBackend:
1. Open "Financial Account" window.
2. Select the "Reconciliations" tab of an account with information in this tab.
3. Select any Reconciliation
4. Press Reconciliation Details or Reconciliation Summary buttons

Verify that the pop-up remains frozen and no report is generated.
Proposed SolutionThe log shows a non-existent jrxml error.
It seems that in version 18Q3 a new class "ReportCompiler.java" was inserted that modified the previous way to compile / execute jasper subreports.
This class extracts the name of the subreports using the name of the parameter. In "ReportReconciliation.java", the name given to the parameter related to the Payments subreport is "SUBREP_PAYMENT".
According to this logic, when compiling the subreport, the jrxml "PAYMENT.jrxml" is searched, when in fact it is called "OutstandingPayment.jrxml".

It works incorrectly from version 18Q3
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0038588 new Triage Omni OMS Refactor reports based on HttpSecureAppServlet.renderJR to avoid manual subreport compilation 
depends on backport 00398693.0PR19Q1 closed AugustoMauch Reconciliation reports not generated 
depends on backport 00398703.0PR18Q4.2 closed AugustoMauch Reconciliation reports not generated 
depends on backport 00398713.0PR18Q3.5 closed AugustoMauch Reconciliation reports not generated 
caused by feature request 0038518 closed caristu Ability of compiling subreports with HttpSecureAppServlet.renderJR 
Attached Filesdiff subReportRefactor.diff (6,786) 2018-12-28 13:19
https://issues.openbravo.com/file_download.php?file_id=12476&type=bug
diff subReportRefactor2.diff (4,395) 2018-12-31 13:47
https://issues.openbravo.com/file_download.php?file_id=12479&type=bug
Issue History
Date ModifiedUsernameFieldChange
2018-12-27 16:30LeyreNew Issue
2018-12-27 16:30LeyreAssigned To => Triage Finance
2018-12-27 16:30LeyreModules => Core
2018-12-27 16:30LeyreRegression date => 2019-01-17
2018-12-27 16:30LeyreRegression introduced in release => 3.0PR18Q3
2018-12-27 16:30LeyreTriggers an Emergency Pack => No
2018-12-27 16:33LeyreResolution time => 1547679600
2018-12-27 16:33LeyreRegression date2019-01-17 =>
2018-12-27 17:07SandrahuguetRelationship addedrelated to 0038518
2018-12-27 17:07SandrahuguetAssigned ToTriage Finance => AugustoMauch
2018-12-28 09:09SandrahuguetSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=18077#r18077
2018-12-28 13:18AugustoMauchAssigned ToAugustoMauch => Sandrahuguet
2018-12-28 13:19AugustoMauchFile Added: subReportRefactor.diff
2018-12-28 14:11AugustoMauchRelationship replacedcaused by 0038518
2018-12-28 14:11AugustoMauchRelationship addedrelated to 0038588
2018-12-28 14:47SandrahuguetRegression level => Production - Confirmed Stable
2018-12-28 14:47SandrahuguetRegression date => 2018-05-18
2018-12-28 14:47SandrahuguetRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/3404760330bcfea180a7869095eefbabf5e760e0 [^]
2018-12-28 14:47SandrahuguetAssigned ToSandrahuguet => platform
2018-12-31 13:47AugustoMauchFile Added: subReportRefactor2.diff
2018-12-31 13:49AugustoMauchAssigned Toplatform => Triage Finance
2018-12-31 15:57AugustoMauchAssigned ToTriage Finance => AugustoMauch
2019-01-02 17:27hgbotCheckin
2019-01-02 17:27hgbotNote Added: 0108789
2019-01-02 17:27hgbotStatusnew => resolved
2019-01-02 17:27hgbotResolutionopen => fixed
2019-01-02 17:27hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/4f9bcd0a6fff981c60cfff6196949beba5800b0d [^]
2019-01-02 17:27AugustoMauchStatusresolved => new
2019-01-02 17:27AugustoMauchResolutionfixed => open
2019-01-02 18:39AugustoMauchStatusnew => scheduled
2019-01-02 18:39AugustoMauchStatusscheduled => resolved
2019-01-02 18:39AugustoMauchResolutionopen => fixed
2019-01-03 08:44hgbotCheckin
2019-01-03 08:44hgbotNote Added: 0108800
2019-01-03 09:12SandrahuguetReview Assigned To => Sandrahuguet
2019-01-03 09:12SandrahuguetNote Added: 0108807
2019-01-03 09:12SandrahuguetStatusresolved => closed
2019-01-03 09:12SandrahuguetFixed in Version => 3.0PR19Q2
2019-03-28 14:55hudsonbotCheckin
2019-03-28 14:55hudsonbotNote Added: 0110715
2019-03-28 14:55hudsonbotCheckin
2019-03-28 14:55hudsonbotNote Added: 0110716

Notes
(0108789)
hgbot   
2019-01-02 17:27   
Repository: erp/devel/pi
Changeset: 4f9bcd0a6fff981c60cfff6196949beba5800b0d
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Mon Dec 31 15:58:22 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/4f9bcd0a6fff981c60cfff6196949beba5800b0d [^]

Fixes issue 39859: Replaces SUBREP_ prefix in report

The SUBREP_ prefix can only be used in a report parameter if there exists a jrxml file wich the same
name as the parameter suffix. Otherwise the report is not generated.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_reports/ReportReconciliation.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_reports/ReportReconciliation.jrxml
---
(0108800)
hgbot   
2019-01-03 08:44   
Repository: erp/devel/pi
Changeset: 8688e0940a7d142419897c77d59840542e1b4d27
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Thu Jan 03 08:43:09 2019 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/8688e0940a7d142419897c77d59840542e1b4d27 [^]

related to issue 39859 update copyright year

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_reports/ReportReconciliation.java
---
(0108807)
Sandrahuguet   
2019-01-03 09:12   
Code review + testing OK
(0110715)
hudsonbot   
2019-03-28 14:55   
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/b2fbc1588df4 [^]
Maturity status: Test
(0110716)
hudsonbot   
2019-03-28 14:55   
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/b2fbc1588df4 [^]
Maturity status: Test