Add the API to update the Link model.

CHANGELOG.org

@@ -23,4 +23,15 @@
     Add the API to create the Links model.
 ** 0.4.2 <2023-07-30 Sun>
     Add the API to delete the Links model.
+** 0.4.3 <2023-07-31 Mon>
+    Rename the controller and the view names to "Link".
+    Move Links to the Links context.
+    Refactor to split the Links to a Link model and a Links logic.
+    Update the controller and the view for updated Link model.
+** 0.4.4 <2023-08-01 Tue>
+    Fix Links.update_one/2.
+** 0.4.5 <2023-08-01 Tue>
+    Fix the route path in LinkController.create/2.
+** 0.4.6 <2023-08-01 Tue>
+    Add the API to update the Link model.
 

lib/link_shortener/links.ex

@@ -1,73 +0,0 @@
-defmodule LinkShortener.Links do
-  use Ecto.Schema
-  import Ecto.Changeset
-  import Ecto.Query
-  alias LinkShortener.Repo
-  alias LinkShortener.Links
-  alias LinkShortener.Generators.SafeString
-
-  schema "links" do
-    field :name, :string
-    field :url, :string
-    field :shorten, :string
-
-    timestamps()
-  end
-
-  @doc false
-  def changeset(link, attrs) do
-    link
-    |> cast(attrs, [:name, :url, :shorten])
-    |> validate_required([:url, :shorten])
-    |> unique_constraint(:shorten)
-  end
-
-  def new_one(), do: Links.changeset(%Links{})
-
-  def create_one(attrs, length \\ 10, generator \\ SafeString) do
-    shorten = generator.generate(length)
-    insert_one(Map.put(attrs, :shorten, shorten))
-  end
-
-  def insert_one(attrs) do
-    %Links{}
-    |> Links.changeset(attrs)
-    |> Repo.insert()
-  end
-
-  def edit_one(id) do
-    get_one(id)
-    |> Links.changeset()
-  end
-
-  def update_one(%Links{} = Links, changes) do
-    Links
-    |> Links.changeset(changes)
-    |> Repo.update()
-  end
-
-  def insert_one(attrs) do
-    %Links{}
-    |> Links.changeset(attrs)
-    |> Repo.insert()
-  end
-
-  def delete_one(%Links{} = Links), do: Repo.delete(Links)
-
-  def get_one_by(attrs) do
-    Repo.get_by(Links, attrs)
-  end
-
-  def get_one_by_shorten(shorten) do
-    get_one_by(%{shorten: shorten})
-  end
-
-  def get_one(id) do
-    Repo.get!(Links, id)
-  end
-
-  def get_all(opts) do
-    from(Links)
-    |> Repo.all()
-  end
-end

lib/link_shortener/links/link.ex

@@ -0,0 +1,21 @@
+defmodule LinkShortener.Links.Link do
+  use Ecto.Schema
+  import Ecto.Changeset
+  import Ecto.Query
+
+  schema "links" do
+    field :name, :string
+    field :url, :string
+    field :shorten, :string
+
+    timestamps()
+  end
+
+  @doc false
+  def changeset(link, attrs) do
+    link
+    |> cast(attrs, [:name, :url, :shorten])
+    |> validate_required([:url, :shorten])
+    |> unique_constraint(:shorten)
+  end
+end

lib/link_shortener/links/links.ex

@@ -0,0 +1,57 @@
+defmodule LinkShortener.Links.Links do
+  use Ecto.Schema
+  import Ecto.Changeset
+  import Ecto.Query
+  alias LinkShortener.Repo
+  alias LinkShortener.Links.Link
+  alias LinkShortener.Generators.SafeString
+
+  def new_one(), do: Link.changeset(%Link{})
+
+  def create_one(attrs, length \\ 10, generator \\ SafeString) do
+    shorten = generator.generate(length)
+    insert_one(Map.put(attrs, :shorten, shorten))
+  end
+
+  def insert_one(attrs) do
+    %Link{}
+    |> Link.changeset(attrs)
+    |> Repo.insert()
+  end
+
+  def edit_one(id) do
+    get_one(id)
+    |> Link.changeset()
+  end
+
+  def update_one(%Link{} = link, changes) do
+    link
+    |> Link.changeset(changes)
+    |> Repo.update()
+  end
+
+  def insert_one(attrs) do
+    %Link{}
+    |> Link.changeset(attrs)
+    |> Repo.insert()
+  end
+
+  def delete_one(%Link{} = link), do: Repo.delete(link)
+
+  def get_one_by(attrs) do
+    Repo.get_by(Link, attrs)
+  end
+
+  def get_one_by_shorten(shorten) do
+    get_one_by(%{shorten: shorten})
+  end
+
+  def get_one(id) do
+    Repo.get!(Link, id)
+  end
+
+  def get_all(opts) do
+    from(Link)
+    |> Repo.all()
+  end
+end

