Bug #25722
action_button: arrivée sur un 404
0%
Description
Quand on clique sur une actions qui n'existe plus (déjà passée) ou qui n'est plus valable (expirée ou pas de saut avec l'identifiant indiqué), on arrive sur une 404.
Je pense qu'il serait préférable de tomber sur une page qui dise plus explicitement "Cette action n'est plus possible." (sans donner plus la raison).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-general-give-a-custom-error-message-on-invalid-actio.patch 0001-general-give-a-custom-error-message-on-invalid-actio.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà un patch, ça reste techniquement une 404 mais ça affiche "This action link has already been used or has expired.".
Mis à jour par Thomas Noël il y a plus de 5 ans
Il y a aussi ce raise :
for item in status.items:
if getattr(item, 'identifier', None) == self.token.context['action_id']:
self.action = item
break
else:
raise errors.TraversalError() <-- ici
quand on clique sur une action alors que la demande a déjà bougé et n'est plus sur un statut qui permet l'action. Je mettrais exactement le même message, ie un «raise MissingOrExpiredToken()» (et le nom colle encore, le "expired" garde son sens).
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-general-give-a-custom-error-message-on-invalid-actio.patch 0001-general-give-a-custom-error-message-on-invalid-actio.patch ajouté
Avec également les tests couvrant ces deux raise.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Super, merci.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6d20a697b04de36c38cd25551a1b1de4e34d2978
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Sep 4 09:42:35 2018 +0200
general: give a custom error message on invalid action link (#25722)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: give a custom error message on invalid action link (#25722)