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!

CommentFileSizeAuthor
drush-status-private-files.patch640 byteskalabro
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greg.1.anderson’s picture

greg.1.anderson CreditAttribution: greg.1.anderson commented
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.

greg.1.anderson’s picture

greg.1.anderson CreditAttribution: greg.1.anderson commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks.

jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
CreditAttribution: jonhattan commented

Backported.

Status: Fixed » Closed (fixed)

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