lib/link_shortener_web/controllers/api/v1/link_controller.ex

@@ -0,0 +1,43 @@
+defmodule LinkShortenerWeb.Api.V1.LinkController do
+  use LinkShortenerWeb, :controller
+
+  alias LinkShortener.Links.Links
+  alias LinkShortener.Links.Link
+
+  action_fallback LinkShortenerWeb.FallbackController
+
+  def index(conn, _params) do
+    links = Links.get_all({})
+    render(conn, "index.json", links: links)
+  end
+
+  def create(conn, %{"link" => link_params}) do
+    with {:ok, %Link{} = link} <- Links.insert_one(link_params) do
+      conn
+      |> put_status(:created)
+      |> put_resp_header("location", Routes.v1_link_path(conn, :show, link))
+      |> render("show.json", link: link)
+    end
+  end
+
+  def show(conn, %{"id" => id}) do
+    link = Links.get_one(id)
+    render(conn, "show.json", link: link)
+  end
+
+  def update(conn, %{"id" => id, "link" => link_params}) do
+    link = Links.get_one(id)
+
+    with {:ok, %Link{} = link} <- Links.update_one(link, link_params) do
+      render(conn, "show.json", link: link)
+    end
+  end
+
+  def delete(conn, %{"id" => id}) do
+    link = Links.get_one(id)
+
+    with {:ok, %Link{}} <- Links.delete_one(link) do
+      send_resp(conn, :no_content, "")
+    end
+  end
+end

lib/link_shortener_web/controllers/api/v1/links_controller.ex

@@ -1,34 +0,0 @@
-defmodule LinkShortenerWeb.Api.V1.LinksController do
-  use LinkShortenerWeb, :controller
-
-  alias LinkShortener.Links
-
-  action_fallback LinkShortenerWeb.FallbackController
-
-  def index(conn, _params) do
-    links = Links.get_all({})
-    render(conn, "index.json", links: links)
-  end
-
-  def create(conn, %{"link" => link_params}) do
-    with {:ok, %Links{} = link} <- Links.insert_one(link_params) do
-      conn
-      |> put_status(:created)
-      |> put_resp_header("location", Routes.links_path(conn, :show, link))
-      |> render("show.json", links: link)
-    end
-  end
-
-  def show(conn, %{"id" => id}) do
-    link = Links.get_one(id)
-    render(conn, "show.json", links: link)
-  end
-
-  def delete(conn, %{"id" => id}) do
-    link = Links.get_one(id)
-
-    with {:ok, %Link{}} <- Links.delete_one(link) do
-      send_resp(conn, :no_content, "")
-    end
-  end
-end

lib/link_shortener_web/router.ex

@@ -26,7 +26,7 @@ defmodule LinkShortenerWeb.Router do
     pipe_through :api
 
     scope "/v1", Api.V1, as: :v1 do
-      resources "/links", LinksController
+      resources "/links", LinkController
     end
   end
 

lib/link_shortener_web/views/api/v1/link_view.ex

@@ -0,0 +1,21 @@
+defmodule LinkShortenerWeb.Api.V1.LinkView do
+  use LinkShortenerWeb, :view
+  alias LinkShortenerWeb.Api.V1.LinkView
+
+  def render("index.json", %{links: links}) do
+    %{data: render_many(links, LinkView, "link.json")}
+  end
+
+  def render("show.json", %{link: link}) do
+    %{data: render_one(link, LinkView, "link.json")}
+  end
+
+  def render("link.json", %{link: link}) do
+    %{
+      id: link.id,
+      name: link.name,
+      url: link.url,
+      shorten: link.shorten
+    }
+  end
+end

lib/link_shortener_web/views/api/v1/links_view.ex

@@ -1,21 +0,0 @@
-defmodule LinkShortenerWeb.Api.V1.LinksView do
-  use LinkShortenerWeb, :view
-  alias LinkShortenerWeb.Api.V1.LinksView
-
-  def render("index.json", %{links: links}) do
-    %{data: render_many(links, LinksView, "link.json")}
-  end
-
-  def render("show.json", %{links: link}) do
-    %{data: render_one(link, LinksView, "link.json")}
-  end
-
-  def render("link.json", %{links: link}) do
-    %{
-      id: link.id,
-      name: link.name,
-      url: link.url,
-      shorten: link.shorten
-    }
-  end
-end