Skip to content
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

Problème de génération des tickets \r\n #1193

Closed
tdelaux opened this issue Nov 12, 2018 · 10 comments · Fixed by #1212
Closed

Problème de génération des tickets \r\n #1193

tdelaux opened this issue Nov 12, 2018 · 10 comments · Fixed by #1212
Assignees
Labels
Milestone

Comments

@tdelaux
Copy link

tdelaux commented Nov 12, 2018

Describe the bug
A clear and concise description of what the bug is.
Bonjour,
depuis la version 2.6.5, les tickets générés comportent les caractères "\r\n" à la place de retour à la ligne dans la description du ticket lorsque le contenu de la destination est créé spécifiquement.

Exemple de Description dans le ticket de destination :


Demandes pour les applications suivantes : ##answer_8##

Pour l'utilisateur : ##answer_7##
Profil identique à l'utilisateur : ##answer_11##

Commentaire : ##answer_14##


ce qui donne comme description du ticket :


Demandes pour les applications suivantes : \r\nMOVEX\r\n\r\nPour l'utilisateur : tech\r\nProfil identique à l'utilisateur : post-only\r\n\r\nCommentaire : ryèu


Cordialement,

@btry btry added this to the 2.7.0 milestone Nov 12, 2018
@btry btry added the bug label Nov 12, 2018
@btry
Copy link
Collaborator

btry commented Nov 14, 2018

Bonjour

Le mode texte riche est il activé dans votre GLPI ?

@btry btry self-assigned this Nov 14, 2018
@SylvainGuibert
Copy link

Bonjour,

Le problème est similaire pour nous depuis le passage en 2.6.5.
Concernant le mode texte riche , il est désactivé de notre coté :

2018-11-15_11h00_35

Je rajouterais en plus des carctères \r\n, le \ qui apparaît devait les simple quote.

ex :

S'agit-il d'un remplacement de matériel ou d'une demande ? : Demande
Téléphonie / Lecteur / Douchette : \r\nLecteur CAB

@tdelaux
Copy link
Author

tdelaux commented Nov 15, 2018

Le mode texte riche est désactivé de mon coté aussi.

@btry
Copy link
Collaborator

btry commented Nov 15, 2018

Bonjour,

Pour faire court, je suis au bout de ce que je peux faire dans la série 2.6.x et sa dette technique. La version 2.7.0 contient des changements majeurs pour s'en débarrasser et éviter les effets de bord qu'on me rapporte après chaque correction liée au rendu des objets cible (ticket, changements).

@SylvainGuibert Pouvez-vous également mener une campagne de tests de la branche develop ? Avoir de nombreux testeurs accélèrera significativement la release 2.7.0.

@SylvainGuibert
Copy link

@btry : Il faut me tutoyer. Oui, je vais prendre le temps de tester sur la branche develop.

@SylvainGuibert
Copy link

@btry : Avec la branche Develop :
Test unitaire sur un formulaire crée dans une version antérieure :

-> Lorsqu'il il y'a une simple ou double quotte (',") le caractères d'échappement '' apparaît. Nous avons donc par exemple : S'agit-il ou "

-> Lorsque le type de question est "boite à cocher", la réponse dans l'élément de destination (ticket) parfait avec les caractères \r\n devant la réponse.
Exemple : 4) Validation responsable : \r\nOui

-> Pas de problème avec les réponses sur des questions de type "boutons radios", "champ libre" ou bien
-> Les caractères type @, # , ! , ? ne semblent pas poser de problème.

@btry
Copy link
Collaborator

btry commented Nov 18, 2018

Bonjour

J'ai préparé une PR qui doit résoudre les soucis de rendu de modèle des cibles; avec tests unitaires pour éliminer les risques de régression.

Tests faits pour les tickets, les changments avec et sans mode texte riche. Ca m'a permis de nettoyer le code à cet endroit, et les tests me semblent bons.

Je suis intéressé par vos retours sur cette branche. Attention : si il y a des problèems de rendus spécifiques à des champs, je les résoudrai après validation de ce correctif.

@SylvainGuibert
Copy link

Bonjour,

Je viens de faire quelques tests (sur le même formulaire que le premier test). Les tests me paraissent également bon. Plus de caractères type \r\n . En revanche, j'ai testé avec quelques caractères spéciaux et les \ ne passent pas correctement. Un des deux est supprimé systématiquement.

Exemple :

Données dans le formulaire :

2018-11-20_09h38_06

Données dans le ticket :

2018-11-20_09h38_21

@btry
Copy link
Collaborator

btry commented Nov 20, 2018

Bonjour

J'ai entamé hier un refactorage afin de mieux gérer la production de contenu selon qu'il faille ou non utiliser le texte riche. Je pense que ce refactorage m'aidera à résoudre ce souci. Je prévois la publication du code courant de la semaine prochaine.

@btry
Copy link
Collaborator

btry commented Nov 23, 2018

Bonjour @SylvainGuibert

J'ai fusionné le refactorage dont j'ai parlé. Il semble qu'il a effectivement contribué à résoudre ce problème de double anti-slash. Pouvez-vous mettre à jour votre copie du plugin et voir si le correctif focntionne pour vous également ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants