Does not work with PHPMailer v2.1.0 beta 2

john ackers - April 19, 2008 - 21:52
Project:Messaging
Version:5.x-1.0-beta1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The Messaging PHPMailer plugin attempts to initialize the $to property of the PHPMailer object which is now private. Line 99 should be commented out, it is not required. Here is the apache error message.

[Sat Apr 19 21:13:47 2008] [error] [client 127.0.0.1] PHP Fatal error: Cannot access private property PHPMailer::$to in /home/work/web/portal/sites/all/modules/messaging/messaging_phpmailer/messaging_phpmailer.module on line 99

#1

jeffmurphy - April 27, 2008 - 03:29
Status:active» reviewed & tested by the community

cd /home/work/web/portal/sites/all/modules/messaging/messaging_phpmailer
vi +93 messaging_phpmailer.module

change the "t" in "to" to "T" so the line reads as:

$mail->To = array();

#2

jeffmurphy - April 28, 2008 - 14:26

You're right, the line apparently can just be commented out without affecting functionality.

#3

m3avrck - April 28, 2008 - 16:30
Status:reviewed & tested by the community» fixed

thanks this is fixed now!

#4

Anonymous (not verified) - May 12, 2008 - 16:32
Status:fixed» closed

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

#5

decibel.places - August 22, 2008 - 18:02

I changed "to" to "To" and also commented out the line

I get these errors:

    * warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.module on line 194.
    * warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.inc on line 55.
    * warning: Invalid argument supplied for foreach() in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.inc on line 21.
    * warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.module on line 194.
    * warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.inc on line 55.
    * warning: Invalid argument supplied for foreach() in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.inc on line 21.
    * warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.module on line 194.
    * warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.inc on line 55.
    * warning: Invalid argument supplied for foreach() in /home2/netsperi/public_html/flydigits/sites/all/modules/mimemail/mimemail.inc on line 21.

upgraded from 5.x-1.0-beta to 5.x-1.0 and it seems fine...

 
 

Drupal is a registered trademark of Dries Buytaert.