Fix crash when shuffling on device with no songs. Close naman14#80 (naman14#81)

Author: iamvukasin

app/src/main/java/com/naman14/timberx/ui/fragments/songs/SongsFragment.kt

@@ -20,6 +20,7 @@ import android.view.View
 import android.view.ViewGroup
 import androidx.recyclerview.widget.LinearLayoutManager
 import com.afollestad.rxkprefs.Pref
+import com.google.android.material.snackbar.Snackbar
 import com.naman14.timberx.PREF_SONG_SORT_ORDER
 import com.naman14.timberx.R
 import com.naman14.timberx.constants.SongSortOrder
@@ -93,7 +94,13 @@ class SongsFragment : MediaItemFragment() {
         override fun shuffleAll() {
             songsAdapter.songs.shuffled().apply {
                 val extras = getExtraBundle(toSongIds(), getString(R.string.all_songs))
-                mainViewModel.mediaItemClicked(this[0], extras)
+
+                if (this.isEmpty()) {
+                    Snackbar.make(recyclerView, R.string.shuffle_no_songs_error, Snackbar.LENGTH_SHORT)
+                            .show()
+                } else {
+                    mainViewModel.mediaItemClicked(this[0], extras)
+                }
             }
         }
 

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

@@ -44,6 +44,7 @@
     <string name="no_song_next">No song up next</string>
 
     <string name="play_song_error">Unable to play song</string>
+    <string name="shuffle_no_songs_error">There\'s no songs to shuffle</string>
     <string name="new_playlist">New Playlist</string>
     <string name="lyrics">Lyrics</string>
     <string name="up_next">Up Next</string>