[ https://issues.apache.org/jira/browse/SLING-10844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17429620#comment-17429620 ]
Aditya Seth commented on SLING-10844: ------------------------------------- PR raised: https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/50 > ResourceMapper.getMapping() returns null for empty path > ------------------------------------------------------- > > Key: SLING-10844 > URL: https://issues.apache.org/jira/browse/SLING-10844 > Project: Sling > Issue Type: Bug > Components: ResourceResolver > Affects Versions: Resource Resolver 1.7.0 > Reporter: Mohit Arora > Priority: Major > Fix For: Resource Resolver 1.7.12 > > > After the bug fix for SLING-9620, the behavior for > ResourceMapper.getAllMappings() was changed such that the mappings list > remains empty if the resourcePath provided is an empty string. Prior to this > bug fix, the mappings list contained a single entry for empty path. > Since mappings list is empty, [ResourceMapper.getMapping() returns > null|https://github.com/apache/sling-org-apache-sling-resourceresolver/blob/master/src/main/java/org/apache/sling/resourceresolver/impl/mapping/ResourceMapperImpl.java#L74-L75]. > The [javadoc of > ResourceMapper.getMapping()|https://github.com/apache/sling-org-apache-sling-api/blob/master/src/main/java/org/apache/sling/api/resource/mapping/ResourceMapper.java#L67] > API mentions that it will return a non null value if the resourcepath is not > null. > We need to decide on the expected behavior here. If mappings list should not > be updated in case of empty resourcePath then the API annotation will have to > be changed and it can potentially be a backwards incompatible issue in theory > as the consumers of the API may not have added a null check to the return > value. -- This message was sent by Atlassian Jira (v8.3.4#803005)