EBloodBank\Controllers\DeleteDonors

Complexity

Total Complexity

17

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	doDeleteAction()	0	34	9
A	__invoke()	0	18	6
A	doActions()	0	6	2

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

use EBloodBank as EBB;

/**
 * Delete donors page controller class
 *
 * @since 1.1
 */
class DeleteDonors 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', 'delete')) {
            $view = $this->viewFactory->forgeView('error-403');

The assignment to $view is dead and can be removed.

        }

        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(
            'delete-donors',
            [
                'donors' => $this->donors,
            ]
        );
    }

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

    /**
     * @return void
     * @since 1.1
     */
    protected function doDeleteAction()
    {
        if (! $this->hasAuthenticatedUser() || ! $this->getAcl()->isUserAllowed($this->getAuthenticatedUser(), 'Donor', 'delete')) {
            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;
        }

        $deletedDonorsCount = 0;

        foreach ($donors as $donor) {
            if ($this->getAcl()->canDeleteEntity($this->getAuthenticatedUser(), $donor)) {
                $this->getEntityManager()->remove($donor);
                $deletedDonorsCount++;
            }
        }

        $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-deleted' => $deletedDonorsCount]
            )
        );
    }
}