RoleUpdateCrudRequest::authorize() - Code Metrics - Inspection of "Merge branch 'MIR24-fix-role-validation'" - Laravel-Backpack/PermissionManager - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2f3f3f...772594 )

by Cristian

created 2019-03-12 09:15 UTC

RoleUpdateCrudRequest::authorize() A

↳ Parent: RoleUpdateCrudRequest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	5
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

namespace Backpack\PermissionManager\app\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class RoleUpdateCrudRequest 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()
    {
        $rules = [
            'name' => 'required|string|max:255',
        ];

        return $rules;
    }
}


1			<?php
2
3			namespace Backpack\PermissionManager\app\Http\Requests;
4
5			use Illuminate\Foundation\Http\FormRequest;
6
7			class RoleUpdateCrudRequest 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 2019-03-12 09:17 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			$rules = [
28			'name' => 'required\|string\|max:255',
29			];
30
31			return $rules;
32			}
33			}
34

Laravel-Backpack / PermissionManager

Scrutinizer GitHub App not installed

Push — master ( 2f3f3f...772594 )

RoleUpdateCrudRequest::authorize() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like