refactor with Account context

Author: SophieDeBenedetto

lib/pointing_party/user.ex renamed to lib/pointing_party/account.ex

@@ -1,25 +1,25 @@
-defmodule PointingParty.User do
+defmodule PointingParty.Account do
   use Ecto.Schema
   import Ecto.Changeset
-  alias PointingParty.User
+  alias PointingParty.Account
 
-  schema "users" do
+  schema "accounts" do
     field :username, :string
   end
 
   def create(attrs) do
-    changeset = changeset(%User{}, attrs)
+    changeset = changeset(%Account{}, attrs)
     if changeset.valid? do
-      user =  apply_changes(changeset)
-      {:ok, user}
+      account = apply_changes(changeset)
+      {:ok, account}
     else
       {:error, %{changeset | action: :insert}}
     end
   end
 
   @doc false
-  def changeset(user, attrs \\ %{}) do
-    user
+  def changeset(account, attrs \\ %{}) do
+    account
     |> cast(attrs, [:username])
     |> validate_required([:username])
   end

lib/pointing_party/account/auth.ex

@@ -0,0 +1,6 @@
+defmodule PointingParty.Account.Auth do
+  alias PointingParty.Account
+  def login(params) do
+    Account.create(params)
+  end
+end

lib/pointing_party_web/controllers/card_controller.ex

@@ -1,18 +1,10 @@
 defmodule PointingPartyWeb.CardController do
   use PointingPartyWeb, :controller
-  plug :authenticate_user
   alias PointingParty.Card
 
   def index(conn, _params) do
     # temporary, just to get something on the page for now
     card = Card.get!(1)
     render(conn, "index.html", card: card)
   end
-
-  def authenticate_user(conn, _params) do
-    case get_session(conn, :username) do
-      nil -> redirect(conn, to: "/login") |> halt()
-      username -> assign(conn, :username, username)
-    end
-  end
 end

lib/pointing_party_web/controllers/session_controller.ex

@@ -1,16 +1,17 @@
 defmodule PointingPartyWeb.SessionController do
   use PointingPartyWeb, :controller
-  alias PointingParty.User
+  alias PointingParty.Account.Auth
+  alias PointingParty.Account
 
   def new(conn, _params) do
-    changeset = User.changeset(%User{})
+    changeset = Account.changeset(%Account{})
     render(conn, "new.html", changeset: changeset)
   end
 
   def create(conn, params) do
-    case User.create(params["user"]) do
-      {:ok, user} ->
-        put_session(conn, :username, user.username)
+    case Auth.login(params["account"]) do
+      {:ok, account} ->
+        put_session(conn, :username, account.username)
         |> redirect(to: "/cards") |> halt()
       {:error, changeset} ->
         render(conn, "new.html", changeset: changeset)

lib/pointing_party_web/plugs/auth.ex

@@ -0,0 +1,17 @@
+defmodule PointingPartyWeb.Plugs.Auth do
+  import Plug.Conn
+  import Phoenix.Controller
+
+  def init(default), do: default
+
+  def call(conn, _default) do
+    case authenticate(conn) do
+      nil -> redirect(conn, to: "/login") |> halt()
+      username -> assign(conn, :username, username)
+    end
+  end
+
+  defp authenticate(conn) do
+     get_session(conn, :username)
+  end
+end

lib/pointing_party_web/router.ex

@@ -19,6 +19,10 @@ defmodule PointingPartyWeb.Router do
     post "/login", SessionController, :create
     delete "/logout", SessionController, :delete
     get "/", PageController, :index
+  end
+
+  scope "/", PointingPartyWeb do
+    pipe_through [:browser, PointingPartyWeb.Plugs.Auth]
     get "/cards", CardController, :index
   end