View GitHub Repository TalentCloud

Schedule Inspection
Subscribe
Completed
Push — dev ( 39ba5f...0b8cdb )
by Tristan
04:57 queued 04:51
created

DegreeValidator::validate()   A

↳ Parent: DegreeValidator

Complexity

Conditions 1
Paths 1

Size

Total Lines 17
Code Lines 10

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 0
Metric Value
eloc 10
dl 0
loc 17
ccs 0
cts 9
cp 0
rs 9.9332
c 0
b 0
f 0
cc 1
nc 1
nop 1
crap 2
1 
<?php
2 
namespace App\Services\Validation;
0 ignored issues
– show
Coding Style introduced by
Missing file doc comment
Loading history...
3 
use Illuminate\Support\Facades\Validator;
4 
use Illuminate\Validation\Rule;
5 
use App\Models\Degree;
6 
use App\Models\Lookup\DegreeType;
7 
use App\Models\Applicant;
8 
use App\Services\Validation\Rules\UniqueApplicantSkillRule;
9 
class DegreeValidator
0 ignored issues
– show
Coding Style introduced by
Missing doc comment for class DegreeValidator
Loading history...
10 
{
11
12 
    protected $applicant;
13 
    protected $degree_type_id;
14
15 
    public function __construct(Applicant $applicant)
0 ignored issues
– show
Coding Style introduced by
Missing doc comment for function __construct()
Loading history...
16 
    {
17 
        $this->applicant = $applicant;
18 
        $this->degree_type_id = DegreeType::all()->pluck('id');
19
20 
    }
21 
    public function validate(DegreeValidator $degreeValidator)
0 ignored issues
– show
Coding Style introduced by
Missing doc comment for function validate()
Loading history...
22 
    {
23 
        $uniqueSkillRule = new UniqueApplicantSkillRule($this->applicant, $degreeValidator->id);
0 ignored issues
– show
Bug Best Practice introduced by
The property id does not exist on App\Services\Validation\DegreeValidator. Did you maybe forget to declare it?
Loading history...
Unused Code introduced by
The assignment to $uniqueSkillRule is dead and can be removed.
Loading history...
24 
        //This array is reset every time because applicants table can change frequently
25 
        $applicant_ids = Applicant::all()->pluck('id');
26 
        //Validate basic data is filled in
27 
        Validator::make($degreeValidator->getAttributes(), [
0 ignored issues
– show
Bug introduced by
The method getAttributes() does not exist on App\Services\Validation\DegreeValidator. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

27 
        Validator::make($degreeValidator->/** @scrutinizer ignore-call */ getAttributes(), [

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
Coding Style introduced by
The opening parenthesis of a multi-line function call should be the last content on the line.
Loading history...
28 
            'applicant_id' => [
29 
            'required',
30 
               Rule::in($applicant_ids->toArray()),
31 
        ],
32 
            'degree_type_id' => [
33 
            'required',
34 
               Rule::in($this->degree_type_id->toArray()),
35 
        ]
36
37 
        ])->validate();
0 ignored issues
– show
Coding Style introduced by
For multi-line function calls, the closing parenthesis should be on a new line.

If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line:

someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
Loading history...
38 
    }
39
40 
}