Add conversation id to activities.

This is for compatibility with the TwAPI.
This commit is contained in:
Roger Braun 2017-03-28 14:40:09 +02:00
parent bb07efdaab
commit e25aea3fa7
3 changed files with 25 additions and 4 deletions

View file

@ -1,8 +1,9 @@
defmodule Pleroma.Web.TwitterAPI.Representers.UserRepresenter do defmodule Pleroma.Web.TwitterAPI.Representers.UserRepresenter do
use Pleroma.Web.TwitterAPI.Representers.BaseRepresenter use Pleroma.Web.TwitterAPI.Representers.BaseRepresenter
alias Pleroma.User alias Pleroma.User
def to_map(user, opts \\ %{}) do def to_map(user, opts) do
image = "https://placehold.it/48x48" image = "https://placehold.it/48x48"

View file

@ -34,11 +34,15 @@ def create_status(user = %User{}, data = %{}) do
|> put_in(["object", "context"], context) |> put_in(["object", "context"], context)
|> put_in(["object", "inReplyTo"], inReplyTo.data["object"]["id"]) |> put_in(["object", "inReplyTo"], inReplyTo.data["object"]["id"])
|> put_in(["object", "inReplyToStatusId"], inReplyToId) |> put_in(["object", "inReplyToStatusId"], inReplyToId)
|> put_in(["statusnet_conversation_id"], inReplyTo.data["statusnet_conversation_id"])
|> put_in(["object", "statusnet_conversation_id"], inReplyTo.data["statusnet_conversation_id"])
else _e -> else _e ->
activity activity
end end
ActivityPub.insert(activity) with {:ok, activity} <- ActivityPub.insert(activity) do
add_conversation_id(activity)
end
end end
def fetch_friend_statuses(user, opts \\ %{}) do def fetch_friend_statuses(user, opts \\ %{}) do
@ -67,6 +71,19 @@ def unfollow(%User{} = follower, followed_id) do
end end
end end
defp add_conversation_id(activity) do
if is_integer(activity.data["statusnet_conversation_id"]) do
{:ok, activity}
else
data = activity.data
|> put_in(["object", "statusnet_conversation_id"], activity.id)
|> put_in(["statusnet_conversation_id"], activity.id)
changeset = Ecto.Changeset.change(activity, data: data)
Repo.update(changeset)
end
end
defp activities_to_statuses(activities, opts) do defp activities_to_statuses(activities, opts) do
Enum.map(activities, fn(activity) -> Enum.map(activities, fn(activity) ->
activity_to_status(activity, opts) activity_to_status(activity, opts)

View file

@ -19,9 +19,11 @@ test "create a status" do
assert Enum.member?(get_in(activity.data, ["to"]), User.ap_followers(user)) assert Enum.member?(get_in(activity.data, ["to"]), User.ap_followers(user))
assert Enum.member?(get_in(activity.data, ["to"]), "https://www.w3.org/ns/activitystreams#Public") assert Enum.member?(get_in(activity.data, ["to"]), "https://www.w3.org/ns/activitystreams#Public")
# Add a context # Add a context + 'statusnet_conversation_id'
assert is_binary(get_in(activity.data, ["context"])) assert is_binary(get_in(activity.data, ["context"]))
assert is_binary(get_in(activity.data, ["object", "context"])) assert is_binary(get_in(activity.data, ["object", "context"]))
assert get_in(activity.data, ["object", "statusnet_conversation_id"]) == activity.id
assert get_in(activity.data, ["statusnet_conversation_id"]) == activity.id
end end
test "create a status that is a reply" do test "create a status that is a reply" do
@ -41,7 +43,8 @@ test "create a status that is a reply" do
assert get_in(reply.data, ["context"]) == get_in(activity.data, ["context"]) assert get_in(reply.data, ["context"]) == get_in(activity.data, ["context"])
assert get_in(reply.data, ["object", "context"]) == get_in(activity.data, ["object", "context"]) assert get_in(reply.data, ["object", "context"]) == get_in(activity.data, ["object", "context"])
assert get_in(reply.data, ["object", "context"]) == get_in(activity.data, ["object", "context"]) assert get_in(reply.data, ["statusnet_conversation_id"]) == get_in(activity.data, ["statusnet_conversation_id"])
assert get_in(reply.data, ["object", "statusnet_conversation_id"]) == get_in(activity.data, ["object", "statusnet_conversation_id"])
assert get_in(reply.data, ["object", "inReplyTo"]) == get_in(activity.data, ["object", "id"]) assert get_in(reply.data, ["object", "inReplyTo"]) == get_in(activity.data, ["object", "id"])
assert get_in(reply.data, ["object", "inReplyToStatusId"]) == activity.id assert get_in(reply.data, ["object", "inReplyToStatusId"]) == activity.id
end end