Prefer naming this function build_image_url/2
authorMark Felder <feld@feld.me>
Sat, 13 Mar 2021 18:27:15 +0000 (12:27 -0600)
committerMark Felder <feld@feld.me>
Sat, 13 Mar 2021 18:27:15 +0000 (12:27 -0600)
lib/pleroma/web/mastodon_api/views/status_view.ex

index 581b4e952322555529dea89215f992156a2b95d7..71f659ba05914e779ac9cf423782a5253edee109 100644 (file)
@@ -386,7 +386,7 @@ defmodule Pleroma.Web.MastodonAPI.StatusView do
         nil
       end
 
-    image_url = get_image_url(image_url_data, page_url_data)
+    image_url = build_image_url(image_url_data, page_url_data)
 
     %{
       type: "link",
@@ -548,8 +548,8 @@ defmodule Pleroma.Web.MastodonAPI.StatusView do
   # Avoid applying URI.merge unless necessary
   # TODO: revert to always attempting URI.merge(image_url_data, page_url_data)
   # when Elixir 1.12 is the minimum supported version
-  @spec get_image_url(struct() | nil, struct()) :: String.t() | nil
-  defp get_image_url(
+  @spec build_image_url(struct() | nil, struct()) :: String.t() | nil
+  defp build_image_url(
          %URI{scheme: image_scheme, host: image_host} = image_url_data,
          %URI{} = _page_url_data
        )
@@ -557,9 +557,9 @@ defmodule Pleroma.Web.MastodonAPI.StatusView do
     image_url_data |> to_string
   end
 
-  defp get_image_url(%URI{} = image_url_data, %URI{} = page_url_data) do
+  defp build_image_url(%URI{} = image_url_data, %URI{} = page_url_data) do
     URI.merge(page_url_data, image_url_data) |> to_string
   end
 
-  defp get_image_url(_, _), do: nil
+  defp build_image_url(_, _), do: nil
 end