danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Improve wishlist track sorting robustness

Browse source 
Refactored the sorting logic to handle cases where 'spotify_data' may be a JSON string or have varying structures. This ensures consistent sorting by artist and track name, even with malformed or unexpected data formats.
This commit is contained in:
Broque Thomas 2025-12-14 12:21:36 -08:00
parent b50f6c8d9f
commit 3c7659feac
1 changed files with 25 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
core/wishlist_service.py
View file

@ -110,10 +110,31 @@ class WishlistService:
# Sort by artist name, then track name for consistent display order # Sort by artist name, then track name for consistent display order
try: try:
wishlist_tracks.sort(key=lambda x: ( def get_sort_key(track):
x['spotify_data'].get('artists', [{}])[0].get('name', '').lower(), spotify_data = track['spotify_data']
x['spotify_data'].get('name', '').lower() # Parse JSON string if needed
)) if isinstance(spotify_data, str):
import json
try:
spotify_data = json.loads(spotify_data)
except:
return ('', '') # Fallback for invalid JSON
artist_name = ''
track_name = ''
if isinstance(spotify_data, dict):
artists = spotify_data.get('artists', [])
if artists and len(artists) > 0:
if isinstance(artists[0], dict):
artist_name = artists[0].get('name', '')
elif isinstance(artists[0], str):
artist_name = artists[0]
track_name = spotify_data.get('name', '')
return (artist_name.lower(), track_name.lower())
wishlist_tracks.sort(key=get_sort_key)
logger.debug(f"Successfully sorted {len(wishlist_tracks)} wishlist tracks by artist/track name") logger.debug(f"Successfully sorted {len(wishlist_tracks)} wishlist tracks by artist/track name")
except Exception as sort_error: except Exception as sort_error:
logger.warning(f"Failed to sort wishlist tracks, using original order: {sort_error}") logger.warning(f"Failed to sort wishlist tracks, using original order: {sort_error}")