Toutateam

Toutateam - Plate-forme collaborative
N'oubliez pas les règles d'or avant de proposer un bug :
1 - Lisez les docs d'installation, la FAQ
2 - Faire une recherche sur le forum AVANT de poser une question
3 - Décrivez clairement le bug ou demande d'évolution, essayez de le preproduire (bug), répondez aux questions des autres utilisateurs
| Liste des tâches |

FS#141 - Les appels à la fonction error_reporting() de PHP devraient être supprimés du code

Concerne le projet: Toutateam
Ouverte par Laurent Léonard (trash) - vendredi, 06 mars 2009, 23:30 GMT+2
Dernière édition par Laurent Léonard (trash) - samedi, 07 mars 2009, 02:03 GMT+2
Type de tâche Anomalie
Catégorie Noyau applicatif
Etat Nouvelle
Assignée à Personne
Système d'exploitation Tous
Sévérité Moyenne
Priorité Normale
Basée sur la version 1.5.6
Due pour la version Non décidé
Date d'échéance Non décidé
Pourcentage achevé: 0%
Votes 0
Privée Non

Détails

Le code de Toutateam devrait être nettoyé des appels à la fonction error_reporting() de PHP. Ceux-ci empêchent un deboguage efficace de l'application en cas de problème. Le niveau de verbosité des messages de PHP devrait être géré par l'administrateur dans la configuration d'Apache/PHP, au sein d'un virtual host ou globalement, mais certainement pas dans le code lui-même.
Cette tâche dépend de

Commentaire de Yannick Bétemps (Yannick) - lundi, 09 mars 2009, 17:59 GMT+2
effectivement les "error_reporting" sont ajoutés un peu de manière aléatoire dans quelques fichiers de Toutateam
Au final il y a tout de même peu d'endroits où le error_reporting est réellement utilisé.
Vu que sur la plupart des hébergement mutualisés cette fonction n'est pas active, cela a peu d'impact.
Cependant pour garder la possibilité de l'utiliser au cas par cas, ou de le conserver désactiver, je pense qu'il faudrait réactualiser le paramètre $error_reporting_level proprement, de manière à ce que :
- si $error_reporting_level contient une chaine vide, il n'y ait aucun error_reporting
- s'il contient un chiffre, c'est ce chiffre qui s'applique partout

De toutes manières pour être plus propre il faut nettoyer tous les error_reporting hors celui qui est dans le lib.inc.php

A noter qu'il y en a un certain nombre dans des fichiers hors TT : FCKEditor, le forum. Je pense qu'il n'est pas nécessaire de les oter.

Pour le debug de l'appli, une fonction est incluse dans Toutateam : Tools::Trace($message);
Elle ajoute la trace dans un fichier tracefile.log écrit à la racine d'exécution du script (racine du site ou racine de la fonction)

Autre "idée" de debug : FirePHP, qui marche pas mal du tout, en complément de Firebug !
Il est possible que j'ajoute la librairie FirePHP au projet pour faciliter le debug.

Chargement...