You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Jyotsna Rathore (JIRA)" <ji...@apache.org> on 2009/06/19 11:11:07 UTC

[jira] Updated: (OFBIZ-512) Unused createCustomer simple methods in CustomerEvents.xml should be removed.

     [ https://issues.apache.org/jira/browse/OFBIZ-512?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jyotsna Rathore updated OFBIZ-512:
----------------------------------

    Attachment: createCustomer.patch

Updated Order::custSettings.ftl and Party::newcustomer.ftl to use UserEvents.createCustomer event. 
Also removed Order:: CustomerEvents.createCustomer, Ecommerce:: CustomerEvents.createCustomer and Party:: CustomerEvent.xml.

> Unused  createCustomer simple methods in CustomerEvents.xml should be removed.
> ------------------------------------------------------------------------------
>
>                 Key: OFBIZ-512
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-512
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: party, specialpurpose/ecommerce
>            Reporter: Anil K Patel
>            Priority: Minor
>         Attachments: createCustomer.patch, createCustomer.patch
>
>
> While working on refactoring of CheckOut process, I came across too many createCustomer simple methods.
> Ecommerce component --> CustomerEvents.xml file has simple method with the name createCustomer.
> Party component --> CustomerEvents.xml file has simple method with the name createCustomer, This is not called. Most of the code in this method is from createUser simple method in Party component.
> New Customer screen in Party uses NewUser form in PartyForms.xml. This Form uses the Field Names that will work with createCustomer simple method. But this form Posts to createCustomer simple method defined in UserEvents.xml file, this simple method is real simple and uses createUser simple method. This makes it clear that Party::CustomerEvents.createCustomer should be removed.
> Order::CustomerEvents.createCustomer method is made for custom "custsettings.ftl" file. I think "custsettings.ftl" file can be modified to use Party::UserEvents.createCustomer.  "newcustomer.ftl" file in ecommerce follows naming of fields that works with createCustomer.  In such situation we can remove Order::CustomerEvents.createCustomer method.
> We can also remove Ecommerce::CustomerEvents.creatCustomer by pointing request to Party::UserEvents.createCustomer.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.