NotifyOrderChangeJob::handle() - Code Metrics - SierraTecnologia/Gamer - Measure and Improve Code Quality continuously with Scrutinizer

NotifyOrderChangeJob::handle() A
last analyzed 2020-09-08 01:28 UTC

↳ Parent: NotifyOrderChangeJob

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?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\Order;
use Log;

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

    protected $order;

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

     */
    public function __construct(Service $order)
    {
        $this->order = $order;
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        // Captura Usuário Responsável
        $user = $this->order->user;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        // Avisa Servidor da alteração no status
        Organizer::getOrganizerServiceForUser(
            $this->order->user, $this->order->companyToken
        )->foundOrganizerDataByToken();
    }
}


1			<?php
2
3			namespace Gamer\Jobs;
4
5			use Illuminate\Bus\Queueable;
6			use Illuminate\Queue\SerializesModels;
7			use Illuminate\Queue\InteractsWithQueue;
8			use Illuminate\Contracts\Queue\ShouldQueue;
9			use Illuminate\Foundation\Bus\Dispatchable;
10
11			use Gamer\Models\Order;
12			use Log;
13
14			class NotifyOrderChangeJob implements ShouldQueue
15			{
16			use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
17
18			protected $order;
19
20			/**
21			* Create a new job instance.
22			*
23			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2020-09-08 01:28 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
24			*/
25			public function __construct(Service $order)
26			{
27			$this->order = $order;
28			}
29
30			/**
31			* Execute the job.
32			*
33			* @return void
34			*/
35			public function handle()
36			{
37			// Captura Usuário Responsável
38			$user = $this->order->user;
			0 ignored issues – show Unused Code introduced 2020-09-08 01:28 UTC by Report Bug Copy Issue Report `$user` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
39
40			// Avisa Servidor da alteração no status
41			Organizer::getOrganizerServiceForUser(
42			$this->order->user, $this->order->companyToken
43			)->foundOrganizerDataByToken();
44			}
45			}
46

SierraTecnologia / Gamer

NotifyOrderChangeJob::handle() A last analyzed 2020-09-08 01:28 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

NotifyOrderChangeJob::handle() A
last analyzed 2020-09-08 01:28 UTC