From 3743729308e5e890e1d7821c3c509d1e3864348e Mon Sep 17 00:00:00 2001
From: Michael Kuperfish Steinberg
 <36902556+Michael-K-Stein@users.noreply.github.com>
Date: Fri, 20 Jan 2023 14:25:53 +0200
Subject: [PATCH 1/3] Fix minor bugs

---
 settings.py              | 3 ++-
 spotify_mass_download.py | 6 ++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/settings.py b/settings.py
index 2e30aa8..602347a 100644
--- a/settings.py
+++ b/settings.py
@@ -1,5 +1,5 @@
 class Settings:
-    DEFAULT_DOWNLOAD_DIRECTORY = 'E:\\Spotify\\'
+    DEFAULT_DOWNLOAD_DIRECTORY = 'music\\'
     ARTIST_IMAGES_SUB_DIR = '_Artists'
     PLAYLIST_METADATA_SUB_DIR = '_Playlists'
     CATEGORY_METADATA_SUB_DIR = '_Categories'
@@ -8,3 +8,4 @@ class Settings:
     FULL_DOWNLOAD_THREAD_LIMIT = 50
     VERBOSE_OUTPUTS = False
     AUTO_DOWNLOAD_PLAYLIST_METADATA = True
+    DOWNLOADS_FILE_SAVE_INTERVAL = 15
diff --git a/spotify_mass_download.py b/spotify_mass_download.py
index d4f3f06..3983608 100644
--- a/spotify_mass_download.py
+++ b/spotify_mass_download.py
@@ -119,7 +119,7 @@ def save_globals_save_file():
             f.write( json.dumps(data) )
         if settings.VERBOSE_OUTPUTS:
             console.log('Saved globals file!')
-        sleep(15)
+        sleep(settings.DOWNLOADS_FILE_SAVE_INTERVAL)
 
 
 def full_download(download_dir: str, identifier: str, recursive_artist: bool=False, recursive_album: bool=False, recursive: bool=False, recursive_limit:int=1024, thread_count:int=5):
