jmstacey and I discovered that the current implementation of mkdir() for stream wrapper that is in core fails when you try recursive directory creation - hence the File APi conversion cannot work without this quick fix.

CommentFileSizeAuthor
#3 recursive-fix-547412-3.patch1.04 KBpwolanin
recursive-fix.patch925 bytespwolanin

Comments

jmstacey’s picture

jmstacey commented
Status: Needs review » Reviewed & tested by the community

Looks good.

pwolanin’s picture

pwolanin commented

by which jmstacey means he tested it and it fixes the bug :)

pwolanin’s picture

pwolanin commented
StatusFileSize
new recursive-fix-547412-3.patch1.04 KB

minor changes suggested by Dries

pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Closed (duplicate)

actually, this is not an issue yet, since the current core implementation of getLocalPath() doesn't have the realpath() check

this fix will jsut be included in http://drupal.org/node/517814