UserPolicy::delete() - Code Metrics - Inspection of "tests - wip" - devpri/tinre-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 9f5635...6b9ecc )

by Paul

created 2020-09-05 08:45 UTC

UserPolicy::delete() A

↳ Parent: UserPolicy

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	5
c	1
b	0
f	0
nc	3
nop	2
dl	0
loc	11
ccs	6
cts	6
cp	1
crap	3
rs	10

<?php

namespace  Devpri\Tinre\Policies;

use Devpri\Tinre\Models\User;
use Illuminate\Auth\Access\HandlesAuthorization;
use Illuminate\Support\Facades\Route;

class UserPolicy
{
    use HandlesAuthorization;

    public function viewAny(User $user): bool
    {
        if ($user->hasPermissionTo('user:view:any')) {
            return true;
        }

        return false;
    }

    public function view(User $user, User $userModel): bool
    {
        if ($user->hasPermissionTo('user:view:any')) {
            return true;
        }

        if (! $user->hasPermissionTo('user:view')) {
            return false;
        }

        if ($user->id === $userModel->id) {
            return true;
        }

        return false;
    }

    public function create(User $user): bool
    {
        if ($user->hasPermissionTo('user:create')) {
            return true;
        }

        return false;
    }

    public function update(User $user, User $userModel): bool
    {
        if ($user->id === $userModel->id) {
            return false;
        }

        if ($user->hasPermissionTo('user:update:any')) {
            return true;
        }

        return false;
    }

    public function updateOwn(User $user, User $userModel): bool
    {
        if (! $user->hasPermissionTo('user:update')) {
            return false;
        }

        if ($user->id === $userModel->id) {
            return true;
        }

        return false;
    }

    public function changeEmail(User $user): bool
    {
        if (! $user->hasPermissionTo('user:change_email')) {
            return false;
        }

        if (Route::has('email.change')) {
            return true;
        }

        return false;
    }

    public function delete(User $user, User $userModel): bool
    {
        if ($user->id === $userModel->id) {
            return false;
        }

        if ($user->hasPermissionTo('user:delete:any')) {
            return true;
        }

        return false;
    }
}


1		<?php
2
3		namespace Devpri\Tinre\Policies;
4
5		use Devpri\Tinre\Models\User;
6		use Illuminate\Auth\Access\HandlesAuthorization;
7		use Illuminate\Support\Facades\Route;
8
9		class UserPolicy
10		{
11		use HandlesAuthorization;
12
13	18	public function viewAny(User $user): bool
14		{
15	18	if ($user->hasPermissionTo('user:view:any')) {
16	8	return true;
17		}
18
19	10	return false;
20		}
21
22	19	public function view(User $user, User $userModel): bool
23		{
24	19	if ($user->hasPermissionTo('user:view:any')) {
25	10	return true;
26		}
27
28	9	if (! $user->hasPermissionTo('user:view')) {
29	1	return false;
30		}
31
32	8	if ($user->id === $userModel->id) {
33	6	return true;
34		}
35
36	2	return false;
37		}
38
39	13	public function create(User $user): bool
40		{
41	13	if ($user->hasPermissionTo('user:create')) {
42	7	return true;
43		}
44
45	6	return false;
46		}
47
48	15	public function update(User $user, User $userModel): bool
49		{
50	15	if ($user->id === $userModel->id) {
51	8	return false;
52		}
53
54	9	if ($user->hasPermissionTo('user:update:any')) {
55	6	return true;
56		}
57
58	3	return false;
59		}
60
61	13	public function updateOwn(User $user, User $userModel): bool
62		{
63	13	if (! $user->hasPermissionTo('user:update')) {
64	1	return false;
65		}
66
67	12	if ($user->id === $userModel->id) {
68	8	return true;
69		}
70
71	6	return false;
72		}
73
74	12	public function changeEmail(User $user): bool
75		{
76	12	if (! $user->hasPermissionTo('user:change_email')) {
77	1	return false;
78		}
79
80	11	if (Route::has('email.change')) {
81	11	return true;
82		}
83
84		return false;
85		}
86
87	15	public function delete(User $user, User $userModel): bool
88		{
89	15	if ($user->id === $userModel->id) {
90	7	return false;
91		}
92
93	10	if ($user->hasPermissionTo('user:delete:any')) {
94	7	return true;
95		}
96
97	3	return false;
98		}
99		}
100

devpri / tinre-core

Push — master ( 9f5635...6b9ecc )

UserPolicy::delete() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like