Shift string literal to left side of equals(avoid null pointer ex) 96/92996/1
authorsindhu3672 <arcot.sindhuri@huawei.com>
Thu, 8 Aug 2019 06:10:48 +0000 (11:40 +0530)
committersindhu3672 <arcot.sindhuri@huawei.com>
Thu, 8 Aug 2019 06:10:48 +0000 (11:40 +0530)
Avoid null pointer exception mso-openstack-adapters classes

Issue-ID: SO-2187

Signed-off-by: sindhu3672 <arcot.sindhuri@huawei.com>
Change-Id: Ie7099318a79162aed26ec3a8e4a013cece367736

adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfCloudifyAdapterImpl.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfPluginAdapterImpl.java

index 96e5db7..f09fa34 100644 (file)
@@ -854,7 +854,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
 
                     // Include aliases.
                     String alias = htp.getParamAlias();
-                    if (alias != null && !alias.equals("") && !params.containsKey(alias)) {
+                    if (alias != null && !"".equals(alias) && !params.containsKey(alias)) {
                         params.put(alias, htp);
                     }
                 }
index d5fe285..41bcc8c 100644 (file)
@@ -323,7 +323,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
         String type = templateParam.getParamType();
         logger.debug("Parameter: {} is of type ", templateParam.getParamName(), type);
 
-        if (type.equalsIgnoreCase("number")) {
+        if ("number".equalsIgnoreCase(type)) {
             try {
                 return Integer.valueOf(inputValue.toString());
             } catch (Exception e) {