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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

CarlHinton’s picture

CarlHinton CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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) CreditAttribution: Anonymous 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?