Plugin applicatifs: ajout de champs

pthoirey · 2009-05-17 18:38:37

Environnement
GLPI SVN 20090517
Plugins:
- applicatifs SVN 20090517
- appweb SVN 20090517
- domain SVN 20090517

Suggestions
1. À l'instar de ce que l'on a dans le cœur de GLPI avec les matériels (ordinateurs, moniteurs, etc.), il serait intéressant de pouvoir visualiser la date de dernière modification pour chaque applicatif. En effet, cela permettrait de mieux suivre l'activité de gestion (ajout de nouveaux applicatifs, mise à jour) de l'inventaire des applicatifs.

2. Pour ceux qui ont un contrat de service dépendant du statut des applicatifs (critique, production, qualification, prototype, test, par exemple) qu'ils gérent, il serait bénéfique de disposer d'un champ de type dropdown pour renseigner le statut.

Last edited by pthoirey (2009-05-19 06:30:33)

pthoirey · 2009-05-20 07:47:08

Précisions
J'ai implémenté le point 2 à partir de la version 1.4 du plugin applicatif. Je peux tenter d'en faire de même avec la version SVN.

Last edited by pthoirey (2009-05-20 21:47:07)

tsmr · 2009-05-25 09:04:23

Un statut différent par entités ? ou commun à toute ? ou le statut de glpi ?

remi · 2009-05-25 09:26:12

Je me pose une question : est-ce l'applicatif qui a un statut (prod, qualif) ou la laison composant/applicatifs (installé sur serveur 1 en prod, sur serveur 2 en qualif)

J'ai l'impression que le statut d'un applicatif (prod, qualif) n'est pas exactement la même notion que le statut d'un matériel (stock, prod, HS)

Pour l'instant je pense qu'il est facile de créer 2 instances, avec le statut dans le nom.

Cette fonctionnalité, assez triviale à implémenter, nécessite une petite réflexion.

+

pthoirey · 2009-05-25 20:22:04

tsmr wrote:

Un statut différent par entités ? ou commun à toute ? ou le statut de glpi ?

Un statut commun à toutes les entités me semble être la bonne solution.

En revanche, je ne pense pas qu'il faille utiliser le statut des matériels de GLPI, ni même inférer du statut d'un matériel associé à un applicatif le statut de l'applicatif. En effet, les préoccupations et objectifs des personnes qui gèrent les matériels ne sont pas les mêmes que celles des personnes qui gèrent les applicatifs. De fait, la typologie des applicatifs (en termes de statut) n'a pas à être identique à celle des matériels. On peut, par exemple, avoir des applicatifs pilote, donc qui ne sont pas en production, sur des serveurs de production (la réciproque n'étant évidemment pas souhaitable).

De plus, les matériels et applicatifs sont de natures différentes. De fait, la typologie des applicatifs (en termes de statut) ne peut pas être identique à celle des matériels. Voir sur ce point la remarque ci dessous.

remi wrote:

J'ai l'impression que le statut d'un applicatif (prod, qualif) n'est pas exactement la même notion que le statut d'un matériel (stock, prod, HS)

pthoirey · 2009-05-26 06:47:32

Ci-joint le patch pour ajouter la date de dernière modification au plugin applicatifs (point 1 de mon message d'origine).

diff -Naur applicatifs/hook.php applicatifs.new/hook.php
--- applicatifs/hook.php        2009-05-25 20:39:28.127101817 +0200
+++ applicatifs.new/hook.php    2009-05-26 06:37:46.683103232 +0200
@@ -199,6 +199,12 @@
                $sopt[PLUGIN_APPLICATIFS_TYPE][8]['field']='name';^M
                $sopt[PLUGIN_APPLICATIFS_TYPE][8]['linkfield']='FK_Groups';^M
                $sopt[PLUGIN_APPLICATIFS_TYPE][8]['name']=$LANG['common'][35];^M
+
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['table']='glpi_plugin_applicatifs';
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['field']='date_mod';
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['linkfield']='date_mod';
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['name']=$LANG['common'][26];
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['datatype']='datetime';
                ^M
                $sopt[PLUGIN_APPLICATIFS_TYPE][30]['table']='glpi_plugin_applicatifs';^M
                $sopt[PLUGIN_APPLICATIFS_TYPE][30]['field']='ID';^M
@@ -821,4 +827,4 @@
        }^M
 }^M
 ^M
-?>
\ Pas de fin de ligne à la fin du fichier.
+?>
diff -Naur applicatifs/inc/plugin_applicatifs-1.5.0-empty.sql applicatifs.new/inc/plugin_applicatifs-1.5.0-empty.sql
--- applicatifs/inc/plugin_applicatifs-1.5.0-empty.sql  2009-05-25 20:39:28.123101919 +0200
+++ applicatifs.new/inc/plugin_applicatifs-1.5.0-empty.sql      2009-05-26 06:37:46.687099754 +0200
@@ -7,6 +7,7 @@
        `recursive` tinyint(1) NOT NULL default '0',
        `name` varchar(255) collate utf8_unicode_ci NOT NULL default '',
        `deleted` smallint(6) NOT NULL default '0',
+       `date_mod` timestamp NOT NULL,
        `type` tinyint(4) NOT NULL default '1',
        `comments` text,
        `notes` LONGTEXT,
@@ -112,4 +113,4 @@
 INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','2','2','0');
 INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','3','3','0');
 INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','4','4','0');
-INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','5','5','0');
\ Pas de fin de ligne à la fin du fichier.
+INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','5','5','0');
diff -Naur applicatifs/inc/plugin_applicatifs-1.5.0-update.sql applicatifs.new/inc/plugin_applicatifs-1.5.0-update.sql
--- applicatifs/inc/plugin_applicatifs-1.5.0-update.sql 2009-05-25 20:39:28.123101919 +0200
+++ applicatifs.new/inc/plugin_applicatifs-1.5.0-update.sql     2009-05-26 06:37:46.687099754 +0200
@@ -2,6 +2,7 @@
 ALTER TABLE `glpi_plugin_applicatifs` ADD INDEX `FK_entities` (`FK_entities`);
 ALTER TABLE `glpi_plugin_applicatifs` ADD INDEX `deleted` (`deleted`);
 ALTER TABLE `glpi_plugin_applicatifs` ADD `FK_groups` int(11) NOT NULL default '0';
+ALTER TABLE `glpi_plugin_applicatifs` ADD date_mod timestamp NOT NULL;
 ALTER TABLE `glpi_dropdown_plugin_applicatifs_type` ADD INDEX `FK_entities` (`FK_entities`);

 -- Cette table sert à enregistrer les champs personnalisables d'un applicatif
@@ -28,4 +29,4 @@
 ALTER TABLE `glpi_plugin_applicatifs_profiles` DROP COLUMN `interface` , DROP COLUMN `is_default`;

 ALTER TABLE `glpi_plugin_applicatifs` CHANGE `tech` `FK_users` int(4);
-ALTER TABLE `glpi_plugin_applicatifs_profiles` ADD `open_ticket` char(1) default NULL;
\ Pas de fin de ligne à la fin du fichier.
+ALTER TABLE `glpi_plugin_applicatifs_profiles` ADD `open_ticket` char(1) default NULL;

A noter: le champ "date_mod" ajouté à la table glpi_plugin_applicatifs est de type "timestamp" plutôt que "datetime". Sa valeur est donc modifié automatiquement à chaque modification d'un champ d'une entrée de la table.

remi · 2009-05-26 06:52:32

Le champ date_mod doit être de type "datetime", sa valeur est positionnée automatiquement par GLPI lors d'une mise à jour (commondbtm / add + update).

