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

["Sebb (JIRA)" <ji...@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.