EBloodBank\Controllers\ActivateUser

Complexity

Total Complexity

16

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	doActivateAction()	0	24	6
A	doActions()	0	6	2
A	__invoke()	0	26	6

<?php
/**
 * Activate user page controller class file
 *
 * @package    EBloodBank
 * @subpackage Controllers
 * @since      1.0
 */
namespace EBloodBank\Controllers;

use EBloodBank as EBB;
use Psr\Container\ContainerInterface;

/**
 * Activate user page controller class
 *
 * @since 1.0
 */
class ActivateUser extends Controller
{
    /**
     * @var int
     * @since 1.6
     */
    protected $userId = 0;

    /**
     * @var \EBloodBank\Models\User|null
     * @since 1.0
     */
    protected $user;

    /**
     * @since 1.0
     */
    public function __construct(ContainerInterface $container, $userID)
    {
        parent::__construct($container);
        if (EBB\isValidID($userID)) {

The function isValidID was not found. Maybe you did not declare it correctly or list all dependencies?

            $this->userId = (int) $userID;
        }
    }

    /**
     * @return void
     * @since 1.0
     */
    public function __invoke()
    {
        if (! $this->hasAuthenticatedUser() || ! $this->getAcl()->isUserAllowed($this->getAuthenticatedUser(), 'User', 'activate')) {
            $this->viewFactory->displayView('error-403');
            return;
        }

        if ($this->userID) {

The property userID does not exist on EBloodBank\Controllers\ActivateUser. Did you mean userId?

            $this->user = $this->getUserRepository()->find($this->userID);
        }

        if (! $this->user) {
            $this->viewFactory->displayView('error-404');
            return;
        }

        if (! $this->getAcl()->canActivateUser($this->getAuthenticatedUser(), $this->user)) {
            $this->viewFactory->displayView('error-403');
            return;
        }

        $this->doActions();
        $this->viewFactory->displayView(
            'activate-user',
            [
                'user' => $this->user,
            ]
        );
    }

    /**
     * @return void
     * @since 1.0
     */
    protected function doActions()
    {
        switch (filter_input(INPUT_POST, 'action')) {
            case 'activate_user':
                $this->doActivateAction();
                break;
        }
    }

    /**
     * @return void
     * @since 1.0
     */
    protected function doActivateAction()
    {
        $actionToken = filter_input(INPUT_POST, 'token');
        $sessionToken = $this->getSession()->getCsrfToken();

        if (! $actionToken || ! $sessionToken->isValid($actionToken)) {
            return;
        }

        if (! $this->hasAuthenticatedUser() || ! $this->getAcl()->canActivateUser($this->getAuthenticatedUser(), $this->user)) {

It seems like $this->user can also be of type null; however, parameter $user of EBloodBank\AclInterface::canActivateUser() does only seem to accept EBloodBank\Models\User, maybe add an additional type check?

            return;
        }

        if (! $this->user->isPending()) {

The method isPending() does not exist on null.

            return;
        }

        $this->user->set('status', 'activated');
        $this->getEntityManager()->flush($this->user);

        EBB\redirect(

The function redirect was not found. Maybe you did not declare it correctly or list all dependencies?

            EBB\addQueryArgs(

The function addQueryArgs was not found. Maybe you did not declare it correctly or list all dependencies?

                EBB\getEditUsersURL(),

The function getEditUsersURL was not found. Maybe you did not declare it correctly or list all dependencies?

                ['flag-activated' => 1]
            )
        );
    }
}