sphericalkat/katbin
1
0
Fork 0
Code Issues 5 Pull Requests 12 Actions Packages Projects Releases Wiki Activity

fix(pastes): replacing a paste with a URL now shortens the URL

Browse Source 
fixes #5

Signed-off-by: SphericalKat <amolele@gmail.com>
This commit is contained in:
Amogh Lele 2021-08-16 13:53:22 +05:30
parent 47acc97955
commit 478bea7efb
No known key found for this signature in database
GPG Key ID: ED5C54FBBB920E51
1 changed files with 30 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
lib/ketbin_web/controllers/page_controller.ex
View File

@ -13,11 +13,14 @@ defmodule KetbinWeb.PageController do
end
def show(%{assigns: %{show_edit: show_edit}} = conn, %{"id" => id}) do
paste = Pastes.get_paste!(id) # fetch paste from db
# fetch paste from db
paste = Pastes.get_paste!(id)
if paste.is_url do # paste is a url, redirect
# paste is a url, redirect
# regular paste, show content
if paste.is_url do
redirect(conn, external: paste.content)
else # regular paste, show content
else
render(conn, "show.html", paste: paste, show_edit: show_edit)
end
end
@ -49,16 +52,20 @@ defmodule KetbinWeb.PageController do
# attempt to create a paste
case Pastes.create_paste(paste_params) do
{:ok, paste} -> # all good, redirect
# all good, redirect
{:ok, paste} ->
unless is_url do
conn
|> redirect(to: Routes.page_path(conn, :show, paste)) # is a regular paste, take to regular route
# is a regular paste, take to regular route
|> redirect(to: Routes.page_path(conn, :show, paste))
else
conn
|> redirect(to: Routes.page_path(conn, :showlink, paste)) # is a url, take to route with /v/ prefix
# is a url, take to route with /v/ prefix
|> redirect(to: Routes.page_path(conn, :showlink, paste))
end
{:error, %Ecto.Changeset{} = changeset} -> # something went wrong, bail
# something went wrong, bail
{:error, %Ecto.Changeset{} = changeset} ->
render(conn, "index.html", changeset: changeset)
end
end
@ -72,11 +79,24 @@ defmodule KetbinWeb.PageController do
def update(conn, %{"id" => id, "paste" => paste_params}) do
paste = Pastes.get_paste!(id)
# check if content is a url
is_url =
Map.get(paste_params, "content")
|> Utils.is_url?()
paste_params = Map.put(paste_params, "is_url", is_url)
case Pastes.update_paste(paste, paste_params) do
{:ok, paste} ->
unless is_url do
conn
|> put_flash(:info, "Paste updated successfully.")
|> redirect(to: Routes.page_path(conn, :show, paste))
else
conn
|> put_flash(:info, "Paste updated successfully.")
|> redirect(to: Routes.page_path(conn, :showlink, paste))
end
{:error, %Ecto.Changeset{} = changeset} ->
render(conn, "edit.html", paste: paste, changeset: changeset)