thebodster / clients-oriented-ftp

Automatically exported from code.google.com/p/clients-oriented-ftp
0 stars 0 forks source link

Possible solution to email notifcations error? #508

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I've recently restarted my attempts to implement ProjectSend on our website, 
having been discouraged by ProjectSend failing to send email notifications. 
(I'd been getting the error "E-mail notification couldn't be sent" ever since 
upgrading from r514 -- see <a 
href="https://code.google.com/p/clients-oriented-ftp/issues/detail?id=439&can=4"
>Issue 439</a>). After some digging this morning, I finally got email 
notifications working by modifying some of your code, so I wanted to report it 
to you.

I had discovered that ProjectSend utilizes PHPMailer, and recalled that we are 
using PHPMailer successfully elsewhere on our site. In that implementation, we 
are not using SMTP username/password authentication, so I aimed to recreate 
those settings within ProjectSend.

Thus in the file includes/classes/send-email.php, after I commented out the 
lines below (property settings on the PHPMailer object), everything worked:

$this->send_mail->SMTPAuth = true;
$this->send_mail->Port = SMTP_PORT;
$this->send_mail->Username = SMTP_USER;
$this->send_mail->Password = SMTP_PASS;

That section of code in send-email.php could be tweaked a little further to 
make SMTP authentication optional (if SMTP_USER and SMTP_PASS are blank then 
they don't get passed to the PHPMailer object and SMTPAuth is not set to true).

Original issue reported on code.google.com by davesto...@gmail.com on 20 Nov 2014 at 4:44