aristath / kirki

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

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 
 		foreach ( array( 'background-image', 'background-color', 'background-repeat', 'background-position', 'background-size', 'background-attachment' ) as $property ) {
 			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' ];
 			}
 		}
 	}

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

Spacing +7 added lines, -7 removed lines

@@ -39,18 +39,18 @@
 
 		foreach ( $value as $key => $sub_value ) {
 
-			$property = ( empty( $output['property'] ) ) ? $key : $output['property'] . '-' . $key;
-			if ( isset( $output['choice'] ) && $output['property'] ) {
-				if ( $key === $output['choice'] ) {
-					$property = $output['property'];
+			$property = ( empty( $output[ 'property' ] ) ) ? $key : $output[ 'property' ] . '-' . $key;
+			if ( isset( $output[ 'choice' ] ) && $output[ 'property' ] ) {
+				if ( $key === $output[ 'choice' ] ) {
+					$property = $output[ 'property' ];
 				} else {
 					continue;
 				}
 			}
-			if ( false !== strpos( $output['property'], '%%' ) ) {
-				$property = str_replace( '%%', $key, $output['property'] );
+			if ( false !== strpos( $output[ 'property' ], '%%' ) ) {
+				$property = str_replace( '%%', $key, $output[ 'property' ] );
 			}
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $property ] = $output['prefix'] . $this->process_property_value( $property, $value[ $key ] ) . $output['suffix'];
+			$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $property ] = $output[ 'prefix' ] . $this->process_property_value( $property, $value[ $key ] ) . $output[ 'suffix' ];
 		}
 	}
 }

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

Spacing +7 added lines, -7 removed lines

@@ -26,28 +26,28 @@
 		foreach ( $value as $key => $sub_value ) {
 
 			// If "element" is not defined, there's no reason to continue.
-			if ( ! isset( $output['element'] ) ) {
+			if ( ! isset( $output[ 'element' ] ) ) {
 				continue;
 			}
 
 			// If the "choice" is not the same as the $key in our loop, there's no reason to proceed.
-			if ( isset( $output['choice'] ) && $key !== $output['choice'] ) {
+			if ( isset( $output[ 'choice' ] ) && $key !== $output[ 'choice' ] ) {
 				continue;
 			}
 
 			// If "property" is not defined, fallback to the $key.
-			$property = ( ! isset( $output['property'] ) || empty( $output['property'] ) ) ? $key : $output['property'];
+			$property = ( ! isset( $output[ 'property' ] ) || empty( $output[ 'property' ] ) ) ? $key : $output[ 'property' ];
 
 			// If "media_query" is not defined, use "global".
-			if ( ! isset( $output['media_query'] ) || empty( $output['media_query'] ) ) {
-				$output['media_query'] = 'global';
+			if ( ! isset( $output[ 'media_query' ] ) || empty( $output[ 'media_query' ] ) ) {
+				$output[ 'media_query' ] = 'global';
 			}
 
 			// If "suffix" is defined, add it to the value.
-			$output['suffix'] = ( isset( $output['suffix'] ) ) ? $output['suffix'] : '';
+			$output[ 'suffix' ] = ( isset( $output[ 'suffix' ] ) ) ? $output[ 'suffix' ] : '';
 
 			// Create the styles.
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $property ] = $sub_value . $output['suffix'];
+			$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $property ] = $sub_value . $output[ 'suffix' ];
 
 		}
 	}

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

Spacing +6 added lines, -6 removed lines

