App\Controller\RegistrationController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	register()	0	46	4

<?php

namespace App\Controller;

use Awurth\Slim\Helper\Controller\RestController;
use Respect\Validation\Validator as V;
use Slim\Http\Request;
use Slim\Http\Response;

class RegistrationController extends RestController
{
    public function register(Request $request, Response $response)
    {
        $username = $request->getParam('username');
        $email = $request->getParam('email');
        $password = $request->getParam('password');

        $this->validator->request($request, [

The property validator does not exist on App\Controller\RegistrationController. Since you implemented __get, consider adding a @property annotation.

            'username' => V::length(3, 25)->alnum('_')->noWhitespace(),
            'email' => V::noWhitespace()->email(),
            'password' => [
                'rules' => V::noWhitespace()->length(6, 25),
                'messages' => [
                    'length' => 'The password length must be between {{minValue}} and {{maxValue}} characters'
                ]
            ],
            'password_confirm' => [
                'rules' => V::equals($password),
                'messages' => [
                    'equals' => 'Passwords don\'t match'
                ]
            ]
        ]);

        if ($this->sentinel->findByCredentials(['login' => $username])) {

The property sentinel does not exist on App\Controller\RegistrationController. Since you implemented __get, consider adding a @property annotation.

            $this->validator->addError('username', 'User already exists with this username.');
        }

        if ($this->sentinel->findByCredentials(['login' => $email])) {
            $this->validator->addError('email', 'User already exists with this email address.');
        }

        if ($this->validator->isValid()) {
            $role = $this->sentinel->findRoleByName('user');

            $user = $this->sentinel->registerAndActivate([
                'username' => $username,
                'email' => $email,
                'password' => $password
            ]);

            $role->users()->attach($user);

            return $this->created($response, 'login');
        }

        return $this->badRequest($response, $this->validator->getErrors());
    }
}