Completed
Pull Request — master (#1839)
by Stephanie
45s
created
classes/models/FrmEmail.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 		$to = $this->prepare_email_setting( $this->settings['email_to'], $user_id_args );
163 163
 		$to = $this->explode_emails( $to );
164 164
 
165
-		$where  = array(
165
+		$where = array(
166 166
 			'it.field_id !' => 0,
167 167
 			'it.item_id'    => $this->entry->id,
168 168
 		);
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
 			'entry'     => $this->entry,
173 173
 			'form'      => $this->form,
174 174
 		);
175
-		$to     = apply_filters( 'frm_to_email', $to, $values, $this->form->id, $args );
175
+		$to = apply_filters( 'frm_to_email', $to, $values, $this->form->id, $args );
176 176
 
177 177
 		$this->to = array_unique( (array) $to );
178 178
 
@@ -702,11 +702,11 @@  discard block
 block discarded – undo
702 702
 				$name = trim( str_replace( $email, '', $val ) );
703 703
 			} else {
704 704
 				// If user enters a name without an email
705
-				unset( $recipients[ $key ] );
705
+				unset( $recipients[$key] );
706 706
 				continue;
707 707
 			}
708 708
 
709
-			$recipients[ $key ] = $this->format_from_email( $name, $email );
709
+			$recipients[$key] = $this->format_from_email( $name, $email );
710 710
 		}//end foreach
711 711
 
712 712
 		return $recipients;
@@ -856,7 +856,7 @@  discard block
 block discarded – undo
856 856
 				);
857 857
 
858 858
 				// Remove phone number from to addresses
859
-				unset( $this->to[ $key ] );
859
+				unset( $this->to[$key] );
860 860
 			}//end if
861 861
 		}//end foreach
862 862
 	}
Please login to merge, or discard this patch.
classes/helpers/FrmFieldsHelper.php 2 patches
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -228,7 +228,8 @@  discard block
 block discarded – undo
228 228
 	 * @param mixed  $value
229 229
 	 */
