Thunder\Shortcode\HandlerContainer\HandlerContainer

Complexity

Total Complexity

14

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	addAlias()	0	10	2
A	add()	0	13	3
A	remove()	0	9	2
A	setDefault()	0	8	1
A	getNames()	0	4	1
A	get()	0	4	2
A	has()	0	4	1
A	guardHandler()	0	6	2

<?php
namespace Thunder\Shortcode\HandlerContainer;

use Thunder\Shortcode\Shortcode\ShortcodeInterface;

/**
 * @author Tomasz Kowalczyk <tomasz@kowalczyk.cc>
 */
final class HandlerContainer implements HandlerContainerInterface
{
    /** @psalm-var array<string,callable(ShortcodeInterface):string> */
    private $handlers = array();
    /** @psalm-var (callable(ShortcodeInterface):string)|null */
    private $default = null;

    /**
     * @param string $name
     * @param callable $handler
     * @psalm-param callable(ShortcodeInterface):string $handler
     *
     * @return $this
     */
    public function add($name, $handler)
    {
        $this->guardHandler($handler);

        if (empty($name) || $this->has($name)) {
            $msg = 'Invalid name or duplicate shortcode handler for %s!';
            throw new \RuntimeException(sprintf($msg, $name));
        }

        $this->handlers[$name] = $handler;

        return $this;
    }

    /**
     * @param string $alias
     * @param string $name
     *
     * @return $this
     */
    public function addAlias($alias, $name)
    {
        if (false === $this->has($name)) {
            $msg = 'Failed to add an alias %s to non existent handler %s!';
            throw new \RuntimeException(sprintf($msg, $alias, $name));
        }

        /** @psalm-suppress PossiblyNullArgument */
        return $this->add($alias, $this->get($name));

It seems like $this->get($name) targeting Thunder\Shortcode\HandlerContainer\HandlerContainer::get() can also be of type null; however, Thunder\Shortcode\HandlerContainer\HandlerContainer::add() does only seem to accept callable, maybe add an additional type check?

    }

    /**
     * @param string $name
     *
     * @return void
     */
    public function remove($name)
    {
        if (false === $this->has($name)) {
            $msg = 'Failed to remove non existent handler %s!';
            throw new \RuntimeException(sprintf($msg, $name));
        }

        unset($this->handlers[$name]);
    }

    /**
     * @param callable $handler
     * @psalm-param callable(ShortcodeInterface):string $handler
     *
     * @return $this
     */
    public function setDefault($handler)
    {
        $this->guardHandler($handler);

        $this->default = $handler;

        return $this;
    }

    /** @return string[] */
    public function getNames()
    {
        return array_keys($this->handlers);
    }

    /**
     * @param string $name
     *
     * @return callable|null
     * @psalm-return (callable(ShortcodeInterface):string)|null
     */
    public function get($name)
    {
        return $this->has($name) ? $this->handlers[$name] : $this->default;
    }

    /**
     * @param string $name
     *
     * @return bool
     */
    public function has($name)
    {
        return array_key_exists($name, $this->handlers);
    }

    /**
     * @param callable $handler
     *
     * @return void
     */
    private function guardHandler($handler)
    {
        if (!is_callable($handler)) {
            throw new \RuntimeException('Shortcode handler must be callable!');
        }
    }
}