Mostafaznv\Larupload\Concerns\Storage\Attachment\AttachmentEvents

Complexity

Total Complexity

7

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	saved()	0	30	5
A	deleted()	0	4	2

<?php

namespace Mostafaznv\Larupload\Concerns\Storage\Attachment;


use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\Storage;
use Mostafaznv\Larupload\Actions\GenerateFileIdAction;

trait AttachmentEvents
{
    public function saved(Model $model): Model
    {
        $this->id = GenerateFileIdAction::make($model, $this->secureIdsMethod, $this->mode, $this->name)->run();

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

        $this->uploaded = true;

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

        if (isset($this->file)) {
            if ($this->file == LARUPLOAD_NULL) {
                $this->clean($this->id);

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

            }
            else {
                if (!$this->keepOldFiles) {
                    $this->clean($this->id);
                }

                $this->setBasicDetails();

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

                $this->setMediaDetails();

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

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

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

                $this->setCover($this->id);

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

                $this->handleStyles($this->id, $model);

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

            }

            $model = $this->setAttributes($model);

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

        }
        else if (isset($this->cover)) {
            $this->setCover($this->id);

            $model = $this->setAttributes($model);
        }

        return $model;
    }

    public function deleted(): void
    {
        if (!$this->preserveFiles) {
            Storage::disk($this->disk)->deleteDirectory("$this->folder/$this->id");
        }
    }
}