Closed (works as designed)
Project:
Chatroom
Version:
6.x-2.9
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Jul 2010 at 13:22 UTC
Updated:
8 Jul 2010 at 09:30 UTC
Jump to comment: Most recent
Comments
Comment #1
Anonymous (not verified) commentedjari - you need to set the variable in $conf in settings.php
$conf['chatroom_module_dir'] = 'path/to/your/chatroom/module/directory';
Comment #2
tormu commentedOk that works. I couldn't find any place where that trick would be mentioned..
Maybe add that to INSTALL.txt or even better, to the settings page in GUI. On the settings page having the possibility to set the variable without touching to settings.php would be great. drupal_get_path('module', 'chatroom') could be used to tell the user that the path should be set or not.
I guess the reason for not using the full bootstrap (so drupal_get_path() could be used) in chatroomread.php is a performance issue?
Comment #3
Anonymous (not verified) commentedjari, thanks for the suggestion, i've updated README.txt.
http://drupal.org/cvs?commit=389632
have a look at the code comment at the top of chatroomread.php. the file is a performance hack. it doesn't make chatroom module scale well, just enough for small rooms with tight integration into drupal.