Development #45846
shown_because_admin ne gère pas le flag cell.restricted_to_unlogged
0%
Description
Ça provoque des incompréhensions (comme dans #45601), il faudrait reproduire le comportement de is_element_visible() (et je pense que le plus simple serait de réutiliser is_element_visible() en y ajoutant un flag pour considérer our pas le flag is_superuser, c'est plus DRY).
Fichiers
Demandes liées
Révisions associées
misc: show unlogged only cells to superuser (#45846)
In this case the .shown-because-admin class is present on the cell.
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-mics-fix-shown_because_admin-when-cell.restricted_to.patch 0001-mics-fix-shown_because_admin-when-cell.restricted_to.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Au passage j'ai aussi corrigé le comportement de element_is_visible() quand on est superuser et qu'une cellule est réservée au public.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Assigné à changé de Valentin Deniaud à Benjamin Dauvergne
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
Les choix posés là peuvent correspondre à des situations passée précises, je ne souhaite pas de changement de comportement. (autre que le fait initial de ce ticket, marquer la cellule comme affichée parce que l'usager est admin).
(je ne sais pas à quoi correspond cette réassignation à Benjamin)
Mis à jour par Frédéric Péters il y a plus de 3 ans
Note aussi, s'il s'agit au final quand même de toucher element_is_visible(), il faudrait regarder le patch de #45741 d'abord.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Frédéric Péters a écrit :
Les choix posés là peuvent correspondre à des situations passée précises, je ne souhaite pas de changement de comportement. (autre que le fait initial de ce ticket, marquer la cellule comme affichée parce que l'usager est admin).
Ok.
(je ne sais pas à quoi correspond cette réassignation à Benjamin)
Que j'étais en train d'écrire le patch quand Valentin se l'est assigné.
Note aussi, s'il s'agit au final quand même de toucher element_is_visible(), il faudrait regarder le patch de #45741 d'abord.
Je peux reprendre l'argument ignore_superuser.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-misc-fix-shown_because_admin-has-no-role-and-cell.re.patch 0001-misc-fix-shown_because_admin-has-no-role-and-cell.re.patch ajouté
- Statut changé de En cours à Solution proposée
Voilà, sans changement de comportement et avec les tests qui n'existaient pas, si on décide de changer le comportement, on le verra.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Lié à Development #45859: Pourquoi les cellules réservées aux utilisateurs non connectés sont-elles invisibles des administrateurs ? ajouté
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-misc-fix-shown_because_admin-has-no-role-and-cell.re.patch 0001-misc-fix-shown_because_admin-has-no-role-and-cell.re.patch ajouté
Patch rebasé, toujours le même souci, le templatetag shown_because_admin()
ne respecte pas les mêmes règles que element_is_visible()
, le mieux étant de réutiliser cette deuxième fonction en ignorant le statut superutilisateur (comportement introduit dans #45741).
Je n'ai pas changé le comportement, les cellules cachées aux connectés sont toujours invisibles même aux administrateurs. Mais par contre les hachures vont apparaître enfin pour des cellules caché aux membres d'un rôle dont l'utilisateur actuel, qui est admin, fait parti, c'est le dernier test ajouté "superuser_with_role".
Maintenant qu'il y a ces tests on pourrait tenter d'afficher les cellules "réservées aux utilisateurs non connectés" en grisé aux administrateurs.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-misc-fix-shown_because_admin-has-no-role-and-cell.re.patch 0001-misc-fix-shown_because_admin-has-no-role-and-cell.re.patch ajouté
- Fichier 0002-misc-show-unlogged-only-cells-to-superuser-45846.patch 0002-misc-show-unlogged-only-cells-to-superuser-45846.patch ajouté
Finalement avec le patch qui modifie le comportement dans le cas "restricted_to_unlogged".
Mis à jour par Serghei Mihai 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 7ce06a36ab895afde6afaf5a846db5f6da01f3f2 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Oct 14 15:45:47 2022 +0200 misc: show unlogged only cells to superuser (#45846) In this case the .shown-because-admin class is present on the cell. commit a6116c5d32b2bf8d76e11aa5050ee410fc177aef Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Aug 12 15:16:30 2020 +0200 misc: fix shown_because_admin has no role and cell.restricted_to_unlogged is True (#45846)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: fix shown_because_admin has no role and cell.restricted_to_unlogged is True (#45846)