aristath / kirki

core/class-kirki-field.php

Spacing +18 added lines, -18 removed lines

@@ -257,13 +257,13 @@ 
 
 		// Get the config arguments, and merge them with the defaults.
 		$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 ];
+		if ( 'global' !== $this->kirki_config && isset( Kirki::$config[$this->kirki_config] ) ) {
+			$config_defaults = Kirki::$config[$this->kirki_config];
 		}
 		$config_defaults = ( is_array( $config_defaults ) ) ? $config_defaults : array();
 		foreach ( $config_defaults as $key => $value ) {
-			if ( isset( $defaults[ $key ] ) && ! empty( $value ) && $value !== $defaults[ $key ] ) {
-				$defaults[ $key ] = $value;
+			if ( isset( $defaults[$key] ) && ! empty( $value ) && $value !== $defaults[$key] ) {
+				$defaults[$key] = $value;
 			}
 		}
 
@@ -308,11 +308,11 @@ 
 		// Get all arguments with their values.
 		$args = get_object_vars( $this );
 		foreach ( array_keys( $args ) as $key ) {
-			$args[ $key ] = $this->$key;
+			$args[$key] = $this->$key;
 		}
 
 		// Add the field to the static $fields variable properly indexed.
-		Kirki::$fields[ $this->settings ] = $args;
+		Kirki::$fields[$this->settings] = $args;
 
 	}
 
@@ -407,7 +407,7 @@ 
 			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 ] );
+				unset( $this->partial_refresh[$id] );
 				continue;
 			}
 		}
@@ -435,10 +435,10 @@ 
 		}
 		$settings = array();
 		foreach ( $this->settings as $setting_key => $setting_value ) {
-			$settings[ $setting_key ] = $setting_value;
+			$settings[$setting_key] = $setting_value;
 			// If we're using serialized options then we need to spice this up.
 			if ( 'option' === $this->option_type && '' !== $this->option_name && ( false === strpos( $setting_key, '[' ) ) ) {
-				$settings[ $setting_key ] = "{$this->option_name}[{$setting_value}]";
+				$settings[$setting_key] = "{$this->option_name}[{$setting_value}]";
 			}
 		}
 		$this->settings = $settings;
@@ -550,27 +550,27 @@ 
 		}
 		foreach ( $this->output as $key => $output ) {
 			if ( empty( $output ) || ! isset( $output['element'] ) ) {
-				unset( $this->output[ $key ] );
+				unset( $this->output[$key] );
 				continue;
 			}
 			if ( ! isset( $output['sanitize_callback'] ) && isset( $output['callback'] ) ) {
-				$this->output[ $key ]['sanitize_callback'] = $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'] );
+				$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'] ) );
 		}
 	}
 

modules/postmessage/class-kirki-modules-postmessage.php

Spacing +5 added lines, -5 removed lines

@@ -142,10 +142,10 @@ 
 			$js_var['index_key'] = $key;
 			$callback = $this->get_callback( $args );
 			if ( is_callable( $callback ) ) {
-				$field['scripts'][ $key ] = call_user_func_array( $callback, array( $js_var, $args ) );
+				$field['scripts'][$key] = call_user_func_array( $callback, array( $js_var, $args ) );
 				continue;
 			}
-			$field['scripts'][ $key ] = $this->script_var( $js_var );
+			$field['scripts'][$key] = $this->script_var( $js_var );
 		}
 		$combo_extra_script = '';
 		$combo_css_script   = '';
