MallardDuck\DynamicEcho\Channels\PrivateUserChannelParameters::__construct()

Complexity

Conditions	1
Paths	1

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	1.048

Importance

Changes

0

<?php
    public function __construct()
    {
        $this->channelType = \Illuminate\Broadcasting\PrivateChannel::class;
        $this->channelAuthName = 'App.Models.User.{userId}';
        $this->channelAuthCallback = static function (User $user, $userId) {
            return (int) $user->id === (int) $userId;
        };
        $this->channelContextBindingCallback = static function (Request $request) {
            return [
                'userId' => $request->user()->id
            ];
        };
        $this->eventChannelIdentifierBindingCallback = static function (HasDynamicChannelFormula $event) {
            $self = $event;
            return [
                'userId' => $self->userId,

Accessing userId on the interface MallardDuck\DynamicEcho\Contracts\HasDynamicChannelFormula suggest that you code against a concrete implementation. How about adding an instanceof check?

            ];
        };
        parent::__construct();
    }