Fix playlist reload

app/src/main/java/code/name/monkey/retromusic/dialogs/CreatePlaylistDialog.kt

@@ -68,17 +68,8 @@ class CreatePlaylistDialog : DialogFragment() {
             ) { _, _ ->
                 val playlistName = playlistView.text.toString()
                 if (!TextUtils.isEmpty(playlistName)) {
-                    lifecycleScope.launch(Dispatchers.IO) {
+                    libraryViewModel.addToPlaylist(playlistName, songs)
-                        if (libraryViewModel.checkPlaylistExists(playlistName).isEmpty()) {
+
-                            val playlistId: Long =
-                                libraryViewModel.createPlaylist(PlaylistEntity(playlistName = playlistName))
-                            libraryViewModel.insertSongs(songs.map { it.toSongEntity(playlistId) })
-                            libraryViewModel.forceReload(Playlists)
-                        } else {
-                            Toast.makeText(requireContext(), "Playlist exists", Toast.LENGTH_SHORT)
-                                .show()
-                        }
-                    }
                 } else {
                     playlistContainer.error = "Playlist is can't be empty"
                 }

app/src/main/java/code/name/monkey/retromusic/fragments/LibraryViewModel.kt

@@ -14,16 +14,32 @@
  */
 package code.name.monkey.retromusic.fragments
 
-import androidx.lifecycle.*
+import android.widget.Toast
+import androidx.lifecycle.LiveData
+import androidx.lifecycle.MutableLiveData
+import androidx.lifecycle.ViewModel
+import androidx.lifecycle.liveData
+import androidx.lifecycle.viewModelScope
+import code.name.monkey.retromusic.App
 import code.name.monkey.retromusic.RECENT_ALBUMS
 import code.name.monkey.retromusic.RECENT_ARTISTS
 import code.name.monkey.retromusic.TOP_ALBUMS
 import code.name.monkey.retromusic.TOP_ARTISTS
-import code.name.monkey.retromusic.db.*
+import code.name.monkey.retromusic.db.PlaylistEntity
+import code.name.monkey.retromusic.db.PlaylistWithSongs
+import code.name.monkey.retromusic.db.SongEntity
+import code.name.monkey.retromusic.db.toSong
+import code.name.monkey.retromusic.db.toSongEntity
 import code.name.monkey.retromusic.fragments.ReloadType.*
 import code.name.monkey.retromusic.helper.MusicPlayerRemote
 import code.name.monkey.retromusic.interfaces.IMusicServiceEventListener
-import code.name.monkey.retromusic.model.*
+import code.name.monkey.retromusic.model.Album
+import code.name.monkey.retromusic.model.Artist
+import code.name.monkey.retromusic.model.Contributor
+import code.name.monkey.retromusic.model.Genre
+import code.name.monkey.retromusic.model.Home
+import code.name.monkey.retromusic.model.Playlist
+import code.name.monkey.retromusic.model.Song
 import code.name.monkey.retromusic.repository.RealRepository
 import code.name.monkey.retromusic.state.NowPlayingPanelState
 import code.name.monkey.retromusic.util.PreferenceUtil
@@ -226,10 +242,10 @@ class LibraryViewModel(
     suspend fun removeSongFromPlaylist(songEntity: SongEntity) =
         repository.removeSongFromPlaylist(songEntity)
 
-    suspend fun checkPlaylistExists(playlistName: String): List<PlaylistEntity> =
+    private suspend fun checkPlaylistExists(playlistName: String): List<PlaylistEntity> =
         repository.checkPlaylistExists(playlistName)
 
-    suspend fun createPlaylist(playlistEntity: PlaylistEntity): Long =
+    private suspend fun createPlaylist(playlistEntity: PlaylistEntity): Long =
         repository.createPlaylist(playlistEntity)
 
     fun importPlaylists() = viewModelScope.launch(IO) {
@@ -303,6 +319,29 @@ class LibraryViewModel(
             searchResults.postValue(emptyList())
         }
     }
+
+    fun addToPlaylist(playlistName: String, songs: List<Song>) {
+        viewModelScope.launch(IO) {
+            val playlists = checkPlaylistExists(playlistName)
+            if (playlists.isEmpty()) {
+                val playlistId: Long = createPlaylist(PlaylistEntity(playlistName = playlistName))
+                insertSongs(songs.map { it.toSongEntity(playlistId) })
+                forceReload(Playlists)
+            } else {
+                val playlist = playlists.firstOrNull()
+                if (playlist != null) {
+                    insertSongs(songs.map {
+                        it.toSongEntity(playListId = playlist.playListId)
+                    })
+                }
+                Toast.makeText(
+                    App.getContext(),
+                    "Adding songs to $playlistName",
+                    Toast.LENGTH_SHORT
+                ).show()
+            }
+        }
+    }
 }
 
 enum class ReloadType {

app/src/main/res/layout-xlarge-land/fragment_blur.xml

@@ -16,54 +16,57 @@
         android:scaleType="centerCrop"
         app:srcCompat="@color/black_color" />
 
-
     <View
         android:layout_width="match_parent"
         android:layout_height="match_parent"
         android:background="@drawable/shadow_up" />
 
-    <LinearLayout
+    <FrameLayout
-        android:id="@+id/playerContent"
+        android:id="@+id/statusBarContainer"
-        android:layout_width="400dp"
+        android:layout_width="match_parent"
-        android:layout_height="0dp"
+        android:layout_height="wrap_content"
-        android:layout_above="@id/playerToolbar"
-        android:layout_gravity="center_horizontal"
-        android:layout_marginStart="8dp"
-        android:layout_marginTop="8dp"
-        android:orientation="vertical"
-        app:layout_constraintBottom_toTopOf="@+id/playerToolbar"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent">
 
-        <code.name.monkey.retromusic.views.WidthFitSquareLayout
+        <include layout="@layout/status_bar" />
-            android:id="@+id/album_cover_container"
+    </FrameLayout>
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:layout_gravity="center_horizontal"
-            android:layout_weight="0">
 
-            <fragment
+    <FrameLayout
-                android:id="@+id/playerAlbumCoverFragment"
+        android:id="@+id/albumCoverContainer"
-                android:name="code.name.monkey.retromusic.fragments.player.PlayerAlbumCoverFragment"
+        android:layout_width="0dp"
-                android:layout_width="match_parent"
+        android:layout_height="0dp"
-                android:layout_height="match_parent"
+        android:layout_gravity="center_horizontal"
-                tools:layout="@layout/fragment_album_cover" />
+        android:layout_weight="0"
-        </code.name.monkey.retromusic.views.WidthFitSquareLayout>
+        app:layout_constraintBottom_toTopOf="@id/playerContent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@id/statusBarContainer">
 
-        <FrameLayout
+        <fragment
+            android:id="@+id/playerAlbumCoverFragment"
+            android:name="code.name.monkey.retromusic.fragments.player.PlayerAlbumCoverFragment"
             android:layout_width="match_parent"
             android:layout_height="match_parent"
-            android:layout_weight="1"
+            tools:layout="@layout/fragment_album_cover" />
-            tools:background="@color/md_white_1000">
+    </FrameLayout>
 
-            <fragment
+    <FrameLayout
-                android:id="@+id/playbackControlsFragment"
+        android:id="@+id/playerContent"
-                android:name="code.name.monkey.retromusic.fragments.player.blur.BlurPlaybackControlsFragment"
+        android:layout_width="400dp"
-                android:layout_width="match_parent"
+        android:layout_height="0dp"
-                android:layout_height="match_parent" />
+        android:layout_above="@id/playerToolbar"
-        </FrameLayout>
+        app:layout_constraintBottom_toTopOf="@+id/playerToolbar"
-    </LinearLayout>
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@id/albumCoverContainer">
+
+        <fragment
+            android:id="@+id/playbackControlsFragment"
+            android:name="code.name.monkey.retromusic.fragments.player.blur.BlurPlaybackControlsFragment"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent" />
+    </FrameLayout>
 
 
     <com.google.android.material.appbar.MaterialToolbar

app/src/main/res/values/styles.xml

@@ -253,9 +253,15 @@
         <item name="strokeColor">?android:attr/colorButtonNormal</item>
         <item name="cardElevation">0dp</item>
     </style>
+
     <!-- This will set the fade in animation on all your activities by default -->
     <style name="WindowAnimationTransition">
         <item name="android:windowEnterAnimation">@android:anim/fade_in</item>
         <item name="android:windowExitAnimation">@android:anim/fade_out</item>
     </style>
+
+    <style name="circleImageView" parent="">
+        <item name="cornerFamily">rounded</item>
+        <item name="cornerSize">5%</item>
+    </style>
 </resources>

build.gradle

@@ -7,11 +7,10 @@ buildscript {
         google()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:4.0.1'
+        classpath 'com.android.tools.build:gradle:4.0.2'
         classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
         def nav_version = "2.3.0"
         classpath "androidx.navigation:navigation-safe-args-gradle-plugin:$nav_version"
-        //noinspection GradleDependency
         classpath "com.diffplug.spotless:spotless-plugin-gradle:4.5.1"
     }
 }