Caractères ? transformés en ' par safeUtf8Decode (V4) #3

Closed
opened 2 years ago by JLuc · 2 comments
JLuc commented 2 years ago
Collaborator

Avec la V4 de facteur : Sur un site en utf8, quand le titre du mail contient un ?, ce caractère est transformé en ' lorsque l'option "mails en ISO" est activée.

C'est la fonction safeUtf8Decode qui fait ça. https://git.spip.net/spip-contrib-extensions/facteur/src/branch/master/inc/Facteur/FacteurMail.php#L430

Elle teste d'abord si is_utf8 et apparemment c'est bien reconnu comme utf8 car ça return pas et aussitôt après $text = str_replace('?', "'", $text); pof ya plus de ?

Avec la V4 de facteur : Sur un site en utf8, quand le titre du mail contient un `?`, ce caractère est transformé en `'` lorsque l'option "mails en ISO" est activée. C'est la fonction safeUtf8Decode qui fait ça. https://git.spip.net/spip-contrib-extensions/facteur/src/branch/master/inc/Facteur/FacteurMail.php#L430 Elle teste d'abord si is_utf8 et apparemment c'est bien reconnu comme utf8 car ça return pas et aussitôt après `$text = str_replace('?', "'", $text);` pof ya plus de `?`
JLuc changed title from Caractères ? transformés en ' par safeUtf8Decode to Caractères ? transformés en ' par safeUtf8Decode (V4) 2 years ago
Owner

c'est un problème de conversion iso-utf du fichier php visiblement, c'est censé être un curly à vérifier dans l'historique du plugin

c'est un problème de conversion iso-utf du fichier php visiblement, c'est censé être un curly `’` à vérifier dans l'historique du plugin
Poster
Collaborator

Effectivement. En v3 c'était bien un ’ dans la fonction qui était alors en snake case : safe_utf8_decode https://git.spip.net/spip-contrib-extensions/facteur/src/branch/v3/classes/facteur.php#L330

Effectivement. En v3 c'était bien un ’ dans la fonction qui était alors en snake case : safe_utf8_decode https://git.spip.net/spip-contrib-extensions/facteur/src/branch/v3/classes/facteur.php#L330
JLuc closed this issue 2 years ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.