Completed
Pull Request — master (#1039)
by Stephanie
01:04
created
classes/controllers/FrmStylesController.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -119,7 +119,8 @@
 block discarded – undo
119 119
 	 * @return void
120 120
 	 */
121 121
 	private static function maybe_hook_into_global_settings_save() {
122
-		if ( empty( $_POST ) || ! isset( $_POST['style'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
122
+		if ( empty( $_POST ) || ! isset( $_POST['style'] ) ) {
123
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
123 124
 			// Avoid changing any style data if the style array is not sent in the request.
124 125
 			return;
125 126
 		}
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -651,7 +651,7 @@  discard block
 block discarded – undo
651 651
 		}
652 652
 
653 653
 		parse_str( $query, $parsed_query );
654
-		$form_id      = ! empty( $parsed_query['form'] ) ? absint( $parsed_query['form'] ) : 0;
654
+		$form_id = ! empty( $parsed_query['form'] ) ? absint( $parsed_query['form'] ) : 0;
655 655
 
656 656
 		$style     = new stdClass();
657 657
 		$style->ID = end( $ids );
@@ -689,8 +689,8 @@  discard block
 block discarded – undo
689 689
 
690 690
 		$forms = FrmForm::get_published_forms();
691 691
 		foreach ( $forms as $form ) {
692
-			$new_style      = ( isset( $_POST['style'] ) && isset( $_POST['style'][ $form->id ] ) ) ? sanitize_text_field( wp_unslash( $_POST['style'][ $form->id ] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
693
-			$previous_style = ( isset( $_POST['prev_style'] ) && isset( $_POST['prev_style'][ $form->id ] ) ) ? sanitize_text_field( wp_unslash( $_POST['prev_style'][ $form->id ] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
692
+			$new_style      = ( isset( $_POST['style'] ) && isset( $_POST['style'][$form->id] ) ) ? sanitize_text_field( wp_unslash( $_POST['style'][$form->id] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
693
+			$previous_style = ( isset( $_POST['prev_style'] ) && isset( $_POST['prev_style'][$form->id] ) ) ? sanitize_text_field( wp_unslash( $_POST['prev_style'][$form->id] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Missing
694 694
 			if ( $new_style == $previous_style ) {
695 695
 				continue;
696 696
 			}
@@ -1064,8 +1064,8 @@  discard block
 block discarded – undo
1064 1064
 	 */
1065 1065
 	public static function get_style_val( $val, $form = 'default' ) {
1066 1066
 		$style = self::get_form_style( $form );
1067
-		if ( $style && isset( $style->post_content[ $val ] ) ) {
1068
-			return $style->post_content[ $val ];
1067
+		if ( $style && isset( $style->post_content[$val] ) ) {
1068
+			return $style->post_content[$val];
1069 1069
 		}
1070 1070
 	}
1071 1071
 
@@ -1084,7 +1084,7 @@  discard block
 block discarded – undo
1084 1084
 			} elseif ( 'alt_bg_color' == $name ) {
1085 1085
 				$setting = 'bg_color_active';
1086 1086
 			}
1087
-			$default_styles[ $name ] = $style->post_content[ $setting ];
1087
+			$default_styles[$name] = $style->post_content[$setting];
1088 1088
 			unset( $name, $val );
1089 1089
 		}
1090 1090
 
Please login to merge, or discard this patch.
classes/models/FrmStyle.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,8 @@
 block discarded – undo
67 67
 			$new_instance = (array) $new_instance;
68 68
 			$this->id     = $new_instance['ID'];
69 69
 
70
-			if ( $id != $this->id || ! $_POST || ! isset( $_POST['frm_style_setting'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
70
+			if ( $id != $this->id || ! $_POST || ! isset( $_POST['frm_style_setting'] ) ) {
71
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
71 72
 				// Don't continue if not saving this style.
72 73
 				continue;
73 74
 			}
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -92,21 +92,21 @@  discard block
 block discarded – undo
92 92
 			$default_settings = $this->get_defaults();
93 93
 
94 94
 			foreach ( $default_settings as $setting => $default ) {
95
-				if ( ! isset( $new_instance['post_content'][ $setting ] ) ) {
96
-					$new_instance['post_content'][ $setting ] = $default;
95
+				if ( ! isset( $new_instance['post_content'][$setting] ) ) {
96
+					$new_instance['post_content'][$setting] = $default;
97 97
 				}
98 98
 
99 99
 				if ( $this->is_color( $setting ) ) {
100
-					$color_val = $new_instance['post_content'][ $setting ];
100
+					$color_val = $new_instance['post_content'][$setting];
101 101
 					if ( $color_val !== '' && false !== strpos( $color_val, 'rgb' ) ) {
102 102
 						// Maybe sanitize if invalid rgba value is entered.
103 103
 						$this->maybe_sanitize_rgba_value( $color_val );
104 104
 					}
105
-					$new_instance['post_content'][ $setting ] = str_replace( '#', '', $color_val );
106
-				} elseif ( in_array( $setting, array( 'submit_style', 'important_style', 'auto_width' ), true ) && ! isset( $new_instance['post_content'][ $setting ] ) ) {
107
-					$new_instance['post_content'][ $setting ] = 0;
105
+					$new_instance['post_content'][$setting] = str_replace( '#', '', $color_val );
106
+				} elseif ( in_array( $setting, array( 'submit_style', 'important_style', 'auto_width' ), true ) && ! isset( $new_instance['post_content'][$setting] ) ) {
107
+					$new_instance['post_content'][$setting] = 0;
108 108
 				} elseif ( $setting === 'font' ) {
109
-					$new_instance['post_content'][ $setting ] = $this->force_balanced_quotation( $new_instance['post_content'][ $setting ] );
109
+					$new_instance['post_content'][$setting] = $this->force_balanced_quotation( $new_instance['post_content'][$setting] );
110 110
 				}
111 111
 			}
112 112
 
@@ -226,10 +226,10 @@  discard block
 block discarded – undo
226 226
 		$valid_keys         = array_keys( $defaults );
227 227
 		$sanitized_settings = array();
228 228
 		foreach ( $valid_keys as $key ) {
229
-			if ( isset( $settings[ $key ] ) ) {
230
-				$sanitized_settings[ $key ] = sanitize_textarea_field( $settings[ $key ] );
229
+			if ( isset( $settings[$key] ) ) {
230
+				$sanitized_settings[$key] = sanitize_textarea_field( $settings[$key] );
231 231
 			} else {
232
-				$sanitized_settings[ $key ] = $defaults[ $key ];
232
+				$sanitized_settings[$key] = $defaults[$key];
233 233
 			}
234 234
 		}
235 235
 		return $sanitized_settings;
@@ -424,13 +424,13 @@  discard block
 block discarded – undo
424 424
 			$style->post_content = $this->override_defaults( $style->post_content );
425 425
 			$style->post_content = wp_parse_args( $style->post_content, $default_values );
426 426
 
427
-			$styles[ $style->ID ] = $style;
427
+			$styles[$style->ID] = $style;
428 428
 		}
429 429
 
430 430
 		if ( ! $default_style ) {
431 431
 			$default_style = reset( $styles );
432 432
 
433
-			$styles[ $default_style->ID ]->menu_order = 1;
433
+			$styles[$default_style->ID]->menu_order = 1;
434 434
 		}
435 435
 
436 436
 		return $styles;
Please login to merge, or discard this patch.
classes/helpers/FrmStylesHelper.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -285,7 +285,8 @@
 block discarded – undo
285 285
 	public static function get_settings_for_output( $style ) {
286 286
 		if ( self::previewing_style() ) {
287 287
 			$frm_style = new FrmStyle();
288
-			if ( isset( $_POST['frm_style_setting'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
288
+			if ( isset( $_POST['frm_style_setting'] ) ) {
289
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
289 290
 
290 291
 				// Sanitizing is done later.
291 292
 				$posted = wp_unslash( $_POST['frm_style_setting'] ); //phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
 				'-' => 'down',
136 136
 				'+' => 'up',
137 137
 			);
138
-			$class = 'frm_arrow' . $arrow[ $icon ];
138
+			$class = 'frm_arrow' . $arrow[$icon];
139 139
 		} else {
140 140
 			//frm_minus1_icon
141 141
 			$key   = str_replace( 'p', '', $key );
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 				'-' => 'minus',
144 144
 				'+' => 'plus',
145 145
 			);
146
-			$class = 'frm_' . $plus[ $icon ];
146
+			$class = 'frm_' . $plus[$icon];
147 147
 		}
148 148
 
149 149
 		if ( $key ) {
@@ -169,8 +169,8 @@  discard block
 block discarded – undo
169 169
 		?>
170 170
 		<div class="btn-group" id="frm_<?php echo esc_attr( $name ); ?>_select">
171 171
 			<button class="multiselect dropdown-toggle btn btn-default" data-toggle="dropdown" type="button">
172
-				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[ $name ], '+', $type ) ); ?>
173
-				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[ $name ], '-', $type ) ); ?>
172
+				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[$name], '+', $type ) ); ?>
173
+				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[$name], '-', $type ) ); ?>
174 174
 				<b class="caret"></b>
175 175
 			</button>
176 176
 			<ul class="multiselect-container frm-dropdown-menu">
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
 					<li <?php echo ( $style->post_content['collapse_icon'] == $key ) ? 'class="active"' : ''; ?>>
179 179
 						<a href="javascript:void(0);">
180 180
 							<label>
181
-								<input type="radio" value="<?php echo esc_attr( $key ); ?>" name="<?php echo esc_attr( $frm_style->get_field_name( $name ) ); ?>" <?php checked( $style->post_content[ $name ], $key ); ?> />
181
+								<input type="radio" value="<?php echo esc_attr( $key ); ?>" name="<?php echo esc_attr( $frm_style->get_field_name( $name ) ); ?>" <?php checked( $style->post_content[$name], $key ); ?> />
182 182
 								<span>
183 183
 									<?php
184 184
 									FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $key, '+', $type ) );
@@ -271,15 +271,15 @@  discard block
 block discarded – undo
271 271
 		$vars   = array_diff( $vars, $remove );
272 272
 
273 273
 		foreach ( $vars as $var ) {
274
-			if ( ! isset( $settings[ $var ] ) ) {
274
+			if ( ! isset( $settings[$var] ) ) {
275 275
 				continue;
276 276
 			}
277
-			if ( ! isset( $defaults[ $var ] ) ) {
278
-				$defaults[ $var ] = '';
277
+			if ( ! isset( $defaults[$var] ) ) {
278
+				$defaults[$var] = '';
279 279
 			}
280
-			$show = empty( $defaults ) || ( $settings[ $var ] !== '' && $settings[ $var ] !== $defaults[ $var ] );
280
+			$show = empty( $defaults ) || ( $settings[$var] !== '' && $settings[$var] !== $defaults[$var] );
281 281
 			if ( $show ) {
282
-				echo '--' . esc_html( str_replace( '_', '-', $var ) ) . ':' . ( $var === 'font' ? FrmAppHelper::kses( $settings[ $var ] ) : esc_html( $settings[ $var ] ) ) . ';'; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
282
+				echo '--' . esc_html( str_replace( '_', '-', $var ) ) . ':' . ( $var === 'font' ? FrmAppHelper::kses( $settings[$var] ) : esc_html( $settings[$var] ) ) . ';'; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
283 283
 			}
284 284
 		}
285 285
 	}
@@ -328,8 +328,8 @@  discard block
 block discarded – undo
328 328
 
329 329
 		$checkbox_opts = array( 'important_style', 'auto_width', 'submit_style', 'collapse_icon', 'center_form' );
330 330
 		foreach ( $checkbox_opts as $opt ) {
331
-			if ( ! isset( $settings[ $opt ] ) ) {
332
-				$settings[ $opt ] = 0;
331
+			if ( ! isset( $settings[$opt] ) ) {
332
+				$settings[$opt] = 0;
333 333
 			}
334 334
 		}
335 335
 
@@ -357,7 +357,7 @@  discard block
 block discarded – undo
357 357
 				$css = '';
358 358
 			}
359 359
 			foreach ( $opts as $opt ) {
360
-				self::get_color_output( $css, $settings[ $opt ] );
360
+				self::get_color_output( $css, $settings[$opt] );
361 361
 			}
362 362
 		}
363 363
 	}
Please login to merge, or discard this patch.
classes/helpers/FrmStylesPreviewHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -168,7 +168,7 @@
 block discarded – undo
168 168
 			return;
169 169
 		}
170 170
 
171
-		unset( $dependencies[ $index ] );
171
+		unset( $dependencies[$index] );
172 172
 		$dependencies = array_values( $dependencies );
173 173
 
174 174
 		$styles->registered['wp-admin']->deps = $dependencies;
Please login to merge, or discard this patch.
classes/helpers/FrmStylesCardHelper.php 2 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -349,12 +349,12 @@
 block discarded – undo
349 349
 		array_walk(
350 350
 			$styles,
351 351
 			/**
352
-			* Echo a style card for a single style in the $styles array.
353
-			*
354
-			* @param WP_Post $style
355
-			* @param int     $count Used for pagination.
356
-			* @return void
357
-			*/
352
+			 * Echo a style card for a single style in the $styles array.
353
+			 *
354
+			 * @param WP_Post $style
355
+			 * @param int     $count Used for pagination.
356
+			 * @return void
357
+			 */
358 358
 			function( $style ) use ( &$count ) {
359 359
 				$hidden = $count > ( self::PAGE_SIZE - 1 );
360 360
 				$this->echo_style_card( $style, $hidden );
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -130,11 +130,11 @@  discard block
 block discarded – undo
130 130
 	private static function has_dark_background( $style ) {
131 131
 		$key = 'fieldset_bg_color';
132 132
 
133
-		if ( empty( $style->post_content[ $key ] ) ) {
133
+		if ( empty( $style->post_content[$key] ) ) {
134 134
 			return false;
135 135
 		}
136 136
 
137
-		$color = $style->post_content[ $key ];
137
+		$color = $style->post_content[$key];
138 138
 
139 139
 		if ( 0 === strpos( $color, 'rgba' ) ) {
140 140
 			preg_match_all( '/([\\d.]+)/', $color, $matches );
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
 					),
205 205
 					'/style-templates/' . $style['slug']
206 206
 				);
207
-				$params['data-requires']    = FrmFormsHelper::get_plan_required( $style );
207
+				$params['data-requires'] = FrmFormsHelper::get_plan_required( $style );
208 208
 				return $params;
209 209
 			};
210 210
 		} else {
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
 				continue;
266 266
 			}
267 267
 
268
-			$value = $style->post_content[ $key ];
268
+			$value = $style->post_content[$key];
269 269
 
270 270
 			$is_hex = in_array( $key, $color_settings, true ) && $value && '#' !== $value[0] && false === strpos( $value, 'rgb' ) && $value !== 'transparent';
271 271
 			if ( $is_hex ) {
@@ -413,7 +413,7 @@  discard block
 block discarded – undo
413 413
 			function( $style ) use ( &$count ) {
414 414
 				$hidden = $count > ( self::PAGE_SIZE - 1 );
415 415
 				$this->echo_style_card( $style, $hidden );
416
-				++$count;
416
+				++ $count;
417 417
 			}
418 418
 		);
419 419
 
Please login to merge, or discard this patch.
classes/helpers/FrmAppHelper.php 1 patch
Spacing   +77 added lines, -77 removed lines patch added patch discarded remove patch
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 			'fill'   => '#4d4d4d',
163 163
 			'orange' => '#f05a24',
164 164
 		);
165
-		$atts     = array_merge( $defaults, $atts );
165
+		$atts = array_merge( $defaults, $atts );
166 166
 
167 167
 		return '<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 599.68 601.37" width="' . esc_attr( $atts['width'] ) . '" height="' . esc_attr( $atts['height'] ) . '">
168 168
 			<path fill="' . esc_attr( $atts['orange'] ) . '" d="M289.6 384h140v76h-140z"/>
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 	 * @return string
370 370
 	 */
371 371
 	public static function get_server_value( $value ) {
372
-		return isset( $_SERVER[ $value ] ) ? wp_strip_all_tags( wp_unslash( $_SERVER[ $value ] ) ) : '';
372
+		return isset( $_SERVER[$value] ) ? wp_strip_all_tags( wp_unslash( $_SERVER[$value] ) ) : '';
373 373
 	}
374 374
 
375 375
 	/**
@@ -392,7 +392,7 @@  discard block
 block discarded – undo
392 392
 		);
393 393
 		$ip = '';
394 394
 		foreach ( $ip_options as $key ) {
395
-			if ( ! isset( $_SERVER[ $key ] ) ) {
395
+			if ( ! isset( $_SERVER[$key] ) ) {
396 396
 				continue;
397 397
 			}
398 398
 
@@ -416,10 +416,10 @@  discard block
 block discarded – undo
416 416
 		}
417 417
 
418 418
 		if ( $src === 'get' ) {
419
-			$value = isset( $_POST[ $param ] ) ? wp_unslash( $_POST[ $param ] ) : ( isset( $_GET[ $param ] ) ? wp_unslash( $_GET[ $param ] ) : $default ); // phpcs:ignore WordPress.Security.NonceVerification.Missing, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
420
-			if ( ! isset( $_POST[ $param ] ) && isset( $_GET[ $param ] ) && ! is_array( $value ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
419
+			$value = isset( $_POST[$param] ) ? wp_unslash( $_POST[$param] ) : ( isset( $_GET[$param] ) ? wp_unslash( $_GET[$param] ) : $default ); // phpcs:ignore WordPress.Security.NonceVerification.Missing, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
420
+			if ( ! isset( $_POST[$param] ) && isset( $_GET[$param] ) && ! is_array( $value ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
421 421
 				// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
422
-				$value = htmlspecialchars_decode( wp_unslash( $_GET[ $param ] ) );
422
+				$value = htmlspecialchars_decode( wp_unslash( $_GET[$param] ) );
423 423
 			}
424 424
 			self::sanitize_value( $sanitize, $value );
425 425
 		} else {
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
 				}
441 441
 
442 442
 				$p     = trim( $p, ']' );
443
-				$value = isset( $value[ $p ] ) ? $value[ $p ] : $default;
443
+				$value = isset( $value[$p] ) ? $value[$p] : $default;
444 444
 			}
445 445
 		}
446 446
 
@@ -496,26 +496,26 @@  discard block
 block discarded – undo
496 496
 			'sanitize' => 'sanitize_text_field',
497 497
 			'serialized' => false,
498 498
 		);
499
-		$args     = wp_parse_args( $args, $defaults );
499
+		$args = wp_parse_args( $args, $defaults );
500 500
 
501 501
 		$value = $args['default'];
502 502
 		if ( $args['type'] == 'get' ) {
503
-			if ( $_GET && isset( $_GET[ $args['param'] ] ) ) {
503
+			if ( $_GET && isset( $_GET[$args['param']] ) ) {
504 504
 				// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
505
-				$value = wp_unslash( $_GET[ $args['param'] ] );
505
+				$value = wp_unslash( $_GET[$args['param']] );
506 506
 			}
507 507
 		} elseif ( $args['type'] == 'post' ) {
508
-			if ( isset( $_POST[ $args['param'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
508
+			if ( isset( $_POST[$args['param']] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
509 509
 				// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
510
-				$value = wp_unslash( $_POST[ $args['param'] ] );
510
+				$value = wp_unslash( $_POST[$args['param']] );
511 511
 				if ( $args['serialized'] === true && is_serialized_string( $value ) && is_serialized( $value ) ) {
512 512
 					self::unserialize_or_decode( $value );
513 513
 				}
514 514
 			}
515 515
 		} else {
516
-			if ( isset( $_REQUEST[ $args['param'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
516
+			if ( isset( $_REQUEST[$args['param']] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
517 517
 				// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized
518
-				$value = wp_unslash( $_REQUEST[ $args['param'] ] );
518
+				$value = wp_unslash( $_REQUEST[$args['param']] );
519 519
 			}
520 520
 		}
521 521
 
@@ -547,7 +547,7 @@  discard block
 block discarded – undo
547 547
 			if ( is_array( $value ) ) {
548 548
 				$temp_values = $value;
549 549
 				foreach ( $temp_values as $k => $v ) {
550
-					self::sanitize_value( $sanitize, $value[ $k ] );
550
+					self::sanitize_value( $sanitize, $value[$k] );
551 551
 				}
552 552
 			} else {
553 553
 				$value = call_user_func( $sanitize, $value );
@@ -558,8 +558,8 @@  discard block
 block discarded – undo
558 558
 	public static function sanitize_request( $sanitize_method, &$values ) {
559 559
 		$temp_values = $values;
560 560
 		foreach ( $temp_values as $k => $val ) {
561
-			if ( isset( $sanitize_method[ $k ] ) ) {
562
-				$values[ $k ] = call_user_func( $sanitize_method[ $k ], $val );
561
+			if ( isset( $sanitize_method[$k] ) ) {
562
+				$values[$k] = call_user_func( $sanitize_method[$k], $val );
563 563
 			}
564 564
 		}
565 565
 	}
@@ -582,7 +582,7 @@  discard block
 block discarded – undo
582 582
 		if ( is_array( $value ) ) {
583 583
 			$temp_values = $value;
584 584
 			foreach ( $temp_values as $k => $v ) {
585
-				self::decode_specialchars( $value[ $k ] );
585
+				self::decode_specialchars( $value[$k] );
586 586
 			}
587 587
 		} else {
588 588
 			self::decode_amp( $value );
@@ -702,7 +702,7 @@  discard block
 block discarded – undo
702 702
 	 * @return array
703 703
 	 */
704 704
 	public static function add_allowed_submit_button_tags( $allowed_html ) {
705
-		$allowed_html['input']                    = array(
705
+		$allowed_html['input'] = array(
706 706
 			'type'           => true,
707 707
 			'value'          => true,
708 708
 			'formnovalidate' => true,
@@ -725,7 +725,7 @@  discard block
 block discarded – undo
725 725
 			$allowed_html = $html;
726 726
 		} elseif ( ! empty( $allowed ) ) {
727 727
 			foreach ( (array) $allowed as $a ) {
728
-				$allowed_html[ $a ] = isset( $html[ $a ] ) ? $html[ $a ] : array();
728
+				$allowed_html[$a] = isset( $html[$a] ) ? $html[$a] : array();
729 729
 			}
730 730
 		}
731 731
 
@@ -904,8 +904,8 @@  discard block
 block discarded – undo
904 904
 		}
905 905
 
906 906
 		global $wp_query;
907
-		if ( isset( $wp_query->query_vars[ $param ] ) ) {
908
-			$value = $wp_query->query_vars[ $param ];
907
+		if ( isset( $wp_query->query_vars[$param] ) ) {
908
+			$value = $wp_query->query_vars[$param];
909 909
 		}
910 910
 
911 911
 		return $value;
@@ -932,9 +932,9 @@  discard block
 block discarded – undo
932 932
 		$deprecated = array(
933 933
 			'frm_clone_solid_icon' => 'frm_clone_icon',
934 934
 		);
935
-		if ( isset( $deprecated[ $icon ] ) ) {
936
-			$icon = $deprecated[ $icon ];
937
-			$class = str_replace( $icon, $deprecated[ $icon ], $class );
935
+		if ( isset( $deprecated[$icon] ) ) {
936
+			$icon = $deprecated[$icon];
937
+			$class = str_replace( $icon, $deprecated[$icon], $class );
938 938
 		}
939 939
 
940 940
 		if ( $icon === $class ) {
@@ -1255,7 +1255,7 @@  discard block
 block discarded – undo
1255 1255
 				'new_file_path' => self::plugin_path() . '/js',
1256 1256
 			)
1257 1257
 		);
1258
-		$new_file  = new FrmCreateFile( $file_atts );
1258
+		$new_file = new FrmCreateFile( $file_atts );
1259 1259
 
1260 1260
 		$files = array(
1261 1261
 			self::plugin_path() . '/js/formidable.min.js',
@@ -1717,8 +1717,8 @@  discard block
 block discarded – undo
1717 1717
 			return $error;
1718 1718
 		}
1719 1719
 
1720
-		$nonce_value = ( $_REQUEST && isset( $_REQUEST[ $nonce_name ] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST[ $nonce_name ] ) ) : '';
1721
-		if ( $_REQUEST && ( ! isset( $_REQUEST[ $nonce_name ] ) || ! wp_verify_nonce( $nonce_value, $nonce ) ) ) {
1720
+		$nonce_value = ( $_REQUEST && isset( $_REQUEST[$nonce_name] ) ) ? sanitize_text_field( wp_unslash( $_REQUEST[$nonce_name] ) ) : '';
1721
+		if ( $_REQUEST && ( ! isset( $_REQUEST[$nonce_name] ) || ! wp_verify_nonce( $nonce_value, $nonce ) ) ) {
1722 1722
 			$frm_settings = self::get_settings();
1723 1723
 			$error        = $frm_settings->admin_permission;
1724 1724
 		}
@@ -1753,7 +1753,7 @@  discard block
 block discarded – undo
1753 1753
 			} else {
1754 1754
 				foreach ( $value as $k => $v ) {
1755 1755
 					if ( ! is_array( $v ) ) {
1756
-						$value[ $k ] = call_user_func( $original_function, $v );
1756
+						$value[$k] = call_user_func( $original_function, $v );
1757 1757
 					}
1758 1758
 				}
1759 1759
 			}
@@ -1778,7 +1778,7 @@  discard block
 block discarded – undo
1778 1778
 				$return = array_merge( $return, self::array_flatten( $value, $keys ) );
1779 1779
 			} else {
1780 1780
 				if ( $keys == 'keep' ) {
1781
-					$return[ $key ] = $value;
1781
+					$return[$key] = $value;
1782 1782
 				} else {
1783 1783
 					$return[] = $value;
1784 1784
 				}
@@ -1831,11 +1831,11 @@  discard block
 block discarded – undo
1831 1831
 		}
1832 1832
 
1833 1833
 		$ver = $default;
1834
-		if ( ! isset( $wp_scripts->registered[ $handle ] ) ) {
1834
+		if ( ! isset( $wp_scripts->registered[$handle] ) ) {
1835 1835
 			return $ver;
1836 1836
 		}
1837 1837
 
1838
-		$query = $wp_scripts->registered[ $handle ];
1838
+		$query = $wp_scripts->registered[$handle];
1839 1839
 		if ( is_object( $query ) && ! empty( $query->ver ) ) {
1840 1840
 			$ver = $query->ver;
1841 1841
 		}
@@ -1941,7 +1941,7 @@  discard block
 block discarded – undo
1941 1941
 			$suffix = 2;
1942 1942
 			do {
1943 1943
 				$key_check = $key . $separator . $suffix;
1944
-				++$suffix;
1944
+				++ $suffix;
1945 1945
 			} while ( in_array( $key_check, $similar_keys, true ) );
1946 1946
 
1947 1947
 			$key = $key_check;
@@ -2042,7 +2042,7 @@  discard block
 block discarded – undo
2042 2042
 
2043 2043
 		foreach ( array( 'name', 'description' ) as $var ) {
2044 2044
 			$default_val    = isset( $record->{$var} ) ? $record->{$var} : '';
2045
-			$values[ $var ] = self::get_param( $var, $default_val, 'get', 'wp_kses_post' );
2045
+			$values[$var] = self::get_param( $var, $default_val, 'get', 'wp_kses_post' );
2046 2046
 			unset( $var, $default_val );
2047 2047
 		}
2048 2048
 
@@ -2100,9 +2100,9 @@  discard block
 block discarded – undo
2100 2100
 			}
2101 2101
 		}
2102 2102
 
2103
-		$field_type = isset( $post_values['field_options'][ 'type_' . $field->id ] ) ? $post_values['field_options'][ 'type_' . $field->id ] : $field->type;
2104
-		if ( isset( $post_values['item_meta'][ $field->id ] ) ) {
2105
-			$new_value = $post_values['item_meta'][ $field->id ];
2103
+		$field_type = isset( $post_values['field_options']['type_' . $field->id] ) ? $post_values['field_options']['type_' . $field->id] : $field->type;
2104
+		if ( isset( $post_values['item_meta'][$field->id] ) ) {
2105
+			$new_value = $post_values['item_meta'][$field->id];
2106 2106
 			self::unserialize_or_decode( $new_value );
2107 2107
 		} else {
2108 2108
 			$new_value = $meta_value;
@@ -2123,7 +2123,7 @@  discard block
 block discarded – undo
2123 2123
 
2124 2124
 		$field_array = array_merge( (array) $field->field_options, $field_array );
2125 2125
 
2126
-		$values['fields'][ $field->id ] = $field_array;
2126
+		$values['fields'][$field->id] = $field_array;
2127 2127
 	}
2128 2128
 
2129 2129
 	/**
@@ -2170,11 +2170,11 @@  discard block
 block discarded – undo
2170 2170
 		}
2171 2171
 
2172 2172
 		foreach ( $form->options as $opt => $value ) {
2173
-			if ( isset( $post_values[ $opt ] ) ) {
2174
-				$values[ $opt ] = $post_values[ $opt ];
2175
-				self::unserialize_or_decode( $values[ $opt ] );
2173
+			if ( isset( $post_values[$opt] ) ) {
2174
+				$values[$opt] = $post_values[$opt];
2175
+				self::unserialize_or_decode( $values[$opt] );
2176 2176
 			} else {
2177
-				$values[ $opt ] = $value;
2177
+				$values[$opt] = $value;
2178 2178
 			}
2179 2179
 		}
2180 2180
 
@@ -2188,8 +2188,8 @@  discard block
 block discarded – undo
2188 2188
 		$form_defaults = FrmFormsHelper::get_default_opts();
2189 2189
 
2190 2190
 		foreach ( $form_defaults as $opt => $default ) {
2191
-			if ( ! isset( $values[ $opt ] ) || $values[ $opt ] == '' ) {
2192
-				$values[ $opt ] = ( $post_values && isset( $post_values['options'][ $opt ] ) ) ? $post_values['options'][ $opt ] : $default;
2191
+			if ( ! isset( $values[$opt] ) || $values[$opt] == '' ) {
2192
+				$values[$opt] = ( $post_values && isset( $post_values['options'][$opt] ) ) ? $post_values['options'][$opt] : $default;
2193 2193
 			}
2194 2194
 
2195 2195
 			unset( $opt, $default );
@@ -2200,8 +2200,8 @@  discard block
 block discarded – undo
2200 2200
 		}
2201 2201
 
2202 2202
 		foreach ( array( 'before', 'after', 'submit' ) as $h ) {
2203
-			if ( ! isset( $values[ $h . '_html' ] ) ) {
2204
-				$values[ $h . '_html' ] = ( isset( $post_values['options'][ $h . '_html' ] ) ? $post_values['options'][ $h . '_html' ] : FrmFormsHelper::get_default_html( $h ) );
2203
+			if ( ! isset( $values[$h . '_html'] ) ) {
2204
+				$values[$h . '_html'] = ( isset( $post_values['options'][$h . '_html'] ) ? $post_values['options'][$h . '_html'] : FrmFormsHelper::get_default_html( $h ) );
2205 2205
 			}
2206 2206
 			unset( $h );
2207 2207
 		}
@@ -2353,25 +2353,25 @@  discard block
 block discarded – undo
2353 2353
 		if ( ! is_numeric( $levels ) ) {
2354 2354
 			// Show time in specified unit.
2355 2355
 			$levels = self::get_unit( $levels );
2356
-			if ( isset( $time_strings[ $levels ] ) ) {
2356
+			if ( isset( $time_strings[$levels] ) ) {
2357 2357
 				$diff = array(
2358 2358
 					$levels => self::time_format( $levels, $diff ),
2359 2359
 				);
2360 2360
 				$time_strings = array(
2361
-					$levels => $time_strings[ $levels ],
2361
+					$levels => $time_strings[$levels],
2362 2362
 				);
2363 2363
 			}
2364 2364
 			$levels = 1;
2365 2365
 		}
2366 2366
 
2367 2367
 		foreach ( $time_strings as $k => $v ) {
2368
-			if ( isset( $diff[ $k ] ) && $diff[ $k ] ) {
2369
-				$time_strings[ $k ] = $diff[ $k ] . ' ' . ( $diff[ $k ] > 1 ? $v[1] : $v[0] );
2370
-			} elseif ( isset( $diff[ $k ] ) && count( $time_strings ) === 1 ) {
2368
+			if ( isset( $diff[$k] ) && $diff[$k] ) {
2369
+				$time_strings[$k] = $diff[$k] . ' ' . ( $diff[$k] > 1 ? $v[1] : $v[0] );
2370
+			} elseif ( isset( $diff[$k] ) && count( $time_strings ) === 1 ) {
2371 2371
 				// Account for 0.
2372
-				$time_strings[ $k ] = $diff[ $k ] . ' ' . $v[1];
2372
+				$time_strings[$k] = $diff[$k] . ' ' . $v[1];
2373 2373
 			} else {
2374
-				unset( $time_strings[ $k ] );
2374
+				unset( $time_strings[$k] );
2375 2375
 			}
2376 2376
 		}
2377 2377
 
@@ -2390,8 +2390,8 @@  discard block
 block discarded – undo
2390 2390
 			'y' => 'y',
2391 2391
 			'd' => 'days',
2392 2392
 		);
2393
-		if ( isset( $return[ $unit ] ) ) {
2394
-			return $diff[ $return[ $unit ] ];
2393
+		if ( isset( $return[$unit] ) ) {
2394
+			return $diff[$return[$unit]];
2395 2395
 		}
2396 2396
 
2397 2397
 		$total = $diff['days'] * self::convert_time( 'd', $unit );
@@ -2399,11 +2399,11 @@  discard block
 block discarded – undo
2399 2399
 		$times = array( 'h', 'i', 's' );
2400 2400
 
2401 2401
 		foreach ( $times as $time ) {
2402
-			if ( ! isset( $diff[ $time ] ) ) {
2402
+			if ( ! isset( $diff[$time] ) ) {
2403 2403
 				continue;
2404 2404
 			}
2405 2405
 
2406
-			$total += $diff[ $time ] * self::convert_time( $time, $unit );
2406
+			$total += $diff[$time] * self::convert_time( $time, $unit );
2407 2407
 		}
2408 2408
 
2409 2409
 		return floor( $total );
@@ -2423,7 +2423,7 @@  discard block
 block discarded – undo
2423 2423
 			'y' => DAY_IN_SECONDS * 365.25,
2424 2424
 		);
2425 2425
 
2426
-		return $convert[ $from ] / $convert[ $to ];
2426
+		return $convert[$from] / $convert[$to];
2427 2427
 	}
2428 2428
 
2429 2429
 	/**
@@ -2431,7 +2431,7 @@  discard block
 block discarded – undo
2431 2431
 	 */
2432 2432
 	private static function get_unit( $unit ) {
2433 2433
 		$units = self::get_time_strings();
2434
-		if ( isset( $units[ $unit ] ) || is_numeric( $unit ) ) {
2434
+		if ( isset( $units[$unit] ) || is_numeric( $unit ) ) {
2435 2435
 			return $unit;
2436 2436
 		}
2437 2437
 
@@ -2542,17 +2542,17 @@  discard block
 block discarded – undo
2542 2542
 
2543 2543
 					case 1:
2544 2544
 						$l2 = $name;
2545
-						self::add_value_to_array( $name, $l2, $this_val, $vars[ $l1 ] );
2545
+						self::add_value_to_array( $name, $l2, $this_val, $vars[$l1] );
2546 2546
 						break;
2547 2547
 
2548 2548
 					case 2:
2549 2549
 						$l3 = $name;
2550
-						self::add_value_to_array( $name, $l3, $this_val, $vars[ $l1 ][ $l2 ] );
2550
+						self::add_value_to_array( $name, $l3, $this_val, $vars[$l1][$l2] );
2551 2551
 						break;
2552 2552
 
2553 2553
 					case 3:
2554 2554
 						$l4 = $name;
2555
-						self::add_value_to_array( $name, $l4, $this_val, $vars[ $l1 ][ $l2 ][ $l3 ] );
2555
+						self::add_value_to_array( $name, $l4, $this_val, $vars[$l1][$l2][$l3] );
2556 2556
 				}
2557 2557
 
2558 2558
 				unset( $this_val, $n );
@@ -2571,8 +2571,8 @@  discard block
 block discarded – undo
2571 2571
 	public static function add_value_to_array( $name, $l1, $val, &$vars ) {
2572 2572
 		if ( $name == '' ) {
2573 2573
 			$vars[] = $val;
2574
-		} elseif ( ! isset( $vars[ $l1 ] ) ) {
2575
-			$vars[ $l1 ] = $val;
2574
+		} elseif ( ! isset( $vars[$l1] ) ) {
2575
+			$vars[$l1] = $val;
2576 2576
 		}
2577 2577
 	}
2578 2578
 
@@ -2588,7 +2588,7 @@  discard block
 block discarded – undo
2588 2588
 			'email_subject' => esc_attr( sprintf( __( 'If you leave the subject blank, the default will be used: %1$s Form submitted on %2$s', 'formidable' ), $form_name, self::site_name() ) ),
2589 2589
 		);
2590 2590
 
2591
-		if ( ! isset( $tooltips[ $name ] ) ) {
2591
+		if ( ! isset( $tooltips[$name] ) ) {
2592 2592
 			return;
2593 2593
 		}
2594 2594
 
@@ -2598,7 +2598,7 @@  discard block
 block discarded – undo
2598 2598
 			echo ' class="frm_help"';
2599 2599
 		}
2600 2600
 
2601
-		echo ' title="' . esc_attr( $tooltips[ $name ] );
2601
+		echo ' title="' . esc_attr( $tooltips[$name] );
2602 2602
 
2603 2603
 		if ( 'open' != $class ) {
2604 2604
 			echo '"';
@@ -2657,13 +2657,13 @@  discard block
 block discarded – undo
2657 2657
 	}
2658 2658
 
2659 2659
 	private static function prepare_action_slashes( $val, $key, &$post_content ) {
2660
-		if ( ! isset( $post_content[ $key ] ) || is_numeric( $val ) ) {
2660
+		if ( ! isset( $post_content[$key] ) || is_numeric( $val ) ) {
2661 2661
 			return;
2662 2662
 		}
2663 2663
 
2664 2664
 		if ( is_array( $val ) ) {
2665 2665
 			foreach ( $val as $k1 => $v1 ) {
2666
-				self::prepare_action_slashes( $v1, $k1, $post_content[ $key ] );
2666
+				self::prepare_action_slashes( $v1, $k1, $post_content[$key] );
2667 2667
 				unset( $k1, $v1 );
2668 2668
 			}
2669 2669
 		} else {
@@ -2671,7 +2671,7 @@  discard block
 block discarded – undo
2671 2671
 			$val = stripslashes( $val );
2672 2672
 
2673 2673
 			// Add backslashes before double quotes and forward slashes only
2674
-			$post_content[ $key ] = addcslashes( $val, '"\\/' );
2674
+			$post_content[$key] = addcslashes( $val, '"\\/' );
2675 2675
 		}
2676 2676
 	}
2677 2677
 
@@ -2734,14 +2734,14 @@  discard block
 block discarded – undo
2734 2734
 				continue;
2735 2735
 			}
2736 2736
 			$key = $input['name'];
2737
-			if ( isset( $formatted[ $key ] ) ) {
2738
-				if ( is_array( $formatted[ $key ] ) ) {
2739
-					$formatted[ $key ][] = $input['value'];
2737
+			if ( isset( $formatted[$key] ) ) {
2738
+				if ( is_array( $formatted[$key] ) ) {
2739
+					$formatted[$key][] = $input['value'];
2740 2740
 				} else {
2741
-					$formatted[ $key ] = array( $formatted[ $key ], $input['value'] );
2741
+					$formatted[$key] = array( $formatted[$key], $input['value'] );
2742 2742
 				}
2743 2743
 			} else {
2744
-				$formatted[ $key ] = $input['value'];
2744
+				$formatted[$key] = $input['value'];
2745 2745
 			}
2746 2746
 		}
2747 2747
 
@@ -3403,8 +3403,8 @@  discard block
 block discarded – undo
3403 3403
 		}
3404 3404
 
3405 3405
 		foreach ( $keys as $key ) {
3406
-			if ( isset( $values[ $key ] ) ) {
3407
-				$values[ $key ] = self::kses( $values[ $key ], 'all' );
3406
+			if ( isset( $values[$key] ) ) {
3407
+				$values[$key] = self::kses( $values[$key], 'all' );
3408 3408
 			}
3409 3409
 		}
3410 3410
 
@@ -3563,7 +3563,7 @@  discard block
 block discarded – undo
3563 3563
 			return 0;
3564 3564
 		}
3565 3565
 
3566
-		return strlen( $parts[ count( $parts ) - 1 ] );
3566
+		return strlen( $parts[count( $parts ) - 1] );
3567 3567
 	}
3568 3568
 
3569 3569
 	/**
Please login to merge, or discard this patch.
classes/models/FrmStyleApi.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
 		// Remove 'Styling Template' from titles.
51 51
 		foreach ( $api_info as $id => $template ) {
52 52
 			if ( isset( $template['name'] ) ) {
53
-				$api_info[ $id ]['name'] = preg_replace( '/(\sStyle|Styling)?(\sTemplate)?$/', '', $template['name'] );
53
+				$api_info[$id]['name'] = preg_replace( '/(\sStyle|Styling)?(\sTemplate)?$/', '', $template['name'] );
54 54
 			}
55 55
 		}
56 56
 
Please login to merge, or discard this patch.
classes/models/fields/FrmFieldType.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -132,8 +132,8 @@  discard block
 block discarded – undo
132 132
 		$field_val = '';
133 133
 		if ( is_object( $this->field ) ) {
134 134
 			$field_val = $this->field->{$column};
135
-		} elseif ( is_array( $this->field ) && isset( $this->field[ $column ] ) ) {
136
-			$field_val = $this->field[ $column ];
135
+		} elseif ( is_array( $this->field ) && isset( $this->field[$column] ) ) {
136
+			$field_val = $this->field[$column];
137 137
 		}
138 138
 
139 139
 		return $field_val;
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 		if ( is_object( $this->field ) ) {
148 148
 			$this->field->{$column} = $value;
149 149
 		} elseif ( is_array( $this->field ) ) {
150
-			$this->field[ $column ] = $value;
150
+			$this->field[$column] = $value;
151 151
 		}
152 152
 	}
153 153
 
@@ -527,7 +527,7 @@  discard block
 block discarded – undo
527 527
 			printf(
528 528
 				/* translators: %s: Field type */
529 529
 				esc_html__( '%s Options', 'formidable' ),
530
-				esc_html( $all_field_types[ $args['display']['type'] ]['name'] )
530
+				esc_html( $all_field_types[$args['display']['type']]['name'] )
531 531
 			);
532 532
 			FrmAppHelper::icon_by_class( 'frmfont frm_arrowdown6_icon', array( 'aria-hidden' => 'true' ) );
533 533
 			?>
@@ -650,8 +650,8 @@  discard block
 block discarded – undo
650 650
 		$fields = FrmField::field_selection();
651 651
 		$fields = array_merge( $fields, FrmField::pro_field_selection() );
652 652
 
653
-		if ( isset( $fields[ $this->type ] ) ) {
654
-			$name = is_array( $fields[ $this->type ] ) ? $fields[ $this->type ]['name'] : $fields[ $this->type ];
653
+		if ( isset( $fields[$this->type] ) ) {
654
+			$name = is_array( $fields[$this->type] ) ? $fields[$this->type]['name'] : $fields[$this->type];
655 655
 		}
656 656
 
657 657
 		return $name;
@@ -662,7 +662,7 @@  discard block
 block discarded – undo
662 662
 	}
663 663
 
664 664
 	public function get_default_field_options() {
665
-		$opts       = array(
665
+		$opts = array(
666 666
 			'size'               => '',
667 667
 			'max'                => '',
668 668
 			'label'              => '',
@@ -1004,7 +1004,7 @@  discard block
 block discarded – undo
1004 1004
 			}
1005 1005
 		} else {
1006 1006
 			$args['save_array'] = $this->is_readonly_array();
1007
-			$hidden             .= $this->show_single_hidden( $selected_value, $args );
1007
+			$hidden .= $this->show_single_hidden( $selected_value, $args );
1008 1008
 		}
1009 1009
 
1010 1010
 		return $hidden;
@@ -1013,7 +1013,7 @@  discard block
 block discarded – undo
1013 1013
 	protected function show_single_hidden( $selected, $args ) {
1014 1014
 		if ( $args['save_array'] ) {
1015 1015
 			$args['field_name'] .= '[]';
1016
-			$id                 = '';
1016
+			$id = '';
1017 1017
 		} else {
1018 1018
 			$id = ' id="' . esc_attr( $args['html_id'] ) . '"';
1019 1019
 		}
@@ -1029,8 +1029,8 @@  discard block
 block discarded – undo
1029 1029
 		$selected = $values['field_value'];
1030 1030
 
1031 1031
 		if ( isset( $values['combo_name'] ) ) {
1032
-			$options  = $options[ $values['combo_name'] ];
1033
-			$selected = ( is_array( $selected ) && isset( $selected[ $values['combo_name'] ] ) ) ? $selected[ $values['combo_name'] ] : '';
1032
+			$options  = $options[$values['combo_name']];
1033
+			$selected = ( is_array( $selected ) && isset( $selected[$values['combo_name']] ) ) ? $selected[$values['combo_name']] : '';
1034 1034
 		}
1035 1035
 
1036 1036
 		$input = $this->select_tag( $values );
@@ -1081,7 +1081,7 @@  discard block
 block discarded – undo
1081 1081
 	}
1082 1082
 
1083 1083
 	protected function fill_display_field_values( $args = array() ) {
1084
-		$defaults        = array(
1084
+		$defaults = array(
1085 1085
 			'field_name'    => 'item_meta[' . $this->get_field_column( 'id' ) . ']',
1086 1086
 			'field_id'      => $this->get_field_column( 'id' ),
1087 1087
 			'field_plus_id' => '',
@@ -1134,7 +1134,7 @@  discard block
 block discarded – undo
1134 1134
 			}
1135 1135
 		}
1136 1136
 
1137
-		if ( isset( $args['errors'][ 'field' . $args['field_id'] ] ) && ! $custom_error_fields ) {
1137
+		if ( isset( $args['errors']['field' . $args['field_id']] ) && ! $custom_error_fields ) {
1138 1138
 			if ( $error_comes_first ) {
1139 1139
 				array_unshift( $describedby, 'frm_error_' . $args['html_id'] );
1140 1140
 			} else {
@@ -1213,11 +1213,11 @@  discard block
 block discarded – undo
1213 1213
 
1214 1214
 		$field_id = $this->get_field_column( 'id' );
1215 1215
 		if ( ! array_key_exists( $field_id, $frm_validated_unique_values ) ) {
1216
-			$frm_validated_unique_values[ $field_id ] = array();
1216
+			$frm_validated_unique_values[$field_id] = array();
1217 1217
 			return false;
1218 1218
 		}
1219 1219
 
1220
-		$already_validated_this_value = in_array( $value, $frm_validated_unique_values[ $field_id ], true );
1220
+		$already_validated_this_value = in_array( $value, $frm_validated_unique_values[$field_id], true );
1221 1221
 		return $already_validated_this_value;
1222 1222
 	}
1223 1223
 
@@ -1242,7 +1242,7 @@  discard block
 block discarded – undo
1242 1242
 	private function value_validated_as_unique( $value ) {
1243 1243
 		global $frm_validated_unique_values;
1244 1244
 		$field_id                                   = $this->get_field_column( 'id' );
1245
-		$frm_validated_unique_values[ $field_id ][] = $value;
1245
+		$frm_validated_unique_values[$field_id][] = $value;
1246 1246
 	}
1247 1247
 
1248 1248
 	public function get_value_to_save( $value, $atts ) {
@@ -1274,8 +1274,8 @@  discard block
 block discarded – undo
1274 1274
 		$value = $this->prepare_display_value( $value, $atts );
1275 1275
 
1276 1276
 		if ( is_array( $value ) ) {
1277
-			if ( isset( $atts['show'] ) && $atts['show'] && isset( $value[ $atts['show'] ] ) ) {
1278
-				$value = $value[ $atts['show'] ];
1277
+			if ( isset( $atts['show'] ) && $atts['show'] && isset( $value[$atts['show']] ) ) {
1278
+				$value = $value[$atts['show']];
1279 1279
 			} elseif ( ! isset( $atts['return_array'] ) || ! $atts['return_array'] ) {
1280 1280
 				$sep   = isset( $atts['sep'] ) ? $atts['sep'] : ', ';
1281 1281
 				$value = implode( $sep, $value );
@@ -1344,8 +1344,8 @@  discard block
 block discarded – undo
1344 1344
 		$saved_entries = $atts['ids'];
1345 1345
 		$new_value     = array();
1346 1346
 		foreach ( (array) $value as $old_child_id ) {
1347
-			if ( isset( $saved_entries[ $old_child_id ] ) ) {
1348
-				$new_value[] = $saved_entries[ $old_child_id ];
1347
+			if ( isset( $saved_entries[$old_child_id] ) ) {
1348
+				$new_value[] = $saved_entries[$old_child_id];
1349 1349
 			}
1350 1350
 		}
1351 1351
 
Please login to merge, or discard this patch.
classes/views/styles/_style-card.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 				'text-color'      => $style->post_content['text_color'],
41 41
 				'submit-bg-color' => $style->post_content['submit_bg_color'],
42 42
 			);
43
-			$index  = 0;
43
+			$index = 0;
44 44
 			foreach ( $colors as $css_var_name => $color ) {
45 45
 				if ( 0 !== strpos( $color, 'rgb' ) ) {
46 46
 					$color = '#' . $color;
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 					'style' => 'background-color: var(--' . $css_var_name . ')',
52 52
 				);
53 53
 
54
-				++$index;
54
+				++ $index;
55 55
 				?>
56 56
 				<div <?php FrmAppHelper::array_to_html_params( $circle_params, true ); ?>></div>
57 57
 				<?php
Please login to merge, or discard this patch.