Openbravo Issue Tracking System - POS2
View Issue Details
0053498POS2POSpublic2023-09-20 10:032024-06-22 11:11
cberner 
AugustoMauch 
normalmajorhave not tried
closedfixed 
5
 
24Q2 
Production - QA Approved
2023-08-10
23Q3
https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/b1b36743cb98217957f2bc92ba97d66c2292935e [^]
No
0053498: Add and remove service associations does not allow selecting lines
It is not possible to select any line in the process to add or remove service associations, resulting in these processes not working properly.

Two gifs have been provided, showing how it works in 23Q2.3 and how it fails to work starting from 23Q3.
1. Login WebPOS (23Q3+): https://livebuilds.openbravo.com/context/pos2_mp_23Q3/web/pos/?terminal=VBS-2 [^]
2. Add a GPS Mini product to the ticket, and any of the warranty services that appear. Finish the service selection.
3. Click on the Warranty service product ticket line, then click on "Remove Associations", see that lines can't be selected.

An environment where it is working properly is available on version 23Q2.3 -> https://livebuilds.openbravo.com/context/pos2_mp_23Q2.3/web/pos/?terminal=VBS-2 [^]
No tags attached.
depends on backport 005581424Q2 closed AugustoMauch Add and remove service associations does not allow selecting lines 
depends on backport 005581524Q1.3 scheduled AugustoMauch Add and remove service associations does not allow selecting lines 
caused by defect 0052435 closed AugustoMauch When a grid has selected records and is sorted, different records are then selected 
Not all the children of this issue are yet resolved or closed.
gif removeAssocitionsFailing.gif (442,589) 2023-09-20 10:03
https://issues.openbravo.com/file_download.php?file_id=18973&type=bug
gif removeAssociationsWorking.gif (876,843) 2023-09-20 10:05
https://issues.openbravo.com/file_download.php?file_id=18974&type=bug
Issue History
2023-09-20 10:03cbernerNew Issue
2023-09-20 10:03cbernerAssigned To => Retail
2023-09-20 10:03cbernerFile Added: removeAssocitionsFailing.gif
2023-09-20 10:03cbernerRegression level => Production - QA Approved
2023-09-20 10:03cbernerRegression date => 2023-08-10
2023-09-20 10:03cbernerRegression introduced in release => 23Q3
2023-09-20 10:03cbernerTriggers an Emergency Pack => No
2023-09-20 10:05cbernerFile Added: removeAssociationsWorking.gif
2023-09-20 10:10cbernerSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=26855#r26855
2023-09-20 13:54sreehariAssigned ToRetail => sreehari
2023-09-22 08:24sreehariStatusnew => scheduled
2023-09-22 11:10sreehariRegression introduced by commit => https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/b1b36743cb98217957f2bc92ba97d66c2292935e [^]
2023-12-14 07:21sreehariRelationship addedcaused by 0052435
2023-12-14 08:53sreehariNote Added: 0158183
2023-12-14 08:54sreehariAssigned Tosreehari => Retail
2023-12-14 08:54sreehariStatusscheduled => acknowledged
2023-12-14 08:55sreehariStatusacknowledged => scheduled
2023-12-14 08:55sreehariStatusscheduled => resolved
2023-12-14 08:55sreehariResolutionopen => fixed
2023-12-14 08:55sreehariStatusresolved => new
2023-12-14 08:55sreehariResolutionfixed => open
2024-05-07 16:27lorenzofidalgoTarget Version => 24Q2
2024-05-22 10:45guilleaerStatusnew => acknowledged
2024-05-22 10:52guilleaerStatusacknowledged => scheduled
2024-05-22 10:52guilleaerStatusscheduled => resolved
2024-05-22 10:52guilleaerResolutionopen => fixed
2024-05-22 10:53guilleaerAssigned ToRetail => Triage Platform Base
2024-05-22 10:53guilleaerStatusresolved => new
2024-05-22 10:53guilleaerResolutionfixed => open
2024-05-30 11:23hgbotNote Added: 0165308
2024-06-10 12:54AugustoMauchAssigned ToTriage Platform Base => AugustoMauch
2024-06-10 12:54AugustoMauchStatusnew => scheduled
2024-06-19 16:23hgbotNote Added: 0166111
2024-06-21 11:19hgbotNote Added: 0166191
2024-06-21 11:19hgbotNote Added: 0166192
2024-06-21 11:19hgbotNote Added: 0166193
2024-06-21 11:19hgbotResolutionopen => fixed
2024-06-21 11:19hgbotStatusscheduled => closed
2024-06-21 11:19hgbotFixed in Version => 24Q3
2024-06-21 11:19hgbotNote Added: 0166194
2024-06-22 11:11AugustoMauchStatusclosed => new
2024-06-22 11:11AugustoMauchResolutionfixed => open
2024-06-22 11:11AugustoMauchFixed in Version24Q3 =>
2024-06-22 11:11AugustoMauchStatusnew => scheduled
2024-06-22 11:11AugustoMauchStatusscheduled => resolved
2024-06-22 11:11AugustoMauchResolutionopen => fixed
2024-06-22 11:11AugustoMauchStatusresolved => closed

