Ticket #625 (assigned enhancement)

Opened 5 months ago

Last modified 1 month ago

Log avec firePHP

Reported by: hadrien Assigned to: bastnic (accepted)
Priority: low Milestone: Jelix 1.2
Component: jelix:utils Version: 1.0.4
Severity: minor Keywords:
Cc: Php version:
Review: review- Hosting Provider:
Documentation needed: 0 Blocking:

Description

Voici un patch sur jLog qui permet de logguer dans firePHP.

Dans la méthode log, on détecte si le navigateur dispose du dit plugin. Si c'est le cas, on renvoit la méthode fireLog qui envoit un log dans les headers. Sinon, on log normalement.

Attachments

firePHP.diff (6.3 kB) - added by hadrien on 06/18/08 16:53:27.
Ajout de fireLog
firePHP.2.diff (6.3 kB) - added by hadrien on 06/18/08 16:54:26.
Ajout de fireLog
firePHP.3.diff (6.4 kB) - added by hadrien on 06/18/08 17:19:04.
Petit fix
firephp-test.diff (30.0 kB) - added by bastnic on 07/20/08 17:36:46.

Change History

06/18/08 16:53:27 changed by hadrien

  • attachment firePHP.diff added.

Ajout de fireLog

06/18/08 16:54:26 changed by hadrien

  • attachment firePHP.2.diff added.

Ajout de fireLog

06/18/08 17:19:04 changed by hadrien

  • attachment firePHP.3.diff added.

Petit fix

06/19/08 10:37:06 changed by laurentj

  • review set to review-.
  • type changed from bug to enhancement.

rejeté. merci de faire le patch à partir du trunk, et non d'une distribution, et de respecter les normes de coding :-p

Sans compter que le ticket est un doublon du #400.

À part ça, je ne vois pas l'interêt des constantes ajoutées, puisque jLog n'est pas utilisé par le gestionnaire d'erreur. Il est donc possible de simplifier le code.

07/20/08 17:36:33 changed by bastnic

  • priority changed from normal to low.
  • review changed from review- to review?.
  • severity changed from normal to minor.
  • milestone set to Jelix 1.1 beta 1.

Je n'ai pas encore ajouté la vérification de l'extension, mais dans l'esprit c'est plutôt un truc comme ça non ?

07/20/08 17:36:46 changed by bastnic

  • attachment firephp-test.diff added.

07/20/08 17:48:34 changed by bastnic

  • status changed from new to assigned.
  • owner set to bastnic.

07/20/08 17:49:30 changed by bastnic

Pour l'activer, dans defaultconfig.ini.php : 

[logfiles]
;default=messages.log
default="!firephp"

07/21/08 00:35:22 changed by laurentj

  • review changed from review? to review-.

Je ne veux pas utiliser les fichiers PHP de firephp : il y a beaucoup de doublon de code avec ce que l'on trouve déjà dans jelix (json_encode et cie). J'en avais discuté avec Hadrien, et son patch me convient mieux dans l'esprit (intégation directe dans jLog), même si en l'état il n'est pas acceptable à cause du coding style, du fait que ce n'est pas configurable comme toi dans la section logfiles etc) etc.. De plus, je pense que ce ne doit pas être jResponseHtml à piloter firephp. Après tout, l'interêt de firePHP, c'est de pouvoir envoyer des logs dans n'importe quel type de réponse (redirection, xul, json etc...).

Bref, faudrait reprendre en gros ce qu'à fait Hadrien et l'améliorer en reprennant ton idée d'activation par la conf, mais aussi en testant si on a déjà ou non envoyé les entêtes, pour pas générer d'erreur. Et que faire quand les entêtes sont déjà envoyé. Genre, je log dans une zone appelée par un plugin {zone} dans un template -> le début de la réponse a déjà été envoyé (problème similaire au ticket #61)

07/21/08 01:13:14 changed by bastnic

On est bien d'accord qu'en mode production, FirePHP ne doit surtout pas être activé ?

09/11/08 00:18:37 changed by laurentj

  • milestone changed from Jelix 1.1 beta 1 to Jelix 1.1 beta 2.

10/22/08 10:43:07 changed by laurentj

  • milestone changed from Jelix 1.1 beta 2 to Jelix 1.2.
Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed