Extract action functions from ExAdmin to ExAdmin.ResourceTitleActions

lib/ex_admin.ex

@@ -279,35 +279,6 @@ defmodule ExAdmin do
     end)
   end
 
-  @doc false
-  def default_resource_title_actions(%Plug.Conn{params: params} = conn, %{resource_model: resource_model} = defn) do
-    singular = ExAdmin.Utils.displayable_name_singular(conn) |> titleize
-    actions = defn.actions
-    case Utils.action_name(conn) do
-      :show ->
-        id = Map.get(params, "id")
-        Enum.reduce([:edit, :new, :delete], [], fn(action, acc) ->
-          if Utils.authorized_action?(conn, action, resource_model) do
-            [{action, action_button(conn, defn, singular, :show, action, actions, id)}|acc]
-          else
-            acc
-          end
-        end)
-        |> add_custom_actions(:show, actions, conn, resource_model, id)
-        |> Enum.reverse
-      action when action in [:index, :edit] ->
-        if Utils.authorized_action?(conn, action, resource_model) do
-          [{:new, action_button(conn, defn, singular, action, :new, actions)}]
-        else
-          []
-        end
-        |> add_custom_actions(action, actions, conn, resource_model)
-        |> Enum.reverse
-      _ ->
-        []
-    end
-  end
-
   @doc false
   def default_page_title_actions(_conn, _) do
     []
@@ -325,62 +296,6 @@ defmodule ExAdmin do
     conn.assigns.theme.name
   end
 
-  def action_button(conn, defn, name, _page, action, actions, id \\ nil) do
-    if action in actions do
-      if ExAdmin.Utils.authorized_action?(conn, action, defn) do
-        action_name = defn.action_labels[action] || humanize(action)
-        [action_link(conn, "#{action_name} #{name}", action, id)]
-      else
-        []
-      end
-    else
-      []
-    end
-  end
-
-  defp add_custom_actions(acc, action, actions, conn, resource_model, id \\ nil)
-  defp add_custom_actions(acc, _action, [], _conn, _resource_model, _id), do: acc
-  defp add_custom_actions(acc, action, [{action, custom_action_name, button} | actions], conn, resource_model, id) do
-    if Utils.authorized_action?(%{action: custom_action_name}, action, resource_model) do
-      eval_custom_function(button, id, acc)
-    else
-      acc
-    end
-    |> add_custom_actions(action, actions, conn, resource_model, id)
-  end
-  defp add_custom_actions(acc, action, [{action, button} | actions], conn, resource_model, id) do
-    if Utils.authorized_action?(conn, action, resource_model) do
-      eval_custom_function(button, id, acc)
-    else
-      acc
-    end
-    |> add_custom_actions(action, actions, conn, resource_model, id)
-  end
-  defp add_custom_actions(acc, action, [_|actions], conn, resource_model, id) do
-    add_custom_actions(acc, action, actions, conn, resource_model, id)
-  end
-
-  def eval_custom_function(button, id, acc) do
-    import ExAdmin.ViewHelpers
-    endpoint()  # remove the compiler warning
-    {fun, _} = Code.eval_quoted button, [id: id], __ENV__
-    cond do
-      is_function(fun, 1) -> [fun.(id) | acc]
-      is_function(fun, 0) -> [fun.() | acc]
-      true                -> acc
-    end
-  end
-
-  defp action_link(conn, name, :delete, _id) do
-    {name,
-      [href: admin_resource_path(conn, :destroy),
-        "data-confirm": Utils.confirm_message,
-        "data-method": :delete, rel: :nofollow]}
-  end
-  defp action_link(conn, name, action, _id) do
-    {name, [href: admin_resource_path(conn, action)]}
-  end
-
   @doc false
   def has_action?(conn, defn, action) do
     if ExAdmin.Utils.authorized_action?(conn, action, defn),

lib/ex_admin/register.ex

@@ -186,7 +186,7 @@ defmodule ExAdmin.Register do
 
       defstruct controller: @controller,
                 controller_methods: Module.get_attribute(__MODULE__, :controller_methods),
-                title_actions: &ExAdmin.default_resource_title_actions/2,
+                title_actions: &ExAdmin.ResourceTitleActions.default/2,
                 type: :resource,
                 resource_model: module,
                 resource_name: resource_name(module),

