Screenshots broken when unused files set to temporary

Created on 1 April 2025, 1 day ago

Problem/Motivation

As per https://www.drupal.org/node/2891902 → there is an option to maintain legacy Drupal behavior that automatically sets unused files to temporary and deletes them on cron by setting $config['file.settings']['make_unused_managed_files_temporary'] = TRUE; With this setting, trying to save a screenshot using website_feedback results in the following error:
"The file used in the Screenshot field may not be referenced."

Steps to reproduce

Install a vanilla Drupal 10 or 11 site.
Install website_feedback, and configure to allow screenshots
Add the line to settings.php: $config['file.settings']['make_unused_managed_files_temporary'] = TRUE;
Attempt to save feedback with a screenshot

Proposed resolution

Manually add a usage to the usage table after saving the screenshot before submitting the feedback form so that the file may be referenced.

🐛 Bug report

Status

Active

Version

1.1

Component

Code

Created by

🇺🇸United States dgroene

Live updates comments and jobs are added and updated live.

Merge Requests

!10Screenshots broken when unused files set to temporary
Open
🇺🇸United States dgroene
updated 1 day ago

Comments & Activities

Issue created by @dgroene
Merge request !10fix: WF-3516719: Fixes issue of screenshot file not saving when set to permanent with no usages → (Open) created by dgroene
Comment 1 day ago →
🇺🇸United States dgroene
I have created a merge request. The issue I think is in the screenshotWidget... the newly created file is being set to permanent (with no usages), but it should initially be set to temporary, as it will get assigned as permanent (with the correct usage) when the parent entity gets saved. I have confirmed that anonymous users have no issues saving screenshots with this change.
Comment about 5 hours ago →
🇺🇸United States dgroene

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024