App\Http\Controllers\Gedcom\Progress::index()

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function index(Request $request)

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

    {
        $user_id = Auth::user()->id;

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $runningjob = ImportJob::orderby('id', 'DESC')->first();
        $slug = null;
        if ($runningjob !== null) {
            $slug = $runningjob->slug;

The property slug does not seem to exist on Illuminate\Database\Eloquent\Relations\HasManyThrough.

The property slug does not seem to exist on Illuminate\Database\Eloquent\Relations\HasOneThrough.

        }
        $ret = [];
        $ret['slug'] = $slug;
        $ret['user'] = $user_id;

        return $ret;
    }