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

how about sanitizing strings in sensors ? #37

Open
ggrandou opened this issue Jul 1, 2024 · 0 comments
Open

how about sanitizing strings in sensors ? #37

ggrandou opened this issue Jul 1, 2024 · 0 comments

Comments

@ggrandou
Copy link

ggrandou commented Jul 1, 2024

les entités de type RegularStrSensor sont retournées telles que fournies par la TIC.

quelques exemples:
"{{ states('sensor.linky_libelle_tarif_fournisseur_en_cours') }}" -> " HP BLEU "
"{{states('sensor.linky_message_court')}}" -> " PAS DE MESSAGE "

Je comprends la volonté de retourner les valeurs brutes, mais en fait ces espaces n'ont pas d'autre but que d'avoir un affichage correctement aligné sur le petit écran LCD de 16x2 du Linky :)

ça complique la vie de l'utilisateur qui doit écrire le bon nombre d'espaces dans les automatisations. il serait plus simple de simplement écrire "HP BLEU" plutôt " HP BLEU ". Je vais devoir d'ailleurs attendre l'arrivée de l'hiver pour connaître le nombre d'espaces pour la chaîne "HP ROUGE" que je n'ai encore jamais vue.

Je pourrais bien sur utiliser un template du style "{{ states('sensor.linky_libelle_tarif_fournisseur_en_cours') | replace(' ', '') }}" mais c'est moins intuitif, et value_template n'est pas dispo partout dans HA.

J'ai un patch rapide qui fonctionne chez moi, si l'idée plaît je peux faire un patch plus travaillé.
Quelle serait la meilleure approche ?

  • modifier RegularStrSensor (ce que j'ai fait en local)
  • créer une class type SanitizedStrSensor pour les sensors NGTF/LTARF/MSG1/MSG2 (et peut-être d'autres ?)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant