Forum GLPI-Project

Joël

Member

Registered: 2006-11-21

Posts: 29

"$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

Je ne pense pas que ceci soit un bug.

Voulant ajouter une colonne "Date fin" dans l'affichage de la liste des contrats avec "glpi/front/contract.php", je suis allé voir le code du fichier php "glpi/front/contract.form.php" pour voir comment cette date est affichée/calculée par gpli.

Dans ce code, j'ai vu une chose étrange :-)

Au début du code la variable "$tab" est initialisée avec le contenu de "$_GET" si cette variable existe ou sinon avec le contenu de "$_POST" si elle existe. Jusque là rien d'anormal. Dans le reste du fichier par contre, il est fait indifféremment appel à "$tab", "$_GET" ou "$POST". Voici quelques exemples.

Ligne 75 utilisation de "$tab" :

logEvent($tab["ID"], "contracts", 4, "financial", $_SESSION["glpiname"]." ".$lang["log"][24]);

Ligne 83 utilisation de "$_POST" :

logEvent($_POST["ID"], "contracts", 4, "financial", $_SESSION["glpiname"]." ".$lang["log"][21]);

Ligne 119 utilisation de "$_GET" :

deleteEnterpriseContract($_GET["ID"]);

Est-ce que l'utilisation des différentes possibilités est faite pour une raison de sécurité (en s'assurant que les variables ont bien été reçues par la bonne méthode POST ou GET), ou pour une autre raison ?

Joël

Informations complémentaires.
* La version de GLPI que vous utilisez (numéro de version) et si vous avez réalisé une mise à jour la version de départ et celle d'arrivée.

0.68.2

* La plateforme sur laquelle vous l'avez installé : win/mandrake/debian ...

Mac OS X 10.4.8 server Intel

* Comment vous l'avez installé (logiciel de décompactage pour Windows)

tar -xzf  etc.

* Les versions  apache, php, mysql ....

Apache/1.3.33 (Darwin)
PHP 4.4.1
MySQL Ver 14.7 Distrib 4.1.13a, for apple-darwin8.0 (powerpc) using  EditLine wrapper

* Navigateur utilisé (IE, Moz...) et version.

Non pertinent

* Les manipulations que vous avez effectuées.

Non pertinent

* La configuration de GLPI : authentification externe......

Non pertinent

* Les messages d'erreur de GLPI en mode debug

Non pertinent

* Toutes informations qui pourraient nous aider à determiner les causes de ce dysfonctionnement de GLPI sur votre installation.

Non pertinent

Last edited by Joël (2006-11-28 12:06:39)

MoYo

GLPI - Lead

From: Poitiers

Registered: 2004-09-13

Posts: 14,513

Website

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

C'est historique. C'estce type de choses que l'on tente de nettoyer au fur et à mesure.

---

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

Joël

Member

Registered: 2006-11-21

Posts: 29

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

Comme je suis dans le fichier, je peux éventuellement remplacer tous les "$_GET" et "$_POST" par "$tab" et vous renvoyer le nouveau fichier. Est-ce que ça peut être utile ? Est-ce qu'il y a un document avec les "coding rules" du projet quelque part ?

PS : je cherche toujours à ajouter une colonne "Date fin" quand même :-)

motsan

Member

From: Marseille

Registered: 2006-04-12

Posts: 528

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

oui, https://dev.indepnet.net:8080/glpi/wiki/CodingStandards

Mais il est en cours de validation. ceci dit, ca ne devrait pas poser de probleme, ce nettoyage s'impose et le code sera + clean.

Maintenant, ya une regle qui dit en gros que $tab n'est pas tres parlant comme nom.
mais nous verrons ca plus tard, quand la validation sera validé

il est plus facile de remplacer $tab partout que de chercher des fois $tab et des fois $_post ou $_get.

---

Serveurs : Debian Wheezy (Apache 2.2.22, Php 5.4.4, mysql 5.5.31).
Logiciels : Firefox 30, 7z 9.20.
Plateforme en exploitation : GLPI 0.84.6/OCS 2.0.5 sur Debian Wheezy.
Plateforme en test : GLPI 0.84.6 sur Windows 7 SP1 x64 / WampServer 2.4 x64

MoYo

GLPI - Lead

From: Poitiers

Registered: 2004-09-13

Posts: 14,513

Website

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

le problème est que le codage tab -> GET et POST ne me plait pas du tout.
C'etait une bidouille pour ne pas s'embeter mais l'utilisation des vraies variables et quand meme plus propre.

---

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

Joël

Member

Registered: 2006-11-21

Posts: 29

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

Bon OK, alors je touche à rien dans ce fichier et je continue à chercher s'il y a un moyen simple (et propre) d'ajouter une colonne "Date fin" dans la liste des contracts.

Merci.

MoYo

GLPI - Lead

From: Poitiers

Registered: 2004-09-13

Posts: 14,513

Website

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

Vous etes simlpement aller voir dans configuration / affichage recherche par defaut ?

---

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

Joël

Member

Registered: 2006-11-21

Posts: 29

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

Alors là je suis comme un fou. C'est vraiment bien ce soft. Dire que je patauge de fichier php en fichier php depuis ce matin ! Le réglage des préférences d'affichage règle mon problème initial. Milles excuses pour le dérangement. Bon alors maintenant je retourne à la saisie de données :-(

JMD

GLPI - Lead

Registered: 2004-09-13

Posts: 9,180

Website

Re: "$tab", "$_GET" et "$_POST" dans "glpi/front/contract.form.php"

ahah

---

JMD / Jean-Mathieu Doléans - Glpi-project.org - Association Indepnet
Apportez votre pierre au  projet GLPI   : Soutenir