Lost a few minutes since I didn't check the permissions for the copied chatroomread.php file; let's add an indication to the README.txt to avoid the hassle to others :).

CommentFileSizeAuthor
README_permissions.patch666 bytesjedihe

Comments

carlhinton’s picture

carlhinton commented

Would it not be better to get the code to work out that it does not sit in sites/all/modules/chatroom

drupal_get_path ('module', 'chatrooms')

should do the trick

dcaillibaud’s picture

dcaillibaud commented

+1

And in this way, drupal_get_path ('module', 'chatroom') should be much better than $conf['chatroom_module_dir'] so that this configuration variable should be useless.

socialnicheguru’s picture

socialnicheguru commented

i just tried it and using drupal_get_path works.

was there a need for the variable? is chatroom called before drupal_get_path is defined?

Anonymous’s picture

Anonymous (not verified) commented
Version: 6.x-2.13 » 6.x-2.x-dev

i like the original patch.

not sure i follow the other suggestions - can one of you elaborate?