App\Http\Controllers\GistsController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	17	1
B	backup()	0	22	4

<?php

namespace App\Http\Controllers;

use App\Helpers\GistBackupHandler;
use App\Helpers\GistFinder;
use Carbon\Carbon;

class GistsController extends Controller
{
    /**
     * Show gists list.
     *
     * @param GistFinder $gistFinder
     *
     * @return \Illuminate\View\View
     */
    public function index(GistFinder $gistFinder)
    {
        $gistsAndTags = $gistFinder->getGistsAndTags();

        $authUser = auth()->user();

The method user does only exist in Illuminate\Contracts\Auth\Guard, but not in Illuminate\Contracts\Auth\Factory.

        $state = collect([
            'user'  => [
                'name'     => $authUser->name,
                'nickname' => $authUser->nickname,
                'avatar'   => $authUser->avatar
            ],
            'gists' => $gistsAndTags['gists'],
            'tags'  => $gistsAndTags['tags']
        ]);

        return view('gists.index', compact('state'));

The expression view('gists.index', compact('state')); of type Illuminate\View\View|Illuminate\Contracts\View\Factory adds the type Illuminate\Contracts\View\Factory to the return on line 33 which is incompatible with the return type documented by App\Http\Controllers\GistsController::index of type Illuminate\View\View.

    }

    /**
     * Backup authorized user gists.
     *
     * @param GistBackupHandler $backupHandler
     *
     * @return \Symfony\Component\HttpFoundation\BinaryFileResponse
     */
    public function backup(GistBackupHandler $backupHandler)
    {
        if (auth()->user()->gists_backup_at) {
            $lastBackup = Carbon::createFromFormat('Y-m-d H:i:s', auth()->user()->gists_backup_at);

The method user does only exist in Illuminate\Contracts\Auth\Guard, but not in Illuminate\Contracts\Auth\Factory.

            if ($lastBackup->gt(Carbon::now()->subMinutes(5))) {
                return response('WAIT_A_MINUTE', 429);
            };
        }

        $zipPath = $backupHandler->backup();

        if ($zipPath === 0) {
            return response('NO_FILES', 404);
        }

        auth()->user()->update(['gists_backup_at' => date('Y-m-d H:i:s')]);

        return response()->download($zipPath, null, [
            'Set-Cookie'   => 'fileDownload=true; path=/',
            'Content-Type' => 'application/zip'
        ])->deleteFileAfterSend(true);
    }
}