ApplicationUserListener::handle() - Code Metrics - Inspection of "add listener for player created event" - remi-san/mini-game-message-app - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 422ca2...2d71c3 )

by Rémi

created 2016-10-13 19:41 UTC

ApplicationUserListener::handle() A

↳ Parent: ApplicationUserListener

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
c	0
b	0
f	0
rs	9.4285
cc	2
eloc	6
nc	2
nop	1

<?php

namespace MiniGameMessageApp\Listener;

use League\Event\EventInterface;
use League\Event\ListenerInterface;
use MiniGame\Event\PlayerCreatedEvent;
use MiniGameMessageApp\Finder\MiniGameUserFinder;
use Psr\Log\LoggerAwareInterface;
use Psr\Log\LoggerAwareTrait;
use Psr\Log\NullLogger;

class ApplicationUserListener implements ListenerInterface, LoggerAwareInterface
{
    use LoggerAwareTrait;

    /**
     * @var MiniGameUserFinder
     */
    private $finder;

    /**
     * Constructor
     *
     * @param MiniGameUserFinder $finder
     */
    public function __construct(
        MiniGameUserFinder $finder
    ) {
        $this->finder = $finder;
        $this->logger = new NullLogger();
    }

    /**
     * Handle an event.
     *
     * @param EventInterface $event
     *
     * @return void
     */
    public function handle(EventInterface $event)
    {
        if (! $event instanceof PlayerCreatedEvent) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            return;
        }

        $applicationUser = $this->finder->find($event->getExternalReference());

        $applicationUser->linkToPlayer($event->getGameId(), $event->getPlayerId());

        $this->finder->save($applicationUser);
    }

    /**
     * Check whether the listener is the given parameter.
     *
     * @param mixed $listener
     *
     * @return bool
     */
    public function isListener($listener)
    {
        return $this == $listener;
    }
}


Push — master ( 422ca2...2d71c3 )

ApplicationUserListener::handle() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

1			<?php
2
3			namespace MiniGameMessageApp\Listener;
4
5			use League\Event\EventInterface;
6			use League\Event\ListenerInterface;
7			use MiniGame\Event\PlayerCreatedEvent;
8			use MiniGameMessageApp\Finder\MiniGameUserFinder;
9			use Psr\Log\LoggerAwareInterface;
10			use Psr\Log\LoggerAwareTrait;
11			use Psr\Log\NullLogger;
12
13			class ApplicationUserListener implements ListenerInterface, LoggerAwareInterface
14			{
15			use LoggerAwareTrait;
16
17			/**
18			* @var MiniGameUserFinder
19			*/
20			private $finder;
21
22			/**
23			* Constructor
24			*
25			* @param MiniGameUserFinder $finder
26			*/
27			public function __construct(
28			MiniGameUserFinder $finder
29			) {
30			$this->finder = $finder;
31			$this->logger = new NullLogger();
32			}
33
34			/**
35			* Handle an event.
36			*
37			* @param EventInterface $event
38			*
39			* @return void
40			*/
41			public function handle(EventInterface $event)
42			{
43			if (! $event instanceof PlayerCreatedEvent) {
			0 ignored issues – show Bug introduced 2016-10-13 19:44 UTC by Report Bug Copy Issue Report The class `MiniGame\Event\PlayerCreatedEvent` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
44			return;
45			}
46
47			$applicationUser = $this->finder->find($event->getExternalReference());
48
49			$applicationUser->linkToPlayer($event->getGameId(), $event->getPlayerId());
50
51			$this->finder->save($applicationUser);
52			}
53
54			/**
55			* Check whether the listener is the given parameter.
56			*
57			* @param mixed $listener
58			*
59			* @return bool
60			*/
61			public function isListener($listener)
62			{
63			return $this == $listener;
64			}
65			}
66

remi-san / mini-game-message-app

Push — master ( 422ca2...2d71c3 )

ApplicationUserListener::handle() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like