Send Email With New Row Data - One Email Only?

[leemo]

The code below currently sends a separate Email with data from each row of a table when a Save All button is pressed. So, if I have 4 rows of data, 4 Email will be sent.

I would prefer to receive one Email with 4 sections of data. Any help greatly appreciated.

$email="to@email.com";
$from="from@email.com";
$msg="";
$subject="Registers Info for: ".$values["first_name"]." ".$values["last_name"]."";
 
$msg.= "Student: ".$values["first_name"]." ".$values["last_name"]."\r\n";
$msg.= "Absent or Present: ".$values["attendance_status"]."\r\n";
$msg.= "Class: ".$values["classname"]."\r\n";
$msg.= "Class Date: ".$values["attendance_date"]."\r\n";
$msg.= "Amount Received: ".$values["cash_received"]."\r\n";
$msg.= "Paid For: ".$values["cash_whatfor"]."\r\n";
 
$ret=runner_mail(array('to' => $email, 'subject' => $subject, 'body' => $msg, 'from'=>$from));
if(!$ret["Sent"])
echo $ret["message"];

 

 

Edited February 27, 2020 by leemo

[requinix]

You haven't shown the loop that must be present, but regardless

Move the variable initialization to before the loop, keep inside the loop the thing that builds the body of the email message, and move the actual sending of the email to after the loop.

[ginerjm]

Build the message while looping thru your query results.  THEN fill out the email and send it.  Simple, huh?

[leemo]

Thanks for your replies. I'm trying so hard as a noob to work my way around things on my own and it's a great feeling when things work. It took me 2 days to get as far as this 🙂

I've tried the magic curly brackets before and after the actual message section, do I also need to add something similar to this:

while ($msg = mysqli_fetch_array($result)) - before the message section and I'm guessing I need something at the end also?
 

[ginerjm]

Don't know what code you are referring to.  Would be nice if you posted that block so we all know what you are looking at.

[leemo]

{
$email="to@email.com";
$from="from@email.com";
$msg="";
$subject="Registers Info for: ".$values["classname"]." ".$values["attendance_date"]."";

while ($msg = mysqli_fetch_array($result))
{
$msg.= "Student: ".$values["first_name"]." ".$values["last_name"]."\r\n";
$msg.= "Absent or Present: ".$values["attendance_status"]."\r\n";
$msg.= "Class: ".$values["classname"]."\r\n";
$msg.= "Class Date: ".$values["attendance_date"]."\r\n";
$msg.= "Amount Received: ".$values["cash_received"]."\r\n";
$msg.= "Paid For: ".$values["cash_whatfor"]."\r\n";
}
$ret=runner_mail(array('to' => $email, 'subject' => $subject, 'body' => $msg, 'from'=>$from));

}

This isn't correct, but I'm doing my best!

Edited March 2, 2020 by leemo

[ginerjm]

I don't know what "runner_mail" is, but the code itself seems pretty ok.  Be sure that your "from" email is using your domain name so that your mail doesn't look like spam

[leemo]

No Emails were sent using the previous block of code.
The code is an After Record Updated event with PHP Runner software

[ginerjm]

Then maybe you need to find a forum that pertains to that product.  I never heard of it much less no how to program around it.

Edited March 2, 2020 by ginerjm

[requinix]

$subject="Registers Info for: ".$values["classname"]." ".$values["attendance_date"]."";

while ($msg = mysqli_fetch_array($result))
{

$values isn't available to use until you get to the while loop.

You didn't post the part of the code that shows what query is being run so I don't know whether you're doing a query that makes sense for that subject line - that is, whether it's a query for a specific class and attendance date. (Which used to be the student name in your original version...?) So if you want the classname and attendance_date, you should get those values from, presumably, the same place that the query was getting them from.

Either that, or you send one email each. You'd be sending multiple emails, but only one for each classname/attendance_date combination. Again, not sure since you haven't posted the code that would help explain what's going on. Hint.

[Barand]

1. As you are putting multiple names in the message it doesn't make sense to include (one of) them in the subject.

2. Your while loop should be

while ($values = mysqli_fetch_array($result))
{

otherwise you trash the $msg variable by overwriting it.

Edited March 2, 2020 by Barand

[leemo]

As stated in the first post I am receiving all the information correctly but in separate Emails. My results are between 1-20 records, which of course is 1-20 Emails.

Requinix I have changed the subject variable to make more sense if receving one Email.

I need to loop the information in the $msg.= sections into one email. I have added:

while ($values = mysqli_fetch_array($result))

and put the area I need looped in curly brackets, not sure if that is the correct way to do it. But, as the code stands below I am not receiving any Emails at all.

$email="lee@fisteps.com";
$from="db@fisteps.com";
$msg="";
$subject="Registers Info for: ".$values["classname"]." ".$values["attendance_date"]."";
while ($values = mysqli_fetch_array($result))
{
$msg.= "Student: ".$values["first_name"]." ".$values["last_name"]."\r\n";
$msg.= "Absent or Present: ".$values["attendance_status"]."\r\n";
$msg.= "Class: ".$values["classname"]."\r\n";
$msg.= "Class Date: ".$values["attendance_date"]."\r\n";
$msg.= "Amount Received: ".$values["cash_received"]."\r\n";
$msg.= "Paid For: ".$values["cash_whatfor"]."\r\n";
}
$ret=runner_mail(array('to' => $email, 'subject' => $subject, 'body' => $msg, 'from'=>$from));

SELECT
attendance.attendance_id,
attendance.student_id,
attendance.class_id,
attendance.attendance_status,
attendance.attendance_date,
attendance.cash_received,
attendance.cash_whatfor,
students.last_name,
students.first_name,
classes.classname
FROM attendance
INNER JOIN classes ON attendance.class_id = classes.class_id
INNER JOIN students ON attendance.student_id = students.student_id
ORDER BY students.last_name

 

[Barand]

Take $values out of the subject

Quote

$subject="Registers Info" for: ".$values["classname"]." ".$values["attendance_date"]."" ;

[phppup]

It sounds like you've got something working. (which is a plus) and now just need to tweak it to for your specifications.

There are many more qualified coders here than me, but here's my opinion.

If I understand your dilemma, you are receiving one email per student but would prefer receive a single email with the information of all the students.

I would suggest adding an IF statement that surrounds the entirety of your applicable code.

You can either ADD another column (maybe MEMBER) and sort through with the IF statement.

Or, analyze an existing column that is applicant (maybe attendanc_date).

So, if attendence_date is March 1 [2020-03-01] THEN put the six lines of $msg into the body of the email, otherwise, skip the row.

You will want to add an additional \n at the end of your $msg to make the email body more presentable and break up the information.

Also, in this instance, make sure your$ret is pristine correctly to avoid sending multiple emails with the same content.

Good luck.

 

Edited March 4, 2020 by phppup

[phppup]

Last line should read:

make sure your $ret is positoned correctly to avoid sending multiple emails with the same content.