User_Capability_Condition - Code Metrics - Inspection of "fix static and dynamic condition evaluation and pa..." - htmlburger/carbon-fields - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — milestone/2_0/container-condit... ( af96ef...c41f81 )

unknown

created 2017-02-28 09:17 UTC

User_Capability_Condition A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

Metric	Value
dl	0
loc	47
rs	10
c	0
b	0
f	0
wmc	8
lcom	1
cbo	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	user_can_any()	0	8	3
B	is_fulfilled()	0	23	5

<?php

namespace Carbon_Fields\Container\Condition;

/**
 * Check if user has a specific capability
 * 
 * Operator "CUSTOM" is passed the user id
 */
class User_Capability_Condition extends Condition {

	/**
	 * Check if a user has any of the supplied capabilities
	 * 
	 * @param  array<string> $capabilities
	 * @return boolean
	 */
	protected function user_can_any( $user_id, $capabilities ) {
		foreach ( $capabilities as $cap ) {
			if ( user_can( $user_id, $cap ) ) {
				return true;
			}
		}
		return false;
	}
	
	/**
	 * Check if the condition is fulfilled
	 * 
	 * @param  array $environment
	 * @return bool
	 */
	public function is_fulfilled( $environment ) {
		$user_id = $environment['user_id'];
		switch ( $this->get_comparison_operator() ) {
			case '=':
				return user_can( $user_id, $this->get_value() );
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case '!=':
				return ! user_can( $user_id, $this->get_value() );
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'IN':
				return $this->user_can_any( $user_id, $this->get_value() );
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'NOT IN':
				return ! $this->user_can_any( $user_id, $this->get_value() );
				break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
		}

		return $this->first_supported_comparer_is_correct(
			$user_id,
			$this->get_comparison_operator(),
			$this->get_value()
		);
	}
}

1			<?php
2
3			namespace Carbon_Fields\Container\Condition;
4
5			/**
6			* Check if user has a specific capability
7			*
8			* Operator "CUSTOM" is passed the user id
9			*/
10			class User_Capability_Condition extends Condition {
11
12			/**
13			* Check if a user has any of the supplied capabilities
14			*
15			* @param array<string> $capabilities
16			* @return boolean
17			*/
18			protected function user_can_any( $user_id, $capabilities ) {
19			foreach ( $capabilities as $cap ) {
20			if ( user_can( $user_id, $cap ) ) {
21			return true;
22			}
23			}
24			return false;
25			}
26
27			/**
28			* Check if the condition is fulfilled
29			*
30			* @param array $environment
31			* @return bool
32			*/
33			public function is_fulfilled( $environment ) {
34			$user_id = $environment['user_id'];
35			switch ( $this->get_comparison_operator() ) {
36			case '=':
37			return user_can( $user_id, $this->get_value() );
38			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:20 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
39			case '!=':
40			return ! user_can( $user_id, $this->get_value() );
41			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:20 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
42			case 'IN':
43			return $this->user_can_any( $user_id, $this->get_value() );
44			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:20 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
45			case 'NOT IN':
46			return ! $this->user_can_any( $user_id, $this->get_value() );
47			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:20 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
48			}
49
50			return $this->first_supported_comparer_is_correct(
51			$user_id,
52			$this->get_comparison_operator(),
53			$this->get_value()
54			);
55			}
56			}

htmlburger / carbon-fields

Push — milestone/2_0/container-condit... ( af96ef...c41f81 )

User_Capability_Condition A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

2 Methods

Duplication Side-by-Side

Filter issues like