PHPMailer looping AddAddress

[tomaszko]

Hello ,
i've got a little problem. I'm checking list of recipents by checkboxes. Then it should send mail to everyony of selected recipents. But it only works with ONE recipent. I know that part:

$wiad->AddAddress ($mail)

needs to be looped, but i can't manage to do that 🙁

<?php include ('conf.php'); ?>
<?php
if(isset($_GET["sortuj"]))
	$sortuj = $_GET["sortuj"];
else
	$sortuj = "id_klient";

// wygeneruj tabelê ze wszystkimi produktami

$zapytanie = "SELECT * FROM klient ORDER BY $sortuj "; 
	//$zapytanie = "SELECT * FROM produkty ORDER BY nazwa" ; 
$wynik = mysql_query($zapytanie) or die ('B³±d: '. mysql_error());

$zapytanie2 = "SELECT * FROM szablon"; 
	//$zapytanie = "SELECT * FROM produkty ORDER BY nazwa" ; 
$szablon = mysql_query($zapytanie2) or die ('B&#322;&#261;d: '. mysql_error());



?>
 <table method='get'>
<tr><th></th><th>
 <a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=imie">Imi&#281;</a>
  </th><th>
 <a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=nazwisko">Nazwisko</a>
  </th><th>
<a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=telefon">Telefon</a></th><th><a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=mail">Mail</a> </th>
<th>
<a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=miasto">Miasto</a></th><th><a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=ulica">Ulica</a> </th>
<th>
<a href="<?php echo $_SERVER['PHP_SELF'];?>?sortuj=kodpocztowy">Kod Pocztowy</a></th><th><a href="#" id="expander">Zwi&#324;</a></th></tr>
<form action="<?php echo $_SERVER["PHP_SELF"]; ?>" method="post">
<?php
while ($rekord = mysql_fetch_assoc($wynik)) {
?>
	<tr>
			<td><input type="checkbox" name="id[]" value="<?php echo $rekord['id_klient']; ?>" /></td>
			<td><?php echo $rekord['imie'];?></td>
			<td><?php echo $rekord['nazwisko'];?></td>
			<td><?php echo $rekord['telefon']; ?></td>
			<td><?php echo $rekord['mail']?></td>
			<td><?php echo $rekord['miasto'];?></td>
			<td><?php echo $rekord['ulica']; ?></td>
			<td><?php echo $rekord['kodpocztowy']; ?></td>

</td>
	</tr>

<?php
}
?></table><br>
<table method='get'>
<tr> <th></th>
<th>
Tytu&#322;</th><th>Tre&#347;&#263;</th></tr>

<?php
while ($rekord = mysql_fetch_assoc($szablon)) {
?>
	<tr>
			<td><input type="radio" name="cont[]" value="<?php echo $rekord['id_szablon']; ?>"/></td>
			<td><?php echo $rekord['tytul'];?></td>
			<td><?php echo $rekord['tresc'];?></td>
	</tr>
<?php
}
?>
</table>

<?php
if(isset($_POST['klik'])) {
	if(isset($_POST['id'])){

	$mail='';
	$nazwisko='';
	$imie='';
foreach($_POST['id'] as $idi){
$sql="SELECT imie,nazwisko,mail FROM klient WHERE id_klient=".$idi;
$res=mysql_query($sql);
$dane_klienta=mysql_fetch_array($res);

$imie=$imie.''.$dane_klienta[0];
$nazwisko=$nazwisko.''.$dane_klienta[1];
$mail=$mail.''.$dane_klienta[2];
}
if(isset($_POST['cont'])){

	$tytul='';
	$tresc='';
foreach($_POST['cont'] as $conti){
$zap="SELECT tytul,tresc FROM szablon WHERE id_szablon=".$conti;
$w=mysql_query($zap);
$cont_szablon=mysql_fetch_array($w);

$tytul=$tytul.''.$cont_szablon[0];
$tresc=$tresc.''.$cont_szablon[1];
}

require_once($_SERVER['DOCUMENT_ROOT'].'/lib/phpmailer/class.phpmailer.php');
require_once($_SERVER['DOCUMENT_ROOT'].'/lib/phpmailer/class.smtp.php');



$wiad = new PHPMailer();  // Instantiate your new class
$wiad->IsSMTP();          // set mailer to use SMTP
$wiad->SMTPAuth = true;   // turn on SMTP authentication
$wiad->Host = "smtp.gmail.com"; // specify main and backup server
$wiad->SMTPSecure= 'ssl'; //  Used instead of TLS when only POP mail is selected
$wiad->Port = 465;        //  Used instead of 587 when only POP mail is selected

$wiad->Username = "xxx";
$wiad->Password = "xxx";

$wiad->From = "xxx"; //Aparently must be the same as the UserName
$wiad->FromName = "xxx";

$wiad->CharSet = "UTF-8";

$wiad->AddAddress ($mail);

$wiad->Body = ($tytul);
$wiad->AltBody = "Wiadomosc txt";
$wiad->Subject = ($tresc);

if(!$wiad->Send())
{
  echo "There was an error sending the message:" . $wiad->ErrorInfo;

  exit;
}
else {	
	$wiad->clearAddresses();
  	echo "Wys&#322;ano";
  	}

}else{echo 'Wybierz odbiorców/szablon.';

}}}
?>

