App\Http\Controllers\TeamsController

Complexity

Total Complexity

12

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	delete()	0	8	2
A	update()	0	17	3
A	deleteConfirm()	0	6	2
A	edit()	0	2	1
A	create()	0	3	1
A	store()	0	18	3

<?php

namespace App\Http\Controllers;

use App\Association;
use App\Team;
use Bouncer;

The type Bouncer was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Http\Request;

class TeamsController extends Controller
{

    public function create(Association $association) {
        return view('team.create', [
            'association' => $association,
        ]);
    }

    public function store(Association $association, Request $request) {
        if (Bouncer::can('create', Team::class)) {
            $validatedData = $request->validate([

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

                'name' => 'required|max:255',
            ]);

            $team = new team;

            $team->name = $request->name;
            $team->association_id = $association->id;
            $team->venue_id = !empty($request->venue_id) ? $request->venue_id : null;

            $team->save();

            return redirect()->route('association.teams', ['association' => $association]);
        }
        else {
            return view('denied');
        }
    }

    public function update(Association $association, Team $team, Request $request) {

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

        if (Bouncer::can('update', Team::class)) {
            $validatedData = $request->validate([

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

                'name' => 'required|max:255',
            ]);

            $team = Team::where(['id' => $team->id])->first();

            $team->name = $request->name;
            $team->venue_id = !empty($request->venue_id) ? $request->venue_id : null;

            $team->save();

            return redirect()->route('association.teams', ['association' => $team->association]);
        }
        else {
            return view('denied');
        }
    }

    public function edit(Association $association, Team $team) {
        return view('team.edit', ['team' => $team]);
    }

    public function deleteConfirm(Association $association, Team $team) {

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

        if (Bouncer::can('delete', $team)) {
            return view('team.delete', ['team' => $team]);
        }
        else {
            return view('denied');
        }
    }

    public function delete(Association $association, Team $team) {
        if (Bouncer::can('delete', $team)) {
            $team->delete();

            return redirect()->route('association.teams', ['association' => $association])->with('success', 'Team deleted successfully.');
        }
        else {
            return view('denied');
        }
    }

}