From 501a7af61e8b68f3229c1c75c521d4c34dc22f2e Mon Sep 17 00:00:00 2001 From: CappielloAntonio Date: Sun, 15 Aug 2021 17:07:03 +0200 Subject: [PATCH] Fix song removing in playlist by position --- .../play/ui/fragment/dialog/PlaylistEditorDialog.java | 2 +- .../play/viewmodel/PlaylistEditorViewModel.java | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/src/main/java/com/cappielloantonio/play/ui/fragment/dialog/PlaylistEditorDialog.java b/app/src/main/java/com/cappielloantonio/play/ui/fragment/dialog/PlaylistEditorDialog.java index 1444e8cc..846ef963 100644 --- a/app/src/main/java/com/cappielloantonio/play/ui/fragment/dialog/PlaylistEditorDialog.java +++ b/app/src/main/java/com/cappielloantonio/play/ui/fragment/dialog/PlaylistEditorDialog.java @@ -156,7 +156,7 @@ public class PlaylistEditorDialog extends DialogFragment { @Override public void onSwiped(RecyclerView.ViewHolder viewHolder, int direction) { - playlistEditorViewModel.removeFromPlaylistSongLiveList(playlistDialogSongHorizontalAdapter.getItem(viewHolder.getBindingAdapterPosition())); + playlistEditorViewModel.removeFromPlaylistSongLiveList(viewHolder.getBindingAdapterPosition()); bind.playlistSongRecyclerView.getAdapter().notifyItemRemoved(viewHolder.getBindingAdapterPosition()); } } diff --git a/app/src/main/java/com/cappielloantonio/play/viewmodel/PlaylistEditorViewModel.java b/app/src/main/java/com/cappielloantonio/play/viewmodel/PlaylistEditorViewModel.java index 6dd2bad7..b2a5efda 100644 --- a/app/src/main/java/com/cappielloantonio/play/viewmodel/PlaylistEditorViewModel.java +++ b/app/src/main/java/com/cappielloantonio/play/viewmodel/PlaylistEditorViewModel.java @@ -74,9 +74,9 @@ public class PlaylistEditorViewModel extends AndroidViewModel { return songLiveList; } - public void removeFromPlaylistSongLiveList(Song song) { + public void removeFromPlaylistSongLiveList(int position) { List songs = songLiveList.getValue(); - Objects.requireNonNull(songs).remove(song); + Objects.requireNonNull(songs).remove(position); songLiveList.postValue(songs); }