Merge pull request #1775 from nextcloud/fix/noid/missing-actor-avatar-on-objectstream

fill status.actor.avatar

lib/Db/SocialCrossQueryBuilder.php

@@ -141,7 +141,11 @@ class SocialCrossQueryBuilder extends SocialCoreQueryBuilder {
 	 * @return Stream
 	 * @throws InvalidResourceException
 	 */
-	public function parseLeftJoinStream(array $data, string $prefix = ''): Stream {
+	public function parseLeftJoinStream(
+		array $data,
+		string $prefix = '',
+		int $exportFormat = 0
+	): Stream {
 		$new = [];
 		foreach ($data as $k => $v) {
 			if (str_starts_with($k, $prefix)) {
@@ -149,13 +153,15 @@ class SocialCrossQueryBuilder extends SocialCoreQueryBuilder {
 			}
 		}
 
-		$stream = new Stream();
-		$stream->importFromDatabase($new);
-		if ($stream->getId() === '') {
+		if (($new['nid'] ?? '') === '') {
 			throw new InvalidResourceException();
 		}
 
-		$actor = $this->parseLeftJoinCacheActors($data, $prefix . 'cacheactor_');
+		$stream = new Stream();
+		$stream->importFromDatabase($new);
+		$stream->setExportFormat($exportFormat);
+
+		$actor = $this->parseLeftJoinCacheActors($data, $prefix . 'cacheactor_', $exportFormat);
 		$stream->setActor($actor);
 
 		return $stream;
@@ -168,7 +174,11 @@ class SocialCrossQueryBuilder extends SocialCoreQueryBuilder {
 	 * @return Person
 	 * @throws InvalidResourceException
 	 */
-	public function parseLeftJoinCacheActors(array $data, string $prefix = ''): Person {
+	public function parseLeftJoinCacheActors(
+		array $data,
+		string $prefix = '',
+		int $exportFormat = 0
+	): Person {
 		$new = [];
 
 		foreach ($data as $k => $v) {
@@ -179,9 +189,10 @@ class SocialCrossQueryBuilder extends SocialCoreQueryBuilder {
 
 		$actor = new Person();
 		$actor->importFromDatabase($new);
+		$actor->setExportFormat($exportFormat);
 
 		if (!AP::$activityPub->isActor($actor)) {
-			throw new InvalidResourceException();
+			throw new InvalidResourceException('actor not actor');
 		}
 
 		try {

lib/Db/StreamRequest.php

@@ -586,6 +586,7 @@ class StreamRequest extends StreamRequestBuilder {
 		$qb->limitToDest($actor->getId(), 'notif', '', 'sd');
 		$qb->linkToCacheActors('ca', 's.attributed_to_prim');
 		$qb->leftJoinStreamAction();
+		$qb->leftJoinObjectStatus();
 
 		return $this->getStreamsFromRequest($qb);
 	}

lib/Db/StreamRequestBuilder.php

@@ -36,6 +36,7 @@ use OCA\Social\Exceptions\InvalidResourceException;
 use OCA\Social\Exceptions\ItemUnknownException;
 use OCA\Social\Exceptions\SocialAppConfigException;
 use OCA\Social\Exceptions\StreamNotFoundException;
+use OCA\Social\Model\ActivityPub\ACore;
 use OCA\Social\Model\ActivityPub\Object\Announce;
 use OCA\Social\Model\ActivityPub\Stream;
 use OCA\Social\Model\InstancePath;
@@ -217,7 +218,7 @@ class StreamRequestBuilder extends CoreRequestBuilder {
 		}
 
 		try {
-			$actor = $qb->parseLeftJoinCacheActors($data, 'cacheactor_');
+			$actor = $qb->parseLeftJoinCacheActors($data, 'cacheactor_', $qb->getFormat());
 			$actor->setExportFormat($qb->getFormat());
 			$item->setCompleteDetails(true);
 			$item->setActor($actor);
@@ -225,7 +226,7 @@ class StreamRequestBuilder extends CoreRequestBuilder {
 		}
 
 		try {
-			$object = $qb->parseLeftJoinStream($data, 'objectstream_');
+			$object = $qb->parseLeftJoinStream($data, 'objectstream_', ACore::FORMAT_LOCAL);
 			$item->setObject($object);
 		} catch (InvalidResourceException $e) {
 		}

lib/Model/ActivityPub/ACore.php

@@ -652,7 +652,9 @@ class ACore extends Item implements JsonSerializable, IQueryRow {
 	 * @return $this
 	 */
 	public function setExportFormat(int $format): self {
-		$this->format = $format;
+		if ($format > 0) {
+			$this->format = $format;
+		}
 
 		return $this;
 	}

lib/Model/ActivityPub/Stream.php

@@ -684,19 +684,11 @@ class Stream extends ACore implements IQueryRow, JsonSerializable {
 				$type = '';
 		}
 
-		$status = null;
-		$statusPost = $this->getDetails('post');
-		if (sizeof($statusPost) > 0) {
-			$status = new Stream();
-			$status->importFromLocal($statusPost);
-			$status->setExportFormat(self::FORMAT_LOCAL);
-		}
-
 		$result = [
 			'id' => (string)$this->getNid(),
 			'type' => $type,
 			'created_at' => date('Y-m-d\TH:i:s', $this->getPublishedTime()) . '.000Z',
-			'status' => $status,
+			'status' => $this->getObject(),
 		];
 
 		if ($this->hasActor()) {