App\Events\LoanTableUpdated

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	broadcastOn()	0	4	1

<?php

namespace App\Events;

use App\Loan;
use Illuminate\Broadcasting\Channel;
use Illuminate\Http\Request;
use Illuminate\Queue\SerializesModels;
use Illuminate\Broadcasting\PrivateChannel;
use Illuminate\Broadcasting\PresenceChannel;
use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Broadcasting\InteractsWithSockets;
use Illuminate\Contracts\Broadcasting\ShouldBroadcast;

class LoanTableUpdated implements ShouldBroadcast
{
    use Dispatchable, InteractsWithSockets, SerializesModels;

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

    /**
     * Action to highlight
     *
     * @var string
     */
    public $action;

    /**
     * Loan to highlight
     *
     * @var string
     */
    public $loan;

    /**
     * Sender window
     *
     * @var string
     */
    public $sender;

    /**
     * Create a new event instance.
     *
     * @param string $action
     * @param Request $request
     * @param Loan|null $loan
     */
    public function __construct(string $action, Request $request, Loan $loan = null)
    {
        $this->action = $action;
        $this->sender = $request->headers->get('x-bibrex-window');
        $this->loan = $loan ? $loan->toArray() : null;

It seems like $loan ? $loan->toArray() : null can also be of type array. However, the property $loan is declared as type string. Maybe add an additional type check?

    }

    /**
     * Get the channels the event should broadcast on.
     *
     * @return \Illuminate\Broadcasting\Channel|array
     */
    public function broadcastOn()
    {
        $library = \Auth::user();
        return new PrivateChannel('loans.' . $library->id);
    }
}