fix: Fix code quality based on review

Signed-off-by: Joas Schilling <[email protected]>

lib/Chat/ChatManager.php

@@ -274,7 +274,7 @@ public function sendMessage(Room $chat, ?Participant $participant, string $actor
 		try {
 			$this->commentsManager->save($comment);
 
-			if ($participant !== null) {
+			if ($participant instanceof Participant) {
 				$this->participantService->updateLastReadMessage($participant, (int) $comment->getId());
 			}
 

lib/Chat/Parser/SystemMessage.php

@@ -566,7 +566,6 @@ public function parseDeletedMessage(Message $chatMessage): void {
 		$parsedParameters = ['actor' => $this->getActor($room, $data['deleted_by_type'], $data['deleted_by_id'])];
 
 		$participant = $chatMessage->getParticipant();
-		$currentActorId = $participant?->getAttendee()->getActorId();
 
 		$authorIsActor = $data['deleted_by_type'] === $chatMessage->getComment()->getActorType()
 			&& $data['deleted_by_id'] === $chatMessage->getComment()->getActorId();
@@ -576,11 +575,11 @@ public function parseDeletedMessage(Message $chatMessage): void {
 		} elseif (!$participant->isGuest()) {
 			$currentUserIsActor = $parsedParameters['actor']['type'] === 'user' &&
 				$participant->getAttendee()->getActorType() === Attendee::ACTOR_USERS &&
-				$currentActorId === $parsedParameters['actor']['id'];
+				$participant->getAttendee()->getActorId() === $parsedParameters['actor']['id'];
 		} else {
 			$currentUserIsActor = $parsedParameters['actor']['type'] === 'guest' &&
 				$participant->getAttendee()->getActorType() === 'guest' &&
-				$currentActorId === $parsedParameters['actor']['id'];
+				$participant->getAttendee()->getActorId() === $parsedParameters['actor']['id'];
 		}
 
 		if ($chatMessage->getMessageType() === ChatManager::VERB_MESSAGE_DELETED) {

lib/Chat/Parser/UserMention.php

@@ -117,7 +117,7 @@ public function parseMessage(Message $chatMessage): void {
 			if ($mention['type'] === 'call') {
 				$userId = '';
 				if ($chatMessage->getParticipant()?->getAttendee()->getActorType() === Attendee::ACTOR_USERS) {
-					$userId = $chatMessage->getParticipant()?->getAttendee()->getActorId();
+					$userId = $chatMessage->getParticipant()->getAttendee()->getActorId();
 				}
 
 				$messageParameters[$mentionParameterId] = [

lib/Migration/Version18000Date20230504205823.php

@@ -66,6 +66,7 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 			]);
 			$table->addColumn('error_count', Types::BIGINT, [
 				'default' => 0,
+				'unsigned' => true,
 			]);
 			$table->addColumn('last_error_date', Types::DATETIME, [
 				'notnull' => false,
@@ -77,6 +78,7 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 			$table->addColumn('state', Types::SMALLINT, [
 				'default' => 0,
 				'notnull' => false,
+				'unsigned' => true,
 			]);
 
 			$table->setPrimaryKey(['id']);
@@ -90,6 +92,7 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 			]);
 			$table->addColumn('bot_id', Types::BIGINT, [
 				'default' => 0,
+				'unsigned' => true,
 			]);
 			$table->addColumn('token', Types::STRING, [
 				'length' => 64,
@@ -98,6 +101,7 @@ public function changeSchema(IOutput $output, Closure $schemaClosure, array $opt
 			$table->addColumn('state', Types::SMALLINT, [
 				'default' => 0,
 				'notnull' => false,
+				'unsigned' => true,
 			]);
 
 			$table->setPrimaryKey(['id']);

lib/Service/BotService.php

@@ -95,7 +95,7 @@ public function afterChatMessageSent(ChatParticipantEvent $event, MessageParser
 				'type' => 'Note',
 				'id' => $event->getComment()->getId(),
 				'name' => 'message',
-				'content' => json_encode($messageData),
+				'content' => json_encode($messageData, JSON_THROW_ON_ERROR),
 				'mediaType' => 'text/markdown', // FIXME or text/plain when markdown is disabled
 			],
 			'target' => [

tests/integration/features/bootstrap/CommandLineTrait.php

@@ -161,7 +161,6 @@ public function theCommandOutputContainsTheText($text) {
 			Assert::assertTrue(false, 'The command did not output the expected text on stdout but stderr');
 		}
 
-		var_dump($this->lastStdOut);
 		Assert::assertStringContainsString($text, $this->lastStdOut, 'The command did not output the expected text on stdout');
 	}