Skip to content

Comments

FOUR-29448: The PMQL to filter in select list control is not working#8738

Open
eiresendez wants to merge 1 commit intodevelopfrom
bugfix/FOUR-29448
Open

FOUR-29448: The PMQL to filter in select list control is not working#8738
eiresendez wants to merge 1 commit intodevelopfrom
bugfix/FOUR-29448

Conversation

@eiresendez
Copy link
Contributor

@eiresendez eiresendez commented Feb 20, 2026

Issue & Reproduction Steps

PMQL in Screen Builder Select List was not applied when using a Data Connector endpoint with params: [].
The request returned all records because dynamic PARAM values (like pmql) were not added to the query string in that case.

Solution

  • Updated MakeHttpRequests.php to preserve legacy behavior for configured endpoint params, and also allow dynamic query params when endpoint params is missing or empty.
  • Prevented overwriting already-defined query keys.
  • Added regression tests in MakeHttpRequestTest.php:
    testRequestConstructionAddsDynamicParamsWhenEndpointParamsAreEmpty
    testRequestConstructionWithEmptyEndpointParamsDoesNotOverwriteExistingQueryParams

How to Test

  • Confirm the Select List is filtered by PMQL in Preview mode.

Related Tickets & Packages

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.

@processmaker-sonarqube
Copy link

@CarliPinell CarliPinell self-requested a review February 21, 2026 01:53
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.

2 participants