Plugin massocsimport: non prise en compte de tag_exclude

pthoirey · 2009-09-12 17:08:31

Environnement
GLPI 0.72
Plugin: Import OCS 1.3.0

Contexte et constat
Le champ TAG de la table accountinfo de ma base de données OCS contient la valeur VM lorsque l'ordinateur correspondant est une instance virtuelle (VMware, QEMU, OpenVZ, VServer). Je peux ainsi définir dans GLPI deux entrées pour une même instance OCS:
- La première entrée va me permettre d'importer et synchroniser les instances physiques en excluant l'importation des entrées OCS ayant le tag VM.
- La deuxième entrée va me permettre d'importer et synchroniser les instances virtuelles en limitant l'importation aux entrées OCS ayant le tag VM.

Ce mécanisme fonctionne parfaitement en manuel via le menu Outils > OCSNG.

Toutefois, si j'utilise le script plugins/massocsimport/scripts/ocsng_fullsync.sh l'exclusion n'est pas opérante. En effet, ma première entrée importe et synchronise toutes les instances sans tenir compte de l'exclusion.

Correctif proposé
Ajouter les lignes de code suivantes au fichier plugins/massocsimport/scripts/ocsng_fullsync.php en ligne 313.

if (!empty ($cfg_ocs["tag_exclude"])) {
    $splitter = explode("$", $cfg_ocs["tag_exclude"]);
    if (count($splitter)) {
        $query_ocs .= " AND accountinfo.TAG NOT IN ('" . $splitter[0] . "'";
        for ($i = 1; $i < count($splitter); $i++) {
            $query_ocs .= ",'" . $splitter[$i] . "'";
        }
        $query_ocs .= ")";
    }
}

wawa · 2009-09-14 08:47:20

Merci du retour et de la proposition de patch
https://forge.indepnet.net/repositories/revision/124/8
en fait il existe une fonction du coeur qui gère ça, mais on avait oublié de l'utiliser dans le plugin

pthoirey · 2009-09-14 17:59:53

J'ai récupéré la nouvelle version (de ce jour) du fichier plugins/massocsimport/scripts/ocsng_fullsync.php (md5sum 0590565c7fc67df69e1862681100d18f) pour constater que la modification apportée ne fonctionne pas: aucun ordinateur n'est importé ou synchronisé via le script alors que j'ai des ordinateurs qui apparaîssent dans l'interface "Outils > OCSNG"!...

remi · 2009-09-14 19:28:11

Il doit manquer le AND entre les conditions....

+

pthoirey · 2009-09-14 19:43:50

En effet, cela marche nettement mieux. Merci.

remi · 2009-09-14 19:58:57

C'est possible de tester la dernière modification pour confirmer la correction ?

Merci d'avance.
https://forge.indepnet.net/repositories/revision/124/9

+

pthoirey · 2009-09-15 08:32:19

Tests effectués avec la nouvelle version du fichier plugins/massocsimport/scripts/ocsng_fullsync.php (md5sum e03b15358ce1972c94b2b6ce7cfef1ee) pour constater que cela fonctionne parfaitement.

Merci.

remi · 2009-09-15 08:40:00

Merci pour le test.

Je ferme le sujet.

Forum GLPI-Project

Announcement

#1 2009-09-12 17:08:31

Plugin massocsimport: non prise en compte de tag_exclude

#2 2009-09-14 08:47:20

Re: Plugin massocsimport: non prise en compte de tag_exclude

#3 2009-09-14 17:59:53

Re: Plugin massocsimport: non prise en compte de tag_exclude

#4 2009-09-14 19:28:11

Re: Plugin massocsimport: non prise en compte de tag_exclude

#5 2009-09-14 19:43:50

Re: Plugin massocsimport: non prise en compte de tag_exclude

#6 2009-09-14 19:58:57

Re: Plugin massocsimport: non prise en compte de tag_exclude

#7 2009-09-15 08:32:19

Re: Plugin massocsimport: non prise en compte de tag_exclude

#8 2009-09-15 08:40:00

Re: Plugin massocsimport: non prise en compte de tag_exclude

Board footer