Crocodicstudio\Crudbooster\Modules\FileManagerModule\AdminFileManagerController

Complexity

Total Complexity

16

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	postCreateDirectory()	0	7	2
A	getDeleteFile()	0	7	2
A	postUpload()	0	17	4
A	getDeleteDirectory()	0	6	1
A	cbInit()	0	2	1
A	getIndex()	0	15	6

<?php

namespace Crocodicstudio\Crudbooster\Modules\FileManagerModule;

use Crocodicstudio\Crudbooster\controllers\CBController;
use Crocodicstudio\Crudbooster\Modules\ModuleGenerator\ControllerGenerator\FieldDetector;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Facades\Request;

class AdminFileManagerController extends CBController
{
    public function cbInit()
    {
    }

    public function getIndex()
    {
        $path = request('path') ? base64_decode(request('path')) : '';

It seems like request('path') can also be of type array; however, parameter $data of base64_decode() does only seem to accept string, maybe add an additional type check?

        if (strpos($path, '..') || $path == '.' || strpos($path, '/.')) {
            return redirect()->route('AdminFileManagerControllerGetIndex');
        }

        $currentPath = $path ?: 'uploads';
        $currentPath = trim($currentPath, '/');

        $directories = Storage::directories($currentPath);
        $files = Storage::files($currentPath);

        return view('CbFileManager::index', ['files' => $files, 'directories' => $directories, 'currentPath' => $currentPath]);
    }

    public function postCreateDirectory()
    {
        $path = (base64_decode(request('path'))) ?: 'uploads';

It seems like request('path') can also be of type array; however, parameter $data of base64_decode() does only seem to accept string, maybe add an additional type check?

        $name = str_slug(request('name'), '_');

It seems like request('name') can also be of type array; however, parameter $title of str_slug() does only seem to accept string, maybe add an additional type check?

        Storage::makeDirectory($path.'/'.$name);

        backWithMsg('The directory has been created!');
    }

    public function postUpload()
    {
        $path = request('path') ? base64_decode(request('path')) : 'uploads';

It seems like request('path') can also be of type array; however, parameter $data of base64_decode() does only seem to accept string, maybe add an additional type check?

        $file = Request::file('userfile');
        if (! $file) {
            return null;
        }

        $fileName = $file->getClientOriginalName();

        if (! FieldDetector::isUploadField($file->getClientOriginalExtension())) {
            backWithMsg('The file '.$fileName.' type is not allowed!', 'warning');
        }

        Storage::putFileAs($path, $file, $fileName);

        backWithMsg('The file '.$fileName.' has been uploaded!');
    }

    public function getDeleteDirectory($dir)
    {
        $dir = base64_decode($dir);
        Storage::deleteDirectory($dir);

        backWithMsg('The directory has been deleted!');
    }

    public function getDeleteFile($file)
    {
        $file = base64_decode($file);
        if(Storage::delete($file))
            backWithMsg('The file has been deleted!');

        backWithMsg('The file did not deleted!', 'warning');
    }
}