CrudInjectable::with() - Code Metrics - Inspection of "Enable unicity checks to crud resources" - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#136)

by Owen

created 2016-10-03 17:28 UTC

CrudInjectable::with() A

↳ Parent: CrudInjectable

Complexity

Conditions	4
Paths	3

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	7
c	1
b	0
f	0
nc	3
nop	1
dl	0
loc	12
rs	9.2

<?php

namespace Backpack\CRUD;

use Backpack\CRUD\Injectables\CheckUnique as CheckUnique;

class CrudInjectable
{
    use CheckUnique;

    protected $requiredInjectables = [];

    protected $name = null;
    protected $options = null;
    protected $controller = null;

    public function __construct($name, $controller, $options)
    {
        $this->name = $name;
        $this->options = $options;
        $this->controller = $controller;
    }

    public function with($injectables)
    {
        if (is_string($injectables)) {
            $this->requiredInjectables[] = 'with'.ucwords($injectables);
        } elseif (is_array($injectables)) {
            foreach ($injectables as $injectable) {
                $this->requiredInjectables = 'with'.ucwords($injectable);

            }
        }

        return $this->inject();
    }

    private function inject()
    {
        foreach ($this->requiredInjectables as $injectable) {
            $this->{$injectable}();
        }
    }

    public function __call($method, $parameters = null)
    {
        if (method_exists($this, $method)) {
            $this->{$method}($parameters);
        }
    }
}


1			<?php
2
3			namespace Backpack\CRUD;
4
5			use Backpack\CRUD\Injectables\CheckUnique as CheckUnique;
6
7			class CrudInjectable
8			{
9			use CheckUnique;
10
11			protected $requiredInjectables = [];
12
13			protected $name = null;
14			protected $options = null;
15			protected $controller = null;
16
17			public function __construct($name, $controller, $options)
18			{
19			$this->name = $name;
20			$this->options = $options;
21			$this->controller = $controller;
22			}
23
24			public function with($injectables)
25			{
26			if (is_string($injectables)) {
27			$this->requiredInjectables[] = 'with'.ucwords($injectables);
28			} elseif (is_array($injectables)) {
29			foreach ($injectables as $injectable) {
30			$this->requiredInjectables = 'with'.ucwords($injectable);
			0 ignored issues – show Documentation Bug introduced 2016-10-03 17:31 UTC by Report Bug Copy Issue Report It seems like `'with' . ucwords($injectable)` of type `string` is incompatible with the declared type `array` of property `$requiredInjectables`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
31			}
32			}
33
34			return $this->inject();
35			}
36
37			private function inject()
38			{
39			foreach ($this->requiredInjectables as $injectable) {
40			$this->{$injectable}();
41			}
42			}
43
44			public function __call($method, $parameters = null)
45			{
46			if (method_exists($this, $method)) {
47			$this->{$method}($parameters);
48			}
49			}
50			}
51

Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Pull Request — master (#136)

CrudInjectable::with() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like