Revert "Merge branch 'feature/attachments-cleanup' into 'develop'"revert/attachment-cleanup

This reverts commit a431e8c9f7033c739e10b0e8b34c75f2cc1d38d4, reversing
changes made to 8b4d81609d5627d62b826bcd3e87290cb513495f.

4 files changed, 1 insertions, 43 deletions

diff --git a/lib/pleroma/object.ex b/lib/pleroma/object.ex
index 38e372f6d..9a91fba9a 100644
--- a/lib/pleroma/object.ex
+++ b/lib/pleroma/object.ex
@@ -83,20 +83,6 @@ defmodule Pleroma.Object do
     Repo.one(from(object in Object, where: fragment("(?)->>'id' = ?", object.data, ^ap_id)))
   end
 
-  @doc """
-  Get a single attachment by it's name and href
-  """
-  @spec get_attachment_by_name_and_href(String.t(), String.t()) :: Object.t() | nil
-  def get_attachment_by_name_and_href(name, href) do
-    query =
-      from(o in Object,
-        where: fragment("(?)->>'name' = ?", o.data, ^name),
-        where: fragment("(?)->>'href' = ?", o.data, ^href)
-      )
-
-    Repo.one(query)
-  end
-
   defp warn_on_no_object_preloaded(ap_id) do
     "Object.normalize() called without preloaded object (#{inspect(ap_id)}). Consider preloading the object"
     |> Logger.debug()
diff --git a/lib/pleroma/uploaders/local.ex b/lib/pleroma/uploaders/local.ex
index 2e6fe3292..36b3c35ec 100644
--- a/lib/pleroma/uploaders/local.ex
+++ b/lib/pleroma/uploaders/local.ex
@@ -5,12 +5,10 @@
 defmodule Pleroma.Uploaders.Local do
   @behaviour Pleroma.Uploaders.Uploader
 
-  @impl true
   def get_file(_) do
     {:ok, {:static_dir, upload_path()}}
   end
 
-  @impl true
   def put_file(upload) do
     {local_path, file} =
       case Enum.reverse(Path.split(upload.path)) do
@@ -35,15 +33,4 @@ defmodule Pleroma.Uploaders.Local do
   def upload_path do
     Pleroma.Config.get!([__MODULE__, :uploads])
   end
-
-  @impl true
-  def delete_file(path) do
-    upload_path()
-    |> Path.join(path)
-    |> File.rm()
-    |> case do
-      :ok -> :ok
-      {:error, posix_error} -> {:error, to_string(posix_error)}
-    end
-  end
 end
diff --git a/lib/pleroma/uploaders/s3.ex b/lib/pleroma/uploaders/s3.ex
index feb89cea6..9876b6398 100644
--- a/lib/pleroma/uploaders/s3.ex
+++ b/lib/pleroma/uploaders/s3.ex
@@ -10,7 +10,6 @@ defmodule Pleroma.Uploaders.S3 do
 
   # The file name is re-encoded with S3's constraints here to comply with previous
   # links with less strict filenames
-  @impl true
   def get_file(file) do
     config = Config.get([__MODULE__])
     bucket = Keyword.fetch!(config, :bucket)
@@ -36,7 +35,6 @@ defmodule Pleroma.Uploaders.S3 do
       ])}}
   end
 
-  @impl true
   def put_file(%Pleroma.Upload{} = upload) do
     config = Config.get([__MODULE__])
     bucket = Keyword.get(config, :bucket)
@@ -71,18 +69,6 @@ defmodule Pleroma.Uploaders.S3 do
     end
   end
 
-  @impl true
-  def delete_file(file) do
-    [__MODULE__, :bucket]
-    |> Config.get()
-    |> ExAws.S3.delete_object(file)
-    |> ExAws.request()
-    |> case do
-      {:ok, %{status_code: 204}} -> :ok
-      error -> {:error, inspect(error)}
-    end
-  end
-
   @regex Regex.compile!("[^0-9a-zA-Z!.*/'()_-]")
   def strict_encode(name) do
     String.replace(name, @regex, "-")
diff --git a/lib/pleroma/uploaders/uploader.ex b/lib/pleroma/uploaders/uploader.ex
index d71e213d2..c0b22c28a 100644
--- a/lib/pleroma/uploaders/uploader.ex
+++ b/lib/pleroma/uploaders/uploader.ex
@@ -36,8 +36,6 @@ defmodule Pleroma.Uploaders.Uploader do
   @callback put_file(Pleroma.Upload.t()) ::
               :ok | {:ok, file_spec()} | {:error, String.t()} | :wait_callback
 
-  @callback delete_file(file :: String.t()) :: :ok | {:error, String.t()}
-
   @callback http_callback(Plug.Conn.t(), Map.t()) ::
               {:ok, Plug.Conn.t()}
               | {:ok, Plug.Conn.t(), file_spec()}
@@ -45,6 +43,7 @@ defmodule Pleroma.Uploaders.Uploader do
   @optional_callbacks http_callback: 2
 
   @spec put_file(module(), Pleroma.Upload.t()) :: {:ok, file_spec()} | {:error, String.t()}
+
   def put_file(uploader, upload) do
     case uploader.put_file(upload) do
       :ok -> {:ok, {:file, upload.path}}