Actually, the <set> element is unnecessary. Mini-language will create a
new Map automatically.
Adrian Crum
Sandglass Software
www.sandglass-software.com
On 9/24/2015 1:28 PM, [email protected] wrote:
Author: jleroux
Date: Thu Sep 24 20:28:21 2015
New Revision: 1705136
URL: http://svn.apache.org/viewvc?rev=1705136&view=rev
Log:
No functional change, while reviewing new OFBIZ-293 proposition I stumbled upon this. The
value="null" is here unnecessary and confusing
Modified:
ofbiz/trunk/specialpurpose/scrum/script/org/ofbiz/scrum/ScrumEvents.xml
Modified:
ofbiz/trunk/specialpurpose/scrum/script/org/ofbiz/scrum/ScrumEvents.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/scrum/script/org/ofbiz/scrum/ScrumEvents.xml?rev=1705136&r1=1705135&r2=1705136&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/scrum/script/org/ofbiz/scrum/ScrumEvents.xml
(original)
+++ ofbiz/trunk/specialpurpose/scrum/script/org/ofbiz/scrum/ScrumEvents.xml Thu
Sep 24 20:28:21 2015
@@ -655,7 +655,7 @@
<set field="custRequestIndexList" type="List"/>
<set field="custRequestList" type="List"/>
<loop count="${rowCount}" field="j">
- <set field="custRequestMap" value="null" type="NewMap"/>
+ <set field="custRequestMap" type="NewMap"/>
<set field="custRequestMap.custRequestId"
from-field="paramMap["custRequestId_o_${j}"]"/>
<set field="custRequestMap.custSequenceNum"
from-field="paramMap["realSequenceNum_o_${j}"]"/>
<field-to-list list="custRequestList"
field="custRequestMap"/>
@@ -664,7 +664,7 @@
<set field="newCustRequestList" type="List"/>
<set field="custRequestListWithNewSeq" type="List"/>
<loop count="${rowCount}" field="k">
- <set field="custRequestMap" value="null" type="NewMap"/>
+ <set field="custRequestMap" type="NewMap"/>
<set field="custRequestMap.custRequestId"
from-field="paramMap["custRequestId_o_${k}"]"/>
<set field="oldSequenceNum"
from-field="paramMap["realSequenceNum_o_${k}"]"/>
<set field="newSequenceNum"
from-field="paramMap["newSequenceNum_o_${k}"]"/>