OFBiz
  1. OFBiz
  2. OFBIZ-3382

inter-app hyperlink generates incorrect url

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 14.12.01, 12.04.06, 13.07.02
    • Component/s: framework
    • Labels:
      None

      Description

      steps to reproduce:

      1, defines a hyper link as the following in component projectmgr:

      <link url-mode="inter-app" target="/mytest/control/main" text="Create my test document" style="smallSubmit">
      <parameter param-name="workEffortId"/>
      <parameter param-name="returnUrl" value="/projectmgr/control/EditTaskContents?workEffortId=$

      {workEffortId}

      "/>
      </link>

      parameter "returnUrl" is the url that external app can return back.

      2, defines a hyper link as the following in component mytest
      <section>
      <condition>
      <not>
      <if-empty field="returnUrl"/>
      </not>
      </condition>
      <widgets>
      <container>
      <link url-mode="inter-app" text="Back"
      target="$

      {returnUrl}

      " style="smallSubmit">
      </link>
      </container>
      </widgets>
      </section>

      3. click generated url from component projectmgr to jump to component mytest, parameter "returnUrl" will be encoded as string: /projectmgr/control/EditTaskContents?workEffortId=10003

      4, from component mytest, we have the incorrectly generated url http://localhost:8080/projectmgr/control/EditTaskContents?workEffortId=10003?externalLoginKey=xxxxxxxx, two "?" characters in the generated url, can not go back the task screen in projectmgr from that.

      1. OFBIZ-3382.patch
        1 kB
        Pranay Pandey
      2. OFBIZ-3382.patch
        1 kB
        Nathan Liang

        Issue Links

          Activity

          Hide
          Nathan Liang added a comment -

          Patch.

          Show
          Nathan Liang added a comment - Patch.
          Hide
          Pranay Pandey added a comment -

          Issue is reproducible with given steps. Supplying slightly updated patch. Thanks Nathan.

          Show
          Pranay Pandey added a comment - Issue is reproducible with given steps. Supplying slightly updated patch. Thanks Nathan.
          Hide
          Ashish Vijaywargiya added a comment -

          Thanks Nathan for creating the issue and Thanks Pranay for the contribution. Changes are committed in trunk at r1632803 and in RB13.07 at r1632804

          Show
          Ashish Vijaywargiya added a comment - Thanks Nathan for creating the issue and Thanks Pranay for the contribution. Changes are committed in trunk at r1632803 and in RB13.07 at r1632804
          Hide
          Jacques Le Roux added a comment -

          Backporte in R12.04 at r1633016

          Show
          Jacques Le Roux added a comment - Backporte in R12.04 at r1633016

            People

            • Assignee:
              Ashish Vijaywargiya
              Reporter:
              Nathan Liang
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development