Record list does not retrieve the selected value/data in the next task. - #1923
Record list does not retrieve the selected value/data in the next task.#1923CarliPinell wants to merge 4 commits into
Conversation
screen-builder
|
||||||||||||||||||||||||||||
| Project |
screen-builder
|
| Branch Review |
bugfix/FOUR-27264
|
| Run status |
|
| Run duration | 44m 58s |
| Commit |
|
| Committer | CarliPinell |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
0
|
|
|
19
|
|
|
0
|
|
|
389
|
| View all changes introduced in this branch ↗︎ | |
eiresendez
left a comment
There was a problem hiding this comment.
@CarliPinell the fix addresses the original issue, and I confirmed it no longer reproduces in manual testing.
However, this PR introduces a new pagination issue through the added selectedRow watcher: the existing @change path already provides a page-relative index, while the watcher passes a global index into the same handler, causing an incorrect selectedRowIndex and duplicate updates on page 2 and beyond. This behavior was not present before this change, could you please review it?
|
eiresendez
left a comment
There was a problem hiding this comment.
The new helper extraction is clean and keeps the component’s selection contract explicit. form-record-list.vue:626 now receives a page-relative index and delegates the single conversion to production-tested code 👍 👍




Solution
How to Test
Related Tickets & Packages
Code Review Checklist