Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0043333 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Sessions | major | have not tried | 2020-02-25 18:57 | 2020-08-04 14:33 | |||
Reporter | adrianromero | View Status | public | |||||
Assigned To | rqueralta | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | RR20Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 3ae476973adb | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Linux 64 bit | Database | PostgreSQL | Java version | 7.x | |||
OS Version | Openbravo Appliance 14.04 | Database version | 9.3.x | Ant version | 1.9.x | |||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | jorge-garcia | |||||||
OBNetwork customer | Gold | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0043333: The login hook is not executed when login after a cashup | |||||||
Description | The module sessions logs out the POS after a cashup. Then when the user logs in again functionality linked using the hook 'OBPOS_LoadPOSWindow' is not executed because the hook OBPOS_LoadPOSWindow implemented in the sessions module does not continue properly the hooks chain executing the function: OB.UTIL.HookManager.callbackExecutor(args, callbacks); | |||||||
Steps To Reproduce | Login in https://livebuilds.openbravo.com/retail_modules_pgsql_pi/web/org.openbravo.retail.posterminal/?terminal=VBS-1 [^] as Vallblanca Execute the cashup process and at the end observe the application goes to the login window. Open the developer tools and set a breakpoint in the lines: OB.UTIL.HookManager.executeHooks('OBPOS_LoadPOSWindow', ... And at the beginning of the callback function: var nextWindow = OB.MobileApp.model.get('nextWindow'); Log in Observe the executeHooks line is executed but the callback function is not. The reason is that the hook OBPOS_LoadPOSWindow in sessions does not invoke the callback executor in this case: https://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/file/tip/web/org.openbravo.retail.sessions/js/components/loginhook.js#l274 [^] This is wrong. The callback executor must be invoked in all cases. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|||||||||||||||
|
![]() |
|
(0118207) hgbot (developer) 2020-02-28 14:59 |
Repository: erp/pmods/org.openbravo.retail.sessions Changeset: 8ef9f48bb1934cd46a5fbaf42bb446cd5dc4447a Author: Rafael Queralta <rafaelcuba81 <at> gmail.com> Date: Thu Feb 27 17:32:17 2020 -0500 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/8ef9f48bb1934cd46a5fbaf42bb446cd5dc4447a [^] Fixed BUG-43333: The login hook is not executed when login after a cashup - Invoked the callback executor in all cases, passing cancellation argument to avoid execute the callback executeHooks if it is necessary --- M web/org.openbravo.retail.sessions/js/components/loginhook.js --- |
(0118208) hgbot (developer) 2020-02-28 15:00 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 0a131525bc281ad0b9938e226a1385545b8e1b83 Author: Rafael Queralta <rafaelcuba81 <at> gmail.com> Date: Thu Feb 27 17:21:31 2020 -0500 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0a131525bc281ad0b9938e226a1385545b8e1b83 [^] Fixed BUG-43333: The login hook is not executed when login after a cashup - Added cancellation argument to callback of executeHooks --- M web/org.openbravo.retail.posterminal/js/login/model/login-model.js --- |
(0118551) jorge-garcia (viewer) 2020-03-12 16:12 |
OB.UTIL.HookManager.callbackExecutor is not being called in all the places that are required in loginhook file |
(0118566) hgbot (developer) 2020-03-12 21:43 |
Repository: erp/pmods/org.openbravo.retail.sessions Changeset: 3ae476973adbea4e2d87dc85bbf917922159fbf0 Author: Rafael Queralta <rafaelcuba81 <at> gmail.com> Date: Thu Mar 12 16:41:12 2020 -0400 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/3ae476973adbea4e2d87dc85bbf917922159fbf0 [^] Fixed BUG-43333: The login hook is not executed when login after a cashup - Added call to OB.UTIL.HookManager.callbackExecutor in others places that are required --- M web/org.openbravo.retail.sessions/js/components/loginhook.js --- |
(0118580) hgbot (developer) 2020-03-13 10:55 |
Repository: erp/pmods/org.openbravo.retail.sessions Changeset: d6ca5aabac3fc4f2ea0ab3e5f3e9634852725e87 Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Fri Mar 13 10:55:14 2020 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/d6ca5aabac3fc4f2ea0ab3e5f3e9634852725e87 [^] Related to BUG-43333: Always cancel callback execution for OBPOS_LoadPOSWindow hook --- M web/org.openbravo.retail.sessions/js/components/loginhook.js --- |
(0118581) jorge-garcia (viewer) 2020-03-13 10:56 |
Code reviewed and tested |
(0118597) hgbot (developer) 2020-03-13 15:28 |
Repository: erp/pmods/org.openbravo.retail.sessions Changeset: cfd879370c5f7b87f0da9862bd832b65dc20f268 Author: Rafael Queralta <rafaelcuba81 <at> gmail.com> Date: Fri Mar 13 10:28:19 2020 -0400 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/cfd879370c5f7b87f0da9862bd832b65dc20f268 [^] Related to BUG-43333: At this point callback execution for OBPOS_LoadPOSWindow hook is required --- M web/org.openbravo.retail.sessions/js/components/loginhook.js --- |
(0118612) hgbot (developer) 2020-03-16 08:58 |
Repository: erp/pmods/org.openbravo.retail.sessions Changeset: adfb84be121fe0892c0fc893b654f1911d7f8db1 Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Mon Mar 16 08:58:30 2020 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/adfb84be121fe0892c0fc893b654f1911d7f8db1 [^] Related to BUG 43333: Backout last changes for issue Backed out changesets cfd879370c5f and d6ca5aabac3f --- M web/org.openbravo.retail.sessions/js/components/loginhook.js --- |
![]() |
|||
Date Modified | Username | Field | Change |
2020-02-25 18:57 | adrianromero | New Issue | |
2020-02-25 18:57 | adrianromero | Assigned To | => Retail |
2020-02-25 18:57 | adrianromero | OBNetwork customer | => No |
2020-02-25 18:57 | adrianromero | Triggers an Emergency Pack | => No |
2020-02-25 19:05 | adrianromero | Steps to Reproduce Updated | View Revisions |
2020-02-26 10:26 | marvintm | OBNetwork customer | No => Gold |
2020-02-26 10:26 | marvintm | Resolution time | => 1584399600 |
2020-02-26 18:55 | rqueralta | Assigned To | Retail => rqueralta |
2020-02-26 18:55 | rqueralta | Status | new => scheduled |
2020-02-28 14:59 | hgbot | Checkin | |
2020-02-28 14:59 | hgbot | Note Added: 0118207 | |
2020-02-28 14:59 | hgbot | Status | scheduled => resolved |
2020-02-28 14:59 | hgbot | Resolution | open => fixed |
2020-02-28 14:59 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/8ef9f48bb1934cd46a5fbaf42bb446cd5dc4447a [^] |
2020-02-28 15:00 | hgbot | Checkin | |
2020-02-28 15:00 | hgbot | Note Added: 0118208 | |
2020-02-28 15:00 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/8ef9f48bb1934cd46a5fbaf42bb446cd5dc4447a [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0a131525bc281ad0b9938e226a1385545b8e1b83 [^] |
2020-03-11 16:35 | jorge-garcia | Review Assigned To | => jorge-garcia |
2020-03-12 16:12 | jorge-garcia | Note Added: 0118551 | |
2020-03-12 16:12 | jorge-garcia | Status | resolved => new |
2020-03-12 16:12 | jorge-garcia | Resolution | fixed => open |
2020-03-12 21:43 | hgbot | Checkin | |
2020-03-12 21:43 | hgbot | Note Added: 0118566 | |
2020-03-12 21:43 | hgbot | Status | new => resolved |
2020-03-12 21:43 | hgbot | Resolution | open => fixed |
2020-03-12 21:43 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0a131525bc281ad0b9938e226a1385545b8e1b83 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/3ae476973adbea4e2d87dc85bbf917922159fbf0 [^] |
2020-03-13 10:55 | hgbot | Checkin | |
2020-03-13 10:55 | hgbot | Note Added: 0118580 | |
2020-03-13 10:56 | jorge-garcia | Note Added: 0118581 | |
2020-03-13 10:56 | jorge-garcia | Status | resolved => closed |
2020-03-13 10:56 | jorge-garcia | Fixed in Version | => RR20Q2 |
2020-03-13 15:28 | hgbot | Checkin | |
2020-03-13 15:28 | hgbot | Note Added: 0118597 | |
2020-03-16 08:58 | hgbot | Checkin | |
2020-03-16 08:58 | hgbot | Note Added: 0118612 | |
2020-04-27 10:26 | adrianromero | Relationship added | causes 0043874 |
2020-08-04 13:58 | adrianromero | Relationship added | blocks 0044565 |
2020-08-04 14:32 | adrianromero | Assigned To | rqueralta => prakashmurugesan88 |
2020-08-04 14:32 | adrianromero | Note Added: 0121805 | |
2020-08-04 14:32 | adrianromero | Status | closed => new |
2020-08-04 14:32 | adrianromero | Resolution | fixed => open |
2020-08-04 14:32 | adrianromero | Fixed in Version | RR20Q2 => |
2020-08-04 14:33 | adrianromero | Assigned To | prakashmurugesan88 => rqueralta |
2020-08-04 14:33 | adrianromero | Status | new => acknowledged |
2020-08-04 14:33 | adrianromero | Status | acknowledged => scheduled |
2020-08-04 14:33 | adrianromero | Status | scheduled => resolved |
2020-08-04 14:33 | adrianromero | Fixed in Version | => RR20Q2 |
2020-08-04 14:33 | adrianromero | Resolution | open => fixed |
2020-08-04 14:33 | adrianromero | Status | resolved => closed |
2020-08-04 14:34 | adrianromero | Note Deleted: 0121805 |
Copyright © 2000 - 2009 MantisBT Group |