Itstructure\MFU\Http\Controllers\Managers\FileListManagerController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	23	5
A	delete()	0	9	3

<?php

namespace Itstructure\MFU\Http\Controllers\Managers;

use Throwable;
use Illuminate\Http\Request;
use Itstructure\GridView\DataProviders\EloquentDataProvider;
use Itstructure\MFU\Facades\Uploader;
use Itstructure\MFU\Http\Controllers\BaseController;
use Itstructure\MFU\Models\Owners\OwnerMediafile;
use Itstructure\MFU\Models\Mediafile;

/**
 * Class FileListManagerController
 * @package Itstructure\MFU\Http\Controllers\Managers
 */
class FileListManagerController extends BaseController
{
    /**
     * @param Request $request
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     */
    public function index(Request $request)
    {
        $requestParams = [];

        if ((null !== $request->get('owner_name') && null !== $request->get('owner_id'))) {
            $requestParams['owner_name'] = $request->get('owner_name');
            $requestParams['owner_id'] = $request->get('owner_id');
        }

        if (null !== $request->get('owner_attribute')) {
            $requestParams['owner_attribute'] = $request->get('owner_attribute');
        }

        if (count($requestParams) > 0) {
            $query = OwnerMediafile::getMediaFilesQuery($requestParams)->orWhereNotIn('id', OwnerMediafile::pluck('mediafile_id')->toArray());
        } else {
            $query = Mediafile::query();
        }

        return view('uploader::managers.file-list', [
            'dataProvider' => new EloquentDataProvider($query),
            'manager' => 'file_list',
            'fromFileSetter' => !empty($request->get('from_file_setter'))
        ]);
    }

    /**
     * @param Request $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function delete(Request $request)
    {
        foreach ($request->items ?? [] as $id) {
            try {
                Uploader::delete($id);

The method delete() does not exist on Itstructure\MFU\Facades\Uploader. Since you implemented __callStatic, consider adding a @method annotation.

            } catch (Throwable $e) {}

Consider adding a comment why this CATCH block is empty.

        }

        return back();
    }
}