App\Http\Controllers\Admin\AdminMusicController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	13	1
C	edit()	0	51	12

<?php

namespace App\Http\Controllers\Admin;

use App\Http\Controllers\BasePageController;
use Blacklight\Genres;
use Blacklight\Music;
use Blacklight\utility\Utility;
use Illuminate\Http\Request;

class AdminMusicController extends BasePageController
{
    /**
     * @throws \Exception
     */
    public function index()
    {
        $this->setAdminPrefs();

        $meta_title = $title = 'Music List';

        $musicList = Utility::getRange('musicinfo');

        $this->smarty->assign('musiclist', $musicList);

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

        $content = $this->smarty->fetch('music-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
     *
     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
     * @throws \Exception
     */
    public function edit(Request $request)
    {
        $this->setAdminPrefs();
        $music = new Music();
        $gen = new Genres();

        $meta_title = $title = 'Music Edit';

        // set the current action
        $action = $request->input('action') ?? 'view';

        if ($request->has('id')) {
            $id = $request->input('id');
            $mus = $music->getMusicInfo($id);

            if (! $mus) {

$mus is of type App\Models\MusicInfo, thus it always evaluated to true.

                $this->show404();
            }

            switch ($action) {
                case 'submit':
                    $coverLoc = resource_path().'/covers/music/'.$id.'.jpg';

                    if ($_FILES['cover']['size'] > 0) {
                        $tmpName = $_FILES['cover']['tmp_name'];
                        $file_info = getimagesize($tmpName);
                        if (! empty($file_info)) {
                            move_uploaded_file($_FILES['cover']['tmp_name'], $coverLoc);
                        }
                    }

                    $request->merge(['cover' => file_exists($coverLoc) ? 1 : 0]);
                    $request->merge(['salesrank' => (empty($request->input('salesrank')) || ! ctype_digit($request->input('salesrank'))) ? 'null' : $request->input('salesrank')]);
                    $request->merge(['releasedate' => (empty($request->input('releasedate')) || ! strtotime($request->input('releasedate'))) ? $mus['releasedate'] : Carbon::parse($request->input('releasedate'))->timestamp]);

The type App\Http\Controllers\Admin\Carbon was not found. Maybe you did not declare it correctly or list all dependencies?

                    $music->update($id, $request->input('title'), $request->input('asin'), $request->input('url'), $request->input('salesrank'), $request->input('artist'), $request->input('publisher'), $request->input('releasedate'), $request->input('year'), $request->input('tracks'), $request->input('cover'), $request->input('genre'));

                    return redirect('admin/music-list');

                    break;

break is not strictly necessary here and could be removed.

                case 'view':
                default:
                    $this->smarty->assign('music', $mus);
                    $this->smarty->assign('genres', $gen->getGenres(Genres::MUSIC_TYPE));
                    break;
            }
        }

        $content = $this->smarty->fetch('music-edit.tpl');
        $this->smarty->assign(compact('title', 'meta_title', 'content'));
        $this->adminrender();
    }
}