AdminAttributesFormRequest::authorize() - Code Metrics - Inspection of "Sanitize input values before submission" - rinvex/cortex-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( fdfb11...a4a128 )

by Abdelrahman

created 2018-04-01 00:08 UTC

AdminAttributesFormRequest::authorize() A

↳ Parent: AdminAttributesFormRequest

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0

<?php

declare(strict_types=1);

namespace Cortex\Auth\Http\Requests\Adminarea;

use Rinvex\Support\Traits\Escaper;
use Illuminate\Foundation\Http\FormRequest;

class AdminAttributesFormRequest extends FormRequest
{
    use Escaper;

    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize(): bool
    {
        return true;
    }

    /**
     * Configure the validator instance.
     *
     * @param \Illuminate\Validation\Validator $validator
     *
     * @return void
     */
    public function withValidator($validator): void
    {
        // Sanitize input data before submission
        $this->replace($this->escape($this->all()));
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules(): array
    {
        $admin = $this->route('admin') ?? app('cortex.auth.admin');

        // Attach attribute rules
        $admin->getEntityAttributes()->each(function ($attribute, $attributeName) use (&$rules) {
            switch ($attribute->type) {
                case 'datetime':
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
                    $type = 'date';
                    break;
                case 'text':
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
                case 'varchar':
                    $type = 'string';
                    break;
                default:
                    $type = $attribute->type;
                    break;
            }

            $rule = ($attribute->is_required ? 'required|' : 'nullable|').$type;
            $rules[$attributeName.($attribute->is_collection ? '.*' : '')] = $rule;
        });

        return $rules ?? [];
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Cortex\Auth\Http\Requests\Adminarea;
6
7			use Rinvex\Support\Traits\Escaper;
8			use Illuminate\Foundation\Http\FormRequest;
9
10			class AdminAttributesFormRequest extends FormRequest
11			{
12			use Escaper;
13
14			/**
15			* Determine if the user is authorized to make this request.
16			*
17			* @return bool
18			*/
19			public function authorize(): bool
20			{
21			return true;
22			}
23
24			/**
25			* Configure the validator instance.
26			*
27			* @param \Illuminate\Validation\Validator $validator
28			*
29			* @return void
30			*/
31			public function withValidator($validator): void
32			{
33			// Sanitize input data before submission
34			$this->replace($this->escape($this->all()));
35			}
36
37			/**
38			* Get the validation rules that apply to the request.
39			*
40			* @return array
41			*/
42			public function rules(): array
43			{
44			$admin = $this->route('admin') ?? app('cortex.auth.admin');
45
46			// Attach attribute rules
47			$admin->getEntityAttributes()->each(function ($attribute, $attributeName) use (&$rules) {
48			switch ($attribute->type) {
49			case 'datetime':
			0 ignored issues – show Coding Style introduced 2018-01-04 01:06 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
50			$type = 'date';
51			break;
52			case 'text':
			0 ignored issues – show Coding Style introduced 2018-01-04 01:06 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
53			case 'varchar':
54			$type = 'string';
55			break;
56			default:
57			$type = $attribute->type;
58			break;
59			}
60
61			$rule = ($attribute->is_required ? 'required\|' : 'nullable\|').$type;
62			$rules[$attributeName.($attribute->is_collection ? '.*' : '')] = $rule;
63			});
64
65			return $rules ?? [];
66			}
67			}
68

rinvex / cortex-auth

Push — develop ( fdfb11...a4a128 )

AdminAttributesFormRequest::authorize() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like