Re: svn commit: r1771782 - /sling/trunk/bundles/api/src/main/java/org/apache/sling/api/resource/ResourceResolverWrapper.java

2016-11-28 Thread Radu Cotescu
Hi Julian, You're right, we should also wrap the returned resources. I've just replied on JIRA a couple of minutes ago. Thanks, Radu On Mon, 28 Nov 2016 at 19:15 Julian Sedding wrote: > Hi Radu > > I would argue that this implementation is incorrect. > > Consider that with

Re: svn commit: r1771782 - /sling/trunk/bundles/api/src/main/java/org/apache/sling/api/resource/ResourceResolverWrapper.java

2016-11-28 Thread Julian Sedding
Hi Radu I would argue that this implementation is incorrect. Consider that with this implementation resolver != resolver.getResource("/foo").getResourceResolver(), which should be the case IMHO. I commented on the issue with two tickets, where the same feature was previously discussed. The