App\Http\Controllers\Admin\AdminShowsController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	destroy()	0	17	2
A	edit()	0	29	6
A	index()	0	18	3

<?php

namespace App\Http\Controllers\Admin;

use App\Http\Controllers\BasePageController;
use App\Models\Release;
use App\Models\Video;
use Illuminate\Http\Request;

class AdminShowsController extends BasePageController
{
    /**
     * @param \Illuminate\Http\Request $request
     * @throws \Exception
     */
    public function index(Request $request)
    {
        $this->setAdminPrefs();

        $meta_title = $title = 'TV Shows List';

        $tvshowname = ($request->has('showname') && ! empty($request->input('showname')) ? $request->input('showname') : '');

        $this->smarty->assign(

The method assign() does not exist on Illuminate\Foundation\Application.

            [
                'showname'          => $tvshowname,
                'tvshowlist'        => Video::getRange($tvshowname),
            ]
        );

        $content = $this->smarty->fetch('show-list.tpl');

The method fetch() does not exist on Illuminate\Foundation\Application.

        $this->smarty->assign(compact('title', 'meta_title', 'content'));
        $this->adminrender();
    }

    /**
     * @param \Illuminate\Http\Request $request
     * @throws \Exception
     */
    public function edit(Request $request)
    {
        $this->setAdminPrefs();

        switch ($request->input('action') ?? 'view') {
            case 'submit':
                if ($request->has('from') && ! empty($request->input('from'))) {
                    header('Location:'.$request->input('from'));
                    exit;
                }

                header('Location:'.url('admin/show-list'));

Are you sure url('admin/show-list') of type Illuminate\Contracts\Routing\UrlGenerator|string can be used in concatenation?

                break;

            case 'view':
            default:
                if ($request->has('id')) {
                    $title = 'TV Show Edit';

The assignment to $title is dead and can be removed.

                    $show = Video::getByVideoID($request->input('id'));
                }
                break;
        }

        $this->smarty->assign('show', $show);

The variable $show does not seem to be defined for all execution paths leading up to this point.

        $meta_title = $title = 'Edit TV Show Data';
        $content = $this->smarty->fetch('show-edit.tpl');
        $this->smarty->assign(compact('title', 'meta_title', 'content'));
        $this->adminrender();
    }

    /**
     * @param $id
     *
     * @throws \Exception
     */
    public function destroy($id)
    {
        $this->setAdminPrefs();

        $success = false;

        if ($id) {
            $success = Release::removeVideoIdFromReleases($id);
            $this->smarty->assign('videoid', $id);
        }

        $this->smarty->assign('success', $success);

        $meta_title = $title = 'Remove Video and Episode IDs from Releases';
        $content = $this->smarty->fetch('show-remove.tpl');
        $this->smarty->assign(compact('title', 'meta_title', 'content'));
        $this->adminrender();
    }
}