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

John D. Ament commented on TAMAYA-181:
--------------------------------------

I agree, the looks of it is that name needs to be populated before the 
properties loaded.  If tests fail because of that, we need to look closely to 
see if the tests are accurate.  Are they checking for null behavior perhaps?  
We can leverage {{Objects.requireNonNull}} ?

> Warning-"Flood" from SimplePropertySource
> -----------------------------------------
>
>                 Key: TAMAYA-181
>                 URL: https://issues.apache.org/jira/browse/TAMAYA-181
>             Project: Tamaya
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 0.2-incubating
>         Environment: any
>            Reporter: Jürg Spiess
>            Priority: Minor
>             Fix For: 0.3-incubating
>
>
> If the constructor
> {code}public SimplePropertySource(URL propertiesLocation){code} is used, 
> STDOUT is flooded with warnings about "missing name".
> Solution:
> {code}
>     public SimplePropertySource(URL propertiesLocation) {
>         super(0);
>         // switch the next 2 lines: name should be set BEFORE load(...)!
>         this.properties = load(propertiesLocation);
>         this.name = propertiesLocation.toString();
>     }
> {code}



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

Reply via email to