Development #27031
Faciliter le diagnostique d'une page manquante
0%
Description
Pour aider le pauvre développeur lors d'une 404, qui pourrait se perdre en cherchant dans pleins de mauvais endroits (les vues, les configurations d'url),
alors que c'est juste une page manquante, à créer.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
Plus d'info sur le problème, les mauvaises pistes, ce qui aurait aidé, etc. ?
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier 0001-give-a-hint-when-a-page-is-missing-27031.patch 0001-give-a-hint-when-a-page-is-missing-27031.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le message de l'exception apparaît sur la page de 404 si DEBUG=True.
Pas réussi à faire un test unitaire (je provoque des 301 au lieu de 404), ça ne me parait nécessaire d'y passer du temps.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Sujet changé de Facilite le diagnostique d'une page manquante à Faciliter le diagnostique d'une page manquante
Mis à jour par Frédéric Péters il y a plus de 5 ans
Sans rien retirer à mon premier commentaire.
requested, et point final.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier 0001-give-a-hint-when-a-page-is-missing-27031.patch 0001-give-a-hint-when-a-page-is-missing-27031.patch ajouté
Frédéric Péters a écrit :
Plus d'info sur le problème, les mauvaises pistes, ce qui aurait aidé, etc. ?
Moi qui me prend une 404 sur un /user/paiements/ (depuis formulaire et workflow crée par moi même en imitant et simplifiant https://demarches-venissieux-test.demarches.sitiv.fr/reservation-de-salle/reservation-de-salle-pour-les-particuliers/)
Et une heure à coups de pdb pour comprendre que c'est juste une page à créer.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Et le /user/paiements vient de là https://demarches-venissieux-test.demarches.sitiv.fr/backoffice/workflows/2/status/25/items/7/.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
301 parce que tu n'as pas de / final j'imagine, tu peux ajouter ça comme test, genre :
--- a/tests/test_public.py +++ b/tests/test_public.py @@ -276,6 +276,11 @@ def test_404(app): resp = app.get('/foobar/', status=404) assert 'Custom 404 Text' in resp.text + with override_settings(DEBUG=True): + # check error page provides an hint when debugging + resp = app.get('/foobar/', status=404) + assert "can't find the requested page" in resp.text + def test_style_demo(app, admin_user):
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier 0001-give-an-hint-when-a-page-is-missing-27031.patch 0001-give-an-hint-when-a-page-is-missing-27031.patch ajouté
- Statut changé de Solution validée à Solution proposée
Merci.
Avec l'apostrophe en représentation numérique dans le test et "an hint" dans le message de commit.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
(passer le statut en "solution validée" pour signifier le ack)
(et je l'avais déjà validé plus haut, repasser par redmine n'était pas nécessaire)
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0c7b57fad091b272c155f19b13d9b5bf5b7f39ee Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Fri Oct 5 14:11:14 2018 +0200 give a hint when a page is missing (#27031)
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
give a hint when a page is missing (#27031)