Zicht\Bundle\PageBundle\Entity\ContentItem

Complexity

Total Complexity

12

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Test Coverage

Coverage

70.97%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getType()	0	3	1
A	getInternalName()	0	3	1
A	getTemplateName()	0	3	1
A	setConvertToType()	0	3	1
A	getRegion()	0	2	1
A	getConvertToType()	0	6	2
A	convert()	0	14	3
A	getShortType()	0	9	2

<?php
/**
 * @author Gerard van Helden <gerard@zicht.nl>
 * @copyright Zicht Online <http://zicht.nl>
 */
namespace Zicht\Bundle\PageBundle\Entity;

use Zicht\Util\Str;
use Zicht\Bundle\PageBundle\Model\ContentItemInterface;

/**
 * Base class for ContentItem entities.
 */
abstract class ContentItem implements ContentItemInterface
{
    /**
     * Returns a (dash notated) short type that can be used in databases, css classes, etc.
     *
     * @param string $infix
     * @return string
     */
    public function getShortType($infix = '-')
    {
        $shortType = Str::infix(lcfirst(Str::classname(Str::rstrip($this->getType(), 'ContentItem'))), $infix);

        if ($template = $this->getTemplateName()) {

Are you sure the assignment to $template is correct as $this->getTemplateName() targeting Zicht\Bundle\PageBundle\Entity\ContentItem::getTemplateName() seems to always return null.

            $shortType .= $infix . $template;

Are you sure $template of type void can be used in concatenation?

        }

        return $shortType;
    }

    /**
     * Copies all properties from the source to the target, if the target has the specified properties.
     *
     * @param ContentItemInterface $from
     * @param ContentItemInterface $to
     * @return ContentItemInterface
     */
    public static function convert(ContentItemInterface $from, ContentItemInterface $to)
    {
        $reflectionFrom = new \ReflectionClass($from);
        $reflectionTo = new \ReflectionClass($to);

        foreach ($reflectionFrom->getProperties() as $property) {
            $property->setAccessible(true);
            $method = 'set' . ucfirst($property->getName());
            if ($reflectionTo->hasMethod($method)) {
                $to->$method($property->getValue($from));
            }
        }

        return $to;
    }

    /**
     * Returns the type of the class.
     *
     * @return string
     */
    final public function getType()
    {
        return get_class($this);
    }


    protected $convertToType = null;

    /**
     * Return string
     *
     * @return string
     */
    final public function getConvertToType()
    {
        if (null !== $this->convertToType) {
            return $this->convertToType;
        }
        return $this->getType();
    }


    /**
     * Set the type to convert to.
     *
     * @param string $type
     * @return void
     */
    public function setConvertToType($type)
    {
        $this->convertToType = $type;
    }

    /**
     * If needed, you can add a custom template name to the content item
     *
     * @return null|string
     */
    public function getTemplateName()
    {
        return null;
    }

    /**
     * @return string
     */
    public function getInternalName()
    {
        return (string)$this;
    }

    /**
     * get region of content item, for BC just
     * setting as empty place holder.
     *
     * @return string
     */
    public function getRegion()
    {
    }
}