Cortex\Tenants\Http\Controllers\Adminarea\TenantsMediaController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	6

Importance

Changes	2
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	authorizeResource()	0	16	5
A	index()	0	10	1
A	store()	0	8	1
A	destroy()	0	9	1

<?php

declare(strict_types=1);

namespace Cortex\Tenants\Http\Controllers\Adminarea;

use Illuminate\Support\Str;
use Rinvex\Tenants\Models\Tenant;
use Spatie\MediaLibrary\Models\Media;
use Cortex\Foundation\DataTables\MediaDataTable;
use Cortex\Foundation\Http\Requests\ImageFormRequest;
use Cortex\Foundation\Http\Controllers\AuthorizedController;

class TenantsMediaController extends AuthorizedController
{
    /**
     * {@inheritdoc}
     */
    protected $resource = 'tenant';

    /**
     * {@inheritdoc}
     */
    public function authorizeResource($model, $parameter = null, array $options = [], $request = null): void
    {
        $middleware = [];
        $parameter = $parameter ?: Str::snake(class_basename($model));

        foreach ($this->mapResourceAbilities() as $method => $ability) {
            $modelName = in_array($method, $this->resourceMethodsWithoutModels()) ? $model : $parameter;

            $middleware["can:update,{$modelName}"][] = $method;
            $middleware["can:{$ability},media"][] = $method;
        }

        foreach ($middleware as $middlewareName => $methods) {
            $this->middleware($middlewareName, $options)->only($methods);
        }
    }

    /**
     * List tenant media.
     *
     * @param \Cortex\Tenants\Models\Tenant                $tenant

Consider making the type for parameter $tenant a bit more specific; maybe use Tenant.

     * @param \Cortex\Foundation\DataTables\MediaDataTable $mediaDataTable
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse

Should the return type not be \Illuminate\Http\JsonResponse|\Illuminate\View\View?

     */
    public function index(Tenant $tenant, MediaDataTable $mediaDataTable)
    {
        return $mediaDataTable->with([
            'resource' => $tenant,
            'tabs' => 'adminarea.tenants.tabs',
            'phrase' => trans('cortex/tenants::common.tenants'),
            'id' => "adminarea-tenants-{$tenant->getKey()}-media-table",
            'url' => route('adminarea.tenants.media.store', ['tenant' => $tenant]),
        ])->render('cortex/foundation::adminarea.pages.datatable-media');
    }

    /**
     * Store new tenant media.
     *
     * @param \Cortex\Foundation\Http\Requests\ImageFormRequest $request
     * @param \Cortex\Tenants\Models\Tenant                     $tenant

Consider making the type for parameter $tenant a bit more specific; maybe use Tenant.

     *
     * @return void
     */
    public function store(ImageFormRequest $request, Tenant $tenant): void

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

    {
        $tenant->addMediaFromRequest('file')
               ->sanitizingFileName(function ($fileName) {
                   return md5($fileName).'.'.pathinfo($fileName, PATHINFO_EXTENSION);
               })
               ->toMediaCollection('default', config('cortex.tenants.media.disk'));
    }

    /**
     * Destroy given tenant media.
     *
     * @param \Cortex\Tenants\Models\Tenant     $tenant

Consider making the type for parameter $tenant a bit more specific; maybe use Tenant.

     * @param \Spatie\MediaLibrary\Models\Media $media
     *
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function destroy(Tenant $tenant, Media $media)
    {
        $tenant->media()->where($media->getKeyName(), $media->getKey())->first()->delete();

        return intend([
            'url' => route('adminarea.tenants.media.index', ['tenant' => $tenant]),
            'with' => ['warning' => trans('cortex/foundation::messages.resource_deleted', ['resource' => 'media', 'id' => $media->getKey()])],

This line exceeds maximum limit of 120 characters; contains 142 characters

        ]);
    }
}