nextcloud / server

apps/user_status/lib/Controller/StatusesController.php

Indentation +62 added lines, -62 removed lines

@@ -35,73 +35,73 @@
 
 class StatusesController extends OCSController {
 
-	/** @var StatusService */
-	private $service;
+    /** @var StatusService */
+    private $service;
 
-	/**
-	 * StatusesController constructor.
-	 *
-	 * @param string $appName
-	 * @param IRequest $request
-	 * @param StatusService $service
-	 */
-	public function __construct(string $appName,
-								IRequest $request,
-								StatusService $service) {
-		parent::__construct($appName, $request);
-		$this->service = $service;
-	}
+    /**
+     * StatusesController constructor.
+     *
+     * @param string $appName
+     * @param IRequest $request
+     * @param StatusService $service
+     */
+    public function __construct(string $appName,
+                                IRequest $request,
+                                StatusService $service) {
+        parent::__construct($appName, $request);
+        $this->service = $service;
+    }
 
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param int|null $limit
-	 * @param int|null $offset
-	 * @return DataResponse
-	 */
-	public function findAll(?int $limit=null, ?int $offset=null): DataResponse {
-		$allStatuses = $this->service->findAll($limit, $offset);
+    /**
+     * @NoAdminRequired
+     *
+     * @param int|null $limit
+     * @param int|null $offset
+     * @return DataResponse
+     */
+    public function findAll(?int $limit=null, ?int $offset=null): DataResponse {
+        $allStatuses = $this->service->findAll($limit, $offset);
 
-		return new DataResponse(array_map(function ($userStatus) {
-			return $this->formatStatus($userStatus);
-		}, $allStatuses));
-	}
+        return new DataResponse(array_map(function ($userStatus) {
+            return $this->formatStatus($userStatus);
+        }, $allStatuses));
+    }
 
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param string $userId
-	 * @return DataResponse
-	 * @throws OCSNotFoundException
-	 */
-	public function find(string $userId): DataResponse {
-		try {
-			$userStatus = $this->service->findByUserId($userId);
-		} catch (DoesNotExistException $ex) {
-			throw new OCSNotFoundException('No status for the requested userId');
-		}
+    /**
+     * @NoAdminRequired
+     *
+     * @param string $userId
+     * @return DataResponse
+     * @throws OCSNotFoundException
+     */
+    public function find(string $userId): DataResponse {
+        try {
+            $userStatus = $this->service->findByUserId($userId);
+        } catch (DoesNotExistException $ex) {
+            throw new OCSNotFoundException('No status for the requested userId');
+        }
 
-		return new DataResponse($this->formatStatus($userStatus));
-	}
+        return new DataResponse($this->formatStatus($userStatus));
+    }
 
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param UserStatus $status
-	 * @return array
-	 */
-	private function formatStatus(UserStatus $status): array {
-		$visibleStatus = $status->getStatus();
-		if ($visibleStatus === 'invisible') {
-			$visibleStatus = 'offline';
-		}
+    /**
+     * @NoAdminRequired
+     *
+     * @param UserStatus $status
+     * @return array
+     */
+    private function formatStatus(UserStatus $status): array {
+        $visibleStatus = $status->getStatus();
+        if ($visibleStatus === 'invisible') {
+            $visibleStatus = 'offline';
+        }
 
-		return [
-			'userId' => $status->getUserId(),
-			'message' => $status->getCustomMessage(),
-			'icon' => $status->getCustomIcon(),
-			'clearAt' => $status->getClearAt(),
-			'status' => $visibleStatus,
-		];
-	}
+        return [
+            'userId' => $status->getUserId(),
+            'message' => $status->getCustomMessage(),
+            'icon' => $status->getCustomIcon(),
+            'clearAt' => $status->getClearAt(),
+            'status' => $visibleStatus,
+        ];
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -59,10 +59,10 @@
 	 * @param int|null $offset
 	 * @return DataResponse
 	 */
-	public function findAll(?int $limit=null, ?int $offset=null): DataResponse {
+	public function findAll(?int $limit = null, ?int $offset = null): DataResponse {
 		$allStatuses = $this->service->findAll($limit, $offset);
 
-		return new DataResponse(array_map(function ($userStatus) {
+		return new DataResponse(array_map(function($userStatus) {
 			return $this->formatStatus($userStatus);
 		}, $allStatuses));
 	}

apps/user_status/lib/Controller/PredefinedStatusController.php

Indentation +23 added lines, -23 removed lines

@@ -37,29 +37,29 @@
  */
 class PredefinedStatusController extends OCSController {
 
-	/** @var PredefinedStatusService */
-	private $predefinedStatusService;
+    /** @var PredefinedStatusService */
+    private $predefinedStatusService;
 
-	/**
-	 * AStatusController constructor.
-	 *
-	 * @param string $appName
-	 * @param IRequest $request
-	 * @param PredefinedStatusService $predefinedStatusService
-	 */
-	public function __construct(string $appName,
-								IRequest $request,
-								PredefinedStatusService $predefinedStatusService) {
-		parent::__construct($appName, $request);
-		$this->predefinedStatusService = $predefinedStatusService;
-	}
+    /**
+     * AStatusController constructor.
+     *
+     * @param string $appName
+     * @param IRequest $request
+     * @param PredefinedStatusService $predefinedStatusService
+     */
+    public function __construct(string $appName,
+                                IRequest $request,
+                                PredefinedStatusService $predefinedStatusService) {
+        parent::__construct($appName, $request);
+        $this->predefinedStatusService = $predefinedStatusService;
+    }
 
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @return DataResponse
-	 */
-	public function findAll():DataResponse {
-		return new DataResponse($this->predefinedStatusService->getDefaultStatuses());
-	}
+    /**
+     * @NoAdminRequired
+     *
+     * @return DataResponse
+     */
+    public function findAll():DataResponse {
+        return new DataResponse($this->predefinedStatusService->getDefaultStatuses());
+    }
 }

apps/user_status/lib/Controller/HeartbeatController.php

Indentation +46 added lines, -46 removed lines

@@ -36,57 +36,57 @@
 
 class HeartbeatController extends Controller {
 
-	/** @var IEventDispatcher */
-	private $eventDispatcher;
+    /** @var IEventDispatcher */
+    private $eventDispatcher;
 
-	/** @var IUserSession */
-	private $userSession;
+    /** @var IUserSession */
+    private $userSession;
 
-	/** @var ITimeFactory */
-	private $timeFactory;
+    /** @var ITimeFactory */
+    private $timeFactory;
 
-	/**
-	 * HeartbeatController constructor.
-	 *
-	 * @param string $appName
-	 * @param IRequest $request
-	 * @param IEventDispatcher $eventDispatcher
-	 */
-	public function __construct(string $appName,
-								IRequest $request,
-								IEventDispatcher $eventDispatcher,
-								IUserSession $userSession,
-								ITimeFactory $timeFactory) {
-		parent::__construct($appName, $request);
-		$this->eventDispatcher = $eventDispatcher;
-		$this->userSession = $userSession;
-		$this->timeFactory = $timeFactory;
-	}
+    /**
+     * HeartbeatController constructor.
+     *
+     * @param string $appName
+     * @param IRequest $request
+     * @param IEventDispatcher $eventDispatcher
+     */
+    public function __construct(string $appName,
+                                IRequest $request,
+                                IEventDispatcher $eventDispatcher,
+                                IUserSession $userSession,
+                                ITimeFactory $timeFactory) {
+        parent::__construct($appName, $request);
+        $this->eventDispatcher = $eventDispatcher;
+        $this->userSession = $userSession;
+        $this->timeFactory = $timeFactory;
+    }
 
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param string $status
-	 * @return JSONResponse
-	 */
-	public function heartbeat(string $status): JSONResponse {
-		if (!\in_array($status, ['online', 'away'])) {
-			return new JSONResponse([], Http::STATUS_BAD_REQUEST);
-		}
+    /**
+     * @NoAdminRequired
+     *
+     * @param string $status
+     * @return JSONResponse
+     */
+    public function heartbeat(string $status): JSONResponse {
+        if (!\in_array($status, ['online', 'away'])) {
+            return new JSONResponse([], Http::STATUS_BAD_REQUEST);
+        }
 
-		$user = $this->userSession->getUser();
-		if ($user === null) {
-			return new JSONResponse([], Http::STATUS_INTERNAL_SERVER_ERROR);
-		}
+        $user = $this->userSession->getUser();
+        if ($user === null) {
+            return new JSONResponse([], Http::STATUS_INTERNAL_SERVER_ERROR);
+        }
 
-		$this->eventDispatcher->dispatchTyped(
-			new UserLiveStatusEvent(
-				$user,
-				$status,
-				$this->timeFactory->getTime()
-			)
-		);
+        $this->eventDispatcher->dispatchTyped(
+            new UserLiveStatusEvent(
+                $user,
+                $status,
+                $this->timeFactory->getTime()
+            )
+        );
 
-		return new JSONResponse([], Http::STATUS_NO_CONTENT);
-	}
+        return new JSONResponse([], Http::STATUS_NO_CONTENT);
+    }
 }

apps/user_status/lib/Controller/UserStatusController.php

Indentation +147 added lines, -147 removed lines

@@ -41,151 +41,151 @@
 
 class UserStatusController extends OCSController {
 
-	/** @var string */
-	private $userId;
-
-	/** @var ILogger */
-	private $logger;
-
-	/** @var StatusService */
-	private $service;
-
-	/**
-	 * StatusesController constructor.
-	 *
-	 * @param string $appName
-	 * @param IRequest $request
-	 * @param string $userId
-	 * @param ILogger $logger;
-	 * @param StatusService $service
-	 */
-	public function __construct(string $appName,
-								IRequest $request,
-								string $userId,
-								ILogger $logger,
-								StatusService $service) {
-		parent::__construct($appName, $request);
-		$this->userId = $userId;
-		$this->logger = $logger;
-		$this->service = $service;
-	}
-
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @return DataResponse
-	 * @throws OCSNotFoundException
-	 */
-	public function getStatus(): DataResponse {
-		try {
-			$userStatus = $this->service->findByUserId($this->userId);
-		} catch (DoesNotExistException $ex) {
-			throw new OCSNotFoundException('No status for the current user');
-		}
-
-		return new DataResponse($this->formatStatus($userStatus));
-	}
-
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param string $statusType
-	 * @return DataResponse
-	 * @throws OCSBadRequestException
-	 */
-	public function setStatus(string $statusType): DataResponse {
-		try {
-			$status = $this->service->setStatus($this->userId, $statusType, null, true);
-			return new DataResponse($this->formatStatus($status));
-		} catch (InvalidStatusTypeException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid status type "' . $statusType . '"');
-			throw new OCSBadRequestException($ex->getMessage(), $ex);
-		}
-	}
-
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param string $messageId
-	 * @param int|null $clearAt
-	 * @return DataResponse
-	 * @throws OCSBadRequestException
-	 */
-	public function setPredefinedMessage(string $messageId,
-										 ?int $clearAt): DataResponse {
-		try {
-			$status = $this->service->setPredefinedMessage($this->userId, $messageId, $clearAt);
-			return new DataResponse($this->formatStatus($status));
-		} catch (InvalidClearAtException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid clearAt value "' . $clearAt . '"');
-			throw new OCSBadRequestException($ex->getMessage(), $ex);
-		} catch (InvalidMessageIdException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid message-id "' . $messageId . '"');
-			throw new OCSBadRequestException($ex->getMessage(), $ex);
-		}
-	}
-
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @param string|null $statusIcon
-	 * @param string $message
-	 * @param int|null $clearAt
-	 * @return DataResponse
-	 * @throws OCSBadRequestException
-	 */
-	public function setCustomMessage(?string $statusIcon,
-									 string $message,
-									 ?int $clearAt): DataResponse {
-		try {
-			$status = $this->service->setCustomMessage($this->userId, $statusIcon, $message, $clearAt);
-			return new DataResponse($this->formatStatus($status));
-		} catch (InvalidClearAtException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid clearAt value "' . $clearAt . '"');
-			throw new OCSBadRequestException($ex->getMessage(), $ex);
-		} catch (InvalidStatusIconException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid icon value "' . $statusIcon . '"');
-			throw new OCSBadRequestException($ex->getMessage(), $ex);
-		} catch (StatusMessageTooLongException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to a too long status message.');
-			throw new OCSBadRequestException($ex->getMessage(), $ex);
-		}
-	}
-
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @return DataResponse
-	 */
-	public function clearStatus(): DataResponse {
-		$this->service->clearStatus($this->userId);
-		return new DataResponse([]);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 *
-	 * @return DataResponse
-	 */
-	public function clearMessage(): DataResponse {
-		$this->service->clearMessage($this->userId);
-		return new DataResponse([]);
-	}
-
-	/**
-	 * @param UserStatus $status
-	 * @return array
-	 */
-	private function formatStatus(UserStatus $status): array {
-		return [
-			'userId' => $status->getUserId(),
-			'message' => $status->getCustomMessage(),
-			'messageId' => $status->getMessageId(),
-			'messageIsPredefined' => $status->getMessageId() !== null,
-			'icon' => $status->getCustomIcon(),
-			'clearAt' => $status->getClearAt(),
-			'status' => $status->getStatus(),
-			'statusIsUserDefined' => $status->getIsUserDefined(),
-		];
-	}
+    /** @var string */
+    private $userId;
+
+    /** @var ILogger */
+    private $logger;
+
+    /** @var StatusService */
+    private $service;
+
+    /**
+     * StatusesController constructor.
+     *
+     * @param string $appName
+     * @param IRequest $request
+     * @param string $userId
+     * @param ILogger $logger;
+     * @param StatusService $service
+     */
+    public function __construct(string $appName,
+                                IRequest $request,
+                                string $userId,
+                                ILogger $logger,
+                                StatusService $service) {
+        parent::__construct($appName, $request);
+        $this->userId = $userId;
+        $this->logger = $logger;
+        $this->service = $service;
+    }
+
+    /**
+     * @NoAdminRequired
+     *
+     * @return DataResponse
+     * @throws OCSNotFoundException
+     */
+    public function getStatus(): DataResponse {
+        try {
+            $userStatus = $this->service->findByUserId($this->userId);
+        } catch (DoesNotExistException $ex) {
+            throw new OCSNotFoundException('No status for the current user');
+        }
+
+        return new DataResponse($this->formatStatus($userStatus));
+    }
+
+    /**
+     * @NoAdminRequired
+     *
+     * @param string $statusType
+     * @return DataResponse
+     * @throws OCSBadRequestException
+     */
+    public function setStatus(string $statusType): DataResponse {
+        try {
+            $status = $this->service->setStatus($this->userId, $statusType, null, true);
+            return new DataResponse($this->formatStatus($status));
+        } catch (InvalidStatusTypeException $ex) {
+            $this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid status type "' . $statusType . '"');
+            throw new OCSBadRequestException($ex->getMessage(), $ex);
+        }
+    }
+
+    /**
+     * @NoAdminRequired
+     *
+     * @param string $messageId
+     * @param int|null $clearAt
+     * @return DataResponse
+     * @throws OCSBadRequestException
+     */
+    public function setPredefinedMessage(string $messageId,
+                                         ?int $clearAt): DataResponse {
+        try {
+            $status = $this->service->setPredefinedMessage($this->userId, $messageId, $clearAt);
+            return new DataResponse($this->formatStatus($status));
+        } catch (InvalidClearAtException $ex) {
+            $this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid clearAt value "' . $clearAt . '"');
+            throw new OCSBadRequestException($ex->getMessage(), $ex);
+        } catch (InvalidMessageIdException $ex) {
+            $this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid message-id "' . $messageId . '"');
+            throw new OCSBadRequestException($ex->getMessage(), $ex);
+        }
+    }
+
+    /**
+     * @NoAdminRequired
+     *
+     * @param string|null $statusIcon
+     * @param string $message
+     * @param int|null $clearAt
+     * @return DataResponse
+     * @throws OCSBadRequestException
+     */
+    public function setCustomMessage(?string $statusIcon,
+                                        string $message,
+                                     ?int $clearAt): DataResponse {
+        try {
+            $status = $this->service->setCustomMessage($this->userId, $statusIcon, $message, $clearAt);
+            return new DataResponse($this->formatStatus($status));
+        } catch (InvalidClearAtException $ex) {
+            $this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid clearAt value "' . $clearAt . '"');
+            throw new OCSBadRequestException($ex->getMessage(), $ex);
+        } catch (InvalidStatusIconException $ex) {
+            $this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid icon value "' . $statusIcon . '"');
+            throw new OCSBadRequestException($ex->getMessage(), $ex);
+        } catch (StatusMessageTooLongException $ex) {
+            $this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to a too long status message.');
+            throw new OCSBadRequestException($ex->getMessage(), $ex);
+        }
+    }
+
+    /**
+     * @NoAdminRequired
+     *
+     * @return DataResponse
+     */
+    public function clearStatus(): DataResponse {
+        $this->service->clearStatus($this->userId);
+        return new DataResponse([]);
+    }
+
+    /**
+     * @NoAdminRequired
+     *
+     * @return DataResponse
+     */
+    public function clearMessage(): DataResponse {
+        $this->service->clearMessage($this->userId);
+        return new DataResponse([]);
+    }
+
+    /**
+     * @param UserStatus $status
+     * @return array
+     */
+    private function formatStatus(UserStatus $status): array {
+        return [
+            'userId' => $status->getUserId(),
+            'message' => $status->getCustomMessage(),
+            'messageId' => $status->getMessageId(),
+            'messageIsPredefined' => $status->getMessageId() !== null,
+            'icon' => $status->getCustomIcon(),
+            'clearAt' => $status->getClearAt(),
+            'status' => $status->getStatus(),
+            'statusIsUserDefined' => $status->getIsUserDefined(),
+        ];
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -98,7 +98,7 @@ 
 			$status = $this->service->setStatus($this->userId, $statusType, null, true);
 			return new DataResponse($this->formatStatus($status));
 		} catch (InvalidStatusTypeException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid status type "' . $statusType . '"');
+			$this->logger->debug('New user-status for "'.$this->userId.'" was rejected due to an invalid status type "'.$statusType.'"');
 			throw new OCSBadRequestException($ex->getMessage(), $ex);
 		}
 	}
@@ -117,10 +117,10 @@ 
 			$status = $this->service->setPredefinedMessage($this->userId, $messageId, $clearAt);
 			return new DataResponse($this->formatStatus($status));
 		} catch (InvalidClearAtException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid clearAt value "' . $clearAt . '"');
+			$this->logger->debug('New user-status for "'.$this->userId.'" was rejected due to an invalid clearAt value "'.$clearAt.'"');
 			throw new OCSBadRequestException($ex->getMessage(), $ex);
 		} catch (InvalidMessageIdException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid message-id "' . $messageId . '"');
+			$this->logger->debug('New user-status for "'.$this->userId.'" was rejected due to an invalid message-id "'.$messageId.'"');
 			throw new OCSBadRequestException($ex->getMessage(), $ex);
 		}
 	}
@@ -141,13 +141,13 @@ 
 			$status = $this->service->setCustomMessage($this->userId, $statusIcon, $message, $clearAt);
 			return new DataResponse($this->formatStatus($status));
 		} catch (InvalidClearAtException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid clearAt value "' . $clearAt . '"');
+			$this->logger->debug('New user-status for "'.$this->userId.'" was rejected due to an invalid clearAt value "'.$clearAt.'"');
 			throw new OCSBadRequestException($ex->getMessage(), $ex);
 		} catch (InvalidStatusIconException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to an invalid icon value "' . $statusIcon . '"');
+			$this->logger->debug('New user-status for "'.$this->userId.'" was rejected due to an invalid icon value "'.$statusIcon.'"');
 			throw new OCSBadRequestException($ex->getMessage(), $ex);
 		} catch (StatusMessageTooLongException $ex) {
-			$this->logger->debug('New user-status for "' . $this->userId . '" was rejected due to a too long status message.');
+			$this->logger->debug('New user-status for "'.$this->userId.'" was rejected due to a too long status message.');
 			throw new OCSBadRequestException($ex->getMessage(), $ex);
 		}
 	}

apps/user_status/lib/Listener/BeforeTemplateRenderedListener.php

Indentation +32 added lines, -32 removed lines

@@ -34,42 +34,42 @@
 
 class BeforeTemplateRenderedListener implements IEventListener {
 
-	/** @var IInitialStateService */
-	private $initialState;
+    /** @var IInitialStateService */
+    private $initialState;
 
-	/** @var JSDataService */
-	private $jsDataService;
+    /** @var JSDataService */
+    private $jsDataService;
 
-	/**
-	 * BeforeTemplateRenderedListener constructor.
-	 *
-	 * @param IInitialStateService $initialState
-	 * @param JSDataService $jsDataService
-	 */
-	public function __construct(IInitialStateService $initialState,
-								JSDataService $jsDataService) {
-		$this->initialState = $initialState;
-		$this->jsDataService = $jsDataService;
-	}
+    /**
+     * BeforeTemplateRenderedListener constructor.
+     *
+     * @param IInitialStateService $initialState
+     * @param JSDataService $jsDataService
+     */
+    public function __construct(IInitialStateService $initialState,
+                                JSDataService $jsDataService) {
+        $this->initialState = $initialState;
+        $this->jsDataService = $jsDataService;
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function handle(Event $event): void {
-		if (!($event instanceof BeforeTemplateRenderedEvent)) {
-			// Unrelated
-			return;
-		}
+    /**
+     * @inheritDoc
+     */
+    public function handle(Event $event): void {
+        if (!($event instanceof BeforeTemplateRenderedEvent)) {
+            // Unrelated
+            return;
+        }
 
-		if (!$event->isLoggedIn()) {
-			return;
-		}
+        if (!$event->isLoggedIn()) {
+            return;
+        }
 
-		$this->initialState->provideLazyInitialState(Application::APP_ID, 'status', function () {
-			return $this->jsDataService;
-		});
+        $this->initialState->provideLazyInitialState(Application::APP_ID, 'status', function () {
+            return $this->jsDataService;
+        });
 
-		\OCP\Util::addScript('user_status', 'user-status-menu');
-		\OCP\Util::addStyle('user_status', 'user-status-menu');
-	}
+        \OCP\Util::addScript('user_status', 'user-status-menu');
+        \OCP\Util::addStyle('user_status', 'user-status-menu');
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 			return;
 		}
 
-		$this->initialState->provideLazyInitialState(Application::APP_ID, 'status', function () {
+		$this->initialState->provideLazyInitialState(Application::APP_ID, 'status', function() {
 			return $this->jsDataService;
 		});
 

apps/user_status/lib/Listener/UserLiveStatusListener.php

Indentation +90 added lines, -90 removed lines

@@ -40,94 +40,94 @@
  */
 class UserLiveStatusListener implements IEventListener {
 
-	/** @var UserStatusMapper */
-	private $mapper;
-
-	/** @var ITimeFactory */
-	private $timeFactory;
-
-	/** @var string[] */
-	private $priorityOrderedStatuses = [
-		'online',
-		'away',
-		'dnd',
-		'invisible',
-		'offline'
-	];
-
-	/** @var string[] */
-	private $persistentUserStatuses = [
-		'away',
-		'dnd',
-		'invisible',
-	];
-
-	/** @var int */
-	private $offlineThreshold = 300;
-
-	/**
-	 * UserLiveStatusListener constructor.
-	 *
-	 * @param UserStatusMapper $mapper
-	 * @param ITimeFactory $timeFactory
-	 */
-	public function __construct(UserStatusMapper $mapper,
-								ITimeFactory $timeFactory) {
-		$this->mapper = $mapper;
-		$this->timeFactory = $timeFactory;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function handle(Event $event): void {
-		if (!($event instanceof UserLiveStatusEvent)) {
-			// Unrelated
-			return;
-		}
-
-		$user = $event->getUser();
-		try {
-			$userStatus = $this->mapper->findByUserId($user->getUID());
-		} catch (DoesNotExistException $ex) {
-			$userStatus = new UserStatus();
-			$userStatus->setUserId($user->getUID());
-			$userStatus->setStatus('offline');
-			$userStatus->setStatusTimestamp(0);
-			$userStatus->setIsUserDefined(false);
-		}
-
-		// If the status is user-defined and one of the persistent statuses, we
-		// will not override it.
-		if ($userStatus->getIsUserDefined() &&
-			\in_array($userStatus->getStatus(), $this->persistentUserStatuses, true)) {
-			return;
-		}
-
-		$needsUpdate = false;
-
-		// If the current status is older than 5 minutes,
-		// treat it as outdated and update
-		if ($userStatus->getStatusTimestamp() < ($this->timeFactory->getTime() - $this->offlineThreshold)) {
-			$needsUpdate = true;
-		}
-
-		// If the emitted status is more important than the current status
-		// treat it as outdated and update
-		if (array_search($event->getStatus(), $this->priorityOrderedStatuses) < array_search($userStatus->getStatus(), $this->priorityOrderedStatuses)) {
-			$needsUpdate = true;
-		}
-
-		if ($needsUpdate) {
-			$userStatus->setStatus($event->getStatus());
-			$userStatus->setStatusTimestamp($event->getTimestamp());
-			$userStatus->setIsUserDefined(false);
-
-			if ($userStatus->getId() === null) {
-				$this->mapper->insert($userStatus);
-			} else {
-				$this->mapper->update($userStatus);
-			}
-		}
-	}
+    /** @var UserStatusMapper */
+    private $mapper;
+
+    /** @var ITimeFactory */
+    private $timeFactory;
+
+    /** @var string[] */
+    private $priorityOrderedStatuses = [
+        'online',
+        'away',
+        'dnd',
+        'invisible',
+        'offline'
+    ];
+
+    /** @var string[] */
+    private $persistentUserStatuses = [
+        'away',
+        'dnd',
+        'invisible',
+    ];
+
+    /** @var int */
+    private $offlineThreshold = 300;
+
+    /**
+     * UserLiveStatusListener constructor.
+     *
+     * @param UserStatusMapper $mapper
+     * @param ITimeFactory $timeFactory
+     */
+    public function __construct(UserStatusMapper $mapper,
+                                ITimeFactory $timeFactory) {
+        $this->mapper = $mapper;
+        $this->timeFactory = $timeFactory;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function handle(Event $event): void {
+        if (!($event instanceof UserLiveStatusEvent)) {
+            // Unrelated
+            return;
+        }
+
+        $user = $event->getUser();
+        try {
+            $userStatus = $this->mapper->findByUserId($user->getUID());
+        } catch (DoesNotExistException $ex) {
+            $userStatus = new UserStatus();
+            $userStatus->setUserId($user->getUID());
+            $userStatus->setStatus('offline');
+            $userStatus->setStatusTimestamp(0);
+            $userStatus->setIsUserDefined(false);
+        }
+
+        // If the status is user-defined and one of the persistent statuses, we
+        // will not override it.
+        if ($userStatus->getIsUserDefined() &&
+            \in_array($userStatus->getStatus(), $this->persistentUserStatuses, true)) {
+            return;
+        }
+
+        $needsUpdate = false;
+
+        // If the current status is older than 5 minutes,
+        // treat it as outdated and update
+        if ($userStatus->getStatusTimestamp() < ($this->timeFactory->getTime() - $this->offlineThreshold)) {
+            $needsUpdate = true;
+        }
+
+        // If the emitted status is more important than the current status
+        // treat it as outdated and update
+        if (array_search($event->getStatus(), $this->priorityOrderedStatuses) < array_search($userStatus->getStatus(), $this->priorityOrderedStatuses)) {
+            $needsUpdate = true;
+        }
+
+        if ($needsUpdate) {
+            $userStatus->setStatus($event->getStatus());
+            $userStatus->setStatusTimestamp($event->getTimestamp());
+            $userStatus->setIsUserDefined(false);
+
+            if ($userStatus->getId() === null) {
+                $this->mapper->insert($userStatus);
+            } else {
+                $this->mapper->update($userStatus);
+            }
+        }
+    }
 }

apps/user_status/lib/Listener/UserDeletedListener.php

Indentation +21 added lines, -21 removed lines

@@ -37,29 +37,29 @@
  */
 class UserDeletedListener implements IEventListener {
 
-	/** @var StatusService */
-	private $service;
+    /** @var StatusService */
+    private $service;
 
-	/**
-	 * UserDeletedListener constructor.
-	 *
-	 * @param StatusService $service
-	 */
-	public function __construct(StatusService $service) {
-		$this->service = $service;
-	}
+    /**
+     * UserDeletedListener constructor.
+     *
+     * @param StatusService $service
+     */
+    public function __construct(StatusService $service) {
+        $this->service = $service;
+    }
 
 
-	/**
-	 * @inheritDoc
-	 */
-	public function handle(Event $event): void {
-		if (!($event instanceof UserDeletedEvent)) {
-			// Unrelated
-			return;
-		}
+    /**
+     * @inheritDoc
+     */
+    public function handle(Event $event): void {
+        if (!($event instanceof UserDeletedEvent)) {
+            // Unrelated
+            return;
+        }
 
-		$user = $event->getUser();
-		$this->service->removeUserStatus($user->getUID());
-	}
+        $user = $event->getUser();
+        $this->service->removeUserStatus($user->getUID());
+    }
 }

apps/user_status/appinfo/routes.php

Indentation +16 added lines, -16 removed lines

@@ -24,20 +24,20 @@
  */
 
 return [
-	'ocs' => [
-		// Routes for querying statuses
-		['name' => 'Statuses#findAll', 'url' => '/api/v1/statuses', 'verb' => 'GET'],
-		['name' => 'Statuses#find', 'url' => '/api/v1/statuses/{userId}', 'verb' => 'GET'],
-		// Routes for manipulating your own status
-		['name' => 'UserStatus#getStatus', 'url' => '/api/v1/user_status', 'verb' => 'GET'],
-		['name' => 'UserStatus#setStatus', 'url' => '/api/v1/user_status/status', 'verb' => 'PUT'],
-		['name' => 'UserStatus#setPredefinedMessage', 'url' => '/api/v1/user_status/message/predefined', 'verb' => 'PUT'],
-		['name' => 'UserStatus#setCustomMessage', 'url' => '/api/v1/user_status/message/custom', 'verb' => 'PUT'],
-		['name' => 'UserStatus#clearMessage', 'url' => '/api/v1/user_status/message', 'verb' => 'DELETE'],
-		// Routes for listing default routes
-		['name' => 'PredefinedStatus#findAll', 'url' => '/api/v1/predefined_statuses/', 'verb' => 'GET']
-	],
-	'routes' => [
-		['name' => 'Heartbeat#heartbeat', 'url' => '/heartbeat', 'verb' => 'PUT'],
-	],
+    'ocs' => [
+        // Routes for querying statuses
+        ['name' => 'Statuses#findAll', 'url' => '/api/v1/statuses', 'verb' => 'GET'],
+        ['name' => 'Statuses#find', 'url' => '/api/v1/statuses/{userId}', 'verb' => 'GET'],
+        // Routes for manipulating your own status
+        ['name' => 'UserStatus#getStatus', 'url' => '/api/v1/user_status', 'verb' => 'GET'],
+        ['name' => 'UserStatus#setStatus', 'url' => '/api/v1/user_status/status', 'verb' => 'PUT'],
+        ['name' => 'UserStatus#setPredefinedMessage', 'url' => '/api/v1/user_status/message/predefined', 'verb' => 'PUT'],
+        ['name' => 'UserStatus#setCustomMessage', 'url' => '/api/v1/user_status/message/custom', 'verb' => 'PUT'],
+        ['name' => 'UserStatus#clearMessage', 'url' => '/api/v1/user_status/message', 'verb' => 'DELETE'],
+        // Routes for listing default routes
+        ['name' => 'PredefinedStatus#findAll', 'url' => '/api/v1/predefined_statuses/', 'verb' => 'GET']
+    ],
+    'routes' => [
+        ['name' => 'Heartbeat#heartbeat', 'url' => '/heartbeat', 'verb' => 'PUT'],
+    ],
 ];

apps/user_status/composer/autoload.php

Spacing +1 added lines, -1 removed lines

@@ -2,6 +2,6 @@
 
 // autoload.php @generated by Composer
 
-require_once __DIR__ . '/composer/autoload_real.php';
+require_once __DIR__.'/composer/autoload_real.php';
 
 return ComposerAutoloaderInitUserStatus::getLoader();