Files
mabibli/MaBibli.Api/Services/Souhaits/ServiceSouhaits.cs
T
mathieuandClaude Opus 5 5cf566bc33 Traiter les retours d'usage du 2026-08-18 (2ᵉ série)
Six lots, issus d'IDEES.md. Les décisions actées sont reportées dans
CLAUDE.md, et les entrées traitées retirées d'IDEES.md.

1. Douchette USB, ISSN et enchaînement du scan
   - Champ ISBN focalisé à l'ouverture : c'est tout ce qu'exige une
     douchette USB, qui se présente au système comme un clavier. Remède
     au scan caméra, qui rate sur la webcam d'un PC (optique, pas
     décodage).
   - Un EAN en 977 est un périodique : il porte un ISSN, donc un titre de
     revue. Intercepté AVANT la validation ISBN (c'est un EAN-13
     parfaitement valide), l'ISSN est déduit du code puis nommé via
     bib.issn. Auparavant la cascade s'exécutait en entier pour finir sur
     « aucun résultat ».
   - L'enchaînement après scan existait déjà mais était invisible : une
     étape « Recherche » affiche désormais le code interrogé.

2. ISBN affichés avec des tirets
   Tables extraites du RangeMessage.xml officiel — plusieurs tranches ne
   sont pas celles qu'on suppose. Le francophone est découpé en entier,
   ailleurs on s'arrête au groupe et à la clé : aucune coupure fausse.
   Corrige au passage l'export CSV, où un ISBN nu était lu comme un
   nombre par Excel.

3. Hors-ligne : la liste d'envies
   ListerSouhaitsAsync était le seul point de lecture hors du dispositif
   hors-ligne, d'où le « 404 Not Found » brut à l'écran. Cinquième
   instantané, écritures refusées, plus aucun message HTTP. Même défaut
   corrigé sur la bibliographie.

4. Navigation par onglets
   Catalogue / Auteurs / Prêts / Envies dans MainLayout ; les barres
   d'actions ne portent plus que des actions. Filtres repliés derrière un
   bouton compteur, ligne « format » masquée quand le fonds n'a qu'un
   format. Une seule entrée d'ajout, désactivée hors-ligne — pas masquée.

5. Liste d'envies : ordre, recherche, couvertures
   Migration RangDesEnvies. Le remplissage reconduit l'ordre affiché
   jusqu'ici : sans lui, les listes existantes se seraient réordonnées
   toutes seules. Réordonnancement par flèches et glisser-déposer (le
   drag & drop HTML5 ne marche pas au doigt). Ajout dans son propre
   écran, avec recherche par titre (bib.title) et couvertures enfin
   alimentées.

6. Bibliographie : une source muette n'est pas une liste vide
   L'écran affichait « BnF injoignable » PUIS « la BnF ne connaît aucun
   livre de cet auteur » — la seconde phrase étant fausse. Les deux cas
   s'excluent désormais, et un bouton Réessayer est offert.

   Vérifié en exécution : le diagnostic d'IDEES.md était faux sur un
   point. Robert A. Harper a bien 7 œuvres à la BnF (85 notices
   annoncées) ; c'était le même délai dépassé observé deux fois, pris
   pour deux causes distinctes.

