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

includeViewParams implicit navigation flag should be faces-include-view-params

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0 Rev a
    • Component/s: Navigation
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      699
    • Status Whiteboard:
      Hide

      cat1 schema changelog

      Show
      cat1 schema changelog

      Description

      When using implicit navigation, you can pass flags through the query string that
      alter the behavior of the navigation. The first is a redirect flag, the second
      is a hint to include view parameters in the outgoing query string.

      • faces-redirect
      • includeViewParams

      As you can see, there are two problems with the includeViewParams flag. First,
      it uses camel case instead of hyphen delimiters between words and it is not
      prefixed with faces-.

      When this feature was being discussed, there was a concern that the hints would
      collide with query string parameters used by the application. As a result, a
      prefix was added to the redirect hint, making it faces-redirect. However,
      includeViewParams was not updated.

      Here's the following change that needs to happen in the spec:

      includeViewParams ==> faces-include-view-params

      And in use:

      /entry?faces-redirect=true&faces-include-view-params=true

      In the long run, we need to find a cleaner (perhaps type-safe) way of indicating
      an implicit navigation. Using the query string really isn't great. Perhaps under
      the covers the implicit navigation is communicated this way, but we provide a
      Java API that generates this in toString()

      return new Navigation("entry", true, true);

        Activity

        Hide
        mojavelinux added a comment -

        P1 and milestone 2.0

        Show
        mojavelinux added a comment - P1 and milestone 2.0
        Hide
        mojavelinux added a comment -

        Update milestone to 2.1 (really 2.0 MR1)

        Show
        mojavelinux added a comment - Update milestone to 2.1 (really 2.0 MR1)
        Hide
        mojavelinux added a comment -

        Update target milestone to 2.0 Rev a

        Show
        mojavelinux added a comment - Update target milestone to 2.0 Rev a
        Hide
        Ed Burns added a comment -

        cat1

        Show
        Ed Burns added a comment - cat1
        Hide
        Ed Burns added a comment -

        schema

        Show
        Ed Burns added a comment - schema
        Hide
        Ed Burns added a comment -

        Fix checked in.

        Show
        Ed Burns added a comment - Fix checked in.
        Hide
        Ed Burns added a comment -

        These are valid 2.0 Rev a issues

        Show
        Ed Burns added a comment - These are valid 2.0 Rev a issues
        Hide
        rogerk added a comment -

        changelog

        Show
        rogerk added a comment - changelog
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            javaserverfowner
            Reporter:
            mojavelinux
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: