Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: SVN trunk
    • Fix Version/s: SVN trunk
    • Labels:
      None
    • Environment:

      demo-trunk

      Description

      1. In OFBIZ-4822 it was decided that request to authview would be removed from the eCommerce customer ftl files because their only purpose was to set URL to https. The set field "donePage" in CustomerScreens.xml editperson effects the same result.

      2. There are brackets in the password.ftl display for the href:
      <a href="<@ofbizUrl>authview/$

      {donePage}

      </@ofbizUrl>" class="button">[$

      {uiLabelMap.CommonGoBack}

      ]</a>

      The brackets are inconsistent with the display for similar elements in the applications

        Activity

        Hide
        Tom Burns added a comment -

        The attached file may be used for testing the changes in OFBIZ-4830 eCommerce Customer Code Cleanup.patch in the Selenium IDE.
        To test :
        Load the file into the Selenium IDE.
        Login to eCommerce as DemoCustomer (using Firefox)
        Invoke the script.

        The script walks through each of the impacted screens and clicks their back buttons.
        As the script runs watch the URL. It should always display https.

        Show
        Tom Burns added a comment - The attached file may be used for testing the changes in OFBIZ-4830 eCommerce Customer Code Cleanup.patch in the Selenium IDE. To test : Load the file into the Selenium IDE. Login to eCommerce as DemoCustomer (using Firefox) Invoke the script. The script walks through each of the impacted screens and clicks their back buttons. As the script runs watch the URL. It should always display https.
        Hide
        Jacques Le Roux added a comment -

        Thanks Tom,

        Your slightly amended patch is in trunk at revision: 1328205.

        It was a real good idea to provide a selenium test suite: you forgot authview/ for the bottom buttons in editperson.ftl and changepassword.ftl

        Show
        Jacques Le Roux added a comment - Thanks Tom, Your slightly amended patch is in trunk at revision: 1328205. It was a real good idea to provide a selenium test suite: you forgot authview/ for the bottom buttons in editperson.ftl and changepassword.ftl

          People

          • Assignee:
            Jacques Le Roux
            Reporter:
            Tom Burns
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development