From 19d38101642fd2e75d74309ee29d76cea49e9ee3 Mon Sep 17 00:00:00 2001 From: william Date: Thu, 27 Aug 2026 06:49:41 +0200 Subject: [PATCH] =?UTF-8?q?Calcule=20rendered=5Fhtml=20pour=20CHAQUE=20?= =?UTF-8?q?=C3=A9l=C3=A9ment,=20pas=20seulement=20le=20premier=20niveau?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cause racine réelle des 2 précédents correctifs (commits 20adfa4, f4a7a73, 5e4e226, f016a81) qui n'avaient AUCUN effet visible malgré des redémarrages en règle : list_elements() ne calculait "rendered_html" que pour les éléments de PREMIER NIVEAU (parent_id NULL). Le nouveau ciblage côté JS (play.html) cherche pourtant à repérer, pour un élément imbriqué comme le Répéteur ou "mail content", s'il porte lui-même un marqueur ou un {{champ}} non résolu — mais côté serveur, ces éléments n'avaient tout simplement PAS de rendered_html du tout : `e.rendered_html` valait `undefined`, donc `hasMarker`/`hasUnresolvedPlaceholder` retombaient toujours à `false` pour eux, laissant SEUL le conteneur racine de premier niveau comme candidat — reproduisant exactement le bug d'origine (tout l'écran régénéré à chaque clic) qu'aucun des correctifs côté JS ne pouvait donc jamais résoudre, quelle que soit la justesse de leur logique de filtrage. Fix : chaque élément (imbriqué ou non) reçoit désormais son propre rendered_html — un élément imbriqué s'y retrouve deux fois (une fois dans le rendered_html de son ancêtre de premier niveau, utilisé pour le rendu HTML initial de la page ; une fois dans le sien propre, utilisé par le ciblage précis côté JS). Vérifié par simulation directe sur le payload réel du jeu de test : les cibles calculées sont maintenant exactement les 3 jauges, le Répéteur et "mail content" — plus jamais le conteneur racine de l'écran. Co-Authored-By: Claude Sonnet 5 --- screens/elements/list_elements.py | 24 ++++++++++++++++-------- 1 file changed, 16 insertions(+), 8 deletions(-) diff --git a/screens/elements/list_elements.py b/screens/elements/list_elements.py index 9025346b..cc755cae 100644 --- a/screens/elements/list_elements.py +++ b/screens/elements/list_elements.py @@ -7,12 +7,21 @@ from ..rendering.render_element_html import render_element_html def list_elements(slug, screen_id, enforce_visibility=False): """Renvoie TOUS les éléments de l'écran, à plat (parent_id indique - l'imbrication) — pratique pour construire l'arbre côté appelant. Seuls - les éléments de premier niveau (parent_id NULL) reçoivent un - "rendered_html" : celui d'un conteneur/répéteur/groupe de champs inclut - déjà, en le générant récursivement, le HTML de tout ce qui est posé à - l'intérieur — les enfants n'ont donc pas besoin d'être dessinés une - deuxième fois séparément. + l'imbrication) — pratique pour construire l'arbre côté appelant. CHAQUE + élément (premier niveau ou imbriqué) reçoit son propre "rendered_html" : + celui d'un conteneur/répéteur/groupe de champs inclut, en le générant + récursivement, le HTML de tout ce qui est posé à l'intérieur — un + élément imbriqué s'y retrouve donc DEUX FOIS (une fois dans le + rendered_html de son ancêtre de premier niveau, utilisé pour le rendu + HTML initial de la page ; une fois dans le sien propre) : c'est ce + dédoublement délibéré qui permet à play.html (voir applyOpenRowBindings/ + refreshRuntimeData) de régénérer ou patcher UN SEUL élément imbriqué + (ex. un panneau de détail posé dans un conteneur) sans reconstruire tout + son ancêtre de premier niveau — potentiellement l'écran entier. Avant ce + changement, seuls les éléments de premier niveau recevaient un + rendered_html ; un élément imbriqué n'en avait AUCUN côté client, qui + retombait donc toujours sur la régénération de son ancêtre de premier + niveau, quelle que soit la finesse du ciblage côté JS. enforce_visibility : True uniquement pour le mode jouable (voir full_game_payload.py) — applique la "condition de visibilité" de chaque @@ -66,6 +75,5 @@ def list_elements(slug, screen_id, enforce_visibility=False): children_map.setdefault(d["parent_id"], []).append(d) play_ctx = {"_forge_play_mode": True} if enforce_visibility else None for d in result: - if not d.get("parent_id"): - d["rendered_html"] = render_element_html(d, slug, children_map, play_ctx) + d["rendered_html"] = render_element_html(d, slug, children_map, play_ctx) return result