Apps\Controller\Front\Profile\ActionUnblock::unblock()

Complexity

Conditions	8
Paths	5

Size

Total Lines	27
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function unblock(string $targetId): ?string
    {
        // check if user is auth
        if (!App::$User->isAuth()) {
            throw new ForbiddenException();
        }

        // check if target is defined
        if (!Any::isInt($targetId) || $targetId < 1 || !App::$User->isExist($targetId)) {
            throw new NotFoundException();
        }

        $user = App::$User->identity();

        // check if target user in blacklist of current user
        if (!Blacklist::have($user->getId(), $targetId)) {
            throw new NotFoundException();
        }

        $model = new FormIgnoreDelete($user, $targetId);

It seems like $user can also be of type null; however, parameter $user of Apps\Model\Front\Profile\FormIgnoreDelete::__construct() does only seem to accept Ffcms\Core\Interfaces\iUser, maybe add an additional type check?

        if ($model->send() && $model->validate()) {
            $model->make();
            $this->response->redirect('profile/ignore');
        }

        return $this->view->render('profile/unblock', [
            'model' => $model
        ]);