svn commit: r1328948 - in /ofbiz/branches/release10.04: ./ applications/accounting/data/DemoFinAccountData.xml

Author: jleroux
Date: Sun Apr 22 19:38:37 2012
New Revision: 1328948

URL: http://svn.apache.org/viewvc?rev=1328948&view=rev
Log:
"Applied fix from trunk for revision: 1328946  "
------------------------------------------------------------------------
r1328946 | jleroux | 2012-04-22 21:37:06 +0200 (dim., 22 avr. 2012) | 31 lines

A patch from Tom Burns for "Update Company Contact Information Fails" https://issues.apache.org/jira/browse/OFBIZ-4804

Updating the Company postal_address in Party Manager fails.
To reproduce:
In Party Manager lookup Party Id "Company"
Click Update in Contact Information > Postal Address
Make a change to the address
Click Save
Note the error message.

A postal_address update triggers an accounting SECA that runs:
PaymentMethodServices.xml (accounting) updatePaymentMethodAddress
services_paymentmethod.xml updateEftAccount
PaymentMethodServices.java updateEftAccount

The purpose of the SECA is to update the contact_mech_id field in CreditCard or EftAccount entities have the optional billing address information.

The updateEftAccount service sets the nameOnAccount field to optional="false".
The demo data that set up two EftAccount's for Company did not include nameOnAccount.
These are the only ETF's set up in demo data.

In the application ETF's are set up in eCommerce which requires "Name On Account" so the problem should not happen with address changes.
See eCommerce:
Login as DemoCustomer
Click "Profile" at the right of the top title bar
Click "Create New EFT Account" at the right of the title bar in the Payment Method Information section.
Note the first file "Name on Account" is required
Note the option to select a billing address.

Changing the DemoFinAccountData.xml file to enter the nameOnAccount field for the Company EftAccount's fixes the problem.

------------------------------------------------------------------------


Modified:
    ofbiz/branches/release10.04/   (props changed)
    ofbiz/branches/release10.04/applications/accounting/data/DemoFinAccountData.xml

Propchange: ofbiz/branches/release10.04/
------------------------------------------------------------------------------
  Merged /ofbiz/trunk:r1328946

Modified: ofbiz/branches/release10.04/applications/accounting/data/DemoFinAccountData.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release10.04/applications/accounting/data/DemoFinAccountData.xml?rev=1328948&r1=1328947&r2=1328948&view=diff
==============================================================================
--- ofbiz/branches/release10.04/applications/accounting/data/DemoFinAccountData.xml (original)
+++ ofbiz/branches/release10.04/applications/accounting/data/DemoFinAccountData.xml Sun Apr 22 19:38:37 2012
@@ -23,12 +23,12 @@ under the License.
             currencyUomId="USD" organizationPartyId="Company" ownerPartyId="Company" postToGlAccountId="111100" isRefundable="Y" replenishLevel="0.00" actualBalance="0.00" availableBalance="0.00"/>
     <FinAccountStatus finAccountId="SC_CHECKING" statusId="FNACT_ACTIVE" statusDate="2009-04-01 00:00:01.000"/>
     <PaymentMethod paymentMethodId="SC_CHECKING" paymentMethodTypeId="COMPANY_CHECK"  partyId="Company" description="Checking Account at Standard Chartered Bank" glAccountId="111100" finAccountId="SC_CHECKING" fromDate="2009-04-01 00:00:00.000"/>
-    <EftAccount paymentMethodId="SC_CHECKING" bankName="Standard Chartered Bank" routingNumber="307075259" accountType="Checking" accountNumber="316253011064226" contactMechId="9000" />
+    <EftAccount paymentMethodId="SC_CHECKING" nameOnAccount="Company" bankName="Standard Chartered Bank" routingNumber="307075259" accountType="Checking" accountNumber="316253011064226" contactMechId="9000" />
     <FinAccount finAccountId="ABN_CHECKING" finAccountTypeId="BANK_ACCOUNT" statusId="FNACT_ACTIVE" finAccountName="ABN AMRO" finAccountCode="6577852521447" finAccountPin="1234" currencyUomId="USD"
             organizationPartyId="Company" ownerPartyId="Company" postToGlAccountId="111100" isRefundable="Y" replenishLevel="0.00" actualBalance="0.00" availableBalance="0.00"/>
     <FinAccountStatus finAccountId="ABN_CHECKING" statusId="FNACT_ACTIVE" statusDate="2009-04-01 00:00:01.000"/>
     <PaymentMethod paymentMethodId="ABN_CHECKING" paymentMethodTypeId="COMPANY_CHECK"  partyId="Company" description="Checking Account at ABN AMRO Bank" glAccountId="111100" finAccountId="ABN_CHECKING" fromDate="2009-04-01 00:00:00.000"/>
-    <EftAccount paymentMethodId="ABN_CHECKING" bankName="ABN AMRO Bank" routingNumber="222371863" accountType="Checking" accountNumber="427253011064227" contactMechId="9000"/>
+    <EftAccount paymentMethodId="ABN_CHECKING" nameOnAccount="Company" bankName="ABN AMRO Bank" routingNumber="222371863" accountType="Checking" accountNumber="427253011064227" contactMechId="9000"/>
     <FinAccount finAccountId="AMEX_01" finAccountTypeId="CREDIT_CARD_ACCOUNT" statusId="FNACT_ACTIVE" finAccountName="Corporate AMEX " finAccountCode="378282246310005" finAccountPin="9876" currencyUomId="USD"
             organizationPartyId="Company" ownerPartyId="Company" postToGlAccountId="231000" isRefundable="Y" replenishLevel="0.00" actualBalance="0.00" availableBalance="0.00"/>
     <FinAccountStatus finAccountId="AMEX_01" statusId="FNACT_ACTIVE" statusDate="2009-04-01 00:00:01.000"/>