lib/ex_admin/resource_title_actions.ex

@@ -0,0 +1,85 @@
+defmodule ExAdmin.ResourceTitleActions do
+  alias ExAdmin.Utils
+  @doc false
+  def default(%Plug.Conn{params: params} = conn, %{resource_model: resource_model} = defn) do
+    singular = Utils.displayable_name_singular(conn) |> Utils.titleize
+    actions = defn.actions
+    case Utils.action_name(conn) do
+      :show ->
+        id = Map.get(params, "id")
+        Enum.reduce([:edit, :new, :delete], [], fn(action, acc) ->
+          if Utils.authorized_action?(conn, action, resource_model) do
+            [{action, action_button(conn, defn, singular, :show, action, actions, id)}|acc]
+          else
+            acc
+          end
+        end)
+        |> add_custom_actions(:show, actions, conn, resource_model, id)
+        |> Enum.reverse
+      action when action in [:index, :edit] ->
+        if Utils.authorized_action?(conn, action, resource_model) do
+          [{:new, action_button(conn, defn, singular, action, :new, actions)}]
+        else
+          []
+        end
+        |> add_custom_actions(action, actions, conn, resource_model)
+        |> Enum.reverse
+      _ ->
+        []
+    end
+  end
+
+  @doc false
+  def action_button(conn, defn, name, _page, action, actions, id \\ nil) do
+    if action in actions do
+      if Utils.authorized_action?(conn, action, defn) do
+        action_name = defn.action_labels[action] || Utils.humanize(action)
+        [action_link(conn, "#{action_name} #{name}", action, id)]
+      else
+        []
+      end
+    else
+      []
+    end
+  end
+
+  defp add_custom_actions(acc, action, actions, conn, resource_model, id \\ nil)
+  defp add_custom_actions(acc, _action, [], _conn, _resource_model, _id), do: acc
+  defp add_custom_actions(acc, action, [{action, custom_action_name, button} | actions], conn, resource_model, id) do
+    %{action: custom_action_name, id: id}
+    |> Utils.authorized_action?(action, resource_model)
+    |> eval_custom_function(button, id, acc)
+    |> add_custom_actions(action, actions, conn, resource_model, id)
+  end
+  defp add_custom_actions(acc, action, [{action, button} | actions], conn, resource_model, id) do
+    conn
+    |> Utils.authorized_action?(action, resource_model)
+    |> eval_custom_function(button, id, acc)
+    |> add_custom_actions(action, actions, conn, resource_model, id)
+  end
+  defp add_custom_actions(acc, action, [_|actions], conn, resource_model, id) do
+    add_custom_actions(acc, action, actions, conn, resource_model, id)
+  end
+
+  defp eval_custom_function(false, _button, _id, acc), do: acc
+  defp eval_custom_function(true, button, id, acc) do
+    import ExAdmin.ViewHelpers
+    endpoint()  # remove the compiler warning
+    {fun, _} = Code.eval_quoted button, [id: id], __ENV__
+    cond do
+      is_function(fun, 1) -> [fun.(id) | acc]
+      is_function(fun, 0) -> [fun.() | acc]
+      true                -> acc
+    end
+  end
+
+  defp action_link(conn, name, :delete, _id) do
+    {name,
+      [href: Utils.admin_resource_path(conn, :destroy),
+        "data-confirm": Utils.confirm_message,
+        "data-method": :delete, rel: :nofollow]}
+  end
+  defp action_link(conn, name, action, _id) do
+    {name, [href: Utils.admin_resource_path(conn, action)]}
+  end
+end

test/ex_admin_test.exs → test/resource_title_actions_test.exs

@@ -15,7 +15,7 @@ defmodule TestExAdmin.ExAdmin.SimpleCustom do
   end
 end
 
-defmodule ExAdminTest do
+defmodule ExAdmin.ResourceTitleActionsTest do
   use ExUnit.Case, async: true
 
   setup config do
@@ -33,50 +33,50 @@ defmodule ExAdminTest do
 
   @tag as_resource: %TestExAdmin.ExAdmin.Simple{}
   test "action_button", %{defn: defn, conn: conn} do
