Xetaravel\Http\Controllers\Blog\ArticleController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

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
A	show()	0	19	2

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

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

class ArticleController extends Controller
{
    /**
     * Constructor.
     */
    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.

    {
        $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'));
        $comments->load('user');

        $breadcrumbs = $this->breadcrumbs->addCrumb("Article : " . e($article->title), $article->article_url);

        return view('Blog::article.show', compact('article', 'comments', 'breadcrumbs'));
    }
}