Strategy11 / formidable-forms

classes/models/FrmEntryShortcodeFormatter.php

Spacing +1 added lines, -1 removed lines

@@ -285,7 +285,7 @@
 			'type'  => $field->type,
 		);
 
-		$this->array_content[ $field->id ] = apply_filters( 'frm_field_shortcodes_for_default_html_email', $array, $field );
+		$this->array_content[$field->id] = apply_filters( 'frm_field_shortcodes_for_default_html_email', $array, $field );
 	}
 
 	/**

classes/models/FrmMigrate.php

Spacing +2 added lines, -2 removed lines

@@ -326,7 +326,7 @@ 
 				continue;
 			}
 
-			$this->maybe_convert_migrated_size( $widgets[ $k ]['size'] );
+			$this->maybe_convert_migrated_size( $widgets[$k]['size'] );
 		}
 		update_option( 'widget_frm_show_form', $widgets );
 	}
@@ -422,7 +422,7 @@ 
 			if ( ! is_array( $widget ) || ! isset( $widget['size'] ) ) {
 				continue;
 			}
-			$this->convert_character_to_px( $widgets[ $k ]['size'] );
+			$this->convert_character_to_px( $widgets[$k]['size'] );
 		}
 		update_option( 'widget_frm_show_form', $widgets );
 	}

classes/models/FrmInstallerSkin.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	 */
 	public function set_upgrader( &$upgrader ) {
 		if ( is_object( $upgrader ) ) {
-			$this->upgrader =& $upgrader;
+			$this->upgrader = & $upgrader;
 		}
 	}
 

classes/models/FrmFieldValue.php

Spacing +2 added lines, -2 removed lines

