Gamer\Jobs\RoutineTrackingUserDeviseJob

Complexity

Total Complexity

3

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2
A	handle()	0	11	1

<?php

namespace Gamer\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Queue\SerializesModels;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;

use Gamer\Models\Customer;
use Log;

class RoutineTrackingUserDeviseJob implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    protected $customer, $trackingType, $trackingData, $timestamp, $íp;

    /**
     * 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(Customer $customer, $trackingType, $trackingData, $timestamp = false, $ip = false)
    {
        $this->customer = $customer;
        $this->trackingType = $trackingType;
        $this->trackingData = $trackingData;
        if (!$timestamp) { $timestamp = time();
        }
        $this->timestamp = $timestamp;
        $this->ip = $ip;

The property ip does not exist. Did you maybe forget to declare it?

    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        $this->customer->tracking->create(
            TrackingType::createToTracking(
                $this->trackingType,
                $this->trackingData,
                $this->timestamp,
                $this->ip
            )
        );
    }
}