Afrittella\BackProject\Http\Requests\RoleEdit

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	authorize()	4	4	1
A	rules()	10	10	1

<?php

namespace Afrittella\BackProject\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

use Afrittella\BackProject\Repositories\Roles;
use Illuminate\Container\Container as App;

class RoleEdit extends FormRequest

This class seems to be duplicated in your project.

{
    protected $roles;
    public function __construct(array $query = array(), array $request = array(), array $attributes = array(), array $cookies = array(), array $files = array(), array $server = array(), $content = null, Roles $roles)
    {
        parent::__construct($query, $request, $attributes, $cookies, $files, $server, $content);
        $this->roles = $roles;
    }

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

    /**
     * Get the validation rules that apply to the request.
     *
     * @return array
     */
    public function rules()
    {

        //$roleRepository = new RoleRepository($this->app);

54% of this comment could be valid code. Did you maybe forget this after debugging?

        $role = $this->roles->findBy('id', $this->route('role'));

        return [
            'name' => 'required|max:255|unique:roles,name,'.$role->id,
        ];
    }
}