search

abarrau / jeedom-plugins-doc

Doc des plugins développés par abarrau/olindote, pour la box domotique Jeedom
3 stars 0 forks source link

[DaikinOnlineCtrl] Problème d'affichage swing 3D #66

Closed vanackej closed 2 years ago

vanackej commented 4 years ago

Salut,

Merci déjà pour le plugin, je sais que c'est beaucoup de temps passé ^^

J'ai un problème avec l'affichage sur le widget du mode Swing 3D, l'affichage pour l'option est vide.

image

J'ai regardé un peu, la classe css fa-arrows est bien présente, donc ça devrait bien être affiché, je ne comprends pas le soucis.

En dehors de ça, étant moi-même développeur, je serais intéressé pour avoir un accès au dépôt de code source, afin de pouvoir proposer des pull request. Je sais que je peux récupérer le code source dans mon installation de Jeedom, mais c'est pas ce qu'il y a de plus pratique.

abarrau commented 4 years ago

Bonjour, Ok merci pour l'information, je vais regarder pour apporter une correction sur ce point; je vous tiens au courant. Cdt, Aurélien

Pingu95 commented 3 years ago

Bonjour Aurélien,

Avant tout merci pour ce plugins très utile quand on est chez Daikin + jeedom.

J'ai le même problème que vanackej lors du passage de v3.55 en V4.1.20 avec en plus un rendu global relativement moche...

Globalement en v4 je n'arrive pas avoir les split de manière agréable comme en v3 sous les design. Et je ne trouve pas ou changer les réglages.

En v3 j'ai cela (serveur de prod) :

image

C’est beau tous les boutons fonctionnent tout va bien (et ce sont les réglages par défaut...).

En v4 (serveur de test) en fonctions des réglages j'ai des résultats comme ça (j'ai changer les affichage des 3 splits pour essayer de me rapproché de la v3) :, j'ai du mal a cliqué sur le soleil par exemple (pour les mode de fonctionnement)

image

Tout est moche police désagréable, icônes semblent déformés, celui de la direction 3D est absent, il ne me manque plus que ça pour basculer en v4, tous mes autres équipements semble bien se comporter. Mais pour les splits du plugins Daikin, rien a faire, ça reste moche.

