WordImpress / Give

includes/admin/admin-actions.php

Spacing +188 added lines, -189 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,19 +20,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -52,13 +52,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -68,21 +68,21 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');
 
 
 /**
@@ -96,25 +96,25 @@ 
  */
 function give_hide_outdated_php_notice() {
 
-	if ( ! isset( $_POST['_give_hide_outdated_php_notices_shortly'] ) ) {
+	if ( ! isset($_POST['_give_hide_outdated_php_notices_shortly'])) {
 		give_die();
 	}
 
 	// Transient key name.
 	$transient_key = "_give_hide_outdated_php_notices_shortly";
 
-	if ( Give_Cache::get( $transient_key, true ) ) {
+	if (Give_Cache::get($transient_key, true)) {
 		return;
 	}
 
 	// Hide notice for 24 hours.
-	Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+	Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 	give_die();
 
 }
 
-add_action( 'wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice' );
+add_action('wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice');
 
 /**
  * Register admin notices.
@@ -123,27 +123,27 @@ 
  */
 function _give_register_admin_notices() {
 	// Bailout.
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
 	// Bulk action notices.
 	if (
-		isset( $_GET['action'] ) &&
-		! empty( $_GET['action'] )
+		isset($_GET['action']) &&
+		! empty($_GET['action'])
 	) {
 
 		// Add payment bulk notice.
 		if (
-			current_user_can( 'edit_give_payments' ) &&
-			isset( $_GET['payment'] ) &&
-			! empty( $_GET['payment'] )
+			current_user_can('edit_give_payments') &&
+			isset($_GET['payment']) &&
+			! empty($_GET['payment'])
 		) {
-			$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+			$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-			switch ( $_GET['action'] ) {
+			switch ($_GET['action']) {
 				case 'delete':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_delete',
 						'type'        => 'updated',
 						'description' => sprintf(
@@ -155,12 +155,12 @@ 
 							),
 							$payment_count ),
 						'show'        => true,
-					) );
+					));
 
 					break;
 
 				case 'resend-receipt':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_resend_receipt',
 						'type'        => 'updated',
 						'description' => sprintf(
@@ -173,7 +173,7 @@ 
 							$payment_count
 						),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'set-status-publish' :
@@ -185,7 +185,7 @@ 
 				case 'set-status-cancelled' :
 				case 'set-status-abandoned' :
 				case 'set-status-preapproval' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_status_change',
 						'type'        => 'updated',
 						'description' => _n(
@@ -195,7 +195,7 @@ 
 							'give'
 						),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 
@@ -203,244 +203,244 @@ 
 	}
 
 	// Add give message notices.
-	if ( ! empty( $_GET['give-message'] ) ) {
+	if ( ! empty($_GET['give-message'])) {
 		// Donation reports errors.
-		if ( current_user_can( 'view_give_reports' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('view_give_reports')) {
+			switch ($_GET['give-message']) {
 				case 'donation_deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been deleted.', 'give' ),
+						'description' => __('The donation has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'email_sent' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-sent',
 						'type'        => 'updated',
-						'description' => __( 'The donation receipt has been resent.', 'give' ),
+						'description' => __('The donation receipt has been resent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'refreshed-reports' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-refreshed-reports',
 						'type'        => 'updated',
-						'description' => __( 'The reports cache has been cleared.', 'give' ),
+						'description' => __('The reports cache has been cleared.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'donation-note-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-note-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been deleted.', 'give' ),
+						'description' => __('The donation note has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Give settings notices and errors.
-		if ( current_user_can( 'manage_give_settings' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('manage_give_settings')) {
+			switch ($_GET['give-message']) {
 				case 'settings-imported' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-settings-imported',
 						'type'        => 'updated',
-						'description' => __( 'The settings have been imported.', 'give' ),
+						'description' => __('The settings have been imported.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-generated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-generated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been generated.', 'give' ),
+						'description' => __('API keys have been generated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-exists' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-exists',
 						'type'        => 'updated',
-						'description' => __( 'The specified user already has API keys.', 'give' ),
+						'description' => __('The specified user already has API keys.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-regenerated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-regenerated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been regenerated.', 'give' ),
+						'description' => __('API keys have been regenerated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-revoked' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-revoked',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been revoked.', 'give' ),
+						'description' => __('API keys have been revoked.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'sent-test-email' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-sent-test-email',
 						'type'        => 'updated',
-						'description' => __( 'The test email has been sent.', 'give' ),
+						'description' => __('The test email has been sent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'matched-success-failure-page':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-matched-success-failure-page',
 						'type'        => 'updated',
-						'description' => __( 'You cannot set the success and failed pages to the same page', 'give' ),
+						'description' => __('You cannot set the success and failed pages to the same page', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 		// Payments errors.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'note-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-note-added',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been added.', 'give' ),
+						'description' => __('The donation note has been added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'payment-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-updated',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been updated.', 'give' ),
+						'description' => __('The donation has been updated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Donor Notices.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'donor-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The selected donor(s) has been deleted.', 'give' ),
+						'description' => __('The selected donor(s) has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'donor-donations-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-donations-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The selected donor(s) and its associated donations has been deleted.', 'give' ),
+						'description' => __('The selected donor(s) and its associated donations has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'confirm-delete-donor' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-confirm-delete-donor',
 						'type'        => 'updated',
-						'description' => __( 'You must confirm to delete the selected donor(s).', 'give' ),
+						'description' => __('You must confirm to delete the selected donor(s).', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'invalid-donor-id' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-invalid-donor-id',
 						'type'        => 'updated',
-						'description' => __( 'Invalid Donor ID.', 'give' ),
+						'description' => __('Invalid Donor ID.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'donor-delete-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-delete-failed',
 						'type'        => 'error',
-						'description' => __( 'Unable to delete selected donor(s).', 'give' ),
+						'description' => __('Unable to delete selected donor(s).', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-added',
 						'type'        => 'updated',
-						'description' => __( 'Donor email added.', 'give' ),
+						'description' => __('Donor email added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-removed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-removed',
 						'type'        => 'updated',
-						'description' => __( 'Donor email removed.', 'give' ),
+						'description' => __('Donor email removed.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-remove-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-remove-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to remove donor email.', 'give' ),
+						'description' => __('Failed to remove donor email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-updated',
 						'type'        => 'updated',
-						'description' => __( 'Primary email updated for donor.', 'give' ),
+						'description' => __('Primary email updated for donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to set primary email.', 'give' ),
+						'description' => __('Failed to set primary email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'reconnect-user' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-reconnect-user',
 						'type'        => 'updated',
-						'description' => __( 'User has been successfully connected with Donor.', 'give' ),
+						'description' => __('User has been successfully connected with Donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'profile-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-profile-updated',
 						'type'        => 'updated',
-						'description' => __( 'Donor information updated successfully.', 'give' ),
+						'description' => __('Donor information updated successfully.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 }
 
-add_action( 'admin_notices', '_give_register_admin_notices', - 1 );
+add_action('admin_notices', '_give_register_admin_notices', - 1);
 
 
 /**
@@ -450,31 +450,30 @@ 
  *
  * @return bool
  */
-function _give_show_test_mode_notice_in_admin_bar( $wp_admin_bar ) {
-	$is_test_mode = ! empty( $_POST['test_mode'] ) ?
-		give_is_setting_enabled( $_POST['test_mode'] ) :
-		give_is_test_mode();
+function _give_show_test_mode_notice_in_admin_bar($wp_admin_bar) {
+	$is_test_mode = ! empty($_POST['test_mode']) ?
+		give_is_setting_enabled($_POST['test_mode']) : give_is_test_mode();
 
 	if (
-		! current_user_can( 'view_give_reports' ) ||
+		! current_user_can('view_give_reports') ||
 		! $is_test_mode
 	) {
 		return false;
 	}
 
 	// Add the main site admin menu item.
-	$wp_admin_bar->add_menu( array(
+	$wp_admin_bar->add_menu(array(
 		'id'     => 'give-test-notice',
-		'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+		'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 		'parent' => 'top-secondary',
-		'title'  => __( 'Give Test Mode Active', 'give' ),
-		'meta'   => array( 'class' => 'give-test-mode-active' ),
-	) );
+		'title'  => __('Give Test Mode Active', 'give'),
+		'meta'   => array('class' => 'give-test-mode-active'),
+	));
 
 	return true;
 }
 
-add_action( 'admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1 );
+add_action('admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1);
 
 /**
  * Add Link to Import page in from donation archive and donation single page
@@ -483,12 +482,12 @@ 
  */
 function give_import_page_link_callback() {
 	?>
-	<a href="<?php echo esc_url( give_import_page_url() ); ?>"
-	   class="page-import-action page-title-action"><?php _e( 'Import Donations', 'give' ); ?></a>
+	<a href="<?php echo esc_url(give_import_page_url()); ?>"
+	   class="page-import-action page-title-action"><?php _e('Import Donations', 'give'); ?></a>
 
 	<?php
 	// Check if view donation single page only.
-	if ( ! empty( $_REQUEST['view'] ) && 'view-payment-details' === (string) give_clean( $_REQUEST['view'] ) && 'give-payment-history' === give_clean( $_REQUEST['page'] ) ) {
+	if ( ! empty($_REQUEST['view']) && 'view-payment-details' === (string) give_clean($_REQUEST['view']) && 'give-payment-history' === give_clean($_REQUEST['page'])) {
 		?>
 		<style type="text/css">
 			.wrap #transaction-details-heading {
@@ -499,7 +498,7 @@ 
 	}
 }
 
-add_action( 'give_payments_page_top', 'give_import_page_link_callback', 11 );
+add_action('give_payments_page_top', 'give_import_page_link_callback', 11);
 
 /**
  * Load donation import ajax callback
@@ -511,9 +510,9 @@ 
  */
 function give_donation_import_callback() {
 	$import_setting = array();
-	$fields         = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
+	$fields         = isset($_POST['fields']) ? $_POST['fields'] : null;
 
-	parse_str( $fields );
+	parse_str($fields);
 
 	$import_setting['create_user'] = $create_user;
 	$import_setting['mode']        = $mode;
@@ -522,54 +521,54 @@ 
 	$import_setting['delete_csv']  = $delete_csv;
 
 	// Parent key id.
-	$main_key = maybe_unserialize( $main_key );
-
-	$current    = absint( $_REQUEST['current'] );
-	$total_ajax = absint( $_REQUEST['total_ajax'] );
-	$start      = absint( $_REQUEST['start'] );
-	$end        = absint( $_REQUEST['end'] );
-	$next       = absint( $_REQUEST['next'] );
-	$total      = absint( $_REQUEST['total'] );
-	$per_page   = absint( $_REQUEST['per_page'] );
-	if ( empty( $delimiter ) ) {
+	$main_key = maybe_unserialize($main_key);
+
+	$current    = absint($_REQUEST['current']);
+	$total_ajax = absint($_REQUEST['total_ajax']);
+	$start      = absint($_REQUEST['start']);
+	$end        = absint($_REQUEST['end']);
+	$next       = absint($_REQUEST['next']);
+	$total      = absint($_REQUEST['total']);
+	$per_page   = absint($_REQUEST['per_page']);
+	if (empty($delimiter)) {
 		$delimiter = ',';
 	}
 
 	// Processing done here.
-	$raw_data = give_get_donation_data_from_csv( $csv, $start, $end, $delimiter );
-	$raw_key  = maybe_unserialize( $mapto );
+	$raw_data = give_get_donation_data_from_csv($csv, $start, $end, $delimiter);
+	$raw_key  = maybe_unserialize($mapto);
 
 	// Prevent normal emails.
-	remove_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
-	remove_action( 'give_insert_user', 'give_new_user_notification', 10 );
-	remove_action( 'give_insert_payment', 'give_payment_save_page_data' );
+	remove_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
+	remove_action('give_insert_user', 'give_new_user_notification', 10);
+	remove_action('give_insert_payment', 'give_payment_save_page_data');
 
-	foreach ( $raw_data as $row_data ) {
-		give_save_import_donation_to_db( $raw_key, $row_data, $main_key, $import_setting );
+	foreach ($raw_data as $row_data) {
+		give_save_import_donation_to_db($raw_key, $row_data, $main_key, $import_setting);
 	}
 
 	// Check if function exists or not.
-	if ( function_exists( 'give_payment_save_page_data' ) ) {
-		add_action( 'give_insert_payment', 'give_payment_save_page_data' );
+	if (function_exists('give_payment_save_page_data')) {
+		add_action('give_insert_payment', 'give_payment_save_page_data');
 	}
-	add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
-	add_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
+	add_action('give_insert_user', 'give_new_user_notification', 10, 2);
+	add_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
 
-	if ( $next == false ) {
+	if ($next == false) {
 		$json_data = array(
 			'success' => true,
-			'message' => __( 'All donation uploaded successfully!', 'give' ),
+			'message' => __('All donation uploaded successfully!', 'give'),
 		);
 	} else {
 		$index_start = $start;
 		$index_end   = $end;
 		$last        = false;
 		$next        = true;
-		if ( $next ) {
+		if ($next) {
 			$index_start = $index_start + $per_page;
-			$index_end   = $per_page + ( $index_start - 1 );
+			$index_end   = $per_page + ($index_start - 1);
 		}
-		if ( $index_end >= $total ) {
+		if ($index_end >= $total) {
 			$index_end = $total;
 			$last      = true;
 		}
@@ -583,27 +582,27 @@ 
 		);
 	}
 
-	$url              = give_import_page_url( array(
+	$url = give_import_page_url(array(
 		'step'          => '4',
 		'importer-type' => 'import_donations',
 		'csv'           => $csv,
 		'total'         => $total,
 		'delete_csv'    => $import_setting['delete_csv'],
-		'success'       => ( isset( $json_data['success'] ) ? $json_data['success'] : '' ),
-	) );
+		'success'       => (isset($json_data['success']) ? $json_data['success'] : ''),
+	));
 	$json_data['url'] = $url;
 
-	$current ++;
+	$current++;
 	$json_data['current'] = $current;
 
-	$percentage              = ( 100 / ( $total_ajax + 1 ) ) * $current;
+	$percentage              = (100 / ($total_ajax + 1)) * $current;
 	$json_data['percentage'] = $percentage;
 	
-	$json_data = apply_filters( 'give_import_ajax_responces', $json_data, $fields );
-	wp_die( json_encode( $json_data ) );
+	$json_data = apply_filters('give_import_ajax_responces', $json_data, $fields);
+	wp_die(json_encode($json_data));
 }
 
-add_action( 'wp_ajax_give_donation_import', 'give_donation_import_callback' );
+add_action('wp_ajax_give_donation_import', 'give_donation_import_callback');
 
 /**
  * Load core settings import ajax callback
@@ -615,8 +614,8 @@ 
  */
 
 function give_core_settings_import_callback() {
-	$fields = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
-	parse_str( $fields, $fields );
+	$fields = isset($_POST['fields']) ? $_POST['fields'] : null;
+	parse_str($fields, $fields);
 
 	$json_data['success'] = false;
 
@@ -631,22 +630,22 @@ 
 	 *
 	 * @return array $fields
 	 */
-	$fields = (array) apply_filters( 'give_import_core_settings_fields', $fields );
+	$fields = (array) apply_filters('give_import_core_settings_fields', $fields);
 
-	$file_name = ( ! empty( $fields['file_name'] ) ? give_clean( $fields['file_name'] ) : false );
+	$file_name = ( ! empty($fields['file_name']) ? give_clean($fields['file_name']) : false);
 
-	if ( ! empty( $file_name ) ) {
-		$type = ( ! empty( $fields['type'] ) ? give_clean( $fields['type'] ) : 'merge' );
+	if ( ! empty($file_name)) {
+		$type = ( ! empty($fields['type']) ? give_clean($fields['type']) : 'merge');
 
 		// Get the json data from the file and then alter it in array format
-		$json_string   = give_get_core_settings_json( $file_name );
-		$json_to_array = json_decode( $json_string, true );
+		$json_string   = give_get_core_settings_json($file_name);
+		$json_to_array = json_decode($json_string, true);
 
 		// get the current settign from the options table.
-		$host_give_options = get_option( 'give_settings', array() );
+		$host_give_options = get_option('give_settings', array());
 
 		// Save old settins for backup.
-		update_option( 'give_settings_old', $host_give_options );
+		update_option('give_settings_old', $host_give_options);
 
 		/**
 		 * Filter to Modify Core Settings that are being going to get import in options table as give settings.
@@ -662,9 +661,9 @@ 
 		 *
 		 * @return array $json_to_array Setting that are being going to get imported
 		 */
-		$json_to_array = (array) apply_filters( 'give_import_core_settings_data', $json_to_array, $type, $host_give_options, $fields );
+		$json_to_array = (array) apply_filters('give_import_core_settings_data', $json_to_array, $type, $host_give_options, $fields);
 
-		update_option( 'give_settings', $json_to_array );
+		update_option('give_settings', $json_to_array);
 
 		$json_data['success'] = true;
 	}
@@ -680,16 +679,16 @@ 
 	 *
 	 * @return array $url
 	 */
-	$json_data['url'] = give_import_page_url( (array) apply_filters( 'give_import_core_settings_success_url', array(
-		'step'          => ( empty( $json_data['success'] ) ? '1' : '3' ),
+	$json_data['url'] = give_import_page_url((array) apply_filters('give_import_core_settings_success_url', array(
+		'step'          => (empty($json_data['success']) ? '1' : '3'),
 		'importer-type' => 'import_core_setting',
-		'success'       => ( empty( $json_data['success'] ) ? '0' : '1' ),
-	) ) );
+		'success'       => (empty($json_data['success']) ? '0' : '1'),
+	)));
 
-	wp_send_json( $json_data );
+	wp_send_json($json_data);
 }
 
-add_action( 'wp_ajax_give_core_settings_import', 'give_core_settings_import_callback' );
+add_action('wp_ajax_give_core_settings_import', 'give_core_settings_import_callback');
 
 /**
  * Initializes blank slate content if a list table is empty.
@@ -701,7 +700,7 @@ 
 	$blank_slate->init();
 }
 
-add_action( 'current_screen', 'give_blank_slate' );
+add_action('current_screen', 'give_blank_slate');
 
 /**
  * Get Array of WP User Roles.
@@ -714,8 +713,8 @@ 
 	$user_roles = array();
 
 	// Loop through User Roles.
-	foreach ( get_editable_roles() as $role_name => $role_info ):
-		$user_roles[ $role_name ] = $role_info['name'];
+	foreach (get_editable_roles() as $role_name => $role_info):
+		$user_roles[$role_name] = $role_info['name'];
 	endforeach;
 
 	return $user_roles;

includes/admin/settings/class-settings-license.php

Spacing +5 added lines, -5 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_License' ) ) :
+if ( ! class_exists('Give_Settings_License')) :
 
 	/**
 	 * Give_Settings_License.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'licenses';
-			$this->label = esc_html__( 'Licenses', 'give' );
+			$this->label = esc_html__('Licenses', 'give');
 
 			parent::__construct();
 		}
@@ -45,7 +45,7 @@ 
 			 * Filter the licenses settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_licenses', $settings );
+			$settings = apply_filters('give_settings_licenses', $settings);
 
 			/**
 			 * Filter the settings.
@@ -54,7 +54,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;

includes/admin/settings/class-settings-advanced.php

Spacing +30 added lines, -30 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Advanced' ) ) :
+if ( ! class_exists('Give_Settings_Advanced')) :
 
 	/**
 	 * Give_Settings_Advanced.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'advanced';
-			$this->label = __( 'Advanced', 'give' );
+			$this->label = __('Advanced', 'give');
 
 			$this->default_tab = 'advanced-options';
 
@@ -45,7 +45,7 @@ 
 
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'advanced-options':
 					$settings = array(
 						array(
@@ -53,8 +53,8 @@ 
 							'type' => 'title',
 						),
 						array(
-							'name'    => __( 'Default User Role', 'give' ),
-							'desc'    => __( 'Assign default user roles for donors when donors opt to register as a WP User.', 'give' ),
+							'name'    => __('Default User Role', 'give'),
+							'desc'    => __('Assign default user roles for donors when donors opt to register as a WP User.', 'give'),
 							'id'      => 'donor_default_user_role',
 							'type'    => 'select',
 							'default' => 'give_donor',
@@ -62,55 +62,55 @@ 
 						),
 						array(
 							/* translators: %s: the_content */
-							'name'    => sprintf( __( '%s filter', 'give' ), '<code>the_content</code>' ),
+							'name'    => sprintf(__('%s filter', 'give'), '<code>the_content</code>'),
 							/* translators: 1: https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content 2: the_content */
-							'desc'    => sprintf( __( 'If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give' ), esc_url( 'https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content' ), '<code>the_content</code>' ),
+							'desc'    => sprintf(__('If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give'), esc_url('https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content'), '<code>the_content</code>'),
 							'id'      => 'the_content_filter',
 							'default' => 'enabled',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Script Loading Location', 'give' ),
-							'desc'    => __( 'This allows you to load your Give scripts either in the <code>&lt;head&gt;</code> or footer of your website.', 'give' ),
+							'name'    => __('Script Loading Location', 'give'),
+							'desc'    => __('This allows you to load your Give scripts either in the <code>&lt;head&gt;</code> or footer of your website.', 'give'),
 							'id'      => 'scripts_footer',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'disabled' => __( 'Head', 'give' ),
-								'enabled'  => __( 'Footer', 'give' ),
+								'disabled' => __('Head', 'give'),
+								'enabled'  => __('Footer', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Akismet SPAM Protection', 'give' ),
-							'desc'    => __( 'Add a layer of SPAM protection to your donation submissions with Akismet. When enabled, donation submissions will be first sent to Akismet\'s API if you have the plugin activated and configured.', 'give' ),
+							'name'    => __('Akismet SPAM Protection', 'give'),
+							'desc'    => __('Add a layer of SPAM protection to your donation submissions with Akismet. When enabled, donation submissions will be first sent to Akismet\'s API if you have the plugin activated and configured.', 'give'),
 							'id'      => 'akismet_spam_protection',
 							'type'    => 'radio_inline',
-							'default' => ( give_check_akismet_key() ) ? 'enabled' : 'disabled',
+							'default' => (give_check_akismet_key()) ? 'enabled' : 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Remove Data on Uninstall', 'give' ),
-							'desc'    => __( 'When the plugin is deleted, completely remove all Give data. This includes all Give settings, forms, form meta, donor, donor data, donations. Everything.', 'give' ),
+							'name'    => __('Remove Data on Uninstall', 'give'),
+							'desc'    => __('When the plugin is deleted, completely remove all Give data. This includes all Give settings, forms, form meta, donor, donor data, donations. Everything.', 'give'),
 							'id'      => 'uninstall_on_delete',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Yes, Remove all data', 'give' ),
-								'disabled' => __( 'No, keep my Give settings and donation data', 'give' ),
+								'enabled'  => __('Yes, Remove all data', 'give'),
+								'disabled' => __('No, keep my Give settings and donation data', 'give'),
 							),
 						),
 						array(
-							'name'  => __( 'Advanced Settings Docs Link', 'give' ),
+							'name'  => __('Advanced Settings Docs Link', 'give'),
 							'id'    => 'advanced_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-advanced' ),
-							'title' => __( 'Advanced Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-advanced'),
+							'title' => __('Advanced Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -126,7 +126,7 @@ 
 			 * Filter the advanced settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_advanced', $settings );
+			$settings = apply_filters('give_settings_advanced', $settings);
 
 			/**
 			 * Filter the settings.
@@ -135,7 +135,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -149,10 +149,10 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'advanced-options' => __( 'Advanced Options', 'give' ),
+				'advanced-options' => __('Advanced Options', 'give'),
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/admin/settings/class-settings-display.php

Spacing +77 added lines, -77 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Display' ) ) :
+if ( ! class_exists('Give_Settings_Display')) :
 
 	/**
 	 * Give_Settings_Display.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'display';
-			$this->label = __( 'Display Options', 'give' );
+			$this->label = __('Display Options', 'give');
 
 			$this->default_tab = 'display-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings        = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'display-settings' :
 					$settings = array(
 						// Section 1: Display
@@ -53,55 +53,55 @@ 
 							'type' => 'title',
 						),
 						array(
-							'name'    => __( 'Default Give Styles', 'give' ),
-							'desc'    => __( 'You can disable Give\'s default styles for donation forms and other frontend elements.', 'give' ),
+							'name'    => __('Default Give Styles', 'give'),
+							'desc'    => __('You can disable Give\'s default styles for donation forms and other frontend elements.', 'give'),
 							'id'      => 'css',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Floating Labels', 'give' ),
+							'name'    => __('Floating Labels', 'give'),
 							/* translators: %s: http://docs.givewp.com/form-floating-labels */
-							'desc'    => sprintf( wp_kses( __( '<a href="%s" target="_blank">Floating labels</a> allows your labels to be inset within the form fields to provide a cleaner form appearance. Note that if the "Disable CSS" option is enabled, you will need to style the floating labels yourself.', 'give' ), array(
+							'desc'    => sprintf(wp_kses(__('<a href="%s" target="_blank">Floating labels</a> allows your labels to be inset within the form fields to provide a cleaner form appearance. Note that if the "Disable CSS" option is enabled, you will need to style the floating labels yourself.', 'give'), array(
 								'a' => array(
 									'href'   => array(),
 									'target' => array(),
 								),
-							) ), esc_url( 'http://docs.givewp.com/form-floating-labels' ) ),
+							)), esc_url('http://docs.givewp.com/form-floating-labels')),
 							'id'      => 'floatlabels',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Welcome Screen', 'give' ),
+							'name'    => __('Welcome Screen', 'give'),
 							/* translators: %s: about page URL */
-							'desc'    => sprintf( wp_kses( __( 'Enable this option if you would like to disable the <a href="%s" target="_blank">Give Welcome screen</a> that displays each time Give is activated or updated.', 'give' ), array(
+							'desc'    => sprintf(wp_kses(__('Enable this option if you would like to disable the <a href="%s" target="_blank">Give Welcome screen</a> that displays each time Give is activated or updated.', 'give'), array(
 								'a' => array(
 									'href'   => array(),
 									'target' => array(),
 								),
-							) ), esc_url( admin_url( 'index.php?page=give-about' ) ) ),
+							)), esc_url(admin_url('index.php?page=give-about'))),
 							'id'      => 'welcome',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'  => __( 'Display Settings Docs Link', 'give' ),
+							'name'  => __('Display Settings Docs Link', 'give'),
 							'id'    => 'display_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/form-display-options' ),
-							'title' => __( 'Display Options Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/form-display-options'),
+							'title' => __('Display Options Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -118,78 +118,78 @@ 
 							'type' => 'title',
 						),
 						array(
-							'name'    => __( 'Form Single Views', 'give' ),
-							'desc'    => __( 'By default, all donation form have single views enabled which creates a specific URL on your website for that form. This option disables the singular posts from being publicly viewable. Note: you will need to embed forms using a shortcode or widget if enabled.', 'give' ),
+							'name'    => __('Form Single Views', 'give'),
+							'desc'    => __('By default, all donation form have single views enabled which creates a specific URL on your website for that form. This option disables the singular posts from being publicly viewable. Note: you will need to embed forms using a shortcode or widget if enabled.', 'give'),
 							'id'      => 'forms_singular',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Form Archives', 'give' ),
-							'desc'    => sprintf( wp_kses( __( 'Archives pages list all the donation forms you have created. This option will disable only the form\'s archive page(s). The single form\'s view will remain in place. Note: you will need to <a href="%s">refresh your permalinks</a> after this option has been enabled.', 'give' ), array(
+							'name'    => __('Form Archives', 'give'),
+							'desc'    => sprintf(wp_kses(__('Archives pages list all the donation forms you have created. This option will disable only the form\'s archive page(s). The single form\'s view will remain in place. Note: you will need to <a href="%s">refresh your permalinks</a> after this option has been enabled.', 'give'), array(
 								'a' => array(
 									'href'   => array(),
 									'target' => array(),
 								),
-							) ), esc_url( admin_url( 'options-permalink.php' ) ) ),
+							)), esc_url(admin_url('options-permalink.php'))),
 							'id'      => 'forms_archives',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Form Excerpts', 'give' ),
-							'desc'    => __( 'The excerpt is an optional summary or description of a donation form; in short, a summary as to why the user should give.', 'give' ),
+							'name'    => __('Form Excerpts', 'give'),
+							'desc'    => __('The excerpt is an optional summary or description of a donation form; in short, a summary as to why the user should give.', 'give'),
 							'id'      => 'forms_excerpt',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Form Featured Image', 'give' ),
-							'desc'    => __( 'If you do not wish to use the featured image functionality you can disable it using this option and it will not be displayed for single donation forms.', 'give' ),
+							'name'    => __('Form Featured Image', 'give'),
+							'desc'    => __('If you do not wish to use the featured image functionality you can disable it using this option and it will not be displayed for single donation forms.', 'give'),
 							'id'      => 'form_featured_img',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Featured Image Size', 'give' ),
-							'desc'    => __( 'The Featured Image is an image that is chosen as the representative image for a donation form. Some themes may have custom featured image sizes. Please select the size you would like to display for your single donation form\'s featured image.', 'give' ),
+							'name'    => __('Featured Image Size', 'give'),
+							'desc'    => __('The Featured Image is an image that is chosen as the representative image for a donation form. Some themes may have custom featured image sizes. Please select the size you would like to display for your single donation form\'s featured image.', 'give'),
 							'id'      => 'featured_image_size',
 							'type'    => 'select',
 							'default' => 'large',
 							'options' => give_get_featured_image_sizes(),
 						),
 						array(
-							'name'    => __( 'Single Form Sidebar', 'give' ),
-							'desc'    => __( 'The sidebar allows you to add additional widgets to the Give single form view. If you don\'t plan on using the sidebar you may disable it with this option.', 'give' ),
+							'name'    => __('Single Form Sidebar', 'give'),
+							'desc'    => __('The sidebar allows you to add additional widgets to the Give single form view. If you don\'t plan on using the sidebar you may disable it with this option.', 'give'),
 							'id'      => 'form_sidebar',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'  => __( 'Post Types Docs Link', 'give' ),
+							'name'  => __('Post Types Docs Link', 'give'),
 							'id'    => 'post_types_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-post-types' ),
-							'title' => __( 'Post Types Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-post-types'),
+							'title' => __('Post Types Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -206,32 +206,32 @@ 
 							'type' => 'title',
 						),
 						array(
-							'name'    => __( 'Form Categories', 'give' ),
-							'desc'    => __( 'Enable Categories for all Give forms.', 'give' ),
+							'name'    => __('Form Categories', 'give'),
+							'desc'    => __('Enable Categories for all Give forms.', 'give'),
 							'id'      => 'categories',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Form Tags', 'give' ),
-							'desc'    => __( 'Enable Tags for all Give forms.', 'give' ),
+							'name'    => __('Form Tags', 'give'),
+							'desc'    => __('Enable Tags for all Give forms.', 'give'),
 							'id'      => 'tags',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'  => __( 'Taxonomies Docs Link', 'give' ),
+							'name'  => __('Taxonomies Docs Link', 'give'),
 							'id'    => 'taxonomies_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-taxonomies' ),
-							'title' => __( 'Taxonomies Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-taxonomies'),
+							'title' => __('Taxonomies Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -248,36 +248,36 @@ 
 							'type' => 'title',
 						),
 						array(
-							'name'    => __( 'Terms and Conditions', 'give' ),
-							'desc'    => __( 'Would you like donors to require that donors agree to your terms when donating? Note: You can enable/disable this option and customize the terms per form as well.', 'give' ),
+							'name'    => __('Terms and Conditions', 'give'),
+							'desc'    => __('Would you like donors to require that donors agree to your terms when donating? Note: You can enable/disable this option and customize the terms per form as well.', 'give'),
 							'id'      => 'terms',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'       => __( 'Agree to Terms Label', 'give' ),
-							'desc'       => __( 'The label shown next to the agree to terms check box. Customize it here or leave blank to use the default placeholder text. Note: You can customize the label per form.', 'give' ),
+							'name'       => __('Agree to Terms Label', 'give'),
+							'desc'       => __('The label shown next to the agree to terms check box. Customize it here or leave blank to use the default placeholder text. Note: You can customize the label per form.', 'give'),
 							'id'         => 'agree_to_terms_label',
 							'attributes' => array(
-								'placeholder' => esc_attr__( 'Agree to Terms?', 'give' ),
+								'placeholder' => esc_attr__('Agree to Terms?', 'give'),
 							),
 							'type'       => 'text',
 						),
 						array(
-							'name' => __( 'Agreement Text', 'give' ),
-							'desc' => __( 'This is the actual text which the user will be asked to agree to in order to donate. Note: You can customize the content per form as needed.', 'give' ),
+							'name' => __('Agreement Text', 'give'),
+							'desc' => __('This is the actual text which the user will be asked to agree to in order to donate. Note: You can customize the content per form as needed.', 'give'),
 							'id'   => 'agreement_text',
 							'type' => 'wysiwyg',
 						),
 						array(
-							'name'  => __( 'Terms and Conditions Docs Link', 'give' ),
+							'name'  => __('Terms and Conditions Docs Link', 'give'),
 							'id'    => 'terms_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-terms' ),
-							'title' => __( 'Terms and Conditions Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-terms'),
+							'title' => __('Terms and Conditions Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -292,7 +292,7 @@ 
 			 * Filter the display options settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_display', $settings );
+			$settings = apply_filters('give_settings_display', $settings);
 
 			/**
 			 * Filter the settings.
@@ -301,7 +301,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -315,13 +315,13 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'display-settings'    => __( 'Display', 'give' ),
-				'post-types'          => __( 'Post Types', 'give' ),
-				'taxonomies'          => __( 'Taxonomies', 'give' ),
-				'term-and-conditions' => __( 'Terms and Conditions', 'give' ),
+				'display-settings'    => __('Display', 'give'),
+				'post-types'          => __('Post Types', 'give'),
+				'taxonomies'          => __('Taxonomies', 'give'),
+				'term-and-conditions' => __('Terms and Conditions', 'give'),
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/admin/settings/class-settings-cmb2-backward-compatibility.php

Spacing +125 added lines, -125 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.8
  */
 
-if ( ! class_exists( 'Give_CMB2_Settings_Loader' ) ) :
+if ( ! class_exists('Give_CMB2_Settings_Loader')) :
 
 	/**
 	 * This class loads the cmb2 settings.
@@ -52,30 +52,30 @@ 
 
 			// Get current tab.
 			$this->current_tab     = give_get_current_setting_tab();
-			$this->current_section = empty( $_REQUEST['section'] ) ? ( current( array_keys( $this->get_sections() ) ) ) : sanitize_title( $_REQUEST['section'] );
+			$this->current_section = empty($_REQUEST['section']) ? (current(array_keys($this->get_sections()))) : sanitize_title($_REQUEST['section']);
 
 			// Tab ID.
 			$this->id = $this->current_tab;
 
 			// Add addon tabs.
-			add_filter( 'give-settings_tabs_array', array( $this, 'add_addon_settings_page' ), 999999 );
+			add_filter('give-settings_tabs_array', array($this, 'add_addon_settings_page'), 999999);
 
 			// Add save hook to addons.
-			add_action( 'give-settings_get_settings_pages', array( $this, 'setup_addon_save_hook' ), 999999 );
+			add_action('give-settings_get_settings_pages', array($this, 'setup_addon_save_hook'), 999999);
 
 			// Add backward compatibility filters plugin settings.
-			$setting_tabs = array( 'general', 'gateways', 'display', 'emails', 'addons', 'licenses' );
+			$setting_tabs = array('general', 'gateways', 'display', 'emails', 'addons', 'licenses');
 
 			// Filter Payment Gateways settings.
-			if ( in_array( $this->current_tab, $setting_tabs ) ) {
-				add_filter( "give_get_settings_{$this->current_tab}", array(
+			if (in_array($this->current_tab, $setting_tabs)) {
+				add_filter("give_get_settings_{$this->current_tab}", array(
 					$this,
 					'get_filtered_addon_settings',
-				), 999999, 1 );
-				add_filter( "give_get_sections_{$this->current_tab}", array(
+				), 999999, 1);
+				add_filter("give_get_sections_{$this->current_tab}", array(
 					$this,
 					'get_filtered_addon_sections',
-				), 999999, 1 );
+				), 999999, 1);
 			}
 		}
 
@@ -88,12 +88,12 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			$default_tab = '';
 
 			// Set default tab to first setting tab.
-			if ( $sections = array_keys( $this->get_sections() ) ) {
-				$default_tab = current( $sections );
+			if ($sections = array_keys($this->get_sections())) {
+				$default_tab = current($sections);
 			}
 
 			return $default_tab;
@@ -108,29 +108,29 @@ 
 		 *
 		 * @return mixed
 		 */
-		function add_addon_settings_page( $pages ) {
+		function add_addon_settings_page($pages) {
 			// Previous setting page.
 			$previous_pages = $this->prev_settings->give_get_settings_tabs();
 
 			// API and System Info setting tab merge to Tools setting tab, so remove them from tabs.
-			unset( $previous_pages['api'] );
-			unset( $previous_pages['system_info'] );
+			unset($previous_pages['api']);
+			unset($previous_pages['system_info']);
 
 			// Tab is not register.
-			$pages_diff = array_keys( array_diff( $previous_pages, $pages ) );
+			$pages_diff = array_keys(array_diff($previous_pages, $pages));
 
 			// Merge old settings with new settings.
-			$pages = array_merge( $pages, $previous_pages );
+			$pages = array_merge($pages, $previous_pages);
 
-			if ( in_array( $this->current_tab, $pages_diff ) ) {
+			if (in_array($this->current_tab, $pages_diff)) {
 				// Filter & actions.
-				add_filter( "give_default_setting_tab_section_{$this->current_tab}", array(
+				add_filter("give_default_setting_tab_section_{$this->current_tab}", array(
 					$this,
 					'set_default_setting_tab',
-				), 10 );
-				add_action( "give-settings_sections_{$this->current_tab}_page", array( $this, 'output_sections' ) );
-				add_action( "give-settings_settings_{$this->current_tab}_page", array( $this, 'output' ), 10 );
-				add_action( "give-settings_save_{$this->current_tab}", array( $this, 'save' ) );
+				), 10);
+				add_action("give-settings_sections_{$this->current_tab}_page", array($this, 'output_sections'));
+				add_action("give-settings_settings_{$this->current_tab}_page", array($this, 'output'), 10);
+				add_action("give-settings_save_{$this->current_tab}", array($this, 'save'));
 			}
 
 			return $pages;
@@ -146,29 +146,29 @@ 
 		 *
 		 * @return mixed
 		 */
-		function setup_addon_save_hook( $pages ) {
+		function setup_addon_save_hook($pages) {
 			$page_ids = array();
 
-			foreach ( $pages as $page ) {
-				$page_ids = $page->add_settings_page( $page_ids );
+			foreach ($pages as $page) {
+				$page_ids = $page->add_settings_page($page_ids);
 			}
 
 			// Previous setting page.
 			$previous_pages = $this->prev_settings->give_get_settings_tabs();
 
 			// API and System Info setting tab merge to Tools setting tab, so remove them from tabs.
-			unset( $previous_pages['api'] );
-			unset( $previous_pages['system_info'] );
+			unset($previous_pages['api']);
+			unset($previous_pages['system_info']);
 
 			// Tab is not register.
-			$pages_diff = array_keys( array_diff( $previous_pages, $page_ids ) );
+			$pages_diff = array_keys(array_diff($previous_pages, $page_ids));
 
 			// Merge old settings with new settings.
 			// $pages = array_merge( $page_ids, $previous_pages );
 
-			if ( in_array( $this->current_tab, $pages_diff ) ) {
+			if (in_array($this->current_tab, $pages_diff)) {
 				// Filter & actions.
-				add_action( "give-settings_save_{$this->current_tab}", array( $this, 'save' ) );
+				add_action("give-settings_save_{$this->current_tab}", array($this, 'save'));
 			}
 
 			return $pages;
@@ -183,16 +183,16 @@ 
 		 *
 		 * @return string
 		 */
-		function get_section_name( $field_name ) {
+		function get_section_name($field_name) {
 			// Bailout.
-			if ( empty( $field_name ) ) {
+			if (empty($field_name)) {
 				return $field_name;
 			}
 
-			$section_name = explode( ' ', $field_name );
+			$section_name = explode(' ', $field_name);
 
 			// Output.
-			return strip_tags( implode( ' ', $section_name ) );
+			return strip_tags(implode(' ', $section_name));
 		}
 
 
@@ -207,7 +207,7 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function en_translation( $translation, $text ) {
+		public function en_translation($translation, $text) {
 			return $text;
 		}
 
@@ -221,52 +221,52 @@ 
 		 *
 		 * @return mixed
 		 */
-		function get_filtered_addon_sections( $sections = array() ) {
+		function get_filtered_addon_sections($sections = array()) {
 			// New sections.
 			$new_sections = array();
-			$sections_ID  = array_keys( $sections );
-			$setting_fields = $this->prev_settings->give_settings( $this->current_tab );
+			$sections_ID  = array_keys($sections);
+			$setting_fields = $this->prev_settings->give_settings($this->current_tab);
 
 			// We need untranslated settings for backward compatibility.
-			add_filter( 'gettext', array( $this, 'en_translation' ), 10, 2 );
-			$en_setting_fields = $this->prev_settings->give_settings( $this->current_tab );
-			remove_filter( 'gettext', array( $this, 'en_translation' ), 10, 2 );
+			add_filter('gettext', array($this, 'en_translation'), 10, 2);
+			$en_setting_fields = $this->prev_settings->give_settings($this->current_tab);
+			remove_filter('gettext', array($this, 'en_translation'), 10, 2);
 			
-			if ( ! empty( $setting_fields ) && ! empty( $setting_fields['fields'] ) ) {
+			if ( ! empty($setting_fields) && ! empty($setting_fields['fields'])) {
 
-				foreach ( $setting_fields['fields'] as $index => $field ) {
+				foreach ($setting_fields['fields'] as $index => $field) {
 					// Collect new sections from addons.
-					if ( 'give_title' !== $field['type'] ) {
+					if ('give_title' !== $field['type']) {
 						continue;
 					}
 
 					// Untranslated setting name.
-					$en_setting_field_name = isset( $en_setting_fields['fields'][ $index ]['name'] ) ? $en_setting_fields['fields'][ $index ]['name'] : '';
+					$en_setting_field_name = isset($en_setting_fields['fields'][$index]['name']) ? $en_setting_fields['fields'][$index]['name'] : '';
 
 					// Section name.
-					$field['name'] = isset( $field['name'] ) ? $field['name'] : '';
-					$section_name  = $this->get_section_name( $field['name'] );
+					$field['name'] = isset($field['name']) ? $field['name'] : '';
+					$section_name  = $this->get_section_name($field['name']);
 
 					// Check if section name exit and section title array is not empty.
-					if ( ! empty( $sections ) && ! empty( $en_setting_field_name ) ) {
+					if ( ! empty($sections) && ! empty($en_setting_field_name)) {
 
 						// Bailout: Do not load section if it is already exist.
 						if (
-							in_array( sanitize_title( $en_setting_field_name ), $sections_ID ) // Check section id.
-							|| in_array( $section_name, $sections )                            // Check section name.
+							in_array(sanitize_title($en_setting_field_name), $sections_ID) // Check section id.
+							|| in_array($section_name, $sections)                            // Check section name.
 						) {
 							continue;
 						}
 					}
 
 					// Collect new sections from addons.
-					$new_sections[ sanitize_title( $field['name'] ) ] = $section_name;
+					$new_sections[sanitize_title($field['name'])] = $section_name;
 				}
 			}
 
 			// Add new section.
-			if ( ! empty( $new_sections ) ) {
-				$sections = array_merge( $sections, $new_sections );
+			if ( ! empty($new_sections)) {
+				$sections = array_merge($sections, $new_sections);
 			}
 
 			// Output.
@@ -284,19 +284,19 @@ 
 		 *
 		 * @return array
 		 */
-		function get_filtered_addon_settings( $settings, $setting_fields = array() ) {
+		function get_filtered_addon_settings($settings, $setting_fields = array()) {
 			global $wp_filter;
 
 			$new_setting_fields = array();
 
-			if ( ! empty( $settings ) ) {
+			if ( ! empty($settings)) {
 				// Bailout: If setting array contain first element of type title then it means it is already created with new setting api (skip this section ).
-				if ( isset( $settings[0]['type'] ) && 'title' == $settings[0]['type'] ) {
-					foreach ( $settings as $setting ) {
+				if (isset($settings[0]['type']) && 'title' == $settings[0]['type']) {
+					foreach ($settings as $setting) {
 						$new_setting_fields[] = $setting;
 
 						// We need setting only till first section end.
-						if ( 'sectionend' === $setting['type'] ) {
+						if ('sectionend' === $setting['type']) {
 							break;
 						}
 					}
@@ -308,24 +308,24 @@ 
 				$prev_title_field_id = '';
 
 				// Create new setting fields.
-				foreach ( $settings as $index => $field ) {
+				foreach ($settings as $index => $field) {
 
 					// Bailout: Must need field type to process.
-					if ( ! isset( $field['type'] ) ) {
+					if ( ! isset($field['type'])) {
 						continue;
 					}
 
 					// Set wrapper class if any.
-					if ( ! empty( $field['row_classes'] ) ) {
+					if ( ! empty($field['row_classes'])) {
 						$field['wrapper_class'] = $field['row_classes'];
-						unset( $field['row_classes'] );
+						unset($field['row_classes']);
 					}
 
-					$field['name'] = ! isset( $field['name'] ) ? '' : $field['name'];
-					$field['desc'] = ! isset( $field['desc'] ) ? '' : $field['desc'];
+					$field['name'] = ! isset($field['name']) ? '' : $field['name'];
+					$field['desc'] = ! isset($field['desc']) ? '' : $field['desc'];
 
 					// Modify cmb2 setting fields.
-					switch ( $field['type'] ) {
+					switch ($field['type']) {
 						case 'text' :
 						case 'file' :
 							$field['css'] = 'width:25em;';
@@ -349,18 +349,18 @@ 
 							$field['type'] = 'title';
 
 							// Since we are showing sections, so there now ned to show horizontal rules.
-							if ( '<hr>' === $field['desc'] ) {
+							if ('<hr>' === $field['desc']) {
 								$field['desc'] = '';
 							}
 
 							break;
 					}
 
-					if ( 'title' === $field['type'] ) {
+					if ('title' === $field['type']) {
 
 						// If we do not have first element as title then these field will be skip from frontend
 						// because there are not belong to any section, so put all abandon fields under first section.
-						if ( $index && empty( $prev_title_field_id ) ) {
+						if ($index && empty($prev_title_field_id)) {
 							array_unshift(
 								$new_setting_fields,
 								array(
@@ -374,7 +374,7 @@ 
 							$prev_title_field_id = $field['id'];
 
 							continue;
-						} elseif ( $index ) {
+						} elseif ($index) {
 							// Section end.
 							$new_setting_fields[] = array(
 								'type' => 'sectionend',
@@ -406,52 +406,52 @@ 
 
 				// Check if setting page has title section or not.
 				// If setting page does not have title section  then add title section to it and fix section end array id.
-				if ( 'title' !== $new_setting_fields[0]['type'] ) {
+				if ('title' !== $new_setting_fields[0]['type']) {
 					array_unshift(
 						$new_setting_fields,
 						array(
-							'title' => ( isset( $settings['give_title'] ) ? $settings['give_title'] : '' ),
+							'title' => (isset($settings['give_title']) ? $settings['give_title'] : ''),
 							'type'  => 'title',
-							'desc'  => ! empty( $setting_fields['desc'] ) ? $setting_fields['desc'] : '',
-							'id'    => ( isset( $settings['id'] ) ? $settings['id'] : '' ),
+							'desc'  => ! empty($setting_fields['desc']) ? $setting_fields['desc'] : '',
+							'id'    => (isset($settings['id']) ? $settings['id'] : ''),
 						)
 					);
 
 					// Update id in section end array if does not contain.
-					if ( empty( $new_setting_fields[ count( $new_setting_fields ) - 1 ]['id'] ) ) {
-						$new_setting_fields[ count( $new_setting_fields ) - 1 ]['id'] = ( isset( $settings['id'] ) ? $settings['id'] : '' );
+					if (empty($new_setting_fields[count($new_setting_fields) - 1]['id'])) {
+						$new_setting_fields[count($new_setting_fields) - 1]['id'] = (isset($settings['id']) ? $settings['id'] : '');
 					}
 				}
 
 				// Return only section related settings.
-				if ( $sections = $this->get_filtered_addon_sections() ) {
-					$new_setting_fields = $this->get_section_settings( $new_setting_fields );
+				if ($sections = $this->get_filtered_addon_sections()) {
+					$new_setting_fields = $this->get_section_settings($new_setting_fields);
 				}
 
 				// Third party plugin backward compatibility.
-				$wp_filter_keys = array_keys( $wp_filter );
-				foreach ( $new_setting_fields as $index => $field ) {
-					if ( ! isset( $field['type'] ) || in_array( $field['type'], array( 'title', 'sectionend' ) ) ) {
+				$wp_filter_keys = array_keys($wp_filter);
+				foreach ($new_setting_fields as $index => $field) {
+					if ( ! isset($field['type']) || in_array($field['type'], array('title', 'sectionend'))) {
 						continue;
 					}
 
 					$cmb2_filter_name = "cmb2_render_{$field['type']}";
 
-					if ( in_array( $cmb2_filter_name, $wp_filter_keys ) ) {
+					if (in_array($cmb2_filter_name, $wp_filter_keys)) {
 
-						if ( 0 >= version_compare( 4.7, get_bloginfo( 'version' ) ) && ! empty( $wp_filter[ $cmb2_filter_name ]->callbacks ) ) {
-							$cmb2_filter_arr = current( $wp_filter[ $cmb2_filter_name ]->callbacks );
+						if (0 >= version_compare(4.7, get_bloginfo('version')) && ! empty($wp_filter[$cmb2_filter_name]->callbacks)) {
+							$cmb2_filter_arr = current($wp_filter[$cmb2_filter_name]->callbacks);
 						} else {
-							$cmb2_filter_arr = current( $wp_filter[ $cmb2_filter_name ] );
+							$cmb2_filter_arr = current($wp_filter[$cmb2_filter_name]);
 						}
 
-						if ( ! empty( $cmb2_filter_arr ) ) {
+						if ( ! empty($cmb2_filter_arr)) {
 							// Note: function can be called either globally or with class object, it depends on how developer invoke it.
-							$new_setting_fields[ $index ]['func'] = current( $cmb2_filter_arr );
-							add_action( "give_admin_field_{$field['type']}", array(
+							$new_setting_fields[$index]['func'] = current($cmb2_filter_arr);
+							add_action("give_admin_field_{$field['type']}", array(
 								$this,
 								'addon_setting_field',
-							), 10, 2 );
+							), 10, 2);
 						}
 					}
 				}
@@ -472,31 +472,31 @@ 
 		 *
 		 * @return array
 		 */
-		function get_section_settings( $tab_settings ) {
+		function get_section_settings($tab_settings) {
 			$current_section = give_get_current_setting_section();
 
 			// Note: If we are opening default tab for addon setting then it is possible that we will get empty string as current section
 			// because default section filter added after save hook fire, so we will always get problem to save first section [default] or if there are only on section
 			// This is hack to fix this.
-			if ( empty( $current_section ) ) {
-				$current_section = $this->set_default_setting_tab( $current_section );
+			if (empty($current_section)) {
+				$current_section = $this->set_default_setting_tab($current_section);
 			}
 
 			$section_start               = false;
 			$section_end                 = false;
 			$section_only_setting_fields = array();
 
-			foreach ( $tab_settings as $field ) {
-				if ( 'title' == $field['type'] && $current_section == sanitize_title( $field['title'] ) ) {
+			foreach ($tab_settings as $field) {
+				if ('title' == $field['type'] && $current_section == sanitize_title($field['title'])) {
 					$section_start = true;
 				}
 
-				if ( ! $section_start || $section_end ) {
+				if ( ! $section_start || $section_end) {
 					continue;
 				}
 
-				if ( $section_start && ! $section_end ) {
-					if ( 'sectionend' == $field['type'] ) {
+				if ($section_start && ! $section_end) {
+					if ('sectionend' == $field['type']) {
 						$section_end = true;
 					}
 					$section_only_setting_fields[] = $field;
@@ -506,7 +506,7 @@ 
 			// Remove title from setting, prevent it from render in setting tab.
 			$section_only_setting_fields[0]['title'] = '';
 
-			return apply_filters( "give_get_settings_{$this->current_tab}_{$current_section}", $section_only_setting_fields, $tab_settings );
+			return apply_filters("give_get_settings_{$this->current_tab}_{$current_section}", $section_only_setting_fields, $tab_settings);
 		}
 
 
@@ -520,17 +520,17 @@ 
 		 *
 		 * @return void
 		 */
-		function addon_setting_field( $field, $saved_value ) {
+		function addon_setting_field($field, $saved_value) {
 			// Create object for cmb2  function callback backward compatibility.
 			// Note: Do not call any cmb2 function on these objects
-			$field_obj      = (object) array( 'args' => $field );
-			$field_type_obj = (object) array( 'field' => $field_obj );
+			$field_obj      = (object) array('args' => $field);
+			$field_type_obj = (object) array('field' => $field_obj);
 
-			switch ( $this->current_tab ) :
+			switch ($this->current_tab) :
 				case 'licenses':
 					?>
 					<div class="give-settings-wrap give-settings-wrap-<?php echo $this->current_tab; ?>">
-						<?php $field['func']['function']( $field_obj, $saved_value, '', '', $field_type_obj ); ?>
+						<?php $field['func']['function']($field_obj, $saved_value, '', '', $field_type_obj); ?>
 					</div>
 					<?php break;
 
@@ -538,22 +538,22 @@ 
 					$colspan = 'colspan="2"';
 					?>
 					<tr valign="top">
-						<?php if ( ! empty( $field['name'] ) && ! in_array( $field['name'], array( '&nbsp;' ) ) ) : ?>
+						<?php if ( ! empty($field['name']) && ! in_array($field['name'], array('&nbsp;'))) : ?>
 							<th scope="row" class="titledesc">
 								<label
-										for="<?php echo esc_attr( $field['name'] ); ?>"><?php echo $field['title']; ?></label>
+										for="<?php echo esc_attr($field['name']); ?>"><?php echo $field['title']; ?></label>
 							</th>
 							<?php $colspan = ''; ?>
 						<?php endif; ?>
 						<td class="give-forminp" <?php echo $colspan; ?>>
 							<?php
-							if ( is_array( $field['func']['function'] ) ) {
+							if (is_array($field['func']['function'])) {
 								$classname = $field['func']['function'][0];
 								$function_name = $field['func']['function'][1];
-								$classname->$function_name( $field_obj, $saved_value, '', '', $field_type_obj );
+								$classname->$function_name($field_obj, $saved_value, '', '', $field_type_obj);
 							} else {
 								$function_name = $field['func']['function'];
-								$function_name( $field_obj, $saved_value, '', '', $field_type_obj );
+								$function_name($field_obj, $saved_value, '', '', $field_type_obj);
 							}
 							?>
 						</td>
@@ -571,10 +571,10 @@ 
 		public function get_sections() {
 			$sections = array();
 
-			if ( ( $setting_fields = $this->prev_settings->give_settings( $this->current_tab ) ) && ! empty( $setting_fields['fields'] ) ) {
-				foreach ( $setting_fields['fields'] as $field ) {
-					if ( 'give_title' == $field['type'] ) {
-						$sections[ sanitize_title( $field['name'] ) ] = $this->get_section_name( $field['name'] );
+			if (($setting_fields = $this->prev_settings->give_settings($this->current_tab)) && ! empty($setting_fields['fields'])) {
+				foreach ($setting_fields['fields'] as $field) {
+					if ('give_title' == $field['type']) {
+						$sections[sanitize_title($field['name'])] = $this->get_section_name($field['name']);
 					}
 				}
 			}
@@ -594,16 +594,16 @@ 
 
 			$new_setting_fields = array();
 
-			if ( $setting_fields = $this->prev_settings->give_settings( $this->current_tab ) ) {
-				if ( isset( $setting_fields['fields'] ) ) {
+			if ($setting_fields = $this->prev_settings->give_settings($this->current_tab)) {
+				if (isset($setting_fields['fields'])) {
 
 					$tab_data = array(
 						'id'         => $setting_fields['id'],
 						'give_title' => $setting_fields['give_title'],
-						'desc'       => ( isset( $setting_fields['desc'] ) ? $setting_fields['desc'] : '' ),
+						'desc'       => (isset($setting_fields['desc']) ? $setting_fields['desc'] : ''),
 					);
 
-					$new_setting_fields = $this->get_filtered_addon_settings( $setting_fields['fields'], $tab_data );
+					$new_setting_fields = $this->get_filtered_addon_settings($setting_fields['fields'], $tab_data);
 				}
 			}
 
@@ -620,24 +620,24 @@ 
 			$sections = $this->get_sections();
 
 			// Show section settings only if setting section exist.
-			if ( $this->current_section && ! in_array( $this->current_section, array_keys( $sections ) ) ) {
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+			if ($this->current_section && ! in_array($this->current_section, array_keys($sections))) {
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
 			echo '<ul class="subsubsub">';
 
-			$array_keys = array_keys( $sections );
+			$array_keys = array_keys($sections);
 
-			foreach ( $sections as $id => $label ) {
-				echo '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=' . $this->current_tab . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $this->current_section == $id ? 'current' : '' ) . '">' . strip_tags( $label ) . '</a> ' . ( end( $array_keys ) == $id ? '' : '|' ) . ' </li>';
+			foreach ($sections as $id => $label) {
+				echo '<li><a href="'.admin_url('edit.php?post_type=give_forms&page=give-settings&tab='.$this->current_tab.'&section='.sanitize_title($id)).'" class="'.($this->current_section == $id ? 'current' : '').'">'.strip_tags($label).'</a> '.(end($array_keys) == $id ? '' : '|').' </li>';
 			}
 
 			echo '</ul><br class="clear" /><hr>';
@@ -652,7 +652,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -664,7 +664,7 @@ 
 		public function save() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
 		}
 	}
 endif;

includes/admin/settings/class-settings-general.php

Spacing +68 added lines, -68 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
 
-if ( ! class_exists( 'Give_Settings_General' ) ) :
+if ( ! class_exists('Give_Settings_General')) :
 
 	/**
 	 * Give_Settings_General.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'general';
-			$this->label = __( 'General', 'give' );
+			$this->label = __('General', 'give');
 
 			$this->default_tab = 'general-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings        = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'access-control':
 					$settings = array(
 						// Section 3: Access control.
@@ -54,20 +54,20 @@ 
 						),
 						array(
 							'id'      => 'session_lifetime',
-							'name'    => __( 'Session Lifetime', 'give' ),
-							'desc'    => __( 'The length of time a user\'s session is kept alive. Give starts a new session per user upon donation. Sessions allow donors to view their donation receipts without being logged in.', 'give' ),
+							'name'    => __('Session Lifetime', 'give'),
+							'desc'    => __('The length of time a user\'s session is kept alive. Give starts a new session per user upon donation. Sessions allow donors to view their donation receipts without being logged in.', 'give'),
 							'type'    => 'select',
 							'options' => array(
-								'86400'  => __( '24 Hours', 'give' ),
-								'172800' => __( '48 Hours', 'give' ),
-								'259200' => __( '72 Hours', 'give' ),
-								'604800' => __( '1 Week', 'give' ),
+								'86400'  => __('24 Hours', 'give'),
+								'172800' => __('48 Hours', 'give'),
+								'259200' => __('72 Hours', 'give'),
+								'604800' => __('1 Week', 'give'),
 							),
 						),
 						array(
 							'id'         => 'limit_display_donations',
-							'name'       => __( 'Limit Donations Displayed', 'give' ),
-							'desc'       => __( 'Adjusts the number of donations displayed to a non logged-in user when they attempt to access the Donation History page without an active session. For security reasons, it\'s best to leave this at 1-3 donations.', 'give' ),
+							'name'       => __('Limit Donations Displayed', 'give'),
+							'desc'       => __('Adjusts the number of donations displayed to a non logged-in user when they attempt to access the Donation History page without an active session. For security reasons, it\'s best to leave this at 1-3 donations.', 'give'),
 							'default'    => '1',
 							'type'       => 'number',
 							'css'        => 'width:50px;',
@@ -77,36 +77,36 @@ 
 							),
 						),
 						array(
-							'name'    => __( 'Email Access', 'give' ),
-							'desc'    => __( 'Would you like your donors to be able to access their donation history using only email? Donors whose sessions have expired and do not have an account may still access their donation history via a temporary email access link.', 'give' ),
+							'name'    => __('Email Access', 'give'),
+							'desc'    => __('Would you like your donors to be able to access their donation history using only email? Donors whose sessions have expired and do not have an account may still access their donation history via a temporary email access link.', 'give'),
 							'id'      => 'email_access',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
 							'id'      => 'recaptcha_key',
-							'name'    => __( 'reCAPTCHA Site Key', 'give' ),
+							'name'    => __('reCAPTCHA Site Key', 'give'),
 							/* translators: %s: https://www.google.com/recaptcha/ */
-							'desc'    => sprintf( __( 'If you would like to prevent spam on the email access form navigate to <a href="%s" target="_blank">the reCAPTCHA website</a> and sign up for an API key and paste your reCAPTCHA site key here. The reCAPTCHA uses Google\'s user-friendly single click verification method.', 'give' ), esc_url( 'http://docs.givewp.com/recaptcha' ) ),
+							'desc'    => sprintf(__('If you would like to prevent spam on the email access form navigate to <a href="%s" target="_blank">the reCAPTCHA website</a> and sign up for an API key and paste your reCAPTCHA site key here. The reCAPTCHA uses Google\'s user-friendly single click verification method.', 'give'), esc_url('http://docs.givewp.com/recaptcha')),
 							'default' => '',
 							'type'    => 'text',
 						),
 						array(
 							'id'      => 'recaptcha_secret',
-							'name'    => __( 'reCAPTCHA Secret Key', 'give' ),
-							'desc'    => __( 'Please paste the reCAPTCHA secret key here from your  reCAPTCHA API Keys panel.', 'give' ),
+							'name'    => __('reCAPTCHA Secret Key', 'give'),
+							'desc'    => __('Please paste the reCAPTCHA secret key here from your  reCAPTCHA API Keys panel.', 'give'),
 							'default' => '',
 							'type'    => 'text',
 						),
 						array(
-							'name'  => __( 'Access Control Docs Link', 'give' ),
+							'name'  => __('Access Control Docs Link', 'give'),
 							'id'    => 'access_control_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-access-control' ),
-							'title' => __( 'Access Control', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-access-control'),
+							'title' => __('Access Control', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -124,14 +124,14 @@ 
 							'id'   => 'give_title_general_settings_2',
 						),
 						array(
-							'name' => __( 'Currency Settings', 'give' ),
+							'name' => __('Currency Settings', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_general_settings_2',
 						),
 						array(
-							'name'    => __( 'Currency', 'give' ),
-							'desc'    => __( 'The donation currency. Note that some payment gateways have currency restrictions.', 'give' ),
+							'name'    => __('Currency', 'give'),
+							'desc'    => __('The donation currency. Note that some payment gateways have currency restrictions.', 'give'),
 							'id'      => 'currency',
 							'class'   => 'give-select-chosen',
 							'type'    => 'select',
@@ -139,47 +139,47 @@ 
 							'default' => 'USD',
 						),
 						array(
-							'name'    => __( 'Currency Position', 'give' ),
-							'desc'    => __( 'The position of the currency symbol.', 'give' ),
+							'name'    => __('Currency Position', 'give'),
+							'desc'    => __('The position of the currency symbol.', 'give'),
 							'id'      => 'currency_position',
 							'type'    => 'select',
 							'options' => array(
 								/* translators: %s: currency symbol */
-								'before' => sprintf( __( 'Before - %s&#x200e;10', 'give' ), give_currency_symbol( give_get_currency() ) ),
+								'before' => sprintf(__('Before - %s&#x200e;10', 'give'), give_currency_symbol(give_get_currency())),
 								/* translators: %s: currency symbol */
-								'after'  => sprintf( __( 'After - 10%s&#x200f;', 'give' ), give_currency_symbol( give_get_currency() ) ),
+								'after'  => sprintf(__('After - 10%s&#x200f;', 'give'), give_currency_symbol(give_get_currency())),
 							),
 							'default' => 'before',
 						),
 						array(
-							'name'    => __( 'Thousands Separator', 'give' ),
-							'desc'    => __( 'The symbol (typically , or .) to separate thousands.', 'give' ),
+							'name'    => __('Thousands Separator', 'give'),
+							'desc'    => __('The symbol (typically , or .) to separate thousands.', 'give'),
 							'id'      => 'thousands_separator',
 							'type'    => 'text',
 							'default' => ',',
 							'css'     => 'width:12em;',
 						),
 						array(
-							'name'    => __( 'Decimal Separator', 'give' ),
-							'desc'    => __( 'The symbol (usually , or .) to separate decimal points.', 'give' ),
+							'name'    => __('Decimal Separator', 'give'),
+							'desc'    => __('The symbol (usually , or .) to separate decimal points.', 'give'),
 							'id'      => 'decimal_separator',
 							'type'    => 'text',
 							'default' => '.',
 							'css'     => 'width:12em;',
 						),
 						array(
-							'name'    => __( 'Number of Decimals', 'give' ),
-							'desc'    => __( 'The number of decimal points displayed in amounts.', 'give' ),
+							'name'    => __('Number of Decimals', 'give'),
+							'desc'    => __('The number of decimal points displayed in amounts.', 'give'),
 							'id'      => 'number_decimals',
 							'type'    => 'text',
 							'default' => 2,
 							'css'     => 'width:12em;',
 						),
 						array(
-							'name'  => __( 'Currency Options Docs Link', 'give' ),
+							'name'  => __('Currency Options Docs Link', 'give'),
 							'id'    => 'currency_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-currency' ),
-							'title' => __( 'Currency Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-currency'),
+							'title' => __('Currency Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -194,7 +194,7 @@ 
 					$country = give_get_country();
 
 					// get the list of the states of which default country is selected.
-					$states = give_get_states( $country );
+					$states = give_get_states($country);
 
 					// Get the country list that does not have any states init.
 					$no_states_country = give_no_states_country_list();
@@ -206,46 +206,46 @@ 
 							'id'   => 'give_title_general_settings_1',
 						),
 						array(
-							'name' => __( 'General Settings', 'give' ),
+							'name' => __('General Settings', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_general_settings_1',
 						),
 						array(
-							'name'    => __( 'Success Page', 'give' ),
+							'name'    => __('Success Page', 'give'),
 							/* translators: %s: [give_receipt] */
-							'desc'    => sprintf( __( 'The page donors are sent to after completing their donations. The %s shortcode should be on this page.', 'give' ), '<code>[give_receipt]</code>' ),
+							'desc'    => sprintf(__('The page donors are sent to after completing their donations. The %s shortcode should be on this page.', 'give'), '<code>[give_receipt]</code>'),
 							'id'      => 'success_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Failed Donation Page', 'give' ),
-							'desc'    => __( 'The page donors are sent to if their donation is cancelled or fails.', 'give' ),
+							'name'    => __('Failed Donation Page', 'give'),
+							'desc'    => __('The page donors are sent to if their donation is cancelled or fails.', 'give'),
 							'id'      => 'failure_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Donation History Page', 'give' ),
+							'name'    => __('Donation History Page', 'give'),
 							/* translators: %s: [donation_history] */
-							'desc'    => sprintf( __( 'The page showing a complete donation history for the current user. The %s shortcode should be on this page.', 'give' ), '<code>[donation_history]</code>' ),
+							'desc'    => sprintf(__('The page showing a complete donation history for the current user. The %s shortcode should be on this page.', 'give'), '<code>[donation_history]</code>'),
 							'id'      => 'history_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Base Country', 'give' ),
-							'desc'    => __( 'The country your site operates from.', 'give' ),
+							'name'    => __('Base Country', 'give'),
+							'desc'    => __('The country your site operates from.', 'give'),
 							'id'      => 'base_country',
 							'type'    => 'select',
 							'options' => give_get_country_list(),
@@ -256,18 +256,18 @@ 
 						 * @since 1.8.14
 						 */
 						array(
-							'wrapper_class' => ( array_key_exists( $country, $no_states_country ) ? 'give-hidden' : '' ),
-							'name'          => __( 'Base State/Province', 'give' ),
-							'desc'          => __( 'The state/province your site operates from.', 'give' ),
+							'wrapper_class' => (array_key_exists($country, $no_states_country) ? 'give-hidden' : ''),
+							'name'          => __('Base State/Province', 'give'),
+							'desc'          => __('The state/province your site operates from.', 'give'),
 							'id'            => 'base_state',
-							'type'          => ( empty( $states ) ? 'text' : 'select' ),
+							'type'          => (empty($states) ? 'text' : 'select'),
 							'options'       => $states,
 						),
 						array(
-							'name'  => __( 'General Options Docs Link', 'give' ),
+							'name'  => __('General Options Docs Link', 'give'),
 							'id'    => 'general_options_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-general' ),
-							'title' => __( 'General Options', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-general'),
+							'title' => __('General Options', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -282,7 +282,7 @@ 
 			 * Filter the general settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_general', $settings );
+			$settings = apply_filters('give_settings_general', $settings);
 
 			/**
 			 * Filter the settings.
@@ -291,7 +291,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -305,12 +305,12 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'general-settings'  => __( 'General', 'give' ),
-				'currency-settings' => __( 'Currency', 'give' ),
-				'access-control'    => __( 'Access Control', 'give' ),
+				'general-settings'  => __('General', 'give'),
+				'currency-settings' => __('Currency', 'give'),
+				'access-control'    => __('Access Control', 'give'),
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/filters.php

Spacing +43 added lines, -45 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,34 +26,34 @@ 
  *
  * @return void
  */
-function give_set_settings_with_disable_prefix( $old_settings, $settings ) {
+function give_set_settings_with_disable_prefix($old_settings, $settings) {
 	// Bailout.
-	if ( ! function_exists( 'give_v18_renamed_core_settings' ) ) {
+	if ( ! function_exists('give_v18_renamed_core_settings')) {
 		return;
 	}
 
 	// Get old setting names.
-	$old_settings   = array_flip( give_v18_renamed_core_settings() );
+	$old_settings   = array_flip(give_v18_renamed_core_settings());
 	$update_setting = false;
 
-	foreach ( $settings as $key => $value ) {
+	foreach ($settings as $key => $value) {
 
 		// Check 1. Check if new option is really updated or not.
 		// Check 2. Continue if key is not renamed.
-		if ( ! isset( $old_settings[ $key ] ) ) {
+		if ( ! isset($old_settings[$key])) {
 			continue;
 		}
 
 		// Set old setting.
-		$settings[ $old_settings[ $key ] ] = 'on';
+		$settings[$old_settings[$key]] = 'on';
 
 		// Do not need to set old setting if new setting is not set.
 		if (
-			( give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'disable_' ) ) )
-			|| ( ! give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'enable_' ) ) )
+			(give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'disable_')))
+			|| ( ! give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'enable_')))
 
 		) {
-			unset( $settings[ $old_settings[ $key ] ] );
+			unset($settings[$old_settings[$key]]);
 		}
 
 		// Tell bot to update setting.
@@ -61,12 +61,12 @@ 
 	}
 
 	// Update setting if any old setting set.
-	if ( $update_setting ) {
-		update_option( 'give_settings', $settings );
+	if ($update_setting) {
+		update_option('give_settings', $settings);
 	}
 }
 
-add_action( 'update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2 );
+add_action('update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2);
 
 /**
  * Check spam through Akismet.
@@ -80,45 +80,45 @@ 
  *
  * @return bool|mixed
  */
-function give_akismet( $spam ) {
+function give_akismet($spam) {
 
 	// Bail out, If spam.
-	if ( $spam ) {
+	if ($spam) {
 		return $spam;
 	}
 
 	// Bail out, if Akismet key not exist.
-	if ( ! give_check_akismet_key() ) {
+	if ( ! give_check_akismet_key()) {
 		return false;
 	}
 
 	// Build args array.
 	$args = array();
 
-	$args['comment_author']       = isset( $_POST['give_first'] ) ? strip_tags( trim( $_POST['give_first'] ) ) : '';
-	$args['comment_author_email'] = isset( $_POST['give_email'] ) ? $_POST['give_email'] : false;
-	$args['blog']                 = get_option( 'home' );
+	$args['comment_author']       = isset($_POST['give_first']) ? strip_tags(trim($_POST['give_first'])) : '';
+	$args['comment_author_email'] = isset($_POST['give_email']) ? $_POST['give_email'] : false;
+	$args['blog']                 = get_option('home');
 	$args['blog_lang']            = get_locale();
-	$args['blog_charset']         = get_option( 'blog_charset' );
+	$args['blog_charset']         = get_option('blog_charset');
 	$args['user_ip']              = $_SERVER['REMOTE_ADDR'];
 	$args['user_agent']           = $_SERVER['HTTP_USER_AGENT'];
 	$args['referrer']             = $_SERVER['HTTP_REFERER'];
 	$args['comment_type']         = 'contact-form';
 
-	$ignore = array( 'HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW' );
+	$ignore = array('HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW');
 
-	foreach ( $_SERVER as $key => $value ) {
-		if ( ! in_array( $key, (array) $ignore ) ) {
+	foreach ($_SERVER as $key => $value) {
+		if ( ! in_array($key, (array) $ignore)) {
 			$args["$key"] = $value;
 		}
 	}
 
 	// It will return Akismet spam detect API response.
-	return give_akismet_spam_check( $args );
+	return give_akismet_spam_check($args);
 
 }
 
-add_filter( 'give_spam', 'give_akismet' );
+add_filter('give_spam', 'give_akismet');
 
 /**
  * Check Akismet API Key.
@@ -128,11 +128,11 @@ 
  * @return bool
  */
 function give_check_akismet_key() {
-	if ( is_callable( array( 'Akismet', 'get_api_key' ) ) ) { // Akismet v3.0+
+	if (is_callable(array('Akismet', 'get_api_key'))) { // Akismet v3.0+
 		return (bool) Akismet::get_api_key();
 	}
 
-	if ( function_exists( 'akismet_get_key' ) ) {
+	if (function_exists('akismet_get_key')) {
 		return (bool) akismet_get_key();
 	}
 
@@ -148,26 +148,26 @@ 
  *
  * @return bool|mixed
  */
-function give_akismet_spam_check( $args ) {
+function give_akismet_spam_check($args) {
 	global $akismet_api_host, $akismet_api_port;
 
 	$spam         = false;
-	$query_string = http_build_query( $args );
+	$query_string = http_build_query($args);
 
-	if ( is_callable( array( 'Akismet', 'http_post' ) ) ) { // Akismet v3.0+
-		$response = Akismet::http_post( $query_string, 'comment-check' );
+	if (is_callable(array('Akismet', 'http_post'))) { // Akismet v3.0+
+		$response = Akismet::http_post($query_string, 'comment-check');
 	} else {
-		$response = akismet_http_post( $query_string, $akismet_api_host,
-			'/1.1/comment-check', $akismet_api_port );
+		$response = akismet_http_post($query_string, $akismet_api_host,
+			'/1.1/comment-check', $akismet_api_port);
 	}
 
 	// It's spam if response status is true.
-	if ( 'true' === $response[1] ) {
+	if ('true' === $response[1]) {
 		$spam = true;
 	}
 
 	// Allow developer to modified Akismet spam detection response.
-	return apply_filters( 'give_akismet_spam_check', $spam, $args );
+	return apply_filters('give_akismet_spam_check', $spam, $args);
 }
 
 /**
@@ -180,15 +180,15 @@ 
  *
  * @return array
  */
-function give_bc_v1817_iranian_currency_code( $currencies ) {
-	if ( ! give_has_upgrade_completed( 'v1817_update_donation_iranian_currency_code' ) ) {
+function give_bc_v1817_iranian_currency_code($currencies) {
+	if ( ! give_has_upgrade_completed('v1817_update_donation_iranian_currency_code')) {
 		$currencies['RIAL'] = $currencies['IRR'];
 	}
 
 	return $currencies;
 }
 
-add_filter( 'give_currencies', 'give_bc_v1817_iranian_currency_code', 0 );
+add_filter('give_currencies', 'give_bc_v1817_iranian_currency_code', 0);
 
 
 /**
@@ -202,22 +202,20 @@ 
  *
  * @return string
  */
-function give_format_price_for_right_to_left_supported_currency( $formatted_amount, $currency_args, $price ) {
-	if ( ! give_is_right_to_left_supported_currency( $currency_args['currency_code'] ) ) {
+function give_format_price_for_right_to_left_supported_currency($formatted_amount, $currency_args, $price) {
+	if ( ! give_is_right_to_left_supported_currency($currency_args['currency_code'])) {
 		return $formatted_amount;
 	}
 
 	$formatted_amount = (
 	'before' === (string) $currency_args['position'] ?
-		'‫' . $price . $currency_args['symbol'] . '‬' :
-		'‪' . $price . $currency_args['symbol'] . '‬'
+		'‫'.$price.$currency_args['symbol'].'‬' : '‪'.$price.$currency_args['symbol'].'‬'
 	);
 
 	$formatted_amount = $currency_args['decode_currency'] ?
-		html_entity_decode( $formatted_amount, ENT_COMPAT, 'UTF-8' ) :
-		$formatted_amount;
+		html_entity_decode($formatted_amount, ENT_COMPAT, 'UTF-8') : $formatted_amount;
 
 	return $formatted_amount;
 }
 
-add_filter( 'give_currency_filter', 'give_format_price_for_right_to_left_supported_currency', 10, 3 );
\ No newline at end of file
+add_filter('give_currency_filter', 'give_format_price_for_right_to_left_supported_currency', 10, 3);
\ No newline at end of file

includes/donors/class-give-donors-query.php

Spacing +49 added lines, -50 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -85,7 +85,7 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'number'     => 20,
 			'offset'     => 0,
@@ -103,7 +103,7 @@ 
 			// 'form'       => array(),
 		);
 
-		$this->args = wp_parse_args( $args, $defaults );
+		$this->args = wp_parse_args($args, $defaults);
 
 		$this->table_name      = Give()->donors->table_name;
 		$this->meta_table_name = Give()->donor_meta->table_name;
@@ -146,12 +146,12 @@ 
 		 *
 		 * @param Give_Donors_Query $this Donors query object.
 		 */
-		do_action( 'give_pre_get_donors', $this );
+		do_action('give_pre_get_donors', $this);
 
-		if ( empty( $this->args['count'] ) ) {
-			$this->donors = $wpdb->get_results( $this->get_sql() );
+		if (empty($this->args['count'])) {
+			$this->donors = $wpdb->get_results($this->get_sql());
 		} else {
-			$this->donors = $wpdb->get_var( $this->get_sql() );
+			$this->donors = $wpdb->get_var($this->get_sql());
 		}
 
 		/**
@@ -161,7 +161,7 @@ 
 		 *
 		 * @param Give_Donors_Query $this Donors query object.
 		 */
-		do_action( 'give_post_get_donors', $this );
+		do_action('give_post_get_donors', $this);
 
 		return $this->donors;
 	}
@@ -178,7 +178,7 @@ 
 	public function get_sql() {
 		global $wpdb;
 
-		if ( $this->args['number'] < 1 ) {
+		if ($this->args['number'] < 1) {
 			$this->args['number'] = 999999999999;
 		}
 
@@ -186,22 +186,22 @@ 
 
 
 		// Set offset.
-		if ( empty( $this->args['offset'] ) && ( 0 < $this->args['paged'] ) ) {
-			$this->args['offset'] = $this->args['number'] * ( $this->args['paged'] - 1 );
+		if (empty($this->args['offset']) && (0 < $this->args['paged'])) {
+			$this->args['offset'] = $this->args['number'] * ($this->args['paged'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$this->table_name}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$this->table_name}.{$this->args['fields']}";
-			} else if ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$this->table_name}." . implode( " , {$this->table_name}.", $this->args['fields'] );
+			} else if (is_array($this->args['fields'])) {
+				$fields = "{$this->table_name}.".implode(" , {$this->table_name}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$this->table_name}.id)";
 		}
 
@@ -209,13 +209,13 @@ 
 
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$this->table_name} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			absint( $this->args['number'] )
+			absint($this->args['offset']),
+			absint($this->args['number'])
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}
@@ -233,15 +233,15 @@ 
 		$where = '';
 
 		// Get sql query for meta.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_object = new WP_Meta_Query( $this->args['meta_query'] );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_object = new WP_Meta_Query($this->args['meta_query']);
 			$meta_query        = $meta_query_object->get_sql(
 				$this->meta_type,
 				$this->table_name,
 				'id'
 			);
 
-			$where = implode( '', $meta_query );
+			$where = implode('', $meta_query);
 		}
 
 		$where .= 'WHERE 1=1 ';
@@ -251,7 +251,7 @@ 
 		$where .= $this->get_where_user();
 		$where .= $this->get_where_date();
 
-		return trim( $where );
+		return trim($where);
 	}
 
 	/**
@@ -268,17 +268,17 @@ 
 
 		$where = '';
 
-		if ( ! empty( $this->args['email'] ) ) {
+		if ( ! empty($this->args['email'])) {
 
-			if ( is_array( $this->args['email'] ) ) {
+			if (is_array($this->args['email'])) {
 
-				$emails_count       = count( $this->args['email'] );
-				$emails_placeholder = array_fill( 0, $emails_count, '%s' );
-				$emails             = implode( ', ', $emails_placeholder );
+				$emails_count       = count($this->args['email']);
+				$emails_placeholder = array_fill(0, $emails_count, '%s');
+				$emails             = implode(', ', $emails_placeholder);
 
-				$where .= $wpdb->prepare( "AND {$this->table_name}.email IN( $emails )", $this->args['email'] );
+				$where .= $wpdb->prepare("AND {$this->table_name}.email IN( $emails )", $this->args['email']);
 			} else {
-				$where .= $wpdb->prepare( "AND {$this->table_name}.email = %s", $this->args['email'] );
+				$where .= $wpdb->prepare("AND {$this->table_name}.email = %s", $this->args['email']);
 			}
 		}
 
@@ -298,11 +298,11 @@ 
 		$where = '';
 
 		// Specific donors.
-		if ( ! empty( $this->args['donor'] ) ) {
-			if ( ! is_array( $this->args['donor'] ) ) {
-				$this->args['donor'] = explode( ',', $this->args['donor'] );
+		if ( ! empty($this->args['donor'])) {
+			if ( ! is_array($this->args['donor'])) {
+				$this->args['donor'] = explode(',', $this->args['donor']);
 			}
-			$donor_ids = implode( ',', array_map( 'intval', $this->args['donor'] ) );
+			$donor_ids = implode(',', array_map('intval', $this->args['donor']));
 
 			$where .= "AND {$this->table_name}.id IN( {$donor_ids} )";
 		}
@@ -323,9 +323,9 @@ 
 		$where = '';
 
 		// Donors created for a specific date or in a date range
-		if ( ! empty( $this->args['date_query'] ) ) {
+		if ( ! empty($this->args['date_query'])) {
 			$date_query_object = new WP_Date_Query(
-				is_array( $this->args['date_query'] ) ? $this->args['date_query'] : wp_parse_args( $this->args['date_query'] ),
+				is_array($this->args['date_query']) ? $this->args['date_query'] : wp_parse_args($this->args['date_query']),
 				"{$this->table_name}.date_created"
 			);
 
@@ -360,11 +360,11 @@ 
 		$where = '';
 
 		// Donors created for a specific date or in a date range
-		if ( ! empty( $this->args['s'] ) && false !== strpos( $this->args['s'], ':' ) ) {
-			$search_parts = explode( ':', $this->args['s'] );
+		if ( ! empty($this->args['s']) && false !== strpos($this->args['s'], ':')) {
+			$search_parts = explode(':', $this->args['s']);
 
-			if ( ! empty( $search_parts[0] ) ) {
-				switch ( $search_parts[0] ) {
+			if ( ! empty($search_parts[0])) {
+				switch ($search_parts[0]) {
 					case 'name':
 						$where = "AND {$this->table_name}.name LIKE '%{$search_parts[1]}%'";
 						break;
@@ -392,11 +392,11 @@ 
 		$where = '';
 
 		// Donors create for specific wp user.
-		if ( ! empty( $this->args['user'] ) ) {
-			if ( ! is_array( $this->args['user'] ) ) {
-				$this->args['user'] = explode( ',', $this->args['user'] );
+		if ( ! empty($this->args['user'])) {
+			if ( ! is_array($this->args['user'])) {
+				$this->args['user'] = explode(',', $this->args['user']);
 			}
-			$user_ids = implode( ',', array_map( 'intval', $this->args['user'] ) );
+			$user_ids = implode(',', array_map('intval', $this->args['user']));
 
 			$where .= "AND {$this->table_name}.user_id IN( {$user_ids} )";
 		}
@@ -415,14 +415,13 @@ 
 	private function get_order_query() {
 		$table_columns = Give()->donors->get_columns();
 
-		$this->args['orderby'] = ! array_key_exists( $this->args['orderby'], $table_columns ) ?
-			'id' :
-			$this->args['orderby'];
+		$this->args['orderby'] = ! array_key_exists($this->args['orderby'], $table_columns) ?
+			'id' : $this->args['orderby'];
 
-		$this->args['orderby'] = esc_sql( $this->args['orderby'] );
-		$this->args['order']   = esc_sql( $this->args['order'] );
+		$this->args['orderby'] = esc_sql($this->args['orderby']);
+		$this->args['order']   = esc_sql($this->args['order']);
 
-		switch ( $table_columns[ $this->args['orderby'] ] ) {
+		switch ($table_columns[$this->args['orderby']]) {
 			case '%d':
 			case '%f':
 				$query = "ORDER BY {$this->table_name}.{$this->args['orderby']}+0";

includes/import-functions.php

Spacing +223 added lines, -223 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,7 +20,7 @@ 
  * @since 1.8.13
  */
 function give_import_donation_report() {
-	return get_option( 'give_import_donation_report', array() );
+	return get_option('give_import_donation_report', array());
 }
 
 
@@ -29,8 +29,8 @@ 
  *
  * @since 1.8.13
  */
-function give_import_donation_report_update( $value = array() ) {
-	update_option( 'give_import_donation_report', $value );
+function give_import_donation_report_update($value = array()) {
+	update_option('give_import_donation_report', $value);
 }
 
 
@@ -40,7 +40,7 @@ 
  * @since 1.8.13
  */
 function give_import_donation_report_reset() {
-	update_option( 'give_import_donation_report', array() );
+	update_option('give_import_donation_report', array());
 }
 
 /**
@@ -52,7 +52,7 @@ 
  *
  * @return array|bool|Give_Donate_Form|int|null|WP_Post
  */
-function give_import_get_form_data_from_csv( $data, $import_setting = array() ) {
+function give_import_get_form_data_from_csv($data, $import_setting = array()) {
 	$new_form = false;
 
 	// Get the import report
@@ -61,58 +61,58 @@ 
 	$form = false;
 	$meta = array();
 
-	if ( ! empty( $data['form_id'] ) ) {
-		$form = new Give_Donate_Form( $data['form_id'] );
+	if ( ! empty($data['form_id'])) {
+		$form = new Give_Donate_Form($data['form_id']);
 		// Add support to older php version.
 		$form_id = $form->get_ID();
-		if ( empty( $form_id ) ) {
-			$report['duplicate_form'] = ( ! empty( $report['duplicate_form'] ) ? ( absint( $report['duplicate_form'] ) + 1 ) : 1 );
+		if (empty($form_id)) {
+			$report['duplicate_form'] = ( ! empty($report['duplicate_form']) ? (absint($report['duplicate_form']) + 1) : 1);
 			$form                     = false;
 		}
 	}
 
-	if ( false === $form && ! empty( $data['form_title'] ) ) {
-		$form = get_page_by_title( $data['form_title'], OBJECT, 'give_forms' );
+	if (false === $form && ! empty($data['form_title'])) {
+		$form = get_page_by_title($data['form_title'], OBJECT, 'give_forms');
 
-		if ( ! empty( $form->ID ) ) {
+		if ( ! empty($form->ID)) {
 
-			$report['duplicate_form'] = ( ! empty( $report['duplicate_form'] ) ? ( absint( $report['duplicate_form'] ) + 1 ) : 1 );
+			$report['duplicate_form'] = ( ! empty($report['duplicate_form']) ? (absint($report['duplicate_form']) + 1) : 1);
 
-			$form = new Give_Donate_Form( $form->ID );
+			$form = new Give_Donate_Form($form->ID);
 		} else {
 			$form                  = new Give_Donate_Form();
 			$args                  = array(
 				'post_title'  => $data['form_title'],
 				'post_status' => 'publish',
 			);
-			$form                  = $form->create( $args );
-			$report['create_form'] = ( ! empty( $report['create_form'] ) ? ( absint( $report['create_form'] ) + 1 ) : 1 );
+			$form                  = $form->create($args);
+			$report['create_form'] = ( ! empty($report['create_form']) ? (absint($report['create_form']) + 1) : 1);
 			$new_form              = true;
 
 		}
 
-		$form = get_page_by_title( $data['form_title'], OBJECT, 'give_forms' );
-		$form = new Give_Donate_Form( $form->ID );
+		$form = get_page_by_title($data['form_title'], OBJECT, 'give_forms');
+		$form = new Give_Donate_Form($form->ID);
 	}
 
-	if ( ! empty( $form ) && $form->get_ID() ) {
-		if ( ! empty( $data['form_level'] ) && 'custom' != (string) strtolower( $data['form_level'] ) ) {
+	if ( ! empty($form) && $form->get_ID()) {
+		if ( ! empty($data['form_level']) && 'custom' != (string) strtolower($data['form_level'])) {
 			$prices     = (array) $form->get_prices();
 			$price_text = array();
-			foreach ( $prices as $key => $price ) {
-				if ( isset( $price['_give_id']['level_id'] ) ) {
-					$price_text[ $price['_give_id']['level_id'] ] = ( ! empty( $price['_give_text'] ) ? $price['_give_text'] : '' );
+			foreach ($prices as $key => $price) {
+				if (isset($price['_give_id']['level_id'])) {
+					$price_text[$price['_give_id']['level_id']] = ( ! empty($price['_give_text']) ? $price['_give_text'] : '');
 				}
 			}
 
-			if ( ! in_array( $data['form_level'], $price_text ) ) {
+			if ( ! in_array($data['form_level'], $price_text)) {
 
 				// For generating unquiet level id.
 				$count     = 1;
-				$new_level = count( $prices ) + $count;
-				while ( array_key_exists( $new_level, $price_text ) ) {
-					$count ++;
-					$new_level = count( $prices ) + $count;
+				$new_level = count($prices) + $count;
+				while (array_key_exists($new_level, $price_text)) {
+					$count++;
+					$new_level = count($prices) + $count;
 				}
 
 				$multi_level_donations = array(
@@ -120,57 +120,57 @@ 
 						'_give_id'     => array(
 							'level_id' => $new_level,
 						),
-						'_give_amount' => give_sanitize_amount_for_db( $data['amount'] ),
+						'_give_amount' => give_sanitize_amount_for_db($data['amount']),
 						'_give_text'   => $data['form_level'],
 					),
 				);
 
-				$price_text[ $new_level ] = $data['form_level'];
+				$price_text[$new_level] = $data['form_level'];
 
-				if ( ! empty( $prices ) && is_array( $prices ) && ! empty( $prices[0] ) ) {
-					$prices = wp_parse_args( $multi_level_donations, $prices );
+				if ( ! empty($prices) && is_array($prices) && ! empty($prices[0])) {
+					$prices = wp_parse_args($multi_level_donations, $prices);
 
 					// Sort $prices by amount in ascending order.
-					$prices = wp_list_sort( $prices, '_give_amount', 'ASC' );
+					$prices = wp_list_sort($prices, '_give_amount', 'ASC');
 				} else {
 					$prices = $multi_level_donations;
 				}
 
 				// Unset _give_default key from $prices.
-				foreach ( $prices as $key => $price ) {
-					if ( isset( $prices[ $key ]['_give_default'] ) ) {
-						unset( $prices[ $key ]['_give_default'] );
+				foreach ($prices as $key => $price) {
+					if (isset($prices[$key]['_give_default'])) {
+						unset($prices[$key]['_give_default']);
 					}
 				}
 
 				// Set the first $price of the $prices as defalut.
 				$prices[0]['_give_default'] = 'default';
 			}
-			$form->price_id = array_search( $data['form_level'], $price_text );
+			$form->price_id = array_search($data['form_level'], $price_text);
 
-			$donation_levels_amounts = wp_list_pluck( $prices, '_give_amount' );
-			$min_amount              = min( $donation_levels_amounts );
-			$max_amount              = max( $donation_levels_amounts );
+			$donation_levels_amounts = wp_list_pluck($prices, '_give_amount');
+			$min_amount              = min($donation_levels_amounts);
+			$max_amount              = max($donation_levels_amounts);
 
 			$meta = array(
 				'_give_levels_minimum_amount' => $min_amount,
 				'_give_levels_maximum_amount' => $max_amount,
 				'_give_price_option'          => 'multi',
-				'_give_donation_levels'       => array_values( $prices ),
+				'_give_donation_levels'       => array_values($prices),
 			);
 		} else {
 			$form->price_id = 'custom';
 		}
 
 		$defaults = array(
-			'_give_set_price'    => give_sanitize_amount_for_db( $data['amount'] ),
+			'_give_set_price'    => give_sanitize_amount_for_db($data['amount']),
 			'_give_price_option' => 'set',
 		);
 
 		// If new form is created.
-		if ( ! empty( $new_form ) ) {
+		if ( ! empty($new_form)) {
 			$new_form = array(
-				'_give_custom_amount_text' => ( ! empty( $data['form_custom_amount_text'] ) ? $data['form_custom_amount_text'] : 'Custom' ),
+				'_give_custom_amount_text' => ( ! empty($data['form_custom_amount_text']) ? $data['form_custom_amount_text'] : 'Custom'),
 				'_give_logged_in_only'     => 'enabled',
 				'_give_custom_amount'      => 'enabled',
 				'_give_payment_import'     => true,
@@ -181,18 +181,18 @@ 
 				'_give_default_gateway'    => 'global',
 				'_give_show_register_form' => 'both',
 			);
-			$defaults = wp_parse_args( $defaults, $new_form );
+			$defaults = wp_parse_args($defaults, $new_form);
 		}
 
-		$meta = wp_parse_args( $meta, $defaults );
+		$meta = wp_parse_args($meta, $defaults);
 
-		foreach ( $meta as $key => $value ) {
-			give_update_meta( $form->get_ID(), $key, $value );
+		foreach ($meta as $key => $value) {
+			give_update_meta($form->get_ID(), $key, $value);
 		}
 	}
 
 	// update the report
-	give_import_donation_report_update( $report );
+	give_import_donation_report_update($report);
 
 	return $form;
 }
@@ -206,30 +206,30 @@ 
  *
  * @return bool|false|WP_User
  */
-function give_import_get_user_from_csv( $data, $import_setting = array() ) {
+function give_import_get_user_from_csv($data, $import_setting = array()) {
 	$report      = give_import_donation_report();
 	$donor_data  = false;
 	$customer_id = false;
 
 	// check if donor id is not empty
-	if ( ! empty( $data['donor_id'] ) ) {
-		$donor_data = new Give_Donor( (int) $data['donor_id'] );
-		if ( ! empty( $donor_data->id ) ) {
-			$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+	if ( ! empty($data['donor_id'])) {
+		$donor_data = new Give_Donor((int) $data['donor_id']);
+		if ( ! empty($donor_data->id)) {
+			$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		}
 	}
 
-	if ( empty( $donor_data->id ) && ! empty( $data['user_id'] ) ) {
+	if (empty($donor_data->id) && ! empty($data['user_id'])) {
 		$user_id    = (int) $data['user_id'];
-		$donor_data = new Give_Donor( $user_id, true );
+		$donor_data = new Give_Donor($user_id, true);
 
 
-		if ( empty( $donor_data->id ) ) {
-			$donor_data = get_user_by( 'id', $user_id );
-			if ( ! empty( $donor_data->ID ) ) {
-				$first_name = ( ! empty( $data['first_name'] ) ? $data['first_name'] : $donor_data->user_nicename );
-				$last_name  = ( ! empty( $data['last_name'] ) ? $data['last_name'] : ( ( $lastname = get_user_meta( $donor_data->ID, 'last_name', true ) ) ? $lastname : '' ) );
-				$name       = $first_name . ' ' . $last_name;
+		if (empty($donor_data->id)) {
+			$donor_data = get_user_by('id', $user_id);
+			if ( ! empty($donor_data->ID)) {
+				$first_name = ( ! empty($data['first_name']) ? $data['first_name'] : $donor_data->user_nicename);
+				$last_name  = ( ! empty($data['last_name']) ? $data['last_name'] : (($lastname = get_user_meta($donor_data->ID, 'last_name', true)) ? $lastname : ''));
+				$name       = $first_name.' '.$last_name;
 				$user_email = $donor_data->user_email;
 				$donor_args = array(
 					'name'    => $name,
@@ -238,44 +238,44 @@ 
 				);
 
 				$donor_data = new Give_Donor();
-				$donor_data->create( $donor_args );
+				$donor_data->create($donor_args);
 
 				// Adding notes that donor is being imported from CSV.
 				$current_user = wp_get_current_user();
-				$donor_data->add_note( esc_html( wp_sprintf( __( 'This donor was imported by %s', 'give' ), $current_user->user_email ) ) );
+				$donor_data->add_note(esc_html(wp_sprintf(__('This donor was imported by %s', 'give'), $current_user->user_email)));
 
 				// Add is used to ensure duplicate emails are not added
-				if ( $user_email != $data['email'] && ! empty( $data['email'] ) ) {
-					$donor_data->add_meta( 'additional_email', $data['email'] );
+				if ($user_email != $data['email'] && ! empty($data['email'])) {
+					$donor_data->add_meta('additional_email', $data['email']);
 				}
 
-				$report['create_donor'] = ( ! empty( $report['create_donor'] ) ? ( absint( $report['create_donor'] ) + 1 ) : 1 );
+				$report['create_donor'] = ( ! empty($report['create_donor']) ? (absint($report['create_donor']) + 1) : 1);
 			} else {
 			}
 		} else {
 			// Add is used to ensure duplicate emails are not added
-			if ( $donor_data->email != $data['email'] ) {
-				$donor_data->add_meta( 'additional_email', ( ! empty( $data['email'] ) ? $data['email'] : $donor_data->email ) );
+			if ($donor_data->email != $data['email']) {
+				$donor_data->add_meta('additional_email', ( ! empty($data['email']) ? $data['email'] : $donor_data->email));
 			}
-			$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+			$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		}
 	}
 
-	if ( empty( $donor_data->id ) && ! empty( $data['email'] ) ) {
+	if (empty($donor_data->id) && ! empty($data['email'])) {
 
-		$donor_data = new Give_Donor( $data['email'] );
-		if ( empty( $donor_data->id ) ) {
-			$donor_data = get_user_by( 'email', $data['email'] );
+		$donor_data = new Give_Donor($data['email']);
+		if (empty($donor_data->id)) {
+			$donor_data = get_user_by('email', $data['email']);
 
-			if ( empty( $donor_data->ID ) && ! empty( $data['first_name'] ) && ! empty( $data['last_name'] ) && isset( $import_setting['create_user'] ) && 1 === absint( $import_setting['create_user'] ) ) {
-				$give_role  = (array) give_get_option( 'donor_default_user_role', get_option( 'default_role', ( ( $give_donor = wp_roles()->is_role( 'give_donor' ) ) && ! empty( $give_donor ) ? 'give_donor' : 'subscriber' ) ) );
+			if (empty($donor_data->ID) && ! empty($data['first_name']) && ! empty($data['last_name']) && isset($import_setting['create_user']) && 1 === absint($import_setting['create_user'])) {
+				$give_role  = (array) give_get_option('donor_default_user_role', get_option('default_role', (($give_donor = wp_roles()->is_role('give_donor')) && ! empty($give_donor) ? 'give_donor' : 'subscriber')));
 				$donor_args = array(
 					'user_login'      => $data['email'],
 					'user_email'      => $data['email'],
-					'user_registered' => date( 'Y-m-d H:i:s' ),
+					'user_registered' => date('Y-m-d H:i:s'),
 					'user_first'      => $data['first_name'],
 					'user_last'       => $data['last_name'],
-					'user_pass'       => wp_generate_password( 8, true ),
+					'user_pass'       => wp_generate_password(8, true),
 					'role'            => $give_role,
 				);
 
@@ -284,56 +284,56 @@ 
 				 *
 				 * @since 1.8.13
 				 */
-				$donor_args = (array) apply_filters( 'give_import_insert_user_args', $donor_args, $data, $import_setting );
+				$donor_args = (array) apply_filters('give_import_insert_user_args', $donor_args, $data, $import_setting);
 
 				// This action was added to remove the login when using the give register function.
-				add_filter( 'give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11 );
-				$customer_id = give_register_and_login_new_user( $donor_args );
-				remove_filter( 'give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11 );
+				add_filter('give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11);
+				$customer_id = give_register_and_login_new_user($donor_args);
+				remove_filter('give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11);
 
-				update_user_meta( $customer_id, '_give_payment_import', true );
-				$donor_data = new Give_Donor( $customer_id, true );
+				update_user_meta($customer_id, '_give_payment_import', true);
+				$donor_data = new Give_Donor($customer_id, true);
 			} else {
-				$customer_id = ( ! empty( $donor_data->ID ) ? $donor_data->ID : false );
+				$customer_id = ( ! empty($donor_data->ID) ? $donor_data->ID : false);
 			}
 
-			if ( ! empty( $customer_id ) || ( isset( $import_setting['create_user'] ) && 0 === absint( $import_setting['create_user'] ) ) ) {
-				$donor_data = new Give_Donor( $customer_id, true );
+			if ( ! empty($customer_id) || (isset($import_setting['create_user']) && 0 === absint($import_setting['create_user']))) {
+				$donor_data = new Give_Donor($customer_id, true);
 
-				if ( empty( $donor_data->id ) ) {
+				if (empty($donor_data->id)) {
 
-					if ( ! empty( $data['form_id'] ) ) {
-						$form = new Give_Donate_Form( $data['form_id'] );
+					if ( ! empty($data['form_id'])) {
+						$form = new Give_Donate_Form($data['form_id']);
 					}
 
-					$payment_title = ( isset( $data['form_title'] ) ? $data['form_title'] : ( isset( $form ) ? $form->get_name() : esc_html__( 'New Form', 'give' ) ) );
+					$payment_title = (isset($data['form_title']) ? $data['form_title'] : (isset($form) ? $form->get_name() : esc_html__('New Form', 'give')));
 					$donor_args    = array(
-						'name'  => ! is_email( $payment_title ) ? $data['first_name'] . ' ' . $data['last_name'] : '',
+						'name'  => ! is_email($payment_title) ? $data['first_name'].' '.$data['last_name'] : '',
 						'email' => $data['email'],
 					);
 
-					if ( ! empty( $customer_id ) ) {
+					if ( ! empty($customer_id)) {
 						$donor_args['user_id'] = $customer_id;
 					}
 
-					$donor_data->create( $donor_args );
+					$donor_data->create($donor_args);
 
 					// Adding notes that donor is being imported from CSV.
 					$current_user = wp_get_current_user();
-					$donor_data->add_note( esc_html( wp_sprintf( __( 'This donor was imported by %s', 'give' ), $current_user->user_email ) ) );
+					$donor_data->add_note(esc_html(wp_sprintf(__('This donor was imported by %s', 'give'), $current_user->user_email)));
 
-					$report['create_donor'] = ( ! empty( $report['create_donor'] ) ? ( absint( $report['create_donor'] ) + 1 ) : 1 );
+					$report['create_donor'] = ( ! empty($report['create_donor']) ? (absint($report['create_donor']) + 1) : 1);
 				} else {
-					$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+					$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 				}
 			}
 		} else {
-			$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+			$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		}
 	}
 
 	// update the report
-	give_import_donation_report_update( $report );
+	give_import_donation_report_update($report);
 
 	return $donor_data;
 }
@@ -351,9 +351,9 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_default_options', array(
-		'' => __( 'Do not import', 'give' ),
-	) );
+	return (array) apply_filters('give_import_default_options', array(
+		'' => __('Do not import', 'give'),
+	));
 }
 
 /**
@@ -369,33 +369,33 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_donations_options', array(
-		'id'          => __( 'Donation ID', 'give' ),
-		'amount'      => __( 'Donation Amount', 'give' ),
-		'post_date'   => __( 'Donation Date', 'give' ),
-		'first_name'  => __( 'Donor First Name', 'give' ),
-		'last_name'   => __( 'Donor Last Name', 'give' ),
-		'line1'       => __( 'Address 1', 'give' ),
-		'line2'       => __( 'Address 2', 'give' ),
-		'city'        => __( 'City', 'give' ),
-		'state'       => __( 'State', 'give' ),
+	return (array) apply_filters('give_import_donations_options', array(
+		'id'          => __('Donation ID', 'give'),
+		'amount'      => __('Donation Amount', 'give'),
+		'post_date'   => __('Donation Date', 'give'),
+		'first_name'  => __('Donor First Name', 'give'),
+		'last_name'   => __('Donor Last Name', 'give'),
+		'line1'       => __('Address 1', 'give'),
+		'line2'       => __('Address 2', 'give'),
+		'city'        => __('City', 'give'),
+		'state'       => __('State', 'give'),
 		'country'     => array(
-			__( 'Country', 'give' ),
-			__( 'County', 'give' ),
-			__( 'Region', 'give' ),
-			__( 'Province', 'give' ),
+			__('Country', 'give'),
+			__('County', 'give'),
+			__('Region', 'give'),
+			__('Province', 'give'),
 		),
 		'zip'         => array(
-			__( 'Zip', 'give' ),
-			__( 'Postal Code', 'give' ),
+			__('Zip', 'give'),
+			__('Postal Code', 'give'),
 		),
-		'email'       => __( 'Donor Email', 'give' ),
-		'post_status' => __( 'Donation Status', 'give' ),
-		'gateway'     => __( 'Payment Method', 'give' ),
-		'notes'       => __( 'Notes', 'give' ),
-		'mode'        => __( 'Test Mode', 'give' ),
-		'post_meta'   => __( 'Import as Meta', 'give' ),
-	) );
+		'email'       => __('Donor Email', 'give'),
+		'post_status' => __('Donation Status', 'give'),
+		'gateway'     => __('Payment Method', 'give'),
+		'notes'       => __('Notes', 'give'),
+		'mode'        => __('Test Mode', 'give'),
+		'post_meta'   => __('Import as Meta', 'give'),
+	));
 }
 
 /**
@@ -411,10 +411,10 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_donor_options', array(
-		'donor_id' => __( 'Donor ID', 'give' ),
-		'user_id'  => __( 'User ID', 'give' ),
-	) );
+	return (array) apply_filters('give_import_donor_options', array(
+		'donor_id' => __('Donor ID', 'give'),
+		'user_id'  => __('User ID', 'give'),
+	));
 }
 
 /**
@@ -430,15 +430,15 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_donation_form_options', array(
+	return (array) apply_filters('give_import_donation_form_options', array(
 		'form_title'              => array(
-			__( 'Donation Form Title', 'give' ),
-			__( 'Donation Form', 'give' ),
+			__('Donation Form Title', 'give'),
+			__('Donation Form', 'give'),
 		),
-		'form_id'                 => __( 'Donation Form ID', 'give' ),
-		'form_level'              => __( 'Donation Level', 'give' ),
-		'form_custom_amount_text' => __( 'Custom Amount Text', 'give' ),
-	) );
+		'form_id'                 => __('Donation Form ID', 'give'),
+		'form_level'              => __('Donation Level', 'give'),
+		'form_custom_amount_text' => __('Custom Amount Text', 'give'),
+	));
 }
 
 /**
@@ -451,7 +451,7 @@ 
  *
  * @return array
  */
-function give_get_donation_data_from_csv( $file_id, $start, $end, $delimiter = 'csv' ) {
+function give_get_donation_data_from_csv($file_id, $start, $end, $delimiter = 'csv') {
 	/**
 	 * Filter to modify delimiter of Import.
 	 *
@@ -460,19 +460,19 @@ 
 	 *
 	 * Return string $delimiter.
 	 */
-	$delimiter = (string) apply_filters( 'give_import_delimiter_set', $delimiter );
+	$delimiter = (string) apply_filters('give_import_delimiter_set', $delimiter);
 
 	$raw_data = array();
-	$file_dir = get_attached_file( $file_id );
+	$file_dir = get_attached_file($file_id);
 	$count    = 0;
-	if ( false !== ( $handle = fopen( $file_dir, 'r' ) ) ) {
-		while ( false !== ( $row = fgetcsv( $handle, 0, $delimiter ) ) ) {
-			if ( $count >= $start && $count <= $end ) {
+	if (false !== ($handle = fopen($file_dir, 'r'))) {
+		while (false !== ($row = fgetcsv($handle, 0, $delimiter))) {
+			if ($count >= $start && $count <= $end) {
 				$raw_data[] = $row;
 			}
-			$count ++;
+			$count++;
 		}
-		fclose( $handle );
+		fclose($handle);
 	}
 
 	return $raw_data;
@@ -488,7 +488,7 @@ 
  *
  * @return bool
  */
-function give_log_user_in_on_register_callback( $value ) {
+function give_log_user_in_on_register_callback($value) {
 	return false;
 }
 
@@ -504,22 +504,22 @@ 
  *
  * @return bool
  */
-function give_save_import_donation_to_db( $raw_key, $row_data, $main_key = array(), $import_setting = array() ) {
-	$data                          = array_combine( $raw_key, $row_data );
+function give_save_import_donation_to_db($raw_key, $row_data, $main_key = array(), $import_setting = array()) {
+	$data                          = array_combine($raw_key, $row_data);
 	$price_id                      = false;
 	$customer_id                   = 0;
-	$import_setting['create_user'] = ( isset( $import_setting['create_user'] ) ? $import_setting['create_user'] : 1 );
+	$import_setting['create_user'] = (isset($import_setting['create_user']) ? $import_setting['create_user'] : 1);
 
-	$data = (array) apply_filters( 'give_save_import_donation_to_db', $data );
+	$data = (array) apply_filters('give_save_import_donation_to_db', $data);
 
-	$data['amount'] = give_maybe_sanitize_amount( $data['amount'] );
+	$data['amount'] = give_maybe_sanitize_amount($data['amount']);
 
 	// Here come the login function.
-	$donor_data = give_import_get_user_from_csv( $data, $import_setting );
-	if ( ! empty( $donor_data->id ) ) {
-		if ( ! empty( $donor_data->user_id ) ) {
+	$donor_data = give_import_get_user_from_csv($data, $import_setting);
+	if ( ! empty($donor_data->id)) {
+		if ( ! empty($donor_data->user_id)) {
 			$customer_id = $donor_data->user_id;
-		} elseif ( ! empty( $data['user_id'] ) ) {
+		} elseif ( ! empty($data['user_id'])) {
 			$customer_id = $data['user_id'];
 		}
 	} else {
@@ -527,95 +527,95 @@ 
 	}
 
 	// get form data or register a form data.
-	$form = give_import_get_form_data_from_csv( $data, $import_setting );
-	if ( false == $form ) {
+	$form = give_import_get_form_data_from_csv($data, $import_setting);
+	if (false == $form) {
 		return false;
 	} else {
-		$price_id = ( ! empty( $form->price_id ) ) ? $form->price_id : false;
+		$price_id = ( ! empty($form->price_id)) ? $form->price_id : false;
 	}
 
 
 	$address = array(
-		'line1'   => ( ! empty( $data['line1'] ) ? give_clean( $data['line1'] ) : '' ),
-		'line2'   => ( ! empty( $data['line1'] ) ? give_clean( $data['line2'] ) : '' ),
-		'city'    => ( ! empty( $data['line1'] ) ? give_clean( $data['city'] ) : '' ),
-		'zip'     => ( ! empty( $data['zip'] ) ? give_clean( $data['zip'] ) : '' ),
-		'state'   => ( ! empty( $data['state'] ) ? give_clean( $data['state'] ) : '' ),
-		'country' => ( ! empty( $data['country'] ) ? ( ( $country_code = array_search( $data['country'], give_get_country_list() ) ) ? $country_code : $data['country'] ) : '' ),
+		'line1'   => ( ! empty($data['line1']) ? give_clean($data['line1']) : ''),
+		'line2'   => ( ! empty($data['line1']) ? give_clean($data['line2']) : ''),
+		'city'    => ( ! empty($data['line1']) ? give_clean($data['city']) : ''),
+		'zip'     => ( ! empty($data['zip']) ? give_clean($data['zip']) : ''),
+		'state'   => ( ! empty($data['state']) ? give_clean($data['state']) : ''),
+		'country' => ( ! empty($data['country']) ? (($country_code = array_search($data['country'], give_get_country_list())) ? $country_code : $data['country']) : ''),
 	);
 
 	//Create payment_data array
 	$payment_data = array(
 		'donor_id'        => $donor_data->id,
 		'price'           => $data['amount'],
-		'status'          => ( ! empty( $data['post_status'] ) ? $data['post_status'] : 'publish' ),
+		'status'          => ( ! empty($data['post_status']) ? $data['post_status'] : 'publish'),
 		'currency'        => give_get_currency(),
 		'user_info'       => array(
 			'id'         => $customer_id,
-			'email'      => ( ! empty( $data['email'] ) ? $data['email'] : ( isset( $donor_data->email ) ? $donor_data->email : false ) ),
-			'first_name' => ( ! empty( $data['first_name'] ) ? $data['first_name'] : ( ! empty( $customer_id ) && ( $first_name = get_user_meta( $customer_id, 'first_name', true ) ) ? $first_name : $donor_data->name ) ),
-			'last_name'  => ( ! empty( $data['last_name'] ) ? $data['last_name'] : ( ! empty( $customer_id ) && ( $last_name = get_user_meta( $customer_id, 'last_name', true ) ) ? $last_name : $donor_data->name ) ),
+			'email'      => ( ! empty($data['email']) ? $data['email'] : (isset($donor_data->email) ? $donor_data->email : false)),
+			'first_name' => ( ! empty($data['first_name']) ? $data['first_name'] : ( ! empty($customer_id) && ($first_name = get_user_meta($customer_id, 'first_name', true)) ? $first_name : $donor_data->name)),
+			'last_name'  => ( ! empty($data['last_name']) ? $data['last_name'] : ( ! empty($customer_id) && ($last_name = get_user_meta($customer_id, 'last_name', true)) ? $last_name : $donor_data->name)),
 			'address'    => $address,
 		),
-		'gateway'         => ( ! empty( $data['gateway'] ) && 'offline' != strtolower( $data['gateway'] ) ? strtolower( $data['gateway'] ) : 'manual' ),
-		'give_form_title' => ( ! empty( $data['form_title'] ) ? $data['form_title'] : $form->get_name() ),
+		'gateway'         => ( ! empty($data['gateway']) && 'offline' != strtolower($data['gateway']) ? strtolower($data['gateway']) : 'manual'),
+		'give_form_title' => ( ! empty($data['form_title']) ? $data['form_title'] : $form->get_name()),
 		'give_form_id'    => (string) $form->get_ID(),
 		'give_price_id'   => $price_id,
-		'purchase_key'    => strtolower( md5( uniqid() ) ),
+		'purchase_key'    => strtolower(md5(uniqid())),
 		'user_email'      => $data['email'],
-		'post_date'       => ( ! empty( $data['post_date'] ) ? mysql2date( 'Y-m-d H:i:s', $data['post_date'] ) : current_time( 'mysql' ) ),
-		'mode'            => ( ! empty( $data['mode'] ) ? ( 'true' == (string) $data['mode'] || 'TRUE' == (string) $data['mode'] ? 'test' : 'live' ) : ( isset( $import_setting['mode'] ) ? ( true == (bool) $import_setting['mode'] ? 'test' : 'live' ) : ( give_is_test_mode() ? 'test' : 'live' ) ) ),
+		'post_date'       => ( ! empty($data['post_date']) ? mysql2date('Y-m-d H:i:s', $data['post_date']) : current_time('mysql')),
+		'mode'            => ( ! empty($data['mode']) ? ('true' == (string) $data['mode'] || 'TRUE' == (string) $data['mode'] ? 'test' : 'live') : (isset($import_setting['mode']) ? (true == (bool) $import_setting['mode'] ? 'test' : 'live') : (give_is_test_mode() ? 'test' : 'live'))),
 	);
 
-	$payment_data = apply_filters( 'give_import_before_import_payment', $payment_data, $data, $donor_data, $form );
+	$payment_data = apply_filters('give_import_before_import_payment', $payment_data, $data, $donor_data, $form);
 
 	// Get the report
 	$report = give_import_donation_report();
 
 	// Check for duplicate code.
-	if ( true === give_check_import_donation_duplicate( $payment_data, $data, $form, $donor_data ) ) {
-		$report['duplicate_donation'] = ( ! empty( $report['duplicate_donation'] ) ? ( absint( $report['duplicate_donation'] ) + 1 ) : 1 );
+	if (true === give_check_import_donation_duplicate($payment_data, $data, $form, $donor_data)) {
+		$report['duplicate_donation'] = ( ! empty($report['duplicate_donation']) ? (absint($report['duplicate_donation']) + 1) : 1);
 	} else {
-		add_action( 'give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1, 1 );
-		add_filter( 'give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11, 2 );
-		add_filter( 'give_update_donor_information', 'give_donation_import_update_donor_information', 11, 3 );
-		add_action( 'give_insert_payment', 'give_import_donation_insert_payment', 11, 2 );
-		$payment = give_insert_payment( $payment_data );
-		remove_action( 'give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1 );
-		remove_filter( 'give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11 );
-		remove_filter( 'give_update_donor_information', 'give_donation_import_update_donor_information', 11 );
-		remove_action( 'give_insert_payment', 'give_import_donation_insert_payment', 11 );
+		add_action('give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1, 1);
+		add_filter('give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11, 2);
+		add_filter('give_update_donor_information', 'give_donation_import_update_donor_information', 11, 3);
+		add_action('give_insert_payment', 'give_import_donation_insert_payment', 11, 2);
+		$payment = give_insert_payment($payment_data);
+		remove_action('give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1);
+		remove_filter('give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11);
+		remove_filter('give_update_donor_information', 'give_donation_import_update_donor_information', 11);
+		remove_action('give_insert_payment', 'give_import_donation_insert_payment', 11);
 
-		if ( $payment ) {
+		if ($payment) {
 
-			$report['create_donation'] = ( ! empty( $report['create_donation'] ) ? ( absint( $report['create_donation'] ) + 1 ) : 1 );
+			$report['create_donation'] = ( ! empty($report['create_donation']) ? (absint($report['create_donation']) + 1) : 1);
 
-			update_post_meta( $payment, '_give_payment_import', true );
+			update_post_meta($payment, '_give_payment_import', true);
 
-			if ( ! empty( $import_setting['csv'] ) ) {
-				update_post_meta( $payment, '_give_payment_import_id', $import_setting['csv'] );
+			if ( ! empty($import_setting['csv'])) {
+				update_post_meta($payment, '_give_payment_import_id', $import_setting['csv']);
 			}
 
 			// Insert Notes.
-			if ( ! empty( $data['notes'] ) ) {
-				give_insert_payment_note( $payment, $data['notes'] );
+			if ( ! empty($data['notes'])) {
+				give_insert_payment_note($payment, $data['notes']);
 			}
 
-			$meta_exists = array_keys( $raw_key, 'post_meta' );
-			if ( ! empty( $main_key ) && ! empty( $meta_exists ) ) {
-				foreach ( $meta_exists as $meta_exist ) {
-					if ( ! empty( $main_key[ $meta_exist ] ) && ! empty( $row_data[ $meta_exist ] ) ) {
-						update_post_meta( $payment, $main_key[ $meta_exist ], $row_data[ $meta_exist ] );
+			$meta_exists = array_keys($raw_key, 'post_meta');
+			if ( ! empty($main_key) && ! empty($meta_exists)) {
+				foreach ($meta_exists as $meta_exist) {
+					if ( ! empty($main_key[$meta_exist]) && ! empty($row_data[$meta_exist])) {
+						update_post_meta($payment, $main_key[$meta_exist], $row_data[$meta_exist]);
 					}
 				}
 			}
 		} else {
-			$report['failed_donation'] = ( ! empty( $report['failed_donation'] ) ? ( absint( $report['failed_donation'] ) + 1 ) : 1 );
+			$report['failed_donation'] = ( ! empty($report['failed_donation']) ? (absint($report['failed_donation']) + 1) : 1);
 		}
 	}
 
 	// update the report
-	give_import_donation_report_update( $report );
+	give_import_donation_report_update($report);
 
 	return true;
 }
@@ -631,12 +631,12 @@ 
  *
  * @return Give_Donor
  */
-function give_donation_import_update_donor_information( $donor, $payment_id, $payment_data ) {
+function give_donation_import_update_donor_information($donor, $payment_id, $payment_data) {
 	$old_donor = $donor;
-	if ( ! empty( $payment_data['donor_id'] ) ) {
-		$donor_id = absint( $payment_data['donor_id'] );
-		$donor    = new Give_Donor( $donor_id );
-		if ( ! empty( $donor->id ) ) {
+	if ( ! empty($payment_data['donor_id'])) {
+		$donor_id = absint($payment_data['donor_id']);
+		$donor    = new Give_Donor($donor_id);
+		if ( ! empty($donor->id)) {
 			return $donor;
 		}
 	}
@@ -649,12 +649,12 @@ 
  *
  * @since 1.8.13
  */
-function give_import_donation_insert_payment( $payment_id, $payment_data ) {
+function give_import_donation_insert_payment($payment_id, $payment_data) {
 	// Update Give Customers purchase_count
-	if ( ! empty( $payment_data['status'] ) && ( 'complete' === (string) $payment_data['status'] || 'publish' === (string) $payment_data['status'] ) ) {
-		$donor_id = (int) get_post_meta( $payment_id, '_give_payment_customer_id', true );
-		if ( ! empty( $donor_id ) ) {
-			$donor = new Give_Donor( $donor_id );
+	if ( ! empty($payment_data['status']) && ('complete' === (string) $payment_data['status'] || 'publish' === (string) $payment_data['status'])) {
+		$donor_id = (int) get_post_meta($payment_id, '_give_payment_customer_id', true);
+		if ( ! empty($donor_id)) {
+			$donor = new Give_Donor($donor_id);
 			$donor->increase_purchase_count();
 		}
 	}
@@ -665,8 +665,8 @@ 
  *
  * @since 1.8.13
  */
-function give_donation_import_give_insert_payment_args( $args, $payment_data ) {
-	if ( ! empty( $payment_data['user_info']['id'] ) ) {
+function give_donation_import_give_insert_payment_args($args, $payment_data) {
+	if ( ! empty($payment_data['user_info']['id'])) {
 		$args['post_author'] = (int) $payment_data['user_info']['id'];
 	}
 
@@ -678,11 +678,11 @@ 
  *
  * @since 1.8.13
  */
-function give_check_import_donation_duplicate( $payment_data, $data, $form, $donor_data ) {
+function give_check_import_donation_duplicate($payment_data, $data, $form, $donor_data) {
 	$return = false;
-	if ( ! empty( $data['post_date'] ) ) {
-		$post_date = mysql2date( 'Y-m-d-H-i-s', $data['post_date'] );
-		$post_date = explode( '-', $post_date );
+	if ( ! empty($data['post_date'])) {
+		$post_date = mysql2date('Y-m-d-H-i-s', $data['post_date']);
+		$post_date = explode('-', $post_date);
 		$args      = array(
 			'post_type'              => 'give_payment',
 			'cache_results'          => false,
@@ -703,7 +703,7 @@ 
 			'meta_query'             => array(
 				array(
 					'key'     => '_give_payment_total',
-					'value'   => preg_replace( '/[\$,]/', '', $payment_data['price'] ),
+					'value'   => preg_replace('/[\$,]/', '', $payment_data['price']),
 					'compare' => 'LIKE',
 				),
 				array(
@@ -720,9 +720,9 @@ 
 			),
 		);
 
-		$payments  = new Give_Payments_Query( $args );
+		$payments  = new Give_Payments_Query($args);
 		$donations = $payments->get_payments();
-		if ( ! empty( $donations ) ) {
+		if ( ! empty($donations)) {
 			return true;
 		}
 	}
@@ -739,9 +739,9 @@ 
  *
  * @return void
  */
-function give_donation_import_insert_default_payment_note( $payment_id ) {
+function give_donation_import_insert_default_payment_note($payment_id) {
 	$current_user = wp_get_current_user();
-	give_insert_payment_note( $payment_id, esc_html( wp_sprintf( __( 'This donation was imported by %s', 'give' ), $current_user->user_email ) ) );
+	give_insert_payment_note($payment_id, esc_html(wp_sprintf(__('This donation was imported by %s', 'give'), $current_user->user_email)));
 }
 
 /**
@@ -753,14 +753,14 @@ 
  *
  * @return string URL
  */
-function give_import_page_url( $parameter = array() ) {
+function give_import_page_url($parameter = array()) {
 	$defalut_query_arg = array(
 		'post_type'     => 'give_forms',
 		'page'          => 'give-tools',
 		'tab'           => 'import',
 		'importer-type' => 'import_donations',
 	);
-	$import_query_arg  = wp_parse_args( $parameter, $defalut_query_arg );
+	$import_query_arg = wp_parse_args($parameter, $defalut_query_arg);
 
-	return add_query_arg( $import_query_arg, admin_url( 'edit.php' ) );
+	return add_query_arg($import_query_arg, admin_url('edit.php'));
 }
\ No newline at end of file