ShareController::index() - Code Metrics - Inspection of "Update[ To laravel 7 ]" - niclasleonbock/hacktoberfest-status - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — develop (#51)

unknown

created 2020-10-29 01:42 UTC

ShareController::index() A

↳ Parent: ShareController

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
eloc	7
c	2
b	0
f	0
dl	0
loc	14
rs	10
cc	2
nc	2
nop	1

<?php

namespace App\Http\Controllers;

use App\Hacktoberfest\GitHub\PullRequestChecker;
use App\User;

class ShareController extends Controller
{
    /**
     * Instance of the 'pull request checker'.
     *
     * @var PullRequestChecker
     */
    protected $checker;

    /**
     * Instance of user model.
     *
     * @var User
     */
    protected $user;


    /**
     * Create a new controller instance.
     *
     * @param PullRequestChecker $checker
     */
    public function __construct(PullRequestChecker $checker, User $user)
    {
        $this->checker     = $checker;
        $this->user        = $user;
        $this->sharingMode = true;

        parent::__construct();
    }

    /**
     * Display a user's PR status if user already authorized this app,
     * otherwise redirect to home page.
     *
     * @param string $github_username
     *
     * @return \Illuminate\Http\RedirectResponse|\Illuminate\View\View
     */
    public function index($github_username)
    {
        $user = $this->user->where('github_username', '=', $github_username)->first();

        if (!$user) {
            return redirect('/');
        }

        $prs = $this->checker->getQualifiedPullRequests($user);

        $viewData = compact('user', 'prs');
        $viewData['sharingMode'] = $this->sharingMode;

        return view('status', $viewData);
    }
}


1			<?php
2
3			namespace App\Http\Controllers;
4
5			use App\Hacktoberfest\GitHub\PullRequestChecker;
6			use App\User;
7
8			class ShareController extends Controller
9			{
10			/**
11			* Instance of the 'pull request checker'.
12			*
13			* @var PullRequestChecker
14			*/
15			protected $checker;
16
17			/**
18			* Instance of user model.
19			*
20			* @var User
21			*/
22			protected $user;
23
24
25			/**
26			* Create a new controller instance.
27			*
28			* @param PullRequestChecker $checker
29			*/
30			public function __construct(PullRequestChecker $checker, User $user)
31			{
32			$this->checker = $checker;
33			$this->user = $user;
34			$this->sharingMode = true;
35
36			parent::__construct();
37			}
38
39			/**
40			* Display a user's PR status if user already authorized this app,
41			* otherwise redirect to home page.
42			*
43			* @param string $github_username
44			*
45			* @return \Illuminate\Http\RedirectResponse\|\Illuminate\View\View
46			*/
47			public function index($github_username)
48			{
49			$user = $this->user->where('github_username', '=', $github_username)->first();
50
51			if (!$user) {
52			return redirect('/');
53			}
54
55			$prs = $this->checker->getQualifiedPullRequests($user);
56
57			$viewData = compact('user', 'prs');
58			$viewData['sharingMode'] = $this->sharingMode;
59
60			return view('status', $viewData);
61			}
62			}
63

niclasleonbock / hacktoberfest-status

Pull Request — develop (#51)

ShareController::index() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like