PonyMirrors/philomena
1
0
Fork 0
mirror of https://github.com/philomena-dev/philomena.git synced 2024-11-23 20:18:00 +01:00
Code Issues Projects Releases Packages Wiki Activity

Somewhat damaged patch 1 (#19)

Browse source 
* Fix for module placement

Correcting my own mistake when I first made these routes

* Correcting module location

* Correcting module location

* Fix

Fixes a potential crash when no posts are returned.
This commit is contained in:
SomewhatDamaged 2020-01-07 18:02:49 +11:00 committed by liamwhite
parent 0c5e0d7fa7
commit e918375c83
3 changed files with 11 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/philomena_web/controllers/api/json/posts_controller.ex → lib/philomena_web/controllers/api/json/forum/topic/post_controller.ex
View file

@ -1,4 +1,4 @@
defmodule PhilomenaWeb.Api.Json.PostController do defmodule PhilomenaWeb.Api.Json.Forum.Topic.PostController do
use PhilomenaWeb, :controller use PhilomenaWeb, :controller
alias PhilomenaWeb.PostJson alias PhilomenaWeb.PostJson
@ -44,6 +44,12 @@ defmodule PhilomenaWeb.Api.Json.PostController do
|> preload([_p, t, _f], topic: t) |> preload([_p, t, _f], topic: t)
|> Repo.all() |> Repo.all()
case posts do
[] ->
json(conn, %{posts: Enum.map(posts, &PostJson.as_json/1), page: page})
_ ->
json(conn, %{posts: Enum.map(posts, &PostJson.as_json/1), page: page, total: hd(posts).topic.post_count}) json(conn, %{posts: Enum.map(posts, &PostJson.as_json/1), page: page, total: hd(posts).topic.post_count})
end end
end
end end

2
lib/philomena_web/controllers/api/json/topics_controller.ex → lib/philomena_web/controllers/api/json/forum/topic_controller.ex
View file

@ -1,4 +1,4 @@
defmodule PhilomenaWeb.Api.Json.TopicController do defmodule PhilomenaWeb.Api.Json.Forum.TopicController do
use PhilomenaWeb, :controller use PhilomenaWeb, :controller
alias PhilomenaWeb.TopicJson alias PhilomenaWeb.TopicJson

4
lib/philomena_web/router.ex
View file

@ -116,8 +116,8 @@ defmodule PhilomenaWeb.Router do
resources "/tags", TagController, only: [:show] resources "/tags", TagController, only: [:show]
resources "/comments", CommentController, only: [:show] resources "/comments", CommentController, only: [:show]
resources "/forums", ForumController, only: [:show, :index] do resources "/forums", ForumController, only: [:show, :index] do
resources "/topics", TopicController, only: [:show, :index] do resources "/topics", Forum.TopicController, only: [:show, :index] do
resources "/posts", PostController, only: [:show, :index] resources "/posts", Forum.Topic.PostController, only: [:show, :index]
end end
end end