Replace map |> join with map_join

benchmarks/load_testing/activities.ex

@@ -224,8 +224,7 @@ defp hell_thread_mentions(users) do
         |> Enum.reduce([users[:user]], fn group, acc ->
           acc ++ Enum.take(users[group], 5)
         end)
-        |> Enum.map(&"@#{&1.nickname}")
+        |> Enum.map_join(", ", &"@#{&1.nickname}")
-        |> Enum.join(", ")
 
       Cachex.put(:user_cache, "hell_thread_mentions", cached)
       cached

lib/mix/tasks/pleroma/instance.ex

@@ -303,7 +303,7 @@ def run(["gen" | rest]) do
     else
       shell_error(
         "The task would have overwritten the following files:\n" <>
-          (Enum.map(will_overwrite, &"- #{&1}\n") |> Enum.join("")) <>
+          Enum.map_join(will_overwrite, "", &"- #{&1}\n") <>
           "Rerun with `--force` to overwrite them."
       )
     end

lib/pleroma/emails/admin_email.ex

@@ -43,7 +43,7 @@ def report(to, reporter, account, statuses, comment) do
       if is_list(statuses) && length(statuses) > 0 do
         statuses_list_html =
           statuses
-          |> Enum.map(fn
+          |> Enum.map_join("\n", fn
             %{id: id} ->
               status_url = Helpers.o_status_url(Pleroma.Web.Endpoint, :notice, id)
               "<li><a href=\"#{status_url}\">#{status_url}</li>"
@@ -54,7 +54,6 @@ def report(to, reporter, account, statuses, comment) do
             id when is_binary(id) ->
               "<li><a href=\"#{id}\">#{id}</li>"
           end)
-          |> Enum.join("\n")
 
         """
         <p> Statuses:

lib/pleroma/emoji.ex

@@ -134,10 +134,9 @@ defp update_emojis(emojis) do
       |> hd()
       |> String.trim()
       |> String.split()
-      |> Enum.map(fn codepoint ->
+      |> Enum.map_join(fn codepoint ->
         <<String.to_integer(codepoint, 16)::utf8>>
       end)
-      |> Enum.join()
     end)
     |> Enum.uniq()
 

lib/pleroma/filter.ex

@@ -207,8 +207,7 @@ def compose_regex(%User{} = user, format) do
   def compose_regex([_ | _] = filters, format) do
     phrases =
       filters
-      |> Enum.map(& &1.phrase)
+      |> Enum.map_join("|", & &1.phrase)
-      |> Enum.join("|")
 
     case format do
       :postgres ->

lib/pleroma/moderation_log.ex

@@ -556,14 +556,12 @@ def get_log_entry_message(%ModerationLog{
 
   defp nicknames_to_string(nicknames) do
     nicknames
-    |> Enum.map(&"@#{&1}")
+    |> Enum.map_join(", ", &"@#{&1}")
-    |> Enum.join(", ")
   end
 
   defp users_to_nicknames_string(users) do
     users
-    |> Enum.map(&"@#{&1["nickname"]}")
+    |> Enum.map_join(", ", &"@#{&1["nickname"]}")
-    |> Enum.join(", ")
   end
 
   defp subject_actor_nickname(%ModerationLog{data: data}, prefix_msg, postfix_msg \\ "") do

lib/pleroma/user/search.ex

@@ -134,8 +134,7 @@ defp to_tsquery(query_string) do
     |> String.replace(~r/[!-\/|@|[-`|{-~|:-?]+/, " ")
     |> String.trim()
     |> String.split()
-    |> Enum.map(&(&1 <> ":*"))
+    |> Enum.map_join(" | ", &(&1 <> ":*"))
-    |> Enum.join(" | ")
   end
 
   # Considers nickname match, localized nickname match, name match; preferences nickname match

lib/pleroma/web/api_spec/render_error.ex

@@ -30,7 +30,7 @@ def call(conn, errors) do
     conn
     |> put_status(:bad_request)
     |> json(%{
-      error: errors |> Enum.map(&message/1) |> Enum.join(" "),
+      error: errors |> Enum.map_join(" ", &message/1),
       errors: errors |> Enum.map(&render_error/1)
     })
   end

lib/pleroma/web/mastodon_api/controllers/search_controller.ex

@@ -167,8 +167,7 @@ defp preprocess_uri_query(query) do
 
   defp joined_tag(tags) do
     tags
-    |> Enum.map(fn tag -> String.capitalize(tag) end)
+    |> Enum.map_join(fn tag -> String.capitalize(tag) end)
-    |> Enum.join()
   end
 
   defp with_fallback(f, fallback \\ []) do

lib/pleroma/web/metadata.ex

@@ -15,8 +15,7 @@ def build_static_tags(params) do
         params
         |> parser.build_tags()
         |> Enum.map(&to_tag/1)
-        |> Enum.map(&HTML.safe_to_string/1)
+        |> Enum.map_join(&HTML.safe_to_string/1)
-        |> Enum.join()
 
       acc <> rendered_html
     end)
@@ -35,8 +34,7 @@ def build_tags(params) do
         params
         |> parser.build_tags()
         |> Enum.map(&to_tag/1)
-        |> Enum.map(&HTML.safe_to_string/1)
+        |> Enum.map_join(&HTML.safe_to_string/1)
-        |> Enum.join()
 
       acc <> rendered_html
     end)

lib/pleroma/web/plugs/rate_limiter.ex

@@ -244,8 +244,7 @@ defp attach_selected_params(input, %{conn_params: conn_params, opts: plug_opts})
       plug_opts
       |> Keyword.get(:params, [])
       |> Enum.sort()
-      |> Enum.map(&Map.get(conn_params, &1, ""))
+      |> Enum.map_join(":", &Map.get(conn_params, &1, ""))
-      |> Enum.join(":")
 
     [input, params_string]
     |> Enum.join(":")

test/pleroma/moderation_log_test.exs

@@ -94,8 +94,7 @@ test "logging user tagged by admin", %{admin: admin, subject1: subject1, subject
 
       users =
         [subject1.nickname, subject2.nickname]
-        |> Enum.map(&"@#{&1}")
+        |> Enum.map_join(", ", &"@#{&1}")
-        |> Enum.join(", ")
 
       tags = ["foo", "bar"] |> Enum.join(", ")
 
@@ -115,8 +114,7 @@ test "logging user untagged by admin", %{admin: admin, subject1: subject1, subje
 
       users =
         [subject1.nickname, subject2.nickname]
-        |> Enum.map(&"@#{&1}")
+        |> Enum.map_join(", ", &"@#{&1}")
-        |> Enum.join(", ")
 
       tags = ["foo", "bar"] |> Enum.join(", ")
 

test/pleroma/web/admin_api/controllers/admin_api_controller_test.exs

@@ -122,8 +122,7 @@ test "it appends specified tags to users with specified nicknames", %{
 
       users =
         [user1.nickname, user2.nickname]
-        |> Enum.map(&"@#{&1}")
+        |> Enum.map_join(", ", &"@#{&1}")
-        |> Enum.join(", ")
 
       tags = ["foo", "bar"] |> Enum.join(", ")
 
@@ -168,8 +167,7 @@ test "it removes specified tags from users with specified nicknames", %{
 
       users =
         [user1.nickname, user2.nickname]
-        |> Enum.map(&"@#{&1}")
+        |> Enum.map_join(", ", &"@#{&1}")
-        |> Enum.join(", ")
 
       tags = ["x", "z"] |> Enum.join(", ")
 

test/pleroma/web/mastodon_api/controllers/conversation_controller_test.exs

@@ -247,8 +247,7 @@ test "Removes a conversation", %{user: user_one, conn: conn} do
   defp create_direct_message(sender, recips) do
     hellos =
       recips
-      |> Enum.map(fn s -> "@#{s.nickname}" end)
+      |> Enum.map_join(", ", fn s -> "@#{s.nickname}" end)
-      |> Enum.join(", ")
 
     CommonAPI.post(sender, %{
       status: "Hi #{hellos}!",

test/pleroma/web/mastodon_api/update_credentials_test.exs

@@ -630,8 +630,8 @@ test "update fields when invalid request", %{conn: conn} do
       name_limit = Pleroma.Config.get([:instance, :account_field_name_length])
       value_limit = Pleroma.Config.get([:instance, :account_field_value_length])
 
-      long_name = Enum.map(0..name_limit, fn _ -> "x" end) |> Enum.join()
+      long_name = Enum.map_join(0..name_limit, fn _ -> "x" end)
-      long_value = Enum.map(0..value_limit, fn _ -> "x" end) |> Enum.join()
+      long_value = Enum.map_join(0..value_limit, fn _ -> "x" end)
 
       fields = [%{name: "foo", value: long_value}]