williamandClaude Sonnet 5
b094342097
Problème remonté : un déclencheur "Au clic" posé sur un Répéteur exécute
le MÊME graphe pour n'importe quelle ligne cliquée - or une Condition
("Si is_opened est égal à Non") ou une action "Modifier une donnée" ne
pouvaient viser qu'une ligne FIXE, choisie à la création du nœud dans
l'éditeur. Impossible donc de dire "modifie le champ DE LA LIGNE QUE JE
VIENS DE CLIQUER", puisque cette ligne n'est justement jamais connue à
l'avance.
Nouvelle valeur sentinelle CLICKED_ROW_ID (-1, screens/flow/constants.py,
ne collisionne jamais avec un vrai id de ligne) proposée en tête de TOUTE
liste déroulante "Ligne concernée" (clause principale et clauses
supplémentaires d'un nœud Condition, cible d'une action "Modifier une
donnée") : "🖱️ Ligne cliquée (Répéteur)".
Résolution au moment de l'exécution, pas à la création du nœud :
- Condition (évaluée côté client) : readFieldValue() (play.html) résout
-1 en window.lastClickedRowId, déjà capturé par bindClicks() au clic sur
une ligne de Répéteur (déjà utilisé par "Ouvrir la ligne cliquée").
- Action "Modifier une donnée" (exécutée côté serveur) : le client envoie
clicked_row_id dans le corps de la requête POST ; flow_node_run_data.py
ne s'en sert que si le nœud vise justement CLICKED_ROW_ID, sinon la
ligne fixe stockée sur le nœud reste utilisée normalement.
Ajoute tests/test_flow_clicked_row.py (ligne cliquée seule modifiée,
absence de clic = no-op plutôt que plantage, non-régression d'une cible
fixe, présence de l'option dans l'éditeur).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
27 lines
1.2 KiB
Python
27 lines
1.2 KiB
Python
from flask import jsonify, request
|
|
|
|
import screens
|
|
|
|
from core.flask_app import app
|
|
|
|
|
|
@app.route("/game/<slug>/flow/nodes/<int:node_id>/run-data", methods=["POST"])
|
|
def flow_node_run_data(slug, node_id):
|
|
"""Exécuté côté serveur au moment du clic, en mode jouable, pour un nœud
|
|
Action de type "Modifier une donnée d'un objet" — identique au
|
|
fonctionnement de l'ancienne action_run_data, mais lit sa config depuis
|
|
un nœud du graphe de logique plutôt que depuis la table _actions.
|
|
|
|
Si le nœud vise CLICKED_ROW_ID (voir "Ligne cliquée (Répéteur)" dans
|
|
screen_edit.html), la vraie ligne n'est connue qu'au moment du clic —
|
|
le client (play.html) la transmet dans le corps JSON de cette requête,
|
|
et elle remplace ici target_row_id avant d'exécuter l'action."""
|
|
node = screens.get_flow_node(slug, node_id)
|
|
if not node or node.get("node_type") != "action" or node.get("action_type") != "modifier_donnee":
|
|
return jsonify({"ok": False}), 400
|
|
if node.get("target_row_id") == screens.CLICKED_ROW_ID:
|
|
body = request.get_json(silent=True) or {}
|
|
node = {**node, "target_row_id": body.get("clicked_row_id")}
|
|
ok = screens.apply_data_action(slug, node)
|
|
return jsonify({"ok": ok})
|