cloud/music_library
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

No need to expose LastFm.Config.new/1

Browse Source
This commit is contained in:
Claudio Ortolina
2024-12-03 09:38:12 +00:00
parent 916c381ed0
commit 669d2558d4
1 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/last_fm/config.ex
+6 -8
View File
@@ -11,19 +11,17 @@ defmodule LastFm.Config do
user: "", user: "",
refresh_interval: 60_000 refresh_interval: 60_000
@spec new(Enumerable.t()) :: t()
def new(opts) do
struct(__MODULE__, opts)
end
@spec enabled?(t) :: boolean() @spec enabled?(t) :: boolean()
def enabled?(config) do def enabled?(config) do
config.api && config.user !== "" && config.api_key !== "" config.api && config.user !== "" && config.api_key !== ""
end end
@spec resolve(atom) :: t
def resolve(otp_app) do def resolve(otp_app) do
otp_app app_config =
|> Application.get_env(LastFm) otp_app
|> new() |> Application.get_env(LastFm)
struct(__MODULE__, app_config)
end end
end end