Openbravo Issue Tracking System - POS2 | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0049394 | POS2 | POS | public | 2022-05-24 18:59 | 2022-05-24 18:59 |
| Reporter | mtaal | ||||
| Assigned To | Retail | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | new | Resolution | open | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | |||||
| Target Version | Fixed in Version | ||||
| Merge Request Status | |||||
| Review Assigned To | |||||
| OBNetwork customer | |||||
| Support ticket | |||||
| Regression level | |||||
| Regression date | |||||
| Regression introduced in release | |||||
| Regression introduced by commit | |||||
| Triggers an Emergency Pack | No | ||||
| Summary | 0049394: New scan event handlers with lower prio than the CrmConnector is never reached | ||||
| Description | The current implementation of the CrmConnectorScanEventHandler does not correctly (imv) handle the case that its scanning didn't return a customer. A eventHandled:true is returned making it impossible to add new scan event handlers with a lower priority. | ||||
| Steps To Reproduce | Add a new scan event handler with a low prio. Run this test code: setTimeout(function() { OB.App.BarcodeScanner.scanCode({code: 'ABC123'}); }, 2000); The new scan event handler is never called as before that the CrmConnector works (if enabled) | ||||
| Proposed Solution | Change the code [1] here to return eventHandled: false [1] https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/blob/master/web-jspack/org.openbravo.pos2/src/services/scan/handlers/CrmConnectorScanEventHandler.js#L48 [^] | ||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2022-05-24 18:59 | mtaal | New Issue | |||
| 2022-05-24 18:59 | mtaal | Assigned To | => Retail | ||
| 2022-05-24 18:59 | mtaal | Triggers an Emergency Pack | => No | ||
| There are no notes attached to this issue. |