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
        Nathan Liang
      2. OFBIZ-3382.patch
        1 kB
        Pranay Pandey

        Issue Links

          Activity

          Nathan Liang created issue -
          Hide
          Nathan Liang added a comment -

          Patch.

          Show
          Nathan Liang added a comment - Patch.
          Nathan Liang made changes -
          Field Original Value New Value
          Attachment OFBIZ-3382.patch [ 12428918 ]
          Gavin made changes -
          Workflow jira [ 12485524 ] OFbiz Workflow [ 12507152 ]
          Jacopo Cappellato made changes -
          Fix Version/s Trunk [ 12311928 ]
          Ashish Vijaywargiya made changes -
          Assignee Ashish Vijaywargiya [ toashishvijay ]
          Ashish Vijaywargiya made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          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.
          Pranay Pandey made changes -
          Attachment OFBIZ-3382.patch [ 12675680 ]
          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
          Ashish Vijaywargiya made changes -
          Status In Progress [ 3 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Jacques Le Roux made changes -
          Fix Version/s Upcoming Branch [ 12327361 ]
          Fix Version/s 13.07.02 [ 12328788 ]
          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
          Jacques Le Roux made changes -
          Fix Version/s 12.04.06 [ 12328243 ]
          Gareth Carter made changes -
          Link This issue breaks OFBIZ-5910 [ OFBIZ-5910 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open In Progress In Progress
          1758d 22h 51m 1 Ashish Vijaywargiya 18/Oct/14 15:54
          In Progress In Progress Closed Closed
          10m 10s 1 Ashish Vijaywargiya 18/Oct/14 16:05

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development