App\Services\Validation\Rules\SkillDeclarationBelongsToUserRule

Complexity

Total Complexity

3

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	message()	0	3	1
A	passes()	0	4	2

<?php

Missing file doc comment

namespace App\Services\Validation\Rules;

use Illuminate\Contracts\Validation\Rule;
use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Lang;
use App\Models\SkillDeclaration;

class SkillDeclarationBelongsToUserRule implements Rule

Missing doc comment for class SkillDeclarationBelongsToUserRule

{

    /**
     * This check passes if the $user has ownership of this skill declaration with id=$value
     * @param  [type] $attribute [description]

The doc comment [type] at position 0 could not be parsed: Unknown type name '[' at position 0 in [type].

There must be exactly one blank line before the tags in a doc comment

     * @param  [type] $value     [description]
     * @return [type]            [description]

The doc comment [type] at position 0 could not be parsed: Unknown type name '[' at position 0 in [type].

Tag cannot be grouped with parameter tags in a doc comment

     */
    public function passes($attribute, $value)
    {
        return SkillDeclaration::find($value) &&
            SkillDeclaration::find($value)->applicant->user->id == Auth::user()->id;

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    public function message()

Missing doc comment for function message()

    {
        return Lang::get('validation.user_owns_skill_declaration');
    }
}