[
https://issues.apache.org/jira/browse/OFBIZ-880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12489067
]
Jacopo Cappellato commented on OFBIZ-880:
-----------------------------------------
Scott,
all your questions are welcome.
I think that the long term goal is to replace all the doubles with BigDecimals,
there are simply too many issues with double arithmetic in Java.
For example, with the following code:
double number = 132.23;
double newNumber = number * 100;
System.out.println("Number: " + number);
System.out.println("Number * 100: " + newNumber);
we get the following output:
Number: 132.23
Number * 100: 13222.999999999998
a bit scary....
However, an incremental approach (small commits etc...) it is the best way to
go.
> Replace calls to deprecated methods in OrderServices.java to avoid warnings
> during compilation.
> -----------------------------------------------------------------------------------------------
>
> Key: OFBIZ-880
> URL: https://issues.apache.org/jira/browse/OFBIZ-880
> Project: OFBiz (The Open for Business Project)
> Issue Type: Improvement
> Components: order
> Reporter: Jacopo Cappellato
> Assigned To: Scott Gray
> Priority: Minor
>
> Replace calls to deprecated methods in OrderServices.java to avoid warnings
> during compilation.
> It would be great to have this fixed before the upcoming release
> (volunteers?).
> Here is the list of the warnings:
> 13 warnings
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1120:
> warning: [deprecation] getOrderGrandTotal() in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> double updatedTotal = orh.getOrderGrandTotal();
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1123:
> warning: [deprecation] getOrderReturnedTotal() in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> double remainingSubTotal = updatedTotal - orh.getOrderReturnedTotal() -
> orh.getOrderNonReturnedTaxAndShipping();
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1123:
> warning: [deprecation] getOrderNonReturnedTaxAndShipping() in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> double remainingSubTotal = updatedTotal - orh.getOrderReturnedTotal() -
> orh.getOrderNonReturnedTaxAndShipping();
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1278:
> warning: [deprecation] getOrderItemsSubTotal(java.util.List,java.util.List)
> in org.ofbiz.order.order.OrderReadHelper has been deprecated
> double orderSubTotal =
> OrderReadHelper.getOrderItemsSubTotal(validOrderItems, allAdjustments);
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1281:
> warning: [deprecation]
> calcOrderAdjustments(java.util.List,double,boolean,boolean,boolean) in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> BigDecimal orderShipping = new
> BigDecimal(OrderReadHelper.calcOrderAdjustments(orderHeaderAdjustments,
> orderSubTotal, false, false, true));
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1289:
> warning: [deprecation]
> getOrderItemSubTotal(org.ofbiz.entity.GenericValue,java.util.List,boolean,boolean)
> in org.ofbiz.order.order.OrderReadHelper has been deprecated
> amounts.add(i, new
> BigDecimal(OrderReadHelper.getOrderItemSubTotal(orderItem, allAdjustments,
> true, false))); // get the item amount
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1290:
> warning: [deprecation]
> getOrderItemAdjustmentsTotal(org.ofbiz.entity.GenericValue,java.util.List,boolean,boolean,boolean)
> in org.ofbiz.order.order.OrderReadHelper has been deprecated
> shipAmts.add(i, new
> BigDecimal(OrderReadHelper.getOrderItemAdjustmentsTotal(orderItem,
> allAdjustments, false, false, true))); // get the shipping amount
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1468:
> warning: [deprecation]
> getAllOrderItemsAdjustmentsTotal(java.util.List,java.util.List,boolean,boolean,boolean)
> in org.ofbiz.order.order.OrderReadHelper has been deprecated
> double currentShipping =
> OrderReadHelper.getAllOrderItemsAdjustmentsTotal(orh.getOrderItemAndShipGroupAssoc(shipGroupSeqId),
> orh.getAdjustments(), false, false, true);
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1469:
> warning: [deprecation] getOrderItemsSubTotal() in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> currentShipping +=
> OrderReadHelper.calcOrderAdjustments(orh.getOrderHeaderAdjustments(shipGroupSeqId),
> orh.getOrderItemsSubTotal(), false, false, true);
> applications/order/src/org/ofbiz/order/order/OrderServices.java:1469:
> warning: [deprecation]
> calcOrderAdjustments(java.util.List,double,boolean,boolean,boolean) in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> currentShipping +=
> OrderReadHelper.calcOrderAdjustments(orh.getOrderHeaderAdjustments(shipGroupSeqId),
> orh.getOrderItemsSubTotal(), false, false, true);
> applications/order/src/org/ofbiz/order/order/OrderServices.java:2355:
> warning: [deprecation] getOrderItemsSubTotal() in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> double orderSubTotal = orh.getOrderItemsSubTotal();
> applications/order/src/org/ofbiz/order/order/OrderServices.java:2357:
> warning: [deprecation]
> getAllOrderItemsAdjustmentsTotal(java.util.List,java.util.List,boolean,boolean,boolean)
> in org.ofbiz.order.order.OrderReadHelper has been deprecated
> double shippingAmount =
> OrderReadHelper.getAllOrderItemsAdjustmentsTotal(orderItems,
> orderAdjustments, false, false, true);
> applications/order/src/org/ofbiz/order/order/OrderServices.java:2358:
> warning: [deprecation]
> calcOrderAdjustments(java.util.List,double,boolean,boolean,boolean) in
> org.ofbiz.order.order.OrderReadHelper has been deprecated
> shippingAmount +=
> OrderReadHelper.calcOrderAdjustments(orderHeaderAdjustments, orderSubTotal,
> false, false, true);
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