Notes
(0158183)
sreehari   
2023-12-14 08:53   
After discussing with Guillermo Alvarez, I'm adding my analysis for this issue, and this needs to be fixed by the Platform team.

Related to this issue, trying to fix this, I found the following causes...

Relted to this Issue https://issues.openbravo.com/view.php?id=52435, [^] the following commits
-https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1098/diffs#ab36ba8b39bdb571549e3add9a1411be653d8a73_231_233 [^]
-https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/blob/master/web-jspack/org.openbravo.core2/src/components/Grid/stories/GridSampleData.js#L402 [^]

the above commits, added a new field id for the rows with checkboxes, and the sample containing id 1,2,3.,,

Due to above fix, in our current issue,
https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/blob/master/web-jspack/org.openbravo.pos2/src/components/Service/ServiceAssociationsGrid/ServiceAssociationsGrid.jsx#L85 [^]
-apiRefs.current -> this object will have keys of the rowIndex selected, which would be 0, 1 2 for number of rows respectively... In our case it would be uuid, and it should expected to work anyways...

But even after my fix, adding the id to the rows, in below commits
https://gitlab.com/openbravo/devel/pmods/org.openbravo.core2/-/commit/1226cdfbfb801aa1a1c84bb42a68cd11adf50937 [^]
https://gitlab.com/openbravo/devel/pmods/org.openbravo.pos2/-/commit/d9d1353baa96cd65a91b7dc0eec950a31562deff [^]

-I'm still unable to fix the issue, since the UUID didnt select all the parent columns respectively.
-The issue after my fixes commits are as follows
    -the collapsible button didnt work to collapse the below rows,
    -the error while clicking the parent checkbox is gone, but still the checkbox remains unchecked
    -though the checkbox is unchecked, it seems when u click the finish button the product gets removed
-Hope this helps.

Video Link after my fix:
https://drive.google.com/file/d/1NaUYtKpYJ0Q85mvu3iUtwm4U3PBLBUuv/view?usp=sharing [^]
(0165308)
hgbot   
2024-05-30 11:23   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/2686 [^]
(0166111)
hgbot   
2024-06-19 16:23   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1521 [^]
(0166191)
hgbot   
2024-06-21 11:19   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1521 [^]
(0166192)
hgbot   
2024-06-21 11:19   
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^]
Changeset: 84c15e5781b5902f0397954cabeff51802cb4808
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 19-06-2024 16:23:03
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/84c15e5781b5902f0397954cabeff51802cb4808 [^]

Related to ISSUE-53498: Clicking on collapse/uncollapse icon should not change the grid selection

---
M web-jspack/org.openbravo.core2/src/components/Grid/DataGrid/DataGrid.jsx
---
(0166193)
hgbot   
2024-06-21 11:19   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/2686 [^]
(0166194)
hgbot   
2024-06-21 11:19   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 5109b141f7369421066ee888abb68f9dd94fa8c1
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 21-06-2024 10:08:05
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/5109b141f7369421066ee888abb68f9dd94fa8c1 [^]

Fixes ISSUE-53498: Adapts service association grids to updated grid API

---
M web-jspack/org.openbravo.pos2/src/components/Service/AddServiceAssociations/AddServiceAssociationsDataSource.js
M web-jspack/org.openbravo.pos2/src/components/Service/RemoveServiceAssociations/RemoveServiceAssociationsDataSource.js
M web-jspack/org.openbravo.pos2/src/components/Service/ServiceAssociationsGrid/ServiceAssociationsGrid.jsx
M web-jspack/org.openbravo.pos2/src/components/Ticket/TicketLinePopover/TicketLinePopover.jsx
---