MyFaces Core
  1. MyFaces Core
  2. MYFACES-2692

Hidden field "javax.faces.encodedURL" should be rendered at begin of h:form

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1
    • Component/s: JSR-314
    • Labels:
      None

      Description

      Checking some stuff related to how to target scripts to <form>, the documentation of javax.faces.Form renderer says this:

      "....Obtain the UIViewRoot view identifier and use it to obtain an action URL by calling ViewHandler.getActionURL. Use the action URL to obtain an encoded action URL by calling ExternalContext.encodeActionURL. Obtain an encoded partial action URL by calling ExternalContext.encodePartialActionURL using action URL as the argument. Compare the result from ExternalContext.encodePartialActionURL with the value from ExternalContext.encodeActionURL(). If they are different, render a hidden field with the name javax.faces.encodedURL and the value of this hidden field as the value from ExternalContext.encodePartialActionURL...."

        Activity

        Hide
        Jakob Korherr added a comment -

        The commit for this issue causes all tests on HtmlFormRendererTest to fail, because encodePartialActionURL() is not implemented on MockExternalContext20.

        I will open an issue on myfaces-test for this and fix it asap. However we have to wait for the next (working) build of myfaces-test until this will work. Until then I will comment-out the three related tests.

        Show
        Jakob Korherr added a comment - The commit for this issue causes all tests on HtmlFormRendererTest to fail, because encodePartialActionURL() is not implemented on MockExternalContext20. I will open an issue on myfaces-test for this and fix it asap. However we have to wait for the next (working) build of myfaces-test until this will work. Until then I will comment-out the three related tests.
        Hide
        Jakob Korherr added a comment -

        The related myfaces-test issue is MYFACESTEST-9.

        Show
        Jakob Korherr added a comment - The related myfaces-test issue is MYFACESTEST-9 .

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Leonardo Uribe
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development