From 6589cd44eb43596077bc8813b8b9a62bb26db361 Mon Sep 17 00:00:00 2001
From: Alexander Capehart <alex@oxycblt.org>
Date: Fri, 21 Feb 2025 11:31:09 -0700
Subject: [PATCH] Revert "detail: fix up navigation"

This reverts commit bfdccd3ba53a26fb5f91d46a567b9c2aa537c2de.
---
 .../java/org/oxycblt/auxio/detail/AlbumDetailFragment.kt  | 2 +-
 .../java/org/oxycblt/auxio/detail/ArtistDetailFragment.kt | 2 +-
 .../main/java/org/oxycblt/auxio/detail/DetailFragment.kt  | 8 --------
 .../java/org/oxycblt/auxio/detail/GenreDetailFragment.kt  | 2 +-
 .../org/oxycblt/auxio/detail/PlaylistDetailFragment.kt    | 3 ++-
 5 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/app/src/main/java/org/oxycblt/auxio/detail/AlbumDetailFragment.kt b/app/src/main/java/org/oxycblt/auxio/detail/AlbumDetailFragment.kt
index 835d9d44e..48785f276 100644
--- a/app/src/main/java/org/oxycblt/auxio/detail/AlbumDetailFragment.kt
+++ b/app/src/main/java/org/oxycblt/auxio/detail/AlbumDetailFragment.kt
@@ -106,7 +106,7 @@ class AlbumDetailFragment : DetailFragment<Album, Song>() {
     private fun updateAlbum(album: Album?) {
         if (album == null) {
             L.d("No album to show, navigating away")
-            exit()
+            findNavController().navigateUp()
             return
         }
 
diff --git a/app/src/main/java/org/oxycblt/auxio/detail/ArtistDetailFragment.kt b/app/src/main/java/org/oxycblt/auxio/detail/ArtistDetailFragment.kt
index b57705ac3..f2db3bab5 100644
--- a/app/src/main/java/org/oxycblt/auxio/detail/ArtistDetailFragment.kt
+++ b/app/src/main/java/org/oxycblt/auxio/detail/ArtistDetailFragment.kt
@@ -113,7 +113,7 @@ class ArtistDetailFragment : DetailFragment<Artist, Music>() {
     private fun updateArtist(artist: Artist?) {
         if (artist == null) {
             L.d("No artist to show, navigating away")
-            exit()
+            findNavController().navigateUp()
             return
         }
         val binding = requireBinding()
diff --git a/app/src/main/java/org/oxycblt/auxio/detail/DetailFragment.kt b/app/src/main/java/org/oxycblt/auxio/detail/DetailFragment.kt
index ca41a753a..906b166cc 100644
--- a/app/src/main/java/org/oxycblt/auxio/detail/DetailFragment.kt
+++ b/app/src/main/java/org/oxycblt/auxio/detail/DetailFragment.kt
@@ -22,7 +22,6 @@ import android.os.Bundle
 import android.view.LayoutInflater
 import androidx.fragment.app.activityViewModels
 import androidx.navigation.findNavController
-import androidx.navigation.fragment.findNavController
 import androidx.recyclerview.widget.GridLayoutManager
 import com.google.android.material.appbar.AppBarLayout
 import com.google.android.material.transition.MaterialSharedAxis
@@ -130,11 +129,4 @@ abstract class DetailFragment<P : MusicParent, C : Music> :
     }
 
     abstract fun onOpenParentMenu()
-
-    protected fun exit() {
-        // For some insane and absurd reason we wont actually navigate away in detail views
-        // until this is called twice.
-        findNavController().navigateUp()
-        findNavController().navigateUp()
-    }
 }
diff --git a/app/src/main/java/org/oxycblt/auxio/detail/GenreDetailFragment.kt b/app/src/main/java/org/oxycblt/auxio/detail/GenreDetailFragment.kt
index 98dd405c7..fdc40cc17 100644
--- a/app/src/main/java/org/oxycblt/auxio/detail/GenreDetailFragment.kt
+++ b/app/src/main/java/org/oxycblt/auxio/detail/GenreDetailFragment.kt
@@ -112,7 +112,7 @@ class GenreDetailFragment : DetailFragment<Genre, Music>() {
     private fun updateGenre(genre: Genre?) {
         if (genre == null) {
             L.d("No genre to show, navigating away")
-            exit()
+            findNavController().navigateUp()
             return
         }
         val binding = requireBinding()
diff --git a/app/src/main/java/org/oxycblt/auxio/detail/PlaylistDetailFragment.kt b/app/src/main/java/org/oxycblt/auxio/detail/PlaylistDetailFragment.kt
index cbad1e7d4..b29ca2bd9 100644
--- a/app/src/main/java/org/oxycblt/auxio/detail/PlaylistDetailFragment.kt
+++ b/app/src/main/java/org/oxycblt/auxio/detail/PlaylistDetailFragment.kt
@@ -184,7 +184,8 @@ class PlaylistDetailFragment :
 
     private fun updatePlaylist(playlist: Playlist?, editedPlaylist: List<Song>?) {
         if (playlist == null) {
-            exit()
+            // Playlist we were showing no longer exists.
+            findNavController().navigateUp()
             return
         }
         val binding = requireBinding()