From ecdae8c6e3d20538609066e360cca6494c917e87 Mon Sep 17 00:00:00 2001
From: Andrew Rabert <ar@nullsum.net>
Date: Sun, 15 Mar 2020 17:06:43 -0400
Subject: [PATCH] Fix switching to playlist on resume Remove a bunch of
 unecessary code

---
 .../fragments/NowPlayingFragment.java         | 55 +------------------
 1 file changed, 3 insertions(+), 52 deletions(-)

diff --git a/app/src/main/java/net/nullsum/audinaut/fragments/NowPlayingFragment.java b/app/src/main/java/net/nullsum/audinaut/fragments/NowPlayingFragment.java
index df3e1a8..878bae2 100644
--- a/app/src/main/java/net/nullsum/audinaut/fragments/NowPlayingFragment.java
+++ b/app/src/main/java/net/nullsum/audinaut/fragments/NowPlayingFragment.java
@@ -65,8 +65,6 @@ import java.util.ArrayList;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.concurrent.Executors;
-import java.util.concurrent.ScheduledExecutorService;
-import java.util.concurrent.ScheduledFuture;
 import java.util.concurrent.TimeUnit;
 
 import static net.nullsum.audinaut.domain.MusicDirectory.Entry;
@@ -105,11 +103,9 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
     private ImageButton repeatButton;
     private View toggleListButton;
 
-    private ScheduledExecutorService executorService;
     private DownloadFile currentPlaying;
     private int swipeDistance;
     private int swipeVelocity;
-    private ScheduledFuture<?> hideControlsFuture;
     private List<DownloadFile> songList;
     private DownloadFileAdapter songListAdapter;
     private boolean seekInProgress = false;
@@ -201,7 +197,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
                     return null;
                 }
             }.execute();
-            setControlsVisible(true);
         });
         previousButton.setOnRepeatListener(() -> changeProgress(true));
 
@@ -214,7 +209,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
                     return true;
                 }
             }.execute();
-            setControlsVisible(true);
         });
         nextButton.setOnRepeatListener(() -> changeProgress(false));
 
@@ -269,12 +263,10 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
                     break;
             }
             updateRepeatButton();
-            setControlsVisible(true);
         });
 
         toggleListButton.setOnClickListener(view -> {
             toggleFullscreenAlbumArt();
-            setControlsVisible(true);
         });
 
         View overlay = rootView.findViewById(R.id.download_overlay_buttons);
@@ -308,7 +300,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
             public void onProgressChanged(final SeekBar seekBar, final int position, final boolean fromUser) {
                 if (fromUser) {
                     positionTextView.setText(Util.formatDuration(position / 1000));
-                    setControlsVisible(true);
                 }
             }
         });
@@ -473,7 +464,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
                     if (controller != null) {
                         SubsonicFragment fragment = new EqualizerFragment();
                         replaceFragment(fragment);
-                        setControlsVisible(true);
 
                         return true;
                     }
@@ -510,19 +500,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
     }
 
     private void onResumeHandlers() {
-        executorService = Executors.newSingleThreadScheduledExecutor();
-        setControlsVisible(true);
-
-        final DownloadService downloadService = getDownloadService();
-        if (downloadService == null || downloadService.getCurrentPlaying() == null || startFlipped) {
-            playlistFlipper.setDisplayedChild(1);
-            startFlipped = false;
-        }
-
-        if (currentPlaying == null && downloadService != null && null == downloadService.getCurrentPlaying()) {
-            setAlbumArt(null, false);
-        }
-
         context.runWhenServiceAvailable(() -> {
             if (primaryFragment) {
                 DownloadService downloadService1 = getDownloadService();
@@ -540,11 +517,9 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
     }
 
     private void onPauseHandlers() {
-        if (executorService != null) {
-            DownloadService downloadService = getDownloadService();
-            if (downloadService != null) {
-                downloadService.removeOnSongChangeListener(this);
-            }
+        DownloadService downloadService = getDownloadService();
+        if (downloadService != null) {
+            downloadService.removeOnSongChangeListener(this);
         }
     }
 
@@ -581,29 +556,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
         return songListAdapter;
     }
 
-    private void scheduleHideControls() {
-        if (hideControlsFuture != null) {
-            hideControlsFuture.cancel(false);
-        }
-
-        final Handler handler = new Handler();
-        Runnable runnable = () -> handler.post(() -> setControlsVisible(false));
-        hideControlsFuture = executorService.schedule(runnable, 3000L, TimeUnit.MILLISECONDS);
-    }
-
-    private void setControlsVisible(boolean visible) {
-        try {
-            long duration = 1700L;
-            FadeOutAnimation.createAndStart(rootView.findViewById(R.id.download_overlay_buttons), !visible, duration);
-
-            if (visible) {
-                scheduleHideControls();
-            }
-        } catch (Exception ignored) {
-
-        }
-    }
-
     // Scroll to current playing/downloading.
     private void scrollToCurrent() {
         if (getDownloadService() == null || songListAdapter == null) {
@@ -693,7 +645,6 @@ public class NowPlayingFragment extends SubsonicFragment implements OnGestureLis
 
     @Override
     public boolean onDown(MotionEvent me) {
-        setControlsVisible(true);
         return false;
     }