From eb9b1866a8313323f7e4123e1baa9db86efac179 Mon Sep 17 00:00:00 2001 From: Claudio Ortolina Date: Sun, 26 Apr 2026 19:05:27 +0100 Subject: [PATCH] Streamline function implementation --- lib/music_library/artists.ex | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/lib/music_library/artists.ex b/lib/music_library/artists.ex index 00faa05b..ad12e53f 100644 --- a/lib/music_library/artists.ex +++ b/lib/music_library/artists.ex @@ -23,15 +23,8 @@ defmodule MusicLibrary.Artists do @spec get_similar_artists(map(), MapSet.t(String.t())) :: {:ok, [map()]} | {:error, term()} def get_similar_artists(artist, collected_artist_ids) do - case LastFm.get_similar_artists(artist.musicbrainz_id, artist.name) do - {:ok, artists} -> - {:ok, - Enum.filter(artists, fn a -> - MapSet.member?(collected_artist_ids, a.musicbrainz_id) - end)} - - error -> - error + with {:ok, artists} <- LastFm.get_similar_artists(artist.musicbrainz_id, artist.name) do + {:ok, Enum.filter(artists, fn a -> a.musicbrainz_id in collected_artist_ids end)} end end