Strategy11 / formidable-forms

stripe/helpers/FrmStrpLiteUrlParamHelper.php

Spacing +3 added lines, -3 removed lines

@@ -23,10 +23,10 @@ 
 	 * @return array|false
 	 */
 	public static function get_details_for_form( $form_id ) {
-		if ( ! isset( self::$details_by_form_id[ $form_id ] ) ) {
+		if ( ! isset( self::$details_by_form_id[$form_id] ) ) {
 			self::set_details_for_form( (int) $form_id );
 		}
-		return isset( self::$details_by_form_id[ $form_id ] ) ? self::$details_by_form_id[ $form_id ] : false;
+		return isset( self::$details_by_form_id[$form_id] ) ? self::$details_by_form_id[$form_id] : false;
 	}
 
 	/**
@@ -87,7 +87,7 @@ 
 			return;
 		}
 
-		self::$details_by_form_id[ $form_id ] = array(
+		self::$details_by_form_id[$form_id] = array(
 			'entry'   => $entry,
 			'intent'  => $intent,
 			'payment' => $payment,

classes/helpers/FrmCurrencyHelper.php

Spacing +4 added lines, -4 removed lines

@@ -16,10 +16,10 @@
 	public static function get_currency( $currency ) {
 		$currency   = strtoupper( $currency );
 		$currencies = self::get_currencies();
-		if ( isset( $currencies[ $currency ] ) ) {
-			$currency = $currencies[ $currency ];
-		} elseif ( isset( $currencies[ strtolower( $currency ) ] ) ) {
-			$currency = $currencies[ strtolower( $currency ) ];
+		if ( isset( $currencies[$currency] ) ) {
+			$currency = $currencies[$currency];
+		} elseif ( isset( $currencies[strtolower( $currency )] ) ) {
+			$currency = $currencies[strtolower( $currency )];
 		} else {
 			$currency = $currencies['USD'];
 		}

stripe/models/FrmTransLitePayment.php

Spacing +3 added lines, -3 removed lines

@@ -126,11 +126,11 @@
 		foreach ( $payments as $payment ) {
 			list( $amount, $currency ) = FrmTransLiteAppHelper::get_amount_and_currency_from_payment( $payment );
 
-			if ( ! isset( $data[ $currency ] ) ) {
-				$data[ $currency ] = 0;
+			if ( ! isset( $data[$currency] ) ) {
+				$data[$currency] = 0;
 			}
 
-			$data[ $currency ] += floatval( $amount );
+			$data[$currency] += floatval( $amount );
 		}
 
 		return $data;

classes/helpers/FrmEmailSummaryHelper.php

Spacing +3 added lines, -3 removed lines

@@ -236,11 +236,11 @@ 
 	 */
 	public static function get_last_sent_date( $type ) {
 		$options = self::get_options();
-		if ( empty( $options[ 'last_' . $type ] ) ) {
+		if ( empty( $options['last_' . $type] ) ) {
 			return false;
 		}
 
-		return $options[ 'last_' . $type ];
+		return $options['last_' . $type];
 	}
 
 	/**
@@ -252,7 +252,7 @@ 
 	public static function set_last_sent_date( $type, $value = null ) {
 		$options = self::get_options();
 
-		$options[ 'last_' . $type ] = null === $value ? self::get_date_from_today() : '';
+		$options['last_' . $type] = null === $value ? self::get_date_from_today() : '';
 		self::save_options( $options );
 	}
 

stripe/helpers/FrmStrpLiteConnectHelper.php

Spacing +2 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
 			'user_id'               => get_current_user_id(),
 			'frm_strp_connect_mode' => $mode,
 		);
-		$data            = self::post_to_connect_server( 'initialize', $additional_body );
+		$data = self::post_to_connect_server( 'initialize', $additional_body );
 
 		if ( is_string( $data ) ) {
 			return $data;
@@ -143,7 +143,7 @@ 
 	 * @return object|string
 	 */
 	private static function post_to_connect_server( $action, $additional_body = array() ) {
-		$body    = array(
+		$body = array(
 			'frm_strp_connect_action' => $action,
 			'frm_strp_connect_mode'   => FrmStrpLiteAppHelper::active_mode(),
 		);

classes/views/frm-fields/front-end/checkbox-field.php

Spacing +1 added lines, -1 removed lines

@@ -87,6 +87,6 @@
 
 		?></div>
 <?php
-		++$option_index;
+		++ $option_index;
 	}//end foreach
 }//end if

classes/views/frm-fields/front-end/combo-field/combo-field.php

Spacing +6 added lines, -6 removed lines

@@ -41,7 +41,7 @@ 
 			$sub_field_class   = "frm_form_field form-field frm_form_subfield-{$name} {$sub_field['wrapper_classes']}";
 			$sub_field_desc    = FrmField::get_option( $field, $name . '_desc' );
 
-			if ( isset( $errors[ 'field' . $field_id . '-' . $name ] ) ) {
+			if ( isset( $errors['field' . $field_id . '-' . $name] ) ) {
 				$sub_field_class .= ' frm_blank_field';
 			}
 			?>
@@ -61,10 +61,10 @@ 
 						<input
 							type="<?php echo esc_attr( $sub_field['type'] ); ?>"
 							id="<?php echo esc_attr( $html_id . '_' . $name ); ?>"
-							value="<?php echo esc_attr( isset( $field_value[ $name ] ) ? $field_value[ $name ] : '' ); ?>"
+							value="<?php echo esc_attr( isset( $field_value[$name] ) ? $field_value[$name] : '' ); ?>"
 							<?php
-							if ( ! empty( $field_value[ $name ] ) ) {
-								echo 'data-frmval="' . esc_attr( $field_value[ $name ] ) . '" ';
+							if ( ! empty( $field_value[$name] ) ) {
+								echo 'data-frmval="' . esc_attr( $field_value[$name] ) . '" ';
 							}
 							if ( empty( $args['remove_names'] ) ) {
 								echo 'name="' . esc_attr( $field_name ) . '[' . esc_attr( $name ) . ']" ';
@@ -81,9 +81,9 @@ 
 				}
 
 				// Don't show individual field errors when there is a combo field error.
-				if ( ! empty( $errors ) && isset( $errors[ 'field' . $field_id . '-' . $name ] ) && ! isset( $errors[ 'field' . $field_id ] ) ) {
+				if ( ! empty( $errors ) && isset( $errors['field' . $field_id . '-' . $name] ) && ! isset( $errors['field' . $field_id] ) ) {
 					?>
-					<div class="frm_error" role="alert"><?php echo esc_html( $errors[ 'field' . $field_id . '-' . $name ] ); ?></div>
+					<div class="frm_error" role="alert"><?php echo esc_html( $errors['field' . $field_id . '-' . $name] ); ?></div>
 				<?php } ?>
 			</div>
 			<?php

classes/controllers/FrmSettingsController.php

Spacing +3 added lines, -3 removed lines

@@ -158,7 +158,7 @@ 
 				$section['function'] = $original;
 			}
 
-			$sections[ $key ] = $section;
+			$sections[$key] = $section;
 		}//end foreach
 
 		return $sections;
@@ -170,11 +170,11 @@ 
 
 		$section  = FrmAppHelper::get_post_param( 'tab', '', 'sanitize_text_field' );
 		$sections = self::get_settings_tabs();
-		if ( ! isset( $sections[ $section ] ) ) {
+		if ( ! isset( $sections[$section] ) ) {
 			wp_die();
 		}
 
-		$section = $sections[ $section ];
+		$section = $sections[$section];
 
 		if ( isset( $section['class'] ) ) {
 			call_user_func( array( $section['class'], $section['function'] ) );

classes/models/FrmEntryValidate.php

Braces +2 added lines, -1 removed lines

@@ -145,7 +145,8 @@
 
 		if ( $posted_field->required == '1' && FrmAppHelper::is_empty_value( $value ) ) {
 			$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' );
-		} elseif ( ! isset( $_POST['item_name'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		} elseif ( ! isset( $_POST['item_name'] ) ) {
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
 			self::maybe_add_item_name( $value, $posted_field );
 		}
 

Spacing +31 added lines, -31 removed lines

@@ -27,7 +27,7 @@ 
 			return $errors;
 		}
 
-		if ( FrmAppHelper::is_admin() && is_user_logged_in() && ( ! isset( $values[ 'frm_submit_entry_' . $values['form_id'] ] ) || ! wp_verify_nonce( $values[ 'frm_submit_entry_' . $values['form_id'] ], 'frm_submit_entry_nonce' ) ) ) {
+		if ( FrmAppHelper::is_admin() && is_user_logged_in() && ( ! isset( $values['frm_submit_entry_' . $values['form_id']] ) || ! wp_verify_nonce( $values['frm_submit_entry_' . $values['form_id']], 'frm_submit_entry_nonce' ) ) ) {
 			$frm_settings   = FrmAppHelper::get_settings();
 			$errors['form'] = $frm_settings->admin_permission;
 		}
@@ -130,7 +130,7 @@ 
 		$args = wp_parse_args( $args, $defaults );
 
 		if ( empty( $args['parent_field_id'] ) ) {
-			$value = isset( $values['item_meta'][ $args['id'] ] ) ? $values['item_meta'][ $args['id'] ] : '';
+			$value = isset( $values['item_meta'][$args['id']] ) ? $values['item_meta'][$args['id']] : '';
 		} else {
 			// value is from a nested form
 			$value = $values;
@@ -151,7 +151,7 @@ 
 		}
 
 		if ( $posted_field->required == '1' && FrmAppHelper::is_empty_value( $value ) ) {
-			$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' );
+			$errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $posted_field, 'blank' );
 		} elseif ( ! isset( $_POST['item_name'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
 			self::maybe_add_item_name( $value, $posted_field );
 		}
@@ -270,7 +270,7 @@ 
 		$values['value'] = $value;
 		FrmFieldsHelper::prepare_new_front_field( $values, $field_object );
 
-		$map_callback = function ( $option ) {
+		$map_callback = function( $option ) {
 			return is_array( $option ) ? $option['value'] : $option;
 		};
 
@@ -338,7 +338,7 @@ 
 			$pattern = self::phone_format( $field );
 
 			if ( ! preg_match( $pattern, $value ) ) {
-				$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $field, 'invalid' );
+				$errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $field, 'invalid' );
 			}
 		}
 	}
@@ -451,7 +451,7 @@ 
 	 */
 	private static function form_is_in_progress( $values ) {
 		return FrmAppHelper::pro_is_installed() &&
-			( isset( $values[ 'frm_page_order_' . $values['form_id'] ] ) || FrmAppHelper::get_post_param( 'frm_next_page' ) ) &&
+			( isset( $values['frm_page_order_' . $values['form_id']] ) || FrmAppHelper::get_post_param( 'frm_next_page' ) ) &&
 			FrmField::get_all_types_in_form( $values['form_id'], 'break' );
 	}
 
@@ -697,9 +697,9 @@ 
 			foreach ( $datas['missing_keys'] as $key_index => $key ) {
 				$found = self::is_akismet_guest_info_value( $key, $value, $field_id, $datas['name_field_ids'], $values );
 				if ( $found ) {
-					$datas[ $key ]             = $value;
+					$datas[$key]             = $value;
 					$datas['frm_duplicated'][] = $field_id;
-					unset( $datas['missing_keys'][ $key_index ] );
+					unset( $datas['missing_keys'][$key_index] );
 				}
 			}
 		}//end foreach
@@ -742,11 +742,11 @@ 
 					if ( 'Name' !== $field->name ) {
 						continue;
 					}
-					if ( isset( $fields[ $index + 1 ] ) && 'Last' === $fields[ $index + 1 ]->name ) {
-						if ( empty( $values[ absint( $fields[ $index + 1 ]->id ) ] ) ) {
+					if ( isset( $fields[$index + 1] ) && 'Last' === $fields[$index + 1]->name ) {
+						if ( empty( $values[absint( $fields[$index + 1]->id )] ) ) {
 							continue;
 						}
-						$value .= ' ' . $values[ $fields[ $index + 1 ]->id ];
+						$value .= ' ' . $values[$fields[$index + 1]->id];
 						return true;
 					}
 				}
@@ -765,13 +765,13 @@ 
 	 */
 	private static function get_name_text_fields( $form_id ) {
 		$name_text_fields_is_initialized = is_array( self::$name_text_fields );
-		if ( $name_text_fields_is_initialized && isset( self::$name_text_fields[ $form_id ] ) ) {
-			return self::$name_text_fields[ $form_id ];
+		if ( $name_text_fields_is_initialized && isset( self::$name_text_fields[$form_id] ) ) {
+			return self::$name_text_fields[$form_id];
 		}
 		if ( ! $name_text_fields_is_initialized ) {
 			self::$name_text_fields = array();
 		}
-		self::$name_text_fields[ $form_id ] = FrmDb::get_results(
+		self::$name_text_fields[$form_id] = FrmDb::get_results(
 			'frm_fields',
 			array(
 				'form_id' => $form_id,
@@ -782,7 +782,7 @@ 
 			array( 'order_by' => 'field_order ASC' )
 		);
 
-		return self::$name_text_fields[ $form_id ];
+		return self::$name_text_fields[$form_id];
 	}
 
 	private static function add_server_values_to_akismet( &$datas ) {
@@ -791,7 +791,7 @@ 
 
 			// Send any potentially useful $_SERVER vars, but avoid sending junk we don't need.
 			if ( $include_value ) {
-				$datas[ $key ] = $value;
+				$datas[$key] = $value;
 			}
 			unset( $key, $value );
 		}
@@ -808,10 +808,10 @@ 
 	private static function add_comment_content_to_akismet( &$datas, $values ) {
 		if ( isset( $datas['frm_duplicated'] ) ) {
 			foreach ( $datas['frm_duplicated'] as $index ) {
-				if ( isset( $values['item_meta'][ $index ] ) ) {
-					unset( $values['item_meta'][ $index ] );
+				if ( isset( $values['item_meta'][$index] ) ) {
+					unset( $values['item_meta'][$index] );
 				} else {
-					unset( $values[ $index ] );
+					unset( $values[$index] );
 				}
 			}
 			unset( $datas['frm_duplicated'] );
@@ -830,14 +830,14 @@ 
 	private static function skip_adding_values_to_akismet( &$values ) {
 		$skipped_fields = self::get_akismet_skipped_field_ids( $values );
 		foreach ( $skipped_fields as $skipped_field ) {
-			if ( ! isset( $values['item_meta'][ $skipped_field->id ] ) ) {
+			if ( ! isset( $values['item_meta'][$skipped_field->id] ) ) {
 				continue;
 			}
 
 			if ( self::should_really_skip_field( $skipped_field, $values ) ) {
-				unset( $values['item_meta'][ $skipped_field->id ] );
-				if ( isset( $values['item_meta']['other'][ $skipped_field->id ] ) ) {
-					unset( $values['item_meta']['other'][ $skipped_field->id ] );
+				unset( $values['item_meta'][$skipped_field->id] );
+				if ( isset( $values['item_meta']['other'][$skipped_field->id] ) ) {
+					unset( $values['item_meta']['other'][$skipped_field->id] );
 				}
 			}
 		}
@@ -873,14 +873,14 @@ 
 		}
 
 		// If a choice field has Other option, but Other is not selected.
-		if ( empty( $values['item_meta']['other'][ $field_data->id ] ) ) {
+		if ( empty( $values['item_meta']['other'][$field_data->id] ) ) {
 			return true;
 		}
 
 		// Check if submitted value is same as one of field option.
 		foreach ( $field_data->options as $option ) {
 			$option_value = ! is_array( $option ) ? $option : ( isset( $option['value'] ) ? $option['value'] : '' );
-			if ( $values['item_meta']['other'][ $field_data->id ] === $option_value ) {
+			if ( $values['item_meta']['other'][$field_data->id] === $option_value ) {
 				return true;
 			}
 		}
@@ -951,7 +951,7 @@ 
 
 			// Convert name array to string.
 			if ( isset( $value['first'] ) && isset( $value['last'] ) ) {
-				$values['item_meta'][ $field_id ] = trim( implode( ' ', $value ) );
+				$values['item_meta'][$field_id] = trim( implode( ' ', $value ) );
 				$values['name_field_ids'][]       = $field_id;
 				continue;
 			}
@@ -972,8 +972,8 @@ 
 						continue;
 					}
 
-					if ( ! isset( $values['item_meta'][ $subsubindex ] ) ) {
-						$values['item_meta'][ $subsubindex ] = array();
+					if ( ! isset( $values['item_meta'][$subsubindex] ) ) {
+						$values['item_meta'][$subsubindex] = array();
 					}
 
 					// Convert name array to string.
@@ -983,13 +983,13 @@ 
 						$values['name_field_ids'][] = $subsubindex;
 					}
 
-					if ( is_array( $values['item_meta'][ $subsubindex ] ) ) {
-						$values['item_meta'][ $subsubindex ][] = $subsubvalue;
+					if ( is_array( $values['item_meta'][$subsubindex] ) ) {
+						$values['item_meta'][$subsubindex][] = $subsubvalue;
 					}
 				}
 			}//end foreach
 
-			unset( $values['item_meta'][ $field_id ] );
+			unset( $values['item_meta'][$field_id] );
 		}//end foreach
 
 		return $form_ids;