Uploaded image for project: 'Portlet Bridge'
  1. Portlet Bridge
  2. PBR-321

The fix of PBR-276 steal the messages from ajax-resource-requests

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 2.3.0.FINAL
    • Fix Version/s: 2.3.1.Final
    • Component/s: Seam
    • Labels:
      None

      Description

      the fix was wrong. it steals the messages for ajax-portlet-calls.
      for example:

      • invoke ajax-portlet. it will pass all jsf-phases 1..6.
        your fix will clear the messages from seam StatusMessages.
        SeamStrategy.java
        	public void afterResourceRequestExecute(FacesContext facesContext) {
        		performPageActions(facesContext);
        		seamListenerWrapper.saveFacesMessages(facesContext);//HERE THE CLEAR WILL BE DONE
        	    super.afterResourceRequestExecute(facesContext);
        	}
        
      AjaxPortletBridge.java
      	public void doFacesRequest(ResourceRequest request,
      			ResourceResponse response) throws BridgeUninitializedException {
      		assertParameters(request, response);
      ...
      					strategy.beforeResourceRequest(facesContext);
      					execute(facesContext);
      					// TODO - create applicable processing function.
      					// processOutgoingParameters(facesContext,request,response);
      //WITH THE FIX THE Seam StatusMessages was cleared
      					strategy.afterResourceRequestExecute(facesContext);
      					if (!facesContext.getResponseComplete()) {
      //BUT HERE WE RENDER THE VIEW AND WE WANT TO SHOW ERROR MESSAGES....BUT the List was cleared.
      						render(facesContext);
      						if (saveWindowStateAfterAction) {
      							windowState.saveRequest(request,facesContext, false);
      						}
      					}
      

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  kenfinni Ken Finnigan
                  Reporter:
                  nias Markus Schulz
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: