processmaker icon indicating copy to clipboard operation
processmaker copied to clipboard

FOUR-5118: Issue - Save search - CreditAccess dev server (for 4.1.develop)

Open danloa opened this issue 3 years ago • 2 comments

Issue & Reproduction Steps

When there are 1000+ requests and in every one of the the "data" json field contains a very large attribute, the endpoint requests returns a 500 error. This is noticeable when accessing the site remotely. In local dev. environments the problem is not as critical as in the remote access. Saved search uses the controller of this endpoint (/requests)

Solution

  • To avoid that Laravel parses the larga data column of tokens and requests, the queries used to filter by request starter and participant were modified to get just ids.

How to Test

Verify that the list requests (endpoint /requests) behavior has not changed.

Related Tickets & Packages

https://processmaker.atlassian.net/browse/FOUR-5118

Code Review Checklist

  • [x] I have pulled this code locally and tested it on my instance, along with any associated packages.
  • [x] 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.
  • [x] This solution fixes the bug reported in the original ticket.
  • [x] This solution does not alter the expected output of a component in a way that would break existing Processes.
  • [x] This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • [x] This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • [x] This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • [x] This ticket conforms to the PRD associated with this part of ProcessMaker.

danloa avatar Jan 28 '22 15:01 danloa

QA server was successfully deployed https://bugfix-four-5118-4-1-develop-processmaker.ci.processmaker.net

pmkerberos avatar Jan 28 '22 15:01 pmkerberos

QA server was successfully deployed https://bugfix-four-5118-4-1-develop-processmaker.ci.processmaker.net

pmkerberos avatar Jan 28 '22 18:01 pmkerberos