UserController::index() - Code Metrics - Inspection of "wip - api" - devpri/tinre-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 72576e...ceb7cb )

by Paul

created 2020-09-04 13:50 UTC

UserController::index() A

↳ Parent: UserController

Complexity

Conditions	3
Paths	4

Size

Total Lines	22
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3.1406

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	11
c	1
b	0
f	0
nc	4
nop	1
dl	0
loc	22
ccs	9
cts	12
cp	0.75
crap	3.1406
rs	9.9

<?php

namespace Devpri\Tinre\Http\Controllers\Api\V1;

use Devpri\Tinre\Http\Controllers\Controller;
use Devpri\Tinre\Http\Resources\Api\V1\User as UserResource;
use Devpri\Tinre\Models\User;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Http\Request;

class UserController extends Controller
{
    public function index(Request $request)
    {
        $user = $request->user();

        if ($user->cant('viewAny', User::class)) {
            abort(401);
        }

        $search = $request->search;

        $query = User::query();

        if ($search) {
            $query->where(function (Builder $query) use ($search) {
                $query->where('name', 'LIKE', "%{$search}%")
                    ->orWhere('email', 'LIKE', "%{$search}%");
            });
        }

        $users = $query->orderBy('created_at', 'DESC')->paginate(30);

        return UserResource::collection($users)->additional(['authorized_actions' => (new User)->authorizedActions()]);
    }

    public function show(Request $request, $id)
    {
        $authUser = $request->user();

        $user = User::where('id', $id)->firstOrFail();

        if ($authUser->cant('view', $user)) {
            abort(401);
        }

        return new UserResource($user);
    }
}


1		<?php
2
3		namespace Devpri\Tinre\Http\Controllers\Api\V1;
4
5		use Devpri\Tinre\Http\Controllers\Controller;
6		use Devpri\Tinre\Http\Resources\Api\V1\User as UserResource;
7		use Devpri\Tinre\Models\User;
8		use Illuminate\Database\Eloquent\Builder;
9		use Illuminate\Http\Request;
10
11		class UserController extends Controller
12		{
13	3	public function index(Request $request)
14		{
15	3	$user = $request->user();
16
17	3	if ($user->cant('viewAny', User::class)) {
18	2	abort(401);
19		}
20
21	1	$search = $request->search;
22
23	1	$query = User::query();
24
25	1	if ($search) {
26		$query->where(function (Builder $query) use ($search) {
27		$query->where('name', 'LIKE', "%{$search}%")
28		->orWhere('email', 'LIKE', "%{$search}%");
29		});
30		}
31
32	1	$users = $query->orderBy('created_at', 'DESC')->paginate(30);
33
34	1	return UserResource::collection($users)->additional(['authorized_actions' => (new User)->authorizedActions()]);
35		}
36
37	3	public function show(Request $request, $id)
38		{
39	3	$authUser = $request->user();
40
41	3	$user = User::where('id', $id)->firstOrFail();
42
43	3	if ($authUser->cant('view', $user)) {
44		abort(401);
45		}
46
47	3	return new UserResource($user);
48		}
49		}
50

devpri / tinre-core

Push — master ( 72576e...ceb7cb )

UserController::index() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like