[ 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)