Forge-Store/Forge-Engine
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Corrige le ciblage élément+ligne transmis par un événement dans un Répéteur

Browse Source 
Signalé par l'utilisateur : quand "Modifier un élément" utilise la case
"Utiliser l'élément transmis par l'événement en cours"
(target_element_from_event), et que cet élément vit à l'intérieur d'un
Répéteur, la résolution ne ciblait jamais que le PREMIER élément
correspondant trouvé dans toute la page — jamais forcément la bonne
ligne.

Vérifié directement sur un rendu réel : chaque ligne d'un Répéteur
rejoue le MÊME modèle (voir render_repeater.py), donc le MÊME
data-element-id se répète à l'identique sur CHAQUE .repeaterItem — seul
data-row-id (posé sur l'enveloppe .repeaterItem) distingue réellement
une ligne d'une autre. Un simple
document.querySelector('[data-element-id]') global tombe donc toujours
sur la première ligne rencontrée dans le DOM, sans rapport avec la ligne
réellement transmise par l'événement (window.lastEventParams.row_id).

Corrigé : quand l'événement transmet aussi une ligne, la recherche est
désormais scopée à l'intérieur du .repeaterItem[data-row-id=...]
correspondant avant d'y chercher l'élément — sinon (élément fixe, ou
événement sans paramètre de ligne), le comportement global d'avant reste
inchangé.

210 tests toujours verts (ce correctif est purement côté client, jamais
couvert par les tests automatisés — vérifié manuellement via un rendu
réel confirmant la structure .repeaterItem/data-row-id décrite
ci-dessus).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
william
2026-08-29 20:23:11 +02:00
co-authored by Claude Sonnet 5
parent f436190f90
commit 666aa892e0
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
templates/play.html
+23 -1
View File
@@ -796,10 +796,32 @@
// l'éditeur — contourne la contrainte de clé étrangère de // l'éditeur — contourne la contrainte de clé étrangère de
// target_element_id, qui empêche d'y stocker un sentinel comme // target_element_id, qui empêche d'y stocker un sentinel comme
// EVENT_ROW_ID directement (voir screens/flow/ensure_flow_schema.py). // EVENT_ROW_ID directement (voir screens/flow/ensure_flow_schema.py).
//
// data-element-id N'EST PAS unique quand l'élément vit dans un
// Répéteur : chaque ligne rejoue le MÊME modèle, donc le MÊME
// data-element-id se répète à l'identique sur chaque ligne (voir
// render_repeater.py, seul data-row-id — posé sur .repeaterItem,
// l'englobant — distingue réellement une ligne d'une autre). Un
// simple document.querySelector('[data-element-id]') global
// prendrait toujours la PREMIÈRE ligne, jamais la bonne — sans
// rapport avec l'événement reçu. Quand l'événement transmet aussi
// une ligne (row_id), la recherche est donc scopée à L'INTÉRIEUR de
// ce .repeaterItem précis ; sinon (élément fixe, ou événement sans
// paramètre de ligne), recherche globale comme avant.
const resolvedElementId = node.target_element_from_event const resolvedElementId = node.target_element_from_event
? (window.lastEventParams ? window.lastEventParams.element_id : null) ? (window.lastEventParams ? window.lastEventParams.element_id : null)
: node.target_element_id; : node.target_element_id;
const targetEl = resolvedElementId ? document.querySelector('[data-element-id="' + resolvedElementId + '"]') : null; const resolvedRowId = node.target_element_from_event && window.lastEventParams
? window.lastEventParams.row_id : null;
let targetEl = null;
if (resolvedElementId != null) {
if (resolvedRowId != null) {
const rowEl = document.querySelector('.repeaterItem[data-row-id="' + resolvedRowId + '"]');
targetEl = rowEl ? rowEl.querySelector('[data-element-id="' + resolvedElementId + '"]') : null;
} else {
targetEl = document.querySelector('[data-element-id="' + resolvedElementId + '"]');
}
}
if (targetEl) applyElementProperty(targetEl, node.element_property, node.element_value); if (targetEl) applyElementProperty(targetEl, node.element_property, node.element_value);
return Promise.resolve(); return Promise.resolve();
} else if (node.action_type === 'declencher_evenement' && node.target_custom_event_id) { } else if (node.action_type === 'declencher_evenement' && node.target_custom_event_id) {