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

Scott Gray commented on OFBIZ-880:
----------------------------------

Hi Jacopo

That sounds like a good way to go, but we need to run searches on all the bsh 
files first as I'm pretty sure some of the deprecated methods are still called 
from them.  But anyway, I'll just keep plugging on a couple of hours a night 
until it's all finished.


> 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.

Reply via email to