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

library page updates

Browse source
This commit is contained in:
Broque Thomas 2025-09-23 15:15:29 -07:00
parent eab17d584d
commit 96043ef530
2 changed files with 80 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
web_server.py
View file

@ -12430,12 +12430,6 @@ def merge_discography_data(owned_releases, spotify_discography):
for spotify_release in spotify_category:
spotify_key = (normalize_title(spotify_release['title']), normalize_year(spotify_release.get('year')))
# Debug logging for Bad Hair Day specifically
if 'bad hair day' in spotify_release['title'].lower():
print(f"🔍 DEBUG: Spotify 'Bad Hair Day' - Title: '{spotify_release['title']}', Year: {spotify_release.get('year')}")
print(f"🔍 DEBUG: Normalized key: {spotify_key}")
print(f"🔍 DEBUG: Available owned keys: {list(owned_map.keys())}")
# Check if we own this release (exact match first)
owned_release = None
matched_key = None
@ -12460,12 +12454,39 @@ def merge_discography_data(owned_releases, spotify_discography):
owned_release['spotify_id'] = spotify_release['spotify_id']
owned_release['owned'] = True
# Calculate track completion using Spotify track count
spotify_track_count = spotify_release.get('track_count', 0)
owned_track_count = owned_release.get('track_count', 0)
if spotify_track_count > 0:
completion_percentage = (owned_track_count / spotify_track_count) * 100
owned_release['track_completion'] = {
'owned_tracks': owned_track_count,
'total_tracks': spotify_track_count,
'percentage': round(completion_percentage, 1),
'missing_tracks': spotify_track_count - owned_track_count
}
else:
# Fallback if no Spotify track count
owned_release['track_completion'] = {
'owned_tracks': owned_track_count,
'total_tracks': owned_track_count,
'percentage': 100.0,
'missing_tracks': 0
}
# Image priority: owned first, then Spotify fallback
if not owned_release.get('image_url') and spotify_release.get('image_url'):
owned_release['image_url'] = spotify_release['image_url']
cards.append(owned_release)
print(f"✅ {category_name}: '{spotify_release['title']}' - OWNED")
# Enhanced logging with track completion
completion = owned_release['track_completion']
if completion['missing_tracks'] > 0:
print(f"✅ {category_name}: '{spotify_release['title']}' - OWNED ({completion['owned_tracks']}/{completion['total_tracks']} tracks, missing {completion['missing_tracks']})")
else:
print(f"✅ {category_name}: '{spotify_release['title']}' - OWNED (complete: {completion['owned_tracks']} tracks)")
# Remove empty lists from map (use the key that actually matched)
if matched_key and not owned_map[matched_key]:

66
webui/static/script.js
View file

@ -400,12 +400,22 @@ async function loadPageData(pageId) {
}
break;
case 'library':
// Initialize and load library data
if (!libraryPageState.isInitialized) {
initializeLibraryPage();
// Check if we should return to artist detail view instead of list
if (artistDetailPageState.currentArtistId && artistDetailPageState.currentArtistName) {
console.log(`🔄 Returning to artist detail: ${artistDetailPageState.currentArtistName}`);
navigateToPage('artist-detail');
if (!artistDetailPageState.isInitialized) {
initializeArtistDetailPage();
}
loadArtistDetailData(artistDetailPageState.currentArtistId, artistDetailPageState.currentArtistName);
} else {
// Refresh data when returning to page
await loadLibraryArtists();
// Initialize and load library data
if (!libraryPageState.isInitialized) {
initializeLibraryPage();
} else {
// Refresh data when returning to page
await loadLibraryArtists();
}
}
break;
case 'artist-detail':
@ -13936,6 +13946,9 @@ function initializeArtistDetailPage() {
if (backBtn) {
backBtn.addEventListener("click", () => {
console.log("🔙 Returning to Library page");
// Clear artist detail state so we go back to the list view
artistDetailPageState.currentArtistId = null;
artistDetailPageState.currentArtistName = null;
navigateToPage('library');
});
}
@ -14289,17 +14302,42 @@ function createReleaseCard(release) {
completionFill.className = "completion-fill";
if (release.owned) {
const percentage = release.track_completion || 100;
completionFill.style.width = `${percentage}%`;
// Handle new detailed track completion object
if (release.track_completion && typeof release.track_completion === 'object') {
const completion = release.track_completion;
const percentage = completion.percentage || 100;
const ownedTracks = completion.owned_tracks || 0;
const totalTracks = completion.total_tracks || 0;
const missingTracks = completion.missing_tracks || 0;
if (percentage === 100) {
completionText.textContent = "Complete";
completionText.className = "completion-text complete";
completionFill.className += " complete";
completionFill.style.width = `${percentage}%`;
if (missingTracks === 0) {
completionText.textContent = `Complete (${ownedTracks})`;
completionText.className = "completion-text complete";
completionFill.className += " complete";
} else {
completionText.textContent = `${ownedTracks}/${totalTracks} tracks`;
completionText.className = "completion-text partial";
completionFill.className += " partial";
// Add missing tracks indicator
completionText.title = `Missing ${missingTracks} track${missingTracks !== 1 ? 's' : ''}`;
}
} else {
completionText.textContent = `${percentage}%`;
completionText.className = "completion-text partial";
completionFill.className += " partial";
// Fallback for legacy simple percentage
const percentage = release.track_completion || 100;
completionFill.style.width = `${percentage}%`;
if (percentage === 100) {
completionText.textContent = "Complete";
completionText.className = "completion-text complete";
completionFill.className += " complete";
} else {
completionText.textContent = `${percentage}%`;
completionText.className = "completion-text partial";
completionFill.className += " partial";
}
}
} else {
completionText.textContent = "Missing";