Ergare17\Articles\Console\Commands\ListArticlesCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	handle()	0	22	4

<?php

namespace Ergare17\Articles\Console\Commands;

use App\User;
use Ergare17\Articles\Models\Article;
use http\Exception;
use Illuminate\Console\Command;

class ListArticlesCommand extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'article:list';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'List articles';

    /**
     * Create a new command instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        try {
            $articles = Article::all();

            $headers = ['id', 'Title','Description', 'Read', 'User id', 'User Name'];
            $fields = [];
            foreach ($articles as $article) {
                $fields[] = [
                    'id:'               => $article->id,
                    'Title:'             => $article->title,
                    'Description:' => $article->description,
                    'Read:'        => $article->read ? 'Yes' : 'No',
                    'User id:'          => $article->user_id,
                    'User name:'        => User::findOrFail($article->user_id)->name,
                ];
            }
        } catch (Exception $e) {

The class http\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $this->error('Error');
        }
        $this->table($headers, $fields);

The variable $headers does not seem to be defined for all execution paths leading up to this point.

The variable $fields does not seem to be defined for all execution paths leading up to this point.

    }
}