[jira] Created: (CONFIGURATION-421) VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (CONFIGURATION-421) VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null

JIRA jira@apache.org
VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null
-------------------------------------------------------------------------------------

                 Key: CONFIGURATION-421
                 URL: https://issues.apache.org/jira/browse/CONFIGURATION-421
             Project: Commons Configuration
          Issue Type: Bug
          Components: File reloading
            Reporter: Sebb


VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null:

{noformat}
        if (configuration.getURL() == null && configuration.getFileName() == null)
        {
            return;
        }
        if (this.configuration == null)
        {
            throw new IllegalStateException("No configuration has been set for this strategy");
        }
{noformat}

The if statements should be swapped.

[Note found by Findbugs]

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.