Bugs in send Client Email

[Rick]

Send client email just displays an alert "clients_email_process.php?id=123" when processing the emails and no email is sent.

There are two bugs which are stopping emails working properly:

1. In clients_email_process.php, after

require("../../include/session.php");

insert the line:

include("../../include/common_functions.php");

because common functions is required to allow the dateToString function to be called later in this routine.

2. In clientemail.js, at line 175, change:

if(response.getElementsByTagName('result')[0].firstChild.data!="sent"){

to

if(response.getElementsByTagName('result')[0].firstChild.data!="Sent"){

[brieb]

Rick,

Is there a way you can submit the changes in a patch?

[dsterry]

I've tested this and it allows me to send emails to multiple recipients but it reports a false error. I get the "Error Sending Email" error next to each recipient even though the email did in fact go out. Must be something with the xml being returned from the process.php file.

[brieb]

Fixed by [252]