template:getData ou la mort silencieuse des pages publiques

[bruno]

Il arrive parfois que les pages publiques d'un blog s'arrêtent brutalement à un endroit donné, sans lever le moindre message d'erreur coté serveur, même avec un error_reporting au maximum.

Exemple flagrant : générer via une balise du code invalide (parse error) dans un fichier inclus : ajouter dans _head.html par exemple : {{tpl:InvalidCode}}

et dans le _public.php du thème :

$core->tpl->addValue('InvalidCode',array('tplTest','invalid'));

class tplTest 
{
	public static function invalid($attr) {
		return '<?php parse_error { ?>';
	}
}

Conséquence : tout le blog affichera page blanche, sans plus râler...

Le coupable : la méthode getData de clearbricks (class.template.php) qui ignore silencieusement toute erreur dans le fichier inclus.

Supprimer le "@" ligne 215 de ce fichier aiderait grandement les gens qui mettent au point leur thème/plugin...

[xave]

Ca ne serait pas plus propre avec un Try/Catch? ?

[bruno]

Pourquoi pas, mais dans ce cas il faudrait définir un errorhandler particulier ( http://fr.php.net/manual/en/class.errorexception.php) pour intercepter une éventuelle erreur, la directive include ne levant pas nativement d'exception.

Par contre, je ne suis pas certain que cela prémunisse de toutes les erreurs. Je pense notamment aux parse error

[bruno]

Confirmation :

"E_ERROR, E_PARSE, E_CORE_ERROR, E_CORE_WARNING, E_COMPILE_ERROR, E_COMPILE_WARNING, and most of E_STRICT" will never reach your custom error handler, and therefore will not be converted into ErrorExceptions.  Plan accordingly.

[xave]

Ah ué. Vivi avait raison : PHP, c'est tout pourri. On peut peut-être simplement se permettre ta première idée alors. Après tout, ça ne sert pas tous les jours. Et en l'occurrence, moins propre, ça fait moins de traitement.

[bruno]

C'est toi qui a les clefs du svn clearbricks... je te laisse commiter la modif ?

[xave]

Bon, au plus j'y réfléchis, au moins je suis convaincu. On prend le risque d'afficher des erreurs en front en production, et ça, c'est mal.

[bruno]

A défaut le permettre via un mode debug ? C'est énormément frustrant de ne pas pouvoir mettre au point un plugin quand on n'a aucune information sur l'erreur qui empêche l'affichage de la page...

[xave]

Je comprends bien (tu noteras que je n'ai pas fermé le ticket.)

J'ai pensé au mode debug, mais je ne sais pas trop comment faire redescendre le debug de Dotclear dans Clearbricks.

[bruno]

J'opterais bien pour un paramètre supplémentaire à getData, du type $verbose_output, par défaut à false...

[xave]

done in Clearbricks commit 240