Openbravo Issue Tracking System - Retail Modules
View Issue Details
0022491Retail ModulesWeb POSpublic2012-12-10 09:252012-12-10 12:24
alostale 
alostale 
immediatemajorhave not tried
closedfixed 
5
 
RMP17 
No
0022491: Rounding issues in packs
Depending on how Packs are defined, rounding issues appear when they are applied.
With Retail Demo data:
* Create a new Pack discount
** Discount Price: 90 EUR
** Products:
*** Alpine poles, 1
*** Alpine ski boot, 1
*** Basecamp lantern, 1
* Log in POS
* Add this pack
* Amount to be paid 90.01 -> ERROR: it should be 90
No tags attached.
jpg image002.jpg (51,638) 2012-12-10 09:27
https://issues.openbravo.com/file_download.php?file_id=5805&type=bug
jpg
Issue History
2012-12-10 09:25alostaleNew Issue
2012-12-10 09:25alostaleAssigned To => alostale
2012-12-10 09:27alostaleFile Added: image002.jpg
2012-12-10 09:32hgbotCheckin
2012-12-10 09:32hgbotNote Added: 0054785
2012-12-10 09:32hgbotStatusnew => resolved
2012-12-10 09:32hgbotResolutionopen => fixed
2012-12-10 09:32hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/6ead8800d5f802c5d65ab574452c2ad08db9ebb9 [^]
2012-12-10 12:24guilleaerNote Added: 0054797
2012-12-10 12:24guilleaerStatusresolved => closed
2012-12-10 12:24guilleaerFixed in Version => RMP17

Notes
(0054785)
hgbot   
2012-12-10 09:32   
Repository: erp/pmods/org.openbravo.retail.discounts
Changeset: 6ead8800d5f802c5d65ab574452c2ad08db9ebb9
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Mon Dec 10 09:30:26 2012 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/6ead8800d5f802c5d65ab574452c2ad08db9ebb9 [^]

fixed bug 22491: Rounding issues in packs
  -Using OB.DEC to proper manage decimal amounts
  -Last line to be discounted is calculated based on pending amount to be discounted
   instead of supposed amount, in this way possible accumulated decimal diffs
   are subsaned in last line.

---
M web/org.openbravo.retail.discounts/js/promotion-pack.js
---
(0054797)
guilleaer   
2012-12-10 12:24   
Tested and verified.

To be included in EMP17.1 emergency pack