Erreur suite à changement printEqLogic

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

Je reviens sur mon sujet d’hier, @kiboost

En fait, c’est un peu des deux, selon mon interprétation.

Si je comprends bien, printEqLogic devient donc asynchrone.

jeedom/core/blob/c0c794673eb3831b3863f132055a31537b1d1ab8/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;')

Donc, quand vient addCmdToTable, un peu après, les commandes sont chargées aux tabpanel précédemment ajoutés.

github.com

jeedom/core/blob/c0c794673eb3831b3863f132055a31537b1d1ab8/core/js/plugin.template.js#L123


      
              if (data.cmd[i].state.length > 50) {
                data.cmd[i]['htmlstate'] += '<br/>' + data.cmd[i].state.replaceAll('"','&quot;')
              }
              data.cmd[i]['htmlstate'] += '" >'
              data.cmd[i]['htmlstate'] += data.cmd[i].state.substring(0, 50) +  ' ' + data.cmd[i].unite
              data.cmd[i]['htmlstate'] += '<span>'
            } else {
              data.cmd[i]['htmlstate'] = '';
            }
            if (typeof addCmdToTable === 'function') {
              addCmdToTable(data.cmd[i])
            } else {
              jeeFrontEnd.pluginTemplate.addCmdToTableDefault(data.cmd[i])
            }
            jeedomUtils.initTooltips()
          }
          document.querySelectorAll('.cmdTableState').forEach(_cmdState => {
            jeedom.cmd.addUpdateFunction(_cmdState.getAttribute('data-cmd_id'), function(_options) {
              _options.display_value = String(_options.display_value).replace(/<[^>]*>?/gm, '')
              let cmd = document.querySelector('.cmdTableState[data-cmd_id="' + _options.cmd_id + '"]')
              if (cmd === null) {

Dans mon cas, j’ajoute des tabpanel dynamiquement en fonction d’une ‹ type › contenu dans la configuration de l’équipement, et par la suite les commandes associées à ce ‹ type › au bon tabpanel. (ex: pour l’équipement four : cavité 0, cavité 1, cavité 2) => dans le but de trier les commandes fournies.
Les tabpanel n’ont donc pas/ou à peine le temps d’être complétés que addCmdToTable est déjà appelé et donc ne peut remplir les données des commandes.
Résultat : tous les tabpanel sont vides. Et j’ai une erreur JS lors de la fonction JS changeType (_cmd vide).

Y a-t-il une astuce, solution, meilleure manière de faire ?

Loic · Février 8, 2023, 8:02

Bonjour
Sur la commande tu sais le tabpanel ? Si c’est le cas tu pourrais dans le addcmd tester sur le tabpanel existe et si non le créer puis faire l’insertion.

Flobul · Février 8, 2023, 8:10

Hello.

Oui, l’info est dans _cmd.configuration.component et toute la commande est affectée comme ça : $(‹ #table_cmd_ › + _cmd.configuration.component + ’ tbody’).html(tr);

J’y avais pas pensé, merci pour l’astuce.
Je teste ça demain

kiboost · Février 8, 2023, 8:14

La fonction print n’est pas asynch le settimeout sans delay attend simplement le prochain render du browser. C’est qqls ms

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

J’ai viré le timeout, à surveiller que çà ne pose pas de probleme sur d’autres plugins.

Flobul · Février 9, 2023, 2:38

J’allais dire qu’il fallait pas faire ça uniquement pour mon plugin, mais je viens de voir le message pour jMQTT.

Merci en tout cas.
Je vais quand même tester de passer sur l’idée de Loic (création à la volée) qui me semble plus correcte que la création au printEqLogic.

Flobul · Février 10, 2023, 10:56

Nickel, merci Loic.
Ca fonctionne impec.

system · Février 11, 2023, 10:57

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.