Merge lp:~openerp-dev/openobject-addons/6.0-opw-17415-nep into lp:openobject-addons/6.0

Proposed by Nehal Panchal (OpenERP)
Status: Approved
Approved by: Rifakat Husen (OpenERP)
Approved revision: 4819
Proposed branch: lp:~openerp-dev/openobject-addons/6.0-opw-17415-nep
Merge into: lp:openobject-addons/6.0
Diff against target: 21 lines (+2/-2)
1 file modified
point_of_sale/point_of_sale.py (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.0-opw-17415-nep
Reviewer Review Type Date Requested Status
Rifakat Husen (OpenERP) (community) Approve
Anup(SerpentCS) Pending
Review via email: mp+74761@code.launchpad.net

Description of the change

Hello,

Subtotal of POS lines does not include tax.

This fixes the issue.

Thanks.

To post a comment you must log in.
Revision history for this message
Rifakat Husen (OpenERP) (rha-openerp) wrote :

Hi Nehal,

Your fix works perfectly.

Thanks a ton,
Rifakat

review: Approve

Unmerged revisions

4819. By Nehal Panchal (OpenERP)

[FIX] point_of_sale : pos lines subtotal does not include tax

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'point_of_sale/point_of_sale.py'
2--- point_of_sale/point_of_sale.py 2011-06-02 05:14:55 +0000
3+++ point_of_sale/point_of_sale.py 2011-09-09 11:34:32 +0000
4@@ -181,7 +181,7 @@
5 line.qty, line.product_id, line.order_id.partner_id)['taxes']:
6 val += c.get('amount', 0.0)
7 res[order.id]['amount_tax'] = cur_obj.round(cr, uid, cur, val)
8- res[order.id]['amount_total'] = res[order.id]['amount_tax'] + cur_obj.round(cr, uid, cur, val1)
9+ res[order.id]['amount_total'] = cur_obj.round(cr, uid, cur, val1)
10 return res
11
12 def _sale_journal_get(self, cr, uid, context=None):
13@@ -1017,7 +1017,7 @@
14 price = line.price_unit * (1 - (line.discount or 0.0) / 100.0)
15 computed_taxes = account_tax_obj.compute_all(cr, uid, taxes, price, line.qty)
16 cur = line.order_id.pricelist_id.currency_id
17- res[line.id][f] = self.pool.get('res.currency').round(cr, uid, cur, computed_taxes['total'])
18+ res[line.id][f] = self.pool.get('res.currency').round(cr, uid, cur, computed_taxes['total_included'])
19 return res
20
21 def price_by_product_multi(self, cr, uid, ids, context=None):