You are not logged in.

Announcement

 Téléchargez la dernière version stable de GLPI      -     Et vous, que pouvez vous faire pour le projet GLPI ? :  Contribuer
 Download last stable version of GLPI                      -     What can you do for GLPI ? :  Contribute

#1 2009-10-30 23:06:02

kabassanov
Member
Registered: 2009-10-06
Posts: 44

Numéro de vlan manquant en cas de création avec gabarit

Bonsoir,

Je me suis rendu compte que quand on utilise des gabarits pour créer un nouvel ordinateur, les connexions réseaux sont bien recopiées dans la nouvelle instance ordinateur, mais en revanche les numéros de vlan affectés ne le sont pas.

Une petite correction donnerait un truc du genre (j'ai utilisé la @version $Id: computer.class.php 8375 2009-06-17 09:21:56Z remi $), mais ca ne change pas grand chose...)

--- glpi/inc/computer.class.php 2009-09-07 09:21:16.000000000 +0200
+++ /usr/local/share/glpi/inc/computer.class.php        2009-10-30 21:55:23.000000000 +0100
@@ -415,7 +415,15 @@ class Computer extends CommonDBTM {
                                        unset($np->fields["ifmac"]);
                                        unset($np->fields["netpoint"]);
                                        $np->fields["on_device"]=$newID;
-                                       $np->addToDB();
+                                       $portnewid=$np->addToDB();
+                                       $query_vlan="SELECT FK_vlan
+                                                    FROM glpi_networking_vlan
+                                                    WHERE FK_port=".$data["ID"].";";
+                                       $result_vlan=$DB->query($query_vlan);
+                                       if ($portnewid && $DB->numrows($result_vlan)>0){
+                                               $data_vlan=$DB->fetch_array($result_vlan);
+                                               assignVlan($portnewid,$data_vlan["FK_vlan"]);
+                                       }
                                }
                        }


Vous en pensez quoi?

Offline

#2 2009-10-31 13:57:29

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Numéro de vlan manquant en cas de création avec gabarit

c'est effectivement quelquechose à corriger.

ticket ajouté pour ne pas oublier.
https://forge.indepnet.net/issues/show/1904


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

#3 2009-11-01 10:21:09

remi
GLPI-DEV
From: Champagne
Registered: 2007-04-28
Posts: 7,127
Website

Re: Numéro de vlan manquant en cas de création avec gabarit

Corrigé et à tester
https://forge.indepnet.net/repositories … i?rev=9195   (ligne 418 et +)

A noter cependant que cette modification a surtout du sens pour les équipements réseaux, car pour les autres matériels, les VLAN sont remplacés par ceux de l'actif lors de la connexion.

+


Dév. Fedora 29 - PHP 5.6/7.0/7.1/7.2/7.3/7.4 - MariaDB 10.3 - GLPI master
Certifié ITILv3 - RPM pour Fedora, RHEL et CentOS sur https://blog.remirepo.net/

Offline

#4 2009-11-01 12:46:36

kabassanov
Member
Registered: 2009-10-06
Posts: 44

Re: Numéro de vlan manquant en cas de création avec gabarit

Je ne suis pas sur de comprendre votre dernière explication...

Je pense que définir un vlan peut avoir du sens pour n'importe quel équipement possédant des interfaces réseau.

Ce qui d'ailleurs me fait penser que la correction dont on parle doit concerner également au moins les imprimantes et les téléphones. Et non pas uniquement les ordinateurs comme j'avais proposé dans le premier post...

Offline

#5 2009-11-01 18:47:40

remi
GLPI-DEV
From: Champagne
Registered: 2007-04-28
Posts: 7,127
Website

Re: Numéro de vlan manquant en cas de création avec gabarit

Ma remarque concernait le fait que créer un port affecté à un VLAN pour un ordinateur présentait un intérêt limité car lorsqu'on connecte sur port sur celui d'un actif réseau, les VLAN sont remplacés par ceux de l'actif.

Effectivement, il faut que je le corrige pour tous les types...

https://forge.indepnet.net/repositories/revision/5/9198

+


Dév. Fedora 29 - PHP 5.6/7.0/7.1/7.2/7.3/7.4 - MariaDB 10.3 - GLPI master
Certifié ITILv3 - RPM pour Fedora, RHEL et CentOS sur https://blog.remirepo.net/

Offline

Board footer

Powered by FluxBB