[MYFACES-3255] UICommand.queueEvent description changed since 2.0

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2.11, 2.0.8, 2.1.2
• Assignee: Leonardo Uribe
• Reporter: Leonardo Uribe
• Created: 02/Aug/11 03:07
• Updated: 23/Aug/11 02:36
• Resolved: 02/Aug/11 03:25

Description

Checking some javadoc for JSF 2.2 "viewAction", I notice our implementation of UICommand.queueEvent does not match with the documentation since JSF 2.0.

Before:

public void queueEvent(FacesEvent e)

Intercept queueEvent and, for ActionEvents, mark the phaseId for the event to be PhaseId.APPLY_REQUEST_VALUES if the immediate flag is true, PhaseId.INVOKE_APPLICATION otherwise.

After:

public void queueEvent(FacesEvent e)

Intercept queueEvent and take the following action. If the event is an ActionEvent, obtain the UIComponent instance from the event. If the component is an ActionSource obtain the value of its "immediate" property. If it is true, mark the phaseId for the event to be PhaseId.APPLY_REQUEST_VALUES otherwise, mark the phaseId to be PhaseId.INVOKE_APPLICATION. The event must be passed on to super.queueEvent() before returning from this method.

Tracking this one I found this issue:

http://java.net/jira/browse/JAVASERVERFACES-622

Since it was added after 1.2 rev A and before 2.0, myfaces code still does what the spec says since 1.2. Anyway, we should change MyFaces code to follow the javadoc description.