Files
mabibli/MaBibli.Api/Services/Souhaits/ServiceRevuesSouhaitees.cs
T
mathieuandClaude Opus 5 94f0ef15c0 Compte les pages d'un livre, et fais des envies une liste à deux moitiés
Lot U — le nombre de pages
`Livre.NombrePages` est nullable, sans valeur par défaut : « 0 page » se lirait
comme une donnée là où l'on veut dire « on ne sait pas ». Même raison que pour
`TypeDocument.NonPrecise` — un défaut qui ne prétend rien n'a rien à reprendre,
d'où une migration réduite à un `AddColumn`. Le service refuse un zéro plutôt
que de l'écrire ; effacer le champ reste la façon de revenir à « inconnu ».

Le préremplissage vient de `dc:format`, que rien ne lisait jusqu'ici. ⚠️ Ce
champ n'est pas un nombre mais une phrase décrivant le support, et les notices
déjà enregistrées sous Fixtures/ le montrent : « 1 vol. (113 p.) : ill., couv.
ill. en coul. ; 18 cm », « 503 p. : couv. ill. ; 17 cm ». Les règles sont donc
étroites — un nombre suivi de « p. » ou de « page(s) », rien d'autre — et tout
le reste rend `null`. L'erreur n'est pas symétrique : un champ vide se remplit à
la main en trois secondes, un chiffre faux s'enregistre sans que personne ne le
voie. « 1 vol. » ne vaut pas 1, « 30 cm » ne vaut pas 30, et « (p. 45-90) »,
qui est une pagination de contribution, ne vaut rien. La valeur reste proposée
dans un champ modifiable, et rien n'est déduit pour un ebook.

Lot X — éditer une envie, et souhaiter une revue
`PUT /api/souhaits/{id}` recalcule la clé d'œuvre et l'auteur normalisé : sans
ce recalcul, le rapprochement « déjà au catalogue » continuerait de se faire sur
l'ancienne forme, et le signalement mentirait sans le dire. Le filtre sur
l'appelant fait partie de la clé de recherche, pas d'une vérification ultérieure
— l'envie d'un autre est introuvable (404), jamais refusée (403).

⚠️ Une édition peut heurter l'unicité (utilisateur, œuvre, auteur), ce qu'un
ajout ne peut pas : renommer une envie en une autre déjà présente répond par un
message lisible, jamais par « UNIQUE constraint failed ». 400 et non 409,
contrairement au doublon du catalogue : là-bas posséder deux exemplaires est
légitime et l'appel se reconfirme, ici l'index l'interdit et il n'y a rien à
confirmer. Le rang n'est pas touché — l'ordre a son propre point d'entrée.

⚠️ La couverture n'est écrite que si la charge utile en porte une. Aucun écran
n'offre de champ « URL de couverture » pour une envie (décision actée), donc un
remplacement inconditionnel l'aurait effacée à la première faute de frappe
corrigée.

`RevueSouhaitee` est une table sœur, et non des colonnes de plus sur
`LivreSouhaite` : un numéro n'a pas d'auteur et se distingue par son numéro,
deux choses que la clé d'unicité des envies de livres ne sait pas exprimer sans
devenir fausse pour tout le monde. `NumeroNormalise` est NOT NULL avec un défaut
vide — SQLite tient deux NULL pour distincts, et « Médor, sans numéro »
s'ajouterait autant de fois qu'on cliquerait. L'ISSN est canonisé avec son tiret,
seul code du projet rangé ainsi.

Le coût de la table sœur est payé partout où il devait l'être : affichage,
`.txt`, `.csv` et instantané hors-ligne `souhaits-revues`. ⚠️ Les revues forment
une SECTION à part plutôt que des lignes entrelacées : chaque table numérote son
rang indépendamment, et mélanger deux suites sans rapport produirait un ordre que
personne n'a choisi. Le `.txt`, groupé par auteur, ne pouvait de toute façon pas
les accueillir — elles n'en ont pas, et « Auteur non précisé » désigne des livres
dont l'auteur est inconnu. Le CSV gagne une colonne « Type » : sans elle, un tri
par titre rendrait revues et livres indiscernables, et la colonne des codes
mêlerait ISBN et ISSN en silence.

