simplify use of AuthPlugOptional in router.ex #65

dwyl · Oct 2, 2020 · 28dc5a6 · 28dc5a6
1 parent 0c883ce
commit 28dc5a6
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 123 deletions.
diff --git a/lib/app/ctx.ex b/lib/app/ctx.ex
@@ -202,102 +202,6 @@ defmodule App.Ctx do
     Status.changeset(status, %{})
   end
 
-  alias App.Ctx.Item
-
-  @doc """
-  Returns the list of items.
-
-  ## Examples
-
-      iex> list_items()
-      [%Item{}, ...]
-
-  """
-  def list_items do
-    Repo.all(Item)
-  end
-
-  @doc """
-  Gets a single item.
-
-  Raises `Ecto.NoResultsError` if the Item does not exist.
-
-  ## Examples
-
-      iex> get_item!(123)
-      %Item{}
-
-      iex> get_item!(456)
-      ** (Ecto.NoResultsError)
-
-  """
-  def get_item!(id), do: Repo.get!(Item, id)
-
-  @doc """
-  Creates a item.
-
-  ## Examples
-
-      iex> create_item(%{field: value})
-      {:ok, %Item{}}
-
-      iex> create_item(%{field: bad_value})
-      {:error, %Ecto.Changeset{}}
-
-  """
-  def create_item(attrs \\ %{}) do
-    %Item{}
-    |> Item.changeset(attrs)
-    |> Repo.insert()
-  end
-
-  @doc """
-  Updates a item.
-
-  ## Examples
-
-      iex> update_item(item, %{field: new_value})
-      {:ok, %Item{}}
-
-      iex> update_item(item, %{field: bad_value})
-      {:error, %Ecto.Changeset{}}
-
-  """
-  def update_item(%Item{} = item, attrs) do
-    item
-    |> Item.changeset(attrs)
-    |> Repo.update()
-  end
-
-  @doc """
-  Deletes a Item.
-
-  ## Examples
-
-      iex> delete_item(item)
-      {:ok, %Item{}}
-
-      iex> delete_item(item)
-      {:error, %Ecto.Changeset{}}
-
-  """
-  def delete_item(%Item{} = item) do
-    Repo.delete(item)
-  end
-
-  @doc """
-  Returns an `%Ecto.Changeset{}` for tracking item changes.
-
-  ## Examples
-
-      iex> change_item(item)
-      %Ecto.Changeset{source: %Item{}}
-
-  """
-  def change_item(%Item{} = item) do
-    Item.changeset(item, %{})
-  end
-
   alias App.Ctx.List
 
   @doc """

diff --git a/lib/app/ctx/session.ex b/lib/app/ctx/session.ex
diff --git a/lib/app_web/router.ex b/lib/app_web/router.ex
@@ -8,7 +8,7 @@ defmodule AppWeb.Router do
     plug :fetch_flash
     plug :protect_from_forgery
     plug :put_secure_browser_headers
-    # plug AppWeb.Auth
+    plug AuthPlugOptional
   end
 
   pipeline :api do
@@ -21,10 +21,8 @@ defmodule AppWeb.Router do
     plug :accepts, ["json"]
   end
 
-  pipeline :auth_optional, do: plug(AuthPlugOptional, %{})
-
   scope "/", AppWeb do
-    pipe_through [:browser, :auth_optional]
+    pipe_through [:browser]
 
     get "/", PageController, :index
     # post "/register", PageController, :index