Kirki_Field_Typography - Code Metrics - aristath/kirki - Measure and Improve Code Quality continuously with Scrutinizer

Kirki_Field_Typography B
last analyzed 2019-01-15 21:51 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	198
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	79
dl	0
loc	198
rs	8.4
c	1
b	0
f	0
wmc	50

7 Methods

Rating	Name	Size	Complexity
B	set_js_vars()	43	8
A	set_sanitize_callback()	8	2
D	sanitize()	61	30
A	__construct()	3	1
A	set_default()	10	6
A	set_type()	2	1
A	set_choices()	10	2

How to fix Complexity

<?php
/**
 * Override field methods
 *
 * @package     Kirki
 * @subpackage  Controls
 * @copyright   Copyright (c) 2017, Aristeides Stathopoulos
 * @license    https://opensource.org/licenses/MIT
 * @since       2.2.7
 */

/**
 * Field overrides.
 */
class Kirki_Field_Typography extends Kirki_Field {

	/**
	 * Sets the control type.
	 *
	 * @access protected
	 */
	protected function set_type() {
		$this->type = 'kirki-typography';
	}

	/**
	 * The class constructor.
	 * Parses and sanitizes all field arguments.
	 * Then it adds the field to Kirki::$fields.
	 *
	 * @access public
	 * @param string $config_id    The ID of the config we want to use.
	 *                             Defaults to "global".
	 *                             Configs are handled by the Kirki_Config class.
	 * @param array  $args         The arguments of the field.
	 */
	public function __construct( $config_id = 'global', $args = array() ) {
		parent::__construct( $config_id, $args );
		$this->set_default();
	}

	/**
	 * Sets the default value.
	 *
	 * @access protected
	 */
	protected function set_default() {

		// Accomodate the use of font-weight and convert to variant.
		if ( isset( $this->default['font-weight'] ) ) {
			$this->default['variant'] = ( 'regular' === $this->default['font-weight'] ) ? 400 : (string) intval( $this->default['font-weight'] );
		}

		// Make sure letter-spacing has units.
		if ( isset( $this->default['letter-spacing'] ) && is_numeric( $this->default['letter-spacing'] ) && $this->default['letter-spacing'] ) {
			$this->default['letter-spacing'] .= 'px';
		}
	}

	/**
	 * Sets the $sanitize_callback
	 *
	 * @access protected
	 */
	protected function set_sanitize_callback() {

		// If a custom sanitize_callback has been defined,
		// then we don't need to proceed any further.
		if ( ! empty( $this->sanitize_callback ) ) {
			return;
		}
		$this->sanitize_callback = array( __CLASS__, 'sanitize' );
	}

	/**
	 * Sets the $js_vars
	 *
	 * @access protected
	 */
	protected function set_js_vars() {
		if ( ! is_array( $this->js_vars ) ) {

			$this->js_vars = array();
		}

		// Check if transport is set to auto.
		// If not, then skip the auto-calculations and exit early.
		if ( 'auto' !== $this->transport ) {
			return;
		}

		// Set transport to refresh initially.
		// Serves as a fallback in case we failt to auto-calculate js_vars.
		$this->transport = 'refresh';

		$js_vars = array();

		// Try to auto-generate js_vars.
		// First we need to check if js_vars are empty, and that output is not empty.
		if ( ! empty( $this->output ) ) {

			// Start going through each item in the $output array.
			foreach ( $this->output as $output ) {

				// If 'element' or 'property' are not defined, skip this.
				if ( ! isset( $output['element'] ) ) {
					continue;
				}
				if ( is_array( $output['element'] ) ) {
					$output['element'] = implode( ',', $output['element'] );
				}

				// If we got this far, it's safe to add this.
				$js_vars[] = $output;
			}

			// Did we manage to get all the items from 'output'?
			// If not, then we're missing something so don't add this.
			if ( count( $js_vars ) !== count( $this->output ) ) {
				return;
			}
			$this->js_vars   = $js_vars;
			$this->transport = 'postMessage';
		}
	}

	/**
	 * Sanitizes typography controls
	 *
	 * @static
	 * @since 2.2.0
	 * @param array $value The value.
	 * @return array
	 */
	public static function sanitize( $value ) {
		if ( ! is_array( $value ) ) {

			return array();
		}

		foreach ( $value as $key => $val ) {
			switch ( $key ) {
				case 'font-family':
					$value['font-family'] = sanitize_text_field( $val );
					break;
				case 'font-weight':
					if ( isset( $value['variant'] ) ) {
						break;
					}
					$value['variant'] = $val;
					if ( isset( $value['font-style'] ) && 'italic' === $value['font-style'] ) {
						$value['variant'] = ( '400' !== $val || 400 !== $val ) ? $value['variant'] . 'italic' : 'italic';
					}
					break;
				case 'variant':
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;

					// Use 'regular' instead of 400 for font-variant.
					$value['variant'] = ( 400 === $val || '400' === $val ) ? 'regular' : $val;

					// Get font-weight from variant.
					$value['font-weight'] = filter_var( $value['variant'], FILTER_SANITIZE_NUMBER_FLOAT, FILTER_FLAG_ALLOW_FRACTION );
					$value['font-weight'] = ( 'regular' === $value['variant'] || 'italic' === $value['variant'] ) ? 400 : absint( $value['font-weight'] );

					// Get font-style from variant.
					if ( ! isset( $value['font-style'] ) ) {
						$value['font-style'] = ( false === strpos( $value['variant'], 'italic' ) ) ? 'normal' : 'italic';
					}
					break;
				case 'font-size':
				case 'letter-spacing':
				case 'word-spacing':
				case 'line-height':
					$value[ $key ] = '' === trim( $value[ $key ] ) ? '' : sanitize_text_field( $val );
					break;
				case 'text-align':
					if ( ! in_array( $val, array( '', 'inherit', 'left', 'center', 'right', 'justify' ), true ) ) {
						$value['text-align'] = '';
					}
					break;
				case 'text-transform':
					if ( ! in_array( $val, array( '', 'none', 'capitalize', 'uppercase', 'lowercase', 'initial', 'inherit' ), true ) ) {
						$value['text-transform'] = '';
					}
					break;
				case 'text-decoration':
					if ( ! in_array( $val, array( '', 'none', 'underline', 'overline', 'line-through', 'initial', 'inherit' ), true ) ) {
						$value['text-transform'] = '';
					}
					break;
				case 'color':
					$value['color'] = '' === $value['color'] ? '' : ariColor::newColor( $val )->toCSS( 'hex' );
					break;
			}
		}

		return $value;
	}

	/**
	 * Sets the $choices
	 *
	 * @access protected
	 * @since 3.0.0
	 */
	protected function set_choices() {
		if ( ! is_array( $this->choices ) ) {

			$this->choices = array();
		}
		$this->choices = wp_parse_args(
			$this->choices, array(
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
);
				'variant' => array(),
				'fonts'   => array(
					'standard' => array(),
					'google'   => array(),
				),
			)
		);
	}
}


