[
https://issues.apache.org/jira/browse/SLING-9714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17187883#comment-17187883
]
Mohit Arora commented on SLING-9714:
------------------------------------
[~kpauls], [~radu] As part of this fix, could you please also get rid of the
[misleading javadoc
comment|https://github.com/apache/sling-org-apache-sling-servlets-resolver/blob/master/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResource.java#L89-L90]
on ServletResource.getResourceSuperType()?
> ServletResourceProviderFactory should check for default resourceSuperType to
> determine if explicit resourceSuperType is provided in a servlet
> ---------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: SLING-9714
> URL: https://issues.apache.org/jira/browse/SLING-9714
> Project: Sling
> Issue Type: Bug
> Components: Servlets
> Affects Versions: Servlets Resolver 2.7.8
> Reporter: Mohit Arora
> Priority: Major
>
> After SLING-8170, ServletResourceProviderFactory checks only for [non empty
> resourceSuperType
> property|https://github.com/apache/sling-org-apache-sling-servlets-resolver/blob/master/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProviderFactory.java#L152]
> to determine whether an explicit resourceSuperType is provided for a
> servlet. However, if no resourceSuperType is provided, sling servlet
> annotations put default resourceSuperType of {{sling/bundle/resource}} on the
> servlet. So in order to determine an explicit resourceSuperType property, the
> factory should also check if the resourceSuperType is not the default one
> applied by the servlet annotations.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
