Add SO Message instead of Exception when vmIds are empty

Add the conditions to log messages instead of null pointer exception
when vmIds or vserverIds are not present.
Issue-ID: SO-2186
Signed-off-by: Bonkur, Venkat (vb8416) <vb8416@att.com>
Change-Id: I27dd8a5480cf5e2d8588cc71ca3bfefaf0e138cc

1 files changed, 16 insertions, 0 deletions

diff --git a/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/client/appc/ApplicationControllerAction.java b/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/client/appc/ApplicationControllerAction.java
index 9828d11186..c05557a317 100644
--- a/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/client/appc/ApplicationControllerAction.java
+++ b/bpmn/MSOCommonBPMN/src/main/java/org/onap/so/client/appc/ApplicationControllerAction.java
@@ -78,8 +78,24 @@ public class ApplicationControllerAction {
                     appCStatus = healthCheckAction(msoRequestId, vnfId, vnfName, vnfHostIpAddress, controllerType);
                     break;
                 case Snapshot:
+                    if (vmIdList.isEmpty()) {
+                        logger.warn("vmIdList is Empty in AppCClient");
+                        break;
+                    }
                     String vmIds = JsonUtils.getJsonValue(vmIdList, "vmIds");
+                    if (vmIds == null) {
+                        logger.warn("vmIds null in AppCClient");
+                        break;
+                    }
+                    if (vserverIdList.isEmpty()) {
+                        logger.warn("vserverIdList is empty in AppCClient");
+                        break;
+                    }
                     String vserverIds = JsonUtils.getJsonValue(vserverIdList, "vserverIds");
+                    if (vserverIds == null) {
+                        logger.warn("vserverIds  null in AppCClient");
+                        break;
+                    }
                     String vmId = "";
                     String vserverId = "";
                     ObjectMapper mapper = new ObjectMapper();