Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0048287POS2Corepublic2021-12-20 16:472021-12-21 16:00
Reportercaristu 
Assigned Tocaristu 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed 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 PackNo
Summary0048287: Need to click clear button twice to clear validated form fields
DescriptionNeed to click clear button twice to clear validated form fields
Steps To Reproduce1) Apply the attached patch into the pos2 module to simulate a time consuming field validation
2) Login in pos2
3) Go to the customers window, click in "New Customer" to open the form
4) Move to any field, for example to the "Email" field and enter a value
5) Click on the "x" button to clear the content. ERROR-> Note that the field is not cleared. We need to click on it a second time to clear the field.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0047232 closed jarmendariz JIRA 2057: CRM connector focus changing and validations 
depends on backport 0048224TAP closed caristu JIRA 2552 - Necessary to click twice when wanting to delete email on new client screen 
Attached Filesdiff timeConsumingValidation.diff (735) 2021-12-20 19:05
https://issues.openbravo.com/file_download.php?file_id=16461&type=bug
Issue History
Date ModifiedUsernameFieldChange
2021-12-20 16:47caristuNew Issue
2021-12-20 16:47caristuAssigned To => Triage Platform Base
2021-12-20 16:47caristuTriggers an Emergency Pack => No
2021-12-20 16:48caristuRelationship addedblocks 0048224
2021-12-20 16:50caristuRelationship addedrelated to 0047232
2021-12-20 16:50caristuAssigned ToTriage Platform Base => caristu
2021-12-20 16:50caristuStatusnew => acknowledged
2021-12-20 16:53caristuRelationship deletedblocks 0048224
2021-12-20 16:53caristuRelationship addeddepends on 0048224
2021-12-20 19:05caristuFile Added: timeConsumingValidation.diff
2021-12-21 11:39hgbotNote Added: 0133873
2021-12-21 16:00hgbotResolutionopen => fixed
2021-12-21 16:00hgbotStatusacknowledged => closed
2021-12-21 16:00hgbotNote Added: 0133878
2021-12-21 16:00hgbotNote Added: 0133879

Notes
(0133873)
hgbot   
2021-12-21 11:39   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/682 [^]
(0133878)
hgbot   
2021-12-21 16:00   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^]
Changeset: 6cc8559bca5cd0b4713914ce3dd11cb600779693
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 21-12-2021 14:59:26
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/6cc8559bca5cd0b4713914ce3dd11cb600779693 [^]

fixes ISSUE-48287: Need to click clear button twice to clear validated form fields

  This problem was caused because the field was being validated when
clicking on the clear button. We already had code to avoid this case in
on blur, but the check if the clear button is the one retrieving the
focus was incorrect.

---
M web-jspack/org.openbravo.core2/src/components/Form/BaseForm.jsx
---
(0133879)
hgbot   
2021-12-21 16:00   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/682 [^]