Hi,

Bertrand Delacretaz schrieb:
> Hi,
> 
> On Fri, Sep 4, 2009 at 11:03 AM, <[email protected]> wrote:
>> ...   
>> sling/trunk/bundles/servlets/get/src/main/java/org/apache/sling/servlets/get/impl/helpers/HtmlRendererServlet.java
> 
>> +        String resourceSuperType = r.getResourceSuperType();
>> +        if (resourceSuperType == null) {
>> +            resourceSuperType = ResourceUtil.getResourceSuperType(
>> +                r.getResourceResolver(), r.getResourceType());
>> +            if (resourceSuperType == null) {
>> +                resourceSuperType = "sling/servlet/default (default 
>> resource super type)";
>> +            } else {
>> +                resourceSuperType += " (from resource type hierarchy)";
>> +            }
> 
> If this is the way of finding a resource's supertype, shouldn't this
> code be part of the ResourceUtil class?
> 
> I'd like to add the info to the RequestProgressTracker as well, and
> duplicating that code does not feel right.

You are perfectly right, and I do not even know our API. There is a
ReousrceUtil.findResourceSuperType method which exactly does that ....
my bad.

Thanks for pointing this out.

Regards
Felix

Reply via email to