svn commit: r1763005 - in /ofbiz/branches: release13.07/applications/order/script/org/ofbiz/order/order/ release14.12/applications/order/script/org/ofbiz/order/order/ release15.12/applications/order/script/org/ofbiz/order/order/

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

svn commit: r1763005 - in /ofbiz/branches: release13.07/applications/order/script/org/ofbiz/order/order/ release14.12/applications/order/script/org/ofbiz/order/order/ release15.12/applications/order/script/org/ofbiz/order/order/

jleroux@apache.org
Author: jleroux
Date: Sat Oct  1 12:23:52 2016
New Revision: 1763005

URL: http://svn.apache.org/viewvc?rev=1763005&view=rev
Log:
"Applied fix from trunk for revision: 1762999  " (conflict handled by hand)
------------------------------------------------------------------------
r1762999 | jleroux | 2016-10-01 13:06:01 +0200 (sam. 01 oct. 2016) | 10 lignes

Fixes: When we pass only "orderId" in "updateOrderHeader" service.
All values are getting null.
(OFBIZ-8339)

If we pass only orderId then rest value should not set empty.

jleroux: I must add it's weird to pass only orderId when updating an Order
Header, but in theory the fix is right, "Errare humanum est"

Thanks: Renuka Srishti
------------------------------------------------------------------------

Modified:
    ofbiz/branches/release13.07/applications/order/script/org/ofbiz/order/order/OrderServices.xml
    ofbiz/branches/release14.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml
    ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml

Modified: ofbiz/branches/release13.07/applications/order/script/org/ofbiz/order/order/OrderServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1763005&r1=1763004&r2=1763005&view=diff
==============================================================================
--- ofbiz/branches/release13.07/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
+++ ofbiz/branches/release13.07/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sat Oct  1 12:23:52 2016
@@ -594,7 +594,7 @@ under the License.
             </add-error>
         </if-empty>
         <check-errors/>
-        <set-nonpk-fields value-field="orderHeader" map="parameters"/>
+        <set-nonpk-fields value-field="orderHeader" map="parameters" set-if-null="false"/>
         <store-value value-field="orderHeader"/>
     </simple-method>
 

Modified: ofbiz/branches/release14.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release14.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1763005&r1=1763004&r2=1763005&view=diff
==============================================================================
--- ofbiz/branches/release14.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
+++ ofbiz/branches/release14.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sat Oct  1 12:23:52 2016
@@ -593,7 +593,7 @@ under the License.
             </add-error>
         </if-empty>
         <check-errors/>
-        <set-nonpk-fields value-field="orderHeader" map="parameters"/>
+        <set-nonpk-fields value-field="orderHeader" map="parameters" set-if-null="false"/>
         <store-value value-field="orderHeader"/>
     </simple-method>
 

Modified: ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1763005&r1=1763004&r2=1763005&view=diff
==============================================================================
--- ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
+++ ofbiz/branches/release15.12/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sat Oct  1 12:23:52 2016
@@ -593,7 +593,7 @@ under the License.
             </add-error>
         </if-empty>
         <check-errors/>
-        <set-nonpk-fields value-field="orderHeader" map="parameters"/>
+        <set-nonpk-fields value-field="orderHeader" map="parameters" set-if-null="false"/>
         <store-value value-field="orderHeader"/>
     </simple-method>