From 5826e300ce202193109062590ab9322d15a6b849 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Sat, 19 Oct 2024 23:04:16 +0200
Subject: [PATCH 1/2] fix(activity): Fix parameter type of call activity

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 lib/Activity/Provider/Base.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/Activity/Provider/Base.php b/lib/Activity/Provider/Base.php
index 9a2817776f1..9597a6169fa 100644
--- a/lib/Activity/Provider/Base.php
+++ b/lib/Activity/Provider/Base.php
@@ -108,7 +108,7 @@ protected function getRoom(Room $room, string $userId): array {
 
 		return [
 			'type' => 'call',
-			'id' => $room->getId(),
+			'id' => (string)$room->getId(),
 			'name' => $room->getDisplayName($userId),
 			'link' => $this->url->linkToRouteAbsolute('spreed.Page.showCall', ['token' => $room->getToken()]),
 			'call-type' => $stringType,

From fa6c521cc018ad234891160066647b61494493da Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Sat, 19 Oct 2024 23:12:00 +0200
Subject: [PATCH 2/2] fix(activity): Also fix former conversations

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 lib/Activity/Provider/Base.php                 |  7 +++----
 lib/Activity/Provider/Invitation.php           |  4 ++--
 tests/php/Activity/Provider/InvitationTest.php | 10 ++++++----
 3 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/lib/Activity/Provider/Base.php b/lib/Activity/Provider/Base.php
index 9597a6169fa..667f010a06f 100644
--- a/lib/Activity/Provider/Base.php
+++ b/lib/Activity/Provider/Base.php
@@ -116,12 +116,11 @@ protected function getRoom(Room $room, string $userId): array {
 		];
 	}
 
-	protected function getFormerRoom(IL10N $l, int $roomId): array {
+	protected function getFormerRoom(IL10N $l): array {
 		return [
-			'type' => 'call',
-			'id' => $roomId,
+			'type' => 'highlight',
+			'id' => 'deleted',
 			'name' => $l->t('a conversation'),
-			'call-type' => Room::TYPE_UNKNOWN,
 		];
 	}
 
diff --git a/lib/Activity/Provider/Invitation.php b/lib/Activity/Provider/Invitation.php
index bd75646cade..5a824867729 100644
--- a/lib/Activity/Provider/Invitation.php
+++ b/lib/Activity/Provider/Invitation.php
@@ -42,11 +42,11 @@ public function parse($language, IEvent $event, ?IEvent $previousEvent = null):
 			$l = $this->languageFactory->get('spreed', $language);
 			$parameters = $event->getSubjectParameters();
 
-			$roomParameter = $this->getFormerRoom($l, (int) $parameters['room']);
 			try {
 				$room = $this->manager->getRoomById((int) $parameters['room']);
 				$roomParameter = $this->getRoom($room, $event->getAffectedUser());
-			} catch (RoomNotFoundException $e) {
+			} catch (RoomNotFoundException) {
+				$roomParameter = $this->getFormerRoom($l);
 			}
 
 			$this->setSubjects($event, $l->t('{actor} invited you to {call}'), [
diff --git a/tests/php/Activity/Provider/InvitationTest.php b/tests/php/Activity/Provider/InvitationTest.php
index f6edb1c18bd..d15eb920e1c 100644
--- a/tests/php/Activity/Provider/InvitationTest.php
+++ b/tests/php/Activity/Provider/InvitationTest.php
@@ -192,6 +192,8 @@ public function testParse($lang, $roomExists, array $params, array $expectedPara
 				->method('getRoom')
 				->with($room, 'user')
 				->willReturn(['call-data']);
+			$provider->expects($this->never())
+				->method('getFormerRoom');
 		} else {
 			$this->manager->expects($this->once())
 				->method('getRoomById')
@@ -200,6 +202,10 @@ public function testParse($lang, $roomExists, array $params, array $expectedPara
 
 			$provider->expects($this->never())
 				->method('getRoom');
+			$provider->expects($this->once())
+				->method('getFormerRoom')
+				->with($l)
+				->willReturn(['call-unknown']);
 		}
 
 		$this->l10nFactory->expects($this->once())
@@ -218,10 +224,6 @@ public function testParse($lang, $roomExists, array $params, array $expectedPara
 			->method('getUser')
 			->with($params['user'])
 			->willReturn(['actor-data']);
-		$provider->expects($this->once())
-			->method('getFormerRoom')
-			->with($l, $params['room'])
-			->willReturn(['call-unknown']);
 
 		$provider->parse($lang, $event);
 	}