Acacha\Events\Http\Controllers\EventsController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	5	1
A	create()	0	4	1
A	store()	0	9	1
A	show()	0	4	1
A	edit()	0	4	1
A	update()	0	7	1
A	destroy()	0	6	1

<?php

namespace Acacha\Events\Http\Controllers;

use Acacha\Events\Models\Event;
use Illuminate\Http\Request;
use Redirect;
use Session;

/**
 * Class EventsController.
 *
 * @package Acacha\Events\Http\Controllers
 */
class EventsController extends Controller
{

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        $events = Event::all();
        return view('events::list_events',compact('events'));
    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        return view('events::create_event');
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        Event::create($request->only(['name','user_id','description']));

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

        Session::flash('status', 'Created ok!');

        return Redirect::to('/events_php/create');

    }

    /**
     * Display the specified resource.
     *
     * @param Event $event
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function show(Event $event)
    {
        return view('events::show_event',compact('event'));
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param Event $event
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function edit(Event $event)
    {
        return view('events::edit_event',['event' => $event]);
    }

    /**
     * Update the specified resource in storage.
     *
     * @param Request $request
     * @param Event $event
     * @return \Illuminate\Http\RedirectResponse
     */
    public function update(Request $request, Event $event)
    {
        $event->update($request->only(['name','description']));

        Session::flash('status', 'Edited ok!');
        return Redirect::to('/events_php/edit/' . $event->id);

The property id does not exist on object<Acacha\Events\Models\Event>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * Remove the specified resource from storage.
     *
     * @param Event $event
     * @return \Illuminate\Http\RedirectResponse
     */
    public function destroy(Event $event)
    {
        $event->delete();
        Session::flash('status', 'Event was deleted successful!');
        return Redirect::to('/events_php');
    }
}