aristath / kirki

modules/css/class-kirki-output.php

Braces +2 added lines, -1 removed lines

@@ -217,7 +217,8 @@
 						}
 						// If 'choice' is defined check for sub-values too.
 						// Fixes https://github.com/aristath/kirki/issues/1416.
-						if ( isset( $output['choice'] ) && isset( $value[ $output['choice'] ] ) && $exclude == $value[ $output['choice'] ] ) { // phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison
+						if ( isset( $output['choice'] ) && isset( $value[ $output['choice'] ] ) && $exclude == $value[ $output['choice'] ] ) {
+// phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison
 							$skip = true;
 						}
 					}

Spacing +30 added lines, -30 removed lines

@@ -83,13 +83,13 @@ 
 	 */
 	protected function apply_sanitize_callback( $output, $value ) {
 
-		if ( isset( $output['sanitize_callback'] ) && null !== $output['sanitize_callback'] ) {
+		if ( isset( $output[ 'sanitize_callback' ] ) && null !== $output[ 'sanitize_callback' ] ) {
 
 			// If the sanitize_callback is invalid, return the value.
-			if ( ! is_callable( $output['sanitize_callback'] ) ) {
+			if ( ! is_callable( $output[ 'sanitize_callback' ] ) ) {
 				return $value;
 			}
-			return call_user_func( $output['sanitize_callback'], $this->value );
+			return call_user_func( $output[ 'sanitize_callback' ], $this->value );
 		}
 
 		return $value;
@@ -105,22 +105,22 @@ 
 	 */
 	protected function apply_value_pattern( $output, $value ) {
 
-		if ( isset( $output['value_pattern'] ) && ! empty( $output['value_pattern'] ) && is_string( $output['value_pattern'] ) ) {
+		if ( isset( $output[ 'value_pattern' ] ) && ! empty( $output[ 'value_pattern' ] ) && is_string( $output[ 'value_pattern' ] ) ) {
 			if ( ! is_array( $value ) ) {
-				$value = str_replace( '$', $value, $output['value_pattern'] );
+				$value = str_replace( '$', $value, $output[ 'value_pattern' ] );
 			}
 			if ( is_array( $value ) ) {
 				foreach ( array_keys( $value ) as $value_k ) {
 					if ( ! is_string( $value[ $value_k ] ) ) {
 						continue;
 					}
-					if ( isset( $output['choice'] ) ) {
-						if ( $output['choice'] === $value_k ) {
-							$value[ $output['choice'] ] = str_replace( '$', $value[ $output['choice'] ], $output['value_pattern'] );
+					if ( isset( $output[ 'choice' ] ) ) {
+						if ( $output[ 'choice' ] === $value_k ) {
+							$value[ $output[ 'choice' ] ] = str_replace( '$', $value[ $output[ 'choice' ] ], $output[ 'value_pattern' ] );
 						}
 						continue;
 					}
-					$value[ $value_k ] = str_replace( '$', $value[ $value_k ], $output['value_pattern'] );
+					$value[ $value_k ] = str_replace( '$', $value[ $value_k ], $output[ 'value_pattern' ] );
 				}
 			}
 			$value = $this->apply_pattern_replace( $output, $value );
@@ -136,14 +136,14 @@ 
 	 * @return string|array
 	 */
 	protected function apply_pattern_replace( $output, $value ) {
-		if ( isset( $output['pattern_replace'] ) && is_array( $output['pattern_replace'] ) ) {
+		if ( isset( $output[ 'pattern_replace' ] ) && is_array( $output[ 'pattern_replace' ] ) ) {
 			$option_type = ( '' !== Kirki::get_config_param( $this->config_id, 'option_type' ) ) ? Kirki::get_config_param( $this->config_id, 'option_type' ) : 'theme_mod';
 			$option_name = Kirki::get_config_param( $this->config_id, 'option_name' );
 			$options     = array();
 			if ( $option_name ) {
 				$options = ( 'site_option' === $option_type ) ? get_site_option( $option_name ) : get_option( $option_name );
 			}
-			foreach ( $output['pattern_replace'] as $search => $replace ) {
+			foreach ( $output[ 'pattern_replace' ] as $search => $replace ) {
 				$replacement = '';
 				switch ( $option_type ) {
 					case 'option':
@@ -203,8 +203,8 @@ 
 			}
 
 			// No need to proceed this if the current value is the same as in the "exclude" value.
-			if ( isset( $output['exclude'] ) && is_array( $output['exclude'] ) ) {
-				foreach ( $output['exclude'] as $exclude ) {
+			if ( isset( $output[ 'exclude' ] ) && is_array( $output[ 'exclude' ] ) ) {
+				foreach ( $output[ 'exclude' ] as $exclude ) {
 					if ( is_array( $value ) ) {
 						if ( is_array( $exclude ) ) {
 							$diff1 = array_diff( $value, $exclude );
@@ -216,7 +216,7 @@ 
 						}
 						// If 'choice' is defined check for sub-values too.
 						// Fixes https://github.com/aristath/kirki/issues/1416.
-						if ( isset( $output['choice'] ) && isset( $value[ $output['choice'] ] ) && $exclude == $value[ $output['choice'] ] ) { // phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison
+						if ( isset( $output[ 'choice' ] ) && isset( $value[ $output[ 'choice' ] ] ) && $exclude == $value[ $output[ 'choice' ] ] ) { // phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison
 							$skip = true;
 						}
 					}
@@ -237,10 +237,10 @@ 
 			// Apply any value patterns defined.
 			$value = $this->apply_value_pattern( $output, $value );
 
-			if ( isset( $output['element'] ) && is_array( $output['element'] ) ) {
-				$output['element'] = array_unique( $output['element'] );
-				sort( $output['element'] );
-				$output['element'] = implode( ',', $output['element'] );
+			if ( isset( $output[ 'element' ] ) && is_array( $output[ 'element' ] ) ) {
+				$output[ 'element' ] = array_unique( $output[ 'element' ] );
+				sort( $output[ 'element' ] );
+				$output[ 'element' ] = implode( ',', $output[ 'element' ] );
 			}
 
 			$value = $this->process_value( $value, $output );
@@ -258,13 +258,13 @@ 
 	 * @return null
 	 */
 	protected function process_output( $output, $value ) {
-		if ( ! isset( $output['element'] ) || ! isset( $output['property'] ) ) {
+		if ( ! isset( $output[ 'element' ] ) || ! isset( $output[ 'property' ] ) ) {
 			return;
 		}
-		$output['media_query'] = ( isset( $output['media_query'] ) ) ? $output['media_query'] : 'global';
-		$output['prefix']      = ( isset( $output['prefix'] ) ) ? $output['prefix'] : '';
-		$output['units']       = ( isset( $output['units'] ) ) ? $output['units'] : '';
-		$output['suffix']      = ( isset( $output['suffix'] ) ) ? $output['suffix'] : '';
+		$output[ 'media_query' ] = ( isset( $output[ 'media_query' ] ) ) ? $output[ 'media_query' ] : 'global';
+		$output[ 'prefix' ]      = ( isset( $output[ 'prefix' ] ) ) ? $output[ 'prefix' ] : '';
+		$output[ 'units' ]       = ( isset( $output[ 'units' ] ) ) ? $output[ 'units' ] : '';
+		$output[ 'suffix' ]      = ( isset( $output[ 'suffix' ] ) ) ? $output[ 'suffix' ] : '';
 
 		// Properties that can accept multiple values.
 		// Useful for example for gradients where all browsers use the "background-image" property
@@ -273,15 +273,15 @@ 
 			'background-image',
 			'background',
 		);
-		if ( in_array( $output['property'], $accepts_multiple, true ) ) {
-			if ( isset( $this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] ) && ! is_array( $this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] ) ) {
-				$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] = (array) $this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ];
+		if ( in_array( $output[ 'property' ], $accepts_multiple, true ) ) {
+			if ( isset( $this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] ) && ! is_array( $this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] ) ) {
+				$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] = (array) $this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ];
 			}
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ][] = $output['prefix'] . $value . $output['units'] . $output['suffix'];
+			$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ][ ] = $output[ 'prefix' ] . $value . $output[ 'units' ] . $output[ 'suffix' ];
 			return;
 		}
 		if ( is_string( $value ) || is_numeric( $value ) ) {
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] = $output['prefix'] . $this->process_property_value( $output['property'], $value ) . $output['units'] . $output['suffix'];
+			$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] = $output[ 'prefix' ] . $this->process_property_value( $output[ 'property' ], $value ) . $output[ 'units' ] . $output[ 'suffix' ];
 		}
 	}
 
@@ -320,8 +320,8 @@ 
 	 * @return string|array
 	 */
 	protected function process_value( $value, $output ) {
-		if ( isset( $output['property'] ) ) {
-			return $this->process_property_value( $output['property'], $value );
+		if ( isset( $output[ 'property' ] ) ) {
+			return $this->process_property_value( $output[ 'property' ], $value );
 		}
 		return $value;
 	}

modules/webfonts/class-kirki-fonts-google-local.php

Spacing +5 added lines, -5 removed lines

@@ -203,13 +203,13 @@ 
 		}
 
 		// Return regular if the one we want could not be found.
-		if ( isset( $local_urls['regular'] ) ) {
-			return $local_urls['regular'];
+		if ( isset( $local_urls[ 'regular' ] ) ) {
+			return $local_urls[ 'regular' ];
 		}
 
 		// Return the first available if all else failed.
 		$vals = array_values( $local_urls );
-		return $vals[0];
+		return $vals[ 0 ];
 	}
 
 	/**
@@ -401,7 +401,7 @@ 
 	public function get_root_path() {
 		// Get the upload directory for this site.
 		$upload_dir = wp_upload_dir();
-		$path       = untrailingslashit( wp_normalize_path( $upload_dir['basedir'] ) ) . '/webfonts';
+		$path       = untrailingslashit( wp_normalize_path( $upload_dir[ 'basedir' ] ) ) . '/webfonts';
 
 		// If the folder doesn't exist, create it.
 		if ( ! file_exists( $path ) ) {
@@ -426,7 +426,7 @@ 
 		$upload_dir = wp_upload_dir();
 
 		// The URL.
-		$url = trailingslashit( $upload_dir['baseurl'] );
+		$url = trailingslashit( $upload_dir[ 'baseurl' ] );
 		// Take care of domain mapping.
 		// When using domain mapping we have to make sure that the URL to the file
 		// does not include the original domain but instead the mapped domain.

Braces +2 added lines, -1 removed lines

@@ -453,7 +453,8 @@
 			$variants = array_keys( $this->files );
 		}
 		foreach ( $this->files as $variant => $file ) {
-			if ( in_array( $variant, $variants ) ) { // phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict
+			if ( in_array( $variant, $variants ) ) {
+// phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict
 				$this->download_font_file( $file );
 			}
 		}

modules/css/field/class-kirki-output-field-background.php

Spacing +3 added lines, -3 removed lines

@@ -36,12 +36,12 @@
 		foreach ( array( 'background-image', 'background-color', 'background-repeat', 'background-position', 'background-size', 'background-attachment' ) as $property ) {
 
 			// See https://github.com/aristath/kirki/issues/1808.
-			if ( 'background-color' === $property && isset( $value['background-color'] ) && $value['background-color'] && ( ! isset( $value['background-image'] ) || empty( $value['background-image'] ) ) ) {
-				$this->styles[ $output['media_query'] ][ $output['element'] ]['background'] = $output['prefix'] . $this->process_property_value( $property, $value[ $property ] ) . $output['suffix'];
+			if ( 'background-color' === $property && isset( $value[ 'background-color' ] ) && $value[ 'background-color' ] && ( ! isset( $value[ 'background-image' ] ) || empty( $value[ 'background-image' ] ) ) ) {
+				$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ 'background' ] = $output[ 'prefix' ] . $this->process_property_value( $property, $value[ $property ] ) . $output[ 'suffix' ];
 			}
 
 			if ( isset( $value[ $property ] ) && ! empty( $value[ $property ] ) ) {
-				$this->styles[ $output['media_query'] ][ $output['element'] ][ $property ] = $output['prefix'] . $this->process_property_value( $property, $value[ $property ] ) . $output['suffix'];
+				$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $property ] = $output[ 'prefix' ] . $this->process_property_value( $property, $value[ $property ] ) . $output[ 'suffix' ];
 			}
 		}
 	}

core/class-kirki-field.php

Spacing +34 added lines, -34 removed lines

@@ -243,28 +243,28 @@ 
 	 */
 	public function __construct( $config_id = 'global', $args = array() ) {
 
-		if ( isset( $args['setting'] ) && ! empty( $args['setting'] ) && ( ! isset( $args['settings'] ) || empty( $args['settings'] ) ) ) {
+		if ( isset( $args[ 'setting' ] ) && ! empty( $args[ 'setting' ] ) && ( ! isset( $args[ 'settings' ] ) || empty( $args[ 'settings' ] ) ) ) {
 			/* translators: %s represents the field ID where the error occurs. */
-			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( 'Typo found in field %s - setting instead of settings.', 'kirki' ), esc_attr( $args['settings'] ) ), '3.0.10' );
-			$args['settings'] = $args['setting'];
-			unset( $args['setting'] );
+			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( 'Typo found in field %s - setting instead of settings.', 'kirki' ), esc_attr( $args[ 'settings' ] ) ), '3.0.10' );
+			$args[ 'settings' ] = $args[ 'setting' ];
+			unset( $args[ 'setting' ] );
 		}
 
 		// In case the user only provides 1 argument,
 		// assume that the provided argument is $args and set $config_id = 'global'.
 		if ( is_array( $config_id ) && empty( $args ) ) {
 			/* translators: %1$s represents the field ID where the error occurs. %2$s is the URL in the documentation site. */
-			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( 'Config not defined for field %1$s - See %2$s for details on how to properly add fields.', 'kirki' ), esc_attr( $args['settings'] ), 'https://aristath.github.io/kirki/docs/getting-started/fields.html' ), '3.0.10' );
+			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( 'Config not defined for field %1$s - See %2$s for details on how to properly add fields.', 'kirki' ), esc_attr( $args[ 'settings' ] ), 'https://aristath.github.io/kirki/docs/getting-started/fields.html' ), '3.0.10' );
 			$args      = $config_id;
 			$config_id = 'global';
 		}
 
-		$args['kirki_config'] = $config_id;
+		$args[ 'kirki_config' ] = $config_id;
 
 		$this->kirki_config = trim( esc_attr( $config_id ) );
 		if ( '' === $config_id ) {
 			/* translators: %1$s represents the field ID where the error occurs. %2$s is the URL in the documentation site. */
-			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( 'Config not defined for field %1$s - See %2$s for details on how to properly add fields.', 'kirki' ), esc_attr( $args['settings'] ), 'https://aristath.github.io/kirki/docs/getting-started/fields.html' ), '3.0.10' );
+			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( 'Config not defined for field %1$s - See %2$s for details on how to properly add fields.', 'kirki' ), esc_attr( $args[ 'settings' ] ), 'https://aristath.github.io/kirki/docs/getting-started/fields.html' ), '3.0.10' );
 			$this->kirki_config = 'global';
 		}
 
@@ -272,7 +272,7 @@ 
 		$defaults = get_class_vars( __CLASS__ );
 
 		// Get the config arguments, and merge them with the defaults.
-		$config_defaults = ( isset( Kirki::$config['global'] ) ) ? Kirki::$config['global'] : array();
+		$config_defaults = ( isset( Kirki::$config[ 'global' ] ) ) ? Kirki::$config[ 'global' ] : array();
 		if ( 'global' !== $this->kirki_config && isset( Kirki::$config[ $this->kirki_config ] ) ) {
 			$config_defaults = Kirki::$config[ $this->kirki_config ];
 		}
@@ -420,7 +420,7 @@ 
 			$this->partial_refresh = array();
 		}
 		foreach ( $this->partial_refresh as $id => $args ) {
-			if ( ! is_array( $args ) || ! isset( $args['selector'] ) || ! isset( $args['render_callback'] ) || ! is_callable( $args['render_callback'] ) ) {
+			if ( ! is_array( $args ) || ! isset( $args[ 'selector' ] ) || ! isset( $args[ 'render_callback' ] ) || ! is_callable( $args[ 'render_callback' ] ) ) {
 				/* translators: %1$s represents the field ID where the error occurs. */
 				_doing_it_wrong( __METHOD__, sprintf( esc_attr__( '"partial_refresh" invalid entry in field %s', 'kirki' ), esc_attr( $this->settings ) ), '3.0.10' );
 				unset( $this->partial_refresh[ $id ] );
@@ -458,8 +458,8 @@ 
 			}
 		}
 		$this->settings = $settings;
-		if ( isset( $this->settings['kirki_placeholder_setting'] ) ) {
-			$this->settings = $this->settings['kirki_placeholder_setting'];
+		if ( isset( $this->settings[ 'kirki_placeholder_setting' ] ) ) {
+			$this->settings = $this->settings[ 'kirki_placeholder_setting' ];
 		}
 	}
 
@@ -481,7 +481,7 @@ 
 						unset( $this->active_callback[ $key ] );
 					}
 				}
-				if ( isset( $this->active_callback[0] ) ) {
+				if ( isset( $this->active_callback[ 0 ] ) ) {
 					$this->required = $this->active_callback;
 				}
 			}
@@ -568,34 +568,34 @@ 
 			);
 		}
 		// Convert to array of arrays if needed.
-		if ( isset( $this->output['element'] ) ) {
+		if ( isset( $this->output[ 'element' ] ) ) {
 			/* translators: The field ID where the error occurs. */
 			_doing_it_wrong( __METHOD__, sprintf( esc_attr__( '"output" invalid format in field %s. The "output" argument should be defined as an array of arrays.', 'kirki' ), esc_attr( $this->settings ) ), '3.0.10' );
 			$this->output = array( $this->output );
 		}
 		foreach ( $this->output as $key => $output ) {
-			if ( empty( $output ) || ! isset( $output['element'] ) ) {
+			if ( empty( $output ) || ! isset( $output[ 'element' ] ) ) {
 				unset( $this->output[ $key ] );
 				continue;
 			}
-			if ( ! isset( $output['sanitize_callback'] ) && isset( $output['callback'] ) ) {
-				$this->output[ $key ]['sanitize_callback'] = $output['callback'];
+			if ( ! isset( $output[ 'sanitize_callback' ] ) && isset( $output[ 'callback' ] ) ) {
+				$this->output[ $key ][ 'sanitize_callback' ] = $output[ 'callback' ];
 			}
 			// Convert element arrays to strings.
-			if ( isset( $output['element'] ) && is_array( $output['element'] ) ) {
-				$this->output[ $key ]['element'] = array_unique( $this->output[ $key ]['element'] );
-				sort( $this->output[ $key ]['element'] );
+			if ( isset( $output[ 'element' ] ) && is_array( $output[ 'element' ] ) ) {
+				$this->output[ $key ][ 'element' ] = array_unique( $this->output[ $key ][ 'element' ] );
+				sort( $this->output[ $key ][ 'element' ] );
 
 				// Trim each element in the array.
-				foreach ( $this->output[ $key ]['element'] as $index => $element ) {
-					$this->output[ $key ]['element'][ $index ] = trim( $element );
+				foreach ( $this->output[ $key ][ 'element' ] as $index => $element ) {
+					$this->output[ $key ][ 'element' ][ $index ] = trim( $element );
 				}
-				$this->output[ $key ]['element'] = implode( ',', $this->output[ $key ]['element'] );
+				$this->output[ $key ][ 'element' ] = implode( ',', $this->output[ $key ][ 'element' ] );
 			}
 
 			// Fix for https://github.com/aristath/kirki/issues/1659#issuecomment-346229751.
-			$this->output[ $key ]['element'] = str_replace( array( "\t", "\n", "\r", "\0", "\x0B" ), ' ', $this->output[ $key ]['element'] );
-			$this->output[ $key ]['element'] = trim( preg_replace( '/\s+/', ' ', $this->output[ $key ]['element'] ) );
+			$this->output[ $key ][ 'element' ] = str_replace( array( "\t", "\n", "\r", "\0", "\x0B" ), ' ', $this->output[ $key ][ 'element' ] );
+			$this->output[ $key ][ 'element' ] = trim( preg_replace( '/\s+/', ' ', $this->output[ $key ][ 'element' ] ) );
 		}
 	}
 
@@ -628,23 +628,23 @@ 
 
 			// Start going through each item in the $output array.
 			foreach ( $this->output as $output ) {
-				$output['function'] = ( isset( $output['function'] ) ) ? $output['function'] : 'style';
+				$output[ 'function' ] = ( isset( $output[ 'function' ] ) ) ? $output[ 'function' ] : 'style';
 
 				// If 'element' or 'property' are not defined, skip this.
-				if ( ! isset( $output['element'] ) || ! isset( $output['property'] ) ) {
+				if ( ! isset( $output[ 'element' ] ) || ! isset( $output[ 'property' ] ) ) {
 					continue;
 				}
-				if ( is_array( $output['element'] ) ) {
-					$output['element'] = implode( ',', $output['element'] );
+				if ( is_array( $output[ 'element' ] ) ) {
+					$output[ 'element' ] = implode( ',', $output[ 'element' ] );
 				}
 
 				// If there's a sanitize_callback defined skip this, unless we also have a js_callback defined.
-				if ( isset( $output['sanitize_callback'] ) && ! empty( $output['sanitize_callback'] ) && ! isset( $output['js_callback'] ) ) {
+				if ( isset( $output[ 'sanitize_callback' ] ) && ! empty( $output[ 'sanitize_callback' ] ) && ! isset( $output[ 'js_callback' ] ) ) {
 					continue;
 				}
 
 				// If we got this far, it's safe to add this.
-				$js_vars[] = $output;
+				$js_vars[ ] = $output;
 			}
 
 			// Did we manage to get all the items from 'output'?
@@ -669,7 +669,7 @@ 
 			$variable        = ( is_string( $this->variables ) && ! empty( $this->variables ) ) ? $this->variables : false;
 			$this->variables = array();
 			if ( $variable && empty( $this->variables ) ) {
-				$this->variables[0]['name'] = $variable;
+				$this->variables[ 0 ][ 'name' ] = $variable;
 			}
 		}
 	}
@@ -717,12 +717,12 @@ 
 		if ( is_string( $this->css_vars ) ) {
 			$this->css_vars = array( $this->css_vars );
 		}
-		if ( isset( $this->css_vars[0] ) && is_string( $this->css_vars[0] ) ) {
+		if ( isset( $this->css_vars[ 0 ] ) && is_string( $this->css_vars[ 0 ] ) ) {
 			$this->css_vars = array( $this->css_vars );
 		}
 		foreach ( $this->css_vars as $key => $val ) {
-			if ( ! isset( $val[1] ) ) {
-				$this->css_vars[ $key ][1] = '$';
+			if ( ! isset( $val[ 1 ] ) ) {
+				$this->css_vars[ $key ][ 1 ] = '$';
 			}
 		}
 	}

core/class-kirki-util.php

Spacing +22 added lines, -22 removed lines

@@ -45,7 +45,7 @@ 
 		$plugins = get_plugins();
 		$_plugin = '';
 		foreach ( $plugins as $plugin => $args ) {
-			if ( ! $is_plugin && isset( $args['Name'] ) && ( 'Kirki' === $args['Name'] || 'Kirki Toolkit' === $args['Name'] ) ) {
+			if ( ! $is_plugin && isset( $args[ 'Name' ] ) && ( 'Kirki' === $args[ 'Name' ] || 'Kirki Toolkit' === $args[ 'Name' ] ) ) {
 				$is_plugin = true;
 				$_plugin   = $plugin;
 			}
@@ -82,26 +82,26 @@ 
 		foreach ( Kirki::$fields as $field ) {
 
 			// Check if we have variables for this field.
-			if ( isset( $field['variables'] ) && $field['variables'] && ! empty( $field['variables'] ) ) {
+			if ( isset( $field[ 'variables' ] ) && $field[ 'variables' ] && ! empty( $field[ 'variables' ] ) ) {
 
 				// Loop through the array of variables.
-				foreach ( $field['variables'] as $field_variable ) {
+				foreach ( $field[ 'variables' ] as $field_variable ) {
 
 					// Is the variable ['name'] defined? If yes, then we can proceed.
-					if ( isset( $field_variable['name'] ) ) {
+					if ( isset( $field_variable[ 'name' ] ) ) {
 
 						// Sanitize the variable name.
-						$variable_name = esc_attr( $field_variable['name'] );
+						$variable_name = esc_attr( $field_variable[ 'name' ] );
 
 						// Do we have a callback function defined? If not then set $variable_callback to false.
-						$variable_callback = ( isset( $field_variable['callback'] ) && is_callable( $field_variable['callback'] ) ) ? $field_variable['callback'] : false;
+						$variable_callback = ( isset( $field_variable[ 'callback' ] ) && is_callable( $field_variable[ 'callback' ] ) ) ? $field_variable[ 'callback' ] : false;
 
 						// If we have a variable_callback defined then get the value of the option
 						// and run it through the callback function.
 						// If no callback is defined (false) then just get the value.
-						$variables[ $variable_name ] = Kirki_Values::get_value( $field['settings'] );
+						$variables[ $variable_name ] = Kirki_Values::get_value( $field[ 'settings' ] );
 						if ( $variable_callback ) {
-							$variables[ $variable_name ] = call_user_func( $field_variable['callback'], Kirki_Values::get_value( $field['settings'] ) );
+							$variables[ $variable_name ] = call_user_func( $field_variable[ 'callback' ], Kirki_Values::get_value( $field[ 'settings' ] ) );
 						}
 					}
 				}
@@ -128,31 +128,31 @@ 
 		if (
 			self::is_plugin() ||
 			false === strpos( $url, 'wordpress.org' ) || (
-				! isset( $request['body'] ) ||
-				! isset( $request['body']['plugins'] ) ||
-				! isset( $request['body']['translations'] ) ||
-				! isset( $request['body']['locale'] ) ||
-				! isset( $request['body']['all'] )
+				! isset( $request[ 'body' ] ) ||
+				! isset( $request[ 'body' ][ 'plugins' ] ) ||
+				! isset( $request[ 'body' ][ 'translations' ] ) ||
+				! isset( $request[ 'body' ][ 'locale' ] ) ||
+				! isset( $request[ 'body' ][ 'all' ] )
 			)
 		) {
 			return $request;
 		}
 
-		$plugins = json_decode( $request['body']['plugins'], true );
-		if ( ! isset( $plugins['plugins'] ) ) {
+		$plugins = json_decode( $request[ 'body' ][ 'plugins' ], true );
+		if ( ! isset( $plugins[ 'plugins' ] ) ) {
 			return $request;
 		}
 		$exists = false;
-		foreach ( $plugins['plugins'] as $plugin ) {
-			if ( isset( $plugin['Name'] ) && 'Kirki Toolkit' === $plugin['Name'] ) {
+		foreach ( $plugins[ 'plugins' ] as $plugin ) {
+			if ( isset( $plugin[ 'Name' ] ) && 'Kirki Toolkit' === $plugin[ 'Name' ] ) {
 				$exists = true;
 			}
 		}
 		// Inject data.
 		if ( ! $exists && defined( 'KIRKI_PLUGIN_FILE' ) ) {
-			$plugins['plugins']['kirki/kirki.php'] = get_plugin_data( KIRKI_PLUGIN_FILE );
+			$plugins[ 'plugins' ][ 'kirki/kirki.php' ] = get_plugin_data( KIRKI_PLUGIN_FILE );
 		}
-		$request['body']['plugins'] = wp_json_encode( $plugins );
+		$request[ 'body' ][ 'plugins' ] = wp_json_encode( $plugins );
 		return $request;
 	}
 
@@ -172,7 +172,7 @@ 
 		// We only need the major version.
 		if ( 'major' === $context ) {
 			$version_parts = explode( '.', $wp_version );
-			return $version_parts[0];
+			return $version_parts[ 0 ];
 		}
 
 		return $wp_version;
@@ -194,7 +194,7 @@ 
 		// We only need the major version.
 		if ( 'major' === $context ) {
 			$version_parts = explode( '.', $wp_version );
-			return absint( $version_parts[0] );
+			return absint( $version_parts[ 0 ] );
 		}
 
 		// If we got this far, we want the full monty.
@@ -202,7 +202,7 @@ 
 		if ( false !== strpos( $wp_version, '-' ) ) {
 			// We're on a dev version.
 			$version_parts = explode( '-', $wp_version );
-			return floatval( $version_parts[0] );
+			return floatval( $version_parts[ 0 ] );
 		}
 		return floatval( $wp_version );
 	}

modules/css-vars/class-kirki-modules-css-vars.php

Spacing +8 added lines, -8 removed lines

@@ -76,15 +76,15 @@ 
 		echo '<style id="kirki-css-vars">';
 		echo ':root{';
 		foreach ( $fields as $id => $args ) {
-			if ( ! isset( $args['css_vars'] ) || empty( $args['css_vars'] ) ) {
+			if ( ! isset( $args[ 'css_vars' ] ) || empty( $args[ 'css_vars' ] ) ) {
 				continue;
 			}
-			$val = Kirki_Values::get_value( $args['kirki_config'], $id );
-			foreach ( $args['css_vars'] as $css_var ) {
-				if ( isset( $css_var[2] ) && is_array( $val ) && isset( $val[ $css_var[2] ] ) ) {
-					echo esc_attr( $css_var[0] ) . ':' . esc_attr( str_replace( '$', $val[ $css_var[2] ], $css_var[1] ) ) . ';';
+			$val = Kirki_Values::get_value( $args[ 'kirki_config' ], $id );
+			foreach ( $args[ 'css_vars' ] as $css_var ) {
+				if ( isset( $css_var[ 2 ] ) && is_array( $val ) && isset( $val[ $css_var[ 2 ] ] ) ) {
+					echo esc_attr( $css_var[ 0 ] ) . ':' . esc_attr( str_replace( '$', $val[ $css_var[ 2 ] ], $css_var[ 1 ] ) ) . ';';
 				} else {
-					echo esc_attr( $css_var[0] ) . ':' . esc_attr( str_replace( '$', $val, $css_var[1] ) ) . ';';
+					echo esc_attr( $css_var[ 0 ] ) . ':' . esc_attr( str_replace( '$', $val, $css_var[ 1 ] ) ) . ';';
 				}
 			}
 		}
@@ -106,8 +106,8 @@ 
 		$fields = Kirki::$fields;
 		$data   = array();
 		foreach ( $fields as $field ) {
-			if ( isset( $field['transport'] ) && 'postMessage' === $field['transport'] && isset( $field['css_vars'] ) && ! empty( $field['css_vars'] ) ) {
-				$data[] = $field;
+			if ( isset( $field[ 'transport' ] ) && 'postMessage' === $field[ 'transport' ] && isset( $field[ 'css_vars' ] ) && ! empty( $field[ 'css_vars' ] ) ) {
+				$data[ ] = $field;
 			}
 		}
 		wp_localize_script( 'kirki_auto_css_vars', 'kirkiCssVarFields', $data );

field/class-kirki-field-typography.php

Spacing +22 added lines, -22 removed lines

@@ -49,13 +49,13 @@ 
 	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'] );
+		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';
+		if ( isset( $this->default[ 'letter-spacing' ] ) && is_numeric( $this->default[ 'letter-spacing' ] ) && $this->default[ 'letter-spacing' ] ) {
+			$this->default[ 'letter-spacing' ] .= 'px';
 		}
 	}
 
@@ -106,15 +106,15 @@ 
 			foreach ( $this->output as $output ) {
 
 				// If 'element' or 'property' are not defined, skip this.
-				if ( ! isset( $output['element'] ) ) {
+				if ( ! isset( $output[ 'element' ] ) ) {
 					continue;
 				}
-				if ( is_array( $output['element'] ) ) {
-					$output['element'] = implode( ',', $output['element'] );
+				if ( is_array( $output[ 'element' ] ) ) {
+					$output[ 'element' ] = implode( ',', $output[ 'element' ] );
 				}
 
 				// If we got this far, it's safe to add this.
-				$js_vars[] = $output;
+				$js_vars[ ] = $output;
 			}
 
 			// Did we manage to get all the items from 'output'?
@@ -146,26 +146,26 @@ 
 		foreach ( $value as $key => $val ) {
 			switch ( $key ) {
 				case 'font-family':
-					$value['font-family'] = esc_attr( $val );
+					$value[ 'font-family' ] = esc_attr( $val );
 					break;
 				case 'font-weight':
-					if ( isset( $value['variant'] ) ) {
+					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';
+					$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':
 					// Use 'regular' instead of 400 for font-variant.
-					$value['variant'] = ( 400 === $val || '400' === $val ) ? 'regular' : $val;
+					$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'] );
+					$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';
+					if ( ! isset( $value[ 'font-style' ] ) ) {
+						$value[ 'font-style' ] = ( false === strpos( $value[ 'variant' ], 'italic' ) ) ? 'normal' : 'italic';
 					}
 					break;
 				case 'font-size':
@@ -176,21 +176,21 @@ 
 					break;
 				case 'text-align':
 					if ( ! in_array( $val, array( '', 'inherit', 'left', 'center', 'right', 'justify' ), true ) ) {
-						$value['text-align'] = '';
+						$value[ 'text-align' ] = '';
 					}
 					break;
 				case 'text-transform':
 					if ( ! in_array( $val, array( '', 'none', 'capitalize', 'uppercase', 'lowercase', 'initial', 'inherit' ), true ) ) {
-						$value['text-transform'] = '';
+						$value[ 'text-transform' ] = '';
 					}
 					break;
 				case 'text-decoration':
 					if ( ! in_array( $val, array( '', 'none', 'underline', 'overline', 'line-through', 'initial', 'inherit' ), true ) ) {
-						$value['text-transform'] = '';
+						$value[ 'text-transform' ] = '';
 					}
 					break;
 				case 'color':
-					$value['color'] = '' === $value['color'] ? '' : ariColor::newColor( $val )->toCSS( 'hex' );
+					$value[ 'color' ] = '' === $value[ 'color' ] ? '' : ariColor::newColor( $val )->toCSS( 'hex' );
 					break;
 			}
 		}

modules/webfonts/class-kirki-fonts-google.php

Spacing +24 added lines, -24 removed lines

@@ -77,7 +77,7 @@ 
 		$config = apply_filters( 'kirki_config', array() );
 
 		// If we have set $config['disable_google_fonts'] to true then do not proceed any further.
-		if ( isset( $config['disable_google_fonts'] ) && true === $config['disable_google_fonts'] ) {
+		if ( isset( $config[ 'disable_google_fonts' ] ) && true === $config[ 'disable_google_fonts' ] ) {
 			return;
 		}
 
@@ -115,57 +115,57 @@ 
 		global $wp_customize;
 
 		// Process typography fields.
-		if ( isset( $args['type'] ) && 'kirki-typography' === $args['type'] ) {
+		if ( isset( $args[ 'type' ] ) && 'kirki-typography' === $args[ 'type' ] ) {
 
 			// Get the value.
 			$value = Kirki_Values::get_sanitized_field_value( $args );
 
-			if ( isset( $value['downloadFont'] ) && $value['downloadFont'] ) {
-				$this->hosted_fonts[] = $value['font-family'];
+			if ( isset( $value[ 'downloadFont' ] ) && $value[ 'downloadFont' ] ) {
+				$this->hosted_fonts[ ] = $value[ 'font-family' ];
 			}
 
 			// If we don't have a font-family then we can skip this.
-			if ( ! $wp_customize && ( ! isset( $value['font-family'] ) || in_array( $value['font-family'], $this->hosted_fonts, true ) ) ) {
+			if ( ! $wp_customize && ( ! isset( $value[ 'font-family' ] ) || in_array( $value[ 'font-family' ], $this->hosted_fonts, true ) ) ) {
 				return;
 			}
 
 			// If not a google-font, then we can skip this.
-			if ( ! isset( $value['font-family'] ) || ! Kirki_Fonts::is_google_font( $value['font-family'] ) ) {
+			if ( ! isset( $value[ 'font-family' ] ) || ! Kirki_Fonts::is_google_font( $value[ 'font-family' ] ) ) {
 				return;
 			}
 
 			// Set a default value for variants.
-			if ( ! isset( $value['variant'] ) ) {
-				$value['variant'] = 'regular';
+			if ( ! isset( $value[ 'variant' ] ) ) {
+				$value[ 'variant' ] = 'regular';
 			}
 
 			// Add the requested google-font.
-			if ( ! isset( $this->fonts[ $value['font-family'] ] ) ) {
-				$this->fonts[ $value['font-family'] ] = array();
+			if ( ! isset( $this->fonts[ $value[ 'font-family' ] ] ) ) {
+				$this->fonts[ $value[ 'font-family' ] ] = array();
 			}
-			if ( ! in_array( $value['variant'], $this->fonts[ $value['font-family'] ], true ) ) {
-				$this->fonts[ $value['font-family'] ][] = $value['variant'];
+			if ( ! in_array( $value[ 'variant' ], $this->fonts[ $value[ 'font-family' ] ], true ) ) {
+				$this->fonts[ $value[ 'font-family' ] ][ ] = $value[ 'variant' ];
 			}
 			// Are we force-loading all variants?
 			if ( true === self::$force_load_all_variants ) {
 				$all_variants               = Kirki_Fonts::get_all_variants();
-				$args['choices']['variant'] = array_keys( $all_variants );
+				$args[ 'choices' ][ 'variant' ] = array_keys( $all_variants );
 			}
 
-			if ( ! empty( $args['choices']['variant'] ) && is_array( $args['choices']['variant'] ) ) {
-				foreach ( $args['choices']['variant'] as $extra_variant ) {
-					$this->fonts[ $value['font-family'] ][] = $extra_variant;
+			if ( ! empty( $args[ 'choices' ][ 'variant' ] ) && is_array( $args[ 'choices' ][ 'variant' ] ) ) {
+				foreach ( $args[ 'choices' ][ 'variant' ] as $extra_variant ) {
+					$this->fonts[ $value[ 'font-family' ] ][ ] = $extra_variant;
 				}
 			}
 			return;
 		}
 
 		// Process non-typography fields.
-		if ( isset( $args['output'] ) && is_array( $args['output'] ) ) {
-			foreach ( $args['output'] as $output ) {
+		if ( isset( $args[ 'output' ] ) && is_array( $args[ 'output' ] ) ) {
+			foreach ( $args[ 'output' ] as $output ) {
 
 				// If we don't have a typography-related output argument we can skip this.
-				if ( ! isset( $output['property'] ) || ! in_array( $output['property'], array( 'font-family', 'font-weight' ), true ) ) {
+				if ( ! isset( $output[ 'property' ] ) || ! in_array( $output[ 'property' ], array( 'font-family', 'font-weight' ), true ) ) {
 					continue;
 				}
 
@@ -173,14 +173,14 @@ 
 				$value = Kirki_Values::get_sanitized_field_value( $args );
 
 				if ( is_string( $value ) ) {
-					if ( 'font-family' === $output['property'] ) {
+					if ( 'font-family' === $output[ 'property' ] ) {
 						if ( ! array_key_exists( $value, $this->fonts ) ) {
 							$this->fonts[ $value ] = array();
 						}
-					} elseif ( 'font-weight' === $output['property'] ) {
+					} elseif ( 'font-weight' === $output[ 'property' ] ) {
 						foreach ( $this->fonts as $font => $variants ) {
 							if ( ! in_array( $value, $variants, true ) ) {
-								$this->fonts[ $font ][] = $value;
+								$this->fonts[ $font ][ ] = $value;
 							}
 						}
 					}
@@ -212,8 +212,8 @@ 
 
 			// Get all valid font variants for this font.
 			$font_variants = array();
-			if ( isset( $this->google_fonts[ $font ]['variants'] ) ) {
-				$font_variants = $this->google_fonts[ $font ]['variants'];
+			if ( isset( $this->google_fonts[ $font ][ 'variants' ] ) ) {
+				$font_variants = $this->google_fonts[ $font ][ 'variants' ];
 			}
 			foreach ( $variants as $variant ) {
 

modules/tooltips/class-kirki-modules-tooltips.php

Spacing +3 added lines, -3 removed lines

@@ -76,13 +76,13 @@
 
 		$fields = Kirki::$fields;
 		foreach ( $fields as $field ) {
-			if ( isset( $field['tooltip'] ) && ! empty( $field['tooltip'] ) ) {
+			if ( isset( $field[ 'tooltip' ] ) && ! empty( $field[ 'tooltip' ] ) ) {
 				// Get the control ID and properly format it for the tooltips.
-				$id = str_replace( '[', '-', str_replace( ']', '', $field['settings'] ) );
+				$id = str_replace( '[', '-', str_replace( ']', '', $field[ 'settings' ] ) );
 				// Add the tooltips content.
 				$this->tooltips_content[ $id ] = array(
 					'id'      => $id,
-					'content' => wp_kses_post( $field['tooltip'] ),
+					'content' => wp_kses_post( $field[ 'tooltip' ] ),
 				);
 			}
 		}