Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0053498
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[POS2] POSmajorhave not tried2023-09-20 10:032024-06-22 11:11
ReportercbernerView Statuspublic 
Assigned ToAugustoMauch 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version24Q2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression levelProduction - QA Approved
Regression date2023-08-10
Regression introduced in release23Q3
Regression introduced by commithttps://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/b1b36743cb98217957f2bc92ba97d66c2292935e [^]
Triggers an Emergency PackNo
Summary

0053498: Add and remove service associations does not allow selecting lines

DescriptionIt 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.
Steps To Reproduce1. 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 [^]
TagsNo tags attached.
Attached Filesgif file icon removeAssocitionsFailing.gif [^] (442,589 bytes) 2023-09-20 10:03
gif file icon removeAssociationsWorking.gif [^] (876,843 bytes) 2023-09-20 10:05

- Relationships Relation Graph ] Dependency Graph ]
depends on backport 005581424Q2 closedAugustoMauch Add and remove service associations does not allow selecting lines 
depends on backport 005581524Q1.3 scheduledAugustoMauch Add and remove service associations does not allow selecting lines 
caused by defect 0052435 closedAugustoMauch 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.

-  Notes
(0158183)
sreehari (developer)
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 (developer)
2024-05-30 11:23

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/2686 [^]
(0166111)
hgbot (developer)
2024-06-19 16:23

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1521 [^]
(0166191)
hgbot (developer)
2024-06-21 11:19

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/1521 [^]
(0166192)
hgbot (developer)
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 (developer)
2024-06-21 11:19

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/2686 [^]
(0166194)
hgbot (developer)
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
---

- Issue History
Date Modified Username Field Change
2023-09-20 10:03 cberner New Issue
2023-09-20 10:03 cberner Assigned To => Retail
2023-09-20 10:03 cberner File Added: removeAssocitionsFailing.gif
2023-09-20 10:03 cberner Regression level => Production - QA Approved
2023-09-20 10:03 cberner Regression date => 2023-08-10
2023-09-20 10:03 cberner Regression introduced in release => 23Q3
2023-09-20 10:03 cberner Triggers an Emergency Pack => No
2023-09-20 10:05 cberner File Added: removeAssociationsWorking.gif
2023-09-20 10:10 cberner Steps to Reproduce Updated View Revisions
2023-09-20 13:54 sreehari Assigned To Retail => sreehari
2023-09-22 08:24 sreehari Status new => scheduled
2023-09-22 11:10 sreehari Regression introduced by commit => https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/b1b36743cb98217957f2bc92ba97d66c2292935e [^]
2023-12-14 07:21 sreehari Relationship added caused by 0052435
2023-12-14 08:53 sreehari Note Added: 0158183
2023-12-14 08:54 sreehari Assigned To sreehari => Retail
2023-12-14 08:54 sreehari Status scheduled => acknowledged
2023-12-14 08:55 sreehari Status acknowledged => scheduled
2023-12-14 08:55 sreehari Status scheduled => resolved
2023-12-14 08:55 sreehari Resolution open => fixed
2023-12-14 08:55 sreehari Status resolved => new
2023-12-14 08:55 sreehari Resolution fixed => open
2024-05-07 16:27 lorenzofidalgo Target Version => 24Q2
2024-05-22 10:45 guilleaer Status new => acknowledged
2024-05-22 10:52 guilleaer Status acknowledged => scheduled
2024-05-22 10:52 guilleaer Status scheduled => resolved
2024-05-22 10:52 guilleaer Resolution open => fixed
2024-05-22 10:53 guilleaer Assigned To Retail => Triage Platform Base
2024-05-22 10:53 guilleaer Status resolved => new
2024-05-22 10:53 guilleaer Resolution fixed => open
2024-05-30 11:23 hgbot Note Added: 0165308
2024-06-10 12:54 AugustoMauch Assigned To Triage Platform Base => AugustoMauch
2024-06-10 12:54 AugustoMauch Status new => scheduled
2024-06-19 16:23 hgbot Note Added: 0166111
2024-06-21 11:19 hgbot Note Added: 0166191
2024-06-21 11:19 hgbot Note Added: 0166192
2024-06-21 11:19 hgbot Note Added: 0166193
2024-06-21 11:19 hgbot Resolution open => fixed
2024-06-21 11:19 hgbot Status scheduled => closed
2024-06-21 11:19 hgbot Fixed in Version => 24Q3
2024-06-21 11:19 hgbot Note Added: 0166194
2024-06-22 11:11 AugustoMauch Status closed => new
2024-06-22 11:11 AugustoMauch Resolution fixed => open
2024-06-22 11:11 AugustoMauch Fixed in Version 24Q3 =>
2024-06-22 11:11 AugustoMauch Status new => scheduled
2024-06-22 11:11 AugustoMauch Status scheduled => resolved
2024-06-22 11:11 AugustoMauch Resolution open => fixed
2024-06-22 11:11 AugustoMauch Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker