Claudio Ortolina
eb8a00ab54
Use Integer.parse/1 instead of String.to_integer/1 on user-supplied and external API input to prevent ArgumentError crashes on non-numeric values. Adds fallback defaults or nil returns at each call site.
52 lines
1.3 KiB
Elixir
52 lines
1.3 KiB
Elixir
defmodule MusicLibraryWeb.CollectionController do
|
|
use MusicLibraryWeb, :controller
|
|
|
|
alias MusicLibrary.Collection
|
|
|
|
def latest(conn, _params) do
|
|
latest_record = Collection.get_latest_record!()
|
|
|
|
render(conn, :show, record: latest_record)
|
|
end
|
|
|
|
def random(conn, _params) do
|
|
random_record = Collection.get_random_record!()
|
|
|
|
render(conn, :show, record: random_record)
|
|
end
|
|
|
|
def on_this_day(conn, params) do
|
|
current_date =
|
|
case Map.get(params, "date") do
|
|
nil -> Date.utc_today()
|
|
date_string -> Date.from_iso8601!(date_string)
|
|
end
|
|
|
|
records_on_this_day = Collection.get_records_on_this_day(current_date)
|
|
|
|
render(conn, :on_this_day, records: records_on_this_day)
|
|
end
|
|
|
|
def index(conn, params) do
|
|
limit = parse_int(params["limit"], 20)
|
|
offset = parse_int(params["offset"], 0)
|
|
|
|
total = Collection.search_records_count("")
|
|
|
|
records = Collection.search_records("", limit: limit, offset: offset)
|
|
|
|
render(conn, :index, total: total, limit: limit, offset: offset, records: records)
|
|
end
|
|
|
|
defp parse_int(nil, default), do: default
|
|
|
|
defp parse_int(value, default) when is_binary(value) do
|
|
case Integer.parse(value) do
|
|
{int, ""} -> int
|
|
_ -> default
|
|
end
|
|
end
|
|
|
|
defp parse_int(_, default), do: default
|
|
end
|