On ?q=admin/user/configure it tries to create the directory %files%/pictures even when I have picture support disabled. Surely this folder should only be created when I have the picture support enabled?

Error in full:

warning: mkdir(files/xxxxxx/pictures): Permission denied in /var/www/html/drupal-4.5.2/includes/file.inc on line 77.

CommentFileSizeAuthor
#2 user_create_image_dir.patch1.35 KBtostinni
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Uwe Hermann’s picture

Version: » x.y.z
Priority: Normal » Minor

Still applies to HEAD.

tostinni’s picture

FileSize
1.35 KB

The path in HEAD is admin/settings/user
Here is a little patch that check first if picture support is enable and then create the path.

Is it so important to check this ?

And I think that your files directory should get write permissions, this is a normal behaviour of Drupal to store its files.

Uwe Hermann’s picture

Status: Active » Reviewed & tested by the community

+1. It's a nice usability improvement, applies cleanly to HEAD, and works as expected.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Anonymous’s picture

Anonymous’s picture

Status: Fixed » Closed (fixed)