commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simone Tripodi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CHAIN-80) NPE when submitting null commands map to CatalogBase
Date Fri, 22 Mar 2013 15:25:16 GMT

    [ https://issues.apache.org/jira/browse/CHAIN-80?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13610374#comment-13610374
] 

Simone Tripodi commented on CHAIN-80:
-------------------------------------

I agree that it would be better if the {{null}} argument would raise an _IllegalArgumentException_
- could you please re-submit 
another patch, with testcases, applying that policy?

Many thanks in advance and thanks for contributing!
                
> NPE when submitting null commands map to CatalogBase
> ----------------------------------------------------
>
>                 Key: CHAIN-80
>                 URL: https://issues.apache.org/jira/browse/CHAIN-80
>             Project: Commons Chain
>          Issue Type: Bug
>    Affects Versions: 2.0
>            Reporter: Steve Westwood
>            Priority: Minor
>              Labels: catalog
>             Fix For: 2.0
>
>         Attachments: CHAIN-80-patch.txt.txt
>
>
> The command relies on _Map.putAll()_ to populate *CatalogBase* when passed into the constructor,
but as the javadocs specify at http://docs.oracle.com/javase/6/docs/api/java/util/Map.html#putAll(java.util.Map),
this throws an NPE. The constructor requires a guard inside to protect it from null objects
being passed as an argument

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message