Hermes (Server)andClaude Opus 5
e6d5f1c9ee
Ergebnis einer Durchsicht der App gegen den Massstab "Apple Music". Alle fuenf Punkte waren keine fehlenden Funktionen, sondern Versprechen, die die Oberflaeche gab und der Code nicht hielt. - POST_NOTIFICATIONS fehlte im Manifest. Auf Android 13+ blieb die Wiedergabe-Benachrichtigung dadurch unsichtbar. Das Recht wird erst erfragt, wenn wirklich ein Titel laeuft. - Der Offline-Schalter wurde nirgends gelesen. Er greift jetzt an allen drei Wegen in die Warteschlange; uebersprungene Titel werden benannt. - CoverImage kannte nur file://, und der Server-Browser gab die Cover-Adresse gar nicht erst mit. Server-Titel waren deshalb immer ein graues Notensymbol. - Warteschlange entfernen/umsortieren baute die ganze Queue neu auf: hoerbarer Aussetzer und doppelte Wiedergabe-Zaehlung. Jetzt mutiert nur der betroffene Eintrag, die Buchfuehrung steht vor dem Player. - Das Herz bei Server-Titeln legte einen Favoriten mit leerer Kennung an, der nie in der Liste auftauchte. Es ist jetzt ausgegraut. Nachgebessert aus dem Code-Review: Offline-Meldung an die Stelle geholt, wo sie greifen kann; Queue-Buchfuehrung vor den Player gezogen; Fehlerpfad fuer die Berechtigungs-Abfrage ergaenzt. 360 Tests gruen (vorher 328), flutter analyze ohne Befund, Release-APK gebaut und POST_NOTIFICATIONS darin nachgeprueft. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013xAHJTJM6UUqmjUgk1PUEd
69 lines
2.6 KiB
Dart
69 lines
2.6 KiB
Dart
import 'package:audio_service/audio_service.dart';
|
|
|
|
import '../player/audio_handler.dart';
|
|
import '../services/navidrome_service.dart';
|
|
import 'database.dart';
|
|
|
|
/// [cover] überschreibt das Coverbild des Songs — damit auf Sperrbildschirm
|
|
/// und in der Benachrichtigung dasselbe Bild erscheint wie in der App
|
|
/// (Einstellung "Gleiche Kategorie = gleiches Coverbild").
|
|
MediaItem songToMediaItem(Song s, {String? cover}) {
|
|
final art = cover ?? s.coverPath;
|
|
return MediaItem(
|
|
id: Uri.file(s.path).toString(),
|
|
title: s.title,
|
|
artist: s.artist,
|
|
album: s.album,
|
|
duration: s.durationMs != null ? Duration(milliseconds: s.durationMs!) : null,
|
|
artUri: art != null ? Uri.file(art) : null,
|
|
extras: {'songId': s.id, 'gainDb': s.gainDb},
|
|
);
|
|
}
|
|
|
|
/// Meldung für Titel, die der Offline-Modus aus der Warteschlange genommen
|
|
/// hat. Der Nutzer soll den Unterschied zwischen "nicht da" und "nicht
|
|
/// heruntergeladen" sehen, statt eine stillschweigend kürzere Liste zu hören.
|
|
String offlineHinweis(int uebersprungen) => uebersprungen == 1
|
|
? 'Offline-Modus: 1 Titel übersprungen — nicht heruntergeladen'
|
|
: 'Offline-Modus: $uebersprungen Titel übersprungen — nicht heruntergeladen';
|
|
|
|
/// Meldung für einen einzelnen Titel, den der Offline-Modus ablehnt.
|
|
const nichtOfflineHinweis =
|
|
'Offline-Modus: dieser Titel ist nicht heruntergeladen';
|
|
|
|
/// Ein Titel vom Navidrome-Server als [MediaItem].
|
|
///
|
|
/// Eine Stelle für beide Aufrufer (Album und Künstler im Server-Browser) —
|
|
/// vorher war der Aufbau doppelt von Hand geschrieben, und beide Kopien
|
|
/// vergaßen das Coverbild.
|
|
MediaItem subsonicToMediaItem(SubsonicSong s, NavidromeService nav) {
|
|
final coverId = s.coverId;
|
|
return MediaItem(
|
|
id: nav.streamUrl(s.id).toString(),
|
|
title: s.titel,
|
|
artist: s.kuenstler,
|
|
album: s.album,
|
|
duration: Duration(seconds: s.dauerSekunden),
|
|
// Das Cover liegt am Server, nicht als Datei auf dem Gerät.
|
|
artUri: coverId == null ? null : nav.coverUrl(coverId),
|
|
// Die Server-ID getrennt von der Abspiel-URI mitgeben: nur damit erkennt
|
|
// der Player einen Server-Titel (Cache, Scrobble, Bookmark).
|
|
extras: {'navidromeId': s.id},
|
|
);
|
|
}
|
|
|
|
/// Spielt [songs] ab [startIndex] ab. [coverOf] liefert je Song das
|
|
/// anzuzeigende Cover (siehe [songToMediaItem]). Gibt zurück, wie viele Titel
|
|
/// der Offline-Modus übersprungen hat.
|
|
Future<int> playSongs(
|
|
MeloAudioHandler handler,
|
|
List<Song> songs,
|
|
int startIndex, {
|
|
String? Function(Song)? coverOf,
|
|
}) {
|
|
return handler.loadPlaylist(
|
|
[for (final s in songs) songToMediaItem(s, cover: coverOf?.call(s))],
|
|
startIndex: startIndex,
|
|
);
|
|
}
|