ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacques Le Roux (JIRA)" <j...@apache.org>
Subject [jira] [Closed] (OFBIZ-9975) ShippingEvents.getShipGroupEstimate() method gives untraceable null pointer exception for BigDecimal comparison
Date Fri, 10 Nov 2017 15:36:01 GMT

     [ https://issues.apache.org/jira/browse/OFBIZ-9975?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Jacques Le Roux closed OFBIZ-9975.
----------------------------------
       Resolution: Fixed
    Fix Version/s: 16.11.04

Thanks Aditya,

Your patch is in
trunk r1814873
R16.11 r1814875


> ShippingEvents.getShipGroupEstimate() method gives untraceable null pointer exception
for BigDecimal comparison
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-9975
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9975
>             Project: OFBiz
>          Issue Type: Bug
>          Components: order
>    Affects Versions: Trunk, Release Branch 16.11
>            Reporter: Aditya Sharma
>            Assignee: Jacques Le Roux
>             Fix For: 16.11.04
>
>         Attachments: OFBIZ-9975.patch
>
>
> The compareTo() method of BigDecimal is not null safe due to which in the following code:
> {code:java}
>         if (BigDecimal.ZERO.compareTo(shippingTotal) < 0 && BigDecimal.ZERO.compareTo(totalAllowance)
< 0) {
>             BigDecimal shippingAllowancePercent = storeShipMethod.getBigDecimal("allowancePercent")
!= null ? storeShipMethod.getBigDecimal("allowancePercent") : BigDecimal.ZERO;
>             totalAllowance = totalAllowance.multiply(shippingAllowancePercent.divide(BigDecimal.valueOf(100)));
>             shippingTotal = shippingTotal.subtract(totalAllowance);
>         }
> {code}
> when totalAllowance value is null it throws null pointer exception which may be difficult
to trace sometimes.
> Valid null pointer checks should be there before using compareTo() if the variable may
have a null value in any of the case.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message