App\Http\Controllers\Equipment\EquipmentCategoryController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	store()	0	8	1
A	edit()	0	7	1
A	update()	0	9	1
A	create()	0	5	1
A	destroy()	0	10	1

<?php

namespace App\Http\Controllers\Equipment;

use App\Events\Equipment\EquipmentCategoryCreatedEvent;
use App\Events\Equipment\EquipmentCategoryDeletedEvent;
use App\Events\Equipment\EquipmentCategoryUpdatedEvent;
use App\Http\Controllers\Controller;
use App\Http\Requests\Equipment\EquipmentCategoryRequest;
use App\Models\EquipmentCategory;
use Inertia\Inertia;

class EquipmentCategoryController extends Controller
{
    /**
     * Show the form for creating a new Equipment Category
     */
    public function create()
    {
        $this->authorize('create', EquipmentCategory::class);

        return Inertia::render('EquipmentCategory/Create');
    }

    /**
     * Store a newly created Equipment Category
     */
    public function store(EquipmentCategoryRequest $request)
    {
        $newCat = EquipmentCategory::create($request->only('name'));

        event(new EquipmentCategoryCreatedEvent($newCat));
        return redirect(route('equipment.index'))->with([
            'message' => 'New Category Created',
            'type'    => 'success',
        ]);
    }

    /**
     * Show the form for editing the Category Name
     */
    public function edit($id)
    {
        $cat = EquipmentCategory::findOrFail($id);
        $this->authorize('update', $cat);

        return Inertia::render('EquipmentCategory/Edit', [
            'category' => $cat,
        ]);
    }

    /**
     * Update the Equipment Category Name
     */
    public function update(EquipmentCategoryRequest $request, $id)
    {
        $cat = EquipmentCategory::find($id);
        $cat->update($request->only('name'));

        event(new EquipmentCategoryUpdatedEvent($cat));

It seems like $cat can also be of type null; however, parameter $category of App\Events\Equipment\EquipmentCategoryUpdatedEvent::__construct() does only seem to accept App\Models\EquipmentCategory, maybe add an additional type check?

        return redirect(route('equipment.index'))->with([
            'message' => 'Category Updated',
            'type'    => 'success',
        ]);
    }

    /**
     * Remove the Equipment Category
     */
    public function destroy($id)
    {
        $cat = EquipmentCategory::find($id);
        $this->authorize('delete', $cat);
        $cat->delete();

        event(new EquipmentCategoryDeletedEvent($cat));

It seems like $cat can also be of type null; however, parameter $category of App\Events\Equipment\EquipmentCategoryDeletedEvent::__construct() does only seem to accept App\Models\EquipmentCategory, maybe add an additional type check?

        return back()->with([
            'message' => 'Equipment Category Deleted',
            'type'    => 'danger',
        ]);
    }
}