Oh! Thanks for the clarification.

-Adrian

On 4/5/2013 8:45 AM, Jacopo Cappellato wrote:
Adrian,

I think that Nicolas was proposing to replace " with ' to increase 
readability: this was not possible with Beanshell but it is now possible after we 
switch to groovy.

Jacopo



On Apr 5, 2013, at 9:26 AM, Adrian Crum <[email protected]> 
wrote:

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 &quot; 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 
&quot; .

productMaintValue.getBigDecimal('intervalQuantity') is really more readable than 
productMaintValue.getBigDecimal(&quot;intervalQuantity&quot;)

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

Cheers,
Nicolas

Le 05/04/2013 00:01, [email protected] 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"/>


Reply via email to