Skip to content

Commit a9a7ba0

Browse files
committed
lint: peek
1 parent 82c4b96 commit a9a7ba0

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/main/java/org/ligoj/app/plugin/id/dao/IdCacheDaoImpl.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -491,19 +491,23 @@ private long updateDelegateDn(final Map<String, ? extends CacheContainer> contai
491491
BiConsumer<DelegateOrg, String> setDn) {
492492
final var updated = new AtomicInteger();
493493
// Get all delegates of he related receiver type
494-
delegateOrgRepository.findAllBy(typePath, type).stream().peek(d -> {
494+
delegateOrgRepository.findAllBy(typePath, type).stream().filter(d -> {
495495
// Consider only the existing ones
496496
final var dn = Optional.ofNullable(containers.get(id.apply(d))).map(CacheContainer::getDescription)
497497
.orElse(null);
498498

499499
// Consider only the dirty one
500500
final var delegateDn = getDn.apply(d);
501+
if (delegateDn == null) {
502+
return false;
503+
}
501504
if (!delegateDn.equalsIgnoreCase(dn)) {
502505
// The delegate DN needed this update
503506
setDn.accept(d, dn);
504507
updated.incrementAndGet();
505508
}
506-
}).filter(d -> getDn.apply(d) == null).forEach(delegateOrgRepository::delete);
509+
return true;
510+
}).forEach(delegateOrgRepository::delete);
507511
return updated.get();
508512
}
509513

0 commit comments

Comments
 (0)