diff --git a/db/constants.py b/db/constants.py index 184f754c..e9f4739e 100644 --- a/db/constants.py +++ b/db/constants.py @@ -18,10 +18,16 @@ FIELD_TYPES = { # Types disponibles pour une variable globale (voir db/global_vars/) — un # sous-ensemble de FIELD_TYPES : ni "relation" (une variable globale n'a pas # d'objet à pointer) ni "texte_long"/"date" (pas gérés par -# apply_variable_action.py, qui ne coerce que ces 4 types). +# apply_variable_action.py, qui ne coerce que ces 4 types). "objet"/ +# "tableau" sont, eux, propres aux variables (pas de colonne SQL réelle, +# contrairement à un champ d'objet) : la valeur est stockée telle quelle +# (colonne TEXT) sous forme de JSON, lue via un chemin ".champ"/"[index]" +# (voir _resolve_variable_path dans screens/rendering/filter_repeater_rows.py). GLOBAL_VARIABLE_TYPES = { "texte": FIELD_TYPES["texte"], "nombre_entier": FIELD_TYPES["nombre_entier"], "nombre_decimal": FIELD_TYPES["nombre_decimal"], "booleen": FIELD_TYPES["booleen"], + "objet": {"label": "Objet (JSON)"}, + "tableau": {"label": "Tableau (JSON)"}, } diff --git a/db/global_vars/coerce_structured_value.py b/db/global_vars/coerce_structured_value.py new file mode 100644 index 00000000..9ab1492c --- /dev/null +++ b/db/global_vars/coerce_structured_value.py @@ -0,0 +1,21 @@ +import json + + +def coerce_structured_value(var_type, value): + """Pour une variable "objet"/"tableau" (voir db/constants.py), la + valeur stockée doit rester du JSON analysable — sinon la prochaine + lecture (_resolve_variable_path, screens/rendering/ + filter_repeater_rows.py) échouerait silencieusement à chaque fois. + Un JSON invalide retombe donc sur un défaut sûr ("{}"/"[]") plutôt que + d'être stocké tel quel. Les autres types ne sont jamais touchés ici.""" + if var_type not in ("objet", "tableau"): + return value + text = (value or "").strip() + fallback = "{}" if var_type == "objet" else "[]" + if not text: + return fallback + try: + json.loads(text) + except (TypeError, ValueError): + return fallback + return text diff --git a/db/global_vars/create_global_variable.py b/db/global_vars/create_global_variable.py index 29501df1..e2a40c0c 100644 --- a/db/global_vars/create_global_variable.py +++ b/db/global_vars/create_global_variable.py @@ -1,5 +1,6 @@ from ..connection import connect from .ensure_global_vars_schema import ensure_global_vars_schema +from .coerce_structured_value import coerce_structured_value def create_global_variable(slug, name, var_type="texte", default_value=""): @@ -11,6 +12,7 @@ def create_global_variable(slug, name, var_type="texte", default_value=""): name = (name or "").strip() if not name: return None + default_value = coerce_structured_value(var_type, default_value) ensure_global_vars_schema(slug) conn = connect(slug) existing = conn.execute("SELECT id FROM _global_variables WHERE name = ?", (name,)).fetchone() diff --git a/db/global_vars/update_global_variable.py b/db/global_vars/update_global_variable.py index 98c26831..93e927c0 100644 --- a/db/global_vars/update_global_variable.py +++ b/db/global_vars/update_global_variable.py @@ -1,16 +1,18 @@ from ..connection import connect from .ensure_global_vars_schema import ensure_global_vars_schema +from .coerce_structured_value import coerce_structured_value def update_global_variable(slug, var_id, var_type, value): - """Modifie le type et la valeur d'une variable existante depuis la page - de gestion (voir game_variables.html) — le NOM reste volontairement - immuable après création : c'est par ce nom qu'une condition de - visibilité ou une action "Modifier une variable" la référence (voir - get_global_variable.py), la renommer casserait ces réglages en - silence.""" + """Modifie le type et la valeur d'une variable existante depuis le + tableau de bord (onglet Variables, game_dashboard.html) — le NOM reste + volontairement immuable après création : c'est par ce nom qu'une + condition de visibilité ou une action "Modifier une variable" la + référence (voir get_global_variable.py), la renommer casserait ces + réglages en silence.""" + value = coerce_structured_value(var_type, str(value)) ensure_global_vars_schema(slug) conn = connect(slug) - conn.execute("UPDATE _global_variables SET type = ?, value = ? WHERE id = ?", (var_type, str(value), var_id)) + conn.execute("UPDATE _global_variables SET type = ?, value = ? WHERE id = ?", (var_type, value, var_id)) conn.commit() conn.close() diff --git a/routes/element_types/element_type_delete.py b/routes/element_types/element_type_delete.py index 7b268ba3..1172c0bf 100644 --- a/routes/element_types/element_type_delete.py +++ b/routes/element_types/element_type_delete.py @@ -9,4 +9,4 @@ from core.flask_app import app def element_type_delete(slug, element_type_id): if screens.element_type_usage_count(slug, element_type_id) == 0: screens.delete_element_type(slug, element_type_id) - return redirect(url_for("element_types", slug=slug)) + return redirect(url_for("game_dashboard", slug=slug, tab="element_types")) diff --git a/routes/element_types/element_type_edit.py b/routes/element_types/element_type_edit.py index a1ce7b90..75660849 100644 --- a/routes/element_types/element_type_edit.py +++ b/routes/element_types/element_type_edit.py @@ -13,4 +13,4 @@ def element_type_edit(slug, element_type_id): name = request.form.get("name", "").strip() or None bound_definition_id = request.form.get("bound_definition_id") or None screens.update_element_type(slug, element_type_id, name=name, bound_definition_id=bound_definition_id) - return redirect(url_for("element_types", slug=slug)) + return redirect(url_for("game_dashboard", slug=slug, tab="element_types")) diff --git a/routes/element_types/element_types.py b/routes/element_types/element_types.py index f00a3c88..f4f9fe6d 100644 --- a/routes/element_types/element_types.py +++ b/routes/element_types/element_types.py @@ -1,6 +1,5 @@ -from flask import render_template, request, redirect, url_for +from flask import request, redirect, url_for -import db import screens from core.flask_app import app @@ -8,18 +7,16 @@ from core.flask_app import app @app.route("/game//element-types", methods=["GET", "POST"]) def element_types(slug): - game = db.game_meta(slug) + """La création/gestion d'un élément de jeu se fait désormais depuis + l'onglet "Éléments de jeu" du tableau de bord (voir game_dashboard.html) + plutôt qu'une page dédiée — cette route ne fait donc plus que traiter la + création (barre de création repliable de cet onglet). Un GET direct + redirige vers le tableau de bord.""" if request.method == "POST": name = request.form.get("name", "").strip() icon = request.form.get("icon", "").strip() or "🧩" bound_definition_id = request.form.get("bound_definition_id") or None if name: screens.create_element_type(slug, name, icon, bound_definition_id) - return redirect(url_for("element_types", slug=slug)) - types = screens.list_element_types(slug) - usage = {t["id"]: screens.element_type_usage_count(slug, t["id"]) for t in types} - definitions = db.list_definitions(slug) - return render_template( - "element_types.html", game=game, types=types, usage=usage, widgets=screens.WIDGETS, - definitions=definitions, - ) + return redirect(url_for("game_dashboard", slug=slug, tab="element_types")) + return redirect(url_for("game_dashboard", slug=slug, tab="element_types")) diff --git a/routes/games/game_dashboard.py b/routes/games/game_dashboard.py index f46eb42d..bbb71a0e 100644 --- a/routes/games/game_dashboard.py +++ b/routes/games/game_dashboard.py @@ -1,12 +1,63 @@ from flask import render_template import db +import screens from core.flask_app import app @app.route("/game/") def game_dashboard(slug): + """Vue d'ensemble du jeu : tout ce qui a été créé (écrans, objets, + éléments de jeu réutilisables, variables), en un coup d'œil, avec un + accès direct à chaque élément — la barre de navigation du jeu (voir + base.html) permet déjà de sauter directement vers chaque éditeur, cette + page sert donc de résumé/point d'entrée plutôt que de répéter les mêmes + liens.""" game = db.game_meta(slug) definitions = db.list_definitions(slug) - return render_template("game_dashboard.html", game=game, definitions=definitions) + # Détail complet (avec ses champs) de chaque objet, pour le panneau + # d'édition déplaçable (voir game_dashboard.html) ouvert par le crayon + # ✏️ de l'onglet Objets — un panneau par objet, pré-rendu ici et + # simplement montré/masqué en JS, plutôt qu'un aller-retour serveur à + # l'ouverture (voir openObjectEditPanel()). + definitions_full = {d["id"]: db.get_definition(slug, d["id"]) for d in definitions} + + # Pour l'onglet "Données" de ce même panneau (remplace data_list.html/ + # object_view.py, retirés) : les lignes déjà enregistrées de chaque + # objet, les libellés lisibles de ses champs relation (pour l'affichage + # du tableau) et les options possibles de ces mêmes champs relation + # (pour le formulaire "Ajouter une entrée"). + rows_by_definition = {} + relation_labels_by_definition = {} + relation_options_by_definition = {} + referenced_by_definition = {} + for d in definitions: + full = definitions_full[d["id"]] + rows_by_definition[d["id"]] = db.list_rows(slug, full) + relation_labels = {} + relation_options = {} + for f in full["fields"]: + if f["type"] == "relation": + options = db.relation_options(slug, f["relation_definition_id"]) + relation_labels[f["id"]] = {opt["id"]: opt["label"] for opt in options} + relation_options[f["id"]] = options + relation_labels_by_definition[d["id"]] = relation_labels + relation_options_by_definition[d["id"]] = relation_options + referenced_by_definition[d["id"]] = db.definitions_referencing(slug, d["id"]) + + screen_list = screens.list_screens(slug) + element_types = screens.list_element_types(slug) + element_type_usage = {t["id"]: screens.element_type_usage_count(slug, t["id"]) for t in element_types} + variables = db.list_global_variables(slug) + return render_template( + "game_dashboard.html", game=game, definitions=definitions, + definitions_full=definitions_full, + rows_by_definition=rows_by_definition, + relation_labels_by_definition=relation_labels_by_definition, + relation_options_by_definition=relation_options_by_definition, + referenced_by_definition=referenced_by_definition, + screen_list=screen_list, element_types=element_types, + element_type_usage=element_type_usage, variables=variables, + variable_types=db.GLOBAL_VARIABLE_TYPES, field_types=db.FIELD_TYPES, + ) diff --git a/routes/global_vars/__init__.py b/routes/global_vars/__init__.py index e076e9b1..0da1055e 100644 --- a/routes/global_vars/__init__.py +++ b/routes/global_vars/__init__.py @@ -1,4 +1,3 @@ from .create_global_var import create_global_var -from .game_variables import game_variables from .global_var_edit import global_var_edit from .global_var_delete import global_var_delete diff --git a/routes/global_vars/create_global_var.py b/routes/global_vars/create_global_var.py index 38e88827..4f50a41f 100644 --- a/routes/global_vars/create_global_var.py +++ b/routes/global_vars/create_global_var.py @@ -15,4 +15,4 @@ def create_global_var(slug): default_value = request.form.get("default_value") or "" if name: db.create_global_variable(slug, name, var_type, default_value) - return redirect(url_for("game_variables", slug=slug)) + return redirect(url_for("game_dashboard", slug=slug, tab="variables")) diff --git a/routes/global_vars/game_variables.py b/routes/global_vars/game_variables.py deleted file mode 100644 index 80ae9ff0..00000000 --- a/routes/global_vars/game_variables.py +++ /dev/null @@ -1,19 +0,0 @@ -from flask import render_template - -import db - -from core.flask_app import app - - -@app.route("/game//variables") -def game_variables(slug): - """Page de gestion des variables globales (voir db/global_vars/) : un - objet "jeu" implicite dont les propriétés/valeurs sont visibles partout - (condition de visibilité, action "Modifier une variable"), plutôt - qu'attachées à un objet de données précis.""" - game = db.game_meta(slug) - variables = db.list_global_variables(slug) - return render_template( - "game_variables.html", game=game, variables=variables, - variable_types=db.GLOBAL_VARIABLE_TYPES, - ) diff --git a/routes/global_vars/global_var_delete.py b/routes/global_vars/global_var_delete.py index 4aa8920d..9a89f507 100644 --- a/routes/global_vars/global_var_delete.py +++ b/routes/global_vars/global_var_delete.py @@ -8,4 +8,4 @@ from core.flask_app import app @app.route("/game//variables//delete", methods=["POST"]) def global_var_delete(slug, var_id): db.delete_global_variable_by_id(slug, var_id) - return redirect(url_for("game_variables", slug=slug)) + return redirect(url_for("game_dashboard", slug=slug, tab="variables")) diff --git a/routes/global_vars/global_var_edit.py b/routes/global_vars/global_var_edit.py index d09ef66f..edf1cd63 100644 --- a/routes/global_vars/global_var_edit.py +++ b/routes/global_vars/global_var_edit.py @@ -10,4 +10,4 @@ def global_var_edit(slug, var_id): var_type = request.form.get("type") or "texte" value = request.form.get("value") or "" db.update_global_variable(slug, var_id, var_type, value) - return redirect(url_for("game_variables", slug=slug)) + return redirect(url_for("game_dashboard", slug=slug, tab="variables")) diff --git a/routes/objects/data_delete.py b/routes/objects/data_delete.py index d0a60a78..e1cda644 100644 --- a/routes/objects/data_delete.py +++ b/routes/objects/data_delete.py @@ -13,8 +13,9 @@ def data_delete(slug, definition_id, row_id): referencing = db.rows_referencing(slug, definition_id, row_id) if referencing: # Suppression bloquée : d'autres lignes pointent vers celle-ci via - # une relation. On revient à la liste, qui explique pourquoi via un - # paramètre de requête simple plutôt qu'une session/flash message. - return redirect(url_for("object_view", slug=slug, definition_id=definition_id, blocked_row=row_id)) + # une relation. On revient au panneau de cet objet (onglet + # Données), qui explique pourquoi via un paramètre de requête + # simple plutôt qu'une session/flash message. + return redirect(url_for("game_dashboard", slug=slug, tab="objects", edit=definition_id, subtab="data", blocked_row=row_id)) db.delete_row(slug, definition, row_id) - return redirect(url_for("object_view", slug=slug, definition_id=definition_id)) + return redirect(url_for("game_dashboard", slug=slug, tab="objects", edit=definition_id, subtab="data")) diff --git a/routes/objects/data_form.py b/routes/objects/data_form.py index 5fbe0d37..49e68601 100644 --- a/routes/objects/data_form.py +++ b/routes/objects/data_form.py @@ -1,32 +1,24 @@ -from flask import render_template, request, redirect, url_for, abort +from flask import request, redirect, url_for, abort import db def _data_form(slug, definition_id, row): - game = db.game_meta(slug) + """Ajouter/modifier une entrée se fait désormais depuis l'onglet + "Données" du panneau déplaçable de son objet (voir game_dashboard.html, + ouvert par le crayon ✏️ de l'onglet Objets) plutôt qu'une page dédiée + (data_form.html, retirée) — cette fonction ne fait donc plus que + traiter la soumission de ce formulaire. Un GET direct redirige vers le + tableau de bord, en rouvrant le panneau de cet objet sur son onglet + Données.""" definition = db.get_definition(slug, definition_id) if not definition: abort(404) - # Pour chaque champ de type relation, on prépare les options du - - {% elif f.type == 'texte_long' %} - - - {% elif f.type == 'nombre_entier' %} - - - {% elif f.type == 'nombre_decimal' %} - - - {% elif f.type == 'booleen' %} - - - {% elif f.type == 'date' %} - - - {% elif f.type == 'relation' %} - {% set current_rel = row[col ~ '_id'] if row else none %} -
- -
- {% if not relation_options.get(f.id) %} -

Aucune entrée n'existe encore pour l'objet lié — crée-en une d'abord si tu veux pouvoir la sélectionner ici.

- {% endif %} - {% endif %} - - - {% endfor %} - - - - -{% endblock %} diff --git a/templates/data_list.html b/templates/data_list.html deleted file mode 100644 index 0f9855e1..00000000 --- a/templates/data_list.html +++ /dev/null @@ -1,128 +0,0 @@ -{% extends "base.html" %} -{% block title %}{{ definition.name }} — {{ game.name }}{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %} -
  • {{ game.name }}
    • -
  • {{ definition.name }}
    • -{% endblock %} -{% block content %} -

    {{ definition.name }} - ✏️ -

    -

    Table SQL : {{ definition.table_name }} — {{ definition.fields|length }} champ(s)

    - -{% if referenced_by %} -
    - ⚠️ D'autres objets ont une relation vers « {{ definition.name }} » : - {{ referenced_by|map(attribute='name')|join(', ') }}. - La suppression de cet objet est bloquée tant que ces relations existent. -
    -{% endif %} - -{% if blocked_row_id %} -
    - ⚠️ Impossible de supprimer l'entrée #{{ blocked_row_id }} : elle est référencée par - {% for ref in blocked_row_refs %}{{ ref.count }} entrée(s) de « {{ ref.definition_name }} » (champ « {{ ref.field_name }} »){% if not loop.last %}, {% endif %}{% endfor %}. - Modifie ou supprime d'abord ces entrées. -
    -{% endif %} - -
    -
    -

    Ajouter

    -
    -

    Une entrée, c'est une ligne de la table SQL « {{ definition.table_name }} ».

    -
    - + Ajouter une entrée -
    -
    -
    - -
    -

    Données enregistrées

    - {% if rows %} - {% macro row_field_display(f, r) %} - {%- if f.type == 'relation' -%} - {{ relation_labels.get(f.id, {}).get(r[(f.name|colname) ~ '_id'], '—') }} - {%- elif f.type == 'booleen' -%} - {{ 'Oui' if r[f.name|colname] else 'Non' }} - {%- else -%} - {{ r[f.name|colname] }} - {%- endif -%} - {% endmacro %} -
    - - - - - {% for f in definition.fields %}{% endfor %} - - - - - {% for r in rows %} - - - {% for f in definition.fields %} - - {% endfor %} - - - {% endfor %} - -
    #{{ f.name }}
    {{ r.id }}{{ row_field_display(f, r) }} -
    - - ✏️ -
    - -
    -
    -
    -
    - - {% for r in rows %} - - {% endfor %} - {% else %} -

    Aucune donnée enregistrée pour l'instant.

    - {% endif %} -
    -
    - - -{% endblock %} diff --git a/templates/element_types.html b/templates/element_types.html deleted file mode 100644 index 7e0c26bb..00000000 --- a/templates/element_types.html +++ /dev/null @@ -1,73 +0,0 @@ -{% extends "base.html" %} -{% block title %}Éléments de jeu — {{ game.name }}{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %} -
  • {{ game.name }}
    • -
  • Éléments de jeu
    • -{% endblock %} -{% block content %} -

    Éléments de jeu

    -

    Un élément de jeu est un conteneur réutilisable.

    - -
    -
    -

    Créer

    -
    -
    -
    - - -
    -
    - - -
    -
    -
    - - -
    -
    -
    -
    - -
    -

    Catalogue

    - {% if types %} -
    - - - - {% for t in types %} - - - - - - {% endfor %} - -
    NomUsage
    {{ t.icon }} {{ t.name }} - {{ usage.get(t.id, 0) }} instance(s) - {% if t.bound_definition_id %} - {% for d in definitions %}{% if d.id == t.bound_definition_id %} · « {{ d.name }} »{% endif %}{% endfor %} - {% endif %} - - ✏️ -
    - -
    -
    -
    - {% else %} -

    Aucun élément de jeu défini pour l'instant.

    - {% endif %} -
    -
    -{% endblock %} diff --git a/templates/game_dashboard.html b/templates/game_dashboard.html index 54fb5922..1fce0814 100644 --- a/templates/game_dashboard.html +++ b/templates/game_dashboard.html @@ -1,63 +1,1116 @@ {% extends "base.html" %} {% block title %}{{ game.name }} — Forge Engine{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %}
  • {{ game.name }}
    • {% endblock %} +{% block content_class %}content-wide{% endblock %} {% block content %} -

    {{ game.name }}

    -

    Base de données : projects/{{ game.slug }}/game.db

    -
    -
    -

    Créer

    -
    - 🖥️ Écrans du jeu - 🧩 Éléments de jeu - 🔧 Variables - ▶️ Jouer - 🗂️ Objet + +
    + + + + +
    + + +
    + {% if request.args.get('blocked_screen') %} +
    + ⚠️ Suppression impossible : au moins un autre écran a un bouton qui cible précisément celui-ci. Retire d'abord cette action. +
    + {% endif %} +
    +

    Écrants

    + +
    + + {% if screen_list %} +
    + + + + {% for s in screen_list %} + + + + + + {% endfor %} + +
    #Nom
    {{ loop.index }} + {{ s.name }} + {% if loop.first %} ★ Départ{% endif %} + +
    +
    + ✏️ +
    + +
    +
    +
    + {% else %} +

    Aucun écran pour l'instant.

    + {% endif %} +
    + + +
    +
    +

    Objets

    + +
    + {% if definitions %} +
    + + + + {% for d in definitions %} + + + + + + {% endfor %} + +
    NomTable SQL
    {{ d.table_name }} + +
    +
    + {% else %} +

    Aucun objet pour l'instant.

    + {% endif %} +
    + + +
    +
    +

    Templates

    + +
    + + {% if element_types %} +
    + + + + {% for t in element_types %} + + + + + + {% endfor %} + +
    NomUsage
    {{ t.icon or '🧩' }} {{ t.name }} + {{ element_type_usage.get(t.id, 0) }} instance(s) + {% if t.bound_definition_id %} + {% for d in definitions %}{% if d.id == t.bound_definition_id %} · « {{ d.name }} »{% endif %}{% endfor %} + {% endif %} + + ✏️ +
    + +
    +
    +
    + {% else %} +

    Aucun template pour l'instant.

    + {% endif %} +
    + + +
    +
    +

    Variables

    + +
    + + {% if variables %} + {% for v in variables %} +
    + {% endfor %} +
    + + + + {% for v in variables %} + + + + + + + {% endfor %} + +
    NomTypeValeur
    {{ v.name }} + + + {% if v.type in ('objet', 'tableau') %} +
    + + +
    + {% else %} + + {% endif %} +     		+ +
    + +
    +
    +
    + {% else %} +

    Aucune variable pour l'instant.

    + {% endif %} +
    + + + + + + + +
    +

    Paramètres du jeu

    +
    + +
    + + +
    +
    +
    + +
    +
    + + + + + + + +{% for d in definitions %} +{% set definition = definitions_full[d.id] %} +{% set existing_definitions = definitions|rejectattr('id', 'equalto', d.id)|list %} + + + + + +{% if definition.fields %} + + + + +{% endif %} +{% endfor %} + + + + + {% endblock %} diff --git a/templates/game_variables.html b/templates/game_variables.html deleted file mode 100644 index 5af74bcf..00000000 --- a/templates/game_variables.html +++ /dev/null @@ -1,75 +0,0 @@ -{% extends "base.html" %} -{% block title %}Variables — {{ game.name }}{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %} -
  • {{ game.name }}
    • -
  • Variables
    • -{% endblock %} -{% block content %} -

    Variables globales

    -

    Des propriétés/valeurs communes à tout le jeu (pas attachées à un objet de données précis) — utile par exemple pour une porte ouverte ou non, un niveau débloqué... Une variable peut être lue dans une "Condition de visibilité" (propriétés d'un élément) et modifiée par une action "Modifier une variable" (Logique de la scène).

    - -
    -
    -

    Créer

    -
    -
    - - -
    -
    - - -
    -
    - - -
    -
    -
    -
    - -
    -

    Variables définies

    - {% if variables %} - {% for v in variables %} -
    - {% endfor %} -
    - - - - {% for v in variables %} - - - - - - - {% endfor %} - -
    NomTypeValeur
    {{ v.name }} - - - -
    - -
    -
    -
    - {% else %} -

    Aucune variable pour l'instant — crée la première à gauche.

    - {% endif %} -
    -
    -{% endblock %} diff --git a/templates/index.html b/templates/index.html index 0f1b71fa..0beacafb 100644 --- a/templates/index.html +++ b/templates/index.html @@ -1,6 +1,5 @@ {% extends "base.html" %} {% block title %}Forge Engine — Mes jeux{% endblock %} -{% block breadcrumb_wrap %}{% endblock %} {% block content_class %}content-wide content-objectEdit{% endblock %} {% block body_class %}objectEditBody{% endblock %} {% block content %} diff --git a/templates/object_edit.html b/templates/object_edit.html deleted file mode 100644 index 8f6db622..00000000 --- a/templates/object_edit.html +++ /dev/null @@ -1,194 +0,0 @@ -{% extends "base.html" %} -{% block title %}Modifier {{ definition.name }} — {{ game.name }}{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %} -
  • {{ game.name }}
    • -
  • {{ definition.name }}
    • -
  • Modifier
    • -{% endblock %} -{% block content %} -

    Modifier l'objet « {{ definition.name }} »

    -

    Table SQL : {{ definition.table_name }}

    - -
    -
    - - -
    -
    - -
    -
    - -
    - -
    -

    Ajouter un champ

    -
    -
    - -
    -
    - -
    - -
    -
    - -
    -
    -
    - - - - - -
    - -
    -
    - -
    -
    -
    - -
    -
    - -
    -

    Supprimer cet objet

    -
    - -
    -
    -
    - - -
    -

    Champs existants

    - {% if definition.fields %} - {% for f in definition.fields %} -
    -
    - {% endfor %} -
    - - - - - - - - - - - - - - {% for f in definition.fields %} - - - - - - - - - - {% endfor %} - -
    Nom du champTypeObjet liéMiniMaxiObligatoire ?
    {{ field_types.get(f.type, {}).get('label', f.type) }} - {% if f.type == 'relation' %} -
    - -
    - {% else %} - — - {% endif %} -     		- {% if f.type in ('nombre_entier', 'nombre_decimal') %} - - {% else %} - — - {% endif %} - - {% if f.type in ('nombre_entier', 'nombre_decimal') %} - - {% else %} - — - {% endif %} - -
    - -
    -     		- - -
    -
    -

    Mini/Maxi (bornage automatique) : laisser vide = pas de limite. S'applique à "Donner cette valeur", "Augmenter de..." et "Diminuer de..." dans le graphe de logique.

    - {% else %} -

    Aucun champ pour l'instant.

    - {% endif %} -
    -
    - - -{% endblock %} diff --git a/templates/object_form.html b/templates/object_form.html deleted file mode 100644 index 4d0fa670..00000000 --- a/templates/object_form.html +++ /dev/null @@ -1,104 +0,0 @@ -{% extends "base.html" %} -{% block title %}Nouvel objet — {{ game.name }}{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %} -
  • {{ game.name }}
    • -
  • Nouvel objet
    • -{% endblock %} -{% block content %} -

    Définir un nouvel objet

    - -
    -
    -
    -

    Objet

    -
    -
    - - -
    -
    -
    -
    - -
    -

    Champs

    - -
    -
    -
    -
    -
    -
    - - - - -{% endblock %} diff --git a/templates/play.html b/templates/play.html index ab635e79..b49828f0 100644 --- a/templates/play.html +++ b/templates/play.html @@ -163,33 +163,69 @@ } function applyAnimationClip(clip) { - var target = document.querySelector('[data-anim-target="' + clip.element_id + '"]'); - if (!target) return; + // querySelectorAll (pas querySelector) : un clip posé dans l'éditeur d'un + // écran-MODÈLE (élément de jeu réutilisable) vise l'un de ses propres + // enfants — un id qui se répète à l'IDENTIQUE à chaque exemplaire de ce + // modèle (rechargé en direct depuis le même écran-modèle, voir + // collectAnimationClips ci-dessous), y compris pour chaque ligne d'un + // Répéteur utilisant ce modèle comme gabarit. Un seul querySelector + // n'aurait animé que le tout premier exemplaire trouvé dans le DOM. + var targets = document.querySelectorAll('[data-anim-target="' + clip.element_id + '"]'); + if (!targets.length) return; var duration = clip.duration || 1, delay = clip.delay || 0; var iterationCount = clip.iteration_count || '1'; var infinite = iterationCount === 'infinite'; - if (clip.kind === 'custom') { - target.style.animation = 'customAnim_' + clip.id + ' ' + duration + 's ' + (clip.easing || 'ease') + - ' ' + delay + 's ' + iterationCount; - if (!infinite) { - target.addEventListener('animationend', function handler() { - target.style.animation = ''; - target.removeEventListener('animationend', handler); - }); - } - } else { - var animClass = 'animate__' + clip.animate_name; - target.style.setProperty('--animate-duration', duration + 's'); - target.style.setProperty('--animate-delay', delay + 's'); - target.style.animationIterationCount = iterationCount; - target.classList.add('animate__animated', animClass); - if (!infinite) { - target.addEventListener('animationend', function handler() { - target.classList.remove('animate__animated', animClass); - target.removeEventListener('animationend', handler); - }); + targets.forEach(function(target) { + if (clip.kind === 'custom') { + target.style.animation = 'customAnim_' + clip.id + ' ' + duration + 's ' + (clip.easing || 'ease') + + ' ' + delay + 's ' + iterationCount; + if (!infinite) { + target.addEventListener('animationend', function handler() { + target.style.animation = ''; + target.removeEventListener('animationend', handler); + }); + } + } else { + var animClass = 'animate__' + clip.animate_name; + target.style.setProperty('--animate-duration', duration + 's'); + target.style.setProperty('--animate-delay', delay + 's'); + target.style.animationIterationCount = iterationCount; + target.classList.add('animate__animated', animClass); + if (!infinite) { + target.addEventListener('animationend', function handler() { + target.classList.remove('animate__animated', animClass); + target.removeEventListener('animationend', handler); + }); + } } + }); + } + + // Rassemble les clips propres à cet écran ET ceux de tout écran-MODÈLE + // utilisé par un de ses éléments (élément de jeu réutilisable posé sur cet + // écran, ex. "mail content") — récursif (un modèle peut lui-même utiliser + // un autre modèle), avec une garde anti-boucle et un dédoublonnage par + // écran (un même modèle posé plusieurs fois ne doit pas répéter ses + // clips). Sans ça, une animation posée directement dans l'éditeur d'un + // modèle ne se jouait jamais quand le modèle est utilisé ailleurs — seul + // l'écran-modèle affiché EN TANT QU'ÉCRAN pouvait la jouer. + function collectAnimationClips(screenId) { + var visited = {}; + var clips = []; + function visit(sid) { + if (visited[sid]) return; + visited[sid] = true; + clips = clips.concat((gameData.animations || {})[String(sid)] || []); + var screenData = screensData.find(function(s){ return String(s.id) === String(sid); }); + if (!screenData) return; + screenData.elements.forEach(function(el) { + if (!el.element_type_id) return; + var templateScreenId = (gameData.element_types || {})[String(el.element_type_id)]; + if (templateScreenId != null) visit(templateScreenId); + }); } + visit(screenId); + return clips; } function runAnimationTimeline(screenId) { @@ -200,7 +236,7 @@ // suite sur le même écran avant la fin de sa timeline. pendingAnimationTimers.forEach(function(t) { clearTimeout(t); }); pendingAnimationTimers = []; - var clips = (gameData.animations || {})[String(screenId)] || []; + var clips = collectAnimationClips(screenId); clips.forEach(function(clip) { var t = setTimeout(function() { applyAnimationClip(clip); }, (clip.start_time || 0) * 1000); pendingAnimationTimers.push(t); @@ -233,11 +269,20 @@ // fois, sur tous les éléments de tous les écrans (déjà tous présents dans // le DOM, seuls les écrans non actifs sont masqués) — chaque identifiant // d'élément est unique dans tout le jeu, donc pas d'ambiguïté possible. + // Cette recherche globale couvre aussi les écrans-MODÈLES (un élément de + // jeu réutilisable, ex. "mail content", posé sur une scène) : un + // déclencheur "Au clic"/"Au survol" posé dans l'éditeur du modèle, sur un + // de SES PROPRES enfants, est donc retrouvé même quand ce modèle est + // utilisé ailleurs — d'où le besoin de renvoyer aussi l'écran d'origine du + // nœud (screenIdKey), pas seulement le nœud : runFlowFrom() doit exécuter + // le graphe dans CET écran-là (celui du modèle), jamais dans celui + // réellement affiché, sans quoi le nœud trouvé n'existerait pas dans le + // graphe utilisé et l'action ne se déclencherait jamais. function findTriggerNode(elementId, event) { for (const screenIdKey in gameData.flows) { const flow = gameData.flows[screenIdKey]; const found = flow.nodes.find(n => n.node_type === 'trigger' && n.trigger_element_id === elementId && (n.trigger_event || 'clic') === event); - if (found) return found; + if (found) return { node: found, screenId: screenIdKey }; } return null; } @@ -290,7 +335,7 @@ const rowEl = ev.target.closest('.repeaterItem'); window.lastClickedRowId = rowEl ? parseInt(rowEl.dataset.rowId, 10) : null; window.lastClickedDefinitionId = elDiv.dataset.definitionId ? parseInt(elDiv.dataset.definitionId, 10) : null; - runFlowFrom(trigger.id); + runFlowFrom(trigger.node.id, trigger.screenId); }); } }); @@ -311,8 +356,8 @@ const leaveTrigger = findTriggerNode(elementId, 'fin_survol'); if (!enterTrigger && !leaveTrigger) return; elDiv.dataset.hoverTriggerBound = '1'; - if (enterTrigger) elDiv.addEventListener('mouseenter', () => runFlowFrom(enterTrigger.id)); - if (leaveTrigger) elDiv.addEventListener('mouseleave', () => runFlowFrom(leaveTrigger.id)); + if (enterTrigger) elDiv.addEventListener('mouseenter', () => runFlowFrom(enterTrigger.node.id, enterTrigger.screenId)); + if (leaveTrigger) elDiv.addEventListener('mouseleave', () => runFlowFrom(leaveTrigger.node.id, leaveTrigger.screenId)); }); } @@ -339,46 +384,124 @@ const screenData = screensData.find(s => s.id === window.currentScreenId); const screenDiv = document.querySelector('.playScreen[data-screen-id="' + window.currentScreenId + '"]'); if (!row || !screenData || !screenDiv) return; - screenData.elements.filter(function(e){ return !e.parent_id; }).forEach(function(el) { - const wrapper = screenDiv.querySelector('.playElement[data-el-id="' + el.id + '"] .playElementInner'); - if (!wrapper || !el.rendered_html || el.rendered_html.indexOf(PLACEHOLDER_OPEN) === -1) return; - // Avant de regénérer ce sous-arbre depuis le HTML D'ORIGINE (nécessaire - // pour ne pas laisser les valeurs d'un mail précédent affichées, voir - // plus haut), sauvegarde l'état visuel TRANSITOIRE posé par une action - // "Modifier un élément" (jamais enregistré en base — voir - // applyElementProperty) sur cet élément ou l'un de ses descendants, - // pour le réappliquer après. Sans ça, un panneau rendu visible par - // cette action (ex. "mail content") redevenait invisible à CHAQUE - // rafraîchissement de données, y compris pour un changement de donnée - // sans rapport avec lui — puisque wrapper.innerHTML = html ci-dessous - // recrée entièrement ses nœuds à partir de leur état PAR DÉFAUT. - const savedState = {}; - wrapper.querySelectorAll('[data-element-id]').forEach(function(node) { - savedState[node.dataset.elementId] = { - style: node.getAttribute('style') || '', - className: node.className, - dataset: Object.assign({}, node.dataset), - }; - }); + // Cible directement, à N'IMPORTE QUEL niveau d'imbrication, le ou les + // éléments qui portent vraiment un {{champ}} non résolu — et non plus + // seulement les éléments de PREMIER NIVEAU de l'écran. Avant ce + // correctif, un panneau de détail niché à 2-3 niveaux (ex. "mail + // content" posé dans un conteneur, lui-même dans un autre) forçait à + // regénérer TOUT son ancêtre de premier niveau — potentiellement + // l'écran entier (jauges, onglets, Répéteur compris) s'il n'y avait + // qu'un seul gros conteneur racine — d'où l'impression trompeuse d'un + // rechargement de page (flash à vide) à chaque clic sur une ligne. + // + // Parmi les éléments qui ont un {{champ}} non résolu, on ne garde que + // les plus SPÉCIFIQUES (les plus profonds) : si un élément ET l'un de + // ses DESCENDANTS sont tous deux candidats, seul le descendant est + // régénéré (rendered_html d'un conteneur inclut TOUJOURS celui de ses + // descendants, donc un conteneur "candidat" ne l'est souvent que parce + // qu'un descendant à lui l'est — le garder en plus regénérerait + // inutilement tout son sous-arbre, à l'exact opposé du but recherché). + // ATTENTION : la version précédente de ce filtre vérifiait l'inverse + // (exclure un élément si un ANCÊTRE est candidat) — comme un ancêtre + // est presque toujours "candidat" dès qu'un descendant l'est (son HTML + // le contient), ce filtre ne gardait en pratique QUE l'ancêtre racine + // de l'écran, annulant complètement l'effet recherché par ce correctif. + const byId = {}; + screenData.elements.forEach(function(e){ byId[e.id] = e; }); + function hasUnresolvedPlaceholder(e) { + return !!(e.rendered_html && e.rendered_html.indexOf(PLACEHOLDER_OPEN) !== -1); + } + function isDescendantOf(e, ancestorId) { + let p = e.parent_id ? byId[e.parent_id] : null; + while (p) { + if (p.id === ancestorId) return true; + p = p.parent_id ? byId[p.parent_id] : null; + } + return false; + } + const candidates = screenData.elements.filter(hasUnresolvedPlaceholder); + const targets = candidates.filter(function(e) { + return !candidates.some(function(other) { return other.id !== e.id && isDescendantOf(other, e.id); }); + }); - let html = el.rendered_html; - Object.keys(row).forEach(function(fieldName) { - if (fieldName === 'id') return; - const value = row[fieldName] == null ? '' : String(row[fieldName]); - const tmp = document.createElement('div'); - tmp.textContent = value; - html = html.split(PLACEHOLDER_OPEN + fieldName + PLACEHOLDER_CLOSE).join(tmp.innerHTML); - }); - wrapper.innerHTML = html; + // Résout {{champ}} EN PLACE dans le DOM déjà affiché, sans jamais + // détruire/recréer le moindre nœud — un peu comme un diff React ciblé, + // mais posé à la main puisqu'il n'y a pas de vrai DOM virtuel ici : + // - dans un nœud TEXTE : {{champ}} est coupé en 3 (texte avant, un + // qui portera la valeur, texte + // après) LA PREMIÈRE FOIS SEULEMENT ; les ouvertures suivantes se + // contentent de changer le textContent de ce span. + // - dans un ATTRIBUT (ex. href d'un lien "{{link_real_url}}") : le + // gabarit d'origine est mémorisé sur un data-bind-attr- lors du + // premier passage, pour pouvoir recalculer et réécrire l'attribut à + // chaque fois sans jamais avoir besoin de reconstruire le nœud. + // Avant ce correctif, tout le sous-arbre ciblé était détruit et recréé + // depuis zéro à chaque clic sur une ligne — ce qui causait un flash + // visuel (perte de tout état transitoire posé par "Modifier un + // élément", donnant l'impression trompeuse d'un rechargement de page). + // Avec la résolution en place, plus aucun nœud du sous-arbre ne bouge : + // rien à sauvegarder ni à restaurer. + const placeholderRe = new RegExp(PLACEHOLDER_OPEN + '([^{}]+)' + PLACEHOLDER_CLOSE, 'g'); - wrapper.querySelectorAll('[data-element-id]').forEach(function(node) { - const saved = savedState[node.dataset.elementId]; - if (!saved) return; - node.setAttribute('style', saved.style); - node.className = saved.className; - Object.assign(node.dataset, saved.dataset); + function resolveTextNodes(root) { + const walker = document.createTreeWalker(root, NodeFilter.SHOW_TEXT); + const toSplit = []; + let node; + while ((node = walker.nextNode())) { + if (node.nodeValue.indexOf(PLACEHOLDER_OPEN) !== -1) toSplit.push(node); + } + toSplit.forEach(function(textNode) { + const raw = textNode.nodeValue; + placeholderRe.lastIndex = 0; + const frag = document.createDocumentFragment(); + let lastIndex = 0, match; + while ((match = placeholderRe.exec(raw))) { + if (match.index > lastIndex) frag.appendChild(document.createTextNode(raw.slice(lastIndex, match.index))); + const span = document.createElement('span'); + span.dataset.bindField = match[1]; + span.textContent = row[match[1]] == null ? '' : String(row[match[1]]); + frag.appendChild(span); + lastIndex = match.index + match[0].length; + } + if (lastIndex < raw.length) frag.appendChild(document.createTextNode(raw.slice(lastIndex))); + textNode.replaceWith(frag); }); + } + + function updateTextSlots(root) { + root.querySelectorAll('[data-bind-field]').forEach(function(slot) { + const value = row[slot.dataset.bindField] == null ? '' : String(row[slot.dataset.bindField]); + if (slot.textContent !== value) slot.textContent = value; + }); + } + + function resolveAttributes(root) { + [root].concat(Array.from(root.querySelectorAll('*'))).forEach(function(node) { + Array.from(node.attributes).forEach(function(attr) { + if (attr.name.indexOf('data-bind-attr-') === 0) return; // gabarit déjà mémorisé, pas un attribut à lire + const bindKey = 'data-bind-attr-' + attr.name; + const template = node.hasAttribute(bindKey) ? node.getAttribute(bindKey) : (attr.value.indexOf(PLACEHOLDER_OPEN) !== -1 ? attr.value : null); + if (template === null) return; + if (!node.hasAttribute(bindKey)) node.setAttribute(bindKey, template); + placeholderRe.lastIndex = 0; + const resolved = template.replace(placeholderRe, function(_, fieldName) { + return row[fieldName] == null ? '' : String(row[fieldName]); + }); + if (attr.value !== resolved) node.setAttribute(attr.name, resolved); + }); + }); + } + + targets.forEach(function(el) { + const target = screenDiv.querySelector('[data-element-id="' + el.id + '"]'); + if (!target) return; + if (target.querySelector('[data-bind-field]')) { + updateTextSlots(target); + } else { + resolveTextNodes(target); + } + resolveAttributes(target); }); } @@ -399,21 +522,78 @@ screensData = gameData.screens; const screenData = screensData.find(s => s.id === window.currentScreenId); const screenDiv = document.querySelector('.playScreen[data-screen-id="' + window.currentScreenId + '"]'); - if (!screenData || !screenDiv) return; - screenData.elements.filter(function(e){ return !e.parent_id; }).forEach(function(el) { - if (!el.rendered_html) return; - // "repeaterItem" (liste de données), "jaugeBar" (2.1 — jauge liée - // à une donnée) et "visibilityGated" (condition de visibilité — - // voir render_element_html.py) sont les types d'éléments dont le - // contenu/l'affichage dépend de la donnée elle-même plutôt que - // d'une action "Modifier un élément" posée à la main dans le - // graphe — eux seuls doivent être régénérés après un changement - // de donnée ou de variable. - if (el.rendered_html.indexOf('repeaterItem') === -1 && el.rendered_html.indexOf('jaugeBar') === -1 && el.rendered_html.indexOf('visibilityGated') === -1) return; - const wrapper = screenDiv.querySelector('.playElement[data-el-id="' + el.id + '"] .playElementInner'); - if (wrapper) wrapper.innerHTML = el.rendered_html; - }); - applyOpenRowBindings(); + // Le bloc ci-dessous (régénération du contenu de l'écran courant) + // est sauté si l'écran n'est pas trouvé, mais bindClicks() et les + // autres réattachements juste après ne doivent JAMAIS dépendre de + // cette condition : un retour anticipé ici a déjà laissé, dans la + // pratique, un Répéteur fraîchement régénéré sans AUCUN gestionnaire + // de clic pour le reste de la partie (plus aucune ligne ne + // répondait, silencieusement, sans erreur) — ces fonctions sont sans + // risque à rappeler même quand rien n'a changé. + if (screenData && screenDiv) { + // try/catch : une erreur inattendue pendant la régénération (ex. + // structure de données imprévue) ne doit jamais empêcher les + // réattachements ci-dessous de s'exécuter — sinon le Répéteur (ou + // tout autre élément cliquable) reste silencieusement inerte pour + // le reste de la partie, sans le moindre message d'erreur visible. + try { + // "repeaterItem" (liste de données), "jaugeBar" (2.1 — jauge liée + // à une donnée) et "visibilityGated" (condition de visibilité — + // voir render_element_html.py) sont les types d'éléments dont le + // contenu/l'affichage dépend de la donnée elle-même plutôt que + // d'une action "Modifier un élément" posée à la main dans le + // graphe — eux seuls doivent être régénérés après un changement + // de donnée ou de variable. + // + // On cible ces éléments à N'IMPORTE QUEL niveau d'imbrication + // (pas seulement le premier niveau de l'écran), via leur + // data-element-id — disponible sur chaque élément rendu, pas + // seulement les éléments de premier niveau. Avant ce correctif, + // un Répéteur niché dans un conteneur (comme ici) n'était JAMAIS + // du premier niveau, donc c'est son ANCÊTRE de premier niveau + // qui portait le marqueur (le Répéteur imbriqué à l'intérieur) + // et se retrouvait régénéré en entier à sa place — potentiellement + // l'écran entier (jauges, onglets... compris) s'il n'y avait + // qu'un seul gros conteneur racine. + // + // Parmi les éléments marqués, on ne garde que les plus + // SPÉCIFIQUES (les plus profonds) : un ancêtre est presque + // toujours "marqué" lui aussi dès qu'un descendant l'est, puisque + // son rendered_html contient celui de ses descendants — le + // garder EN PLUS régénérerait tout son sous-arbre pour rien. + // (Une version antérieure de ce filtre vérifiait l'inverse — + // exclure un élément si un ANCÊTRE est marqué — ce qui ne + // gardait en pratique que l'ancêtre racine de l'écran et + // annulait complètement l'effet recherché.) + const byId = {}; + screenData.elements.forEach(function(e){ byId[e.id] = e; }); + function hasMarker(e) { + return !!(e.rendered_html && (e.rendered_html.indexOf('repeaterItem') !== -1 || e.rendered_html.indexOf('jaugeBar') !== -1 || e.rendered_html.indexOf('visibilityGated') !== -1)); + } + function isDescendantOf(e, ancestorId) { + let p = e.parent_id ? byId[e.parent_id] : null; + while (p) { + if (p.id === ancestorId) return true; + p = p.parent_id ? byId[p.parent_id] : null; + } + return false; + } + const markedCandidates = screenData.elements.filter(hasMarker); + markedCandidates.filter(function(e) { + return !markedCandidates.some(function(other) { return other.id !== e.id && isDescendantOf(other, e.id); }); + }).forEach(function(el) { + const target = screenDiv.querySelector('[data-element-id="' + el.id + '"]'); + if (!target) return; + const container = document.createElement('div'); + container.innerHTML = el.rendered_html; + const newNode = container.firstElementChild; + if (newNode) target.replaceWith(newNode); + }); + applyOpenRowBindings(); + } catch (err) { + console.error('refreshRuntimeData: erreur pendant la régénération de l\'écran', err); + } + } bindClicks(); bindHoverTexts(); bindHoverTriggers(); @@ -533,6 +713,18 @@ } showScreen(node.target_screen_id); applyOpenRowBindings(); + // Si l'élément "ouvert" (ou un ancêtre commun avec lui) contient AUSSI + // un Répéteur — ex. un panneau de détail posé à côté d'une liste dans + // le même conteneur — applyOpenRowBindings() vient de régénérer son + // sous-arbre, lignes du Répéteur comprises, avec des nœuds tout neufs + // SANS AUCUN écouteur de clic (voir bindClicks : le garde-fou anti- + // doublon repose sur elDiv.dataset.clickBound, absent sur un nœud + // fraîchement créé). Sans ce rappel, un clic sur "ouvrir la ligne + // cliquée" combiné à un Répéteur sur le même écran rendait TOUTES les + // lignes muettes dès le clic suivant, sans la moindre erreur. + bindClicks(); + bindHoverTexts(); + bindHoverTriggers(); return Promise.resolve(); } else if (node.action_type === 'modifier_element' && node.target_element_id) { const targetEl = document.querySelector('[data-element-id="' + node.target_element_id + '"]'); @@ -672,8 +864,15 @@ // seulement le premier. Chaque nœud Action produit son effet puis // continue sur son (ou ses) fil de sortie ; chaque nœud Condition évalue // une donnée du jeu et suit le(s) fil(s) "Vrai" ou "Faux" en conséquence. - function runFlowFrom(nodeId) { - const flow = gameData.flows[String(window.currentScreenId)]; + // flowScreenId : l'écran PROPRIÉTAIRE du graphe à exécuter — par défaut + // l'écran affiché, mais peut être un écran-MODÈLE (voir findTriggerNode) + // quand le nœud de départ vient d'un déclencheur posé dans l'éditeur d'un + // élément de jeu réutilisable, sur un de ses propres enfants. Sans ce + // paramètre, un tel déclencheur était bien retrouvé (findTriggerNode + // cherche dans tous les écrans) mais son graphe n'existait jamais dans + // l'écran réellement affiché : rien ne se déclenchait, silencieusement. + function runFlowFrom(nodeId, flowScreenId) { + const flow = gameData.flows[String(flowScreenId != null ? flowScreenId : window.currentScreenId)]; if (!flow) return; const nodesById = {}; flow.nodes.forEach(n => { nodesById[n.id] = n; }); diff --git a/templates/screen_edit.html b/templates/screen_edit.html index 6d4102a7..25e9c038 100644 --- a/templates/screen_edit.html +++ b/templates/screen_edit.html @@ -6,10 +6,6 @@ {% endblock %} - -{% block breadcrumb_wrap %}{% endblock %} {% block content %} {% if screen.is_template %}

    @@ -145,7 +141,7 @@ {% endfor %}

    {% else %} -

    Pas encore d'élément de jeu personnalisé — crées-en un.

    +

    Pas encore d'élément de jeu personnalisé — crées-en un.

    {% endif %}
    @@ -160,20 +156,23 @@ + switchBuilderTab() plus bas). Présents aussi sur un écran-modèle + (élément de jeu réutilisable, ex. "mail content") : ses enfants + gardent désormais TOUJOURS le même id, à chaque exemplaire posé sur + n'importe quelle scène (rechargés en direct depuis ce modèle à + chaque affichage, voir _render_element_type_children côté serveur — + l'ancien mécanisme qui les copiait avec un nouvel id à chaque pose, + instantiate_template_tree, a été retiré). Un déclencheur/une + animation posé ici, sur un enfant du modèle, s'exécute donc bien + partout où ce modèle est utilisé (voir findTriggerNode/ + collectAnimationClips dans templates/play.html). -->
    {% set current_aspect = screen.preview_aspect or '9/16' %} - {% if not screen.is_template %}
    - {% endif %}
    @@ -190,7 +189,6 @@

    Glisse pour déplacer, tire le coin pour redimensionner, clique pour sélectionner.

    - {% if not screen.is_template %}
    @@ -222,7 +220,7 @@
    - {% endif %}
    + {% if not global_variables %} -

    Aucune variable globale pour l'instant — crées-en une ↗

    +

    Aucune variable globale pour l'instant — crées-en une ↗

    {% endif %}
    @@ -811,7 +812,7 @@
    {% if c.key == 'visibilite_cond_variable' %} -

    Pas la variable qu'il te faut ? Gérer les variables du jeu ↗

    +

    Pas la variable qu'il te faut ? Gérer les variables du jeu ↗

    {% endif %} {% endif %} {% endif %} @@ -1717,8 +1718,10 @@ var _FILTER_REF_RE = new RegExp('^\\{\\{\\s*([^.{}]+)\\.([^.{}]+)\\s*\\}\\}$'); // Une paire d'accolades doublées entourant "$nom_variable" (voir // _VAR_REF_PATTERN côté Python, filter_repeater_rows.py) référence une // variable globale — le "$" la distingue sans ambiguïté de _FILTER_REF_RE -// ci-dessus, qui attend toujours un point ("Objet.champ"). -var _VAR_REF_RE = new RegExp('^\\{\\{\\s*\\$([^.{}]+)\\s*\\}\\}$'); +// ci-dessus, qui attend toujours un point ("Objet.champ"). Groupe 2 : un +// chemin optionnel ".champ"/"[index]" (chaînable) à l'intérieur de la +// variable, utile pour une variable "objet"/"tableau". +var _VAR_REF_RE = new RegExp('^\\{\\{\\s*\\$([^.{}\\[\\]]+)((?:\\.[^.{}\\[\\]]+|\\[\\d+\\])*)\\s*\\}\\}$'); function _filterValueDefinitionId(objName) { var opt = document.querySelector('.filterValueObjSel option[value="' + CSS.escape(objName) + '"]'); @@ -1742,8 +1745,10 @@ function updateFilterValueFromDynamic(wrap) { function updateFilterValueFromVariable(wrap) { var varSel = wrap.querySelector('.filterValueVarSel'); + var pathInput = wrap.querySelector('.filterValueVarPath'); var input = wrap.querySelector('.filterValueFixe input'); - input.value = varSel.value ? (_FILTER_REF_OPEN + '$' + varSel.value + _FILTER_REF_CLOSE) : ''; + var path = pathInput ? pathInput.value.trim() : ''; + input.value = varSel.value ? (_FILTER_REF_OPEN + '$' + varSel.value + path + _FILTER_REF_CLOSE) : ''; input.dispatchEvent(new Event('input', { bubbles: true })); } @@ -1779,6 +1784,7 @@ function initFilterValuePickers() { var objSel = wrap.querySelector('.filterValueObjSel'); var fieldSel = wrap.querySelector('.filterValueFieldSel'); var varSel = wrap.querySelector('.filterValueVarSel'); + var varPathInput = wrap.querySelector('.filterValueVarPath'); var current = (wrap.dataset.currentValue || '').trim(); var varMatch = _VAR_REF_RE.exec(current); var objMatch = _FILTER_REF_RE.exec(current); @@ -1787,6 +1793,7 @@ function initFilterValuePickers() { wrap.querySelector('.filterValueFixe').style.display = 'none'; wrap.querySelector('.filterValueVariable').style.display = 'block'; varSel.value = varMatch[1].trim(); + if (varPathInput) varPathInput.value = (varMatch[2] || '').trim(); } else if (objMatch) { modeSel.value = 'dynamique'; wrap.querySelector('.filterValueFixe').style.display = 'none'; diff --git a/templates/screens_list.html b/templates/screens_list.html deleted file mode 100644 index ad429be1..00000000 --- a/templates/screens_list.html +++ /dev/null @@ -1,62 +0,0 @@ -{% extends "base.html" %} -{% block title %}Écrans — {{ game.name }}{% endblock %} -{% block content_class %}content-wide content-objectEdit{% endblock %} -{% block body_class %}objectEditBody{% endblock %} -{% block breadcrumb %} -
  • {{ game.name }}
    • -
  • Écrans
    • -{% endblock %} -{% block content %} -

    Écrans du jeu

    -{% if request.args.get('blocked_screen') %} -
    - ⚠️ Suppression impossible : au moins un autre écran a un bouton qui cible précisément celui-ci. Retire d'abord cette action. -
    -{% endif %} - -
    -
    -

    Créer

    -
    -
    - - -
    -
    -
    -
    - -
    -

    Enchaînement des écrans

    - {% if screen_list %} -
    - - - - {% for s in screen_list %} - - - - - - {% endfor %} - -
    #Nom
    {{ loop.index }} - {{ s.name }} - {% if loop.first %} ★ Départ{% endif %} - -
    -
    - ✏️ -
    - -
    -
    -
    - {% else %} -

    Aucun écran pour l'instant — crée le premier à gauche.

    - {% endif %} -
    -
    -{% endblock %} diff --git a/tests/test_screens_and_elements.py b/tests/test_screens_and_elements.py index e61e4816..42b463fd 100644 --- a/tests/test_screens_and_elements.py +++ b/tests/test_screens_and_elements.py @@ -100,15 +100,17 @@ def test_element_type_template_case_a_and_case_b(client, game, mail_object): follow_redirects=False, ) assert resp.status_code == 302 - html = client.get(f"/game/{slug}/element-types").data.decode() + html = client.get(f"/game/{slug}").data.decode() m = re.search(r"/screens/(\d+)/edit", html) assert m, "le lien vers l'écran-modèle doit apparaître dans le catalogue" template_screen_id = int(m.group(1)) # L'écran-modèle est un écran caché (is_template) : absent de la liste - # normale des écrans jouables. - screens_html = client.get(f"/game/{slug}/screens").data.decode() - assert "CarteMail" not in screens_html or "Modèle" not in screens_html + # normale des écrans jouables (onglet "Écrans du jeu" du tableau de + # bord) — son nom complet ("Modèle : CarteMail") ne doit donc apparaître + # nulle part sur cette page, contrairement à "CarteMail" seul (le nom de + # l'élément de jeu lui-même, affiché dans l'onglet "Éléments de jeu"). + assert "Modèle : CarteMail" not in html # Construit le contenu du modèle : un titre lié au champ "sujet". resp = client.post( @@ -266,7 +268,7 @@ def test_screen_editor_property_panels_closed_by_default(client, game): def test_delete_element_type_removes_hidden_template_screen(client, game): resp = client.post(f"/game/{game}/element-types", data={"name": "ToDelete", "icon": "🧩"}, follow_redirects=False) - html = client.get(f"/game/{game}/element-types").data.decode() + html = client.get(f"/game/{game}").data.decode() m = re.search(r"/screens/(\d+)/edit", html) template_screen_id = int(m.group(1)) element_type_id = int(re.search(r"element-types/(\d+)/delete", html).group(1)) diff --git a/tests/test_variable_object_array.py b/tests/test_variable_object_array.py new file mode 100644 index 00000000..0a134f4c --- /dev/null +++ b/tests/test_variable_object_array.py @@ -0,0 +1,121 @@ +"""Variables globales de type "objet"/"tableau" (db/constants.py) : la +valeur est du JSON, lisible via un chemin ".champ"/"[index]" chaînable +(_resolve_variable_path, screens/rendering/filter_repeater_rows.py) — +aussi bien dans un filtre de Répéteur ({{$var.chemin}}) que dans une +Condition de visibilité en mode "variable" (contrôle séparé +"visibilite_cond_variable_chemin").""" +import json +import re + +import db + + +def _create_screen(client, slug, name="Accueil"): + resp = client.post(f"/game/{slug}/screens/new", data={"name": name}, follow_redirects=False) + return int(re.search(r"/screens/(\d+)/edit", resp.headers["Location"]).group(1)) + + +def _add_element(client, slug, screen_id, widget="texte"): + resp = client.post(f"/game/{slug}/screens/{screen_id}/elements/add", data={"widget": widget}, follow_redirects=False) + return int(re.search(r"selected=(\d+)", resp.headers["Location"]).group(1)) + + +def _rendered_html(client, slug): + payload = client.get(f"/game/{slug}/runtime-payload").get_json() + return payload["screens"][0]["elements"][0]["rendered_html"] + + +def _create_niveau_object(client, slug): + resp = client.post(f"/game/{slug}/objects/new", data={ + "object_name": "Niveau", "field_name[]": ["numero"], "field_type[]": ["nombre_entier"], + "field_relation[]": [""], "field_required[]": ["0"], "field_min[]": [""], "field_max[]": [""], + }, follow_redirects=False) + def_id = int(resp.headers["Location"].rstrip("/").split("/")[-1]) + client.post(f"/game/{slug}/objects/{def_id}/data/new", data={"numero": "12"}) + client.post(f"/game/{slug}/objects/{def_id}/data/new", data={"numero": "99"}) + return def_id + + +def test_create_and_update_validate_json_with_safe_fallback(client, game): + client.post(f"/game/{game}/variables/new", data={"name": "perso", "type": "objet", "default_value": "not json"}) + variable = db.get_global_variable(game, "perso") + assert variable["value"] == "{}" # JSON invalide -> défaut sûr, pas la chaîne brute + + client.post(f"/game/{game}/variables/new", data={"name": "scores", "type": "tableau", "default_value": ""}) + variable = db.get_global_variable(game, "scores") + assert variable["value"] == "[]" # vide -> défaut sûr propre au type + + db.update_global_variable(game, db.get_global_variable(game, "perso")["id"], "objet", json.dumps({"nom": "Alice"})) + assert json.loads(db.get_global_variable(game, "perso")["value"]) == {"nom": "Alice"} + + +def test_repeater_filter_reads_a_field_inside_an_object_variable(client, game): + client.post(f"/game/{game}/variables/new", data={ + "name": "perso", "type": "objet", + "default_value": json.dumps({"nom": "Alice", "arme": {"degats": 12}}), + }) + def_id = _create_niveau_object(client, game) + screen_id = _create_screen(client, game) + resp = client.post(f"/game/{game}/screens/{screen_id}/elements/add", data={"widget": "repeteur_donnees"}, follow_redirects=False) + rep_id = int(re.search(r"selected=(\d+)", resp.headers["Location"]).group(1)) + + # {{$perso.arme.degats}} (chemin chaîné) doit résoudre à 12, et donc ne + # garder que la ligne "numero = 12". + client.post(f"/game/{game}/elements/{rep_id}/save", data={ + "ctrl_definition_id": str(def_id), + "ctrl_filtre_champ": "numero", + "ctrl_filtre_operateur": "egal", + "ctrl_filtre_valeur": "{{$perso.arme.degats}}", + }) + html = _rendered_html(client, game) + assert html.count("repeaterItem") == 1 + assert 'data-row-id="1"' in html + assert 'data-row-id="2"' not in html + + +def test_visibility_condition_reads_an_array_element_by_index(client, game): + client.post(f"/game/{game}/variables/new", data={ + "name": "scores", "type": "tableau", "default_value": json.dumps([10, 20, 30]), + }) + screen_id = _create_screen(client, game) + element_id = _add_element(client, game, screen_id) + + # scores[1] == 20 -> visible. + client.post(f"/game/{game}/elements/{element_id}/save", data={ + "ctrl_visibilite_cond_mode": "variable", + "ctrl_visibilite_cond_variable": "scores", + "ctrl_visibilite_cond_variable_chemin": "[1]", + "ctrl_visibilite_cond_operateur": "egal", + "ctrl_visibilite_cond_valeur": "20", + }) + html = _rendered_html(client, game) + assert "display:none" not in html + + # scores[0] == 10, pas 20 -> masqué. + client.post(f"/game/{game}/elements/{element_id}/save", data={ + "ctrl_visibilite_cond_mode": "variable", + "ctrl_visibilite_cond_variable": "scores", + "ctrl_visibilite_cond_variable_chemin": "[0]", + "ctrl_visibilite_cond_operateur": "egal", + "ctrl_visibilite_cond_valeur": "20", + }) + html = _rendered_html(client, game) + assert "display:none" in html + + +def test_invalid_or_missing_path_never_crashes(client, game): + client.post(f"/game/{game}/variables/new", data={ + "name": "perso", "type": "objet", "default_value": json.dumps({"nom": "Alice"}), + }) + screen_id = _create_screen(client, game) + element_id = _add_element(client, game, screen_id) + client.post(f"/game/{game}/elements/{element_id}/save", data={ + "ctrl_visibilite_cond_mode": "variable", + "ctrl_visibilite_cond_variable": "perso", + "ctrl_visibilite_cond_variable_chemin": ".champ_inexistant", + "ctrl_visibilite_cond_operateur": "egal", + "ctrl_visibilite_cond_valeur": "peu importe", + }) + resp = client.get(f"/game/{game}/runtime-payload") + assert resp.status_code == 200 + assert "display:none" in resp.get_json()["screens"][0]["elements"][0]["rendered_html"]