MarkAsReadSubscriber - Code Metrics - sfcod/socketio - Measure and Improve Code Quality continuously with Scrutinizer

MarkAsReadSubscriber A
last analyzed 2020-08-30 00:27 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

Metric	Value
wmc	3
lcom	0
cbo	1
dl	0
loc	28
rs	10
c	0
b	0
f	0

3 Methods

Rating	Name	Size	Complexity
A	broadcastOn()	4	1
A	name()	4	1
A	handle()	6	1

<?php

namespace SfCod\SocketIoBundle\Tests\Data;

use SfCod\SocketIoBundle\Events\AbstractEvent;
use SfCod\SocketIoBundle\Events\EventInterface;
use SfCod\SocketIoBundle\Events\EventSubscriberInterface;
use SfCod\SocketIoBundle\Service\Broadcast;

class MarkAsReadSubscriber extends AbstractEvent implements EventInterface, EventSubscriberInterface
{
    /**
     * Changel name. For client side this is nsp.
     */
    public static function broadcastOn(): array
    {
        return ['notifications'];
    }

    /**
     * Event name.
     */
    public static function name(): string
    {
        return 'mark_as_read_notification';
    }

    /**
     * Emit client event.
     */
    public function handle()
    {
        // Mark notification as read
        // And call client update
        $this->container->get(Broadcast::class)->emit('update_notification_count', ['some key' => 'some value']);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }
}


1			<?php
2
3			namespace SfCod\SocketIoBundle\Tests\Data;
4
5			use SfCod\SocketIoBundle\Events\AbstractEvent;
6			use SfCod\SocketIoBundle\Events\EventInterface;
7			use SfCod\SocketIoBundle\Events\EventSubscriberInterface;
8			use SfCod\SocketIoBundle\Service\Broadcast;
9
10			class MarkAsReadSubscriber extends AbstractEvent implements EventInterface, EventSubscriberInterface
11			{
12			/**
13			* Changel name. For client side this is nsp.
14			*/
15			public static function broadcastOn(): array
16			{
17			return ['notifications'];
18			}
19
20			/**
21			* Event name.
22			*/
23			public static function name(): string
24			{
25			return 'mark_as_read_notification';
26			}
27
28			/**
29			* Emit client event.
30			*/
31			public function handle()
32			{
33			// Mark notification as read
34			// And call client update
35			$this->container->get(Broadcast::class)->emit('update_notification_count', ['some key' => 'some value']);
			0 ignored issues – show Documentation introduced 2018-06-15 11:18 UTC by Report Bug Copy Issue Report The property `container` does not exist on `object<SfCod\SocketIoBun...a\MarkAsReadSubscriber>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
36			}
37			}
38

sfcod / socketio

MarkAsReadSubscriber A last analyzed 2020-08-30 00:27 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

MarkAsReadSubscriber A
last analyzed 2020-08-30 00:27 UTC