Strategy11 / formidable-forms

classes/helpers/FrmStylesPreviewHelper.php

Spacing +6 added lines, -6 removed lines

@@ -77,7 +77,7 @@ 
 			 *
 			 * @return string
 			 */
-			function ( $position, $field ) {
+			function( $position, $field ) {
 				if ( is_array( $field ) ) {
 					$this->default_label_position_field_ids[] = (int) $field['id'];
 				}
@@ -95,7 +95,7 @@ 
 			 *
 			 * @return string
 			 */
-			function ( $classes, $field ) {
+			function( $classes, $field ) {
 				if ( is_array( $field ) && in_array( (int) $field['id'], $this->default_label_position_field_ids, true ) ) {
 					$classes .= ' frm-default-label-position';
 				}
@@ -123,7 +123,7 @@ 
 			 *
 			 * @return bool
 			 */
-			function ( $show, $field_type ) {
+			function( $show, $field_type ) {
 				if ( 'captcha' === $field_type ) {
 					$show = false;
 				}
@@ -152,7 +152,7 @@ 
 			 *
 			 * @return stdClass|null
 			 */
-			function ( $form ) {
+			function( $form ) {
 				if ( is_object( $form ) && is_array( $form->options ) ) {
 					$form->options['js_validate'] = false;
 				}
@@ -202,7 +202,7 @@ 
 
 		$frm_settings = FrmAppHelper::get_settings();
 		if ( 'none' === $frm_settings->load_style ) {
-			$notes[] = function () {
+			$notes[] = function() {
 				printf(
 					// translators: %1$s: Anchor tag open, %2$s: Anchor tag close.
 					esc_html__( 'Formidable styles are disabled. This needs to be enabled in %1$sGlobal Settings%2$s.', 'formidable' ),
@@ -393,7 +393,7 @@ 
 			return;
 		}
 
-		unset( $dependencies[ $index ] );
+		unset( $dependencies[$index] );
 		$dependencies = array_values( $dependencies );
 
 		$styles->registered['wp-admin']->deps = $dependencies;

phpcs-sniffs/Formidable/Sniffs/WhiteSpace/BlankLineAfterClosingBraceSniff.php

Spacing +8 added lines, -8 removed lines

@@ -75,14 +75,14 @@ 
 			T_COALESCE_EQUAL,
 		);
 
-		if ( ! in_array( $tokens[ $nextToken ]['code'], $assignmentTokens, true ) ) {
+		if ( ! in_array( $tokens[$nextToken]['code'], $assignmentTokens, true ) ) {
 			return;
 		}
 
 		// Find the first token on the current line.
 		$firstOnLine = $stackPtr;
-		for ( $i = $stackPtr - 1; $i >= 0; $i-- ) {
-			if ( $tokens[ $i ]['line'] < $tokens[ $stackPtr ]['line'] ) {
+		for ( $i = $stackPtr - 1; $i >= 0; $i -- ) {
+			if ( $tokens[$i]['line'] < $tokens[$stackPtr]['line'] ) {
 				break;
 			}
 			$firstOnLine = $i;
@@ -95,7 +95,7 @@ 
 		}
 
 		// Now find what's on the previous line.
-		$currentLine  = $tokens[ $stackPtr ]['line'];
+		$currentLine  = $tokens[$stackPtr]['line'];
 		$previousLine = $currentLine - 1;
 
 		if ( $previousLine < 1 ) {
@@ -104,11 +104,11 @@ 
 
 		// Find the last non-whitespace token on the previous line.
 		$lastTokenOnPrevLine = null;
-		for ( $i = $stackPtr - 1; $i >= 0; $i-- ) {
-			if ( $tokens[ $i ]['line'] < $previousLine ) {
+		for ( $i = $stackPtr - 1; $i >= 0; $i -- ) {
+			if ( $tokens[$i]['line'] < $previousLine ) {
 				break;
 			}
-			if ( $tokens[ $i ]['line'] === $previousLine && $tokens[ $i ]['code'] !== T_WHITESPACE ) {
+			if ( $tokens[$i]['line'] === $previousLine && $tokens[$i]['code'] !== T_WHITESPACE ) {
 				$lastTokenOnPrevLine = $i;
 				break;
 			}
@@ -121,7 +121,7 @@ 
 		}
 
 		// Check if the last token on the previous line is a closing brace.
-		if ( $tokens[ $lastTokenOnPrevLine ]['code'] !== T_CLOSE_CURLY_BRACKET ) {
+		if ( $tokens[$lastTokenOnPrevLine]['code'] !== T_CLOSE_CURLY_BRACKET ) {
 			return;
 		}
 

classes/views/frm-settings/payments.php

Spacing +2 added lines, -2 removed lines

@@ -25,8 +25,8 @@
 		$input_params['checked'] = 'checked';
 	}
 
-	$other_section_selectors      = array_map(
-		function ( $section ) {
+	$other_section_selectors = array_map(
+		function( $section ) {
 			return "#frm_{$section}_settings_section";
 		},
 		array_diff( array_keys( $payment_sections ), array( $key ) )

classes/models/fields/FrmFieldEmail.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 
 		$errors = array();
 		if ( false !== strpos( $args['value'], '.@' ) || ! is_email( $args['value'] ) ) {
-			$errors[ 'field' . $args['id'] ] = FrmFieldsHelper::get_error_msg( $this->field, 'invalid' );
+			$errors['field' . $args['id']] = FrmFieldsHelper::get_error_msg( $this->field, 'invalid' );
 		}
 		return $errors;
 	}

classes/models/FrmDb.php

Spacing +10 added lines, -10 removed lines

@@ -355,8 +355,8 @@ 
 
 		$where_is = strtolower( $where_is );
 
-		if ( isset( $switch_to[ $where_is ] ) ) {
-			return ' ' . $switch_to[ $where_is ];
+		if ( isset( $switch_to[$where_is] ) ) {
+			return ' ' . $switch_to[$where_is];
 		}
 
 		// > and < need a little more work since we don't want them switched to >= and <=
@@ -436,14 +436,14 @@ 
 
 		foreach ( $temp_args as $k => $v ) {
 			if ( $v == '' ) {
-				unset( $args[ $k ] );
+				unset( $args[$k] );
 				continue;
 			}
 
 			$db_name = strtoupper( str_replace( '_', ' ', $k ) );
 
 			if ( strpos( $v, $db_name ) === false ) {
-				$args[ $k ] = $db_name . ' ' . $v;
+				$args[$k] = $db_name . ' ' . $v;
 			}
 		}
 
@@ -514,13 +514,13 @@ 
 	private static function esc_query_args( &$args ) {
 		foreach ( $args as $param => $value ) {
 			if ( $param === 'order_by' ) {
-				$args[ $param ] = self::esc_order( $value );
+				$args[$param] = self::esc_order( $value );
 			} elseif ( $param === 'limit' ) {
-				$args[ $param ] = self::esc_limit( $value );
+				$args[$param] = self::esc_limit( $value );
 			}
 
-			if ( $args[ $param ] == '' ) {
-				unset( $args[ $param ] );
+			if ( $args[$param] == '' ) {
+				unset( $args[$param] );
 			}
 		}
 	}
@@ -617,7 +617,7 @@ 
 
 		foreach ( $limit as $k => $l ) {
 			if ( is_numeric( $l ) ) {
-				$limit[ $k ] = $l;
+				$limit[$k] = $l;
 			}
 		}
 
@@ -795,7 +795,7 @@ 
 	 */
 	public static function add_key_to_group_cache( $key, $group ) {
 		$cached         = self::get_group_cached_keys( $group );
-		$cached[ $key ] = $key;
+		$cached[$key] = $key;
 		wp_cache_set( 'cached_keys', $cached, $group, 300 );
 	}
 

classes/models/FrmUsage.php

Spacing +9 added lines, -9 removed lines

@@ -131,7 +131,7 @@ 
 		);
 
 		foreach ( $skipped_keys as $skipped_key ) {
-			unset( $data[ $skipped_key ] );
+			unset( $data[$skipped_key] );
 		}
 
 		return $data;
@@ -239,7 +239,7 @@ 
 
 		foreach ( $pass_settings as $setting ) {
 			if ( isset( $settings_list->$setting ) ) {
-				$settings[ $setting ] = $this->maybe_json( $settings_list->$setting );
+				$settings[$setting] = $this->maybe_json( $settings_list->$setting );
 			}
 		}
 
@@ -277,7 +277,7 @@ 
 		$message_settings = array();
 
 		foreach ( $messages as $message ) {
-			$message_settings[ 'changed-' . $message ] = $settings_list->$message === $default[ $message ] ? 0 : 1;
+			$message_settings['changed-' . $message] = $settings_list->$message === $default[$message] ? 0 : 1;
 		}
 
 		return $message_settings;
@@ -298,7 +298,7 @@ 
 
 		foreach ( $frm_roles as $frm_role => $frm_role_description ) {
 			if ( isset( $settings_list->$frm_role ) ) {
-				$permissions[ $frm_role ] = $settings_list->$frm_role;
+				$permissions[$frm_role] = $settings_list->$frm_role;
 			}
 		}
 
@@ -371,9 +371,9 @@ 
 			);
 
 			foreach ( $settings as $setting ) {
-				if ( isset( $form->options[ $setting ] ) ) {
+				if ( isset( $form->options[$setting] ) ) {
 					if ( 'custom_style' === $setting ) {
-						$style->id = $form->options[ $setting ];
+						$style->id = $form->options[$setting];
 
 						if ( ! $style->id ) {
 							$style_name = 0;
@@ -384,9 +384,9 @@ 
 							$style_name = $style_post ? $style_post->post_name : 'formidable-style';
 						}
 
-						$new_form[ $setting ] = $style_name;
+						$new_form[$setting] = $style_name;
 					} else {
-						$new_form[ $setting ] = $this->maybe_json( $form->options[ $setting ] );
+						$new_form[$setting] = $this->maybe_json( $form->options[$setting] );
 					}
 				}
 			}
@@ -454,7 +454,7 @@ 
 
 		foreach ( $fields as $k => $field ) {
 			FrmAppHelper::unserialize_or_decode( $field->field_options );
-			$fields[ $k ]->field_options = json_encode( $field->field_options );
+			$fields[$k]->field_options = json_encode( $field->field_options );
 		}
 		return $fields;
 	}

classes/models/FrmFormApi.php

Spacing +4 added lines, -4 removed lines

@@ -143,13 +143,13 @@ 
 				$cats = array_intersect( $this->skip_categories(), $addon['categories'] );
 
 				if ( ! empty( $cats ) ) {
-					unset( $addons[ $k ] );
+					unset( $addons[$k] );
 					continue;
 				}
 			}
 
 			if ( ! array_key_exists( 'is_new', $addon ) && array_key_exists( 'released', $addon ) ) {
-				$addons[ $k ]['is_new'] = $this->is_new( $addon );
+				$addons[$k]['is_new'] = $this->is_new( $addon );
 			}
 		}
 
@@ -266,8 +266,8 @@ 
 					return $addon;
 				}
 			}
-		} elseif ( isset( $addons[ $download_id ] ) ) {
-			$plugin = $addons[ $download_id ];
+		} elseif ( isset( $addons[$download_id] ) ) {
+			$plugin = $addons[$download_id];
 		}
 
 		return $plugin;

classes/models/FrmFormMigrator.php

Spacing +11 added lines, -11 removed lines

@@ -38,7 +38,7 @@ 
 	/**
 	 * @var array
 	 */
-	protected $fields_map          = array();
+	protected $fields_map = array();
 
 	/**
 	 * @var mixed
@@ -48,7 +48,7 @@ 
 	/**
 	 * @var array
 	 */
-	protected $current_section     = array();
+	protected $current_section = array();
 
 	/**
 	 * Define required properties.
@@ -319,7 +319,7 @@ 
 
 			// This may occasionally skip one level/order e.g. after adding a
 			// list field, as field_order would already be prepared to be used.
-			++$field_order;
+			++ $field_order;
 
 			if ( ! empty( $new_field['fields'] ) && is_array( $new_field['fields'] ) ) {
 				// we have (inner) fields to merge
@@ -371,7 +371,7 @@ 
 		$order = 0;
 
 		foreach ( $fields as $field ) {
-			++$order;
+			++ $order;
 			$type     = $this->get_field_type( $field );
 			$new_type = $this->convert_field_type( $type, $field );
 
@@ -407,7 +407,7 @@ 
 		$sub['name'] = __( 'Section Buttons', 'formidable' );
 		$subs        = array( $sub );
 		$this->insert_fields_in_array( $subs, $order, 0, $fields );
-		++$order;
+		++ $order;
 	}
 
 	/**
@@ -521,7 +521,7 @@ 
 	protected function create_fields( $form_id, &$form ) {
 		foreach ( $form['fields'] as $key => $new_field ) {
 			$new_field['form_id']         = $form_id;
-			$form['fields'][ $key ]['id'] = FrmField::create( $new_field );
+			$form['fields'][$key]['id'] = FrmField::create( $new_field );
 		}
 	}
 
@@ -564,9 +564,9 @@ 
 			} elseif ( $key === 'the_post_title' ) {
 				$new_action->post_content['post_title'] = $value;
 			} elseif ( is_string( $value ) ) {
-				$new_action->post_content[ $key ] = $this->replace_smart_tags( $value, $form['fields'] );
+				$new_action->post_content[$key] = $this->replace_smart_tags( $value, $form['fields'] );
 			} else {
-				$new_action->post_content[ $key ] = $value;
+				$new_action->post_content[$key] = $value;
 			}
 		}
 
@@ -587,7 +587,7 @@ 
 
 		$imported = $this->get_tracked_import();
 
-		$imported[ $this->slug ][ $new_form_id ] = $source_id;
+		$imported[$this->slug][$new_form_id] = $source_id;
 
 		update_option( $this->tracking, $imported, false );
 	}
@@ -608,11 +608,11 @@ 
 		$imported    = $this->get_tracked_import();
 		$new_form_id = 0;
 
-		if ( ! isset( $imported[ $this->slug ] ) || ! in_array( $source_id, $imported[ $this->slug ] ) ) {
+		if ( ! isset( $imported[$this->slug] ) || ! in_array( $source_id, $imported[$this->slug] ) ) {
 			return $new_form_id;
 		}
 
-		$new_form_id = array_search( $source_id, array_reverse( $imported[ $this->slug ], true ) );
+		$new_form_id = array_search( $source_id, array_reverse( $imported[$this->slug], true ) );
 
 		if ( ! empty( $new_form_id ) && empty( FrmForm::get_key_by_id( $new_form_id ) ) ) {
 			// Allow reimport if the form was deleted.

classes/models/FrmSettings.php

Spacing +4 added lines, -4 removed lines

@@ -393,8 +393,8 @@ 
 		}
 
 		foreach ( $settings as $setting => $default ) {
-			if ( isset( $params[ 'frm_' . $setting ] ) ) {
-				$this->{$setting} = $params[ 'frm_' . $setting ];
+			if ( isset( $params['frm_' . $setting] ) ) {
+				$this->{$setting} = $params['frm_' . $setting];
 			} elseif ( ! isset( $this->{$setting} ) ) {
 				$this->{$setting} = $default;
 			}
@@ -594,7 +594,7 @@ 
 		);
 
 		foreach ( $checkboxes as $set ) {
-			$this->$set = isset( $params[ 'frm_' . $set ] ) ? absint( $params[ 'frm_' . $set ] ) : 0;
+			$this->$set = isset( $params['frm_' . $set] ) ? absint( $params['frm_' . $set] ) : 0;
 		}
 	}
 
@@ -610,7 +610,7 @@ 
 		$roles     = get_editable_roles();
 
 		foreach ( $frm_roles as $frm_role => $frm_role_description ) {
-			$this->$frm_role = (array) ( $params[ $frm_role ] ?? 'administrator' );
+			$this->$frm_role = (array) ( $params[$frm_role] ?? 'administrator' );
 
 			// Make sure administrators always have permissions
 			if ( ! in_array( 'administrator', $this->$frm_role, true ) ) {