Openbravo Issue Tracking System - Retail Modules
View Issue Details
0052486Retail ModulesWeb POSpublic2023-05-18 23:572023-05-22 08:34
axelmercado 
ranjith_qualiantech_com 
normalmajorsometimes
closedfixed 
5
 
RR23Q3 
approved
OBPS
64736
No
0052486: Problem with ISSUE 50603 change: If the line has no product, then it fails in custom customer modules.
We have seen that in one of the new updates you have added the following in the ob-keyboard.js file:

      if (actualChar && actualChar.match(this.keyMatcher)) {
        const hasGroupedProduct = this.line
          ? this.line.get('product').get('groupProduct')
          : true;
        if ((actualChar === '+' || actualChar === '-') && hasGroupedProduct) {

The problem is that we have our own module that uses the globalKeypressHandler and when pressing a key it gives error because in the this.line the product attribute does not exist.
N/A
To solve it we have added the attached .diff, this way we ensure that if this attribute does not exist it does not give error.
No tags attached.
caused by defect 0050603 closed radhakrishnan + key enabled when typing while editing a line 
diff ob-keyboard.diff (837) 2023-05-18 23:57
https://issues.openbravo.com/file_download.php?file_id=18548&type=bug
Issue History
2023-05-18 23:57axelmercadoNew Issue
2023-05-18 23:57axelmercadoAssigned To => Retail
2023-05-18 23:57axelmercadoFile Added: ob-keyboard.diff
2023-05-18 23:57axelmercadoOBNetwork customer => OBPS
2023-05-18 23:57axelmercadoSupport ticket => 64736
2023-05-18 23:57axelmercadoTriggers an Emergency Pack => No
2023-05-18 23:57axelmercadoRelationship addedcaused by 0050603
2023-05-19 07:06PracticsIssue Monitored: Practics
2023-05-20 11:38ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2023-05-20 11:38ranjith_qualiantech_comStatusnew => scheduled
2023-05-21 16:35hgbotMerge Request Status => open
2023-05-21 16:35hgbotNote Added: 0150006
2023-05-22 08:34hgbotMerge Request Statusopen => approved
2023-05-22 08:34hgbotResolutionopen => fixed
2023-05-22 08:34hgbotStatusscheduled => closed
2023-05-22 08:34hgbotNote Added: 0150035
2023-05-22 08:34hgbotFixed in Version => RR23Q3
2023-05-22 08:34hgbotNote Added: 0150036

Notes
(0150006)
hgbot   
2023-05-21 16:35   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/526 [^]
(0150035)
hgbot   
2023-05-22 08:34   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/526 [^]
(0150036)
hgbot   
2023-05-22 08:34   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: d56346627516611900bfdd342678a286b306eadc
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 20-05-2023 15:33:43
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/d56346627516611900bfdd342678a286b306eadc [^]

Fixed ISSUE-52486: Added line validation in the global keypress event handler

---
M web/org.openbravo.mobile.core/source/component/ob-keyboard.js
---