add option to not unfollow on block, and option to not federate outgoing blocks

config/config.exs

@@ -57,7 +57,10 @@
   public: true,
   quarantined_instances: []
 
-config :pleroma, :activitypub, accept_blocks: true
+config :pleroma, :activitypub,
+  accept_blocks: true,
+  unfollow_blocked: true,
+  outgoing_blocks: true
 
 config :pleroma, :mrf_simple,
   media_removal: [],

lib/pleroma/web/activity_pub/activity_pub.ex

@@ -240,17 +240,27 @@ def delete(%Object{data: %{"id" => id, "actor" => actor}} = object, local \\ tru
     end
   end
 
-  def block(blocker, blocked, activity_id \\ nil, local \\ true) do
+  @ap_config Application.get_env(:pleroma, :activitypub)
-    follow_activity = fetch_latest_follow(blocker, blocked)
+  @unfollow_blocked Keyword.get(@ap_config, :unfollow_blocked)
+  @outgoing_blocks Keyword.get(@ap_config, :outgoing_blocks)
 
-    if follow_activity do
+  def block(blocker, blocked, activity_id \\ nil, local \\ true) do
-      unfollow(blocker, blocked, nil, local)
+
+    with true <- unfollow_blocked do
+      follow_activity = fetch_latest_follow(blocker, blocked)
+      if follow_activity do
+        unfollow(blocker, blocked, nil, local)
+      end
     end
 
-    with block_data <- make_block_data(blocker, blocked, activity_id),
+    with true <- outgoing_blocks do
-         {:ok, activity} <- insert(block_data, local),
+      with block_data <- make_block_data(blocker, blocked, activity_id),
-         :ok <- maybe_federate(activity) do
+           {:ok, activity} <- insert(block_data, local),
-      {:ok, activity}
+           :ok <- maybe_federate(activity) do
+        {:ok, activity}
+      end
+    else
+      {:ok, nil}
     end
   end