Meema\MediaRecognition\Traits\CanRecognizeImages

Complexity

Total Complexity

7

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	blob()	0	5	1
A	setImageSettings()	0	23	3
A	detectImageFaces()	0	11	1
A	detectImageLabels()	0	15	2

<?php

namespace Meema\MediaRecognition\Traits;

use Exception;
use Meema\MediaRecognition\Models\MediaRecognition;

trait CanRecognizeImages
{
    /**
     * The input image as base64-encoded bytes.
     *
     * @var string|null
     */
    protected ?string $blob = null;

    /**
     * Set the base64 encoded image.
     *
     * @param string $blob
     *
     * @return $this
     */
    public function blob(string $blob)
    {
        $this->blob = $blob;

        return $this;
    }

    /**
     * Sets the image to be analyzed.
     *
     * @return void
     * @throws \Exception
     */
    protected function setImageSettings(): void
    {
        $this->ensureSourceIsNotNull();

It seems like ensureSourceIsNotNull() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (is_string($this->blob)) {
            $this->settings['Image'] = [

The property settings does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                'Bytes' => $this->blob,
            ];

            return;
        }

        $disk = $this->disk ?? config('media-recognition.disk');
        $bucketName = config("filesystems.disks.$disk.bucket");

        if (! $bucketName) {
            throw new Exception('Please make sure to set a S3 bucket name.');
        }

        $this->settings['Image'] = [
            'S3Object' => [
                'Bucket' => $bucketName,
                'Name' => $this->source,
            ],
        ];
    }

    /**
     * @param $mediaId
     * @param null $minConfidence

Are you sure the doc-type for parameter $minConfidence is correct as it would always require null to be passed?

     * @param null $maxLabels

Are you sure the doc-type for parameter $maxLabels is correct as it would always require null to be passed?

     * @return mixed
     * @throws \Exception
     */
    public function detectImageLabels($mediaId, $minConfidence = null, $maxLabels = null)
    {
        $this->setImageSettings();

        $this->settings['MinConfidence'] = $minConfidence ?? config('media-recognition.min_confidence');

The property settings does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        if (is_int($maxLabels)) {

The condition is_int($maxLabels) is always false.

            $this->settings['MaxLabels'] = $maxLabels;
        }

        $results = $this->client->detectLabels($this->settings);

        $this->updateOrCreate('labels', $mediaId, $results);

It seems like updateOrCreate() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $results;
    }

    /**
     * @param int|null $mediaId
     * @param array $attributes
     * @return mixed
     * @throws \Exception
     */
    public function detectImageFaces($mediaId = null, $attributes = ['DEFAULT'])
    {
        $this->setImageSettings();

        $this->settings['Attributes'] = $attributes;

The property settings does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $results = $this->client->detectFaces($this->settings);

        $this->updateOrCreate('faces', $mediaId, $results);

        return $results;
    }
}