Acacha\Events\Http\Controllers\APIEventsController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	4	1
A	show()	0	4	1
A	store()	0	4	1
A	update()	0	5	1
A	destroy()	0	5	1

<?php

namespace Acacha\Events\Http\Controllers;

use Acacha\Events\Http\Requests\DestroyEvent;
use Acacha\Events\Http\Requests\ListEvents;
use Acacha\Events\Http\Requests\ShowEvent;
use Acacha\Events\Http\Requests\StoreEvent;
use Acacha\Events\Http\Requests\UpdateEvent;
use Acacha\Events\Models\Event;

/**
 * Class APIEventsController.
 * 
 * @package App\Http\Controllers
 */
class APIEventsController extends Controller
{
    /**
     * Show list of events.
     *
     * @param ListEvents $request
     * @return \Illuminate\Database\Eloquent\Collection|static[]
     */
    public function index(ListEvents $request)

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

    {
        return Event::all();
    }

    /**
     * Show an event.
     *
     * @param ShowEvent $request
     * @param Event $event
     * @return Event
     */
    public function show(ShowEvent $request, Event $event)

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

    {
        return $event;
    }

    /**
     * Store an event.
     *
     * @param StoreEvent $request
     * @return mixed
     */
    public function store(StoreEvent $request)
    {
        return Event::create($request->only(['name','user_id','description']));

The method create() does not exist on Acacha\Events\Models\Event. Did you maybe mean created()?

    }

    /**
     * Update an event.
     *
     * @param UpdateEvent $request
     * @param Event $event
     * @return Event
     */
    public function update(UpdateEvent $request , Event $event)
    {
        $event->update($request->only(['name','description']));
        return $event;
    }

    /**
     * Destroy an event.
     *
     * @param DestroyEvent $request
     * @param Event $event
     * @return Event
     */
    public function destroy(DestroyEvent $request, Event $event)

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

    {
        $event->delete();
        return $event;
    }

}