php emailed for w/ mysql supplied data

[jojocircus]

So I am going thorugh another tutorial, but it appears that it may be outdated.
basically the script gets data from mysql db (title and content) and inserts into an email message for users also provided by mysql (whose subscribe status = 1)

I am not getting any mysql errors, so am thinking that the tut (I) may be using has outtdated code.

I am just getting presented with the final "Email sent to" line, but no $email variable is placed, and no email gets sent.

Your help and time is greatly appreciated,
jojo

?php
// set time limit to 15 minutes (900/60)
set_time_limit(900);
include("connect.php");
        $id = preg_replace("/'\/<>\"/","",$_GET['id']);
        if (empty($id))
        die("Invalid ID");
        $link = "SELECT * FROM newsletters WHERE id='$id'";
        $res = mysql_query($link) or die(mysql_error());
        $r = mysql_fetch_assoc($res);
        $subject = $r['name'];
        $message = $r['content'];

    $headers  = 'MIME-Version: 1.0' . "\r\n";
    $headers .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";
    $headers .= 'From: [e-mail]' . "\r\n" .
    'Reply-To: [e-mail]' . "\r\n" .
    //'X-Mailer: PHP/' . phpversion();

    $link = "SELECT * FROM users WHERE status='subscribed'";
    $res = mysql_query($link) or die(mysql_error());
    while ($r = mysql_fetch_row($res))
    {
            $email = $r['email'];
            $mail = mail($email, $subject, $message, $headers);
    }

if ($mail)
{
echo "Email sent to " . $email . '<br>';
die;
}
else
{
echo "Error in mailing " . $email . '<br>';
die;
}

?>

[alimadzi]

What do you get when you run the query directly in your database? Do you have subscribed users? Is 'email' an actual column in the table? Just trying to isolate the real issue.

[jojocircus]

running
SELECT * FROM users WHERE status='subscribed'
returns my single user whose status = subscribed

namef namel email date status unsu id
Jim Jones [e-mail] 2** sub 0 2

my columns;
namef
namel
email
status
unsubscribed
id(pk)
date

are you a bronco?

[alimadzi]

Try changing mysql_fetch_row() to mysql_fetch_assoc(). The first function just grabs one row while the second will get all of them.

jojocircus wrote:

are you a bronco?

I'm kind of a Bronco by association... living in Boise and working for the paper. At heart, though, I'm a Husky (U of Washington).

[jojocircus]

mysql_fetch_assoc() populates $email, but still no email sent.

I think I need to review the X-mailer.

Thanks for your help,
jj

[alimadzi]

jojocircus wrote:

mysql_fetch_assoc() populates $email, but still no email sent.

I think I need to review the X-mailer.

Thanks for your help,
jj

No problem. Best of luck...