EditUserRequest::rules() - Code Metrics - Inspection of "Updated Modify User Pages" - butcherman/Tech_Bench - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dev5a ( 352680...01b7fe )

by Ron

created 2020-05-28 23:49 UTC

EditUserRequest::rules() A

↳ Parent: EditUserRequest

Complexity

Conditions	1
Paths	1

Size

Total Lines	16
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	13
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	16
ccs	11
cts	11
cp	1
crap	1
rs	9.8333

<?php

namespace App\Http\Requests\Admin;

use App\User;
use Illuminate\Validation\Rule;
use Illuminate\Foundation\Http\FormRequest;

class EditUserRequest extends FormRequest
{
    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        return $this->user()->can('hasAccess', 'Manage Users');
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'user_id'    => 'required|exists:users',
            'role_id'    => 'required|numeric|exists:user_role_types,role_id',
            'username'   => [
                'required',
                'regex:/^[a-zA-Z0-9_]*$/',
                Rule::unique('users')->ignore(User::findOrFail($this->request->get('user_id'))),
            ],
            'first_name' => 'required|string',
            'last_name'  => 'required|string',
            'email'      => [
                'required',
                'email',
                Rule::unique('users')->ignore(User::findOrFail($this->request->get('user_id'))),
            ],
        ];
    }
}


1		<?php
2
3		namespace App\Http\Requests\Admin;
4
5		use App\User;
6		use Illuminate\Validation\Rule;
7		use Illuminate\Foundation\Http\FormRequest;
8
9		class EditUserRequest extends FormRequest
10		{
11		/**
12		* Determine if the user is authorized to make this request.
13		*
14		* @return bool
15		*/
16	4	public function authorize()
17		{
18	4	return $this->user()->can('hasAccess', 'Manage Users');
19		}
20
21		/**
22		* Get the validation rules that apply to the request.
23		*
24		* @return array
25		*/
26	4	public function rules()
27		{
28		return [
29	4	'user_id' => 'required\|exists:users',
30	4	'role_id' => 'required\|numeric\|exists:user_role_types,role_id',
31		'username' => [
32	4	'required',
33	4	'regex:/^[a-zA-Z0-9_]*$/',
34	4	Rule::unique('users')->ignore(User::findOrFail($this->request->get('user_id'))),
35		],
36	4	'first_name' => 'required\|string',
37	4	'last_name' => 'required\|string',
38		'email' => [
39	4	'required',
40	4	'email',
41	4	Rule::unique('users')->ignore(User::findOrFail($this->request->get('user_id'))),
42		],
43		];
44		}
45		}
46

butcherman / Tech_Bench

Push — dev5a ( 352680...01b7fe )

EditUserRequest::rules() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like