Uploaded image for project: 'MyFaces Portlet Bridge'
  1. MyFaces Portlet Bridge
  2. PORTLETBRIDGE-201

Proposal for 3.0 IMPL: Preserve and restore Flash scope within BridgeRequestScope

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 3.0.0
    • Fix Version/s: None
    • Component/s: General
    • Labels:
      None

      Description

      In order to make the JSF2 Flash scope work with navigation-rules that take place within the Portlet 2 ACTION_PHASE, it is necessary to preserve the Flash scope in the BridgeRequestScope so that it can survive into the RENDER_PHASE.

      This proposal would require the BridgeRequestScope to remember the PortletMode so that mode changes can be detected:

      public interface BridgeRequestScope

      { ... Flash getFlash(); void setFlash(Flash flash); ... }

      The implementation of Bridge.doFacesRequest(ActionRequest actionRequest, ActionResponse actionResponse) would be required to call bridgeRequestScope.setFlash(facesContext.getExternalContext().getFlash());

      The implementation of Bridge.doFacesRequest(EventRequest eventRequest, EventResponse eventResponse) would be required to call bridgeRequestScope.setFlash(facesContext.getExternalContext().getFlash());

      The implementation of Bridge.doFacesRequest(RenderRequest renderRequest, RenderResponse renderResponse) would be required to call externalContext.setFlash(bridgeRequestScope.getFlash());

        Issue Links

          Activity

          Hide
          ngriffin7a Neil Griffin added a comment -

          Closing as "Won't Fix" since FacesBridge work is now taking place with JSR 378. However, this issue will be addressed in FACES-3034.

          Show
          ngriffin7a Neil Griffin added a comment - Closing as "Won't Fix" since FacesBridge work is now taking place with JSR 378. However, this issue will be addressed in FACES-3034 .

            People

            • Assignee:
              mike_freedman Michael Freedman
              Reporter:
              ngriffin7a Neil Griffin
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development