Yajra\CMS\Http\Controllers\ThemesController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes

0

4 Methods

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

<?php

namespace Yajra\CMS\Http\Controllers;

use Illuminate\Http\Request;
use Yajra\CMS\Entities\Configuration;
use Yajra\CMS\Repositories\Theme\Repository;

class ThemesController extends Controller
{
    /**
     * @var \Yajra\CMS\Repositories\Theme\Repository
     */
    protected $themes;

    /**
     * ThemesController constructor.
     *
     * @param \Yajra\CMS\Repositories\Theme\Repository $themes
     */
    public function __construct(Repository $themes)
    {
        $this->authorizePermissionResource('theme');
        $this->themes = $themes;
    }

    /**
     * Display themes resource.
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function index()
    {
        $themes = $this->themes->all();

        return view('administrator.themes.index', compact('themes'));
    }

    /**
     * Store new default theme.
     *
     * @param \Illuminate\Http\Request $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function store(Request $request)
    {
        $this->validate($request, [
            'theme' => 'required',
        ]);

        /** @var Configuration $config */
        $config        = Configuration::query()->firstOrCreate(['key' => 'theme.frontend']);
        $config->value = $request->get('theme');
        $config->save();

        flash()->success(trans('cms::theme.success', ['theme' => $request->get('theme')]));

It seems like trans('cms::theme.success', array('theme' => $request->get('theme'))) 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?

        return back();
    }

    /**
     * Uninstall a theme.
     *
     * @param string $theme
     * @return \Illuminate\Http\RedirectResponse
     */
    public function destroy($theme)
    {
        if ($this->themes->uninstall($theme)) {
            flash()->success(trans('cms::theme.deleted', compact('theme')));

It seems like trans('cms::theme.deleted', compact('theme')) 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::theme.error', compact('theme')));

It seems like trans('cms::theme.error', compact('theme')) 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();
    }
}