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

Pour le select prenez pas la bonne valeur pour faire la comparaison donc il faut prendre la 2éme valeur #2666

Closed
wants to merge 103 commits into from

Conversation

shun84
Copy link
Contributor

@shun84 shun84 commented Jun 3, 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.

Loïc and others added 30 commits March 29, 2023 14:41
Update network.class.php
zoic21 and others added 26 commits February 7, 2024 09:11
Bugfix for issue on mobile widget
Correct bug on getIsVisible send null
…raison donc il faut prendre la 2éme valeur"

This reverts commit fdf89e8.
@Hotfirenet
Copy link
Contributor

@shun84 peux tu refaire ta PR depuis une alpha a jour, Merci par avace

@shun84
Copy link
Contributor Author

shun84 commented Jun 5, 2024 via email

@shun84 shun84 closed this Jun 5, 2024
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.

8 participants