SocketCluster\SocketCluster

Complexity

Total Complexity

5

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	4
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	error()	0	4	1
A	publish()	0	9	1
A	emit()	0	21	2

<?php
namespace SocketCluster;

use WebSocket\Client;
use Closure;
use Exception;

class SocketCluster
{
    /**
     * @var WebSocket\Client
     */
    protected $websocket;

    /**
     * @var string
     */
    protected $error;

    /**
     * Construct
     *
     * @param WebSocket\Client $websocket
     */
    public function __construct(Client $websocket)
    {
        $this->websocket = $websocket;

It seems like $websocket of type object<WebSocket\Client> is incompatible with the declared type object<SocketCluster\WebSocket\Client> of property $websocket.

    }

    /**
     * Publish Channel
     *
     * @param string       $channel
     * @param mixed        $data
     * @param Closure|null $callback
     *
     * @return boolean
     */
    public function publish($channel, $data, Closure $callback = null)
    {
        $pubData = [
            'channel' => $channel,
            'data' => $data,
        ];
        
        return $this->emit('#publish', $pubData, $callback);
    }

    /**
     * Emit Event
     *
     * @param string       $event
     * @param array        $data
     * @param Closure|null $callback
     *
     * @return boolean
     */
    protected function emit($event, array $data, Closure $callback = null)

The parameter $callback is not used and could be removed.

    {
        try {
            $eventData = [
                'event' => $event,
                'data' => $data,
            ];

            $sendData = (string) @json_encode($eventData);

            $this->websocket->send($sendData);

            $this->error = null;

            return true;

        } catch (Exception $e) {
            $this->error = $e->getMessage();
            return false;
        }
    }

    /**
     * Get Error
     *
     * @return string
     */
    public function error()
    {
        return $this->error;
    }
}