FormRequest - Code Metrics - Inspection of "Updating the package" - ARCANEDEV/Support - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#24)

by ARCANEDEV

created 2018-07-17 16:28 UTC

FormRequest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
dl	0
loc	37
rs	10
c	0
b	0
f	0
ccs	7
cts	7
cp	1
wmc	3
lcom	0
cbo	1

3 Methods

Rating	Name	Size	Complexity
	rules()	1	?
A	authorize()	4	1
A	validateResolved()	7	2

<?php namespace Arcanedev\Support\Http;

use Illuminate\Foundation\Http\FormRequest as BaseFormRequest;

/**
 * Class     FormRequest
 *
 * @package  Arcanedev\Support
 * @author   ARCANEDEV <[email protected]>
 */
abstract class FormRequest extends BaseFormRequest
{
    /* -----------------------------------------------------------------
     |  Main Methods
     | -----------------------------------------------------------------
     */

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

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

    /**
     * Validate the class instance.
     *
     * @return void
     */
    public function validateResolved()
    {
        parent::validateResolved();

        if (method_exists($this, 'prepareAfterValidation'))
            $this->prepareAfterValidation();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }
}


1		<?php namespace Arcanedev\Support\Http;
2
3		use Illuminate\Foundation\Http\FormRequest as BaseFormRequest;
4
5		/**
6		* Class FormRequest
7		*
8		* @package Arcanedev\Support
9		* @author ARCANEDEV <[email protected]>
10		*/
11		abstract class FormRequest extends BaseFormRequest
12		{
13		/* -----------------------------------------------------------------
14		\| Main Methods
15		\| -----------------------------------------------------------------
16		*/
17
18		/**
19		* Determine if the user is authorized to make this request.
20		*
21		* @return bool
22		*/
23	6	public function authorize()
24		{
25	6	return true;
26		}
27
28		/**
29		* Get the validation rules that apply to the request.
30		*
31		* @return array
32		*/
33		abstract public function rules();
34
35		/**
36		* Validate the class instance.
37		*
38		* @return void
39		*/
40	6	public function validateResolved()
41		{
42	6	parent::validateResolved();
43
44	6	if (method_exists($this, 'prepareAfterValidation'))
45	6	$this->prepareAfterValidation();
		0 ignored issues – show Documentation Bug introduced 2018-07-17 16:36 UTC by Report Bug Copy Issue Report The method `prepareAfterValidation` does not exist on `object<Arcanedev\Support\Http\FormRequest>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
46	6	}
47		}
48

ARCANEDEV / Support

Pull Request — master (#24)

FormRequest A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

3 Methods

Duplication Side-by-Side

Filter issues like