Larrock\Core\Events\ComponentItemDestroyed

Complexity

Total Complexity

1

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

1 Method

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

<?php

namespace Larrock\Core\Events;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Http\Request;
use Illuminate\Queue\SerializesModels;
use Larrock\Core\Component;

/**
 * Выбрасываемое событие на удаление материала из компонента
 * Class ComponentItemDestroyed
 * @package Larrock\Core\Events
 */
class ComponentItemDestroyed
{
    use SerializesModels;

    /** @var Component  */
    public $component;

    /** @var Model  */
    public $model;

    /** @var Request  */
    public $request;

    /**
     * Create a new event instance.
     *
     * @param Component $component
     * @param Model $data
     * @param Request $request
     */
    public function __construct(Component $component, Model $data, Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->component = $component;
        $this->model = $data;
        $this->request = $request;
    }
}