Add reindex_after_update helper to comments, images, tags contexts

lib/philomena/comments.ex

@@ -166,15 +166,7 @@ defmodule Philomena.Comments do
     comment
     |> Comment.unhide_changeset()
     |> Repo.update()
-    |> case do
-      {:ok, comment} ->
-        reindex_comment(comment)
-
-        {:ok, comment}
-
-      error ->
-        error
-    end
+    |> reindex_after_update()
   end
 
   @doc """
@@ -190,6 +182,19 @@ defmodule Philomena.Comments do
     comment
     |> Comment.destroy_changeset()
     |> Repo.update()
+    |> reindex_after_update()
+  end
+
+  defp reindex_after_update(result) do
+    case result do
+      {:ok, comment} ->
+        reindex_comment(comment)
+
+        {:ok, comment}
+
+      error ->
+        error
+    end
   end
 
   @doc """

lib/philomena/images.ex

@@ -289,6 +289,7 @@ defmodule Philomena.Images do
     image
     |> Image.lock_comments_changeset(locked)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -304,6 +305,7 @@ defmodule Philomena.Images do
     image
     |> Image.lock_description_changeset(locked)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -319,6 +321,7 @@ defmodule Philomena.Images do
     image
     |> Image.lock_tags_changeset(locked)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -335,6 +338,7 @@ defmodule Philomena.Images do
     image
     |> Image.remove_hash_changeset()
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -350,6 +354,7 @@ defmodule Philomena.Images do
     image
     |> Image.scratchpad_changeset(attrs)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -366,6 +371,7 @@ defmodule Philomena.Images do
     |> Repo.preload(:source_changes)
     |> Image.remove_source_history_changeset()
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -447,6 +453,7 @@ defmodule Philomena.Images do
     image
     |> Image.changeset(attrs)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -462,6 +469,7 @@ defmodule Philomena.Images do
     image
     |> Image.description_changeset(attrs)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -570,6 +578,7 @@ defmodule Philomena.Images do
     |> Repo.preload(:locked_tags)
     |> Image.locked_tags_changeset(attrs, new_tags)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -751,6 +760,7 @@ defmodule Philomena.Images do
     image
     |> Image.uploader_changeset(attrs)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -766,6 +776,7 @@ defmodule Philomena.Images do
     image
     |> Image.anonymous_changeset(attrs)
     |> Repo.update()
+    |> reindex_after_update()
   end
 
   @doc """
@@ -784,7 +795,11 @@ defmodule Philomena.Images do
     image
     |> Image.hide_reason_changeset(attrs)
     |> Repo.update()
-    |> case do
+    |> reindex_after_update()
+  end
+
+  defp reindex_after_update(result) do
+    case result do
       {:ok, image} ->
         reindex_image(image)
 

lib/philomena/tags.ex

@@ -176,7 +176,11 @@ defmodule Philomena.Tags do
     tag
     |> Tag.changeset(attrs, implied_tags)
     |> Repo.update()
-    |> case do
+    |> reindex_after_update()
+  end
+
+  defp reindex_after_update(result) do
+    case result do
       {:ok, tag} ->
         reindex_tag(tag)
 
@@ -483,6 +487,7 @@ defmodule Philomena.Tags do
     |> case do
       {:ok, _} = result ->
         reindex_tag_images(former_alias)
+        reindex_tags([tag, former_alias])
 
         result