A bug ? Missing argument for variable_get()

boriso - June 4, 2008 - 05:39
Project:MeBeam Chat
Version:5.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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.

#1

regx - June 6, 2008 - 06:34

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.

#2

boriso - June 7, 2008 - 13:40

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.

#3

regx - June 26, 2008 - 08:06

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.

#4

regx - June 30, 2008 - 07:15
Status:active» fixed

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

#5

Anonymous (not verified) - July 14, 2008 - 07:22
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.