[jira] Resolved: (MYFACES-1411) Lifecycle phase executions repetitions

["Matthias Weßendorf (JIRA)" <de...@myfaces.apache.org>]

     [ https://issues.apache.org/jira/browse/MYFACES-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matthias Weßendorf resolved MYFACES-1411.
-----------------------------------------

    Resolution: Fixed

ICLA listed here:
http://people.apache.org/~jim/committers.html

> Lifecycle phase executions repetitions
> --------------------------------------
>
>                 Key: MYFACES-1411
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1411
>             Project: MyFaces Core
>          Issue Type: Improvement
>          Components: JSR-127
>    Affects Versions: 1.1.4
>            Reporter: Nikolay Petrov
>         Assigned To: Martin Marinschek
>             Fix For: 1.1.5-SNAPSHOT
>
>         Attachments: ApplyRequestValuesExecutor.java, InvokeApplicationExecutor.java, LifecycleImpl.java, PhaseExecutor.java, ProcessValidationsExecutor.java, RenderResponseExecutor.java, RestoreViewExecutor.java, UpdateModelValuesExecutor.java
>
>
> Every phase in LifecycleImpl looks like:
>     private boolean applyRequestValues(FacesContext facesContext, PhaseListenerManager phaseListenerMgr)
>         throws FacesException
>     {
>     		boolean skipFurtherProcessing = false;
>         if (log.isTraceEnabled()) log.trace("entering applyRequestValues in " + LifecycleImpl.class.getName());
>         try {
>             phaseListenerMgr.informPhaseListenersBefore(PhaseId.APPLY_REQUEST_VALUES);
>             if(isResponseComplete(facesContext, "applyRequestValues", true))
>             {
>                             // have to return right away
>                             return true;
>             }
>             if(shouldRenderResponse(facesContext, "applyRequestValues", true))
>             {
>                             skipFurtherProcessing = true;
>             }
>             facesContext.getViewRoot().processDecodes(facesContext);
>         } finally {
>             phaseListenerMgr.informPhaseListenersAfter(PhaseId.APPLY_REQUEST_VALUES);
>         }
>         if (isResponseComplete(facesContext, "applyRequestValues", false)
> 				|| shouldRenderResponse(facesContext, "applyRequestValues", false))
>         {
>         		// since this phase is completed we don't need to return right away even if the response is completed
>         		skipFurtherProcessing = true;
>         }
>         if (!skipFurtherProcessing && log.isTraceEnabled())
> 			log.trace("exiting applyRequestValues in "
> 					+ LifecycleImpl.class.getName());
>         return skipFurtherProcessing;
>     }
> And that is repeated as many times as phases are. The fix will be to extract the common behavior in a method, that receives one additional parameter - PhaseExecutor and delegate to it the real execution.

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