From 752a76eb443c1b78a804b2c391cd5f903f8eca75 Mon Sep 17 00:00:00 2001
From: decarvalhobo <marcio.decarvalho@student.uclouvain.be>
Date: Wed, 4 Dec 2019 19:24:34 +0100
Subject: [PATCH] Usage of drawable instead of remote image + refactor the
 append to an empty playlist by just updating the thumbnail before adding the
 item in it.

---
 .../local/dialog/PlaylistAppendDialog.java       | 16 ++++++----------
 .../local/playlist/LocalPlaylistFragment.java    |  2 +-
 2 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/app/src/main/java/org/schabi/newpipe/local/dialog/PlaylistAppendDialog.java b/app/src/main/java/org/schabi/newpipe/local/dialog/PlaylistAppendDialog.java
index f34e016b7..884fec599 100644
--- a/app/src/main/java/org/schabi/newpipe/local/dialog/PlaylistAppendDialog.java
+++ b/app/src/main/java/org/schabi/newpipe/local/dialog/PlaylistAppendDialog.java
@@ -152,20 +152,16 @@ public final class PlaylistAppendDialog extends PlaylistDialog {
         final Toast successToast = Toast.makeText(getContext(),
                 R.string.playlist_add_stream_success, Toast.LENGTH_SHORT);
 
-        if(playlist.thumbnailUrl.equals("https://i.ytimg.com/")){   
-            playlistDisposables.add(manager.createPlaylist(playlist.name, streams)
-                    .observeOn(AndroidSchedulers.mainThread())
-                    .subscribe(ignored -> successToast.show()));
-            playlistDisposables.add(manager.deletePlaylist(playlist.uid)
-                    .observeOn(AndroidSchedulers.mainThread())
-                    .subscribe(ignored -> successToast.show()));
-        }
-        else {
-            playlistDisposables.add(manager.appendToPlaylist(playlist.uid, streams)
+        if(playlist.thumbnailUrl.equals("drawable://" + R.drawable.dummy_thumbnail_playlist)){
+            playlistDisposables.add(manager.changePlaylistThumbnail(playlist.uid,streams.get(0).getThumbnailUrl())
                     .observeOn(AndroidSchedulers.mainThread())
                     .subscribe(ignored -> successToast.show()));
         }
 
+        playlistDisposables.add(manager.appendToPlaylist(playlist.uid, streams)
+                .observeOn(AndroidSchedulers.mainThread())
+                .subscribe(ignored -> successToast.show()));
+
         getDialog().dismiss();
     }
 }
diff --git a/app/src/main/java/org/schabi/newpipe/local/playlist/LocalPlaylistFragment.java b/app/src/main/java/org/schabi/newpipe/local/playlist/LocalPlaylistFragment.java
index 34278382a..af9ef0037 100644
--- a/app/src/main/java/org/schabi/newpipe/local/playlist/LocalPlaylistFragment.java
+++ b/app/src/main/java/org/schabi/newpipe/local/playlist/LocalPlaylistFragment.java
@@ -419,7 +419,7 @@ public class LocalPlaylistFragment extends BaseLocalListFragment<List<PlaylistSt
         if(!itemListAdapter.getItemsList().isEmpty()){
             newThumbnailUrl = ((PlaylistStreamEntry)itemListAdapter.getItemsList().get(0)).thumbnailUrl;
         }
-        else newThumbnailUrl = "https://i.ytimg.com/";
+        else newThumbnailUrl = "drawable://" + R.drawable.dummy_thumbnail_playlist;
 
         changeThumbnailUrl(newThumbnailUrl);
     }