Rename to make it obvious this is for images not videos
authorMark Felder <feld@FreeBSD.org>
Wed, 26 Aug 2020 21:40:13 +0000 (16:40 -0500)
committerMark Felder <feld@FreeBSD.org>
Wed, 26 Aug 2020 21:40:13 +0000 (16:40 -0500)
lib/pleroma/helpers/media_helper.ex

index 3256802a043eed23e869e4e3ca182a1a90be6b1a..fe11dd4603c15e23c9eead91c75714e7b8ef1a29 100644 (file)
@@ -11,7 +11,7 @@ defmodule Pleroma.Helpers.MediaHelper do
 
   def image_resize(url, options) do
     with executable when is_binary(executable) <- System.find_executable("convert"),
-         {:ok, args} <- prepare_resize_args(options),
+         {:ok, args} <- prepare_image_resize_args(options),
          url = Pleroma.Web.MediaProxy.url(url),
          {:ok, env} <- Pleroma.HTTP.get(url),
          {:ok, fifo_path} <- mkfifo()
@@ -23,7 +23,7 @@ defmodule Pleroma.Helpers.MediaHelper do
     end
   end
 
-  defp prepare_resize_args(%{max_width: max_width, max_height: max_height} = options) do
+  defp prepare_image_resize_args(%{max_width: max_width, max_height: max_height} = options) do
     quality = options[:quality] || 85
     resize = Enum.join([max_width, "x", max_height, ">"])
     args = [
@@ -34,7 +34,7 @@ defmodule Pleroma.Helpers.MediaHelper do
     {:ok, args}
   end
 
-  defp prepare_resize_args(_), do: {:error, :missing_options}
+  defp prepare_image_resize_args(_), do: {:error, :missing_options}
 
   defp run_fifo(fifo_path, env, executable, args) do
     args = List.flatten([fifo_path, args, "jpg:-"])