Skip to content

FOUR-29372: Show persisted multi-instance form data#8900

Open
eiresendez wants to merge 1 commit into
developfrom
bugfix/FOUR-29372
Open

FOUR-29372: Show persisted multi-instance form data#8900
eiresendez wants to merge 1 commit into
developfrom
bugfix/FOUR-29372

Conversation

@eiresendez

@eiresendez eiresendez commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

Multi-instance task forms in the Request Forms tab could render blank even when submitted form data existed in the database. This was reproduced using the imported test loop process: the details-screen-request API returned the submitted textarea values only under _parent, while the screen renderer expects field values at the root of data.

Solution

  • Include token_properties when loading request tokens for the details-screen-request API.
  • Preserve the existing status and element filters for the Forms tab endpoint.
  • Add a feature test covering multi-instance token data so persisted form values are returned at data.<field_name> while _parent remains available.

How to Test

  • Ran ./vendor/bin/phpunit --filter testScreenRequestedReturnsMultiInstanceTokenPropertiesData tests/Feature/Api/ProcessRequestsTest.php
  • Ran ./vendor/bin/phpunit --filter '/testScreenRequested$|testScreenRequestedReturnsMultiInstanceTokenPropertiesData/' tests/Feature/Api/ProcessRequestsTest.php
  • Manually verify and confirm completed multi-instance form rows with persisted textarea data render values in the Forms tab.

Related Tickets & Packages

ci:deploy

@cursor

cursor Bot commented Jul 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for this team, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@eiresendez eiresendez self-assigned this Jul 9, 2026
@processmaker-sonarqube

Copy link
Copy Markdown

@vladyrichter

Copy link
Copy Markdown

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

@CarliPinell
CarliPinell self-requested a review July 9, 2026 21:05

@CarliPinell CarliPinell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants