Strategy11 / formidable-forms

classes/helpers/FrmStylesCardHelper.php

Spacing +10 added lines, -10 removed lines

@@ -145,11 +145,11 @@ 
 	private static function has_dark_background( $style ) {
 		$key = 'fieldset_bg_color';
 
-		if ( empty( $style->post_content[ $key ] ) ) {
+		if ( empty( $style->post_content[$key] ) ) {
 			return false;
 		}
 
-		$color = $style->post_content[ $key ];
+		$color = $style->post_content[$key];
 
 		if ( 0 === strpos( $color, 'rgba' ) ) {
 			preg_match_all( '/([\\d.]+)/', $color, $matches );
@@ -213,7 +213,7 @@ 
 			 * @param stdClass $style_object
 			 * @param array    $style
 			 */
-			$param_filter = function ( $params, $args ) use ( $style_object, $style ) {
+			$param_filter = function( $params, $args ) use ( $style_object, $style ) {
 				if ( $args['style'] !== $style_object ) {
 					return $params;
 				}
@@ -226,7 +226,7 @@ 
 					),
 					'/style-templates/' . $style['slug']
 				);
-				$params['data-requires']    = FrmFormsHelper::get_plan_required( $style );
+				$params['data-requires'] = FrmFormsHelper::get_plan_required( $style );
 				return $params;
 			};
 		} else {
@@ -238,7 +238,7 @@ 
 			 *
 			 * @return array
 			 */
-			$param_filter = function ( $params ) use ( $style ) {
+			$param_filter = function( $params ) use ( $style ) {
 				$params['data-template-key'] = $style['slug'];
 				return $params;
 			};
@@ -291,7 +291,7 @@ 
 				continue;
 			}
 
-			$value = $style->post_content[ $key ];
+			$value = $style->post_content[$key];
 
 			$is_hex = in_array( $key, $color_settings, true ) && $value && '#' !== $value[0] && false === strpos( $value, 'rgb' ) && $value !== 'transparent';
 
@@ -414,7 +414,7 @@ 
 			 *
 			 * @return void
 			 */
-			function ( $style, $key ) {
+			function( $style, $key ) {
 				if ( ! is_numeric( $key ) ) {
 					// Skip active_sub/expires keys.
 					return;
@@ -443,10 +443,10 @@ 
 			 *
 			 * @return void
 			 */
-			function ( $style ) use ( &$count ) {
+			function( $style ) use ( &$count ) {
 				$hidden = $count > self::PAGE_SIZE - 1;
 				$this->echo_style_card( $style, $hidden );
-				++$count;
+				++ $count;
 			}
 		);
 
@@ -507,7 +507,7 @@ 
 			 *
 			 * @return bool
 			 */
-			function ( $style ) {
+			function( $style ) {
 				return $this->default_style->ID !== $style->ID;
 			}
 		);

classes/helpers/FrmListHelper.php

Spacing +15 added lines, -15 removed lines

@@ -246,8 +246,8 @@ 
 			return $this->get_pagenum();
 		}
 
-		if ( isset( $this->_pagination_args[ $key ] ) ) {
-			return $this->_pagination_args[ $key ];
+		if ( isset( $this->_pagination_args[$key] ) ) {
+			return $this->_pagination_args[$key];
 		}
 	}
 
@@ -297,8 +297,8 @@ 
 	 * @return void
 	 */
 	private function hidden_search_inputs( $param_name ) {
-		if ( ! empty( $_REQUEST[ $param_name ] ) ) {
-			$value = sanitize_text_field( wp_unslash( $_REQUEST[ $param_name ] ) );
+		if ( ! empty( $_REQUEST[$param_name] ) ) {
+			$value = sanitize_text_field( wp_unslash( $_REQUEST[$param_name] ) );
 			echo '<input type="hidden" name="' . esc_attr( $param_name ) . '" value="' . esc_attr( $value ) . '" />';
 		}
 	}
@@ -341,7 +341,7 @@ 
 		echo "<ul class='subsubsub'>\n";
 
 		foreach ( $views as $class => $view ) {
-			$views[ $class ] = "\t" . '<li class="' . esc_attr( $class ) . '">' . $view;
+			$views[$class] = "\t" . '<li class="' . esc_attr( $class ) . '">' . $view;
 		}
 		echo implode( " |</li>\n", $views ) . "</li>\n"; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 		echo '</ul>';
@@ -511,7 +511,7 @@ 
 		$out = '<div class="' . ( $always_visible ? 'row-actions visible' : 'row-actions' ) . '">';
 
 		foreach ( $actions as $action => $link ) {
-			++$i;
+			++ $i;
 			$sep  = $i === $action_count ? '' : ' | ';
 			$out .= "<span class='$action'>$link$sep</span>";
 		}
@@ -751,7 +751,7 @@ 
 			'next'  => __( 'Next page', 'formidable' ),
 		);
 
-		return $labels[ $link ];
+		return $labels[$link];
 	}
 
 	private function current_url() {
@@ -856,7 +856,7 @@ 
 
 		// If the primary column doesn't exist fall back to the
 		// first non-checkbox column.
-		if ( ! isset( $columns[ $default ] ) ) {
+		if ( ! isset( $columns[$default] ) ) {
 			$default = self::get_default_primary_column_name();
 		}
 
@@ -870,7 +870,7 @@ 
 		 */
 		$column = apply_filters( 'list_table_primary_column', $default, $this->screen->id );
 
-		if ( empty( $column ) || ! isset( $columns[ $column ] ) ) {
+		if ( empty( $column ) || ! isset( $columns[$column] ) ) {
 			$column = $default;
 		}
 
@@ -892,7 +892,7 @@ 
 			$column_headers = array( array(), array(), array(), $this->get_primary_column_name() );
 
 			foreach ( $this->_column_headers as $key => $value ) {
-				$column_headers[ $key ] = $value;
+				$column_headers[$key] = $value;
 			}
 
 			return $column_headers;
@@ -927,7 +927,7 @@ 
 				$data[1] = false;
 			}
 
-			$sortable[ $id ] = $data;
+			$sortable[$id] = $data;
 		}
 
 		$primary = $this->get_primary_column_name();
@@ -986,7 +986,7 @@ 
 			static $cb_counter = 1;
 			$columns['cb']     = '<label class="screen-reader-text" for="cb-select-all-' . $cb_counter . '">' . __( 'Select All', 'formidable' ) . '</label>';
 			$columns['cb']    .= '<input id="cb-select-all-' . esc_attr( $cb_counter ) . '" type="checkbox" />';
-			++$cb_counter;
+			++ $cb_counter;
 		}
 
 		foreach ( $columns as $column_key => $column_display_name ) {
@@ -1008,8 +1008,8 @@ 
 				$class[] = 'column-primary';
 			}
 
-			if ( isset( $sortable[ $column_key ] ) ) {
-				list( $orderby, $desc_first ) = $sortable[ $column_key ];
+			if ( isset( $sortable[$column_key] ) ) {
+				list( $orderby, $desc_first ) = $sortable[$column_key];
 
 				if ( $current_orderby == $orderby ) {
 					// The sorted column. The `aria-sort` attribute must be set only on the sorted column.
@@ -1129,7 +1129,7 @@ 
 	 * @return bool Returns true if the setting is not set or if it is not false; otherwise, returns false.
 	 */
 	protected function should_display( $args, $settings ) {
-		return ! isset( $args[ $settings ] ) || false !== $args[ $settings ];
+		return ! isset( $args[$settings] ) || false !== $args[$settings];
 	}
 
 	/**

classes/helpers/FrmAddonsHelper.php

Spacing +2 added lines, -2 removed lines

@@ -178,7 +178,7 @@ 
 			'convertkit'                      => 'convertkit',
 		);
 
-		$icon = array_key_exists( $slug, $icons_map ) ? 'frm_' . $icons_map[ $slug ] . '_icon' : 'frm_logo_icon';
+		$icon = array_key_exists( $slug, $icons_map ) ? 'frm_' . $icons_map[$slug] . '_icon' : 'frm_logo_icon';
 
 		if ( 'ai' === $slug ) {
 			$icon = str_replace( '_', '-', $icon );
@@ -291,7 +291,7 @@ 
 		?>
 		<span style="color: <?php echo esc_attr( $color ); ?>;">
 			<?php
-			for ( $i = 0; $i < 5; $i++ ) {
+			for ( $i = 0; $i < 5; $i ++ ) {
 				echo $icon; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 			}
 			?>

classes/helpers/FrmOnSubmitHelper.php

Spacing +11 added lines, -11 removed lines

@@ -247,17 +247,17 @@ 
 		$opt = 'update' === $event ? 'edit_' : 'success_';
 
 		if ( ! $action || ! is_object( $action ) ) {
-			$form_options[ $opt . 'action' ] = self::get_default_action_type();
-			$form_options[ $opt . 'msg' ]    = self::get_default_msg();
+			$form_options[$opt . 'action'] = self::get_default_action_type();
+			$form_options[$opt . 'msg']    = self::get_default_msg();
 
 			return;
 		}
 
-		$form_options[ $opt . 'action' ] = $action->post_content['success_action'] ?? 'message';
+		$form_options[$opt . 'action'] = $action->post_content['success_action'] ?? 'message';
 
-		switch ( $form_options[ $opt . 'action' ] ) {
+		switch ( $form_options[$opt . 'action'] ) {
 			case 'redirect':
-				$form_options[ $opt . 'url' ]        = $action->post_content['success_url'] ?? '';
+				$form_options[$opt . 'url']        = $action->post_content['success_url'] ?? '';
 				$form_options['open_in_new_tab']     = ! empty( $action->post_content['open_in_new_tab'] );
 				$form_options['redirect_delay']      = ! empty( $action->post_content['redirect_delay'] );
 				$form_options['redirect_delay_time'] = $action->post_content['redirect_delay_time'];
@@ -265,11 +265,11 @@ 
 				break;
 
 			case 'page':
-				$form_options[ $opt . 'page_id' ] = $action->post_content['success_page_id'] ?? '';
+				$form_options[$opt . 'page_id'] = $action->post_content['success_page_id'] ?? '';
 				break;
 
 			default:
-				$form_options[ $opt . 'msg' ] = ! empty( $action->post_content['success_msg'] ) ? $action->post_content['success_msg'] : self::get_default_msg();
+				$form_options[$opt . 'msg'] = ! empty( $action->post_content['success_msg'] ) ? $action->post_content['success_msg'] : self::get_default_msg();
 				$form_options['show_form']    = ! empty( $action->post_content['show_form'] );
 		}
 	}
@@ -363,20 +363,20 @@ 
 	private static function get_on_submit_action_data_from_form_options( $form_options, $event = 'create' ) {
 		$opt  = 'update' === $event ? 'edit_' : 'success_';
 		$data = array(
-			'success_action' => $form_options[ $opt . 'action' ] ?? self::get_default_action_type(),
+			'success_action' => $form_options[$opt . 'action'] ?? self::get_default_action_type(),
 		);
 
 		switch ( $data['success_action'] ) {
 			case 'redirect':
-				$data['success_url'] = $form_options[ $opt . 'url' ] ?? '';
+				$data['success_url'] = $form_options[$opt . 'url'] ?? '';
 				break;
 
 			case 'page':
-				$data['success_page_id'] = $form_options[ $opt . 'page_id' ] ?? '';
+				$data['success_page_id'] = $form_options[$opt . 'page_id'] ?? '';
 				break;
 
 			default:
-				$data['success_msg'] = $form_options[ $opt . 'msg' ] ?? self::get_default_msg();
+				$data['success_msg'] = $form_options[$opt . 'msg'] ?? self::get_default_msg();
 				$data['show_form']   = ! empty( $form_options['show_form'] );
 		}
 

classes/helpers/FrmFieldGridHelper.php

Spacing +1 added lines, -1 removed lines

@@ -230,7 +230,7 @@
 		}
 
 		if ( false !== $this->parent_li ) {
-			++$this->current_field_count;
+			++ $this->current_field_count;
 			$this->current_list_size += $this->active_field_size;
 
 			if ( 12 === $this->current_list_size ) {

classes/helpers/FrmStringReaderHelper.php

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
 	public function read_until( $char ) {
 		$value = '';
 
-		while ( $this->pos <= $this->max && ( $one = $this->string[ $this->pos++ ] ) !== $char ) {
+		while ( $this->pos <= $this->max && ( $one = $this->string[$this->pos ++] ) !== $char ) {
 			$value .= $one;
 		}
 		return $value;
@@ -70,10 +70,10 @@ 
 	public function read( $count ) {
 		$value = '';
 
-		while ( $count > 0 && $this->pos <= $this->max && ( ( $one = $this->string[ $this->pos ] ) || '0' === $one ) ) {
+		while ( $count > 0 && $this->pos <= $this->max && ( ( $one = $this->string[$this->pos] ) || '0' === $one ) ) {
 			$value     .= $one;
 			$this->pos += 1;
-			--$count;
+			-- $count;
 		}
 
 		/**

stripe/models/FrmStrpLiteAuth.php

Spacing +15 added lines, -15 removed lines

@@ -61,7 +61,7 @@ 
 
 		if ( $intent_is_processing ) {
 			// Append an additional processing message to the end of the success message.
-			$filter = function ( $message ) {
+			$filter = function( $message ) {
 				$stripe_settings = FrmStrpLiteAppHelper::get_settings();
 				$message        .= '<p>' . esc_html( $stripe_settings->settings->processing_message ) . '</p>';
 				return $message;
@@ -171,7 +171,7 @@ 
 		$atts['form']        = FrmForm::getOne( $atts['entry']->form_id );
 		$atts['entry_id']    = $atts['entry']->id;
 		$opt                 = 'success_action';
-		$atts['conf_method'] = ! empty( $atts['form']->options[ $opt ] ) ? $atts['form']->options[ $opt ] : 'message';
+		$atts['conf_method'] = ! empty( $atts['form']->options[$opt] ) ? $atts['form']->options[$opt] : 'message';
 
 		$actions = FrmFormsController::get_met_on_submit_actions( $atts, 'create' );
 
@@ -269,11 +269,11 @@ 
 	 */
 	public static function get_payment_intents( $name ) {
 		// phpcs:ignore WordPress.Security.NonceVerification.Missing
-		if ( ! isset( $_POST[ $name ] ) ) {
+		if ( ! isset( $_POST[$name] ) ) {
 			return array();
 		}
 
-		$intents = $_POST[ $name ]; // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
+		$intents = $_POST[$name]; // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
 		FrmAppHelper::sanitize_value( 'sanitize_text_field', $intents );
 		return $intents;
 	}
@@ -301,7 +301,7 @@ 
 		self::format_form_data( $form );
 
 		$form_id = absint( $form['form_id'] );
-		$intents = $form[ 'frmintent' . $form_id ] ?? array();
+		$intents = $form['frmintent' . $form_id] ?? array();
 
 		if ( empty( $intents ) ) {
 			wp_die();
@@ -311,8 +311,8 @@ 
 			$intents = array( $intents );
 		} else {
 			foreach ( $intents as $k => $intent ) {
-				if ( is_array( $intent ) && isset( $intent[ $k ] ) ) {
-					$intents[ $k ] = $intent[ $k ];
+				if ( is_array( $intent ) && isset( $intent[$k] ) ) {
+					$intents[$k] = $intent[$k];
 				}
 			}
 		}
@@ -374,7 +374,7 @@ 
 				if ( $saved->metadata->action != $action->ID ) {
 					continue;
 				}
-				$intents[ $k ] = array(
+				$intents[$k] = array(
 					'id'     => $intent,
 					'action' => $action->ID,
 				);
@@ -421,7 +421,7 @@ 
 			if ( $k === 'item_meta' ) {
 				foreach ( $v as $f => $value ) {
 					FrmAppHelper::sanitize_value( 'wp_kses_post', $value );
-					$entry->metas[ absint( $f ) ] = $value;
+					$entry->metas[absint( $f )] = $value;
 				}
 			} else {
 				FrmAppHelper::sanitize_value( 'wp_kses_post', $v );
@@ -447,14 +447,14 @@ 
 		foreach ( $form as $input ) {
 			$key = $input['name'];
 
-			if ( isset( $formatted[ $key ] ) ) {
-				if ( is_array( $formatted[ $key ] ) ) {
-					$formatted[ $key ][] = $input['value'];
+			if ( isset( $formatted[$key] ) ) {
+				if ( is_array( $formatted[$key] ) ) {
+					$formatted[$key][] = $input['value'];
 				} else {
-					$formatted[ $key ] = array( $formatted[ $key ], $input['value'] );
+					$formatted[$key] = array( $formatted[$key], $input['value'] );
 				}
 			} else {
-				$formatted[ $key ] = $input['value'];
+				$formatted[$key] = $input['value'];
 			}
 		}
 
@@ -706,7 +706,7 @@ 
 
 		foreach ( $actions as $k => $action ) {
 			$amount                                = self::get_amount_before_submit( compact( 'action', 'form' ) );
-			$actions[ $k ]->post_content['amount'] = $amount;
+			$actions[$k]->post_content['amount'] = $amount;
 		}
 	}
 

stripe/models/FrmTransLiteDb.php

Spacing +4 added lines, -4 removed lines

@@ -295,14 +295,14 @@
 		$defaults = $this->get_defaults();
 
 		foreach ( $defaults as $val => $default ) {
-			if ( isset( $values[ $val ] ) ) {
+			if ( isset( $values[$val] ) ) {
 				if ( $default['sanitize'] === 'float' ) {
-					$new_values[ $val ] = (float) $values[ $val ];
+					$new_values[$val] = (float) $values[$val];
 				} elseif ( ! empty( $default['sanitize'] ) ) {
-					$new_values[ $val ] = call_user_func( $default['sanitize'], $values[ $val ] );
+					$new_values[$val] = call_user_func( $default['sanitize'], $values[$val] );
 				}
 			} elseif ( $values['action'] === 'create' ) {
-				$new_values[ $val ] = $default['default'];
+				$new_values[$val] = $default['default'];
 			}
 		}
 	}

stripe/controllers/FrmStrpLiteActionsController.php

Spacing +9 added lines, -9 removed lines

@@ -78,7 +78,7 @@ 
 			$is_stripe = $gateway === 'stripe' || ( is_array( $gateway ) && in_array( 'stripe', $gateway, true ) );
 
 			if ( ! $is_stripe || empty( $payment_action->post_content['amount'] ) ) {
-				unset( $payment_actions[ $k ] );
+				unset( $payment_actions[$k] );
 			}
 		}
 		return $payment_actions;
@@ -117,7 +117,7 @@ 
 			'run_triggers' => false,
 			'show_errors'  => true,
 		);
-		$atts     = compact( 'action', 'entry', 'form' );
+		$atts = compact( 'action', 'entry', 'form' );
 
 		$amount = self::prepare_amount( $action->post_content['amount'], $atts );
 
@@ -317,7 +317,7 @@ 
 
 		// Gateway is a radio button but it should always be an array in the database for
 		// compatibility with the payments submodule where it is a checkbox.
-		$settings['gateway']  = ! empty( $settings['gateway'] ) ? (array) $settings['gateway'] : array( 'stripe' );
+		$settings['gateway'] = ! empty( $settings['gateway'] ) ? (array) $settings['gateway'] : array( 'stripe' );
 
 		$is_stripe = in_array( 'stripe', $settings['gateway'], true );
 
@@ -523,7 +523,7 @@ 
 
 		foreach ( $settings as $k => $s ) {
 			if ( is_string( $s ) ) {
-				$settings[ $k ] = str_replace( $disallowed, '', $s );
+				$settings[$k] = str_replace( $disallowed, '', $s );
 			}
 		}
 
@@ -652,7 +652,7 @@ 
 	 * @return array
 	 */
 	public static function remove_cc_validation( $errors, $field, $values ) {
-		$has_processed = isset( $_POST[ 'frmintent' . $field->form_id ] ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
+		$has_processed = isset( $_POST['frmintent' . $field->form_id] ); // phpcs:ignore WordPress.Security.NonceVerification.Missing
 
 		if ( ! $has_processed ) {
 			return $errors;
@@ -660,12 +660,12 @@ 
 
 		$field_id = $field->temp_id ?? $field->id;
 
-		if ( isset( $errors[ 'field' . $field_id . '-cc' ] ) ) {
-			unset( $errors[ 'field' . $field_id . '-cc' ] );
+		if ( isset( $errors['field' . $field_id . '-cc'] ) ) {
+			unset( $errors['field' . $field_id . '-cc'] );
 		}
 
-		if ( isset( $errors[ 'field' . $field_id ] ) ) {
-			unset( $errors[ 'field' . $field_id ] );
+		if ( isset( $errors['field' . $field_id] ) ) {
+			unset( $errors['field' . $field_id] );
 		}
 
 		return $errors;