-    result = ExAdmin.action_button(conn, defn, "Simple", :show, :edit, defn.actions, "17")
+    result = ExAdmin.ResourceTitleActions.action_button(conn, defn, "Simple", :show, :edit, defn.actions, "17")
     assert result == [{"Edit Simple", [href: "/admin/simples/1/edit"]}]
 
-    result = ExAdmin.action_button(conn, defn, "Simple", :show, :new, defn.actions, "17")
+    result = ExAdmin.ResourceTitleActions.action_button(conn, defn, "Simple", :show, :new, defn.actions, "17")
     assert result == [{"New Simple", [href: "/admin/simples/new"]}]
 
-    result = ExAdmin.action_button(conn, defn, "Simple", :show, :delete, defn.actions, "17")
+    result = ExAdmin.ResourceTitleActions.action_button(conn, defn, "Simple", :show, :delete, defn.actions, "17")
     assert result == [{"Delete Simple", [href: "/admin/simples/1", "data-confirm": "Are you sure you want to delete this?", "data-method": :delete, rel: :nofollow]}]
 
-    result = ExAdmin.action_button(conn, defn, "Simple", :index, :new, defn.actions, "17")
+    result = ExAdmin.ResourceTitleActions.action_button(conn, defn, "Simple", :index, :new, defn.actions, "17")
     assert result == [{"New Simple", [href: "/admin/simples/new"]}]
 
-    result = ExAdmin.action_button(conn, defn, "Simple", :edit, :new, defn.actions, "17")
+    result = ExAdmin.ResourceTitleActions.action_button(conn, defn, "Simple", :edit, :new, defn.actions, "17")
     assert result == [{"New Simple", [href: "/admin/simples/new"]}]
   end
 
   @tag as_resource: %TestExAdmin.ExAdmin.Simple{}
-  test "default_resource_title_actions", %{defn: defn, conn: conn} do
+  test "default actions", %{defn: defn, conn: conn} do
     conn = struct(conn, private: %{phoenix_action: :show})
-    result = ExAdmin.default_resource_title_actions(conn, defn)
+    result = ExAdmin.ResourceTitleActions.default(conn, defn)
     assert result ==  [edit: [{"Edit Simple", [href: "/admin/simples/1/edit"]}], new: [{"New Simple", [href: "/admin/simples/new"]}], delete: [{"Delete Simple", [href: "/admin/simples/1", "data-confirm": "Are you sure you want to delete this?", "data-method": :delete, rel: :nofollow]}]]
 
     conn = struct(conn, private: %{phoenix_action: :index})
-    result = ExAdmin.default_resource_title_actions(conn, defn)
+    result = ExAdmin.ResourceTitleActions.default(conn, defn)
 
     assert result == [new: [{"New Simple", [href: "/admin/simples/new"]}]]
     conn = struct(conn, private: %{phoenix_action: :edit})
 
-    result = ExAdmin.default_resource_title_actions(conn, defn)
+    result = ExAdmin.ResourceTitleActions.default(conn, defn)
     assert result == [new: [{"New Simple", [href: "/admin/simples/new"]}]]
   end
 
   @tag as_resource: %TestExAdmin.ExAdmin.SimpleCustom{}
-  test "default_resource_title_actions custom actions", %{defn: defn, conn: conn} do
+  test "default custom actions", %{defn: defn, conn: conn} do
     conn = struct(conn, private: %{phoenix_action: :index})
-    result = ExAdmin.default_resource_title_actions(conn, defn)
+    result = ExAdmin.ResourceTitleActions.default(conn, defn)
     assert result ==  [
       new: [{"New Simple", [href: "/admin/simples/new"]}],
       custom: [{"Public Bulk", [href: "/admin/simples/collection/public_bulk"]}],
       custom: [{"Custom Action", [href: "/custom"]}]
     ]
 
     conn = struct(conn, private: %{phoenix_action: :show})
-    result = ExAdmin.default_resource_title_actions(conn, defn)
+    result = ExAdmin.ResourceTitleActions.default(conn, defn)
     assert result ==  [
       edit: [{"Edit Simple", [href: "/admin/simples/1/edit"]}],
       new: [{"New Simple", [href: "/admin/simples/new"]}],