Passed
Push — develop ( b8eab4...c3f711 )
by Aristeides
08:33
created
modules/css/class-kirki-output.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -83,13 +83,13 @@  discard block
 block discarded – undo
83 83
 	 */
84 84
 	protected function apply_sanitize_callback( $output, $value ) {
85 85
 
86
-		if ( isset( $output['sanitize_callback'] ) && null !== $output['sanitize_callback'] ) {
86
+		if ( isset( $output[ 'sanitize_callback' ] ) && null !== $output[ 'sanitize_callback' ] ) {
87 87
 
88 88
 			// If the sanitize_callback is invalid, return the value.
89
-			if ( ! is_callable( $output['sanitize_callback'] ) ) {
89
+			if ( ! is_callable( $output[ 'sanitize_callback' ] ) ) {
90 90
 				return $value;
91 91
 			}
92
-			return call_user_func( $output['sanitize_callback'], $this->value );
92
+			return call_user_func( $output[ 'sanitize_callback' ], $this->value );
93 93
 		}
94 94
 
95 95
 		return $value;
@@ -105,22 +105,22 @@  discard block
 block discarded – undo
105 105
 	 */
106 106
 	protected function apply_value_pattern( $output, $value ) {
107 107
 
108
-		if ( isset( $output['value_pattern'] ) && ! empty( $output['value_pattern'] ) && is_string( $output['value_pattern'] ) ) {
108
+		if ( isset( $output[ 'value_pattern' ] ) && ! empty( $output[ 'value_pattern' ] ) && is_string( $output[ 'value_pattern' ] ) ) {
109 109
 			if ( ! is_array( $value ) ) {
110
-				$value = str_replace( '$', $value, $output['value_pattern'] );
110
+				$value = str_replace( '$', $value, $output[ 'value_pattern' ] );
111 111
 			}
112 112
 			if ( is_array( $value ) ) {
113 113
 				foreach ( array_keys( $value ) as $value_k ) {
114 114
 					if ( ! is_string( $value[ $value_k ] ) ) {
115 115
 						continue;
116 116
 					}
117
-					if ( isset( $output['choice'] ) ) {
118
-						if ( $output['choice'] === $value_k ) {
119
-							$value[ $output['choice'] ] = str_replace( '$', $value[ $output['choice'] ], $output['value_pattern'] );
117
+					if ( isset( $output[ 'choice' ] ) ) {
118
+						if ( $output[ 'choice' ] === $value_k ) {
119
+							$value[ $output[ 'choice' ] ] = str_replace( '$', $value[ $output[ 'choice' ] ], $output[ 'value_pattern' ] );
120 120
 						}
121 121
 						continue;
122 122
 					}
123
-					$value[ $value_k ] = str_replace( '$', $value[ $value_k ], $output['value_pattern'] );
123
+					$value[ $value_k ] = str_replace( '$', $value[ $value_k ], $output[ 'value_pattern' ] );
124 124
 				}
125 125
 			}
126 126
 			$value = $this->apply_pattern_replace( $output, $value );
@@ -136,14 +136,14 @@  discard block
 block discarded – undo
136 136
 	 * @return string|array
137 137
 	 */
138 138
 	protected function apply_pattern_replace( $output, $value ) {
139
-		if ( isset( $output['pattern_replace'] ) && is_array( $output['pattern_replace'] ) ) {
139
+		if ( isset( $output[ 'pattern_replace' ] ) && is_array( $output[ 'pattern_replace' ] ) ) {
140 140
 			$option_type = ( '' !== Kirki::get_config_param( $this->config_id, 'option_type' ) ) ? Kirki::get_config_param( $this->config_id, 'option_type' ) : 'theme_mod';
141 141
 			$option_name = Kirki::get_config_param( $this->config_id, 'option_name' );
142 142
 			$options     = array();
143 143
 			if ( $option_name ) {
144 144
 				$options = ( 'site_option' === $option_type ) ? get_site_option( $option_name ) : get_option( $option_name );
145 145
 			}
146
-			foreach ( $output['pattern_replace'] as $search => $replace ) {
146
+			foreach ( $output[ 'pattern_replace' ] as $search => $replace ) {
147 147
 				$replacement = '';
148 148
 				switch ( $option_type ) {
149 149
 					case 'option':
@@ -204,8 +204,8 @@  discard block
 block discarded – undo
204 204
 			}
205 205
 
206 206
 			// No need to proceed this if the current value is the same as in the "exclude" value.
207
-			if ( isset( $output['exclude'] ) && is_array( $output['exclude'] ) ) {
208
-				foreach ( $output['exclude'] as $exclude ) {
207
+			if ( isset( $output[ 'exclude' ] ) && is_array( $output[ 'exclude' ] ) ) {
208
+				foreach ( $output[ 'exclude' ] as $exclude ) {
209 209
 					if ( is_array( $value ) ) {
210 210
 						if ( is_array( $exclude ) ) {
211 211
 							$diff1 = array_diff( $value, $exclude );
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
 						// If 'choice' is defined check for sub-values too.
219 219
 						// Fixes https://github.com/aristath/kirki/issues/1416.
220 220
 						// @codingStandardsIgnoreLine WordPress.PHP.StrictComparisons.LooseComparison
221
-						if ( isset( $output['choice'] ) && isset( $value[ $output['choice'] ] ) && $exclude == $value[ $output['choice'] ] ) { // phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison
221
+						if ( isset( $output[ 'choice' ] ) && isset( $value[ $output[ 'choice' ] ] ) && $exclude == $value[ $output[ 'choice' ] ] ) { // phpcs:ignore WordPress.PHP.StrictComparisons.LooseComparison
222 222
 							$skip = true;
223 223
 						}
224 224
 					}
@@ -239,10 +239,10 @@  discard block
 block discarded – undo
239 239
 			// Apply any value patterns defined.
240 240
 			$value = $this->apply_value_pattern( $output, $value );
241 241
 
242
-			if ( isset( $output['element'] ) && is_array( $output['element'] ) ) {
243
-				$output['element'] = array_unique( $output['element'] );
244
-				sort( $output['element'] );
245
-				$output['element'] = implode( ',', $output['element'] );
242
+			if ( isset( $output[ 'element' ] ) && is_array( $output[ 'element' ] ) ) {
243
+				$output[ 'element' ] = array_unique( $output[ 'element' ] );
244
+				sort( $output[ 'element' ] );
245
+				$output[ 'element' ] = implode( ',', $output[ 'element' ] );
246 246
 			}
247 247
 
248 248
 			$value = $this->process_value( $value, $output );
@@ -260,13 +260,13 @@  discard block
 block discarded – undo
260 260
 	 * @return null
261 261
 	 */
262 262
 	protected function process_output( $output, $value ) {
263
-		if ( ! isset( $output['element'] ) || ! isset( $output['property'] ) ) {
263
+		if ( ! isset( $output[ 'element' ] ) || ! isset( $output[ 'property' ] ) ) {
264 264
 			return;
265 265
 		}
266
-		$output['media_query'] = ( isset( $output['media_query'] ) ) ? $output['media_query'] : 'global';
267
-		$output['prefix']      = ( isset( $output['prefix'] ) ) ? $output['prefix'] : '';
268
-		$output['units']       = ( isset( $output['units'] ) ) ? $output['units'] : '';
269
-		$output['suffix']      = ( isset( $output['suffix'] ) ) ? $output['suffix'] : '';
266
+		$output[ 'media_query' ] = ( isset( $output[ 'media_query' ] ) ) ? $output[ 'media_query' ] : 'global';
267
+		$output[ 'prefix' ]      = ( isset( $output[ 'prefix' ] ) ) ? $output[ 'prefix' ] : '';
268
+		$output[ 'units' ]       = ( isset( $output[ 'units' ] ) ) ? $output[ 'units' ] : '';
269
+		$output[ 'suffix' ]      = ( isset( $output[ 'suffix' ] ) ) ? $output[ 'suffix' ] : '';
270 270
 
271 271
 		// Properties that can accept multiple values.
272 272
 		// Useful for example for gradients where all browsers use the "background-image" property
@@ -275,15 +275,15 @@  discard block
 block discarded – undo
275 275
 			'background-image',
276 276
 			'background',
277 277
 		);
278
-		if ( in_array( $output['property'], $accepts_multiple, true ) ) {
279
-			if ( isset( $this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] ) && ! is_array( $this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] ) ) {
280
-				$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] = (array) $this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ];
278
+		if ( in_array( $output[ 'property' ], $accepts_multiple, true ) ) {
279
+			if ( isset( $this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] ) && ! is_array( $this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] ) ) {
280
+				$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] = (array) $this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ];
281 281
 			}
282
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ][] = $output['prefix'] . $value . $output['units'] . $output['suffix'];
282
+			$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ][ ] = $output[ 'prefix' ] . $value . $output[ 'units' ] . $output[ 'suffix' ];
283 283
 			return;
284 284
 		}
285 285
 		if ( is_string( $value ) ) {
286
-			$this->styles[ $output['media_query'] ][ $output['element'] ][ $output['property'] ] = $output['prefix'] . $this->process_property_value( $output['property'], $value ) . $output['units'] . $output['suffix'];
286
+			$this->styles[ $output[ 'media_query' ] ][ $output[ 'element' ] ][ $output[ 'property' ] ] = $output[ 'prefix' ] . $this->process_property_value( $output[ 'property' ], $value ) . $output[ 'units' ] . $output[ 'suffix' ];
287 287
 		}
288 288
 	}
289 289
 
@@ -322,8 +322,8 @@  discard block
 block discarded – undo
322 322
 	 * @return string|array
323 323
 	 */
324 324
 	protected function process_value( $value, $output ) {
325
-		if ( isset( $output['property'] ) ) {
326
-			return $this->process_property_value( $output['property'], $value );
325
+		if ( isset( $output[ 'property' ] ) ) {
326
+			return $this->process_property_value( $output[ 'property' ], $value );
327 327
 		}
328 328
 		return $value;
329 329
 	}
Please login to merge, or discard this patch.