X-Git-Url: http://git.squeep.com/?a=blobdiff_plain;f=lib%2Fpleroma%2Fweb%2Fostatus%2Fhandlers%2Fnote_handler.ex;h=770a71a0a3c22c4d93a026331c30519ca87d7866;hb=92a0210fb03ca3e0aefe769fb6b0ab7bda6e5336;hp=38f9fc478e381857a00663472bace236297a4d3e;hpb=611ca385dea3d611a97579000311cc42684305e6;p=akkoma diff --git a/lib/pleroma/web/ostatus/handlers/note_handler.ex b/lib/pleroma/web/ostatus/handlers/note_handler.ex index 38f9fc478..770a71a0a 100644 --- a/lib/pleroma/web/ostatus/handlers/note_handler.ex +++ b/lib/pleroma/web/ostatus/handlers/note_handler.ex @@ -1,10 +1,17 @@ +# Pleroma: A lightweight social networking server +# Copyright © 2017-2019 Pleroma Authors +# SPDX-License-Identifier: AGPL-3.0-only + defmodule Pleroma.Web.OStatus.NoteHandler do require Logger - alias Pleroma.Web.{XML, OStatus} - alias Pleroma.{Object, User, Activity} + + alias Pleroma.Activity + alias Pleroma.Object alias Pleroma.Web.ActivityPub.ActivityPub alias Pleroma.Web.ActivityPub.Utils alias Pleroma.Web.CommonAPI + alias Pleroma.Web.OStatus + alias Pleroma.Web.XML @doc """ Get the context for this note. Uses this: @@ -12,50 +19,57 @@ defmodule Pleroma.Web.OStatus.NoteHandler do 2. The conversation reference in the ostatus xml 3. A newly generated context id. """ - def get_context(entry, inReplyTo) do - context = ( - XML.string_from_xpath("//ostatus:conversation[1]", entry) - || XML.string_from_xpath("//ostatus:conversation[1]/@ref", entry) - || "") |> String.trim + def get_context(entry, in_reply_to) do + context = + (XML.string_from_xpath("//ostatus:conversation[1]", entry) || + XML.string_from_xpath("//ostatus:conversation[1]/@ref", entry) || "") + |> String.trim() - with %{data: %{"context" => context}} <- Object.get_cached_by_ap_id(inReplyTo) do + with %{data: %{"context" => context}} <- Object.get_cached_by_ap_id(in_reply_to) do context - else _e -> - if String.length(context) > 0 do - context - else - Utils.generate_context_id - end + else + _e -> + if String.length(context) > 0 do + context + else + Utils.generate_context_id() + end end end def get_people_mentions(entry) do - :xmerl_xpath.string('//link[@rel="mentioned" and @ostatus:object-type="http://activitystrea.ms/schema/1.0/person"]', entry) - |> Enum.map(fn(person) -> XML.string_from_xpath("@href", person) end) + :xmerl_xpath.string( + '//link[@rel="mentioned" and @ostatus:object-type="http://activitystrea.ms/schema/1.0/person"]', + entry + ) + |> Enum.map(fn person -> XML.string_from_xpath("@href", person) end) end def get_collection_mentions(entry) do transmogrify = fn - ("http://activityschema.org/collection/public") -> + "http://activityschema.org/collection/public" -> "https://www.w3.org/ns/activitystreams#Public" - (group) -> + + group -> group end - :xmerl_xpath.string('//link[@rel="mentioned" and @ostatus:object-type="http://activitystrea.ms/schema/1.0/collection"]', entry) - |> Enum.map(fn(collection) -> XML.string_from_xpath("@href", collection) |> transmogrify.() end) + :xmerl_xpath.string( + '//link[@rel="mentioned" and @ostatus:object-type="http://activitystrea.ms/schema/1.0/collection"]', + entry + ) + |> Enum.map(fn collection -> XML.string_from_xpath("@href", collection) |> transmogrify.() end) end def get_mentions(entry) do - (get_people_mentions(entry) - ++ get_collection_mentions(entry)) - |> Enum.filter(&(&1)) + (get_people_mentions(entry) ++ get_collection_mentions(entry)) + |> Enum.filter(& &1) end def get_emoji(entry) do try do :xmerl_xpath.string('//link[@rel="emoji"]', entry) - |> Enum.reduce(%{}, fn(emoji, acc) -> + |> Enum.reduce(%{}, fn emoji, acc -> Map.put(acc, XML.string_from_xpath("@name", emoji), XML.string_from_xpath("@href", emoji)) end) rescue @@ -74,13 +88,14 @@ defmodule Pleroma.Web.OStatus.NoteHandler do Map.put(note, "external_url", url) end - def fetch_replied_to_activity(entry, inReplyTo) do - with %Activity{} = activity <- Activity.get_create_activity_by_object_ap_id(inReplyTo) do + def fetch_replied_to_activity(entry, in_reply_to) do + with %Activity{} = activity <- Activity.get_create_by_object_ap_id(in_reply_to) do activity else _e -> - with inReplyToHref when not is_nil(inReplyToHref) <- XML.string_from_xpath("//thr:in-reply-to[1]/@href", entry), - {:ok, [activity | _]} <- OStatus.fetch_activity_from_url(inReplyToHref) do + with in_reply_to_href when not is_nil(in_reply_to_href) <- + XML.string_from_xpath("//thr:in-reply-to[1]/@href", entry), + {:ok, [activity | _]} <- OStatus.fetch_activity_from_url(in_reply_to_href) do activity else _e -> nil @@ -91,34 +106,55 @@ defmodule Pleroma.Web.OStatus.NoteHandler do # TODO: Clean this up a bit. def handle_note(entry, doc \\ nil) do with id <- XML.string_from_xpath("//id", entry), - activity when is_nil(activity) <- Activity.get_create_activity_by_object_ap_id(id), + activity when is_nil(activity) <- Activity.get_create_by_object_ap_id(id), [author] <- :xmerl_xpath.string('//author[1]', doc), {:ok, actor} <- OStatus.find_make_or_update_user(author), content_html <- OStatus.get_content(entry), cw <- OStatus.get_cw(entry), - inReplyTo <- XML.string_from_xpath("//thr:in-reply-to[1]/@ref", entry), - inReplyToActivity <- fetch_replied_to_activity(entry, inReplyTo), - inReplyTo <- (inReplyToActivity && inReplyToActivity.data["object"]["id"]) || inReplyTo, + in_reply_to <- XML.string_from_xpath("//thr:in-reply-to[1]/@ref", entry), + in_reply_to_activity <- fetch_replied_to_activity(entry, in_reply_to), + in_reply_to <- + (in_reply_to_activity && in_reply_to_activity.data["object"]["id"]) || in_reply_to, attachments <- OStatus.get_attachments(entry), - context <- get_context(entry, inReplyTo), + context <- get_context(entry, in_reply_to), tags <- OStatus.get_tags(entry), mentions <- get_mentions(entry), to <- make_to_list(actor, mentions), date <- XML.string_from_xpath("//published", entry), unlisted <- XML.string_from_xpath("//mastodon:scope", entry) == "unlisted", cc <- if(unlisted, do: ["https://www.w3.org/ns/activitystreams#Public"], else: []), - note <- CommonAPI.Utils.make_note_data(actor.ap_id, to, context, content_html, attachments, inReplyToActivity, [], cw), + note <- + CommonAPI.Utils.make_note_data( + actor.ap_id, + to, + context, + content_html, + attachments, + in_reply_to_activity, + [], + cw + ), note <- note |> Map.put("id", id) |> Map.put("tag", tags), note <- note |> Map.put("published", date), note <- note |> Map.put("emoji", get_emoji(entry)), note <- add_external_url(note, entry), note <- note |> Map.put("cc", cc), # TODO: Handle this case in make_note_data - note <- (if inReplyTo && !inReplyToActivity, do: note |> Map.put("inReplyTo", inReplyTo), else: note) - do - res = ActivityPub.create(%{to: to, actor: actor, context: context, object: note, published: date, local: false, additional: %{"cc" => cc}}) - User.increase_note_count(actor) - res + note <- + if( + in_reply_to && !in_reply_to_activity, + do: note |> Map.put("inReplyTo", in_reply_to), + else: note + ) do + ActivityPub.create(%{ + to: to, + actor: actor, + context: context, + object: note, + published: date, + local: false, + additional: %{"cc" => cc} + }) else %Activity{} = activity -> {:ok, activity} e -> {:error, e}