Pour que MySQL s'occupe de la MAJ, il faudrait utiliser les attributs
- DEFAULT CURRENT_TIMESTAMP (lors d'un ajout)
- ON UPDATE CURRENT_TIMESTAMP (lors d'une mise à jour)

GLPI n'utilise pas ces attributs, ce qui permet notamment de gérer une liste de champs n'entrainant pas la modification de date_mod (ex la date de dernière connexion d'un utilisateur)

+

pthoirey · 2009-05-26 08:52:35

remi wrote:

Pour que MySQL s'occupe de la MAJ, il faudrait utiliser les attributs
- DEFAULT CURRENT_TIMESTAMP (lors d'un ajout)
- ON UPDATE CURRENT_TIMESTAMP (lors d'une mise à jour)

Effectivement, mais à partir de MySQL 5.

Pour le reste, merci pour l'explication concernant la gestion des dates par le coeur de GLPI.

PS Dois-je poster un nouveau patch?

pthoirey · 2009-06-04 09:10:30

Ci-joint le patch qui intègre les deux ajouts de champ suggérés dans mon message d'origine (date de modification et champ statut). Ce patch s'appuie sur la version SVN du plugin applicatifs du 03/06/2009.

J'espère que ces modifications seront intégrées.

diff -Naur applicatifs/hook.php applicatifs.new/hook.php
--- applicatifs/hook.php        2009-06-04 08:59:42.517970805 +0200
+++ applicatifs.new/hook.php    2009-06-04 08:59:24.457473190 +0200
@@ -121,6 +121,7 @@
        if ($plugin->isActivated("applicatifs"))
                return array(
                        "glpi_dropdown_plugin_applicatifs_type"=>array("glpi_plugin_applicatifs"=>"type"),
+                       "glpi_dropdown_plugin_applicatifs_state"=>array("glpi_plugin_applicatifs"=>"state"),
                        "glpi_entities"=>array("glpi_plugin_applicatifs"=>"FK_entities",
                                                                "glpi_dropdown_plugin_applicatifs_type"=>"FK_entities"),
                        "glpi_plugin_applicatifs"=>array("glpi_plugin_applicatifs_device"=>"FK_applicatif"),
@@ -138,7 +139,7 @@

        $plugin = new Plugin();
        if ($plugin->isActivated("applicatifs"))
-               return array("glpi_dropdown_plugin_applicatifs_type"=>$LANG['plugin_applicatifs']['setup'][2]);
+               return array("glpi_dropdown_plugin_applicatifs_type"=>$LANG['plugin_applicatifs']['setup'][2],"glpi_dropdown_plugin_applicatifs_state"=>$LANG['state'][0]);
                // Harded coded : no edit possible
                //      "glpi_dropdown_plugin_applicatifs_relationtype"=>$LANG['plugin_applicatifs']['setup'][15]);
        else
@@ -199,6 +200,17 @@
                $sopt[PLUGIN_APPLICATIFS_TYPE][8]['field']='name';
                $sopt[PLUGIN_APPLICATIFS_TYPE][8]['linkfield']='FK_Groups';
                $sopt[PLUGIN_APPLICATIFS_TYPE][8]['name']=$LANG['common'][35];
+
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['table']='glpi_plugin_applicatifs';
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['field']='date_mod';
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['linkfield']='date_mod';
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['name']=$LANG['common'][26];
+               $sopt[PLUGIN_APPLICATIFS_TYPE][9]['datatype']='datetime';
+
+               $sopt[PLUGIN_APPLICATIFS_TYPE][10]['table']='glpi_dropdown_plugin_applicatifs_state';
+                $sopt[PLUGIN_APPLICATIFS_TYPE][10]['field']='name';
+                $sopt[PLUGIN_APPLICATIFS_TYPE][10]['linkfield']='state';
+                $sopt[PLUGIN_APPLICATIFS_TYPE][10]['name']=$LANG['state'][0];

                $sopt[PLUGIN_APPLICATIFS_TYPE][30]['table']='glpi_plugin_applicatifs';
                $sopt[PLUGIN_APPLICATIFS_TYPE][30]['field']='ID';
@@ -831,4 +843,4 @@
        }
 }

-?>
\ Pas de fin de ligne à la fin du fichier.
+?>
diff -Naur applicatifs/inc/plugin_applicatifs-1.5.0-empty.sql applicatifs.new/inc/plugin_applicatifs-1.5.0-empty.sql
--- applicatifs/inc/plugin_applicatifs-1.5.0-empty.sql  2009-06-03 21:14:30.378071493 +0200
+++ applicatifs.new/inc/plugin_applicatifs-1.5.0-empty.sql      2009-06-04 08:59:24.461471680 +0200
@@ -14,6 +14,8 @@
        `FK_users` INT( 4 ) NOT NULL,
        `FK_groups` int(11) NOT NULL default '0',
        `relationtype` INT( 4 ) NOT NULL,
+       `date_mod` datetime default NULL,
+       `state` int( 4 ) NOT NULL default '0' COMMENT 'RELATION to glpi_dropdown_plugin_applicatifs_state (ID)',
        PRIMARY KEY  (`ID`),
        KEY `FK_entities` (`FK_entities`),
        KEY `deleted` (`deleted`),
@@ -51,6 +53,17 @@
        KEY `FK_entities` (`FK_entities`)
 ) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

+-- Cette table contient la liste des statuts des applicatifs
+-- This table contains the list of applications' statuses.
+DROP TABLE IF EXISTS `glpi_dropdown_plugin_applicatifs_state`;
+CREATE TABLE `glpi_dropdown_plugin_applicatifs_state` (
+  `ID` int(11) NOT NULL auto_increment,
+  `name` varchar(255) collate utf8_unicode_ci default NULL,
+  `comments` text collate utf8_unicode_ci,
+  PRIMARY KEY  (`ID`),
+  KEY `name` (`name`)
+) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;
+
 -- Cette table contient la liste des droits (aucun, lecture, écriture)
 -- associé à un profil utilisateur pour le plugin Applicatifs.
 -- This table contains the list of rights (none, read, write)
@@ -112,4 +125,4 @@
 INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','2','2','0');
 INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','3','3','0');
 INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','4','4','0');
-INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','5','5','0');
\ Pas de fin de ligne à la fin du fichier.
+INSERT INTO `glpi_display` ( `ID` , `type` , `num` , `rank` , `FK_users` )  VALUES (NULL,'1200','5','5','0');
diff -Naur applicatifs/inc/plugin_applicatifs-1.5.0-update.sql applicatifs.new/inc/plugin_applicatifs-1.5.0-update.sql
--- applicatifs/inc/plugin_applicatifs-1.5.0-update.sql 2009-06-03 21:14:30.378071493 +0200
+++ applicatifs.new/inc/plugin_applicatifs-1.5.0-update.sql     2009-06-04 08:59:24.461471680 +0200
@@ -2,7 +2,20 @@
 ALTER TABLE `glpi_plugin_applicatifs` ADD INDEX `FK_entities` (`FK_entities`);
 ALTER TABLE `glpi_plugin_applicatifs` ADD INDEX `deleted` (`deleted`);
 ALTER TABLE `glpi_plugin_applicatifs` ADD `FK_groups` int(11) NOT NULL default '0';
+ALTER TABLE `glpi_plugin_applicatifs` ADD `date_mod` datetime default NULL;
 ALTER TABLE `glpi_dropdown_plugin_applicatifs_type` ADD INDEX `FK_entities` (`FK_entities`);
+ALTER TABLE `glpi_plugin_applicatifs` ADD `state` int( 4 ) NOT NULL default '0' COMMENT 'RELATION to glpi_dropdown_plugins_applicatifs_state (ID)';
+
+-- Cette table contient la liste des statuts des applicatifs
+-- This table contains the list of applications' statuses.
+DROP TABLE IF EXISTS `glpi_dropdown_plugin_applicatifs_state`;
+CREATE TABLE `glpi_dropdown_plugin_applicatifs_state` (
+  `ID` int(11) NOT NULL auto_increment,
+  `name` varchar(255) collate utf8_unicode_ci default NULL,
+  `comments` text collate utf8_unicode_ci,
+  PRIMARY KEY  (`ID`),
+  KEY `name` (`name`)
+) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;
 -- Cette table sert à enregistrer les champs personnalisables d'un applicatif
 CREATE TABLE IF NOT EXISTS `glpi_plugin_applicatifs_optvalues` (
@@ -28,4 +41,4 @@
 ALTER TABLE `glpi_plugin_applicatifs_profiles` DROP COLUMN `interface` , DROP COLUMN `is_default`;

 ALTER TABLE `glpi_plugin_applicatifs` CHANGE `tech` `FK_users` int(4);
-ALTER TABLE `glpi_plugin_applicatifs_profiles` ADD `open_ticket` char(1) default NULL;
\ Pas de fin de ligne à la fin du fichier.
+ALTER TABLE `glpi_plugin_applicatifs_profiles` ADD `open_ticket` char(1) default NULL;
diff -Naur applicatifs/inc/plugin_applicatifs.applicatif.classes.php applicatifs.new/inc/plugin_applicatifs.applicatif.classes.php
--- applicatifs/inc/plugin_applicatifs.applicatif.classes.php   2009-06-03 21:14:30.378071493 +0200
+++ applicatifs.new/inc/plugin_applicatifs.applicatif.classes.php       2009-06-04 08:59:24.461471680 +0200
@@ -148,6 +148,10 @@
                        echo "<td>";
                        autocompletionTextField("name","glpi_plugin_applicatifs","name",$this->fields["name"],20,$this->fields["FK_entities"]);
                        echo "</td></tr>";
+
+                       echo "<tr><td>".$LANG['state'][0].":      </td><td>";
+                       dropdownValue("glpi_dropdown_plugin_applicatifs_state", "state", $this->fields["state"]);
+                       echo "</td></tr>";

                        echo "<tr><td>".$LANG['plugin_applicatifs'][20].":      </td><td>";
                        dropdownValue("glpi_dropdown_plugin_applicatifs_type", "type", $this->fields["type"],1,$this->fields["FK_entities"]);
@@ -327,4 +331,4 @@
        }
 }

-?>
\ Pas de fin de ligne à la fin du fichier.
+?>

pthoirey · 2009-06-09 21:37:45

Y a-t-il d'autres personnes et organisations intéressées par l'ajout d'un champ "statut" et d'un champ "dernière modification"?

En outre:
- Pour ce qui est du champ "statut", il me semble également pertinent pour le plugin "appweb".
- Pour ce qui est du champ "dernière modification", il me semble être intéressant pour un grand nombre de plugins. Il va, en effet et entre autres, permettre de mesurer le caractère vivant de l'inventaire, c'est-à-dire sa pertinence.

pthoirey · 2009-06-14 18:25:35

Avez-vous testé?
Qu'en pensez-vous?

tsmr · 2009-06-17 21:53:32

dsl pas eu le temps de regarder. Mais je garde le thread sous le coude.

Par contre a mon avis il va manquer un truc pour que date_mod se mette à jour.

pthoirey · 2009-06-26 06:31:25

tsmr wrote:

Par contre a mon avis il va manquer un truc pour que date_mod se mette à jour.

J'ai de nouveau testé ce point pour constater que les dates se mettent bien à jour.

remi · 2009-06-26 07:07:50

Pour le date_mod : pas de prob pour moi.

Pour le status, il serait bien de trouver un autre nom pour éviter les confusions.
ITIL considère cette donnée comme essentielle, mais ne prévoit pas plusieurs listes
Il faudrait voir du côté des autres méthodologie (genre Prince) la terminologie utilisée.

Pourquoi pas : phase (gestion projet) ? environnement (utilisé par les exploitants) ? cycle de vie ?

Donc je suis toujours perplexe sur la meilleur solution pour intégrer cette notion.

+

P.S. il faudra pas oublier la modification du PDF généré.

pthoirey · 2009-06-26 09:07:21

remi wrote:

Pourquoi pas : phase (gestion projet) ? environnement (utilisé par les exploitants) ? cycle de vie ?

Environnement me paraît être le meilleur vocable si Statut ne convient pas. En effet, l'inventaire est géré par des exploitants.

tsmr · 2009-07-09 22:26:55

https://dev.indepnet.net/plugins/changeset/11252

Merci pour ton patch.
Je clos.

Pour le date_mod dans les autres plugins. On le fera aprs la sortie de la 0.72

Forum GLPI-Project

Announcement

#1 2009-05-17 18:38:37

Plugin applicatifs: ajout de champs

#2 2009-05-20 07:47:08

Re: Plugin applicatifs: ajout de champs

#3 2009-05-25 09:04:23

Re: Plugin applicatifs: ajout de champs

#4 2009-05-25 09:26:12

Re: Plugin applicatifs: ajout de champs

#5 2009-05-25 20:22:04

Re: Plugin applicatifs: ajout de champs

#6 2009-05-26 06:47:32

Re: Plugin applicatifs: ajout de champs

#7 2009-05-26 06:52:32

Re: Plugin applicatifs: ajout de champs

#8 2009-05-26 08:52:35

Re: Plugin applicatifs: ajout de champs

#9 2009-06-04 09:10:30

Re: Plugin applicatifs: ajout de champs

#10 2009-06-09 21:37:45

Re: Plugin applicatifs: ajout de champs

#11 2009-06-14 18:25:35

Re: Plugin applicatifs: ajout de champs

#12 2009-06-17 21:53:32

Re: Plugin applicatifs: ajout de champs

#13 2009-06-26 06:31:25

Re: Plugin applicatifs: ajout de champs

#14 2009-06-26 07:07:50

Re: Plugin applicatifs: ajout de champs

#15 2009-06-26 09:07:21

Re: Plugin applicatifs: ajout de champs

#16 2009-07-09 22:26:55

Re: Plugin applicatifs: ajout de champs

Board footer