joerghoh commented on code in PR #171:
URL:
https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/171#discussion_r2056390674
##########
src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverImpl.java:
##########
@@ -928,20 +928,21 @@ private Resource getChildInternal(final Resource parent,
final String childName)
// we do not have a child with the exact name, so we look for
// a child, whose alias matches the childName
if (factory.getMapEntries().isOptimizeAliasResolutionEnabled()) {
- logger.debug("getChildInternal: Optimize Alias Resolution is
Enabled");
- // optimization made in SLING-2521
+ final String parentPath = parent.getPath();
+ logger.debug(
+ "getChildInternal: Optimize Alias Resolution is Enabled,
looking up {} in {}",
+ childName,
+ parentPath);
+
+ // optimized alias resolution: aliases are cached by MapEntries
final Optional<String> aliasedResourceName =
-
factory.getMapEntries().getAliasMap(parent.getPath()).entrySet().stream()
+
factory.getMapEntries().getAliasMap(parentPath).entrySet().stream()
.filter(e -> e.getValue().contains(childName))
.findFirst()
.map(Map.Entry::getKey);
if (aliasedResourceName.isPresent()) {
- final String aliasPath;
- if (aliasedResourceName.get().startsWith("/")) {
- aliasPath = aliasedResourceName.get();
- } else {
- aliasPath = parent.getPath() + '/' +
aliasedResourceName.get();
- }
+ // we know that MapEntries already has checked for valid
aliases
+ final String aliasPath = parent.getPath() + '/' +
aliasedResourceName.get();
Review Comment:
can't you use ``parentPath`` here as well?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@sling.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
