[jira] [Commented] (OFBIZ-11862) BigDecimal casting in Groovy

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-11862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17148667#comment-17148667 ]

ASF subversion and git services commented on OFBIZ-11862:
---------------------------------------------------------

Commit df179777b68f8d9411b84b3e11d38e0bda5c0bb4 in ofbiz-framework's branch refs/heads/trunk from Jacques Le Roux
[ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=df17977 ]

Fixed: BigDecimal casting in Groovy (OFBIZ-11862)

Sometimes use wrong type castings for BigDecimal:

I have observed that for typecasting in groovy, we have used the wrong coding
pattern like

(BigDecimal) 0
It should be like
0 as BigDecimal

Note that this does not apply to decimal because in case of decimal, the default
type is always BigDecimal in Groovy.

Thanks: Pritam Kute for spotting the issue in OFBIZ-11843

> BigDecimal casting in Groovy
> ----------------------------
>
>                 Key: OFBIZ-11862
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11862
>             Project: OFBiz
>          Issue Type: Bug
>          Components: bi, order, product
>    Affects Versions: Trunk
>            Reporter: Jacques Le Roux
>            Priority: Major
>
> [Pritam Kute noticed|https://issues.apache.org/jira/browse/OFBIZ-11843?focusedCommentId=17146911&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17146911] in OFBIZ-11843 that we sometimes use wrong type castings for BigDecimal:
> {quote}
> I have observed that for typecasting in groovy, we have used the wrong coding pattern like
> {code:java}
> (BigDecimal) -0.01
> {code}
> It should be like
> {code:java}
> -0.01 as BigDecimal
> {code}
> {quote}
> Actually in this case we don't need to cast, [because in case of decimal, the default type is always BigDecimal|https://www.logicbig.com/tutorials/misc/groovy/numbers-types-suffix.html].
> But there are Integers in trunk (mostly 0) and 2 cases where it's useless to cast in R18, nothing in R17

--
This message was sent by Atlassian Jira
(v8.3.4#803005)