[
https://issues.apache.org/jira/browse/SLING-6327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15693783#comment-15693783
]
Julian Sedding commented on SLING-6327:
---------------------------------------
Based on our discussion on the mailing list, I think the following assertion
should also hold:
{code}
assertTrue(areResourceTypesEqual("/libs/foo/bar", "/apps/foo/bar", String[]{
"/libs", "/apps" }))
{code}
Other than that the patch looks fine.
> ResourceResolverImpl.isResourceType() should consider resource resolver's
> search path
> -------------------------------------------------------------------------------------
>
> Key: SLING-6327
> URL: https://issues.apache.org/jira/browse/SLING-6327
> Project: Sling
> Issue Type: Improvement
> Components: ResourceResolver
> Affects Versions: API 2.15.0, Resource Resolver 1.5.2
> Reporter: Konrad Windszus
> Assignee: Konrad Windszus
> Attachments: SLING-6327-v01.patch
>
>
> Currently the following two expressions return false
> {{ResourceResolverImpl.isResourceType(<Resource with
> resourceType="sling/some/type">, "/libs/sling/some/type"}}
> {{ResourceResolverImpl.isResourceType(<Resource with
> resourceType="/libs/sling/some/type">, "sling/some/type"}}
> Since it cannot always be influenced whether the given resource is absolute
> or relative (because both usually works from a rendering perspective when you
> talk about the current request's resource), both cases should actually return
> {{true}}.
> See also the related discussion at
> http://www.mail-archive.com/[email protected]/msg62351.html
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)