Move external links definition to context
This commit is contained in:
@@ -163,6 +163,22 @@ defmodule MusicLibrary.Artists do
|
|||||||
|> Repo.update()
|
|> Repo.update()
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@external_link_patterns %{
|
||||||
|
"ProgArchives" => "progarchives.com"
|
||||||
|
}
|
||||||
|
|
||||||
|
def external_links(artist_info) do
|
||||||
|
Enum.reduce(@external_link_patterns, [], fn {name, pattern}, acc ->
|
||||||
|
case ArtistInfo.relation_urls(artist_info, pattern) do
|
||||||
|
[] ->
|
||||||
|
acc
|
||||||
|
|
||||||
|
[url | _rest] ->
|
||||||
|
[%{name: name, url: url} | acc]
|
||||||
|
end
|
||||||
|
end)
|
||||||
|
end
|
||||||
|
|
||||||
def favicon_url(external_link), do: Favicon.favicon_url(external_link.url)
|
def favicon_url(external_link), do: Favicon.favicon_url(external_link.url)
|
||||||
|
|
||||||
defp get_collected_artist_ids do
|
defp get_collected_artist_ids do
|
||||||
|
|||||||
@@ -205,7 +205,7 @@ defmodule MusicLibraryWeb.ArtistLive.Show do
|
|||||||
|> assign(:current_section, :artists)
|
|> assign(:current_section, :artists)
|
||||||
|> assign(:artist, artist)
|
|> assign(:artist, artist)
|
||||||
|> assign(:artist_info, artist_info)
|
|> assign(:artist_info, artist_info)
|
||||||
|> assign(:external_links, external_links(artist_info))
|
|> assign(:external_links, Artists.external_links(artist_info))
|
||||||
|> assign(:country, ArtistInfo.country(artist_info))
|
|> assign(:country, ArtistInfo.country(artist_info))
|
||||||
|> assign_async(:lastfm_artist_info, fn ->
|
|> assign_async(:lastfm_artist_info, fn ->
|
||||||
with {:ok, lastfm_artist_info} <- LastFm.get_artist_info(artist.musicbrainz_id, artist.name) do
|
with {:ok, lastfm_artist_info} <- LastFm.get_artist_info(artist.musicbrainz_id, artist.name) do
|
||||||
@@ -363,20 +363,4 @@ defmodule MusicLibraryWeb.ArtistLive.Show do
|
|||||||
attributes: [class: "mt-2 text-sm/7"]
|
attributes: [class: "mt-2 text-sm/7"]
|
||||||
)
|
)
|
||||||
end
|
end
|
||||||
|
|
||||||
@external_link_patterns %{
|
|
||||||
"ProgArchives" => "progarchives.com"
|
|
||||||
}
|
|
||||||
|
|
||||||
defp external_links(artist_info) do
|
|
||||||
Enum.reduce(@external_link_patterns, [], fn {name, pattern}, acc ->
|
|
||||||
case ArtistInfo.relation_urls(artist_info, pattern) do
|
|
||||||
[] ->
|
|
||||||
acc
|
|
||||||
|
|
||||||
[url | _rest] ->
|
|
||||||
[%{name: name, url: url} | acc]
|
|
||||||
end
|
|
||||||
end)
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user