When I'm yet to activate phpmailer after installation of the module 'admin/settings/phpmailer' has no error but when saving the configuration including test email recipient it shows the error. I placed class.phpmailer.php and class.smtp.php in sites/all/libraries/libraries/phpmailer

CommentFileSizeAuthor
#7 phpmailer_plugindir.patch2.52 KBsmk-ka
#6 phpmailer-HEAD.plugindir.patch699 bytessun

Comments

smk-ka’s picture

smk-ka commented
Status: Active » Postponed (maintainer needs more info)

Support for Libraries API only exists in the 3.x branch of this module, but not in 2.x. Please update if you wish to make use of it. The PHPMailer class files should be placed in sites/all/libraries/phpmailer (note that 'libraries' appears only once in the path). Does this fix your issue?

pyxio’s picture

pyxio commented

Hi

I have the same error. I'm trying to run 6.x-2.1 which don't require the libraries so I'm not sure why I'm seen the white screen. Thanks. Kevin

smk-ka’s picture

smk-ka commented

Could you please try the latest development snapshot of this module? Given the fact that there were so many changes since 2.1, and the next stable releases 2.2 and 3.0 are really close to be released, I'd like to see if this error still appears with these versions. I'm using the 2.x snapshot in several projects without issues, and the 2.x branch will still be maintained in the future. It is identical to the 3.x branch minus Libraries API support.

njathan’s picture

njathan commented

I too had the same problem. As per this post copying the two files - class.phpmailer.php & class.smtp.php - in sites/all/modules/phpmailer/phpmailer solved the problem. Probably the readme should be updated.

Anonymous’s picture

Anonymous (not verified) commented

I had the same problem and did a bit of digging. Commenting out line 45 of includes/phpmailer.class.inc:

// Adjust path to SMTP class.
// $this->PluginDir = drupal_get_path('module', 'phpmailer') . '/phpmailer/';

did the trick. It looks like the module is still overriding PHPMailer's settings with its own location, which it shouldn't have to do anymore with the Libraries API.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: Error "Class 'PHPMailer' not found" using PHPMailer 6.x-2.1 with or without Libraries API » "Class 'PHPMailer' not found" with or without Libraries API
Status: Postponed (maintainer needs more info) » Patch (to be ported)
StatusFileSize
new phpmailer-HEAD.plugindir.patch699 bytes

Confirmed.

Thanks for reporting, reviewing, and testing! Committed attached patch to HEAD.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Probably needs to be backported to 2.x.

smk-ka’s picture

smk-ka commented
Status: Patch (to be ported) » Fixed
StatusFileSize
new phpmailer_plugindir.patch2.52 KB

Just committed a follow-up patch. Two notes:
1) We can't just remove setting $this->PluginDir altogether as per #5, since this only works if the PHP INI variable include_path is set to search the current directory.
2) Using the Libraries API is still optional in 3.x, we must therefore also check the local module directory in addition to the libraries path.

Status: Fixed » Closed (fixed)

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