if possible, we should reuse drupal's usual tmp directory instead of declaring our own. we can use a subdir there. for example, one should not see this message: sites/default/modules/chatroom/chat_cache directory is not writable by php

see file.inc, or the file system section of admin/settings page.

CommentFileSizeAuthor
#6 chatroom.install.patch1.12 KBdarren oh

Comments

Anonymous’s picture

Anonymous (not verified) commented
Assigned: Unassigned »

sounds like a good suggestion - i'll try and code this up tomorrow evening sydney time.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

hi moshe,

good suggestion - committed fix for this.

cheers
justin

moshe weitzman’s picture

moshe weitzman commented
Status: Fixed » Active

i think we should just create these tmp dirs instead of asking user to do so via an error msg

Anonymous’s picture

Anonymous (not verified) commented
Assigned: » Unassigned
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Assigned: Unassigned » darren oh
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Active » Fixed
StatusFileSize
new chatroom.install.patch1.12 KB

Fixed in CVS.

Anonymous’s picture

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