@@ -59,8 +59,8 @@
 	protected function init_saved_value( $entry ) {
 		if ( $this->field->type === 'html' ) {
 			$this->saved_value = $this->field->description;
-		} elseif ( isset( $entry->metas[ $this->field->id ] ) ) {
-			$this->saved_value = $entry->metas[ $this->field->id ];
+		} elseif ( isset( $entry->metas[$this->field->id] ) ) {
+			$this->saved_value = $entry->metas[$this->field->id];
 		} else {
 			$this->saved_value = '';
 		}

classes/controllers/FrmEntriesController.php

Braces +2 added lines, -1 removed lines

@@ -491,7 +491,8 @@
 
 	public static function process_entry( $errors = '', $ajax = false ) {
 		$form_id = FrmAppHelper::get_post_param( 'form_id', '', 'absint' );
-		if ( FrmAppHelper::is_admin() || empty( $_POST ) || empty( $form_id ) || ! isset( $_POST['item_key'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		if ( FrmAppHelper::is_admin() || empty( $_POST ) || empty( $form_id ) || ! isset( $_POST['item_key'] ) ) {
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
 			return;
 		}
 

Spacing +27 added lines, -27 removed lines

@@ -170,20 +170,20 @@ 
 		global $frm_vars;
 		$form_id = FrmForm::get_current_form_id();
 
-		$columns[ $form_id . '_id' ]       = 'ID';
-		$columns[ $form_id . '_item_key' ] = esc_html__( 'Entry Key', 'formidable' );
+		$columns[$form_id . '_id']       = 'ID';
+		$columns[$form_id . '_item_key'] = esc_html__( 'Entry Key', 'formidable' );
 
 		if ( $form_id ) {
 			self::get_columns_for_form( $form_id, $columns );
 		} else {
-			$columns[ $form_id . '_form_id' ] = esc_html__( 'Form', 'formidable' );
-			$columns[ $form_id . '_name' ]    = esc_html__( 'Entry Name', 'formidable' );
-			$columns[ $form_id . '_user_id' ] = esc_html__( 'Created By', 'formidable' );
+			$columns[$form_id . '_form_id'] = esc_html__( 'Form', 'formidable' );
+			$columns[$form_id . '_name']    = esc_html__( 'Entry Name', 'formidable' );
+			$columns[$form_id . '_user_id'] = esc_html__( 'Created By', 'formidable' );
 		}
 
-		$columns[ $form_id . '_is_draft' ]   = esc_html__( 'Entry Status', 'formidable' );
-		$columns[ $form_id . '_created_at' ] = __( 'Entry creation date', 'formidable' );
-		$columns[ $form_id . '_updated_at' ] = __( 'Entry update date', 'formidable' );
+		$columns[$form_id . '_is_draft']   = esc_html__( 'Entry Status', 'formidable' );
+		$columns[$form_id . '_created_at'] = __( 'Entry creation date', 'formidable' );
+		$columns[$form_id . '_updated_at'] = __( 'Entry update date', 'formidable' );
 		self::maybe_add_ip_col( $form_id, $columns );
 
 		$frm_vars['cols'] = $columns;
@@ -256,10 +256,10 @@ 
 
 		foreach ( $sub_form_cols as $k => $sub_form_col ) {
 			if ( FrmField::is_no_save_field( $sub_form_col->type ) ) {
-				unset( $sub_form_cols[ $k ] );
+				unset( $sub_form_cols[$k] );
 				continue;
 			}
-			$columns[ $form_id . '_' . $sub_form_col->field_key . '-_-' . $field->id ] = FrmAppHelper::truncate( $sub_form_col->name, 35 );
+			$columns[$form_id . '_' . $sub_form_col->field_key . '-_-' . $field->id] = FrmAppHelper::truncate( $sub_form_col->name, 35 );
 			unset( $sub_form_col );
 		}
 	}
@@ -285,10 +285,10 @@ 
 		$include_column_for_sep_val = $has_separate_value && ! $is_post_status;
 
 		if ( $include_column_for_sep_val ) {
-			$columns[ $form_id . '_frmsep_' . $col_id ] = self::maybe_format_field_name_for_column_title( $field, $include_column_for_sep_val );
+			$columns[$form_id . '_frmsep_' . $col_id] = self::maybe_format_field_name_for_column_title( $field, $include_column_for_sep_val );
 		}
 
-		$columns[ $form_id . '_' . $col_id ] = self::maybe_format_field_name_for_column_title( $field, $include_column_for_sep_val, false );
+		$columns[$form_id . '_' . $col_id] = self::maybe_format_field_name_for_column_title( $field, $include_column_for_sep_val, false );
 	}
 
 	/**
@@ -322,7 +322,7 @@ 
 	 */
 	private static function maybe_add_ip_col( $form_id, &$columns ) {
 		if ( FrmAppHelper::ips_saved() ) {
-			$columns[ $form_id . '_ip' ] = 'IP';
+			$columns[$form_id . '_ip'] = 'IP';
 		}
 	}
 
@@ -382,7 +382,7 @@ 
 		foreach ( $meta_value as $mk => $mv ) {
 			// Remove blank values.
 			if ( empty( $mv ) ) {
-				unset( $meta_value[ $mk ] );
+				unset( $meta_value[$mk] );
 			}
 		}
 
@@ -479,14 +479,14 @@ 
 		);
 
 		if ( ! $form_id ) {
-			$columns[ $form_id . '_user_id' ] = 'user_id';
-			$columns[ $form_id . '_name' ]    = 'name';
-			$columns[ $form_id . '_form_id' ] = 'form_id';
+			$columns[$form_id . '_user_id'] = 'user_id';
+			$columns[$form_id . '_name']    = 'name';
+			$columns[$form_id . '_form_id'] = 'form_id';
 		}
 
 		foreach ( $fields as $field ) {
 			if ( self::field_supports_sorting( $field ) ) {
-				$columns[ $form_id . '_' . $field->field_key ] = 'meta_' . $field->id;
+				$columns[$form_id . '_' . $field->field_key] = 'meta_' . $field->id;
 			}
 		}
 
@@ -583,7 +583,7 @@ 
 			$atts['form_id'] . '_item_key' => '',
 			$atts['form_id'] . '_id'       => '',
 		);
-		$cols         = $remove_first + array_reverse( $frm_vars['cols'], true );
+		$cols = $remove_first + array_reverse( $frm_vars['cols'], true );
 
 		$i = $atts['i'];
 
@@ -594,7 +594,7 @@ 
 
 			if ( ! $result || ! in_array( $col_key, $result, true ) ) {
 				$result[] = $col_key;
-				--$i;
+				-- $i;
 			}
 
 			unset( $col_key, $col );
@@ -773,9 +773,9 @@ 
 		if ( ! isset( $frm_vars['form_params'] ) ) {
 			$frm_vars['form_params'] = array();
 		}
-		$frm_vars['form_params'][ $form->id ] = $params;
+		$frm_vars['form_params'][$form->id] = $params;
 
-		if ( isset( $frm_vars['created_entries'][ $form_id ] ) ) {
+		if ( isset( $frm_vars['created_entries'][$form_id] ) ) {
 			return;
 		}
 
@@ -792,17 +792,17 @@ 
 		 */
 		$errors = apply_filters( 'frm_entries_before_create', $errors, $form );
 
-		$frm_vars['created_entries'][ $form_id ] = array( 'errors' => $errors );
+		$frm_vars['created_entries'][$form_id] = array( 'errors' => $errors );
 
 		if ( ! $errors ) {
 			$_POST['frm_skip_cookie'] = 1;
 			$do_success               = false;
 
 			if ( $params['action'] === 'create' ) {
-				if ( apply_filters( 'frm_continue_to_create', true, $form_id ) && ! isset( $frm_vars['created_entries'][ $form_id ]['entry_id'] ) ) {
-					$frm_vars['created_entries'][ $form_id ]['entry_id'] = FrmEntry::create( $_POST ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
+				if ( apply_filters( 'frm_continue_to_create', true, $form_id ) && ! isset( $frm_vars['created_entries'][$form_id]['entry_id'] ) ) {
+					$frm_vars['created_entries'][$form_id]['entry_id'] = FrmEntry::create( $_POST ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
 
-					$params['id'] = $frm_vars['created_entries'][ $form_id ]['entry_id'];
+					$params['id'] = $frm_vars['created_entries'][$form_id]['entry_id'];
 					$do_success   = true;
 				}
 			}
@@ -930,7 +930,7 @@ 
 		$atts = shortcode_atts( $defaults, $atts );
 
 		if ( $atts['default_email'] ) {
-			$shortcode_atts  = array(
+			$shortcode_atts = array(
 				'format'     => $atts['format'],
 				'plain_text' => $atts['plain_text'],
 			);

classes/models/FrmSolution.php

Braces +2 added lines, -1 removed lines

@@ -42,7 +42,8 @@
 		}
 
 		// Only do this for single site installs.
-		if ( isset( $_GET['activate-multi'] ) || is_network_admin() ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		if ( isset( $_GET['activate-multi'] ) || is_network_admin() ) {
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
 			return;
 		}
 

Spacing +10 added lines, -10 removed lines

@@ -207,7 +207,7 @@ 
 	 */
 	public function add_settings( $sections ) {
 		wp_enqueue_style( 'formidable-pro-fields' );
-		$sections[ $this->plugin_slug ] = array(
+		$sections[$this->plugin_slug] = array(
 			'class'    => $this,
 			'function' => 'settings_page',
 			'name'     => $this->plugin_name(),
@@ -380,9 +380,9 @@ 
 			// Set the current step.
 			if ( ! isset( $step['current'] ) ) {
 				if ( $step['complete'] ) {
-					$steps[ $k ]['current'] = false;
+					$steps[$k]['current'] = false;
 				} else {
-					$steps[ $k ]['current'] = ! $has_current;
+					$steps[$k]['current'] = ! $has_current;
 					$has_current            = true;
 				}
 			} elseif ( $step['current'] ) {
@@ -393,10 +393,10 @@ 
 			$class  = $step['button_class'] ?? '';
 			$class .= ' button-primary frm-button-primary';
 
-			if ( ! $steps[ $k ]['current'] ) {
+			if ( ! $steps[$k]['current'] ) {
 				$class .= ' frm_grey disabled';
 			}
-			$steps[ $k ]['button_class'] = $class;
+			$steps[$k]['button_class'] = $class;
 		}//end foreach
 
 		return $steps;
@@ -567,7 +567,7 @@ 
 		$addons = $api->get_api_info();
 
 		$id       = $this->download_id();
-		$has_file = isset( $addons[ $id ] ) && isset( $addons[ $id ]['beta'] );
+		$has_file = isset( $addons[$id] ) && isset( $addons[$id]['beta'] );
 
 		if ( ! $step['current'] ) {
 			?>
@@ -582,10 +582,10 @@ 
 
 		if ( ! $has_file ) {
 			echo '<p class="frm_error_style">' . esc_html__( 'We didn\'t find anything to import. Please contact our team.', 'formidable' ) . '</p>';
-		} elseif ( ! isset( $addons[ $id ]['beta']['package'] ) ) {
+		} elseif ( ! isset( $addons[$id]['beta']['package'] ) ) {
 			echo '<p class="frm_error_style">' . esc_html__( 'Looks like you may not have a current subscription for this solution. Please check your account.', 'formidable' ) . '</p>';
 		} else {
-			$xml = $addons[ $id ]['beta']['package'];
+			$xml = $addons[$id]['beta']['package'];
 
 			if ( is_array( $xml ) ) {
 				$xml = reset( $xml );
@@ -666,7 +666,7 @@ 
 		foreach ( $options as $info ) {
 			// Count the number of options displayed for css.
 			if ( $count > 1 && ! isset( $info['img'] ) ) {
-				--$count;
+				-- $count;
 			}
 		}
 
@@ -764,7 +764,7 @@ 
 			$was_imported = isset( $form['form'] ) ? FrmForm::get_id_by_key( $form['form'] ) : false;
 
 			if ( $was_imported ) {
-				$imported[ $form['form'] ] = $was_imported;
+				$imported[$form['form']] = $was_imported;
 			}
 		}
 

classes/helpers/FrmStylesHelper.php

Braces +2 added lines, -1 removed lines

@@ -336,7 +336,8 @@
 		if ( self::previewing_style() ) {
 
 			$frm_style = new FrmStyle();
-			if ( isset( $_POST['frm_style_setting'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Missing
+			if ( isset( $_POST['frm_style_setting'] ) ) {
+// phpcs:ignore WordPress.Security.NonceVerification.Missing
 
 				// Sanitizing is done later.
 				$posted = wp_unslash( $_POST['frm_style_setting'] ); //phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing

Spacing +20 added lines, -20 removed lines

@@ -162,7 +162,7 @@ 
 				'-' => 'down',
 				'+' => 'up',
 			);
-			$class = 'frm_arrow' . $arrow[ $icon ];
+			$class = 'frm_arrow' . $arrow[$icon];
 		} else {
 			// frm_minus1_icon.
 			$key   = str_replace( 'p', '', $key );
@@ -170,7 +170,7 @@ 
 				'-' => 'minus',
 				'+' => 'plus',
 			);
-			$class = 'frm_' . $plus[ $icon ];
+			$class = 'frm_' . $plus[$icon];
 		}
 
 		if ( $key ) {
@@ -196,8 +196,8 @@ 
 		?>
 		<div class="btn-group" id="frm_<?php echo esc_attr( $name ); ?>_select">
 			<button class="multiselect dropdown-toggle btn btn-default" data-toggle="dropdown" type="button">
-				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[ $name ], '+', $type ) ); ?>
-				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[ $name ], '-', $type ) ); ?>
+				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[$name], '+', $type ) ); ?>
+				<?php FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $style->post_content[$name], '-', $type ) ); ?>
 				<b class="caret"></b>
 			</button>
 			<ul class="multiselect-container frm-dropdown-menu">
@@ -205,7 +205,7 @@ 
 					<li <?php echo $style->post_content['collapse_icon'] == $key ? 'class="active"' : ''; // phpcs:ignore Universal.Operators.StrictComparisons ?>>
 						<a href="javascript:void(0);">
 							<label>
-								<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 ); ?> />
+								<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 ); ?> />
 								<span>
 									<?php
 									FrmAppHelper::icon_by_class( 'frmfont ' . self::icon_key_to_class( $key, '+', $type ) );
@@ -478,12 +478,12 @@ 
 		$vars = array_diff( $vars, self::get_style_keys_to_remove_from_output_vars() );
 
 		foreach ( $vars as $var ) {
-			if ( ! isset( $settings[ $var ] ) || ! self::css_key_is_valid( $var ) ) {
+			if ( ! isset( $settings[$var] ) || ! self::css_key_is_valid( $var ) ) {
 				continue;
 			}
 
-			if ( ! isset( $defaults[ $var ] ) ) {
-				$defaults[ $var ] = '';
+			if ( ! isset( $defaults[$var] ) ) {
+				$defaults[$var] = '';
 			}
 
 			$prepared_value = '';
@@ -544,7 +544,7 @@ 
 			return false;
 		}
 
-		if ( $defaults && $defaults[ $var ] === $prepared_value ) {
+		if ( $defaults && $defaults[$var] === $prepared_value ) {
 			return false;
 		}
 
@@ -623,7 +623,7 @@ 
 	 * @return string
 	 */
 	private static function css_var_prepare_value( $settings, $key ) {
-		$value = $settings[ $key ];
+		$value = $settings[$key];
 
 		if ( ! is_string( $value ) && ! is_numeric( $value ) ) {
 			return '';
@@ -713,8 +713,8 @@ 
 		$checkbox_opts = array( 'important_style', 'auto_width', 'submit_style', 'collapse_icon', 'center_form' );
 
 		foreach ( $checkbox_opts as $opt ) {
-			if ( ! isset( $settings[ $opt ] ) ) {
-				$settings[ $opt ] = 0;
+			if ( ! isset( $settings[$opt] ) ) {
+				$settings[$opt] = 0;
 			}
 		}
 
@@ -764,9 +764,9 @@ 
 		);
 
 		array_map(
-			function ( $key ) use ( $defaults, $font_size, $base_font_size, &$settings ) {
-				if ( isset( $settings[ $key ] ) ) {
-					$settings[ $key ] = round( self::get_base_font_size_scale( $key, $font_size, $defaults ) * $base_font_size ) . 'px';
+			function( $key ) use ( $defaults, $font_size, $base_font_size, &$settings ) {
+				if ( isset( $settings[$key] ) ) {
+					$settings[$key] = round( self::get_base_font_size_scale( $key, $font_size, $defaults ) * $base_font_size ) . 'px';
 				}
 			},
 			$font_sizes_to_update
@@ -787,11 +787,11 @@ 
 	 * @return float
 	 */
 	private static function get_base_font_size_scale( $key, $value, $defaults ) {
-		if ( empty( $defaults[ $key ] ) || ! is_numeric( (int) $defaults[ $key ] ) || ! is_numeric( (int) $value ) || 0 === (int) $value ) {
+		if ( empty( $defaults[$key] ) || ! is_numeric( (int) $defaults[$key] ) || ! is_numeric( (int) $value ) || 0 === (int) $value ) {
 			return 1;
 		}
 
-		return round( (int) $defaults[ $key ] / (int) $value, 2 );
+		return round( (int) $defaults[$key] / (int) $value, 2 );
 	}
 
 	/**
@@ -811,7 +811,7 @@ 
 			}
 
 			foreach ( $opts as $opt ) {
-				self::get_color_output( $css, $settings[ $opt ] );
+				self::get_color_output( $css, $settings[$opt] );
 			}
 		}
 	}
@@ -1039,13 +1039,13 @@ 
 	 */
 	private static function get_default_style_count( $style_id, $conversational_style_id ) {
 		$substrings = array_map(
-			function ( $value ) {
+			function( $value ) {
 				$substring = serialize( array( 'custom_style' => $value ) );
 				return substr( $substring, 5, -1 );
 			},
 			array( '1', 1 )
 		);
-		$where      = array(
+		$where = array(
 			'status' => 'published',
 			0        => array(
 				'options NOT LIKE' => 'custom_style',

classes/models/FrmStyleApi.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 		// Remove 'Styling Template' from titles.
 		foreach ( $api_info as $id => $template ) {
 			if ( isset( $template['name'] ) ) {
-				$api_info[ $id ]['name'] = preg_replace( '/(\sStyle|Styling)?(\sTemplate)?$/', '', $template['name'] );
+				$api_info[$id]['name'] = preg_replace( '/(\sStyle|Styling)?(\sTemplate)?$/', '', $template['name'] );
 			}
 		}
 

classes/models/FrmReviews.php

Spacing +4 added lines, -4 removed lines

@@ -80,12 +80,12 @@ 
 		$show_intervals = array( 50, 200, 500 );
 		$asked          = $this->review_status['asked'];
 
-		if ( ! isset( $show_intervals[ $asked ] ) ) {
+		if ( ! isset( $show_intervals[$asked] ) ) {
 			return;
 		}
 
 		$entries = FrmEntry::getRecordCount();
-		$count   = $show_intervals[ $asked ];
+		$count   = $show_intervals[$asked];
 		$user    = wp_get_current_user();
 
 		// Only show review request if the site has collected enough entries
@@ -134,7 +134,7 @@ 
 		$requests = $message->get_messages();
 		$key      = $this->inbox_key . ( $asked ? $asked : '' );
 
-		if ( isset( $requests[ $key ] ) ) {
+		if ( isset( $requests[$key] ) ) {
 			return;
 		}
 
@@ -173,7 +173,7 @@ 
 	 * @return bool
 	 */
 	private function has_later_request( $requests, $asked ) {
-		return isset( $requests[ $this->inbox_key . ( $asked + 1 ) ] ) || isset( $requests[ $this->inbox_key . ( $asked + 2 ) ] );
+		return isset( $requests[$this->inbox_key . ( $asked + 1 )] ) || isset( $requests[$this->inbox_key . ( $asked + 2 )] );
 	}
 
 	/**