fix(room): Fix undefined variable in federated calls

Signed-off-by: Joas Schilling <coding@schilljs.com>
nextcloud · Nov 21, 2024 · 90b6a0c · 90b6a0c
1 parent 7d9f7e3
commit 90b6a0c
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/lib/Federation/Proxy/TalkV1/Controller/CallController.php b/lib/Federation/Proxy/TalkV1/Controller/CallController.php
@@ -32,7 +32,7 @@ public function __construct(
 	}
 
 	/**
-	 * @see \OCA\Talk\Controller\RoomController::getPeersForCall()
+	 * @see \OCA\Talk\Controller\CallController::getPeersForCall()
 	 *
 	 * @param Room $room the federated room to get the call peers
 	 * @param Participant $participant the federated user to get the call peers
@@ -64,7 +64,7 @@ public function getPeersForCall(Room $room, Participant $participant): DataRespo
 	}
 
 	/**
-	 * @see \OCA\Talk\Controller\RoomController::joinFederatedCall()
+	 * @see \OCA\Talk\Controller\CallController::joinFederatedCall()
 	 *
 	 * @param Room $room the federated room to join the call in
 	 * @param Participant $participant the federated user that will join the
@@ -111,7 +111,7 @@ public function joinFederatedCall(Room $room, Participant $participant, int $fla
 	}
 
 	/**
-	 * @see \OCA\Talk\Controller\RoomController::ringAttendee()
+	 * @see \OCA\Talk\Controller\CallController::ringAttendee()
 	 *
 	 * @param int $attendeeId ID of the attendee to ring
 	 * @return DataResponse<Http::STATUS_OK|Http::STATUS_NOT_FOUND, null, array{}>|DataResponse<Http::STATUS_BAD_REQUEST, array{error: string}, array{}>
@@ -144,7 +144,7 @@ public function ringAttendee(Room $room, Participant $participant, int $attendee
 	}
 
 	/**
-	 * @see \OCA\Talk\Controller\RoomController::updateFederatedCallFlags()
+	 * @see \OCA\Talk\Controller\CallController::updateFederatedCallFlags()
 	 *
 	 * @param Room $room the federated room to update the call flags in
 	 * @param Participant $participant the federated user to update the call
@@ -181,7 +181,7 @@ public function updateFederatedCallFlags(Room $room, Participant $participant, i
 	}
 
 	/**
-	 * @see \OCA\Talk\Controller\RoomController::leaveFederatedCall()
+	 * @see \OCA\Talk\Controller\CallController::leaveFederatedCall()
 	 *
 	 * @param Room $room the federated room to leave the call in
 	 * @param Participant $participant the federated user that will leave the

diff --git a/lib/Federation/Proxy/TalkV1/Controller/ReactionController.php b/lib/Federation/Proxy/TalkV1/Controller/ReactionController.php
@@ -38,6 +38,8 @@ public function __construct(
 	 * 201: Reaction added successfully
 	 * 400: Adding reaction is not possible
 	 * 404: Message not found
+	 *
+	 * @see \OCA\Talk\Controller\ReactionController::react()
 	 */
 	public function react(Room $room, Participant $participant, int $messageId, string $reaction, string $format): DataResponse {
 		$proxy = $this->proxy->post(
@@ -78,6 +80,8 @@ public function react(Room $room, Participant $participant, int $messageId, stri
 	 * 200: Reaction deleted successfully
 	 * 400: Deleting reaction is not possible
 	 * 404: Message not found
+	 *
+	 * @see \OCA\Talk\Controller\ReactionController::delete()
 	 */
 	public function delete(Room $room, Participant $participant, int $messageId, string $reaction, string $format): DataResponse {
 		$proxy = $this->proxy->delete(
@@ -118,6 +122,8 @@ public function delete(Room $room, Participant $participant, int $messageId, str
 	 *
 	 * 200: Reactions returned
 	 * 404: Message or reaction not found
+	 *
+	 * @see \OCA\Talk\Controller\ReactionController::getReactions()
 	 */
 	public function getReactions(Room $room, Participant $participant, int $messageId, ?string $reaction, string $format): DataResponse {
 		$proxy = $this->proxy->get(

diff --git a/lib/Service/RoomFormatter.php b/lib/Service/RoomFormatter.php
@@ -284,6 +284,7 @@ public function formatRoomV4(
 
 		$roomData['canStartCall'] = $currentParticipant->canStartCall($this->serverConfig);
 
+		$currentUser = null;
 		if ($attendee->getActorType() === Attendee::ACTOR_USERS) {
 			$currentUser = $this->userManager->get($attendee->getActorId());
 			if ($room->isFederatedConversation()) {
@@ -396,6 +397,7 @@ public function formatRoomV4(
 			} catch (DoesNotExistException) {
 			}
 		}
+
 		if ($currentUser instanceof IUser
 			&& $attendee->getActorType() === Attendee::ACTOR_USERS
 			&& $roomData['lastReadMessage'] === ChatManager::UNREAD_FIRST_MESSAGE