Bugfix/FOUR-32292-b: Validation of required fields does not work correctly in Record List controls with checkboxes

[CarliPinell]

Solution

Problems addressed

• Required Record Form fields were not validated correctly on parent Submit (values live in list rows, not at root).
• Add/Edit modal renderers polluted global valid/submitted state, so empty modal defaults could block parent Submit.
• Error message double-counted the same controls (vdata + schema / vocabularies → e.g. 4 fields → “8 validation errors”).
• PageNavigate on the Record Form page (e.g. “back to page 0”) pulled the main page into Record List row rules, creating ghost listName__* keys and inflating counts further (e.g. 10 vs 6).

Expected behavior after fix

• N required checkboxes (root + Record Form) → about N errors when invalid (not 2×N).
• Screens with PageNavigate on the Record Form page no longer inflate the count.
• Modal Add/Edit still validates locally; parent Submit validates list rows correctly.

How to Test

Follow steps detailed in ticket

Related Tickets & Packages

• https://processmaker.atlassian.net/browse/FOUR-32292

Code Review Checklist

• I have pulled this code locally and tested it on my instance, along with any associated packages.
• This code adheres to ProcessMaker Coding Guidelines.
• This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
• This solution fixes the bug reported in the original ticket.
• This solution does not alter the expected output of a component in a way that would break existing Processes.
• This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
• This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
• This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
• This ticket conforms to the PRD associated with this part of ProcessMaker.

ci:deploy

[cypress]

screen-builder    Run #2511

Run Properties:   Passed #2511  •  35e7407d6d: Bugfix/FOUR-32292-b: Validation of required fields does not work correctly in Re...

Project	screen-builder
Branch Review	bugfix/FOUR-32292-b
Run status	Passed #2511
Run duration	12m 17s
Commit	35e7407d6d: Bugfix/FOUR-32292-b: Validation of required fields does not work correctly in Re...
Committer	CarliPinell
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	19
Skipped	0
Passing	393
View all changes introduced in this branch ↗︎

[nolanpro]

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

[nolanpro]

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

[eiresendez]

Manual testing confirms that the reported scenario is fixed 👍 I'm just leaving a few observations:

1. Remove first-row-dependent conditional validation
2. Clarify empty-list semantics

[nolanpro]

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

[eiresendez]

hi @CarliPinell thanks for fixing the previous issue, the new commit introduced this new observation!

[nolanpro]

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

[eiresendez]

There are no remaining code-review blockers based on the diff and manual QA 👍👍

[nolanpro]

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