Kirki_Active_Callback::compare() - Code Metrics - Inspection of "remove unnecessary condition" - aristath/kirki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( d9c9c1...345147 )

by Aristeides

created 2017-04-23 11:40 UTC

Kirki_Active_Callback::compare() C

↳ Parent: Kirki_Active_Callback

Complexity

Conditions	40
Paths	124

Size

Total Lines	66
Code Lines	56

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	40
eloc	56
nc	124
nop	3
dl	0
loc	66
rs	5.1724
c	0
b	0
f	0

How to fix Long Method Complexity

<?php
/**
 * Active callback used with the "required" argument in fields
 *
 * @package     Kirki
 * @category    Core
 * @author      Aristeides Stathopoulos
 * @copyright   Copyright (c) 2016, Aristeides Stathopoulos
 * @license     http://opensource.org/licenses/https://opensource.org/licenses/MIT
 */

/**
 * Callback class for use with the "required" argument
 */
class Kirki_Active_Callback {

	/**
	 * Figure out whether the current object should be displayed or not.
	 *
	 * @param  WP_Customize_Setting $object The current field.
	 * @return boolean
	 */
	public static function evaluate( $object ) {

		$show = true;

		// Get all fields.
		$fields = Kirki::$fields;

		// Make sure the current object matches a registered field.
		if ( ! isset( $object->setting->id ) || ! isset( $fields[ $object->setting->id ] ) ) {
			return true;
		}

		$field = $fields[ $object->setting->id ];

		if ( isset( $field['required'] ) ) {

			foreach ( $field['required'] as $requirement ) {
				// Handles "AND" functionality.
				$show = self::evaluate_requirement( $object, $field, $requirement );
				// No need to process further if one requirement returns false.
				if ( ! $show ) {
					return false;
				}
			}
		}

		return true;

	}

	/**
	 * Figure out whether the current object should be displayed or not.
	 * We're only parsing a single requirement here from the array of requirements.
	 * This is a proxy function that facilitates evaluating and/or conditions.
	 *
	 * @param  WP_Customize_Setting $object      The current field.
	 * @param  object               $field       The current object.
	 * @param  array                $requirement A single requirement.
	 * @return boolean
	 */
	private static function evaluate_requirement( $object, $field, $requirement ) {

		// Test for callables first.
		if ( is_callable( $requirement ) ) {
			return call_user_func_array( $requirement, array( $field, $object ) );
		}

		// Look for comparison array.
		if ( is_array( $requirement ) && isset( $requirement['operator'], $requirement['value'], $requirement['setting'] ) ) {

			if ( isset( $field['option_name'] ) && '' !== $field['option_name'] ) {
				if ( false === strpos( $requirement['setting'], '[' ) ) {
					$requirement['setting'] = $field['option_name'] . '[' . $requirement['setting'] . ']';
				}
			}

			$current_setting = $object->manager->get_setting( $requirement['setting'] );

			/**
			 * Depending on the 'operator' argument we use,
			 * we'll need to perform the appropriate comparison
			 * and figure out if the control will be shown or not.
			 */
			$show = self::compare( $requirement['value'], $current_setting->value(), $requirement['operator'] );

		} else {
			if ( ! is_array( $requirement ) ) {
				return true;
			}

			// Handles "OR" functionality.
			$show = false;
			foreach ( $requirement as $sub_requirement ) {
				$show = self::evaluate_requirement( $object, $field, $sub_requirement );
				// No need to go on if one sub_requirement returns true.
				if ( $show ) {
					return true;
				}
			}
		}

		return $show;
	}

	/**
	 * Compares the 2 values given the condition
	 *
	 * @param mixed  $value1   The 1st value in the comparison.
	 * @param mixed  $value2   The 2nd value in the comparison.
	 * @param string $operator The operator we'll use for the comparison.
	 * @return boolean whether The comparison has succeded (true) or failed (false).
	 */
	public static function compare( $value1, $value2, $operator ) {
		switch ( $operator ) {
			case '===':
				$show = ( $value1 === $value2 ) ? true : false;
				break;
			case '==':
			case '=':
			case 'equals':
			case 'equal':
				$show = ( $value1 == $value2 ) ? true : false;
				break;
			case '!==':
				$show = ( $value1 !== $value2 ) ? true : false;
				break;
			case '!=':
			case 'not equal':
				$show = ( $value1 != $value2 ) ? true : false;
				break;
			case '>=':
			case 'greater or equal':
			case 'equal or greater':
				$show = ( $value1 >= $value2 ) ? true : false;
				break;
			case '<=':
			case 'smaller or equal':
			case 'equal or smaller':
				$show = ( $value1 <= $value2 ) ? true : false;
				break;
			case '>':
			case 'greater':
				$show = ( $value1 > $value2 ) ? true : false;
				break;
			case '<':
			case 'smaller':
				$show = ( $value1 < $value2 ) ? true : false;
				break;
			case 'contains':
			case 'in':
				if ( is_array( $value1 ) && ! is_array( $value2 ) ) {
					$array  = $value1;
					$string = $value2;
				} elseif ( is_array( $value2 ) && ! is_array( $value1 ) ) {
					$array  = $value2;
					$string = $value1;
				}
				if ( isset( $array ) && isset( $string ) ) {
					if ( ! in_array( $string, $array ) ) {
						$show = false;
					}
				} else {
					if ( false === strrpos( $value1, $value2 ) && false === strpos( $value2, $value1 ) ) {
						$show = false;
					}
				}
				break;
			default:
				$show = ( $value1 == $value2 ) ? true : false;

		} // End switch().

		if ( isset( $show ) ) {
			return $show;
		}

		return true;
	}
}


