You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2016/10/01 11:06:01 UTC

svn commit: r1762999 - /ofbiz/trunk/applications/order/minilang/order/OrderServices.xml

Author: jleroux
Date: Sat Oct  1 11:06:01 2016
New Revision: 1762999

URL: http://svn.apache.org/viewvc?rev=1762999&view=rev
Log:
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/trunk/applications/order/minilang/order/OrderServices.xml

Modified: ofbiz/trunk/applications/order/minilang/order/OrderServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/minilang/order/OrderServices.xml?rev=1762999&r1=1762998&r2=1762999&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/minilang/order/OrderServices.xml (original)
+++ ofbiz/trunk/applications/order/minilang/order/OrderServices.xml Sat Oct  1 11:06:01 2016
@@ -545,7 +545,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>