380 tests, dont un qui applique réellement la migration (EnsureCreated
n'en joue aucune) et un qui verrouille les messages atteignant
l'utilisateur.

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

243 lines
9.1 KiB
C#

using MaBibli.Api.Data;
using MaBibli.Shared.Dtos;
using MaBibli.Shared.Entites;
using MaBibli.Shared.Isbn;
using MaBibli.Shared.Textes;
using Microsoft.EntityFrameworkCore;
namespace MaBibli.Api.Services.Souhaits;
/// <summary>Issue d'un ajout d'envie : soit la ligne créée, soit un message pour l'utilisateur.</summary>
public readonly record struct ResultatSouhait(SouhaitDto? Souhait, string? Erreur)
{
public static ResultatSouhait Ok(SouhaitDto souhait) => new(souhait, null);
public static ResultatSouhait Invalide(string message) => new(null, message);
}
public interface IServiceSouhaits
{
Task<IReadOnlyList<SouhaitDto>> ListerAsync(string? utilisateur, CancellationToken ct = default);
Task<ResultatSouhait> AjouterAsync(
EnregistrementSouhait saisie, string? utilisateur, CancellationToken ct = default);
Task<bool> SupprimerAsync(int id, string? utilisateur, CancellationToken ct = default);
/// <summary>
/// Réordonne la liste de l'appelant selon la suite d'identifiants fournie.
/// </summary>
/// <remarks>
/// Prend la liste <b>entière</b> plutôt qu'un déplacement unitaire : c'est ce dont a besoin
/// le glisser-déposer, et les flèches monter/descendre s'y ramènent sans effort. L'opération
/// est idempotente — réenvoyer le même ordre ne change rien.
/// </remarks>
Task<bool> ReordonnerAsync(
IReadOnlyList<int> idsOrdonnes, string? utilisateur, CancellationToken ct = default);
}
/// <summary>
/// La liste d'envies, <b>personnelle</b>.
/// </summary>
/// <remarks>
/// ⚠️ <b>Toute méthode filtre sur <paramref name="utilisateur"/>, sans exception.</b> C'est la
/// différence de fond avec <c>ServiceCatalogue</c>, dont les lectures ne filtrent jamais sur
/// <c>AjoutePar</c> : le catalogue est commun, la liste d'envies ne l'est pas. Une envie qui
/// fuiterait vers un autre membre du foyer gâcherait exactement ce que la fonctionnalité sert
/// à préparer.
/// <para>
/// Sans identité (ni en-tête SSOwat ni utilisateur simulé), il n'y a pas de liste : on ne rend
/// rien et on n'écrit rien, plutôt que de rattacher des envies à un propriétaire inventé.
/// </para>
/// </remarks>
public sealed class ServiceSouhaits(MaBibliDbContext db) : IServiceSouhaits
{
public async Task<IReadOnlyList<SouhaitDto>> ListerAsync(
string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return [];
}
// L'ordre est celui que l'utilisateur a choisi, du plus désiré au moins désiré. Il a
// remplacé un tri par auteur puis titre : classer une liste d'envies par ordre
// alphabétique répondait à une question que personne ne se pose.
// L'Id départage les rangs égaux — deux lignes de même rang ne devraient pas exister,
// mais un tri instable ferait sautiller la liste entre deux affichages.
var souhaits = await db.LivresSouhaites
.AsNoTracking()
.Where(s => s.Utilisateur == utilisateur)
.OrderBy(s => s.Rang)
.ThenBy(s => s.Id)
.ToListAsync(ct);
return souhaits.Select(Projeter).ToList();
}
public async Task<ResultatSouhait> AjouterAsync(
EnregistrementSouhait saisie, string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return ResultatSouhait.Invalide(
"Impossible d'ajouter une envie sans savoir à qui elle appartient.");
}
if (string.IsNullOrWhiteSpace(saisie.Titre))
{
return ResultatSouhait.Invalide("Le titre est obligatoire.");
}
// L'ISBN reste facultatif — on souhaite souvent une œuvre sans avoir choisi son édition —
// mais s'il est saisi il doit tenir debout, comme pour un livre du catalogue.
string? isbn = null;
if (!string.IsNullOrWhiteSpace(saisie.Isbn))
{
var normalise = IsbnUtils.Normaliser(saisie.Isbn);
if (!IsbnUtils.EstValide(normalise))
{
return ResultatSouhait.Invalide(
$"« {saisie.Isbn} » n'est pas un ISBN valide. Laissez le champ vide si vous ne l'avez pas.");
}
isbn = normalise;
}
var souhait = new LivreSouhaite
{
Utilisateur = utilisateur,
Titre = saisie.Titre,
Auteur = saisie.Auteur,
Editeur = Vide(saisie.Editeur),
Annee = Vide(saisie.Annee),
Isbn = isbn,
CoverUrl = Vide(saisie.CoverUrl),
Note = Vide(saisie.Note),
DateAjout = DateTime.UtcNow,
};
souhait.RecalculerFormes();
// Le doublon est refusé avec un message plutôt que laissé à l'index unique : l'écran de
// bibliographie rend le double clic facile, et « UNIQUE constraint failed » ne veut rien
// dire pour l'utilisateur.
var deja = await db.LivresSouhaites.AnyAsync(
s => s.Utilisateur == utilisateur
&& s.TitreNormalise == souhait.TitreNormalise
&& s.AuteurNormalise == souhait.AuteurNormalise,
ct);
if (deja)
{
return ResultatSouhait.Invalide($"« {souhait.Titre} » est déjà dans votre liste d'envies.");
}
// En fin de liste : une envie qu'on vient de noter n'est pas déclarée plus désirable
// que celles déjà classées. La poser en tête déclasserait le choix de l'utilisateur
// à chaque ajout.
var dernierRang = await db.LivresSouhaites
.Where(s => s.Utilisateur == utilisateur)
.Select(s => (int?)s.Rang)
.MaxAsync(ct);
souhait.Rang = (dernierRang ?? -1) + 1;
db.LivresSouhaites.Add(souhait);
await db.SaveChangesAsync(ct);
return ResultatSouhait.Ok(Projeter(souhait));
}
public async Task<bool> SupprimerAsync(int id, string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return false;
}
// Le filtre sur l'utilisateur fait partie de la CLÉ de recherche, pas d'une vérification
// ultérieure : ainsi personne ne peut supprimer l'envie d'un autre en devinant son
// identifiant. Une envie inexistante et une envie appartenant à un autre sont
// indiscernables de l'extérieur, ce qui est bien le comportement voulu.
var souhait = await db.LivresSouhaites
.FirstOrDefaultAsync(s => s.Id == id && s.Utilisateur == utilisateur, ct);
if (souhait is null)
{
return false;
}
db.LivresSouhaites.Remove(souhait);
await db.SaveChangesAsync(ct);
return true;
}
public async Task<bool> ReordonnerAsync(
IReadOnlyList<int> idsOrdonnes, string? utilisateur, CancellationToken ct = default)
{
if (utilisateur is null)
{
return false;
}
// ⚠️ On relit TOUTE la liste de l'appelant, et on ne se fie pas à ce que le client
// envoie. Deux raisons : un identifiant appartenant à quelqu'un d'autre ne doit pas
// pouvoir être renuméroté (le filtre fait partie de la clé, comme pour la suppression),
// et la liste du client peut être périmée — une envie ajoutée depuis un autre appareil
// n'y figure pas, et la perdre serait pire que de mal la classer.
var siennes = await db.LivresSouhaites
.Where(s => s.Utilisateur == utilisateur)
.ToListAsync(ct);
if (siennes.Count == 0)
{
return false;
}
var parId = siennes.ToDictionary(s => s.Id);
var rang = 0;
foreach (var id in idsOrdonnes.Distinct())
{
if (parId.Remove(id, out var souhait))
{
souhait.Rang = rang++;
}
// Un identifiant inconnu — supprimé entre-temps, ou appartenant à un autre — est
// simplement ignoré : réordonner n'est pas une occasion de découvrir des erreurs.
}
// Ce que le client ne connaissait pas se range à la suite, dans son ordre précédent,
// plutôt que d'être renuméroté au hasard.
foreach (var oublie in parId.Values.OrderBy(s => s.Rang).ThenBy(s => s.Id))
{
oublie.Rang = rang++;
}
await db.SaveChangesAsync(ct);
return true;
}
private static string? Vide(string? valeur) =>
string.IsNullOrWhiteSpace(valeur) ? null : valeur.Trim();
/// <summary>
/// Projection vers le DTO. <b>Sans le champ <c>Utilisateur</c></b> : il ne sort jamais de
/// l'API, pour qu'aucun client ne puisse croire qu'il désigne de qui il parle.
/// </summary>
internal static SouhaitDto Projeter(LivreSouhaite souhait) => new()
{
Id = souhait.Id,
Titre = souhait.Titre,
Auteur = souhait.Auteur,
Editeur = souhait.Editeur,
Annee = souhait.Annee,
Isbn = souhait.Isbn,
CoverUrl = souhait.CoverUrl,
Note = souhait.Note,
DateAjout = souhait.DateAjout,
};
}