Development #70042
l'erreur 403 ne reprend pas le menu latéral publik
0%
Description
Ça donne une pleine page avec la barre d'entête et
Vous n’êtes pas autorisé à voir cette page. [Accueil] [Retour]
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-manager-add-a-permission-denied-view-70042.patch 0001-manager-add-a-permission-denied-view-70042.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Il manquait une vue 403 spécifique pour le manage, on pourra l'améliorer pour par exemple indiquer les permissions qu'il aurait fallu avoir (via request.resolver_match.view.permissions mais qui n'est pas présent sur toutes les vues, la gestion des permissions n'étant plus uniforme avec les vues authenticators).
Mis à jour par Valentin Deniaud il y a plus d'un an
Benjamin Dauvergne a écrit :
la gestion des permissions n'étant plus uniforme avec les vues authenticators
#66984 rendrait ça à nouveau uniforme ?
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Valentin Deniaud a écrit :
Benjamin Dauvergne a écrit :
la gestion des permissions n'étant plus uniforme avec les vues authenticators
#66984 rendrait ça à nouveau uniforme ?
Le problème n'est pas le choix de la permission mais la façon dont c'est implémenté avec des checks explicites plutôt que de réutiliser PermissionMixin, donc oui on peut ré-introduire PermissionMixin au passage simplifier/améliorer et changer la permission.
PS: loin de moi l'idée que l'une ou l'autre façon soit mieux, c'est juste pas pareil pour le même résultat à la fin.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à En cours
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-manager-add-a-permission-denied-view-70042.patch 0001-manager-add-a-permission-denied-view-70042.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Emmanuel Cazenave il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit f4979a4f5ea9aaacd56e709364766edb8a568a8f
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Mon Oct 10 19:09:00 2022 +0200
manager: add a permission denied view (#70042)Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
manager: add a permission denied view (#70042)