HasQueryFilterTrait::getQueryFilter() - 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

HasQueryFilterTrait::getQueryFilter() A

↳ Parent: HasQueryFilterTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

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

<?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\Requests\RequestInterface;
use WPEmerge\Routing\Conditions\UrlCondition;

/**
 * Represent an object which has a WordPress query filter attribute.
 */
trait HasQueryFilterTrait {
	/**
	 * Query filter.
	 *
	 * @var callable|null
	 */
	protected $query_filter = null;

	/**
	 * Get attribute.
	 *
	 * @param  string $attribute
	 * @param  mixed  $default
	 * @return mixed
	 */
	public abstract function getAttribute( $attribute, $default = '' );

	/**
	 * Apply the query filter, if any.
	 *
	 * @param  RequestInterface $request
	 * @param  array            $query_vars
	 * @return array
	 */
	public function applyQueryFilter( $request, $query_vars ) {
		$query = $this->getAttribute( 'query' );
		$condition = $this->getAttribute( 'condition' );

		if ( $query === null ) {
			return $query_vars;
		}

		if ( ! $condition instanceof UrlCondition ) {
			throw new ConfigurationException(
				'Only routes with a URL condition can use queries. ' .
				'Make sure your route has a URL condition and it is not in a non-URL route group.'
			);
		}

		return call_user_func_array(
			$query,
			array_merge(
				[$query_vars],
				array_values( $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\Requests\RequestInterface;
14		use WPEmerge\Routing\Conditions\UrlCondition;
15
16		/**
17		* Represent an object which has a WordPress query filter attribute.
18		*/
19		trait HasQueryFilterTrait {
20		/**
21		* Query filter.
22		*
23		* @var callable\|null
24		*/
25		protected $query_filter = null;
26
27		/**
28		* Get attribute.
29		*
30		* @param string $attribute
31		* @param mixed $default
32		* @return mixed
33		*/
34		public abstract function getAttribute( $attribute, $default = '' );
35
36		/**
37		* Apply the query filter, if any.
38		*
39		* @param RequestInterface $request
40		* @param array $query_vars
41		* @return array
42		*/
43	3	public function applyQueryFilter( $request, $query_vars ) {
44	3	$query = $this->getAttribute( 'query' );
45	3	$condition = $this->getAttribute( 'condition' );
46
47	3	if ( $query === null ) {
48	1	return $query_vars;
49		}
50
51	2	if ( ! $condition instanceof UrlCondition ) {
52	1	throw new ConfigurationException(
53		'Only routes with a URL condition can use queries. ' .
54	1	'Make sure your route has a URL condition and it is not in a non-URL route group.'
55		);
56		}
57
58	1	return call_user_func_array(
59	1	$query,
60	1	array_merge(
61	1	[$query_vars],
62	1	array_values( $condition->getArguments( $request ) )
63		)
64		);
65		}
66		}
67

htmlburger / wpemerge

Branch — try/refactor-route (c848cf)

HasQueryFilterTrait::getQueryFilter() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like