App\Http\Controllers\NewsController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	126
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

7 Methods

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

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use App\Http\Requests;

use Session;

use App\News;

class NewsController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        //$news = News::all();

50% of this comment could be valid code. Did you maybe forget this after debugging?

        // using paginate function to show 3 news items per page
        $itemsPerPage = 3;
        $news = News::orderBy('created_at', 'desc')->paginate($itemsPerPage);

        return view('news.index', array('news' => $news, 'title' => 'News Display'));
    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        return view('news.create', array('title' => 'Add News'));
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        $this->validate($request, array(
                                'title' => 'required',
                                'slug' => 'required',
                                'short_description' => 'required',
                                'full_content' => 'required',
                            )
                        );

        $input = $request->all();
        //dd($input); // dd() helper function is print_r alternative

67% of this comment could be valid code. Did you maybe forget this after debugging?

        News::create($input);

        Session::flash('flash_message', 'News added successfully!');

        //return redirect()->back();

70% of this comment could be valid code. Did you maybe forget this after debugging?

        //return redirect('news');

72% of this comment could be valid code. Did you maybe forget this after debugging?

        return redirect()->route('news.index');
    }

    /**
     * Display the specified resource.
     *
     * @param  string  $slug
     * @return \Illuminate\Http\Response
     */
    public function show($slug)
    {
        $news = News::where('slug', $slug)->first();
        return view('news.show', array('news' => $news));
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function edit($id)
    {
        $news = News::findOrFail($id);

        return view('news.edit', array('news' => $news, 'title' => 'Edit News'));
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)
    {
        $news = News::findOrFail($id);

        $this->validate($request, array(
                                'title' => 'required',
                                'slug' => 'required',
                                'short_description' => 'required',
                                'full_content' => 'required',
                            )
                        );

        $input = $request->all();

        $news->fill($input)->save();

        Session::flash('flash_message', 'News updated successfully!');

        return redirect()->back();
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    {
        $news = News::findOrFail($id);

        $news->delete();

        Session::flash('flash_message', 'News deleted successfully!');

        return redirect()->route('news.index');
    }
}