From 0ce67b6355cb77f7587be7517884d397a860c76b Mon Sep 17 00:00:00 2001 From: LSmyrnaios Date: Wed, 29 Mar 2023 12:08:43 +0300 Subject: [PATCH] Avoid assigning an empty list, in case the "reportItems" is null, inside "SushiliteServiceImpl.getReportResults()". --- .../manager/service/sushilite/SushiliteServiceImpl.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/eu/dnetlib/repo/manager/service/sushilite/SushiliteServiceImpl.java b/src/main/java/eu/dnetlib/repo/manager/service/sushilite/SushiliteServiceImpl.java index bd00c7c..d38c306 100644 --- a/src/main/java/eu/dnetlib/repo/manager/service/sushilite/SushiliteServiceImpl.java +++ b/src/main/java/eu/dnetlib/repo/manager/service/sushilite/SushiliteServiceImpl.java @@ -93,8 +93,8 @@ public class SushiliteServiceImpl implements SushiliteService { List requestedItemList = new ArrayList<>(); Customer customer = reportResponseWrapper.getReportResponse().getReportWrapper().getReport().getCustomer(); - List allReportItems = customer.getReportItems(); + List allReportItems = customer.getReportItems(); if ( allReportItems != null ) { try { int totalItems = allReportItems.size(); @@ -111,9 +111,9 @@ public class SushiliteServiceImpl implements SushiliteService { logger.error("Exception on getReportResults - trying to cast strings to integers", e); throw e; } - } - customer.setReportItems(requestedItemList); // Setting the reportItems to the "customer"-reference, updates the "reportResponseWrapper" object. + customer.setReportItems(requestedItemList); // Setting the reportItems to the "customer"-reference, updates the "reportResponseWrapper" object. + } return reportResponseWrapper; }