Produits Entr'ouvert » Combo

Je pense que c'est parce qu'on cherche les liens dans for cell in context.get('page_cells', []): alors qu'on n'est plus sur la page de la cellule (celle où elle est instanciée).

J'ai l'impression que c'est juste ça, ou bien je rate une subtilité ?

diff --git a/combo/data/models.py b/combo/data/models.py
index a295707f..6f9581de 100644
--- a/combo/data/models.py
+++ b/combo/data/models.py
@@ -1241,11 +1241,7 @@ class LinkListCell(CellBase):
     def get_cell_extra_context(self, context):
         extra_context = super(LinkListCell, self).get_cell_extra_context(context)
         links = []
-        for cell in context.get('page_cells', []):
-            if not hasattr(cell, 'add_as_link_label'):
-                continue
-            if not cell.placeholder == self.link_placeholder:
-                continue
+        for cell in self.get_items():
             links.append(cell.get_cell_extra_context(context))
         extra_context['links'] = links
         extra_context['more_links'] = []

J'ai l'impression que c'est juste ça, ou bien je rate une subtilité ?

Minimiser le nombre de requêtes.

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

J'ai l'impression que c'est juste ça, ou bien je rate une subtilité ?

Minimiser le nombre de requêtes.

Yep.

Tenté sur http://git.entrouvert.org/combo.git/?h=wip%2F41276-linklist-in-parentcell

Et oui ça requête plus et donc le test « assert len(ctx.captured_queries) == 31 » n'est pas content.

Ceci étant c'est a priori le seul pépin.

Si quelqu'un a une autre idée...

Et là le test c'est sur la page du manager, donc moins important que le front côté nombre de requêtes. (mais modif sans effet vu que pas de problème côté manager).

Ce qu'il faut (d'un rapide regard) c'est adapter le get_parent_cells(), modifier

CellBase.get_cells(placeholder=self.placeholder, page__in=pages)

pour qu'il récupère aussi les cellules faisant partie de "liste de liens". (et une approximation genre placeholder__startswith='_' doit être ok, je pense).

A priori le seen ajouté dans combo/data/models.py ne sert à rien (sans doute un reste d'idée ?)

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

Voilà, avec un moche mais efficace,

• c'est pour garder l'ordre que tu n'utilise pas un set ? Ou bien pour une question d'efficacité ?
• et aussi, pourquoi enlever les doublons ?

(et aussi le remplissage du seen dans la méthode get_parents_cells qui semble ne pas servir à grand chose :) )

A priori le seen ajouté dans combo/data/models.py ne sert à rien (sans doute un reste d'idée ?)

Yes, c'était dans l'idée de faire ça à cet endroit, mais ça ne marche pas.

Si j'ai compris il s'agit de ne pas avoir de doublons. On (Lauréline et moi) a deux questions :

c'est pour garder l'ordre que tu n'utilise pas un set ? Ou bien pour une question d'efficacité ?

Pour le seen ? C'est par habitude et certitude concernant les performances.

et aussi, pourquoi enlever les doublons ?

Quand on a une ParentContentCell, extend_with_parent_cells() est appelé pour étendre la liste des cellules figurant sur la page; l'appel faisait placeholder=self.placeholder et donc à étendre la zone "pied de page" on trouvait uniquement les cellules du pied de page, très bien.

À chercher également les cellules avec un placeholder commençant par un _, on attrape les "sous-cellules" (lien d'une cellule liste de liens, en l'occurence).

Les doublons arrivent s'il y a une seconde ParentContentCell; elle va aussi faire le même job, récupérer les mêmes "sous-cellules", et celles-ci vont se trouver deux fois dans la liste des cellules figurant dans la page.

Ensuite, la cellule LinkListCell va faire

        links = []
        for cell in context.get('page_cells', []):

et si dans page_cells on avait laissé les cellules en double, elles seraient ici reprises deux fois.