From d8da5808696b32195dcbd061a672f317e2a88399 Mon Sep 17 00:00:00 2001
From: Michael <heluecht@pirati.ca>
Date: Fri, 22 Nov 2019 08:01:23 +0000
Subject: [PATCH] Fix mail delivery via AP when the contact is hidden

---
 src/Protocol/ActivityPub/Transmitter.php | 19 ++++++++++---------
 src/Worker/Notifier.php                  |  6 ++++++
 2 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/src/Protocol/ActivityPub/Transmitter.php b/src/Protocol/ActivityPub/Transmitter.php
index 5d3094339c..7fc0f5d990 100644
--- a/src/Protocol/ActivityPub/Transmitter.php
+++ b/src/Protocol/ActivityPub/Transmitter.php
@@ -498,13 +498,7 @@ class Transmitter
 			}
 		}
 
-		$receivers = ['to' => array_values($data['to']), 'cc' => array_values($data['cc']), 'bcc' => array_values($data['bcc'])];
-
-		if (!$blindcopy) {
-			unset($receivers['bcc']);
-		}
-
-		return $receivers;
+		return ['to' => array_values($data['to']), 'cc' => array_values($data['cc']), 'bcc' => array_values($data['bcc'])];
 	}
 
 	/**
@@ -699,8 +693,15 @@ class Transmitter
 		$mail = self::ItemArrayFromMail($mail_id);
 		$object = self::createNote($mail);
 
-		$object['to'] = $object['cc'];
-		unset($object['cc']);
+		if (!empty($object['cc'])) {
+			$object['to'] = array_merge($object['to'], $object['cc']);
+			unset($object['cc']);
+		}
+
+		if (!empty($object['bcc'])) {
+			$object['to'] = array_merge($object['to'], $object['bcc']);
+			unset($object['bcc']);
+		}
 
 		$object['tag'] = [['type' => 'Mention', 'href' => $object['to'][0], 'name' => 'test']];
 
diff --git a/src/Worker/Notifier.php b/src/Worker/Notifier.php
index 8b6be76b50..3e4efb5f3c 100644
--- a/src/Worker/Notifier.php
+++ b/src/Worker/Notifier.php
@@ -51,6 +51,8 @@ class Notifier
 
 		$delivery_contacts_stmt = null;
 		$target_item = [];
+		$parent = [];
+		$thr_parent = [];
 		$items = [];
 		$delivery_queue_count = 0;
 
@@ -594,6 +596,10 @@ class Notifier
 	 */
 	private static function skipDFRN($contact, $item, $parent, $thr_parent, $cmd)
 	{
+		if (empty($parent['network'])) {
+			return false;
+		}
+
 		// Don't skip when the starting post is delivered via Diaspora
 		if ($parent['network'] == Protocol::DIASPORA) {
 			return false;