@@ -23,7 +23,7 @@ 
 	 */
 	protected function process_output( $output, $value ) {
 
-		if ( ! isset( $output['element'] ) || ! isset( $output['property'] ) ) {
+		if ( ! isset( $output[ 'element' ] ) || ! isset( $output[ 'property' ] ) ) {
 			return;
 		}
 		$output = wp_parse_args( $output, array(
@@ -33,16 +33,16 @@ 
 			'suffix'      => '',
 		) );
 		if ( is_array( $value ) ) {
-			if ( isset( $output['choice'] ) && $output['choice'] ) {
-				$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] = $output['prefix'] . $this->process_property_value( $output['property'], $value[ $output['choice'] ] ) . $output['units'] . $output['suffix'];
+			if ( isset( $output[ 'choice' ] ) && $output[ 'choice' ] ) {
+				$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] = $output[ 'prefix' ] . $this->process_property_value( $output[ 'property' ], $value[ $output[ 'choice' ] ] ) . $output[ 'units' ] . $output[ 'suffix' ];
 				return;
 			}
-			if ( isset( $value['url'] ) ) {
-				$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] = $output['prefix'] . $this->process_property_value( $output['property'], $value['url'] ) . $output['units'] . $output['suffix'];
+			if ( isset( $value[ 'url' ] ) ) {
+				$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] = $output[ 'prefix' ] . $this->process_property_value( $output[ 'property' ], $value[ 'url' ] ) . $output[ 'units' ] . $output[ 'suffix' ];
 				return;
 			}
 			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' ];
 	}
 }

modules/css/class-kirki-output.php

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':
@@ -204,8 +204,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 );
@@ -217,7 +217,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;
 						}
 					}
@@ -238,10 +238,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 );
@@ -259,13 +259,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
@@ -274,14 +274,14 @@ 
 			'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;
 		}
