Skip to content

Commit 699b2af

Browse files
committed
fix(profile): allow user to have no posts/threads when querying posts/threads by username
1 parent 669c3d8 commit 699b2af

File tree

2 files changed

+2
-10
lines changed

2 files changed

+2
-10
lines changed

lib/epochtalk_server_web/controllers/post.ex

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -387,8 +387,7 @@ defmodule EpochtalkServerWeb.Controllers.Post do
387387
per_page: limit,
388388
desc: desc
389389
),
390-
count <- Profile.post_count_by_username(username),
391-
{:has_posts, true} <- {:has_posts, posts != []} do
390+
count <- Profile.post_count_by_username(username) do
392391
render(conn, :by_username, %{
393392
posts: posts,
394393
user: user,
@@ -400,9 +399,6 @@ defmodule EpochtalkServerWeb.Controllers.Post do
400399
desc: desc
401400
})
402401
else
403-
{:has_posts, false} ->
404-
ErrorHelpers.render_json_error(conn, 404, "Error, requested posts not found")
405-
406402
{:view_deleted_users, false} ->
407403
ErrorHelpers.render_json_error(conn, 400, "Account not found")
408404

lib/epochtalk_server_web/controllers/thread.ex

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -255,8 +255,7 @@ defmodule EpochtalkServerWeb.Controllers.Thread do
255255
Thread.page_by_username(username, priority, page,
256256
per_page: limit + 1,
257257
desc: desc
258-
),
259-
{:has_threads, true} <- {:has_threads, threads != []} do
258+
) do
260259
render(conn, :by_username, %{
261260
threads: threads,
262261
user: user,
@@ -267,9 +266,6 @@ defmodule EpochtalkServerWeb.Controllers.Thread do
267266
page: page
268267
})
269268
else
270-
{:has_threads, false} ->
271-
ErrorHelpers.render_json_error(conn, 404, "Error, requested threads not found")
272-
273269
{:view_deleted_users, false} ->
274270
ErrorHelpers.render_json_error(conn, 400, "Account not found")
275271

0 commit comments

Comments
 (0)