Case "Afficher menu développement" présente pour les admins restreints. #3855

Closed
opened 6 years ago by Yohooo · 5 comments
Yohooo commented 6 years ago

La case Case "Afficher menu développement" est visible pour les admins restreint sur la page ?exec=configurer_preferences alors que ceux-ci ne peuvent voir ce menu.

Soit il faut supprimer la case, soit il faut permettre aux admins restreint de voir le menu (je plaide pour la première solution).

La case Case "Afficher menu développement" est visible pour les admins restreint sur la page _?exec=configurer_preferences_ alors que ceux-ci ne peuvent voir ce menu. Soit il faut supprimer la case, soit il faut permettre aux admins restreint de voir le menu (je plaide pour la première solution).
b_b commented 6 years ago
Owner

Ha oui, bien vu, Eric a oublié de conditionner l'affichage de la case à cocher dans r21657 il devrait pouvoir fixer ça rapidement :)
Assigné à Eric

Ha oui, bien vu, Eric a oublié de conditionner l'affichage de la case à cocher dans r21657 il devrait pouvoir fixer ça rapidement :) **Assigné à Eric**
Owner

Hello,

Ben en fait c'est fait sur la branche 3.2 et d'ailleurs c'est ton commit ;-).
Il n'a pas été backporté en 3.1.
La solution est de rajouter un test d'autorisation webmestre.

Mais je me demande si il ne faudrait pas aussi rajouter par précaution l'autorisation webmestre dans l'autorisation d'affichage du menu développement lui-même?

Hello, Ben en fait c'est fait sur la branche 3.2 et d'ailleurs c'est ton commit ;-). Il n'a pas été backporté en 3.1. La solution est de rajouter un test d'autorisation webmestre. Mais je me demande si il ne faudrait pas aussi rajouter par précaution l'autorisation webmestre dans l'autorisation d'affichage du menu développement lui-même?
b_b commented 6 years ago
Owner

Il manque donc un report de r22711 (merci pour la référence :p) dans la branche 3.1. Faut-il que je m'en occupe ou tu le fais ?

Il manque donc un report de r22711 (merci pour la référence :p) dans la branche 3.1. Faut-il que je m'en occupe ou tu le fais ?

Appliqué par commit r23246.
Statut changé à Fermé

Appliqué par commit r23246. **Statut changé à Fermé**
b_b commented 6 years ago
Owner
There is no content yet.
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.