App\Controller\SignupController::signup()

Complexity

Conditions	6
Paths	20

Size

Total Lines	43
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function signup(RequestInterface $request, IdentityRepositoryInterface $identityRepository, ORMInterface $orm, UrlGeneratorInterface $urlGenerator, LoggerInterface $logger): ResponseInterface
    {
        $body = $request->getParsedBody();

The method getParsedBody() does not exist on Psr\Http\Message\RequestInterface. It seems like you code against a sub-type of Psr\Http\Message\RequestInterface such as Psr\Http\Message\ServerRequestInterface.

        $error = null;

        if ($request->getMethod() === Method::POST) {
            try {
                foreach (['login', 'password'] as $name) {
                    if (empty($body[$name])) {
                        throw new \InvalidArgumentException(ucfirst($name) . ' is required.');
                    }
                }

                /** @var \App\Entity\User $identity */
                $identity = $identityRepository->findByLogin($body['login']);
                if ($identity !== null) {
                    throw new \InvalidArgumentException('Unable to register user with such username.');
                }

                $user = new User($body['login'], $body['password']);

                $transaction = new Transaction($orm);
                $transaction->persist($user);

                $transaction->run();
                return $this->responseFactory
                    ->createResponse(302)
                    ->withHeader(
                        'Location',
                        $urlGenerator->generate('site/index')
                    );
            } catch (\Throwable $e) {
                $logger->error($e);
                $error = $e->getMessage();
            }
        }

        return $this->render(
            'signup',
            [
                'body' => $body,
                'error' => $error,
                'csrf' => $request->getAttribute('csrf_token'),

The method getAttribute() does not exist on Psr\Http\Message\RequestInterface. It seems like you code against a sub-type of Psr\Http\Message\RequestInterface such as Psr\Http\Message\ServerRequestInterface.

            ]