fix: selection preset quality returning null

android/.gitignore

@@ -11,3 +11,4 @@ GeneratedPluginRegistrant.java
 key.properties
 **/*.keystore
 **/*.jks
+.kotlin

lib/models/metadata/audio_source.dart

@@ -44,7 +44,7 @@ class SpotubeAudioLossyContainerQuality
 
   @override
   toString() {
-    return "${oneOptionalDecimalFormatter.format(bitrate)}kbps";
+    return "${oneOptionalDecimalFormatter.format(bitrate / 1000)}kbps";
   }
 }
 

lib/modules/metadata_plugins/installed_plugin.dart

@@ -52,9 +52,10 @@ class MetadataInstalledPluginItem extends HookConsumerWidget {
         ref.watch(metadataPluginAuthenticatedProvider);
     final isAudioSourceAuthenticatedSnapshot =
         ref.watch(audioSourcePluginAuthenticatedProvider);
-    final isAuthenticated =
+    final isAuthenticated = (isDefaultMetadata &&
-        isMetadataAuthenticatedSnapshot.asData?.value == true ||
+            isMetadataAuthenticatedSnapshot.asData?.value == true) ||
-            isAudioSourceAuthenticatedSnapshot.asData?.value == true;
+        (isDefaultAudioSource &&
+            isAudioSourceAuthenticatedSnapshot.asData?.value == true);
 
     final metadataUpdateAvailable =
         ref.watch(metadataPluginUpdateCheckerProvider);

lib/provider/metadata_plugin/audio_source/quality_presets.dart

@@ -6,6 +6,7 @@ import 'package:shared_preferences/shared_preferences.dart';
 import 'package:spotube/models/metadata/metadata.dart';
 import 'package:spotube/provider/metadata_plugin/metadata_plugin_provider.dart';
 import 'package:spotube/services/audio_player/audio_player.dart';
+import 'package:spotube/services/metadata/metadata.dart';
 
 part 'quality_presets.g.dart';
 part 'quality_presets.freezed.dart';
@@ -28,9 +29,13 @@ class AudioSourceAvailableQualityPresetsNotifier
     extends Notifier<AudioSourcePresetsState> {
   @override
   build() {
-    ref.watch(audioSourcePluginProvider);
+    final audioSourceSnapshot = ref.watch(audioSourcePluginProvider);
+    final audioSourceConfigSnapshot = ref.watch(
+      metadataPluginsProvider.select((data) =>
+          data.whenData((value) => value.defaultAudioSourcePluginConfig)),
+    );
 
-    _initialize();
+    _initialize(audioSourceSnapshot, audioSourceConfigSnapshot);
 
     listenSelf((previous, next) {
       final isNewLossless =
@@ -49,12 +54,12 @@ class AudioSourceAvailableQualityPresetsNotifier
     return AudioSourcePresetsState();
   }
 
-  void _initialize() async {
+  void _initialize(
-    final audioSource = await ref.read(audioSourcePluginProvider.future);
+    AsyncValue<MetadataPlugin?> audioSourceSnapshot,
-    final audioSourceConfig = await ref.read(
+    AsyncValue<PluginConfiguration?> audioSourceConfigSnapshot,
-      metadataPluginsProvider
+  ) async {
-          .selectAsync((data) => data.defaultAudioSourcePluginConfig),
+    audioSourceConfigSnapshot.whenData((audioSourceConfig) {
-    );
+      audioSourceSnapshot.whenData((audioSource) async {
         if (audioSource == null || audioSourceConfig == null) {
           throw Exception("Dude wat?");
         }
@@ -63,12 +68,18 @@ class AudioSourceAvailableQualityPresetsNotifier
             preferences.getString("audioSourceState-${audioSourceConfig.slug}");
 
         if (persistedStateStr != null) {
-      state = AudioSourcePresetsState.fromJson(jsonDecode(persistedStateStr));
+          state =
+              AudioSourcePresetsState.fromJson(jsonDecode(persistedStateStr))
+                  .copyWith(
+            presets: audioSource.audioSource.supportedPresets,
+          );
         } else {
           state = AudioSourcePresetsState(
             presets: audioSource.audioSource.supportedPresets,
           );
         }
+      });
+    });
   }
 
   void setSelectedStreamingContainerIndex(int index) {

lib/provider/metadata_plugin/core/auth.dart

@@ -65,6 +65,6 @@ class AudioSourcePluginAuthenticatedNotifier extends AsyncNotifier<bool> {
 }
 
 final audioSourcePluginAuthenticatedProvider =
-    AsyncNotifierProvider<MetadataPluginAuthenticatedNotifier, bool>(
+    AsyncNotifierProvider<AudioSourcePluginAuthenticatedNotifier, bool>(
-  MetadataPluginAuthenticatedNotifier.new,
+  AudioSourcePluginAuthenticatedNotifier.new,
 );

lib/provider/server/routes/playback.dart

@@ -48,7 +48,7 @@ class ServerPlaybackRoutes {
     return join(
       await UserPreferencesNotifier.getMusicCacheDir(),
       ServiceUtils.sanitizeFilename(
-        '${track.query.name} - ${track.query.artists.join(",")} (${track.info.id}).${track.qualityPreset!.name}',
+        '${track.query.name} - ${track.query.artists.map((d) => d.name).join(",")} (${track.info.id}).${track.qualityPreset!.name}',
       ),
     );
   }
@@ -288,7 +288,9 @@ class ServerPlaybackRoutes {
           imageBytes: imageBytes,
           fileLength: fileLength,
         ),
-      );
+      ).catchError((e, stackTrace) {
+        AppLogger.reportError(e, stackTrace);
+      });
     }
 
     return (bytes: bytes, response: res);

lib/services/sourced_track/sourced_track.dart

@@ -306,6 +306,8 @@ class SourcedTrack extends BasicSourcedTrack {
     SpotubeAudioSourceContainerPreset preset,
     int qualityIndex,
   ) {
+    if (sources.isEmpty) return null;
+
     final quality = preset.qualities[qualityIndex];
 
     final exactMatch = sources.firstWhereOrNull(
@@ -317,7 +319,7 @@ class SourcedTrack extends BasicSourcedTrack {
               source.bitDepth == quality.bitDepth;
         } else {
           return source.bitrate ==
-              (preset as SpotubeAudioLossyContainerQuality).bitrate;
+              (quality as SpotubeAudioLossyContainerQuality).bitrate;
         }
       },
     );
@@ -326,45 +328,24 @@ class SourcedTrack extends BasicSourcedTrack {
       return exactMatch;
     }
 
-    // Find the closest to preset
+    // Find the preset with closest quality to the supplied quality
-    SpotubeAudioSourceStreamObject? closest;
+    return sources.where((source) {
-    for (final source in sources) {
+      return source.container == preset.name;
-      if (source.container != preset.name) continue;
+    }).reduce((prev, curr) {
-
+      if (quality is SpotubeAudioLosslessContainerQuality) {
-      if (quality case SpotubeAudioLosslessContainerQuality()) {
+        final prevDiff = ((prev.sampleRate ?? 0) - quality.sampleRate).abs() +
-        final sourceBps = (source.bitDepth ?? 0) * (source.sampleRate ?? 0);
+            ((prev.bitDepth ?? 0) - quality.bitDepth).abs();
-        final qualityBps = quality.bitDepth * quality.sampleRate;
+        final currDiff = ((curr.sampleRate ?? 0) - quality.sampleRate).abs() +
-        final closestBps =
+            ((curr.bitDepth ?? 0) - quality.bitDepth).abs();
-            (closest?.bitDepth ?? 0) * (closest?.sampleRate ?? 0);
+        return currDiff < prevDiff ? curr : prev;
-
-        if (sourceBps == qualityBps) {
-          closest = source;
-          break;
-        }
-        final closestDiff = (closestBps - qualityBps).abs();
-        final sourceDiff = (sourceBps - qualityBps).abs();
-
-        if (sourceDiff < closestDiff) {
-          closest = source;
-        }
       } else {
-        final presetBitrate =
+        final prevDiff = ((prev.bitrate ?? 0) -
-            (preset as SpotubeAudioLossyContainerQuality).bitrate;
+                (quality as SpotubeAudioLossyContainerQuality).bitrate)
-        if (presetBitrate == source.bitrate) {
+            .abs();
-          closest = source;
+        final currDiff = ((curr.bitrate ?? 0) - quality.bitrate).abs();
-          break;
+        return currDiff < prevDiff ? curr : prev;
       }
-
+    });
-        final closestDiff = (closest?.bitrate ?? 0) - presetBitrate;
-        final sourceDiff = (source.bitrate ?? 0) - presetBitrate;
-
-        if (sourceDiff < closestDiff) {
-          closest = source;
-        }
-      }
-    }
-
-    return closest;
   }
 
   String? getUrlOfQuality(