Route::getArguments() - Code Metrics - Inspection of "Fix mismatched method signature" - htmlburger/wpemerge - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Branch — try/refactor-route (c848cf)

by Atanas

created 2020-04-19 23:41 UTC

Route::getArguments() A

↳ Parent: Route

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2.032

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	4
c	1
b	0
f	0
dl	0
loc	8
ccs	4
cts	5
cp	0.8
rs	10
cc	2
nc	2
nop	1
crap	2.032

<?php
/**
 * @package   WPEmerge
 * @author    Atanas Angelov <[email protected]>
 * @copyright 2017-2019 Atanas Angelov
 * @license   https://www.gnu.org/licenses/gpl-2.0.html GPL-2.0
 * @link      https://wpemerge.com/
 */

namespace WPEmerge\Routing;

use WPEmerge\Exceptions\ConfigurationException;
use WPEmerge\Helpers\HasAttributesTrait;
use WPEmerge\Requests\RequestInterface;
use WPEmerge\Routing\Conditions\ConditionInterface;

/**
 * Represent a route
 */
class Route implements RouteInterface, HasQueryFilterInterface {
	use HasAttributesTrait;
	use HasQueryFilterTrait;

	/**
	 * {@inheritDoc}
	 */
	public function isSatisfied( RequestInterface $request ) {
		$methods = $this->getAttribute( 'methods', [] );
		$condition = $this->getAttribute( 'condition' );

		if ( ! in_array( $request->getMethod(), $methods ) ) {
			return false;
		}

		if ( ! $condition instanceof ConditionInterface ) {
			throw new ConfigurationException( 'Route does not have a condition.' );
		}

		return $condition->isSatisfied( $request );
	}

	/**
	 * {@inheritDoc}
	 */
	public function getArguments( RequestInterface $request ) {
		$condition = $this->getAttribute( 'condition' );

		if ( ! $condition instanceof ConditionInterface ) {
			throw new ConfigurationException( 'Route does not have a condition.' );
		}

		return $condition->getArguments( $request );
	}
}


1		<?php
2		/**
3		* @package WPEmerge
4		* @author Atanas Angelov <[email protected]>
5		* @copyright 2017-2019 Atanas Angelov
6		* @license https://www.gnu.org/licenses/gpl-2.0.html GPL-2.0
7		* @link https://wpemerge.com/
8		*/
9
10		namespace WPEmerge\Routing;
11
12		use WPEmerge\Exceptions\ConfigurationException;
13		use WPEmerge\Helpers\HasAttributesTrait;
14		use WPEmerge\Requests\RequestInterface;
15		use WPEmerge\Routing\Conditions\ConditionInterface;
16
17		/**
18		* Represent a route
19		*/
20		class Route implements RouteInterface, HasQueryFilterInterface {
21		use HasAttributesTrait;
22		use HasQueryFilterTrait;
23
24		/**
25		* {@inheritDoc}
26		*/
27	2	public function isSatisfied( RequestInterface $request ) {
28	2	$methods = $this->getAttribute( 'methods', [] );
29	2	$condition = $this->getAttribute( 'condition' );
30
31	2	if ( ! in_array( $request->getMethod(), $methods ) ) {
32	1	return false;
33		}
34
35	2	if ( ! $condition instanceof ConditionInterface ) {
36		throw new ConfigurationException( 'Route does not have a condition.' );
37		}
38
39	2	return $condition->isSatisfied( $request );
40		}
41
42		/**
43		* {@inheritDoc}
44		*/
45	1	public function getArguments( RequestInterface $request ) {
46	1	$condition = $this->getAttribute( 'condition' );
47
48	1	if ( ! $condition instanceof ConditionInterface ) {
49		throw new ConfigurationException( 'Route does not have a condition.' );
50		}
51
52	1	return $condition->getArguments( $request );
53		}
54		}
55

htmlburger / wpemerge

Branch — try/refactor-route (c848cf)

Route::getArguments() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like