From c92b1b5aef22b9e7fb896b85c3e62ba8f2a84e19 Mon Sep 17 00:00:00 2001
From: Dan Barber <github@danbarber.me>
Date: Fri, 2 Mar 2018 16:55:11 -0500
Subject: [PATCH] Add name to user

---
 lib/chess/auth/user.ex                                   | 9 ++++++---
 lib/chess_web/templates/layout/app.html.eex              | 2 +-
 lib/chess_web/templates/registration/new.html.eex        | 3 +++
 priv/repo/migrations/20180302195534_add_name_to_user.exs | 9 +++++++++
 4 files changed, 19 insertions(+), 4 deletions(-)
 create mode 100644 priv/repo/migrations/20180302195534_add_name_to_user.exs

diff --git a/lib/chess/auth/user.ex b/lib/chess/auth/user.ex
index 5e8f22c..5148183 100644
--- a/lib/chess/auth/user.ex
+++ b/lib/chess/auth/user.ex
@@ -6,9 +6,10 @@ defmodule Chess.Auth.User do
   alias Comeonin.Argon2
 
   schema "users" do
+    field :name, :string
+    field :username, :string
     field :password, :string, virtual: true
     field :password_hash, :string
-    field :username, :string
 
     has_many :games, Chess.Store.Game
     has_many :games_as_opponent, Chess.Store.Game, foreign_key: :opponent_id
@@ -19,8 +20,8 @@ defmodule Chess.Auth.User do
   @doc false
   def changeset(struct, params \\ %{}) do
     struct
-    |> cast(params, [:username, :password])
-    |> validate_required([:username, :password])
+    |> cast(params, required_attrs())
+    |> validate_required(required_attrs())
     |> unique_constraint(:username)
     |> hash_password()
   end
@@ -34,4 +35,6 @@ defmodule Chess.Auth.User do
       changeset
     end
   end
+
+  defp required_attrs, do: ~w[name username password]a
 end
diff --git a/lib/chess_web/templates/layout/app.html.eex b/lib/chess_web/templates/layout/app.html.eex
index 90f24cf..54b1120 100644
--- a/lib/chess_web/templates/layout/app.html.eex
+++ b/lib/chess_web/templates/layout/app.html.eex
@@ -16,7 +16,7 @@
       <header role="banner">
         <nav role="user">
           <%= if current_user(@conn) do %>
-            <%= current_user(@conn).username %>
+            <%= current_user(@conn).name %> (<%= current_user(@conn).username %>)
             |
             <%= link("Log out", to: session_path(@conn, :delete), method: :delete) %>
           <% else %>
diff --git a/lib/chess_web/templates/registration/new.html.eex b/lib/chess_web/templates/registration/new.html.eex
index f0e9ffd..61d72e2 100644
--- a/lib/chess_web/templates/registration/new.html.eex
+++ b/lib/chess_web/templates/registration/new.html.eex
@@ -8,6 +8,9 @@
   <% end %>
 
   <div class="form-group">
+    <div class="form-field">
+      <%= input f, :name %>
+    </div>
     <div class="form-field">
       <%= input f, :username %>
     </div>
diff --git a/priv/repo/migrations/20180302195534_add_name_to_user.exs b/priv/repo/migrations/20180302195534_add_name_to_user.exs
new file mode 100644
index 0000000..7afa104
--- /dev/null
+++ b/priv/repo/migrations/20180302195534_add_name_to_user.exs
@@ -0,0 +1,9 @@
+defmodule Chess.Repo.Migrations.AddNameToUser do
+  use Ecto.Migration
+
+  def change do
+    alter table("users") do
+      add :name, :string
+    end
+  end
+end