OFBiz
  1. OFBiz
  2. OFBIZ-2618

double forward slashes in ecommerce emails

    Details

      Description

      in the widets
      <set field="baseEcommerceSecureUrl"
      value="$

      {baseSecureUrl}

      /ecommerce/control/"/>
      has a trailing forward slash
      then the added url has a leading forward slash
      /orderstatus?orderId=
      so you get
      /ecommerce/control//orderstatus?orderId=

      removed the trailing forward slash

        Activity

        Hide
        BJ Freeman added a comment -

        patch

        Show
        BJ Freeman added a comment - patch
        Hide
        Jacques Le Roux added a comment -

        HI Bj,

        I can't commit your patch as is. At least 2 reasons

        1. Please don't add the following lines in the header
          ### Eclipse Workspace Patch 1.0
          #P ofbiz_09_04
          

          For this in the create patch wizard choose your project and not the workspace

        2. Don't cut your lines since this generate conflicts when paching (in Eclipse at least), example
          +                <property-map resource="EcommerceUiLabels" map-name="uiLabelMap"
          +                    global="true"/>
          

          should be

          +                <property-map resource="EcommerceUiLabels" map-name="uiLabelMap" global="true"/>
          

        Thanks

        Show
        Jacques Le Roux added a comment - HI Bj, I can't commit your patch as is. At least 2 reasons Please don't add the following lines in the header ### Eclipse Workspace Patch 1.0 #P ofbiz_09_04 For this in the create patch wizard choose your project and not the workspace Don't cut your lines since this generate conflicts when paching (in Eclipse at least), example + <property-map resource= "EcommerceUiLabels" map-name= "uiLabelMap" + global= " true " /> should be + <property-map resource= "EcommerceUiLabels" map-name= "uiLabelMap" global= " true " /> Thanks

          People

          • Assignee:
            Unassigned
            Reporter:
            BJ Freeman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development