This is an automated email from the ASF dual-hosted git repository.
reschke pushed a commit to branch master
in repository
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-resourceresolver.git
The following commit(s) were added to refs/heads/master by this push:
new 844b238 SLING-12401: MapEntries.isAliasValid() misnamed - rename to
isAliasInvalid() (#121)
844b238 is described below
commit 844b2381d095d3bd386f70bb2734736b09b9db22
Author: Julian Reschke <[email protected]>
AuthorDate: Thu Aug 8 16:37:54 2024 +0200
SLING-12401: MapEntries.isAliasValid() misnamed - rename to
isAliasInvalid() (#121)
---
.../org/apache/sling/resourceresolver/impl/mapping/MapEntries.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java
b/src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java
index 792ec7b..73fed11 100644
---
a/src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java
+++
b/src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java
@@ -1269,7 +1269,7 @@ public class MapEntries implements
log.debug("Found alias, total size {}", aliasArray.length);
// the order matters here, the first alias in the array must
come first
for (final String alias : aliasArray) {
- if (isAliasValid(alias)) {
+ if (isAliasInvalid(alias)) {
log.warn("Encountered invalid alias {} under parent
path {}. Refusing to use it.", alias, parentPath);
} else {
Map<String, Collection<String>> parentMap =
map.computeIfAbsent(parentPath, key -> new ConcurrentHashMap<>());
@@ -1297,7 +1297,7 @@ public class MapEntries implements
/**
* Check alias syntax
*/
- private static boolean isAliasValid(String alias) {
+ private static boolean isAliasInvalid(String alias) {
boolean invalid = alias.equals("..") || alias.equals(".");
if (!invalid) {
for (final char c : alias.toCharArray()) {
