Skip to content

FOUR-25776: Record List UI Unchecks Selections When Using PMQL with Multiple Records Mode #1865

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

FOUR-25776: Record List UI Unchecks Selections When Using PMQL with Multiple Records Mode #1865

wants to merge 2 commits into from
+77 −0

Conversation

@eiresendez
Copy link
Contributor

@eiresendez eiresendez commented Oct 2, 2025 •
edited by sanjacornelius
Loading

Issue & Reproduction Steps

  • Using a record list configured with Collection + PMQL filter + multiple-records selection.
  • Checkboxes deselect immediately after data reloads triggered by PMQL evaluation, filtering, or pagination.
  • Visual state and payload drift: UI showed no selections although the emitted IDs remained.

Solution

  • Rehydrated selectedRows after collection refreshes by matching refreshed rows against previously selected entries.
  • Added guard helpers so the new behavior only applies to Collection sources using PMQL with multi-record selection.
  • Preserved selectedRowsIndex metadata to keep pagination-aware indices in sync.
Screen.Recording.2025-10-02.at.12.29.37.p.m.mov

How to Test

  • Configure a screen as above (Collection source, PMQL filter, multiple-record selection).
  • Select several rows, then change filter inputs to trigger a PMQL refresh; confirm the original checkboxes remain checked.
  • Page forward and back after selecting rows and verify selections persist visually and in the emitted payload.

Related Tickets & Packages

ci:deploy
.

@cypress
Copy link

cypress bot commented Oct 2, 2025 •
edited
Loading

screen-builder    Run #2059

Run Properties:  status check passed Passed #2059  •  git commit e001b9c740: FOUR-25776: Record List UI Unchecks Selections When Using PMQL with Multiple Rec...
Project screen-builder
Branch Review defect/FOUR-25776
Run status status check passed Passed #2059
Run duration 09m 34s
Commit git commit e001b9c740: FOUR-25776: Record List UI Unchecks Selections When Using PMQL with Multiple Rec...
Committer Eleazar Resendez
View all properties for this run ↗︎
Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 19
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 389
View all changes introduced in this branch ↗︎

@eiresendez eiresendez requested a review from CarliPinell October 2, 2025 20:22
CarliPinell
CarliPinell approved these changes Oct 2, 2025
View reviewed changes
Copy link
Contributor

@CarliPinell CarliPinell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@vladyrichter
Copy link

vladyrichter commented Oct 4, 2025

QA server K8S was successfully deployed https://ci-faaa114067.engk8s.processmaker.net

@processmaker-sonarqube
Copy link

processmaker-sonarqube bot commented Oct 4, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CarliPinell CarliPinell CarliPinell approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@eiresendez @vladyrichter @CarliPinell