Openbravo Issue Tracking System - Retail Modules | |||||||||||||||||||
View Issue Details | |||||||||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||||||||
0032277 | Retail Modules | Web POS | public | 2016-02-18 16:35 | 2016-02-25 13:53 | ||||||||||||||
Reporter | marvintm | ||||||||||||||||||
Assigned To | jorge-garcia | ||||||||||||||||||
Priority | normal | Severity | major | Reproducibility | always | ||||||||||||||
Status | closed | Resolution | fixed | ||||||||||||||||
Platform | OS | 5 | OS Version | ||||||||||||||||
Product Version | |||||||||||||||||||
Target Version | pi | Fixed in Version | RR16Q2 | ||||||||||||||||
Merge Request Status | |||||||||||||||||||
Review Assigned To | guilleaer | ||||||||||||||||||
OBNetwork customer | No | ||||||||||||||||||
Support ticket | |||||||||||||||||||
Regression level | Packaging and release | ||||||||||||||||||
Regression date | 2015-12-15 | ||||||||||||||||||
Regression introduced in release | pi | ||||||||||||||||||
Regression introduced by commit | https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/5521 [^] | ||||||||||||||||||
Triggers an Emergency Pack | No | ||||||||||||||||||
Summary | 0032277: When a discretionary discount is removed, the total amount of the ticket is not computed | ||||||||||||||||||
Description | When a discretionary discount is added to a line, the total amount of the ticket is correctly calculated. However, when this discount is removed, the total doesn't change, and this is wrong. | ||||||||||||||||||
Steps To Reproduce | - Add a line to a ticket - Add a discretionary discount (using Receipt discounts) - Select the line, and in the Edit panel, click on "Remove discount" button - Verify that the discount is correctly removed from the line, but the total amount of the ticket hasn't changed. | ||||||||||||||||||
Proposed Solution | This problem comes from the removal of these two lines in editline.js: +++ b/web/org.openbravo.retail.posterminal/js/pointofsale/view/editline.js Tue Dec 15 13:23:54 2015 +0100 @@ -263,8 +263,6 @@ tap: function () { if (this.owner.owner && this.owner.owner.line && this.owner.owner.line.get('promotions')) { this.owner.owner.line.unset('promotions'); - OB.Model.Discounts.applyPromotions(this.model.get('order')); - this.model.get('order').calculateGross(); this.hide(); } }, The solution is to replace them by a call to the new calculateReceipt() function. | ||||||||||||||||||
Additional Information | |||||||||||||||||||
Tags | No tags attached. | ||||||||||||||||||
Relationships |
| ||||||||||||||||||
Attached Files | |||||||||||||||||||
Issue History | |||||||||||||||||||
Date Modified | Username | Field | Change | ||||||||||||||||
2016-02-18 16:35 | marvintm | New Issue | |||||||||||||||||
2016-02-18 16:35 | marvintm | Assigned To | => jorge-garcia | ||||||||||||||||
2016-02-18 16:35 | marvintm | OBNetwork customer | => No | ||||||||||||||||
2016-02-18 16:35 | marvintm | Regression level | => Packaging and release | ||||||||||||||||
2016-02-18 16:35 | marvintm | Regression date | => 2015-12-15 | ||||||||||||||||
2016-02-18 16:35 | marvintm | Regression introduced in release | => pi | ||||||||||||||||
2016-02-18 16:35 | marvintm | Regression introduced by commit | => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/5521 [^] | ||||||||||||||||
2016-02-18 16:35 | marvintm | Triggers an Emergency Pack | => No | ||||||||||||||||
2016-02-18 16:37 | marvintm | Status | new => scheduled | ||||||||||||||||
2016-02-19 10:04 | ranjith_qualiantech_com | Relationship added | has duplicate 0032271 | ||||||||||||||||
2016-02-23 15:38 | hgbot | Checkin | |||||||||||||||||
2016-02-23 15:38 | hgbot | Note Added: 0084436 | |||||||||||||||||
2016-02-23 15:38 | hgbot | Status | scheduled => resolved | ||||||||||||||||
2016-02-23 15:38 | hgbot | Resolution | open => fixed | ||||||||||||||||
2016-02-23 15:38 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4f3983e69d7f656bd5c8b58e975b999f5d089cd6 [^] | ||||||||||||||||
2016-02-23 15:38 | hgbot | Checkin | |||||||||||||||||
2016-02-23 15:38 | hgbot | Note Added: 0084437 | |||||||||||||||||
2016-02-25 13:53 | guilleaer | Review Assigned To | => guilleaer | ||||||||||||||||
2016-02-25 13:53 | guilleaer | Status | resolved => closed | ||||||||||||||||
2016-02-25 13:53 | guilleaer | Fixed in Version | => RR16Q2 |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|