0033226: ConcatPDFReport has a null pointer exception if is used without reportConfiguration parameter

Description

ConcatPDFReport has a null pointer exception if is used without reportConfiguration parameter
Method concatPDFReport(List<JasperPrint> jasperPrintList, boolean createBookmarks,
OutputStream outputStream)
is creating a null pointer exception

Steps To Reproduce

Create a call on concatPDFReport(List<JasperPrint> jasperPrintList, boolean createBookmarks,
OutputStream outputStream)
Without any SimplePdfExporterConfiguration

Proposed Solution

On ReportingUtils.java:
   public static void concatPDFReport(List<JasperPrint> jasperPrintList, boolean createBookmarks,
      OutputStream outputStream, SimplePdfExporterConfiguration reportConfiguration)
      throws JRException {

    JRPdfExporter exporter = new JRPdfExporter();
    SimpleOutputStreamExporterOutput exporterOutput = new SimpleOutputStreamExporterOutput(
        outputStream);
    SimplePdfExporterConfiguration configuration = reportConfiguration != null ? reportConfiguration
        : new SimplePdfExporterConfiguration();

    reportConfiguration.setCreatingBatchModeBookmarks(createBookmarks);
    exporter.setConfiguration(configuration);
    exporter.setExporterInput(SimpleExporterInput.getInstance(jasperPrintList));
    exporter.setExporterOutput(exporterOutput);

    exporter.exportReport();
  }

Line:
    reportConfiguration.setCreatingBatchModeBookmarks(createBookmarks);
Should be:
    configuration.setCreatingBatchModeBookmarks(createBookmarks);

Tags

No tags attached.

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

blocks

defect

0033222

closed

caristu

ConcatPDFReport has a null pointer exception if is used without reportConfiguration parameter

Notes
(0087192) hgbot (developer) 2016-06-13 11:00	Repository: erp/backports/3.0PR16Q1.4 Changeset: fe177b6118d7d599ab1fbc21fc7750f8f4dddead Author: Carlos Aristu <carlos.aristu <at> openbravo.com> Date: Mon Jun 13 10:59:22 2016 +0200 URL: http://code.openbravo.com/erp/backports/3.0PR16Q1.4/rev/fe177b6118d7d599ab1fbc21fc7750f8f4dddead [^] fixes issue 33226: ConcatPDFReport can throw a null pointer exception Use the correct variable name (configuration instead of reportConfiguration) when using the setCreatingBatchModeBookmarks method --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/report/ReportingUtils.java ---

(0087668) alostale (manager) 2016-06-20 13:06	code reviewed + tested

Issue History
Date Modified	Username	Field	Change
2016-06-13 09:49	alostale	Type	defect => backport
2016-06-13 09:49	alostale	Target Version	=> 3.0PR16Q1.4
2016-06-13 11:00	hgbot	Checkin
2016-06-13 11:00	hgbot	Note Added: 0087192
2016-06-13 11:00	hgbot	Status	scheduled => resolved
2016-06-13 11:00	hgbot	Resolution	open => fixed
2016-06-13 11:00	hgbot	Fixed in SCM revision	=> http://code.openbravo.com/erp/backports/3.0PR16Q1.4/rev/fe177b6118d7d599ab1fbc21fc7750f8f4dddead [^]
2016-06-13 11:21	caristu	Review Assigned To	=> alostale
2016-06-20 13:06	alostale	Note Added: 0087668
2016-06-20 13:06	alostale	Status	resolved => closed
2016-06-20 13:06	alostale	Fixed in Version	=> 3.0PR16Q1.4