Backpack\CRUD\app\Library\Uploaders\MultipleFiles::uploadFile()

Complexity

Conditions	10
Paths	24

Size

Total Lines	33
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function uploadFile(Model $entry, $value = null)
    {
        $filesToDelete = CRUD::getRequest()->get('clear_'.$this->getName());

The method getRequest() does not exist on Backpack\CRUD\app\Library\CrudPanel\CrudPanelFacade. Since you implemented __callStatic, consider adding a @method annotation.

        $value = $value ?? CRUD::getRequest()->file($this->getName());
        $previousFiles = $entry->getOriginal($this->getName()) ?? [];

        if (! is_array($previousFiles) && is_string($previousFiles)) {
            $previousFiles = json_decode($previousFiles, true);
        }

        if ($filesToDelete) {
            foreach ($previousFiles as $previousFile) {
                if (in_array($previousFile, $filesToDelete)) {
                    Storage::disk($this->getDisk())->delete($previousFile);

                    $previousFiles = Arr::where($previousFiles, function ($value, $key) use ($previousFile) {

The parameter $key is not used and could be removed.

                        return $value != $previousFile;
                    });
                }
            }
        }

        foreach ($value ?? [] as $file) {
            if ($file && is_file($file)) {
                $fileName = $this->getFileName($file);

                $file->storeAs($this->getPath(), $fileName, $this->getDisk());

                $previousFiles[] = $this->getPath().$fileName;
            }
        }

        return isset($entry->getCasts()[$this->getName()]) ? $previousFiles : json_encode($previousFiles);

The expression return IssetNode ? $previousFiles : json_encode($previousFiles) also could return the type array|string which is incompatible with the return type mandated by Backpack\CRUD\app\Library\Uploaders\Support\Interfaces\UploaderInterface::uploadFile() of void.

    }