[ 
https://issues.apache.org/jira/browse/SLING-5610?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Oliver Lietz closed SLING-5610.
-------------------------------

> ResourceUtil.getValueMap() contains a wrong @Nonnull annotation 
> ----------------------------------------------------------------
>
>                 Key: SLING-5610
>                 URL: https://issues.apache.org/jira/browse/SLING-5610
>             Project: Sling
>          Issue Type: Bug
>          Components: API
>    Affects Versions: API 2.9.0
>            Reporter: Dominik Lienemann
>            Assignee: Konrad Windszus
>             Fix For: API 2.12.0
>
>
> *org.apache.sling.api.resource.ResourceUtil.getValueMap(res)* performs a null 
> check as first action (*if ( res == null ) {*). Also, the javadoc states that 
> this method is null safe.
> But the method contains a @Nonnull annotation for the res parameter 
> (*@Nonnull final Resource res*) which forces the calling methods to do an 
> additional null check, even if the method handles it internally.
> I suggest to remove the annotation for the parameter.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to