OFBiz
  1. OFBiz
  2. OFBIZ-1705

Merging of processCustomerSettings and createUpdateUser method

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 14.12.01
    • Labels:
      None

      Description

      Merge code of "processCustoemrSettings" and "createUpdateUser" method in new method "createUpdateCustomer " in org/ofbiz/ecommerce/customer/QuickAnonCustomerEvents.xml.
      While we do this make sure that the removal of these two methods is not breaking anything.

      1. Marg_simple_method.patch
        18 kB
        Brajesh Patel
      2. OFBIZ-1705.patch
        4 kB
        Arun Patidar

        Activity

        Hide
        Brajesh Patel added a comment -

        I have done work of merging two simple methods("processCustoemrSettings" and "createUpdateUser") in new simple method "createUpdateCustomer" in QuickAnonCustomerEvents.xml. While making new method, i have checked every thing related to these methods and done well testing.

        Show
        Brajesh Patel added a comment - I have done work of merging two simple methods("processCustoemrSettings" and "createUpdateUser") in new simple method "createUpdateCustomer" in QuickAnonCustomerEvents.xml. While making new method, i have checked every thing related to these methods and done well testing.
        Hide
        Jacques Le Roux added a comment -

        Nobody interested in this ?

        Show
        Jacques Le Roux added a comment - Nobody interested in this ?
        Hide
        Arun Patidar added a comment -

        Attached is the patch for the same.

        Steps to test:
        1. Go to ecommerce front store and add items into cart.
        2. Click on "Quick Checkout" button.
        4. In this screen under "Checkout Without Login" section click on "Quick Checkout" button.
        5. Fill the personal, billing and shipping address related information and click on continue. (This is place where createUpdateCustomer method is used.)
        6. Select the shipping options and place the order.

        Expected Result: Order should get successfully placed with anonymous user.

        Show
        Arun Patidar added a comment - Attached is the patch for the same. Steps to test: 1. Go to ecommerce front store and add items into cart. 2. Click on "Quick Checkout" button. 4. In this screen under "Checkout Without Login" section click on "Quick Checkout" button. 5. Fill the personal, billing and shipping address related information and click on continue. (This is place where createUpdateCustomer method is used.) 6. Select the shipping options and place the order. Expected Result: Order should get successfully placed with anonymous user.
        Hide
        Ashish Vijaywargiya added a comment -

        Thanks Brajesh for reporting the issue and Thanks Arun for providing the patch. Changes are committed in trunk at r1626417.


        Ashish

        Show
        Ashish Vijaywargiya added a comment - Thanks Brajesh for reporting the issue and Thanks Arun for providing the patch. Changes are committed in trunk at r1626417. – Ashish

          People

          • Assignee:
            Ashish Vijaywargiya
            Reporter:
            Brajesh Patel
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development