LAG\AdminBundle\Message\MessageHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	73
Duplicated Lines	35.62 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	handleError()	13	13	2
A	handleSuccess()	13	13	2

<?php

namespace LAG\AdminBundle\Message;

use Psr\Log\LoggerInterface;
use Symfony\Component\HttpFoundation\Session\Session;
use Symfony\Component\Translation\TranslatorInterface;

class MessageHandler implements MessageHandlerInterface
{
    /**
     * @var LoggerInterface
     */
    protected $logger;

    /**
     * @var Session
     */
    protected $session;

    /**
     * @var TranslatorInterface
     */
    protected $translator;

    /**
     * ErrorHandler constructor
     *
     * @param LoggerInterface $logger
     * @param Session $session
     * @param TranslatorInterface $translator
     */
    public function __construct(LoggerInterface $logger, Session $session, TranslatorInterface $translator)
    {
        $this->logger = $logger;
        $this->session = $session;
        $this->translator = $translator;
    }

    /**
     * Create an flash message into the session. It will be translated. If $logMessage is defined, it will be logged
     * using the logger using the error channel
     *
     * @param string $flashMessage
     * @param string|null $logMessage
     */
    public function handleError($flashMessage, $logMessage = null)

This method seems to be duplicated in your project.

    {
        $this
            ->session
            ->getFlashBag()
            ->add('error', $this->translator->trans($flashMessage));

        if ($logMessage) {
            $this
                ->logger
                ->error($logMessage);
        }
    }

    /**
     * Create an flash message into the session. It will be translated. If $logMessage is defined, it will be logged
     * using the logger using info channel
     *
     * @param $flashMessage
     * @param null $logMessage
     */
    public function handleSuccess($flashMessage, $logMessage = null)

This method seems to be duplicated in your project.

    {
        $this
            ->session
            ->getFlashBag()
            ->add('info', $this->translator->trans($flashMessage));

        if ($logMessage) {
            $this
                ->logger
                ->info($logMessage);
        }
    }
}