[
https://issues.apache.org/jira/browse/SLING-13018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18041890#comment-18041890
]
Konrad Windszus commented on SLING-13018:
-----------------------------------------
Isn't this a duplicate of SLING-11067?
> URIProvider.toURI() has to throw IAE to signal failure to adapt
> ---------------------------------------------------------------
>
> Key: SLING-13018
> URL: https://issues.apache.org/jira/browse/SLING-13018
> Project: Sling
> Issue Type: Task
> Components: API, JCR
> Reporter: Joerg Hoh
> Priority: Major
>
> {{JcrNodeResource.convertToPublicURI()}} iterates through a list of
> registered URIProviders to find if an URIProvider can handle this.
> Unfortunately the indication that an URIProvider cannot handle this is by
> throwing an {{IllegalArgumentException}}, which can consume quite some time.
> For that I suggest to extend the {{URIProvider}} interface with a new method:
> {noformat}
> URI toURIOrNull(@NotNull Resource resource, @NotNull URIProvider.Scope
> scope, @NotNull URIProvider.Operation operation);
> {noformat}
> which will return null if that URIProvider cannot handle the provided
> resource. If we provide a default implementation, so it will be only a minor
> version update on the package. And it's possible to override it on
> URIProviders with an improved version avoiding the creation of this exception.
> Of course it also requires a small change in the JCRNodeResource to use this
> new method.
> (In Sling/AEM this code path is used by JS Scripting.)
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
