Shamaseen\Laravel\Ratchet\Traits\WebSocketMessagesManager

Complexity

Total Complexity

4

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	error()	0	12	1
A	sendToWebSocketUser()	0	4	1
A	sendToUser()	0	7	1
A	sendBack()	0	4	1

<?php
/**
 * Created by PhpStorm.
 * User: shanmaseen
 * Date: 23/03/19
 * Time: 10:15 م
 */

namespace Shamaseen\Laravel\Ratchet\Traits;


use Shamaseen\Laravel\Ratchet\Exceptions\WebSocketException;
use Ratchet\ConnectionInterface;

/**
 * Trait WebSocketMessagesManager
 * @package App\WebSockets
 */
trait WebSocketMessagesManager
{
    /**
     * @param $msg
     * @param $from
     * @param $error
     * @throws WebSocketException
     */
    function error($msg,ConnectionInterface $from, $error)

It is generally recommended to explicitly declare the visibility for methods.

    {
        echo 'Error: ';
        echo $error."\n";
        print_r($msg);
        $data = [
            'type'=>'error',
            'message'=>$error
        ];
        $this->sendToWebSocketUser($from,$data);
        throw new WebSocketException();
    }

    /**
     * @param ConnectionInterface $conn
     * @param $data
     */
    function sendToWebSocketUser(ConnectionInterface $conn,$data)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $conn->send(json_encode($data));
    }

    /**
     * @param int $user_id
     * @param array $data
     */
    function sendToUser($user_id,$data)

It is generally recommended to explicitly declare the visibility for methods.

The variable $user_id is not named in camelCase.

    {
        $resourceId = $this->userAuthSocketMapper[$user_id];

The property userAuthSocketMapper does not exist. Did you maybe forget to declare it?

        /** @var ConnectionInterface $conn */
        $conn = $this->clients[$resourceId]->conn;

The property clients does not exist. Did you maybe forget to declare it?

        $this->sendToWebSocketUser($conn,$data);
    }

    /**
     * @param $data
     */
    function sendBack($data)

It is generally recommended to explicitly declare the visibility for methods.

    {
       $this->sendToWebSocketUser($this->conn,$data);

The property conn does not exist. Did you maybe forget to declare it?

    }

}