230 230
 	private static function get_posted_field_setting( $setting, &$value ) {
231
-		if ( ! isset( $_POST['field_options'][ $setting ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
231
+		if ( ! isset( $_POST['field_options'][ $setting ] ) ) {
232
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
232 233
 			return;
233 234
 		}
234 235
 
@@ -1251,7 +1252,8 @@  discard block
 block discarded – undo
1251 1252
 
1252 1253
 		// Check posted vals before checking saved values
1253 1254
 		// For fields inside repeating sections - note, don't check if $pointer is true because it will often be zero
1254
-		if ( $parent && isset( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
1255
+		if ( $parent && isset( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ] ) ) {
1256
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
1255 1257
 			if ( FrmField::is_field_with_multiple_values( $field ) ) {
1256 1258
 				// phpcs:ignore WordPress.Security.NonceVerification.Missing
1257 1259
 				$other_val = isset( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ][ $opt_key ] ) ? sanitize_text_field( wp_unslash( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ][ $opt_key ] ) ) : '';
@@ -1262,7 +1264,8 @@  discard block
 block discarded – undo
1262 1264
 			return $other_val;
1263 1265
 		}
1264 1266
 
1265
-		if ( isset( $field['id'] ) && isset( $_POST['item_meta']['other'][ $field['id'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
1267
+		if ( isset( $field['id'] ) && isset( $_POST['item_meta']['other'][ $field['id'] ] ) ) {
1268
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
1266 1269
 			// For normal fields
1267 1270
 
1268 1271
 			if ( FrmField::is_field_with_multiple_values( $field ) ) {
Please login to merge, or discard this patch.
Spacing   +48 added lines, -48 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 			if ( in_array( $type, array( 'data', 'lookup' ), true ) ) {
32 32
 				$values['field_options']['data_type'] = $setting;
33 33
 			} else {
34
-				$values['field_options'][ $setting ] = 1;
34
+				$values['field_options'][$setting] = 1;
35 35
 			}
36 36
 		}
37 37
 
@@ -149,10 +149,10 @@  discard block
 block discarded – undo
149 149
 		}
150 150
 
151 151
 		foreach ( $defaults as $opt => $default ) {
152
-			$values[ $opt ] = isset( $field->field_options[ $opt ] ) ? $field->field_options[ $opt ] : $default;
152
+			$values[$opt] = isset( $field->field_options[$opt] ) ? $field->field_options[$opt] : $default;
153 153
 
154 154
 			if ( $check_post ) {
155
-				self::get_posted_field_setting( $opt . '_' . $field->id, $values[ $opt ] );
155
+				self::get_posted_field_setting( $opt . '_' . $field->id, $values[$opt] );
156 156
 			}
157 157
 
158 158
 			unset( $opt, $default );
@@ -228,18 +228,18 @@  discard block
 block discarded – undo
228 228
 	 * @param mixed  $value
229 229
 	 */
230 230
 	private static function get_posted_field_setting( $setting, &$value ) {
231
-		if ( ! isset( $_POST['field_options'][ $setting ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
231
+		if ( ! isset( $_POST['field_options'][$setting] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
232 232
 			return;
233 233
 		}
234 234
 
235 235
 		if ( strpos( $setting, 'html' ) !== false ) {
236 236
 			// Strip slashes from HTML but not regex or script tags.
237
-			$value = wp_unslash( $_POST['field_options'][ $setting ] ); // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
237
+			$value = wp_unslash( $_POST['field_options'][$setting] ); // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
238 238
 		} elseif ( strpos( $setting, 'format_' ) === 0 ) {
239 239
 			// TODO: Remove stripslashes on output, and use on input only.
240
-			$value = sanitize_text_field( $_POST['field_options'][ $setting ] ); // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.NonceVerification.Missing
240
+			$value = sanitize_text_field( $_POST['field_options'][$setting] ); // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.NonceVerification.Missing
241 241
 		} else {
242
-			$value = wp_unslash( $_POST['field_options'][ $setting ] ); // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
242
+			$value = wp_unslash( $_POST['field_options'][$setting] ); // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
243 243
 			FrmAppHelper::sanitize_value( 'wp_kses_post', $value );
244 244
 		}
245 245
 	}
@@ -312,7 +312,7 @@  discard block
 block discarded – undo
312 312
 		$values['default_value'] = FrmAppHelper::maybe_json_encode( $field->default_value );
313 313
 
314 314
 		foreach ( array( 'name', 'description', 'type', 'field_order', 'field_options', 'required' ) as $col ) {
315
-			$values[ $col ] = $field->{$col};
315
+			$values[$col] = $field->{$col};
316 316
 		}
317 317
 	}
318 318
 
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
 		);
351 351
 
352 352
 		$msg = FrmField::get_option( $field, $error );
353
-		$msg = empty( $msg ) ? $defaults[ $error ]['part'] : $msg;
353
+		$msg = empty( $msg ) ? $defaults[$error]['part'] : $msg;
354 354
 		$msg = do_shortcode( $msg );
355 355
 
356 356
 		$msg = self::maybe_replace_substrings_with_field_name( $msg, $error, $field );
@@ -618,7 +618,7 @@  discard block
 block discarded – undo
618 618
 			'title'        => '',
619 619
 			'inside_class' => 'inside',
620 620
 		);
621
-		$args     = array_merge( $defaults, $args );
621
+		$args = array_merge( $defaults, $args );
622 622
 
623 623
 		include FrmAppHelper::plugin_path() . '/classes/views/frm-fields/back-end/inline-modal.php';
624 624
 	}
@@ -853,7 +853,7 @@  discard block
 block discarded – undo
853 853
 				continue;
854 854
 			}
855 855
 
856
-			$atts = FrmShortcodeHelper::get_shortcode_attribute_array( $shortcodes[3][ $short_key ] );
856
+			$atts = FrmShortcodeHelper::get_shortcode_attribute_array( $shortcodes[3][$short_key] );
857 857
 			$tag  = FrmShortcodeHelper::get_shortcode_tag( $shortcodes, $short_key );
858 858
 
859 859
 			$atts['entry'] = $entry;
@@ -863,7 +863,7 @@  discard block
 block discarded – undo
863 863
 			if ( $replace_with !== null ) {
864 864
 				$replace_with = self::trigger_shortcode_atts( $replace_with, $atts );
865 865
 				self::sanitize_embedded_shortcodes( compact( 'entry' ), $replace_with );
866
-				$content = str_replace( $shortcodes[0][ $short_key ], $replace_with, $content );
866
+				$content = str_replace( $shortcodes[0][$short_key], $replace_with, $content );
867 867
 			}
868 868
 
869 869
 			unset( $atts, $replace_with );
@@ -881,7 +881,7 @@  discard block
 block discarded – undo
881 881
 		$supported_atts = array( 'remove_accents', 'sanitize', 'sanitize_url' );
882 882
 		$included_atts  = array_intersect( $supported_atts, array_keys( $atts ) );
883 883
 		foreach ( $included_atts as $included_att ) {
884
-			if ( '0' === $atts[ $included_att ] ) {
884
+			if ( '0' === $atts[$included_att] ) {
885 885
 				// Skip any option that uses 0 so sanitize_url=0 does not encode.
886 886
 				continue;
887 887
 			}
@@ -954,8 +954,8 @@  discard block
 block discarded – undo
954 954
 
955 955
 		$dynamic_default = array( 'admin_email', 'siteurl', 'frmurl', 'sitename', 'get' );
956 956
 
957
-		if ( isset( $shortcode_values[ $atts['tag'] ] ) ) {
958
-			$replace_with = $shortcode_values[ $atts['tag'] ];
957
+		if ( isset( $shortcode_values[$atts['tag']] ) ) {
958
+			$replace_with = $shortcode_values[$atts['tag']];
959 959
 		} elseif ( in_array( $atts['tag'], $dynamic_default, true ) ) {
960 960
 			$replace_with = self::dynamic_default_values( $atts['tag'], $atts );
961 961
 		} elseif ( $clean_tag === 'user_agent' ) {
@@ -1171,8 +1171,8 @@  discard block
 block discarded – undo
1171 1171
 			self::field_types_for_input( $single_input, $field_selection, $field_types );
1172 1172
 		} elseif ( in_array( $type, $multiple_input, true ) ) {
1173 1173
 			self::field_types_for_input( $multiple_input, $field_selection, $field_types );
1174
-		} elseif ( isset( $field_selection[ $type ] ) ) {
1175
-			$field_types[ $type ] = $field_selection[ $type ];
1174
+		} elseif ( isset( $field_selection[$type] ) ) {
1175
+			$field_types[$type] = $field_selection[$type];
1176 1176
 		}
1177 1177
 
1178 1178
 		$field_types = apply_filters( 'frm_switch_field_types', $field_types, compact( 'type', 'field_selection' ) );
@@ -1213,7 +1213,7 @@  discard block
 block discarded – undo
1213 1213
 		foreach ( $inputs as $input ) {
1214 1214
 			// This may not be set if a field type was removed using the frm_available_fields or frm_pro_available_fields filters.
1215 1215
 			if ( array_key_exists( $input, $fields ) ) {
1216
-				$field_types[ $input ] = $fields[ $input ];
1216
+				$field_types[$input] = $fields[$input];
1217 1217
 			}
1218 1218
 			unset( $input );
1219 1219
 		}
@@ -1246,7 +1246,7 @@  discard block
 block discarded – undo
1246 1246
 			'parent'  => false,
1247 1247
 			'pointer' => false,
1248 1248
 		);
1249
-		$args     = wp_parse_args( $args, $defaults );
1249
+		$args = wp_parse_args( $args, $defaults );
1250 1250
 
1251 1251
 		$opt_key   = $args['opt_key'];
1252 1252
 		$field     = $args['field'];
@@ -1262,25 +1262,25 @@  discard block
 block discarded – undo
1262 1262
 
1263 1263
 		// Check posted vals before checking saved values
1264 1264
 		// For fields inside repeating sections - note, don't check if $pointer is true because it will often be zero
1265
-		if ( $parent && isset( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
1265
+		if ( $parent && isset( $_POST['item_meta'][$parent][$pointer]['other'][$field['id']] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
1266 1266
 			if ( FrmField::is_field_with_multiple_values( $field ) ) {
1267 1267
 				// phpcs:ignore WordPress.Security.NonceVerification.Missing
1268
-				$other_val = isset( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ][ $opt_key ] ) ? sanitize_text_field( wp_unslash( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ][ $opt_key ] ) ) : '';
1268
+				$other_val = isset( $_POST['item_meta'][$parent][$pointer]['other'][$field['id']][$opt_key] ) ? sanitize_text_field( wp_unslash( $_POST['item_meta'][$parent][$pointer]['other'][$field['id']][$opt_key] ) ) : '';
1269 1269
 			} else {
1270
-				$other_val = sanitize_text_field( wp_unslash( $_POST['item_meta'][ $parent ][ $pointer ]['other'][ $field['id'] ] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
1270
+				$other_val = sanitize_text_field( wp_unslash( $_POST['item_meta'][$parent][$pointer]['other'][$field['id']] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
1271 1271
 			}
1272 1272
 
1273 1273
 			return $other_val;
1274 1274
 		}
1275 1275
 
1276
-		if ( isset( $field['id'] ) && isset( $_POST['item_meta']['other'][ $field['id'] ] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
1276
+		if ( isset( $field['id'] ) && isset( $_POST['item_meta']['other'][$field['id']] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
1277 1277
 			// For normal fields
1278 1278
 
1279 1279
 			if ( FrmField::is_field_with_multiple_values( $field ) ) {
1280 1280
 				// phpcs:ignore WordPress.Security.NonceVerification.Missing
1281
-				$other_val = isset( $_POST['item_meta']['other'][ $field['id'] ][ $opt_key ] ) ? sanitize_text_field( wp_unslash( $_POST['item_meta']['other'][ $field['id'] ][ $opt_key ] ) ) : '';
1281
+				$other_val = isset( $_POST['item_meta']['other'][$field['id']][$opt_key] ) ? sanitize_text_field( wp_unslash( $_POST['item_meta']['other'][$field['id']][$opt_key] ) ) : '';
1282 1282
 			} else {
1283
-				$other_val = sanitize_text_field( wp_unslash( $_POST['item_meta']['other'][ $field['id'] ] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
1283
+				$other_val = sanitize_text_field( wp_unslash( $_POST['item_meta']['other'][$field['id']] ) ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
1284 1284
 			}
1285 1285
 
1286 1286
 			return $other_val;
@@ -1290,8 +1290,8 @@  discard block
 block discarded – undo
1290 1290
 		if ( $field['type'] === 'checkbox' && is_array( $field['value'] ) ) {
1291 1291
 			// Check if there is an "other" val in saved value and make sure the
1292 1292
 			// "other" val is not equal to the Other checkbox option
1293
-			if ( isset( $field['value'][ $opt_key ] ) && $field['options'][ $opt_key ] != $field['value'][ $opt_key ] ) {
1294
-				$other_val = $field['value'][ $opt_key ];
1293
+			if ( isset( $field['value'][$opt_key] ) && $field['options'][$opt_key] != $field['value'][$opt_key] ) {
1294
+				$other_val = $field['value'][$opt_key];
1295 1295
 			}
1296 1296
 		} else {
1297 1297
 			/**
@@ -1303,8 +1303,8 @@  discard block
 block discarded – undo
1303 1303
 				// Multi-select dropdowns - key is not preserved
1304 1304
 				if ( is_array( $field['value'] ) ) {
1305 1305
 					$o_key = array_search( $temp_val, $field['value'] );
1306
-					if ( isset( $field['value'][ $o_key ] ) ) {
1307
-						unset( $field['value'][ $o_key ], $o_key );
1306
+					if ( isset( $field['value'][$o_key] ) ) {
1307
+						unset( $field['value'][$o_key], $o_key );
1308 1308
 					}
1309 1309
 				} elseif ( $temp_val == $field['value'] ) {
1310 1310
 					// For radio and regular dropdowns
@@ -1512,11 +1512,11 @@  discard block
 block discarded – undo
1512 1512
 			foreach ( $val as $k => $v ) {
1513 1513
 				if ( is_string( $v ) ) {
1514 1514
 					if ( 'custom_html' === $k ) {
1515
-						$val[ $k ] = self::switch_ids_except_strings( $replace, $replace_with, array( '[if description]', '[description]', '[/if description]' ), $v );
1515
+						$val[$k] = self::switch_ids_except_strings( $replace, $replace_with, array( '[if description]', '[description]', '[/if description]' ), $v );
1516 1516
 						unset( $k, $v );
1517 1517
 						continue;
1518 1518
 					}
1519
-					$val[ $k ] = str_replace( $replace, $replace_with, $v );
1519
+					$val[$k] = str_replace( $replace, $replace_with, $v );
1520 1520
 					unset( $k, $v );
1521 1521
 				}
1522 1522
 			}
@@ -1545,8 +1545,8 @@  discard block
 block discarded – undo
1545 1545
 			if ( false === $index ) {
1546 1546
 				continue;
1547 1547
 			}
1548
-			unset( $replace[ $index ] );
1549
-			unset( $replace_with[ $index ] );
1548
+			unset( $replace[$index] );
1549
+			unset( $replace_with[$index] );
1550 1550
 		}
1551 1551
 		$value = str_replace( $replace, $replace_with, $value );
1552 1552
 		return $value;
@@ -1894,7 +1894,7 @@  discard block
 block discarded – undo
1894 1894
 			$countries['class'] = 'frm-countries-opts';
1895 1895
 		}
1896 1896
 
1897
-		$prepop[ __( 'Countries', 'formidable' ) ] = $countries;
1897
+		$prepop[__( 'Countries', 'formidable' )] = $countries;
1898 1898
 
1899 1899
 		// State abv.
1900 1900
 		$states    = self::get_us_states();
@@ -1904,7 +1904,7 @@  discard block
 block discarded – undo
1904 1904
 			$state_abv['class'] = 'frm-state-abv-opts';
1905 1905
 		}
1906 1906
 
1907
-		$prepop[ __( 'U.S. State Abbreviations', 'formidable' ) ] = $state_abv;
1907
+		$prepop[__( 'U.S. State Abbreviations', 'formidable' )] = $state_abv;
1908 1908
 
1909 1909
 		// States.
1910 1910
 		$states = array_values( $states );
@@ -1913,7 +1913,7 @@  discard block
 block discarded – undo
1913 1913
 			$states['class'] = 'frm-states-opts';
1914 1914
 		}
1915 1915
 
1916
-		$prepop[ __( 'U.S. States', 'formidable' ) ] = $states;
1916
+		$prepop[__( 'U.S. States', 'formidable' )] = $states;
1917 1917
 		unset( $state_abv, $states );
1918 1918
 
1919 1919
 		// Age.
@@ -1931,7 +1931,7 @@  discard block
 block discarded – undo
1931 1931
 			$ages['class'] = 'frm-age-opts';
1932 1932
 		}
1933 1933
 
1934
-		$prepop[ __( 'Age', 'formidable' ) ] = $ages;
1934
+		$prepop[__( 'Age', 'formidable' )] = $ages;
1935 1935
 
1936 1936
 		// Satisfaction.
1937 1937
 		$satisfaction = array(
@@ -1946,7 +1946,7 @@  discard block
 block discarded – undo
1946 1946
 			$satisfaction['class'] = 'frm-satisfaction-opts';
1947 1947
 		}
1948 1948
 
1949
-		$prepop[ __( 'Satisfaction', 'formidable' ) ] = $satisfaction;
1949
+		$prepop[__( 'Satisfaction', 'formidable' )] = $satisfaction;
1950 1950
 
1951 1951
 		// Importance.
1952 1952
 		$importance = array(
@@ -1961,7 +1961,7 @@  discard block
 block discarded – undo
1961 1961
 			$importance['class'] = 'frm-importance-opts';
1962 1962
 		}
1963 1963
 
1964
-		$prepop[ __( 'Importance', 'formidable' ) ] = $importance;
1964
+		$prepop[__( 'Importance', 'formidable' )] = $importance;
1965 1965
 
1966 1966
 		// Agreement.
1967 1967
 		$agreement = array(
@@ -1976,7 +1976,7 @@  discard block
 block discarded – undo
1976 1976
 			$agreement['class'] = 'frm-agreement-opts';
1977 1977
 		}
1978 1978
 
1979
-		$prepop[ __( 'Agreement', 'formidable' ) ] = $agreement;
1979
+		$prepop[__( 'Agreement', 'formidable' )] = $agreement;
1980 1980
 
1981 1981
 		// Likely.
1982 1982
 		$likely = array(
@@ -1991,7 +1991,7 @@  discard block
 block discarded – undo
1991 1991
 			$likely['class'] = 'frm-likely-opts';
1992 1992
 		}
1993 1993
 
1994
-		$prepop[ __( 'Likely', 'formidable' ) ] = $likely;
1994
+		$prepop[__( 'Likely', 'formidable' )] = $likely;
1995 1995
 
1996 1996
 		$prepop = apply_filters( 'frm_bulk_field_choices', $prepop );
1997 1997
 	}
@@ -2212,16 +2212,16 @@  discard block
 block discarded – undo
2212 2212
 	 */
2213 2213
 	private static function fill_image_setting_options( $options, &$args ) {
2214 2214
 		foreach ( $options as $key => $option ) {
2215
-			$args['options'][ $key ] = $option;
2215
+			$args['options'][$key] = $option;
2216 2216
 
2217 2217
 			if ( ! empty( $option['addon'] ) ) {
2218
-				$args['options'][ $key ]['custom_attrs'] = self::fill_image_setting_addon_link( $option );
2218
+				$args['options'][$key]['custom_attrs'] = self::fill_image_setting_addon_link( $option );
2219 2219
 			}
2220 2220
 
2221
-			unset( $args['options'][ $key ]['addon'] );
2221
+			unset( $args['options'][$key]['addon'] );
2222 2222
 			$fill = array( 'upgrade', 'message', 'content' );
2223 2223
 			foreach ( $fill as $f ) {
2224
-				unset( $args['options'][ $key ][ $f ], $f );
2224
+				unset( $args['options'][$key][$f], $f );
2225 2225
 			}
2226 2226
 		}
2227 2227
 	}
@@ -2242,8 +2242,8 @@  discard block
 block discarded – undo
2242 2242
 
2243 2243
 		$fill = array( 'upgrade', 'message', 'content' );
2244 2244
 		foreach ( $fill as $f ) {
2245
-			if ( isset( $option[ $f ] ) ) {
2246
-				$custom_attrs[ 'data-' . $f ] = $option[ $f ];
2245
+			if ( isset( $option[$f] ) ) {
2246
+				$custom_attrs['data-' . $f] = $option[$f];
2247 2247
 			}
2248 2248
 		}
2249 2249
 
@@ -2308,7 +2308,7 @@  discard block
 block discarded – undo
2308 2308
 
2309 2309
 		return array_filter(
2310 2310
 			$rows,
2311
-			function ( $row ) {
2311
+			function( $row ) {
2312 2312
 				FrmAppHelper::unserialize_or_decode( $row->field_options );
2313 2313
 				return is_array( $row->field_options ) && ! empty( $row->field_options['draft'] );
2314 2314
 			}
Please login to merge, or discard this patch.
classes/views/xml/posts_xml.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -113,8 +113,8 @@
 block discarded – undo
113 113
 	$label         = 'category' === $term->taxonomy || 'tag' === $term->taxonomy ? $term->taxonomy : 'term';
114 114
 	?>
115 115
 	<term><term_id><?php echo esc_html( $term->term_id ); ?></term_id><term_taxonomy><?php echo esc_html( $term->taxonomy ); ?></term_taxonomy><?php
116
-	if ( ! empty( $parent_slugs[ $term->parent ] ) ) {
117
-		echo '<term_parent>' . esc_html( $parent_slugs[ $term->parent ] ) . '</term_parent>';
116
+	if ( ! empty( $parent_slugs[$term->parent] ) ) {
117
+		echo '<term_parent>' . esc_html( $parent_slugs[$term->parent] ) . '</term_parent>';
118 118
 	}
119 119
 	if ( ! empty( $term->name ) ) {
120 120
 		echo '<term_name>' . FrmXMLHelper::cdata( $term->name ) . '</term_name>'; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
Please login to merge, or discard this patch.
classes/helpers/FrmCSVExportHelper.php 1 patch
Spacing   +54 added lines, -54 removed lines patch added patch discarded remove patch
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
 		unset( $filename );
130 130
 
131
-		$comment_count       = FrmDb::get_count(
131
+		$comment_count = FrmDb::get_count(
132 132
 			'frm_item_metas',
133 133
 			array(
134 134
 				'item_id'         => $atts['entry_ids'],
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
 	private static function prepare_csv_headings() {
247 247
 		$headings = array();
248 248
 		self::csv_headings( $headings );
249
-		$headings       = apply_filters(
249
+		$headings = apply_filters(
250 250
 			'frm_csv_columns',
251 251
 			$headings,
252 252
 			self::$form_id,
@@ -270,10 +270,10 @@  discard block
 block discarded – undo
270 270
 		$field_headings  = array();
271 271
 		$separate_values = array( 'user_id', 'file', 'data', 'date' );
272 272
 		if ( ! empty( $col->field_options['separate_value'] ) && ! in_array( $col->type, $separate_values, true ) ) {
273
-			$field_headings[ $col->id . '_label' ] = strip_tags( $col->name . ' ' . __( '(label)', 'formidable' ) );
273
+			$field_headings[$col->id . '_label'] = strip_tags( $col->name . ' ' . __( '(label)', 'formidable' ) );
274 274
 		}
275 275
 
276
-		$field_headings[ $col->id ] = strip_tags( $col->name );
276
+		$field_headings[$col->id] = strip_tags( $col->name );
277 277
 		$field_headings             = apply_filters(
278 278
 			'frm_csv_field_columns',
279 279
 			$field_headings,
@@ -294,14 +294,14 @@  discard block
 block discarded – undo
294 294
 			if ( self::is_the_child_of_a_repeater( $col ) ) {
295 295
 				$repeater_id = $col->field_options['in_section'];
296 296
 				// Set a placeholder to maintain order for repeater fields.
297
-				$headings[ 'repeater' . $repeater_id ] = array();
297
+				$headings['repeater' . $repeater_id] = array();
298 298
 
299
-				if ( ! isset( $fields_by_repeater_id[ $repeater_id ] ) ) {
300
-					$fields_by_repeater_id[ $repeater_id ] = array();
299
+				if ( ! isset( $fields_by_repeater_id[$repeater_id] ) ) {
300
+					$fields_by_repeater_id[$repeater_id] = array();
301 301
 					$repeater_ids[]                        = $repeater_id;
302 302
 				}
303 303
 
304
-				$fields_by_repeater_id[ $repeater_id ][] = $col;
304
+				$fields_by_repeater_id[$repeater_id][] = $col;
305 305
 
306 306
 				continue;
307 307
 			}
@@ -320,8 +320,8 @@  discard block
 block discarded – undo
320 320
 				$end    = strpos( $row->meta_value, ':{' );
321 321
 				$length = substr( $row->meta_value, $start, $end - $start );
322 322
 
323
-				if ( $length > $max[ $row->field_id ] ) {
324
-					$max[ $row->field_id ] = $length;
323
+				if ( $length > $max[$row->field_id] ) {
324
+					$max[$row->field_id] = $length;
325 325
 				}
326 326
 			}
327 327
 			unset( $start, $end, $length, $row, $repeater_meta, $where );
@@ -332,17 +332,17 @@  discard block
 block discarded – undo
332 332
 					$repeater_id = str_replace( 'repeater', '', $key );
333 333
 
334 334
 					$repeater_headings = array();
335
-					foreach ( $fields_by_repeater_id[ $repeater_id ] as $col ) {
335
+					foreach ( $fields_by_repeater_id[$repeater_id] as $col ) {
336 336
 						$repeater_headings += self::field_headings( $col );
337 337
 					}
338 338
 
339
-					for ( $i = 0; $i < $max[ $repeater_id ]; $i++ ) {
339
+					for ( $i = 0; $i < $max[$repeater_id]; $i ++ ) {
340 340
 						foreach ( $repeater_headings as $repeater_key => $repeater_name ) {
341
-							$flat[ $repeater_key . '[' . $i . ']' ] = $repeater_name;
341
+							$flat[$repeater_key . '[' . $i . ']'] = $repeater_name;
342 342
 						}
343 343
 					}
344 344
 				} else {
345
-					$flat[ $key ] = $heading;
345
+					$flat[$key] = $heading;
346 346
 				}
347 347
 			}
348 348
 
@@ -355,10 +355,10 @@  discard block
 block discarded – undo
355 355
 		}//end if
356 356
 
357 357
 		if ( self::$comment_count ) {
358
-			for ( $i = 0; $i < self::$comment_count; $i++ ) {
359
-				$headings[ 'comment' . $i ]            = __( 'Comment', 'formidable' );
360
-				$headings[ 'comment_user_id' . $i ]    = __( 'Comment User', 'formidable' );
361
-				$headings[ 'comment_created_at' . $i ] = __( 'Comment Date', 'formidable' );
358
+			for ( $i = 0; $i < self::$comment_count; $i ++ ) {
359
+				$headings['comment' . $i]            = __( 'Comment', 'formidable' );
360
+				$headings['comment_user_id' . $i]    = __( 'Comment User', 'formidable' );
361
+				$headings['comment_created_at' . $i] = __( 'Comment Date', 'formidable' );
362 362
 			}
363 363
 			unset( $i );
364 364
 		}
@@ -403,7 +403,7 @@  discard block
 block discarded – undo
403 403
 
404 404
 	private static function prepare_next_csv_rows( $next_set ) {
405 405
 		// order by parent_item_id so children will be first
406
-		$where   = array(
406
+		$where = array(
407 407
 			'or'             => 1,
408 408
 			'id'             => $next_set,
409 409
 			'parent_item_id' => $next_set,
@@ -449,32 +449,32 @@  discard block
 block discarded – undo
449 449
 					continue;
450 450
 				}
451 451
 
452
-				if ( ! isset( $entries[ self::$entry->parent_item_id ] ) ) {
453
-					$entries[ self::$entry->parent_item_id ]        = new stdClass();
454
-					$entries[ self::$entry->parent_item_id ]->metas = array();
452
+				if ( ! isset( $entries[self::$entry->parent_item_id] ) ) {
453
+					$entries[self::$entry->parent_item_id]        = new stdClass();
454
+					$entries[self::$entry->parent_item_id]->metas = array();
455 455
 				}
456 456
 
457
-				if ( ! isset( $entries[ self::$entry->parent_item_id ]->metas[ $meta_id ] ) ) {
458
-					$entries[ self::$entry->parent_item_id ]->metas[ $meta_id ] = array();
459
-				} elseif ( ! is_array( $entries[ self::$entry->parent_item_id ]->metas[ $meta_id ] ) ) {
457
+				if ( ! isset( $entries[self::$entry->parent_item_id]->metas[$meta_id] ) ) {
458
+					$entries[self::$entry->parent_item_id]->metas[$meta_id] = array();
459
+				} elseif ( ! is_array( $entries[self::$entry->parent_item_id]->metas[$meta_id] ) ) {
460 460
 					// if the data is here, it should be an array but if this field has collected data
461 461
 					// both while inside and outside of the repeating section, it's possible this is a string.
462
-					$entries[ self::$entry->parent_item_id ]->metas[ $meta_id ] = (array) $entries[ self::$entry->parent_item_id ]->metas[ $meta_id ];
462
+					$entries[self::$entry->parent_item_id]->metas[$meta_id] = (array) $entries[self::$entry->parent_item_id]->metas[$meta_id];
463 463
 				}
464 464
 
465 465
 				// Add the repeated values.
466
-				$entries[ self::$entry->parent_item_id ]->metas[ $meta_id ][] = $meta_value;
466
+				$entries[self::$entry->parent_item_id]->metas[$meta_id][] = $meta_value;
467 467
 			}//end foreach
468 468
 
469 469
 			self::$entry->metas                              = self::fill_missing_repeater_metas( self::$entry->metas, $entries );
470
-			$entries[ self::$entry->parent_item_id ]->metas += self::$entry->metas;
470
+			$entries[self::$entry->parent_item_id]->metas += self::$entry->metas;
471 471
 		}//end if
472 472
 
473 473
 		// add the embedded form id
474
-		if ( ! isset( $entries[ self::$entry->parent_item_id ]->embedded_fields ) ) {
475
-			$entries[ self::$entry->parent_item_id ]->embedded_fields = array();
474
+		if ( ! isset( $entries[self::$entry->parent_item_id]->embedded_fields ) ) {
475
+			$entries[self::$entry->parent_item_id]->embedded_fields = array();
476 476
 		}
477
-		$entries[ self::$entry->parent_item_id ]->embedded_fields[ self::$entry->id ] = self::$entry->form_id;
477
+		$entries[self::$entry->parent_item_id]->embedded_fields[self::$entry->id] = self::$entry->form_id;
478 478
 	}
479 479
 
480 480
 	/**
@@ -495,19 +495,19 @@  discard block
 block discarded – undo
495 495
 		}
496 496
 
497 497
 		$repeater_id = $field->field_options['in_section'];
498
-		if ( ! isset( self::$fields_by_repeater_id[ $repeater_id ] ) ) {
498
+		if ( ! isset( self::$fields_by_repeater_id[$repeater_id] ) ) {
499 499
 			return $metas;
500 500
 		}
501 501
 
502
-		foreach ( self::$fields_by_repeater_id[ $repeater_id ] as $repeater_child ) {
503
-			if ( ! isset( $metas[ $repeater_child->id ] ) ) {
504
-				$metas[ $repeater_child->id ] = '';
502
+		foreach ( self::$fields_by_repeater_id[$repeater_id] as $repeater_child ) {
503
+			if ( ! isset( $metas[$repeater_child->id] ) ) {
504
+				$metas[$repeater_child->id] = '';
505 505
 
506
-				if ( ! isset( $entries[ self::$entry->parent_item_id ]->metas[ $repeater_child->id ] ) || ! is_array( $entries[ self::$entry->parent_item_id ]->metas[ $repeater_child->id ] ) ) {
507
-					$entries[ self::$entry->parent_item_id ]->metas[ $repeater_child->id ] = array();
506
+				if ( ! isset( $entries[self::$entry->parent_item_id]->metas[$repeater_child->id] ) || ! is_array( $entries[self::$entry->parent_item_id]->metas[$repeater_child->id] ) ) {
507
+					$entries[self::$entry->parent_item_id]->metas[$repeater_child->id] = array();
508 508
 				}
509 509
 
510
-				$entries[ self::$entry->parent_item_id ]->metas[ $repeater_child->id ][] = '';
510
+				$entries[self::$entry->parent_item_id]->metas[$repeater_child->id][] = '';
511 511
 			}
512 512
 		}
513 513
 
@@ -526,7 +526,7 @@  discard block
 block discarded – undo
526 526
 
527 527
 	private static function add_field_values_to_csv( &$row ) {
528 528
 		foreach ( self::$fields as $col ) {
529
-			$field_value = isset( self::$entry->metas[ $col->id ] ) ? self::$entry->metas[ $col->id ] : false;
529
+			$field_value = isset( self::$entry->metas[$col->id] ) ? self::$entry->metas[$col->id] : false;
530 530
 
531 531
 			FrmFieldsHelper::prepare_field_value( $field_value, $col->type );
532 532
 			self::add_array_values_to_columns( $row, compact( 'col', 'field_value' ) );
@@ -545,20 +545,20 @@  discard block
 block discarded – undo
545 545
 			if ( ! empty( $col->field_options['separate_value'] ) ) {
546 546
 				$label_key = $col->id . '_label';
547 547
 				if ( self::is_the_child_of_a_repeater( $col ) ) {
548
-					$row[ $label_key ] = array();
548
+					$row[$label_key] = array();
549 549
 
550 550
 					if ( is_array( $field_value ) ) {
551 551
 						foreach ( $field_value as $value ) {
552
-							$row[ $label_key ][] = self::get_separate_value_label( $value, $col );
552
+							$row[$label_key][] = self::get_separate_value_label( $value, $col );
553 553
 						}
554 554
 					}
555 555
 				} else {
556
-					$row[ $label_key ] = self::get_separate_value_label( $field_value, $col );
556
+					$row[$label_key] = self::get_separate_value_label( $field_value, $col );
557 557
 				}
558 558
 				unset( $label_key );
559 559
 			}
560 560
 
561
-			$row[ $col->id ] = $field_value;
561
+			$row[$col->id] = $field_value;
562 562
 
563 563
 			unset( $col, $field_value );
564 564
 		}//end foreach
@@ -581,7 +581,7 @@  discard block
 block discarded – undo
581 581
 				'show_icon'         => false,
582 582
 				'entry_id'          => self::$entry->id,
583 583
 				'sep'               => self::$separator,
584
-				'embedded_field_id' => isset( self::$entry->embedded_fields ) && isset( self::$entry->embedded_fields[ self::$entry->id ] ) ? 'form' . self::$entry->embedded_fields[ self::$entry->id ] : 0,
584
+				'embedded_field_id' => isset( self::$entry->embedded_fields ) && isset( self::$entry->embedded_fields[self::$entry->id] ) ? 'form' . self::$entry->embedded_fields[self::$entry->id] : 0,
585 585
 			)
586 586
 		);
587 587
 	}
@@ -596,8 +596,8 @@  discard block
 block discarded – undo
596 596
 					// This is combo field inside repeater. The heading key has this format: [86_first[0]].
597 597
 					foreach ( $sub_value as $sub_key => $sub_sub_value ) {
598 598
 						$column_key = $atts['col']->id . '_' . $sub_key . '[' . $key . ']';
599
-						if ( ! is_numeric( $sub_key ) && isset( self::$headings[ $column_key ] ) ) {
600
-							$row[ $column_key ] = $sub_sub_value;
599
+						if ( ! is_numeric( $sub_key ) && isset( self::$headings[$column_key] ) ) {
600
+							$row[$column_key] = $sub_sub_value;
601 601
 						}
602 602
 					}
603 603
 
@@ -605,8 +605,8 @@  discard block
 block discarded – undo
605 605
 				}
606 606
 
607 607
 				$column_key = $atts['col']->id . '_' . $key;
608
-				if ( ! is_numeric( $key ) && isset( self::$headings[ $column_key ] ) ) {
609
-					$row[ $column_key ] = $sub_value;
608
+				if ( ! is_numeric( $key ) && isset( self::$headings[$column_key] ) ) {
609
+					$row[$column_key] = $sub_value;
610 610
 				}
611 611
 			}
612 612
 		}
@@ -631,18 +631,18 @@  discard block
 block discarded – undo
631 631
 		$echo = 'echo' === self::$mode;
632 632
 
633 633
 		foreach ( self::$headings as $k => $heading ) {
634
-			if ( isset( $rows[ $k ] ) ) {
635
-				$row = $rows[ $k ];
634
+			if ( isset( $rows[$k] ) ) {
635
+				$row = $rows[$k];
636 636
 			} else {
637 637
 				$row = '';
638 638
 				// array indexed data is not at $rows[ $k ]
639
-				if ( $k[ strlen( $k ) - 1 ] === ']' ) {
639
+				if ( $k[strlen( $k ) - 1] === ']' ) {
640 640
 					$start = strrpos( $k, '[' );
641
-					$key   = substr( $k, 0, $start++ );
641
+					$key   = substr( $k, 0, $start ++ );
642 642
 					$index = substr( $k, $start, strlen( $k ) - 1 - $start );
643 643
 
644
-					if ( isset( $rows[ $key ] ) && isset( $rows[ $key ][ $index ] ) ) {
645
-						$row = $rows[ $key ][ $index ];
644
+					if ( isset( $rows[$key] ) && isset( $rows[$key][$index] ) ) {
645
+						$row = $rows[$key][$index];
646 646
 					}
647 647
 
648 648
 					unset( $start, $key, $index );
Please login to merge, or discard this patch.
classes/helpers/FrmStylesCardHelper.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -138,11 +138,11 @@  discard block
 block discarded – undo
138 138
 	private static function has_dark_background( $style ) {
139 139
 		$key = 'fieldset_bg_color';
140 140
 
141
-		if ( empty( $style->post_content[ $key ] ) ) {
141
+		if ( empty( $style->post_content[$key] ) ) {
142 142
 			return false;
143 143
 		}
144 144
 
145
-		$color = $style->post_content[ $key ];
145
+		$color = $style->post_content[$key];
146 146
 
147 147
 		if ( 0 === strpos( $color, 'rgba' ) ) {
148 148
 			preg_match_all( '/([\\d.]+)/', $color, $matches );
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 			 * @param stdClass $style_object
202 202
 			 * @param array    $style
203 203
 			 */
204
-			$param_filter = function ( $params, $args ) use ( $style_object, $style ) {
204
+			$param_filter = function( $params, $args ) use ( $style_object, $style ) {
205 205
 				if ( $args['style'] !== $style_object ) {
206 206
 					return $params;
207 207
 				}
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 					),
215 215
 					'/style-templates/' . $style['slug']
216 216
 				);
217
-				$params['data-requires']    = FrmFormsHelper::get_plan_required( $style );
217
+				$params['data-requires'] = FrmFormsHelper::get_plan_required( $style );
218 218
 				return $params;
219 219
 			};
220 220
 		} else {
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 			 * @param array $style
226 226
 			 * @return array
227 227
 			 */
228
-			$param_filter = function ( $params ) use ( $style ) {
228
+			$param_filter = function( $params ) use ( $style ) {
229 229
 				$params['data-template-key'] = $style['slug'];
230 230
 				return $params;
231 231
 			};
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
 				continue;
278 278
 			}
279 279
 
280
-			$value = $style->post_content[ $key ];
280
+			$value = $style->post_content[$key];
281 281
 
282 282
 			$is_hex = in_array( $key, $color_settings, true ) && $value && '#' !== $value[0] && false === strpos( $value, 'rgb' ) && $value !== 'transparent';
283 283
 			if ( $is_hex ) {
@@ -396,7 +396,7 @@  discard block
 block discarded – undo
396 396
 			 * @param int          $count Used for pagination.
397 397
 			 * @return void
398 398
 			 */
399
-			function ( $style, $key ) {
399
+			function( $style, $key ) {
400 400
 				if ( ! is_numeric( $key ) ) {
401 401
 					// Skip active_sub/expires keys.
402 402
 					return;
@@ -423,10 +423,10 @@  discard block
 block discarded – undo
423 423
 			 * @param int     $count Used for pagination.
424 424
 			 * @return void
425 425
 			 */
426
-			function ( $style ) use ( &$count ) {
426
+			function( $style ) use ( &$count ) {
427 427
 				$hidden = $count > self::PAGE_SIZE - 1;
428 428
 				$this->echo_style_card( $style, $hidden );
429
-				++$count;
429
+				++ $count;
430 430
 			}
431 431
 		);
432 432
 
@@ -484,7 +484,7 @@  discard block
 block discarded – undo
484 484
 			 * @param WP_Post $style
485 485
 			 * @return bool
486 486
 			 */
487
-			function ( $style ) {
487
+			function( $style ) {
488 488
 				return $this->default_style->ID !== $style->ID;
489 489
 			}
490 490
 		);
Please login to merge, or discard this patch.
classes/helpers/FrmStylesPreviewHelper.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 			 * @param array|object $field
75 75
 			 * @return string
76 76
 			 */
77
-			function ( $position, $field ) {
77
+			function( $position, $field ) {
78 78
 				if ( is_array( $field ) ) {
79 79
 					$this->default_label_position_field_ids[] = (int) $field['id'];
80 80
 				}
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 			 * @param array|object $field
92 92
 			 * @return string
93 93
 			 */
94
-			function ( $classes, $field ) {
94
+			function( $classes, $field ) {
95 95
 				if ( is_array( $field ) && in_array( (int) $field['id'], $this->default_label_position_field_ids, true ) ) {
96 96
 					$classes .= ' frm-default-label-position';
97 97
 				}
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
 			 * @param string $target_field_type
119 119
 			 * @return bool
120 120
 			 */
121
-			function ( $show, $field_type ) {
121
+			function( $show, $field_type ) {
122 122
 				if ( 'captcha' === $field_type ) {
123 123
 					$show = false;
124 124
 				}
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 			 * @param int           $form_id
147 147
 			 * @return stdClass|null
148 148
 			 */
149
-			function ( $form ) {
149
+			function( $form ) {
150 150
 				if ( is_object( $form ) && is_array( $form->options ) ) {
151 151
 					$form->options['js_validate'] = false;
152 152
 				}
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
 
197 197
 		$frm_settings = FrmAppHelper::get_settings();
198 198
 		if ( 'none' === $frm_settings->load_style ) {
199
-			$notes[] = function () {
199
+			$notes[] = function() {
200 200
 				printf(
201 201
 					// translators: %1$s: Anchor tag open, %2$s: Anchor tag close.
202 202
 					esc_html__( 'Formidable styles are disabled. This needs to be enabled in %1$sGlobal Settings%2$s.', 'formidable' ),
@@ -354,7 +354,7 @@  discard block
 block discarded – undo
354 354
 			return;
355 355
 		}
356 356
 
357
-		unset( $dependencies[ $index ] );
357
+		unset( $dependencies[$index] );
358 358
 		$dependencies = array_values( $dependencies );
359 359
 
360 360
 		$styles->registered['wp-admin']->deps = $dependencies;
Please login to merge, or discard this patch.
classes/helpers/FrmListHelper.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -223,8 +223,8 @@  discard block
 block discarded – undo
223 223
 			return $this->get_pagenum();
224 224
 		}
225 225
 
226
-		if ( isset( $this->_pagination_args[ $key ] ) ) {
227
-			return $this->_pagination_args[ $key ];
226
+		if ( isset( $this->_pagination_args[$key] ) ) {
227
+			return $this->_pagination_args[$key];
228 228
 		}
229 229
 	}
230 230
 
@@ -269,8 +269,8 @@  discard block
 block discarded – undo
269 269
 	}
270 270
 
271 271
 	private function hidden_search_inputs( $param_name ) {
272
-		if ( ! empty( $_REQUEST[ $param_name ] ) ) {
273
-			$value = sanitize_text_field( wp_unslash( $_REQUEST[ $param_name ] ) );
272
+		if ( ! empty( $_REQUEST[$param_name] ) ) {
273
+			$value = sanitize_text_field( wp_unslash( $_REQUEST[$param_name] ) );
274 274
 			echo '<input type="hidden" name="' . esc_attr( $param_name ) . '" value="' . esc_attr( $value ) . '" />';
275 275
 		}
276 276
 	}
@@ -312,7 +312,7 @@  discard block
 block discarded – undo
312 312
 
313 313
 		echo "<ul class='subsubsub'>\n";
314 314
 		foreach ( $views as $class => $view ) {
315
-			$views[ $class ] = "\t" . '<li class="' . esc_attr( $class ) . '">' . $view;
315
+			$views[$class] = "\t" . '<li class="' . esc_attr( $class ) . '">' . $view;
316 316
 		}
317 317
 		echo implode( " |</li>\n", $views ) . "</li>\n"; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
318 318
 		echo '</ul>';
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
 
462 462
 		$out = '<div class="' . ( $always_visible ? 'row-actions visible' : 'row-actions' ) . '">';
463 463
 		foreach ( $actions as $action => $link ) {
464
-			++$i;
464
+			++ $i;
465 465
 			$sep  = $i == $action_count ? '' : ' | ';
466 466
 			$out .= "<span class='$action'>$link$sep</span>";
467 467
 		}
@@ -684,7 +684,7 @@  discard block
 block discarded – undo
684 684
 			'next'  => __( 'Next page', 'formidable' ),
685 685
 		);
686 686
 
687
-		return $labels[ $link ];
687
+		return $labels[$link];
688 688
 	}
689 689
 
690 690
 	private function current_url() {
@@ -774,7 +774,7 @@  discard block
 block discarded – undo
774 774
 
775 775
 		// If the primary column doesn't exist fall back to the
776 776
 		// first non-checkbox column.
777
-		if ( ! isset( $columns[ $default ] ) ) {
777
+		if ( ! isset( $columns[$default] ) ) {
778 778
 			$default = self::get_default_primary_column_name();
779 779
 		}
780 780
 
@@ -788,7 +788,7 @@  discard block
 block discarded – undo
788 788
 		 */
789 789
 		$column = apply_filters( 'list_table_primary_column', $default, $this->screen->id );
790 790
 
791
-		if ( empty( $column ) || ! isset( $columns[ $column ] ) ) {
791
+		if ( empty( $column ) || ! isset( $columns[$column] ) ) {
792 792
 			$column = $default;
793 793
 		}
794 794
 
@@ -809,7 +809,7 @@  discard block
 block discarded – undo
809 809
 			// In 4.3, we added a fourth argument for primary column.
810 810
 			$column_headers = array( array(), array(), array(), $this->get_primary_column_name() );
811 811
 			foreach ( $this->_column_headers as $key => $value ) {
812
-				$column_headers[ $key ] = $value;
812
+				$column_headers[$key] = $value;
813 813
 			}
814 814
 
815 815
 			return $column_headers;
@@ -842,7 +842,7 @@  discard block
 block discarded – undo
842 842
 				$data[1] = false;
843 843
 			}
844 844
 
845
-			$sortable[ $id ] = $data;
845
+			$sortable[$id] = $data;
846 846
 		}
847 847
 
848 848
 		$primary = $this->get_primary_column_name();
@@ -898,7 +898,7 @@  discard block
 block discarded – undo
898 898
 			static $cb_counter = 1;
899 899
 			$columns['cb']     = '<label class="screen-reader-text" for="cb-select-all-' . $cb_counter . '">' . __( 'Select All', 'formidable' ) . '</label>';
900 900
 			$columns['cb']    .= '<input id="cb-select-all-' . esc_attr( $cb_counter ) . '" type="checkbox" />';
901
-			++$cb_counter;
901
+			++ $cb_counter;
902 902
 		}
903 903
 
904 904
 		foreach ( $columns as $column_key => $column_display_name ) {
@@ -920,8 +920,8 @@  discard block
 block discarded – undo
920 920
 				$class[] = 'column-primary';
921 921
 			}
922 922
 
923
-			if ( isset( $sortable[ $column_key ] ) ) {
924
-				list( $orderby, $desc_first ) = $sortable[ $column_key ];
923
+			if ( isset( $sortable[$column_key] ) ) {
924
+				list( $orderby, $desc_first ) = $sortable[$column_key];
925 925
 
926 926
 				if ( $current_orderby == $orderby ) {
927 927
 					// The sorted column. The `aria-sort` attribute must be set only on the sorted column.
@@ -1034,7 +1034,7 @@  discard block
 block discarded – undo
1034 1034
 	 * @return bool Returns true if the setting is not set or if it is not false; otherwise, returns false.
1035 1035
 	 */
1036 1036
 	protected function should_display( $args, $settings ) {
1037
-		return ! isset( $args[ $settings ] ) || false !== $args[ $settings ];
1037
+		return ! isset( $args[$settings] ) || false !== $args[$settings];
1038 1038
 	}
1039 1039
 
1040 1040
 	/**
Please login to merge, or discard this patch.
classes/views/dashboard/templates/counters.php 1 patch
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,20 +14,26 @@
 block discarded – undo
14 14
 				<h4><?php echo esc_html( $counter['heading'] ); ?></h4>
15 15
 				<?php if ( isset( $counter['cta'] ) && isset( $counter['cta']['display'] ) && true === $counter['cta']['display'] ) : ?>
16 16
 					<a href="<?php echo esc_url( $counter['cta']['link'] ); ?>"><?php echo esc_html( $counter['cta']['title'] ); ?></a>
17
-				<?php else : ?>
17
+				<?php else {
18
+	: ?>
18 19
 					<?php if ( 'currency' === $counter['type'] ) : ?>
19 20
 						<div class="frm-flex-box frm-gap-md">
20 21
 							<?php foreach ( $counter['items'] as $item ) : ?>
21 22
 								<b>
22
-									<?php echo esc_attr( $item['counter_label']['symbol_left'] ); ?>
23
+									<?php echo esc_attr( $item['counter_label']['symbol_left'] );
24
+}
25
+?>
23 26
 									<span class="frm-counter" data-type="<?php echo esc_attr( $counter['type'] ); ?>" data-locale="<?php echo esc_attr( get_locale() ); ?>" data-counter="<?php echo (int) $item['counter']; ?>"><?php echo (int) $item['counter']; ?></span>
24 27
 									<?php echo esc_attr( $item['counter_label']['symbol_right'] ); ?>
25 28
 								</b>
26 29
 							<?php endforeach; ?>
27 30
 						</div>
28
-					<?php else : ?>
31
+					<?php else {
32
+	: ?>
29 33
 						<b>
30
-							<span class="frm-counter" data-type="<?php echo esc_attr( $counter['type'] ); ?>" data-locale="<?php echo esc_attr( get_locale() ); ?>" data-counter="<?php echo (int) $counter['counter']; ?>"><?php echo (int) $counter['counter']; ?></span>
34
+							<span class="frm-counter" data-type="<?php echo esc_attr( $counter['type'] );
35
+}
36
+?>" data-locale="<?php echo esc_attr( get_locale() ); ?>" data-counter="<?php echo (int) $counter['counter']; ?>"><?php echo (int) $counter['counter']; ?></span>
31 37
 						</b>
32 38
 					<?php endif; ?>
33 39
 				<?php endif; ?>
Please login to merge, or discard this patch.
classes/controllers/FrmXMLController.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 			return;
164 164
 		}
165 165
 
166
-		$selected_xml = isset( $form['xml'] ) && isset( $form['xml'][ $selected_form ] ) ? $form['xml'][ $selected_form ] : '';
166
+		$selected_xml = isset( $form['xml'] ) && isset( $form['xml'][$selected_form] ) ? $form['xml'][$selected_form] : '';
167 167
 		if ( empty( $selected_xml ) || strpos( $selected_xml, 'http' ) !== 0 ) {
168 168
 			return;
169 169
 		}
@@ -178,8 +178,8 @@  discard block
 block discarded – undo
178 178
 	 * @param string $value
179 179
 	 */
180 180
 	private static function get_selected_in_form( $form, $value = 'form' ) {
181
-		if ( ! empty( $form ) && ! empty( $form[ $value ] ) ) {
182
-			return $form[ $value ];
181
+		if ( ! empty( $form ) && ! empty( $form[$value] ) ) {
182
+			return $form[$value];
183 183
 		}
184 184
 
185 185
 		return '';
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
 			}
209 209
 
210 210
 			if ( $for === 'view' ) {
211
-				$item_key  = is_array( $view_keys ) ? $view_keys[ $form_key ] : $view_keys;
211
+				$item_key  = is_array( $view_keys ) ? $view_keys[$form_key] : $view_keys;
212 212
 				$shortcode = '[display-frm-data id=%1$s filter=limited]';
213 213
 			} elseif ( $for === 'form' ) {
214 214
 				$item_key  = $form_key;
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
 				continue;
224 224
 			}
225 225
 
226
-			$page_ids[ $for ] = wp_insert_post(
226
+			$page_ids[$for] = wp_insert_post(
227 227
 				array(
228 228
 					'post_title'   => $name,
229 229
 					'post_type'    => 'page',
@@ -376,7 +376,7 @@  discard block
 block discarded – undo
376 376
 		// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.NonceVerification.Missing
377 377
 		$file_type = sanitize_option( 'upload_path', $_FILES['frm_import_file']['name'] );
378 378
 		$file_type = strtolower( pathinfo( $file_type, PATHINFO_EXTENSION ) );
379
-		if ( 'xml' !== $file_type && isset( $export_format[ $file_type ] ) ) {
379
+		if ( 'xml' !== $file_type && isset( $export_format[$file_type] ) ) {
380 380
 			// allow other file types to be imported
381 381
 			do_action( 'frm_before_import_' . $file_type );
382 382
 
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
 		foreach ( $type as $tb_type ) {
471 471
 			$where = array();
472 472
 			$join  = '';
473
-			$table = $tables[ $tb_type ];
473
+			$table = $tables[$tb_type];
474 474
 
475 475
 			$select     = $table . '.id';
476 476
 			$query_vars = array();
@@ -485,7 +485,7 @@  discard block
 block discarded – undo
485 485
 							$table . '.parent_form_id' => $args['ids'],
486 486
 						);
487 487
 					} else {
488
-						$where[ $table . '.status !' ] = 'draft';
488
+						$where[$table . '.status !'] = 'draft';
489 489
 					}
490 490
 					break;
491 491
 				case 'actions':
@@ -498,7 +498,7 @@  discard block
 block discarded – undo
498 498
 				case 'items':
499 499
 					// $join = "INNER JOIN {$wpdb->prefix}frm_item_metas im ON ($table.id = im.item_id)";
500 500
 					if ( $args['ids'] ) {
501
-						$where[ $table . '.form_id' ] = $args['ids'];
501
+						$where[$table . '.form_id'] = $args['ids'];
502 502
 					}
503 503
 					break;
504 504
 				case 'styles':
@@ -539,7 +539,7 @@  discard block
 block discarded – undo
539 539
 					}
540 540
 			}//end switch
541 541
 
542
-			$records[ $tb_type ] = FrmDb::get_col( $table . $join, $where, $select );
542
+			$records[$tb_type] = FrmDb::get_col( $table . $join, $where, $select );
543 543
 			unset( $tb_type );
544 544
 		}//end foreach
545 545
 
@@ -734,7 +734,7 @@  discard block
 block discarded – undo
734 734
 		$no_export_fields = FrmField::no_save_fields();
735 735
 		foreach ( $csv_fields as $k => $f ) {
736 736
 			if ( in_array( $f->type, $no_export_fields, true ) ) {
737
-				unset( $csv_fields[ $k ] );
737
+				unset( $csv_fields[$k] );
738 738
 			}
739 739
 		}
740 740
 
Please login to merge, or discard this patch.