Mostafaznv\Larupload\Concerns\Storage\Attachment\AttachmentActions::attach()

Complexity

Conditions	8
Paths	16

Size

Total Lines	25
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function attach(mixed $file, ?UploadedFile $cover = null): bool
    {
        $fileIsAttachable = (file_has_value($file) or $file == LARUPLOAD_NULL);
        $coverIsAttachable = (file_has_value($cover) or $cover == null);

        if ($fileIsAttachable and $coverIsAttachable) {
            file_is_valid($file, $this->name, 'file');
            file_is_valid($cover, $this->name, 'cover');

            $this->file = $file;

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

            $this->uploaded = false;

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

            if ($file != LARUPLOAD_NULL) {
                $this->cover = $cover;

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

                $this->type = GuessLaruploadFileTypeAction::make($file)->calc();

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

                if ($this->type === LaruploadFileType::IMAGE && $this->optimizeImage) {
                    $this->file = OptimizeImageAction::make($file)->process();
                }
            }

            return true;
        }

        return false;
    }