diff options
| author | Joren Van Onder <jov@odoo.com> | 2015-08-27 15:59:04 +0200 |
|---|---|---|
| committer | Cedric Snauwaert <csn@openerp.com> | 2015-08-27 17:11:07 +0200 |
| commit | dbb77c5f504bfb1b459ef51ca71862640ebf7c90 (patch) | |
| tree | 15db77416a68fa619ce771316e9228dd743f577d /addons | |
| parent | 65dbf8f11740dda608fe8e7789f7652d97b656bb (diff) | |
[FIX] point_of_sale: take quantity into account when calculating taxes
Fixes a bug introduced in 5a8a0d252cc735de4c95748c90f34ccbf50349b5.
Before this, creating an order with two of the same products would
result in a total price to pay which would only include the tax of one
product. This would cause point of sale orders in the backend which
weren't fully paid.
Diffstat (limited to 'addons')
| -rw-r--r-- | addons/point_of_sale/static/src/js/models.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/addons/point_of_sale/static/src/js/models.js b/addons/point_of_sale/static/src/js/models.js index 2ae832f8c94..0891ed037c5 100644 --- a/addons/point_of_sale/static/src/js/models.js +++ b/addons/point_of_sale/static/src/js/models.js @@ -1275,13 +1275,13 @@ exports.Orderline = Backbone.Model.extend({ return base_amount >= 0 ? ret : ret * -1; } if ((tax.amount_type === 'percent' && !tax.price_include) || (tax.amount_type === 'division' && tax.price_include)){ - return base_amount * tax.amount / 100; + return (base_amount * tax.amount / 100) * quantity; } if (tax.amount_type === 'percent' && tax.price_include){ - return base_amount - (base_amount / (1 + tax.amount / 100)); + return (base_amount - (base_amount / (1 + tax.amount / 100))) * quantity; } if (tax.amount_type === 'division' && !tax.price_include) { - return base_amount / (1 - tax.amount / 100) - base_amount; + return (base_amount / (1 - tax.amount / 100) - base_amount) * quantity; } return false; }, |