Drupal 5.1
Configuration file Protected
Cron maintenance tasks Last run 21 min 33 sec ago
You can run cron manually.
Database schema Up to date
File system Writable (public download method)
GD library bundled (2.0.28 compatible)
MySQL database 5.0.27
PHP 5.2.0
Unicode library PHP Mbstring Extension
Web server Apache/2.2.3 (Win32) DAV/2 mod_ssl/2.2.3 OpenSSL/0.9.8d mod_autoindex_color PHP/5.2.0

I'm using the SMTP module and I'm trying this out with a google groups list.

I'm getting this error:

warning: mail() [function.mail]: Failed to connect to mailserver at "localhost" port 25, verify your "SMTP" and "smtp_port" setting in php.ini or use ini_set() in D:\Server\xampp\htdocs\drupal\sites\devsheffuncen.ath.cx\modules\mailman_manager\mailman_manager.module on line 456.

My settings are correct for my e-mail server in the SMTP module which is using a gmail server using port 465.

if there's a minor fix like editing a line somewhere just let me know and I can go ahead and do that. I'm not able to use patches. :)

Have a good day!

CommentFileSizeAuthor
#5 mailman_manager_patch_drupal_mail_rev.patch2.48 KBarturoramos
#4 mailman_manager_patch_drupal_mail.patch2.29 KBarturoramos

Comments

arturoramos’s picture

arturoramos commented
Assigned: Unassigned » arturoramos
Status: Active » Postponed (maintainer needs more info)

I am not familiar with the SMTP module, but this module uses the default PHP mail() function:

http://us2.php.net/mail

From what you are explaining you have another module that somehow is acting as an SMTP server. You would need to alter line 456 of mailman_manager.module to call the appropriate hook in your SMTP module instead of using the PHP mail() function:

mail($commandaddress,'',$command,$headers);

There are several other places in the module where the php mail() function is used as well that would also need to be changed.

Miss Essence’s picture

Miss Essence commented

Crud.

PHP Mail is blocked by my ISP so that's a no go. Plus I don't know enough about the SMTP module myself to go in and edit yours that much to make it work with it.

LukeLast’s picture

LukeLast commented

This is not the first module to suffer from this problem. The issue is using the php mail() function instead of the drupal_mail() function.

http://api.drupal.org/api/HEAD/function/drupal_mail

If you replace the php mail() function with the drupal_mail() function and add the additional arguments it should work.

arturoramos’s picture

arturoramos commented
StatusFileSize
new mailman_manager_patch_drupal_mail.patch2.29 KB

I have made what I believe are the necessary changes to use the drupal_mail() function instead of the php mail() function. I would appreciate someone testing for me before incorporation into the next release.

arturoramos’s picture

arturoramos commented
StatusFileSize
new mailman_manager_patch_drupal_mail_rev.patch2.48 KB

Revision to patch... please ignore previously posted patch for testing

arturoramos’s picture

arturoramos commented
Status: Postponed (maintainer needs more info) » Needs review

Changing status of bug request to patched... needs review.

Miss Essence’s picture

Miss Essence commented

As much as I'd love to see if this works. I've tried the "patching" before with other stuff and it never quite worked for me. Maybe I'm too dense *grin*.

Is there another way to test the changes that need made? I wouldn't mind doing it manually if I could understand what needed changing where.

Miss Essence’s picture

Miss Essence commented

* warning: array_merge() [function.array-merge]: Argument #2 is not an array in D:\Server\xampp\htdocs\drupal\includes\common.inc on line 1929.
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in D:\Server\xampp\htdocs\drupal\includes\common.inc on line 1929.

Made the changes in the patch file and added two lists. Attempted to join and got those two errors.

Miss Essence’s picture

Miss Essence commented

Had a non-admin member go to the mailing list edit page.

They saw:
warning: array_merge() [function.array-merge]: Argument #2 is not an array in D:\Server\xampp\htdocs\drupal\includes\common.inc on line 1929.
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in D:\Server\xampp\htdocs\drupal\includes\common.inc on line 1929.
* warning: array_merge_recursive() [function.array-merge-recursive]: Argument #1 is not an array in D:\Server\xampp\htdocs\drupal\sites\all\modules\user\user.module on line 2435.
* warning: uasort() [function.uasort]: The argument should be an array in D:\Server\xampp\htdocs\drupal\sites\all\modules\user\user.module on line 2438

Miss Essence’s picture

Miss Essence commented

Said member -only- sees warnings and nothing else on the mailing lists page.

arturoramos’s picture

arturoramos commented
Status: Needs review » Fixed

Errors had to do with admin address not being set and the module sending a non-arrayed blank variable to the drupal_mail() function. This has been resolved and revised patch has been integrated into next release.

Anonymous’s picture

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