[BETA] Vue JSON ne fonctionne pas/plus

jerryzz · Février 8, 2023, 5:56

Bonjour,

Je suis en

jeedom alpha 4.4.0
jmqtt beta version du 2023-02-07 01:01:23

Depuis quelques temps (impossible de dire quand), je n’ai plus la vue JSON.
Qu’importe la vue choisie, la mise en page ne change pas.
Que j’ajoute manuellement une ou des commande info ou action, cela reste sans effet lorsque je sauvegarde.

Vue classique:

Vu JSON:

Que faire ?

Merci pour votre aide

Bad · Février 8, 2023, 7:39

Hello,

Dans ce post tu semblais dire que c’était OK pour la sauvegarde avec la version 2023-02-04, non ?

C’est KO depuis la version du 2023-02-07 ?
As-tu désactivé jQuery ? (je ne pense pas)
Peux-tu me fournir d’autres traces intéressantes et captures d’écran stp ?

Bad

Phpvarious · Février 8, 2023, 7:42

Bonsoir,
@Bad, je confirme aussi.
Je n’avais pas testé sur le json.

Bad · Février 8, 2023, 7:57

OK, je n’arrive pas à reproduire en v4.3.15 et c’est plutôt pas mal.

Je pense que ça vient de ce commit :

Ca remarche en supprimant les tests if ($("#table_cmd").sortable("instance")), mais ce commit était justement là pour corriger un problème en v4.4…

EDIT : Pas trop de temps cette semaine, si ça ne concerne que l’alpha, je regarde de près ce weekend.

Phpvarious · Février 8, 2023, 8:25

Pour info, même en commentant la totalité du PR + sortable , JSON → HS sans aucune erreur js :

jMQTT_1

jerryzz · Février 8, 2023, 9:03

Oui, j’ai exactement le même symptôme (le petit décalage)

jerryzz · Février 8, 2023, 9:03

Ok, merci

Phpvarious · Février 8, 2023, 9:37

J’ai peut-être une piste…

@kiboost peut sans doute confirmer

Voici l’ordre des évènements :

en stable : printEqLogic puis addCmdToTable
en alpha : addCmdToTable puis printEqLogic

C’est donc peut-être le core le « problème »

Edit :
J’ai trouvé, c’est le setTimeout() qui rend printEqlogic() asynchrone :

github.com

