williamandClaude Sonnet 5
c57420c8c9
Config strictement stricte partout (ruff, mypy --strict, bandit, vulture, import-linter, eslint, stylelint), aucune regle desactivee "pour ne pas casser le build" - l'existant a ete corrige pour la satisfaire plutot que l'inverse. Hooks pre-commit locaux (language: system) bloquants. - Typage mypy --strict propage a tout le moteur (db, screens, auth, core, ai, routes, puis publish/scripts/tests/app.py/build_css.py). - Securite : fuite de handle fichier Windows corrigee dans l'export SCORM (routes/publish/export_scorm.py), CSRF/RNG non-crypto/xAPI documentes (# nosec, # NOSONAR justifies), nouveau db.json_for_script() (echappe "</script>" dans le JSON embarque en <script>, 25 sites). - Architecture : imports circulaires/F811 nettoyes, contrats import-linter respectes, code mort retire (vulture). - Accessibilite : 69 champs de formulaire sans label correctement associe corriges (for/id ou aria-label) sur 11 templates. - ESLint/Stylelint : lot mecanique JS/CSS, regles ajustees puis appliquees (aucune desactivee sans verification individuelle). - Tests : isolation du compte admin partage (nettoyage ponctuel + fixture de teardown automatique en filet de securite), suite complete verte (591 tests Python, 241 tests JS). - SonarQube Community Build self-heberge (Docker + PostgreSQL) : rapport complet analyse point par point, faux positifs documentes. - .gitattributes ajoute (LF force) : core.autocrlf=true sur cette machine faisait echouer ESLint (linebreak-style) via un bug connu de git (checkout "en place" qui ignore l'eol force sur un fichier deja present sur disque - contourne en supprimant puis recreant chaque fichier suivi). djLint (H021, styles inline) volontairement saute pour ce commit - backlog assume, deja documente, traite dans un lot separe. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
63 lines
2.7 KiB
Python
63 lines
2.7 KiB
Python
from typing import Any
|
|
|
|
import db
|
|
|
|
from .compute_operation import compute_new_value
|
|
|
|
|
|
def apply_data_action(slug: str, action: dict[str, Any], player_id: str = db.PLAYER_SHARED) -> bool:
|
|
"""Exécute au moment du clic (mode jouable) une action "modifier_donnee" :
|
|
lit l'objet/la ligne/le champ visés dans l'action, calcule la nouvelle
|
|
valeur selon l'opération choisie, et l'enregistre — un vrai UPDATE SQL,
|
|
limité à cette seule colonne (les autres champs de la ligne ne sont pas
|
|
touchés). `player_id` (état par joueur, Phase 1) : la ligne visée doit
|
|
appartenir à CE joueur pour un objet per_player (voir get_row.py/
|
|
update_row_field.py — garde-fou contre un row_id d'un autre joueur)."""
|
|
definition_id = action.get("target_definition_id")
|
|
row_id = action.get("target_row_id")
|
|
field_name = action.get("target_field")
|
|
operation = action.get("data_operation")
|
|
raw_value = action.get("data_value")
|
|
if not (definition_id and row_id and field_name and operation):
|
|
return False
|
|
definition = db.get_definition(slug, definition_id)
|
|
if not definition:
|
|
return False
|
|
field_def = next((f for f in definition["fields"] if f["name"] == field_name), None)
|
|
if not field_def:
|
|
return False
|
|
row = db.get_row(slug, definition, row_id, player_id)
|
|
if not row:
|
|
return False
|
|
col = db.slugify(field_def["name"]).replace("-", "_")
|
|
if field_def["type"] == "relation":
|
|
col = col + "_id"
|
|
current = row.get(col)
|
|
|
|
try:
|
|
new_value = compute_new_value(operation, current, raw_value, field_def["type"] == "nombre_decimal")
|
|
except ValueError:
|
|
return False
|
|
|
|
new_value = _clamp_to_field_bounds(new_value, field_def)
|
|
db.update_row_field(slug, definition, row_id, field_def, new_value, player_id)
|
|
return True
|
|
|
|
|
|
def _clamp_to_field_bounds(value: Any, field_def: dict[str, Any]) -> Any:
|
|
"""2.2 — bornage automatique : ramène la valeur dans [min_value,
|
|
max_value] si l'un ou l'autre est réglé sur ce champ, pour qu'une
|
|
jauge (voir render_jauge.py) ne puisse jamais dépasser 100 % ni
|
|
descendre sous 0 % (ou toute autre borne définie), quel que soit le
|
|
nombre d'actions "Augmenter de..."/"Diminuer de..." posées dans le
|
|
graphe de logique. Ne s'applique qu'aux champs numériques — un champ
|
|
non numérique n'a pas de min_value/max_value (voir create_definition)."""
|
|
if field_def["type"] not in ("nombre_entier", "nombre_decimal"):
|
|
return value
|
|
min_v, max_v = field_def.get("min_value"), field_def.get("max_value")
|
|
if min_v is not None and value < min_v:
|
|
value = min_v
|
|
if max_v is not None and value > max_v:
|
|
value = max_v
|
|
return value if field_def["type"] == "nombre_decimal" else int(value)
|