Merge pull request #6888 from annando/issue-6167-a

Issue 6167: Don't publish non visible content

src/Protocol/ActivityPub/Transmitter.php

@@ -152,7 +152,7 @@ class Transmitter
 
 		$condition = ['uid' => 0, 'contact-id' => $public_contact, 'author-id' => $public_contact,
 			'private' => false, 'gravity' => [GRAVITY_PARENT, GRAVITY_COMMENT],
-			'deleted' => false, 'visible' => true];
+			'deleted' => false, 'visible' => true, 'moderated' => false];
 		$count = DBA::count('item', $condition);
 
 		$data = ['@context' => ActivityPub::CONTEXT];

src/Protocol/DFRN.php

@@ -254,7 +254,7 @@ class DFRN
 			FROM `item` USE INDEX (`uid_wall_changed`) $sql_post_table
 			STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
 			WHERE `item`.`uid` = %d AND `item`.`wall` AND `item`.`changed` > '%s'
-			$sql_extra
+			AND `item`.`visible` $sql_extra
 			ORDER BY `item`.`parent` ".$sort.", `item`.`created` ASC LIMIT 0, 300",
 			intval($owner_id),
 			DBA::escape($check_date),

src/Worker/Delivery.php

@@ -60,7 +60,7 @@ class Delivery extends BaseObject
 			}
 			$parent_id = intval($item['parent']);
 
-			$condition = ['id' => [$target_id, $parent_id], 'moderated' => false];
+			$condition = ['id' => [$target_id, $parent_id], 'visible' => true, 'moderated' => false];
 			$params = ['order' => ['id']];
 			$itemdata = Model\Item::select([], $condition, $params);
 

src/Worker/Notifier.php

@@ -99,11 +99,6 @@ class Notifier
 				return;
 			}
 
-			// Issue 6167: We don't distribute image uploads
-			if ($target_item['post-type'] == Item::PT_IMAGE) {
-				return;
-			}
-
 			if (!empty($target_item['contact-uid'])) {
 				$uid = $target_item['contact-uid'];
 			} elseif (!empty($target_item['uid'])) {