williamandClaude Sonnet 5
ecd483f352
Build and deploy / test-python (push) Successful in 7m48s
Build and deploy / test-js (push) Successful in 52s
Build and deploy / lint-python (push) Successful in 5m44s
Build and deploy / lint-js (push) Failing after 1m52s
Build and deploy / build-and-push (push) Skipped
Build and deploy / deploy (push) Skipped
Build and deploy / sonarqube (push) Failing after 5m5s
Retour utilisateur : "il faut implementer un systeme de page". Un support est desormais compose de PLUSIEURS pages (_document_pages), chacune un document independant affiche seul sur le canevas -- chaque element appartient a exactement une page via page_id (document_engine/ elements/*, routes/document/document_element_add.py/document_render.py revalident desormais un page_id explicite). Migration automatique et silencieuse pour les supports crees avant cette fonctionnalite (db/supports/ensure_document_pages_schema.py, meme convention que les ensure_X_schema.py existants) : leurs elements deviennent tous les enfants d'une "Page 1" creee a la volee, aucune perte de contenu. Nouveau paquet document_engine/pages/ (add/list/get/rename/move/delete) et 4 routes dediees (routes/document/document_page_*.py) -- supprimer la DERNIERE page restante est refuse (garde-fou pose a la route, meme decoupage que routes/game/screens/screen_delete.py cote jeu, jamais dans la fonction bas niveau). Cote editeur : une bande d'ONGLETS au-dessus du canevas (jamais un panneau lateral, choix explicite de l'utilisateur) -- clic pour changer de page, double-clic pour renommer (contenteditable), glisser pour reordonner, "+" pour ajouter, "x" pour supprimer. Changer de page vide la pile Annuler/Retablir (une commande empilee sur une autre page n'a plus de sens). Mode Apercu : navigation Page precedente/suivante avec indicateur "Page X / N" (choix explicite : page par page, pas de defilement continu), jamais affichee s'il n'y a qu'une seule page. Verifie : suite pytest complete (702 tests, dont 14 nouveaux pour les routes de pages), simulation DOM reelle (jsdom, 25 assertions couvrant tout le cycle de vie cote client -- creation/bascule/renommage/ reordonnancement/suppression de page, portee correcte des elements par page, pile Annuler/Retablir videe au changement de page, pilule de navigation en Apercu), et un test de fumee HTTP reel contre le serveur de dev en marche (creation/ajout d'element/rendu/renommage/suppression d'une page, refus de supprimer la derniere page, page inconnue -> 404). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
57 lines
2.6 KiB
Markdown
57 lines
2.6 KiB
Markdown
# document_engine/pages/
|
|
|
|
CRUD des pages d'un support de formation (`_document_pages`, voir
|
|
`db/supports/create_support.py`) — retour utilisateur du 21/09/2026:
|
|
"il faut implémenter un système de page". Un support est désormais
|
|
composé de plusieurs pages, chacune portant son propre flux d'éléments
|
|
(voir `document_engine/elements/`, filtré par `page_id`). Un support a
|
|
TOUJOURS au moins une page (`create_support` en crée une par défaut,
|
|
`ensure_document_pages_schema` en garantit une pour les supports plus
|
|
anciens) — la garde "jamais supprimer la dernière page" est un
|
|
garde-fou métier posé par l'appelant (voir
|
|
`routes/document/document_page_delete.py`), pas une contrainte de ce
|
|
paquet.
|
|
|
|
## `add_document_page(slug: str, title: str | None = None) -> int`
|
|
Ajoute une page en fin de la bande d'onglets. `title` par défaut :
|
|
"Page N" (N = position 1-indexée + 1) si non fourni.
|
|
- **Retour** : l'`id` de la nouvelle page.
|
|
- **Exceptions** : aucune levée explicitement.
|
|
|
|
## `list_document_pages(slug: str) -> list[dict[str, Any]]`
|
|
Toutes les pages du support, triées par `order_index`.
|
|
- **Retour** : liste de dicts (une ligne de table chacun).
|
|
- **Exceptions** : aucune.
|
|
|
|
## `get_document_page(slug: str, page_id: int) -> dict[str, Any] | None`
|
|
Récupère une seule page par id.
|
|
- **Retour** : le dict de la page, ou `None` si l'id n'existe pas.
|
|
- **Exceptions** : aucune.
|
|
|
|
## `update_document_page(slug: str, page_id: int, title: str) -> None`
|
|
Renomme une page. Un titre vide (une fois `.strip()`-é) est
|
|
silencieusement ignoré — la page garde son titre précédent plutôt que de
|
|
se retrouver sans nom dans la bande d'onglets.
|
|
- **Retour** : aucun.
|
|
- **Exceptions** : aucune.
|
|
|
|
## `move_document_page(slug: str, page_id: int, new_index: int) -> None`
|
|
Réordonne une page dans la bande d'onglets (glisser-déposer) —
|
|
renumérote intégralement `order_index` sur toutes les pages, même
|
|
principe que `document_engine.move_document_element` (jamais un
|
|
décalage un par un).
|
|
- **Retour** : aucun.
|
|
- **Exceptions** : aucune.
|
|
|
|
## `delete_document_page(slug: str, page_id: int) -> None`
|
|
Supprime une page ET ses éléments (`DELETE FROM _document_elements
|
|
WHERE page_id = ?` explicite — la contrainte `FOREIGN KEY ... ON DELETE
|
|
CASCADE` n'existe que pour les supports créés après l'ajout des pages,
|
|
voir `db/supports/ensure_document_pages_schema.py` pour les anciens).
|
|
Ne refuse JAMAIS de supprimer la dernière page restante — cette règle
|
|
est posée par l'appelant, pas par cette fonction bas niveau (même
|
|
découpage que `routes/game/screens/screen_delete.py` côté jeu, où le
|
|
garde-fou vit aussi dans la route).
|
|
- **Retour** : aucun.
|
|
- **Exceptions** : aucune.
|