After Mollom has been enabled and configured, I noticed the admin/user/captcha page was blank. Then, I started noticing the following message:

warning: Missing argument 2 for mollom_captcha() in mollom.module on line 113.

I thought it might be a conflict with reCAPTCHA, so I disabled it, but that didn't help.

Comments

Gurpartap Singh’s picture

It's likely a problem with your session IDs.

wim leers’s picture

Category: bug » support
Status: Active » Closed (fixed)

Closing due to lack of response. Feel free to reopen!

BioALIEN’s picture

Version: 6.x-1.1 » 6.x-1.x-dev
Category: support » bug
Status: Closed (fixed) » Active

This is still valid, and I just came across this in my logs. Here are my steps which may help reproduce:
1. Enabled captcha module
2. Configured captcha to protect registration & forgot password forms.
3. Enabled mollom module
4. Configured mollom to protect registration & forgot password forms.
5. At this point visiting captcha module's settings via admin resulted in WSOD.
6. I proceeded to the module page and deactivated captcha module while leaving mollom enabled.
7. I may have visited mollom module's settings via admin after the above step
8. Visiting admin/reports/dblog shows mollom throwing the php error above.

So the original poster was spot on in his observation except he was using reCAPTCHA. Maybe mollom doesn't play nice with other captcha modules?

errement’s picture

Assigned: Unassigned » errement

I had the same problem getting warning: Missing argument 2 for mollom_captcha() and my solution for it was:

-Uninstall Mollom module

Just this!!
And all went well again.

It's a pitty getting this problem as i also like very much the reCaptcha being able to help digitize books!!
I think both modules are not compatible if both are doing same comands.

errement’s picture

Assigned: errement » Unassigned
LeonidShamis’s picture

Version: 6.x-1.x-dev » 6.x-1.3

Same problem here:
Missing argument 2 for mollom_captcha() in .../sites/all/modules/contrib/mollom/mollom.module on line 113.
My configuration:
drupal-6.4
captcha-6.x-1.0-rc2.tar.gz
recaptcha-6.x-1.0.tar.gz (with recaptcha-php-1.10.zip)
mollom-6.x-1.3.tar.gz
Just installed and enabled all modules and attempted to access admin/user/captcha to configure CAPTCHA/reCAPTCHA.
Mollom configuration was successful though and I could set Mollom to protect new User Registration form - Mollom was inserting its own captcha.
I'm afraid I have to follow the advice above and... uninstall Mollom.

LeonidShamis’s picture

Just confirming: uninstalling Mollom resolves the issue and allows access to CAPTCHA configuration at admin/user/captcha

Yura’s picture

Disable of the mollum module is dissolving the problem. But I want to use mollom too!?

m3avrck’s picture

Status: Active » Needs review

Having the same problem.

I have the latest Captcha and Mollom 6.1.3 installed.

warning: Missing argument 2 for mollom_captcha() in /home/gntvejaa/web/public/sites/all/modules/contrib/mollom/mollom.module on line 113.

Error only happens admin/user/captcha the recaptcha page is fine.

The problem is because captcha module implements:

Default implementation of hook_captcha()

The easy fix is for Mollom to rename molllom_captcha() to something else.

I propose the following patch:

Index: mollom.module
===================================================================
--- mollom.module	(revision 13)
+++ mollom.module	(working copy)
@@ -91,7 +91,7 @@
   // A menu callback that is used for AJAX purposes:
   $items['mollom/captcha/%/%'] = array(
     'title' => 'Request CAPTCHA',
-    'page callback' => 'mollom_captcha',
+    'page callback' => 'mollom_get_captcha',
     'page arguments' => array(2, 3),
     'access callback' => TRUE,
     'type' => MENU_CALLBACK,
@@ -110,7 +110,7 @@
 /**
  * An AJAX callback to retrieve a CAPTCHA.
  */
-function mollom_captcha($type, $session_id) {
+function mollom_get_captcha($type, $session_id) {
   
   // TODO: add error handling
   $output = '';
errement’s picture

@m3avrck,

First of all, thanks for the patch!!

But i think that the decision on renaming molllom_captcha() to something else, does compete to Dries —or this is what i think—

What do you —community— think?

BioALIEN’s picture

@errement, there's a clear conflict between the two modules. Since captcha.module has been around for longer, mollom IMO has to give!

errement’s picture

@BioALIEN,

Thanq for your clear explanation, i indeed confirm what you said about the conflict here!

From the beginning i was sure the conflct existed but i also think that both modules should not work together as they actually do same thing, but in different ways.

Personally i use mollom as i don't have to obligate —but trust— my site users entering a captcha everytime they post, only no-human ones —SPAM— should do so. For me this is first!

Meanwhile, i do like the idea —as i said before— of reCaptcha being able to help digitize books. But, one should decide which is better to use on his projects.

I appreciate your concern explaning me that.

Thx again,
;-))

N.B.: Maybe the solution for this issue is very easy, but this is something between wundo & dries, that i won't discuss.

dave reid’s picture

Status: Needs review » Fixed

This has been fixed (the function has been renamed to mollom_captcha_js). Please upgrade to the latest version (6.x-1.7).

Status: Fixed » Closed (fixed)

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