Merge pull request #363 from avdept/album-description

Add overview for albums

Author: avdept

lib/src/data/api/subsonic/subsonic_api.dart

@@ -101,6 +101,11 @@ class SubsonicApi {
     ),
   );
 
+  Future<SubsonicAlbumInfoDTO> getAlbumInfo2(String id) async =>
+      SubsonicAlbumInfoDTO.fromJson(
+        _section(await call('getAlbumInfo2', {'id': id}), 'albumInfo'),
+      );
+
   Future<SubsonicAlbumDTO> getAlbum(String id) async =>
       SubsonicAlbumDTO.fromJson(
         _section(await call('getAlbum', {'id': id}), 'album'),

lib/src/data/backend/subsonic/subsonic_client.dart

@@ -677,7 +677,7 @@ class SubsonicClient implements MediaServerClient {
         ItemKind.song => (await _api.getSong(
           itemId,
         )).toLibraryItem(lyricsAvailable: capabilities.lyrics),
-        ItemKind.album => (await _api.getAlbum(itemId)).toLibraryItem(),
+        ItemKind.album => await _albumWithInfo(itemId),
         ItemKind.artist => await _artistWithInfo(itemId),
         ItemKind.playlist => (await _api.getPlaylist(itemId)).toLibraryItem(),
         ItemKind.genre => LibraryItem(
@@ -696,6 +696,19 @@ class SubsonicClient implements MediaServerClient {
               .toLibraryItem(),
       };
 
+  Future<LibraryItem> _albumWithInfo(String id) async {
+    final results = await Future.wait<Object?>([
+      _api.getAlbum(id),
+      _api
+          .getAlbumInfo2(id)
+          .then<Object?>((info) => info, onError: (_) => null),
+    ]);
+    final album = (results[0]! as SubsonicAlbumDTO).toLibraryItem();
+    final info = results[1] as SubsonicAlbumInfoDTO?;
+    final notes = subsonicPlainText(info?.notes);
+    return notes == null ? album : album.copyWith(overview: notes);
+  }
+
   Future<LibraryItem> _artistWithInfo(String id) async {
     final results = await Future.wait<Object?>([
       _api.getArtist(id),

lib/src/data/dto/subsonic/subsonic_dto.dart

@@ -168,6 +168,18 @@ abstract class SubsonicArtistInfoDTO with _$SubsonicArtistInfoDTO {
       _$SubsonicArtistInfoDTOFromJson(json);
 }
 
+@freezed
+abstract class SubsonicAlbumInfoDTO with _$SubsonicAlbumInfoDTO {
+  const factory SubsonicAlbumInfoDTO({
+    String? notes,
+    String? musicBrainzId,
+    String? lastFmUrl,
+  }) = _SubsonicAlbumInfoDTO;
+
+  factory SubsonicAlbumInfoDTO.fromJson(Map<String, dynamic> json) =>
+      _$SubsonicAlbumInfoDTOFromJson(json);
+}
+
 @freezed
 abstract class SubsonicIndexDTO with _$SubsonicIndexDTO {
   const factory SubsonicIndexDTO({