Strategy11 / formidable-forms

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/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 ) ) {

classes/models/FrmFormState.php

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
 	 * @return void
 	 */
 	public function set( $key, $value ) {
-		$this->state[ $key ] = $value;
+		$this->state[$key] = $value;
 	}
 
 	/**
@@ -84,7 +84,7 @@ 
 	 * @return mixed
 	 */
 	public function get( $key, $default ) {
-		return $this->state[ $key ] ?? $default;
+		return $this->state[$key] ?? $default;
 	}
 
 	/**
@@ -207,7 +207,7 @@ 
 		$compressed = array();
 
 		foreach ( $this->state as $key => $value ) {
-			$compressed[ self::compressed_key( $key ) ] = $value;
+			$compressed[self::compressed_key( $key )] = $value;
 		}
 		return $compressed;
 	}

classes/models/fields/FrmFieldCaptcha.php

Spacing +6 added lines, -6 removed lines

@@ -310,8 +310,8 @@ 
 
 		if ( is_wp_error( $resp ) ) {
 			$error_string                     = $resp->get_error_message();
-			$errors[ 'field' . $args['id'] ]  = __( 'There was a problem verifying your captcha', 'formidable' );
-			$errors[ 'field' . $args['id'] ] .= ' ' . $error_string;
+			$errors['field' . $args['id']]  = __( 'There was a problem verifying your captcha', 'formidable' );
+			$errors['field' . $args['id']] .= ' ' . $error_string;
 			return $errors;
 		}
 
@@ -339,7 +339,7 @@ 
 			if ( $invalid_message === __( 'The reCAPTCHA was not entered correctly', 'formidable' ) ) {
 				$invalid_message = '';
 			}
-			$errors[ 'field' . $args['id'] ] = ( $invalid_message === '' ? $frm_settings->re_msg : $invalid_message );
+			$errors['field' . $args['id']] = ( $invalid_message === '' ? $frm_settings->re_msg : $invalid_message );
 		}
 
 		return $errors;
@@ -359,8 +359,8 @@ 
 
 		$form_id = is_object( $this->field ) ? $this->field->form_id : $this->field['form_id'];
 
-		if ( ! isset( $frm_vars['captcha_scores'][ $form_id ] ) ) {
-			$frm_vars['captcha_scores'][ $form_id ] = $score;
+		if ( ! isset( $frm_vars['captcha_scores'][$form_id] ) ) {
+			$frm_vars['captcha_scores'][$form_id] = $score;
 		}
 	}
 
@@ -391,7 +391,7 @@ 
 	protected static function post_data_includes_token() {
 		$settings = FrmCaptchaFactory::get_settings_object();
 		// phpcs:ignore WordPress.Security.NonceVerification.Missing
-		return ! empty( $_POST[ $settings->token_field ] );
+		return ! empty( $_POST[$settings->token_field] );
 	}
 
 	/**

classes/models/fields/FrmFieldName.php

Spacing +13 added lines, -13 removed lines

@@ -68,19 +68,19 @@ 
 		$result = array();
 
 		foreach ( $names as $name ) {
-			if ( empty( $this->sub_fields[ $name ] ) ) {
+			if ( empty( $this->sub_fields[$name] ) ) {
 				continue;
 			}
 
-			if ( ! isset( $this->sub_fields[ $name ]['wrapper_classes'] ) ) {
-				$this->sub_fields[ $name ]['wrapper_classes'] = $col_class;
-			} elseif ( is_array( $this->sub_fields[ $name ]['wrapper_classes'] ) ) {
-				$this->sub_fields[ $name ]['wrapper_classes'] = implode( ' ', $this->sub_fields[ $name ]['wrapper_classes'] ) . ' ' . $col_class;
+			if ( ! isset( $this->sub_fields[$name]['wrapper_classes'] ) ) {
+				$this->sub_fields[$name]['wrapper_classes'] = $col_class;
+			} elseif ( is_array( $this->sub_fields[$name]['wrapper_classes'] ) ) {
+				$this->sub_fields[$name]['wrapper_classes'] = implode( ' ', $this->sub_fields[$name]['wrapper_classes'] ) . ' ' . $col_class;
 			} else {
-				$this->sub_fields[ $name ]['wrapper_classes'] .= ' ' . $col_class;
+				$this->sub_fields[$name]['wrapper_classes'] .= ' ' . $col_class;
 			}
 
-			$result[ $name ] = $this->sub_fields[ $name ];
+			$result[$name] = $this->sub_fields[$name];
 		}
 
 		return $result;
@@ -112,7 +112,7 @@ 
 
 		// Default desc.
 		foreach ( $this->sub_fields as $name => $sub_field ) {
-			$extra_options[ $name . '_desc' ] = $sub_field['label'];
+			$extra_options[$name . '_desc'] = $sub_field['label'];
 		}
 
 		return $extra_options;
@@ -151,7 +151,7 @@ 
 		$name_layout = $this->get_name_layout();
 
 		if ( ! empty( $atts['show'] ) ) {
-			return $value[ $atts['show'] ] ?? '';
+			return $value[$atts['show']] ?? '';
 		}
 
 		$value = wp_parse_args(
@@ -231,7 +231,7 @@ 
 			$args['sub_fields'] = $this->sub_fields;
 
 			foreach ( $hidden_fields as $name => $hidden_field ) {
-				$args['sub_fields'][ $name ]['wrapper_classes'] .= ' frm_hidden';
+				$args['sub_fields'][$name]['wrapper_classes'] .= ' frm_hidden';
 			}
 		}
 	}
@@ -316,7 +316,7 @@ 
 	public static function track_first_name_field( $fields ) {
 		foreach ( $fields as $field ) {
 			if ( 'name' === $field->type ) {
-				self::$first_name_field_ids[ $field->form_id ] = $field->id;
+				self::$first_name_field_ids[$field->form_id] = $field->id;
 				return;
 			}
 		}
@@ -337,7 +337,7 @@ 
 
 		$form_id = (int) ( is_array( $args['field'] ) ? $args['field']['form_id'] : $args['field']->form_id );
 
-		if ( ! self::$first_name_field_ids || empty( self::$first_name_field_ids[ $form_id ] ) ) {
+		if ( ! self::$first_name_field_ids || empty( self::$first_name_field_ids[$form_id] ) ) {
 			return $attrs;
 		}
 
@@ -350,7 +350,7 @@ 
 
 		$field_id = (int) ( is_array( $args['field'] ) ? $args['field']['id'] : $args['field']->id );
 
-		if ( intval( self::$first_name_field_ids[ $form_id ] ) === $field_id ) {
+		if ( intval( self::$first_name_field_ids[$form_id] ) === $field_id ) {
 			if ( 'first' === $sub_field['name'] ) {
 				$attrs['autocomplete'] = 'given-name';
 			} elseif ( 'last' === $sub_field['name'] ) {

classes/models/fields/FrmFieldCombo.php

Spacing +16 added lines, -16 removed lines

@@ -63,12 +63,12 @@ 
 			if ( is_array( $sub_field ) ) {
 				$sub_field                 = wp_parse_args( $sub_field, $defaults );
 				$sub_field['name']         = $name;
-				$this->sub_fields[ $name ] = $sub_field;
+				$this->sub_fields[$name] = $sub_field;
 				continue;
 			}
 
 			if ( is_string( $sub_field ) ) {
-				$this->sub_fields[ $name ] = wp_parse_args(
+				$this->sub_fields[$name] = wp_parse_args(
 					array(
 						'name'  => $name,
 						'label' => $sub_field,
@@ -91,7 +91,7 @@ 
 	 */
 	public function set_aria_invalid_error( &$shortcode_atts, $args ) {
 		foreach ( $this->get_sub_fields() as $sub_field ) {
-			$shortcode_atts[ 'aria-invalid-' . $sub_field['name'] ] = isset( $args['errors'][ 'field' . $this->field_id . '-' . $sub_field['name'] ] ) ? 'true' : 'false';
+			$shortcode_atts['aria-invalid-' . $sub_field['name']] = isset( $args['errors']['field' . $this->field_id . '-' . $sub_field['name']] ) ? 'true' : 'false';
 		}
 	}
 
@@ -137,9 +137,9 @@ 
 				}
 
 				if ( is_string( $option ) ) {
-					$extra_options[ $key . '_' . $option ] = '';
+					$extra_options[$key . '_' . $option] = '';
 				} elseif ( ! empty( $option['name'] ) ) {
-					$extra_options[ $key . '_' . $option['name'] ] = '';
+					$extra_options[$key . '_' . $option['name']] = '';
 				}
 			}
 		}
