Xuandung38\LaravelFilemanager\Controllers\UploadController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	upload()	0	31	7
A	__construct()	0	4	1

<?php

namespace Xuandung38\LaravelFilemanager\Controllers;

use Illuminate\Support\Facades\Log;
use Xuandung38\LaravelFilemanager\Events\ImageIsUploading;
use Xuandung38\LaravelFilemanager\Events\ImageWasUploaded;
use Xuandung38\LaravelFilemanager\Lfm;

class UploadController extends LfmController
{
    protected $errors;

    public function __construct()
    {
        parent::__construct();
        $this->errors = [];
    }

    /**
     * Upload files
     *
     * @param void
     * @return string
     */
    public function upload()
    {
        $uploaded_files = request()->file('upload');

The function request was not found. Maybe you did not declare it correctly or list all dependencies?

        $error_bag = [];
        $new_filename = null;

        foreach (is_array($uploaded_files) ? $uploaded_files : [$uploaded_files] as $file) {
            try {
                $new_filename = $this->lfm->upload($file);

The method upload() does not exist on null.

The property lfm does not exist on Xuandung38\LaravelFilemanager\Controllers\UploadController. Since you implemented __get, consider adding a @property annotation.

            } catch (\Exception $e) {
                Log::error($e->getMessage(), [
                    'file' => $e->getFile(),
                    'line' => $e->getLine(),
                    'trace' => $e->getTraceAsString()
                ]);
                array_push($error_bag, $e->getMessage());
            }
        }

        if (is_array($uploaded_files)) {
            $response = count($error_bag) > 0 ? $error_bag : parent::$success_response;
        } else { // upload via ckeditor 'Upload' tab
            if (is_null($new_filename)) {
                $response = $error_bag[0];
            } else {
                $response = view(Lfm::PACKAGE_NAME . '::use')

The function view was not found. Maybe you did not declare it correctly or list all dependencies?

                    ->withFile($this->lfm->setName($new_filename)->url());
            }
        }

        return $response;
    }
}