Remove duplication from LastFm.Refresh module
This commit is contained in:
+4
-14
@@ -21,22 +21,12 @@ defmodule LastFm.Refresh do
|
|||||||
end
|
end
|
||||||
|
|
||||||
@impl true
|
@impl true
|
||||||
def handle_continue(:refresh, config) do
|
def handle_continue(:refresh, config), do: refresh(config)
|
||||||
case config.api.get_recent_tracks(config.user, config.api_key) do
|
|
||||||
{:ok, tracks} ->
|
|
||||||
Feed.update(tracks)
|
|
||||||
Process.send_after(self(), :refresh, config.refresh_interval)
|
|
||||||
{:noreply, config}
|
|
||||||
|
|
||||||
{:error, _reason} ->
|
|
||||||
# TODO: think about failure scenario - error is logged at the API level
|
|
||||||
Process.send_after(self(), :refresh, config.refresh_interval)
|
|
||||||
{:noreply, config}
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
@impl true
|
@impl true
|
||||||
def handle_info(:refresh, config) do
|
def handle_info(:refresh, config), do: refresh(config)
|
||||||
|
|
||||||
|
defp refresh(config) do
|
||||||
case config.api.get_recent_tracks(config.user, config.api_key) do
|
case config.api.get_recent_tracks(config.user, config.api_key) do
|
||||||
{:ok, tracks} ->
|
{:ok, tracks} ->
|
||||||
Feed.update(tracks)
|
Feed.update(tracks)
|
||||||
|
|||||||
Reference in New Issue
Block a user