Problem/Motivation

If a file is uploaded to the root of a stream, such as public or private, with no sub-directory, an extra / is added after the stream wrapper. In the case of private files Drupal then fails to serve them.

Steps to reproduce

* Create a file field with no subdirectory (ideally use private file scheme)
* Upload a file that requires transliteration
* File is stored with a uri private:///
* File gives 403 as it can't be found

Proposed resolution

* Check if the directory already ends in /
* Alternative: Check if the directory is a stream wrapper

Remaining tasks

* MR opened already
* Confirm approach
* Testing

Issue fork filename_transliteration-3370400

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:

  • 3370400-file-move-doesnt Comparechanges, plain diff MR !2
  • 1.1.x Comparecompare
  • 1.0.x Comparecompare
About issue forks

Comments

Alexj12 created an issue. See original summary.

Alexj12 opened merge request !2

alexj12’s picture

alexj12 commented
Issue summary: View changes
alexj12’s picture

alexj12 commented
Status: Active » Needs review

jwilson3 made their first commit to this issue’s fork.

  • jwilson3 committed 5c86f4f1 on 1.1.x authored by Alexj12 
    Issue #3370400: File move doesn't take into account if the file is in...
jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

Merged. This will be included in the final release 1.2.0

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.