Project

General

Profile

Development #45846

shown_because_admin ne gère pas le flag cell.restricted_to_unlogged

Added by Benjamin Dauvergne about 1 year ago. Updated about 1 year ago.

Status:
Solution proposée
Priority:
Normal
Target version:
-
Start date:
12 Aug 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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).


Files


Related issues

Related to Combo - Development #45859: Pourquoi les cellules réservées aux utilisateurs non connectés sont-elles invisibles des administrateurs ?Fermé12 Aug 2020

Actions

History

#2

Updated by Valentin Deniaud about 1 year ago

  • Assignee set to Valentin Deniaud
#3

Updated by Benjamin Dauvergne about 1 year ago

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.

#4

Updated by Benjamin Dauvergne about 1 year ago

  • Assignee changed from Valentin Deniaud to Benjamin Dauvergne
#5

Updated by Frédéric Péters about 1 year ago

  • Status changed from Solution proposée to 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)

#6

Updated by Frédéric Péters about 1 year ago

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.

#7

Updated by Benjamin Dauvergne about 1 year ago

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.

#8

Updated by Benjamin Dauvergne about 1 year ago

Voilà, sans changement de comportement et avec les tests qui n'existaient pas, si on décide de changer le comportement, on le verra.

#9

Updated by Benjamin Dauvergne about 1 year ago

  • Related to Development #45859: Pourquoi les cellules réservées aux utilisateurs non connectés sont-elles invisibles des administrateurs ? added

Also available in: Atom PDF