Development #59920
quand on teste dans l'inspect, remonter dans l'UI l'erreur de modèle de fiche absent
0%
Description
On attrape l'erreur pour ne plus faire planter en usage normal (#59661) mais dans l'inspect, ça serait utile de 1/ ne pas logguer ça comme erreur, 2/ d'informer dans l'interface de l'exception.
(sans doute en poussant dans le contexte l'info comme quoi on est en mode inspect).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-backoffice-display-missing-carddef-formdef-in-inspec.patch 0001-backoffice-display-missing-carddef-formdef-in-inspec.patch ajouté
- Fichier Screenshot 2021-12-20 at 14-59-39 Backoffice de Auquo - geoloc sans carte.png Screenshot 2021-12-20 at 14-59-39 Backoffice de Auquo - geoloc sans carte.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Plutôt que taper dans le contexte directement brutal l'attribut dans la requête.
Dan le patch on voit un </div> apparaitre, il manquait avant.
Mis à jour par Thomas Noël il y a plus de 2 ans
Je pensais qu'on avait déjà une magie quelque part qui n'enregistrait pas les erreurs quand on joue dans l'inspect ? J'ai rêvé ?... (parce que sioui, j'imaginais que tu la réutiliserais ici)
Mis à jour par Frédéric Péters il y a plus de 2 ans
Non il n'y pas de magie mais il y a un autre ticket. (#57553, compliqué à trouver parce que inpecteur).
Mis à jour par Thomas Noël il y a plus de 2 ans
Frédéric Péters a écrit :
Non il n'y pas de magie mais il y a un autre ticket. (#57553, compliqué à trouver parce que inpecteur).
Ok, j'avais rêvé le patch mais pas le ticket... mais de là, il n'y aurait pas un chemin plus large, de ne jamais logguer les erreurs et toujours les afficher ? (et pas uniquement pour FormDefDoesNotExist et CardDefDoesNotExist ?
Mis à jour par Frédéric Péters il y a plus de 2 ans
ne jamais logguer les erreurs et toujours les afficher
J'ai wip/57553-do-not-record-errors-in-inspect pour ne pas logguer pour l'autre ticket mais je ne vais pas m'avancer à autre chose (le ticket ne mentionne pas l'erreur et le ticket lié contient ensuite lui-même des liens et je ne vais pas chercher ce qui était à l'origine).
Mis à jour par Thomas Noël il y a plus de 2 ans
Ca me va ainsi.
Juste un détail : est qu'une idée derrière Exception::get_inspect_error_message serait qu'on ait aussi un jour un message clair qui serait utilisable dans les erreurs enregistrée ? Dans ce cas, je propose dès maintenant d'appeler la méthode get_human_error_message
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-backoffice-display-missing-carddef-formdef-in-inspec.patch 0001-backoffice-display-missing-carddef-formdef-in-inspec.patch ajouté
Il n'y avait pas cette ambition mais oui ça pourrait tout à fait servir aussi là. Pas enthousiaste sur get_human_error_message, j'ai juste tapé dans la branche la méthode renommée en get_error_message().
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4e6125718e985739a126d238ff50e92001e5c7cc Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Dec 20 14:16:30 2021 +0100 backoffice: display missing carddef/formdef in inspect template test (#59920)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: display missing carddef/formdef in inspect template test (#59920)