@@ -166,7 +166,7 @@ 
 			$sub_field['name'] = $name;
 			$wrapper_classes   = 'frm_grid_container frm_sub_field_options frm_sub_field_options-' . $sub_field['name'];
 
-			if ( ! isset( $processed_sub_fields[ $name ] ) ) {
+			if ( ! isset( $processed_sub_fields[$name] ) ) {
 				// Options for this subfield should be hidden.
 				$wrapper_classes .= ' frm_hidden';
 			}
@@ -196,7 +196,7 @@ 
 			$default_value = array();
 
 			foreach ( $this->sub_fields as $name => $sub_field ) {
-				$default_value[ $name ] = '';
+				$default_value[$name] = '';
 			}
 
 			return $default_value;
@@ -381,8 +381,8 @@ 
 		if ( in_array( 'placeholder', $sub_field['options'], true ) ) {
 			$placeholders = FrmField::get_option( $field, 'placeholder' );
 
-			if ( ! empty( $placeholders[ $sub_field['name'] ] ) ) {
-				$field['placeholder'] = $placeholders[ $sub_field['name'] ];
+			if ( ! empty( $placeholders[$sub_field['name']] ) ) {
+				$field['placeholder'] = $placeholders[$sub_field['name']];
 			}
 		}
 
@@ -445,9 +445,9 @@ 
 
 		// Validate not empty.
 		foreach ( $sub_fields as $name => $sub_field ) {
-			if ( empty( $sub_field['optional'] ) && empty( $args['value'][ $name ] ) ) {
-				$errors[ 'field' . $args['id'] . '-' . $name ] = '';
-				$errors[ 'field' . $args['id'] ]               = $blank_msg;
+			if ( empty( $sub_field['optional'] ) && empty( $args['value'][$name] ) ) {
+				$errors['field' . $args['id'] . '-' . $name] = '';
+				$errors['field' . $args['id']]               = $blank_msg;
 			}
 		}
 
@@ -467,7 +467,7 @@ 
 		$sub_fields = $this->get_processed_sub_fields();
 
 		foreach ( $sub_fields as $name => $sub_field ) {
-			$headings[ $field_id . '_' . $name ] = $field_name . ' (' . $field_key . ') - ' . $sub_field['label'];
+			$headings[$field_id . '_' . $name] = $field_name . ' (' . $field_key . ') - ' . $sub_field['label'];
 		}
 
 		return $headings;
@@ -531,9 +531,9 @@ 
 			'value' => '',
 		);
 
-		if ( ! empty( $args['field']['value'][ $sub_field['name'] ] ) ) {
-			$attrs['value']       = $args['field']['value'][ $sub_field['name'] ];
-			$attrs['data-frmval'] = $args['field']['value'][ $sub_field['name'] ];
+		if ( ! empty( $args['field']['value'][$sub_field['name']] ) ) {
+			$attrs['value']       = $args['field']['value'][$sub_field['name']];
+			$attrs['data-frmval'] = $args['field']['value'][$sub_field['name']];
 		}
 
 		if ( empty( $args['remove_names'] ) ) {

classes/models/FrmInbox.php

Spacing +32 added lines, -32 removed lines

@@ -116,7 +116,7 @@ 
 			return;
 		}
 
-		if ( isset( self::$messages[ $message['key'] ] ) && ! isset( $message['force'] ) ) {
+		if ( isset( self::$messages[$message['key']] ) && ! isset( $message['force'] ) ) {
 			// Don't replace messages unless required.
 			return;
 		}
@@ -125,13 +125,13 @@ 
 			return;
 		}
 
-		if ( isset( self::$messages[ $message['key'] ] ) ) {
+		if ( isset( self::$messages[$message['key']] ) ) {
 			// Move up and mark as new.
-			unset( self::$messages[ $message['key'] ] );
+			unset( self::$messages[$message['key']] );
 		}
 
 		$this->fill_message( $message );
-		self::$messages[ $message['key'] ] = $message;
+		self::$messages[$message['key']] = $message;
 
 		$this->update_list();
 
@@ -169,11 +169,11 @@ 
 		$removed = false;
 
 		foreach ( self::$messages as $t => $message ) {
-			$read      = ! empty( $message['read'] ) && isset( $message['read'][ get_current_user_id() ] ) && $message['read'][ get_current_user_id() ] < strtotime( '-1 month' );
-			$dismissed = ! empty( $message['dismissed'] ) && isset( $message['dismissed'][ get_current_user_id() ] ) && $message['dismissed'][ get_current_user_id() ] < strtotime( '-1 week' );
+			$read      = ! empty( $message['read'] ) && isset( $message['read'][get_current_user_id()] ) && $message['read'][get_current_user_id()] < strtotime( '-1 month' );
+			$dismissed = ! empty( $message['dismissed'] ) && isset( $message['dismissed'][get_current_user_id()] ) && $message['dismissed'][get_current_user_id()] < strtotime( '-1 week' );
 
 			if ( $read || $dismissed || ! $this->within_valid_timeframe( $message ) ) {
-				unset( self::$messages[ $t ] );
+				unset( self::$messages[$t] );
 				$removed = true;
 			}
 		}
@@ -193,12 +193,12 @@ 
 		$user_id = get_current_user_id();
 
 		foreach ( $messages as $k => $message ) {
-			$dismissed = isset( $message['dismissed'] ) && isset( $message['dismissed'][ $user_id ] );
+			$dismissed = isset( $message['dismissed'] ) && isset( $message['dismissed'][$user_id] );
 
 			if ( empty( $k ) || ! $this->within_valid_timeframe( $message ) || ( $type === 'dismissed' ) !== $dismissed ) {
-				unset( $messages[ $k ] );
+				unset( $messages[$k] );
 			} elseif ( ! $this->is_for_user( $message ) ) {
-				unset( $messages[ $k ] );
+				unset( $messages[$k] );
 			}
 		}
 
@@ -276,14 +276,14 @@ 
 	 * @return void
 	 */
 	public function mark_read( $key ) {
-		if ( ! $key || ! isset( self::$messages[ $key ] ) ) {
+		if ( ! $key || ! isset( self::$messages[$key] ) ) {
 			return;
 		}
 
-		if ( ! isset( self::$messages[ $key ]['read'] ) ) {
-			self::$messages[ $key ]['read'] = array();
+		if ( ! isset( self::$messages[$key]['read'] ) ) {
+			self::$messages[$key]['read'] = array();
 		}
-		self::$messages[ $key ]['read'][ get_current_user_id() ] = time();
+		self::$messages[$key]['read'][get_current_user_id()] = time();
 
 		$this->update_list();
 	}
@@ -296,10 +296,10 @@ 
 	 * @return void
 	 */
 	public function mark_unread( $key ) {
-		$is_read = isset( self::$messages[ $key ] ) && isset( self::$messages[ $key ]['read'] ) && isset( self::$messages[ $key ]['read'][ get_current_user_id() ] );
+		$is_read = isset( self::$messages[$key] ) && isset( self::$messages[$key]['read'] ) && isset( self::$messages[$key]['read'][get_current_user_id()] );
 
 		if ( $is_read ) {
-			unset( self::$messages[ $key ]['read'][ get_current_user_id() ] );
+			unset( self::$messages[$key]['read'][get_current_user_id()] );
 			$this->update_list();
 		}
 	}
@@ -315,14 +315,14 @@ 
 			return;
 		}
 
-		if ( ! isset( self::$messages[ $key ] ) ) {
+		if ( ! isset( self::$messages[$key] ) ) {
 			return;
 		}
 
-		if ( ! isset( self::$messages[ $key ]['dismissed'] ) ) {
-			self::$messages[ $key ]['dismissed'] = array();
+		if ( ! isset( self::$messages[$key]['dismissed'] ) ) {
+			self::$messages[$key]['dismissed'] = array();
 		}
-		self::$messages[ $key ]['dismissed'][ get_current_user_id() ] = time();
+		self::$messages[$key]['dismissed'][get_current_user_id()] = time();
 
 		$this->update_list();
 	}
@@ -337,11 +337,11 @@ 
 
 		foreach ( self::$messages as $key => $message ) {
 			if ( ! isset( $message['dismissed'] ) ) {
-				self::$messages[ $key ]['dismissed'] = array();
+				self::$messages[$key]['dismissed'] = array();
 			}
 
-			if ( ! isset( $message['dismissed'][ $user_id ] ) ) {
-				self::$messages[ $key ]['dismissed'][ $user_id ] = time();
+			if ( ! isset( $message['dismissed'][$user_id] ) ) {
+				self::$messages[$key]['dismissed'][$user_id] = time();
 			}
 		}
 		$this->update_list();
@@ -355,8 +355,8 @@ 
 		$user_id  = get_current_user_id();
 
 		foreach ( $messages as $t => $message ) {
-			if ( isset( $message['read'] ) && isset( $message['read'][ $user_id ] ) ) {
-				unset( $messages[ $t ] );
+			if ( isset( $message['read'] ) && isset( $message['read'][$user_id] ) ) {
+				unset( $messages[$t] );
 			}
 		}
 		return $messages;
@@ -398,8 +398,8 @@ 
 	 * @return void
 	 */
 	public function remove( $key ) {
-		if ( isset( self::$messages[ $key ] ) ) {
-			unset( self::$messages[ $key ] );
+		if ( isset( self::$messages[$key] ) ) {
+			unset( self::$messages[$key] );
 			$this->update_list();
 		}
 	}
@@ -448,7 +448,7 @@ 
 			 *
 			 * @return string
 			 */
-			function ( $matches ) {
+			function( $matches ) {
 				$url   = $matches[2];
 				$parts = parse_url( $url );
 
@@ -514,8 +514,8 @@ 
 		$inbox = new self();
 		return array_filter(
 			$inbox->get_messages( 'filter' ),
-			function ( $message ) use ( $key ) {
-				return ! empty( $message[ $key ] );
+			function( $message ) use ( $key ) {
+				return ! empty( $message[$key] );
 			}
 		);
 	}
@@ -562,8 +562,8 @@ 
 
 		return array_reduce(
 			$keys_to_return,
-			function ( $total, $key ) use ( $message ) {
-				$total[ $key ] = $message[ $key ] ?? '';
+			function( $total, $key ) use ( $message ) {
+				$total[$key] = $message[$key] ?? '';
 				return $total;
 			},
 			array()

classes/models/FrmSalesApi.php

Spacing +9 added lines, -9 removed lines

@@ -113,8 +113,8 @@ 
 		$cross_sell_links = $data['cross_sell_link'];
 		$index            = self::determine_cross_sell_index( $cross_sell_text );
 
-		self::$cross_sell_text = sanitize_text_field( $cross_sell_text[ $index ] );
-		self::$cross_sell_link = esc_url_raw( $cross_sell_links[ $index ] );
+		self::$cross_sell_text = sanitize_text_field( $cross_sell_text[$index] );
+		self::$cross_sell_link = esc_url_raw( $cross_sell_links[$index] );
 	}
 
 	/**
@@ -161,13 +161,13 @@ 
 		}
 
 		foreach ( $cross_sell_text as $index => $current_text ) {
-			if ( ! isset( $cross_sell_settings[ $current_text ] ) ) {
-				$cross_sell_settings[ $current_text ] = time();
+			if ( ! isset( $cross_sell_settings[$current_text] ) ) {
+				$cross_sell_settings[$current_text] = time();
 				update_option( $option_name, $cross_sell_settings );
 				return $index;
 			}
 
-			$time_elapsed = time() - $cross_sell_settings[ $current_text ];
+			$time_elapsed = time() - $cross_sell_settings[$current_text];
 
 			if ( $time_elapsed < DAY_IN_SECONDS * 30 ) {
 				return $index;
@@ -201,7 +201,7 @@ 
 			return;
 		}
 
-		self::$sales[ $sale['key'] ] = $this->fill_sale( $sale );
+		self::$sales[$sale['key']] = $this->fill_sale( $sale );
 	}
 
 	/**
@@ -312,7 +312,7 @@ 
 
 		$sale = self::$instance->get_best_sale();
 
-		return is_array( $sale ) && ! empty( $sale[ $key ] ) ? $sale[ $key ] : false;
+		return is_array( $sale ) && ! empty( $sale[$key] ) ? $sale[$key] : false;
 	}
 
 	/**
@@ -520,14 +520,14 @@ 
 			esc_html( self::$cross_sell_text ),
 			'activate_plugins',
 			'frm-sales-api-cross-sell',
-			function () {
+			function() {
 				// There is no page. The redirect logic is handled below, before this callback is triggered.
 			}
 		);
 
 		add_action(
 			'admin_init',
-			function () {
+			function() {
 				if ( ! current_user_can( 'activate_plugins' ) ) {
 					return;
 				}

classes/controllers/FrmXMLController.php

Spacing +11 added lines, -11 removed lines

@@ -177,7 +177,7 @@ 
 			return;
 		}
 
-		$selected_xml = isset( $form['xml'] ) && isset( $form['xml'][ $selected_form ] ) ? $form['xml'][ $selected_form ] : '';
+		$selected_xml = isset( $form['xml'] ) && isset( $form['xml'][$selected_form] ) ? $form['xml'][$selected_form] : '';
 
 		if ( empty( $selected_xml ) || strpos( $selected_xml, 'http' ) !== 0 ) {
 			return;
@@ -195,8 +195,8 @@ 
 	 * @return string
 	 */
 	private static function get_selected_in_form( $form, $value = 'form' ) {
-		if ( ! empty( $form ) && ! empty( $form[ $value ] ) ) {
-			return $form[ $value ];
+		if ( ! empty( $form ) && ! empty( $form[$value] ) ) {
+			return $form[$value];
 		}
 
 		return '';
@@ -226,7 +226,7 @@ 
 			}
 
 			if ( $for === 'view' ) {
-				$item_key  = is_array( $view_keys ) ? $view_keys[ $form_key ] : $view_keys;
+				$item_key  = is_array( $view_keys ) ? $view_keys[$form_key] : $view_keys;
 				$shortcode = '[display-frm-data id=%1$s filter=limited]';
 			} elseif ( $for === 'form' ) {
 				$item_key  = $form_key;
@@ -241,7 +241,7 @@ 
 				continue;
 			}
 
-			$page_ids[ $for ] = wp_insert_post(
+			$page_ids[$for] = wp_insert_post(
 				array(
 					'post_title'   => $name,
 					'post_type'    => 'page',
@@ -400,7 +400,7 @@ 
 		$file_type = sanitize_option( 'upload_path', $_FILES['frm_import_file']['name'] );
 		$file_type = strtolower( pathinfo( $file_type, PATHINFO_EXTENSION ) );
 
-		if ( 'xml' !== $file_type && isset( $export_format[ $file_type ] ) ) {
+		if ( 'xml' !== $file_type && isset( $export_format[$file_type] ) ) {
 			// allow other file types to be imported
 			do_action( 'frm_before_import_' . $file_type );
 
@@ -495,7 +495,7 @@ 
 		foreach ( $type as $tb_type ) {
 			$where = array();
 			$join  = '';
-			$table = $tables[ $tb_type ];
+			$table = $tables[$tb_type];
 
 			$select     = $table . '.id';
 			$query_vars = array();
@@ -510,7 +510,7 @@ 
 							$table . '.parent_form_id' => $args['ids'],
 						);
 					} else {
-						$where[ $table . '.status !' ] = 'draft';
+						$where[$table . '.status !'] = 'draft';
 					}
 					break;
 				case 'actions':
@@ -524,7 +524,7 @@ 
 				case 'items':
 					// $join = "INNER JOIN {$wpdb->prefix}frm_item_metas im ON ($table.id = im.item_id)";
 					if ( $args['ids'] ) {
-						$where[ $table . '.form_id' ] = $args['ids'];
+						$where[$table . '.form_id'] = $args['ids'];
 					}
 					break;
 				case 'styles':
@@ -568,7 +568,7 @@ 
 					}
 			}//end switch
 
-			$records[ $tb_type ] = FrmDb::get_col( $table . $join, $where, $select );
+			$records[$tb_type] = FrmDb::get_col( $table . $join, $where, $select );
 			unset( $tb_type );
 		}//end foreach
 
@@ -781,7 +781,7 @@ 
 
 		foreach ( $csv_fields as $k => $f ) {
 			if ( in_array( $f->type, $no_export_fields, true ) ) {
-				unset( $csv_fields[ $k ] );
+				unset( $csv_fields[$k] );
 			}
 		}