WordImpress / Give

includes/admin/admin-pages.php

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,7 +32,7 @@ 
 	global $give_settings_page, $give_payments_page, $give_reports_page, $give_add_ons_page, $give_donors_page, $give_tools_page;
 
 	//Payments
-	$give_payment       = get_post_type_object( 'give_payment' );
+	$give_payment       = get_post_type_object('give_payment');
 	$give_payments_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
 		$give_payment->labels->name,
@@ -45,8 +45,8 @@ 
 	//Donors
 	$give_donors_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Donors', 'give' ),
-		esc_html__( 'Donors', 'give' ),
+		esc_html__('Donors', 'give'),
+		esc_html__('Donors', 'give'),
 		'view_give_reports',
 		'give-donors',
 		'give_donors_page'
@@ -55,8 +55,8 @@ 
 	//Reports`
 	$give_reports_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Donation Reports', 'give' ),
-		esc_html__( 'Reports', 'give' ),
+		esc_html__('Donation Reports', 'give'),
+		esc_html__('Reports', 'give'),
 		'view_give_reports',
 		'give-reports',
 		array(
@@ -68,8 +68,8 @@ 
 	//Settings
 	$give_settings_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Settings', 'give' ),
-		esc_html__( 'Settings', 'give' ),
+		esc_html__('Give Settings', 'give'),
+		esc_html__('Settings', 'give'),
 		'manage_give_settings',
 		'give-settings',
 		array(
@@ -81,8 +81,8 @@ 
 	//Tools.
 	$give_tools_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Tools', 'give' ),
-		esc_html__( 'Tools', 'give' ),
+		esc_html__('Give Tools', 'give'),
+		esc_html__('Tools', 'give'),
 		'manage_give_settings',
 		'give-tools',
 		array(
@@ -94,15 +94,15 @@ 
 	//Add-ons
 	$give_add_ons_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Add-ons', 'give' ),
-		esc_html__( 'Add-ons', 'give' ),
+		esc_html__('Give Add-ons', 'give'),
+		esc_html__('Add-ons', 'give'),
 		'install_plugins',
 		'give-addons',
 		'give_add_ons_page'
 	);
 }
 
-add_action( 'admin_menu', 'give_add_options_links', 10 );
+add_action('admin_menu', 'give_add_options_links', 10);
 
 /**
  *  Determines whether the current admin page is a Give admin page.
@@ -118,28 +118,28 @@ 
  *
  * @return bool True if Give admin page.
  */
-function give_is_admin_page( $passed_page = '', $passed_view = '' ) {
+function give_is_admin_page($passed_page = '', $passed_view = '') {
 	global $pagenow, $typenow;
 
 	$found          = true;
-	$get_query_args = ! empty( $_GET ) ? @array_map( 'strtolower', $_GET ) : array();
+	$get_query_args = ! empty($_GET) ? @array_map('strtolower', $_GET) : array();
 
 	// Set default argument, if not passed.
-	$query_args = wp_parse_args( $get_query_args, array_fill_keys( array( 'post_type', 'action', 'taxonomy', 'page', 'view', 'tab' ), false ) );
+	$query_args = wp_parse_args($get_query_args, array_fill_keys(array('post_type', 'action', 'taxonomy', 'page', 'view', 'tab'), false));
 
-	switch ( $passed_page ) {
+	switch ($passed_page) {
 		case 'categories':
 		case 'tags':
-			$has_view = in_array( $passed_view, array( 'list-table', 'edit', 'new' ), true );
+			$has_view = in_array($passed_view, array('list-table', 'edit', 'new'), true);
 
 			if (
-				! in_array( $query_args['taxonomy'], array( 'give_forms_category', 'give_forms_tag' ), true ) &&
+				! in_array($query_args['taxonomy'], array('give_forms_category', 'give_forms_tag'), true) &&
 				'edit-tags.php' !== $pagenow &&
 				(
 					$has_view ||
 					(
-						( in_array( $passed_view, array( 'list-table', 'new' ), true ) && 'edit' === $query_args['action'] ) ||
-						( 'edit' !== $passed_view && 'edit' !== $query_args['action'] ) &&
+						(in_array($passed_view, array('list-table', 'new'), true) && 'edit' === $query_args['action']) ||
+						('edit' !== $passed_view && 'edit' !== $query_args['action']) &&
 						! $has_view
 					)
 				)
@@ -149,18 +149,18 @@ 
 			break;
 		// Give Donation form page.
 		case 'give_forms':
-			$has_view = in_array( $passed_view, array( 'new', 'list-table', 'edit' ), true );
+			$has_view = in_array($passed_view, array('new', 'list-table', 'edit'), true);
 
 			if (
 				'give_forms' !== $typenow &&
 				(
-					( 'list-table' !== $passed_view && 'edit.php' !== $pagenow ) &&
-					( 'edit' !== $passed_view && 'post.php' !== $pagenow ) &&
-					( 'new' !== $passed_view && 'post-new.php' !== $pagenow )
+					('list-table' !== $passed_view && 'edit.php' !== $pagenow) &&
+					('edit' !== $passed_view && 'post.php' !== $pagenow) &&
+					('new' !== $passed_view && 'post-new.php' !== $pagenow)
 				) ||
 				(
 					! $has_view &&
-					( 'post-new.php' !== $pagenow && 'give_forms' !== $query_args['post_type'] )
+					('post-new.php' !== $pagenow && 'give_forms' !== $query_args['post_type'])
 				)
 			) {
 				$found = false;
@@ -168,13 +168,13 @@ 
 			break;
 		// Give Donors page.
 		case 'donors':
-			$has_view = array_intersect( array( $passed_view, $query_args['view'] ), array( 'list-table', 'overview', 'notes' ) );
+			$has_view = array_intersect(array($passed_view, $query_args['view']), array('list-table', 'overview', 'notes'));
 
 			if (
-				( 'give-donors' !== $query_args['page'] || 'edit.php' !== $pagenow ) &&
+				('give-donors' !== $query_args['page'] || 'edit.php' !== $pagenow) &&
 				(
-					( $passed_view !== $query_args['view'] || ! empty( $has_view ) ) ||
-					( false !== $query_args['view'] && 'list-table' !== $passed_view )
+					($passed_view !== $query_args['view'] || ! empty($has_view)) ||
+					(false !== $query_args['view'] && 'list-table' !== $passed_view)
 				)
 			) {
 				$found = false;
@@ -183,12 +183,12 @@ 
 		// Give Donations page.
 		case 'payments':
 			if (
-				( 'give-payment-history' !== $query_args['page'] || 'edit.php' !== $pagenow ) &&
+				('give-payment-history' !== $query_args['page'] || 'edit.php' !== $pagenow) &&
 				(
-					! in_array( $passed_view, array( 'list-table', 'edit' ), true ) ||
+					! in_array($passed_view, array('list-table', 'edit'), true) ||
 					(
-						( 'list-table' !== $passed_view && false !== $query_args['view'] ) ||
-						( 'edit' !== $passed_view && 'view-payment-details' !== $query_args['view'] )
+						('list-table' !== $passed_view && false !== $query_args['view']) ||
+						('edit' !== $passed_view && 'view-payment-details' !== $query_args['view'])
 					)
 				)
 			) {
@@ -199,20 +199,20 @@ 
 		case 'settings':
 		case 'addons':
 			// Get current tab.
-			$current_tab       = empty( $passed_view ) ? $query_args['tab'] : $passed_view;
-			$give_setting_page = in_array( $query_args['page'], array( 'give-reports', 'give-settings', 'give-addons' ), true );
+			$current_tab       = empty($passed_view) ? $query_args['tab'] : $passed_view;
+			$give_setting_page = in_array($query_args['page'], array('give-reports', 'give-settings', 'give-addons'), true);
 
 			// Check if it's Give Setting page or not.
 			if (
-				( 'edit.php' !== $pagenow || ! $give_setting_page ) &&
-				! Give_Admin_Settings::is_setting_page( $current_tab )
+				('edit.php' !== $pagenow || ! $give_setting_page) &&
+				! Give_Admin_Settings::is_setting_page($current_tab)
 			) {
 				$found = false;
 			}
 			break;
 		default:
 			global $give_payments_page, $give_settings_page, $give_reports_page, $give_system_info_page, $give_add_ons_page, $give_settings_export, $give_donors_page, $give_tools_page;
-			$admin_pages = apply_filters( 'give_admin_pages', array(
+			$admin_pages = apply_filters('give_admin_pages', array(
 				$give_payments_page,
 				$give_settings_page,
 				$give_reports_page,
@@ -222,11 +222,11 @@ 
 				$give_donors_page,
 				$give_tools_page,
 				'widgets.php',
-			) );
+			));
 
-			$found = ( 'give_forms' === $typenow || in_array( $pagenow, array_merge( $admin_pages, array( 'index.php', 'post-new.php', 'post.php' ) ), true ) ) ? true : false;
+			$found = ('give_forms' === $typenow || in_array($pagenow, array_merge($admin_pages, array('index.php', 'post-new.php', 'post.php')), true)) ? true : false;
 	}
-	return (bool) apply_filters( 'give_is_admin_page', $found, $query_args['page'], $query_args['view'], $passed_page, $passed_view );
+	return (bool) apply_filters('give_is_admin_page', $found, $query_args['page'], $query_args['view'], $passed_page, $passed_view);
 }
 
 /**
@@ -236,37 +236,37 @@ 
  * @param  array $settings
  * @return array
  */
-function give_settings_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
-	include( 'settings/class-settings-cmb2-backward-compatibility.php' );
+function give_settings_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
+	include('settings/class-settings-cmb2-backward-compatibility.php');
 
 	$settings = array(
 		// General settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-general.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-general.php'),
 
 		// Payment Gateways Settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-gateways.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-gateways.php'),
 
 		// Display settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-display.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-display.php'),
 
 		// Emails settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-email.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-email.php'),
 
 		// Addons settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-addon.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-addon.php'),
 
 		// License settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-license.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-license.php'),
 
 		// Advanced settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-advanced.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-advanced.php'),
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-settings_get_settings_pages', 'give_settings_page_pages', 0, 1 );
+add_filter('give-settings_get_settings_pages', 'give_settings_page_pages', 0, 1);
 
 
 /**
@@ -276,25 +276,25 @@ 
  * @param  array $settings
  * @return array
  */
-function give_reports_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
+function give_reports_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
 
 	$settings = array(
 		// Earnings.
-		include( 'reports/class-earnings-report.php' ),
+		include('reports/class-earnings-report.php'),
 
 		// Forms.
-		include( 'reports/class-forms-report.php' ),
+		include('reports/class-forms-report.php'),
 
 		// Gateways.
-		include( 'reports/class-gateways-report.php' ),
+		include('reports/class-gateways-report.php'),
 
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-reports_get_settings_pages', 'give_reports_page_pages', 0, 1 );
+add_filter('give-reports_get_settings_pages', 'give_reports_page_pages', 0, 1);
 
 /**
  * Add setting tab to give-settings page
@@ -303,34 +303,34 @@ 
  * @param  array $settings
  * @return array
  */
-function give_tools_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
+function give_tools_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
 
 	$settings = array(
 
 		// Export.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-export.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-export.php'),
 
 		// Import
-		include_once( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-import.php' ),
+		include_once(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-import.php'),
 
 		// Logs.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-logs.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-logs.php'),
 
 		// API.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-api.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-api.php'),
 
 		// Data.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-data.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-data.php'),
 
 		// System Info.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-system-info.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-system-info.php'),
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-tools_get_settings_pages', 'give_tools_page_pages', 0, 1 );
+add_filter('give-tools_get_settings_pages', 'give_tools_page_pages', 0, 1);
 
 /**
  * Set default tools page tab.
@@ -339,10 +339,10 @@ 
  * @param  string $default_tab Default tab name.
  * @return string
  */
-function give_set_default_tab_form_tools_page( $default_tab ) {
+function give_set_default_tab_form_tools_page($default_tab) {
 	return 'export';
 }
-add_filter( 'give_default_setting_tab_give-tools', 'give_set_default_tab_form_tools_page', 10, 1 );
+add_filter('give_default_setting_tab_give-tools', 'give_set_default_tab_form_tools_page', 10, 1);
 
 
 /**
@@ -352,10 +352,10 @@ 
  * @param  string $default_tab Default tab name.
  * @return string
  */
-function give_set_default_tab_form_reports_page( $default_tab ) {
+function give_set_default_tab_form_reports_page($default_tab) {
 	return 'earnings';
 }
-add_filter( 'give_default_setting_tab_give-reports', 'give_set_default_tab_form_reports_page', 10, 1 );
+add_filter('give_default_setting_tab_give-reports', 'give_set_default_tab_form_reports_page', 10, 1);
 
 
 /**
@@ -368,19 +368,19 @@ 
  *
  * @return array
  */
-function give_add_display_page_states( $post_states, $post ) {
+function give_add_display_page_states($post_states, $post) {
 
-	switch ( $post->ID ) {
-		case give_get_option( 'success_page' ):
-			$post_states['give_successfully_page'] = __( 'Donation Success Page', 'give' );
+	switch ($post->ID) {
+		case give_get_option('success_page'):
+			$post_states['give_successfully_page'] = __('Donation Success Page', 'give');
 			break;
 
-		case give_get_option( 'failure_page' ):
-			$post_states['give_failure_page'] = __( 'Donation Failed Page', 'give' );
+		case give_get_option('failure_page'):
+			$post_states['give_failure_page'] = __('Donation Failed Page', 'give');
 			break;
 
-		case give_get_option( 'history_page' ):
-			$post_states['give_history_page'] = __( 'Donation History Page', 'give' );
+		case give_get_option('history_page'):
+			$post_states['give_history_page'] = __('Donation History Page', 'give');
 			break;
 	}
 
@@ -388,4 +388,4 @@ 
 }
 
 // Add a post display state for special Give pages.
-add_filter( 'display_post_states', 'give_add_display_page_states', 10, 2 );
+add_filter('display_post_states', 'give_add_display_page_states', 10, 2);

includes/admin/class-api-keys-table.php

Spacing +66 added lines, -66 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -53,11 +53,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'API Key', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'API Keys', 'give' ),    // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => esc_html__('API Key', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('API Keys', 'give'), // Plural name of the listed records
 			'ajax'     => false, // Does this table support ajax?
-		) );
+		));
 
 		$this->query();
 	}
@@ -73,8 +73,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		return $item[ $column_name ];
+	public function column_default($item, $column_name) {
+		return $item[$column_name];
 	}
 
 	/**
@@ -87,8 +87,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_key( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['key'] ) . '"/>';
+	public function column_key($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['key']).'"/>';
 	}
 
 	/**
@@ -101,8 +101,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_token( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['token'] ) . '"/>';
+	public function column_token($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['token']).'"/>';
 	}
 
 	/**
@@ -115,8 +115,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_secret( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['secret'] ) . '"/>';
+	public function column_secret($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['secret']).'"/>';
 	}
 
 	/**
@@ -126,46 +126,46 @@ 
 	 * @since  1.1
 	 * @return string
 	 */
-	public function column_user( $item ) {
+	public function column_user($item) {
 
 		$actions = array();
 
-		if ( apply_filters( 'give_api_log_requests', true ) ) {
+		if (apply_filters('give_api_log_requests', true)) {
 			$actions['view'] = sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( add_query_arg( array(
+				esc_url(add_query_arg(array(
 					'section'   => 'api_requests',
 					'post_type' => 'give_forms',
 					'page'      => 'give-tools',
 					'tab'       => 'logs',
 					's'         => $item['email'],
-				), 'edit.php' ) ),
-				esc_html__( 'View API Log', 'give' )
+				), 'edit.php')),
+				esc_html__('View API Log', 'give')
 			);
 		}
 
 		$actions['reissue'] = sprintf(
 			'<a href="%s" class="give-regenerate-api-key">%s</a>',
-			esc_url( wp_nonce_url( add_query_arg( array(
+			esc_url(wp_nonce_url(add_query_arg(array(
 				'user_id'          => $item['id'],
 				'give_action'      => 'process_api_key',
 				'give_api_process' => 'regenerate',
-			) ), 'give-api-nonce' ) ),
-			esc_html__( 'Reissue', 'give' )
+			)), 'give-api-nonce')),
+			esc_html__('Reissue', 'give')
 		);
-		$actions['revoke']  = sprintf(
+		$actions['revoke'] = sprintf(
 			'<a href="%s" class="give-revoke-api-key give-delete">%s</a>',
-			esc_url( wp_nonce_url( add_query_arg( array(
+			esc_url(wp_nonce_url(add_query_arg(array(
 				'user_id'          => $item['id'],
 				'give_action'      => 'process_api_key',
 				'give_api_process' => 'revoke',
-			) ), 'give-api-nonce' ) ),
-			esc_html__( 'Revoke', 'give' )
+			)), 'give-api-nonce')),
+			esc_html__('Revoke', 'give')
 		);
 
-		$actions = apply_filters( 'give_api_row_actions', array_filter( $actions ) );
+		$actions = apply_filters('give_api_row_actions', array_filter($actions));
 
-		return sprintf( '%1$s %2$s', $item['user'], $this->row_actions( $actions ) );
+		return sprintf('%1$s %2$s', $item['user'], $this->row_actions($actions));
 	}
 
 	/**
@@ -189,10 +189,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'user'   => esc_html__( 'Username', 'give' ),
-			'key'    => esc_html__( 'Public Key', 'give' ),
-			'token'  => esc_html__( 'Token', 'give' ),
-			'secret' => esc_html__( 'Secret Key', 'give' ),
+			'user'   => esc_html__('Username', 'give'),
+			'key'    => esc_html__('Public Key', 'give'),
+			'token'  => esc_html__('Token', 'give'),
+			'secret' => esc_html__('Secret Key', 'give'),
 		);
 
 		return $columns;
@@ -206,20 +206,20 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -237,18 +237,18 @@ 
 	 *
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place.
 		static $give_api_is_bottom;
 
-		if ( $give_api_is_bottom ) {
+		if ($give_api_is_bottom) {
 			return;
 		}
 		?>
 		<input type="hidden" name="give_action" value="process_api_key"/>
 		<input type="hidden" name="give_api_process" value="generate"/>
 		<?php
-		wp_nonce_field( 'give-api-nonce' );
+		wp_nonce_field('give-api-nonce');
 		/**
 		 * API Key user search.
 		 */
@@ -257,9 +257,9 @@ 
 			'name' => 'user_id',
 		);
 
-		echo Give()->html->ajax_user_search( $args );
+		echo Give()->html->ajax_user_search($args);
 
-		submit_button( esc_html__( 'Generate New API Keys', 'give' ), 'secondary', 'submit', false );
+		submit_button(esc_html__('Generate New API Keys', 'give'), 'secondary', 'submit', false);
 
 		$give_api_is_bottom = true;
 	}
@@ -272,7 +272,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -283,21 +283,21 @@ 
 	 * @return array
 	 */
 	public function query() {
-		$users = get_users( array(
+		$users = get_users(array(
 			'meta_value' => 'give_user_secret_key',
 			'number'     => $this->per_page,
-			'offset'     => $this->per_page * ( $this->get_paged() - 1 ),
-		) );
-		$keys  = array();
-
-		foreach ( $users as $user ) {
-			$keys[ $user->ID ]['id']    = $user->ID;
-			$keys[ $user->ID ]['email'] = $user->user_email;
-			$keys[ $user->ID ]['user']  = '<a href="' . add_query_arg( 'user_id', $user->ID, 'user-edit.php' ) . '"><strong>' . $user->user_login . '</strong></a>';
-
-			$keys[ $user->ID ]['key']    = Give()->api->get_user_public_key( $user->ID );
-			$keys[ $user->ID ]['secret'] = Give()->api->get_user_secret_key( $user->ID );
-			$keys[ $user->ID ]['token']  = Give()->api->get_token( $user->ID );
+			'offset'     => $this->per_page * ($this->get_paged() - 1),
+		));
+		$keys = array();
+
+		foreach ($users as $user) {
+			$keys[$user->ID]['id']    = $user->ID;
+			$keys[$user->ID]['email'] = $user->user_email;
+			$keys[$user->ID]['user']  = '<a href="'.add_query_arg('user_id', $user->ID, 'user-edit.php').'"><strong>'.$user->user_login.'</strong></a>';
+
+			$keys[$user->ID]['key']    = Give()->api->get_user_public_key($user->ID);
+			$keys[$user->ID]['secret'] = Give()->api->get_user_secret_key($user->ID);
+			$keys[$user->ID]['token']  = Give()->api->get_token($user->ID);
 		}
 
 		return $keys;
@@ -314,9 +314,9 @@ 
 	public function total_items() {
 		global $wpdb;
 
-		$total_items = Give_Cache::get( 'give_total_api_keys', true );
+		$total_items = Give_Cache::get('give_total_api_keys', true);
 
-		if ( ! $total_items ) {
+		if ( ! $total_items) {
 			$total_items = $wpdb->get_var(
 				$wpdb->prepare(
 					"SELECT count(user_id)
@@ -325,7 +325,7 @@ 
 				)
 			);
 
-			Give_Cache::set( 'give_total_api_keys', $total_items, HOUR_IN_SECONDS, true );
+			Give_Cache::set('give_total_api_keys', $total_items, HOUR_IN_SECONDS, true);
 		}
 
 		return $total_items;
@@ -344,7 +344,7 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = array(); // Not sortable... for now
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$data = $this->query();
 
@@ -352,10 +352,10 @@ 
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $total_items,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $total_items / $this->per_page ),
-		) );
+			'total_pages' => ceil($total_items / $this->per_page),
+		));
 	}
 }

includes/admin/donors/donors.php

Spacing +218 added lines, -218 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  *
  * @return string
  */
-function __give_get_format_address( $address, $address_args = array() ) {
+function __give_get_format_address($address, $address_args = array()) {
 	$address_html = '';
 	$address_args = wp_parse_args(
 		$address_args,
@@ -40,7 +40,7 @@ 
 	$address_id = $address_args['type'];
 
 	// Bailout.
-	if ( empty( $address ) || ! is_array( $address ) ) {
+	if (empty($address) || ! is_array($address)) {
 		return $address_html;
 	}
 
@@ -49,19 +49,19 @@ 
 	$address_html .= sprintf(
 		'<span data-address-type="line1">%1$s</span>%2$s',
 		$address['line1'],
-		( ! empty( $address['line2'] ) ? '<br>' : '' )
+		( ! empty($address['line2']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="line2">%1$s</span>%2$s',
 		$address['line2'],
-		( ! empty( $address['city'] ) ? '<br>' : '' )
+		( ! empty($address['city']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="city">%1$s</span><span data-address-type="state">%2$s</span><span data-address-type="zip">%3$s</span>%4$s',
 		$address['city'],
-		( ! empty( $address['state'] ) ? ", {$address['state']}" : '' ),
-		( ! empty( $address['zip'] ) ? " {$address['zip']}" : '' ),
-		( ! empty( $address['country'] ) ? '<br>' : '' )
+		( ! empty($address['state']) ? ", {$address['state']}" : ''),
+		( ! empty($address['zip']) ? " {$address['zip']}" : ''),
+		( ! empty($address['country']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="country">%s</span><br>',
@@ -71,8 +71,8 @@ 
 	// Address action.
 	$address_html .= sprintf(
 		'<br><a href="#" class="js-edit">%1$s</a> | <a href="#" class="js-remove">%2$s</a>',
-		__( 'Edit', 'give' ),
-		__( 'Remove', 'give' )
+		__('Edit', 'give'),
+		__('Remove', 'give')
 	);
 
 	/**
@@ -80,14 +80,14 @@ 
 	 *
 	 * @since 2.0
 	 */
-	$address_label = apply_filters( "give_donor_{$address_args['type']}_address_label", ucfirst( $address_args['type'] ), $address_args );
+	$address_label = apply_filters("give_donor_{$address_args['type']}_address_label", ucfirst($address_args['type']), $address_args);
 
 	// Set unique id and index for multi type address.
-	if ( isset( $address_args['index'] ) ) {
+	if (isset($address_args['index'])) {
 		$address_label = "{$address_label} #{$address_args['index']}";
 	}
 
-	if ( isset( $address_args['id'] ) ) {
+	if (isset($address_args['id'])) {
 		$address_id = "{$address_id}_{$address_args['id']}";
 	}
 
@@ -112,9 +112,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -130,7 +130,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -144,7 +144,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -156,7 +156,7 @@ 
  */
 function give_donors_list() {
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/donors/class-donor-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/donors/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -169,12 +169,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
 
 		<hr class="wp-header-end">
-		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
-			<?php $donors_table->search_box( __( 'Search Donors', 'give' ), 'give-donors' ); ?>
+		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
+			<?php $donors_table->search_box(__('Search Donors', 'give'), 'give-donors'); ?>
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-donors" />
 			<input type="hidden" name="view" value="donors" />
@@ -191,7 +191,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -207,33 +207,33 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id          = (int) $_GET['id'];
-	$reconnect_user_id = ! empty( $_GET['user_id'] ) ? (int) $_GET['user_id'] : '';
-	$donor             = new Give_Donor( $donor_id );
+	$reconnect_user_id = ! empty($_GET['user_id']) ? (int) $_GET['user_id'] : '';
+	$donor             = new Give_Donor($donor_id);
 
 	// Reconnect User with Donor profile.
-	if ( $reconnect_user_id ) {
-		give_connect_user_donor_profile( $donor, array( 'user_id' => $reconnect_user_id ), array() );
+	if ($reconnect_user_id) {
+		give_connect_user_donor_profile($donor, array('user_id' => $reconnect_user_id), array());
 	}
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -242,9 +242,9 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 
@@ -252,7 +252,7 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor first name */
-				__( 'Edit Donor: %s %s', 'give' ),
+				__('Edit Donor: %s %s', 'give'),
 				$donor->get_first_name(),
 				$donor->get_last_name()
 			);
@@ -261,26 +261,26 @@ 
 
 		<hr class="wp-header-end">
 
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 				<?php
-				foreach ( $donor_tabs as $key => $tab ) :
+				foreach ($donor_tabs as $key => $tab) :
 					$active = $key === $view ? true : false;
 					$class  = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 					printf(
-						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-						esc_attr( $class ),
-						sanitize_html_class( $tab['dashicon'] ),
-						esc_html( $tab['title'] )
+						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+						esc_attr($class),
+						sanitize_html_class($tab['dashicon']),
+						esc_html($tab['title'])
 					);
 				endforeach;
 				?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -300,9 +300,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -311,11 +311,11 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 
 	// Set Read only to the fields which needs to be locked.
 	$read_only = '';
-	if ( $donor->user_id ) {
+	if ($donor->user_id) {
 		$read_only = 'readonly="readonly"';
 	}
 	?>
@@ -323,32 +323,32 @@ 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
 		<form id="edit-donor-info" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
 						<span class="donor-name info-item edit-item">
 							<input <?php echo $read_only; ?> size="15" data-key="first_name"
 							                                 name="customerinfo[first_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_first_name() ); ?>"
-							                                 placeholder="<?php _e( 'First Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_first_name()); ?>"
+							                                 placeholder="<?php _e('First Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
 							<?php endif; ?>
 							<input <?php echo $read_only; ?> size="15" data-key="last_name"
 							                                 name="customerinfo[last_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_last_name() ); ?>"
-							                                 placeholder="<?php _e( 'Last Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_last_name()); ?>"
+							                                 placeholder="<?php _e('Last Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
@@ -358,12 +358,12 @@ 
 									data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php _e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php _e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
+					<?php if (current_user_can($donor_edit_role)) : ?>
 						<a href="#" id="edit-donor"
-						   class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+						   class="button info-item editable donor-edit-link"><?php _e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -373,11 +373,11 @@ 
 					<table class="widefat striped">
 						<tbody>
 						<tr>
-							<th scope="col"><label for="tablecell"><?php _e( 'Donor ID:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('Donor ID:', 'give'); ?></label></th>
 							<td><?php echo $donor->id; ?></td>
 						</tr>
 						<tr>
-							<th scope="col"><label for="tablecell"><?php _e( 'User ID:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('User ID:', 'give'); ?></label></th>
 							<td>
 									<span class="donor-user-id info-item edit-item">
 										<?php
@@ -394,22 +394,22 @@ 
 											'data'  => $data_atts,
 										);
 
-										if ( ! empty( $user_id ) ) {
-											$userdata              = get_userdata( $user_id );
+										if ( ! empty($user_id)) {
+											$userdata              = get_userdata($user_id);
 											$user_args['selected'] = $user_id;
 										}
 
-										echo Give()->html->ajax_user_search( $user_args );
+										echo Give()->html->ajax_user_search($user_args);
 										?>
 									</span>
 
 								<span class="donor-user-id info-item editable">
-										<?php if ( ! empty( $userdata ) ) : ?>
-											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+										<?php if ( ! empty($userdata)) : ?>
+											<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'Unregistered', 'give' ); ?></span>
+											<span data-key="user_id"><?php _e('Unregistered', 'give'); ?></span>
 										<?php endif; ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0):
 
 										echo sprintf(
 											'- <span class="disconnect-user">
@@ -418,11 +418,11 @@ 
 				                                       | <span class="view-user-profile">
  											                <a id="view-user-profile" href="%3$s" aria-label="%4$s">%5$s</a>
  										               </span>',
-											__( 'Disconnects the current user ID from this donor record.', 'give' ),
-											__( 'Disconnect User', 'give' ),
-											'user-edit.php?user_id=' . $donor->user_id,
-											__( 'View User Profile of current user ID.', 'give' ),
-											__( 'View User Profile', 'give' )
+											__('Disconnects the current user ID from this donor record.', 'give'),
+											__('Disconnect User', 'give'),
+											'user-edit.php?user_id='.$donor->user_id,
+											__('View User Profile of current user ID.', 'give'),
+											__('View User Profile', 'give')
 										);
 
 										endif; ?>
@@ -431,13 +431,13 @@ 
 						</tr>
 
 						<?php
-						$donor_company = $donor->get_meta( '_give_donor_company', true );
+						$donor_company = $donor->get_meta('_give_donor_company', true);
 
-						if ( ! empty( $donor_company ) ) {
+						if ( ! empty($donor_company)) {
 							?>
 							<tr class="alternate">
 								<th scope="col">
-									<label for="tablecell"><?php _e( 'Company Name:', 'give' ); ?></label>
+									<label for="tablecell"><?php _e('Company Name:', 'give'); ?></label>
 								</th>
 								<td><?php echo $donor_company; ?></td>
 							</tr>
@@ -453,11 +453,11 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>"/>
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor"/>
 				<input type="submit" id="give-edit-donor-save" class="button-secondary"
-				       value="<?php _e( 'Update Donor', 'give' ); ?>"/>
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+				       value="<?php _e('Update Donor', 'give'); ?>"/>
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -472,24 +472,24 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations.
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->get_total_donation_amount(), array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->get_total_donation_amount(), array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -501,7 +501,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -514,11 +514,11 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_address', $donor );
+	do_action('give_donor_before_address', $donor);
 	?>
 
 	<div id="donor-address-wrapper" class="donor-section clear">
-		<h3><?php _e( 'Addresses', 'give' ); ?></h3>
+		<h3><?php _e('Addresses', 'give'); ?></h3>
 
 		<div class="postbox">
 			<div class="give-spinner-wrapper">
@@ -528,16 +528,16 @@ 
 				<div class="all-address">
 					<div class="give-grid-row">
 						<?php
-						if ( ! empty( $donor->address ) ) :
+						if ( ! empty($donor->address)) :
 							// Default address always will be at zero array index.
 							$is_set_as_default = null;
 
-							foreach ( $donor->address as $address_type => $addresses ) {
+							foreach ($donor->address as $address_type => $addresses) {
 
-								switch ( true ) {
-									case is_array( end( $addresses ) ):
+								switch (true) {
+									case is_array(end($addresses)):
 										$index = 1;
-										foreach ( $addresses as $id => $address ) {
+										foreach ($addresses as $id => $address) {
 											echo __give_get_format_address(
 												$address,
 												array(
@@ -547,11 +547,11 @@ 
 												)
 											);
 
-											$index ++;
+											$index++;
 										}
 										break;
 
-									case is_string( end( $addresses ) ):
+									case is_string(end($addresses)):
 										echo __give_get_format_address(
 											$addresses,
 											array(
@@ -564,13 +564,13 @@ 
 						endif;
 						?>
 					</div>
-					<span class="give-no-address-message<?php if ( ! empty( $donor->address ) ) {
+					<span class="give-no-address-message<?php if ( ! empty($donor->address)) {
 						echo ' give-hidden';
 					} ?>">
-						<?php _e( 'This donor does not have any addresses saved.', 'give' ); ?>
+						<?php _e('This donor does not have any addresses saved.', 'give'); ?>
 					</span>
 					<button class="button add-new-address">
-						<?php _e( 'Add Address', 'give' ); ?>
+						<?php _e('Add Address', 'give'); ?>
 					</button>
 				</div>
 
@@ -580,26 +580,26 @@ 
 							<tbody>
 							<tr>
 								<th class="col">
-									<label class="country"><?php esc_html_e( 'Country:', 'give' ); ?></label>
+									<label class="country"><?php esc_html_e('Country:', 'give'); ?></label>
 								</th>
 								<td>
 									<?php
-									echo Give()->html->select( array(
+									echo Give()->html->select(array(
 										'options'          => give_get_country_list(),
 										'name'             => 'country',
-										'selected'         => give_get_option( 'base_country' ),
+										'selected'         => give_get_option('base_country'),
 										'show_option_all'  => false,
 										'show_option_none' => false,
 										'chosen'           => true,
-										'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-										'data'             => array( 'search-type' => 'no_ajax' ),
-									) );
+										'placeholder'      => esc_attr__('Select a country', 'give'),
+										'data'             => array('search-type' => 'no_ajax'),
+									));
 									?>
 								</td>
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line1"><?php esc_html_e( 'Address 1:', 'give' ); ?></label>
+									<label for="line1"><?php esc_html_e('Address 1:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line1" name="line1" type="text" class="medium-text"/>
@@ -607,7 +607,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line2"><?php esc_html_e( 'Address 2:', 'give' ); ?></label>
+									<label for="line2"><?php esc_html_e('Address 2:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line2" type="text" name="line2" value="" class="medium-text"/>
@@ -616,7 +616,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="city"><?php esc_html_e( 'City:', 'give' ); ?></label>
+									<label for="city"><?php esc_html_e('City:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="city" type="text" name="city" value="" class="medium-text"/>
@@ -624,41 +624,41 @@ 
 							</tr>
 							<?php
 							$no_states_country = give_no_states_country_list();
-							$base_country      = give_get_option( 'base_country' );
-							if ( ! array_key_exists( $base_country, $no_states_country ) ) {
+							$base_country      = give_get_option('base_country');
+							if ( ! array_key_exists($base_country, $no_states_country)) {
 								?>
 								<tr class="give-field-wrap">
 									<th class="col">
-										<label for="state"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+										<label for="state"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 									</th>
 									<td>
 										<?php
-										$states     = give_get_states( $base_country );
+										$states     = give_get_states($base_country);
 										$state_args = array(
 											'name'  => 'state',
 											'class' => 'regular-text',
 										);
 
-										if ( empty( $states ) ) {
+										if (empty($states)) {
 
 											// Show Text field, if empty states.
-											$state_args = wp_parse_args( $state_args, array(
-												'value' => give_get_option( 'base_state' ),
-											) );
-											echo Give()->html->text( $state_args );
+											$state_args = wp_parse_args($state_args, array(
+												'value' => give_get_option('base_state'),
+											));
+											echo Give()->html->text($state_args);
 										} else {
 
 											// Show Chosen DropDown, if states are not empty.
-											$state_args = wp_parse_args( $state_args, array(
+											$state_args = wp_parse_args($state_args, array(
 												'options'          => $states,
-												'selected'         => give_get_option( 'base_state' ),
+												'selected'         => give_get_option('base_state'),
 												'show_option_all'  => false,
 												'show_option_none' => false,
 												'chosen'           => true,
-												'placeholder'      => __( 'Select a state', 'give' ),
-												'data'             => array( 'search-type' => 'no_ajax' ),
-											) );
-											echo Give()->html->select( $state_args );
+												'placeholder'      => __('Select a state', 'give'),
+												'data'             => array('search-type' => 'no_ajax'),
+											));
+											echo Give()->html->select($state_args);
 										}
 										?>
 									</td>
@@ -668,7 +668,7 @@ 
 							?>
 							<tr>
 								<th class="col">
-									<label for="zip"><?php esc_html_e( 'Zip / Postal Code:', 'give' ); ?></label>
+									<label for="zip"><?php esc_html_e('Zip / Postal Code:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="zip" type="text" name="zip" value="" class="medium-text"/>
@@ -676,12 +676,12 @@ 
 							</tr>
 							<tr>
 								<td colspan="2">
-									<?php wp_nonce_field( 'give-manage-donor-addresses', '_wpnonce', false ); ?>
+									<?php wp_nonce_field('give-manage-donor-addresses', '_wpnonce', false); ?>
 									<input type="hidden" name="address-action" value="add">
 									<input type="hidden" name="address-id" value="">
 									<input type="submit" class="button button-primary js-save"
-									       value="<?php _e( 'Save', 'give' ); ?>">&nbsp;&nbsp;<button
-											class="button js-cancel"><?php _e( 'Cancel', 'give' ); ?></button>
+									       value="<?php _e('Save', 'give'); ?>">&nbsp;&nbsp;<button
+											class="button js-cancel"><?php _e('Cancel', 'give'); ?></button>
 								</td>
 							</tr>
 							</tbody>
@@ -700,7 +700,7 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -713,46 +713,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 			<tr>
-				<th><?php _e( 'Email', 'give' ); ?></th>
-				<th><?php _e( 'Actions', 'give' ); ?></th>
+				<th><?php _e('Email', 'give'); ?></th>
+				<th><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 
 			<tbody>
-			<?php if ( ! empty( $donor->emails ) ) { ?>
+			<?php if ( ! empty($donor->emails)) { ?>
 
-				<?php foreach ( $donor->emails as $key => $email ) : ?>
+				<?php foreach ($donor->emails as $key => $email) : ?>
 					<tr data-key="<?php echo $key; ?>">
 						<td>
 							<?php echo $email; ?>
-							<?php if ( 'primary' === $key ) : ?>
+							<?php if ('primary' === $key) : ?>
 								<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 							<?php endif; ?>
 						</td>
 						<td>
-							<?php if ( 'primary' !== $key ) : ?>
+							<?php if ('primary' !== $key) : ?>
 								<?php
-								$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-								$promote_url = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+								$promote_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'set_donor_primary_email',
-								), $base_url ), 'give-set-donor-primary-email' );
-								$remove_url  = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								), $base_url), 'give-set-donor-primary-email');
+								$remove_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'remove_donor_email',
-								), $base_url ), 'give-remove-donor-email' );
+								), $base_url), 'give-remove-donor-email');
 								?>
-								<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+								<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 								&nbsp;|&nbsp;
-								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 							<?php endif; ?>
 						</td>
 					</tr>
@@ -762,14 +762,14 @@ 
 					<td colspan="2" class="add-donor-email-td">
 						<div class="add-donor-email-wrapper">
 							<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>"/>
-							<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
+							<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
 							<input type="email" name="additional-email" value=""
-							       placeholder="<?php _e( 'Email Address', 'give' ); ?>"/>&nbsp;
+							       placeholder="<?php _e('Email Address', 'give'); ?>"/>&nbsp;
 							<input type="checkbox" name="make-additional-primary" value="1"
 							       id="make-additional-primary"/>&nbsp;<label
-									for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
+									for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
 							<button class="button-secondary give-add-donor-email"
-							        id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+							        id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 							<span class="spinner"></span>
 						</div>
 						<div class="notice-wrap"></div>
@@ -777,50 +777,50 @@ 
 				</tr>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No Emails Found', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php _e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('ID', 'give'); ?></th>
+				<th scope="col"><?php _e('Amount', 'give'); ?></th>
+				<th scope="col"><?php _e('Date', 'give'); ?></th>
+				<th scope="col"><?php _e('Status', 'give'); ?></th>
+				<th scope="col"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
-						<td><?php echo Give()->seq_donation_number->get_serial_code( $payment->ID ); ?></td>
-						<td><?php echo give_donation_amount( $payment->ID, array( 'currency' => true, 'amount' => true, 'type' => 'donor' ) ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo Give()->seq_donation_number->get_serial_code($payment->ID); ?></td>
+						<td><?php echo give_donation_amount($payment->ID, array('currency' => true, 'amount' => true, 'type' => 'donor')); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 								/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								__( 'View Donation', 'give' )
+								__('View Donation', 'give')
 							);
 							?>
 
@@ -835,47 +835,47 @@ 
 							 * @param object $donor   The donor object being displayed.
 							 * @param object $payment The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php _e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php _e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php _e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 								/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								__( 'View Form', 'give' )
+								__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -883,7 +883,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -897,7 +897,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -910,7 +910,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -923,31 +923,31 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged       = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged       = absint( $paged );
+	$paged       = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged       = absint($paged);
 	$note_count  = $donor->get_notes_count();
-	$per_page    = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page    = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php _e( 'Notes', 'give' ); ?></h3>
+		<h3><?php _e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
 				<form id="give-add-donor-note" method="post"
-				      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br/>
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>"/>
 					<input type="hidden" name="give_action" value="add-donor-note"/>
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note"/>
 				</form>
 			</div>
@@ -962,26 +962,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php _e( 'No donor notes found.', 'give' ); ?>
+					<?php _e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -997,9 +997,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -1008,16 +1008,16 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
 		<form id="delete-donor" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -1025,20 +1025,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php _e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php _e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php _e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php _e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -1051,19 +1051,19 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>"/>
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor"/>
 					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary"
-					       value="<?php _e( 'Delete Donor', 'give' ); ?>"/>
+					       value="<?php _e('Delete Donor', 'give'); ?>"/>
 					<a id="give-delete-donor-cancel"
-					   href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>"
-					   class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+					   href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>"
+					   class="delete"><?php _e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -1079,5 +1079,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

Braces +5 added lines, -2 removed lines

@@ -406,8 +406,11 @@
 								<span class="donor-user-id info-item editable">
 										<?php if ( ! empty( $userdata ) ) : ?>
 											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
-										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'Unregistered', 'give' ); ?></span>
+										<?php else {
+	: ?>
+											<span data-key="user_id"><?php _e( 'Unregistered', 'give' );
+}
+?></span>
 										<?php endif; ?>
 									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
 

includes/admin/add-ons.php

Spacing +15 added lines, -15 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
 function give_add_ons_page() {
 	?>
 	<div class="wrap" id="give-add-ons">
-		<h1><?php echo esc_html( get_admin_page_title() ); ?>
-			&nbsp;&mdash;&nbsp;<a href="https://givewp.com/addons/" class="button-primary give-view-addons-all" target="_blank"><?php esc_html_e( 'View All Add-ons', 'give' ); ?>
+		<h1><?php echo esc_html(get_admin_page_title()); ?>
+			&nbsp;&mdash;&nbsp;<a href="https://givewp.com/addons/" class="button-primary give-view-addons-all" target="_blank"><?php esc_html_e('View All Add-ons', 'give'); ?>
 				<span class="dashicons dashicons-external"></span></a>
 		</h1>
 
 		<hr class="wp-header-end">
 
-		<p><?php esc_html_e( 'The following Add-ons extend the functionality of Give.', 'give' ); ?></p>
+		<p><?php esc_html_e('The following Add-ons extend the functionality of Give.', 'give'); ?></p>
 		<?php give_add_ons_feed(); ?>
 	</div>
 	<?php
@@ -49,27 +49,27 @@ 
 function give_add_ons_feed() {
 
 	$addons_debug = false; //set to true to debug
-	$cache        = Give_Cache::get( 'give_add_ons_feed', true );
+	$cache        = Give_Cache::get('give_add_ons_feed', true);
 
-	if ( false === $cache || ( true === $addons_debug && true === WP_DEBUG ) ) {
-		if ( function_exists( 'vip_safe_wp_remote_get' ) ) {
-			$feed = vip_safe_wp_remote_get( 'https://givewp.com/downloads/feed/', false, 3, 1, 20, array( 'sslverify' => false ) );
+	if (false === $cache || (true === $addons_debug && true === WP_DEBUG)) {
+		if (function_exists('vip_safe_wp_remote_get')) {
+			$feed = vip_safe_wp_remote_get('https://givewp.com/downloads/feed/', false, 3, 1, 20, array('sslverify' => false));
 		} else {
-			$feed = wp_remote_get( 'https://givewp.com/downloads/feed/', array( 'sslverify' => false ) );
+			$feed = wp_remote_get('https://givewp.com/downloads/feed/', array('sslverify' => false));
 		}
 
-		if ( ! is_wp_error( $feed ) && ! empty( $feed ) ) {
-			if ( isset( $feed['body'] ) && strlen( $feed['body'] ) > 0 ) {
-				$cache = wp_remote_retrieve_body( $feed );
-				Give_Cache::set( 'give_add_ons_feed', $cache, HOUR_IN_SECONDS, true );
+		if ( ! is_wp_error($feed) && ! empty($feed)) {
+			if (isset($feed['body']) && strlen($feed['body']) > 0) {
+				$cache = wp_remote_retrieve_body($feed);
+				Give_Cache::set('give_add_ons_feed', $cache, HOUR_IN_SECONDS, true);
 			}
 		} else {
 			$cache = sprintf(
 				'<div class="error"><p>%s</p></div>',
-				esc_html__( 'There was an error retrieving the Give Add-ons list from the server. Please try again later.', 'give' )
+				esc_html__('There was an error retrieving the Give Add-ons list from the server. Please try again later.', 'give')
 			);
 		}
 	}
 
-	echo wp_kses_post( $cache );
+	echo wp_kses_post($cache);
 }

includes/admin/class-blank-slate.php

Spacing +55 added lines, -55 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.8.13
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -75,63 +75,63 @@ 
 	 */
 	public function init() {
 		// Bail early if screen cannot be detected.
-		if ( empty( $this->screen ) ) {
+		if (empty($this->screen)) {
 			return null;
 		}
 
 		$content = array();
 
 		// Define content and hook into the appropriate action.
-		switch ( $this->screen ) {
+		switch ($this->screen) {
 			// Forms screen.
 			case 'edit-give_forms':
-				$this->form = $this->post_exists( 'give_forms' );
+				$this->form = $this->post_exists('give_forms');
 
-				if ( $this->form ) {
+				if ($this->form) {
 					// Form exists. Bail out.
 					return false;
 				} else {
 					// No forms exist.
-					$content = $this->get_content( 'no_forms' );
+					$content = $this->get_content('no_forms');
 				}
 
-				add_action( 'manage_posts_extra_tablenav', array( $this, 'render' ) );
+				add_action('manage_posts_extra_tablenav', array($this, 'render'));
 				break;
 			// Donations screen.
 			case 'give_forms_page_give-payment-history':
-				$this->form     = $this->post_exists( 'give_forms' );
-				$this->donation = $this->post_exists( 'give_payment' );
+				$this->form     = $this->post_exists('give_forms');
+				$this->donation = $this->post_exists('give_payment');
 
-				if ( $this->donation ) {
+				if ($this->donation) {
 					// Donation exists. Bail out.
 					return false;
-				} elseif ( ! $this->form ) {
+				} elseif ( ! $this->form) {
 					// No forms and no donations exist.
-					$content = $this->get_content( 'no_donations_or_forms' );
+					$content = $this->get_content('no_donations_or_forms');
 				} else {
 					// No donations exist but a form does exist.
-					$content = $this->get_content( 'no_donations' );
+					$content = $this->get_content('no_donations');
 				}
 
-				add_action( 'give_payments_page_bottom', array( $this, 'render' ) );
+				add_action('give_payments_page_bottom', array($this, 'render'));
 				break;
 			// Donors screen.
 			case 'give_forms_page_give-donors':
-				$this->form  = $this->post_exists( 'give_forms' );
+				$this->form  = $this->post_exists('give_forms');
 				$this->donor = $this->donor_exists();
 
-				if ( $this->donor ) {
+				if ($this->donor) {
 					// Donor exists. Bail out.
 					return false;
-				} elseif ( ! $this->form ) {
+				} elseif ( ! $this->form) {
 					// No forms and no donors exist.
-					$content = $this->get_content( 'no_donors_or_forms' );
+					$content = $this->get_content('no_donors_or_forms');
 				} else {
 					// No donors exist but a form does exist.
-					$content = $this->get_content( 'no_donors' );
+					$content = $this->get_content('no_donors');
 				}
 
-				add_action( 'give_donors_table_bottom', array( $this, 'render' ) );
+				add_action('give_donors_table_bottom', array($this, 'render'));
 				break;
 			default:
 				return null;
@@ -140,7 +140,7 @@ 
 		$this->content = $content;
 
 		// Hide non-essential UI elements.
-		add_action( 'admin_head', array( $this, 'hide_ui' ) );
+		add_action('admin_head', array($this, 'hide_ui'));
 	}
 
 	/**
@@ -150,9 +150,9 @@ 
 	 *
 	 * @param string $which The location of the list table hook: 'top' or 'bottom'.
 	 */
-	public function render( $which = 'bottom' ) {
+	public function render($which = 'bottom') {
 		// Bail out to prevent content from rendering twice.
-		if ( 'top' === $which ) {
+		if ('top' === $which) {
 			return null;
 		}
 
@@ -177,9 +177,9 @@ 
 		 *
 		 * @param string $screen The current screen ID.
 		 */
-		$content = apply_filters( 'give_blank_slate_content', $this->content, $screen );
+		$content = apply_filters('give_blank_slate_content', $this->content, $screen);
 
-		$template_path = GIVE_PLUGIN_DIR . 'includes/admin/views/blank-slate.php';
+		$template_path = GIVE_PLUGIN_DIR.'includes/admin/views/blank-slate.php';
 
 		include $template_path;
 	}
@@ -214,17 +214,17 @@ 
 	 * @param string $post_type Post type used in the query.
 	 * @return bool True if post exists, otherwise false.
 	 */
-	private function post_exists( $post_type ) {
+	private function post_exists($post_type) {
 		// Attempt to get a single post of the post type.
-		$query = new WP_Query( array(
+		$query = new WP_Query(array(
 			'post_type'              => $post_type,
 			'posts_per_page'         => 1,
 			'no_found_rows'          => false,
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
 			'fields'                 => 'ids',
-			'post_status'            => array( 'any', 'trash' ),
-		) );
+			'post_status'            => array('any', 'trash'),
+		));
 
 		return $query->have_posts();
 	}
@@ -237,9 +237,9 @@ 
 	 * @return bool True if donor exists, otherwise false.
 	 */
 	private function donor_exists() {
-		$donors = Give()->donors->get_donors( array( 'number' => 1 ) );
+		$donors = Give()->donors->get_donors(array('number' => 1));
 
-		return ! empty( $donors );
+		return ! empty($donors);
 	}
 
 	/**
@@ -250,18 +250,18 @@ 
 	 * @param string $context The key used to determine which content is returned.
 	 * @return array Blank slate content.
 	 */
-	private function get_content( $context ) {
+	private function get_content($context) {
 		// Define default content.
 		$defaults = array(
-			'image_url' => GIVE_PLUGIN_URL . 'assets/dist/images/give-icon-full-circle.svg',
-			'image_alt' => __( 'Give Icon', 'give' ),
-			'heading'   => __( 'No donation forms  found.', 'give' ),
-			'message'   => __( 'The first step towards accepting online donations is to create a form.', 'give' ),
-			'cta_text'  => __( 'Create Donation Form', 'give' ),
-			'cta_link'  => admin_url( 'post-new.php?post_type=give_forms' ),
+			'image_url' => GIVE_PLUGIN_URL.'assets/dist/images/give-icon-full-circle.svg',
+			'image_alt' => __('Give Icon', 'give'),
+			'heading'   => __('No donation forms  found.', 'give'),
+			'message'   => __('The first step towards accepting online donations is to create a form.', 'give'),
+			'cta_text'  => __('Create Donation Form', 'give'),
+			'cta_link'  => admin_url('post-new.php?post_type=give_forms'),
 			'help'      => sprintf(
 				/* translators: 1: Opening anchor tag. 2: Closing anchor tag. */
-				__( 'Need help? Get started with %1$sGive 101%2$s.', 'give' ),
+				__('Need help? Get started with %1$sGive 101%2$s.', 'give'),
 				'<a href="http://docs.givewp.com/give101/" target="_blank">',
 				'</a>'
 			),
@@ -270,42 +270,42 @@ 
 		// Define contextual content.
 		$content = array(
 			'no_donations_or_forms' => array(
-				'heading' => __( 'No donations found.', 'give' ),
-				'message' => __( 'Your donation history will appear here, but first, you need a donation form!', 'give' ),
+				'heading' => __('No donations found.', 'give'),
+				'message' => __('Your donation history will appear here, but first, you need a donation form!', 'give'),
 			),
 			'no_donations'          => array(
-				'heading'  => __( 'No donations found.', 'give' ),
-				'message'  => __( 'When your first donation arrives, a record of the donation will appear here.', 'give' ),
-				'cta_text' => __( 'View All Forms', 'give' ),
-				'cta_link' => admin_url( 'edit.php?post_type=give_forms' ),
+				'heading'  => __('No donations found.', 'give'),
+				'message'  => __('When your first donation arrives, a record of the donation will appear here.', 'give'),
+				'cta_text' => __('View All Forms', 'give'),
+				'cta_link' => admin_url('edit.php?post_type=give_forms'),
 				'help'     => sprintf(
 					/* translators: 1: Opening anchor tag. 2: Closing anchor tag. */
-					__( 'Need help? Learn more about %1$sDonations%2$s.', 'give' ),
+					__('Need help? Learn more about %1$sDonations%2$s.', 'give'),
 					'<a href="http://docs.givewp.com/core-donations/">',
 					'</a>'
 				),
 			),
 			'no_donors_or_forms'    => array(
-				'heading' => __( 'No donors  found.', 'give' ),
-				'message' => __( 'Your donor history will appear here, but first, you need a donation form!', 'give' ),
+				'heading' => __('No donors  found.', 'give'),
+				'message' => __('Your donor history will appear here, but first, you need a donation form!', 'give'),
 			),
 			'no_donors'             => array(
-				'heading'  => __( 'No donors found.', 'give' ),
-				'message'  => __( 'When your first donation arrives, the donor will appear here.', 'give' ),
-				'cta_text' => __( 'View All Forms', 'give' ),
-				'cta_link' => admin_url( 'edit.php?post_type=give_forms' ),
+				'heading'  => __('No donors found.', 'give'),
+				'message'  => __('When your first donation arrives, the donor will appear here.', 'give'),
+				'cta_text' => __('View All Forms', 'give'),
+				'cta_link' => admin_url('edit.php?post_type=give_forms'),
 				'help'     => sprintf(
 					/* translators: 1: Opening anchor tag. 2: Closing anchor tag. */
-					__( 'Need help? Learn more about %1$sDonors%2$s.', 'give' ),
+					__('Need help? Learn more about %1$sDonors%2$s.', 'give'),
 					'<a href="http://docs.givewp.com/core-donors/">',
 					'</a>'
 				),
 			),
 		);
 
-		if ( isset( $content[ $context ] ) ) {
+		if (isset($content[$context])) {
 			// Merge contextual content with defaults.
-			return wp_parse_args( $content[ $context ], $defaults );
+			return wp_parse_args($content[$context], $defaults);
 		} else {
 			// Return defaults if context is undefined.
 			return $defaults;

includes/admin/tools/data/class-give-tools-recount-form-stats.php

Spacing +56 added lines, -56 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -69,43 +69,43 @@ 
 	 */
 	public function get_data() {
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_form_stats' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_form_stats');
 		}
 
-		$totals = $this->get_stored_data( 'give_temp_recount_form_stats' );
+		$totals = $this->get_stored_data('give_temp_recount_form_stats');
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array(
 				'earnings' => (float) 0,
 				'sales'    => 0,
 			);
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 		}
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'give_forms' => $this->form_id,
 			'number'     => $this->per_step,
 			'status'     => $accepted_statuses,
 			'paged'      => $this->step,
 			'fields'     => 'ids',
-		) );
+		));
 
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		if ( $payments ) {
-			foreach ( $payments as $payment ) {
+		if ($payments) {
+			foreach ($payments as $payment) {
 
 				// Ensure acceptable status only.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					continue;
 				}
 
 				// Ensure only payments for this form are counted.
-				if ( $payment->form_id != $this->form_id ) {
+				if ($payment->form_id != $this->form_id) {
 					continue;
 				}
 
@@ -116,25 +116,25 @@ 
 				 */
 				$earning_amount = apply_filters(
 					'give_donation_amount',
-					give_format_amount( $payment->total, array( 'donation_id' => $payment->ID ) ),
+					give_format_amount($payment->total, array('donation_id' => $payment->ID)),
 					$payment->total,
 					$payment->ID,
-					array( 'type' => 'stats', 'currency' => false, 'amount' => false )
+					array('type' => 'stats', 'currency' => false, 'amount' => false)
 				);
 
-				$totals['sales'] ++;
-				$totals['earnings'] += (float) give_maybe_sanitize_amount( $earning_amount );
+				$totals['sales']++;
+				$totals['earnings'] += (float) give_maybe_sanitize_amount($earning_amount);
 
 			}
 
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 
 			return true;
 		}
 
 
-		give_update_meta( $this->form_id, '_give_form_sales', $totals['sales'] );
-		give_update_meta( $this->form_id, '_give_form_earnings', give_sanitize_amount_for_db( $totals['earnings'] ) );
+		give_update_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		give_update_meta($this->form_id, '_give_form_earnings', give_sanitize_amount_for_db($totals['earnings']));
 
 		return false;
 	}
@@ -146,35 +146,35 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
+		if ($this->step == 1) {
+			$this->delete_data('give_recount_total_'.$this->form_id);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
-		$total             = $this->get_stored_data( 'give_recount_total_' . $this->form_id );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
+		$total             = $this->get_stored_data('give_recount_total_'.$this->form_id);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = 0;
-			$args  = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args  = apply_filters('give_recount_form_stats_total_args', array(
 				'give_forms' => $this->form_id,
-				'number'     => - 1,
+				'number'     => -1,
 				'status'     => $accepted_statuses,
 				'fields'     => 'ids',
-			) );
+			));
 
-			$payments = new Give_Payments_Query( $args );
-			$total    = count( $payments->get_payments() );
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$payments = new Give_Payments_Query($args);
+			$total    = count($payments->get_payments());
+			$this->store_data('give_recount_total_'.$this->form_id, $total);
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -188,8 +188,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -200,21 +200,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_temp_recount_form_stats' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_temp_recount_form_stats');
 			$this->done    = true;
-			$this->message = sprintf( esc_html__( 'Donation counts and income amount statistics successfully recounted for "%s".', 'give' ), get_the_title( $this->form_id ) );
+			$this->message = sprintf(esc_html__('Donation counts and income amount statistics successfully recounted for "%s".', 'give'), get_the_title($this->form_id));
 
 			return false;
 		}
@@ -248,17 +248,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -274,10 +274,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -291,7 +291,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -303,9 +303,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }
\ No newline at end of file

includes/admin/tools/data/class-give-tools-reset-stats.php

Spacing +64 added lines, -64 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -49,8 +49,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -67,16 +67,16 @@ 
 	public function get_data() {
 		global $wpdb;
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids' );
+		$items = $this->get_stored_data('give_temp_reset_ids');
 
-		if ( ! is_array( $items ) ) {
+		if ( ! is_array($items)) {
 			return false;
 		}
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $items, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
 
-		if ( $step_items ) {
+		if ($step_items) {
 
 			$step_ids = array(
 				'customers' => array(),
@@ -84,9 +84,9 @@ 
 				'other'     => array(),
 			);
 
-			foreach ( $step_items as $item ) {
+			foreach ($step_items as $item) {
 
-				switch ( $item['type'] ) {
+				switch ($item['type']) {
 					case 'customer':
 						$step_ids['customers'][] = $item['id'];
 						break;
@@ -94,24 +94,24 @@ 
 						$step_ids['give_forms'][] = $item['id'];
 						break;
 					default:
-						$item_type                = apply_filters( 'give_reset_item_type', 'other', $item );
-						$step_ids[ $item_type ][] = $item['id'];
+						$item_type                = apply_filters('give_reset_item_type', 'other', $item);
+						$step_ids[$item_type][] = $item['id'];
 						break;
 				}
 			}
 
 			$sql = array();
-			$meta_table = __give_v20_bc_table_details('form' );
+			$meta_table = __give_v20_bc_table_details('form');
 
-			foreach ( $step_ids as $type => $ids ) {
+			foreach ($step_ids as $type => $ids) {
 
-				if ( empty( $ids ) ) {
+				if (empty($ids)) {
 					continue;
 				}
 
-				$ids = implode( ',', $ids );
+				$ids = implode(',', $ids);
 
-				switch ( $type ) {
+				switch ($type) {
 					case 'customers':
 						$sql[]           = "DELETE FROM $wpdb->donors WHERE id IN ($ids)";
 						$table_name      = $wpdb->donors;
@@ -131,16 +131,16 @@ 
 						break;
 				}
 
-				if ( ! in_array( $type, array( 'customers', 'forms', 'other' ) ) ) {
+				if ( ! in_array($type, array('customers', 'forms', 'other'))) {
 					// Allows other types of custom post types to filter on their own post_type
 					// and add items to the query list, for the IDs found in their post type.
-					$sql = apply_filters( "give_reset_add_queries_{$type}", $sql, $ids );
+					$sql = apply_filters("give_reset_add_queries_{$type}", $sql, $ids);
 				}
 			}
 
-			if ( ! empty( $sql ) ) {
-				foreach ( $sql as $query ) {
-					$wpdb->query( $query );
+			if ( ! empty($sql)) {
+				foreach ($sql as $query) {
+					$wpdb->query($query);
 				}
 			}
 
@@ -160,16 +160,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids' );
-		$total = count( $items );
+		$items = $this->get_stored_data('give_temp_reset_ids');
+		$total = count($items);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -183,7 +183,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -194,26 +194,26 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to reset data.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to reset data.', 'give'), esc_html__('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', 0 );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', 0);
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_data( 'give_temp_reset_ids' );
+			$this->delete_data('give_temp_reset_ids');
 
 			$this->done    = true;
-			$this->message = esc_html__( 'Donation forms, income, donations counts, and logs successfully reset.', 'give' );
+			$this->message = esc_html__('Donation forms, income, donations counts, and logs successfully reset.', 'give');
 
 			return false;
 		}
@@ -246,26 +246,26 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_reset_ids' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_reset_ids');
 		}
 
-		$items = get_option( 'give_temp_reset_ids', false );
+		$items = get_option('give_temp_reset_ids', false);
 
-		if ( false === $items ) {
+		if (false === $items) {
 			$items = array();
 
-			$give_types_for_reset = array( 'give_forms', 'give_payment' );
-			$give_types_for_reset = apply_filters( 'give_reset_store_post_types', $give_types_for_reset );
+			$give_types_for_reset = array('give_forms', 'give_payment');
+			$give_types_for_reset = apply_filters('give_reset_store_post_types', $give_types_for_reset);
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_type'      => $give_types_for_reset,
 				'post_status'    => 'any',
-				'posts_per_page' => - 1,
-			) );
+				'posts_per_page' => -1,
+			));
 
-			$posts = get_posts( $args );
-			foreach ( $posts as $post ) {
+			$posts = get_posts($args);
+			foreach ($posts as $post) {
 				$items[] = array(
 					'id'   => (int) $post->ID,
 					'type' => $post->post_type,
@@ -273,10 +273,10 @@ 
 			}
 
 			$donor_args = array(
-				'number' => - 1,
+				'number' => -1,
 			);
-			$donors     = Give()->donors->get_donors( $donor_args );
-			foreach ( $donors as $donor ) {
+			$donors     = Give()->donors->get_donors($donor_args);
+			foreach ($donors as $donor) {
 				$items[] = array(
 					'id'   => (int) $donor->id,
 					'type' => 'customer',
@@ -285,9 +285,9 @@ 
 
 			// Allow filtering of items to remove with an unassociative array for each item
 			// The array contains the unique ID of the item, and a 'type' for you to use in the execution of the get_data method
-			$items = apply_filters( 'give_reset_items', $items );
+			$items = apply_filters('give_reset_items', $items);
 
-			$this->store_data( 'give_temp_reset_ids', $items );
+			$this->store_data('give_temp_reset_ids', $items);
 		}// End if().
 
 	}
@@ -301,17 +301,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database.
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -327,10 +327,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -344,7 +344,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -356,11 +356,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array(
+		$wpdb->delete($wpdb->options, array(
 			'option_name' => $key,
-		) );
+		));
 	}
 
 }

includes/admin/tools/data/class-give-tools-recount-all-stats.php

Spacing +100 added lines, -102 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -73,8 +73,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -89,108 +89,106 @@ 
 	 */
 	public function get_data() {
 
-		$totals             = $this->get_stored_data( 'give_temp_recount_all_stats' );
-		$payment_items      = $this->get_stored_data( 'give_temp_payment_items' );
-		$processed_payments = $this->get_stored_data( 'give_temp_processed_payments' );
-		$accepted_statuses  = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$totals             = $this->get_stored_data('give_temp_recount_all_stats');
+		$payment_items      = $this->get_stored_data('give_temp_payment_items');
+		$processed_payments = $this->get_stored_data('give_temp_processed_payments');
+		$accepted_statuses  = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array();
 		}
 
-		if ( false === $payment_items ) {
+		if (false === $payment_items) {
 			$payment_items = array();
 		}
 
-		if ( false === $processed_payments ) {
+		if (false === $processed_payments) {
 			$processed_payments = array();
 		}
 
-		$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+		$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-		$payments = $this->get_stored_data( 'give_temp_all_payments_data' );
+		$payments = $this->get_stored_data('give_temp_all_payments_data');
 
-		if ( empty( $payments ) ) {
-			$args = apply_filters( 'give_recount_form_stats_args', array(
+		if (empty($payments)) {
+			$args = apply_filters('give_recount_form_stats_args', array(
 				'give_forms' => $all_forms,
 				'number'     => $this->per_step,
 				'status'     => $accepted_statuses,
 				'paged'      => $this->step,
 				'output'     => 'give_payments',
-			) );
+			));
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 		}
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
 			//Loop through payments
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$payment_id = ( ! empty( $payment['ID'] ) ? absint( $payment['ID'] ) : ( ! empty( $payment->ID ) ? absint( $payment->ID ) : false ) );
-				$payment = new Give_Payment( $payment_id );
+				$payment_id = ( ! empty($payment['ID']) ? absint($payment['ID']) : ( ! empty($payment->ID) ? absint($payment->ID) : false));
+				$payment = new Give_Payment($payment_id);
 
 				// Prevent payments that have all ready been retrieved from a previous sales log from counting again.
-				if ( in_array( $payment->ID, $processed_payments ) ) {
+				if (in_array($payment->ID, $processed_payments)) {
 					continue;
 				}
 
 				// Verify accepted status.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					$processed_payments[] = $payment->ID;
 					continue;
 				}
 
-				$payment_item = $payment_items[ $payment->ID ];
+				$payment_item = $payment_items[$payment->ID];
 
-				$form_id = isset( $payment_item['id'] ) ? $payment_item['id'] : '';
+				$form_id = isset($payment_item['id']) ? $payment_item['id'] : '';
 
 				// Must have a form ID.
-				if ( empty( $form_id ) ) {
+				if (empty($form_id)) {
 					continue;
 				}
 
 				// Form ID must be within $all_forms array to be validated.
-				if ( ! in_array( $form_id, $all_forms ) ) {
+				if ( ! in_array($form_id, $all_forms)) {
 					continue;
 				}
 
 				// Set Sales count
-				$totals[ $form_id ]['sales'] = isset( $totals[ $form_id ]['sales'] ) ?
-					++ $totals[ $form_id ]['sales'] :
-					1;
+				$totals[$form_id]['sales'] = isset($totals[$form_id]['sales']) ?
+					++$totals[$form_id]['sales'] : 1;
 
 				// Set Total Earnings
-				$totals[ $form_id ]['earnings'] = isset( $totals[ $form_id ]['earnings'] ) ?
-					( $totals[ $form_id ]['earnings'] + $payment_item['price'] ) :
-					$payment_item['price'];
+				$totals[$form_id]['earnings'] = isset($totals[$form_id]['earnings']) ?
+					($totals[$form_id]['earnings'] + $payment_item['price']) : $payment_item['price'];
 
 				$processed_payments[] = $payment->ID;
 			}
 
 			// Get the list of form ids which does not contain any payment record.
-			$remaining_form_ids = array_diff( $all_forms, array_keys( $totals ) );
-			foreach ( $remaining_form_ids as $form_id ) {
+			$remaining_form_ids = array_diff($all_forms, array_keys($totals));
+			foreach ($remaining_form_ids as $form_id) {
 				//If array key doesn't exist, create it
-				if ( ! array_key_exists( $form_id, $totals ) ) {
-					$totals[ $form_id ] = array(
+				if ( ! array_key_exists($form_id, $totals)) {
+					$totals[$form_id] = array(
 						'sales'    => (int) 0,
 						'earnings' => (float) 0,
 					);
 				}
 			}
 
-			$this->store_data( 'give_temp_processed_payments', $processed_payments );
-			$this->store_data( 'give_temp_recount_all_stats', $totals );
+			$this->store_data('give_temp_processed_payments', $processed_payments);
+			$this->store_data('give_temp_recount_all_stats', $totals);
 
 			return true;
 		}
 
 
-		foreach ( $totals as $key => $stats ) {
-			give_update_meta( $key, '_give_form_sales', $stats['sales'] );
-			give_update_meta( $key, '_give_form_earnings', give_sanitize_amount_for_db( $stats['earnings'] ) );
+		foreach ($totals as $key => $stats) {
+			give_update_meta($key, '_give_form_sales', $stats['sales']);
+			give_update_meta($key, '_give_form_earnings', give_sanitize_amount_for_db($stats['earnings']));
 		}
 
 		return false;
@@ -205,20 +203,20 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_all_total' );
+		$total = $this->get_stored_data('give_recount_all_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$this->pre_fetch();
-			$total = $this->get_stored_data( 'give_recount_all_total' );
+			$total = $this->get_stored_data('give_recount_all_total');
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -232,8 +230,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -244,25 +242,25 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_form_ids' );
-			$this->delete_data( 'give_temp_processed_payments' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_form_ids');
+			$this->delete_data('give_temp_processed_payments');
 			$this->done    = true;
-			$this->message = esc_html__( 'Donation form income amounts and donation counts stats successfully recounted.', 'give' );
+			$this->message = esc_html__('Donation form income amounts and donation counts stats successfully recounted.', 'give');
 
 			return false;
 		}
@@ -298,69 +296,69 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( 1 == $this->step ) {
-			$this->delete_data( 'give_temp_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_processed_payments' );
-			$this->delete_data( 'give_temp_all_payments_data' );
+		if (1 == $this->step) {
+			$this->delete_data('give_temp_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_processed_payments');
+			$this->delete_data('give_temp_all_payments_data');
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
-		$total             = $this->get_stored_data( 'give_temp_recount_all_total' );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
+		$total             = $this->get_stored_data('give_temp_recount_all_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 
-			$payment_items = $this->get_stored_data( 'give_temp_payment_items' );
+			$payment_items = $this->get_stored_data('give_temp_payment_items');
 
-			if ( false === $payment_items ) {
+			if (false === $payment_items) {
 				$payment_items = array();
-				$this->store_data( 'give_temp_payment_items', $payment_items );
+				$this->store_data('give_temp_payment_items', $payment_items);
 			}
 
 			$args = array(
 				'post_status'    => 'publish',
 				'post_type'      => 'give_forms',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'fields'         => 'ids',
 			);
 
-			$all_forms = get_posts( $args );
+			$all_forms = get_posts($args);
 
-			$this->store_data( 'give_temp_form_ids', $all_forms );
+			$this->store_data('give_temp_form_ids', $all_forms);
 
-			$args = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args = apply_filters('give_recount_form_stats_total_args', array(
 				'give_forms' => $all_forms,
 				'number'     => $this->per_step,
 				'status'     => $accepted_statuses,
 				'page'       => $this->step,
 				'output'     => 'payments',
-			) );
+			));
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 
-			$total = wp_count_posts( 'give_payment' )->publish;
+			$total = wp_count_posts('give_payment')->publish;
 
-			$this->store_data( 'give_temp_all_payments_data', $payments );
+			$this->store_data('give_temp_all_payments_data', $payments);
 
-			if ( $payments ) {
+			if ($payments) {
 
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 
 					$form_id = $payment->form_id;
 
 					//If for some reason somehow the form_ID isn't set check payment meta
-					if ( empty( $payment->form_id ) ) {
+					if (empty($payment->form_id)) {
 						$payment_meta = $payment->get_meta();
-						$form_id      = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
+						$form_id      = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
 					}
 
-					if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+					if ( ! in_array($payment->post_status, $accepted_statuses)) {
 						continue;
 					}
 
-					if ( ! array_key_exists( $payment->ID, $payment_items ) ) {
+					if ( ! array_key_exists($payment->ID, $payment_items)) {
 
 						/**
 						 * Filter the payment amount.
@@ -369,22 +367,22 @@ 
 						 */
 						$payment_total = apply_filters(
 							'give_donation_amount',
-							give_format_amount( $payment->total, array( 'donation_id' => $payment->ID ) ),
+							give_format_amount($payment->total, array('donation_id' => $payment->ID)),
 							$payment->total,
 							$payment->ID,
-							array( 'type' => 'stats', 'currency' => false, 'amount' => false )
+							array('type' => 'stats', 'currency' => false, 'amount' => false)
 						);
 
-						$payment_items[ $payment->ID ] = array(
+						$payment_items[$payment->ID] = array(
 							'id'         => $form_id,
 							'payment_id' => $payment->ID,
-							'price'      => (float) give_maybe_sanitize_amount( $payment_total ),
+							'price'      => (float) give_maybe_sanitize_amount($payment_total),
 						);
 					}
 				}
 			}
-			$this->store_data( 'give_temp_payment_items', $payment_items );
-			$this->store_data( 'give_recount_all_total', $total );
+			$this->store_data('give_temp_payment_items', $payment_items);
+			$this->store_data('give_recount_all_total', $total);
 		}
 
 	}
@@ -398,17 +396,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
-		if ( empty( $value ) ) {
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
+		$maybe_json = json_decode($value);
 
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -424,10 +422,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -441,7 +439,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -453,9 +451,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

includes/admin/tools/data/class-give-tools-recount-income.php

Spacing +50 added lines, -50 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,8 +46,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -61,33 +61,33 @@ 
 	 */
 	public function get_data() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_earnings' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_earnings');
 		}
 
-		$total = get_option( 'give_temp_recount_earnings', false );
+		$total = get_option('give_temp_recount_earnings', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = (float) 0;
-			$this->store_data( 'give_temp_recount_earnings', $total );
+			$this->store_data('give_temp_recount_earnings', $total);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		$args = apply_filters( 'give_recount_earnings_args', array(
+		$args = apply_filters('give_recount_earnings_args', array(
 			'number' => $this->per_step,
 			'page'   => $this->step,
 			'status' => $accepted_statuses,
 			'fields' => 'ids'
-		) );
+		));
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 				// Get the payment amount.
-				$payment_amount = give_get_meta( $payment->ID, '_give_payment_total', true );
+				$payment_amount = give_get_meta($payment->ID, '_give_payment_total', true);
 
 				/**
 				 * Filter the payment amount.
@@ -96,28 +96,28 @@ 
 				 */
 				$donation_amount = apply_filters(
 					'give_donation_amount',
-					give_format_amount( $payment_amount, array( 'donation_id' => $payment->ID ) ),
+					give_format_amount($payment_amount, array('donation_id' => $payment->ID)),
 					$payment->total,
 					$payment->ID,
-					array( 'type' => 'stats', 'currency' => false, 'amount' => false )
+					array('type' => 'stats', 'currency' => false, 'amount' => false)
 				);
 
-				$total += (float) give_maybe_sanitize_amount( $donation_amount );
+				$total += (float) give_maybe_sanitize_amount($donation_amount);
 			}
 
-			if ( $total < 0 ) {
+			if ($total < 0) {
 				$totals = 0;
 			}
 
-			$total = round( $total, give_get_price_decimals() );
+			$total = round($total, give_get_price_decimals());
 
-			$this->store_data( 'give_temp_recount_earnings', $total );
+			$this->store_data('give_temp_recount_earnings', $total);
 
 			return true;
 
 		}
 
-		update_option( 'give_earnings_total', $total );
+		update_option('give_earnings_total', $total);
 
 		return false;
 
@@ -131,25 +131,25 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_earnings_total' );
+		$total = $this->get_stored_data('give_recount_earnings_total');
 
-		if ( false === $total ) {
-			$args = apply_filters( 'give_recount_earnings_total_args', array() );
+		if (false === $total) {
+			$args = apply_filters('give_recount_earnings_total_args', array());
 
-			$counts = give_count_payments( $args );
-			$total  = absint( $counts->publish );
-			$total  = apply_filters( 'give_recount_store_earnings_total', $total );
+			$counts = give_count_payments($args);
+			$total  = absint($counts->publish);
+			$total  = apply_filters('give_recount_store_earnings_total', $total);
 
-			$this->store_data( 'give_recount_earnings_total', $total );
+			$this->store_data('give_recount_earnings_total', $total);
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -163,7 +163,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -174,21 +174,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_earnings_total' );
-			$this->delete_data( 'give_temp_recount_earnings' );
+			$this->delete_data('give_recount_earnings_total');
+			$this->delete_data('give_temp_recount_earnings');
 			$this->done    = true;
-			$this->message = esc_html__( 'Income stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Income stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -225,17 +225,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -251,10 +251,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -268,7 +268,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -280,9 +280,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }