Eviter d'avoir des entités typo dans les titres d'email. #84
Merged
cerdic
merged 1 commits from maieul/spip:bugfix_nettoyer_titre_email
into master
2 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'maieul/spip:bugfix_nettoyer_titre_email'
Deleting a branch is permanent. It CANNOT be undone. Continue?
La function nettoyer_titre_email() utilise corriger_typo, laquelle
intègre des entités HTML, pas les bienvenues dans un titre de mail.
On supprime ces entités HTML.
Exemple de cas où cela posait problème.
$envoyer_mail = charger_fonction('envoyer_mail', 'inc');
$envoyer_mail ('truc@truc.net', 'toto — hop', 'toto — hop');
Une raison pour ne pas avoir utilisé la fonction
nettoyer_caracteres_mail()
qui est juste en dessous et qui fait le boulot propremenet en tenant compte du charset ?aucune, si ce n'est la fatigue.
J'ai corrigé. Cela étant je ne t'en voudrais pas si tu commite directement la bonne version sans même me mettre dans les co-authors ;-)
protip : sur les PR plutot que d'ajouter un nouveau commit dans ce genre de cas, tu fais un
git commit -a --amend
en local, sur ta branche PR, et donc tu modifie le commit précedent avec le bon code, puis ensuite tu fais ungit push --force
ce qui va ecraser la branche de la PR avec ton nouveau commit et du coup la PR sera mise à jour(il ne faut faire le --force que sur les PR, jamais sur une branche utile !)
@cerdic a oui, c'est juste que sur mes propres depots j'ai bloqué tout les forcages, et comme je ne savais pas la politique ici. Mais oui. Je vais donc envoyer une branche propre.
ec5376e8e9
toabf70b058a
2 years agodonc voilà, c'est fait. (bon après comme les PR sont coincés...)
2ca8687392
into master 2 years ago2ca8687392
.