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

Current wording in renderkit docs leads to double encoding of query parameters

    Details

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

      Operating System: All
      Platform: All

    • Issuezilla Id:
      436
    • Status Whiteboard:
      Hide

      cat1 renderkitdocs changelog

      Show
      cat1 renderkitdocs changelog

      Description

      Hi

      Looking on the jsf tld documentation (1.1 and 1.2), it says something like this
      (for h:commandLink):

      "............The name and the value (of UIParameter child components) must be
      URLEncoded.......".

      It says the same as h:outputLink.

      Testing on jsf ri 1.2_09 for example a <f:param name="paramName" value="modalità">

      Is added on commandLink javascript as:

      if(typeof jsfcljs ==
      'function'){jsfcljs(document.getElementById('mainForm'),

      {'mainForm:link':'mainForm:link','paramName':'modalità'}

      ,'');}return
      false

      This behavior suggest that no URLEncoding is used for f:param inside
      h:commandLink (instead the encoding used by ResponseWriter.writeAttribute). Is
      this the correct behavior? The tld api doc is wrong?

      ----------------------------------

      Speaking with Ed over the phone about this we came to the agreement that
      the content pertaining to URLEncoding isn't necessary since writeAttribute
      should do that for us.

        Activity

        Hide
        Ed Burns added a comment -

        EL

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

        Fixed

        Show
        Ed Burns added a comment - Fixed
        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:
            Ryan Lubke
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: