DavideCasiraghi\LaravelSmartBlog\Http\Controllers\BlogController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	160
Duplicated Lines	0 %

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	12	1
A	create()	0	6	1
A	destroy()	0	6	1
A	show()	0	3	1
A	edit()	0	6	1
A	saveOnDb()	0	28	1
A	store()	0	17	2
A	__construct()	0	3	1
A	update()	0	11	1

<?php

namespace DavideCasiraghi\LaravelSmartBlog\Http\Controllers;

use Validator;
use Illuminate\Support\Str;
use Illuminate\Http\Request;
use DavideCasiraghi\LaravelSmartBlog\Models\Blog;
use DavideCasiraghi\LaravelSmartBlog\Models\Category;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization;

class BlogController extends Controller
{
    /* Restrict the access to this resource just to logged in users */
    public function __construct()
    {
        $this->middleware('admin');
    }
    
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        $categories = Category::get();
        $blogs = Blog::latest()->paginate(10);

        // Countries available for translations
        $countriesAvailableForTranslations = LaravelLocalization::getSupportedLocales();
    
        return view('laravel-smart-blog::blogs.index', compact('blogs'))
            ->with('i', (request()->input('page', 1) - 1) * 10)
            ->with('categories', $categories)
            ->with('countriesAvailableForTranslations', $countriesAvailableForTranslations);
    }
    
    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        $categories = Category::get();
        
        return view('laravel-smart-blog::blogs.create')
            ->with('categories', $categories);
    }
    
    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        // Validate form datas
        $validator = Validator::make($request->all(), [
                'category_id' => 'required',
                'layout' => 'required',
            ]);
        if ($validator->fails()) {
            return back()->withErrors($validator)->withInput();
        }

        $blog = new Blog();

        $this->saveOnDb($request, $blog);

        return redirect()->route('blogs.index')
                        ->with('success', __('messages.blog_added_successfully'));
    }

    /**
     * Display the specified resource.
     *
     * @param  \DavideCasiraghi\LaravelSmartBlog\Models\Blog  $blog
     * @return \Illuminate\Http\Response
     */
    public function show(Blog $blog)
    {
        return view('laravel-smart-blog::blogs.show', compact('blog'));
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  \App\Blog  $blog

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

     * @return \Illuminate\Http\Response
     */
    public function edit(Blog $blog)
    {
        $categories = Category::get();
        
        return view('laravel-smart-blog::blogs.edit', compact('blog'))
                    ->with('categories', $categories);
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \DavideCasiraghi\LaravelSmartBlog\Models\Blog  $blog
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, Blog $blog)
    {
        request()->validate([
            'category_id' => 'required',
            'layout' => 'required',
        ]);

        $this->saveOnDb($request, $blog);

        return redirect()->route('blogs.index')
                        ->with('success', __('messages.blog_updated_successfully'));
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  \DavideCasiraghi\LaravelSmartBlog\Models\Blog  $blog
     * @return \Illuminate\Http\Response
     */
    public function destroy(Blog $blog)
    {
        $blog->delete();

        return redirect()->route('blogs.index')
                        ->with('success', __('messages.blog_deleted_successfully'));
    }

    /***************************************************************************/

    /**
     * Save/Update the record on DB.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return void
     */
    public function saveOnDb($request, $blog)
    {
        $blog->category_id = $request->get('category_id');
        $blog->layout = $request->get('layout');
        $blog->columns = $request->get('columns');
        $blog->article_order = $request->get('article_order');
        
        $blog->pagination = $request->get('pagination');
        $blog->featured_articles = $request->get('featured_articles');
        $blog->show_category_title = $request->get('show_category_title');
        $blog->show_category_subtitle = $request->get('show_category_subtitle');
        $blog->show_category_description = $request->get('show_category_description');
        $blog->show_category_image = $request->get('show_category_image');
        
        $blog->show_post_title = $request->get('show_post_title');
        $blog->post_linked_titles = $request->get('post_linked_titles');
        $blog->show_post_intro_text = $request->get('show_post_intro_text');
        $blog->show_post_author = $request->get('show_post_author');
        $blog->link_post_author = $request->get('link_post_author');
        
        $blog->show_create_date = $request->get('show_create_date');
        $blog->show_modify_date = $request->get('show_modify_date');
        $blog->show_publish_date = $request->get('show_publish_date');
        $blog->show_read_more = $request->get('show_read_more');
        
        $blog->created_by = \Auth::user()->id;

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $blog->save();
    }
}