diff --git a/config/container.php b/config/container.php index 3c03b935..ace4700c 100644 --- a/config/container.php +++ b/config/container.php @@ -191,5 +191,5 @@ // Add shared UserNetworkSessionData to the container so that the DTO can be populated and used in different places UserNetworkSessionData::class => function () { return new UserNetworkSessionData(); - } + }, ]; diff --git a/src/Application/Action/Authentication/Ajax/AccountUnlockProcessAction.php b/src/Application/Action/Authentication/Ajax/AccountUnlockProcessAction.php index 4010c658..2ab7cfb4 100644 --- a/src/Application/Action/Authentication/Ajax/AccountUnlockProcessAction.php +++ b/src/Application/Action/Authentication/Ajax/AccountUnlockProcessAction.php @@ -20,7 +20,7 @@ public function __construct( private RedirectHandler $redirectHandler, private SessionManagerInterface $sessionManager, private SessionInterface $session, - private AccountUnlockTokenVerifier $accountUnlockTokenVerifier + private AccountUnlockTokenVerifier $accountUnlockTokenVerifier, ) { } diff --git a/src/Application/Action/Authentication/Ajax/RegisterVerifyProcessAction.php b/src/Application/Action/Authentication/Ajax/RegisterVerifyProcessAction.php index 3dd033c7..52b6c188 100644 --- a/src/Application/Action/Authentication/Ajax/RegisterVerifyProcessAction.php +++ b/src/Application/Action/Authentication/Ajax/RegisterVerifyProcessAction.php @@ -25,7 +25,7 @@ public function __construct( private RouteParserInterface $routeParser, private SessionManagerInterface $sessionManager, private SessionInterface $session, - private RegisterTokenVerifier $registerTokenVerifier + private RegisterTokenVerifier $registerTokenVerifier, ) { } diff --git a/src/Application/Action/Client/Ajax/ClientCreateAction.php b/src/Application/Action/Client/Ajax/ClientCreateAction.php index 70be30ba..e2206269 100644 --- a/src/Application/Action/Client/Ajax/ClientCreateAction.php +++ b/src/Application/Action/Client/Ajax/ClientCreateAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $clientValues = (array)$request->getParsedBody(); diff --git a/src/Application/Action/Client/Ajax/ClientCreateDropdownOptionsFetchAction.php b/src/Application/Action/Client/Ajax/ClientCreateDropdownOptionsFetchAction.php index 5cf13bcc..076e277b 100644 --- a/src/Application/Action/Client/Ajax/ClientCreateDropdownOptionsFetchAction.php +++ b/src/Application/Action/Client/Ajax/ClientCreateDropdownOptionsFetchAction.php @@ -27,7 +27,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $dropdownOptions = $this->clientUtilFinder->findClientDropdownValues(); diff --git a/src/Application/Action/Client/Ajax/ClientDeleteAction.php b/src/Application/Action/Client/Ajax/ClientDeleteAction.php index a99b0d15..b2c62c00 100644 --- a/src/Application/Action/Client/Ajax/ClientDeleteAction.php +++ b/src/Application/Action/Client/Ajax/ClientDeleteAction.php @@ -20,7 +20,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $clientId = (int)$args['client_id']; diff --git a/src/Application/Action/Client/Ajax/ClientFetchListAction.php b/src/Application/Action/Client/Ajax/ClientFetchListAction.php index 3fb0af23..18e7406e 100644 --- a/src/Application/Action/Client/Ajax/ClientFetchListAction.php +++ b/src/Application/Action/Client/Ajax/ClientFetchListAction.php @@ -21,7 +21,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { try { // Retrieve clients with given filter values (or none) diff --git a/src/Application/Action/Client/Ajax/ClientUpdateAction.php b/src/Application/Action/Client/Ajax/ClientUpdateAction.php index 094427f7..bc325248 100644 --- a/src/Application/Action/Client/Ajax/ClientUpdateAction.php +++ b/src/Application/Action/Client/Ajax/ClientUpdateAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $clientId = (int)$args['client_id']; $clientValues = (array)$request->getParsedBody(); diff --git a/src/Application/Action/Client/Api/ApiClientCreateAction.php b/src/Application/Action/Client/Api/ApiClientCreateAction.php index 11d4f27c..0327f9c6 100644 --- a/src/Application/Action/Client/Api/ApiClientCreateAction.php +++ b/src/Application/Action/Client/Api/ApiClientCreateAction.php @@ -21,7 +21,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $clientValues = (array)$request->getParsedBody(); diff --git a/src/Application/Action/Client/Page/ClientListPageAction.php b/src/Application/Action/Client/Page/ClientListPageAction.php index c3a08980..40689fbc 100644 --- a/src/Application/Action/Client/Page/ClientListPageAction.php +++ b/src/Application/Action/Client/Page/ClientListPageAction.php @@ -14,14 +14,14 @@ public function __construct( private TemplateRenderer $templateRenderer, private ClientListFilterChipProvider $clientListFilterChipGetter, - private ClientPermissionVerifier $clientPermissionVerifier + private ClientPermissionVerifier $clientPermissionVerifier, ) { } public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { // Clients are loaded dynamically with js after page load for a faster loading time // Retrieving available filters diff --git a/src/Application/Action/Client/Page/ClientReadPageAction.php b/src/Application/Action/Client/Page/ClientReadPageAction.php index a923c91b..6a593cd5 100644 --- a/src/Application/Action/Client/Page/ClientReadPageAction.php +++ b/src/Application/Action/Client/Page/ClientReadPageAction.php @@ -20,7 +20,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $clientReadResult = $this->clientFinder->findClientReadAggregate((int)$args['client_id']); $dropdownValues = $this->clientUtilFinder->findClientDropdownValues($clientReadResult->userId); diff --git a/src/Application/Action/Dashboard/DashboardPageAction.php b/src/Application/Action/Dashboard/DashboardPageAction.php index 5540291d..536ffd56 100644 --- a/src/Application/Action/Dashboard/DashboardPageAction.php +++ b/src/Application/Action/Dashboard/DashboardPageAction.php @@ -23,7 +23,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $dashboards = $this->dashboardGetter->getAuthorizedDashboards(); diff --git a/src/Application/Action/Dashboard/DashboardTogglePanelProcessAction.php b/src/Application/Action/Dashboard/DashboardTogglePanelProcessAction.php index eca6ea80..a5da2918 100644 --- a/src/Application/Action/Dashboard/DashboardTogglePanelProcessAction.php +++ b/src/Application/Action/Dashboard/DashboardTogglePanelProcessAction.php @@ -24,7 +24,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $params = (array)$request->getParsedBody(); // As there is no other validation the request body is checked for valid keys here diff --git a/src/Application/Action/Note/Ajax/NoteCreateAction.php b/src/Application/Action/Note/Ajax/NoteCreateAction.php index cb569074..07ffaebd 100644 --- a/src/Application/Action/Note/Ajax/NoteCreateAction.php +++ b/src/Application/Action/Note/Ajax/NoteCreateAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $noteValues = (array)$request->getParsedBody(); diff --git a/src/Application/Action/Note/Ajax/NoteDeleteAction.php b/src/Application/Action/Note/Ajax/NoteDeleteAction.php index cda430f4..411cb1e7 100644 --- a/src/Application/Action/Note/Ajax/NoteDeleteAction.php +++ b/src/Application/Action/Note/Ajax/NoteDeleteAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $noteId = (int)$args['note_id']; diff --git a/src/Application/Action/Note/Ajax/NoteFetchListAction.php b/src/Application/Action/Note/Ajax/NoteFetchListAction.php index 366100b6..d153d68c 100644 --- a/src/Application/Action/Note/Ajax/NoteFetchListAction.php +++ b/src/Application/Action/Note/Ajax/NoteFetchListAction.php @@ -20,7 +20,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { try { // Retrieve notes with given filter values (or none) diff --git a/src/Application/Action/Note/Ajax/NoteUpdateAction.php b/src/Application/Action/Note/Ajax/NoteUpdateAction.php index 094412da..ae36b9f9 100644 --- a/src/Application/Action/Note/Ajax/NoteUpdateAction.php +++ b/src/Application/Action/Note/Ajax/NoteUpdateAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $noteIdToChange = (int)$args['note_id']; $noteValues = (array)$request->getParsedBody(); diff --git a/src/Application/Action/Note/Page/NoteReadPageAction.php b/src/Application/Action/Note/Page/NoteReadPageAction.php index b34a271c..c7868455 100644 --- a/src/Application/Action/Note/Page/NoteReadPageAction.php +++ b/src/Application/Action/Note/Page/NoteReadPageAction.php @@ -22,7 +22,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $noteData = $this->noteFinder->findNote((int)$args['note_id']); if ($noteData->id) { diff --git a/src/Application/Action/User/Ajax/PasswordChangeSubmitAction.php b/src/Application/Action/User/Ajax/PasswordChangeSubmitAction.php index 083cbe7c..5606a16b 100644 --- a/src/Application/Action/User/Ajax/PasswordChangeSubmitAction.php +++ b/src/Application/Action/User/Ajax/PasswordChangeSubmitAction.php @@ -18,7 +18,7 @@ public function __construct( private JsonResponder $jsonResponder, private SessionManagerInterface $sessionManager, private SessionInterface $session, - private PasswordChanger $passwordChanger + private PasswordChanger $passwordChanger, ) { } diff --git a/src/Application/Action/User/Ajax/UserActivityFetchListAction.php b/src/Application/Action/User/Ajax/UserActivityFetchListAction.php index 1ae07c15..f6231188 100644 --- a/src/Application/Action/User/Ajax/UserActivityFetchListAction.php +++ b/src/Application/Action/User/Ajax/UserActivityFetchListAction.php @@ -30,7 +30,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $queryParams = $request->getQueryParams(); // User ids may be an array or a single value diff --git a/src/Application/Action/User/Ajax/UserCreateDropdownOptionsFetchAction.php b/src/Application/Action/User/Ajax/UserCreateDropdownOptionsFetchAction.php index 66a8e802..f745063c 100644 --- a/src/Application/Action/User/Ajax/UserCreateDropdownOptionsFetchAction.php +++ b/src/Application/Action/User/Ajax/UserCreateDropdownOptionsFetchAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $dropdownOptions = $this->userUtilFinder->findUserDropdownValues(); diff --git a/src/Application/Action/User/Ajax/UserDeleteAction.php b/src/Application/Action/User/Ajax/UserDeleteAction.php index ea2c52c1..5da15f5d 100644 --- a/src/Application/Action/User/Ajax/UserDeleteAction.php +++ b/src/Application/Action/User/Ajax/UserDeleteAction.php @@ -33,7 +33,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $userIdToDelete = (int)$args['user_id']; // Delete user diff --git a/src/Application/Action/User/Ajax/UserFetchListAction.php b/src/Application/Action/User/Ajax/UserFetchListAction.php index d3a9d0ac..4d67ad53 100644 --- a/src/Application/Action/User/Ajax/UserFetchListAction.php +++ b/src/Application/Action/User/Ajax/UserFetchListAction.php @@ -19,7 +19,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { // Retrieve users $userResultDataArray = $this->userFinder->findAllUsersResultDataForList(); diff --git a/src/Application/Action/User/Ajax/UserUpdateAction.php b/src/Application/Action/User/Ajax/UserUpdateAction.php index 56a50381..5662e722 100644 --- a/src/Application/Action/User/Ajax/UserUpdateAction.php +++ b/src/Application/Action/User/Ajax/UserUpdateAction.php @@ -18,7 +18,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { // Key 'user_id' for the user id in the URL is defined in the route definition in routes.php $userIdToChange = (int)$args['user_id']; diff --git a/src/Application/Action/User/Page/UserListPageAction.php b/src/Application/Action/User/Page/UserListPageAction.php index 8308e515..a92a48ef 100644 --- a/src/Application/Action/User/Page/UserListPageAction.php +++ b/src/Application/Action/User/Page/UserListPageAction.php @@ -19,7 +19,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { if ($this->userPermissionVerifier->isGrantedToRead()) { return $this->templateRenderer->render($response, 'user/user-list.html.php'); diff --git a/src/Application/Action/User/Page/UserReadPageAction.php b/src/Application/Action/User/Page/UserReadPageAction.php index f2422e8b..e8b51647 100644 --- a/src/Application/Action/User/Page/UserReadPageAction.php +++ b/src/Application/Action/User/Page/UserReadPageAction.php @@ -23,7 +23,7 @@ public function __construct( public function __invoke( ServerRequestInterface $request, ResponseInterface $response, - array $args + array $args, ): ResponseInterface { $authenticatedUserId = $this->session->get('user_id'); $userId = (int)($args['user_id'] ?? $authenticatedUserId); diff --git a/src/Application/ErrorRenderer/ProdErrorPageRenderer.php b/src/Application/ErrorRenderer/ProdErrorPageRenderer.php index a20d4c4e..7427c536 100644 --- a/src/Application/ErrorRenderer/ProdErrorPageRenderer.php +++ b/src/Application/ErrorRenderer/ProdErrorPageRenderer.php @@ -26,7 +26,7 @@ public function __construct( public function renderHtmlProdErrorPage( int $statusCode, ?string $safeExceptionMessage, - ?string $errorReportEmailAddress + ?string $errorReportEmailAddress, ): string { return $this->phpRenderer->fetch('error/error-page.html.php', [ 'statusCode' => $statusCode, diff --git a/src/Application/Middleware/PhpViewMiddleware.php b/src/Application/Middleware/PhpViewMiddleware.php index 73d3bf33..cbbe3d16 100644 --- a/src/Application/Middleware/PhpViewMiddleware.php +++ b/src/Application/Middleware/PhpViewMiddleware.php @@ -35,7 +35,7 @@ public function __construct( private readonly JsImportCacheBuster $jsImportCacheBuster, Settings $settings, private readonly UserPermissionVerifier $userPermissionVerifier, - private readonly RouteParserInterface $routeParser + private readonly RouteParserInterface $routeParser, ) { $this->publicSettings = $settings->get('public'); $this->deploymentSettings = $settings->get('deployment'); diff --git a/src/Application/Responder/JsonResponder.php b/src/Application/Responder/JsonResponder.php index 45a418f3..79453608 100644 --- a/src/Application/Responder/JsonResponder.php +++ b/src/Application/Responder/JsonResponder.php @@ -18,7 +18,7 @@ public function encodeAndAddToResponse( ResponseInterface $response, mixed $data = null, - int $status = 200 + int $status = 200, ): ResponseInterface { $response->getBody()->write((string)json_encode($data, JSON_UNESCAPED_SLASHES | JSON_PARTIAL_OUTPUT_ON_ERROR)); $response = $response->withStatus($status); diff --git a/src/Application/Responder/RedirectHandler.php b/src/Application/Responder/RedirectHandler.php index 80d13d29..742742b2 100644 --- a/src/Application/Responder/RedirectHandler.php +++ b/src/Application/Responder/RedirectHandler.php @@ -26,7 +26,7 @@ public function __construct(private RouteParserInterface $routeParser) public function redirectToUrl( ResponseInterface $response, string $destination, - array $queryParams = [] + array $queryParams = [], ): ResponseInterface { if ($queryParams) { $destination = sprintf('%s?%s', $destination, http_build_query($queryParams)); @@ -52,7 +52,7 @@ public function redirectToRouteName( ResponseInterface $response, string $routeName, array $data = [], - array $queryParams = [] + array $queryParams = [], ): ResponseInterface { return $this->redirectToUrl($response, $this->routeParser->urlFor($routeName, $data, $queryParams)); } diff --git a/src/Domain/Authentication/Repository/UserRoleFinderRepository.php b/src/Domain/Authentication/Repository/UserRoleFinderRepository.php index 22628b80..44259121 100644 --- a/src/Domain/Authentication/Repository/UserRoleFinderRepository.php +++ b/src/Domain/Authentication/Repository/UserRoleFinderRepository.php @@ -9,7 +9,7 @@ final readonly class UserRoleFinderRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenCreatorRepository.php b/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenCreatorRepository.php index 1eacf387..fd41ad9f 100644 --- a/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenCreatorRepository.php +++ b/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenCreatorRepository.php @@ -7,7 +7,7 @@ final readonly class VerificationTokenCreatorRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenDeleterRepository.php b/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenDeleterRepository.php index 6e1df4d7..4b012993 100644 --- a/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenDeleterRepository.php +++ b/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenDeleterRepository.php @@ -7,7 +7,7 @@ final readonly class VerificationTokenDeleterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenFinderRepository.php b/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenFinderRepository.php index f315afff..af87a4be 100644 --- a/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenFinderRepository.php +++ b/src/Domain/Authentication/Repository/VerificationToken/VerificationTokenFinderRepository.php @@ -12,7 +12,7 @@ class VerificationTokenFinderRepository { public function __construct( - private readonly QueryFactory $queryFactory + private readonly QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Authentication/Service/LoginMailSender.php b/src/Domain/Authentication/Service/LoginMailSender.php index de5ea7e6..87a65079 100644 --- a/src/Domain/Authentication/Service/LoginMailSender.php +++ b/src/Domain/Authentication/Service/LoginMailSender.php @@ -19,7 +19,7 @@ class LoginMailSender public function __construct( private readonly Mailer $mailer, private readonly LocaleConfigurator $localeConfigurator, - Settings $settings + Settings $settings, ) { $settings = $settings->get('public')['email']; // Create email object diff --git a/src/Domain/Authentication/Service/LoginNonActiveUserHandler.php b/src/Domain/Authentication/Service/LoginNonActiveUserHandler.php index d46373e9..e0773f59 100644 --- a/src/Domain/Authentication/Service/LoginNonActiveUserHandler.php +++ b/src/Domain/Authentication/Service/LoginNonActiveUserHandler.php @@ -44,7 +44,7 @@ public function __construct( public function handleLoginAttemptFromNonActiveUser( UserData $dbUser, array $queryParams, - ?string $captcha = null + ?string $captcha = null, ): void { // DTO values may be null and these values are required if (!isset($dbUser->id, $dbUser->email)) { @@ -123,7 +123,7 @@ private function handleUnverifiedUserLoginAttempt( int $userId, string $email, string $fullName, - array $queryParams = [] + array $queryParams = [], ): void { // Create a verification token, so they don't have to register again $queryParams = $this->verificationTokenCreator->createUserVerification($userId, $queryParams); @@ -170,7 +170,7 @@ private function handleLockedUserLoginAttempt( int $userId, string $email, string $fullName, - array $queryParams = [] + array $queryParams = [], ): void { // Create verification token to unlock account $queryParams = $this->verificationTokenCreator->createUserVerification($userId, $queryParams); diff --git a/src/Domain/Authentication/Service/RegistrationMailSender.php b/src/Domain/Authentication/Service/RegistrationMailSender.php index b797c2aa..0fd4840a 100644 --- a/src/Domain/Authentication/Service/RegistrationMailSender.php +++ b/src/Domain/Authentication/Service/RegistrationMailSender.php @@ -24,7 +24,7 @@ class RegistrationMailSender public function __construct( private readonly Mailer $mailer, private readonly LocaleConfigurator $localeConfigurator, - Settings $settings + Settings $settings, ) { $settings = $settings->get('public')['email']; // Create email object diff --git a/src/Domain/Client/Repository/ClientCreatorRepository.php b/src/Domain/Client/Repository/ClientCreatorRepository.php index 9d3a7141..5cb9234b 100644 --- a/src/Domain/Client/Repository/ClientCreatorRepository.php +++ b/src/Domain/Client/Repository/ClientCreatorRepository.php @@ -7,7 +7,7 @@ final readonly class ClientCreatorRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Client/Repository/ClientDeleterRepository.php b/src/Domain/Client/Repository/ClientDeleterRepository.php index 0d630401..4ef43397 100644 --- a/src/Domain/Client/Repository/ClientDeleterRepository.php +++ b/src/Domain/Client/Repository/ClientDeleterRepository.php @@ -7,7 +7,7 @@ final readonly class ClientDeleterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Client/Repository/ClientFinderRepository.php b/src/Domain/Client/Repository/ClientFinderRepository.php index f1c279b9..f42d3141 100644 --- a/src/Domain/Client/Repository/ClientFinderRepository.php +++ b/src/Domain/Client/Repository/ClientFinderRepository.php @@ -12,7 +12,7 @@ { public function __construct( private QueryFactory $queryFactory, - private Hydrator $hydrator + private Hydrator $hydrator, ) { } diff --git a/src/Domain/Client/Repository/ClientUpdaterRepository.php b/src/Domain/Client/Repository/ClientUpdaterRepository.php index 3053b294..f083a352 100644 --- a/src/Domain/Client/Repository/ClientUpdaterRepository.php +++ b/src/Domain/Client/Repository/ClientUpdaterRepository.php @@ -7,7 +7,7 @@ final readonly class ClientUpdaterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Client/Service/Authorization/ClientPermissionVerifier.php b/src/Domain/Client/Service/Authorization/ClientPermissionVerifier.php index e256b6c3..9e7e68c6 100644 --- a/src/Domain/Client/Service/Authorization/ClientPermissionVerifier.php +++ b/src/Domain/Client/Service/Authorization/ClientPermissionVerifier.php @@ -83,7 +83,7 @@ public function isGrantedToCreate(?ClientData $client = null): bool public function isGrantedToAssignUserToClient( ?int $assignedUserId, ?int $authenticatedUserRoleHierarchy = null, - ?array $userRoleHierarchies = null + ?array $userRoleHierarchies = null, ) { if ($this->loggedInUserId === null) { $this->logger->error( @@ -279,7 +279,7 @@ public function removeNonAuthorizedClientsFromList(?array $clients): array public function isGrantedToRead( ?int $ownerId, string|\DateTimeImmutable|null $deletedAt = null, - bool $log = true + bool $log = true, ): bool { if ($this->loggedInUserId !== null) { $authenticatedUserRoleHierarchy = $this->userRoleFinderRepository->getRoleHierarchyByUserId( diff --git a/src/Domain/Client/Service/ClientListFilter/ClientListFilterChipProvider.php b/src/Domain/Client/Service/ClientListFilter/ClientListFilterChipProvider.php index e1245fbb..54991161 100644 --- a/src/Domain/Client/Service/ClientListFilter/ClientListFilterChipProvider.php +++ b/src/Domain/Client/Service/ClientListFilter/ClientListFilterChipProvider.php @@ -20,7 +20,7 @@ public function __construct( private UserFinderRepository $userFinderRepository, private SessionInterface $session, private AuthorizationChecker $authorizationChecker, - private FilterSettingFinder $filterSettingFinder + private FilterSettingFinder $filterSettingFinder, ) { } diff --git a/src/Domain/Note/Repository/NoteCreatorRepository.php b/src/Domain/Note/Repository/NoteCreatorRepository.php index 92bfdafa..f722b15e 100644 --- a/src/Domain/Note/Repository/NoteCreatorRepository.php +++ b/src/Domain/Note/Repository/NoteCreatorRepository.php @@ -7,7 +7,7 @@ final readonly class NoteCreatorRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Note/Repository/NoteDeleterRepository.php b/src/Domain/Note/Repository/NoteDeleterRepository.php index 0d74be18..927e68dc 100644 --- a/src/Domain/Note/Repository/NoteDeleterRepository.php +++ b/src/Domain/Note/Repository/NoteDeleterRepository.php @@ -7,7 +7,7 @@ final readonly class NoteDeleterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Note/Repository/NoteFinderRepository.php b/src/Domain/Note/Repository/NoteFinderRepository.php index 41db108e..dd3497e8 100644 --- a/src/Domain/Note/Repository/NoteFinderRepository.php +++ b/src/Domain/Note/Repository/NoteFinderRepository.php @@ -23,7 +23,7 @@ class NoteFinderRepository public function __construct( private readonly QueryFactory $queryFactory, - private readonly Hydrator $hydrator + private readonly Hydrator $hydrator, ) { } diff --git a/src/Domain/Note/Repository/NoteUpdaterRepository.php b/src/Domain/Note/Repository/NoteUpdaterRepository.php index 1536e187..01bc18e7 100644 --- a/src/Domain/Note/Repository/NoteUpdaterRepository.php +++ b/src/Domain/Note/Repository/NoteUpdaterRepository.php @@ -7,7 +7,7 @@ final readonly class NoteUpdaterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Note/Service/Authorization/NotePermissionVerifier.php b/src/Domain/Note/Service/Authorization/NotePermissionVerifier.php index 379575d5..9bfc9184 100644 --- a/src/Domain/Note/Service/Authorization/NotePermissionVerifier.php +++ b/src/Domain/Note/Service/Authorization/NotePermissionVerifier.php @@ -34,7 +34,7 @@ public function isGrantedToRead( ?int $clientOwnerId = null, ?int $isHidden = null, bool $isDeleted = false, - bool $log = true + bool $log = true, ): bool { if (($loggedInUserId = (int)$this->session->get('user_id')) !== 0) { $authenticatedUserRoleHierarchy = $this->userRoleFinderRepository->getRoleHierarchyByUserId( @@ -117,7 +117,7 @@ public function isGrantedToUpdate( int $isMain, ?int $noteOwnerId = null, ?int $clientOwnerId = null, - bool $log = true + bool $log = true, ): bool { if (($loggedInUserId = (int)$this->session->get('user_id')) !== 0) { $authenticatedUserRoleHierarchy = $this->userRoleFinderRepository->getRoleHierarchyByUserId( @@ -161,7 +161,7 @@ public function isGrantedToUpdate( public function isGrantedToDelete( ?int $noteOwnerId = null, ?int $clientOwnerId = null, - bool $log = true + bool $log = true, ): bool { if (($loggedInUserId = (int)$this->session->get('user_id')) !== 0) { $authenticatedUserRoleHierarchy = $this->userRoleFinderRepository->getRoleHierarchyByUserId( diff --git a/src/Domain/Note/Service/NoteCreator.php b/src/Domain/Note/Service/NoteCreator.php index a1749ab6..24b3f75c 100644 --- a/src/Domain/Note/Service/NoteCreator.php +++ b/src/Domain/Note/Service/NoteCreator.php @@ -20,7 +20,7 @@ public function __construct( private UserActivityLogger $userActivityLogger, private UserNetworkSessionData $userNetworkSessionData, private UserFinder $userFinder, - private NoteFinder $noteFinder + private NoteFinder $noteFinder, ) { } diff --git a/src/Domain/Note/Service/NoteFinder.php b/src/Domain/Note/Service/NoteFinder.php index b0e89f28..18eb70a0 100644 --- a/src/Domain/Note/Service/NoteFinder.php +++ b/src/Domain/Note/Service/NoteFinder.php @@ -61,7 +61,7 @@ public function findAllNotesExceptMainFromUser(int $userId): array private function setNotePrivilegeAndRemoveMessageOfHidden( array $notes, ?int $clientOwnerId = null, - ?int $clientId = null + ?int $clientId = null, ): void { $randomText = 'Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua. At vero eos et accusam et justo diff --git a/src/Domain/Security/Exception/SecurityException.php b/src/Domain/Security/Exception/SecurityException.php index e785a8c1..e834b5d0 100644 --- a/src/Domain/Security/Exception/SecurityException.php +++ b/src/Domain/Security/Exception/SecurityException.php @@ -12,7 +12,7 @@ class SecurityException extends \RuntimeException public function __construct( private readonly int|string $remainingDelay, private readonly SecurityType $securityType, - string $message = 'Security check failed.' + string $message = 'Security check failed.', ) { parent::__construct($message); } diff --git a/src/Domain/Security/Repository/AuthenticationLoggerRepository.php b/src/Domain/Security/Repository/AuthenticationLoggerRepository.php index 9ad64d50..88ff8575 100644 --- a/src/Domain/Security/Repository/AuthenticationLoggerRepository.php +++ b/src/Domain/Security/Repository/AuthenticationLoggerRepository.php @@ -7,7 +7,7 @@ final readonly class AuthenticationLoggerRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Security/Repository/EmailLogFinderRepository.php b/src/Domain/Security/Repository/EmailLogFinderRepository.php index 70939095..50fcca53 100644 --- a/src/Domain/Security/Repository/EmailLogFinderRepository.php +++ b/src/Domain/Security/Repository/EmailLogFinderRepository.php @@ -8,7 +8,7 @@ class EmailLogFinderRepository { public function __construct( - private readonly QueryFactory $queryFactory + private readonly QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Security/Repository/EmailLoggerRepository.php b/src/Domain/Security/Repository/EmailLoggerRepository.php index 5d54865a..8d505910 100644 --- a/src/Domain/Security/Repository/EmailLoggerRepository.php +++ b/src/Domain/Security/Repository/EmailLoggerRepository.php @@ -7,7 +7,7 @@ final readonly class EmailLoggerRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } @@ -25,7 +25,7 @@ public function logEmailRequest( string $fromEmail, string $toEmail, string $subject, - int|string|null $userId + int|string|null $userId, ): int { $query = $this->queryFactory->insertQuery(); diff --git a/src/Domain/Security/Repository/LoginLogFinderRepository.php b/src/Domain/Security/Repository/LoginLogFinderRepository.php index 00d8733b..8bc34ca3 100644 --- a/src/Domain/Security/Repository/LoginLogFinderRepository.php +++ b/src/Domain/Security/Repository/LoginLogFinderRepository.php @@ -8,7 +8,7 @@ class LoginLogFinderRepository { public function __construct( - private readonly QueryFactory $queryFactory + private readonly QueryFactory $queryFactory, ) { } diff --git a/src/Domain/Security/Service/EmailRequestFinder.php b/src/Domain/Security/Service/EmailRequestFinder.php index 06b02347..918bb3f0 100644 --- a/src/Domain/Security/Service/EmailRequestFinder.php +++ b/src/Domain/Security/Service/EmailRequestFinder.php @@ -11,7 +11,7 @@ class EmailRequestFinder public function __construct( private readonly EmailLogFinderRepository $emailRequestFinderRepository, - Settings $settings + Settings $settings, ) { $this->securitySettings = $settings->get('security'); } diff --git a/src/Domain/Security/Service/LoginRequestFinder.php b/src/Domain/Security/Service/LoginRequestFinder.php index 0e18784b..03f04b25 100644 --- a/src/Domain/Security/Service/LoginRequestFinder.php +++ b/src/Domain/Security/Service/LoginRequestFinder.php @@ -13,7 +13,7 @@ class LoginRequestFinder public function __construct( private readonly LoginLogFinderRepository $loginRequestFinderRepository, private readonly UserNetworkSessionData $ipAddressData, - Settings $settings + Settings $settings, ) { $this->securitySettings = $settings->get('security'); } diff --git a/src/Domain/Security/Service/SecurityCaptchaVerifier.php b/src/Domain/Security/Service/SecurityCaptchaVerifier.php index 566d8242..d6bf52af 100644 --- a/src/Domain/Security/Service/SecurityCaptchaVerifier.php +++ b/src/Domain/Security/Service/SecurityCaptchaVerifier.php @@ -11,7 +11,7 @@ class SecurityCaptchaVerifier private array $googleSettings; public function __construct( - Settings $settings + Settings $settings, ) { $this->googleSettings = $settings->get('google'); } diff --git a/src/Domain/Security/Service/SecurityEmailChecker.php b/src/Domain/Security/Service/SecurityEmailChecker.php index ccc30825..98854daf 100644 --- a/src/Domain/Security/Service/SecurityEmailChecker.php +++ b/src/Domain/Security/Service/SecurityEmailChecker.php @@ -15,7 +15,7 @@ public function __construct( private readonly SecurityCaptchaVerifier $captchaVerifier, private readonly EmailRequestFinder $emailRequestFinder, private readonly EmailLogFinderRepository $requestFinderRepository, - Settings $settings + Settings $settings, ) { $this->securitySettings = $settings->get('security'); } @@ -74,7 +74,7 @@ public function performEmailAbuseCheck(?string $email, ?string $reCaptchaRespons */ private function performEmailRequestsCheck( int $emailsAmount, - string $email + string $email, ): void { if (isset($this->securitySettings['user_email_throttle_rule'])) { // Reverse order to compare fails the longest delay first and then go down from there diff --git a/src/Domain/Security/Service/SecurityLoginChecker.php b/src/Domain/Security/Service/SecurityLoginChecker.php index e58be2c7..99b626db 100644 --- a/src/Domain/Security/Service/SecurityLoginChecker.php +++ b/src/Domain/Security/Service/SecurityLoginChecker.php @@ -16,7 +16,7 @@ public function __construct( private readonly SecurityCaptchaVerifier $captchaVerifier, private readonly LoginRequestFinder $loginRequestFinder, private readonly LoginLogFinderRepository $loginRequestFinderRepository, - Settings $settings + Settings $settings, ) { $this->securitySettings = $settings->get('security'); } diff --git a/src/Domain/User/Repository/UserCreatorRepository.php b/src/Domain/User/Repository/UserCreatorRepository.php index 61894e2b..8876a567 100644 --- a/src/Domain/User/Repository/UserCreatorRepository.php +++ b/src/Domain/User/Repository/UserCreatorRepository.php @@ -7,7 +7,7 @@ final readonly class UserCreatorRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/User/Repository/UserDeleterRepository.php b/src/Domain/User/Repository/UserDeleterRepository.php index daf74127..545a4ad3 100644 --- a/src/Domain/User/Repository/UserDeleterRepository.php +++ b/src/Domain/User/Repository/UserDeleterRepository.php @@ -7,7 +7,7 @@ final readonly class UserDeleterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/User/Repository/UserFinderRepository.php b/src/Domain/User/Repository/UserFinderRepository.php index 06ffbd90..97c5e0ff 100644 --- a/src/Domain/User/Repository/UserFinderRepository.php +++ b/src/Domain/User/Repository/UserFinderRepository.php @@ -25,7 +25,7 @@ class UserFinderRepository public function __construct( private readonly QueryFactory $queryFactory, - private readonly Hydrator $hydrator + private readonly Hydrator $hydrator, ) { } diff --git a/src/Domain/User/Repository/UserUpdaterRepository.php b/src/Domain/User/Repository/UserUpdaterRepository.php index 40442e47..46232b4b 100644 --- a/src/Domain/User/Repository/UserUpdaterRepository.php +++ b/src/Domain/User/Repository/UserUpdaterRepository.php @@ -8,7 +8,7 @@ final readonly class UserUpdaterRepository { public function __construct( - private QueryFactory $queryFactory + private QueryFactory $queryFactory, ) { } diff --git a/src/Domain/User/Service/Authorization/UserPermissionVerifier.php b/src/Domain/User/Service/Authorization/UserPermissionVerifier.php index 19342bf1..4f7f67b3 100644 --- a/src/Domain/User/Service/Authorization/UserPermissionVerifier.php +++ b/src/Domain/User/Service/Authorization/UserPermissionVerifier.php @@ -254,7 +254,7 @@ public function isGrantedToUpdate(array $userDataToUpdate, string|int $userIdToU */ public function isGrantedToDelete( int $userIdToDelete, - bool $log = true + bool $log = true, ): bool { if ($this->loggedInUserId === null) { $this->logger->error( @@ -342,7 +342,7 @@ public function isGrantedToRead(?int $userIdToRead = null, bool $log = true): bo */ public function isGrantedToReadUserActivity( int $userIdToRead, - bool $log = true + bool $log = true, ): bool { if ($this->loggedInUserId === null) { $this->logger->error( diff --git a/src/Infrastructure/Service/Mailer.php b/src/Infrastructure/Service/Mailer.php index 2b9c3c37..8307e7c4 100644 --- a/src/Infrastructure/Service/Mailer.php +++ b/src/Infrastructure/Service/Mailer.php @@ -22,7 +22,7 @@ public function __construct( private MailerInterface $mailer, private PhpRenderer $phpRenderer, private EmailLoggerRepository $emailLoggerRepository, - private UserNetworkSessionData $userNetworkSessionData + private UserNetworkSessionData $userNetworkSessionData, ) { $this->loggedInUserId = $this->userNetworkSessionData->userId ?? null; } diff --git a/tests/Integration/Authentication/AccountUnlockActionTest.php b/tests/Integration/Authentication/AccountUnlockActionTest.php index 0d971c6a..abe821e0 100644 --- a/tests/Integration/Authentication/AccountUnlockActionTest.php +++ b/tests/Integration/Authentication/AccountUnlockActionTest.php @@ -83,7 +83,7 @@ public function testAccountUnlockAction(UserVerificationData $verification, stri #[DataProviderExternal(UserVerificationProvider::class, 'userVerificationInvalidTokenProvider')] public function testAccountUnlockActionInvalidExpiredToken( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // Insert locked user $userRow = $this->insertFixture( @@ -129,7 +129,7 @@ public function testAccountUnlockActionInvalidExpiredToken( #[DataProviderExternal(UserVerificationProvider::class, 'userVerificationProvider')] public function testAccountUnlockActionAlreadyUnlocked( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // Insert locked user $userRow = $this->insertFixture( diff --git a/tests/Integration/Authentication/PasswordForgottenEmailSubmitActionTest.php b/tests/Integration/Authentication/PasswordForgottenEmailSubmitActionTest.php index f8a8f02a..cffcee3e 100644 --- a/tests/Integration/Authentication/PasswordForgottenEmailSubmitActionTest.php +++ b/tests/Integration/Authentication/PasswordForgottenEmailSubmitActionTest.php @@ -117,7 +117,7 @@ public function testPasswordForgottenEmailSubmit(): void public function testPasswordForgottenEmailSubmitSecurityThrottling( int|string $delay, int $emailLogAmountInTimeSpan, - array $securitySettings + array $securitySettings, ): void { // Insert user $userRow = $this->insertFixture(UserFixture::class); diff --git a/tests/Integration/Authentication/PasswordResetSubmitActionTest.php b/tests/Integration/Authentication/PasswordResetSubmitActionTest.php index d9077766..d0ab513d 100644 --- a/tests/Integration/Authentication/PasswordResetSubmitActionTest.php +++ b/tests/Integration/Authentication/PasswordResetSubmitActionTest.php @@ -81,7 +81,7 @@ public function testResetPasswordSubmit(UserVerificationData $verification, stri #[DataProviderExternal(UserVerificationProvider::class, 'userVerificationInvalidTokenProvider')] public function testResetPasswordSubmitInvalidToken( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // User needed to insert verification $userRow = $this->insertFixture( @@ -135,7 +135,7 @@ public function testResetPasswordSubmitInvalidToken( #[DataProviderExternal(UserVerificationProvider::class, 'userVerificationProvider')] public function testResetPasswordSubmitInvalidData( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // Insert user id 2 role: user $userRow = $this->insertFixture(UserFixture::class, ['id' => $verification->userId]); diff --git a/tests/Integration/Authentication/RegisterVerifyActionTest.php b/tests/Integration/Authentication/RegisterVerifyActionTest.php index 34d66057..97132be3 100644 --- a/tests/Integration/Authentication/RegisterVerifyActionTest.php +++ b/tests/Integration/Authentication/RegisterVerifyActionTest.php @@ -82,7 +82,7 @@ public function testRegisterVerification(UserVerificationData $verification, str #[DataProviderExternal(\App\Test\Provider\Authentication\UserVerificationProvider::class, 'userVerificationProvider')] public function testRegisterVerificationAlreadyVerified( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // User needed to insert verification $userRow = $this->insertFixture( @@ -124,7 +124,7 @@ public function testRegisterVerificationAlreadyVerified( #[DataProviderExternal(\App\Test\Provider\Authentication\UserVerificationProvider::class, 'userVerificationProvider')] public function testRegisterVerificationAlreadyVerifiedAndAuthenticated( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // User needed to insert verification $userRow = $this->insertFixture( @@ -168,7 +168,7 @@ public function testRegisterVerificationAlreadyVerifiedAndAuthenticated( #[DataProviderExternal(\App\Test\Provider\Authentication\UserVerificationProvider::class, 'userVerificationInvalidTokenProvider')] public function testRegisterVerificationInvalidUsedExpiredToken( UserVerificationData $verification, - string $clearTextToken + string $clearTextToken, ): void { // User needed to insert verification $userRow = $this->insertFixture( diff --git a/tests/Integration/Client/ApiClientCreateActionTest.php b/tests/Integration/Client/ApiClientCreateActionTest.php index 28b3d560..3e31a290 100644 --- a/tests/Integration/Client/ApiClientCreateActionTest.php +++ b/tests/Integration/Client/ApiClientCreateActionTest.php @@ -110,7 +110,7 @@ public function testApiClientSubmitCreateAction(): void #[DataProviderExternal(\App\Test\Provider\Client\ApiClientCreateProvider::class, 'invalidApiClientCreationValues')] public function testApiClientSubmitCreateActionInvalid( array $requestBody, - array $jsonResponse + array $jsonResponse, ): void { // Insert action pending client status because it's needed by the service function $this->insertFixture( diff --git a/tests/Integration/Client/ClientCreateActionTest.php b/tests/Integration/Client/ClientCreateActionTest.php index fefd45f5..6709b854 100644 --- a/tests/Integration/Client/ClientCreateActionTest.php +++ b/tests/Integration/Client/ClientCreateActionTest.php @@ -48,7 +48,7 @@ class ClientCreateActionTest extends TestCase public function testClientSubmitCreateActionAuthorization( ?array $userLinkedToClientRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userLinkedToClientRow); diff --git a/tests/Integration/Client/ClientCreateDropdownOptionsTest.php b/tests/Integration/Client/ClientCreateDropdownOptionsTest.php index 6fcb7fed..fe384386 100644 --- a/tests/Integration/Client/ClientCreateDropdownOptionsTest.php +++ b/tests/Integration/Client/ClientCreateDropdownOptionsTest.php @@ -35,10 +35,10 @@ class ClientCreateDropdownOptionsTest extends TestCase * @param array $otherUserRow another user (that appears in dropdown) attributes containing the user_role_id * @param array $expectedUserNames * - * @return void * @throws ContainerExceptionInterface - * * @throws NotFoundExceptionInterface + * + * @return void */ #[DataProviderExternal(\App\Test\Provider\Client\ClientCreateProvider::class, 'clientCreationDropdownOptionsCases')] public function testClientCreateAssignedUserDropdownOptionsAuthorization( diff --git a/tests/Integration/Client/ClientDeleteActionTest.php b/tests/Integration/Client/ClientDeleteActionTest.php index deba4738..d87e8879 100644 --- a/tests/Integration/Client/ClientDeleteActionTest.php +++ b/tests/Integration/Client/ClientDeleteActionTest.php @@ -49,7 +49,7 @@ class ClientDeleteActionTest extends TestCase public function testClientSubmitDeleteActionAuthorization( array $userLinkedToClientRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userLinkedToClientRow); @@ -116,7 +116,7 @@ public function testClientSubmitUndeleteActionAuthorization( array $userLinkedToClientRow, array $authenticatedUserRow, array $requestData, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userLinkedToClientRow); diff --git a/tests/Integration/Client/ClientUpdateActionTest.php b/tests/Integration/Client/ClientUpdateActionTest.php index 8d20e332..495b447e 100644 --- a/tests/Integration/Client/ClientUpdateActionTest.php +++ b/tests/Integration/Client/ClientUpdateActionTest.php @@ -55,7 +55,7 @@ public function testClientSubmitUpdateActionAuthorization( array $userLinkedToClientRow, array $authenticatedUserRow, array $requestData, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userLinkedToClientRow); diff --git a/tests/Integration/Note/NoteCreateActionTest.php b/tests/Integration/Note/NoteCreateActionTest.php index 0f902221..1ee39c5c 100644 --- a/tests/Integration/Note/NoteCreateActionTest.php +++ b/tests/Integration/Note/NoteCreateActionTest.php @@ -51,7 +51,7 @@ class NoteCreateActionTest extends TestCase public function testNoteSubmitCreateActionAuthorization( array $linkedUserRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $linkedUserRow); @@ -152,7 +152,7 @@ public function testNoteSubmitCreateActionUnauthenticated(): void public function testNoteCreateSubmitActionInvalid( array $invalidRequestBody, bool $existingMainNote, - array $expectedResponseData + array $expectedResponseData, ): void { // Insert user authorized to create $clientOwnerId = $this->insertFixture( diff --git a/tests/Integration/Note/NoteDeleteActionTest.php b/tests/Integration/Note/NoteDeleteActionTest.php index cb868a3b..f1a557c4 100644 --- a/tests/Integration/Note/NoteDeleteActionTest.php +++ b/tests/Integration/Note/NoteDeleteActionTest.php @@ -49,7 +49,7 @@ class NoteDeleteActionTest extends TestCase public function testNoteSubmitDeleteActionAuthorization( array $linkedUserRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $linkedUserRow); diff --git a/tests/Integration/Note/NoteListActionTest.php b/tests/Integration/Note/NoteListActionTest.php index 6ff3a80a..f42163fe 100644 --- a/tests/Integration/Note/NoteListActionTest.php +++ b/tests/Integration/Note/NoteListActionTest.php @@ -55,7 +55,7 @@ public function testNoteListActionAuthorization( array $userLinkedToNoteRow, array $authenticatedUserRow, ?int $noteHidden, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userLinkedToNoteRow); diff --git a/tests/Integration/Note/NoteUpdateActionTest.php b/tests/Integration/Note/NoteUpdateActionTest.php index 8cc23afd..3b648747 100644 --- a/tests/Integration/Note/NoteUpdateActionTest.php +++ b/tests/Integration/Note/NoteUpdateActionTest.php @@ -50,7 +50,7 @@ class NoteUpdateActionTest extends TestCase public function testNoteSubmitUpdateActionAuthorization( array $linkedUserRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $linkedUserRow); diff --git a/tests/Integration/User/PasswordChangeSubmitActionTest.php b/tests/Integration/User/PasswordChangeSubmitActionTest.php index ee6b2ef9..6611c75e 100644 --- a/tests/Integration/User/PasswordChangeSubmitActionTest.php +++ b/tests/Integration/User/PasswordChangeSubmitActionTest.php @@ -44,7 +44,7 @@ class PasswordChangeSubmitActionTest extends TestCase public function testChangePasswordSubmitActionAuthorization( array $userToUpdateRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userToUpdateRow); diff --git a/tests/Integration/User/UserCreateActionTest.php b/tests/Integration/User/UserCreateActionTest.php index 5642e8ea..5dae33a6 100644 --- a/tests/Integration/User/UserCreateActionTest.php +++ b/tests/Integration/User/UserCreateActionTest.php @@ -48,7 +48,7 @@ class UserCreateActionTest extends TestCase public function testUserSubmitCreateAuthorization( array $authenticatedUserAttr, ?UserRole $newUserRole, - array $expectedResult + array $expectedResult, ): void { $userRoleFinderRepository = $this->container->get(UserRoleFinderRepository::class); // Insert authenticated user and user linked to resource with given attributes containing the user role diff --git a/tests/Integration/User/UserCreateDropdownOptionsTest.php b/tests/Integration/User/UserCreateDropdownOptionsTest.php index ee2bb2c7..b6026131 100644 --- a/tests/Integration/User/UserCreateDropdownOptionsTest.php +++ b/tests/Integration/User/UserCreateDropdownOptionsTest.php @@ -34,10 +34,10 @@ class UserCreateDropdownOptionsTest extends TestCase * user create form are the right ones for different * authenticated user roles. * - * @return void * @throws NotFoundExceptionInterface - * * @throws ContainerExceptionInterface + * + * @return void */ #[DataProviderExternal(\App\Test\Provider\User\UserCreateProvider::class, 'userCreationDropdownOptionsCases')] public function testUserCreateDropdownOptionsAuthorization( diff --git a/tests/Integration/User/UserDeleteActionTest.php b/tests/Integration/User/UserDeleteActionTest.php index d121c475..6589ac7a 100644 --- a/tests/Integration/User/UserDeleteActionTest.php +++ b/tests/Integration/User/UserDeleteActionTest.php @@ -43,7 +43,7 @@ class UserDeleteActionTest extends TestCase public function testUserSubmitDeleteActionAuthorization( array $userToDeleteRow, array $authenticatedUserRow, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user to delete with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userToDeleteRow); diff --git a/tests/Integration/User/UserUpdateActionTest.php b/tests/Integration/User/UserUpdateActionTest.php index 5ea075fa..615ff963 100644 --- a/tests/Integration/User/UserUpdateActionTest.php +++ b/tests/Integration/User/UserUpdateActionTest.php @@ -51,7 +51,7 @@ public function testUserSubmitUpdateAuthorization( array $userToChangeRow, array $authenticatedUserRow, array $requestData, - array $expectedResult + array $expectedResult, ): void { // Insert authenticated user and user linked to resource with given attributes containing the user role $this->insertUserFixtures($authenticatedUserRow, $userToChangeRow); diff --git a/tests/Trait/AppTestTrait.php b/tests/Trait/AppTestTrait.php index 39d6268d..622b37dc 100644 --- a/tests/Trait/AppTestTrait.php +++ b/tests/Trait/AppTestTrait.php @@ -21,7 +21,7 @@ trait AppTestTrait { use ContainerTestTrait; - /** @var App<\Psr\Container\ContainerInterface> $app */ + /** @var App<\Psr\Container\ContainerInterface> */ protected App $app; /** diff --git a/tests/Unit/Security/SecurityEmailCheckerTest.php b/tests/Unit/Security/SecurityEmailCheckerTest.php index cc703e0b..3316633a 100644 --- a/tests/Unit/Security/SecurityEmailCheckerTest.php +++ b/tests/Unit/Security/SecurityEmailCheckerTest.php @@ -44,7 +44,7 @@ class SecurityEmailCheckerTest extends TestCase public function testPerformEmailAbuseCheckIndividual( int|string $delay, int $emailLogAmountInTimeSpan, - array $securitySettings + array $securitySettings, ): void { // Settings for login throttling $this->mock(Settings::class)->method('get')->willReturn($securitySettings); @@ -99,7 +99,7 @@ public function testPerformEmailAbuseCheckIndividual( public function testPerformEmailAbuseCheckGlobal( int $todayEmailAmount, int $thisMonthEmailAmount, - array $securitySettings + array $securitySettings, ): void { // Settings for login throttling $this->mock(Settings::class)->method('get')->willReturn($securitySettings); diff --git a/tests/Unit/Security/SecurityLoginCheckerTest.php b/tests/Unit/Security/SecurityLoginCheckerTest.php index 9f60b102..66c9b147 100644 --- a/tests/Unit/Security/SecurityLoginCheckerTest.php +++ b/tests/Unit/Security/SecurityLoginCheckerTest.php @@ -57,7 +57,7 @@ class SecurityLoginCheckerTest extends TestCase public function testPerformLoginSecurityCheckIndividual( int|string $delay, array $ipAndEmailLogSummary, - array $securitySettings + array $securitySettings, ): void { $loginLogFinderRepository = $this->mock(LoginLogFinderRepository::class);