Mostafaznv\Larupload\Concerns\Storage\Attachment\StyleAttachment

Complexity

Total Complexity

21

Size/Duplication

Total Lines	108
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
C	prepareStylePath()	0	36	12
A	handleVideoStyles()	0	17	3
A	handleStyles()	0	27	6

<?php

namespace Mostafaznv\Larupload\Concerns\Storage\Attachment;

use Illuminate\Support\Facades\Storage;
use Mostafaznv\Larupload\Enums\LaruploadFileType;
use Mostafaznv\Larupload\Larupload;
use Mostafaznv\Larupload\Actions\FixExceptionNamesAction;

trait StyleAttachment
{
    /**
     * Handle styles
     * resize, crop and generate styles from original file
     *
     * @param string $id
     * @param string $class
     * @param bool $standalone
     */
    protected function handleStyles(string $id, string $class, bool $standalone = false): void
    {
        switch ($this->type) {
            case LaruploadFileType::IMAGE:
                foreach ($this->imageStyles as $name => $style) {
                    $path = $this->getBasePath($id, $name);

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

                    $saveTo = $path . '/' . FixExceptionNamesAction::make($this->output['name'], $name)->run();

                    Storage::disk($this->disk)->makeDirectory($path);
                    $this->img($this->file)->resize($saveTo, $style);

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

                }

                break;

            case LaruploadFileType::VIDEO:
                if ($this->ffmpegQueue) {
                    if ($this->driverIsNotLocal()) {

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

                        $this->uploadOriginalFile($id, $this->localDisk);

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

                    }

                    $this->initializeFFMpegQueue($id, $class, $standalone);

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

                }
                else {
                    $this->handleVideoStyles($id);
                }

                break;
        }
    }

    /**
     * Handle styles for videos
     *
     * @param $id
     */
    protected function handleVideoStyles($id): void
    {
        foreach ($this->videoStyles as $name => $style) {
            $path = $this->getBasePath($id, $name);
            Storage::disk($this->disk)->makeDirectory($path);
            $saveTo = "$path/{$this->output['name']}";

            $this->ffmpeg()->manipulate($style, $saveTo);

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

        }

        if (count($this->streams)) {
            $fileName = pathinfo($this->output['name'], PATHINFO_FILENAME) . '.m3u8';

Are you sure pathinfo($this->output['name'], Mostafaznv\Larupload\Concerns\Storage\Attachment\PATHINFO_FILENAME) of type array|string can be used in concatenation?

            $path = $this->getBasePath($id, Larupload::STREAM_FOLDER);
            Storage::disk($this->disk)->makeDirectory($path);

            $this->ffmpeg()->stream($this->streams, $path, $fileName);
        }
    }

    /**
     * Prepare style path
     * this function will use to prepare full path of given style to generate url/download response
     *
     * @param string $style
     * @return string|null
     */
    protected function prepareStylePath(string $style): ?string
    {
        $staticStyles = [
            Larupload::ORIGINAL_FOLDER,
            Larupload::COVER_FOLDER,
            Larupload::STREAM_FOLDER
        ];

        if (isset($this->id) and (in_array($style, $staticStyles) or array_key_exists($style, $this->imageStyles) or array_key_exists($style, $this->videoStyles))) {
            $name = $style == Larupload::COVER_FOLDER
                ? $this->output['cover']
                : $this->output['name'];

            $type = $this->output['type']
                ? LaruploadFileType::from($this->output['type'])
                : null;

            if ($name and $style == Larupload::STREAM_FOLDER) {
                if ($type === LaruploadFileType::VIDEO) {
                    $name = pathinfo($name, PATHINFO_FILENAME) . '.m3u8';

Are you sure pathinfo($name, Mostafaznv\Larupload\Concerns\Storage\Attachment\PATHINFO_FILENAME) of type array|string can be used in concatenation?

                    $path = $this->getBasePath($this->id, $style);

                    return "$path/$name";
                }

                return null;
            }
            else if ($name and $this->styleHasFile($style)) {

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

                $name = FixExceptionNamesAction::make($name, $style)->run();
                $path = $this->getBasePath($this->id, $style);

                return "$path/$name";
            }
        }

        return null;
    }
}