1 files changed, 17 insertions, 15 deletions

diff --git a/apps/dav/lib/CalDAV/Status/StatusService.php b/apps/dav/lib/CalDAV/Status/StatusService.php
index be8ad90dd45..9ee0e9bf356 100644
--- a/apps/dav/lib/CalDAV/Status/StatusService.php
+++ b/apps/dav/lib/CalDAV/Status/StatusService.php
@@ -27,35 +27,37 @@ use Sabre\CalDAV\Xml\Property\ScheduleCalendarTransp;
 
 class StatusService {
 	private ICache $cache;
-	public function __construct(private ITimeFactory $timeFactory,
+	public function __construct(
+		private ITimeFactory $timeFactory,
 		private IManager $calendarManager,
 		private IUserManager $userManager,
 		private UserStatusService $userStatusService,
 		private IAvailabilityCoordinator $availabilityCoordinator,
 		private ICacheFactory $cacheFactory,
-		private LoggerInterface $logger) {
+		private LoggerInterface $logger,
+	) {
 		$this->cache = $cacheFactory->createLocal('CalendarStatusService');
 	}
 
 	public function processCalendarStatus(string $userId): void {
 		$user = $this->userManager->get($userId);
-		if($user === null) {
+		if ($user === null) {
 			return;
 		}
 
 		$availability = $this->availabilityCoordinator->getCurrentOutOfOfficeData($user);
-		if($availability !== null && $this->availabilityCoordinator->isInEffect($availability)) {
+		if ($availability !== null && $this->availabilityCoordinator->isInEffect($availability)) {
 			$this->logger->debug('An Absence is in effect, skipping calendar status check', ['user' => $userId]);
 			return;
 		}
 
 		$calendarEvents = $this->cache->get($userId);
-		if($calendarEvents === null) {
+		if ($calendarEvents === null) {
 			$calendarEvents = $this->getCalendarEvents($user);
 			$this->cache->set($userId, $calendarEvents, 300);
 		}
 
-		if(empty($calendarEvents)) {
+		if (empty($calendarEvents)) {
 			try {
 				$this->userStatusService->revertUserStatus($userId, IUserStatus::MESSAGE_CALENDAR_BUSY);
 			} catch (Exception $e) {
@@ -81,9 +83,9 @@ class StatusService {
 			$currentStatus = null;
 		}
 
-		if($currentStatus !== null && $currentStatus->getMessageId() === IUserStatus::MESSAGE_CALL
-			|| $currentStatus !== null && $currentStatus->getStatus() === IUserStatus::DND
-			|| $currentStatus !== null && $currentStatus->getStatus() === IUserStatus::INVISIBLE) {
+		if (($currentStatus !== null && $currentStatus->getMessageId() === IUserStatus::MESSAGE_CALL)
+			|| ($currentStatus !== null && $currentStatus->getStatus() === IUserStatus::DND)
+			|| ($currentStatus !== null && $currentStatus->getStatus() === IUserStatus::INVISIBLE)) {
 			// We don't overwrite the call status, DND status or Invisible status
 			$this->logger->debug('Higher priority status detected, skipping calendar status change', ['user' => $userId]);
 			return;
@@ -101,7 +103,7 @@ class StatusService {
 			if (isset($component['DTSTART']) && $userStatusTimestamp !== null) {
 				/** @var DateTimeImmutable $dateTime */
 				$dateTime = $component['DTSTART'][0];
-				if($dateTime instanceof DateTimeImmutable && $userStatusTimestamp > $dateTime->getTimestamp()) {
+				if ($dateTime instanceof DateTimeImmutable && $userStatusTimestamp > $dateTime->getTimestamp()) {
 					return false;
 				}
 			}
@@ -112,7 +114,7 @@ class StatusService {
 			return true;
 		});
 
-		if(empty($applicableEvents)) {
+		if (empty($applicableEvents)) {
 			try {
 				$this->userStatusService->revertUserStatus($userId, IUserStatus::MESSAGE_CALENDAR_BUSY);
 			} catch (Exception $e) {
@@ -130,7 +132,7 @@ class StatusService {
 		}
 
 		// Only update the status if it's neccesary otherwise we mess up the timestamp
-		if($currentStatus === null || $currentStatus->getMessageId() !== IUserStatus::MESSAGE_CALENDAR_BUSY) {
+		if ($currentStatus === null || $currentStatus->getMessageId() !== IUserStatus::MESSAGE_CALENDAR_BUSY) {
 			// One event that fulfills all status conditions is enough
 			// 1. Not an OOO event
 			// 2. Current user status (that is not a calendar status) was not set after the start of this event
@@ -139,7 +141,7 @@ class StatusService {
 			$this->logger->debug("Found $count applicable event(s), changing user status", ['user' => $userId]);
 			$this->userStatusService->setUserStatus(
 				$userId,
-				IUserStatus::AWAY,
+				IUserStatus::BUSY,
 				IUserStatus::MESSAGE_CALENDAR_BUSY,
 				true
 			);
@@ -148,7 +150,7 @@ class StatusService {
 
 	private function getCalendarEvents(User $user): array {
 		$calendars = $this->calendarManager->getCalendarsForPrincipal('principals/users/' . $user->getUID());
-		if(empty($calendars)) {
+		if (empty($calendars)) {
 			return [];
 		}
 
@@ -172,7 +174,7 @@ class StatusService {
 		$dtEnd = DateTimeImmutable::createFromMutable($this->timeFactory->getDateTime('+5 minutes'));
 
 		// Only query the calendars when there's any to search
-		if($query instanceof CalendarQuery && !empty($query->getCalendarUris())) {
+		if ($query instanceof CalendarQuery && !empty($query->getCalendarUris())) {
 			// Query the next hour
 			$query->setTimerangeStart($dtStart);
 			$query->setTimerangeEnd($dtEnd);