Connaissez vous les quelques réglage à mettre en v4 pour avoir pour les split Daikin le même affichage qu'en thème Darksobre de la v3 qui n'existe pas vraiment en v4 au final... (surtout dans les design je n'utilise que ça au final...) ?

Par avance merci, très bonne journée en tout cas, et bon courage dans les différent dev.

En attendant je reste en sur la branche v3 où tout marche bien mais j'ai peur qu'a un moment elle ne soit plus maintenu.

A+

abarrau commented 2 years ago

Bonjour, Correction prévue dans une version 1.4.x (prévue en sortie d'ici la fin du mois) :

Désolé pour le délais de traitement. Cdt, Aurélien

abarrau commented 2 years ago

version 1.4.0 poussée en béta sur le market

Pingu95 commented 2 years ago

Bonjour Aurélien,

Merci pour les modifications. J'ai pu faire le passage en v4 car cela est déjà plus jolie a présente, pas de souci d'affichage des température et les icônes sont jolie. C'est donc mieux pour l'affichage global mais j'ai toujours deux petits problèmes. Je ne sais pas si c'est identique chez vous...

Premier problème : la zone de click du bouton des modes (et uniquement des modes, celui de direction et vitesse est ok sur toute la zone) est fonctionnel que sur la partie droite du bouton. (la partie gauche n'est pas cliquable). Deuxième petit souci : le bouton direction 3D n'as pas d'icône.

Ci-dessous une capture détaillant la chose. En haut les zones de click NOK et OK pour le bouton mode (RAS pour les deux autres boutons). En bas le bouton (direction 3d) manquant. J'espère que c'est assez clair :)

image

par avance merci !

Cdlt, Sébastien

abarrau commented 2 years ago

Bonjour Sébastien,

1/ Merci pour ce retour du problème de zone cliquable ; j'avais remarqué aussi, mais ce n'était pas systématique ; j'ai trouvé enfin le pblm.

Fichier : /plugins/DaikinOnlineCtrl/core/template/dashboard/DaikinOnlineCtrl.html Ligne : 474 Modifier par : <span class="statusCmd" style="position:absolute;left:5px;bottom:0px;width:30px;z-index:1030;height:30px;"></span>

2/ J'ai l'impression que la mise à jour n'a pas bien fonctionné pour toi. J'ai eu ce problème également, mais après plusieurs tentatives c'était passé ... peut-être un problème de cache (?)

Dans un scénario, bloc "code", tu peux passer ce bout de code : 

foreach (eqLogic::byType('DaikinOnlineCtrl') as $DaikinOnlineCtrl) {
    // mise à jour des icons "direction" //
    $oCmd = $DaikinOnlineCtrl->getCmd('action', 'DOC-sci-f_dir0-action4');
    $oCmd->setDisplay('icon', "<i class='fas fa-pause'></i>");
    $oCmd->save();
    $oCmd = $DaikinOnlineCtrl->getCmd('action', 'DOC-sci-f_dir1-action4');
    $oCmd->setDisplay('icon', "<i class='fas fa-arrows-alt-h'></i>");
    $oCmd->save();
    $oCmd = $DaikinOnlineCtrl->getCmd('action', 'DOC-sci-f_dir2-action4');
    $oCmd->setDisplay('icon', "<i class='fas fa-arrows-alt-v'></i>");
    $oCmd->save();
    $oCmd = $DaikinOnlineCtrl->getCmd('action', 'DOC-sci-f_dir3-action4');
    $oCmd->setDisplay('icon', "<i class='fas fa-arrows-alt'></i>");
    $oCmd->save();
    $DaikinOnlineCtrl->setConfiguration('temperatureStep',1);
    $DaikinOnlineCtrl->save();
}

Si tu peux passer ces 2 éléments, tu devrais revenir à une situation normale. Je sortirais une 1.4.1 dans la semaine.

Cdt, Aurélien

Pingu95 commented 2 years ago

Merci pour ton retour. Pour la zone de clique sur le bonton mode c'est good après la modification de la ligne.

Par contre pour le bouton 3d c'est toujours NOK, j'ai tenté ton scénario (bloc code) mais je ne sais pas si il est passé correctement. Dans les logs il me met :

[2022-03-28 11:57:25][SCENARIO] -- Start : Scenario lance manuellement. [2022-03-28 11:57:25][SCENARIO] - Exécution du sous-élément de type [action] : code [2022-03-28 11:57:25][SCENARIO] Exécution d'un bloc code [2022-03-28 11:57:25][SCENARIO] Call to a member function setDisplay() on bool [2022-03-28 11:57:25][SCENARIO] Fin correcte du scénario

N'étant pas expert php je ne sais pas dire si le call to a member... est une erreur ou si c'est un warning. Peut être dois-je éditer un autre fichier à la main ? j'ai recharger tout (vider le cache & co) mais toujours pas d'icône 3d.

Merci.

Sébastien

Pingu95 commented 2 years ago

j'ai contourné le truc en modifiant l'icone de chaque unité directement a la main ici :

image

Je ne sais pas si c'est conventionnel de le faire à la main, mais ça résout le problème :)

abarrau commented 2 years ago

Sébastien, Etrange qu'avec le bloc code ça passe pas ... Sinon ta 2nde méthode est très bien, j'ai aussi fait une adaptation pour permettre l'utilisation de cette fonction "icone" ; et ça allait être mon recours ... mais t'as trouvé avant ;) Cdt, Aurélien

Pingu95 commented 2 years ago

Ok ça marche :) Pour moi c'est donc tout bon :)

Merci encore pour la maj !

Cdlt, Sébastien

Pingu95 commented 2 years ago

Hello Aurélien,

