Thunder\Shortcode\ShortcodeFacade

Complexity

Total Complexity

23

Size/Duplication

Total Lines	128
Duplicated Lines	15.63 %

Coupling/Cohesion

Components	2
Dependencies	12

Test Coverage

Coverage

100%

Importance

Changes	12
Bugs	3	Features	2

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	14	1
A	create()	0	10	1
A	rebuildProcessor()	0	5	1
A	process()	0	4	1
A	parse()	0	4	1
A	setParser()	0	7	1
A	addHandler()	0	6	1
A	addHandlerAlias()	0	6	1
A	addEventHandler()	0	6	1
B	serialize()	10	10	5
B	unserialize()	10	10	5
A	serializeToText()	0	1	1
A	serializeToJson()	0	1	1
A	unserializeFromText()	0	1	1
A	unserializeFromJson()	0	1	1

<?php
namespace Thunder\Shortcode;

use Thunder\Shortcode\EventContainer\EventContainer;
use Thunder\Shortcode\HandlerContainer\HandlerContainer;
use Thunder\Shortcode\HandlerContainer\HandlerContainerInterface;
use Thunder\Shortcode\Parser\ParserInterface;
use Thunder\Shortcode\Parser\RegularParser;
use Thunder\Shortcode\Processor\Processor;
use Thunder\Shortcode\Processor\ProcessorInterface;
use Thunder\Shortcode\Serializer\JsonSerializer;
use Thunder\Shortcode\Serializer\SerializerInterface;
use Thunder\Shortcode\Serializer\TextSerializer;
use Thunder\Shortcode\Serializer\XmlSerializer;
use Thunder\Shortcode\Serializer\YamlSerializer;
use Thunder\Shortcode\Shortcode\ShortcodeInterface;
use Thunder\Shortcode\Syntax\CommonSyntax;
use Thunder\Shortcode\Syntax\SyntaxInterface;

/**
 * @author Tomasz Kowalczyk <tomasz@kowalczyk.cc>
 */
class ShortcodeFacade
{
    /** @var ProcessorInterface */
    private $processor;
    /** @var ParserInterface */
    private $parser;
    /** @var SyntaxInterface */
    private $syntax;

    /** @var HandlerContainer */
    private $handlers;
    /** @var EventContainer */
    private $events;

    /** @var SerializerInterface */
    private $textSerializer;
    /** @var SerializerInterface */
    private $jsonSerializer;
    /** @var SerializerInterface */
    private $xmlSerializer;
    /** @var SerializerInterface */
    private $yamlSerializer;

    public function __construct()
    {
        $this->syntax = new CommonSyntax();
        $this->handlers = new HandlerContainer();
        $this->events = new EventContainer();

        $this->parser = new RegularParser($this->syntax);
        $this->rebuildProcessor();

        $this->textSerializer = new TextSerializer();
        $this->jsonSerializer = new JsonSerializer();
        $this->yamlSerializer = new YamlSerializer();
        $this->xmlSerializer = new XmlSerializer();
    }

    /** @deprecated use constructor and customize using exposed methods */
    public static function create(HandlerContainerInterface $handlers, SyntaxInterface $syntax)
    {
        $self = new self();

        $self->handlers = $handlers;

$handlers is of type object<Thunder\Shortcode\HandlerContainer\HandlerContainerInterface>, but the property $handlers was declared to be of type object<Thunder\Shortcode\HandlerContainer\HandlerContainer>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $self->syntax = $syntax;
        $self->rebuildProcessor();

        return $self;
    }

    private function rebuildProcessor()
    {
        $this->processor = new Processor($this->parser, $this->handlers);
        $this->processor = $this->processor->withEventContainer($this->events);
    }

    public function process($text)
    {
        return $this->processor->process($text);
    }

    public function parse($text)
    {
        return $this->parser->parse($text);
    }

    public function setParser(ParserInterface $parser)
    {
        $this->parser = $parser;
        $this->rebuildProcessor();

        return $this;
    }

    public function addHandler($name, $handler)
    {
        $this->handlers->add($name, $handler);

        return $this;
    }

    public function addHandlerAlias($alias, $name)
    {
        $this->handlers->addAlias($alias, $name);

        return $this;
    }

    public function addEventHandler($name, $handler)
    {
        $this->events->addListener($name, $handler);

        return $this;
    }

    /* --- SERIALIZATION --------------------------------------------------- */

    public function serialize(ShortcodeInterface $shortcode, $format)

This method seems to be duplicated in your project.

    {
        switch($format) {
            case 'text': return $this->textSerializer->serialize($shortcode);
            case 'json': return $this->jsonSerializer->serialize($shortcode);
            case 'yaml': return $this->yamlSerializer->serialize($shortcode);
            case 'xml': return $this->xmlSerializer->serialize($shortcode);
            default: throw new \InvalidArgumentException(sprintf('Invalid serialization format %s!', $format));
        }
    }

    public function unserialize($text, $format)

This method seems to be duplicated in your project.

    {
        switch($format) {
            case 'text': return $this->textSerializer->unserialize($text);
            case 'json': return $this->jsonSerializer->unserialize($text);
            case 'yaml': return $this->yamlSerializer->unserialize($text);
            case 'xml': return $this->xmlSerializer->unserialize($text);
            default: throw new \InvalidArgumentException(sprintf('Invalid unserialization format %s!', $format));
        }
    }

    /** @deprecated use serialize($shortcode, $format) */
    public function serializeToText(ShortcodeInterface $s) { return $this->serialize($s, 'text'); }
    /** @deprecated use serialize($shortcode, $format) */
    public function serializeToJson(ShortcodeInterface $s) { return $this->serialize($s, 'json'); }
    /** @deprecated use unserialize($shortcode, $format) */
    public function unserializeFromText($text) { return $this->unserialize($text, 'text'); }
    /** @deprecated use unserialize($shortcode, $format) */
    public function unserializeFromJson($text) { return $this->unserialize($text, 'json'); }
}