sending emails to recordset - adding dynamic field to Subject line

Hi Ray, I have an automated email which sends by looping through a recordset. All working fine, however if I add a dynamic value into the Subject field it always shows the value from the first record it finds and doesn't loop through. Dynamic values included in the email body are all correct. How would I fix the attached page so that the subject line will correctly include the info from the relevant record?

Thank you. I used your file to identify and fix an issue with the extension. I've updated the WA_email.php file to address the problem. If you re-install databrige then the file will be updated on your system. For now I have also attached the updated file so you overwrite and test it manually.

Thanks for this, thought I was missing something really obvious! However there's no file attached - could you attach it so that I don't have to reinstall?

It should be in the previous thread, not attached to the email.

Sorry yes, I see it now - will run a couple of tests and confirm it's working!

The subject line is now showing the correct info, however I noticed that two duplicate records are being written to the database - can't see what might be causing that?

I'd have to debug it. It could be the page loaded twice, or the recordset is returning duplicate records.

I think it is because the page is sometimes reloading or refreshing - I usually run this via a scheduled task on the server but for testing purposes I'm loading it in the browser. I tried loading it in a different browser and it only created one database record so looks like it's fine. Thanks again.

Hi Ray, I've just been made aware of an issue with the above file that you kindly updated for me which I think is probably the same issue as I had with the dynamic subject line. When the emails are being sent from a recordset, the cc email is being sent to the first record in the recordset, not the one linked to the correct record. So for example, the email code is:

$Email->addToFromRS("rsReminders","user_email");
$Email->addCCFromRS("rsReminders","EmailCopy");

The first email in the loop goes to the correct people, but the second and subsequent emails all get cc'd to the EmailCopy address in the first record. Hope you can help quite urgently as I have a bunch of these email scripts that run via a scheduled task every day at midday and I want to sort it out before tomorrow! I've attached a sample email page.

I see. It doesn't really work like that by default. You would have to hand code to get that working and put things in a different order in the code. Try this:

<?php

if ("" === "")     {  //WA Universal Email (hand coded)

  $Email = new WA_Email("event-reminder-email");

  $Email->From = "Paradigm <info@paradigm.co.uk>";

  $Email->addToFromRS("rsReminders","user_email");  

  $Email->BodyFile = "../../../webassist/email/diary-regdata-reporting-half-year.php";

  if (function_exists("rel2abs") && $Email->Redirect) $Email->Redirect = $Email->Redirect?rel2abs($Email->Redirect,dirname(__FILE__)):"";

  for ($emailGroup=0; $emailGroup<sizeof($Email->To); $emailGroup++) {

    $Email->CC = array();

    $Email->addCC($rsReminders->getColumnVal("EmailCopy",false));

    $Email->Subject = "Compliance Diary Reminder - RegData Reporting (Half Year)";

    $Email->onSend = new Email_Log("insert","calendar_reminders","Email",$Composite);

    $Email->onSend->Query->bindColumn("Email", "s", "".($rsReminders->getColumnVal("user_email"))  ."", "WA_BLANK");

    $Email->onSend->Query->bindColumn("Phone", "s", "".($rsReminders->getColumnVal("PhoneNo"))  ."", "WA_BLANK");

    $Email->onSend->Query->bindColumn("DisplayName", "s", "".($rsReminders->getColumnVal("display_name"))  ."", "WA_BLANK");

    $Email->onSend->Query->bindColumn("EventName", "s", "".($rsReminders->getColumnVal("EventName"))  ."", "WA_BLANK");

    $Email->send($emailGroup);

  }

  $Email->close();

}

?>

I moved the CC line into the loop that sends the email so that it gets reset for every email and I can set it to the recordset value manually inside the loop so it will send to the corresponding row. I think that will work.