New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
— dans le titre des notifications #211
Comments
Plus précisément, cela vient de notre utilisation de |
texte brut pour le sujet des notifications pour ne pas y envoyer d'entité html, facteur gère ça pour nous
Corrigé, à voir s'il faudrait appliquer ça ici aussi https://github.com/seenthis/seenthis/blob/spip31/inc/seenthis_notifier.php#L345 mais j'en doute. |
J'ouvre de nouveau car @Fil me dit que ça reste transcodé cf :
Pourtant, je viens de tester sur mon instance locale avec le même titre et je ne reproduis pas :\ |
Avec le test suivant j'obtiens ceci https://seenthis.net/spip.php?page=bb
Donc si c'est encore transcodé, ça doit venir de facteur, mais je l'utilise aussi sur mon instance de test... |
Ha, je crois que j'ai trouvé le fautif ! https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/typo_guillemets/plugin.xml#L18 Comme facteur génère des mails html, typo guillemets entre en action dans ce squelette : https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/facteur/trunk/emails/texte.html On pourrait rétablir les mails texte uniquement en suivant l'astuce donnée dans le commentaire :
|
Ça vient de ce commit https://zone.spip.net/trac/spip-zone/changeset/59363/spip-zone |
Avec le retour des mails de notifications au format texte brut on devrait être bons (j'espère), je ferme le ticket. |
Et c'est reparti... reçu à l'instant Si quelqu'un est intéressé pour y regarder de plus prêt, de mon côté j'ai juste envie de désactiver le plugin guillemets typographiques /me blasé |
Comme je ne lâche pas l'affaire comme ça, j'y ai passé encore "un peu" de temps, et je découvre que c'est
En effet, Je proposerais bien de passer outre en insérant de force notre sujet de mail par le biais du pipeline |
Et donc tout simplement parce que nettoyer_titre_email() utilise corriger_typo() qui est le cœur de typo() et c'est là qu'il y a les pipelines pre_typo et post_typo, donc que le plugin Orthotypo ou assimilé fait son œuvre. |
Plutôt que de forcer un autre titre, alors que ce qui a été fait avant peu contenir des choses intéressantes, y compris de correction typo indispensable (espaces, etc), est-ce qu'on ne peut pas simplement prendre ce qui a été produit et passer dessus une fonction qui remet les entités en vrais caractères UTF ? |
Pour comparaison, en 2.1 on n'utilisait pas facteur et donc on utilisait Fonction qui avait "le bon goût" de remplacer les D'ailleurs si je reprends l'exemple précédent, on obtient |
@rastapopougros quand tu dis :
Je doute sur le fait d'appeler une fonction pour défaire ce qui a été fait plus haut. À quelle fonction penses-tu ? |
@brunob bé : |
Dans ce cas on pourrait utiliser la méthode Ou celle proposée par la classe facteur, mais celle-ci semble bien plus complexe https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/facteur/trunk/facteur_fonctions.php#L307 et elle renvoie du markdown. Sinon, un simple |
Normalement, à l'endroit dont on parle, il n'y a jamais de HTML, de balises, donc html2text ou l'autre c'est pas pour ça. Là on parle juste d'entités uniquement, à priori. |
Je parlais de cette méthode car elle appelle justement |
Proposition de patch : diff --git a/seenthis_pipelines.php b/seenthis_pipelines.php
index 6277467..e956343 100644
--- a/seenthis_pipelines.php
+++ b/seenthis_pipelines.php
@@ -21,5 +21,7 @@ function seenthis_facteur_pre_envoi($facteur){
// focer les mails au format txt
$facteur->ContentType = 'text/plain';
$facteur->AltBody = '';
+ // décoder les entités html du sujet (insérées par typo_guillemets dans post_typo par exemple)
+ $facteur->Subject = html_entity_decode($facteur->Subject, ENT_QUOTES, $facteur->CharSet);
return $facteur;
} |
On peut déjà faire ça, et puis si on voit d'autres problèmes arriver on complètera. |
devrait enfin régler pour de bon seenthis/seenthis_squelettes#211
Patch commité, up en prod, je ferme et on y reviendra si nécessaire. |
Le mail de notification de https://seenthis.net/messages/791824 a pour sujet
Project Raven — Inside the UAE’s secret hacking team of American mercenaries
. Cela vient deextraire_titre()
https://github.com/seenthis/seenthis/blob/spip31/seenthis_options.php#L671The text was updated successfully, but these errors were encountered: