Yajra\CMS\Http\Controllers\ExtensionsController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	index()	0	4	1
A	store()	0	10	2
A	destroy()	0	15	3

<?php

namespace Yajra\CMS\Http\Controllers;

use Illuminate\Http\Request;
use Yajra\CMS\DataTables\ExtensionsDataTable;
use Yajra\CMS\Repositories\Extension\Repository;

class ExtensionsController extends Controller
{
    /**
     * @var \Yajra\CMS\Repositories\Extension\Repository
     */
    protected $repository;

    /**
     * ExtensionsController constructor.
     *
     * @param \Yajra\CMS\Repositories\Extension\Repository $repository
     */
    public function __construct(Repository $repository)
    {
        $this->authorizePermissionResource('extension');
        $this->repository = $repository;
    }

    /**
     * Display extensions resource.
     *
     * @param \Yajra\CMS\DataTables\ExtensionsDataTable $dataTable
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function index(ExtensionsDataTable $dataTable)
    {
        return $dataTable->render('administrator.extensions.index');
    }

    public function store(Request $request)
    {
        $extension          = $this->repository->findOrFail($request->get('id'));
        $extension->enabled = ! $extension->enabled;
        $extension->save();

        $response = 'cms::extension.' . ($extension->enabled ? 'enabled' : 'disabled');

        return $this->notifySuccess(trans($response));
    }

    /**
     * Uninstall an extension.
     *
     * @param string $id
     * @return \Illuminate\Http\RedirectResponse
     */
    public function destroy($id)
    {
        $extension = $this->repository->findOrFail($id);
        if ($extension->protected) {
            return $this->notifyError(trans('cms::extension.protected'));
        }

        if ($this->repository->uninstall($id)) {
            flash()->success(trans('cms::extension.deleted', compact('id')));

It seems like trans('cms::extension.deleted', compact('id')) targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, Laracasts\Flash\FlashNotifier::success() does only seem to accept string, maybe add an additional type check?

        } else {
            flash()->error(trans('cms::extension.error', compact('id')));

It seems like trans('cms::extension.error', compact('id')) targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, Laracasts\Flash\FlashNotifier::error() does only seem to accept string, maybe add an additional type check?

        }

        return back();
    }
}