From a0a6a8d15b32bf4f95dda7c35934f8aaad8478a5 Mon Sep 17 00:00:00 2001 From: Robert Honz Date: Tue, 12 Aug 2025 10:18:20 +0200 Subject: [PATCH] refactor(helper): improve ID format handling logic MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Refactors the ID formatting logic to extend support for `video_id` and improve clarity. - ✨ **Feature:** Adds support for `video_id` in ID formatting logic. - ♻️ **Refactor:** Updates conditional checks for better readability. - 🔧 **Metadata:** Updates `poetry.lock` content hash. These changes enhance functionality and maintainability by accommodating additional media types. --- poetry.lock | 2 +- tidal_dl_ng/helper/path.py | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/poetry.lock b/poetry.lock index 19a650a..d8927a1 100644 --- a/poetry.lock +++ b/poetry.lock @@ -2358,4 +2358,4 @@ gui = ["pyqtdarktheme-fork", "pyside6"] [metadata] lock-version = "2.1" python-versions = ">=3.12,<3.13" -content-hash = "e2545be9d3073f35d9f926d7c45b97e2e53f66447132e0f4eeb2330cb2291246" +content-hash = "e27796fbb2cf94075e1304607b89c746b9e2f4a4cda8edb216a9025a8d6fc4fa" diff --git a/tidal_dl_ng/helper/path.py b/tidal_dl_ng/helper/path.py index 12ffdce..0f7de31 100644 --- a/tidal_dl_ng/helper/path.py +++ b/tidal_dl_ng/helper/path.py @@ -279,8 +279,10 @@ def _format_ids( str | None: The formatted ID or None if the format string is not ID-related. """ # Handle track and playlist IDs - if (name == "track_id" and isinstance(media, Track | Video)) or ( - name == "playlist_id" and isinstance(media, Playlist) + if ( + (name == "track_id" and isinstance(media, Track)) + or (name == "playlist_id" and isinstance(media, Playlist)) + or (name == "video_id" and isinstance(media, Video)) ): return str(media.id) # Handle album IDs