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 6 months ago
maieul commented 7 months ago

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');

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');
Owner

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 ?

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 ?
Poster

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 ;-)

> 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 ;-)
Owner

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 un git 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 !)

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 un `git 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 !)
Poster

@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.

@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.
maieul force-pushed bugfix_nettoyer_titre_email from ec5376e8e9 to abf70b058a 6 months ago
Poster

donc voilà, c'est fait. (bon après comme les PR sont coincés...)

donc voilà, c'est fait. (bon après comme les PR sont coincés...)
cerdic merged commit 2ca8687392 into master 6 months ago
The pull request has been merged as 2ca8687392.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.