<input type="submit" name="klik" value="Wyslij" onClick='fir

[big_nerd]

With phpMailer, AddAddress is a method, or function.

// ... around here

$allEmails = array(); // This will hold ALL email addresses you want to send to 

foreach($_POST['id'] as $idi){
$sql="SELECT imie,nazwisko,mail FROM klient WHERE id_klient=".$idi;
$res=mysql_query($sql);
$dane_klienta=mysql_fetch_array($res);

$imie=$imie.''.$dane_klienta[0];
$nazwisko=$nazwisko.''.$dane_klienta[1];
$mail=$mail.''.$dane_klienta[2];

$allEmails[] = $mail; // add this address (which I assume is $mail)
} 

$wiad = new PHPMailer();  // Instantiate your new class
$wiad->IsSMTP();          // set mailer to use SMTP
$wiad->SMTPAuth = true;   // turn on SMTP authentication
$wiad->Host = "smtp.gmail.com"; // specify main and backup server
$wiad->SMTPSecure= 'ssl'; //  Used instead of TLS when only POP mail is selected
$wiad->Port = 465;        //  Used instead of 587 when only POP mail is selected

$wiad->Username = "xxx";
$wiad->Password = "xxx";

// From == EMAIL address, so [e-mail]
// Just noticed -- since you are using Gmail as the SMTP agent this must match your gmail username.
$wiad->From = "xxx"; //Aparently must be the same as the UserName

// FromName == "Your Website" - if above is an email address this will work so "My Website"
$wiad->FromName = "xxx";

$wiad->CharSet = "UTF-8";


// Now LOOP THROUGH EACH EMAIL and use AddAddress on it.
foreach ($allEmails as $mail) {
    $wiad->AddAddress ($mail);
}

$wiad->Body = ($tytul);
$wiad->AltBody = "Wiadomosc txt";
$wiad->Subject = ($tresc); 

I hope I've read your code correctly!

Best of luck...

[tomaszko]

Ok, it's sort of working. Sending mail to first address in a que. And i get this message

Invalid address: [e-mail]@gmail.comMessage sent!

Here is entire code

if(isset($_POST['klik'])) {
	if(isset($_POST['id'])){

	$mail='';
	$nazwisko='';
	$imie='';

$allEmails = array(); // This will hold ALL email addresses you want to send to 
foreach($_POST['id'] as $idi){
$sql="SELECT imie,nazwisko,mail FROM klient WHERE id_klient=".$idi;
$res=mysql_query($sql);
$dane_klienta=mysql_fetch_array($res);

$imie=$imie.''.$dane_klienta[0];
$nazwisko=$nazwisko.''.$dane_klienta[1];
$mail=$mail.''.$dane_klienta[2];

$allEmails[] = $mail; // add this address (which I assume is $mail) 
}
if(isset($_POST['cont'])){

	$tytul='';
	$tresc='';
foreach($_POST['cont'] as $conti){
$zap="SELECT tytul,tresc FROM szablon WHERE id_szablon=".$conti;
$w=mysql_query($zap);
$cont_szablon=mysql_fetch_array($w);

$tytul=$tytul.''.$cont_szablon[0];
$tresc=$tresc.''.$cont_szablon[1];
}

require_once($_SERVER['DOCUMENT_ROOT'].'/lib/phpmailer/class.phpmailer.php');
require_once($_SERVER['DOCUMENT_ROOT'].'/lib/phpmailer/class.smtp.php');



$wiad = new PHPMailer();  // Instantiate your new class
$wiad->IsSMTP();          // set mailer to use SMTP
$wiad->SMTPAuth = true;   // turn on SMTP authentication
$wiad->Host = "smtp.gmail.com"; // specify main and backup server
$wiad->SMTPSecure= 'ssl'; //  Used instead of TLS when only POP mail is selected
$wiad->Port = 465;        //  Used instead of 587 when only POP mail is selected

$wiad->Username = "[e-mail]";
$wiad->Password = "jelcar1234";

$wiad->From = "[e-mail]"; //Aparently must be the same as the UserName
$wiad->FromName = "JelCar";

$wiad->CharSet = "UTF-8";

foreach ($allEmails as $mail) { 
    $wiad->AddAddress ($mail); 
} 

$wiad->Body = ($tytul);
$wiad->AltBody = "Wiadomosc txt";
$wiad->Subject = ($tresc);


if(!$wiad->Send())
{
  echo "There was an error sending the message:" . $wiad->ErrorInfo;

  exit;
}
else {	
	$wiad->clearAddresses();
  	echo "Wysłano";
  	}

}else{echo 'Wybierz odbiorców/szablon.';

}}}

[tomaszko]

I think now it's acumulating value of $mail, example: [e-mail] -> [email][e-mail]@gmail.com[/email] -> [email][e-mail]@[e-mail][/email]. And that's why it's sending to first person and not to the other becouse of wrong address.

[big_nerd]

tomaszko,

This part:

$mail=$mail.''.$dane_klienta[2]; 

Change to

$mail=$dane_klienta[2]; 

You were appending each email as you went on, so you if you just set it as the email address (which I assume is $dane_klienta[2]) it should work.