Cleanup

lib/addict/crypto.ex

@@ -25,16 +25,4 @@ Signs and verifies text
   defp do_verify(false) do
     {:error, [{:token, "Password reset token not valid."}]}
   end
-
-  defp decode_key(key) do
-    Base.decode64(key) |> do_decode_key
-  end
-
-  defp do_decode_key(:error) do
-    {:error, ["Addict secret_key", "Addict.Configs.secret_key is invalid"]}
-  end
-
-  defp do_decode_key(key) do
-    key
-  end
 end

lib/addict/interactors/validate_password.ex

@@ -10,10 +10,10 @@ defmodule Addict.Interactors.ValidatePassword do
     call(changeset, [])
   end
 
-  def call(changeset, strategies \\ []) do
+  def call(changeset, strategies) do
     if Enum.count(strategies) == 0, do: strategies = [:default]
 
-    messages = strategies
+    strategies
     |> Enum.reduce(changeset, fn (strategy, acc) ->
       validate(strategy, acc)
     end)
@@ -33,7 +33,7 @@ defmodule Addict.Interactors.ValidatePassword do
   end
 
   defp validate(:default, changeset) do
-    Ecto.Changeset.validate_change(changeset, :password, fn (field, value) ->
+    Ecto.Changeset.validate_change(changeset, :password, fn (_field, value) ->
       validate(:default, value)
     end).errors
   end

lib/addict/interactors/validate_user_for_registration.ex

@@ -7,13 +7,13 @@ defmodule Addict.PasswordUser do
   end
 end
 
+defmodule Addict.Interactors.ValidateUserForRegistration do
 @doc """
 Validates if the user is valid for insertion.
 Checks if `password` is valid and if `email` is well formatted and unique.
 
 Returns `{:ok, []}` or `{:error, [errors]}`
 """
-defmodule Addict.Interactors.ValidateUserForRegistration do
   import Ecto.Changeset
   alias Addict.Interactors.ValidatePassword
   def call(user_params, configs \\ Addict.Configs) do

lib/addict/interactors/verify_password.ex

@@ -1,6 +1,4 @@
 defmodule Addict.Interactors.VerifyPassword do
-  import Ecto.Query
-
   @doc """
   Verifies if the password for the user is valid
 

lib/addict/mailers/generic.ex

@@ -1,6 +1,6 @@
+defmodule Addict.Mailers.Generic do
 @moduledoc """
 Defines the required behaviour for e-mail providers
 """
-defmodule Addict.Mailers.Generic do
   @callback send_email(String.t, String.t, String.t, String.t) :: any
 end

lib/addict/mailers/mail_sender.ex

@@ -10,7 +10,7 @@ defmodule Addict.Mailers.MailSender do
     user = user_params |> convert_to_list
     html_body = EEx.eval_string(template, user)
     from_email = Addict.Configs.from_email || "no-reply@addict.github.io"
-    Addict.Mailers.send_email(user_params["email"], Addict.Configs.from_email, subject, html_body)
+    Addict.Mailers.send_email(user_params["email"], from_email, subject, html_body)
   end
 
   def send_reset_token(email, path, host \\ Addict.Configs.host) do
@@ -20,7 +20,7 @@ defmodule Addict.Mailers.MailSender do
     params = %{"email" => email, "path" => path} |> convert_to_list
     html_body = EEx.eval_string(template, params)
     from_email = Addict.Configs.from_email || "no-reply@addict.github.io"
-    Addict.Mailers.send_email(email, Addict.Configs.from_email, subject, html_body)
+    Addict.Mailers.send_email(email, from_email, subject, html_body)
   end
 
   defp convert_to_list(params) do