[jira] [Commented] (OFBIZ-9204) Cancelled OrderItems are still included in orderGrandTotal

2017-02-07 Thread Michael Brohl (JIRA)

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

Michael Brohl commented on OFBIZ-9204:
--

Yes, please file a new issue, thanks Maurice!

> Cancelled OrderItems are still included in orderGrandTotal
> --
>
> Key: OFBIZ-9204
> URL: https://issues.apache.org/jira/browse/OFBIZ-9204
> Project: OFBiz
>  Issue Type: Bug
>Affects Versions: Release Branch 14.12, Release Branch 15.12, Upcoming 
> Release, Release Branch 16.11
>Reporter: Maurice Meyer
>Assignee: Michael Brohl
> Fix For: Release Branch 14.12, Release Branch 15.12, Upcoming 
> Release, 16.11.02
>
> Attachments: OFBIZ-9204.patch
>
>
> Cancelled OrderItems are still included in the calculation of the order price.
> Tested with release 16.11, revision: 1781126
> Steps to reproduce:
> 1. Setup a fresh OFBiz and do loadDefault
> 2. Go to the Order Manager and cancel the only existing demo order named 
> DEMO10090 here: 
> https://localhost:8443/ordermgr/control/orderview?orderId=DEMO10090
> 3. The "Total Due" at the bottom right still states 38.36$. It should be 0$ 
> because all the items got cancelled.
> The error is probably in OrderView.groovy line 119: 
> OrderReadHelper.getOrderGrandTotal(orderItems, orderAdjustments)
> This method apparently includes cancelled OrderItems in the grandTotal.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (OFBIZ-9204) Cancelled OrderItems are still included in orderGrandTotal

2017-02-07 Thread Maurice Meyer (JIRA)

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

Maurice Meyer commented on OFBIZ-9204:
--

Great, thanks for comitting.

What about the tax issue? With the patch applied the "Total Due" of the 
cancelled order "DEMO10090" is - 0.04$. That's not working as intended, is it? 
Should a new Ticket be opened for that?

> Cancelled OrderItems are still included in orderGrandTotal
> --
>
> Key: OFBIZ-9204
> URL: https://issues.apache.org/jira/browse/OFBIZ-9204
> Project: OFBiz
>  Issue Type: Bug
>Affects Versions: Release Branch 14.12, Release Branch 15.12, Upcoming 
> Release, Release Branch 16.11
>Reporter: Maurice Meyer
>Assignee: Michael Brohl
> Fix For: Release Branch 14.12, Release Branch 15.12, Upcoming 
> Release, 16.11.02
>
> Attachments: OFBIZ-9204.patch
>
>
> Cancelled OrderItems are still included in the calculation of the order price.
> Tested with release 16.11, revision: 1781126
> Steps to reproduce:
> 1. Setup a fresh OFBiz and do loadDefault
> 2. Go to the Order Manager and cancel the only existing demo order named 
> DEMO10090 here: 
> https://localhost:8443/ordermgr/control/orderview?orderId=DEMO10090
> 3. The "Total Due" at the bottom right still states 38.36$. It should be 0$ 
> because all the items got cancelled.
> The error is probably in OrderView.groovy line 119: 
> OrderReadHelper.getOrderGrandTotal(orderItems, orderAdjustments)
> This method apparently includes cancelled OrderItems in the grandTotal.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (OFBIZ-9204) Cancelled OrderItems are still included in orderGrandTotal

2017-02-06 Thread Jacques Le Roux (JIRA)

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

Jacques Le Roux commented on OFBIZ-9204:


I guess Deepak means
bq. orderReadHelper.getOrderGrandTotal()

> Cancelled OrderItems are still included in orderGrandTotal
> --
>
> Key: OFBIZ-9204
> URL: https://issues.apache.org/jira/browse/OFBIZ-9204
> Project: OFBiz
>  Issue Type: Bug
>Affects Versions: Release Branch 16.11
>Reporter: Maurice Meyer
>Assignee: Michael Brohl
>
> Cancelled OrderItems are still included in the calculation of the order price.
> Tested with release 16.11, revision: 1781126
> Steps to reproduce:
> 1. Setup a fresh OFBiz and do loadDefault
> 2. Go to the Order Manager and cancel the only existing demo order named 
> DEMO10090 here: 
> https://localhost:8443/ordermgr/control/orderview?orderId=DEMO10090
> 3. The "Total Due" at the bottom right still states 38.36$. It should be 0$ 
> because all the items got cancelled.
> The error is probably in OrderView.groovy line 119: 
> OrderReadHelper.getOrderGrandTotal(orderItems, orderAdjustments)
> This method apparently includes cancelled OrderItems in the grandTotal.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (OFBIZ-9204) Cancelled OrderItems are still included in orderGrandTotal

2017-02-06 Thread Deepak Dixit (JIRA)

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

Deepak Dixit commented on OFBIZ-9204:
-

Hi Maurice,

Better fix is to use OrderReadHelper method directly to get grandTotal.
{code}
orh,getOrderGrandTotal()
{code}

You can create patch with these changes. 

> Cancelled OrderItems are still included in orderGrandTotal
> --
>
> Key: OFBIZ-9204
> URL: https://issues.apache.org/jira/browse/OFBIZ-9204
> Project: OFBiz
>  Issue Type: Bug
>Affects Versions: Release Branch 16.11
>Reporter: Maurice Meyer
>Assignee: Michael Brohl
>
> Cancelled OrderItems are still included in the calculation of the order price.
> Tested with release 16.11, revision: 1781126
> Steps to reproduce:
> 1. Setup a fresh OFBiz and do loadDefault
> 2. Go to the Order Manager and cancel the only existing demo order named 
> DEMO10090 here: 
> https://localhost:8443/ordermgr/control/orderview?orderId=DEMO10090
> 3. The "Total Due" at the bottom right still states 38.36$. It should be 0$ 
> because all the items got cancelled.
> The error is probably in OrderView.groovy line 119: 
> OrderReadHelper.getOrderGrandTotal(orderItems, orderAdjustments)
> This method apparently includes cancelled OrderItems in the grandTotal.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (OFBIZ-9204) Cancelled OrderItems are still included in orderGrandTotal

2017-02-06 Thread Michael Brohl (JIRA)

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

Michael Brohl commented on OFBIZ-9204:
--

Hi Maurice,

thanks for bringing this up. Is it possible for you to provide a patch for this 
issue?

Thanks and regards,

Michael

> Cancelled OrderItems are still included in orderGrandTotal
> --
>
> Key: OFBIZ-9204
> URL: https://issues.apache.org/jira/browse/OFBIZ-9204
> Project: OFBiz
>  Issue Type: Bug
>Affects Versions: Release Branch 16.11
>Reporter: Maurice Meyer
>Assignee: Michael Brohl
>
> Cancelled OrderItems are still included in the calculation of the order price.
> Tested with release 16.11, revision: 1781126
> Steps to reproduce:
> 1. Setup a fresh OFBiz and do loadDefault
> 2. Go to the Order Manager and cancel the only existing demo order named 
> DEMO10090 here: 
> https://localhost:8443/ordermgr/control/orderview?orderId=DEMO10090
> 3. The "Total Due" at the bottom right still states 38.36$. It should be 0$ 
> because all the items got cancelled.
> The error is probably in OrderView.groovy line 119: 
> OrderReadHelper.getOrderGrandTotal(orderItems, orderAdjustments)
> This method apparently includes cancelled OrderItems in the grandTotal.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)