App\Console\Commands\ListTasksCommand::handle()

Complexity

Conditions	4
Paths	7

Size

Total Lines	20
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle()
    {
        try {
            $tasks = Task::all();

            $headers = ['id', 'Name', 'Completed', 'User id', 'User Name'];
            $fields = [];
            foreach ($tasks as $task) {
                $fields[] = [
                    'id:'               => $task->id,
                    'Name:'             => $task->name,
                    'Completed:'        => $task->completed ? 'Yes' : 'No',
                    'User id:'          => $task->user_id,
                    'User name:'        => User::findOrFail($task->user_id)->name,

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

                ];
            }
        } catch (Exception $e) {
            $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.

    }