New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SLING-10447 Improve the querys that are used to load vanity paths, by specifying path restrictions #46
Conversation
… specifying path restrictions
…torage # Conflicts: # src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java # src/test/java/org/apache/sling/resourceresolver/impl/mapping/InMemoryResourceProvider.java
…s now superseded by the alias, vanity & mapping root settings
Thanks for the PR @henrykuijpers ! I'll not merge it immediately as @kwin has some concerns on whether this is the right thing to do. If we end up going this way, we will need the branch to be rebased as it has conflicts. |
src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverFactoryConfig.java
Show resolved
Hide resolved
src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java
Outdated
Show resolved
Hide resolved
src/main/java/org/apache/sling/resourceresolver/impl/mapping/MapEntries.java
Outdated
Show resolved
Hide resolved
…te sets, make /jcr:system exclude default in ResourceResolverFactoryActivator, make query building code easier to read/understand
Just did a commit with some fixes, @jsedding . :) Can you have another look? |
src/main/java/org/apache/sling/resourceresolver/impl/CommonResourceResolverFactoryImpl.java
Show resolved
Hide resolved
src/test/java/org/apache/sling/resourceresolver/impl/mapping/MapEntriesTest.java
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! |
@@ -707,7 +748,7 @@ public void onChange(final List<ResourceChange> changes) { | |||
log.debug("onChange, type={}, path={}", rc.getType(), path); | |||
|
|||
// don't care for system area | |||
if (path.startsWith(JCR_SYSTEM_PREFIX)) { | |||
if (path.startsWith(JCR_SYSTEM_PATH + '/')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
while this is not a big deal, why did you change this from a constant to an expression that is executed each time?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is an oversight: I think I inlined a few values, I'll change it back to a constant.
Kudos, SonarCloud Quality Gate passed! |
This PR can be closed, right? (see https://issues.apache.org/jira/browse/SLING-10447) |
No description provided.