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"/>
