Refactor ostatus_controller, extract metatags redirection to Redirector itself. Set...

[akkoma] / lib / pleroma / web / ostatus / metadata.ex

diff --git a/lib/pleroma/web/ostatus/metadata.ex b/lib/pleroma/web/ostatus/metadata.ex
index dd099e2ad7dd5aad4aaef5f73e80d09e75314008..4f319b3607003da149ccddd96e82fd817890cfeb 100644 (file)
--- a/lib/pleroma/web/ostatus/metadata.ex
+++ b/lib/pleroma/web/ostatus/metadata.ex
@@ -4,10 +4,10 @@ defmodule Pleroma.Web.Metadata do
   alias Pleroma.{User, Activity}
   alias Pleroma.Web.MediaProxy
 
-  def build_tags(activity, user, url) do
+  def build_tags(request_url, params) do
     Enum.concat([
-      if(meta_enabled?(:opengraph), do: opengraph_tags(activity, user), else: []),
-      if(meta_enabled?(:oembed), do: oembed_links(url), else: [])
+      if(meta_enabled?(:opengraph), do: opengraph_tags(params), else: []),
+      if(meta_enabled?(:oembed), do: oembed_links(request_url), else: [])
     ])
     |> Enum.map(&to_tag/1)
     |> Enum.map(&HTML.safe_to_string/1)
@@ -19,39 +19,38 @@ defmodule Pleroma.Web.Metadata do
     Keyword.get(config, type, false)
   end
 
-  def to_tag(data) do
-    with {name, attrs, _content = []} <- data do
-      HTML.Tag.tag(name, attrs)
-    else
-      {name, attrs, content} ->
-        HTML.Tag.content_tag(name, content, attrs)
-
-      _ ->
-        raise ArgumentError, message: "make_tag invalid args"
-    end
-  end
-
-  defp oembed_links(url) do
-    Enum.map(["xml", "json"], fn format ->
-      href = HTML.raw(oembed_path(url, format))
-      { :link, [ type: ["application/#{format}+oembed"], href: href, rel: 'alternate'], [] }
-    end)
-  end
-
-  defp opengraph_tags(activity, user) do
-    with image = User.avatar_url(user) |> MediaProxy.url(),
-         truncated_content = Formatter.truncate(activity.data["object"]["content"]),
-         domain = Pleroma.Config.get([:instance, :domain], "UNKNOWN_DOMAIN") do
+  # opengraph for single status
+  defp opengraph_tags(%{activity: activity, user: user}) do
+    with truncated_content = Formatter.truncate(activity.data["object"]["content"]) do
       [
         {:meta,
           [
             property: "og:title",
-            content: "#{user.name} (@#{user.nickname}@#{domain}) post ##{activity.id}"
+            content: "#{user.name} (@#{user.nickname}@#{pleroma_domain()}) post ##{activity.id}"
           ], []},
         {:meta, [property: "og:url", content: activity.data["id"]], []},
         {:meta, [property: "og:description", content: truncated_content],
           []},
-        {:meta, [property: "og:image", content: image], []},
+        {:meta, [property: "og:image", content: user_avatar_url(user)], []},
+        {:meta, [property: "og:image:width", content: 120], []},
+        {:meta, [property: "og:image:height", content: 120], []},
+        {:meta, [property: "twitter:card", content: "summary"], []}
+      ]
+    end
+  end
+
+  # opengraph for user card
+  defp opengraph_tags(%{user: user}) do
+    with truncated_bio = Formatter.truncate(user.bio) do
+      [
+        {:meta,
+          [
+            property: "og:title",
+            content: "#{user.name} (@#{user.nickname}@#{pleroma_domain()}) profile"
+          ], []},
+        {:meta, [property: "og:url", content: User.profile_url(user)], []},
+        {:meta, [property: "og:description", content: truncated_bio], []},
+        {:meta, [property: "og:image", content: user_avatar_url(user)], []},
         {:meta, [property: "og:image:width", content: 120], []},
         {:meta, [property: "og:image:height", content: 120], []},
         {:meta, [property: "twitter:card", content: "summary"], []}
@@ -59,8 +58,35 @@ defmodule Pleroma.Web.Metadata do
     end
   end
 
+  defp oembed_links(url) do
+    Enum.map(["xml", "json"], fn format ->
+      href = HTML.raw(oembed_path(url, format))
+      { :link, [ type: ["application/#{format}+oembed"], href: href, rel: 'alternate'], [] }
+    end)
+  end
+
+  def to_tag(data) do
+    with {name, attrs, _content = []} <- data do
+      HTML.Tag.tag(name, attrs)
+    else
+      {name, attrs, content} ->
+        HTML.Tag.content_tag(name, content, attrs)
+
+      _ ->
+        raise ArgumentError, message: "make_tag invalid args"
+    end
+  end
+
   defp oembed_path(url, format) do
     query = URI.encode_query(%{url: url, format: format})
     "#{Web.base_url()}/oembed?#{query}"
   end
+
+  defp user_avatar_url(user) do
+    User.avatar_url(user) |> MediaProxy.url()
+  end
+
+  def pleroma_domain do
+    Pleroma.Config.get([:instance, :domain], "UNKNOWN_DOMAIN")
+  end
 end
\ No newline at end of file