UserUpdateCrudRequest::rules() - Code Metrics - Inspection of "Allow for use of custom guard on request." - Laravel-Backpack/PermissionManager - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#160)

unknown

created 2018-06-20 01:43 UTC

UserUpdateCrudRequest::rules() A

↳ Parent: UserUpdateCrudRequest

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	8
rs	9.4285
cc	1
eloc	5
nc	1
nop	0

<?php

namespace Backpack\PermissionManager\app\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class UserUpdateCrudRequest extends FormRequest
{
	/**
	 * Determine if the user is authorized to make this request.
	 *
	 * @return bool
	 */
	public function authorize()
	{
		// only allow updates if the user is logged in
		return backpack_auth()->check();
	}

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array

     */
    public function rules()
    {
        return [
            'email'    => 'required',
            'name'     => 'required',
            'password' => 'confirmed',
        ];
    }
}


1			<?php
2
3			namespace Backpack\PermissionManager\app\Http\Requests;
4
5			use Illuminate\Foundation\Http\FormRequest;
6
7			class UserUpdateCrudRequest extends FormRequest
8			{
9			/**
10			* Determine if the user is authorized to make this request.
11			*
12			* @return bool
13			*/
14			public function authorize()
15			{
16			// only allow updates if the user is logged in
17			return backpack_auth()->check();
18			}
19
20			/**
21			* Get the validation rules that apply to the request.
22			*
23			* @return array
			0 ignored issues – show Documentation introduced 2016-05-18 14:25 UTC by Report Bug Copy Issue Report Consider making the return type a bit more specific; maybe use `array<string,string>`. This check looks for the generic type `array` as a return type and suggests a more specific type. This type is inferred from the actual code. Loading history...
24			*/
25			public function rules()
26			{
27			return [
28			'email' => 'required',
29			'name' => 'required',
30			'password' => 'confirmed',
31			];
32			}
33			}
34

Laravel-Backpack / PermissionManager

Scrutinizer GitHub App not installed

Pull Request — master (#160)

UserUpdateCrudRequest::rules() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like