jeedom/core/blob/alpha/core/js/plugin.template.js#L104

      
        
            },
            success: function(data) {
              document.getElementById('div_pageContainer').querySelectorAll('.eqLogicAttr').jeeValue('')
              if (isset(data) && isset(data.timeout) && data.timeout == 0) {
                data.timeout = ''
              }
              document.getElementById('div_mainContainer').setJeeValues(data, '.eqLogicAttr')
              if (!isset(data.category.opening)) try { document.querySelector('input[data-l2key="opening"]').checked = false } catch (e) { }
            
            
  if (typeof printEqLogic === 'function') {
                setTimeout(() => { printEqLogic(data) })
              }
              document.querySelectorAll('.cmd').remove()
              for (var i in data.cmd) {
                if (data.cmd[i].type == 'info') {
                  data.cmd[i].state = String(data.cmd[i].state).replace(/<[^>]*>?/gm, '')
                  data.cmd[i]['htmlstate'] =  '<span class="cmdTableState"'
                  data.cmd[i]['htmlstate'] += 'data-cmd_id="' + data.cmd[i].id+ '"'
                  data.cmd[i]['htmlstate'] += 'title="{{Date de valeur}} : ' + data.cmd[i].valueDate + '<br/>{{Date de collecte}} : ' + data.cmd[i].collectDate
                  if (data.cmd[i].state.length > 50) {
                    data.cmd[i]['htmlstate'] += '<br/>' + data.cmd[i].state.replaceAll('"','&quot;')

En mettant simplement printEqLogic(data) c’est good.

kiboost · Février 9, 2023, 7:20

Faudrait s’assurer que ce ne pose pas de problème sur d’autres plugins.

PS : j’ai supprimé le timeout, à surveiller …

jerryzz · Février 9, 2023, 1:03

Je viens de faire cette modif.

Et cela refonctionne de nouveau !

Je vous tiens au courant s’il y a des effets de bords sur d’autres plugins.

Merci

jerryzz · Février 9, 2023, 2:17

Bon, je me suis peut-être emaballé trop vite: La sauvegarde ne fonctionne pas lorsque j’ajoute deux entrées:

Alors, du coup, je supprime toutes les lignes (sauf celles que je veux garder)

Et lorsque je sauvegarde, il me dit bien que 2 commandes sont ajoutées, mais elles ne s’affichent pas:

Phpvarious · Février 9, 2023, 8:23

Bonsoir,

J’ai une piste :

dans la function saveEqLogic du plugin il y a une boucle qui permet de supprimer les commandes qui ont un id et name vide :

github.com

Domochip/jMQTT/blob/beta/desktop/js/jMQTT.js#L727


      
          		for (var c of _eqLogic.cmd) {
          			if (!inArray(new_cmds, c)) {
          				if (c.type == 'info') {
          					//console.log('loop: add info ' + c.configuration.topic + c.configuration.jsonPath);
          					recursiveAddCmdFromTopic(c.configuration.topic, c.configuration.jsonPath);
          				}
          				else {
          					// Action commands are added directly
          					addCmd(c);
          				}
          			}
          		}
          
          		_eqLogic.cmd = new_cmds;
          
          		// JSON view: disable the sortable functionality
          		jmqtt.setCmdsSortable(false);
          	} else {
          		// CLASSIC view button is active
          		for (var c of _eqLogic.cmd) {
          			c.tree_id = (n_cmd++).toString();

hors le core semble maintenant envoyer un null pour l’id
En remplaçant par
if ((_eqLogic.cmd[i].id == "" || _eqLogic.cmd[i].id === null) && _eqLogic.cmd[i].name == "") {

il n’y a plus l’erreur « Le nom de la commande ne peut pas être vide … »

Mais, et oui il y a un mais

Lors du save, la commande n’est pas réellement save, une nouvelle erreur dans le http.log :

PHP Notice: Array to string conversion in /var/www/html/core/class/DB.class.php on line 84
PHP Notice: Array to string conversion in /var/www/html/core/php/utils.inc.php on line 140

En mettant quelque log supplémentaire, je me suis aperçu qu’après ce traitement :

github.com

Domochip/jMQTT/blob/beta/desktop/js/jMQTT.js#L735


      
          					addCmd(c);
          				}
          			}
          		}
          
          		_eqLogic.cmd = new_cmds;
          
          		// JSON view: disable the sortable functionality
          		jmqtt.setCmdsSortable(false);
          	} else {
          		// CLASSIC view button is active
          		for (var c of _eqLogic.cmd) {
          			c.tree_id = (n_cmd++).toString();
          		}
          
          		// Classical view: enable the sortable functionality
          		jmqtt.setCmdsSortable(true);
          	}
          
          	// Show UI elements depending on the type
          	if ((_eqLogic.configuration.type == 'eqpt' && (_eqLogic.configuration.eqLogic == undefined || _eqLogic.configuration.eqLogic < 0))

_eqLogic.cmd, ne semble plus un array :

Avant

Après :

En commentant cette ligne _eqLogic = jmqtt.substractKeys(_eqLogic, $('#brokertab').getValues('.eqLogicAttr')[0]);

plus aucun soucis.

@Bad , je te laisse voir ceci, car je ne comprend pas a quoi sert cette ligne
du moins j’ai bien trouvé la fonction dans jMQTT.function.js, mais je ne la comprend pas

jerryzz · Février 9, 2023, 8:29

je suis épaté par tant de précisions. Cela dépasse mes compétences techniques, et de loin

Phpvarious · Février 9, 2023, 8:51

Re @kiboost , je revient vers toi pour la nouvelle méthode SortableJS.
Si je comprend bien, en 4.4, il n’est plus possible de faire un enable/disable.
J’ai regardé la lib, est j’ai pas trouvé d’équivalent.

Ne peut-on pas mettre une class en plus (Ex: .static) dans le filter du Sortable.create pour laisser au Dev la possibilité de désactiver le dragg sur certains éléments ?.

J’ai fait le test sur jMQTT, et c’est convainquant

kiboost · Février 9, 2023, 8:57

Je peux registerer le sortable dans jeeFrontEnd.pluginTemplate.cmdSortable

Pour le desactiver : jeeFrontEnd.pluginTemplate.cmdSortable.options.disabled = true

kiboost · Février 9, 2023, 9:01

C’est poussé, je le store aussi sur la table :

document.getElementById(‹ table_cmd ›)._sortable

en verifiant qu’il n’est pas null

Phpvarious · Février 9, 2023, 9:38

Je vient de tester… et c’est good comme d’habitude
Merci, je veut dire Bad te remercie

Bad · Février 11, 2023, 12:50

Hello,

Je viens d’implémenter le correctif suivant en m’inspirant de votre travail :

Si vous pouviez tester ce serait top

Si c’est OK pour vous, je le passe en stable cette nuit.

Phpvarious · Février 11, 2023, 1:20

Hello @Bad,

Testé sur la beta du 2023-02-07 01:01:23 avec Alpha
Testé sur stable du 2023-02-07 01:01:25 avec 4.3.15

Phpvarious · Février 11, 2023, 1:31

Par contre sur Alpha toujours les problème cité ci-dessus.