Platine\Framework\Demo\Action\User\CreateAction::handle()

Complexity

Conditions	4
Paths	4

Size

Total Lines	59
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        if ($request->getMethod() === 'GET') {
            return new TemplateResponse(
                $this->template,
                'user/create',
                [
                    'param' => new UserParam([])
                ]
            );
        }

        $param = new RequestData($request);
        $formParam = new UserParam($param->posts());
        $validator = new UserValidator($formParam);

        if (!$validator->validate()) {
            return new TemplateResponse(
                $this->template,
                'user/create',
                [
                    'errors' => $validator->getErrors(),
                    'param' => $formParam
                ]
            );
        }

        $username = $param->post('username');
        $userExist = $this->userRepository->findBy(['username' => $username]);

        if ($userExist) {
            $this->logger->error('User with username {username} already exists', ['username' => $username]);
            return new TemplateResponse(
                $this->template,
                'user/create',
                [
                   'param' => $formParam
                ]
            );
        }

        $password = $param->post('password');

        $hash = new BcryptHash();
        $passwordHash = $hash->hash($password);

It seems like $password can also be of type null; however, parameter $plain of Platine\Security\Hash\BcryptHash::hash() does only seem to accept string, maybe add an additional type check?

        $user = $this->userRepository->create([
            'username' => $formParam->getUsername(),
            'fname' => $formParam->getFirstname(),
            'lname' => $formParam->getLastname(),
            'password' => $passwordHash,
            'status' => 1,
            'age' => (int) $formParam->getAge(),
            'deleted' => 0,
        ]);

        $this->userRepository->save($user);

        return (new RedirectResponse('list'))->redirect();
    }