`ServiceRenormalisation` connaît la nouvelle table, avec la règle de collision
déjà en place. ⚠️ L'ISSN y est canonisé à part : `Renormaliser` n'applique rien
quand la clé ne bouge pas, un ISSN mal formé sur une ligne au titre inchangé y
échapperait.

`RevueSouhaitee` ne porte PAS de `CoverUrl` : rien à ajouter au garde de
`GET /api/couvertures`.

Vérifié en exécution : ISSN « 24666718 » rangé « 2466-6718 », édition de l'envie
d'un autre en 404, et les deux exports portant bien les deux moitiés.
602 tests au vert (552 au départ), aucun avertissement de compilation.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-21 14:02:09 +02:00

209 lines
7.3 KiB
C#

using MaBibli.Api.Data;
using MaBibli.Shared.Dtos;
using MaBibli.Shared.Entites;
using Microsoft.EntityFrameworkCore;
namespace MaBibli.Api.Services.Souhaits;
/// <summary>Issue d'une écriture d'envie de revue : la ligne, un message, ou rien du tout.</summary>
public readonly record struct ResultatRevueSouhaitee(
RevueSouhaiteeDto? Envie, string? Erreur, bool NExistePas = false)
{
public static readonly ResultatRevueSouhaitee Introuvable = new(null, null, true);
public static ResultatRevueSouhaitee Ok(RevueSouhaiteeDto envie) => new(envie, null);
public static ResultatRevueSouhaitee Invalide(string message) => new(null, message);
}
public interface IServiceRevuesSouhaitees
{
Task<IReadOnlyList<RevueSouhaiteeDto>> ListerAsync(
string? utilisateur, CancellationToken ct = default);
Task<ResultatRevueSouhaitee> AjouterAsync(
EnregistrementRevueSouhaitee saisie, string? utilisateur, CancellationToken ct = default);
Task<ResultatRevueSouhaitee> ModifierAsync(
int id, EnregistrementRevueSouhaitee saisie, string? utilisateur, CancellationToken ct = default);
Task<bool> SupprimerAsync(int id, string? utilisateur, CancellationToken ct = default);
}
/// <summary>
/// Les envies de revues et de numéros, <b>personnelles</b>.
/// </summary>
/// <remarks>
/// Mêmes règles que <see cref="ServiceSouhaits"/>, sans exception : toute méthode filtre sur
/// l'appelant, aucun point d'entrée n'accepte de nom d'utilisateur, et l'envie d'un autre est
/// simplement introuvable.
/// <para>
/// ⚠️ Il n'y a <b>pas</b> de rapprochement avec les revues possédées, contrairement aux envies
/// de livres. Ce n'est pas un oubli : « je possède la revue mais pas ce numéro » et « je possède
/// ce numéro » demandent deux réponses différentes, et un signalement approximatif sur une liste
/// d'envies coûte plus qu'il ne rapporte.
/// </para>
/// </remarks>
public sealed class ServiceRevuesSouhaitees(MaBibliDbContext db) : IServiceRevuesSouhaitees
{
public async Task<IReadOnlyList<RevueSouhaiteeDto>> ListerAsync(
string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return [];
}
var envies = await db.RevuesSouhaitees
.AsNoTracking()
.Where(e => e.Utilisateur == utilisateur)
.OrderBy(e => e.Rang)
.ThenBy(e => e.Id)
.ToListAsync(ct);
return envies.Select(Projeter).ToList();
}
public async Task<ResultatRevueSouhaitee> AjouterAsync(
EnregistrementRevueSouhaitee saisie, string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return ResultatRevueSouhaitee.Invalide(
"Impossible d'ajouter une envie sans savoir à qui elle appartient.");
}
if (string.IsNullOrWhiteSpace(saisie.Titre))
{
return ResultatRevueSouhaitee.Invalide("Le titre de la revue est obligatoire.");
}
var envie = new RevueSouhaitee
{
Utilisateur = utilisateur,
Titre = saisie.Titre,
Numero = saisie.Numero,
Issn = saisie.Issn,
Note = Vide(saisie.Note),
DateAjout = DateTime.UtcNow,
};
// ⚠️ Canonise l'ISSN au passage : « 24666718 » devient « 2466-6718 », la seule forme
// rangée en base et la seule que la BnF interroge.
envie.RecalculerFormes();
if (await ExisteDejaAsync(envie, null, ct))
{
return ResultatRevueSouhaitee.Invalide(Deja(envie));
}
// En fin de section, comme une envie de livre : on vient de la noter, on n'a pas dit
// qu'on la voulait plus que les autres.
var dernierRang = await db.RevuesSouhaitees
.Where(e => e.Utilisateur == utilisateur)
.Select(e => (int?)e.Rang)
.MaxAsync(ct);
envie.Rang = (dernierRang ?? -1) + 1;
db.RevuesSouhaitees.Add(envie);
await db.SaveChangesAsync(ct);
return ResultatRevueSouhaitee.Ok(Projeter(envie));
}
public async Task<ResultatRevueSouhaitee> ModifierAsync(
int id, EnregistrementRevueSouhaitee saisie, string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return ResultatRevueSouhaitee.Introuvable;
}
// Le filtre fait partie de la clé de recherche : l'envie d'un autre n'est pas refusée,
// elle n'est pas trouvée.
var envie = await db.RevuesSouhaitees
.FirstOrDefaultAsync(e => e.Id == id && e.Utilisateur == utilisateur, ct);
if (envie is null)
{
return ResultatRevueSouhaitee.Introuvable;
}
if (string.IsNullOrWhiteSpace(saisie.Titre))
{
return ResultatRevueSouhaitee.Invalide("Le titre de la revue est obligatoire.");
}
envie.Titre = saisie.Titre;
envie.Numero = saisie.Numero;
envie.Issn = saisie.Issn;
envie.Note = Vide(saisie.Note);
envie.RecalculerFormes();
if (await ExisteDejaAsync(envie, envie.Id, ct))
{
return ResultatRevueSouhaitee.Invalide(Deja(envie));
}
// Rang inchangé : corriger un numéro ne dit rien de l'ordre voulu.
await db.SaveChangesAsync(ct);
return ResultatRevueSouhaitee.Ok(Projeter(envie));
}
public async Task<bool> SupprimerAsync(int id, string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return false;
}
var envie = await db.RevuesSouhaitees
.FirstOrDefaultAsync(e => e.Id == id && e.Utilisateur == utilisateur, ct);
if (envie is null)
{
return false;
}
db.RevuesSouhaitees.Remove(envie);
await db.SaveChangesAsync(ct);
return true;
}
/// <summary>
/// Le doublon est refusé <b>avec un message</b>, jamais laissé à l'index unique.
/// </summary>
/// <remarks>
/// ⚠️ L'entité est déjà rattachée au contexte lors d'une modification : d'où l'exclusion
/// explicite de sa propre ligne, sans quoi toute édition se dirait doublon d'elle-même.
/// </remarks>
private Task<bool> ExisteDejaAsync(RevueSouhaitee envie, int? saufId, CancellationToken ct) =>
db.RevuesSouhaitees.AnyAsync(
e => e.Utilisateur == envie.Utilisateur
&& e.TitreNormalise == envie.TitreNormalise
&& e.NumeroNormalise == envie.NumeroNormalise
&& (saufId == null || e.Id != saufId),
ct);
private static string Deja(RevueSouhaitee envie) =>
envie.Numero is null
? $"« {envie.Titre} » est déjà dans votre liste d'envies."
: $"« {envie.Titre} n° {envie.Numero} » est déjà dans votre liste d'envies.";
private static string? Vide(string? valeur) =>
string.IsNullOrWhiteSpace(valeur) ? null : valeur.Trim();
/// <summary>Projection vers le DTO, <b>sans</b> le champ <c>Utilisateur</c>.</summary>
private static RevueSouhaiteeDto Projeter(RevueSouhaitee envie) => new()
{
Id = envie.Id,
Titre = envie.Titre,
Numero = envie.Numero,
Issn = envie.Issn,
Note = envie.Note,
DateAjout = envie.DateAjout,
};
}