diff --git a/lib/memex/accounts.ex b/lib/memex/accounts.ex index 6ddbc49..c0cc6cf 100644 --- a/lib/memex/accounts.ex +++ b/lib/memex/accounts.ex @@ -363,10 +363,16 @@ defmodule Memex.Accounts do """ @spec is_admin?(User.t()) :: boolean() def is_admin?(%User{id: user_id}) do - Repo.one(from u in User, where: u.id == ^user_id and u.role == :admin) - |> is_nil() + Repo.exists?(from u in User, where: u.id == ^user_id and u.role == :admin) end + @doc """ + Checks to see if user has the admin role + """ + @spec is_already_admin?(User.t() | nil) :: boolean() + def is_already_admin?(%User{role: :admin}), do: true + def is_already_admin?(_invalid_user), do: false + ## Confirmation @doc """ diff --git a/lib/memex_web/components/topbar.ex b/lib/memex_web/components/topbar.ex index 48e24dc..abaa8c7 100644 --- a/lib/memex_web/components/topbar.ex +++ b/lib/memex_web/components/topbar.ex @@ -65,7 +65,7 @@ defmodule MemexWeb.Components.Topbar do