EBloodBank\Controllers\ApproveDonors

Complexity

Total Complexity

18

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	doActions()	0	6	2
B	doApproveAction()	0	37	10
A	__invoke()	0	19	6

<?php
/**
 * Approve donors page controller class file
 *
 * @package    EBloodBank
 * @subpackage Controllers
 * @since      1.1
 */
namespace EBloodBank\Controllers;

use EBloodBank as EBB;

/**
 * Approve donors page controller class
 *
 * @since 1.1
 */
class ApproveDonors extends Controller
{
    /**
     * @var \EBloodBank\Models\Donor[]
     * @since 1.1
     */
    protected $donors = [];

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

        if (filter_has_var(INPUT_POST, 'donors')) {
            $donorsIDs = filter_input(INPUT_POST, 'donors', FILTER_SANITIZE_NUMBER_INT, FILTER_REQUIRE_ARRAY);
            if (! empty($donorsIDs) && is_array($donorsIDs)) {
                $this->donors = $this->getDonorRepository()->findBy(['id' => $donorsIDs]);
            }
        }

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

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

    /**
     * @return void
     * @since 1.1
     */
    protected function doApproveAction()
    {
        if (! $this->hasAuthenticatedUser() || ! $this->getAcl()->isUserAllowed($this->getAuthenticatedUser(), 'Donor', 'approve')) {
            return;
        }

        $actionToken = filter_input(INPUT_POST, 'token');
        $sessionToken = $this->getSession()->getCsrfToken();

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

        $donors = $this->donors;

        if (! $donors || ! is_array($donors)) {

The expression $donors of type EBloodBank\Models\Donor[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return;
        }

        $approvedDonorsCount = 0;

        foreach ($donors as $donor) {
            if (! $donor->isPending()) {
                continue;
            }
            if ($this->getAcl()->canApproveDonor($this->getAuthenticatedUser(), $donor)) {
                $donor->set('status', 'approved');
                $approvedDonorsCount++;
            }
        }

        $this->getEntityManager()->flush();

        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\getEditDonorsURL(),

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

                ['flag-approved' => $approvedDonorsCount]
            )
        );
    }
}