Added null check on removing id in dependsOn

Nakul Sabharwal · Nakul Sabharwal · commit fbe12a7b8e45 · 2019-03-08T13:44:26.000+05:30
diff --git a/src/main/java/com/microsoft/graph/content/MSBatchRequestContent.java b/src/main/java/com/microsoft/graph/content/MSBatchRequestContent.java
@@ -63,7 +63,9 @@ public boolean removeBatchRequestStepWithId(String requestId) {
 			batchRequestStepsHashMap.remove(requestId);
 			removed = true;
 			for(Map.Entry<String, MSBatchRequestStep> steps : batchRequestStepsHashMap.entrySet()) {
-				while(steps.getValue().getArrayOfDependsOnIds().remove(requestId));
+				if(steps.getValue() != null && steps.getValue().getArrayOfDependsOnIds() != null) {
+					while(steps.getValue().getArrayOfDependsOnIds().remove(requestId));
+				}
 			}
 		}
 		return removed;

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,9 @@ public boolean removeBatchRequestStepWithId(String requestId) {`
`63`	`63`	`batchRequestStepsHashMap.remove(requestId);`
`64`	`64`	`removed = true;`
`65`	`65`	`for(Map.Entry<String, MSBatchRequestStep> steps : batchRequestStepsHashMap.entrySet()) {`
`66`		`- while(steps.getValue().getArrayOfDependsOnIds().remove(requestId));`
	`66`	`+ if(steps.getValue() != null && steps.getValue().getArrayOfDependsOnIds() != null) {`
	`67`	`+ while(steps.getValue().getArrayOfDependsOnIds().remove(requestId));`
	`68`	`+ }`
`67`	`69`	`}`
`68`	`70`	`}`
`69`	`71`	`return removed;`