EWW\Dpf\ViewHelpers\ShowDocumentTypeViewHelper

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	render()	0	10	2

<?php
namespace EWW\Dpf\ViewHelpers;

/*
 * This file is part of the TYPO3 CMS project.
 *
 * It is free software; you can redistribute it and/or modify it under
 * the terms of the GNU General Public License, either version 2
 * of the License, or any later version.
 *
 * For the full copyright and license information, please read the
 * LICENSE.txt file that was distributed with this source code.
 *
 * The TYPO3 project - inspiring people to share!
 */

class ShowDocumentTypeViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\AbstractViewHelper
{
    /**
     * documentTypeRepository
     *
     * @var \EWW\Dpf\Domain\Repository\DocumentTypeRepository
     * @inject
     */
    protected $documentTypeRepository = null;


    /**
     * Gets the localized display name of the given document type.
     *
     * @param string $docType
     * @return string
     */
    public function render($docType)
    {
        /** @var \EWW\Dpf\Domain\Model\DocumentType $documentType */
        $documentType = $this->documentTypeRepository->findOneByName($docType);

The method findOneByName() does not exist on EWW\Dpf\Domain\Repository\DocumentTypeRepository. Since you implemented __call, consider adding a @method annotation.

        if ($documentType) {

$documentType is of type EWW\Dpf\Domain\Model\DocumentType, thus it always evaluated to true.

            return $documentType->getDisplayName();
        }

        return "-";
    }
}