Bug #15768
cellule json précablée : un template par défaut
0%
Description
En test, quand une ConfigJsonCell ne trouve pas son template :
E TemplateDoesNotExist: combo/json/foobar.html, combo/configjsoncell.html
Je me dis qu'on pourrait livrer avec combo un configjsoncell.html qui dise juste "missing template" ?
Fichiers
Historique
Mis à jour par Frédéric Péters il y a environ 7 ans
Template qui en mode DEBUG donnerait quand même le nom du fichier à utiliser. (parce que je trouve pratique d'avoir l'info dans le message d'erreur).
Mis à jour par Thomas Noël il y a environ 7 ans
- Fichier 0001-add-fail-over-template-for-config-json-cell-15768.patch 0001-add-fail-over-template-for-config-json-cell-15768.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
(à appliquer après #15770)
Mis à jour par Thomas Noël il y a environ 7 ans
- Fichier 0001-add-error-template-on-cell-renderer-15768.patch 0001-add-error-template-on-cell-renderer-15768.patch ajouté
Pour le fun j'ai généricisé le truc à toutes les cellules (tant qu'elles passent bien par le render de CellBase) et j'ai ajouté la gestion des erreurs de syntaxe. C'est sommaire mais bon.
Mis à jour par Thomas Noël il y a environ 7 ans
- Fichier 0001-add-error-template-on-cell-renderer-15768.patch 0001-add-error-template-on-cell-renderer-15768.patch ajouté
Rajout du fichier badsyntax.html oublié
Mis à jour par Frédéric Péters il y a environ 7 ans
<h2> autour de {% trans "template error:" %} {{ exception_type }} ?
Et dans le except:, if not settings.DEBUG: raise ? comme ça on se tape l'info dans une trace plutôt qu'afficher un message à l'usager ?
Mis à jour par Thomas Noël il y a environ 7 ans
Frédéric Péters a écrit :
<h2> autour de {% trans "template error:" %} {{ exception_type }} ?
Oui.
Et dans le except:, if not settings.DEBUG: raise ? comme ça on se tape l'info dans une trace plutôt qu'afficher un message à l'usager ?
Ah ben alors tu m'embêtes à ajouter un «if debug» dans le template, et puis en fait non ? :-)
Je suis un peu partagé. Ca m'intéresse quand même qu'un client "power user" puisse un jour jouer des templates sur ses cellules JSON (voire les template_string de la cellule de base) et voit les erreurs (en plus ou au lieu d'un mail qui arrive chez nous). Peut-être qu'on pourrait afficher l'erreur seulement pour les is_staff/is_admin ?
Mis à jour par Frédéric Péters il y a environ 7 ans
Peut-être qu'on pourrait afficher l'erreur seulement pour les is_staff/is_admin ?
Ouaip, ça le ferait.
Mis à jour par Frédéric Péters il y a environ 4 ans
Je reviens ici dessus, avec l'expérience, je pense que ma préférence irait à un template de base qui ressemble à :
h2. {{cell.get_label}} Cette cellule n'a pas de gabarit associé, les gabarits suivants ont été essayés : <liste>
et je serais pour laisser planter le cas template.TemplateSyntaxError (comme plante n'importe quelle autre cellule qui recevrait un gabarit invalide).