Zwei Namen für denselben Titel brachen den ganzen Lauf ab
Der Eindeutigkeitsindex aus Punkt 3 stellte die Regel auf, dichtete aber den einen Weg nicht ab, auf dem sie verletzt wird: der Enricher setzt die TMDB-Kennung, und sobald zwei Einträge derselben Mediathek dieselbe bekamen, brach EF die gesamte Speicherung ab — mit "An error occurred while saving the entity changes", einer Meldung, die den Grund nicht nennt und den ganzen Scan mitnimmt. Jetzt wird die Kennung nicht blind gesetzt, sondern vorgemerkt und nach der Schleife zusammengelegt. Zwei Tests halten den Fall fest: der Anbieter antwortet auf jede Anfrage mit demselben Film beziehungsweise derselben Serie, also genau die Lage, die vorher den Lauf zerriss. Dazu der wahrscheinliche Grund für die siebenhundert Serien: der Serientitel kam ausschliesslich aus dem Dateinamen. Heisst eine Datei nur "S01E01.mkv", steht vor der Folgennummer nichts, und der ganze Dateiname wurde zum Serientitel — zwanzig Folgen ergaben zwanzig Serien namens "S01E01", "S01E02", und so weiter. Jetzt sagt der Ordner den Namen, wenn die Datei ihn nicht nennt; der Staffelordner wird dabei übersprungen, sonst hiesse die Serie "Staffel 1". Plex und Jellyfin lesen ihn seit je so. Dazu die Prüfliste: Folgen werden nach Serie gebündelt, weil zwanzig neue Folgen eine Entscheidung sind und nicht zwanzig. Mit Mehrfachauswahl — anhaken und gemeinsam freigeben oder auf einen Zeitpunkt legen. Und der Zeitpunkt ist ein datetime-local statt eines Textfelds mit Formvorschrift, bei dem jeder Tippfehler erst nach dem Absenden auffiel. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Scarriffle
co-authored by
Claude Opus 5
Claude Opus 5
parent
228c318ad2
commit
fa18c0d7fa
@@ -85,13 +85,32 @@ public sealed record Bildauswahl(
|
||||
public sealed record BildWahlRequest(string Kind, string? Url);
|
||||
|
||||
/// <summary>Ein Titel, der noch wartet oder geplant ist.</summary>
|
||||
///
|
||||
/// <remarks>
|
||||
/// <b>Eine Zeile je Serie, nicht je Folge.</b> Bei zwanzig neuen Folgen
|
||||
/// derselben Serie wären zwanzig Zeilen zwanzig Entscheidungen für eine
|
||||
/// Sache. Die Folgen stecken in <see cref="Ids"/>; freigegeben werden sie
|
||||
/// gemeinsam.
|
||||
/// </remarks>
|
||||
public sealed record Pruefeintrag(
|
||||
string Kind, Guid Id, string Title, int? Year, string? PosterImage,
|
||||
/// <summary><c>movie</c> oder <c>series</c>.</summary>
|
||||
string Kind,
|
||||
Guid Id, string Title, int? Year, string? PosterImage,
|
||||
/// <summary><c>null</c> heisst: wartet. Ein Zeitpunkt in der Zukunft heisst: geplant.</summary>
|
||||
///
|
||||
/// <remarks>
|
||||
/// Bei einer Serie nur gesetzt, wenn alle Folgen denselben Termin tragen —
|
||||
/// sonst wäre es eine Behauptung über zwanzig Folgen, von denen eine
|
||||
/// anders liegt.
|
||||
/// </remarks>
|
||||
DateTimeOffset? PublishAt,
|
||||
string? Path,
|
||||
/// <summary>Ob eine Zuordnung besteht — ohne sie fehlen Titel und Bilder.</summary>
|
||||
bool Matched);
|
||||
bool Matched,
|
||||
/// <summary>Wie viele Titel dahinterstecken — bei einer Serie die Zahl der Folgen.</summary>
|
||||
int Count,
|
||||
/// <summary>Worauf eine Freigabe wirkt.</summary>
|
||||
IReadOnlyList<Guid> Ids);
|
||||
|
||||
public sealed record FreigabeRequest(IReadOnlyList<Guid>? Ids, DateTimeOffset? PublishAt);
|
||||
|
||||
|
||||
@@ -26,24 +26,56 @@ public static class FreigabeEndpoints
|
||||
.Select(m => new Pruefeintrag(
|
||||
"movie", m.Id, m.Title, m.Year, m.PosterImage, m.PublishAt,
|
||||
db.MediaFiles.Where(f => f.MovieId == m.Id).Select(f => f.Path).FirstOrDefault(),
|
||||
m.TmdbId != null))
|
||||
m.TmdbId != null, 1, new List<Guid> { m.Id }))
|
||||
.ToListAsync(ct);
|
||||
|
||||
var folgen = await db.Episodes.AsNoTracking()
|
||||
// **Folgen werden nach Serie gebündelt.** Zwanzig neue Folgen
|
||||
// derselben Serie sind eine Entscheidung, nicht zwanzig.
|
||||
var offeneFolgen = await db.Episodes.AsNoTracking()
|
||||
.Where(f => f.PublishAt == null || f.PublishAt > jetzt)
|
||||
.OrderBy(f => f.SeasonNumber).ThenBy(f => f.EpisodeNumber)
|
||||
.Select(f => new Pruefeintrag(
|
||||
"episode", f.Id,
|
||||
db.Series.Where(s => s.Id == f.SeriesId).Select(s => s.Title).First()
|
||||
+ $" · S{f.SeasonNumber}E{f.EpisodeNumber}",
|
||||
null,
|
||||
db.Series.Where(s => s.Id == f.SeriesId).Select(s => s.PosterImage).First(),
|
||||
f.PublishAt,
|
||||
db.MediaFiles.Where(d => d.EpisodeId == f.Id).Select(d => d.Path).FirstOrDefault(),
|
||||
true))
|
||||
.Select(f => new
|
||||
{
|
||||
f.Id, f.SeriesId, f.SeasonNumber, f.EpisodeNumber, f.PublishAt,
|
||||
Pfad = db.MediaFiles.Where(d => d.EpisodeId == f.Id)
|
||||
.Select(d => d.Path).FirstOrDefault(),
|
||||
})
|
||||
.ToListAsync(ct);
|
||||
|
||||
return Results.Ok(filme.Concat(folgen).ToList());
|
||||
var serienIds = offeneFolgen.Select(f => f.SeriesId).Distinct().ToList();
|
||||
var serien = await db.Series.AsNoTracking()
|
||||
.Where(s => serienIds.Contains(s.Id))
|
||||
.Select(s => new { s.Id, s.Title, s.Year, s.PosterImage, Zugeordnet = s.TmdbId != null })
|
||||
.ToDictionaryAsync(s => s.Id, ct);
|
||||
|
||||
var gebuendelt = offeneFolgen
|
||||
.GroupBy(f => f.SeriesId)
|
||||
.Select(g =>
|
||||
{
|
||||
var serie = serien.GetValueOrDefault(g.Key);
|
||||
var termine = g.Select(f => f.PublishAt).Distinct().ToList();
|
||||
|
||||
return new Pruefeintrag(
|
||||
"series", g.Key,
|
||||
serie?.Title ?? "Unbekannte Serie",
|
||||
serie?.Year,
|
||||
serie?.PosterImage,
|
||||
// Nur wenn alle denselben Termin tragen — sonst wäre es
|
||||
// eine Behauptung über Folgen, die anders liegen.
|
||||
termine.Count == 1 ? termine[0] : null,
|
||||
Pfad(g.Select(f => f.Pfad).FirstOrDefault()),
|
||||
serie?.Zugeordnet ?? false,
|
||||
g.Count(),
|
||||
g.OrderBy(f => f.SeasonNumber).ThenBy(f => f.EpisodeNumber)
|
||||
.Select(f => f.Id).ToList());
|
||||
})
|
||||
.OrderBy(e => e.Title)
|
||||
.ToList();
|
||||
|
||||
return Results.Ok(filme.Concat(gebuendelt).ToList());
|
||||
|
||||
// Bei einer Serie sagt der Ordner mehr als die erste Folgendatei.
|
||||
static string? Pfad(string? datei) =>
|
||||
datei is null ? null : System.IO.Path.GetDirectoryName(datei);
|
||||
});
|
||||
|
||||
// Veröffentlichen oder planen. Ohne Zeitpunkt heisst: jetzt.
|
||||
|
||||
@@ -1,3 +1,4 @@
|
||||
using Microsoft.AspNetCore.DataProtection;
|
||||
using System.Threading.RateLimiting;
|
||||
using Flaro.Api;
|
||||
using Flaro.Api.Auth;
|
||||
@@ -99,6 +100,22 @@ builder.Services.AddSingleton<PackageCoordinator>();
|
||||
// ueber den Augenblick. Nach einem Neustart lautet die Antwort: niemand.
|
||||
builder.Services.AddSingleton<PlaybackSessions>();
|
||||
builder.Services.AddSingleton<Tagebuch>();
|
||||
|
||||
// **Die Schlüssel der Datenverschlüsselung gehören ins Volume.**
|
||||
//
|
||||
// Ohne diese Zeile legt ASP.NET sie unter `~/.aspnet/DataProtection-Keys` ab —
|
||||
// also im Container, der beim nächsten Update weg ist. Heute fällt das nicht
|
||||
// auf, weil Flaros Sitzungs-Cookie das Gerätetoken selbst trägt und nichts
|
||||
// davon verschlüsselt wird. Es fällt erst auf, wenn irgendwann etwas doch
|
||||
// darauf baut, und dann sieht es aus wie ein Fehler anderswo.
|
||||
//
|
||||
// Nebenbei verschwindet damit die Warnung, die bei jedem Start im Protokoll
|
||||
// stand und nach einem Problem aussah, das keines war.
|
||||
var schluesselOrdner = Path.Combine(pfade.Data, "keys");
|
||||
Directory.CreateDirectory(schluesselOrdner);
|
||||
builder.Services.AddDataProtection()
|
||||
.PersistKeysToFileSystem(new DirectoryInfo(schluesselOrdner))
|
||||
.SetApplicationName("Flaro");
|
||||
builder.Services.AddProblemDetails();
|
||||
|
||||
var app = builder.Build();
|
||||
|
||||
@@ -206,13 +206,61 @@ public static partial class FileNames
|
||||
var (worteDanach, _) = Zerlegen(danach, istDateiname: false);
|
||||
var (folgenTitel, _) = TitelBis(worteDanach, worteDanach.Length);
|
||||
|
||||
// **Steht im Dateinamen keine Serie, sagt der Ordner sie.**
|
||||
//
|
||||
// Bei „Dark/Staffel 1/S01E01.mkv" steht vor der Folgennummer nichts —
|
||||
// und bis hierher wurde dann der ganze Dateiname zum Serientitel.
|
||||
// Jede so benannte Datei ergab damit eine eigene Serie: aus zwanzig
|
||||
// Folgen wurden zwanzig Serien namens „S01E01", „S01E02", …
|
||||
//
|
||||
// Plex und Jellyfin lesen den Namen seit je aus dem Ordner. Der
|
||||
// Staffelordner wird dabei übersprungen, sonst hiesse die Serie
|
||||
// „Staffel 1".
|
||||
var ausOrdner = serie.Length > 0 ? null : SerieAusOrdner(dateiname);
|
||||
|
||||
return new EpisodeName(
|
||||
SeriesTitle: serie.Length > 0 ? serie : Rohname(dateiname),
|
||||
SeriesTitle: serie.Length > 0 ? serie : ausOrdner ?? Rohname(dateiname),
|
||||
Season: staffel,
|
||||
Episode: folge,
|
||||
EpisodeTitle: folgenTitel.Length > 0 ? folgenTitel : null);
|
||||
}
|
||||
|
||||
/// <summary>Der Serienname aus dem Pfad — Staffelordner übersprungen.</summary>
|
||||
///
|
||||
/// <remarks>
|
||||
/// <b>Höchstens zwei Ebenen hoch.</b> Weiter oben steht der Wurzelordner
|
||||
/// der Mediathek, und „Serien" ist kein Serientitel.
|
||||
/// </remarks>
|
||||
private static string? SerieAusOrdner(string pfad)
|
||||
{
|
||||
var ordner = Path.GetDirectoryName(pfad);
|
||||
|
||||
for (var ebene = 0; ebene < 2 && !string.IsNullOrEmpty(ordner); ebene++)
|
||||
{
|
||||
var name = Path.GetFileName(ordner);
|
||||
if (name.Length == 0) return null;
|
||||
|
||||
if (!IstStaffelordner(name))
|
||||
{
|
||||
var (worte, _) = Zerlegen(name, istDateiname: false);
|
||||
var (titel, _) = TitelBis(worte, worte.Length);
|
||||
return titel.Length > 0 ? titel : null;
|
||||
}
|
||||
|
||||
ordner = Path.GetDirectoryName(ordner);
|
||||
}
|
||||
|
||||
return null;
|
||||
}
|
||||
|
||||
/// <summary>„Staffel 1", „Season 02", „S3" — Ordner, die keine Serie benennen.</summary>
|
||||
private static bool IstStaffelordner(string name) =>
|
||||
StaffelOrdnerName().IsMatch(name.Trim());
|
||||
|
||||
[GeneratedRegex(@"^(staffel|season|series|s)[\s._-]*\d{1,3}$|^(specials?|extras?)$",
|
||||
RegexOptions.IgnoreCase)]
|
||||
private static partial Regex StaffelOrdnerName();
|
||||
|
||||
/// <summary>Nimmt Worte, bis das erste Beiwerk oder eine Jahreszahl kommt.</summary>
|
||||
private static (string Titel, int? Jahr) TitelBis(string[] worte, int grenze)
|
||||
{
|
||||
|
||||
@@ -30,6 +30,7 @@ public sealed class MetadataEnricher(
|
||||
IImageStore bilder,
|
||||
IImageReader bildLeser,
|
||||
ISidecarStore beiwerk,
|
||||
Zusammenlegung zusammenlegung,
|
||||
ILogger<MetadataEnricher> log)
|
||||
{
|
||||
/// <param name="erneut">
|
||||
@@ -81,6 +82,14 @@ public sealed class MetadataEnricher(
|
||||
var zugeordnet = 0;
|
||||
var durch = 0;
|
||||
|
||||
// Wer welche Kennung schon hat — auch die, die dieser Lauf gerade
|
||||
// vergeben hat und die deshalb noch nicht in der Datenbank stehen.
|
||||
var vergeben = await db.Movies
|
||||
.Where(m => m.LibraryId == libraryId && m.TmdbId != null)
|
||||
.ToDictionaryAsync(m => m.TmdbId!.Value, m => m.Id, ct);
|
||||
|
||||
var zusammenzulegen = new List<(Guid Sieger, Guid Verlierer)>();
|
||||
|
||||
foreach (var film in offen)
|
||||
{
|
||||
ct.ThrowIfCancellationRequested();
|
||||
@@ -101,6 +110,24 @@ public sealed class MetadataEnricher(
|
||||
|
||||
if (gefunden is null) continue;
|
||||
|
||||
// **Zwei Einträge, derselbe Film — und das fällt erst hier auf.**
|
||||
//
|
||||
// Beim Scan heisst der eine „Ich einfach unverbesserlich" und der
|
||||
// andere „Despicable Me"; dass es derselbe ist, verrät allein die
|
||||
// TMDB-Kennung. Die hier blind zu setzen verletzt den
|
||||
// Eindeutigkeitsindex, und EF bricht die ganze Speicherung ab —
|
||||
// mit „An error occurred while saving the entity changes", einer
|
||||
// Meldung, die den Grund nicht nennt und den ganzen Lauf mitnimmt.
|
||||
//
|
||||
// Also nicht setzen, sondern vormerken: zusammengelegt wird nach
|
||||
// der Schleife, wenn nichts mehr dazwischenkommt.
|
||||
if (vergeben.TryGetValue(gefunden.ProviderId, out var schonDa))
|
||||
{
|
||||
zusammenzulegen.Add((Sieger: schonDa, Verlierer: film.Id));
|
||||
continue;
|
||||
}
|
||||
|
||||
vergeben[gefunden.ProviderId] = film.Id;
|
||||
film.TmdbId = gefunden.ProviderId;
|
||||
film.ImdbId = gefunden.ImdbId;
|
||||
film.Title = gefunden.Title;
|
||||
@@ -128,6 +155,23 @@ public sealed class MetadataEnricher(
|
||||
}
|
||||
|
||||
if (zugeordnet > 0) await db.SaveChangesAsync(ct);
|
||||
|
||||
// **Erst speichern, dann zusammenlegen.** Das Verschmelzen löscht
|
||||
// Zeilen am Änderungsverfolger vorbei; stünde es davor, schriebe das
|
||||
// folgende Speichern sie wieder hin.
|
||||
foreach (var (sieger, verlierer) in zusammenzulegen)
|
||||
{
|
||||
await zusammenlegung.FilmVerschmelzenAsync(sieger, verlierer, ct);
|
||||
}
|
||||
|
||||
if (zusammenzulegen.Count > 0)
|
||||
{
|
||||
await db.SaveChangesAsync(ct);
|
||||
log.LogInformation(
|
||||
"{Anzahl} Film(e) waren derselbe Titel unter anderem Namen und wurden zusammengelegt.",
|
||||
zusammenzulegen.Count);
|
||||
}
|
||||
|
||||
return zugeordnet;
|
||||
}
|
||||
|
||||
@@ -160,6 +204,12 @@ public sealed class MetadataEnricher(
|
||||
var zugeordnet = 0;
|
||||
var durch = 0;
|
||||
|
||||
var vergeben = await db.Series
|
||||
.Where(x => x.LibraryId == libraryId && x.TmdbId != null)
|
||||
.ToDictionaryAsync(x => x.TmdbId!.Value, x => x.Id, ct);
|
||||
|
||||
var zusammenzulegen = new List<(Guid Sieger, Guid Verlierer)>();
|
||||
|
||||
foreach (var serie in offen)
|
||||
{
|
||||
ct.ThrowIfCancellationRequested();
|
||||
@@ -181,6 +231,17 @@ public sealed class MetadataEnricher(
|
||||
|
||||
if (gefunden is null) continue;
|
||||
|
||||
// Wie beim Film: zwei Einträge, dieselbe Serie. Bei Serien ist das
|
||||
// noch häufiger — derselbe Ordner unter zwei Schreibweisen, oder
|
||||
// ein Staffelordner, der versehentlich als eigene Serie gelesen
|
||||
// wurde.
|
||||
if (vergeben.TryGetValue(gefunden.ProviderId, out var schonDa))
|
||||
{
|
||||
zusammenzulegen.Add((Sieger: schonDa, Verlierer: serie.Id));
|
||||
continue;
|
||||
}
|
||||
|
||||
vergeben[gefunden.ProviderId] = serie.Id;
|
||||
serie.TmdbId = gefunden.ProviderId;
|
||||
serie.ImdbId = gefunden.ImdbId;
|
||||
serie.Title = gefunden.Title;
|
||||
@@ -208,6 +269,20 @@ public sealed class MetadataEnricher(
|
||||
}
|
||||
|
||||
if (zugeordnet > 0) await db.SaveChangesAsync(ct);
|
||||
|
||||
foreach (var (sieger, verlierer) in zusammenzulegen)
|
||||
{
|
||||
await zusammenlegung.SerieVerschmelzenAsync(sieger, verlierer, ct);
|
||||
}
|
||||
|
||||
if (zusammenzulegen.Count > 0)
|
||||
{
|
||||
await db.SaveChangesAsync(ct);
|
||||
log.LogInformation(
|
||||
"{Anzahl} Serie(n) waren dieselbe unter anderem Namen und wurden zusammengelegt.",
|
||||
zusammenzulegen.Count);
|
||||
}
|
||||
|
||||
return zugeordnet;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user