Ordre des commandes des équipements modifié lors de la sauvegarde de l'objet

Phpvarious · Août 19, 2024, 7:35

Bonsoir,

@Loic j’ai peut être un début de piste :

Déja le Core commence les order a 0 normalement, hors en js il commence a 1:

jeedom/core/blob/alpha/desktop/common/js/ui.js#L99


      
          @view
          save eqlogics/scenarios size/order
          */
          saveWidgetDisplay: function(_params) {
            if (init(_params) == '') {
              _params = {}
            }
            var cmds = []
            //Get all commands for display default eqLogic:
            document.querySelectorAll('div.eqLogic-widget:not(div.eqLogic_layout_table)').forEach(function(el_eqLogic) {
              var order = 1
              el_eqLogic.querySelectorAll('.cmd').forEach(function(el_cmd) {
                var cmd = {}
                cmd.id = el_cmd.getAttribute('data-cmd_id') || undefined
                cmd.order = order
                cmds.push(cmd)
                order++
              })
            })
            //Get all commands for display table eqLogic:
            document.querySelectorAll('div.eqLogic-widget.eqLogic_layout_table').forEach(function(el_eqLogic) {

Mais même si il commence a 0, il y a toujours un soucis car les cmd qui sont masquées ne sont pas considérées car pas dans le DOM, et donc lors du save (Exit edit mode) les commandes sont réordonnées et cela peut créer un order identique d’une autre commande.

Exemple Virtuel :

Si je vais en config avancé → Informations de chaque commande je constate bien

order 1 pour Etat
order 2 pour on
order 3 pour off

Maintenant si j’édite le dashboard et quitte le mode édition (j’ai modifier var order = 0 dans le core) :

config avancé → Informations de chaque commande je constate bien

order 0 pour Etat
order 1 pour on
order 1 pour off

On voit bien que on et off on switchés visuellement, et que il y a 2x order 1

En faite en regardant de plus près le code je pense qu’il ne faudrait pas faire une save des order des cmd lorsqu’on sort du mode édition en dashboard car si l’user veut modifier l’ordre des cmd en dashboard il utilisera la modale avec sont propre save :

Il faudrait conditionner cette partie :

github.com

jeedom/core/blob/alpha/desktop/common/js/ui.js#L98-L120


      
          document.querySelectorAll('div.eqLogic-widget:not(div.eqLogic_layout_table)').forEach(function(el_eqLogic) {
            var order = 1
            el_eqLogic.querySelectorAll('.cmd').forEach(function(el_cmd) {
              var cmd = {}
              cmd.id = el_cmd.getAttribute('data-cmd_id') || undefined
              cmd.order = order
              cmds.push(cmd)
              order++
            })
          })
          //Get all commands for display table eqLogic:
          document.querySelectorAll('div.eqLogic-widget.eqLogic_layout_table').forEach(function(el_eqLogic) {
            var order = 1
            el_eqLogic.querySelectorAll('.cmd').forEach(function(el_cmd) {
              var cmd = {}
              cmd.id = el_cmd.getAttribute('data-cmd_id') || undefined
              cmd.line = el_cmd.closest('td')?.getAttribute('data-line') || undefined
              cmd.column = el_cmd.closest('td')?.getAttribute('data-column') || undefined
              cmd.order = order
              cmds.push(cmd)

This file has been truncated. show original

avec un if (init(_params['dashboard'], 0) == 0) { par exemple.