Strategy11 / formidable-forms

classes/controllers/FrmSettingsController.php

Spacing +3 added lines, -3 removed lines

@@ -115,7 +115,7 @@ 
 				$section['function'] = $original;
 			}
 
-			$sections[ $key ] = $section;
+			$sections[$key] = $section;
 		}
 
 		return $sections;
@@ -127,11 +127,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/controllers/FrmFormActionsController.php

Spacing +17 added lines, -17 removed lines

@@ -98,19 +98,19 @@ 
 		}
 
 		foreach ( $action_controls as $action ) {
-			if ( isset( $groups[ $action->id_base ] ) || in_array( $action->id_base, $grouped ) ) {
+			if ( isset( $groups[$action->id_base] ) || in_array( $action->id_base, $grouped ) ) {
 				continue;
 			}
 
 			$this_group = $action->action_options['group'];
-			if ( ! isset( $groups[ $this_group ] ) ) {
+			if ( ! isset( $groups[$this_group] ) ) {
 				$this_group = 'misc';
 			}
 
-			if ( ! isset( $groups[ $this_group ]['actions'] ) ) {
-				$groups[ $this_group ]['actions'] = array();
+			if ( ! isset( $groups[$this_group]['actions'] ) ) {
+				$groups[$this_group]['actions'] = array();
 			}
-			$groups[ $this_group ]['actions'][] = $action->id_base;
+			$groups[$this_group]['actions'][] = $action->id_base;
 
 			unset( $action );
 		}
@@ -241,7 +241,7 @@ 
 				return $a;
 			}
 
-			$actions[ $a->id_base ] = $a;
+			$actions[$a->id_base] = $a;
 		}
 
 		return $actions;
@@ -272,16 +272,16 @@ 
 		$action_map = array();
 
 		foreach ( $action_controls as $key => $control ) {
-			$action_map[ $control->id_base ] = $key;
+			$action_map[$control->id_base] = $key;
 		}
 
 		foreach ( $form_actions as $action ) {
-			if ( ! isset( $action_map[ $action->post_excerpt ] ) ) {
+			if ( ! isset( $action_map[$action->post_excerpt] ) ) {
 				// don't try and show settings if action no longer exists
 				continue;
 			}
 
-			self::action_control( $action, $form, $action->ID, $action_controls[ $action_map[ $action->post_excerpt ] ], $values );
+			self::action_control( $action, $form, $action->ID, $action_controls[$action_map[$action->post_excerpt]], $values );
 		}
 	}
 
@@ -489,8 +489,8 @@ 
 			}
 
 			// Store actions so they can be triggered with the correct priority.
-			$stored_actions[ $action->ID ]  = $action;
-			$action_priority[ $action->ID ] = $link_settings[ $action->post_excerpt ]->action_options['priority'];
+			$stored_actions[$action->ID]  = $action;
+			$action_priority[$action->ID] = $link_settings[$action->post_excerpt]->action_options['priority'];
 
 			unset( $action );
 		}
@@ -502,7 +502,7 @@ 
 			new FrmNotification();
 
 			foreach ( $action_priority as $action_id => $priority ) {
-				$action = $stored_actions[ $action_id ];
+				$action = $stored_actions[$action_id];
 				do_action( 'frm_trigger_' . $action->post_excerpt . '_action', $action, $entry, $form, $event );
 				do_action( 'frm_trigger_' . $action->post_excerpt . '_' . $event . '_action', $action, $entry, $form );
 
@@ -549,12 +549,12 @@ 
 	}
 
 	public function register( $action_class ) {
-		$this->actions[ $action_class ] = new $action_class();
+		$this->actions[$action_class] = new $action_class();
 	}
 
 	public function unregister( $action_class ) {
-		if ( isset( $this->actions[ $action_class ] ) ) {
-			unset( $this->actions[ $action_class ] );
+		if ( isset( $this->actions[$action_class] ) ) {
+			unset( $this->actions[$action_class] );
 		}
 	}
 
@@ -563,8 +563,8 @@ 
 
 		foreach ( $keys as $key ) {
 			// don't register new action if old action with the same id is already registered
-			if ( ! isset( $this->actions[ $key ] ) ) {
-				$this->actions[ $key ]->_register();
+			if ( ! isset( $this->actions[$key] ) ) {
+				$this->actions[$key]->_register();
 			}
 		}
 	}

classes/models/FrmUsage.php

Spacing +7 added lines, -7 removed lines

@@ -139,7 +139,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 );
 			}
 		}
 
@@ -171,7 +171,7 @@ 
 
 		$message_settings = array();
 		foreach ( $messages as $message ) {
-			$message_settings[ $message ] = $settings_list->$message;
+			$message_settings[$message] = $settings_list->$message;
 		}
 
 		return $message_settings;
@@ -189,7 +189,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;
 			}
 		}
 
@@ -256,8 +256,8 @@ 
 			);
 
 			foreach ( $settings as $setting ) {
-				if ( isset( $form->options[ $setting ] ) ) {
-					$new_form[ $setting ] = $this->maybe_json( $form->options[ $setting ] );
+				if ( isset( $form->options[$setting] ) ) {
+					$new_form[$setting] = $this->maybe_json( $form->options[$setting] );
 				}
 			}
 
@@ -308,7 +308,7 @@ 
 	 * @return array
 	 */
 	private function fields() {
-		$args   = array(
+		$args = array(
 			'limit'    => 50,
 			'order_by' => 'id DESC',
 		);
@@ -316,7 +316,7 @@ 
 		$fields = FrmDb::get_results( 'frm_fields', array(), 'form_id, name, type, field_options', $args );
 		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/FrmMigrate.php

Spacing +3 added lines, -3 removed lines

@@ -329,7 +329,7 @@ 
 
 		foreach ( (array) $fields as $f ) {
 			FrmAppHelper::unserialize_or_decode( $f->field_options );
-			$size             = $f->field_options['size'];
+			$size = $f->field_options['size'];
 			$this->maybe_convert_migrated_size( $size );
 
 			if ( $size === $f->field_options['size'] ) {
@@ -392,7 +392,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 );
 	}
@@ -491,7 +491,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/FrmEntryMeta.php

Spacing +10 added lines, -10 removed lines

@@ -54,7 +54,7 @@ 
 
 		global $wpdb;
 
-		$values               = array(
+		$values = array(
 			'item_id'  => $entry_id,
 			'field_id' => $field_id,
 		);
@@ -128,7 +128,7 @@ 
 
 				if ( ( is_array( $meta_value ) && empty( $meta_value ) ) || ( ! is_array( $meta_value ) && trim( $meta_value ) == '' ) ) {
 					// remove blank fields
-					unset( $values[ $field_id ] );
+					unset( $values[$field_id] );
 				} else {
 					// if value exists, then update it
 					self::update_entry_meta( $entry_id, $field_id, '', $meta_value );
@@ -193,7 +193,7 @@ 
 	 */
 	public static function get_meta_value( $entry, $field_id ) {
 		if ( isset( $entry->metas ) ) {
-			return isset( $entry->metas[ $field_id ] ) ? $entry->metas[ $field_id ] : false;
+			return isset( $entry->metas[$field_id] ) ? $entry->metas[$field_id] : false;
 		} else {
 			return self::get_entry_meta_by_field( $entry->id, $field_id );
 		}
@@ -211,8 +211,8 @@ 
 			$cached   = FrmDb::check_cache( $entry_id, 'frm_entry' );
 		}
 
-		if ( $cached && isset( $cached->metas ) && isset( $cached->metas[ $field_id ] ) ) {
-			$result = $cached->metas[ $field_id ];
+		if ( $cached && isset( $cached->metas ) && isset( $cached->metas[$field_id] ) ) {
+			$result = $cached->metas[$field_id];
 
 			return wp_unslash( $result );
 		}
@@ -222,7 +222,7 @@ 
 		if ( is_numeric( $field_id ) ) {
 			$query['field_id'] = $field_id;
 		} else {
-			$get_table             .= ' it LEFT OUTER JOIN ' . $wpdb->prefix . 'frm_fields fi ON it.field_id=fi.id';
+			$get_table .= ' it LEFT OUTER JOIN ' . $wpdb->prefix . 'frm_fields fi ON it.field_id=fi.id';
 			$query['fi.field_key'] = $field_id;
 		}
 
@@ -240,7 +240,7 @@ 
 			'stripslashes' => true,
 			'is_draft'     => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args( $args, $defaults );
 
 		$query = array();
 		self::meta_field_query( $field_id, $order, $limit, $args, $query );
@@ -255,7 +255,7 @@ 
 
 		foreach ( $values as $k => $v ) {
 			FrmAppHelper::unserialize_or_decode( $v );
-			$values[ $k ] = $v;
+			$values[$k] = $v;
 			unset( $k, $v );
 		}
 
@@ -312,7 +312,7 @@ 
 
 		foreach ( $results as $k => $result ) {
 			FrmAppHelper::unserialize_or_decode( $result->meta_value );
-			$results[ $k ]->meta_value = wp_unslash( $result->meta_value );
+			$results[$k]->meta_value = wp_unslash( $result->meta_value );
 			unset( $k, $result );
 		}
 
@@ -325,7 +325,7 @@ 
 			'user_id'  => '',
 			'group_by' => '',
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args( $args, $defaults );
 
 		$query = array();
 		self::get_ids_query( $where, $order_by, $limit, $unique, $args, $query );

classes/models/FrmEmail.php

Spacing +6 added lines, -6 removed lines

@@ -85,7 +85,7 @@ 
 		$to = $this->prepare_email_setting( $this->settings['email_to'], $user_id_args );
 		$to = $this->explode_emails( $to );
 
-		$where  = array(
+		$where = array(
 			'it.field_id !' => 0,
 			'it.item_id'    => $this->entry->id,
 		);
@@ -95,7 +95,7 @@ 
 			'entry'     => $this->entry,
 			'form'      => $this->form,
 		);
-		$to     = apply_filters( 'frm_to_email', $to, $values, $this->form->id, $args );
+		$to = apply_filters( 'frm_to_email', $to, $values, $this->form->id, $args );
 
 		$this->to = array_unique( (array) $to );
 
@@ -298,7 +298,7 @@ 
 
 		// Add the user info if it isn't already included
 		if ( $this->include_user_info && $prev_mail_body === $mail_body ) {
-			$data      = $this->entry->description;
+			$data = $this->entry->description;
 			$mail_body .= "\r\n\r\n" . __( 'User Information', 'formidable' ) . "\r\n";
 			$this->maybe_add_ip( $mail_body );
 			$mail_body .= __( 'User-Agent (Browser/OS)', 'formidable' ) . ': ' . FrmEntriesHelper::get_browser( $data['browser'] ) . "\r\n";
@@ -563,12 +563,12 @@ 
 					$name = trim( str_replace( $email, '', $val ) );
 				} else {
 					// If user enters a name without an email
-					unset( $recipients[ $key ] );
+					unset( $recipients[$key] );
 					continue;
 				}
 			}
 
-			$recipients[ $key ] = $this->format_from_email( $name, $email );
+			$recipients[$key] = $this->format_from_email( $name, $email );
 		}
 
 		return $recipients;
@@ -709,7 +709,7 @@ 
 				);
 
 				// Remove phone number from to addresses
-				unset( $this->to[ $key ] );
+				unset( $this->to[$key] );
 			}
 		}
 	}

classes/views/frm-forms/add_field_links.php

Spacing +3 added lines, -3 removed lines

@@ -49,10 +49,10 @@
 foreach ( $pro_fields as $field_key => $field_type ) {
 
 	if ( isset( $field_type['section'] ) ) {
-		if ( ! isset( $field_sections[ $field_type['section'] ] ) ) {
-			$field_sections[ $field_type['section'] ] = array();
+		if ( ! isset( $field_sections[$field_type['section']] ) ) {
+			$field_sections[$field_type['section']] = array();
 		}
-		$field_sections[ $field_type['section'] ][ $field_key ] = $field_type;
+		$field_sections[$field_type['section']][$field_key] = $field_type;
 		continue;
 	}
 

classes/models/FrmFormMigrator.php

Spacing +6 added lines, -6 removed lines

@@ -396,7 +396,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 );
 		}
 	}
 
@@ -426,9 +426,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;
 			}
 		}
 
@@ -447,7 +447,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 );
 	}
@@ -467,11 +467,11 @@ 
 	private function is_imported( $source_id ) {
 		$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.
 			$new_form_id = 0;

classes/controllers/FrmSMTPController.php

Spacing +1 added lines, -1 removed lines

@@ -235,7 +235,7 @@
 		if ( is_array( $upsell ) ) {
 			foreach ( $upsell as $k => $plugin ) {
 				if ( strpos( $plugin['slug'], 'wpforms' ) !== false ) {
-					unset( $upsell[ $k ] );
+					unset( $upsell[$k] );
 				}
 			}
 		}