Pierstoval\Bundle\CharacterManagerBundle\Registry\ActionsRegistry

Complexity

Total Complexity

6

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addStepAction()	0	4	1
A	getAction()	0	26	5

<?php

/**
 * This file is part of the PierstovalCharacterManagerBundle package.
 *
 * (c) Alexandre Rock Ancelet <pierstoval@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Pierstoval\Bundle\CharacterManagerBundle\Registry;

use Pierstoval\Bundle\CharacterManagerBundle\Action\StepActionInterface;

class ActionsRegistry implements ActionsRegistryInterface
{
    /**
     * @var StepActionInterface[][]
     */
    private $actions = [];

    public function addStepAction(string $manager, StepActionInterface $action): void
    {
        $this->actions[$manager][$action->getStep()->getName()] = $action;
    }

    public function getAction(string $stepName, string $manager = null): StepActionInterface
    {
        if (!$this->actions) {

The expression $this->actions of type Pierstoval\Bundle\CharacterManagerBundle\Action\StepActionInterface[][] 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.

            throw new \RuntimeException('No actions in the registry.');
        }

        if ($manager === null) {
            $manager = array_keys($this->actions)[0];
        }

        if (!isset($this->actions[$manager])) {
            throw new \InvalidArgumentException(\sprintf(
                'Manager "%s" does not exist.',
                $manager
            ));
        }

        if (!isset($this->actions[$manager][$stepName])) {
                throw new \InvalidArgumentException(\sprintf(
                'Step "%s" not found in manager "%s".',
                $stepName, $manager
            ));
        }

        return $this->actions[$manager][$stepName];
    }
}