MessageApp\Listener\UserEventHandler

Complexity

Total Complexity

8

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	12

Importance

Changes	4
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
B	handle()	0	26	4
A	isValidEvent()	0	4	3

<?php

namespace MessageApp\Listener;

use League\Event\EventInterface;
use MessageApp\Event\UserEvent;
use MessageApp\Finder\MessageFinder;
use MessageApp\Message\DefaultMessage;
use MessageApp\Message\Sender\MessageSender;
use MessageApp\User\Finder\AppUserFinder;
use Psr\Log\LoggerAwareInterface;
use Psr\Log\LoggerAwareTrait;
use Psr\Log\NullLogger;
use RemiSan\Context\Context;

class UserEventHandler implements MessageEventHandler, LoggerAwareInterface
{
    use LoggerAwareTrait;

    /**
     * @var AppUserFinder
     */
    private $userFinder;

    /**
     * @var MessageFinder
     */
    private $messageFinder;

    /**
     * @var MessageSender
     */
    private $messageSender;

    /**
     * Constructor
     *
     * @param AppUserFinder $userFinder
     * @param MessageFinder $messageFinder
     * @param MessageSender $messageSender
     */
    public function __construct(
        AppUserFinder $userFinder,
        MessageFinder $messageFinder,
        MessageSender $messageSender
    ) {
        $this->userFinder = $userFinder;
        $this->messageFinder = $messageFinder;
        $this->messageSender = $messageSender;
        $this->logger = new NullLogger();
    }

    /**
     * Handle an event.
     *
     * @param EventInterface $event
     * @param Context        $context
     *
     * @return void
     */
    public function handle(EventInterface $event, Context $context = null)
    {
        if (! self::isValidEvent($event)) {
            return;
        }

        // Build message
        $user = $this->userFinder->find($event->getUserId());

The method getUserId() does not seem to exist on object<League\Event\EventInterface>.

        $this->logger->info(
            'Send message',
            [
                'user' => $user->getName(),
                'message' => $event->getAsMessage(),

The method getAsMessage() does not seem to exist on object<League\Event\EventInterface>.

                'type' => $event->getName()
            ]
        );

        $message = null;

$message is not used, you could remove the assignment.

        if ($context) {
            $messageContext = $this->messageFinder->findByReference($context->getValue());
        }

        $message = new DefaultMessage($user, $event->getAsMessage());

The method getAsMessage() does not seem to exist on object<League\Event\EventInterface>.

        $this->messageSender->send($message, ($messageContext)?$messageContext->getSource():null);

The variable $messageContext does not seem to be defined for all execution paths leading up to this point.

    }

    /**
     * @param EventInterface $event
     *
     * @return bool
     */
    private static function isValidEvent(EventInterface $event)
    {
        return $event instanceof UserEvent && $event->getUserId() && $event->getAsMessage();
    }
}