1			<?php
2			/**
3			* Override field methods
4			*
5			* @package Kirki
6			* @subpackage Controls
7			* @copyright Copyright (c) 2017, Aristeides Stathopoulos
8			* @license https://opensource.org/licenses/MIT
9			* @since 2.2.7
10			*/
11
12			/**
13			* Field overrides.
14			*/
15			class Kirki_Field_Typography extends Kirki_Field {
16
17			/**
18			* Sets the control type.
19			*
20			* @access protected
21			*/
22			protected function set_type() {
23			$this->type = 'kirki-typography';
24			}
25
26			/**
27			* The class constructor.
28			* Parses and sanitizes all field arguments.
29			* Then it adds the field to Kirki::$fields.
30			*
31			* @access public
32			* @param string $config_id The ID of the config we want to use.
33			* Defaults to "global".
34			* Configs are handled by the Kirki_Config class.
35			* @param array $args The arguments of the field.
36			*/
37			public function __construct( $config_id = 'global', $args = array() ) {
38			parent::__construct( $config_id, $args );
39			$this->set_default();
40			}
41
42			/**
43			* Sets the default value.
44			*
45			* @access protected
46			*/
47			protected function set_default() {
48
49			// Accomodate the use of font-weight and convert to variant.
50			if ( isset( $this->default['font-weight'] ) ) {
51			$this->default['variant'] = ( 'regular' === $this->default['font-weight'] ) ? 400 : (string) intval( $this->default['font-weight'] );
52			}
53
54			// Make sure letter-spacing has units.
55			if ( isset( $this->default['letter-spacing'] ) && is_numeric( $this->default['letter-spacing'] ) && $this->default['letter-spacing'] ) {
56			$this->default['letter-spacing'] .= 'px';
57			}
58			}
59
60			/**
61			* Sets the $sanitize_callback
62			*
63			* @access protected
64			*/
65			protected function set_sanitize_callback() {
66
67			// If a custom sanitize_callback has been defined,
68			// then we don't need to proceed any further.
69			if ( ! empty( $this->sanitize_callback ) ) {
70			return;
71			}
72			$this->sanitize_callback = array( __CLASS__, 'sanitize' );
73			}
74
75			/**
76			* Sets the $js_vars
77			*
78			* @access protected
79			*/
80			protected function set_js_vars() {
81			if ( ! is_array( $this->js_vars ) ) {
			0 ignored issues – show introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report The condition `is_array($this->js_vars)` is always `true`. Loading history...
82			$this->js_vars = array();
83			}
84
85			// Check if transport is set to auto.
86			// If not, then skip the auto-calculations and exit early.
87			if ( 'auto' !== $this->transport ) {
88			return;
89			}
90
91			// Set transport to refresh initially.
92			// Serves as a fallback in case we failt to auto-calculate js_vars.
93			$this->transport = 'refresh';
94
95			$js_vars = array();
96
97			// Try to auto-generate js_vars.
98			// First we need to check if js_vars are empty, and that output is not empty.
99			if ( ! empty( $this->output ) ) {
100
101			// Start going through each item in the $output array.
102			foreach ( $this->output as $output ) {
103
104			// If 'element' or 'property' are not defined, skip this.
105			if ( ! isset( $output['element'] ) ) {
106			continue;
107			}
108			if ( is_array( $output['element'] ) ) {
109			$output['element'] = implode( ',', $output['element'] );
110			}
111
112			// If we got this far, it's safe to add this.
113			$js_vars[] = $output;
114			}
115
116			// Did we manage to get all the items from 'output'?
117			// If not, then we're missing something so don't add this.
118			if ( count( $js_vars ) !== count( $this->output ) ) {
119			return;
120			}
121			$this->js_vars = $js_vars;
122			$this->transport = 'postMessage';
123			}
124			}
125
126			/**
127			* Sanitizes typography controls
128			*
129			* @static
130			* @since 2.2.0
131			* @param array $value The value.
132			* @return array
133			*/
134			public static function sanitize( $value ) {
135			if ( ! is_array( $value ) ) {
			0 ignored issues – show introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report The condition `is_array($value)` is always `true`. Loading history...
136			return array();
137			}
138
139			foreach ( $value as $key => $val ) {
140			switch ( $key ) {
141			case 'font-family':
142			$value['font-family'] = sanitize_text_field( $val );
143			break;
144			case 'font-weight':
145			if ( isset( $value['variant'] ) ) {
146			break;
147			}
148			$value['variant'] = $val;
149			if ( isset( $value['font-style'] ) && 'italic' === $value['font-style'] ) {
150			$value['variant'] = ( '400' !== $val \|\| 400 !== $val ) ? $value['variant'] . 'italic' : 'italic';
151			}
152			break;
153			case 'variant':
			0 ignored issues – show Coding Style introduced 2018-10-31 19:34 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
154
155			// Use 'regular' instead of 400 for font-variant.
156			$value['variant'] = ( 400 === $val \|\| '400' === $val ) ? 'regular' : $val;
157
158			// Get font-weight from variant.
159			$value['font-weight'] = filter_var( $value['variant'], FILTER_SANITIZE_NUMBER_FLOAT, FILTER_FLAG_ALLOW_FRACTION );
160			$value['font-weight'] = ( 'regular' === $value['variant'] \|\| 'italic' === $value['variant'] ) ? 400 : absint( $value['font-weight'] );
161
162			// Get font-style from variant.
163			if ( ! isset( $value['font-style'] ) ) {
164			$value['font-style'] = ( false === strpos( $value['variant'], 'italic' ) ) ? 'normal' : 'italic';
165			}
166			break;
167			case 'font-size':
168			case 'letter-spacing':
169			case 'word-spacing':
170			case 'line-height':
171			$value[ $key ] = '' === trim( $value[ $key ] ) ? '' : sanitize_text_field( $val );
172			break;
173			case 'text-align':
174			if ( ! in_array( $val, array( '', 'inherit', 'left', 'center', 'right', 'justify' ), true ) ) {
175			$value['text-align'] = '';
176			}
177			break;
178			case 'text-transform':
179			if ( ! in_array( $val, array( '', 'none', 'capitalize', 'uppercase', 'lowercase', 'initial', 'inherit' ), true ) ) {
180			$value['text-transform'] = '';
181			}
182			break;
183			case 'text-decoration':
184			if ( ! in_array( $val, array( '', 'none', 'underline', 'overline', 'line-through', 'initial', 'inherit' ), true ) ) {
185			$value['text-transform'] = '';
186			}
187			break;
188			case 'color':
189			$value['color'] = '' === $value['color'] ? '' : ariColor::newColor( $val )->toCSS( 'hex' );
190			break;
191			}
192			}
193
194			return $value;
195			}
196
197			/**
198			* Sets the $choices
199			*
200			* @access protected
201			* @since 3.0.0
202			*/
203			protected function set_choices() {
204			if ( ! is_array( $this->choices ) ) {
			0 ignored issues – show introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report The condition `is_array($this->choices)` is always `true`. Loading history...
205			$this->choices = array();
206			}
207			$this->choices = wp_parse_args(
208			$this->choices, array(
			0 ignored issues – show Coding Style introduced 2018-10-23 18:27 UTC by Report Bug Copy Issue Report For multi-line function calls, each argument should be on a separate line. For a function calls that spawns multiple lines, the coding style suggests to split arguments to separate lines like this: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); Loading history...
209			'variant' => array(),
210			'fonts' => array(
211			'standard' => array(),
212			'google' => array(),
213			),
214			)
215			);
216			}
217			}
218

aristath / kirki

Kirki_Field_Typography B last analyzed 2019-01-15 21:51 UTC

Complexity

Size/Duplication

Importance

7 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like

Kirki_Field_Typography B
last analyzed 2019-01-15 21:51 UTC