Openbravo Issue Tracking System - Retail Modules
View Issue Details
0035322Retail ModulesWeb POSpublic2017-02-21 12:142017-03-13 09:00
adrianromero 
mario_castello 
urgentmajorhave not tried
closedfixed 
5
 
RR17Q2 
marvintm
No
0035322: [SER-QA 2479] Credit amount calculation and formatting is not done properly
Credit amount calculation and comparison must be done using OB.DEC.sub and OB.DEC.compare: https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/796f4eab7e93/web/org.openbravo.retail.posterminal/js/model/order.js#l4507 [^]
Displaying credit amount must be done using OB.I18N.formatCurrency
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/796f4eab7e93/web/org.openbravo.retail.posterminal/js/components/order.js#l650 [^]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/796f4eab7e93/web/org.openbravo.retail.posterminal/js/components/order.js#l663 [^]
In description
See attached patch
SER-QA
? issue35322.export (3,265) 2017-02-21 12:20
https://issues.openbravo.com/file_download.php?file_id=10470&type=bug
Issue History
2017-02-21 12:14adrianromeroNew Issue
2017-02-21 12:14adrianromeroAssigned To => Retail
2017-02-21 12:14adrianromeroResolution time => 1488841200
2017-02-21 12:14adrianromeroTriggers an Emergency Pack => No
2017-02-21 12:17adrianromeroTag Attached: SER-QA
2017-02-21 12:20adrianromeroFile Added: issue35322.export
2017-02-21 19:26mario_castelloAssigned ToRetail => mario_castello
2017-02-21 19:27mario_castelloStatusnew => acknowledged
2017-02-21 19:27mario_castelloStatusacknowledged => scheduled
2017-03-10 21:10hgbotCheckin
2017-03-10 21:10hgbotNote Added: 0094953
2017-03-10 21:10hgbotStatusscheduled => resolved
2017-03-10 21:10hgbotResolutionopen => fixed
2017-03-10 21:10hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/e020e081b20457c52de805c4179eae52ecb34f89 [^]
2017-03-10 21:16hgbotCheckin
2017-03-10 21:16hgbotNote Added: 0094954
2017-03-13 09:00marvintmReview Assigned To => marvintm
2017-03-13 09:00marvintmStatusresolved => closed
2017-03-13 09:00marvintmFixed in Version => RR17Q2

Notes
(0094953)
hgbot   
2017-03-10 21:10   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: e020e081b20457c52de805c4179eae52ecb34f89
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Fri Mar 10 09:45:47 2017 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/e020e081b20457c52de805c4179eae52ecb34f89 [^]

Fixed issue 0035322: [SER-QA 2479] Credit amount calculation and formatting is not done properly

- Refactor the calculate of credit amount and the comparison using OB.DEC.sub and OB.DEC.compare
- Added correct format when show the value

---
M web/org.openbravo.retail.posterminal/js/components/order.js
M web/org.openbravo.retail.posterminal/js/model/order.js
---
(0094954)
hgbot   
2017-03-10 21:16   
Repository: tools/automation/pi-mobile
Changeset: fa3bd34c92b70251e3576923873614e786006fdd
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Fri Mar 10 10:13:15 2017 -0600
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/fa3bd34c92b70251e3576923873614e786006fdd [^]

Verifies issue 35322: Fixed affected tests by this changes

- Updated Copyright
- Refactor and fixed test ReversePaymentOnPartiallyPaidOnCredit
- Refactor and fixed test ReversePaymentPaidOnCreditWithCredit
- Refactor and fixed test I31600_VerfiyPaidOnCredit

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/reversepayments/ReversePaymentOnPartiallyPaidOnCredit.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/reversepayments/ReversePaymentPaidOnCreditWithCredit.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I31600_VerfiyPaidOnCredit.java
---