J'ai un nouveau petit souci, je pense que ce n'est pas grand chose mais quand j'ajoute sur un design une commande unique d'un bloque de clim (et non l'équipement complet qui lui ne pose pas problème) j'ai une erreur sur la page du design indiquant :

Fichier Ligne Message
https://192.168.1.xxx/index.php?v=d&p=plan&plan_id=12&fullscreen=0 10 SyntaxError: missing ) after condition

Que ce soit un mode (chauffage/clim/ventilation) ou une vitesse ou n'importe quel commande. Je suppose qu'il manque une ( quelque part :)

image

image

Si tu as une idée si tu sais reproduire le souci... Ce n'est pas très bloquant c'est juste un warning la commande marche bien mais bon, c'est pas l'idéale vu qu'il y a une erreur.

Très bonne journée.

Cdlt, Sébastien

abarrau commented 2 years ago

Brj Sébastien, Je pense avoir trouvé d'où cela pouvait venir; mais ça va pas être évident à corriger. C'est un problème au niveau de l'affichage des icones. Normalement si tu reprends la modification des icones, avec la procédure que tu avais proposé plus haut (mise à jour de l'icone, via la config de la commande), le problème n'existe pas. Je vais réfléchir à ce que je peux faire ... Cdt, Aurélien

Pingu95 commented 2 years ago

Bonjour Aurélien,

En effet ton fixe fonctionne bien. C'est bizarre car contrairement a la direction 3D où il n'y avait pas d'icone affecté (cf plus haut) les cmd que j’ appel en directe avaient elles, déjà leur icônes comme il faut. Il a fallut le "supprimer" (en cliquant dessus) et en remettant le même... Donc visuellement sur la commande niveau config (gui en tout cas) ça ne change rien, par contre ça doit réécrire une entrée quelque part qui fixe l'erreur de ")" ...

Je n'utilise que 2 cmd en direct sur un seul de mes 3 splits donc cette solution est jouable à mon niveau sans problème.

Après c'est peut être mieux pour ton plugins quand tu le passera en version stable si tu trouve une solution plus global. Après je ne sais pas si grand monde s'amuse à faire des appels à des commendes en direct le widget de base est déjà très bien et fait le job à la perfection :)

Bref, pour moi c'est tout bon !

Très bonne journée et encore merci pour les tips/fix !

a+ Sébastien

abarrau commented 2 years ago

Bjr Sébastien, Je viens de passer une 1.4.1 qui reprend les corrections identifiées ensemble et j'ai rajouté une mise à jour complète des "generic_type". J'espère que ça passera chez vous. Je passerais en stable. Cdt, Aurélien

Pingu95 commented 2 years ago

Bonjour Aurélien,

Je viens de pousser la mise à jour. Pas de souci a première vu pour tout ce que nous avons vu ensemble ça reste pareil :) Le click sur l'icone des modes est toujours ok et les icone sont toujours là. Par contre l'appel à une cmd directement depuis un design me met toujours l'erreur : https://192.168.1.101/index.php?v=d&p=plan&plan_id=14&fullscreen=0 | 10 | SyntaxError: missing ) after condition

Comme je viens d'une v3 maj en v4.2 directement il y a peut être des choses qui se sont mal passé et je t'avoue que j'ai vraiment pas le courage de supprimer le plugin pour tout refaire :( Le souci de cette erreur est je pense relativement peu impactant c'est juste un warning et je pense que les gens ne font pas spécialement appel a des cmd en direct. Est ce que tu arrives à reproduire cette erreur chez toi ? Sinon c'est que ma mise à jour qui a du poser problème et du coup ne t'embête pas avec ça. En modifiant l’icône a la main ça supprime toujours bien l'erreur (je t'avoue que je ne comprend pas trop pourquoi car l'icône est bien présent a la base, donc c'est l'action de supprimer l’icône de la config et de le remettre (le même) qui doit faire une écriture quelque part en plus... A mon niveau en tout cas, ce n'est plus un problème, j'ai fais le nécessaire pour les 3-4 cmd que j'appelle en direct. A toi de voir si tu veux la passer en stable ou continuer de débug un peu... A part ces appels en direct, tout le reste est ok je pense.

Encore merci.

A+ Sébastien

abarrau commented 2 years ago

Hello, vraiment très étrange cette histoire, c'est comme si la mise à jour ne se faisait pas avec le bon script chez vous. bon l'essentiel était de donner la main aux utilisateurs, comme tu l'as fait. Je vais le passer en stable cette semaine. Cdt, Aurélien

abarrau commented 2 years ago

En stable depuis le 14/07/22