Skip to content

FOUR-32224: Prevent Web Entry temporary file download errors#1918

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

FOUR-32224: Prevent Web Entry temporary file download errors#1918
eiresendez wants to merge 1 commit into
developfrom
bugfix/FOUR-32224

Conversation

@eiresendez

@eiresendez eiresendez commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

In an intermediate Web Entry task, configure File Upload, File Preview, and File Download controls to use the same variable. Open the Web Entry and upload a file. File Preview loads successfully, but File Download treats the temporary webentry_* identifier as a persisted Request Media ID, calls the persisted-file endpoint, receives a 404, and displays an error alert.

Solution

  • Detect temporary Web Entry file identifiers before requesting persisted file metadata.
  • Clear stale File Download metadata and skip the invalid persisted-file request for webentry_* values.
  • Preserve the existing behavior for persisted Request Media IDs.
  • Add unit coverage for temporary, persisted, empty, and unsupported identifier values.

Downloading temporary files before submission remains outside the scope of this fix.

How to Test

  1. Run npx jest tests/unit/FileDownloadUtils.spec.js --no-coverage --runInBand.
  2. Run npx eslint src/components/renderer/file-download.vue tests/unit/FileDownloadUtils.spec.js.
  3. Run npm run build.
  4. Configure an intermediate Web Entry with File Upload, File Preview, and File Download controls using the same variable.
  5. Upload a file and verify that File Preview loads, no error alert appears, and no persisted-file request containing a webentry_* identifier is made.
  6. Verify that File Download shows no available files until the upload is persisted.
  7. Verify that an existing persisted Request file remains downloadable.

Manual local verification was completed successfully.

Related Tickets & Packages

ci:deploy

@eiresendez eiresendez self-assigned this Jul 15, 2026
@eiresendez

Copy link
Copy Markdown
Contributor Author

Solution:

Screen.Recording.2026-07-15.at.11.38.02.a.m.mov

@cypress

cypress Bot commented Jul 15, 2026

Copy link
Copy Markdown

screen-builder    Run #2295

Run Properties:  status check passed Passed #2295  •  git commit cdb37a372b: FOUR-32224: Prevent Web Entry temporary file download errors
Project screen-builder
Branch Review bugfix/FOUR-32224
Run status status check passed Passed #2295
Run duration 09m 51s
Commit git commit cdb37a372b: FOUR-32224: Prevent Web Entry temporary file download errors
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 ↗︎

@processmaker-sonarqube

Copy link
Copy Markdown

@vladyrichter

Copy link
Copy Markdown

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

@eiresendez
eiresendez requested a review from mcraeteisha July 17, 2026 18:04
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.

3 participants