search

fusioninventory / fusioninventory-for-glpi

FusionInventory plugin for GLPI
http://www.FusionInventory.org/
GNU Affero General Public License v3.0
362 stars 150 forks source link

Bug moniteur déjà connecté #2225

Open demonisto opened 7 years ago

demonisto commented 7 years ago

Bonjour,

Si on déplace un écran d'un poste Pa sur un autre poste Pb et si on ne déconnecte pas l'écran du poste Pa. Alors l'écran ne remontera jamais sur le poste Pb, même si on force une remontée d'inventaire X fois.

Voir les sujet : http://forum.glpi-project.org/viewtopic.php?id=157806 http://forum.glpi-project.org/viewtopic.php?id=155003

demonisto commented 7 years ago

Bonjour, je me suis apperçu il y a peu d'une erreur venant de ce code. Je ne tenais pas compte des ordinateurs qui n'avait pas d'écran et cela remplissait le fichier sql_error.log

J'ai indiqué les modifications qu'il faut faire sur les sujets.

Mais voici les modifications : Le code après modification :

A la ligne 1311, je déclare une variable qui permettra de faire une requete sur les écrans connectés sur d'autre ordinateur plus tard :

   ` $a_monitors_for_condition = 0;`

A la ligne 1325, j'alimente la variable à ce moment pour éviter de faire une boucle sur après sur le tableau "$a_monitor":

 if($a_monitors_for_condition != 0){
                        $a_monitors_for_condition .= ',';
                    }
                    $a_monitors_for_condition .= $data['found_equipment'][0];

En gras les modifications apportés $a_monitors_for_condition = 0; if($a_monitors_for_condition != 0){