Skip to content

BugFix sur la conversion des caractères

Maïeul a demandé de fusionner gh-0e04adce/9/unknown/refs/pull/9/head vers master

La conversion en ISO doit se faire juste avant l'envoi de l'email, et non pas dans FacteurMail->setMessage(), car les entetes sont potentiellement changés après dans envoyer_mail().

Exemple de cas qui buguait

$envoyer_mail = charger_fonction('envoyer_mail', 'inc');

$envoyer_mail('toto@truc.com', 'un sujet',
	array(
		'text' => 'Le corps de message',
		'html' => 'Corps HTML',
		'from' => 'toto@truc.com',
		'nom_envoyeur' => 'Un envoyeur avec une apostrophe U+2019(’)'
		)
	);

Je n'ai pas commité direct pour 2 raisons :

  • je me demande s'il ne ne faudrait pas plutot faire la conversion dans envoyer_mail avant l'appel au pipeline pre_envoi
  • je suis tout de même étonné qu'on modifie ainsi brutalement des attributs dans envoyer_mail, sans passer par un setter

Rapports de requête de fusion