StartLabelDetection::handle() - Code Metrics - meemalabs/laravel-media-recognition - Measure and Improve Code Quality continuously with Scrutinizer

StartLabelDetection::handle() A
last analyzed 2021-01-16 04:52 UTC

↳ Parent: StartLabelDetection

Complexity

Conditions	3
Paths	3

Size

Total Lines	21
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	9
c	0
b	0
f	0
nc	3
nop	0
dl	0
loc	21
rs	9.9666

<?php

namespace Meema\MediaRecognition\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Str;
use Meema\MediaRecognition\Events\LabelAnalysisCompleted;
use Meema\MediaRecognition\Facades\Recognize;

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


    private string $path;

    /**
     * $mimeType may be 'image', 'video' or the actual mime type of the file.
     * It allows us to reduce an HTTP request to check for the mime type.
     * If not assigned, it will check the mime type for whether it is an image or a video source.
     *
     * @var string|null
     */
    private ?string $mimeType = null;

    private ?int $mediaId = null;

    private ?int $minConfidence;

    private int $maxResults;

    /**
     * Create a new job instance.
     *
     * @param string $path
     * @param string|null $mimeType
     * @param int|null $mediaId
     * @param int|null $minConfidence
     * @param int $maxResults
     */
    public function __construct(string $path, $mimeType = null, $mediaId = null, $minConfidence = null, $maxResults = 1000)
    {
        $this->path = $path;
        $this->mimeType = $mimeType;
        $this->mediaId = $mediaId;
        $this->minConfidence = $minConfidence;
        $this->maxResults = $maxResults;
    }

    /**
     * Execute the job.
     *
     * @return void
     * @throws \Exception
     */
    public function handle()
    {
        $this->ensureMimeTypeIsSet();

        if (Str::contains($this->mimeType, 'image')) {
            $result = Recognize::source($this->path, $this->mimeType, $this->mediaId)->detectImageLabels($this->minConfidence, $this->maxResults);

            // we need to manually fire the event for image analyses because unlike the video analysis,
            // AWS is not sending a webhook upon completion of the image analysis
            event(new LabelAnalysisCompleted($result, $this->mediaId));

            return;
        }

        if (Str::contains($this->mimeType, 'video')) {
            Recognize::source($this->path, $this->mimeType, $this->mediaId)->detectVideoLabels($this->minConfidence, $this->maxResults);

            return;
        }

        throw new \Exception('$mimeType does neither indicate being a video nor an image');
    }

    protected function ensureMimeTypeIsSet()
    {
        if (is_null($this->mimeType)) {
            $this->mimeType = Storage::disk(config('media-recognition.disk'))->mimeType($this->path);
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        }
    }
}


1			<?php
2
3			namespace Meema\MediaRecognition\Jobs;
4
5			use Illuminate\Bus\Queueable;
6			use Illuminate\Contracts\Queue\ShouldQueue;
7			use Illuminate\Foundation\Bus\Dispatchable;
8			use Illuminate\Queue\InteractsWithQueue;
9			use Illuminate\Queue\SerializesModels;
10			use Illuminate\Support\Facades\Storage;
11			use Illuminate\Support\Str;
12			use Meema\MediaRecognition\Events\LabelAnalysisCompleted;
13			use Meema\MediaRecognition\Facades\Recognize;
14
15			class StartLabelDetection implements ShouldQueue
16			{
17			use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
			0 ignored issues – show introduced 2021-01-02 00:25 UTC by Report Bug Copy Issue Report The trait `Illuminate\Queue\SerializesModels` requires some properties which are not provided by `Meema\MediaRecognition\Jobs\StartLabelDetection`: `$id`, `$relations`, `$class`, `$keyBy` Loading history...
18
19			private string $path;
20
21			/**
22			* $mimeType may be 'image', 'video' or the actual mime type of the file.
23			* It allows us to reduce an HTTP request to check for the mime type.
24			* If not assigned, it will check the mime type for whether it is an image or a video source.
25			*
26			* @var string\|null
27			*/
28			private ?string $mimeType = null;
29
30			private ?int $mediaId = null;
31
32			private ?int $minConfidence;
33
34			private int $maxResults;
35
36			/**
37			* Create a new job instance.
38			*
39			* @param string $path
40			* @param string\|null $mimeType
41			* @param int\|null $mediaId
42			* @param int\|null $minConfidence
43			* @param int $maxResults
44			*/
45			public function __construct(string $path, $mimeType = null, $mediaId = null, $minConfidence = null, $maxResults = 1000)
46			{
47			$this->path = $path;
48			$this->mimeType = $mimeType;
49			$this->mediaId = $mediaId;
50			$this->minConfidence = $minConfidence;
51			$this->maxResults = $maxResults;
52			}
53
54			/**
55			* Execute the job.
56			*
57			* @return void
58			* @throws \Exception
59			*/
60			public function handle()
61			{
62			$this->ensureMimeTypeIsSet();
63
64			if (Str::contains($this->mimeType, 'image')) {
65			$result = Recognize::source($this->path, $this->mimeType, $this->mediaId)->detectImageLabels($this->minConfidence, $this->maxResults);
66
67			// we need to manually fire the event for image analyses because unlike the video analysis,
68			// AWS is not sending a webhook upon completion of the image analysis
69			event(new LabelAnalysisCompleted($result, $this->mediaId));
70
71			return;
72			}
73
74			if (Str::contains($this->mimeType, 'video')) {
75			Recognize::source($this->path, $this->mimeType, $this->mediaId)->detectVideoLabels($this->minConfidence, $this->maxResults);
76
77			return;
78			}
79
80			throw new \Exception('$mimeType does neither indicate being a video nor an image');
81			}
82
83			protected function ensureMimeTypeIsSet()
84			{
85			if (is_null($this->mimeType)) {
86			$this->mimeType = Storage::disk(config('media-recognition.disk'))->mimeType($this->path);
			0 ignored issues – show Documentation Bug introduced 2021-01-02 01:25 UTC by Report Bug Copy Issue Report It seems like `Illuminate\Support\Facad...->mimeType($this->path)` can also be of type `false`. However, the property `$mimeType` is declared as type `null\|string`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
87			}
88			}
89			}
90

meemalabs / laravel-media-recognition

StartLabelDetection::handle() A last analyzed 2021-01-16 04:52 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

StartLabelDetection::handle() A
last analyzed 2021-01-16 04:52 UTC