Shamaseen\Laravel\Ratchet\Traits\WebSocketMessagesManager

Complexity

Total Complexity

13

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	sendBack()	0	4	1
A	getSenderData()	0	7	1
A	error()	0	17	2
A	sendToWebSocketUser()	0	15	5
A	sendToUser()	0	13	2
A	isOnline()	0	9	2

<?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 $request
     * @param ConnectionInterface $from
     * @param $error
     * @throws WebSocketException
     */
    function error($request,ConnectionInterface $from, $error)

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

    {
        if(env('APP_DEBUG'))
        {
            echo 'User error: ';
            echo $error."\n";
            print_r($request);
            echo " ============================================================== \n \n \n";
        }

        $data = [
            'event'=>'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.

    {
        if(!is_array($data))
            $data = ['msg'=>$data,'event'=>'default'];

        if(!isset($data['event']))
        {
            $data['event'] = 'default';
        }

        if(isset($this->route) && !array_key_exists('sender',$data))
            $data['sender'] = $this->getSenderData();

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

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

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

    {
        if(!$this->isOnline($user_id))
        {
            return false;
        }

        $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);
        return true;
    }

    /**
     * @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?

    }

    /**
     *
     * @return array
     */
    function getSenderData()

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

    {
        return [
            'id' => \Auth::id(),
            'name'=> \Auth::user()->name
        ];
    }

    function isOnline($user_id)

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

    {
        if(array_key_exists($user_id,$this->userAuthSocketMapper))
        {
            return true;
        }

        return false;
    }

}