MAbadir\ElasticLaravel\Jobs\IndexModel

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

2 Methods

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

<?php

namespace MAbadir\ElasticLaravel\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Queue\SerializesModels;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Contracts\Queue\ShouldQueue;
use MAbadir\ElasticLaravel\ElasticClient;

class IndexModel implements ShouldQueue
{
    use InteractsWithQueue, Queueable, SerializesModels;

    public $model, $params, $operation;

It is generally advisable to only define one property per statement.

    /**
     * Create a new job instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($model, $params, $operation)
    {
        $this->model = $model;
        $this->params = $params;
        $this->operation = $operation;
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        $client = new ElasticClient();
        $client->{$this->operation}($this->params);
    }
}