Xetaravel\Http\Controllers\Blog\ArticleController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	55
Duplicated Lines	49.09 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	index()	0	7	1
B	show()	27	27	2

<?php
namespace Xetaravel\Http\Controllers\Blog;

use Xetaravel\Http\Controllers\Controller;
use Xetaravel\Models\Article;
use Illuminate\Http\Request;

class ArticleController extends Controller
{
    public function __construct()
    {
        parent::__construct();

        $this->breadcrumbs->addCrumb('Blog', route('blog_article_index'));

The property breadcrumbs does not exist. Did you maybe forget to declare it?

    }

    /**
     * Show the list of all articles.
     *
     * @return \Illuminate\Http\Response

Should the return type not be \Illuminate\View\View|\Illuminate\Contracts\View\Factory?

     */
    public function index()
    {
        $articles = Article::with('category', 'user')

The method paginate does only exist in Illuminate\Database\Eloquent\Builder, but not in Illuminate\Database\Eloquent\Model.

            ->paginate(config('xetaravel.pagination.blog.article_per_page'));

        return view('Blog::article.index', ['articles' => $articles, 'breadcrumbs' => $this->breadcrumbs]);
    }

    /**
     * Show the article by his id.
     *
     * @return \Illuminate\Http\Response

Should the return type not be \Illuminate\Http\RedirectResponse|\Illuminate\View\View|\Illuminate\Contracts\View\Factory?

     */
    public function show(Request $request, $slug, $id)

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

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

This method seems to be duplicated in your project.

    {
        $article = Article::with('category', 'user', 'comments')

The method where does only exist in Illuminate\Database\Eloquent\Builder, but not in Illuminate\Database\Eloquent\Model.

            ->where('id', $id)
            ->first();

        if (is_null($article)) {
            return redirect()
                ->route('blog_article_index')
                ->with('danger', 'This article doesn\'t exist or has been deleted !');
        }

        $comments = $article->comments()->paginate(config('xetaravel.pagination.blog.comment_per_page'));

        $this->breadcrumbs->addCrumb(
            "Article : " . e($article->title),
            route(
                'blog_article_show',
                ['slug' => $article->category->slug, 'id' => $article->category->id]
            )
        );

        return view(
            'Blog::article.show',
            ['article' => $article, 'comments' => $comments, 'breadcrumbs' => $this->breadcrumbs]
        );
    }
}