Bonjour,
J’ai répondu à une remarque sur l’utilisation de booléen dans un bloc SI pour le cas « true » mais je suis surpris de ne pas trouver dans la doc le cas « false ».
Je viens de tester et apparemment on peut écrire :
« SI NOT booléen » comme dans un bloc SI normalement constitué.
J’estime mais peut-être à tort qu’il bon de le faire figurer dans la doc.
Merci d’avance pour votre attention.
3 « J'aime »
Salut @olive,
Je n’ai pas voulu utiliser le tag « amélioration » puisque c’est déjà en place.
Peut-être qu’un tag « suggestion » aurait été plus approprié.
Salut @mich0111, je ne sais pas si tu es familier avec GitHub, (j’imagine que oui avec ton avatar de codeur !) mais j’ai réussi il y a quelques jours à enrichir la doc de Jeedom avec un Pull Request qui a été accepté par Kiboost, on en parle ici.
Si tu en as le temps tu peux peut-être le proposer directement via Github ?
Je veux bien mais c’est quand même la base.
Les tests sur booléen ne sont pas mentionnés, ni en « true » ni en « false ».
Je me suis rendu compte de ça lorsque quelqu’un m’a indiqué qu’un test était mal écrit parce qu’il manquait le « == 1 » sur un test de booléen.
Du coup, j’ai eu !a curiosité de lire cette partie de la doc, et je me suis rendu compte que cela n’était pas mentionné.
Les codeurs ne seront pas gênés parce que c’est évident mais les autres …
Et puis en relisant cette doc, à part créer un paragraphe dédié, je me demande où mettre
Les tests sur booléens sont applicables dans un bloc SI. On peut écrire :
SI boolean
Ou
SI NOT boolean
Finalement, j’ai suivi ton conseil et proposé un. PR.
Top 
C’est la meilleur des choses je pense
Cela permet d’aider la team à notre niveau, ils ne peuvent pas être partout tout le temps c’est sûr !