aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorlain <lain@soykaf.club>2020-04-09 16:59:49 +0200
committerlain <lain@soykaf.club>2020-04-09 16:59:49 +0200
commite8fd0dd689be0c7bbca006f7267955329279da98 (patch)
treeca8115003cb5d6cd3dfe060cc92773562b67f7d6
parent68abea313d0be49aa6b8d4b980aa361383f991a7 (diff)
downloadpleroma-e8fd0dd689be0c7bbca006f7267955329279da98.tar.gz
ChatController: Basic support for returning messages.
-rw-r--r--lib/pleroma/web/pleroma_api/controllers/chat_controller.ex40
-rw-r--r--lib/pleroma/web/router.ex1
-rw-r--r--test/web/pleroma_api/controllers/chat_controller_test.exs28
3 files changed, 69 insertions, 0 deletions
diff --git a/lib/pleroma/web/pleroma_api/controllers/chat_controller.ex b/lib/pleroma/web/pleroma_api/controllers/chat_controller.ex
index 0ee8bea33..de23b9a22 100644
--- a/lib/pleroma/web/pleroma_api/controllers/chat_controller.ex
+++ b/lib/pleroma/web/pleroma_api/controllers/chat_controller.ex
@@ -5,10 +5,50 @@ defmodule Pleroma.Web.PleromaAPI.ChatController do
use Pleroma.Web, :controller
alias Pleroma.Chat
+ alias Pleroma.Object
alias Pleroma.Repo
import Ecto.Query
+ def messages(%{assigns: %{user: %{id: user_id} = user}} = conn, %{"id" => id}) do
+ with %Chat{} = chat <- Repo.get_by(Chat, id: id, user_id: user_id) do
+ messages =
+ from(o in Object,
+ where: fragment("?->>'type' = ?", o.data, "ChatMessage"),
+ where:
+ fragment(
+ """
+ (?->>'actor' = ? and ?->'to' = ?)
+ OR (?->>'actor' = ? and ?->'to' = ?)
+ """,
+ o.data,
+ ^user.ap_id,
+ o.data,
+ ^[chat.recipient],
+ o.data,
+ ^chat.recipient,
+ o.data,
+ ^[user.ap_id]
+ ),
+ order_by: [desc: o.id]
+ )
+ |> Repo.all()
+
+ represented_messages =
+ messages
+ |> Enum.map(fn message ->
+ %{
+ actor: message.data["actor"],
+ id: message.id,
+ content: message.data["content"]
+ }
+ end)
+
+ conn
+ |> json(represented_messages)
+ end
+ end
+
def index(%{assigns: %{user: %{id: user_id}}} = conn, _params) do
chats =
from(c in Chat,
diff --git a/lib/pleroma/web/router.ex b/lib/pleroma/web/router.ex
index 18ce9ee4b..368e77d3e 100644
--- a/lib/pleroma/web/router.ex
+++ b/lib/pleroma/web/router.ex
@@ -289,6 +289,7 @@ defmodule Pleroma.Web.Router do
post("/chats/by-ap-id/:ap_id", ChatController, :create)
get("/chats", ChatController, :index)
+ get("/chats/:id/messages", ChatController, :messages)
end
scope [] do
diff --git a/test/web/pleroma_api/controllers/chat_controller_test.exs b/test/web/pleroma_api/controllers/chat_controller_test.exs
index 40c09d1cd..6b2db5064 100644
--- a/test/web/pleroma_api/controllers/chat_controller_test.exs
+++ b/test/web/pleroma_api/controllers/chat_controller_test.exs
@@ -5,9 +5,37 @@ defmodule Pleroma.Web.PleromaAPI.ChatControllerTest do
use Pleroma.Web.ConnCase, async: true
alias Pleroma.Chat
+ alias Pleroma.Web.CommonAPI
import Pleroma.Factory
+ describe "GET /api/v1/pleroma/chats/:id/messages" do
+ # TODO
+ # - Test that statuses don't show
+ # - Test the case where it's not the user's chat
+ # - Test the returned data
+ test "it returns the messages for a given chat", %{conn: conn} do
+ user = insert(:user)
+ other_user = insert(:user)
+ third_user = insert(:user)
+
+ {:ok, _} = CommonAPI.post_chat_message(user, other_user, "hey")
+ {:ok, _} = CommonAPI.post_chat_message(user, third_user, "hey")
+ {:ok, _} = CommonAPI.post_chat_message(user, other_user, "how are you?")
+ {:ok, _} = CommonAPI.post_chat_message(other_user, user, "fine, how about you?")
+
+ chat = Chat.get(user.id, other_user.ap_id)
+
+ result =
+ conn
+ |> assign(:user, user)
+ |> get("/api/v1/pleroma/chats/#{chat.id}/messages")
+ |> json_response(200)
+
+ assert length(result) == 3
+ end
+ end
+
describe "POST /api/v1/pleroma/chats/by-ap-id/:id" do
test "it creates or returns a chat", %{conn: conn} do
user = insert(:user)