Projet

Général

Profil

Bug #57163

Bouton "Calendrier d’indisponibilités" visible même si on n'a pas les droits

Ajouté par Marie Kuntz il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Catégorie:
-
Version cible:
-
Début:
21 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Quand on n'a pas accès aux agendas, l'entrée de menu s'affiche quand même, et sur la page agenda, on voit le bouton "Calendrier d’indisponibilités", mais quand on clique dessus, on a une erreur 403 "accès interdit". Il faudrait ne pas voir le bouton.

Fichiers

0001-manager-adapt-responses-to-restricted-access-on-home.patch (6,27 ko) 0001-manager-adapt-responses-to-restricted-access-on-home.patch Frédéric Péters, 22 septembre 2021 07:42

Révisions associées

Révision acd6cdb9 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

manager: serve a specific template for home 403 (#57163)

Révision d3554167 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

Revert "manager: serve a specific template for home 403 (#57163)"

This reverts commit acd6cdb94f7445b24af4d3f90dc4d0a081d6c914 as it was
not the correct commit to merge :/

Révision 494a2b37 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

manager: adapt responses to restricted access on home (#57163)

Historique

#3

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Assigné à mis à Frédéric Péters

l'entrée de menu s'affiche quand même

ce bout est dans #57165.

sur la page agenda, on voit le bouton

Et celui-ci est une autre conséquence de #48367 qui fait afficher la page pour avoir un message d'erreur "propre".

#4

Mis à jour par Frédéric Péters il y a plus de 2 ans

Ma première approche ici a été de servir un gabarit différent et limité au message "pas d'accès" mais ça a échoué sur test_unavailability_calendar_homepage_permission.

Ce test m'a paru bizarre il vérifie qu'on obtient une 403 mais qu'il y a le bouton "Unavailability calendars" dessus.

Version révisée attachée qui, quand l'utilisateur peut accéder Chrono uniquement parce qu'il peut accéder à un UnavailabilityCalendar, retourne une redirection vers l'écran listant ceux-ci.

+ ne pas afficher le bouton pour les usagers qui auraient accès à un agenda mais aucun UnavailabilityCalendar.

+ adaptation au menu.json pour prendre en compte les accès à UnavailabilityCalendar.

#7

Mis à jour par Nicolas Roche il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
#8

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 acd6cdb94f7445b24af4d3f90dc4d0a081d6c914
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Sep 21 20:20:06 2021 +0200

    manager: serve a specific template for home 403 (#57163)
#9

Mis à jour par Frédéric Péters il y a plus de 2 ans

Je n'ai pas mergé la bonne branche :/

commit 494a2b37984c0ad205f4ff0afd51accc1a8da490
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Sep 21 20:20:06 2021 +0200

    manager: adapt responses to restricted access on home (#57163)

commit d3554167b9c6faae6190f2647b0f1a9e1567daa0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Oct 5 22:22:54 2021 +0200

    Revert "manager: serve a specific template for home 403 (#57163)" 

    This reverts commit acd6cdb94f7445b24af4d3f90dc4d0a081d6c914 as it was
    not the correct commit to merge :/
#10

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

Formats disponibles : Atom PDF