Commit 5dca6446 authored by Kourser's avatar Kourser
Browse files

Un modèle de message porte un libellé, pas un nom 

La lecture à version triait par « nom » toutes les tables du dossier sauf
deux, nommément écartées. modele_message est arrivé avec les modèles sans
rejoindre cette liste, et sa table ne porte pas de colonne « nom » — elle a un
libelle. Toute lecture du dossier échouait donc sur « column "nom" does not
exist », et « pnpm verif » s'arrêtait là.

Ce que le tri par défaut supposait
  Que toute table de préparation ait un nom. C'était vrai des quatre qui
  décrivent des gens et des organisations — personne, tiers,
  application_metier, partie_prenante — et l'exception se traitait par une
  liste d'écarts plutôt que par une déclaration.

  Un défaut qui vaut pour quatre cas sur sept n'est plus un défaut, c'est un
  piège tendu au prochain ajout. La branche pour modele_message est écrite, et
  le commentaire dit ce que la prochaine table devra faire : déclarer son tri.

Contrôlé
  891 garanties vérifiées, aucune en échec. La suite va désormais jusqu'au
  bout.

Co-Authored-By: default avatarClaude Opus 5 <noreply@anthropic.com>
parent f019b09b

apps/api/src/preparation/dossier.ts

+5 −0
Changes for apps/api/src/preparation/dossier.ts: 5 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -142,6 +142,11 @@ export async function lire<T extends object>( 
               and (jusqu_a_version is null or jusqu_a_version > ${version})`}

     order by ${table === "fiche_reflexe" ? sql`ordre, titre`

                : table === "document_kit" ? sql`categorie, titre`

                // Un modèle de message porte un libellé, pas un nom : ce sont

                // les gens et les organisations qui ont des noms. Toute table

                // ajoutée ici doit déclarer son tri — « nom » n'est pas un

                // défaut universel, c'est celui des quatre tables qui en ont un.

                : table === "modele_message" ? sql`libelle`

                : sql`nom`}`;

}