BeyondCode\LaravelWebSockets\Events\UnsubscribedFromChannel

Complexity

Total Complexity

1

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1

<?php

namespace BeyondCode\LaravelWebSockets\Events;

use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Queue\SerializesModels;
use stdClass;

class UnsubscribedFromChannel
{
    use Dispatchable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by BeyondCode\LaravelWebSockets\Events\UnsubscribedFromChannel: $id, $relations, $class, $connection, $keyBy

    /**
     * The WebSockets app id that the user connected to.
     *
     * @var string
     */
    public $appId;

    /**
     * The Socket ID associated with the connection.
     *
     * @var string
     */
    public $socketId;

    /**
     * The channel name.
     *
     * @var string
     */
    public $channelName;

    /**
     * The user received on presence channel.
     *
     * @var string
     */
    public $user;

    /**
     * Create a new event instance.
     *
     * @param  string  $appId
     * @param  string  $socketId
     * @param  string  $channelName
     * @param  stdClass|null  $user
     * @return void
     */
    public function __construct(string $appId, string $socketId, string $channelName, ?stdClass $user = null)
    {
        $this->appId = $appId;
        $this->socketId = $socketId;
        $this->channelName = $channelName;
        $this->user = $user;

It seems like $user can also be of type stdClass. However, the property $user is declared as type string. Maybe add an additional type check?

    }
}