Category::find_all() - Code Metrics - Inspection of "Merge branch 'feature/default_classes' into develo..." - Apiki/wpsteak-plugin - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — develop ( d7a728...605806 )

by Elvis Henrique

created 2019-01-17 23:59 UTC

Category::find_all() A

↳ Parent: Category

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
eloc	7
dl	0
loc	15
c	0
b	0
f	0
ccs	7
cts	7
cp	1
rs	10
cc	2
nc	2
nop	0
crap	2

<?php
/**
 * Category.
 *
 * @package App
 */

declare(strict_types=1);

namespace App\Repositories;

use App\Entities\Collection;
use App\Entities\Category as Entity;

/**
 * Category class.
 */
class Category extends AbstractTerm {

	/**
	 * Find by id.
	 *
	 * @param integer $id Id.
	 * @return Entity|null
	 */
	public function find_by_id( int $id ) : ?Entity {
		$term = $this->get_term( $id );

		if ( ! $term ) {

			return null;
		}

		return ( new Entity() )->set_term( $term );
	}

	/**
	 * Find all.
	 *
	 * @return Collection
	 */
	public function find_all() : Collection {
		$terms = $this->get_terms(
			[
				'taxonomy'   => Entity::TAXONOMY,
				'hide_empty' => false,
			]
		);

		$collection = new Collection();

		foreach ( $terms as $term ) {
			$collection->add_entity( ( new Entity() )->set_term( $term ) );
		}

		return $collection;
	}
}


1		<?php
2		/**
3		* Category.
4		*
5		* @package App
6		*/
7
8		declare(strict_types=1);
9
10		namespace App\Repositories;
11
12		use App\Entities\Collection;
13		use App\Entities\Category as Entity;
14
15		/**
16		* Category class.
17		*/
18		class Category extends AbstractTerm {
19
20		/**
21		* Find by id.
22		*
23		* @param integer $id Id.
24		* @return Entity\|null
25		*/
26	2	public function find_by_id( int $id ) : ?Entity {
27	2	$term = $this->get_term( $id );
28
29	2	if ( ! $term ) {
		0 ignored issues – show introduced 2019-01-18 00:03 UTC by Report Bug Copy Issue Report `$term` is of type `WP_term`, thus it always evaluated to `true`. Loading history...
30	1	return null;
31		}
32
33	1	return ( new Entity() )->set_term( $term );
34		}
35
36		/**
37		* Find all.
38		*
39		* @return Collection
40		*/
41	1	public function find_all() : Collection {
42	1	$terms = $this->get_terms(
43		[
44	1	'taxonomy' => Entity::TAXONOMY,
45		'hide_empty' => false,
46		]
47		);
48
49	1	$collection = new Collection();
50
51	1	foreach ( $terms as $term ) {
52	1	$collection->add_entity( ( new Entity() )->set_term( $term ) );
53		}
54
55	1	return $collection;
56		}
57		}
58

Apiki / wpsteak-plugin

Push — develop ( d7a728...605806 )

Category::find_all() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like