Model::validate() - Code Metrics - Inspection of "test" - AnuTiger/F3 - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (03df00)

by Anu

created 2017-10-07 19:56 UTC

Model::validate() B

↳ Parent: Project

Complexity

Conditions	9
Paths	10

Size

Total Lines	17
Code Lines	12

Duplication

Lines	4
Ratio	23.53 %

Importance

Changes

Metric	Value
cc	9
eloc	12
nc	10
nop	2
dl	4
loc	17
rs	7.756
c	0
b	0
f	0

<?


abstract class Model  extends Cortex {
filter:
    dependency_paths: ["lib/*"]
	protected $app;
	protected $db = 'DB';

	protected $fieldConf = array(
		'created_at' => array(
			'type' => Schema::DT_TIMESTAMP,
filter:
    dependency_paths: ["lib/*"]
			'default' => Schema::DF_CURRENT_TIMESTAMP
			),
		'updated_at' => array(
			'type' => Schema::DT_TIMESTAMP,
			'default' => '0-0-0 0:0:0'
			),
		'deleted_at' => array(
			'type' => Schema::DT_TIMESTAMP,
			'default' => '0-0-0 0:0:0'
			)
		);

	public function __construct() {
		if(property_exists($this, 'fields')) {
			$this->fieldConf = array_merge($this->fields, $this->fieldConf);
		}

		parent::__construct();
		$this->app = f3();
		//$this->beforesave($this->validate(get_called_class()));
	}

	/*private function validate($caller, $parent) {
		$valid = true;
		foreach($this->getFieldConfiguration() as $field => $conf) {
			if(isset($conf['type']) && !isset($conf['relType'])){
				$val = $this->get($field);
				$model = strtolower(str_replace('\\','.',$class));
				// check required fields
				if ($valid && isset($conf['required']))
					$valid = \Validation::instance()->required($val,$field,'error.'.$model.'.'.$field);
				// check unique
				if ($valid && isset($conf['unique']))
					$valid = \Validation::instance()->unique($self,$val,$field,'error.'.$model.'.'.$field);
				if (!$valid)
					break;
			}
		}
		return $valid;
	}*/
}

1			<?
			0 ignored issues – show Security Best Practice introduced 2017-10-07 19:32 UTC by Report Bug Copy Issue Report It is not recommended to use PHP's short opening tag `<?`, better use `<?php`, or `<?=` in case of outputting. Short opening tags are disabled in PHP?s default configuration. In such a case, all content of this file is output verbatim to the browser without being parsed, or executed. As a precaution to avoid these problems better use the long opening tag `<?php`. Loading history...
2
3			abstract class Model extends Cortex {
			0 ignored issues – show Bug introduced 2017-10-07 19:32 UTC by Report Bug Copy Issue Report The type `Cortex` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
4			protected $app;
5			protected $db = 'DB';
			0 ignored issues – show Comprehensibility introduced 2017-10-07 19:57 UTC by Report Bug Copy Issue Report Avoid variables with short names like $db. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history...
6			protected $fieldConf = array(
7			'created_at' => array(
8			'type' => Schema::DT_TIMESTAMP,
			0 ignored issues – show Bug introduced 2017-10-07 19:32 UTC by Report Bug Copy Issue Report The type `Schema` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
9			'default' => Schema::DF_CURRENT_TIMESTAMP
10			),
11			'updated_at' => array(
12			'type' => Schema::DT_TIMESTAMP,
13			'default' => '0-0-0 0:0:0'
14			),
15			'deleted_at' => array(
16			'type' => Schema::DT_TIMESTAMP,
17			'default' => '0-0-0 0:0:0'
18			)
19			);
20
21			public function __construct() {
22			if(property_exists($this, 'fields')) {
23			$this->fieldConf = array_merge($this->fields, $this->fieldConf);
24			}
25
26			parent::__construct();
27			$this->app = f3();
28			//$this->beforesave($this->validate(get_called_class()));
29			}
30
31			/*private function validate($caller, $parent) {
32			$valid = true;
33			foreach($this->getFieldConfiguration() as $field => $conf) {
34			if(isset($conf['type']) && !isset($conf['relType'])){
35			$val = $this->get($field);
36			$model = strtolower(str_replace('\\','.',$class));
37			// check required fields
38			if ($valid && isset($conf['required']))
39			$valid = \Validation::instance()->required($val,$field,'error.'.$model.'.'.$field);
40			// check unique
41			if ($valid && isset($conf['unique']))
42			$valid = \Validation::instance()->unique($self,$val,$field,'error.'.$model.'.'.$field);
43			if (!$valid)
44			break;
45			}
46			}
47			return $valid;
48			}*/
49			}

AnuTiger / F3

Branch — master (03df00)

Model::validate() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like