WordImpress / Give

includes/plugin-compatibility.php

Indentation +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 	) {
 		// If Yoast SEO or Yoast SEO Premium plugin exists, then update seo sitemap cache.
 		if ( (is_plugin_active( 'wordpress-seo/wp-seo.php' )
-		     || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ))
+			 || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ))
 			&& class_exists( 'WPSEO_Sitemaps_Cache' )
 		) {
 			$yoast_sitemaps_cache = new WPSEO_Sitemaps_Cache();

Spacing +11 added lines, -11 removed lines

@@ -19,9 +19,9 @@ 
  * @return void
  */
 function give_disable_mandrill_nl2br() {
-	add_filter( 'mandrill_nl2br', '__return_false' );
+	add_filter('mandrill_nl2br', '__return_false');
 }
-add_action( 'give_email_send_before', 'give_disable_mandrill_nl2br' );
+add_action('give_email_send_before', 'give_disable_mandrill_nl2br');
 
 
 /**
@@ -33,26 +33,26 @@ 
  */
 function give_clear_seo_sitemap_cache_on_settings_change() {
 	// Load required file if the fn 'is_plugin_active' doesn't exists.
-	if ( ! function_exists( 'is_plugin_active' ) ) {
-		require_once ABSPATH . 'wp-admin/includes/plugin.php';
+	if ( ! function_exists('is_plugin_active')) {
+		require_once ABSPATH.'wp-admin/includes/plugin.php';
 	}
 
 	// If there is change detected for Single Form View and Form Archives options then proceed.
 	if (
-		( $_POST['forms_singular'] != give_get_option( 'forms_singular' ) ) ||
-		( $_POST['forms_archives'] != give_get_option( 'forms_archives' ) )
+		($_POST['forms_singular'] != give_get_option('forms_singular')) ||
+		($_POST['forms_archives'] != give_get_option('forms_archives'))
 	) {
 		// If Yoast SEO or Yoast SEO Premium plugin exists, then update seo sitemap cache.
-		if ( (is_plugin_active( 'wordpress-seo/wp-seo.php' )
-		     || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ))
-			&& class_exists( 'WPSEO_Sitemaps_Cache' )
+		if ((is_plugin_active('wordpress-seo/wp-seo.php')
+		     || is_plugin_active('wordpress-seo-premium/wp-seo-premium.php'))
+			&& class_exists('WPSEO_Sitemaps_Cache')
 		) {
 			$yoast_sitemaps_cache = new WPSEO_Sitemaps_Cache();
-			if ( method_exists( $yoast_sitemaps_cache, 'clear' ) ) {
+			if (method_exists($yoast_sitemaps_cache, 'clear')) {
 				WPSEO_Sitemaps_Cache::clear();
 			}
 		}
 	}
 }
 
-add_action( 'give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change' );
+add_action('give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change');

includes/class-give-db-donor-meta.php

Spacing +24 added lines, -24 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,7 +33,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->customermeta = $this->table_name  = $wpdb->prefix . 'give_customermeta';
+		$wpdb->customermeta = $this->table_name = $wpdb->prefix.'give_customermeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -71,13 +71,13 @@ 
 	 *
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single is true.
 	 */
-	public function get_meta( $donor_id = 0, $meta_key = '', $single = false ) {
-		$donor_id = $this->sanitize_donor_id( $donor_id );
-		if ( false === $donor_id ) {
+	public function get_meta($donor_id = 0, $meta_key = '', $single = false) {
+		$donor_id = $this->sanitize_donor_id($donor_id);
+		if (false === $donor_id) {
 			return false;
 		}
 
-		return get_metadata( 'customer', $donor_id, $meta_key, $single );
+		return get_metadata('customer', $donor_id, $meta_key, $single);
 	}
 
 	/**
@@ -95,13 +95,13 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function add_meta( $donor_id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$donor_id = $this->sanitize_donor_id( $donor_id );
-		if ( false === $donor_id ) {
+	public function add_meta($donor_id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$donor_id = $this->sanitize_donor_id($donor_id);
+		if (false === $donor_id) {
 			return false;
 		}
 
-		return add_metadata( 'customer', $donor_id, $meta_key, $meta_value, $unique );
+		return add_metadata('customer', $donor_id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -124,13 +124,13 @@ 
 	 *
 	 * @return  bool                  False on failure, true if success.
 	 */
-	public function update_meta( $donor_id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$donor_id = $this->sanitize_donor_id( $donor_id );
-		if ( false === $donor_id ) {
+	public function update_meta($donor_id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$donor_id = $this->sanitize_donor_id($donor_id);
+		if (false === $donor_id) {
 			return false;
 		}
 
-		return update_metadata( 'customer', $donor_id, $meta_key, $meta_value, $prev_value );
+		return update_metadata('customer', $donor_id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -151,8 +151,8 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $donor_id = 0, $meta_key = '', $meta_value = '' ) {
-		return delete_metadata( 'customer', $donor_id, $meta_key, $meta_value );
+	public function delete_meta($donor_id = 0, $meta_key = '', $meta_value = '') {
+		return delete_metadata('customer', $donor_id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -165,7 +165,7 @@ 
 	 */
 	public function create_table() {
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
 		$sql = "CREATE TABLE {$this->table_name} (
 			meta_id bigint(20) NOT NULL AUTO_INCREMENT,
@@ -177,9 +177,9 @@ 
 			KEY meta_key (meta_key)
 			) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 	/**
@@ -192,23 +192,23 @@ 
 	 *
 	 * @return int|bool                The normalized donor ID or false if it's found to not be valid.
 	 */
-	private function sanitize_donor_id( $donor_id ) {
-		if ( ! is_numeric( $donor_id ) ) {
+	private function sanitize_donor_id($donor_id) {
+		if ( ! is_numeric($donor_id)) {
 			return false;
 		}
 
 		$donor_id = (int) $donor_id;
 
 		// We were given a non positive number.
-		if ( absint( $donor_id ) !== $donor_id ) {
+		if (absint($donor_id) !== $donor_id) {
 			return false;
 		}
 
-		if ( empty( $donor_id ) ) {
+		if (empty($donor_id)) {
 			return false;
 		}
 
-		return absint( $donor_id );
+		return absint($donor_id);
 
 	}
 

includes/admin/class-i18n-module.php

Spacing +45 added lines, -45 removed lines

@@ -65,18 +65,18 @@ 
 	 *
 	 * @param $args
 	 */
-	public function __construct( $args ) {
+	public function __construct($args) {
 
 		// Only for admins.
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
-		foreach ( $args as $key => $arg ) {
+		foreach ($args as $key => $arg) {
 			$this->$key = $arg;
 		}
 
-		add_action( 'admin_init', array( $this, 'init' ) );
+		add_action('admin_init', array($this, 'init'));
 
 
 	}
@@ -87,15 +87,15 @@ 
 	function init() {
 
 		// First get user's locale (4.7+).
-		$this->locale = function_exists( 'get_user_locale' ) ? get_user_locale() : get_locale();
+		$this->locale = function_exists('get_user_locale') ? get_user_locale() : get_locale();
 
 		// This plugin is en_US native.
-		if ( 'en_US' === $this->locale ) {
+		if ('en_US' === $this->locale) {
 			return;
 		}
 
-		if ( ! $this->hide_promo() ) {
-			add_action( $this->hook, array( $this, 'promo' ) );
+		if ( ! $this->hide_promo()) {
+			add_action($this->hook, array($this, 'promo'));
 		}
 	}
 
@@ -108,11 +108,11 @@ 
 	 * @return bool
 	 */
 	private function hide_promo() {
-		$hide_promo = Give_Cache::get( 'give_i18n_give_promo_hide', true );
-		if ( ! $hide_promo ) {
-			if ( filter_input( INPUT_GET, 'remove_i18n_promo', FILTER_VALIDATE_INT ) === 1 ) {
+		$hide_promo = Give_Cache::get('give_i18n_give_promo_hide', true);
+		if ( ! $hide_promo) {
+			if (filter_input(INPUT_GET, 'remove_i18n_promo', FILTER_VALIDATE_INT) === 1) {
 				// No expiration time, so this would normally not expire, but it wouldn't be copied to other sites etc.
-				Give_Cache::set( 'give_i18n_give_promo_hide', true, null, true );
+				Give_Cache::set('give_i18n_give_promo_hide', true, null, true);
 				$hide_promo = true;
 			}
 		}
@@ -131,20 +131,20 @@ 
 		$message = false;
 
 		// Using a translation less than 90% complete.
-		if ( $this->translation_exists && $this->translation_loaded && $this->percent_translated < 90 ) {
-			$message = __( 'As you can see, there is a translation of this plugin in %1$s. This translation is currently %3$d%% complete. We need your help to make it complete and to fix any errors. Please register at %4$s to help %5$s to %1$s!', 'give' );
-		} elseif ( ! $this->translation_loaded && $this->translation_exists ) {
-			$message = __( 'You\'re using WordPress in %1$s. While %2$s has been %3$d%% translated to %1$s, it has not been shipped with the plugin yet. You can help! Register at %4$s to help complete the translation to %1$s!', 'give' );
-		} elseif ( ! $this->translation_exists ) {
-			$message = __( 'You\'re using WordPress in a language we don\'t support yet. We\'d love for %2$s to be translated in that language too, but unfortunately, it isn\'t right now. You can change that! Register at %4$s to help translate it!', 'give' );
+		if ($this->translation_exists && $this->translation_loaded && $this->percent_translated < 90) {
+			$message = __('As you can see, there is a translation of this plugin in %1$s. This translation is currently %3$d%% complete. We need your help to make it complete and to fix any errors. Please register at %4$s to help %5$s to %1$s!', 'give');
+		} elseif ( ! $this->translation_loaded && $this->translation_exists) {
+			$message = __('You\'re using WordPress in %1$s. While %2$s has been %3$d%% translated to %1$s, it has not been shipped with the plugin yet. You can help! Register at %4$s to help complete the translation to %1$s!', 'give');
+		} elseif ( ! $this->translation_exists) {
+			$message = __('You\'re using WordPress in a language we don\'t support yet. We\'d love for %2$s to be translated in that language too, but unfortunately, it isn\'t right now. You can change that! Register at %4$s to help translate it!', 'give');
 		}
 
 		// Links.
-		$registration_link = sprintf( '<a href="%1$s" target="_blank">%2$s</a>', 'https://wordpress.org/support/register.php', esc_html__( 'WordPress.org', 'give' ) );
-		$translations_link = sprintf( '<a href="%1$s" target="_blank">%2$s</a>', 'https://translate.wordpress.org/projects/wp-plugins/give', esc_html__( 'complete the translation', 'give' ) );
+		$registration_link = sprintf('<a href="%1$s" target="_blank">%2$s</a>', 'https://wordpress.org/support/register.php', esc_html__('WordPress.org', 'give'));
+		$translations_link = sprintf('<a href="%1$s" target="_blank">%2$s</a>', 'https://translate.wordpress.org/projects/wp-plugins/give', esc_html__('complete the translation', 'give'));
 
 		// Message.
-		$message = sprintf( $message, $this->locale_name, 'Give', $this->percent_translated, $registration_link, $translations_link );
+		$message = sprintf($message, $this->locale_name, 'Give', $this->percent_translated, $registration_link, $translations_link);
 
 		return $message;
 
@@ -158,7 +158,7 @@ 
 		$this->translation_details();
 		$message = $this->promo_message();
 
-		if ( $message ) { ?>
+		if ($message) { ?>
 
 			<style>
 				/* Banner specific styles */
@@ -223,12 +223,12 @@ 
 				                                                                                                                             style="font-size: 110px; text-decoration: none;"></span></a>
 
 				<div class="give-i18n-notice-content">
-					<a href="<?php echo esc_url( add_query_arg( array( 'remove_i18n_promo' => '1' ) ) ); ?>" class="dismiss"><span class="dashicons dashicons-dismiss"></span></a>
+					<a href="<?php echo esc_url(add_query_arg(array('remove_i18n_promo' => '1'))); ?>" class="dismiss"><span class="dashicons dashicons-dismiss"></span></a>
 
-					<h2 style="margin: 10px 0;"><?php printf( esc_html__( 'Help Translate Give to %s', 'give' ), $this->locale_name ); ?></h2>
+					<h2 style="margin: 10px 0;"><?php printf(esc_html__('Help Translate Give to %s', 'give'), $this->locale_name); ?></h2>
 					<p><?php echo $message; ?></p>
 					<p>
-						<a href="https://wordpress.org/support/register.php" target="_blank"><?php _e( 'Register now &raquo;', 'give' ); ?></a>
+						<a href="https://wordpress.org/support/register.php" target="_blank"><?php _e('Register now &raquo;', 'give'); ?></a>
 					</p>
 				</div>
 			</div>
@@ -245,11 +245,11 @@ 
 	 */
 	private function find_or_initialize_translation_details() {
 
-		$set = Give_Cache::get( "give_i18n_give_{$this->locale}", true );
+		$set = Give_Cache::get("give_i18n_give_{$this->locale}", true);
 
-		if ( ! $set ) {
+		if ( ! $set) {
 			$set = $this->retrieve_translation_details();
-			Give_Cache::set( "give_i18n_give_{$this->locale}", $set, DAY_IN_SECONDS, true );
+			Give_Cache::set("give_i18n_give_{$this->locale}", $set, DAY_IN_SECONDS, true);
 		}
 
 		return $set;
@@ -263,10 +263,10 @@ 
 	private function translation_details() {
 		$set = $this->find_or_initialize_translation_details();
 
-		$this->translation_exists = ! is_null( $set );
-		$this->translation_loaded = is_textdomain_loaded( 'give' );
+		$this->translation_exists = ! is_null($set);
+		$this->translation_loaded = is_textdomain_loaded('give');
 
-		$this->parse_translation_set( $set );
+		$this->parse_translation_set($set);
 	}
 
 	/**
@@ -278,26 +278,26 @@ 
 	 */
 	private function retrieve_translation_details() {
 
-		$api_url = trailingslashit( $this->glotpress_url );
+		$api_url = trailingslashit($this->glotpress_url);
 
-		$resp = wp_remote_get( $api_url );
+		$resp = wp_remote_get($api_url);
 
-		if ( is_wp_error( $resp ) || wp_remote_retrieve_response_code( $resp ) === '404' ) {
+		if (is_wp_error($resp) || wp_remote_retrieve_response_code($resp) === '404') {
 			return null;
 		}
 
-		$body = wp_remote_retrieve_body( $resp );
-		unset( $resp );
+		$body = wp_remote_retrieve_body($resp);
+		unset($resp);
 
-		if ( $body ) {
-			$body = json_decode( $body );
+		if ($body) {
+			$body = json_decode($body);
 
-			foreach ( $body->translation_sets as $set ) {
-				if ( ! property_exists( $set, 'wp_locale' ) ) {
+			foreach ($body->translation_sets as $set) {
+				if ( ! property_exists($set, 'wp_locale')) {
 					continue;
 				}
 
-				if ( $this->locale == $set->wp_locale ) {
+				if ($this->locale == $set->wp_locale) {
 					return $set;
 				}
 			}
@@ -313,8 +313,8 @@ 
 	 *
 	 * @access private
 	 */
-	private function parse_translation_set( $set ) {
-		if ( $this->translation_exists && is_object( $set ) ) {
+	private function parse_translation_set($set) {
+		if ($this->translation_exists && is_object($set)) {
 			$this->locale_name        = $set->name;
 			$this->percent_translated = $set->percent_translated;
 		} else {
@@ -324,7 +324,7 @@ 
 	}
 }
 
-$give_i18n = new Give_i18n_Banner( array(
+$give_i18n = new Give_i18n_Banner(array(
 		'hook'          => 'give_forms_page_give-settings',
 		'glotpress_url' => 'https://translate.wordpress.org/api/projects/wp-plugins/give/stable/',
-	) );
+	));

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;
 }
 
@@ -58,16 +58,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(),
@@ -75,9 +75,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;
@@ -85,25 +85,25 @@ 
 						$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();
 
-			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':
-						$table_name = $wpdb->prefix . 'give_customers';
+						$table_name = $wpdb->prefix.'give_customers';
 						$sql[]      = "DELETE FROM $table_name WHERE id IN ($ids)";
 						break;
 					case 'forms':
@@ -118,16 +118,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);
 				}
 			}
 
@@ -147,16 +147,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;
 		}
 
@@ -170,7 +170,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -181,31 +181,31 @@ 
 	 */
 	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');
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$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;
 		}
@@ -215,10 +215,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -242,26 +242,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,
@@ -269,10 +269,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',
@@ -281,9 +281,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().
 
 	}
@@ -297,11 +297,11 @@ 
 	 *
 	 * @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));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -314,10 +314,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -331,7 +331,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -343,11 +343,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,
-		) );
+		));
 	}
 
 }
\ No newline at end of file

includes/admin/tools/export/export-actions.php

Spacing +27 added lines, -27 removed lines

@@ -10,7 +10,7 @@ 
  * @license     https://opensource.org/licenses/gpl-license GNU Public License
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,13 +22,13 @@ 
  */
 function give_process_batch_export_form() {
 
-	if ( ! wp_verify_nonce( $_REQUEST['nonce'], 'give-batch-export' ) ) {
-		wp_die( esc_html__( 'Nonce verification failed.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($_REQUEST['nonce'], 'give-batch-export')) {
+		wp_die(esc_html__('Nonce verification failed.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export.php';
 
 	/**
 	 * Fires before batch export.
@@ -37,14 +37,14 @@ 
 	 *
 	 * @param string $class Export class.
 	 */
-	do_action( 'give_batch_export_class_include', $_REQUEST['class'] );
+	do_action('give_batch_export_class_include', $_REQUEST['class']);
 
 	$export = new $_REQUEST['class'];
 	$export->export();
 
 }
 
-add_action( 'give_form_batch_export', 'give_process_batch_export_form' );
+add_action('give_form_batch_export', 'give_process_batch_export_form');
 
 /**
  * Exports earnings for a specified time period.
@@ -55,14 +55,14 @@ 
  * @return void
  */
 function give_export_earnings() {
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-export-earnings.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-export-earnings.php';
 
 	$earnings_export = new Give_Earnings_Export();
 
 	$earnings_export->export();
 }
 
-add_action( 'give_earnings_export', 'give_export_earnings' );
+add_action('give_earnings_export', 'give_export_earnings');
 
 
 /**
@@ -72,7 +72,7 @@ 
  * @return void
  */
 function give_register_batch_exporters() {
-	if ( is_admin() ) {
+	if (is_admin()) {
 		/**
 		 * Fires in the admin, while plugins loaded.
 		 *
@@ -82,11 +82,11 @@ 
 		 *
 		 * @param string $class Export class.
 		 */
-		do_action( 'give_register_batch_exporter' );
+		do_action('give_register_batch_exporter');
 	}
 }
 
-add_action( 'plugins_loaded', 'give_register_batch_exporters' );
+add_action('plugins_loaded', 'give_register_batch_exporters');
 
 /**
  * Register the payments batch exporter
@@ -94,10 +94,10 @@ 
  * @since  1.5
  */
 function give_register_payments_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_payments_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_payments_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_payments_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_payments_batch_export', 10);
 
 /**
  * Loads the payments batch process if needed
@@ -108,10 +108,10 @@ 
  *
  * @return void
  */
-function give_include_payments_batch_processor( $class ) {
+function give_include_payments_batch_processor($class) {
 
-	if ( 'Give_Batch_Payments_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export-payments.php';
+	if ('Give_Batch_Payments_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export-payments.php';
 	}
 
 }
@@ -122,10 +122,10 @@ 
  * @since  1.5.2
  */
 function give_register_donors_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_donors_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_donors_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_donors_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_donors_batch_export', 10);
 
 /**
  * Loads the donors batch process if needed.
@@ -136,10 +136,10 @@ 
  *
  * @return void
  */
-function give_include_donors_batch_processor( $class ) {
+function give_include_donors_batch_processor($class) {
 
-	if ( 'Give_Batch_Donors_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export-donors.php';
+	if ('Give_Batch_Donors_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export-donors.php';
 	}
 
 }
@@ -150,10 +150,10 @@ 
  * @since  1.5
  */
 function give_register_forms_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_forms_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_forms_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_forms_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_forms_batch_export', 10);
 
 /**
  * Loads the file downloads batch process if needed
@@ -164,10 +164,10 @@ 
  *
  * @return void
  */
-function give_include_forms_batch_processor( $class ) {
+function give_include_forms_batch_processor($class) {
 
-	if ( 'Give_Batch_Forms_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export-forms.php';
+	if ('Give_Batch_Forms_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export-forms.php';
 	}
 
 }

includes/admin/tools/export/export-functions.php

Spacing +27 added lines, -28 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_do_ajax_export() {
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export.php';
 
-	parse_str( $_POST['form'], $form );
+	parse_str($_POST['form'], $form);
 
 	$_REQUEST = $form = (array) $form;
 
-	if ( ! wp_verify_nonce( $_REQUEST['give_ajax_export'], 'give_ajax_export' ) ) {
-		die( '-2' );
+	if ( ! wp_verify_nonce($_REQUEST['give_ajax_export'], 'give_ajax_export')) {
+		die('-2');
 	}
 
 	/**
@@ -42,28 +42,28 @@ 
 	 *
 	 * @param string $class Export class.
 	 */
-	do_action( 'give_batch_export_class_include', $form['give-export-class'] );
+	do_action('give_batch_export_class_include', $form['give-export-class']);
 
-	$step   = absint( $_POST['step'] );
-	$class  = sanitize_text_field( $form['give-export-class'] );
+	$step   = absint($_POST['step']);
+	$class  = sanitize_text_field($form['give-export-class']);
 
 	/* @var Give_Batch_Export $export */
-	$export = new $class( $step );
+	$export = new $class($step);
 
-	if ( ! $export->can_export() ) {
-		die( '-1' );
+	if ( ! $export->can_export()) {
+		die('-1');
 	}
 
-	if ( ! $export->is_writable ) {
+	if ( ! $export->is_writable) {
 		$json_args = array(
 			'error'   => true,
-			'message' => esc_html__( 'Export location or file not writable.', 'give' )
+			'message' => esc_html__('Export location or file not writable.', 'give')
 		);
 		echo json_encode($json_args);
 		exit;
 	}
 
-	$export->set_properties( give_clean( $_REQUEST ) );
+	$export->set_properties(give_clean($_REQUEST));
 
 	$export->pre_fetch();
 
@@ -71,7 +71,7 @@ 
 
 	$percentage = $export->get_percentage_complete();
 
-	if ( $ret ) {
+	if ($ret) {
 
 		$step += 1;
 		$json_data = array(
@@ -79,18 +79,17 @@ 
 			'percentage' => $percentage
 		);
 
-	} elseif ( true === $export->is_empty ) {
+	} elseif (true === $export->is_empty) {
 
 		$json_data = array(
 			'error'   => true,
-			'message' => esc_html__( 'No data found for export parameters.', 'give' )
+			'message' => esc_html__('No data found for export parameters.', 'give')
 		);
 
-	} elseif ( true === $export->done && true === $export->is_void ) {
+	} elseif (true === $export->done && true === $export->is_void) {
 
-		$message = ! empty( $export->message ) ?
-			$export->message :
-			esc_html__( 'Batch Processing Complete', 'give' );
+		$message = ! empty($export->message) ?
+			$export->message : esc_html__('Batch Processing Complete', 'give');
 
 		$json_data = array(
 			'success' => true,
@@ -99,23 +98,23 @@ 
 
 	} else {
 		
-		$args = array_merge( $_REQUEST, array(
+		$args = array_merge($_REQUEST, array(
 			'step'        => $step,
 			'class'       => $class,
-			'nonce'       => wp_create_nonce( 'give-batch-export' ),
+			'nonce'       => wp_create_nonce('give-batch-export'),
 			'give_action' => 'form_batch_export',
-		) );
+		));
 
 		$json_data = array(
 			'step' => 'done',
-			'url' => add_query_arg( $args, admin_url() )
+			'url' => add_query_arg($args, admin_url())
 		);
 
 	}
 
-	$export->unset_properties( give_clean( $_REQUEST ), $export );
-	echo json_encode( $json_data );
+	$export->unset_properties(give_clean($_REQUEST), $export);
+	echo json_encode($json_data);
 	exit;
 }
 
-add_action( 'wp_ajax_give_do_ajax_export', 'give_do_ajax_export' );
+add_action('wp_ajax_give_do_ajax_export', 'give_do_ajax_export');

includes/admin/tools/export/class-batch-export-donors.php

Indentation +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 
 					// Continue if donor already included.
 					if ( empty( $payment->customer_id ) ||
-					     in_array( $payment->customer_id, $cached_donor_ids )
+						 in_array( $payment->customer_id, $cached_donor_ids )
 					) {
 						continue;
 					}

Spacing +102 added lines, -104 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -70,23 +70,23 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 
 		// Set data from form submission
-		if ( isset( $_POST['form'] ) ) {
-			parse_str( $_POST['form'], $this->data );
+		if (isset($_POST['form'])) {
+			parse_str($_POST['form'], $this->data);
 		}
 
 		$this->form = $this->data['forms'];
 
 		// Setup donor ids cache.
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 			// Cache donor ids to output unique list of donor.
-			$this->query_id = give_clean( $_REQUEST['give_export_option']['query_id'] );
+			$this->query_id = give_clean($_REQUEST['give_export_option']['query_id']);
 			$this->cache_donor_ids();
 		}
 
-		$this->price_id = ! empty( $request['give_price_option'] ) && 'all' !== $request['give_price_option'] ? absint( $request['give_price_option'] ) : null;
+		$this->price_id = ! empty($request['give_price_option']) && 'all' !== $request['give_price_option'] ? absint($request['give_price_option']) : null;
 
 	}
 
@@ -101,12 +101,12 @@ 
 		// Fetch already cached donor ids.
 		$donor_ids = $this->donor_ids;
 
-		if ( $cached_donor_ids = Give_Cache::get( $this->query_id, true ) ) {
-			$donor_ids = array_unique( array_merge( $cached_donor_ids, $this->donor_ids ) );
+		if ($cached_donor_ids = Give_Cache::get($this->query_id, true)) {
+			$donor_ids = array_unique(array_merge($cached_donor_ids, $this->donor_ids));
 		}
 
-		$donor_ids = array_values( $donor_ids );
-		Give_Cache::set( $this->query_id, $donor_ids, HOUR_IN_SECONDS, true );
+		$donor_ids = array_values($donor_ids);
+		Give_Cache::set($this->query_id, $donor_ids, HOUR_IN_SECONDS, true);
 	}
 
 	/**
@@ -118,14 +118,14 @@ 
 	 */
 	public function csv_cols() {
 
-		$columns = isset( $this->data['give_export_option'] ) ? $this->data['give_export_option'] : array();
+		$columns = isset($this->data['give_export_option']) ? $this->data['give_export_option'] : array();
 
 		// We need columns.
-		if ( empty( $columns ) ) {
+		if (empty($columns)) {
 			return false;
 		}
 
-		$cols = $this->get_cols( $columns );
+		$cols = $this->get_cols($columns);
 
 		return $cols;
 	}
@@ -137,41 +137,41 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_cols( $columns ) {
+	private function get_cols($columns) {
 
 		$cols = array();
 
-		foreach ( $columns as $key => $value ) {
+		foreach ($columns as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 				case 'full_name' :
-					$cols['full_name'] = esc_html__( 'Full Name', 'give' );
+					$cols['full_name'] = esc_html__('Full Name', 'give');
 					break;
 				case 'email' :
-					$cols['email'] = esc_html__( 'Email Address', 'give' );
+					$cols['email'] = esc_html__('Email Address', 'give');
 					break;
 				case 'address' :
-					$cols['address_line1']   = esc_html__( 'Address', 'give' );
-					$cols['address_line2']   = esc_html__( 'Address 2', 'give' );
-					$cols['address_city']    = esc_html__( 'City', 'give' );
-					$cols['address_state']   = esc_html__( 'State', 'give' );
-					$cols['address_zip']     = esc_html__( 'Zip', 'give' );
-					$cols['address_country'] = esc_html__( 'Country', 'give' );
+					$cols['address_line1']   = esc_html__('Address', 'give');
+					$cols['address_line2']   = esc_html__('Address 2', 'give');
+					$cols['address_city']    = esc_html__('City', 'give');
+					$cols['address_state']   = esc_html__('State', 'give');
+					$cols['address_zip']     = esc_html__('Zip', 'give');
+					$cols['address_country'] = esc_html__('Country', 'give');
 					break;
 				case 'userid' :
-					$cols['userid'] = esc_html__( 'User ID', 'give' );
+					$cols['userid'] = esc_html__('User ID', 'give');
 					break;
 				case 'donation_form' :
-					$cols['donation_form'] = esc_html__( 'Donation Form', 'give' );
+					$cols['donation_form'] = esc_html__('Donation Form', 'give');
 					break;
 				case 'date_first_donated' :
-					$cols['date_first_donated'] = esc_html__( 'First Donation Date', 'give' );
+					$cols['date_first_donated'] = esc_html__('First Donation Date', 'give');
 					break;
 				case 'donations' :
-					$cols['donations'] = esc_html__( 'Number of Donations', 'give' );
+					$cols['donations'] = esc_html__('Number of Donations', 'give');
 					break;
 				case 'donation_sum' :
-					$cols['donation_sum'] = esc_html__( 'Sum of Donations', 'give' );
+					$cols['donation_sum'] = esc_html__('Sum of Donations', 'give');
 					break;
 			}
 		}
@@ -192,29 +192,29 @@ 
 		$i = 0;
 
 		$data             = array();
-		$cached_donor_ids = Give_Cache::get( $this->query_id, true );
+		$cached_donor_ids = Give_Cache::get($this->query_id, true);
 
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 
 			// Export donors for a specific donation form and also within specified timeframe
 			$args = array(
 				'output'     => 'payments', // Use 'posts' to get standard post objects
-				'post_type'  => array( 'give_payment' ),
+				'post_type'  => array('give_payment'),
 				'number'     => 30,
 				'paged'      => $this->step,
 				'status'     => 'publish',
 				'meta_key'   => '_give_payment_form_id',
-				'meta_value' => absint( $this->form ),
+				'meta_value' => absint($this->form),
 			);
 
 			// Check for date option filter
-			if ( ! empty( $this->data['donor_export_start_date'] ) || ! empty( $this->data['donor_export_end_date'] ) ) {
-				$args['start_date'] = ! empty( $this->data['donor_export_start_date'] ) ? date( 'Y-n-d 00:00:00', strtotime( $this->data['donor_export_start_date'] ) ) : date( 'Y-n-d 23:59:59', '1970-1-01 00:00:00' );
-				$args['end_date']   = ! empty( $this->data['donor_export_end_date'] ) ? date( 'Y-n-d 23:59:59', strtotime( $this->data['donor_export_end_date'] ) ) : date( 'Y-n-d 23:59:59', current_time( 'timestamp' ) );
+			if ( ! empty($this->data['donor_export_start_date']) || ! empty($this->data['donor_export_end_date'])) {
+				$args['start_date'] = ! empty($this->data['donor_export_start_date']) ? date('Y-n-d 00:00:00', strtotime($this->data['donor_export_start_date'])) : date('Y-n-d 23:59:59', '1970-1-01 00:00:00');
+				$args['end_date']   = ! empty($this->data['donor_export_end_date']) ? date('Y-n-d 23:59:59', strtotime($this->data['donor_export_end_date'])) : date('Y-n-d 23:59:59', current_time('timestamp'));
 			}
 
 			// Check for price option.
-			if ( null !== $this->price_id ) {
+			if (null !== $this->price_id) {
 				$args['meta_query'] = array(
 					array(
 						'key'   => '_give_payment_price_id',
@@ -223,45 +223,43 @@ 
 				);
 			}
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 
-			if ( $payments ) {
+			if ($payments) {
 				/* @var Give_Payment $payment */
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 					// Set donation sum.
-					$this->payment_stats[ $payment->customer_id ]['donation_sum'] = isset( $this->payment_stats[ $payment->customer_id ]['donation_sum'] ) ?
-						$this->payment_stats[ $payment->customer_id ]['donation_sum'] :
-						0;
-					$this->payment_stats[ $payment->customer_id ]['donation_sum'] += $payment->total;
+					$this->payment_stats[$payment->customer_id]['donation_sum'] = isset($this->payment_stats[$payment->customer_id]['donation_sum']) ?
+						$this->payment_stats[$payment->customer_id]['donation_sum'] : 0;
+					$this->payment_stats[$payment->customer_id]['donation_sum'] += $payment->total;
 
 					// Set donation count.
-					$this->payment_stats[ $payment->customer_id ]['donations'] = isset( $this->payment_stats[ $payment->customer_id ]['donations'] ) ?
-						++ $this->payment_stats[ $payment->customer_id ]['donations'] :
-						1;
+					$this->payment_stats[$payment->customer_id]['donations'] = isset($this->payment_stats[$payment->customer_id]['donations']) ?
+						++$this->payment_stats[$payment->customer_id]['donations'] : 1;
 
 					// Set donation form name.
-					$this->payment_stats[ $payment->customer_id ]['form_title'] = $payment->form_title;
+					$this->payment_stats[$payment->customer_id]['form_title'] = $payment->form_title;
 
 					// Continue if donor already included.
-					if ( empty( $payment->customer_id ) ||
-					     in_array( $payment->customer_id, $cached_donor_ids )
+					if (empty($payment->customer_id) ||
+					     in_array($payment->customer_id, $cached_donor_ids)
 					) {
 						continue;
 					}
 
 					$this->donor_ids[] = $cached_donor_ids[] = $payment->customer_id;
 
-					$i ++;
+					$i++;
 				}
 
-				if ( ! empty( $this->donor_ids ) ) {
-					foreach ( $this->donor_ids as $donor_id ) {
-						$donor                      = Give()->donors->get_donor_by( 'id', $donor_id );
-						$donor->donation_form_title = $this->payment_stats[ $donor_id ]['form_title'];
-						$donor->purchase_count      = $this->payment_stats[ $donor_id ]['donations'];
-						$donor->purchase_value      = $this->payment_stats[ $donor_id ]['donation_sum'];
-						$data[]                     = $this->set_donor_data( $i, $data, $donor );
+				if ( ! empty($this->donor_ids)) {
+					foreach ($this->donor_ids as $donor_id) {
+						$donor                      = Give()->donors->get_donor_by('id', $donor_id);
+						$donor->donation_form_title = $this->payment_stats[$donor_id]['form_title'];
+						$donor->purchase_count      = $this->payment_stats[$donor_id]['donations'];
+						$donor->purchase_value      = $this->payment_stats[$donor_id]['donation_sum'];
+						$data[]                     = $this->set_donor_data($i, $data, $donor);
 					}
 
 					// Cache donor ids only if admin export donor for specific form.
@@ -271,7 +269,7 @@ 
 		} else {
 
 			// Export all donors.
-			$offset = 30 * ( $this->step - 1 );
+			$offset = 30 * ($this->step - 1);
 
 			$args = array(
 				'number' => 30,
@@ -279,31 +277,31 @@ 
 			);
 
 			// Check for date option filter
-			if ( ! empty( $this->data['donor_export_start_date'] ) || ! empty( $this->data['donor_export_end_date'] ) ) {
+			if ( ! empty($this->data['donor_export_start_date']) || ! empty($this->data['donor_export_end_date'])) {
 				$args['date'] = array(
-					'start' => ! empty( $this->data['donor_export_start_date'] ) ? date( 'Y-n-d 00:00:00', strtotime( $this->data['donor_export_start_date'] ) ) : date( 'Y-n-d 23:59:59', '1970-1-01 00:00:00' ),
-					'end'   => ! empty( $this->data['donor_export_end_date'] ) ? date( 'Y-n-d 23:59:59', strtotime( $this->data['donor_export_end_date'] ) ) : date( 'Y-n-d 23:59:59', current_time( 'timestamp' ) ),
+					'start' => ! empty($this->data['donor_export_start_date']) ? date('Y-n-d 00:00:00', strtotime($this->data['donor_export_start_date'])) : date('Y-n-d 23:59:59', '1970-1-01 00:00:00'),
+					'end'   => ! empty($this->data['donor_export_end_date']) ? date('Y-n-d 23:59:59', strtotime($this->data['donor_export_end_date'])) : date('Y-n-d 23:59:59', current_time('timestamp')),
 				);
 			}
 
-			$donors = Give()->donors->get_donors( $args );
+			$donors = Give()->donors->get_donors($args);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
 				// Continue if donor already included.
-				if ( empty( $donor->id ) || empty( $donor->payment_ids ) ) {
+				if (empty($donor->id) || empty($donor->payment_ids)) {
 					continue;
 				}
 
-				$payment                    = new Give_Payment( $donor->payment_ids );
+				$payment                    = new Give_Payment($donor->payment_ids);
 				$donor->donation_form_title = $payment->form_title;
-				$data[]                     = $this->set_donor_data( $i, $data, $donor );
-				$i ++;
+				$data[]                     = $this->set_donor_data($i, $data, $donor);
+				$i++;
 			}
 		}// End if().
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 		return $data;
 	}
@@ -319,18 +317,18 @@ 
 		$percentage = 0;
 
 		// We can't count the number when getting them for a specific form.
-		if ( empty( $this->form ) ) {
+		if (empty($this->form)) {
 
 			$total = Give()->donors->count();
 
-			if ( $total > 0 ) {
+			if ($total > 0) {
 
-				$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+				$percentage = ((30 * $this->step) / $total) * 100;
 
 			}
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -346,49 +344,49 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function set_donor_data( $i, $data, $donor ) {
+	private function set_donor_data($i, $data, $donor) {
 
 		$columns = $this->csv_cols();
 
 		// Set address variable
 		$address = '';
-		if ( isset( $donor->user_id ) && $donor->user_id > 0 ) {
-			$address = give_get_donor_address( $donor->user_id );
+		if (isset($donor->user_id) && $donor->user_id > 0) {
+			$address = give_get_donor_address($donor->user_id);
 		}
 
 		// Set columns
-		if ( ! empty( $columns['full_name'] ) ) {
-			$data[ $i ]['full_name'] = $donor->name;
+		if ( ! empty($columns['full_name'])) {
+			$data[$i]['full_name'] = $donor->name;
 		}
-		if ( ! empty( $columns['email'] ) ) {
-			$data[ $i ]['email'] = $donor->email;
+		if ( ! empty($columns['email'])) {
+			$data[$i]['email'] = $donor->email;
 		}
-		if ( ! empty( $columns['address_line1'] ) ) {
-
-			$data[ $i ]['address_line1']   = isset( $address['line1'] ) ? $address['line1'] : '';
-			$data[ $i ]['address_line2']   = isset( $address['line2'] ) ? $address['line2'] : '';
-			$data[ $i ]['address_city']    = isset( $address['city'] ) ? $address['city'] : '';
-			$data[ $i ]['address_state']   = isset( $address['state'] ) ? $address['state'] : '';
-			$data[ $i ]['address_zip']     = isset( $address['zip'] ) ? $address['zip'] : '';
-			$data[ $i ]['address_country'] = isset( $address['country'] ) ? $address['country'] : '';
+		if ( ! empty($columns['address_line1'])) {
+
+			$data[$i]['address_line1']   = isset($address['line1']) ? $address['line1'] : '';
+			$data[$i]['address_line2']   = isset($address['line2']) ? $address['line2'] : '';
+			$data[$i]['address_city']    = isset($address['city']) ? $address['city'] : '';
+			$data[$i]['address_state']   = isset($address['state']) ? $address['state'] : '';
+			$data[$i]['address_zip']     = isset($address['zip']) ? $address['zip'] : '';
+			$data[$i]['address_country'] = isset($address['country']) ? $address['country'] : '';
 		}
-		if ( ! empty( $columns['userid'] ) ) {
-			$data[ $i ]['userid'] = ! empty( $donor->user_id ) ? $donor->user_id : '';
+		if ( ! empty($columns['userid'])) {
+			$data[$i]['userid'] = ! empty($donor->user_id) ? $donor->user_id : '';
 		}
-		if ( ! empty( $columns['donation_form'] ) ) {
-			$data[ $i ]['donation_form'] = ! empty( $donor->donation_form_title ) ? $donor->donation_form_title : '';
+		if ( ! empty($columns['donation_form'])) {
+			$data[$i]['donation_form'] = ! empty($donor->donation_form_title) ? $donor->donation_form_title : '';
 		}
-		if ( ! empty( $columns['date_first_donated'] ) ) {
-			$data[ $i ]['date_first_donated'] = date_i18n( give_date_format(), strtotime( $donor->date_created ) );
+		if ( ! empty($columns['date_first_donated'])) {
+			$data[$i]['date_first_donated'] = date_i18n(give_date_format(), strtotime($donor->date_created));
 		}
-		if ( ! empty( $columns['donations'] ) ) {
-			$data[ $i ]['donations'] = $donor->purchase_count;
+		if ( ! empty($columns['donations'])) {
+			$data[$i]['donations'] = $donor->purchase_count;
 		}
-		if ( ! empty( $columns['donation_sum'] ) ) {
-			$data[ $i ]['donation_sum'] = give_format_amount( $donor->purchase_value, array( 'sanitize' => false ) );
+		if ( ! empty($columns['donation_sum'])) {
+			$data[$i]['donation_sum'] = give_format_amount($donor->purchase_value, array('sanitize' => false));
 		}
 
-		return $data[ $i ];
+		return $data[$i];
 
 	}
 
@@ -398,9 +396,9 @@ 
 	 * @param array             $request
 	 * @param Give_Batch_Export $export
 	 */
-	public function unset_properties( $request, $export ) {
-		if ( $export->done ) {
-			Give_Cache::delete( "give_cache_{$this->query_id}" );
+	public function unset_properties($request, $export) {
+		if ($export->done) {
+			Give_Cache::delete("give_cache_{$this->query_id}");
 		}
 	}
 }
\ No newline at end of file

includes/admin/tools/export/class-batch-export.php

Spacing +38 added lines, -38 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -118,14 +118,14 @@ 
 	 *
 	 * @param int $_step
 	 */
-	public function __construct( $_step = 1 ) {
+	public function __construct($_step = 1) {
 
 		$upload_dir     = wp_upload_dir();
 		$this->filetype = '.csv';
-		$this->filename = 'give-' . $this->export_type . $this->filetype;
-		$this->file     = trailingslashit( $upload_dir['basedir'] ) . $this->filename;
+		$this->filename = 'give-'.$this->export_type.$this->filetype;
+		$this->file     = trailingslashit($upload_dir['basedir']).$this->filename;
 
-		if ( ! is_writeable( $upload_dir['basedir'] ) ) {
+		if ( ! is_writeable($upload_dir['basedir'])) {
 			$this->is_writable = false;
 		}
 
@@ -141,22 +141,22 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to export data.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to export data.', 'give'), esc_html__('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
-		if ( $this->step < 2 ) {
+		if ($this->step < 2) {
 
 			// Make sure we start with a fresh file on step 1.
-			@unlink( $this->file );
+			@unlink($this->file);
 			$this->print_csv_cols();
 		}
 
 		$rows = $this->print_csv_rows();
 
-		if ( $rows ) {
+		if ($rows) {
 			return true;
 		} else {
 			return false;
@@ -176,14 +176,14 @@ 
 		$col_data = '';
 		$cols     = $this->get_csv_cols();
 		$i        = 1;
-		foreach ( $cols as $col_id => $column ) {
-			$col_data .= '"' . addslashes( $column ) . '"';
-			$col_data .= $i == count( $cols ) ? '' : ',';
-			$i ++;
+		foreach ($cols as $col_id => $column) {
+			$col_data .= '"'.addslashes($column).'"';
+			$col_data .= $i == count($cols) ? '' : ',';
+			$i++;
 		}
 		$col_data .= "\r\n";
 
-		$this->stash_step_data( $col_data );
+		$this->stash_step_data($col_data);
 
 		return $col_data;
 
@@ -202,23 +202,23 @@ 
 		$data     = $this->get_data();
 		$cols     = $this->get_csv_cols();
 
-		if ( $data ) {
+		if ($data) {
 
 			// Output each row
-			foreach ( $data as $row ) {
+			foreach ($data as $row) {
 				$i = 1;
-				foreach ( $row as $col_id => $column ) {
+				foreach ($row as $col_id => $column) {
 					// Make sure the column is valid
-					if ( array_key_exists( $col_id, $cols ) ) {
-						$row_data .= '"' . addslashes( preg_replace( '/"/', "'", $column ) ) . '"';
-						$row_data .= $i == count( $cols ) ? '' : ',';
-						$i ++;
+					if (array_key_exists($col_id, $cols)) {
+						$row_data .= '"'.addslashes(preg_replace('/"/', "'", $column)).'"';
+						$row_data .= $i == count($cols) ? '' : ',';
+						$i++;
 					}
 				}
 				$row_data .= "\r\n";
 			}
 
-			$this->stash_step_data( $row_data );
+			$this->stash_step_data($row_data);
 
 			return $row_data;
 		}
@@ -246,18 +246,18 @@ 
 
 		$file = '';
 
-		if ( @file_exists( $this->file ) ) {
+		if (@file_exists($this->file)) {
 
-			if ( ! is_writeable( $this->file ) ) {
+			if ( ! is_writeable($this->file)) {
 				$this->is_writable = false;
 			}
 
-			$file = @file_get_contents( $this->file );
+			$file = @file_get_contents($this->file);
 
 		} else {
 
-			@file_put_contents( $this->file, '' );
-			@chmod( $this->file, 0664 );
+			@file_put_contents($this->file, '');
+			@chmod($this->file, 0664);
 
 		}
 
@@ -273,18 +273,18 @@ 
 	 *
 	 * @return void
 	 */
-	protected function stash_step_data( $data = '' ) {
+	protected function stash_step_data($data = '') {
 
 		$file = $this->get_file();
 		$file .= $data;
-		@file_put_contents( $this->file, $file );
+		@file_put_contents($this->file, $file);
 
 		// If we have no rows after this step, mark it as an empty export.
-		$file_rows    = file( $this->file, FILE_SKIP_EMPTY_LINES );
+		$file_rows    = file($this->file, FILE_SKIP_EMPTY_LINES);
 		$default_cols = $this->get_csv_cols();
-		$default_cols = empty( $default_cols ) ? 0 : 1;
+		$default_cols = empty($default_cols) ? 0 : 1;
 
-		$this->is_empty = count( $file_rows ) == $default_cols ? true : false;
+		$this->is_empty = count($file_rows) == $default_cols ? true : false;
 
 	}
 
@@ -302,7 +302,7 @@ 
 
 		$file = $this->get_file();
 
-		@unlink( $this->file );
+		@unlink($this->file);
 
 		echo $file;
 
@@ -311,7 +311,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( 'give_file_export_complete', $_REQUEST );
+		do_action('give_file_export_complete', $_REQUEST);
 
 		give_die();
 	}
@@ -323,7 +323,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -334,7 +334,7 @@ 
 	 * @param array             $request The Form Data passed into the batch processing.
 	 * @param Give_Batch_Export $export
 	 */
-	public function unset_properties( $request, $export ) {
+	public function unset_properties($request, $export) {
 	}
 
 	/**

includes/admin/tools/views/html-admin-page-exports.php

Spacing +56 added lines, -56 removed lines

@@ -3,7 +3,7 @@ 
  * Admin View: Exports
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 } ?>
 
@@ -18,14 +18,14 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_top' );
+				do_action('give_tools_tab_export_content_top');
 				?>
 
 				<table class="widefat export-options-table give-table">
 					<thead>
 					<tr>
-						<th scope="col"><?php esc_html_e( 'Export Type', 'give' ); ?></th>
-						<th scope="col"><?php esc_html_e( 'Export Options', 'give' ); ?></th>
+						<th scope="col"><?php esc_html_e('Export Type', 'give'); ?></th>
+						<th scope="col"><?php esc_html_e('Export Options', 'give'); ?></th>
 					</tr>
 					</thead>
 					<tbody>
@@ -38,42 +38,42 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_top' );
+					do_action('give_tools_tab_export_table_top');
 					?>
 					<tr class="give-export-pdf-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export PDF of Donations and Income', 'give' ); ?></span>
+								<span><?php esc_html_e('Export PDF of Donations and Income', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a PDF of Donations and Income reports for all forms for the current year.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a PDF of Donations and Income reports for all forms for the current year.', 'give'); ?></p>
 						</td>
 						<td>
-							<a class="button" href="<?php echo wp_nonce_url( add_query_arg( array( 'give-action' => 'generate_pdf' ) ), 'give_generate_pdf' ); ?>">
-								<?php esc_html_e( 'Generate PDF', 'give' ); ?>
+							<a class="button" href="<?php echo wp_nonce_url(add_query_arg(array('give-action' => 'generate_pdf')), 'give_generate_pdf'); ?>">
+								<?php esc_html_e('Generate PDF', 'give'); ?>
 							</a>
 						</td>
 					</tr>
 					<tr class="alternate give-export-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Income and Donation Stats', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Income and Donation Stats', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a CSV of income and donations over time.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a CSV of income and donations over time.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post">
 								<?php
 								printf(
 								/* translators: 1: start date dropdown 2: end date dropdown */
-									esc_html__( '%1$s to %2$s', 'give' ),
-									Give()->html->year_dropdown( 'start_year' ) . ' ' . Give()->html->month_dropdown( 'start_month' ),
-									Give()->html->year_dropdown( 'end_year' ) . ' ' . Give()->html->month_dropdown( 'end_month' )
+									esc_html__('%1$s to %2$s', 'give'),
+									Give()->html->year_dropdown('start_year').' '.Give()->html->month_dropdown('start_month'),
+									Give()->html->year_dropdown('end_year').' '.Give()->html->month_dropdown('end_month')
 								);
 								?>
 								<input type="hidden" name="give-action"
 								       value="earnings_export"/>
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 							</form>
 						</td>
@@ -81,37 +81,37 @@ 
 					<tr class="give-export-payment-history">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Donation History', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Donation History', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a CSV of all donations recorded.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a CSV of all donations recorded.', 'give'); ?></p>
 						</td>
 						<td>
 							<form id="give-export-payments"
 							      class="give-export-form" method="post">
 								<?php
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give-payment-export-start',
 									'name'        => 'start',
-									'placeholder' => esc_attr__( 'Start date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('Start date', 'give'),
+								));
 
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give-payment-export-end',
 									'name'        => 'end',
-									'placeholder' => esc_attr__( 'End date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('End date', 'give'),
+								));
 								?>
 								<select name="status">
-									<option value="any"><?php esc_html_e( 'All Statuses', 'give' ); ?></option>
+									<option value="any"><?php esc_html_e('All Statuses', 'give'); ?></option>
 									<?php
 									$statuses = give_get_payment_statuses();
-									foreach ( $statuses as $status => $label ) {
-										echo '<option value="' . $status . '">' . $label . '</option>';
+									foreach ($statuses as $status => $label) {
+										echo '<option value="'.$status.'">'.$label.'</option>';
 									}
 									?>
 								</select>
 								<?php
-								if ( give_is_setting_enabled( give_get_option( 'categories' ) ) ) {
+								if (give_is_setting_enabled(give_get_option('categories'))) {
 									echo Give()->html->category_dropdown(
 										'give_forms_categories[]',
 										0,
@@ -121,12 +121,12 @@ 
 											'multiple'        => true,
 											'selected'        => array(),
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from categories', 'give' ),
+											'placeholder'     => __('Choose one or more from categories', 'give'),
 										)
 									);
 								}
 
-								if ( give_is_setting_enabled( give_get_option( 'tags' ) ) ) {
+								if (give_is_setting_enabled(give_get_option('tags'))) {
 									echo Give()->html->tags_dropdown(
 										'give_forms_tags[]',
 										0,
@@ -136,18 +136,18 @@ 
 											'multiple'        => true,
 											'selected'        => array(),
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from tags', 'give' ),
+											'placeholder'     => __('Choose one or more from tags', 'give'),
 										)
 									);
 								}
 
-								wp_nonce_field( 'give_ajax_export', 'give_ajax_export' );
+								wp_nonce_field('give_ajax_export', 'give_ajax_export');
 								?>
 								<input type="hidden" name="give-export-class"
 								       value="Give_Batch_Payments_Export"/>
 								<span>
 									<input type="submit"
-									       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+									       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 									       class="button-secondary"/>
 									<span class="spinner"></span>
 								</span>
@@ -157,9 +157,9 @@ 
 					<tr class="alternate give-export-donors">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Donors in CSV', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Donors in CSV', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download an export of donors for all donation forms or only those who have given to a particular form.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download an export of donors for all donation forms or only those who have given to a particular form.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post" id="give_donor_export"
@@ -167,99 +167,99 @@ 
 
 								<?php
 								// Start Date form field for donors
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give_donor_export_start_date',
 									'name'        => 'donor_export_start_date',
-									'placeholder' => esc_attr__( 'Start date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('Start date', 'give'),
+								));
 
 								// End Date form field for donors
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give_donor_export_end_date',
 									'name'        => 'donor_export_end_date',
-									'placeholder' => esc_attr__( 'End date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('End date', 'give'),
+								));
 
 								// Donation forms dropdown for donors export
-								echo Give()->html->forms_dropdown( array(
+								echo Give()->html->forms_dropdown(array(
 									'name'   => 'forms',
 									'id'     => 'give_donor_export_form',
 									'chosen' => true,
-								) );
+								));
 								?>
 
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 
 								<div id="export-donor-options-wrap"
 								     class="give-clearfix">
-									<p><?php esc_html_e( 'Export Columns:', 'give' ); ?></p>
+									<p><?php esc_html_e('Export Columns:', 'give'); ?></p>
 									<ul id="give-export-option-ul">
 										<li>
 											<label for="give-export-fullname">
 												<input type="checkbox" checked
 												       name="give_export_option[full_name]"
-												       id="give-export-fullname"><?php esc_html_e( 'Name', 'give' ); ?>
+												       id="give-export-fullname"><?php esc_html_e('Name', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-email">
 												<input type="checkbox" checked
 												       name="give_export_option[email]"
-												       id="give-export-email"><?php esc_html_e( 'Email', 'give' ); ?>
+												       id="give-export-email"><?php esc_html_e('Email', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-address">
 												<input type="checkbox" checked
 												       name="give_export_option[address]"
-												       id="give-export-address"><?php esc_html_e( 'Address', 'give' ); ?>
+												       id="give-export-address"><?php esc_html_e('Address', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-userid">
 												<input type="checkbox" checked
 												       name="give_export_option[userid]"
-												       id="give-export-userid"><?php esc_html_e( 'User ID', 'give' ); ?>
+												       id="give-export-userid"><?php esc_html_e('User ID', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-form">
 												<input type="checkbox" checked
 												       name="give_export_option[donation_form]"
-												       id="give-export-donation-form"><?php esc_html_e( 'Donation Form', 'give' ); ?>
+												       id="give-export-donation-form"><?php esc_html_e('Donation Form', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-first-donation-date">
 												<input type="checkbox" checked
 												       name="give_export_option[date_first_donated]"
-												       id="give-export-first-donation-date"><?php esc_html_e( 'First Donation Date', 'give' ); ?>
+												       id="give-export-first-donation-date"><?php esc_html_e('First Donation Date', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-number">
 												<input type="checkbox" checked
 												       name="give_export_option[donations]"
-												       id="give-export-donation-number"><?php esc_html_e( 'Number of Donations', 'give' ); ?>
+												       id="give-export-donation-number"><?php esc_html_e('Number of Donations', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-sum">
 												<input type="checkbox" checked
 												       name="give_export_option[donation_sum]"
-												       id="give-export-donation-sum"><?php esc_html_e( 'Total Donated', 'give' ); ?>
+												       id="give-export-donation-sum"><?php esc_html_e('Total Donated', 'give'); ?>
 											</label>
 										</li>
 									</ul>
 								</div>
-								<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+								<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 								<input type="hidden" name="give-export-class"
 								       value="Give_Batch_Donors_Export"/>
 								<input type="hidden"
 								       name="give_export_option[query_id]"
-								       value="<?php echo uniqid( 'give_' ); ?>"/>
+								       value="<?php echo uniqid('give_'); ?>"/>
 							</form>
 						</td>
 					</tr>
@@ -272,7 +272,7 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_bottom' );
+					do_action('give_tools_tab_export_table_bottom');
 					?>
 					</tbody>
 				</table>
@@ -283,7 +283,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_bottom' );
+				do_action('give_tools_tab_export_content_bottom');
 				?>
 
 			</div>