[
https://issues.apache.org/jira/browse/SLING-5025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14745172#comment-14745172
]
Carsten Ziegeler commented on SLING-5025:
-----------------------------------------
For 2. : I think you want to check if it's not live and then throw :)
For 1. : I think we should change the description of when the RR is considered
closed: it's either closed if done by calling the method or if the factory gets
inactive etc. This saves us from having the rather longthy description on each
method. WDYT?
> Make ResourceResolver methods throw ISE in case the according factory is
> unregistered
> -------------------------------------------------------------------------------------
>
> Key: SLING-5025
> URL: https://issues.apache.org/jira/browse/SLING-5025
> Project: Sling
> Issue Type: Improvement
> Components: ResourceResolver
> Affects Versions: Resource Resolver 1.2.6
> Reporter: Konrad Windszus
> Attachments: SLING-5025-api.v01.diff,
> SLING-5025-resourceresolver.v01.diff
>
>
> In SLING-4360 the {{ResourceResolver.isLive()}} method was extended to check
> if the {{ResourceResolverFactory}} has been unregistered in the mean time.
> That check should be done in addition for every method on the resource
> resolver and should throw an ISE in case the factory is no longer registered.
> See also the discussion at
> http://www.mail-archive.com/[email protected]/msg49364.html.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
