Download & Extend

Check private files variable 'file_private_path' correctly

Project:Drush
Component:Config
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

If we save 'file_private_path' variable on 'admin/config/media/file-system' page as empty string, private file system becomes unavailable in Drupal: http://drupalcode.org/project/drupal.git/blob/b9127101ffeca819e74a03fa9f...
The same should be done in Drush.
Initial patch for branch 7.x-5.x attached.

Why I decided to fix it:
drush perms command (#990812: Add a "permissions" subcommand to fix/set all file permissions) takes private path '' (if it is an empty string), convert to absolute path and sets permissions for whole drupal directory as for private folder. Not so cool.

Thanks!

AttachmentSize
drush-status-private-files.patch640 bytes

Comments

#1

Status:needs review» reviewed & tested by the community

I agree that this should be committed; the Drush permissions command should also be fixed to ignore empty relative paths.

#2

Status:reviewed & tested by the community» fixed

Committed. Thanks.

#3

Backported.

#4

Status:fixed» closed (fixed)

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