When creating directories and files, the permissions need to be set so that the web server can read them. This is handled properly when creating new directories, but not when creating new files.

I think that the proper place to do this is in imageapi_image_close, rather than hook_image_close, because the file permissions issue seems more like a generic web server issue, than a specific toolkit issue. If you do it there, then it only needs to be done in one place, than but maybe you have other ideas.

Patch is attached.

CommentFileSizeAuthor
#1 236857.patch1.44 KBdouggreen
chmod.patch1.45 KBdouggreen

Comments

douggreen’s picture

douggreen commented
StatusFileSize
new 236857.patch1.44 KB

doh, try this one instead, it changes the perms of the destination file and NOT the source file.

dopry’s picture

dopry commented
Status: Active » Closed (fixed)

committed to HEAD.

dopry’s picture

dopry commented
Status: Closed (fixed) » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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