add the albumhash prop to the fallback track object

- plus other tiny changes

server/app/api/track.py

@@ -17,14 +17,18 @@ def send_track_file(trackid):
     Returns an audio file that matches the passed id to the client.
     """
     track = instances.tracks_instance.get_track_by_id(trackid)
+    msg = {"msg": "File Not Found"}
 
     if track is None:
-        return "File not found", 404
+        return msg, 404
 
     track = models.Track(track)
     type = track.filepath.split(".")[-1]
 
-    return send_file(track.filepath, mimetype=f"audio/{type}")
+    try:
+        return send_file(track.filepath, mimetype=f"audio/{type}")
+    except FileNotFoundError:
+        return msg, 404
 
 
 @track_bp.route("/sample")

server/app/functions.py

@@ -79,7 +79,6 @@ class getArtistImage:
 
 
 class useImageDownloader:
-
     def __init__(self, url: str, dest: str) -> None:
         self.url = url
         self.dest = dest
@@ -96,7 +95,6 @@ class useImageDownloader:
 
 
 class CheckArtistImages:
-
     def __init__(self):
         self.artists: list[str] = []
         print("Checking for artist images")
@@ -121,8 +119,12 @@ class CheckArtistImages:
         :param artistname: The artist name
         """
 
-        img_path = (settings.APP_DIR + "/images/artists/" +
-                    helpers.create_safe_name(artistname) + ".webp")
+        img_path = (
+            settings.APP_DIR
+            + "/images/artists/"
+            + helpers.create_safe_name(artistname)
+            + ".webp"
+        )
 
         if cls.check_if_exists(img_path):
             return "exists"
@@ -139,7 +141,7 @@ class CheckArtistImages:
 
         with ThreadPoolExecutor() as pool:
             iter = pool.map(self.download_image, self.artists)
-            [print(i) for i in iter]
+            [i for i in iter]
 
         print("Done fetching images")
 
@@ -149,7 +151,8 @@ def fetch_album_bio(title: str, albumartist: str) -> str | None:
     Returns the album bio for a given album.
     """
     last_fm_url = "http://ws.audioscrobbler.com/2.0/?method=album.getinfo&api_key={}&artist={}&album={}&format=json".format(
-        settings.LAST_FM_API_KEY, albumartist, title)
+        settings.LAST_FM_API_KEY, albumartist, title
+    )
 
     try:
         response = requests.get(last_fm_url)
@@ -158,8 +161,7 @@ def fetch_album_bio(title: str, albumartist: str) -> str | None:
         return None
 
     try:
-        bio = data["album"]["wiki"]["summary"].split(
-            '<a href="https://www.last.fm/')[0]
+        bio = data["album"]["wiki"]["summary"].split('<a href="https://www.last.fm/')[0]
     except KeyError:
         bio = None
 

server/app/lib/watchdoge.py

@@ -126,7 +126,3 @@ class Handler(PatternMatchingEventHandler):
 
 
 watch = OnMyWatch()
-
-# TODO
-# When removing a track, check if there are other tracks in the same album,
-# if it was the last one, remove the album.