Jeder Titel war seine eigene Dublette, und das seit Monaten
„Metadaten auffrischen" hat an allem, was schon zugeordnet war, schlicht nichts getan. Die Dublettenprüfung schlägt jeden Titel nach, der eine TMDB-Kennung hat — beim Auffrischen also auch den gerade bearbeiteten. Ohne Vergleich auf Gleichheit fand sich jeder als "Dublette seiner selbst", wurde übersprungen und nie fortgeschrieben. Titel, Beschreibung und Bilder blieben für immer so, wie sie beim allerersten Lauf hereinkamen. In der Oberfläche sah das aus, als wirke die Spracheinstellung nicht — und genau so wurde es gemeldet. Gemessen, Ende zu Ende: Serversprache en-US: 101 Dalmatians · The Lion King Serversprache de-DE: 101 Dalmatiner · Der König der Löwen Damit das nie wieder unbemerkt bleibt, weiss der Server jetzt, in welcher Sprache er die Titel hat: eine Spalte MetadataLanguage, gesetzt beim Anreichern, und in der Verwaltung eine Zeile mit der Verteilung. Weicht etwas ab, frischt ein Knopf genau diese Titel auf — nicht den ganzen Bestand, und ohne das Dateisystem noch einmal abzulaufen. Dazu zwei Dinge, die ohne das nicht gingen: ein Titel lässt sich von Hand setzen und wird dabei festgeschrieben wie ein Plakat, mit einem Weg zurück. Und ein Titel, der auf Freigabe wartet, trägt jetzt dasselbe Dreipunktemenü wie eine Kachel — Bilder, Zuordnung, Titel. Ausgerechnet dort fehlte es, obwohl das der Eintrag ist, den man vor der Freigabe geradeziehen will. 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
ce38da1376
commit
d2fbaad871
@@ -621,6 +621,23 @@ export const pruefungSetzen = (bibliothek: string, an: boolean) =>
|
||||
anfrage<void>(`/libraries/${bibliothek}/review`,
|
||||
{ method: 'PUT', body: JSON.stringify({ reviewNewItems: an }) });
|
||||
|
||||
export type Sprachanteil = { sprache: string | null; anzahl: number };
|
||||
export type Sprachverteilung = {
|
||||
serversprache: string;
|
||||
anteile: Sprachanteil[];
|
||||
/** Wie viele Titel nicht in der Serversprache vorliegen. */
|
||||
abweichend: number;
|
||||
};
|
||||
|
||||
/** In welcher Sprache der Bestand vorliegt — die Zahl, die die Frage beendet. */
|
||||
export const sprachverteilung = () =>
|
||||
anfrage<Sprachverteilung>('/settings/metadata/languages');
|
||||
|
||||
/** Frischt genau die Titel auf, die nicht zur Serversprache passen. */
|
||||
export const abweichendeAuffrischen = () =>
|
||||
anfrage<{ started: number; language: string }>(
|
||||
'/settings/metadata/languages/refresh', { method: 'POST' });
|
||||
|
||||
export type Speicherort = {
|
||||
name: string; variable: string; pfad: string;
|
||||
belegt: number; frei: number | null; bilder: number | null;
|
||||
@@ -720,6 +737,15 @@ export const metadatenSetzen = (werte: { apiKey?: string; language?: string }) =
|
||||
export const vorschlaege = (art: 'movies' | 'series', id: string, suche?: string, jahr?: number) =>
|
||||
anfrage<Vorschlag[]>(`/${art}/${id}/matches${abfrage({ query: suche, year: jahr })}`);
|
||||
|
||||
/** Setzt den Titel von Hand und schreibt ihn fest. */
|
||||
export const titelSetzen = (art: 'movies' | 'series', id: string, titel: string) =>
|
||||
anfrage<void>(`/${art}/${id}/title`,
|
||||
{ method: 'PUT', body: JSON.stringify({ title: titel }) });
|
||||
|
||||
/** Gibt den Titel wieder frei — das nächste Auffrischen holt ihn neu. */
|
||||
export const titelFreigeben = (art: 'movies' | 'series', id: string) =>
|
||||
anfrage<void>(`/${art}/${id}/title`, { method: 'DELETE' });
|
||||
|
||||
/** Woher ein Titel stammt — Ordner und ein paar Dateinamen. */
|
||||
export type Herkunft = { ordner: string | null; dateien: number; beispiele: string[] };
|
||||
|
||||
|
||||
@@ -7,6 +7,7 @@ import { icons } from './icons';
|
||||
import { kontextmenue, menueAnZiel, type Menuepunkt } from './menue';
|
||||
import { bilddialog } from './bilder';
|
||||
import { zuordnungsdialog } from './zuordnung';
|
||||
import { titeldialog } from './titel';
|
||||
|
||||
/** Was eine Kachel zeigt — unabhängig davon, woher sie kommt. */
|
||||
export type Kachel = {
|
||||
@@ -267,6 +268,17 @@ function kachelpunkte(
|
||||
icon: 'stift',
|
||||
tun: () => bilddialog(art, k.id, k.titel),
|
||||
});
|
||||
punkte.push({
|
||||
text: 'Titel ändern',
|
||||
icon: 'stift',
|
||||
// **Nicht jeder Titel steht bei TMDB.** Ein Mitschnitt hat gar keinen,
|
||||
// und mancher heisst im Haushalt anders. Bisher blieb nur der Dateiname.
|
||||
tun: () => titeldialog(art, k.id, k.titel, neu => {
|
||||
k.titel = neu;
|
||||
const text = knopf.querySelector('.kacheltext strong');
|
||||
if (text) text.textContent = neu;
|
||||
}),
|
||||
});
|
||||
punkte.push({
|
||||
text: 'Zuordnung korrigieren',
|
||||
icon: 'suche',
|
||||
|
||||
@@ -47,6 +47,8 @@ export function dialog(optionen: {
|
||||
gefaehrlich?: boolean;
|
||||
/** Wird mit den Werten gerufen; wirft sie, bleibt der Dialog offen. */
|
||||
tun: (wert: Dialogwerte) => Promise<void>;
|
||||
/** Ein dritter Knopf links, für den Weg zurück — etwa „Wieder automatisch". */
|
||||
weiterer?: { text: string; tun: () => Promise<void> };
|
||||
}): void {
|
||||
const d = document.createElement('dialog');
|
||||
d.className = 'dlg';
|
||||
@@ -154,6 +156,25 @@ export function dialog(optionen: {
|
||||
abbrechen.onclick = () => d.close();
|
||||
const los = knopf(optionen.bestaetigen,
|
||||
optionen.gefaehrlich ? 'btn btn-gefahr' : 'btn btn-primary');
|
||||
|
||||
// Der dritte Knopf steht links aussen, abgesetzt von Abbrechen und
|
||||
// Bestätigen — er gehört zu keinem von beiden.
|
||||
if (optionen.weiterer) {
|
||||
const dritter = knopf(optionen.weiterer.text, 'btn dlg-weiterer');
|
||||
dritter.onclick = async () => {
|
||||
dritter.disabled = true;
|
||||
try { await optionen.weiterer!.tun(); d.close(); }
|
||||
catch (fehler) {
|
||||
const m = document.createElement('div');
|
||||
m.className = 'fehler';
|
||||
m.textContent = meldung(fehler);
|
||||
fehlerplatz.appendChild(m);
|
||||
dritter.disabled = false;
|
||||
}
|
||||
};
|
||||
fuss.appendChild(dritter);
|
||||
}
|
||||
|
||||
fuss.append(abbrechen, los);
|
||||
|
||||
los.onclick = async () => {
|
||||
|
||||
@@ -1059,6 +1059,10 @@ h2 { font-size: var(--t-abschnitt); font-weight: 700; letter-spacing: -.015em; m
|
||||
* ein Element ohne Höhe kreuzt die Sichtgrenze nie zuverlässig. */
|
||||
.nachlader { height: 1px; }
|
||||
|
||||
/* Der dritte Knopf im Dialogfuss steht links aussen — er gehört weder zu
|
||||
* Abbrechen noch zu Bestätigen. */
|
||||
.dlg-weiterer { margin-right: auto; }
|
||||
|
||||
.bildblock { display: flex; flex-direction: column; gap: var(--s-snug); }
|
||||
.bildblock h3 { margin: 0; font-size: var(--t-gross); font-weight: 600; flex: 1; }
|
||||
|
||||
|
||||
@@ -0,0 +1,42 @@
|
||||
import { titelSetzen, titelFreigeben } from './api';
|
||||
import { dialog } from './dialog';
|
||||
|
||||
/** Den Titel eines Eintrags von Hand setzen.
|
||||
*
|
||||
* **Warum es das braucht.** TMDB kennt nicht jeden Titel so, wie er im
|
||||
* Haushalt heisst, und bei einem Mitschnitt gibt es gar keinen. Ohne diesen
|
||||
* Weg bleibt nur der Dateiname oder ein falscher Treffer — und beides liess
|
||||
* sich bisher nirgends geraderücken.
|
||||
*
|
||||
* **Gesetzt heisst festgeschrieben.** Sonst überschreibt das nächste
|
||||
* „Metadaten auffrischen" die Korrektur, und man tippt sie nach jedem Lauf
|
||||
* neu. Den Weg zurück gibt es trotzdem: der Knopf unten gibt ihn wieder frei.
|
||||
*/
|
||||
export function titeldialog(
|
||||
art: 'movies' | 'series',
|
||||
id: string,
|
||||
jetzt: string,
|
||||
fertig?: (neu: string) => void,
|
||||
): void {
|
||||
dialog({
|
||||
titel: 'Titel ändern',
|
||||
text: 'Ein Titel von Hand bleibt stehen, auch wenn die Metadaten erneut '
|
||||
+ 'geholt werden. „Wieder automatisch" gibt ihn frei — dann holt ihn der '
|
||||
+ 'nächste Lauf wieder von der Metadatenquelle.',
|
||||
felder: [{ name: 'titel', beschriftung: 'Titel', wert: jetzt }],
|
||||
bestaetigen: 'Speichern',
|
||||
tun: async wert => {
|
||||
const neu = wert('titel').trim();
|
||||
if (neu.length === 0) throw new Error('Ein Titel darf nicht leer sein.');
|
||||
await titelSetzen(art, id, neu);
|
||||
fertig?.(neu);
|
||||
},
|
||||
weiterer: {
|
||||
text: 'Wieder automatisch',
|
||||
tun: async () => {
|
||||
await titelFreigeben(art, id);
|
||||
fertig?.(jetzt);
|
||||
},
|
||||
},
|
||||
});
|
||||
}
|
||||
@@ -10,11 +10,16 @@ import {
|
||||
servernameSetzen, systemauskunft,
|
||||
laufendeWiedergaben, serverzustand, bildAdresse,
|
||||
speicherorte, bildspeicherLeeren, unzugeordnet,
|
||||
sprachverteilung, abweichendeAuffrischen,
|
||||
type Bibliothek, type Wiedergabe, ApiFehler,
|
||||
} from '../api';
|
||||
import { esc, dauer, groesse, knopf, fehlerzeile, leer, meldung } from '../util';
|
||||
import { dialog, nachfragen } from '../dialog';
|
||||
import { icon } from '../icons';
|
||||
import { icon, icons } from '../icons';
|
||||
import { kontextmenue, menueAnZiel, type Menuepunkt } from '../menue';
|
||||
import { bilddialog } from '../bilder';
|
||||
import { zuordnungsdialog } from '../zuordnung';
|
||||
import { titeldialog } from '../titel';
|
||||
|
||||
/** Die Verwaltung.
|
||||
*
|
||||
@@ -321,9 +326,30 @@ function pruefbereich(geaendert: () => void): HTMLElement {
|
||||
catch (fehler) { d.appendChild(fehlerzeile(meldung(fehler))); nur.disabled = false; }
|
||||
};
|
||||
|
||||
// **Dasselbe Dreipunktemenü wie auf einer Kachel.**
|
||||
//
|
||||
// Hier stand bisher nur „Veröffentlichen" — ausgerechnet bei dem Titel,
|
||||
// den man vor der Freigabe geradeziehen will. Bilder, Zuordnung und Titel
|
||||
// waren an einem wartenden Eintrag überhaupt nicht erreichbar.
|
||||
const art = e.kind === 'series' ? 'series' : 'movies';
|
||||
const mehr = document.createElement('button');
|
||||
mehr.className = 'btn rund';
|
||||
mehr.innerHTML = icons.punkte;
|
||||
mehr.setAttribute('aria-label', `Mehr zu „${e.title}“`);
|
||||
const punkte = (): Menuepunkt[] => [
|
||||
{ text: 'Titel ändern', icon: 'stift',
|
||||
tun: () => titeldialog(art, e.id, e.title, () => void laden()) },
|
||||
{ text: 'Bilder bearbeiten', icon: 'stift',
|
||||
tun: () => bilddialog(art, e.id, e.title) },
|
||||
{ text: 'Zuordnung korrigieren', icon: 'suche',
|
||||
tun: () => zuordnungsdialog(art, e.id, e.title, e.year, () => void laden()) },
|
||||
];
|
||||
mehr.onclick = () => menueAnZiel(mehr, punkte());
|
||||
kontextmenue(mehr, punkte);
|
||||
|
||||
const oben = document.createElement('div');
|
||||
oben.className = 'knopfreihe';
|
||||
oben.append(haken, bild, text, nur);
|
||||
oben.append(haken, bild, text, mehr, nur);
|
||||
|
||||
if (geplant) {
|
||||
const zurueck = knopf('Zurückziehen', 'btn');
|
||||
@@ -1140,6 +1166,45 @@ function sprachbereich(): HTMLElement {
|
||||
|
||||
// **Die eigene Sprache, getrennt von der des Servers.** Die obere
|
||||
// bestimmt, was geholt wird; diese nur, was man selbst zu sehen bekommt.
|
||||
// **Was tatsächlich vorliegt — nicht was eingestellt ist.**
|
||||
//
|
||||
// Der eigentliche Fehler war nicht, dass Titel englisch blieben, sondern
|
||||
// dass sich nicht feststellen liess, ob sie es sind: nirgends stand, in
|
||||
// welcher Sprache ein Titel geholt wurde. Hier steht es, als Zahl.
|
||||
void sprachverteilung().then(v => {
|
||||
const benennen = (k: string | null) => k === null
|
||||
? 'unbekannt (vor dieser Fassung geholt)'
|
||||
: sprachen.find(x => x.wert === k)?.text ?? k;
|
||||
|
||||
const zeileBestand = zeile(
|
||||
'Bestand',
|
||||
v.anteile.length === 0
|
||||
? 'Noch nichts zugeordnet'
|
||||
: v.anteile.map(a => `${a.anzahl} × ${benennen(a.sprache)}`).join(' · '),
|
||||
v.abweichend === 0
|
||||
? 'Alles liegt in der Serversprache vor.'
|
||||
: `${v.abweichend} Titel liegen in einer anderen Sprache vor. Das Auffrischen `
|
||||
+ 'holt genau diese neu — nicht den ganzen Bestand.');
|
||||
|
||||
if (v.abweichend > 0) {
|
||||
const los = knopf(`${v.abweichend} auffrischen`, 'btn btn-primary');
|
||||
los.onclick = async () => {
|
||||
los.disabled = true;
|
||||
try {
|
||||
await abweichendeAuffrischen();
|
||||
los.textContent = 'Läuft…';
|
||||
} catch (fehler) {
|
||||
zeileBestand.appendChild(fehlerzeile(meldung(fehler)));
|
||||
los.disabled = false;
|
||||
}
|
||||
};
|
||||
zeileBestand.querySelector('.knopfreihe')?.appendChild(los);
|
||||
}
|
||||
|
||||
// Direkt unter die Serversprache, denn dort wird die Frage gestellt.
|
||||
liste.insertBefore(zeileBestand, liste.children[1] ?? null);
|
||||
}).catch(() => { /* Ohne Verteilung fehlt nur die Zeile. */ });
|
||||
|
||||
// **Weicht sie ab, steht das deutlich dabei.**
|
||||
//
|
||||
// Eine eigene Sprache ändert nicht, was der Server holt — nur was man
|
||||
|
||||
Reference in New Issue
Block a user