diff options
author | Parshad Patel <pars.patel@samsung.com> | 2020-04-08 14:02:26 +0900 |
---|---|---|
committer | Parshad Patel <pars.patel@samsung.com> | 2020-08-27 18:45:59 +0900 |
commit | c7cdb80d8e7ac1ea8aaaa9e5875ad1071af07440 (patch) | |
tree | 8ee901992cf44a33040e4ff6732fb2fe5156a5fa /ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition | |
parent | 23b262748c6fc10ddc8eb55036b82be7f71e120b (diff) |
NPE issue fix
Fix A "NullPointerException" could be thrown
Either log or rethrow this exception
Issue-ID: PORTAL-562
Change-Id: Iaaba9f5db6a82655739ebb96dddad928ac864b09
Signed-off-by: Parshad Patel <pars.patel@samsung.com>
Diffstat (limited to 'ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition')
-rw-r--r-- | ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition/ReportDefinition.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition/ReportDefinition.java b/ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition/ReportDefinition.java index 16129b98..8b1cc8cb 100644 --- a/ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition/ReportDefinition.java +++ b/ecomp-sdk/epsdk-analytics/src/main/java/org/onap/portalsdk/analytics/model/definition/ReportDefinition.java @@ -332,7 +332,7 @@ public class ReportDefinition extends ReportWrapper implements Serializable { DbUtils.commitTransaction(connection); } catch (RaptorException e) { - e.printStackTrace(); + logger.debug(EELFLoggerDelegate.errorLogger, "RaptorException in persistReport", e); DbUtils.rollbackTransaction(connection); throw e; } finally { @@ -664,6 +664,7 @@ public class ReportDefinition extends ReportWrapper implements Serializable { try { dct.getColFilterList().getColFilter().remove(filterPos); } catch (IndexOutOfBoundsException e) { + logger.debug(EELFLoggerDelegate.errorLogger, "IndexOutOfBoundsException in removeColumnFilter", e); } resetCache(true); @@ -796,6 +797,7 @@ public class ReportDefinition extends ReportWrapper implements Serializable { sem.setSemaphoreId(generateNewSemaphoreId()); sem.setSemaphoreName(semaphoreName); } catch (JAXBException ex) { + logger.debug(EELFLoggerDelegate.errorLogger, "JAXBException in addSemaphore", ex); throw new RaptorException(ex.getMessage(), ex.getCause()); } @@ -941,7 +943,7 @@ public class ReportDefinition extends ReportWrapper implements Serializable { fft.setRangeStartDate(DatatypeFactory.newInstance() .newXMLGregorianCalendar(rangeEndDate.YEAR, rangeEndDate.MONTH, rangeEndDate.DAY_OF_WEEK, rangeEndDate.HOUR, rangeEndDate.MINUTE, rangeEndDate.SECOND, rangeEndDate.MILLISECOND, rangeEndDate.ZONE_OFFSET)); } catch (DatatypeConfigurationException ex) { - + logger.debug(EELFLoggerDelegate.errorLogger, "DatatypeConfigurationException in addFormFieldType", ex); } fft.setRangeStartDateSQL(rangeStartDateSQL); |