developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Last modified 9 years ago

#625 confirmed enhancement

Log avec firePHP

Reported by: hadrien Owned by:
Priority: low Milestone:
Component: jelix:plugins:logger Version: 1.0.4
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

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 (4)

firePHP.diff (6.3 KB) - added by hadrien 12 years ago.
Ajout de fireLog
firePHP.2.diff (6.3 KB) - added by hadrien 12 years ago.
Ajout de fireLog
firePHP.3.diff (6.4 KB) - added by hadrien 12 years ago.
Petit fix
firephp-test.diff (30.0 KB) - added by bastnic 12 years ago.

Download all attachments as: .zip

Change History (16)

Changed 12 years ago by hadrien

Ajout de fireLog

Changed 12 years ago by hadrien

Ajout de fireLog

Changed 12 years ago by hadrien

Petit fix

comment:1 Changed 12 years ago 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.

comment:2 Changed 12 years ago by bastnic

  • Milestone set to Jelix 1.1 beta 1
  • Priority changed from normal to low
  • review changed from review- to review?
  • Severity changed from normal to minor

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 ?

Changed 12 years ago by bastnic

comment:3 Changed 12 years ago by bastnic

  • Owner set to bastnic
  • Status changed from new to assigned

comment:4 Changed 12 years ago by bastnic

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

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

comment:5 Changed 12 years ago 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)

comment:6 Changed 12 years ago by bastnic

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

comment:7 Changed 12 years ago by laurentj

  • Milestone changed from Jelix 1.1 beta 1 to Jelix 1.1 beta 2

comment:8 Changed 12 years ago by laurentj

  • Milestone changed from Jelix 1.1 beta 2 to Jelix 1.2

comment:9 Changed 11 years ago by laurentj

  • Milestone Jelix 1.2 beta deleted

comment:10 Changed 10 years ago by laurentj

  • Component changed from jelix:utils to jelix:plugins:logger
  • Milestone set to Jelix 1.3

comment:11 Changed 10 years ago by laurentj

  • Milestone Jelix 1.3 deleted

comment:12 Changed 9 years ago by laurentj

  • Owner bastnic deleted
  • review review- deleted
  • Status changed from assigned to confirmed
Note: See TracTickets for help on using tickets.