Hi !

I just installed the MeBeam module on a test site. As soon as I try to set up the MeBeam settings on Drupal, I get the following warning message :

* warning: Missing argument 2 for variable_get(), called in /home/libr5009/public_html/fenestro.net/modules/contrib/mebeam_chat/mebeam_chat.module on line 186 and defined in /home/libr5009/public_html/fenestro.net/includes/bootstrap.inc on line 412.
* warning: Missing argument 2 for variable_get(), called in /home/libr5009/public_html/fenestro.net/modules/contrib/mebeam_chat/mebeam_chat.module on line 186 and defined in /home/libr5009/public_html/fenestro.net/includes/bootstrap.inc on line 412.

Also, when I try to create a room with a random name, leaving the "room name" field empty, with the randomize box checked, as suggested I get the following message :

* warning: Missing argument 2 for variable_get(), called in /home/libr5009/public_html/fenestro.net/modules/contrib/mebeam_chat/mebeam_chat.module on line 186 and defined in /home/libr5009/public_html/fenestro.net/includes/bootstrap.inc on line 412.
* MeBeam Room Name must be more than 4 characters.

I am not sure how critical these issues are. Just thought of letting you know. I use Drupal 5.7. The only other module I installed is webcams.

Comments

regx’s picture

regx commented

I probably broke this when I tried to make it auto-install using sensible defaults without any user setup.

Were you able to get it working? If not I will simplify the install and make it require setup.

boriso’s picture

boriso commented

Even after installing, I still get the following message when I enter Mebeam chat on my drupal site :

warning: Missing argument 2 for variable_get(), called in /home/libr5009/public_html/fenestro.net/modules/contrib/mebeam_chat/mebeam_chat.module on line 114 and defined in /home/libr5009/public_html/fenestro.net/includes/bootstrap.inc on line 412.

It does seem that the module is working correctly though (I can get into a mebeam room with my cam), but I can't be 100% sure that I am not missing any functionality. I haven't tested interacting with other users yet.

regx’s picture

regx commented

The second arg to variable_get is the default value.
I will fix this as soon as I get a chance, but it should not affect functionality.
Simply adding any default value to the variable_get call on line 412 will fix the error.

regx’s picture

regx commented
Status: Active » Fixed

This has been fixed.
Just do a CVS update, or grab the latest cvs version.

Anonymous’s picture

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

Automatically closed -- issue fixed for two weeks with no activity.