You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by da...@apache.org on 2025/03/07 08:07:30 UTC
(jackrabbit-oak) 01/01: OAK-11562 : removed usage of Guava's Iterables.addAll
This is an automated email from the ASF dual-hosted git repository.
daim pushed a commit to branch OAK-11562
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git
commit d173ca19219a198b50e3fabdac6937a096846e9f
Author: Rishabh Kumar <di...@adobe.com>
AuthorDate: Fri Mar 7 13:37:16 2025 +0530
OAK-11562 : removed usage of Guava's Iterables.addAll
---
.../spi/security/authorization/cug/impl/CugAccessControlManager.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugAccessControlManager.java b/oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugAccessControlManager.java
index 96bcf54137..75ae64237d 100644
--- a/oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugAccessControlManager.java
+++ b/oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugAccessControlManager.java
@@ -369,7 +369,7 @@ class CugAccessControlManager extends AbstractAccessControlManager implements Cu
String path = eval.remove();
Tree t = immutableRoot.getTree(path);
if (PathUtils.denotesRoot(path)) {
- Iterables.addAll(eval, nestedCugPaths(t));
+ nestedCugPaths(t).forEach(eval::add);
}
if (CugUtil.isSupportedPath(path, supportedPaths)) {
Tree cug = CugUtil.getCug(t);
@@ -378,7 +378,7 @@ class CugAccessControlManager extends AbstractAccessControlManager implements Cu
if (!Collections.disjoint(ImmutableSet.copyOf(principalNames), ImmutableSet.copyOf(pNames.getValue(Type.STRINGS)))) {
candidates.add(path);
}
- Iterables.addAll(eval, nestedCugPaths(cug));
+ nestedCugPaths(cug).forEach(eval::add);
}
}
}