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

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Fix Version/s: 2.x
• Component/s: File reloading
• Assignee: Unassigned
• Reporter: Sebb
• Created: 08/Jun/10 12:31
• Updated: 24/Mar/16 20:12

Description

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

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

The if statements should be swapped.

[Note found by Findbugs]