Good Day,

enabled the module and configured the settings when I save the settings I get the following messages

PHPMailer is currently disabled.
Unable to send email
The test configuration has been save. A test e-mail has been sent

Having a hard time to get this going any advice?

Thank you,

Mark

CommentFileSizeAuthor
#4 D7-not-sending-mail-1018226.patch16.58 KBMichael Zetterberg fd. Lopez
phpmail3.png14.04 KBmrgoltra
phpmail2.png18.19 KBmrgoltra
phpmail1.png35.55 KBmrgoltra

Comments

pluess’s picture

pluess commented

Same Problem here.

It doesn't happen all the time. But setting / unsetting the enable check box several times makes the gui and the sttings getting out of sync. Email Sending is not working relyably with the test mail function 9 out of 10 times it does not work.

mrgoltra’s picture

mrgoltra commented

I decided to use SMTP module instead. I couldn't get phpmailer to work. Double checked the settings.

Michael Zetterberg fd. Lopez’s picture

Michael Zetterberg fd. Lopez commented

The 7.x-3.x-dev (from today june 3) seems to still use the smtp_library variable, which only drupal_mail_send seems to use. D7's drupal_mail_system does not, so currently the 7.x-3.x-dev version won't work (default mail will be used).
Hopefully I can get a patch going if I get some time this weekend.

Michael Zetterberg fd. Lopez’s picture

Michael Zetterberg fd. Lopez commented
Component: Miscellaneous » Code
Category: support » bug
Priority: Normal » Major
Status: Active » Needs review
StatusFileSize
new D7-not-sending-mail-1018226.patch16.58 KB

Here is a stab at a patch. I only tested with my local ISP's outgoing mailserver and it works well. Mail gets sent (and received).
I cleaned out some warnings I got too.

I'm new to Git so hopefully I did it right. If not please let me know so I can improve :)

I based the patch on master, I guess that is 7.x-3.x-dev, following the guidelines here http://drupal.org/node/1054616 ?

Bumped to major because mail not getting sent for a module that sends mail seems bad :)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Closed (duplicate)

Thanks for taking the time to report this issue.

However, marking as duplicate of #811774: Port to D7. You can follow up on that issue to track its status instead. If any information from this issue is missing in the other issue, please make sure you provide it over there.

Also thanks for the patch. Note that all of the clean-up changes should have been a separate issue/patch. Don't do this when porting modules to a new major version of Drupal core.

Michael Zetterberg fd. Lopez’s picture

Michael Zetterberg fd. Lopez commented

Ok! Thanks for your comments. I'll look over at #811774.