Fix a potential NPE 93/9093/1
authorDeterme, Sebastien (sd378r) <sd378r@intl.att.com>
Tue, 29 Aug 2017 11:43:14 +0000 (04:43 -0700)
committerDeterme, Sebastien (sd378r) <sd378r@intl.att.com>
Tue, 29 Aug 2017 11:43:14 +0000 (04:43 -0700)
The code must verify if the policy has been found or not

Change-Id: I41343aec40643c29180c892996e4a65d0805193e
Issue-Id: CLAMP-43
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>
src/main/java/org/onap/clamp/clds/client/OperationalPolicyDeleteDelegate.java

index e7b0be5..2c60941 100644 (file)
@@ -56,13 +56,15 @@ public class OperationalPolicyDeleteDelegate implements JavaDelegate {
         prop.setCurrentModelElementId(policy.getId());
 
         String responseMessage = "";
-        for (PolicyChain policyChain : policy.getPolicyChains()) {
-            prop.setPolicyUniqueId(policyChain.getPolicyId());
-            responseMessage = policyClient.deleteBrms(prop);
-        }
-        if (responseMessage != null) {
-            execution.setVariable("operationalPolicyDeleteResponseMessage", responseMessage.getBytes());
-        }
+               if(policy.isFound()){
+               for (PolicyChain policyChain : policy.getPolicyChains()) {
+                   prop.setPolicyUniqueId(policyChain.getPolicyId());
+                   responseMessage = policyClient.deleteBrms(prop);
+               }
+               if (responseMessage != null) {
+                   execution.setVariable("operationalPolicyDeleteResponseMessage", responseMessage.getBytes());
+               }
+           }
     }
 
 }