Implemented caching of title, artist and album of a track as WOBUZZM3U parameters.

wobuzz/player/playlist.py

@@ -5,7 +5,7 @@ import threading
 
 from PyQt6.QtCore import Qt
 from PyQt6.QtWidgets import QAbstractItemView
-from .track import Track
+from .track import Track, TrackMetadata
 
 
 class Playlist:
@@ -38,9 +38,7 @@ class Playlist:
         self.loaded = False
         self.loading = False
 
-        self.path = os.path.expanduser(
-            f"{app.settings.library_path}/playlists/{self.title.replace(" ", "_")}.wbz.m3u"
-        )
+        self.path = self.path_from_title(title)
 
     def clear(self):
         self.sorting: list[Qt.SortOrder] | None = None
@@ -162,6 +160,8 @@ class Playlist:
             lambda: i
         )
 
+        track_metadata = TrackMetadata()  # cached track metadata from WOBUZZM3U
+
         while i < num_lines:
             line = lines[i]
 
@@ -177,16 +177,30 @@ class Playlist:
                     # convert these from strings back to int and bool and append them to the sorting
                     self.sorting.append((int(sorting[0]), sorting[1] == "True"))
 
+                elif line.startswith("#TRACK_TITLE: "):
+                    track_metadata.title = line[14:]
+
+                elif line.startswith("#TRACK_ARTIST: "):
+                    track_metadata.artist = line[15:]
+
+                elif line.startswith("#TRACK_ALBUM: "):
+                    track_metadata.album = line[14:]
+
                 i += 1
 
                 continue
 
-            elif line.startswith("http"):  # filter out urls
+            elif line.startswith("http"):  # ignore urls
                 i += 1
 
                 continue
 
-            self.append_track(Track(self.app, line, cache=i == 0))  # first track is cached
+            track_metadata.path = line
+            track_metadata.add_missing()
+
+            self.append_track(Track(self.app, line, cache=i == 0, metadata=track_metadata))  # first track is cached
+
+            track_metadata = TrackMetadata()  # metadata for next track
 
             i += 1
 
@@ -275,6 +289,11 @@ class Playlist:
             wbz_data += f"#SORT: {sort_column}, {order}\n"
 
         for track in self.tracks:
+            # cache track metadata
+            wbz_data += f"#TRACK_TITLE: {track.metadata.title}\n"
+            wbz_data += f"#TRACK_ARTIST: {track.metadata.artist}\n"
+            wbz_data += f"#TRACK_ALBUM: {track.metadata.album}\n"
+
             wbz_data += f"{track.path}\n"
 
         wbz = open(self.path, "w")
@@ -288,9 +307,7 @@ class Playlist:
         old_title = self.title
         self.title = self.app.utils.unique_name(title, ignore=old_title)
 
-        self.path = os.path.expanduser(
-            f"{self.app.settings.library_path}/playlists/{self.title.replace(" ", "_")}.wbz.m3u"
-        )
+        self.path = self.path_from_title(self.title)
 
         # make sure the playlist is not referenced anymore as the temporary playlist
         if self == self.app.library.temporary_playlist:
@@ -334,3 +351,9 @@ class Playlist:
         if len(self.tracks) > 1:
             return self.tracks[-2]
 
+    def path_from_title(self, title):
+        path = os.path.expanduser(
+            f"{self.app.settings.library_path}/playlists/{title.replace(" ", "_")}.wbz.m3u"
+        )
+
+        return path