Bug #61520
action commentaire : perte totale du format saisi (sauts de ligne, notamment)
0%
Description
L'action commentaire est souvent utilisée pour envoyer des mails au demandeur. On permet aux agents de passer des lignes pour formatter un peu la réponse.
Mais avec la saisie riche (godo) il semble que tout formatage soit devenu impossible : on perd même les sauts de ligne.
Ainsi, si je saisi dans le champ :
Ligne 1
Ligne 2
Ligne 3
la variable form_workflow_data_comment_xxx contient « Ligne 1Ligne 2Ligne 3 ».
Et l'envoi par mail de ce commentaire donne un mauvais résultat. L'affichage avec le filtre "linebreaks" ne fonctionne plus non plus.
Il faudrait a minima ne pas supprimer les sauts de lignes
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-workflows-convert-paragraphs-into-newlines-for-plain.patch 0001-workflows-convert-paragraphs-into-newlines-for-plain.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le strip_tags un peu violent; voilà une interprétation très sophistiquée de l'HTML.
À noter qu'idéalement il y aurait également à permettre la saisie de retours à la ligne, j'ai créé #61527 pour cette partie dans godo.
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Le strip_tags un peu violent; voilà une interprétation très sophistiquée de l'HTML.
;-)
À noter qu'idéalement il y aurait également à permettre la saisie de retours à la ligne, j'ai créé #61527 pour cette partie dans godo.
Est-ce qu'on ne voudrait pas les imaginer dès à présent possible, et les les gérer ici avec :
...replace('</p><p>', '\n\n').remplace('<br>', '\n\n')
(Et là je me demande s'il faudra les remplacer par un double retour-chariot, mais c'est parce que je ne pense qu'aux mails et à reST...)
Bref, les <br> ça sera un autre sujet, je valide déjà ceci parce que je pense que ça va sauver quelques situations.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
Et là je me demande s'il faudra les remplacer par un double retour-chariot
Chariot simple plutôt. Mais j'ai préféré ne pas anticiper.
commit 690f04e506d3d017df40fcf590f96264e41b91b4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Feb 8 08:20:26 2022 +0100 workflows: convert paragraphs into newlines for plaintext comments (#61520)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: convert paragraphs into newlines for plaintext comments (#61520)