Marmelator/persistence-service
1
0
Fork 0
Code Issues Pull Requests 17 Actions Packages Projects Releases 1.1k Wiki Activity

DM-498 - Fix NPE for component overrides with includeDetails flag to true #136

Closed
andrei.isvoran.ext wants to merge 1 commits from DM-498 into master
pull from: DM-498
merge into: Marmelator:master
Conversation 2 Commits 1 Files Changed 1 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
persistence-service-v1/persistence-service-external-api-impl-v2/src/main/java/com/iqser/red/persistence/service/v2/external/api/impl/controller/ComponentControllerV2.java vendored
View File

@ -66,7 +66,7 @@ public class ComponentControllerV2 implements ComponentResource {
}
Map<String, List<com.iqser.red.service.persistence.service.v2.api.external.model.Component>> componentsDetails = new HashMap<>();
if (includeDetails) {
if (includeDetails && overrides.getComponentOverrides() != null) {
componentLog.getComponents().forEach(c -> {
componentsDetails.computeIfAbsent(c.getName(), (x) -> new ArrayList<>())
.addAll(c.getComponentValues().stream().map(entry -> convert(entry, overrides, c.getName())).toList());