diff --git a/php/psalm-baseline.xml b/php/psalm-baseline.xml index fcf5ebf023e9..3dc505755b44 100644 --- a/php/psalm-baseline.xml +++ b/php/psalm-baseline.xml @@ -1,46 +1,3 @@ - - - $args - $args - $args - $args - $request - $request - $request - $response - $response - $response - - - - - $args - - - - - $args - $args - $args - $args - $args - $args - $args - $args - - - - - $args - $args - $args - - - - - $object - - diff --git a/php/public/index.php b/php/public/index.php index aaf829c86330..f9d4e94d9c65 100644 --- a/php/public/index.php +++ b/php/public/index.php @@ -12,6 +12,8 @@ use Slim\Factory\AppFactory; use Slim\Views\Twig; use Slim\Views\TwigMiddleware; +use Psr\Http\Message\ResponseInterface as Response; +use Psr\Http\Message\ServerRequestInterface as Request; require __DIR__ . '/../vendor/autoload.php'; @@ -65,7 +67,7 @@ $app->post('/api/configuration', \AIO\Controller\ConfigurationController::class . ':SetConfig'); // Views -$app->get('/containers', function ($request, $response, $args) use ($container) { +$app->get('/containers', function (Request $request, Response $response, mixed $args) use ($container) { $view = Twig::fromRequest($request); /** @var \AIO\Data\ConfigurationManager $configurationManager */ $configurationManager = $container->get(\AIO\Data\ConfigurationManager::class); @@ -110,7 +112,7 @@ 'additional_backup_directories' => $configurationManager->GetAdditionalBackupDirectoriesString(), ]); })->setName('profile'); -$app->get('/login', function ($request, $response, $args) use ($container) { +$app->get('/login', function (Request $request, Response $response, mixed $args) use ($container) { $view = Twig::fromRequest($request); /** @var \AIO\Docker\DockerActionManager $dockerActionManger */ $dockerActionManger = $container->get(\AIO\Docker\DockerActionManager::class); @@ -118,7 +120,7 @@ 'is_login_allowed' => $dockerActionManger->isLoginAllowed(), ]); }); -$app->get('/setup', function ($request, $response, $args) use ($container) { +$app->get('/setup', function (Request $request, Response $response, mixed $args) use ($container) { $view = Twig::fromRequest($request); /** @var \AIO\Data\Setup $setup */ $setup = $container->get(\AIO\Data\Setup::class); @@ -140,7 +142,7 @@ }); // Auth Redirector -$app->get('/', function (\Psr\Http\Message\RequestInterface $request, \Psr\Http\Message\ResponseInterface $response, $args) use ($container) { +$app->get('/', function (\Psr\Http\Message\RequestInterface $request, \Psr\Http\Message\ResponseInterface $response, mixed $args) use ($container) { $authManager = $container->get(\AIO\Auth\AuthManager::class); /** @var \AIO\Data\Setup $setup */ diff --git a/php/src/Controller/ConfigurationController.php b/php/src/Controller/ConfigurationController.php index a06cc6b9a087..95981a6af98b 100644 --- a/php/src/Controller/ConfigurationController.php +++ b/php/src/Controller/ConfigurationController.php @@ -19,7 +19,7 @@ public function __construct( $this->configurationManager = $configurationManager; } - public function SetConfig(Request $request, Response $response, $args) : Response { + public function SetConfig(Request $request, Response $response, mixed $args) : Response { try { if (isset($request->getParsedBody()['domain'])) { $domain = $request->getParsedBody()['domain'] ?? ''; diff --git a/php/src/Controller/DockerController.php b/php/src/Controller/DockerController.php index 7cf152260d46..0ef76e4cc899 100644 --- a/php/src/Controller/DockerController.php +++ b/php/src/Controller/DockerController.php @@ -49,7 +49,7 @@ private function PerformRecursiveContainerStart(string $id, bool $pullContainer $this->dockerActionManager->ConnectContainerToNetwork($container); } - public function GetLogs(Request $request, Response $response, $args) : Response + public function GetLogs(Request $request, Response $response, mixed $args) : Response { $id = $request->getQueryParams()['id']; if (str_starts_with($id, 'nextcloud-aio-')) { @@ -67,7 +67,7 @@ public function GetLogs(Request $request, Response $response, $args) : Response ->withHeader('Content-Disposition', 'inline'); } - public function StartBackupContainerBackup(Request $request, Response $response, $args) : Response { + public function StartBackupContainerBackup(Request $request, Response $response, mixed $args) : Response { $this->startBackup(); return $response->withStatus(201)->withHeader('Location', '/'); } @@ -84,7 +84,7 @@ public function startBackup() : void { $this->PerformRecursiveContainerStart($id); } - public function StartBackupContainerCheck(Request $request, Response $response, $args) : Response { + public function StartBackupContainerCheck(Request $request, Response $response, mixed $args) : Response { $this->checkBackup(); return $response->withStatus(201)->withHeader('Location', '/'); } @@ -98,7 +98,7 @@ public function checkBackup() : void { $this->PerformRecursiveContainerStart($id); } - public function StartBackupContainerRestore(Request $request, Response $response, $args) : Response { + public function StartBackupContainerRestore(Request $request, Response $response, mixed $args) : Response { $config = $this->configurationManager->GetConfig(); $config['backup-mode'] = 'restore'; $config['selected-restore-time'] = $request->getParsedBody()['selected_restore_time'] ?? ''; @@ -113,7 +113,7 @@ public function StartBackupContainerRestore(Request $request, Response $response return $response->withStatus(201)->withHeader('Location', '/'); } - public function StartBackupContainerTest(Request $request, Response $response, $args) : Response { + public function StartBackupContainerTest(Request $request, Response $response, mixed $args) : Response { $config = $this->configurationManager->GetConfig(); $config['backup-mode'] = 'test'; $config['instance_restore_attempt'] = 0; @@ -128,7 +128,7 @@ public function StartBackupContainerTest(Request $request, Response $response, $ return $response->withStatus(201)->withHeader('Location', '/'); } - public function StartContainer(Request $request, Response $response, $args) : Response + public function StartContainer(Request $request, Response $response, mixed $args) : Response { $uri = $request->getUri(); $host = $uri->getHost(); @@ -165,7 +165,7 @@ public function startTopContainer(bool $pullContainer) : void { $this->PerformRecursiveContainerStart($id, $pullContainer); } - public function StartWatchtowerContainer(Request $request, Response $response, $args) : Response { + public function StartWatchtowerContainer(Request $request, Response $response, mixed $args) : Response { $this->startWatchtower(); return $response->withStatus(201)->withHeader('Location', '/'); } @@ -188,7 +188,7 @@ private function PerformRecursiveContainerStop(string $id) : void $this->dockerActionManager->StopContainer($container); } - public function StopContainer(Request $request, Response $response, $args) : Response + public function StopContainer(Request $request, Response $response, mixed $args) : Response { $id = self::TOP_CONTAINER; $this->PerformRecursiveContainerStop($id); diff --git a/php/src/Controller/LoginController.php b/php/src/Controller/LoginController.php index bd7cb0b5e077..d37b47a8d8fe 100644 --- a/php/src/Controller/LoginController.php +++ b/php/src/Controller/LoginController.php @@ -19,7 +19,7 @@ public function __construct(AuthManager $authManager, DockerActionManager $docke $this->dockerActionManager = $dockerActionManager; } - public function TryLogin(Request $request, Response $response, $args) : Response { + public function TryLogin(Request $request, Response $response, mixed $args) : Response { if (!$this->dockerActionManager->isLoginAllowed()) { return $response->withHeader('Location', '/')->withStatus(302); } @@ -32,7 +32,7 @@ public function TryLogin(Request $request, Response $response, $args) : Response return $response->withHeader('Location', '/')->withStatus(302); } - public function GetTryLogin(Request $request, Response $response, $args) : Response { + public function GetTryLogin(Request $request, Response $response, mixed $args) : Response { $token = $request->getQueryParams()['token'] ?? ''; if($this->authManager->CheckToken($token)) { $this->authManager->SetAuthState(true); @@ -42,7 +42,7 @@ public function GetTryLogin(Request $request, Response $response, $args) : Respo return $response->withHeader('Location', '/')->withStatus(302); } - public function Logout(Request $request, Response $response, $args) : Response + public function Logout(Request $request, Response $response, mixed $args) : Response { $this->authManager->SetAuthState(false); return $response diff --git a/php/src/Twig/ClassExtension.php b/php/src/Twig/ClassExtension.php index 9204aee1d890..ff5ffe44b043 100644 --- a/php/src/Twig/ClassExtension.php +++ b/php/src/Twig/ClassExtension.php @@ -14,7 +14,7 @@ public function getFunctions() : array ); } - public function getClassName($object) : ?string + public function getClassName(mixed $object) : ?string { if (!is_object($object)) { return null;