Musixmatch provider no longer supports retry

Lambada10 · Oct 24, 2024 · 6116e6c · 6116e6c
1 parent 76f661b
commit 6116e6c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/MusixmatchAPI.kt b/app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/MusixmatchAPI.kt
@@ -45,13 +45,7 @@ class MusixmatchAPI {
         if (response.status.value !in 200..299)
             return null
 
-        val json = json.decodeFromString<MusixmatchSearchResponse>(responseBody)
-
-        val result = try {
-            json[offset]
-        } catch (e: IndexOutOfBoundsException) {
-            return null
-        }
+        val result = json.decodeFromString<MusixmatchSearchResponse>(responseBody)
 
         return SongInfo(
             songName = result.songName,

diff --git a/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/LyricsFetchScreen.kt b/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/LyricsFetchScreen.kt
@@ -175,7 +175,9 @@ fun SharedTransitionScope.LyricsFetchScreen(
                         lyricsFetchState = viewModel.lyricsFetchState,
                         animatedVisibilityScope = animatedVisibilityScope,
                         disableMarquee = viewModel.userSettingsController.disableMarquee,
-                        allowTryingAgain = viewModel.userSettingsController.selectedProvider != Providers.APPLE
+                        allowTryingAgain =
+                            viewModel.userSettingsController.selectedProvider != Providers.APPLE &&
+                            viewModel.userSettingsController.selectedProvider != Providers.MUSIXMATCH
                     )
 
                     is QueryStatus.Failed -> FailedDialogue(