fix(rest): avoid NPE in obligation Boolean filter

YianZhao · GMishx · commit 98c91155d6e4 · 2026-04-06T11:54:33.000+05:30
Signed-off-by: Alex &lt;alextanzhao22@gmail.com&gt;
diff --git a/rest/resource-server/src/main/java/org/eclipse/sw360/rest/resourceserver/core/RestControllerHelper.java b/rest/resource-server/src/main/java/org/eclipse/sw360/rest/resourceserver/core/RestControllerHelper.java
@@ -549,10 +549,10 @@ public void addEmbeddedLicenses(HalResource<Release> halComponent, Set<String> l
     }
 
     public Set<String> getObligationIdsFromRequestWithValueTrue(Map<String, Boolean> reqBodyMaps) {
-        Map<String, Boolean> obligationIdsRequest = reqBodyMaps.entrySet().stream()
-                .filter(reqBodyMap-> reqBodyMap.getValue().equals(true))
-                .collect(Collectors.toMap(reqBodyMap-> reqBodyMap.getKey(),reqBodyMap -> reqBodyMap.getValue()));
-        return obligationIdsRequest.keySet();
+        return reqBodyMaps.entrySet().stream()
+                .filter(entry -> Boolean.TRUE.equals(entry.getValue()))
+                .map(Map.Entry::getKey)
+                .collect(Collectors.toSet());
     }
 
     public boolean checkDuplicateLicense(List<License> licenses, String licenseId) {

Original file line number	Diff line number	Diff line change
`@@ -549,10 +549,10 @@ public void addEmbeddedLicenses(HalResource<Release> halComponent, Set<String> l`
`549`	`549`	`}`
`550`	`550`
`551`	`551`	`public Set<String> getObligationIdsFromRequestWithValueTrue(Map<String, Boolean> reqBodyMaps) {`
`552`		`- Map<String, Boolean> obligationIdsRequest = reqBodyMaps.entrySet().stream()`
`553`		`- .filter(reqBodyMap-> reqBodyMap.getValue().equals(true))`
`554`		`- .collect(Collectors.toMap(reqBodyMap-> reqBodyMap.getKey(),reqBodyMap -> reqBodyMap.getValue()));`
`555`		`- return obligationIdsRequest.keySet();`
	`552`	`+ return reqBodyMaps.entrySet().stream()`
	`553`	`+ .filter(entry -> Boolean.TRUE.equals(entry.getValue()))`
	`554`	`+ .map(Map.Entry::getKey)`
	`555`	`+ .collect(Collectors.toSet());`
`556`	`556`	`}`
`557`	`557`
`558`	`558`	`public boolean checkDuplicateLicense(List<License> licenses, String licenseId) {`