ModelFields - Code Metrics - anavel/crud - Measure and Improve Code Quality continuously with Scrutinizer

ModelFields A
last analyzed 2017-09-28 22:36 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
dl	0
loc	47
ccs	21
cts	21
cp	1
rs	10
c	0
b	0
f	0
wmc	11
lcom	1
cbo	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	readConfig()	0	8	6
B	setConfig()	0	20	5

<?php

namespace Anavel\Crud\Abstractor\Eloquent\Traits;

trait ModelFields
{
    protected $fieldsPresentation = [];
    protected $formTypes = [];
    protected $validationRules = [];
    protected $functions = [];
    protected $defaults = [];

    /**
     * @param string $action
     */
    public function readConfig($action)
    {
        $this->fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->formTypes = $this->getConfigValue($action, 'form_types') ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->validationRules = $this->getConfigValue($action, 'validation') ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->functions = $this->getConfigValue($action, 'functions') ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->defaults = $this->getConfigValue($action, 'defaults') ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * @param array  $config
     * @param string $columnName
     *
     * @return array
     */
    public function setConfig(array $config, $columnName)
    {
        if (array_key_exists($columnName, $this->formTypes)) {
            $config['form_type'] = $this->formTypes[$columnName];
        }

        if (array_key_exists($columnName, $this->validationRules)) {
            $config['validation'] = $this->validationRules[$columnName];
        }

        if (array_key_exists($columnName, $this->functions)) {
            $config['functions'] = $this->functions[$columnName];
        }

        if (array_key_exists($columnName, $this->defaults)) {
            $config['defaults'] = $this->defaults[$columnName];
        }

        return $config;
    }
}


1		<?php
2
3		namespace Anavel\Crud\Abstractor\Eloquent\Traits;
4
5		trait ModelFields
6		{
7		protected $fieldsPresentation = [];
8		protected $formTypes = [];
9		protected $validationRules = [];
10		protected $functions = [];
11		protected $defaults = [];
12
13		/**
14		* @param string $action
15		*/
16	14	public function readConfig($action)
17		{
18	14	$this->fieldsPresentation = $this->getConfigValue('fields_presentation') ?: [];
		0 ignored issues – show Bug introduced 2016-02-12 13:12 UTC by Report Bug Copy Issue Report It seems like `getConfigValue()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
19	14	$this->formTypes = $this->getConfigValue($action, 'form_types') ?: [];
		0 ignored issues – show Bug introduced 2016-02-12 13:12 UTC by Report Bug Copy Issue Report It seems like `getConfigValue()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
20	14	$this->validationRules = $this->getConfigValue($action, 'validation') ?: [];
		0 ignored issues – show Bug introduced 2016-02-12 13:12 UTC by Report Bug Copy Issue Report It seems like `getConfigValue()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
21	14	$this->functions = $this->getConfigValue($action, 'functions') ?: [];
		0 ignored issues – show Bug introduced 2016-02-12 13:12 UTC by Report Bug Copy Issue Report It seems like `getConfigValue()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
22	14	$this->defaults = $this->getConfigValue($action, 'defaults') ?: [];
		0 ignored issues – show Bug introduced 2016-02-12 13:12 UTC by Report Bug Copy Issue Report It seems like `getConfigValue()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
23	14	}
24
25		/**
26		* @param array $config
27		* @param string $columnName
28		*
29		* @return array
30		*/
31	14	public function setConfig(array $config, $columnName)
32		{
33	14	if (array_key_exists($columnName, $this->formTypes)) {
34	4	$config['form_type'] = $this->formTypes[$columnName];
35	4	}
36
37	14	if (array_key_exists($columnName, $this->validationRules)) {
38	4	$config['validation'] = $this->validationRules[$columnName];
39	4	}
40
41	14	if (array_key_exists($columnName, $this->functions)) {
42	4	$config['functions'] = $this->functions[$columnName];
43	4	}
44
45	14	if (array_key_exists($columnName, $this->defaults)) {
46	4	$config['defaults'] = $this->defaults[$columnName];
47	4	}
48
49	14	return $config;
50		}
51		}
52

anavel / crud

ModelFields A last analyzed 2017-09-28 22:36 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

2 Methods

Duplication Side-by-Side

Filter issues like

ModelFields A
last analyzed 2017-09-28 22:36 UTC