Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0049268POS2Corepublic2022-05-09 15:162022-06-10 09:27
Reporterandre_montenegro 
Assigned Toablasco 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tojarmendariz
OBNetwork customer
Support ticket
Regression levelAutomated tests
Regression date
Regression introduced in releasepi
Regression introduced by commithttps://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/a2ccac66e2463cc5835f525caad36abe4a94ee7c [^]
Triggers an Emergency PackNo
Summary0049268: TypeError after matching Initial Count after use
DescriptionThe error has been mainly observed while matching the initial count at the beginning of the automated tests, after use the button "Match initial count" for each Payment Method and click "Finish" button, a "TypeError: cannot read properties of null (reading 'value')" popup is displayed, then a full screen with the same error may be displayed.

Take in account that the behavior has been also observed in different scenarios after using keypad buttons.

For now, it has not been reproduced manually, but the behavior can be observed in the video attached: quotation.spec.js.mp4.

All Cypress automated tests are failing due this regression caused by commits which fix ISSUE-48923: minValue and maxValue is checked when Focus is out from... . Related commits: 365632fd and a2ccac66

Mentioned commits must be reverted until a new solution can be implemented in order to unblock Cypress tests executions.
Steps To Reproduce"Quotation" Cypress test steps:
- Login POS2
- Click button "Show All Payment Mt"
- Match Initial Count through button "Match initial Count" for each Payment Method
- Click "Finish" button.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
caused by defect 0048923 closed ablasco useEditNumber is not working fine if used with a keypad and minimum amount has two or more digits 
Attached Filespng ErrorAfterCommit_a2ccac66.png (237,149) 2022-05-09 15:19
https://issues.openbravo.com/file_download.php?file_id=16998&type=bug
png

png ErrorAfterCommit_365632fd.png (237,660) 2022-05-09 15:19
https://issues.openbravo.com/file_download.php?file_id=16999&type=bug
png

? quotation.spec.js_1.mp4 (266,234) 2022-05-09 15:21
https://issues.openbravo.com/file_download.php?file_id=17000&type=bug
Issue History
Date ModifiedUsernameFieldChange
2022-05-09 15:16andre_montenegroNew Issue
2022-05-09 15:16andre_montenegroAssigned To => Retail
2022-05-09 15:16andre_montenegroTriggers an Emergency Pack => No
2022-05-09 15:16andre_montenegroRelationship addedcaused by 0048923
2022-05-09 15:19andre_montenegroFile Added: ErrorAfterCommit_a2ccac66.png
2022-05-09 15:19andre_montenegroFile Added: ErrorAfterCommit_365632fd.png
2022-05-09 15:21andre_montenegroFile Added: quotation.spec.js_1.mp4
2022-05-09 15:22andre_montenegroSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=24053#r24053
2022-05-09 15:23andre_montenegroPrioritynormal => immediate
2022-05-09 15:23andre_montenegroReproducibilityhave not tried => always
2022-05-09 15:23andre_montenegroSummaryCypress tests fails due Type uncaught Error caused since commit 365632fd and a2ccac66 => Regression introduced by commits 365632fd and a2ccac66
2022-05-09 15:23andre_montenegroDescription Updatedbug_revision_view_page.php?rev_id=24055#r24055
2022-05-09 15:24andre_montenegroRegression level => Automated tests
2022-05-09 15:24andre_montenegroRegression introduced in release => pi
2022-05-09 15:32andre_montenegroDescription Updatedbug_revision_view_page.php?rev_id=24056#r24056
2022-05-09 15:32andre_montenegroSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=24057#r24057
2022-05-09 15:32andre_montenegroProposed Solution updated
2022-05-09 15:32andre_montenegroDescription Updatedbug_revision_view_page.php?rev_id=24058#r24058
2022-05-09 16:10andre_montenegroRegression introduced by commit => https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/a2ccac66e2463cc5835f525caad36abe4a94ee7c [^]
2022-05-09 16:10andre_montenegroAssigned ToRetail => jarmendariz
2022-05-09 16:10andre_montenegroversion => pi
2022-05-09 16:10andre_montenegroSummaryRegression introduced by commits 365632fd and a2ccac66 => [POS2] Regression: TypeError after matching Initial Count after use
2022-05-09 16:10andre_montenegroDescription Updatedbug_revision_view_page.php?rev_id=24059#r24059
2022-05-09 16:10andre_montenegroSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=24060#r24060
2022-05-09 16:10andre_montenegroProposed Solution updated
2022-05-09 16:12andre_montenegroDescription Updatedbug_revision_view_page.php?rev_id=24061#r24061
2022-05-09 16:12andre_montenegroSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=24062#r24062
2022-05-09 16:18andre_montenegroSummary[POS2] Regression: TypeError after matching Initial Count after use => TypeError after matching Initial Count after use
2022-05-09 17:00jarmendarizAssigned Tojarmendariz => Triage Platform Base
2022-05-09 17:26jarmendarizAssigned ToTriage Platform Base => ablasco
2022-05-10 11:28jarmendarizNote Added: 0137112
2022-05-10 11:28jarmendarizStatusnew => scheduled
2022-06-10 09:26jarmendarizStatusscheduled => resolved
2022-06-10 09:26jarmendarizResolutionopen => fixed
2022-06-10 09:27jarmendarizReview Assigned To => jarmendariz
2022-06-10 09:27jarmendarizStatusresolved => closed

Notes
(0137112)
jarmendariz   
2022-05-10 11:28   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/746 [^]