As I see in image.module, the Image modules defines its temp folder inside the image folder.

Personally I would prefer that the temp folder is the one defined in Drupal's settings (returned by file_directory_temp). This way, I can make the image folder available to the public, while I put the temp path outside the htdocs folder (an certainly outside the images folder).

I've attached a diff with the three changes I've made.

Andrei.

CommentFileSizeAuthor
#1 diff.txt306 bytesabautu
diff.txt302 bytesabautu

Comments

abautu’s picture

StatusFileSize
new306 bytes

I forgot a slash. Here it is.

drewish’s picture

definitely won't be changed in the 1.x branch. 2.x is where the new features are going. but i don't think it'll work to use the core temp directory since we need to serve files out of it for previews.

drewish’s picture

Version: 5.x-1.6 » 5.x-2.x-dev
Status: Needs review » Needs work
sun’s picture

Category: bug » feature
Status: Needs work » Closed (won't fix)

Using an own temp directory instead of Drupal's default actually was a nice trick. We should bear in mind image previews when working on #232129: Merge Image with Imagefield/cache in 2.x.