Just some formatting change and utilities for the next step

This commit is contained in:
thecookingsenpai 2024-01-30 15:51:05 +01:00
parent d411f04e98
commit e0581259a6
2 changed files with 8 additions and 2 deletions

View File

@ -104,8 +104,9 @@ class Populate:
log.error( log.error(
"Internet connection lost. Downloading artist images stopped." "Internet connection lost. Downloading artist images stopped."
) )
log.error(e) # REVIEW More informations = good
else: else:
log.warning(f"No internet connection. Downloading artist images stopped!") log.warning("No internet connection. Downloading artist images stopped!")
# Re-process the new artist images. # Re-process the new artist images.
if tried_to_download_new_images: if tried_to_download_new_images:
@ -135,6 +136,7 @@ class Populate:
unmodified_paths.add(track.filepath) unmodified_paths.add(track.filepath)
continue continue
except (FileNotFoundError, OSError) as e: except (FileNotFoundError, OSError) as e:
log.warning(e) # REVIEW More informations = good
TrackStore.remove_track_obj(track) TrackStore.remove_track_obj(track)
remove_tracks_by_filepaths(track.filepath) remove_tracks_by_filepaths(track.filepath)

View File

@ -242,6 +242,9 @@ class Keys:
@classmethod @classmethod
def load(cls): def load(cls):
# TODO Remove this. Just an handy flag to test the app without the API key
# IS_BUILD = True
if IS_BUILD: if IS_BUILD:
cls.LASTFM_API_KEY = configs.LASTFM_API_KEY cls.LASTFM_API_KEY = configs.LASTFM_API_KEY
cls.PLUGIN_LYRICS_AUTHORITY = configs.PLUGIN_LYRICS_AUTHORITY cls.PLUGIN_LYRICS_AUTHORITY = configs.PLUGIN_LYRICS_AUTHORITY
@ -253,7 +256,8 @@ class Keys:
@classmethod @classmethod
def verify_keys(cls): def verify_keys(cls):
if not cls.LASTFM_API_KEY: if not cls.LASTFM_API_KEY:
print("ERROR: LASTFM_API_KEY not set in environment") # REVIEW Ideally, this shouldn't be fatal
print("WARNING: LASTFM_API_KEY not set in environment. Experimental API calls will be implemented")
sys.exit(0) sys.exit(0)
@classmethod @classmethod