imp: reduce useless LDAP deletion notice

fdaugan · fdaugan · commit 9848c2f505c7 · 2025-11-27T20:36:18.000+01:00
diff --git a/src/main/java/org/ligoj/app/plugin/id/dao/IdCacheDaoImpl.java b/src/main/java/org/ligoj/app/plugin/id/dao/IdCacheDaoImpl.java
@@ -365,8 +365,10 @@ public void removeUserFromGroup(final UserOrg user, final GroupOrg group) {
 	}
 
 	private void deleteBatch(Class<?> cls, List<String> ids, Consumer<List<String>> batchConsumer) {
-		log.info("Deleting removed cache {} {} entries", cls.getSimpleName(), ids.size());
-		ListUtils.partition(ids, 1000).forEach(batchConsumer);
+		if (!ids.isEmpty()) {
+			log.info("Deleting removed cache {} {} entries", cls.getSimpleName(), ids.size());
+			ListUtils.partition(ids, 1000).forEach(batchConsumer);
+		}
 	}
 
 	@Override

Original file line number	Diff line number	Diff line change
`@@ -365,8 +365,10 @@ public void removeUserFromGroup(final UserOrg user, final GroupOrg group) {`
`365`	`365`	`}`
`366`	`366`
`367`	`367`	`private void deleteBatch(Class<?> cls, List<String> ids, Consumer<List<String>> batchConsumer) {`
`368`		`- log.info("Deleting removed cache {} {} entries", cls.getSimpleName(), ids.size());`
`369`		`- ListUtils.partition(ids, 1000).forEach(batchConsumer);`
	`368`	`+ if (!ids.isEmpty()) {`
	`369`	`+ log.info("Deleting removed cache {} {} entries", cls.getSimpleName(), ids.size());`
	`370`	`+ ListUtils.partition(ids, 1000).forEach(batchConsumer);`
	`371`	`+ }`
`370`	`372`	`}`
`371`	`373`
`372`	`374`	`@Override`