From 4cfd7cc77d0cf85ba50f3b4edc1bd7dba2053d65 Mon Sep 17 00:00:00 2001
From: Christian Soffke <christian.soffke@gmail.com>
Date: Fri, 13 Oct 2023 19:30:09 +0200
Subject: Track Info [Playlist] field: Add playlist name

For any selected track that is part of a playlist,
additional info about the list is now provided in
the [Playlist] field of the Track Info screen.

1) Asterisk indicates if playlist has been modified.
2) Playlist filename is visible, unless the current
playlist is *not* associated with a file on disk, in
which case the following will be shown instead:

- (Folder)    for unmodified folder playlists.
- (Dynamic)   for playlists that are neither associated
              with a playlist file, nor with a folder.

Change-Id: I9dcf7cbba4ac2e37b23413180a2b2bf4bbe5ee2a
---
 apps/playlist_viewer.c |  2 +-
 apps/screens.c         | 44 ++++++++++++++++++++++++++++++++++++++++----
 apps/screens.h         |  4 ++++
 3 files changed, 45 insertions(+), 5 deletions(-)

diff --git a/apps/playlist_viewer.c b/apps/playlist_viewer.c
index d780bfb7e9..8219aa5feb 100644
--- a/apps/playlist_viewer.c
+++ b/apps/playlist_viewer.c
@@ -523,7 +523,7 @@ static enum pv_onplay_result show_track_info(const struct playlist_entry *curren
     }
 
     return id3_retrieval_successful &&
-            browse_id3(&id3, current_track->index + 1,
+            browse_id3_ex(&id3, viewer.playlist, current_track->index + 1,
             viewer.num_tracks, NULL, 1) ? PV_ONPLAY_USB : PV_ONPLAY_UNCHANGED;
 }
 
diff --git a/apps/screens.c b/apps/screens.c
index 50b7711fa7..c11e052bce 100644
--- a/apps/screens.c
+++ b/apps/screens.c
@@ -397,6 +397,7 @@ struct id3view_info {
     struct tm *modified;
     int track_ct;
     int count;
+    struct playlist_info *playlist;
     int playlist_display_index;
     int playlist_amount;
     int info_id[ARRAYLEN(id3_headers)];
@@ -496,6 +497,7 @@ static const char * id3_get_or_speak_info(int selected_item, void* data,
     const unsigned char * const *unit;
     unsigned int unit_ct;
     unsigned long length;
+    bool pl_modified;
     struct tm *tm = info->modified;
     int info_no=selected_item/2;
     if(!(selected_item%2))
@@ -618,14 +620,39 @@ static const char * id3_get_or_speak_info(int selected_item, void* data,
             case LANG_ID3_PLAYLIST:
                 if (info->playlist_display_index == 0 || info->playlist_amount == 0 )
                     return NULL;
-                snprintf(buffer, buffer_len, "%d/%d",
-                         info->playlist_display_index, info->playlist_amount);
-                val=buffer;
+
+                pl_modified = playlist_modified(info->playlist);
+
+                snprintf(buffer, buffer_len, "%d/%d%s",
+                         info->playlist_display_index, info->playlist_amount,
+                         pl_modified ? "* " :"  ");
+                val = buffer;
+                size_t prefix_len = strlen(buffer);
+                buffer += prefix_len;
+                buffer_len -= prefix_len;
+
+                if (info->playlist)
+                     playlist_name(info->playlist, buffer, buffer_len);
+                else
+                {
+                    if (playlist_allow_dirplay(NULL))
+                        strmemccpy(buffer, "(Folder)", buffer_len);
+                    else if (playlist_dynamic_only())
+                        strmemccpy(buffer, "(Dynamic)", buffer_len);
+                    else
+                        playlist_name(NULL, buffer, buffer_len);
+                }
+
                 if(say_it)
                 {
                     talk_number(info->playlist_display_index, true);
                     talk_id(VOICE_OF, true);
                     talk_number(info->playlist_amount, true);
+
+                    if (pl_modified)
+                        talk_spell("Modified", true);
+                    if (buffer) /* playlist name */
+                        talk_spell(buffer, true);
                 }
                 break;
             case LANG_FORMAT:
@@ -750,7 +777,8 @@ static int id3_speak_item(int selected_item, void* data)
 /* Note: If track_ct > 1, filesize value will be treated as
  * KiB (instead of Bytes), and length as s instead of ms.
  */
-bool browse_id3(struct mp3entry *id3, int playlist_display_index, int playlist_amount,
+bool browse_id3_ex(struct mp3entry *id3, struct playlist_info *playlist,
+                int playlist_display_index, int playlist_amount,
                 struct tm *modified, int track_ct)
 {
     struct gui_synclist id3_lists;
@@ -760,6 +788,7 @@ bool browse_id3(struct mp3entry *id3, int playlist_display_index, int playlist_a
     info.id3 = id3;
     info.modified = modified;
     info.track_ct = track_ct;
+    info.playlist = playlist;
     info.playlist_amount = playlist_amount;
     bool ret = false;
     int curr_activity = get_current_activity();
@@ -821,6 +850,13 @@ refresh_info:
     return ret;
 }
 
+bool browse_id3(struct mp3entry *id3, int playlist_display_index, int playlist_amount,
+                struct tm *modified, int track_ct)
+{
+    return browse_id3_ex(id3, NULL, playlist_display_index, playlist_amount,
+                         modified, track_ct);
+}
+
 static const char* runtime_get_data(int selected_item, void* data,
                                     char* buffer, size_t buffer_len)
 {
diff --git a/apps/screens.h b/apps/screens.h
index bcd06655ad..1cfd7dcdcf 100644
--- a/apps/screens.h
+++ b/apps/screens.h
@@ -24,6 +24,7 @@
 #include "config.h"
 #include "timefuncs.h"
 #include "metadata.h"
+#include "playlist.h"
 
 struct screen;
 
@@ -41,6 +42,9 @@ bool set_time_screen(const char* title, struct tm *tm, bool set_date);
 
 bool browse_id3(struct mp3entry *id3, int playlist_display_index, int playlist_amount,
                 struct tm *modified, int track_ct);
+bool browse_id3_ex(struct mp3entry *id3, struct playlist_info *playlist,
+                int playlist_display_index, int playlist_amount,
+                struct tm *modified, int track_ct);
 int  view_runtime(void);
 
 #ifdef HAVE_TOUCHSCREEN
-- 
cgit v1.2.3