At the risk of annoying the author, I've reopened this and am attaching a patch to change the behavior of the latest release just slightly....

The current code requires that both the config.php file be writable and the gallery module directory be writable. I don't like that arrangement. If there is no config.php, then the module needs write permission on the directory in order to create one. But if there is one, then it only needs write permission on the file to update it, not on the directory. That's how I have my installation configured and IMHO that it is preferred to having the whole directory open....

The attached patch accomplishes this.

CommentFileSizeAuthor
gallery.config-permission.patch.txt862 bytesrbroberts

Comments

rbroberts’s picture

rbroberts commented
Title: Gallery g2image config.php permission warning when permissions are adequate » g2image image assist config.php permission warning when permissions are adequate

I didn't say that quite right in the original message. The permission check in on the g2image config.php in TinyMCE, but the principle is the same; only the file itself needs write permission, not the whole directory.

kiz_0987’s picture

kiz_0987 commented
Status: Active » Fixed

Committed to HEAD.

Anonymous’s picture

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