Discussion :

[gtraxx]

Bonjour a tous, j'aurai besoin que mon script utilisant phpmailer envoi un mail a toutes les personnes de la liste.
Je m'apercois que je n'envoi le mail qu'as la dernière personne dans la base de donnée en effectuant un var_dump.
Voici le script qui envoi le mail:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
function loadMailUsers(){
		$alllist = new DbUsersAdmin;
		foreach ($alllist->loadUsersFrontend() as $item => $email){
			$listmail = $email['email'];
		}
		return $listmail;
	}

voici ma fonction pour la base de donnée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

j'ai retiré mon code :D

J'ai beau me creuser je ne vois pas ....

Voici le retour de mon var_dump:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

string(25) "momail@gmail.com"

hors il devrai y en avoir deux ???

[sabotage]

je n'ai pas regardé tout le detail mais ce n'est pas un truc comme ca qu'il te faudrait plutot ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
foreach ($alllist->loadUsersFrontend() as $item => $email){
			$listmail .= $email['email'] . ',';
		}

[Sub0]

-> Ajouter une virgule sauf pour le dernier de la liste...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
$listmail='';
foreach ($alllist->loadUsersFrontend() as $item => $email){
  if(!empty($listmail)) $listmail.=' ,';
  $listmail.=$email['email'];
}

Pour vérifier que la liste des destinataires est correcte :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

echo $listmail;

[Sub0]

Ce sujet pourrait t'intéresser :
http://www.developpez.net/forums/d59...-envoi-e-mail/

[gtraxx]

non didju exact, j'avais beau retourner la question dans ma tête pas moyen de tomber sur la soluce.
le pire c'est que j'ai fais une correction sur un autre forum pour le même prob honte a moi.
merci a vous deux

[gtraxx]

correction j'ai tout de même un probléme lors de la reception du mail :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
his is an automatically generated Delivery Status Notification
 
Delivery to the following recipient failed permanently:
 
    test@gmail.com ,test2@gmail.com
 
Technical details of permanent failure: 
Probe failed: Illegal To: address (invalid domain name):  test@gmail.com ,test2@gmail.com

[sabotage]

ton hebergeur a peut etre des restrictions sur les destinataires de mail.
qu'est ce cela donne avec ton adresse sur l'hebergeur ?

[gtraxx]

ben en faites la je test en local avec l'adresse en question .
Donc si vous voulez j'envoi le mail via le formulaire sur ce meme mail via smtp.
Je vais tester avec un autre serveur smtp pour voir mais sinon en principe j'utilise OVH
C'est surement cela le probléme?

[gtraxx]

si je ne mets qu'une seul adresse sa fonctionne par contre si j'en mets plus d'une j'ai droit a l'erreur ????

Voila ce que me mets phpmailer :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Erreur SMTP : Les destinataires suivants sont en erreur :

Donc il y as bien un probléme avec la virgule ou espace entre les destinataires??
En tout cas la connexion ssl ce fais parfaitement
on dirai que c'est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$mail->AddAddress($this->loadMailUsers());

Qui ne fonctionne pas correctement et je me demand s'il ne doit pas être bouclé, car dans la doc j'ai cru voir :

$mail->AddAddress(mail1@mail.com);
$mail->AddAddress(mail2@mail.com);
$mail->AddAddress(mail3@mail.com);

une idée pour une boucle correct alors que j'ai déjà un tableau des adresses

[daajack]

Hello,

Je ne connais pas cette classe, je ne peux donc pas répondre à ta dernière question, mais pour ce que j'ai vu avant, il me semble que les espaces fonctionnent aussi pour mettre plusieurs destinataires, et tu peux utiliser une fonction méconnue, et pourtant au combien pratique : implode, ce qui nous donnerait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$stringDest = implode(' ', $alllist->loadUsersFrontend());

[gtraxx]

sa ne change pas grand chose, j'ai un souci avec implode il m'indique :

Erreur SMTP : Les destinataires suivants sont en erreur : monmail1@mail.comArray ,Arraymonmail2@mail.com

j'ai mis ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
function loadMailUsers(){
		$alllist = new DbUsersAdmin;
		$listmail='';
		foreach ($alllist->loadUsersFrontend() as $item => $email){
 
		  if(!empty($listmail)) $listmail .= implode(' ,', $alllist->loadUsersFrontend());
		  $listmail.=$email['email'];
		}
 
		return $listmail;
	}

[daajack]

Non, l'idée c'est justement de remplacer la boucle :
Je t'avertis c'est pas censé résoudre ton problème, c'était juste pour te montrer ma culture infinie...

Tu remplaces :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$listmail='';
foreach ($alllist->loadUsersFrontend() as $item => $email){
 
  if(!empty($listmail)) $listmail .= implode(' ,', $alllist->loadUsersFrontend());
  $listmail.=$email['email'];
}
 
return $listmail;

Par ceci :
Apparemment c'est un tableau de tableaux associatif, dont seul la clé 'email' nous intéresse...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$emails = Array();
foreach ($alllist->loadUsersFrontend() as $user) $emails[] = $user['email'];
return implode(' ', $emails);

Du coup y'a quand même une boucle, mais c'est plus stylé que de faire de la mise en forme de chaîne avec des méthodes de barbare

[lepotier]

si je ne mets qu'une seul adresse sa fonctionne par contre si j'en mets plus d'une j'ai droit a l'erreur ????

Voila ce que me mets phpmailer :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Erreur SMTP : Les destinataires suivants sont en erreur :

Donc il y as bien un probléme avec la virgule ou espace entre les destinataires??
En tout cas la connexion ssl ce fais parfaitement
on dirai que c'est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$mail->AddAddress($this->loadMailUsers());

Qui ne fonctionne pas correctement et je me demand s'il ne doit pas être bouclé, car dans la doc j'ai cru voir :

une idée pour une boucle correct alors que j'ai déjà un tableau des adresses

C'est mon code et il marche, si ça peut t'aider...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
$length = sizeof($email);
	$m=1;
	while ($m<$length+1){
		$subject = stripslashes($_POST['sujet']);
		$message = "";
 
		$mail = new PHPmailer();
		$mail->IsHTML(true);
		$mail->FromName='ATL';
		$mail->From='from@adresse.mail';
		$mail->AddAddress($email[$m]); //$email est un array avec mes adresses

[gtraxx]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
function loadMailUsers(){
		$emails = Array();
		foreach ($this->dbh->loadUsersFrontend() as $user) $emails[] = $user['email'];
		return implode(' ', $emails);
	}

Ceci ne change rien j'ai toujours le même message d'erreur

Erreur SMTP : Les destinataires suivants sont en erreur :
monmail@mail.com monmail2@gmail.com

[daajack]

Ceci ne change rien j'ai toujours le même message d'erreur

Ben non...

Je t'avertis c'est pas censé résoudre ton problème, c'était juste pour te montrer ma culture infinie...

...alors forcément.

T'as essayer ',' et ';' ?

[gtraxx]

pour l'instant sa a l'air de fonctionner avec ';'
je vous remercie grandement
mes cheveux blanc peuvent s'en aller