[#394] Refactoring.

lib/pleroma/user.ex

@@ -846,12 +846,7 @@ defp tag_or_untag([hd | _] = users, tags, action) when is_map(hd) do
 
     Repo.transaction(fn ->
       for user <- users do
-        new_tags =
+        new_tags = mutate_tags(user, tags, action)
-          if action == :tag do
-            Enum.uniq(user.tags ++ tags)
-          else
-            user.tags -- tags
-          end
 
         {:ok, updated_user} =
           user
@@ -862,4 +857,8 @@ defp tag_or_untag([hd | _] = users, tags, action) when is_map(hd) do
       end
     end)
   end
+
+  defp mutate_tags(user, tags, :tag), do: Enum.uniq(user.tags ++ tags)
+
+  defp mutate_tags(user, tags, :untag), do: user.tags -- tags
 end

lib/pleroma/web/mastodon_api/views/account_view.ex

@@ -59,7 +59,7 @@ def render("account.json", %{user: user} = opts) do
         privacy: user_info.default_scope,
         sensitive: false
       },
-      # Note: Mastodon does not return this field:
+      # Pleroma extension
       tags: user.tags
     }
   end

lib/pleroma/web/twitter_api/views/user_view.ex

@@ -78,7 +78,7 @@ def render("user.json", %{user: user = %User{}} = assigns) do
       "default_scope" => user.info.default_scope,
       "no_rich_text" => user.info.no_rich_text,
       "fields" => fields,
-      # Note: twitter.com does not return this field:
+      # Pleroma extension
       "tags" => user.tags
     }