Notifications : Bug si nom affecté à une commande contient un guillemet

Bonjour,

Juste pour signaler un petit bug qui fait que la notification n’est pas envoyée si le nom affecté à une commande contient un guillemet (2 erreur différentes si 1 ou plusieures commandes ayant un guillemet dans le nom que l’on affecte)

Jeedom Core : 4.2.11
Version JC : 1.1.0

Equipements :
  JeedomConnect - Fred : v1.1.0 sur android (ws)

Comportement avec 2 commandes :

image

Log :

[2022-02-11 10:31:32][INFO] : Send notification with data {"type":"DISPLAY_NOTIF","payload":{"cmdId":"6649","title":"test","message":"test","answer":null,"timeout":null,"notificationId":16445718922601,"otherAskCmdId":null,"options":[],"files":[],"time":1644571892,"channel":"default","update":false,"critical":false,"criticalVolume":null,"color":"#0000ff","image":{"source":"user","name":"Logo Jeedom bulle.png"},"actions":[{"type":"cmd","id":"655","name":"Allume l'entr\u00e9e","index":0},{"type":"cmd","id":"657","name":"Eteint l'entr\u00e9e","index":1}],"index":6,"id":"notif-7"}}
[2022-02-11 10:31:32][DEBUG] : Send output : 2022/02/11 10:31:32 JSON_PARSING_ERROR: Unexpected token END OF FILE at position 652. 

Comportement avec une seule commande :

image

Log :

[2022-02-11 09:28:55][INFO] : Send notification with data {"type":"DISPLAY_NOTIF","payload":{"cmdId":"6649","title":"test","message":"test","answer":null,"timeout":null,"notificationId":16445681356028,"otherAskCmdId":null,"options":[],"files":[],"time":1644568135,"channel":"default","update":false,"critical":false,"criticalVolume":null,"color":"#0000ff","image":{"source":"user","name":"Logo Jeedom bulle.png"},"actions":[{"type":"cmd","id":"655","name":"Allume l'entr\u00e9e","index":0},{"type":"cmd","id":"657","name":"Eteint entr\u00e9e","index":1}],"index":6,"id":"notif-7"}}
[2022-02-11 09:28:55][INFO] : Error while sending notification

Rien de bloquant bien sûr :slight_smile:

Cordialement
Frédéric

Bonjour,

premiere partie manquante :

++
sur le partage des logs, merci d’utiliser le bouton </> afin de formater correctement les texte et que ca devienne lisible

merci pour les corrections !

je vais essayer de reproduire le cas, et corriger ça !

salut @FredT ,

en partie corrigé dans la beta 1.1.2
les notifications sont bien envoyées à l’appli.

il restera un probleme d’affichage dans l’appli qui sera réglé dans la prochaine version

mais comme tu es en stable ca devrait pas trop te gener :slight_smile:

Merci,

Pas de soucis, y’a pas d’urgence, j’attendrais la prochaine stable :slight_smile:

Corrigé dans l appli egalement :wink:
For the next one

:+1:
Merci