Fix tag reading exceptions (#157)

The disc and track numbers are often stored as strings containing e.g.
"1/16" which leads to exceptions.
Also taglib sometimes returns empty lists which also produces an
exception.

This fix avoids some common exceptions.

diff --git a/music_assistant/providers/file/__init__.py b/music_assistant/providers/file/__init__.py
index 7a17e8ee79b1a9243e0a0147c6be4d988cc5b36f..d12f5b1018171de82d95b15a89f5b368ce9338a6 100644 (file)
--- a/music_assistant/providers/file/__init__.py
+++ b/music_assistant/providers/file/__init__.py
@@ -2,6 +2,7 @@
 import base64
 import logging
 import os
+import re
 from typing import List, Optional
 
 import taglib
@@ -368,12 +369,14 @@ class FileProvider(MusicProvider):
             track.artists.add(artist)
         if "GENRE" in song.tags:
             track.metadata["genres"] = song.tags["GENRE"]
-        if "ISRC" in song.tags:
+        if "ISRC" in song.tags and song.tags["ISRC"]:
             track.isrc = song.tags["ISRC"][0]
-        if "DISCNUMBER" in song.tags:
-            track.disc_number = int(song.tags["DISCNUMBER"][0])
-        if "TRACKNUMBER" in song.tags:
-            track.track_number = int(song.tags["TRACKNUMBER"][0])
+        if "DISCNUMBER" in song.tags and song.tags["DISCNUMBER"]:
+            regexp_numbers = re.findall(r'\d+', song.tags["DISCNUMBER"][0])
+            track.disc_number = int(regexp_numbers[0] if regexp_numbers else "0")
+        if "TRACKNUMBER" in song.tags and song.tags["TRACKNUMBER"]:
+            regexp_numbers = re.findall(r'\d+', song.tags["TRACKNUMBER"][0])
+            track.track_number = int(regexp_numbers[0] if regexp_numbers else "0")
         quality_details = ""
         if filename.endswith(".flac"):
             # TODO: get bit depth