williamandClaude Sonnet 5
236d6b4b46
Fil directeur du plan : chaque variable globale et chaque objet de données gagne un player_id (sentinelle PLAYER_SHARED='__shared__' par défaut partout — l'aperçu créateur et tous les tests existants continuent de fonctionner À L'IDENTIQUE, aucune signature appelée sans argument explicite ne change de comportement), plus un réglage per_player choisi une fois à la création : - per_player=1 (par défaut) : chaque joueur a sa propre valeur/ses propres lignes. - per_player=0 : valeur/lignes partagées par tous les joueurs (ex. un compteur de visiteurs global, un catalogue commun). db/global_vars/ : _global_variables passe de UNIQUE(name) à UNIQUE(name, player_id) — SQLite ne permet pas de modifier une contrainte UNIQUE via ALTER TABLE, reconstruction de la table détectée et faite une seule fois (ensure_global_vars_schema.py) pour les jeux créés avant cette phase. La ligne "modèle" (player_id=PLAYER_SHARED, créée par le créateur) porte le réglage per_player et sert de valeur PAR DÉFAUT : la première écriture d'un joueur sur une variable per_player crée paresseusement SA propre ligne (copiée depuis le modèle) ; une lecture sans ligne encore écrite retombe sur le modèle (nouveau resolve_player_key.py). list_global_variables() (tableau de bord) ne montre toujours que les lignes modèles ; nouveau list_global_variables_for_player() expose la valeur EFFECTIVE d'un joueur au runtime (full_game_payload.py). db/definitions/ + db/rows/ : chaque table d'objet généré (create_definition.py) gagne une colonne player_id (ADD COLUMN simple, pas de contrainte UNIQUE en jeu ici) ; _definitions gagne per_player. Contrairement aux variables, PAS de repli sur une ligne "modèle" pour les lignes d'un objet per_player — une LISTE n'a pas de valeur par défaut unique à copier comme un scalaire, un nouvel objet per_player démarre VIDE pour chaque joueur (nouveau resolve_row_player_key.py). get_row/update_row/update_row_field/delete_row filtrent aussi par player_id (pas seulement id) : garde-fou contre un row_id d'un AUTRE joueur, nécessaire dès qu'un objet per_player sera exposé sur la future route publique /jouer/<slug>. Migration : nouveau ensure_player_id_column(slug, table_name), appelé avant toute requête sur une table d'objet créée avant cette phase. Chaîne de rendu (screens/elements/list_elements.py -> screens/rendering/render_element_html.py -> render_repeater.py/ render_jauge.py/resolve_bound_row.py/visibility_condition.py/ filter_repeater_rows.py) : player_id transite dans le ctx déjà utilisé partout pour "champ en cours" (ctx["_forge_player_id"], même patron que ctx["_forge_play_mode"], posé une seule fois par list_elements quand enforce_visibility=True) — pas de nouveau paramètre positionnel à threader dans chaque fonction, juste une clé de plus dans un mécanisme déjà en place. Nouveau tests/test_player_state.py : verrouille à la fois le nouveau comportement (deux joueurs => valeurs/lignes indépendantes ; per_player=0 => partagé ; nouveau joueur => valeur par défaut pour une variable, liste VIDE pour un objet ; get_row ne fuite jamais vers un autre joueur) et la non-régression de l'aperçu créateur (comportement historique inchangé). Vérifié : 232 tests passent (9 nouveaux). Reste à faire (prochains commits) : route publique /jouer/<slug>, identité visiteur (cookie), bascule "Publier en ligne" dans le tableau de bord. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
70 lines
3.7 KiB
Python
70 lines
3.7 KiB
Python
import db
|
|
|
|
from .filter_repeater_rows import _resolve_filter_value, _resolve_variable_path, _compare, _field_column
|
|
|
|
|
|
def is_element_visible(slug, attrs, ctx=None):
|
|
"""Condition de visibilité (panneau de propriétés, tout widget — voir
|
|
VISIBILITY_CONDITION_CONTROLS) : masque un élément selon soit une
|
|
VARIABLE GLOBALE (une seule valeur pour tout le jeu, voir
|
|
db/global_vars/), soit le champ d'un objet de données. Absente de
|
|
réglage (_visibilite_cond_mode vide) => toujours visible, comportement
|
|
inchangé pour tous les éléments déjà créés. Réutilise
|
|
_compare/_resolve_filter_value/_field_column de filter_repeater_rows.py
|
|
pour rester cohérent avec les mêmes opérateurs et la même syntaxe
|
|
"{{Objet.champ}}" pour une valeur de comparaison dynamique.
|
|
|
|
`ctx` porte les {{champ}} de la ligne en cours (voir render_repeater.py)
|
|
quand cet élément est descendant d'un Répéteur de données — en mode
|
|
"objet", si le champ réglé s'y trouve déjà, sa valeur est utilisée telle
|
|
quelle plutôt que d'aller chercher la ligne la plus récente de l'objet
|
|
en base : sans ça, TOUS les exemplaires d'un même modèle de ligne (ex.
|
|
une "Mail card" répétée pour chaque e-mail) évalueraient la même ligne
|
|
(la plus récente de l'objet) au lieu de chacun la sienne, et
|
|
n'afficheraient donc jamais rien de cohérent. Hors Répéteur (mode
|
|
"objet" sur un élément posé directement sur l'écran), le champ n'est
|
|
pas dans ctx et on retombe sur la convention "1 seule ligne = état de
|
|
partie" (déjà utilisée par la Jauge/Donnée liée) : la ligne la plus
|
|
récente de l'objet choisi."""
|
|
player_id = (ctx or {}).get("_forge_player_id", db.PLAYER_SHARED)
|
|
mode = attrs.get("_visibilite_cond_mode")
|
|
if mode not in ("variable", "objet"):
|
|
return True
|
|
operator = attrs.get("_visibilite_cond_operateur") or "egal"
|
|
expected = _resolve_filter_value(slug, attrs.get("_visibilite_cond_valeur") or "", player_id)
|
|
|
|
if mode == "variable":
|
|
var_name = attrs.get("_visibilite_cond_variable")
|
|
if not var_name:
|
|
return True
|
|
variable = db.get_global_variable(slug, var_name, player_id)
|
|
if not variable:
|
|
return True # variable supprimée depuis : ne bloque jamais l'affichage
|
|
value = variable.get("value")
|
|
# "Chemin dans la variable" (voir visibility_condition_controls.py)
|
|
# : nécessaire pour comparer un CHAMP d'une variable "objet" ou un
|
|
# ÉLÉMENT d'une variable "tableau", pas seulement la variable
|
|
# entière — même syntaxe ".champ"/"[index]" que {{$var.chemin}}.
|
|
path = attrs.get("_visibilite_cond_variable_chemin")
|
|
if path:
|
|
value = _resolve_variable_path(value, path)
|
|
return _compare(value, operator, expected, variable.get("type") or "texte")
|
|
|
|
definition_id = attrs.get("_visibilite_cond_definition_id")
|
|
field_name = attrs.get("_visibilite_cond_champ")
|
|
if not definition_id or not field_name:
|
|
return True
|
|
definition = db.get_definition(slug, int(definition_id))
|
|
if not definition:
|
|
return True
|
|
field_type = next((f["type"] for f in definition["fields"] if f["name"] == field_name), "texte")
|
|
|
|
if ctx and field_name in ctx:
|
|
return _compare(ctx.get(field_name), operator, expected, field_type)
|
|
|
|
rows = db.list_rows(slug, definition, player_id)
|
|
if not rows:
|
|
return True # objet sans donnée pour l'instant : ne bloque pas
|
|
row = rows[0] # convention "état de partie" : une seule ligne, voir data_definition_options.py
|
|
return _compare(row.get(_field_column(definition, field_name)), operator, expected, field_type)
|