OOOOPS!!! Help with email, please! This one is urgent.

[Dolemite50]

Hi,

This seems to be working for sending emails but on my test of just sending to the first 2 members, I noticed that the additional email addresses are being added to the TO: line like so:

TO: [e-mail]; [e-mail]; [e-mail]
Thats not good. 🙂

Can anybody please show me how to alter this code so that it won't do that?

Thanks alot.

$oneday = mysql_query("SELECT username, user_email FROM table WHERE user_id <= 3"); 

 $name="management";
$fromemail = "[e-mail]";
while($rowday = mysql_fetch_array($oneday))  

    {  

$loginName = $rowday['username'];  

$to = $rowday['user_email'];  

$subject="Hello $loginName!"; 
$headers .= "MIME-Version: 1.0\r\n";  

$headers .= "Content-type: text/html; charset=iso-8859-1\r\n";  

$headers .= "From: MIXPOSURE<[e-mail]>\r\n";  

$headers .= "To: <".$to.">\r\n";  

$headers .= "Reply-To: <[e-mail]>\r\n";   

$headers .= "X-Priority: 1\r\n";  

$headers .= "X-MSMail-Priority: High\r\n";  

$headers .= "X-Mailer: Microsoft Outlook Express 6.00.2800.1409";
$message = "Hello $loginName!<br> blah blah blah";
		$bounce = "[e-mail]"; 
mail($to, $subject, $message, $headers, $bounce);         

echo $to."<br>"; 
echo $message."<br>"; 
echo $loginName."<br><br>"; 
$to = "";
     }  

[Weedpacket]

You never reset your headers - they just accumulate duplicates over and aover with each iteration.

You don't need to build the headers inside the loop anyway: the only header that changes for each mail is the To: header, and that's put in by the mail() function anyway (that's why your [e-mail] shows up twice).

So build the headers once, outside the loop, and that should be problem solved.

[Dolemite50]

My hero once again WP!

Thank you so much for all of your kind help, I hope I can repay you someday.

[Dolemite50]

Just one more thing,

This is working beautifully but would like to add a .jpg as an attachment. Can anybody PLEASE show me how to do this? I found a lot of posts on it but it looks like the order of the headers has to be perfect and nobody's script looked like mine.

The image is on my server in /MiscPics/mypic.jpg

Working version of email script that needs altered:

$headers .= "MIME-Version: 1.0\r\n";  

$headers .= "Content-type: text/html; charset=iso-8859-1\r\n";  

$headers .= "From: MIXPOSURE<[e-mail]>\r\n";  

$headers .= "Reply-To: <[e-mail]>\r\n";   

$headers .= "X-Priority: 1\r\n";  

$headers .= "X-MSMail-Priority: High\r\n";  

$headers .= "X-Mailer: Microsoft Outlook Express 6.00.2800.1409";
$oneday = mysql_query("SELECT username, user_email FROM phpbb_users WHERE user_id > 3000"); 
while($rowday = mysql_fetch_array($oneday))  

    {  

$username = $rowday['username'];  

$to = $rowday['user_email'];  

$subject="Hello $username!"; 
$message = "YADA YADA";
$bounce = "[e-mail]"; 
mail($to, $subject, $message, $headers, $bounce);   

}

[Weedpacket]

Well, it's not just the headers, it's the whole MIME multipart attachment deal (email, after all, is a text-only medium, so binaries have to be converted into text, and bits have to be inserted to say where one part of the mail leaves off and the attachment begins, and those have to be text, plus you have to have headers inside the body of the mail to describe the type and name of each part....).

I've got a link at the top of this forum to a post that includes information on attachments - the practical upshot is that unless you want to nut this out yourself you'd be better off using something pre-existing like what is at PHPGuru (the "HTML MIME" class - though you can have plain text+attachment as well).