Faulancer\View\Helper\FlashMessage

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	16	2

<?php
/**
 * Class FlashMessage | FlashMessagesage.php
 * @package Faulancer\View\Helper
 * @author  Florian Knapp <office@florianknapp.de>
 */
namespace Faulancer\View\Helper;

use Faulancer\Service\SessionManagerService;
use Faulancer\Translate\Translator;
use Faulancer\View\AbstractViewHelper;
use Faulancer\View\ViewController;

/**
 * Class FlashMessage
 */
class FlashMessage extends AbstractViewHelper
{

    /**
     * @param ViewController $view
     * @param string         $key
     * @param string         $type
     * @return string
     */
    public function __invoke(ViewController $view, string $key, $type = 'default')
    {
        $result = '';

        /** @var Translator $translator */
        $translator = $this->getServiceLocator()->get(Translator::class);

        /** @var SessionManagerService $sessionManager */
        $sessionManager = $this->getServiceLocator()->get(SessionManagerService::class);

        if ($sessionManager->hasFlashMessage($key)) {
            $result = '<span class="flash-message ' . $type . '">' . $translator->translate($sessionManager->getFlashMessage($key)) . '</span>';

It seems like $sessionManager->getFlashMessage($key) targeting Faulancer\Session\SessionManager::getFlashMessage() can also be of type array or null; however, Faulancer\Translate\Translator::translate() does only seem to accept string, maybe add an additional type check?

        }

        return $result;
    }

}