-		$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' ];
 	}
 
 	/**
@@ -319,8 +319,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/css/class-kirki-modules-css-generator.php

Spacing +6 added lines, -6 removed lines

@@ -146,10 +146,10 @@ 
 	public static function css( $field ) {
 
 		// Set class vars.
-		self::$settings   = $field['settings'];
-		self::$callback   = $field['sanitize_callback'];
-		self::$field_type = $field['type'];
-		self::$output     = $field['output'];
+		self::$settings   = $field[ 'settings' ];
+		self::$callback   = $field[ 'sanitize_callback' ];
+		self::$field_type = $field[ 'type' ];
+		self::$output     = $field[ 'output' ];
 		if ( ! is_array( self::$output ) ) {
 			self::$output = array(
 				array(
@@ -164,7 +164,7 @@ 
 
 		// Find the class that will handle the outpout for this field.
 		$classname = 'Kirki_Output';
-		$field_output_classes = apply_filters( "kirki/{$field['kirki_config']}/output/control-classnames", array(
+		$field_output_classes = apply_filters( "kirki/{$field[ 'kirki_config' ]}/output/control-classnames", array(
 			'kirki-background'  => 'Kirki_Output_Field_Background',
 			'kirki-dimensions'  => 'Kirki_Output_Field_Dimensions',
 			'kirki-image'       => 'Kirki_Output_Field_Image',
@@ -174,7 +174,7 @@ 
 		if ( array_key_exists( self::$field_type, $field_output_classes ) ) {
 			$classname = $field_output_classes[ self::$field_type ];
 		}
-		$obj = new $classname( $field['kirki_config'], self::$output, self::$value, $field );
+		$obj = new $classname( $field[ 'kirki_config' ], self::$output, self::$value, $field );
 		return $obj->get_styles();
 
 	}

modules/css/class-kirki-css-to-file.php

Spacing +5 added lines, -5 removed lines

@@ -53,15 +53,15 @@ 
 		$upload_dir = wp_upload_dir();
 
 		$paths = array(
-			'file'   => wp_normalize_path( $upload_dir['basedir'] . '/kirki-css/styles.css' ),
-			'folder' => wp_normalize_path( $upload_dir['basedir'] . '/kirki-css' ),
+			'file'   => wp_normalize_path( $upload_dir[ 'basedir' ] . '/kirki-css/styles.css' ),
+			'folder' => wp_normalize_path( $upload_dir[ 'basedir' ] . '/kirki-css' ),
 		);
 
 		if ( 'file' === $context ) {
-			return $paths['file'];
+			return $paths[ 'file' ];
 		}
 		if ( 'folder' === $context ) {
-			return $paths['folder'];
+			return $paths[ 'folder' ];
 		}
 		return $paths;
 
@@ -77,7 +77,7 @@ 
 	public function get_url() {
 
 		$upload_dir = wp_upload_dir();
-		return esc_url_raw( $upload_dir['baseurl'] . '/kirki-css/styles.css' );
+		return esc_url_raw( $upload_dir[ 'baseurl' ] . '/kirki-css/styles.css' );
 
 	}
 

modules/webfonts/class-kirki-modules-webfonts-embed.php

Spacing +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
 				return false;
 			}
 
-			if ( ! isset( $response['response'] ) || ! is_array( $response['response'] ) || ! isset( $response['response']['code'] ) || 200 !== $response['response']['code'] ) {
+			if ( ! isset( $response[ 'response' ] ) || ! is_array( $response[ 'response' ] ) || ! isset( $response[ 'response' ][ 'code' ] ) || 200 !== $response[ 'response' ][ 'code' ] ) {
 				return false;
 			}
 

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

Spacing +33 added lines, -33 removed lines

@@ -76,7 +76,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;
 		}
 
@@ -113,64 +113,64 @@ 
 	public function generate_google_font( $args ) {
 
 		// 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 we don't have a font-family then we can skip this.
-			if ( ! isset( $value['font-family'] ) ) {
+			if ( ! isset( $value[ 'font-family' ] ) ) {
 				return;
 			}
 
 			// If not a google-font, then we can skip this.
-			if ( ! Kirki_Fonts::is_google_font( $value['font-family'] ) ) {
+			if ( ! 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';
 			}
-			if ( isset( $value['subsets'] ) ) {
+			if ( isset( $value[ 'subsets' ] ) ) {
 
 				// Add the subset directly to the array of subsets in the Kirki_GoogleFonts_Manager object.
 				// Subsets must be applied to ALL fonts if possible.
-				if ( ! is_array( $value['subsets'] ) ) {
-					$this->subsets[] = $value['subsets'];
+				if ( ! is_array( $value[ 'subsets' ] ) ) {
+					$this->subsets[ ] = $value[ 'subsets' ];
 				} else {
-					foreach ( $value['subsets'] as $subset ) {
-						$this->subsets[] = $subset;
+					foreach ( $value[ 'subsets' ] as $subset ) {
+						$this->subsets[ ] = $subset;
 					}
 				}
 			}
 
 			// 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;
 				}
 			}
 		} else {
 
 			// 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', 'font-subset', 'subset', 'subsets' ), true ) ) {
+					if ( ! isset( $output[ 'property' ] ) || ! in_array( $output[ 'property' ], array( 'font-family', 'font-weight', 'font-subset', 'subset', 'subsets' ), true ) ) {
 						continue;
 					}
 
@@ -178,25 +178,25 @@ 
 					$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;
 								}
 							}
-						} elseif ( 'font-subset' === $output['property'] || 'subset' === $output['property'] || 'subsets' === $output['property'] ) {
+						} elseif ( 'font-subset' === $output[ 'property' ] || 'subset' === $output[ 'property' ] || 'subsets' === $output[ 'property' ] ) {
 							if ( ! is_array( $value ) ) {
 								if ( ! in_array( $value, $this->subsets, true ) ) {
-									$this->subsets[] = $value;
+									$this->subsets[ ] = $value;
 								}
 							} else {
 								foreach ( $value as $subset ) {
 									if ( ! in_array( $subset, $this->subsets, true ) ) {
-										$this->subsets[] = $subset;
+										$this->subsets[ ] = $subset;
 									}
 								}
 							}
@@ -231,8 +231,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 ) {
 
@@ -247,10 +247,10 @@ 
 
 			// Check if the selected subsets exist, even in one of the selected fonts.
 			// If they don't, then they have to be removed otherwise the link will fail.
-			if ( isset( $this->google_fonts[ $font ]['subsets'] ) ) {
+			if ( isset( $this->google_fonts[ $font ][ 'subsets' ] ) ) {
 				foreach ( $this->subsets as $subset ) {
-					if ( in_array( $subset, $this->google_fonts[ $font ]['subsets'], true ) ) {
-						$valid_subsets[] = $subset;
+					if ( in_array( $subset, $this->google_fonts[ $font ][ 'subsets' ], true ) ) {
+						$valid_subsets[ ] = $subset;
 					}
 				}
 			}