1			<?php
2			/**
3			* Active callback used with the "required" argument in fields
4			*
5			* @package Kirki
6			* @category Core
7			* @author Aristeides Stathopoulos
8			* @copyright Copyright (c) 2016, Aristeides Stathopoulos
9			* @license http://opensource.org/licenses/https://opensource.org/licenses/MIT
10			*/
11
12			/**
13			* Callback class for use with the "required" argument
14			*/
15			class Kirki_Active_Callback {
16
17			/**
18			* Figure out whether the current object should be displayed or not.
19			*
20			* @param WP_Customize_Setting $object The current field.
21			* @return boolean
22			*/
23			public static function evaluate( $object ) {
24
25			$show = true;
26
27			// Get all fields.
28			$fields = Kirki::$fields;
29
30			// Make sure the current object matches a registered field.
31			if ( ! isset( $object->setting->id ) \|\| ! isset( $fields[ $object->setting->id ] ) ) {
32			return true;
33			}
34
35			$field = $fields[ $object->setting->id ];
36
37			if ( isset( $field['required'] ) ) {
38
39			foreach ( $field['required'] as $requirement ) {
40			// Handles "AND" functionality.
41			$show = self::evaluate_requirement( $object, $field, $requirement );
42			// No need to process further if one requirement returns false.
43			if ( ! $show ) {
44			return false;
45			}
46			}
47			}
48
49			return true;
50
51			}
52
53			/**
54			* Figure out whether the current object should be displayed or not.
55			* We're only parsing a single requirement here from the array of requirements.
56			* This is a proxy function that facilitates evaluating and/or conditions.
57			*
58			* @param WP_Customize_Setting $object The current field.
59			* @param object $field The current object.
60			* @param array $requirement A single requirement.
61			* @return boolean
62			*/
63			private static function evaluate_requirement( $object, $field, $requirement ) {
64
65			// Test for callables first.
66			if ( is_callable( $requirement ) ) {
67			return call_user_func_array( $requirement, array( $field, $object ) );
68			}
69
70			// Look for comparison array.
71			if ( is_array( $requirement ) && isset( $requirement['operator'], $requirement['value'], $requirement['setting'] ) ) {
72
73			if ( isset( $field['option_name'] ) && '' !== $field['option_name'] ) {
74			if ( false === strpos( $requirement['setting'], '[' ) ) {
75			$requirement['setting'] = $field['option_name'] . '[' . $requirement['setting'] . ']';
76			}
77			}
78
79			$current_setting = $object->manager->get_setting( $requirement['setting'] );
80
81			/**
82			* Depending on the 'operator' argument we use,
83			* we'll need to perform the appropriate comparison
84			* and figure out if the control will be shown or not.
85			*/
86			$show = self::compare( $requirement['value'], $current_setting->value(), $requirement['operator'] );
87
88			} else {
89			if ( ! is_array( $requirement ) ) {
90			return true;
91			}
92
93			// Handles "OR" functionality.
94			$show = false;
95			foreach ( $requirement as $sub_requirement ) {
96			$show = self::evaluate_requirement( $object, $field, $sub_requirement );
97			// No need to go on if one sub_requirement returns true.
98			if ( $show ) {
99			return true;
100			}
101			}
102			}
103
104			return $show;
105			}
106
107			/**
108			* Compares the 2 values given the condition
109			*
110			* @param mixed $value1 The 1st value in the comparison.
111			* @param mixed $value2 The 2nd value in the comparison.
112			* @param string $operator The operator we'll use for the comparison.
113			* @return boolean whether The comparison has succeded (true) or failed (false).
114			*/
115			public static function compare( $value1, $value2, $operator ) {
116			switch ( $operator ) {
117			case '===':
118			$show = ( $value1 === $value2 ) ? true : false;
119			break;
120			case '==':
121			case '=':
122			case 'equals':
123			case 'equal':
124			$show = ( $value1 == $value2 ) ? true : false;
125			break;
126			case '!==':
127			$show = ( $value1 !== $value2 ) ? true : false;
128			break;
129			case '!=':
130			case 'not equal':
131			$show = ( $value1 != $value2 ) ? true : false;
132			break;
133			case '>=':
134			case 'greater or equal':
135			case 'equal or greater':
136			$show = ( $value1 >= $value2 ) ? true : false;
137			break;
138			case '<=':
139			case 'smaller or equal':
140			case 'equal or smaller':
141			$show = ( $value1 <= $value2 ) ? true : false;
142			break;
143			case '>':
144			case 'greater':
145			$show = ( $value1 > $value2 ) ? true : false;
146			break;
147			case '<':
148			case 'smaller':
149			$show = ( $value1 < $value2 ) ? true : false;
150			break;
151			case 'contains':
152			case 'in':
153			if ( is_array( $value1 ) && ! is_array( $value2 ) ) {
154			$array = $value1;
155			$string = $value2;
156			} elseif ( is_array( $value2 ) && ! is_array( $value1 ) ) {
157			$array = $value2;
158			$string = $value1;
159			}
160			if ( isset( $array ) && isset( $string ) ) {
161			if ( ! in_array( $string, $array ) ) {
162			$show = false;
163			}
164			} else {
165			if ( false === strrpos( $value1, $value2 ) && false === strpos( $value2, $value1 ) ) {
166			$show = false;
167			}
168			}
169			break;
170			default:
171			$show = ( $value1 == $value2 ) ? true : false;
172
173			} // End switch().
174
175			if ( isset( $show ) ) {
176			return $show;
177			}
178
179			return true;
180			}
181			}
182

aristath / kirki

Push — develop ( d9c9c1...345147 )

Kirki_Active_Callback::compare() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like