fix(logos): 404 „logoGone“ beim Löschen gilt als Erfolg
Schlussreview F5. Geht die Antwort einer Löschanfrage verloren und der Nutzer klickt noch einmal auf „Löschen“, antwortet der Dienst mit 404 `logoGone` (das Logo ist ja schon weg). Die Infozeile zeigte dann „Dieses Logo ist nicht (mehr) verfügbar.“ als Fehler, die Kachel blieb stehen. - Dieser Fehler (ApiError mit Status 404 und Schlüssel `logoGone`) gilt wie ein gelungenes Löschen: forgetLogo, die Kachel verschwindet, eine Auswahl wird aufgehoben (onSelect(null)), kein Fehlertext – wie bei „Zurückziehen“ einer schon zurückgezogenen Karte. - Jeder andere Fehler bleibt einer, auch ein 404 mit anderem oder ohne Schlüssel (etwa ein Dienst, der die Anfrage nicht kennt). - Tests in logo-info.test.ts (Hochlader, Admin, Seite verlassen, Fehler, die Fehler bleiben) und im Picker mit Raster und Auswahl. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
1 parent
4aedb1d322
commit
df1cf29d42
3 files changed
+80
-1
No files matched your search
@@ -208,7 +208,13 @@ export function createLogoInfo(host: LogoInfoHost, deps: LogoInfoDeps, signal: A
|
||||
);
|
||||
// Während der Rückfrage kann die Seite verlassen oder die Sitzung gewechselt haben: dann nichts mehr senden.
|
||||
if (!confirmed || signal.aborted || !mayDelete(logo)) return;
|
||||
await deps.deleteLogo(logo.id);
|
||||
try {
|
||||
await deps.deleteLogo(logo.id);
|
||||
} catch (err) {
|
||||
// Das Logo gibt es schon nicht mehr (die Antwort einer ersten Anfrage ging verloren und der Nutzer versucht es noch einmal, oder es
|
||||
// wurde anderswo gelöscht): Ziel erreicht. Nur der Schlüssel `logoGone` zählt, jedes andere 404 ist ein echter Fehler.
|
||||
if (!(err instanceof ApiError && err.code === 404 && err.serverCode === 'logoGone')) throw err;
|
||||
}
|
||||
forgetLogo(logo.id); // auch nach dem Verlassen der Seite: der Zwischenspeicher darf das gelöschte Logo nicht behalten
|
||||
if (signal.aborted) return;
|
||||
host.removeLogo(logo.id);
|
||||
|
||||
Reference in new issue
Block a user