From: varun gudisena Date: Fri, 22 Sep 2017 23:58:06 +0000 (+0000) Subject: Merge "Fix for Sonar Blocker issues" X-Git-Tag: v1.0.1~38 X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=648be3c324aa6d9c5b868106cffbf1c97c94c046;hp=f5b284e5e0eacfec3312d3318ec769269e1ec2a5;p=dmaap%2Fmessagerouter%2Fmessageservice.git Merge "Fix for Sonar Blocker issues" --- diff --git a/src/main/java/com/att/nsa/dmaap/filemonitor/ServicePropertiesMap.java b/src/main/java/com/att/nsa/dmaap/filemonitor/ServicePropertiesMap.java index 731428d..0f7744c 100644 --- a/src/main/java/com/att/nsa/dmaap/filemonitor/ServicePropertiesMap.java +++ b/src/main/java/com/att/nsa/dmaap/filemonitor/ServicePropertiesMap.java @@ -52,12 +52,15 @@ public class ServicePropertiesMap */ public static void refresh(File file) throws Exception { + String filePath= null; try { logger.info("Loading properties - " + (file != null?file.getName():"")); //Store .json & .properties files into map of maps - String filePath = file.getPath(); + if (file != null) { + filePath = file.getPath(); + } if(filePath.lastIndexOf(".json")>0){ @@ -65,9 +68,10 @@ public class ServicePropertiesMap TypeReference> typeRef = new TypeReference>() {}; HashMap propMap = om.readValue(file, typeRef); - HashMap lcasePropMap = new HashMap(); - for (String key : propMap.keySet() ) + HashMap lcasePropMap = new HashMap<>(); + for (Map.Entry entry : propMap.entrySet()) { + String key = entry.getKey(); String lcaseKey = ifNullThenEmpty(key); lcasePropMap.put(lcaseKey, propMap.get(key)); } @@ -81,7 +85,7 @@ public class ServicePropertiesMap prop.load(fis); @SuppressWarnings("unchecked") - HashMap propMap = new HashMap((Map)prop); + HashMap propMap = new HashMap<>((Map)prop); mapOfMaps.put(file.getName(), propMap); } @@ -111,7 +115,7 @@ public class ServicePropertiesMap * @param fileName fileName * @return mapProp */ - public static HashMap getProperties(String fileName){ + public static Map getProperties(String fileName){ return mapOfMaps.get(fileName); }