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

Do not include empty view params using `includeViewParams=true`

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Ajax/JavaScript, Navigation
    • Labels:
      None
    • Environment:

      Glassfish 3.12

      Description

      Having a command component with an action attribute and using 'includeViewParams=true' using Navigation will include all view parameters defined in `f:metadata`.

      But empty parameters are passed as String representations creating ugly url's.

      'site.xhtml?key1=value1&key2=&key3='

        Activity

        djmj created issue -
        Hide
        djmj added a comment -

        In my Converter's I return now null instead of empty String. Since it seems null value's won't be included.

        But this leads to other issue: https://java.net/jira/browse/JAVASERVERFACES_SPEC_PUBLIC-1187

        Show
        djmj added a comment - In my Converter's I return now null instead of empty String. Since it seems null value's won't be included. But this leads to other issue: https://java.net/jira/browse/JAVASERVERFACES_SPEC_PUBLIC-1187
        Ed Burns made changes -
        Field Original Value New Value
        Assignee rogerk [ rogerk ]
        Hide
        Ed Burns added a comment -

        Set priority to baseline ahead of JSF 2.3 triage. Priorities will be assigned accurately after this exercise.

        Show
        Ed Burns added a comment - Set priority to baseline ahead of JSF 2.3 triage. Priorities will be assigned accurately after this exercise.
        Ed Burns made changes -
        Priority Major [ 3 ] Trivial [ 5 ]
        Ed Burns made changes -
        Priority Trivial [ 5 ] Minor [ 4 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            djmj
          • Votes:
            2 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: