svn commit: r934389 - /ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r934389 - /ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml

ashish-18
Author: ashish
Date: Thu Apr 15 12:38:15 2010
New Revision: 934389

URL: http://svn.apache.org/viewvc?rev=934389&view=rev
Log:
Applied patch from jira issue OFBIZ-3703 - updatePartyContactMech service creates new record while contactMechId is unchange.
Thanks Arun for the contribution & also thanks to Scott & Pranay for providing your comments after review.

Modified:
    ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml

Modified: ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml?rev=934389&r1=934388&r2=934389&view=diff
==============================================================================
--- ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml (original)
+++ ofbiz/trunk/applications/party/script/org/ofbiz/party/contact/PartyContactMechServices.xml Thu Apr 15 12:38:15 2010
@@ -107,10 +107,9 @@ under the License.
         <now-timestamp field="newPartyContactMech.fromDate"/>
         <now-timestamp field="partyContactMech.thruDate"/>
 
-        <store-value value-field="partyContactMech"/>
-        <create-value value-field="newPartyContactMech"/>
-
         <if-compare-field to-field="newPartyContactMech.contactMechId" field="parameters.contactMechId" operator="not-equals">
+            <store-value value-field="partyContactMech"/>
+            <create-value value-field="newPartyContactMech"/>
             <get-related value-field="partyContactMech" relation-name="PartyContactMechPurpose" list="partyContactMechPurposes"/>
             <filter-list-by-date list="partyContactMechPurposes"/>
             <iterate entry="partyContactMechPurposeOld" list="partyContactMechPurposes">