fix: radio history tile images after new station uuid mapping

ubuntu-flutter-community · Nov 5, 2024 · a1be4b3 · a1be4b3
1 parent ce9cf1b
commit a1be4b3
Show file tree

Hide file tree

Showing 11 changed files with 132 additions and 99 deletions.
diff --git a/lib/main.dart b/lib/main.dart
@@ -34,6 +34,7 @@ import 'player/player_service.dart';
 import 'podcasts/download_model.dart';
 import 'podcasts/podcast_model.dart';
 import 'podcasts/podcast_service.dart';
+import 'radio/online_art_model.dart';
 import 'radio/online_art_service.dart';
 import 'radio/radio_model.dart';
 import 'radio/radio_service.dart';
@@ -182,6 +183,12 @@ void registerServicesAndViewModels({
       )..init(),
       dispose: (s) => s.dispose(),
     )
+    ..registerLazySingleton<OnlineArtModel>(
+      () => OnlineArtModel(
+        onlineArtService: di<OnlineArtService>(),
+      )..init(),
+      dispose: (s) => s.dispose(),
+    )
     ..registerLazySingleton<AppModel>(
       () => AppModel(
         appVersion: version,
@@ -207,7 +214,6 @@ void registerServicesAndViewModels({
     ..registerLazySingleton<RadioModel>(
       () => RadioModel(
         radioService: di<RadioService>(),
-        onlineArtService: di<OnlineArtService>(),
       ),
       dispose: (s) => s.dispose(),
     )

diff --git a/lib/player/player_model.dart b/lib/player/player_model.dart
@@ -123,6 +123,9 @@ class PlayerModel extends SafeChangeNotifier {
 
   int getRadioHistoryLength({String? filter}) =>
       filteredRadioHistory(filter: filter).length;
+  MpvMetaData? getMetadata(String? icyTitle) =>
+      icyTitle == null ? null : _playerService.radioHistory[icyTitle];
+
   Iterable<MapEntry<String, MpvMetaData>> filteredRadioHistory({
     required String? filter,
   }) {

diff --git a/lib/player/player_service.dart b/lib/player/player_service.dart
@@ -81,13 +81,13 @@ class PlayerService {
     }
     if (validHistoryElement) {
       _addRadioHistoryElement(
-        icyTitle: mpvMetaData!.icyTitle.everyWordCapitalized,
+        icyTitle: mpvMetaData!.icyTitle,
         mpvMetaData: mpvMetaData!.copyWith(
           icyName: audio?.title?.trim() ?? _mpvMetaData?.icyName ?? '',
         ),
       );
 
-      await _processParsedIcyTitle(mpvMetaData!.icyTitle.everyWordCapitalized);
+      await _processParsedIcyTitle(mpvMetaData!.icyTitle);
     }
     _propertiesChangedController.add(true);
   }

diff --git a/lib/radio/online_art_model.dart b/lib/radio/online_art_model.dart
@@ -0,0 +1,26 @@
+import 'dart:async';
+
+import 'package:safe_change_notifier/safe_change_notifier.dart';
+
+import 'online_art_service.dart';
+
+class OnlineArtModel extends SafeChangeNotifier {
+  OnlineArtModel({
+    required OnlineArtService onlineArtService,
+  }) : _onlineArtService = onlineArtService;
+
+  final OnlineArtService _onlineArtService;
+  StreamSubscription<bool>? _propertiesChangedSub;
+  String? getCover(String icyTitle) => _onlineArtService.get(icyTitle);
+
+  void init() {
+    _propertiesChangedSub ??=
+        _onlineArtService.propertiesChanged.listen((_) => notifyListeners());
+  }
+
+  @override
+  Future<void> dispose() async {
+    await _propertiesChangedSub?.cancel();
+    super.dispose();
+  }
+}
diff --git a/lib/radio/online_art_service.dart b/lib/radio/online_art_service.dart
@@ -14,10 +14,14 @@ const _kCoverArtArchiveAddress = 'https://coverartarchive.org/release/';
 class OnlineArtService {
   OnlineArtService({required Dio dio}) : _dio = dio;
   final Dio _dio;
+  final _propertiesChangedController = StreamController<bool>.broadcast();
+  Stream<bool> get propertiesChanged => _propertiesChangedController.stream;
+  final _errorController = StreamController<String?>.broadcast();
+  Stream<String?> get error => _errorController.stream;
 
   Future<String?> fetchAlbumArt(String icyTitle) async {
     _errorController.add(null);
-    return get(icyTitle) ??
+    final albumArtUrl = get(icyTitle) ??
         put(
           key: icyTitle,
           url: await compute(
@@ -29,23 +33,24 @@ class OnlineArtService {
             return null;
           }),
         );
+    _propertiesChangedController.add(true);
+
+    return albumArtUrl;
   }
 
-  final _value = <String, String?>{};
+  final _store = <String, String?>{};
 
   String? put({required String key, String? url}) {
-    return _value.containsKey(key)
-        ? _value.update(key, (value) => url)
-        : _value.putIfAbsent(key, () => url);
+    return _store.containsKey(key)
+        ? _store.update(key, (value) => url)
+        : _store.putIfAbsent(key, () => url);
   }
 
-  String? get(String? icyTitle) => icyTitle == null ? null : _value[icyTitle];
-
-  final _errorController = StreamController<String?>.broadcast();
-  Stream<String?> get error => _errorController.stream;
+  String? get(String? icyTitle) => icyTitle == null ? null : _store[icyTitle];
 
   Future<void> dispose() async {
     await _errorController.close();
+    await _propertiesChangedController.close();
   }
 }
 

diff --git a/lib/radio/radio_model.dart b/lib/radio/radio_model.dart
@@ -4,20 +4,13 @@ import 'package:safe_change_notifier/safe_change_notifier.dart';
 
 import '../common/data/audio.dart';
 import '../l10n/l10n.dart';
-import 'online_art_service.dart';
 import 'radio_service.dart';
 
 class RadioModel extends SafeChangeNotifier {
   final RadioService _radioService;
-  final OnlineArtService _onlineArtService;
 
-  RadioModel({
-    required RadioService radioService,
-    required OnlineArtService onlineArtService,
-  })  : _radioService = radioService,
-        _onlineArtService = onlineArtService;
-
-  String? getCover(String icyTitle) => _onlineArtService.get(icyTitle);
+  RadioModel({required RadioService radioService})
+      : _radioService = radioService;
 
   Future<void> clickStation(Audio? station) async {
     if (station?.description != null) {

diff --git a/lib/radio/view/icy_image.dart b/lib/radio/view/icy_image.dart
@@ -1,27 +1,26 @@
 import 'package:flutter/material.dart';
 import 'package:watch_it/watch_it.dart';
 
-import '../../common/data/mpv_meta_data.dart';
 import '../../common/view/icons.dart';
 import '../../common/view/mpv_metadata_dialog.dart';
 import '../../common/view/safe_network_image.dart';
 import '../../constants.dart';
 import '../../l10n/l10n.dart';
 import '../../player/player_model.dart';
-import '../radio_model.dart';
+import '../online_art_model.dart';
 
 class IcyImage extends StatelessWidget with WatchItMixin {
   const IcyImage({
     super.key,
-    required this.mpvMetaData,
+    required this.icyTitle,
     this.height = kAudioTrackWidth,
     this.width = kAudioTrackWidth,
     this.borderRadius,
     this.fallBackWidget,
     this.fit,
   });
 
-  final MpvMetaData mpvMetaData;
+  final String? icyTitle;
 
   final double height, width;
   final BorderRadius? borderRadius;
@@ -31,28 +30,35 @@ class IcyImage extends StatelessWidget with WatchItMixin {
   @override
   Widget build(BuildContext context) {
     final bR = borderRadius ?? BorderRadius.circular(4);
-    watchPropertyValue(
-      (PlayerModel m) => m.remoteImageUrl,
-    );
+    final imageUrl = icyTitle == null
+        ? null
+        : watchPropertyValue(
+            (OnlineArtModel m) => m.getCover(icyTitle!),
+          );
+    // watchPropertyValue((PlayerModel m) => m.remoteImageUrl);
 
     return Tooltip(
       message: context.l10n.metadata,
       child: ClipRRect(
         borderRadius: bR,
         child: InkWell(
           borderRadius: bR,
-          onTap: () => showDialog(
-            context: context,
-            builder: (context) => MpvMetadataDialog(
-              image: di<RadioModel>().getCover(mpvMetaData.icyTitle),
-              mpvMetaData: mpvMetaData,
-            ),
-          ),
+          onTap: () {
+            final metadata = di<PlayerModel>().getMetadata(icyTitle);
+            if (metadata == null) return;
+            showDialog(
+              context: context,
+              builder: (context) => MpvMetadataDialog(
+                image: imageUrl,
+                mpvMetaData: metadata,
+              ),
+            );
+          },
           child: SizedBox(
             height: height,
             width: width,
             child: SafeNetworkImage(
-              url: di<RadioModel>().getCover(mpvMetaData.icyTitle),
+              url: imageUrl,
               fallBackIcon: fallBackWidget ?? Icon(Iconz.radio),
               filterQuality: FilterQuality.medium,
               fit: fit ?? BoxFit.fitHeight,

diff --git a/lib/radio/view/radio_history_list.dart b/lib/radio/view/radio_history_list.dart
@@ -53,12 +53,12 @@ class RadioHistoryList extends StatelessWidget with WatchItMixin {
                   .elementAt(reversedIndex);
               return simpleList
                   ? RadioHistoryTile.simple(
-                      entry: e,
+                      icyTitle: e.key,
                       selected: current?.icyTitle != null &&
                           current?.icyTitle == e.value.icyTitle,
                     )
                   : RadioHistoryTile(
-                      entry: e,
+                      icyTitle: e.key,
                       selected: current?.icyTitle != null &&
                           current?.icyTitle == e.value.icyTitle,
                     );
@@ -77,12 +77,14 @@ class SliverRadioHistoryList extends StatelessWidget with WatchItMixin {
     this.emptyMessage,
     this.padding,
     this.emptyIcon,
+    required this.allowNavigation,
   });
 
   final String? filter;
   final Widget? emptyMessage;
   final Widget? emptyIcon;
   final EdgeInsetsGeometry? padding;
+  final bool allowNavigation;
 
   @override
   Widget build(BuildContext context) {
@@ -109,9 +111,10 @@ class SliverRadioHistoryList extends StatelessWidget with WatchItMixin {
               .filteredRadioHistory(filter: filter)
               .elementAt(reversedIndex);
           return RadioHistoryTile(
-            entry: e,
+            icyTitle: e.key,
             selected: current?.icyTitle != null &&
                 current?.icyTitle == e.value.icyTitle,
+            allowNavigation: allowNavigation,
           );
         },
         childCount: length,