jMailer and tpl

Ça peut être sympa en effet.

Hop voila un patch pour ce ticket

Une démo a cette adresse http://jelix.toopi.info/index.php?module=mail&action=default:index

A-t-on vraiment besoin de réinjecter les informations du mail dans le template (tout les $mailtpl->assign()) ? Tu as des exemples où c'est utile ?

Je n'aime pas ta fonction decAdrs. Une syntaxe courante (qui d'ailleurs fait peut être partie des specs RFC quelque part), est "nom <email>". Donc il faut que decAdrs parse ce genre de chaîne. (Avec des noms de variables en anglais ;-) ). Et je pense que parseEmailAddress est mieux comme nom.

   function parseEmailAddress ($email) {
     ..
     return array('email', 'name');
   }

Le test "if( isset($this->BodyTpl?) ) { " n'est pas bon. Il vaux mieux tester avec la valeur vide.

$mailtpl->assign('Priority', $this->Priority );

Permet d'afficher un message indiquant la priorité du mail :

/!\ Ce mail doit être traité en priorité.

$mailtpl->assign('From', $this->From ); $mailtpl->assign('FromName?', $this->FromName? ); $mailtpl->assign('Sender', $this->Sender );

Afficher dynamiquemment le nom de l'expéditeur :

Mr {$FromName} vous souhait une agréable journée vous pouvez lui répondre a l'adresse {$sender}

$mailtpl->assign('to', $this->to );

Cette e-mail a était expédié a {boucle sur $to}

$mailtpl->assign('cc', $this->cc );

Une copie de cette e-mail a était expédié a {boucle sur $cc}

$mailtpl->assign('bcc', $this->bcc );

Une copie de cette e-mail a était expédié a {boucle sur $bcc}, mais vous pouvez pas le savoir AHAHAH ! (rire satanique)

$mailtpl->assign('ReplyTo?', $this->ReplyTo? );

Cette e-mail est envoyé par un robot, pour toutes demande de renseignement utiliser plutôt {$ReplyTo}

---

Certain exemple sont bidon, mais je pense que ça peut être utile de rendre ces info disponibles.

Je n'aime pas ta fonction decAdrs (...)

J'avais fait ce que tu propose au début, et puis je l'ais changer pour ce ":" Je ne sait plus exactement pourquoi, mais je crois que je voulais éviter les "<>".

Le test "if( isset( ... (...)

tu veut dire ??

if( !isset($this->BodyTpl) ) {
    ;
} else {
(...)
}

ok pour les assign :-)

Pour le test, je préfère un if($this->bodyTpl != ) plutôt que de faire un isset. Parce que si on donne une chaine vide, ton test sera true, ce qui n'est en fait pas vrai

arf, trac m'a bouffé ma chaine :-) Je voulais dire if($this->bodyTpl != "")

ok c'est partie.

Je n'aime pas ta fonction decAdrs. Une syntaxe courante (qui d'ailleurs fait peut être partie des specs RFC quelque part), est "nom <email>".

Bien vu RFC MSG Le Format des Messages de l'Internet ( 3.4 Spécification de l’adresse )

I prefer this signature :

   function getAddrName($address)

and it returns an array($address, $name). Don't call methods, but store data directly in the property, it's easier. For example, instead of doing :

   $adr = $this->getAddrName($address);
   $this->AddCC($adr[0], $adr[1]);

Just do

    $this->cc[] = $this->getAddrName($address);

Same thing for AddAddress?, AddBcc? etc..

Use a regular expression (preg_match) in getAddrName, it's better than to do some explode etc, and do a trim on the address before to return it.

Carefull about the word address : it's not adresse or addresse ;-)

I don't know why you do this :

@@ -445,17 +522,17 @@
             $rt = @mail($to, $this->EncodeHeader($this->Subject), $body,
                         $header, $params);
         }
-        else
-            $rt = @mail($to, $this->EncodeHeader($this->Subject), $body, $header);
+        else {
+            try {
+                $rt = mail($to, $this->EncodeHeader($this->Subject), $body, $header);
+            } catch ( Exeptions $e ) {
+                throw new jException('jelix~errors.mail.instantiate', $e->getMessage());
+            }
+        }
 
         if (isset($old_from))
             ini_set("sendmail_from", $old_from);
 
-        if(!$rt)
-        {
-            throw new jException('jelix~errors.mail.instantiate');
-        }
-

According to the documentation, there isn't exception thrown in the mail function if something goes wrong. (And you forgot to do the same thing with mail call just above the one you change)

Juste pour savoir où en est ce ticket avec le patch à corriger suite à la review de laurentj.

lipki just disappeared, i will look at this one.

ro l'autre j'ai même pas disparu en plus ... :)

je check ça dans la journée.

For the last point, I do not know why I change, but I knew that I had created errors in the local file.

I removal.

Lipki : il ne faut pas mettre review+ ;)

I need it, so I just commited it. Thanks Kevin