Produits Entr'ouvert » Combo

Ça m'a l'air tout ok vu de loin.

---

Frédéric Péters a écrit :

Côté code au final le seul truc majeur qui change, c'est Page.get_as_reordered_flat_hierarchy qui est refait de manière bien différente.

Je ne sais pas si c'est voulu mais le fait de rebaser page.level au minimum dans un cas comme :

A1
 B1
  C1
   D1
 B2
  C2

où l'utilisateur a un droit d'édition sur les sous-pages de C1 et B2 ça va donner :

 C1
  D1
B2
 C2

je pense qu'il faudrait rebaser le niveau de chaque branche indépendante à 0, un peu comme ça :

visible = set()
for page_path in page_hierarchy:
    page = page_path[-1][-1]
    if len(path_path) < 2:
        page.level = 0
    else:
        parent = page_path[-2][-1]
        if parent in visible:
            page.level = parent.level + 1
        else:
            page.level = 0
    visible.add(page)

• Une petite remarque sur la précédence entre page_edit_role et subpages_edit_role, à en lire le bout de code :
  

              pages_hierarchy = [
                  x
                  for x in pages_hierarchy
                  if x[-1][-1].edit_role_id in group_ids
                  or any(y[-1].subpages_edit_role_id in group_ids for y in x[:-1])
              ]
  

  
  on comprend que subpages_edit_role s’applique à toutes les pages descendantes dans la hiérarchie, même si un autre subpages_edit_role à été défini pour une page intermédiaire dans la hiérarchie.
  À l’inverse, on pourrait s’attendre à ce que edit_role donne automatiquement le droit d’éditer des sous-pages mais il semblerait que non. Est-ce que c’est grave docteur ?
  Est-ce qu’on ne va pas avoir le cas d’un utilisateur avec le rôle A permettant d’éditer une page, qui souhaite laisser aux utilisateurs ayant le rôle B le droit d’éditer les sous-pages de ladite page, et ceci sans pour autant posséder le rôle B mais en souhaitant quand même garder le droit d’édition des sous-pages ?

Formulé autrement, est-ce la précédence de subpages_edit_role d’une page aux subpages_edit_role(s) ses descendants ne doit pas aussi aussi s’appliquer pour page_edit_role de la page par rapport aux subpages_edit_role de ses descendants ?

• Rien à voir, mais je me pose la question des perfs de
  

              group_ids = [x.id for x in user.groups.all()]           
              if Page.objects.filter(
                  Q(edit_role_id__in=group_ids) | Q(subpages_edit_role_id__in=group_ids)
              ).exists():
                  return True
  

  
  dans le décorateur sans trop savoir si je suis parano ou non. J’ai peur que ça tape un peu au vu de l’usage qui en est fait.
  Est-ce qu’il n’y aurait pas simplement moyen de tenir à jour l'ensemble des rôles autorisant l’édition d’une quelconque page ou de ses sous-pages, et dans ce décorateur simplement tester l’intersection de cet ensemble avec celui des rôles de l’usager ?

Bref juste des questions pour se mettre d’accord, ensuite je testerai en local.

1/ edit_role c'est pour une page la page en question. subpage_edit_role c'est pour les pages "enfants" de la page en question. Rien de plus rien de moins.

2/ on parle d'un appel par requête HTTP, vers le backoffice, ce n'est pas là qu'il y aura à travailler les perfs en premier.

Frédéric Péters a écrit :

1/ edit_role c'est pour une page la page en question. subpage_edit_role c'est pour les pages "enfants" de la page en question. Rien de plus rien de moins.

Ok c’est le “rien de plus” pour lequel j’avais un doute, mais si c’est voulu ainsi alors très bien.

2/ on parle d'un appel par requête HTTP, vers le backoffice, ce n'est pas là qu'il y aura à travailler les perfs en premier.

Ok.