Flagbit\Plantuml\TokenReflection\WriterAbstract

Complexity

Total Complexity

16

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

81.25%

Importance

Changes	9
Bugs	3	Features	4

6 Methods

Rating	Name	Duplication	Size	Complexity
A	formatLine()	0	4	1
A	__construct()	0	8	2
A	setIndent()	0	4	1
A	expandNamespaceAlias()	0	11	3
A	formatClassName()	0	8	2
C	formatValue()	0	30	7

<?php

namespace Flagbit\Plantuml\TokenReflection;

use TokenReflection\IReflectionClass;

abstract class WriterAbstract
{

    /**
     * @param Flagbit\Plantuml\TokenReflection\WriterOptions $writerOptions
     */
    public function __construct(WriterOptions $writerOptions=null)
    {
        if (!is_null($writerOptions)) {
            $this->writerOptions = $writerOptions;

It seems like $writerOptions of type object<Flagbit\Plantuml\TokenReflection\WriterOptions> is incompatible with the declared type object<Flagbit\Plantuml\TokenReflection\Flagbit\Plantuml\TokenReflection\WriterOptions> of property $writerOptions.

        } else {
            $this->writerOptions = new WriterOptions();

It seems like new \Flagbit\Plantuml\TokenReflection\WriterOptions() of type object<Flagbit\Plantuml\TokenReflection\WriterOptions> is incompatible with the declared type object<Flagbit\Plantuml\TokenReflection\Flagbit\Plantuml\TokenReflection\WriterOptions> of property $writerOptions.

        }
    }
    /**
     * @var string
     */
    private $indent = '    ';

    /**
     * @var string
     */
    private $linebreak = "\n";

    /**
     * @var  Flagbit\Plantuml\TokenReflection\WriterOptions
     */
    protected $writerOptions = null;

    /**
     * @param string $indent
     */
    protected function setIndent($indent = '    ')
    {
        $this->indent = (string) $indent;
    }

    /**
     * @param string $string
     * @return string
     */
    public function formatLine($string)
    {
        return $this->indent . $string . $this->linebreak;
    }

    /**
     * @param \TokenReflection\IReflectionClass $declaringClass The class using the namespace aliases
     * @param string $aliasedClassName The class name used in the declaring class
     * @return string
     */
    protected function expandNamespaceAlias(IReflectionClass $declaringClass, $aliasedClassName)
    {
        $aliasedClassName = trim($aliasedClassName);
        foreach ($declaringClass->getNamespaceAliases() as $namespaceAlias) {
            if (1 === preg_match('/\\\\' . preg_quote($aliasedClassName) . '$/', $namespaceAlias)) {
                $aliasedClassName = $namespaceAlias;
                break;
            }
        }
        return $aliasedClassName;
    }

    /**
     * @param string $className
     * @return string
     */
    protected function formatClassName($className)
    {
        $className = str_replace('\\', '.', trim($className));
        if ('.' === $className[0]) {
            $className = substr($className, 1);
        }
        return $className;
    }

    /**
     * @param mixed $value
     * @return string
     */
    protected function formatValue($value)
    {
        if (is_null($value)) {
            $value = 'null';
        }
        else if (is_array($value)) {
            $formattedValues = array();
            foreach ($value as $currentValue) {
                $formattedValues[] = $this->formatValue($currentValue);
            }
            $value = '[' .implode(', ', $formattedValues) . ']';
        }
        else if (is_numeric($value)) {
            // nothing to do here
        }
        else if (is_bool($value)) {
            $value = $value ? 'true' : 'false';
        }
        else {
            // make sure we receive two backslashes in the output as
            // plantuml needs them escaped as well
            $value = strtr($value, array(

It seems like $value can also be of type object; however, strtr() does only seem to accept string, maybe add an additional type check?

                "\n" => '\\\\n',
                "\r" => '\\\\r',
                "\t" => '\\\\t',
            ));
            $value = '"' . $value . '"';
        }
        return $value;
    }
}