Mostafaznv\Larupload\Concerns\Storage\Attachment\AttachmentEvents::saved()

Complexity

Conditions	5
Paths	5

Size

Total Lines	30
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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;
    }