ContinentSerializerTrait::getAttributes() - Code Metrics - Inspection of "moved serializer to own root package" - keeko/core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a7ae23...b30b3b )

by Thomas

created 2016-03-09 15:46 UTC

ContinentSerializerTrait::getAttributes() A

↳ Parent: ContinentSerializerTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	8
rs	9.4285
cc	1
eloc	6
nc	1
nop	2

<?php
namespace keeko\core\serializer\base;

/**
 */
trait ContinentSerializerTrait {

	/**
	 * @param mixed $model
	 * @param array $fields
	 */
	public function getAttributes($model, array $fields = null) {

		return [
			'id' => $model->Id(),
			'parent_id' => $model->ParentId(),
			'numeric' => $model->Numeric(),
			'name' => $model->Name(),
		];
	}

	/**
	 */
	public function getFields() {
		return ['id', 'parent_id', 'numeric', 'name'];
	}

	/**
	 * @param mixed $model
	 */
	public function getId($model) {

		return $model->getId();
	}

	/**
	 */
	public function getSortFields() {
		return ['id', 'parent_id', 'numeric', 'name'];
	}

	/**
	 * @param mixed $model
	 */
	public function getType($model) {

		return 'core/continent';
	}

	/**
	 * @param mixed $model
	 * @param mixed $data
	 */
	public function hydrate($model, $data) {

		// this model is read-only!
		return $model;
	}
}


1			<?php
2			namespace keeko\core\serializer\base;
3
4			/**
5			*/
6			trait ContinentSerializerTrait {
7
8			/**
9			* @param mixed $model
10			* @param array $fields
11			*/
12			public function getAttributes($model, array $fields = null) {
			1 ignored issue – show Unused Code introduced 2016-03-09 15:56 UTC by Report Bug Copy Issue Report The parameter `$fields` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
13			return [
14			'id' => $model->Id(),
15			'parent_id' => $model->ParentId(),
16			'numeric' => $model->Numeric(),
17			'name' => $model->Name(),
18			];
19			}
20
21			/**
22			*/
23			public function getFields() {
24			return ['id', 'parent_id', 'numeric', 'name'];
25			}
26
27			/**
28			* @param mixed $model
29			*/
30			public function getId($model) {
			0 ignored issues – show Documentation introduced 2016-03-09 15:56 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
31			return $model->getId();
32			}
33
34			/**
35			*/
36			public function getSortFields() {
37			return ['id', 'parent_id', 'numeric', 'name'];
38			}
39
40			/**
41			* @param mixed $model
42			*/
43			public function getType($model) {
			1 ignored issue – show Unused Code introduced 2016-03-09 15:56 UTC by Report Bug Copy Issue Report The parameter `$model` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
44			return 'core/continent';
45			}
46
47			/**
48			* @param mixed $model
49			* @param mixed $data
50			*/
51			public function hydrate($model, $data) {
			1 ignored issue – show Unused Code introduced 2016-03-09 15:56 UTC by Report Bug Copy Issue Report The parameter `$data` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
52			// this model is read-only!
53			return $model;
54			}
55			}
56

keeko / core

Push — master ( a7ae23...b30b3b )

ContinentSerializerTrait::getAttributes() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like