From e0d2439f56262b03b83496a9547ca2bcdb9c6cda Mon Sep 17 00:00:00 2001
From: vicrodh <103399283+vicrodh@users.noreply.github.com>
Date: Mon, 26 Jan 2026 20:30:11 -0600
Subject: [PATCH] fix: correct Discogs adapter to use extended search result

- Remove duplicate discogs_release_to_search_result function
- Use existing discogs_extended_to_search_result which properly maps:
  year, country, label, catno, format
- MusicBrainz already maps label_info with catalog_number
---
 src-tauri/src/library/remote_metadata/mod.rs | 27 --------------------
 1 file changed, 27 deletions(-)

diff --git a/src-tauri/src/library/remote_metadata/mod.rs b/src-tauri/src/library/remote_metadata/mod.rs
index 47187030f..1a63fba15 100644
--- a/src-tauri/src/library/remote_metadata/mod.rs
+++ b/src-tauri/src/library/remote_metadata/mod.rs
@@ -128,33 +128,6 @@ pub fn musicbrainz_release_to_search_result(
 
 // ============ Discogs Adapter ============
 
-/// Convert Discogs release search result to unified DTO
-pub fn discogs_release_to_search_result(
-    result: &crate::discogs::SearchResult,
-) -> RemoteAlbumSearchResult {
-    // Discogs title format is usually "Artist - Album"
-    let (artist, title) = if let Some(pos) = result.title.find(" - ") {
-        let (a, t) = result.title.split_at(pos);
-        (a.to_string(), t.trim_start_matches(" - ").to_string())
-    } else {
-        ("Unknown Artist".to_string(), result.title.clone())
-    };
-
-    RemoteAlbumSearchResult {
-        provider: RemoteProvider::Discogs,
-        provider_id: result.id.to_string(),
-        title,
-        artist,
-        year: None, // Not in search result, need to fetch details
-        track_count: None,
-        country: None,
-        label: None,
-        catalog_number: None,
-        confidence: None,
-        format: None,
-    }
-}
-
 /// Parse Discogs track position to (disc_number, track_number)
 /// Handles formats: "1", "A1", "1-1", "CD1-1", "1.1"
 pub fn parse_discogs_position(position: &str) -> (u8, u8) {