App\Controller\UserController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	137
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	12

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	actionCreate()	0	20	2
A	actionUpdate()	0	23	3
B	actionRequestResetPassword()	0	37	5
A	actionResetPassword()	0	19	3

<?php
namespace App\Controller;

use App\Model\User;
use App\Common\JsonException;

use App\Requests\RequestResetPasswordRequest;
use App\Requests\ResetPasswordRequest;
use App\Requests\UserCreateRequest;
use App\Requests\UserUpdateRequest;

use Slim\Http\Request;
use Slim\Http\Response;

final class UserController extends CrudController
{
    /**
     * @param Request  $request
     * @param Response $response
     * @param array    $args
     *
     * @return mixed

Consider making the return type a bit more specific; maybe use \Psr\Http\Message\ResponseInterface.

     * @throws JsonException
     */
    public function actionCreate(Request $request, Response $response, $args)
    {
        $params = $request->getParsedBody();

        $this->validationRequest($params, $args['entity'], new UserCreateRequest());

        $exist = User::exist($params['data']['attributes']['email']);

        if ($exist) {
            throw new JsonException($args['entity'], 400, 'User already exists', 'User already exists');
        }

        $user = User::create($params['data']['attributes']);
        $user->setPassword($params['data']['attributes']['password']);
        $user->save();

        $result = $this->encode($request, $user);

        return $this->renderer->jsonApiRender($response, 200, $result);
    }

    /**
     * @param Request  $request
     * @param Response $response
     * @param array    $args
     *
     * @return mixed

Consider making the return type a bit more specific; maybe use \Psr\Http\Message\ResponseInterface.

     * @throws JsonException
     */
    public function actionUpdate(Request $request, Response $response, $args)
    {
        $user = User::find($args['id']);

        if (!$user) {
            throw new JsonException($args['entity'], 404, 'Not found', 'Entity not found');
        }

        $params = $request->getParsedBody();

        $this->validationRequest($params, $args['entity'], new UserUpdateRequest());

        $user->update($params['data']['attributes']);

        if (isset($params['data']['attributes']['password'])) {
            $user->setPassword($params['data']['attributes']['password']);
            $user->save();
        }

        $result = $this->encode($request, $user);

        return $this->renderer->jsonApiRender($response, 200, $result);
    }

    /**
     * @param Request  $request
     * @param Response $response
     * @param array    $args
     *
     * @return mixed

Consider making the return type a bit more specific; maybe use \Psr\Http\Message\ResponseInterface.

     * @throws JsonException
     */
    public function actionRequestResetPassword(Request $request, Response $response, $args)
    {
        $params = $request->getParsedBody();

        $this->validationRequest($params, $args['entity'], new RequestResetPasswordRequest());

        $user = User::findUserByEmail($params['data']['attributes']['email']);

        if (!$user) {
            throw new JsonException($args['entity'], 400, 'Bad request', 'Bad request');
        }

        if (!User::isPasswordResetTokenValid($user->password_reset_token)) {
            $user->generatePasswordResetToken();
        }

        if (!$user->save()) {
            throw new JsonException($args['entity'], 400, 'Bad request', 'Bad request');
        }

        $message = \Swift_Message::newInstance('Восстановление пароля для доступа в example.com')
            ->setFrom(['no-reply@example.com' => 'Почтовик example.com'])
            ->setTo([$user->email => $user->full_name])
            ->setBody($this->mailRenderer->render(
                '/RequestResetPassword.php',
                [
                    'host' => $this->settings['params']['host'],
                    'token' => $user->password_reset_token
                ]
            ), 'text/html');

        if ($this->mailer->send($message)) {

$message is of type object<Swift_Mime_MimePart>, but the function expects a object<Swift_Mime_Message>.

            return $this->renderer->jsonApiRender($response, 204);
        };

        throw new JsonException($args['entity'], 400, 'Bad request', 'Bad request');
    }

    /**
     * @param Request  $request
     * @param Response $response
     * @param array    $args
     *
     * @return mixed

Consider making the return type a bit more specific; maybe use \Psr\Http\Message\ResponseInterface.

     * @throws JsonException
     */
    public function actionResetPassword(Request $request, Response $response, $args)
    {
        $params = $request->getParsedBody();

        $this->validationRequest($params, $args['entity'], new ResetPasswordRequest());

        $user = User::findByPasswordResetToken($params['data']['attributes']['token']);

        if ($user) {
            $user->setPassword($params['data']['attributes']['password']);
            $user->removePasswordResetToken();

            if($user->save()){
                return $this->renderer->jsonApiRender($response, 204);
            };
        }

        throw new JsonException($args['entity'], 400, 'Bad request', 'Bad request');
    }
}