From 2a9605c66ad8f4865035383c4b72a37b0dd3934e Mon Sep 17 00:00:00 2001
From: miffy <miffy-griffy@riseup.net>
Date: Sun, 8 Sep 2019 02:06:28 +0200
Subject: [PATCH] Refactoring of Admin context

---
 lib/mobilizon/admin/action_log.ex  | 23 +++++++++++++++++++----
 lib/mobilizon/{ => admin}/admin.ex | 30 +++++++++---------------------
 2 files changed, 28 insertions(+), 25 deletions(-)
 rename lib/mobilizon/{ => admin}/admin.ex (52%)

diff --git a/lib/mobilizon/admin/action_log.ex b/lib/mobilizon/admin/action_log.ex
index 2b3f80f7..68d1f189 100644
--- a/lib/mobilizon/admin/action_log.ex
+++ b/lib/mobilizon/admin/action_log.ex
@@ -1,27 +1,42 @@
 defmodule Mobilizon.Admin.ActionLog do
   @moduledoc """
-  ActionLog entity schema
+  Represents an action log entity.
   """
+
   use Ecto.Schema
+
   import Ecto.Changeset
+
   alias Mobilizon.Actors.Actor
 
-  @required_attrs [:action, :target_type, :target_id, :changes, :actor_id]
+  @type t :: %__MODULE__{
+          action: String.t(),
+          target_type: String.t(),
+          target_id: integer,
+          changes: map,
+          actor: Actor.t()
+        }
+
+  @required_attrs [:action, :target_type, :target_id, :actor_id]
+  @optional_attrs [:changes]
+  @attrs @required_attrs ++ @optional_attrs
 
   schema "admin_action_logs" do
     field(:action, :string)
     field(:target_type, :string)
     field(:target_id, :integer)
     field(:changes, :map)
+
     belongs_to(:actor, Actor)
 
     timestamps()
   end
 
   @doc false
+  @spec changeset(t | Ecto.Changeset.t(), map) :: Ecto.Changeset.t()
   def changeset(action_log, attrs) do
     action_log
-    |> cast(attrs, @required_attrs)
-    |> validate_required(@required_attrs -- [:changes])
+    |> cast(attrs, @attrs)
+    |> validate_required(@required_attrs)
   end
 end
diff --git a/lib/mobilizon/admin.ex b/lib/mobilizon/admin/admin.ex
similarity index 52%
rename from lib/mobilizon/admin.ex
rename to lib/mobilizon/admin/admin.ex
index 47eb3097..aa348cfa 100644
--- a/lib/mobilizon/admin.ex
+++ b/lib/mobilizon/admin/admin.ex
@@ -9,39 +9,27 @@ defmodule Mobilizon.Admin do
   alias Mobilizon.Storage.{Page, Repo}
 
   @doc """
-  Returns the list of action_logs.
-
-  ## Examples
-
-      iex> list_action_logs()
-      [%ActionLog{}, ...]
-
+  Returns the list of action logs.
   """
-  @spec list_action_logs(integer(), integer()) :: list(ActionLog.t())
+  @spec list_action_logs(integer | nil, integer | nil) :: [ActionLog.t()]
   def list_action_logs(page \\ nil, limit \\ nil) do
-    from(
-      r in ActionLog,
-      preload: [:actor]
-    )
+    list_action_logs_query()
     |> Page.paginate(page, limit)
     |> Repo.all()
   end
 
   @doc """
   Creates a action_log.
-
-  ## Examples
-
-      iex> create_action_log(%{field: value})
-      {:ok, %ActionLog{}}
-
-      iex> create_action_log(%{field: bad_value})
-      {:error, %Ecto.Changeset{}}
-
   """
+  @spec create_action_log(map) :: {:ok, ActionLog.t()} | {:error, Ecto.Changeset.t()}
   def create_action_log(attrs \\ %{}) do
     %ActionLog{}
     |> ActionLog.changeset(attrs)
     |> Repo.insert()
   end
+
+  @spec list_action_logs_query :: Ecto.Query.t()
+  defp list_action_logs_query do
+    from(r in ActionLog, preload: [:actor])
+  end
 end