Hi,

I'm using the latest CVS update, with the file engine. In the create_directory() function the line

            @chmod($directory, 0775); // Necessary for non-webserver users.

Should read 

            @chmod($dir, 0775); // Necessary for non-webserver users.

Thank you :)

CommentFileSizeAuthor
#3 file.php_.patch3.47 KBflexer
#2 file.php_.patch2.47 KBflexer

Comments

flexer’s picture

flexer commented

Btw, I think that the chmod 775 is a *good* idea (I was going to add the feature myself, when I discovered the bug), but what do you think about also creating the single files with that mask?

flexer’s picture

flexer commented
StatusFileSize
new file.php_.patch2.47 KB

I added some minor tweaks (some @s) and cooked a patch that fixes this issue and the other I filed http://drupal.org/node/354554

Hope it will be applied :)

Thank you

flexer’s picture

flexer commented
StatusFileSize
new file.php_.patch3.47 KB

I've found another bug in the file engine, when a delete('*') is issued.

(cumulative) Patch attached

Bye :)

slantview’s picture

slantview commented
Status: Active » Fixed

Applied to all 3 branches. Will be in next release. Thanks so much.

Status: Fixed » Closed (fixed)

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