From dd5a118d8cfc035c00b95312a5dce28e8991b3ea Mon Sep 17 00:00:00 2001
From: Liam <byteslice@airmail.cc>
Date: Mon, 15 Jul 2024 20:43:45 -0400
Subject: [PATCH] Simplify moderation logs context

---
 lib/philomena/moderation_logs.ex              | 41 ++++---------------
 .../controllers/moderation_log_controller.ex  |  2 +-
 2 files changed, 10 insertions(+), 33 deletions(-)

diff --git a/lib/philomena/moderation_logs.ex b/lib/philomena/moderation_logs.ex
index dcdb53f4..b84d37ad 100644
--- a/lib/philomena/moderation_logs.ex
+++ b/lib/philomena/moderation_logs.ex
@@ -9,40 +9,24 @@ defmodule Philomena.ModerationLogs do
   alias Philomena.ModerationLogs.ModerationLog
 
   @doc """
-  Returns the list of moderation_logs.
+  Returns a paginated list of moderation logs as a `m:Scrivener.Page`.
 
   ## Examples
 
-      iex> list_moderation_logs()
+      iex> list_moderation_logs(page_size: 15)
       [%ModerationLog{}, ...]
 
   """
-  def list_moderation_logs(conn) do
+  def list_moderation_logs(pagination) do
     ModerationLog
-    |> where([ml], ml.created_at > ago(2, "week"))
+    |> where([ml], ml.created_at >= ago(2, "week"))
     |> preload(:user)
     |> order_by(desc: :created_at)
-    |> Repo.paginate(conn.assigns.scrivener)
+    |> Repo.paginate(pagination)
   end
 
   @doc """
-  Gets a single moderation_log.
-
-  Raises `Ecto.NoResultsError` if the Moderation log does not exist.
-
-  ## Examples
-
-      iex> get_moderation_log!(123)
-      %ModerationLog{}
-
-      iex> get_moderation_log!(456)
-      ** (Ecto.NoResultsError)
-
-  """
-  def get_moderation_log!(id), do: Repo.get!(ModerationLog, id)
-
-  @doc """
-  Creates a moderation_log.
+  Creates a moderation log.
 
   ## Examples
 
@@ -60,21 +44,14 @@ defmodule Philomena.ModerationLogs do
   end
 
   @doc """
-  Deletes a moderation_log.
+  Removes moderation logs created more than 2 weeks ago.
 
   ## Examples
 
-      iex> delete_moderation_log(moderation_log)
-      {:ok, %ModerationLog{}}
-
-      iex> delete_moderation_log(moderation_log)
-      {:error, %Ecto.Changeset{}}
+      iex> cleanup!()
+      {31, nil}
 
   """
-  def delete_moderation_log(%ModerationLog{} = moderation_log) do
-    Repo.delete(moderation_log)
-  end
-
   def cleanup! do
     ModerationLog
     |> where([ml], ml.created_at < ago(2, "week"))
diff --git a/lib/philomena_web/controllers/moderation_log_controller.ex b/lib/philomena_web/controllers/moderation_log_controller.ex
index 3d9e7699..a3f202b1 100644
--- a/lib/philomena_web/controllers/moderation_log_controller.ex
+++ b/lib/philomena_web/controllers/moderation_log_controller.ex
@@ -9,7 +9,7 @@ defmodule PhilomenaWeb.ModerationLogController do
     preload: [:user]
 
   def index(conn, _params) do
-    moderation_logs = ModerationLogs.list_moderation_logs(conn)
+    moderation_logs = ModerationLogs.list_moderation_logs(conn.assigns.scrivener)
     render(conn, "index.html", title: "Moderation Logs", moderation_logs: moderation_logs)
   end
 end