Produits Entr'ouvert » Combo

Je pas pouvoir finir la relecture, je bloque un peu pour l'instant sur le "for cell in cells:" qui modifie cells. En général, c'est mal.

Oui, ça serait plus sûr avec cells[:].

Dans get_cells, je trouve que c'est correct : il n'y aura toujours qu'une seule ParentContentCell, donc "break" dès que la première est rencontrée :

        for i, cell in enumerate(cells):
            if not isinstance(cell, ParentContentCell):
                continue
            cells[i:i+1] = cell.get_cells()
            break

Je serai pour prendre le même principe dans extend_with_parent_cells.

Autre chose : lors de PageSelectTemplateView, est-ce qu'on veut ajouter des ParentContentCell sur de nouveaux «placeholders acquired» éventuels, comme dans le nouvel PageAddView.form_valid ? (selon moi non, mais je voulais savoir si tu y as réfléchi aussi)

Je serai pour prendre le même principe dans extend_with_parent_cells.

On ne peut pas faire de break là parce qu'on peut avoir des "parent cell" dans différents placeholders de la page.

Autre chose : lors de PageSelectTemplateView, est-ce qu'on veut ajouter des ParentContentCell sur de nouveaux «placeholders acquired» éventuels, comme dans le nouvel PageAddView.form_valid ? (selon moi non, mais je voulais savoir si tu y as réfléchi aussi)

Lorsqu'on change de template si le nouveau possède des placeholders marqués acquired qui n'existaient pas avant, y mettre le ParentContentCell. Je pense que oui, il faudrait.

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

Je serai pour prendre le même principe dans extend_with_parent_cells.

On ne peut pas faire de break là parce qu'on peut avoir des "parent cell" dans différents placeholders de la page.

Effectivement.

Autre chose : lors de PageSelectTemplateView, est-ce qu'on veut ajouter des ParentContentCell sur de nouveaux «placeholders acquired» éventuels, comme dans le nouvel PageAddView.form_valid ? (selon moi non, mais je voulais savoir si tu y as réfléchi aussi)

Lorsqu'on change de template si le nouveau possède des placeholders marqués acquired qui n'existaient pas avant, y mettre le ParentContentCell. Je pense que oui, il faudrait.

Je pensais ce code délicat à faire. Je me trompe.

Je ne suis pas complètement concentrable non plus, mais ça me semble bon.