@@ -177,7 +177,7 @@ 
 	 */
 	protected function script_html_var( $args ) {
 
-		$script  = ( isset( $args['choice'] ) ) ? "newval=newval['{$args['choice']}'];" : '';
+		$script = ( isset( $args['choice'] ) ) ? "newval=newval['{$args['choice']}'];" : '';
 
 		// Apply the value_pattern.
 		if ( isset( $args['value_pattern'] ) && '' !== $args['value_pattern'] ) {
@@ -251,7 +251,7 @@ 
 		$property_script = '';
 
 		// Define choice.
-		$choice  = ( isset( $args['choice'] ) && '' !== $args['choice'] ) ? $args['choice'] : '';
+		$choice = ( isset( $args['choice'] ) && '' !== $args['choice'] ) ? $args['choice'] : '';
 
 		$value_key = 'newval' . $args['index_key'];
 		$property_script .= $value_key . '=newval;';
@@ -345,7 +345,7 @@ 
 			'font-weight'     => 'fontWeight',
 			'font-style'      => 'fontStyle',
 		);
-		$choice_condition = ( isset( $args['choice'] ) && '' !== $args['choice'] && isset( $css_build_array[ $args['choice'] ] ) );
+		$choice_condition = ( isset( $args['choice'] ) && '' !== $args['choice'] && isset( $css_build_array[$args['choice']] ) );
 		$script .= ( ! $choice_condition ) ? $webfont_loader : '';
 		foreach ( $css_build_array as $property => $var ) {
 			if ( $choice_condition && $property !== $args['choice'] ) {

docs/files/class-kirki-installer-section.php

Spacing +1 added lines, -1 removed lines

@@ -212,7 +212,7 @@
 	 */
 	function kirki_installer_dismiss() {
 		check_ajax_referer( 'dismiss-kirki-recommendation', 'security' );
-		$user_id   = get_current_user_id();
+		$user_id = get_current_user_id();
 		// @codingStandardsIgnoreLine WordPress.VIP.RestrictedFunctions.user_meta_update_user_meta
 		if ( update_user_meta( $user_id, 'dismiss-kirki-recommendation', true ) ) {
 			echo 'success! :-)';

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

Spacing +4 added lines, -4 removed lines

@@ -47,12 +47,12 @@ 
 		foreach ( $properties as $property ) {
 
 			// Early exit if the value is not in the defaults.
-			if ( ! isset( $this->field['default'][ $property ] ) ) {
+			if ( ! isset( $this->field['default'][$property] ) ) {
 				continue;
 			}
 
 			// Early exit if the value is not saved in the values.
-			if ( ! isset( $value[ $property ] ) || ! $value[ $property ] ) {
+			if ( ! isset( $value[$property] ) || ! $value[$property] ) {
 				continue;
 			}
 
@@ -61,7 +61,7 @@ 
 				continue;
 			}
 
-			$property_value = $this->process_property_value( $property, $value[ $property ] );
+			$property_value = $this->process_property_value( $property, $value[$property] );
 			if ( 'font-family' === $property ) {
 				$value['font-backup'] = ( isset( $value['font-backup'] ) ) ? $value['font-backup'] : '';
 				$property_value = $this->process_property_value( $property, array(
@@ -71,7 +71,7 @@ 
 			}
 			$property = ( isset( $output['choice'] ) && isset( $output['property'] ) ) ? $output['property'] : $property;
 			$property_value = ( is_array( $property_value ) && isset( $property_value[0] ) ) ? $property_value[0] : $property_value;
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $property ] = $output['prefix'] . $property_value . $output['suffix'];
+			$this->styles[$output['media_query']][$output['element']][$property] = $output['prefix'] . $property_value . $output['suffix'];
 		}
 	}
 }

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

Spacing +9 added lines, -9 removed lines

@@ -172,7 +172,7 @@ 
 			'kirki-multicolor'  => 'Kirki_Output_Field_Multicolor',
 		) );
 		if ( array_key_exists( self::$field_type, $field_output_classes ) ) {
-			$classname = $field_output_classes[ self::$field_type ];
+			$classname = $field_output_classes[self::$field_type];
 		}
 		$obj = new $classname( $field['kirki_config'], self::$output, self::$value, $field );
 		return $obj->get_styles();
@@ -247,10 +247,10 @@ 
 							'transition',
 							'transition-property',
 						), true ) ) {
-							unset( $css[ $media_query ][ $element ][ $property ] );
-							$css[ $media_query ][ $element ][ '-webkit-' . $property ] = $value;
-							$css[ $media_query ][ $element ][ '-moz-' . $property ]    = $value;
-							$css[ $media_query ][ $element ][ $property ]              = $value;
+							unset( $css[$media_query][$element][$property] );
+							$css[$media_query][$element]['-webkit-' . $property] = $value;
+							$css[$media_query][$element]['-moz-' . $property]    = $value;
+							$css[$media_query][$element][$property]              = $value;
 						}
 
 						// Add -ms-* and -o-*.
@@ -260,10 +260,10 @@ 
 							'transition',
 							'transition-property',
 						), true ) ) {
-							unset( $css[ $media_query ][ $element ][ $property ] );
-							$css[ $media_query ][ $element ][ '-ms-' . $property ] = $value;
-							$css[ $media_query ][ $element ][ '-o-' . $property ]  = $value;
-							$css[ $media_query ][ $element ][ $property ]          = $value;
+							unset( $css[$media_query][$element][$property] );
+							$css[$media_query][$element]['-ms-' . $property] = $value;
+							$css[$media_query][$element]['-o-' . $property]  = $value;
+							$css[$media_query][$element][$property]          = $value;
 						}
 					}
 				}

controls/php/class-kirki-control-typography.php

Spacing +9 added lines, -9 removed lines

@@ -56,16 +56,16 @@ 
 
 		if ( is_array( $this->json['value'] ) ) {
 			foreach ( array_keys( $this->json['value'] ) as $key ) {
-				if ( ! in_array( $key, array( 'variant', 'font-weight', 'font-style' ), true ) && ! isset( $this->json['default'][ $key ] ) ) {
-					unset( $this->json['value'][ $key ] );
+				if ( ! in_array( $key, array( 'variant', 'font-weight', 'font-style' ), true ) && ! isset( $this->json['default'][$key] ) ) {
+					unset( $this->json['value'][$key] );
 				}
 				// Fix for https://wordpress.org/support/topic/white-font-after-updateing-to-3-0-16.
-				if ( ! isset( $this->json['default'][ $key ] ) ) {
-					unset( $this->json['value'][ $key ] );
+				if ( ! isset( $this->json['default'][$key] ) ) {
+					unset( $this->json['value'][$key] );
 				}
 				// Fix for https://github.com/aristath/kirki/issues/1405.
-				if ( isset( $this->json['default'][ $key ] ) && false === $this->json['default'][ $key ] ) {
-					unset( $this->json['value'][ $key ] );
+				if ( isset( $this->json['default'][$key] ) && false === $this->json['default'][$key] ) {
+					unset( $this->json['value'][$key] );
 				}
 			}
 		}
@@ -281,7 +281,7 @@ 
 			if ( is_string( $variant ) ) {
 				$final_variants[] = array(
 					'id'    => $variant,
-					'label' => isset( $all_variants[ $variant ] ) ? $all_variants[ $variant ] : $variant,
+					'label' => isset( $all_variants[$variant] ) ? $all_variants[$variant] : $variant,
 				);
 			} elseif ( is_array( $variant ) && isset( $variant['id'] ) && isset( $variant['label'] ) ) {
 				$final_variants[] = $variant;
@@ -364,7 +364,7 @@ 
 					if ( array_key_exists( $variant, $all_variants ) ) {
 						$available_variants[] = array(
 							'id' => $variant,
-							'label' => $all_variants[ $variant ],
+							'label' => $all_variants[$variant],
 						);
 					}
 				}
@@ -376,7 +376,7 @@ 
 					if ( array_key_exists( $subset, $all_subsets ) ) {
 						$available_subsets[] = array(
 							'id' => $subset,
-							'label' => $all_subsets[ $subset ],
+							'label' => $all_subsets[$subset],
 						);
 					}
 				}

core/class-kirki.php

Spacing +5 added lines, -5 removed lines

@@ -114,7 +114,7 @@ 
 
 		$config = Kirki_Config::get_instance( $config_id, $args );
 		$config_args = $config->get_config();
-		self::$config[ $config_args['id'] ] = $config_args;
+		self::$config[$config_args['id']] = $config_args;
 	}
 
 	/**
@@ -133,7 +133,7 @@ 
 		$args['type']        = ( isset( $args['type'] ) ) ? $args['type'] : 'default';
 		$args['type']        = 'kirki-' . $args['type'];
 
-		self::$panels[ $args['id'] ] = $args;
+		self::$panels[$args['id']] = $args;
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 		$args['type']        = ( isset( $args['type'] ) ) ? $args['type'] : 'default';
 		$args['type']        = 'kirki-' . $args['type'];
 
-		self::$sections[ $args['id'] ] = $args;
+		self::$sections[$args['id']] = $args;
 	}
 
 	/**
@@ -205,9 +205,9 @@ 
 	 */
 	public static function get_config_param( $id, $param ) {
 
-		if ( ! isset( self::$config[ $id ] ) || ! isset( self::$config[ $id ][ $param ] ) ) {
+		if ( ! isset( self::$config[$id] ) || ! isset( self::$config[$id][$param] ) ) {
 			return '';
 		}
-		return self::$config[ $id ][ $param ];
+		return self::$config[$id][$param];
 	}
 }

modules/css/class-kirki-output.php

Spacing +14 added lines, -14 removed lines

@@ -103,16 +103,16 @@ 
 			}
 			if ( is_array( $value ) ) {
 				foreach ( array_keys( $value ) as $value_k ) {
-					if ( ! is_string( $value[ $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'] );
+							$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 );
@@ -142,16 +142,16 @@ 
 						if ( is_array( $options ) ) {
 							if ( $option_name ) {
 								$subkey = str_replace( array( $option_name, '[', ']' ), '', $replace );
-								$replacement = ( isset( $options[ $subkey ] ) ) ? $options[ $subkey ] : '';
+								$replacement = ( isset( $options[$subkey] ) ) ? $options[$subkey] : '';
 								break;
 							}
-							$replacement = ( isset( $options[ $replace ] ) ) ? $options[ $replace ] : '';
+							$replacement = ( isset( $options[$replace] ) ) ? $options[$replace] : '';
 							break;
 						}
 						$replacement = get_option( $replace );
 						break;
 					case 'site_option':
-						$replacement = ( is_array( $options ) && isset( $options[ $replace ] ) ) ? $options[ $replace ] : get_site_option( $replace );
+						$replacement = ( is_array( $options ) && isset( $options[$replace] ) ) ? $options[$replace] : get_site_option( $replace );
 						break;
 					case 'user_meta':
 						$user_id = get_current_user_id();
@@ -166,8 +166,8 @@ 
 				$replacement = ( false === $replacement ) ? '' : $replacement;
 				if ( is_array( $value ) ) {
 					foreach ( $value as $k => $v ) {
-						$_val = ( isset( $value[ $v ] ) ) ? $value[ $v ] : $v;
-						$value[ $k ] = str_replace( $search, $replacement, $_val );
+						$_val = ( isset( $value[$v] ) ) ? $value[$v] : $v;
+						$value[$k] = str_replace( $search, $replacement, $_val );
 					}
 					return $value;
 				}
@@ -209,7 +209,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'] ] ) {
+						if ( isset( $output['choice'] ) && isset( $value[$output['choice']] ) && $exclude == $value[$output['choice']] ) {
 							$skip = true;
 						}
 					}
@@ -267,13 +267,13 @@ 
 			'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 ( 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;
 		}
-		$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'];
 	}
 
 	/**
@@ -295,7 +295,7 @@ 
 			)
 		);
 		if ( array_key_exists( $property, $properties ) ) {
-			$classname = $properties[ $property ];
+			$classname = $properties[$property];
 			$obj = new $classname( $property, $value );
 			return $obj->get_value();
 		}