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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        2381d 17h 22m 1 Arun Patidar 20/Sep/14 11:44
        Patch Available Patch Available In Progress In Progress
        11m 17s 1 Anil K Patel 20/Sep/14 11:55
        In Progress In Progress Closed Closed
        6m 2s 1 Ashish Vijaywargiya 20/Sep/14 12:01
        Jacopo Cappellato made changes -
        Fix Version/s Upcoming Branch [ 12327361 ]
        Ashish Vijaywargiya made changes -
        Status In Progress [ 3 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        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
        Ashish Vijaywargiya made changes -
        Assignee Anil K Patel [ anilpatel ] Ashish Vijaywargiya [ toashishvijay ]
        Anil K Patel made changes -
        Status Patch Available [ 10002 ] In Progress [ 3 ]
        Arun Patidar made changes -
        Attachment OFBIZ-1705.patch [ 12670205 ]
        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.
        Arun Patidar made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Jacopo Cappellato made changes -
        Fix Version/s Trunk [ 12311928 ]
        Gavin made changes -
        Workflow jira [ 12426016 ] OFbiz Workflow [ 12505751 ]
        Anil K Patel made changes -
        Assignee Anil K Patel [ anilpatel ]
        Hide
        Jacques Le Roux added a comment -

        Nobody interested in this ?

        Show
        Jacques Le Roux added a comment - Nobody interested in this ?
        Brajesh Patel made changes -
        Attachment Marg_simple_method.patch [ 12377813 ]
        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.
        Brajesh Patel made changes -
        Field Original Value New Value
        Description Merge code of processCustoemrSettings and createUpdateUser method as 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.
        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.
        Brajesh Patel created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development