I am unable to use the imap_auth module. After installing I get:

warning: Missing argument 3 for imap_auth_form_alter() in /var/www/www.204productions.com/drupal-5.1/sites/all/modules/imap_auth/imap_auth.... on line 66.

on each page.
The server runs php 4.3.10-22 and apache2

Comments

bigcrunch’s picture

it happens to me the same

bigcrunch’s picture

Category: support » bug

I thinks it's a bug, not a support request.

mrf’s picture

I am also seeing the same thing. I recently moved the site to a new testing server, and began seeing the error.

kevingolde’s picture

This error will go away by changing argument 3 to have a default value if it is missing.
That is, on line 66 we have:

function imap_auth_form_alter(&$form, $form_state, $form_id) {

change it to:

function imap_auth_form_alter(&$form, $form_state, $form_id='') {

jpmoo’s picture

Priority: Normal » Critical
StatusFileSize
new52.26 KB

Implemented kevingolde's fix eliminates the error, but I do not see any administrative page under Administer > Site configuration. Going directly to "admin/settings/imap_auth" brings up the attached. We're running in IIS on Windows 2003. PHP IMAP functions are installed and working correctly in other (non-Drupal) applications.

wildturkey’s picture

I have the same issue with the missing site conf page same as jpmoo, on 2 servers apache2, php4, mysql3 and apache2.2, php5 and postgres8.1. Both of these machine have php imap installed and have working copies of a php web mail program

jpmoo’s picture

Deactivated, uninstalled and then reinstalled the module to see if that would help. No change. Still no settings page.

ij’s picture

Same error here with Drupal 5.3, imap_auth 5.x-1.x-dev and postgresql 8.1 on Debian Etch.

oadaeh’s picture

Title: Missing argument 3 for imap_auth_form_alter() » Module no longer compatible with Drupal 5

The problem is that the 5.x version is being pulled from HEAD [0] and on September 2 yecarrillo started converting HEAD to be compatible with Drupal version 6 [1].

As far as I am concerned, this module is no longer compatible with Drupal 5.

[0] http://drupal.org/node/110510
[1] http://drupal.org/cvs?commit=79823

owen barton’s picture

StatusFileSize
new6.44 KB

Here is how to create a DRUPAL-5 version from a HEAD CVS checkout:

cvs up -r1.5 imap_auth.module
cvs up -r1.2 imap_auth.info

This needs to be fixed by someone with cvs commit access to the project. The simplest way would be to do the above, and then commit as a DRUPAL-5 branch.

Attached is a tarball which should hopefully work for non-techy folks in the meantime.

owen barton’s picture

Assigned: wildturkey » Unassigned
Status: Active » Needs work

The 'work' in this case being one of the project devs committing the DRUPAL-5 branch ;)

owen barton’s picture

StatusFileSize
new5.81 KB

Here is an updated tarball that fixes a bug in the above one, and also includes the patch from http://drupal.org/node/211288

davisben’s picture

Status: Needs work » Closed (won't fix)

The 5.x-1.x branch is no longer supported, cleaning up the issue queue.