Net\Bazzline\Component\Template\RuntimeContentBasedTemplate

Complexity

Total Complexity

14

Size/Duplication

Total Lines	127
Duplicated Lines	10.24 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	__invoke()	0	6	1
A	setClosingDelimiter()	0	4	1
A	setContent()	0	12	2
A	setOpeningDelimiter()	0	4	1
A	getContent()	0	10	2
A	getVariables()	13	13	2
A	setPropertiesIfProvided()	0	14	4

<?php
/**
 * @author stev leibelt <artodeto@bazzline.net>
 * @since: 2015-10-02
 */
namespace Net\Bazzline\Component\Template;

use InvalidArgumentException;
use RuntimeException;

class RuntimeContentBasedTemplate extends AbstractTemplate implements DelimiterInterface
{
    /** @var string */
    private $closingDelimiter;

    /** @var null|string */
    private $content;

    /** @var string */
    private $openingDelimiter;

    /**
     * @param array $variables
     * @param null|string $content
     * @param string $openingDelimiter
     * @param string $closingDelimiter
     * @throws InvalidArgumentException
     */
    public function __construct($variables = array(), $content = null, $openingDelimiter = '{', $closingDelimiter = '}')
    {
        parent::__construct($variables);

        $this->setPropertiesIfProvided($content, $closingDelimiter, $openingDelimiter);

It seems like $content defined by parameter $content on line 29 can also be of type string; however, Net\Bazzline\Component\Template\RuntimeContentBasedTemplate::setPropertiesIfProvided() does only seem to accept null, maybe add an additional type check?

    }

    /**
     * @param array $variables
     * @param null|string $content
     * @param null|string $openingDelimiter
     * @param null|string $closingDelimiter
     * @return string
     * @throws InvalidArgumentException
     * @throws RuntimeException
     */
    public function __invoke($variables = array(), $content = null, $openingDelimiter = null, $closingDelimiter = null)
    {
        $this->setPropertiesIfProvided($content, $closingDelimiter, $openingDelimiter);

It seems like $content defined by parameter $content on line 45 can also be of type string; however, Net\Bazzline\Component\Template\RuntimeContentBasedTemplate::setPropertiesIfProvided() does only seem to accept null, maybe add an additional type check?

        return parent::__invoke($variables);
    }

    /**
     * @param string $closingDelimiter
     */
    public function setClosingDelimiter($closingDelimiter)
    {
        $this->closingDelimiter = $closingDelimiter;
    }

    /**
     * @param string $content
     * @throws InvalidArgumentException
     */
    public function setContent($content)
    {
        $content = (string) $content;

        if (strlen($content) < 1) {
            throw new InvalidArgumentException(
                'content must have a string length of at least one character'
            );
        }

        $this->content = $content;
    }

    /**
     * @param string $openingDelimiter
     */
    public function setOpeningDelimiter($openingDelimiter)
    {
        $this->openingDelimiter = $openingDelimiter;
    }

    /**
     * @return string
     * @throws RuntimeException
     */
    protected function getContent()
    {
        if (is_null($this->content)) {
            throw new RuntimeException(
                'no content provided'
            );
        }

        return $this->content;
    }

    /**
     * @return array
     */
    protected function getVariables()

This method seems to be duplicated in your project.

    {
        $prefix     = $this->openingDelimiter;
        $suffix     = $this->closingDelimiter;
        $variables  = parent::getVariables();
        $array      = array();  //@todo find a better name

        foreach ($variables as $key => $value) {
            $array[$prefix . $key . $suffix] = $value;
        }

        return $array;
    }

    /**
     * @param null $content
     * @param null|string $closingDelimiter
     * @param null|string $openingDelimiter
     * @throws InvalidArgumentException
     */
    private function setPropertiesIfProvided($content = null, $closingDelimiter = null, $openingDelimiter = null)
    {
        if (!is_null($content)) {
            $this->setContent($content);
        }

        if (!is_null($closingDelimiter)) {
            $this->setClosingDelimiter($closingDelimiter);
        }

        if (!is_null($openingDelimiter)) {
            $this->setOpeningDelimiter($openingDelimiter);
        }
    }
}