Openbravo Issue Tracking System - Retail Modules |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0037057 | Retail Modules | Web POS | public | 2017-10-11 15:55 | 2017-10-27 12:49 |
|
| Reporter | aaroncalero | |
| Assigned To | ranjith_qualiantech_com | |
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | RR18Q1 | |
| Merge Request Status | |
| Review Assigned To | marvintm |
| OBNetwork customer | |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0037057: Mandatory Services search should be blocked while the calculateReceipt function is being executed |
| Description | When a product is added to a ticket and there are mandatory services related to said product, the mandatory services search is opened and the services are listed right away, while the calculateReceipt function might still be executing.
Some functionalities might require the total of the lines to be calculated (i.e. the Automatic Service Price Rule Calculation: org.openbravo.retail.services.autoprice) before showing the service list |
| Steps To Reproduce | Login in web pos
Open the browser console and execute this[1] code to add a hook that slows down the calculateReceipt process.
Go to the Search tab, search for the Blue Trousers product and add it to the ticket.
The Mandatory services search will be shown, and after 5 seconds the total of the ticket will be correctly calculated.
[1] OB.UTIL.HookManager.registerHook('OBPOS_PreApplyAutomaticDiscount', function(args, callback){
setTimeout(function(){
OB.UTIL.HookManager.callbackExecutor(args, callback);
}, 5000)
}) |
| Proposed Solution | Attached diffs with the following changes:
Postermina: When the calculateReceipt has finished, trigger the 'calculatedReceipt' event on the receipt model.
Mobile Core: Instead of launching the service search right away, if the calculateReceipt is still pending wait for the 'calculatedReceipt' event to trigger the search. |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files |  Issue37057_Posterminal.diff (757) 2017-10-11 16:07
https://issues.openbravo.com/file_download.php?file_id=11171&type=bug
Issue37057_MobileCore.diff (1,784) 2017-10-11 16:08
https://issues.openbravo.com/file_download.php?file_id=11172&type=bug |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2017-10-11 15:55 | aaroncalero | New Issue | |
| 2017-10-11 15:55 | aaroncalero | Assigned To | => Retail |
| 2017-10-11 15:55 | aaroncalero | Resolution time | => 1508882400 |
| 2017-10-11 15:55 | aaroncalero | Triggers an Emergency Pack | => No |
| 2017-10-11 16:07 | aaroncalero | File Added: Issue37057_Posterminal.diff | |
| 2017-10-11 16:08 | aaroncalero | File Added: Issue37057_MobileCore.diff | |
| 2017-10-19 11:55 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
| 2017-10-19 11:55 | ranjith_qualiantech_com | Status | new => scheduled |
| 2017-10-23 13:00 | hgbot | Checkin | |
| 2017-10-23 13:00 | hgbot | Note Added: 0099992 | |
| 2017-10-23 13:00 | hgbot | Status | scheduled => resolved |
| 2017-10-23 13:00 | hgbot | Resolution | open => fixed |
| 2017-10-23 13:00 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/fa427f5256aa97b295fb15aa846ea97bf4be2c6b [^] |
| 2017-10-24 12:05 | hgbot | Checkin | |
| 2017-10-24 12:05 | hgbot | Note Added: 0100018 | |
| 2017-10-27 12:49 | marvintm | Review Assigned To | => marvintm |
| 2017-10-27 12:49 | marvintm | Status | resolved => closed |
| 2017-10-27 12:49 | marvintm | Fixed in Version | => RR18Q1 |