Valeurs de calendrier dans une scène et caractères spécaux/accentés

Développement
,
1

Bonjour à tous (et toutes …),

Gladys m’informe constamment (vocalement et via un bot Télégram) de mes prochains RDV grâce au calendrier, et je vous en remercie une nouvelle fois, c’est vraiment très pratique.

Je constate toutefois un problème depuis quelques temps (je ne saurais pas dire depuis quand exactement …) : dans le nom de l’événement apparaissent les codes HTML des ponctuations et caractères spéciaux, ce qui rends le texte inaudible et moins lisible.

Serait-il possible d’apporter une correction ?

Je vous en remercie par avance,

Belle fin de journée,

Jean

2

Salut @jean_bruder,

Merci pour ton retour !

En fait, tout dépend de la source du calendrier, certains outils ajoutent des caractères spéciaux HTML dans les titres / descriptions.

Tu as raison, on pourrait retirer ces caractères spéciaux pour que ce soit plus propre :slight_smile:

@bertrandda je pense qu’on pourrait modifier calendar.formaters.js pour décoder name, location et description, tu en penses quoi ?

3

Salut,
j’ai regardé rapidement je pense avoir réussi à reproduire et peut être que je me trompe mais je ne pense pas que ce soit caldav qui pose problème. J’ai l’impression que dès qu’un apostrophe ou autre caractère spécial est dans une variable « compilé » par la librairie Handlebars, la lib échappe le caractère.

Exemple (screenshot en dessous), quand je regarde en base de données l’apostrophe est seule si je console.log avant le compile() l’apostrophe est seule également par contre juste après dans le résultat si je console.log ça met des caractères supplémentaires.

J’ai proposé une PR, dit moi ce que tu en penses.

Voilà un screenshot (premier message avant le fix, deuxième message après le fix)

image
image1348×320 33.7 KB

4

Merci d’avoir investigué, ça me paraît bien !

Par contre, je pense que tu peux étendre ce comportement aux autres endroits concernés, sinon on risque de rencontrer le même problème ailleurs :grinning_face_with_smiling_eyes:

Je vois encore pas mal de Handlebars.compile dans le projet.

Et n’hésite pas à ajouter un petit test au passage, comme ça on s’assure que le comportement reste stable dans le temps :wink:

5

@bertrandda en fait j’avais pas vu mais c’était bien le cas dans ta PR, j’ai mergé, ça partira dans la prochaine version de Gladys :slight_smile:

6

Le correctif est disponible dans Gladys Assistant 4.81 :

7

Non tu avais raison la PR n’était pas complète. J’ai ajouté le paramètre noescape dans les Handlebars.compile restants. J’ai également ajouté un test

8

Bonjour @pierre-gilles et @bertrandda :slight_smile:

Je confirme qu’avec la version 4.81.0 qui tourne chez moi, l’échappement des caractères spéciaux est corrigé : plus de choses étranges envoyées par message et de prononciation incohérente des événements à présent !

Une fois de plus, je ne peux que vous remercier :heart:

Jean

9

Excellent ! Dans la prochaine version de Gladys on va corriger ce bug à d’autres endroits car c’était pas le seul endroit.

Merci d’avoir remonté le bug et merci à @bertrandda pour le correctif :blush: