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

Issue fork flysystem-3204882

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

SpadXIII created an issue. See original summary.

SpadXIII opened merge request !2

spadxiii’s picture

spadxiii commented
Status: Active » Needs review
spadxiii’s picture

spadxiii commented
Version: 8.x-1.x-dev » 2.0.x-dev

just a quick note: patches still apply to 2.0

spadxiii’s picture

spadxiii commented
Version: 2.0.x-dev » 2.1.x-dev

just a quick note: patches still apply to 2.1

lisa.rae’s picture

lisa.rae commented

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.

  • lhridley committed 87c4d265 on 2.1.x 
    Issue #3204882 by SpadXIII: hook_file_download triggers warnings for non...
lisa.rae’s picture

lisa.rae commented
Status: Needs review » Fixed
lisa.rae’s picture

lisa.rae commented
Status: Fixed » Closed (fixed)