Produits Entr'ouvert » Combo

(juste en passant, je veux bien prendre du temps là-dessus à mon retour, bien sûr).

Le truc important de la description : chaque lien serait défini par un type (ex: "url" ou "formulaire wcs"), genre il faudrait que combo.apps.wcs puisse exposer l'info qu'il existe des liens de type "démarches", qui correspondraient à ce qu'on a dans les attributs de WcsFormCell, des liens de type "pages" qui correspondait à l'id de la page, des liens de type "url" qui auraient titre/url, etc. il y a peut-être un truc à inventer en réutilisant/qualifiant les cellules LinkCell/WcsFormCell comme étant des cellules "liens", qu'elles soient créées/mises dans un placeholder au nom particulier, etc.

(désolé encore d'être bref ici)

J'espérais stopper un peu les choses en posant mon commentaire précédent :) Comment je vois les choses :

En gros, une cellule, LinkListCell, ok très bien.

Mais ce qui va dedans, ça ne doit pas être un nouvel objet, ça doit être basé sur ce qu'on a déjà, LinkCell, WcsFormCell.

À ces classes on ajouterait un attribut quelconque, add_as_link_label = _('add a link'), add_as_link_label = _('add a form link').

En bas de la cellule, à la place du "Add a link", on aurait donc une série de ces libellés.

Cliquer dessus, popup, qui affichera le formulaire du get_default_form_class() de la classe.

Validation de la popup, enregistrement de l'objet ("cellule"), avec dedans quelque chose comme placeholder = 'linkslist:%(pk)s'.

Affichage dans la cellule "liste de liens" comme tu le fais, en prenant la liste des cellules de la page qui ont placeholder == '...'.

Sans doute ajouter aux différentes cellules une interface commune, get_label(), get_target(), get_description(), qui seront appelés pour le rendu de la cellule.

Réfléchir aux différentes trucs, export/import, duplication de page, comme tu l'as déjà fait.

Bref, je dirais remplacer LinkItem par les objets existants, et la liaison ForeignKey qui se faisait dedans par une valeur dans l'attribut placeholder.

Je n'ai touché à rien depuis ton commentaire précédent :)

Ok merci pour les éclaircissements, je vais partir là-dessus

n'importe quoi, mauvais ticket ...
je nettoie

+        return CellBase.get_cells(page=self.page, placeholder=self.link_placeholder)

Tu peux ici ajouter un paramètre cell_filter, genre cell_filter=lambda x: getattr(x, 'add_link_label'), pour limiter les requêtes SQL qui auront lieu aux seuls modèles possibles. Même si en passant page=self.page on obtient déjà un filtre aux seuls types de cellule présents sur la page. Et écrivant ça je me demande si justement c'est souhaitable, si ça ne va pas faire foirer le cas où la cellule "liste de liens" est posée sur une page parente, dans un placeholder "hérité".

De là je me demande si de toute façon ce qu'il faudrait arriver à faire ne serait pas dans get_cell_extra_context() exploiter "page_cells" qui doit être dans le contexte (via publish_page), et reprendre toutes les cellules, directement sur la page, ou héritées, plutôt que passer par get_items() qui fera de nouvelles requêtes dans la db.

_~~

Sur le reste je ne pense pas avoir de remarque importante, pour le rendu, tu peux adapter le style de la liste des liens en faisant :

+<div class="links-list">
 <ul>
 {% for link in links %}
   <li><a href="{{ link.url }}">{{ link.title }}</a></li>
 {% endfor %}
 </ul>
+</div>

et regarder le rendu me fait penser qu'on devrait avoir un champ titre (optionnel) sur cette cellule.