Re: svn commit: r1464779 - in /ofbiz/trunk: applications/accounting/script/org/ofbiz/accounting/fixedasset/ applications/accounting/script/org/ofbiz/accounting/invoice/ applications/accounting/script/org/ofbiz/accounting/ledger/ applications/accounting/scr...

[Adrian Crum]

We will see if anyone reports a problem. I ran unit tests after the 
replacement (the unit tests used the bsh: scriptlets a lot) and there 
were no errors reported.

-Adrian
On 4/5/2013 7:59 AM, Nicolas Malin wrote:
Hi adrian,

There is a risk to change " by ' when we change bsh by groovy 
engine on flexible string ?
When I convert old customer specific code as you did, I replace also 
the string " .

productMaintValue.getBigDecimal('intervalQuantity') is really more 
readable than 
productMaintValue.getBigDecimal("intervalQuantity")

Just to be sure that I didn't to take a wrong way.

Cheers,
Nicolas

Le 05/04/2013 00:01, adri...@apache.org a écrit :
-            <set field="nextIntervalQty" 
value="${bsh:maxIntervalQty.add(productMaintValue.getBigDecimal(&quot;intervalQuantity&quot;));}" 
type="BigDecimal"/>
+            <set field="nextIntervalQty" 
value="${groovy:maxIntervalQty.add(productMaintValue.getBigDecimal(&quot;intervalQuantity&quot;));}" 
type="BigDecimal"/>