hook_file_download triggers warnings for non-existing files

Open on Drupal.org →

Created on 22 March 2021, about 4 years ago

Updated 3 August 2023, over 1 year ago

Problem/Motivation

The hook_file_download triggers warnings for non-existing files.

Steps to reproduce

Configure flysystem to serve files. Upload a file to drupal and delete it manually. Try to download the file through drupal.

Proposed resolution

Add a file_exists check in flysystem_file_download.

Remaining tasks

none

User interface changes

none

API changes

none

Data model changes

none

🐛 Bug report

Status

Fixed

Version

2.1

Component

Code

Created by

🇳🇱Netherlands spadxiii

Live updates comments and jobs are added and updated live.

Incomplete comments

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment almost 2 years ago →
🇳🇱Netherlands spadxiii
just a quick note: patches still apply to 2.1
Comment almost 2 years ago →
🇺🇸United States lhridley
Please change MR to merge into branch 2.1.x, and I'll test to see if I can duplicate based on your sparse testing instructions above.

Comment over 1 year ago →

lhridley → committed 87c4d265 on 2.1.x

Issue #3204882 by SpadXIII: hook_file_download triggers warnings for non...

Status changed to Fixed over 1 year ago7:38pm 30 July 2023
Comment over 1 year ago →
🇺🇸United States lhridley
Status changed to Fixed over 1 year ago11:35pm 3 August 2023
Comment over 1 year ago →
🇺🇸United States lhridley

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024