App\Console\Commands\ShowTaskCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	19	4

<?php

namespace App\Console\Commands;

use App\Console\Commands\Traits\AsksForTasks;
use App\Task;
use App\User;
use Illuminate\Console\Command;
use Mockery\Exception;

/**
 * Class ShowTaskCommand.
 */
class ShowTaskCommand extends Command
{
    use AsksForTasks;

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'task:show {id? : The task id to show}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Show a task';

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $id = $this->argument('id') ? $this->argument('id') : $this->askForTasks();
        $task = Task::findOrFail($id);
        $user = User::findOrFail($task->user_id);

        try {
            $headers = ['Key', 'Value'];

            $fields = [
              ['Name:', $task->name],
                ['Completed:', $task->completed ? 'Yes' : 'No'],
              ['User id:', $task->user_id],
              ['User name:', $user->name],

The property name does not exist on App\User. Since you implemented __get, consider adding a @property annotation.

            ];

            $this->table($headers, $fields);
        } catch (Exception $e) {
            $this->error('Error');
        }
    }
}