FINERACT-1345: Fixing support for parameterType reports
This commit is contained in:
committed by
Petri Tuomola
parent
bd177bd206
commit
f87b66d4e1
+1
-1
@@ -105,7 +105,7 @@ public class RunreportsApiResource {
|
|||||||
// Pass through isSelfServiceUserReport so that ReportingProcessService implementations can use it
|
// Pass through isSelfServiceUserReport so that ReportingProcessService implementations can use it
|
||||||
queryParams.putSingle(IS_SELF_SERVICE_USER_REPORT_PARAMETER, Boolean.toString(isSelfServiceUserReport));
|
queryParams.putSingle(IS_SELF_SERVICE_USER_REPORT_PARAMETER, Boolean.toString(isSelfServiceUserReport));
|
||||||
|
|
||||||
String reportType = this.readExtraDataAndReportingService.getReportType(reportName, isSelfServiceUserReport);
|
String reportType = this.readExtraDataAndReportingService.getReportType(reportName, isSelfServiceUserReport, parameterType);
|
||||||
ReportingProcessService reportingProcessService = this.reportingProcessServiceProvider.findReportingProcessService(reportType);
|
ReportingProcessService reportingProcessService = this.reportingProcessServiceProvider.findReportingProcessService(reportType);
|
||||||
if (reportingProcessService == null) {
|
if (reportingProcessService == null) {
|
||||||
throw new PlatformServiceUnavailableException("err.msg.report.service.implementation.missing",
|
throw new PlatformServiceUnavailableException("err.msg.report.service.implementation.missing",
|
||||||
|
|||||||
+1
-1
@@ -34,7 +34,7 @@ public interface ReadReportingService {
|
|||||||
|
|
||||||
ReportData retrieveReport(Long id);
|
ReportData retrieveReport(Long id);
|
||||||
|
|
||||||
String getReportType(String reportName, boolean isSelfServiceUserReport);
|
String getReportType(String reportName, boolean isSelfServiceUserReport, boolean isParameterType);
|
||||||
|
|
||||||
Collection<ReportParameterData> getAllowedParameters();
|
Collection<ReportParameterData> getAllowedParameters();
|
||||||
|
|
||||||
|
|||||||
+5
-1
@@ -212,7 +212,11 @@ public class ReadReportingServiceImpl implements ReadReportingService {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public String getReportType(final String reportName, final boolean isSelfServiceUserReport) {
|
public String getReportType(final String reportName, final boolean isSelfServiceUserReport, final boolean isParameterType) {
|
||||||
|
if (isParameterType) {
|
||||||
|
return "Table";
|
||||||
|
}
|
||||||
|
|
||||||
final String sql = "SELECT ifNull(report_type,'') AS report_type FROM `stretchy_report` WHERE report_name = ? AND self_service_user_report = ?";
|
final String sql = "SELECT ifNull(report_type,'') AS report_type FROM `stretchy_report` WHERE report_name = ? AND self_service_user_report = ?";
|
||||||
|
|
||||||
final String sqlWrapped = this.genericDataService.wrapSQL(sql);
|
final String sqlWrapped = this.genericDataService.wrapSQL(sql);
|
||||||
|
|||||||
+1
-1
@@ -441,7 +441,7 @@ public class ReportMailingJobWritePlatformServiceImpl implements ReportMailingJo
|
|||||||
|
|
||||||
try {
|
try {
|
||||||
final boolean isSelfServiceUserReport = false;
|
final boolean isSelfServiceUserReport = false;
|
||||||
final String reportType = this.readReportingService.getReportType(reportName, isSelfServiceUserReport);
|
final String reportType = this.readReportingService.getReportType(reportName, isSelfServiceUserReport, false);
|
||||||
final ReportingProcessService reportingProcessService = this.reportingProcessServiceProvider
|
final ReportingProcessService reportingProcessService = this.reportingProcessServiceProvider
|
||||||
.findReportingProcessService(reportType);
|
.findReportingProcessService(reportType);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user