Compiler warning about unsafe variable no more present?

[kokolegorille]

This topic might be related…

The variable "user" is unsafe as it has been set inside a case/cond/receive/if/&&/||

The code is from Phoenix Guardian example. I think it’s cleaner than using multiple case logics but the compiler give you this annoying warning. Is the compiler right and is there a better way of writing it? defp create_user_from_auth(auth, current_user, repo) do user = current_user if !user, do: user = repo.get_by(User, email: auth.info.email) if !user, do: user = create_user(auth, repo) authorization_from_auth(user, auth, repo) {:ok, user} end warning: the variable "u…