Skip to content

Conversation

@shun84
Copy link
Contributor

@shun84 shun84 commented Jun 7, 2024

Je pense qu'il y a un soucis de comparaison pour la commande Select pour la fonction toHtml sur la ligne 1661 if ($cmdValue->execCmd() == $coupleArray[0]) { pour $coupleArray[0] en passant de 0 à 1, on a la bonne comparaison pour prends la valeur de la commande et non l'élément

Avec ce changement la liste se construit correctement sauf ensuite ça ne s'affichage pas avec la présélection car dans le fichier cmd.action.select.select.html dans la partie dashboard la fonction jeedom.cmd.refreshValue fait un changement, je n'arrive pas à trouver d'où ça vient pour le corriger si vous pouvez voir au passage si vous trouvez avant moi.

@zoic21
Copy link
Contributor

zoic21 commented Jun 10, 2024

Bonjour,
Il faut garder la comparaison avec le clef [0] aussi, certain plugin renvoi la clef et non la valeur. Il faut donc comparer avec les deux (clef + valeur, le 0 et 1 donc).

zoic21 added a commit that referenced this pull request Jun 11, 2024
@zoic21
Copy link
Contributor

zoic21 commented Jun 11, 2024

J'ai fais la modification en 4.4.8 (alpha pour le moment)

@zoic21 zoic21 closed this Jun 11, 2024
@Flobul
Copy link
Contributor

Flobul commented Apr 18, 2025

Cette PR casse les commandes action select dans ce cas de figure :
#listValue# = 1|7;2|6;3|5;4|4;5|3;6|2;7|1

(ici les clés renvoient à des températures :
1 => 7°C
2 => 6°C
...
7 => 1°C)

Si la valeur de la commande info liée est la valeur et non la clé, la valeur à chercher serait dans $coupleArray[1] effectivement.

Si la valeur de la commande info liée est la clé, la valeur à chercher serait dans $coupleArray[0].

Dans le cas de valeurs et clé du même ordre (comme mon exemple), la comparaison if ($cmdValue->execCmd() == $coupleArray[0] || $cmdValue->execCmd() == $coupleArray[1]) { est erroné et va chercher dans la clé alors qu'elle devrait dans la valeur et vis versa en fonction du type de valeur de la commande info liée (clé ou valeur)

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

Successfully merging this pull request may close these issues.

3 participants