From: r.bogacki <r.bogacki@samsung.com>
Date: Tue, 7 May 2019 06:22:38 +0000 (+0200)
Subject: Removed null checks from ResourceRequestBuilder
X-Git-Tag: 1.5.2~361^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F09%2F87109%2F1;p=so.git

Removed null checks from ResourceRequestBuilder

Removed unnecessary null checks from ResourceRequestBuilder
according to comments from https://gerrit.onap.org/r/#/c/87001/

Change-Id: I21140d812982713229c322d772e6a10f7c7c044d
Issue-ID: SO-1498
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
---

diff --git a/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/bpmn/common/resource/ResourceRequestBuilder.java b/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/bpmn/common/resource/ResourceRequestBuilder.java
index 0df25be714..8aab4ec818 100644
--- a/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/bpmn/common/resource/ResourceRequestBuilder.java
+++ b/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/bpmn/common/resource/ResourceRequestBuilder.java
@@ -151,13 +151,13 @@ public class ResourceRequestBuilder {
             Map<String, String> resourceInputData = getResourceInputStr(serviceVnfCust, resourceCustomizationUuid);
 
             // find match in network resource
-            if (resourceInputData != null && resourceInputData.isEmpty()) {
+            if (resourceInputData.isEmpty()) {
                 List<Map<String, Object>> serviceNetworkCust =
                         (List<Map<String, Object>>) serviceResources.get("serviceNetworks");
                 resourceInputData = getResourceInputStr(serviceNetworkCust, resourceCustomizationUuid);
 
                 // find match in AR resource
-                if (resourceInputData == null) {
+                if (resourceInputData.isEmpty()) {
                     List<Map<String, Object>> serviceArCust =
                             (List<Map<String, Object>>) serviceResources.get("serviceAllottedResources");
                     resourceInputData = getResourceInputStr(serviceArCust, resourceCustomizationUuid);
@@ -166,7 +166,7 @@ public class ResourceRequestBuilder {
 
             String resourceInputStr = null;
             ResourceLevel resourceLevel = null;
-            if (resourceInputData != null) {
+            if (!resourceInputData.isEmpty()) {
                 resourceInputStr = resourceInputData.get("resourceInput");
                 resourceLevel = ResourceLevel.valueOf(resourceInputData.get("nodeType"));
             }
@@ -200,7 +200,7 @@ public class ResourceRequestBuilder {
                 return resourceInputMap;
             }
         }
-        return null;
+        return new HashMap<>();
     }
 
     // this method combines resource input with service input