UpdateManagerProfileRequest::rules() - Code Metrics - Inspection of "1.0.17 (#2441)" - GCTC-NTGC/TalentCloud - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6ceee8...d4902a )

by Tristan

created 2020-01-02 20:36 UTC

UpdateManagerProfileRequest::rules() A

↳ Parent: UpdateManagerProfileRequest

Complexity

Conditions	1
Paths	1

Size

Total Lines	56
Code Lines	42

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	42
c	1
b	0
f	0
dl	0
loc	56
rs	9.248
cc	1
nc	1
nop	0

How to fix Long Method

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;
use App\Services\Validation\Rules\ValidIdRule;
use App\Models\Lookup\Frequency;
use App\Models\Lookup\Department;
use App\Services\Validation\Rules\LinkedInUrlRule;
use Illuminate\Validation\Rule;
use App\Services\Validation\Rules\PasswordCorrectRule;
use App\Services\Validation\Rules\PasswordFormatRule;
use App\Services\Validation\Rules\TwitterHandleRule;

class UpdateManagerProfileRequest extends FormRequest
{
    /**
     * Validator instance updated on failedValidation.
     *
     * @var \Illuminate\Contracts\Validation\Validator
     */
    public $validator = null;

    /**
     * Determine if the user is authorized to make this request.
     *
     * @return boolean
     */
    public function authorize()
    {
        return $this->manager !== null;
    }

    /**
     * Override Handle a failed validation attempt.
     *
     * @return void
     */
    protected function failedValidation(\Illuminate\Contracts\Validation\Validator $validator)
    {
        $this->validator = $validator;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {
        return [
            'first_name' => 'required|string|max:191',
            'last_name' => 'required|string|max:191',
            'email' => [
                'required',
                'string',
                'max:191',
                'email',
                // Email may match existing email for this user,
                // but must be unique if changed.
                Rule::unique('users', 'email')->ignore($this->manager->user->id)
            ],
            'gov_email' => [
                'nullable',
                'string',
                'max:191',
                'email',
            ],
            // Password validation
            'current_password' => [
                'nullable',
                'required_with:new_password',
                new PasswordCorrectRule
            ],
            'new_password' => [
                'nullable',
                'min:8',
                new PasswordFormatRule,
                'confirmed'
            ],

            'department_id' => ['required', new ValidIdRule(Department::class)],

            'years_experience' => 'nullable|numeric|min:0',
            'gc_directory_url' => 'nullable|url',

            '*.about_me' => 'nullable|string',
            '*.career_journey' => 'nullable|string',
            '*.division' => 'nullable|string',
            '*.education' => 'nullable|string',
            '*.employee_learning' => 'nullable|string',
            '*.expectations' => 'nullable|string',
            '*.greatest_accomplishment' => 'nullable|string',
            '*.leadership_style' => 'nullable|string',
            '*.learning_path' => 'nullable|string',
            '*.position' => 'nullable|string',

            'twitter_username' => [
                'nullable', // Some people may not have a handle.
                new TwitterHandleRule,
            ],
            'linkedin_url' => [
                'nullable', // Some people may not be on LinkedIn.
                new LinkedInUrlRule,
            ],
        ];
    }
}


1			<?php
2
3			namespace App\Http\Requests;
4
5			use Illuminate\Foundation\Http\FormRequest;
6			use App\Services\Validation\Rules\ValidIdRule;
7			use App\Models\Lookup\Frequency;
8			use App\Models\Lookup\Department;
9			use App\Services\Validation\Rules\LinkedInUrlRule;
10			use Illuminate\Validation\Rule;
11			use App\Services\Validation\Rules\PasswordCorrectRule;
12			use App\Services\Validation\Rules\PasswordFormatRule;
13			use App\Services\Validation\Rules\TwitterHandleRule;
14
15			class UpdateManagerProfileRequest extends FormRequest
16			{
17			/**
18			* Validator instance updated on failedValidation.
19			*
20			* @var \Illuminate\Contracts\Validation\Validator
21			*/
22			public $validator = null;
23
24			/**
25			* Determine if the user is authorized to make this request.
26			*
27			* @return boolean
28			*/
29			public function authorize()
30			{
31			return $this->manager !== null;
32			}
33
34			/**
35			* Override Handle a failed validation attempt.
36			*
37			* @return void
38			*/
39			protected function failedValidation(\Illuminate\Contracts\Validation\Validator $validator)
40			{
41			$this->validator = $validator;
42			}
43
44			/**
45			* Get the validation rules that apply to the request.
46			*
47			* @return array
48			*/
49			public function rules()
50			{
51			return [
52			'first_name' => 'required\|string\|max:191',
53			'last_name' => 'required\|string\|max:191',
54			'email' => [
55			'required',
56			'string',
57			'max:191',
58			'email',
59			// Email may match existing email for this user,
60			// but must be unique if changed.
61			Rule::unique('users', 'email')->ignore($this->manager->user->id)
62			],
63			'gov_email' => [
64			'nullable',
65			'string',
66			'max:191',
67			'email',
68			],
69			// Password validation
70			'current_password' => [
71			'nullable',
72			'required_with:new_password',
73			new PasswordCorrectRule
74			],
75			'new_password' => [
76			'nullable',
77			'min:8',
78			new PasswordFormatRule,
79			'confirmed'
80			],
81
82			'department_id' => ['required', new ValidIdRule(Department::class)],
83
84			'years_experience' => 'nullable\|numeric\|min:0',
85			'gc_directory_url' => 'nullable\|url',
86
87			'*.about_me' => 'nullable\|string',
88			'*.career_journey' => 'nullable\|string',
89			'*.division' => 'nullable\|string',
90			'*.education' => 'nullable\|string',
91			'*.employee_learning' => 'nullable\|string',
92			'*.expectations' => 'nullable\|string',
93			'*.greatest_accomplishment' => 'nullable\|string',
94			'*.leadership_style' => 'nullable\|string',
95			'*.learning_path' => 'nullable\|string',
96			'*.position' => 'nullable\|string',
97
98			'twitter_username' => [
99			'nullable', // Some people may not have a handle.
100			new TwitterHandleRule,
101			],
102			'linkedin_url' => [
103			'nullable', // Some people may not be on LinkedIn.
104			new LinkedInUrlRule,
105			],
106			];
107			}
108			}
109

GCTC-NTGC / TalentCloud

Push — master ( 6ceee8...d4902a )

UpdateManagerProfileRequest::rules() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like