[MYFACES-1883] /WEB-INF/faces-config.xml Loaded twice if logging disabled.

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.2.2, 1.2.3
• Fix Version/s: 1.1.6
• Component/s: General
• Assignee: Leonardo Uribe
• Reporter: David Waters
• Created: 16/Jun/08 15:25
• Updated: 12/Aug/08 21:03
• Resolved: 19/Jun/08 16:45

Description

Bug introduced in FacesConfigurator Version 580395 by mmarinschek 28/Sep/07
While working on MYFACES-1689 mmarinschek introduced an action based on a check if the provided value was the default and not load this file if it was. Unfortunitly this was reusing an existing check that also check to see if log.isWarnEnbaled, so the code will behavoe materilly differently if logging is enabled.

see FacesConfigurator.getConfigFilesList()

WARN FacesConfigurator - /WEB-INF/faces-config.xml has been specified in the javax.faces.CONFIG_FILES context parameter of the deployment descriptor. This will automatically be removed, if we wouldn't do this, it would be loaded twice. See JSF spec 1.1, 10.3.2