javaserverfaces-spec-public
  1. javaserverfaces-spec-public
  2. JAVASERVERFACES_SPEC_PUBLIC-1072

Add javax.faces.application.NavigationHandlerWrapper to JSF API

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: 2.2
    • Component/s: None
    • Labels:
      None

      Description

      This is related to Spec bug JAVASERVERFACES_SPEC_PUBLIC-1066 but I thought it was best to create a separate issue as a formal request to add a javax.faces.application.NavigationHandlerWrapper to the JSF API.

      There is precedent set for this with javax.faces.application.ResourceHandlerWrapper:

      http://javaserverfaces.java.net/nonav/docs/2.0/javadocs/javax/faces/application/ResourceHandlerWrapper.html

      NavigationHandlerWrapper would be useful in the case of a JSF Portlet Bridge, because it would give the bridge the ability to walk the decorator pattern by calling getWrapped().

        Issue Links

          Activity

          Hide
          Ed Burns added a comment -

          Because you used the word "useful" in the report, I feel justified in setting the priority to Minor. We'll probably get to it in 2.2, though.

          Show
          Ed Burns added a comment - Because you used the word "useful" in the report, I feel justified in setting the priority to Minor. We'll probably get to it in 2.2, though.
          Hide
          Neil Griffin added a comment -

          Thanks Ed. At a minimum, it would make the behavior of Application#getNavigationHandler() and Application#getResourceHandler() more consistent which is good too.

          Show
          Neil Griffin added a comment - Thanks Ed. At a minimum, it would make the behavior of Application#getNavigationHandler() and Application#getResourceHandler() more consistent which is good too.
          Hide
          Manfred Riem added a comment -

          Closing resolved issue out

          Show
          Manfred Riem added a comment - Closing resolved issue out

            People

            • Assignee:
              Ed Burns
              Reporter:
              Neil Griffin
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 30 minutes
                30m