Shopware\WebInstaller\Controller\FinishController::default()

Complexity

Conditions	3
Paths	3

Size

Total Lines	28
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    #[Route('/finish', name: 'finish', defaults: ['step' => 3])]
    public function default(Request $request, #[Autowire('%kernel.cache_dir%')] string $cacheDir): Response
    {
        // @codeCoverageIgnoreStart
        if ($request->getMethod() === Request::METHOD_POST) {
            if ($request->hasSession()) {
                $request->getSession()->invalidate();
            }

            $self = $_SERVER['SCRIPT_FILENAME'];
            \assert(\is_string($self));

            $redirectUrl = $request->getBasePath() . '/admin';

            // Cleanup our generated cache dir in system temporary directory
            $fs = new Filesystem();
            $fs->remove($cacheDir);

            // Below this line call only php native functions as we deleted our own files already
            unlink($self);

            header('Content-Type: text/html; charset=utf-8');
            echo '<script>window.location.href = "' . $redirectUrl . '" </script>';
            exit;

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Symfony\Component\HttpFoundation\Response. Consider adding a return statement or allowing null as return value.

Using exit here is not recommended.

        }
        // @codeCoverageIgnoreEnd

        return $this->render('finish.html.twig');
    }