Send Me An Email Of Birthdays

[tomyknoker]

Any ideas about this one?

[thetestingsite]

Well, it should be

\r\n

If you wanted to put the <br> in it, then you would have to put in the MIME type for the email itself. Try this instead:

$message = 'Below is a list of the members and their birthdays that are coming up in the next couple of weeks. \r\n';

$message .= 'The birthdays are as follows:
';

$message .= '-----------------------------------------

';

$message .= $bdays.' 
';

Notice the spaces between the lines. Hope this helps.

or try something like this:

$messages = <<<HERE
Below is a list of the members and their birthdays that are coming up in the next couple of weeks.
The birthdays are as follows:
-----------------------------------------

$bdays
HERE;

[tomyknoker]

Ah ok the first option didn't change anything, the second option did put the first 2 lines seperately but still displayed the birthdays all together... Any ideas?

[thetestingsite]

Try replacing this line:

$bdays .= 'Name: '.$qry['Name'].' ('.$qry['birthday'].') \r\n';

with this:

$bdays .= <<<HERE
Name: $qry['Name'] ($qry['birthday'])

HERE;

That should do it. Hope this helps.

[tomyknoker]

Hmmmm new error is this...

Parse error: parse error, expecting `T_STRING' or `T_VARIABLE' or `T_NUM_STRING' in \emailBirthdays.php on line 24
PHP Parse error: parse error, expecting `T_STRING' or `T_VARIABLE' or `T_NUM_STRING' in \emailBirthdays.php on line 24

Any ideas?

[thetestingsite]

What's line around line 24? If it is the $bdays variable, try taking out the single quotes in the $qry variables to see if that is it. Either way, post the code so we can see what could be the cause of it.

[tomyknoker]

Sorry only just saw this... Ok the whole page looks like this

<?php
$to = "email@mydomain.com"; //your email address
$subject = "Member's birthdays"; //subject of email

/* connect to the mysql database and use a query to get the members info */

include 'library/config.php';
include 'library/opendb.php';

$year = date('Y');
$month = date('m');

$results = mysql_query("SELECT * FROM `tblmembers` WHERE `DateOfBirth` LIKE '%$month%'");

if (mysql_num_rows($results) < 1) {
die('There are no members with birthdays this month');
}

else {

  while ($qry = mysql_fetch_array($results)) {
      if (strtotime($qry['DateOfBirth']) <= (time() + 86400*14)) {
         $bdays .= 'Name: '.$qry['FirstName'].' '.$qry['LastName'].' ('.$qry['DateOfBirth'].') '.$qry['Email'].' \r\n';
      }
  }

$message = <<<HERE
Below is a list of the members and their birthdays that are coming up in the next couple of weeks.
The birthdays are as follows:
-----------------------------------------

$bdays
HERE;

}

$from = "From: PHP Script <noreply@mydomain.com";
{
mail($to, $subject, $message, $from);
}
?>

[thetestingsite]

on this line:

$bdays .= 'Name: '.$qry['FirstName'].' '.$qry['LastName'].' ('.$qry['DateOfBirth'].') '.$qry['Email'].' \r\n';

Change to this:

$bdays .= 'Name: '.$qry["FirstName"].' '.$qry["LastName"].' ('.$qry["DateOfBirth"].') '.$qry["Email"].' \r\n';

That should take care of the error message you get. Hope this helps.

[tomyknoker]

Thanks that fixed the error, but it still just displays them all one after the other on one big line and shows the "\r\n", I was hoping to be able to get them, to display like and also with the DOB lookinbg like that below, it displays it like YYYY-MM-DD...

Name: John Smith (17-01-1980) johnsmith@domain.com

Thanks again!

[thetestingsite]

Try replacing the while loop:

while ($qry = mysql_fetch_array($results)) {
      if (strtotime($qry['DateOfBirth']) <= (time() + 86400*14)) {
         $bdays .= 'Name: '.$qry['FirstName'].' '.$qry['LastName'].' ('.$qry['DateOfBirth'].') '.$qry['Email'].' \r\n';
      }
  }

with this:

  while ($qry = mysql_fetch_array($results)) {
      if (strtotime($qry['DateOfBirth']) <= (time() + 86400*14)) {
         $bdays .= 'Name: '.$qry["FirstName"].' '.$qry["LastName"].' ('.date('d-m-Y', strtotime($qry["DateOfBirth"]).') '.$qry["Email"].' 
';
      }
   }

Not tested, but should work. Hope this helps