feat(email): Allow inviting email guests to private conversations

Signed-off-by: Joas Schilling <coding@schilljs.com>

Author: nickvergessen

lib/Controller/RoomController.php

@@ -1426,13 +1426,6 @@ public function addParticipantToRoom(string $newParticipant, string $source = 'u
 
 			$this->participantService->addCircle($this->room, $circle, $participants);
 		} elseif ($source === 'emails') {
-			$data = [];
-			try {
-				$this->roomService->setType($this->room, Room::TYPE_PUBLIC);
-				$data = ['type' => $this->room->getType()];
-			} catch (TypeException) {
-			}
-
 			$email = strtolower($newParticipant);
 			$actorId = hash('sha256', $email);
 			try {
@@ -1442,7 +1435,7 @@ public function addParticipantToRoom(string $newParticipant, string $source = 'u
 				$this->guestManager->sendEmailInvitation($this->room, $participant);
 			}
 
-			return new DataResponse($data);
+			return new DataResponse([]);
 		} elseif ($source === 'federated_users') {
 			if (!$this->talkConfig->isFederationEnabled()) {
 				return new DataResponse(['error' => 'federation'], Http::STATUS_NOT_IMPLEMENTED);

lib/Service/RoomService.php

@@ -688,11 +688,13 @@ public function setType(Room $room, int $newType, bool $allowSwitchingOneToOne =
 		$room->setType($newType);
 
 		if ($oldType === Room::TYPE_PUBLIC) {
-			// Kick all guests and users that were not invited
+			// Kick all guests that are not email invited
+			// and all users that joined the public link
 			$delete = $this->db->getQueryBuilder();
 			$delete->delete('talk_attendees')
 				->where($delete->expr()->eq('room_id', $delete->createNamedParameter($room->getId(), IQueryBuilder::PARAM_INT)))
-				->andWhere($delete->expr()->in('participant_type', $delete->createNamedParameter([Participant::GUEST, Participant::GUEST_MODERATOR, Participant::USER_SELF_JOINED], IQueryBuilder::PARAM_INT_ARRAY)));
+				->andWhere($delete->expr()->in('participant_type', $delete->createNamedParameter([Participant::GUEST, Participant::GUEST_MODERATOR, Participant::USER_SELF_JOINED], IQueryBuilder::PARAM_INT_ARRAY)))
+				->andWhere($delete->expr()->neq('actor_type', $delete->createNamedParameter(Attendee::ACTOR_EMAILS, IQueryBuilder::PARAM_INT)));
 			$delete->executeStatement();
 		}