@@ -133,7 +133,6 @@ def full_download(download_dir: str, identifier: str, recursive_artist: bool=Fal
 
         client.refresh_tokens()
         console.log(f'Recieved scrape command on identifier: {identifier}, {recursive=}, {recursive_artist=}, {recursive_album=}, {recursive_limit=}, {thread_count=}')
-        #console.log(f'Scraping on identifier: {identifier} yielded {len(track_list)} tracks!')
         download_threads = []
         track_list = []
         for track in scraper.scrape_tracks(identifier, console=console):
@@ -177,12 +176,11 @@ def download_all_categories_playlists(download_meta_data_only=True, query:str=''
     random.shuffle(categories)
     for category_index, category in enumerate(categories):
         console.log(f'Scraping playlists from category {category.name} ({category_index + 1}/{len(categories)})')
-        #category.download_metadata(scraper=scraper)
+        category.download_metadata(scraper=scraper)
         try:
             thread = Thread(target=download_category_playlists, args=(category.spotify_id, category_index, categories, download_meta_data_only))
             thread.start()
             threads.append(thread)
-            #download_category_playlists(category_id, category_index=category_index, category_ids=category_ids, download_meta_data_only=download_meta_data_only)
         except Exception as ex:
                 console.error(f'Scraping categories exception: {ex}')
 

From d4d4c4815098712d2cb4d5f9bae489a7ac11dc76 Mon Sep 17 00:00:00 2001
From: Michael Kuperfish Steinberg
 <36902556+Michael-K-Stein@users.noreply.github.com>
Date: Fri, 20 Jan 2023 14:27:21 +0200
Subject: [PATCH 2/3] Fix typos

---
 auto_compressor.py   | 2 +-
 templates/index.html | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/auto_compressor.py b/auto_compressor.py
index c3693fd..7e5906a 100644
--- a/auto_compressor.py
+++ b/auto_compressor.py
@@ -72,4 +72,4 @@ if __name__ == '__main__':
     if len(sys.argv) == 1:
         zip_bunches()
     else:
-        zip_bunches(sys.argv[1])
\ No newline at end of file
+        zip_bunches(sys.argv[1])
diff --git a/templates/index.html b/templates/index.html
index 171da3a..bed0de3 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -5,7 +5,7 @@
     <meta http-equiv="X-UA-Compatible" content="IE=edge">
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
     <link rel="stylesheet" href="/static/css/base.css">
-    <title>Spotifile</title>
+    <title>SpotiFile</title>
     
     <script>
         let offset = 0;

From fe43b16edf602efda7a4791ea196726be8edd6e4 Mon Sep 17 00:00:00 2001
From: Michael Kuperfish Steinberg
 <36902556+Michael-K-Stein@users.noreply.github.com>
Date: Fri, 20 Jan 2023 15:17:11 +0200
Subject: [PATCH 3/3] Fix scraper crash on local files

---
 spotify_scraper.py     | 2 +-
 utils/spotify_album.py | 5 ++++-
 utils/spotify_track.py | 8 +++++++-
 3 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/spotify_scraper.py b/spotify_scraper.py
index 771eff0..326538e 100644
--- a/spotify_scraper.py
+++ b/spotify_scraper.py
@@ -39,7 +39,7 @@ class SpotifyScraper:
     def extract_id_from_link(self, link: str) -> str:
         return link[link.rindex('/') + 1:]
 
-    def scrape_tracks(self, link: str, console=None) -> list:
+    def scrape_tracks(self, link: str, console=None) -> list[SpotifyTrack]:
         id_type = self.identify_link_type(link)
         if id_type == self.IDTypes.Playlist:
             return self.scrape_playlist_tracks(self.extract_id_from_link(link))
diff --git a/utils/spotify_album.py b/utils/spotify_album.py
index 47ba26d..48f4100 100644
--- a/utils/spotify_album.py
+++ b/utils/spotify_album.py
@@ -14,8 +14,11 @@ class SpotifyAlbum:
             self.load_from_data(album_data)
 
     def load_from_data(self, data):
+        if not data['album_type']:
+            return
         self.title = data['name']
-        self.thumbnail_href = data['images'][0]['url']
+        if len(data['images']) > 0:
+            self.thumbnail_href = data['images'][0]['url']
         self.track_count = data['total_tracks']
         try:
             self.release_date = time.mktime(datetime.datetime.strptime(data['release_date'], "%Y-%m-%d").timetuple())
diff --git a/utils/spotify_track.py b/utils/spotify_track.py
index a117467..b4fac4e 100644
--- a/utils/spotify_track.py
+++ b/utils/spotify_track.py
@@ -48,7 +48,9 @@ class SpotifyTrack:
         self.explicit = data['explicit']
         self.href = data['href']
         self.popularity = data['popularity']
-        self.isrc = data['external_ids']['isrc']
+        if 'isrc' in data['external_ids']:
+            # isrc is not available for local files
+            self.isrc = data['external_ids']['isrc']
 
     def __str__(self) -> str:
         return f'SpotifyTrack< {self.title} >'
@@ -65,9 +67,13 @@ class SpotifyTrack:
         return scraper.get(self.thumbnail_href).content
 
     def get_download_link(self, scraper) -> str:
+        if not self.isrc:
+            return ''
         return Deezer.get_track_download_url(Deezer.get_track_data(Deezer.get_track_id_from_isrc(self.isrc)))[0]
 
     def download(self, scraper) -> bytes:
+        if not self.isrc:
+            raise SpotifyTrackException(f'Cannot download local file {self.title}!')
         try:
             download_link = self.get_download_link(scraper)
             data = Deezer.decrypt_download_data(requests.get(download_link, headers={'Accept':'*/*'}), self.isrc)