There is several ways to handle this issue:

1. Lookup under core/ and if not found, assume it is in the include_path.
2. Remove the full path (i.e. rever 0003021) and assume it will always be in the include_path.
3. Add phpMailer path to the configs.
4. Use symbolic links on Unix/Linux systems to phpMailer.
5. Just copy phpMailer files as suggested in the "additional information".

I found a related problem where if a path is used to include class.phpmailer.php, and SMTP is used, an error is generated where class.smtp.php can not be included (assuming it is not in the include path). The possible fixes to this error are:

• Set the PluginDir attribute on the phpmailer class to the path that has class.smtp.php.
• Or include the path in the include_path.

The phpmailer files should be assumed to be in the include path. It is up to the end-user to make sure that 'include_path' is set correctly in php.ini. This is true about every 3rd party library or header-file that is required by any software package. The line in email_api.php should be:

include_once('class.phpmailer.php');

In other words, in the first bugnote, only solution 2 is viable. The other alternative solutions are hacks. If a user cannot set the include_path correctly (e.g. no write permissions) then symlinking these files, is, I guess, acceptable.

edited on: 07-23-03 16:58

I think we should go with option 1. The user may not have access to changing php.ini (shared hosting), and there must be a way for them to still use phpMailer.

I am starting to think that option 3 (adding a new config option) is the way to go. The configuration option will be used as follows:

include_once( config_get('phpmailer_path') . 'class.phpmailer.php' );
$mail = new phpmailer;
$mail->PluginDir = config_get('phpmailer_path');
.....

The default value for the path can be:

1. Empty (i.e. use include path by default)
   or
2. Core path (for backward compatability).

comments?

Fixed in CVS. Will be available in 0.18.0a5. See my last bugnote for details about the fix.

edited on: 07-27-03 05:19