InvitesController::create() - Code Metrics - doesangueorg/doesangue-core - Measure and Improve Code Quality continuously with Scrutinizer

InvitesController::create() A
last analyzed 2019-03-08 14:47 UTC

↳ Parent: InvitesController

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	17
ccs	0
cts	9
cp	0
rs	9.7
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	2

<?php

namespace DoeSangue\Http\Controllers\API\V1;

use Illuminate\Http\Request;
use DoeSangue\Http\Controllers\Controller;
use Tymon\JWTAuth\Facades\JWTAuth;
use DoeSangue\Repositories\Invites;
use DoeSangue\Models\Invite;
use Hash;

class InvitesController extends Controller
{
    public function __construct()
    {
        $this->middleware('jwt.auth', [ 'except' => [ 'invite' ] ]);
    }

    public function create(Request $request)
    {
        // Determines the user/owner of invite.
        $user = JWTAuth::parseToken()->authenticate();

        $invite = new Invite();
        $invite->invite_code = Hash::make($request[ 'invite_code' ]);
        $invite->user_id = $user->id;
        $invite->save();

        return response()->json(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            [
            'status_code' => 201,
            'message' => 'Invite created!'
            ], 201
        );
    }

    public function show($id)
    {
        $invite = Invite::find($id);

        return response()->json($invite, 200);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }

}


1			<?php
2
3			namespace DoeSangue\Http\Controllers\API\V1;
4
5			use Illuminate\Http\Request;
6			use DoeSangue\Http\Controllers\Controller;
7			use Tymon\JWTAuth\Facades\JWTAuth;
8			use DoeSangue\Repositories\Invites;
9			use DoeSangue\Models\Invite;
10			use Hash;
11
12			class InvitesController extends Controller
13			{
14			public function __construct()
15			{
16			$this->middleware('jwt.auth', [ 'except' => [ 'invite' ] ]);
17			}
18
19			public function create(Request $request)
20			{
21			// Determines the user/owner of invite.
22			$user = JWTAuth::parseToken()->authenticate();
23
24			$invite = new Invite();
25			$invite->invite_code = Hash::make($request[ 'invite_code' ]);
26			$invite->user_id = $user->id;
27			$invite->save();
28
29			return response()->json(
			0 ignored issues – show Bug introduced 2019-03-03 06:57 UTC by Report Bug Copy Issue Report The method `json` does only exist in `Illuminate\Contracts\Routing\ResponseFactory`, but not in `Illuminate\Http\Response`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
30			[
31			'status_code' => 201,
32			'message' => 'Invite created!'
33			], 201
34			);
35			}
36
37			public function show($id)
38			{
39			$invite = Invite::find($id);
40
41			return response()->json($invite, 200);
			0 ignored issues – show Bug introduced 2019-03-03 06:57 UTC by Report Bug Copy Issue Report The method `json` does only exist in `Illuminate\Contracts\Routing\ResponseFactory`, but not in `Illuminate\Http\Response`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
42			}
43
44			}
45

doesangueorg / doesangue-core

Scrutinizer GitHub App not installed

InvitesController::create() A last analyzed 2019-03-08 14:47 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

InvitesController::create() A
last analyzed 2019-03-08 14:47 UTC