From 317908e8c31ad07d1cfeaf0e97d9e95bc9fcc99d Mon Sep 17 00:00:00 2001 From: Christian Soffke Date: Mon, 15 Feb 2021 13:55:30 +0100 Subject: Warn before permanently reshuffling the current playlist Reshuffling a playlist using the WPS context menu for the current playlist is a destructive operation since the previous order of items can't be restored. A warning should be displayed, so that the user can still cancel the operation in case they intended to select another menu item (e.g. one of the harmless "Playing Time" or "Save Current Playlist" items right next to it). Change-Id: Ib9fc379439d155be907872d77cf54d5035bd08a0 --- apps/onplay.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'apps') diff --git a/apps/onplay.c b/apps/onplay.c index 03470a279f..e7f7e589db 100644 --- a/apps/onplay.c +++ b/apps/onplay.c @@ -446,6 +446,8 @@ static bool playing_time(void) /* CONTEXT_WPS playlist options */ static bool shuffle_playlist(void) { + if (!warn_on_pl_erase()) + return false; playlist_sort(NULL, true); playlist_randomise(NULL, current_tick, true); -- cgit v1.2.3