User_Capability_Condition::is_fulfilled() - Code Metrics - Inspection of "reduce duplication in user id condition" - htmlburger/carbon-fields - Measure and Improve Code Quality continuously with Scrutinizer

Completed

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

unknown

created 2017-02-28 09:35 UTC

User_Capability_Condition::is_fulfilled() B

↳ Parent: User_Capability_Condition

Complexity

Conditions	5
Paths	5

Size

Total Lines	23
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	19
nc	5
nop	1
dl	0
loc	23
rs	8.5906
c	0
b	0
f	0

<?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 {

	/**
	 * Get user id from environment
	 * 
	 * @param  array         $environment
	 * @return array<string>
	 */
	protected function get_user_id( $environment ) {
		return $environment['user_id'];
	}

	/**
	 * 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 = $this->get_user_id( $environment );
		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			* Get user id from environment
14			*
15			* @param array $environment
16			* @return array<string>
17			*/
18			protected function get_user_id( $environment ) {
19			return $environment['user_id'];
20			}
21
22			/**
23			* Check if a user has any of the supplied capabilities
24			*
25			* @param array<string> $capabilities
26			* @return boolean
27			*/
28			protected function user_can_any( $user_id, $capabilities ) {
29			foreach ( $capabilities as $cap ) {
30			if ( user_can( $user_id, $cap ) ) {
31			return true;
32			}
33			}
34			return false;
35			}
36
37			/**
38			* Check if the condition is fulfilled
39			*
40			* @param array $environment
41			* @return bool
42			*/
43			public function is_fulfilled( $environment ) {
44			$user_id = $this->get_user_id( $environment );
45			switch ( $this->get_comparison_operator() ) {
46			case '=':
47			return user_can( $user_id, $this->get_value() );
48			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:40 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...
49			case '!=':
50			return ! user_can( $user_id, $this->get_value() );
51			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:40 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...
52			case 'IN':
53			return $this->user_can_any( $user_id, $this->get_value() );
54			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:40 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...
55			case 'NOT IN':
56			return ! $this->user_can_any( $user_id, $this->get_value() );
57			break;
			1 ignored issue – show Unused Code introduced 2017-02-28 09:40 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...
58			}
59
60			return $this->first_supported_comparer_is_correct(
61			$user_id,
62			$this->get_comparison_operator(),
63			$this->get_value()
64			);
65			}
66			}

htmlburger / carbon-fields

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

User_Capability_Condition::is_fulfilled() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like