WordImpress / Give

includes/admin/reporting/tools.php

Spacing +26 added lines, -26 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,7 +22,7 @@ 
  */
 function give_tools_recount_stats_display() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 
@@ -31,27 +31,27 @@ 
 	 *
 	 * @since 1.5
 	 */
-	do_action( 'give_tools_recount_stats_before' );
+	do_action('give_tools_recount_stats_before');
 	?>
 	<div id="poststuff">
 		<div class="postbox">
 
-			<h2 class="hndle ui-sortable-handle"><span><?php esc_html_e( 'Recount Stats', 'give' ); ?></span></h2>
+			<h2 class="hndle ui-sortable-handle"><span><?php esc_html_e('Recount Stats', 'give'); ?></span></h2>
 
 			<div class="inside recount-stats-controls">
-				<p><?php esc_html_e( 'Use these tools to recount stats, delete test transactions, or reset stats.', 'give' ); ?></p>
+				<p><?php esc_html_e('Use these tools to recount stats, delete test transactions, or reset stats.', 'give'); ?></p>
 				<form method="post" id="give-tools-recount-form" class="give-export-form">
 
-					<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+					<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 
 					<select name="give-export-class" id="recount-stats-type">
-						<option value="0" selected="selected" disabled="disabled"><?php esc_html_e( 'Please select an option', 'give' ); ?></option>
-						<option data-type="recount-stats" value="Give_Tools_Recount_Income"><?php esc_html_e( 'Recalculate Total Donation Income Amount', 'give' ); ?></option>
-						<option data-type="recount-form" value="Give_Tools_Recount_Form_Stats"><?php esc_html_e( 'Recalculate Income Amount and Donation Counts for a Form', 'give' ); ?></option>
-						<option data-type="recount-all" value="Give_Tools_Recount_All_Stats"><?php esc_html_e( 'Recalculate Income Amount and Donation Counts for All Forms', 'give' ); ?></option>
-						<option data-type="recount-customer-stats" value="Give_Tools_Recount_Customer_Stats"><?php esc_html_e( 'Recalculate Donor Statistics', 'give' ); ?></option>
-						<option data-type="delete-test-transactions" value="Give_Tools_Delete_Test_Transactions"><?php esc_html_e( 'Delete Test Transactions', 'give' ); ?></option>
-						<option data-type="reset-stats" value="Give_Tools_Reset_Stats"><?php esc_html_e( 'Delete All Data', 'give' ); ?></option>
+						<option value="0" selected="selected" disabled="disabled"><?php esc_html_e('Please select an option', 'give'); ?></option>
+						<option data-type="recount-stats" value="Give_Tools_Recount_Income"><?php esc_html_e('Recalculate Total Donation Income Amount', 'give'); ?></option>
+						<option data-type="recount-form" value="Give_Tools_Recount_Form_Stats"><?php esc_html_e('Recalculate Income Amount and Donation Counts for a Form', 'give'); ?></option>
+						<option data-type="recount-all" value="Give_Tools_Recount_All_Stats"><?php esc_html_e('Recalculate Income Amount and Donation Counts for All Forms', 'give'); ?></option>
+						<option data-type="recount-customer-stats" value="Give_Tools_Recount_Customer_Stats"><?php esc_html_e('Recalculate Donor Statistics', 'give'); ?></option>
+						<option data-type="delete-test-transactions" value="Give_Tools_Delete_Test_Transactions"><?php esc_html_e('Delete Test Transactions', 'give'); ?></option>
+						<option data-type="reset-stats" value="Give_Tools_Reset_Stats"><?php esc_html_e('Delete All Data', 'give'); ?></option>
 						<?php
 						/**
 						 * Fires in the recount stats selectbox.
@@ -60,7 +60,7 @@ 
 						 *
 						 * @since 1.5
 						 */
-						do_action( 'give_recount_tool_options' );
+						do_action('give_recount_tool_options');
 						?>
 					</select>
 
@@ -68,22 +68,22 @@ 
 						<?php
 						$args = array(
 							'name'   => 'form_id',
-							'number' => - 1,
+							'number' => -1,
 							'chosen' => true,
 						);
-						echo Give()->html->forms_dropdown( $args );
+						echo Give()->html->forms_dropdown($args);
 						?>
 					</span>
 
-					<input type="submit" id="recount-stats-submit" value="<?php esc_attr_e( 'Submit', 'give' ); ?>" class="button-secondary"/>
+					<input type="submit" id="recount-stats-submit" value="<?php esc_attr_e('Submit', 'give'); ?>" class="button-secondary"/>
 
 					<br/>
 
 					<span class="give-recount-stats-descriptions">
-						<span id="recount-stats"><?php esc_html_e( 'Recalculates the overall donation income amount.', 'give' ); ?></span>
-						<span id="recount-form"><?php esc_html_e( 'Recalculates the donation and income stats for a specific form.', 'give' ); ?></span>
-						<span id="recount-all"><?php esc_html_e( 'Recalculates the earnings and sales stats for all forms.', 'give' ); ?></span>
-						<span id="recount-customer-stats"><?php esc_html_e( 'Recalculates the lifetime value and donation counts for all donors.', 'give' ); ?></span>
+						<span id="recount-stats"><?php esc_html_e('Recalculates the overall donation income amount.', 'give'); ?></span>
+						<span id="recount-form"><?php esc_html_e('Recalculates the donation and income stats for a specific form.', 'give'); ?></span>
+						<span id="recount-all"><?php esc_html_e('Recalculates the earnings and sales stats for all forms.', 'give'); ?></span>
+						<span id="recount-customer-stats"><?php esc_html_e('Recalculates the lifetime value and donation counts for all donors.', 'give'); ?></span>
 						<?php
 						/**
 						 * Fires in the recount stats description area.
@@ -92,10 +92,10 @@ 
 						 *
 						 * @since 1.5
 						 */
-						do_action( 'give_recount_tool_descriptions' );
+						do_action('give_recount_tool_descriptions');
 						?>
-						<span id="delete-test-transactions"><?php _e( '<strong>Deletes</strong> all TEST donations, donors, and related log entries.', 'give' ); ?></span>
-						<span id="reset-stats"><?php _e( '<strong>Deletes</strong> ALL donations, donors, and related log entries regardless of test or live mode.', 'give' ); ?></span>
+						<span id="delete-test-transactions"><?php _e('<strong>Deletes</strong> all TEST donations, donors, and related log entries.', 'give'); ?></span>
+						<span id="reset-stats"><?php _e('<strong>Deletes</strong> ALL donations, donors, and related log entries regardless of test or live mode.', 'give'); ?></span>
 					</span>
 
 					<span class="spinner"></span>
@@ -109,7 +109,7 @@ 
 				 *
 				 * @since 1.5
 				 */
-				do_action( 'give_tools_recount_forms' );
+				do_action('give_tools_recount_forms');
 				?>
 			</div><!-- .inside -->
 		</div><!-- .postbox -->
@@ -120,5 +120,5 @@ 
 	 *
 	 * @since 1.5
 	 */
-	do_action( 'give_tools_recount_stats_after' );
+	do_action('give_tools_recount_stats_after');
 }
\ No newline at end of file

includes/admin/views/html-admin-settings.php

Spacing +15 added lines, -15 removed lines

@@ -2,13 +2,13 @@ 
 /**
  * Admin View: Settings
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 
 // Bailout: Do not output anything if setting tab is not defined.
-if( ! empty( $tabs ) ) :
+if ( ! empty($tabs)) :
 	/**
 	 * Filter the form action.
 	 *
@@ -18,7 +18,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_method    = apply_filters( self::$setting_filter_prefix . '_form_method_tab_' . $current_tab, 'post' );
+	$form_method = apply_filters(self::$setting_filter_prefix.'_form_method_tab_'.$current_tab, 'post');
 
 	/**
 	 * Filter the main form tab.
@@ -30,15 +30,15 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_open_tag  = apply_filters( self::$setting_filter_prefix . '_open_form', '<form method="' . $form_method  . '" id="give-mainform" action="" enctype="multipart/form-data">' );
-	$form_close_tag = apply_filters( self::$setting_filter_prefix . '_close_form', '</form>' );
+	$form_open_tag  = apply_filters(self::$setting_filter_prefix.'_open_form', '<form method="'.$form_method.'" id="give-mainform" action="" enctype="multipart/form-data">');
+	$form_close_tag = apply_filters(self::$setting_filter_prefix.'_close_form', '</form>');
 	?>
-	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix . '-setting-page'; ?>">
+	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix.'-setting-page'; ?>">
 		<?php echo $form_open_tag; ?>
 			<h2 class="nav-tab-wrapper woo-nav-tab-wrapper">
 				<?php
-				foreach ( $tabs as $name => $label ) {
-					echo '<a href="' . admin_url( "edit.php?post_type=give_forms&page=" . self::$setting_filter_prefix . "&tab={$name}" ) . '" class="nav-tab ' . ( $current_tab == $name ? 'nav-tab-active' : '' ) . '">' . $label . '</a>';
+				foreach ($tabs as $name => $label) {
+					echo '<a href="'.admin_url("edit.php?post_type=give_forms&page=".self::$setting_filter_prefix."&tab={$name}").'" class="nav-tab '.($current_tab == $name ? 'nav-tab-active' : '').'">'.$label.'</a>';
 				}
 
 				/**
@@ -50,10 +50,10 @@ 
 				 *
 				 * @since 1.8
 				 */
-				do_action( self::$setting_filter_prefix . '_tabs' );
+				do_action(self::$setting_filter_prefix.'_tabs');
 				?>
 			</h2>
-			<h1 class="screen-reader-text"><?php echo esc_html( $tabs[ $current_tab ] ); ?></h1>
+			<h1 class="screen-reader-text"><?php echo esc_html($tabs[$current_tab]); ?></h1>
 			<?php
 
 			/**
@@ -65,7 +65,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . "_sections_{$current_tab}_page" );
+			do_action(self::$setting_filter_prefix."_sections_{$current_tab}_page");
 
 			
 			// Show messages.
@@ -81,12 +81,12 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . "_settings_{$current_tab}_page" );
+			do_action(self::$setting_filter_prefix."_settings_{$current_tab}_page");
 
-			if ( empty( $GLOBALS['give_hide_save_button'] ) ) : ?>
+			if (empty($GLOBALS['give_hide_save_button'])) : ?>
 				<div class="give-submit-wrap">
-					<?php wp_nonce_field( 'give-save-settings', '_give-save-settings' ); ?>
-					<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e( 'Save changes', 'give' ); ?>" />
+					<?php wp_nonce_field('give-save-settings', '_give-save-settings'); ?>
+					<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e('Save changes', 'give'); ?>" />
 				</div>
 			<?php endif; ?>
 		<?php echo $form_close_tag; ?>

includes/admin/tools/class-settings-api.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_API' ) ) :
+if ( ! class_exists('Give_Settings_API')) :
 
 	/**
 	 * Give_Settings_API.
@@ -43,10 +43,10 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'api';
-			$this->label = esc_html__( 'API', 'give' );
+			$this->label = esc_html__('API', 'give');
 
-			add_filter( 'give-tools_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-tools_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter('give-tools_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-tools_settings_{$this->id}_page", array($this, 'output'));
 		}
 
 		/**
@@ -56,8 +56,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -73,7 +73,7 @@ 
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Get settings.
-			$settings = apply_filters( 'give_settings_system', array(
+			$settings = apply_filters('give_settings_system', array(
 				array(
 					'id'   => 'give_tools_api',
 					'type' => 'title',
@@ -81,7 +81,7 @@ 
 				),
 				array(
 					'id'   => 'api',
-					'name' => esc_html__( 'API', 'give' ),
+					'name' => esc_html__('API', 'give'),
 					'type' => 'api',
 				),
 				array(
@@ -97,7 +97,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -112,7 +112,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 	}
 

includes/admin/tools/class-settings-data.php

Spacing +14 added lines, -14 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Data' ) ) :
+if ( ! class_exists('Give_Settings_Data')) :
 
 	/**
 	 * Give_Settings_Data.
@@ -43,15 +43,15 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'data';
-			$this->label = esc_html__( 'Data', 'give' );
+			$this->label = esc_html__('Data', 'give');
 
-			add_filter( 'give-tools_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-tools_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter('give-tools_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-tools_settings_{$this->id}_page", array($this, 'output'));
 
 			// Do not use main form for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( "give-tools_open_form", '__return_empty_string' );
-				add_action( "give-tools_close_form", '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action("give-tools_open_form", '__return_empty_string');
+				add_action("give-tools_close_form", '__return_empty_string');
 			}
 		}
 
@@ -62,8 +62,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -79,7 +79,7 @@ 
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Get settings.
-			$settings = apply_filters( 'give_settings_system', array(
+			$settings = apply_filters('give_settings_system', array(
 				array(
 					'id'   => 'give_tools_tools',
 					'type' => 'title',
@@ -87,7 +87,7 @@ 
 				),
 				array(
 					'id'   => 'api',
-					'name' => esc_html__( 'Tools', 'give' ),
+					'name' => esc_html__('Tools', 'give'),
 					'type' => 'data',
 				),
 				array(
@@ -103,7 +103,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -118,7 +118,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 	}
 

includes/filters.php

Spacing +11 added lines, -11 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,28 +26,28 @@ 
  *
  * @return void
  */
-function give_set_settings_with_disable_prefix( $settings, $option_name ) {
+function give_set_settings_with_disable_prefix($settings, $option_name) {
 	// Get old setting names.
 	$old_settings   = give_v18_renamed_core_settings();
 	$update_setting = false;
 
-	foreach ( $settings as $key => $value ) {
+	foreach ($settings as $key => $value) {
 
 		// Check 1. Check if new option is really updated or not.
 		// Check 2. Continue if key is not renamed.
 		if (
-			! isset( $_POST[ $key ] )
-			|| false === ( $old_setting_name = array_search( $key, $old_settings ) )
+			! isset($_POST[$key])
+			|| false === ($old_setting_name = array_search($key, $old_settings))
 		) {
 			continue;
 		}
 
 		// Set old setting.
-		$settings[ $old_setting_name ] = 'on';
+		$settings[$old_setting_name] = 'on';
 
 		// Do not need to set old setting if new setting is not set.
-		if ( give_is_setting_enabled( $value ) ) {
-			unset( $settings[ $old_setting_name ] );
+		if (give_is_setting_enabled($value)) {
+			unset($settings[$old_setting_name]);
 		}
 
 		// Tell bot to update setting.
@@ -55,9 +55,9 @@ 
 	}
 
 	// Update setting if any old setting set.
-	if ( $update_setting ) {
-		update_option( $option_name, $settings );
+	if ($update_setting) {
+		update_option($option_name, $settings);
 	}
 }
 
-add_filter( 'give_save_settings_give_settings', 'give_set_settings_with_disable_prefix', 10, 2 );
+add_filter('give_save_settings_give_settings', 'give_set_settings_with_disable_prefix', 10, 2);

includes/class-give-logging.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 	 *
 	 * Display admin bar when active
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function give_admin_bar_menu() {
 		global $wp_admin_bar;

Spacing +68 added lines, -68 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
 	 */
 	public function __setup_hooks() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache') );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache') );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache') );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache') );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
 	}
 
 	/**
@@ -67,7 +67,7 @@ 
 	public function register_post_type() {
 		/* Logs post type */
 		$log_args = array(
-			'labels'              => array( 'name' => esc_html__( 'Logs', 'give' ) ),
+			'labels'              => array('name' => esc_html__('Logs', 'give')),
 			'public'              => false,
 			'exclude_from_search' => true,
 			'publicly_queryable'  => false,
@@ -75,11 +75,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true,
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -93,7 +93,7 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array( 'public' => false ) );
+		register_taxonomy('give_log_type', 'give_log', array('public' => false));
 	}
 
 	/**
@@ -113,7 +113,7 @@ 
 			'api_request',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -128,8 +128,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -148,7 +148,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -156,7 +156,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -173,12 +173,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type,
-		) );
+		));
 	}
 
 	/**
@@ -192,7 +192,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -201,7 +201,7 @@ 
 			'log_type'     => false,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		/**
 		 * Fires before inserting log entry.
@@ -211,20 +211,20 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -237,7 +237,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -253,7 +253,7 @@ 
 	 *
 	 * @return bool|null       True if successful, false otherwise.
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
 		/**
 		 * Fires before updating log entry.
@@ -263,7 +263,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -271,15 +271,15 @@ 
 			'post_parent' => 0,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
@@ -293,7 +293,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -308,19 +308,19 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false,
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -330,9 +330,9 @@ 
 			);
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -355,17 +355,17 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -375,25 +375,25 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
 
 		// Get cache key for current query.
-		$cache_key =  give_get_cache_key( 'get_log_count', $query_args );
+		$cache_key = give_get_cache_key('get_log_count', $query_args);
 
 		// check if cache already exist or not.
-		if( ! ( $logs_count = get_option( $cache_key ) ) ) {
-			$logs = new WP_Query( $query_args );
+		if ( ! ($logs_count = get_option($cache_key))) {
+			$logs = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 
 			// Cache results.
-			add_option( $cache_key, $logs_count, '', 'no' );
+			add_option($cache_key, $logs_count, '', 'no');
 		}
 
 		return $logs_count;
@@ -413,16 +413,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -432,15 +432,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -453,8 +453,8 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
-		wp_schedule_single_event( time(), 'give_delete_log_cache' );
+	public function background_process_delete_cache($post_id) {
+		wp_schedule_single_event(time(), 'give_delete_log_cache');
 	}
 
 	/**
@@ -476,14 +476,14 @@ 
 		);
 
 		// Bailout.
-		if( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
 
 		// Delete log cache.
-		foreach ( $cache_option_names as $option_name ) {
-			delete_option( $option_name['option_name'] );
+		foreach ($cache_option_names as $option_name) {
+			delete_option($option_name['option_name']);
 		}
 	}
 }
@@ -506,10 +506,10 @@ 
  *
  * @return int             ID of the new log entry.
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	/* @var Give_Logging $give_logs */
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }

includes/class-give-cli-commands.php

Spacing +196 added lines, -196 removed lines

@@ -7,12 +7,12 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Add give command.
-WP_CLI::add_command( 'give', 'GIVE_CLI_COMMAND' );
+WP_CLI::add_command('give', 'GIVE_CLI_COMMAND');
 
 
 /**
@@ -68,8 +68,8 @@ 
 	 *
 	 * @subcommand    logo
 	 */
-	public function ascii( $args, $assoc_args ) {
-		WP_CLI::log( file_get_contents( GIVE_PLUGIN_DIR . 'assets/images/give-ascii-logo.txt' ) );
+	public function ascii($args, $assoc_args) {
+		WP_CLI::log(file_get_contents(GIVE_PLUGIN_DIR.'assets/images/give-ascii-logo.txt'));
 	}
 
 
@@ -94,54 +94,54 @@ 
 	 *
 	 * @subcommand    details
 	 */
-	public function details( $args, $assoc_args ) {
+	public function details($args, $assoc_args) {
 
 		/**
 		 * Plugin Information
 		 */
-		WP_CLI::log( $this->color_message( __( 'Give Version: ', 'give' ) ) . GIVE_VERSION );
+		WP_CLI::log($this->color_message(__('Give Version: ', 'give')).GIVE_VERSION);
 
 		/**
 		 * General Information.
 		 */
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'General information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('General information', 'give')).'   ####');
 
-		$success_page = give_get_option( 'success_page' );
-		$failure_page = give_get_option( 'failure_page' );
-		$history_page = give_get_option( 'history_page' );
+		$success_page = give_get_option('success_page');
+		$failure_page = give_get_option('failure_page');
+		$history_page = give_get_option('history_page');
 
-		WP_CLI::log( $this->color_message( sprintf( __( 'Success Page: ', 'give' ) ) ) . ( $success_page ? "[{$success_page}] " . get_permalink( $success_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Failed Donation Page: ', 'give' ) ) . ( $failure_page ? "[{$failure_page}] " . get_permalink( $failure_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Donation History Page: ', 'give' ) ) . ( $history_page ? "[{$history_page}] " . get_permalink( $history_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Country: ', 'give' ) ) . give_get_country() );
+		WP_CLI::log($this->color_message(sprintf(__('Success Page: ', 'give'))).($success_page ? "[{$success_page}] ".get_permalink($success_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Failed Donation Page: ', 'give')).($failure_page ? "[{$failure_page}] ".get_permalink($failure_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Donation History Page: ', 'give')).($history_page ? "[{$history_page}] ".get_permalink($history_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Country: ', 'give')).give_get_country());
 
 		/**
 		 * Currency Information.
 		 */
-		$default_gateway = give_get_option( 'default_gateway' );
+		$default_gateway = give_get_option('default_gateway');
 
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'Currency Information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('Currency Information', 'give')).'   ####');
 
-		WP_CLI::log( $this->color_message( __( 'Currency: ', 'give' ), give_get_currency() ) );
-		WP_CLI::log( $this->color_message( __( 'Currency Position: ', 'give' ), give_get_currency_position() ) );
-		WP_CLI::log( $this->color_message( __( 'Thousand Separator: ', 'give' ), give_get_price_thousand_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Decimal Separator: ', 'give' ), give_get_price_decimal_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Number of Decimals: ', 'give' ), give_get_price_decimals() ) );
-		WP_CLI::log( $this->color_message( __( 'Test Mode: ', 'give' ), ( give_get_option( 'test_mode' ) ? __( 'Yes', 'give' ) : __( 'No', 'give' ) ) ) );
-		WP_CLI::log( $this->color_message( __( 'Default Gateway: ', 'give' ), ( $default_gateway ? $default_gateway : __( 'Not Set', 'give' ) ) ) );
+		WP_CLI::log($this->color_message(__('Currency: ', 'give'), give_get_currency()));
+		WP_CLI::log($this->color_message(__('Currency Position: ', 'give'), give_get_currency_position()));
+		WP_CLI::log($this->color_message(__('Thousand Separator: ', 'give'), give_get_price_thousand_separator()));
+		WP_CLI::log($this->color_message(__('Decimal Separator: ', 'give'), give_get_price_decimal_separator()));
+		WP_CLI::log($this->color_message(__('Number of Decimals: ', 'give'), give_get_price_decimals()));
+		WP_CLI::log($this->color_message(__('Test Mode: ', 'give'), (give_get_option('test_mode') ? __('Yes', 'give') : __('No', 'give'))));
+		WP_CLI::log($this->color_message(__('Default Gateway: ', 'give'), ($default_gateway ? $default_gateway : __('Not Set', 'give'))));
 
 		// Payment gateways Information.
-		$gateways = give_get_ordered_payment_gateways( give_get_payment_gateways() );
-		WP_CLI::log( $this->color_message( __( 'Enabled Gateways: ', 'give' ) ) );
+		$gateways = give_get_ordered_payment_gateways(give_get_payment_gateways());
+		WP_CLI::log($this->color_message(__('Enabled Gateways: ', 'give')));
 
-		if ( ! empty( $gateways ) ) {
+		if ( ! empty($gateways)) {
 			self::$counter = 1;
-			foreach ( $gateways as $gateway ) {
-				WP_CLI::log( '  ' . $this->color_message( self::$counter, $gateway['admin_label'] ) );
-				self::$counter ++;
+			foreach ($gateways as $gateway) {
+				WP_CLI::log('  '.$this->color_message(self::$counter, $gateway['admin_label']));
+				self::$counter++;
 			}
 		} else {
-			WP_CLI::log( __( 'Not any payment gateways found', 'give' ) );
+			WP_CLI::log(__('Not any payment gateways found', 'give'));
 		}
 	}
 
@@ -173,15 +173,15 @@ 
 	 *
 	 * @subcommand    forms
 	 */
-	public function forms( $args, $assoc_args ) {
+	public function forms($args, $assoc_args) {
 		global $wp_query;
-		$form_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$number  = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? absint( $assoc_args['number'] ) : 10;
+		$form_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$number  = isset($assoc_args) && array_key_exists('number', $assoc_args) ? absint($assoc_args['number']) : 10;
 		$start   = time();
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -190,22 +190,22 @@ 
 		$wp_query->query_vars['number'] = $number;
 
 		// Get forms.
-		$forms = $form_id ? $this->api->get_forms( $form_id ) : $this->api->get_forms();
+		$forms = $form_id ? $this->api->get_forms($form_id) : $this->api->get_forms();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
 		// Bailout.
-		if ( array_key_exists( 'error', $forms ) ) {
+		if (array_key_exists('error', $forms)) {
 
-			WP_CLI::warning( $forms['error'] );
+			WP_CLI::warning($forms['error']);
 
 			return;
-		} elseif ( empty( $forms['forms'] ) ) {
+		} elseif (empty($forms['forms'])) {
 
-			WP_CLI::error( __( 'No forms found.', 'give' ) );
+			WP_CLI::error(__('No forms found.', 'give'));
 
 			return;
 		}
@@ -213,25 +213,25 @@ 
 		// Param to check if form typeis already showed or not.
 		$is_show_form_type = false;
 
-		if ( 1 === count( $forms ) && $form_id ) {
+		if (1 === count($forms) && $form_id) {
 			// Show single form.
-			foreach ( $forms['forms'][0] as $key => $info ) {
-				switch ( $key ) {
+			foreach ($forms['forms'][0] as $key => $info) {
+				switch ($key) {
 					case 'stats':
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
-						foreach ( $info as $heading => $data ) {
-							$this->color_sub_heading( ucfirst( $heading ) );
-							switch ( $heading ) {
+						foreach ($info as $heading => $data) {
+							$this->color_sub_heading(ucfirst($heading));
+							switch ($heading) {
 								default:
-									foreach ( $data as $subheading => $subdata ) {
+									foreach ($data as $subheading => $subdata) {
 
-										switch ( $subheading ) {
+										switch ($subheading) {
 											case 'earnings':
-												WP_CLI::log( $this->color_message( $subheading . ': ', give_currency_filter( $subdata ) ) );
+												WP_CLI::log($this->color_message($subheading.': ', give_currency_filter($subdata)));
 												break;
 											default:
-												WP_CLI::log( $this->color_message( $subheading . ': ', $subdata ) );
+												WP_CLI::log($this->color_message($subheading.': ', $subdata));
 										}
 									}
 							}
@@ -241,26 +241,26 @@ 
 					case 'pricing':
 					case 'info':
 					default:
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
 						// Show form type.
-						if ( ! $is_show_form_type ) {
-							$form              = new Give_Donate_Form( $form_id );
+						if ( ! $is_show_form_type) {
+							$form              = new Give_Donate_Form($form_id);
 							$is_show_form_type = true;
 
-							WP_CLI::log( $this->color_message( __( 'form type', 'give' ), $form->get_type() ) );
+							WP_CLI::log($this->color_message(__('form type', 'give'), $form->get_type()));
 						}
 
-						foreach ( $info as $heading => $data ) {
+						foreach ($info as $heading => $data) {
 
-							switch ( $heading ) {
+							switch ($heading) {
 								case 'id':
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									WP_CLI::log($this->color_message($heading, $data));
 									break;
 
 								default:
-									$data = empty( $data ) ? __( 'Not set', 'give' ) : $data;
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									$data = empty($data) ? __('Not set', 'give') : $data;
+									WP_CLI::log($this->color_message($heading, $data));
 							}
 						}
 				}
@@ -271,37 +271,37 @@ 
 			$is_table_first_row_set = false;
 			$table_column_count     = 0;
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%d donation forms found', 'give' ), count( $forms['forms'] ) ), '', false ) );
+			WP_CLI::line($this->color_message(sprintf(__('%d donation forms found', 'give'), count($forms['forms'])), '', false));
 
-			foreach ( $forms['forms'] as $index => $form_data ) {
+			foreach ($forms['forms'] as $index => $form_data) {
 
 				// Default table data.
 				$table_first_row = array();
 				$table_row       = array();
 
-				foreach ( $form_data['info'] as $key => $form ) {
+				foreach ($form_data['info'] as $key => $form) {
 
 					// Do not show thumbnail, content and link in table.
-					if ( in_array( $key, array( 'content', 'thumbnail', 'link' ), true ) ) {
+					if (in_array($key, array('content', 'thumbnail', 'link'), true)) {
 						continue;
 					}
 
-					if ( ! $is_table_first_row_set ) {
+					if ( ! $is_table_first_row_set) {
 						$table_first_row[] = $key;
 					}
 
 					$table_row[] = $form;
 
-					if ( 'status' === $key ) {
+					if ('status' === $key) {
 						// First array item will be an form id in our case.
-						$form = new Give_Donate_Form( absint( $table_row[0] ) );
+						$form = new Give_Donate_Form(absint($table_row[0]));
 
 						$table_row[] = $form->get_type();
 					}
 				}
 
 				// Set table first row.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 
 					// Add extra column to table.
 					$table_first_row[] = 'type';
@@ -314,7 +314,7 @@ 
 				$table_data[] = $table_row;
 			}
 
-			$this->display_table( $table_data );
+			$this->display_table($table_data);
 		}
 	}
 
@@ -366,37 +366,37 @@ 
 	 *
 	 * @subcommand    donors
 	 */
-	public function donors( $args, $assoc_args ) {
+	public function donors($args, $assoc_args) {
 		global $wp_query;
-		$donor_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$email    = isset( $assoc_args ) && array_key_exists( 'email', $assoc_args ) ? $assoc_args['email'] : false;
-		$name     = isset( $assoc_args ) && array_key_exists( 'name', $assoc_args ) ? $assoc_args['name'] : '';
-		$create   = isset( $assoc_args ) && array_key_exists( 'create', $assoc_args ) ? $assoc_args['create'] : false;
-		$number   = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
-		$form_id  = isset( $assoc_args ) && array_key_exists( 'form-id', $assoc_args ) ? $assoc_args['form-id'] : 0;
-		$format   = isset( $assoc_args ) && array_key_exists( 'format', $assoc_args ) ? $assoc_args['format'] : 'table';
+		$donor_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$email    = isset($assoc_args) && array_key_exists('email', $assoc_args) ? $assoc_args['email'] : false;
+		$name     = isset($assoc_args) && array_key_exists('name', $assoc_args) ? $assoc_args['name'] : '';
+		$create   = isset($assoc_args) && array_key_exists('create', $assoc_args) ? $assoc_args['create'] : false;
+		$number   = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
+		$form_id  = isset($assoc_args) && array_key_exists('form-id', $assoc_args) ? $assoc_args['form-id'] : 0;
+		$format   = isset($assoc_args) && array_key_exists('format', $assoc_args) ? $assoc_args['format'] : 'table';
 		$start    = time();
 
-		if ( $create ) {
+		if ($create) {
 			$number = 1;
 
-			if ( isset( $assoc_args['email'] ) && ! is_email( $email ) ) {
-				WP_CLI::warning( 'Wrong email address provided.', 'give' );
+			if (isset($assoc_args['email']) && ! is_email($email)) {
+				WP_CLI::warning('Wrong email address provided.', 'give');
 
 				return;
 			}
 
 			// Create one or more donors.
-			if ( ! $email ) {
+			if ( ! $email) {
 				// If no email is specified, look to see if we are generating arbitrary donor accounts.
-				$number = is_numeric( $create ) ? absint( $create ) : 1;
+				$number = is_numeric($create) ? absint($create) : 1;
 			}
 
-			for ( $i = 0; $i < $number; $i ++ ) {
-				if ( ! $email ) {
+			for ($i = 0; $i < $number; $i++) {
+				if ( ! $email) {
 
 					// Generate fake email.
-					$email = 'customer-' . uniqid() . '@test.com';
+					$email = 'customer-'.uniqid().'@test.com';
 				}
 
 				$args = array(
@@ -404,19 +404,19 @@ 
 					'name'  => $name,
 				);
 
-				$customer_id = Give()->customers->add( $args );
+				$customer_id = Give()->customers->add($args);
 
-				if ( $customer_id ) {
-					WP_CLI::line( $this->color_message( sprintf( __( 'Donor #%d created successfully', 'give' ), $customer_id ) ) );
+				if ($customer_id) {
+					WP_CLI::line($this->color_message(sprintf(__('Donor #%d created successfully', 'give'), $customer_id)));
 				} else {
-					WP_CLI::error( __( 'Failed to create donor', 'give' ) );
+					WP_CLI::error(__('Failed to create donor', 'give'));
 				}
 
 				// Reset email to false so it is generated on the next loop (if creating donors).
 				$email = false;
 			}
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%1$d donors created in %2$d seconds', 'give' ), $number, time() - $start ) ) );
+			WP_CLI::line($this->color_message(sprintf(__('%1$d donors created in %2$d seconds', 'give'), $number, time() - $start)));
 
 		} else {
 			// Counter.
@@ -430,7 +430,7 @@ 
 			 */
 			// Cache previous number query var.
 			$is_set_number = $cache_per_page = false;
-			if ( isset( $wp_query->query_vars['number'] ) ) {
+			if (isset($wp_query->query_vars['number'])) {
 				$cache_per_page = $wp_query->query_vars['number'];
 				$is_set_number  = true;
 			}
@@ -439,24 +439,24 @@ 
 			$wp_query->query_vars['number'] = $number;
 
 			// Get donors.
-			if ( $form_id ) {
+			if ($form_id) {
 				// @TODO: Allow user to get a list of donors by donation status.
-				$donors = $this->get_donors_by_form_id( $form_id );
+				$donors = $this->get_donors_by_form_id($form_id);
 			} else {
-				$donors = $this->api->get_customers( $search );
+				$donors = $this->api->get_customers($search);
 			}
 
 			// Reset number query var.
-			if ( $is_set_number ) {
+			if ($is_set_number) {
 				$wp_query->query_vars['number'] = $cache_per_page;
 			}
 
-			if ( isset( $donors['error'] ) ) {
-				WP_CLI::error( $donors['error'] );
+			if (isset($donors['error'])) {
+				WP_CLI::error($donors['error']);
 			}
 
-			if ( empty( $donors ) ) {
-				WP_CLI::error( __( 'No donors found.', 'give' ) );
+			if (empty($donors)) {
+				WP_CLI::error(__('No donors found.', 'give'));
 
 				return;
 			}
@@ -464,24 +464,24 @@ 
 			$table_data             = array();
 			$is_table_first_row_set = false;
 
-			foreach ( $donors['donors'] as $donor_data ) {
+			foreach ($donors['donors'] as $donor_data) {
 				// Set default table row data.
-				$table_first_row = array( __( 's_no', 'give' ) );
-				$table_row       = array( self::$counter );
+				$table_first_row = array(__('s_no', 'give'));
+				$table_row       = array(self::$counter);
 
-				foreach ( $donor_data as $key => $donor ) {
-					switch ( $key ) {
+				foreach ($donor_data as $key => $donor) {
+					switch ($key) {
 						case 'stats':
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
-								switch ( $heading ) {
+								switch ($heading) {
 									case 'total_spent':
-										$table_row[] = give_currency_filter( $data );
+										$table_row[] = give_currency_filter($data);
 										break;
 
 									default:
@@ -492,10 +492,10 @@ 
 
 						case 'info':
 						default:
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
@@ -505,7 +505,7 @@ 
 				}
 
 				// Add first row data to table data.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 					$table_data[]           = $table_first_row;
 					$is_table_first_row_set = true;
 				}
@@ -514,44 +514,44 @@ 
 				$table_data[] = $table_row;
 
 				// Increase counter.
-				self::$counter ++;
+				self::$counter++;
 			}
 
-			switch ( $format ) {
+			switch ($format) {
 				case 'json':
 					$table_column_name = $table_data[0];
-					unset( $table_data[0] );
+					unset($table_data[0]);
 
 					$new_table_data = array();
-					foreach ( $table_data as $index => $data ) {
-						foreach ( $data as $key => $value ) {
-							$new_table_data[ $index ][ $table_column_name[ $key ] ] = $value;
+					foreach ($table_data as $index => $data) {
+						foreach ($data as $key => $value) {
+							$new_table_data[$index][$table_column_name[$key]] = $value;
 						}
 					}
 
-					WP_CLI::log( json_encode( $new_table_data ) );
+					WP_CLI::log(json_encode($new_table_data));
 					break;
 
 				case 'csv':
-					$file_path = trailingslashit( WP_CONTENT_DIR ) . 'uploads/give_donors_' . date( 'Y_m_d_s', current_time( 'timestamp' ) ) . '.csv';
-					$fp        = fopen( $file_path, 'w' );
+					$file_path = trailingslashit(WP_CONTENT_DIR).'uploads/give_donors_'.date('Y_m_d_s', current_time('timestamp')).'.csv';
+					$fp        = fopen($file_path, 'w');
 
-					if ( is_writable( $file_path ) ) {
-						foreach ( $table_data as $fields ) {
-							fputcsv( $fp, $fields );
+					if (is_writable($file_path)) {
+						foreach ($table_data as $fields) {
+							fputcsv($fp, $fields);
 						}
 
-						fclose( $fp );
+						fclose($fp);
 
-						WP_CLI::success( "Donors list csv created successfully: {$file_path}" );
+						WP_CLI::success("Donors list csv created successfully: {$file_path}");
 					} else {
-						WP_CLI::warning( "Unable to create donors list csv file: {$file_path} (May folder do not have write permission)" );
+						WP_CLI::warning("Unable to create donors list csv file: {$file_path} (May folder do not have write permission)");
 					}
 
 					break;
 
 				default:
-					$this->display_table( $table_data );
+					$this->display_table($table_data);
 			}
 		}
 	}
@@ -581,13 +581,13 @@ 
 	 *
 	 * @subcommand    donations
 	 */
-	public function donations( $args, $assoc_args ) {
+	public function donations($args, $assoc_args) {
 		global $wp_query;
-		$number = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
+		$number = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -599,46 +599,46 @@ 
 		$donations = $this->api->get_recent_donations();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
-		if ( empty( $donations ) ) {
-			WP_CLI::error( __( 'No donations found.', 'give' ) );
+		if (empty($donations)) {
+			WP_CLI::error(__('No donations found.', 'give'));
 
 			return;
 		}
 
 		self::$counter = 1;
 
-		foreach ( $donations['donations'] as $key => $donation ) {
-			$this->color_main_heading( sprintf( __( '%1$s. Donation #%2$s', 'give' ), self::$counter, $donation['ID'] ), 'Y' );
-			self::$counter ++;
+		foreach ($donations['donations'] as $key => $donation) {
+			$this->color_main_heading(sprintf(__('%1$s. Donation #%2$s', 'give'), self::$counter, $donation['ID']), 'Y');
+			self::$counter++;
 
-			foreach ( $donation as $column => $data ) {
+			foreach ($donation as $column => $data) {
 
-				if ( is_array( $data ) ) {
-					$this->color_sub_heading( $column );
-					foreach ( $data as $subcolumn => $subdata ) {
+				if (is_array($data)) {
+					$this->color_sub_heading($column);
+					foreach ($data as $subcolumn => $subdata) {
 
 						// Decode html codes.
-						switch ( $subcolumn ) {
+						switch ($subcolumn) {
 							case 'name':
-								$subdata = html_entity_decode( $subdata );
+								$subdata = html_entity_decode($subdata);
 								break;
 						}
 
 						// @TODO Check if multi dimension array information is importent to show or not. For example inside donation array we have array for fees data inside payment meta.
-						if ( is_array( $subdata ) ) {
+						if (is_array($subdata)) {
 							continue;
 						}
 
-						WP_CLI::log( $this->color_message( $subcolumn, $subdata ) );
+						WP_CLI::log($this->color_message($subcolumn, $subdata));
 					}
 					continue;
 				}
 
-				WP_CLI::log( $this->color_message( $column, $data ) );
+				WP_CLI::log($this->color_message($column, $data));
 			}
 		}
 	}
@@ -677,27 +677,27 @@ 
 	 *
 	 * @return        void
 	 */
-	public function report( $args, $assoc_args ) {
+	public function report($args, $assoc_args) {
 		$stats      = new Give_Payment_Stats();
-		$date       = isset( $assoc_args ) && array_key_exists( 'date', $assoc_args ) ? $assoc_args['date'] : false;
-		$start_date = isset( $assoc_args ) && array_key_exists( 'start-date', $assoc_args ) ? $assoc_args['start-date'] : false;
-		$end_date   = isset( $assoc_args ) && array_key_exists( 'end-date', $assoc_args ) ? $assoc_args['end-date'] : false;
-		$form_id    = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? $assoc_args['id'] : 0;
+		$date       = isset($assoc_args) && array_key_exists('date', $assoc_args) ? $assoc_args['date'] : false;
+		$start_date = isset($assoc_args) && array_key_exists('start-date', $assoc_args) ? $assoc_args['start-date'] : false;
+		$end_date   = isset($assoc_args) && array_key_exists('end-date', $assoc_args) ? $assoc_args['end-date'] : false;
+		$form_id    = isset($assoc_args) && array_key_exists('id', $assoc_args) ? $assoc_args['id'] : 0;
 
-		if ( ! empty( $date ) ) {
+		if ( ! empty($date)) {
 			$start_date = $date;
 			$end_date   = false;
-		} elseif ( empty( $date ) && empty( $start_date ) ) {
+		} elseif (empty($date) && empty($start_date)) {
 			$start_date = 'this_month';
 			$end_date   = false;
 		}
 
 		// Get stats.
-		$earnings = $stats->get_earnings( $form_id, $start_date, $end_date );
-		$sales    = $stats->get_sales( $form_id, $start_date, $end_date );
+		$earnings = $stats->get_earnings($form_id, $start_date, $end_date);
+		$sales    = $stats->get_sales($form_id, $start_date, $end_date);
 
-		WP_CLI::line( $this->color_message( __( 'Earnings', 'give' ), give_currency_filter( $earnings ) ) );
-		WP_CLI::line( $this->color_message( __( 'Sales', 'give' ), $sales ) );
+		WP_CLI::line($this->color_message(__('Earnings', 'give'), give_currency_filter($earnings)));
+		WP_CLI::line($this->color_message(__('Sales', 'give'), $sales));
 	}
 
 
@@ -724,27 +724,27 @@ 
 	 *
 	 * @subcommand    cache
 	 */
-	public function cache( $args, $assoc_args ) {
-		$action = isset( $assoc_args ) && array_key_exists( 'action', $assoc_args ) ? $assoc_args['action'] : false;
+	public function cache($args, $assoc_args) {
+		$action = isset($assoc_args) && array_key_exists('action', $assoc_args) ? $assoc_args['action'] : false;
 
 		// Bailout.
-		if ( ! $action || ! in_array( $action, array( 'delete' ), true ) ) {
-			WP_CLI::warning( __( 'Type wp give cache --action=delete to delete all stat transients', 'give' ) );
+		if ( ! $action || ! in_array($action, array('delete'), true)) {
+			WP_CLI::warning(__('Type wp give cache --action=delete to delete all stat transients', 'give'));
 
 			return;
 		}
 
-		switch ( $action ) {
+		switch ($action) {
 			case 'delete' :
 				// Reset counter.
 				self::$counter = 1;
 
-				if ( $this->delete_stats_transients() ) {
+				if ($this->delete_stats_transients()) {
 					// Report .eading.
-					WP_CLI::success( 'Give cache deleted.' );
+					WP_CLI::success('Give cache deleted.');
 				} else {
 					// Report .eading.
-					WP_CLI::warning( 'We did not find any Give plugin cache to delete :)' );
+					WP_CLI::warning('We did not find any Give plugin cache to delete :)');
 				}
 				break;
 		}
@@ -773,27 +773,27 @@ 
 			ARRAY_A
 		);
 
-		if ( ! empty( $stat_option_names ) ) {
+		if ( ! empty($stat_option_names)) {
 
-			foreach ( $stat_option_names as $option_name ) {
+			foreach ($stat_option_names as $option_name) {
 				$error       = false;
 				$option_name = $option_name['option_name'];
 
-				switch ( true ) {
-					case ( false !== strpos( $option_name, 'transient' ) ):
-						$option_name = str_replace( '_transient_', '', $option_name );
-						$error       = delete_transient( $option_name );
+				switch (true) {
+					case (false !== strpos($option_name, 'transient')):
+						$option_name = str_replace('_transient_', '', $option_name);
+						$error       = delete_transient($option_name);
 						break;
 
 					default:
-						$error = delete_option( $option_name );
+						$error = delete_option($option_name);
 				}
 
-				if ( $error ) {
-					WP_CLI::log( $this->color_message( self::$counter, $option_name ) );
-					self::$counter ++;
+				if ($error) {
+					WP_CLI::log($this->color_message(self::$counter, $option_name));
+					self::$counter++;
 				} else {
-					WP_CLI::log( $this->color_message( __( 'Error while deleting this transient', 'give' ), $option_name ) );
+					WP_CLI::log($this->color_message(__('Error while deleting this transient', 'give'), $option_name));
 				}
 			}
 
@@ -814,13 +814,13 @@ 
 	 *
 	 * @return    mixed
 	 */
-	private function color_message( $heading, $message = '', $colon = true, $color = 'g' ) {
+	private function color_message($heading, $message = '', $colon = true, $color = 'g') {
 		// Add colon.
-		if ( $colon ) {
-			$heading = $heading . ': ';
+		if ($colon) {
+			$heading = $heading.': ';
 		}
 
-		return WP_CLI::colorize( "%{$color}" . $heading . '%n' ) . $message;
+		return WP_CLI::colorize("%{$color}".$heading.'%n').$message;
 	}
 
 
@@ -835,8 +835,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_main_heading( $heading, $color = 'g' ) {
-		WP_CLI::log( "\n######   " . $this->color_message( $heading, '', false, $color ) . '   ######' );
+	private function color_main_heading($heading, $color = 'g') {
+		WP_CLI::log("\n######   ".$this->color_message($heading, '', false, $color).'   ######');
 	}
 
 	/**
@@ -849,8 +849,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_sub_heading( $subheading ) {
-		WP_CLI::log( "\n--->" . $subheading . '', '', false );
+	private function color_sub_heading($subheading) {
+		WP_CLI::log("\n--->".$subheading.'', '', false);
 	}
 
 
@@ -864,17 +864,17 @@ 
 	 *
 	 * @return    void
 	 */
-	private function display_table( $data ) {
+	private function display_table($data) {
 		$table = new \cli\Table();
 
 		// Set table header.
-		$table->setHeaders( $data[0] );
+		$table->setHeaders($data[0]);
 
 		// Remove table header.
-		unset( $data[0] );
+		unset($data[0]);
 
 		// Set table data.
-		$table->setRows( $data );
+		$table->setRows($data);
 
 		// Display table.
 		$table->display();
@@ -891,14 +891,14 @@ 
 	 * @return array
 	 */
 
-	private function get_donors_by_form_id( $form_id ) {
+	private function get_donors_by_form_id($form_id) {
 		$donors = array();
 
 		$donations = new Give_Payments_Query(
 			array(
-				'give_forms' => array( $form_id ),
-				'number'     => - 1,
-				'status'     => array( 'publish' ),
+				'give_forms' => array($form_id),
+				'number'     => -1,
+				'status'     => array('publish'),
 			)
 		);
 
@@ -906,16 +906,16 @@ 
 		$skip_donors = array();
 
 		/* @var Give_Payment|object $donation Payment object. */
-		foreach ( $donations as $donation ) {
+		foreach ($donations as $donation) {
 
-			if ( in_array( $donation->customer_id, $skip_donors ) ) {
+			if (in_array($donation->customer_id, $skip_donors)) {
 				continue;
 			}
 
-			if ( ! empty( $donors ) ) {
-				$donors['donors'][] = current( current( $this->api->get_customers( (int) $donation->customer_id ) ) );
+			if ( ! empty($donors)) {
+				$donors['donors'][] = current(current($this->api->get_customers((int) $donation->customer_id)));
 			} else {
-				$donors = array_merge( $donors, $this->api->get_customers( (int) $donation->customer_id ) );
+				$donors = array_merge($donors, $this->api->get_customers((int) $donation->customer_id));
 			}
 
 			$skip_donors[] = $donation->customer_id;

includes/emails/template.php

Spacing +104 added lines, -104 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -41,8 +41,8 @@ 
  *
  * @return string $message Fully formatted message
  */
-function give_email_template_tags( $message, $payment_data, $payment_id, $admin_notice = false ) {
-	return give_do_email_tags( $message, $payment_id );
+function give_email_template_tags($message, $payment_data, $payment_id, $admin_notice = false) {
+	return give_do_email_tags($message, $payment_id);
 }
 
 /**
@@ -56,42 +56,42 @@ 
  *
  * @return string $message Fully formatted message
  */
-function give_email_preview_template_tags( $message ) {
+function give_email_preview_template_tags($message) {
 
-	$price = give_currency_filter( give_format_amount( 10.50 ) );
+	$price = give_currency_filter(give_format_amount(10.50));
 
 	$gateway = 'PayPal';
 
-	$receipt_id = strtolower( md5( uniqid() ) );
+	$receipt_id = strtolower(md5(uniqid()));
 
-	$payment_id = rand( 1, 100 );
+	$payment_id = rand(1, 100);
 
-	$receipt_link_url = esc_url( add_query_arg( array( 'payment_key' => $receipt_id, 'give_action' => 'view_receipt' ), home_url() ) );
+	$receipt_link_url = esc_url(add_query_arg(array('payment_key' => $receipt_id, 'give_action' => 'view_receipt'), home_url()));
 	$receipt_link = sprintf(
 		'<a href="%1$s">%2$s</a>',
 		$receipt_link_url,
-		esc_html__( 'View the receipt in your browser &raquo;', 'give' )
+		esc_html__('View the receipt in your browser &raquo;', 'give')
 	);
 
 	$user = wp_get_current_user();
 
-	$message = str_replace( '{name}', $user->display_name, $message );
-	$message = str_replace( '{fullname}', $user->display_name, $message );
-	$message = str_replace( '{username}', $user->user_login, $message );
-	$message = str_replace( '{date}', date( give_date_format(), current_time( 'timestamp' ) ), $message );
-	$message = str_replace( '{amount}', $price, $message );
-	$message = str_replace( '{price}', $price, $message );
-	$message = str_replace( '{donation}', esc_html__( 'Sample Donation Form Title', 'give' ), $message );
-	$message = str_replace( '{form_title}', esc_html__( 'Sample Donation Form Title - Sample Donation Level', 'give' ), $message );
-	$message = str_replace( '{receipt_id}', $receipt_id, $message );
-	$message = str_replace( '{payment_method}', $gateway, $message );
-	$message = str_replace( '{sitename}', get_bloginfo( 'name' ), $message );
-	$message = str_replace( '{payment_id}', $payment_id, $message );
-	$message = str_replace( '{receipt_link}', $receipt_link, $message );
-	$message = str_replace( '{receipt_link_url}', $receipt_link_url, $message );
-	$message = str_replace( '{pdf_receipt}', '<a href="#">Download Receipt</a>', $message );
-
-	return wpautop( apply_filters( 'give_email_preview_template_tags', $message ) );
+	$message = str_replace('{name}', $user->display_name, $message);
+	$message = str_replace('{fullname}', $user->display_name, $message);
+	$message = str_replace('{username}', $user->user_login, $message);
+	$message = str_replace('{date}', date(give_date_format(), current_time('timestamp')), $message);
+	$message = str_replace('{amount}', $price, $message);
+	$message = str_replace('{price}', $price, $message);
+	$message = str_replace('{donation}', esc_html__('Sample Donation Form Title', 'give'), $message);
+	$message = str_replace('{form_title}', esc_html__('Sample Donation Form Title - Sample Donation Level', 'give'), $message);
+	$message = str_replace('{receipt_id}', $receipt_id, $message);
+	$message = str_replace('{payment_method}', $gateway, $message);
+	$message = str_replace('{sitename}', get_bloginfo('name'), $message);
+	$message = str_replace('{payment_id}', $payment_id, $message);
+	$message = str_replace('{receipt_link}', $receipt_link, $message);
+	$message = str_replace('{receipt_link_url}', $receipt_link_url, $message);
+	$message = str_replace('{pdf_receipt}', '<a href="#">Download Receipt</a>', $message);
+
+	return wpautop(apply_filters('give_email_preview_template_tags', $message));
 }
 
 /**
@@ -103,23 +103,23 @@ 
  * @since  1.0
  * @return array|bool
  */
-function give_email_template_preview( $array ) {
+function give_email_template_preview($array) {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return false;
 	}
 	$custom_field = array(
-		'name' => esc_html__( 'Preview Email', 'give' ),
-		'desc' => esc_html__( 'Click the buttons to preview emails.', 'give' ),
+		'name' => esc_html__('Preview Email', 'give'),
+		'desc' => esc_html__('Click the buttons to preview emails.', 'give'),
 		'id'   => 'give_email_preview_buttons',
 		'type' => 'email_preview_buttons'
 	);
 
-	return give_settings_array_insert( $array, 'donation_subject', array( $custom_field ) );
+	return give_settings_array_insert($array, 'donation_subject', array($custom_field));
 
 }
 
-add_filter( 'give_settings_emails', 'give_email_template_preview' );
+add_filter('give_settings_emails', 'give_email_template_preview');
 
 /**
  * Output Email Template Preview Buttons.
@@ -131,12 +131,12 @@ 
 function give_email_preview_buttons_callback() {
 	ob_start();
 	?>
-	<a href="<?php echo esc_url( add_query_arg( array( 'give_action' => 'preview_email' ), home_url() ) ); ?>" class="button-secondary" target="_blank"><?php esc_html_e( 'Preview Donation Receipt', 'give' ); ?></a>
-	<a href="<?php echo wp_nonce_url( add_query_arg( array(
+	<a href="<?php echo esc_url(add_query_arg(array('give_action' => 'preview_email'), home_url())); ?>" class="button-secondary" target="_blank"><?php esc_html_e('Preview Donation Receipt', 'give'); ?></a>
+	<a href="<?php echo wp_nonce_url(add_query_arg(array(
 		'give_action'  => 'send_test_email',
 		'give-message' => 'sent-test-email',
 		'tag'          => 'emails'
-	) ), 'give-test-email' ); ?>" aria-label="<?php esc_attr_e( 'Send demo donation receipt to the emails listed below.', 'give' ); ?>" class="button-secondary"><?php esc_html_e( 'Send Test Email', 'give' ); ?></a>
+	)), 'give-test-email'); ?>" aria-label="<?php esc_attr_e('Send demo donation receipt to the emails listed below.', 'give'); ?>" class="button-secondary"><?php esc_html_e('Send Test Email', 'give'); ?></a>
 	<?php
 	echo ob_get_clean();
 }
@@ -149,46 +149,46 @@ 
  */
 function give_display_email_template_preview() {
 
-	if ( empty( $_GET['give_action'] ) ) {
+	if (empty($_GET['give_action'])) {
 		return;
 	}
 
-	if ( 'preview_email' !== $_GET['give_action'] ) {
+	if ('preview_email' !== $_GET['give_action']) {
 		return;
 	}
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 
 
-	Give()->emails->heading = esc_html__( 'Donation Receipt', 'give' );
+	Give()->emails->heading = esc_html__('Donation Receipt', 'give');
 
-	$payment_id = (int) isset( $_GET['preview_id'] ) ? $_GET['preview_id'] : '';
+	$payment_id = (int) isset($_GET['preview_id']) ? $_GET['preview_id'] : '';
 
 	echo give_get_preview_email_header();
 
 	//Are we previewing an actual payment?
-	if ( ! empty( $payment_id ) ) {
+	if ( ! empty($payment_id)) {
 
-		$content = give_get_email_body_content( $payment_id );
+		$content = give_get_email_body_content($payment_id);
 
-		$preview_content = give_do_email_tags( $content, $payment_id );
+		$preview_content = give_do_email_tags($content, $payment_id);
 
 	} else {
 
 		//No payment ID, use sample preview content
-		$preview_content = give_email_preview_template_tags( give_get_email_body_content( 0, array() ) );
+		$preview_content = give_email_preview_template_tags(give_get_email_body_content(0, array()));
 	}
 
 
-	echo Give()->emails->build_email( $preview_content );
+	echo Give()->emails->build_email($preview_content);
 
 	exit;
 
 }
 
-add_action( 'init', 'give_display_email_template_preview' );
+add_action('init', 'give_display_email_template_preview');
 
 /**
  * Email Template Body.
@@ -200,18 +200,18 @@ 
  *
  * @return string $email_body Body of the email
  */
-function give_get_email_body_content( $payment_id = 0, $payment_data = array() ) {
+function give_get_email_body_content($payment_id = 0, $payment_data = array()) {
 
 	$default_email_body = give_get_default_donation_receipt_email();
 
-	$email_content = give_get_option( 'donation_receipt' );
-	$email_content = isset( $email_content ) ? stripslashes( $email_content ) : $default_email_body;
+	$email_content = give_get_option('donation_receipt');
+	$email_content = isset($email_content) ? stripslashes($email_content) : $default_email_body;
 
-	$email_body = wpautop( $email_content );
+	$email_body = wpautop($email_content);
 
-	$email_body = apply_filters( 'give_donation_receipt_' . Give()->emails->get_template(), $email_body, $payment_id, $payment_data );
+	$email_body = apply_filters('give_donation_receipt_'.Give()->emails->get_template(), $email_body, $payment_id, $payment_data);
 
-	return apply_filters( 'give_donation_receipt', $email_body, $payment_id, $payment_data );
+	return apply_filters('give_donation_receipt', $email_body, $payment_id, $payment_data);
 }
 
 /**
@@ -224,36 +224,36 @@ 
  *
  * @return string $email_body Body of the email
  */
-function give_get_donation_notification_body_content( $payment_id = 0, $payment_data = array() ) {
+function give_get_donation_notification_body_content($payment_id = 0, $payment_data = array()) {
 
-	$user_info = maybe_unserialize( $payment_data['user_info'] );
-	$email     = give_get_payment_user_email( $payment_id );
+	$user_info = maybe_unserialize($payment_data['user_info']);
+	$email     = give_get_payment_user_email($payment_id);
 
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 ) {
-		$user_data = get_userdata( $user_info['id'] );
+	if (isset($user_info['id']) && $user_info['id'] > 0) {
+		$user_data = get_userdata($user_info['id']);
 		$name      = $user_data->display_name;
-	} elseif ( isset( $user_info['first_name'] ) && isset( $user_info['last_name'] ) ) {
-		$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
+	} elseif (isset($user_info['first_name']) && isset($user_info['last_name'])) {
+		$name = $user_info['first_name'].' '.$user_info['last_name'];
 	} else {
 		$name = $email;
 	}
 
-	$gateway = give_get_gateway_admin_label( get_post_meta( $payment_id, '_give_payment_gateway', true ) );
+	$gateway = give_get_gateway_admin_label(get_post_meta($payment_id, '_give_payment_gateway', true));
 
-	$default_email_body = esc_html__( 'Hello', 'give' ) . "\n\n";
-	$default_email_body .= esc_html__( 'A donation has been made.', 'give' ) . "\n\n";
-	$default_email_body .= esc_html__( 'Donation:', 'give' ) . "\n\n";
-	$default_email_body .= esc_html__( 'Donor:', 'give' ) . ' ' . html_entity_decode( $name, ENT_COMPAT, 'UTF-8' ) . "\n";
-	$default_email_body .= esc_html__( 'Amount:', 'give' ) . ' ' . html_entity_decode( give_currency_filter( give_format_amount( give_get_payment_amount( $payment_id ) ) ), ENT_COMPAT, 'UTF-8' ) . "\n";
-	$default_email_body .= esc_html__( 'Payment Method:', 'give' ) . ' ' . $gateway . "\n\n";
-	$default_email_body .= esc_html__( 'Thank you', 'give' );
+	$default_email_body = esc_html__('Hello', 'give')."\n\n";
+	$default_email_body .= esc_html__('A donation has been made.', 'give')."\n\n";
+	$default_email_body .= esc_html__('Donation:', 'give')."\n\n";
+	$default_email_body .= esc_html__('Donor:', 'give').' '.html_entity_decode($name, ENT_COMPAT, 'UTF-8')."\n";
+	$default_email_body .= esc_html__('Amount:', 'give').' '.html_entity_decode(give_currency_filter(give_format_amount(give_get_payment_amount($payment_id))), ENT_COMPAT, 'UTF-8')."\n";
+	$default_email_body .= esc_html__('Payment Method:', 'give').' '.$gateway."\n\n";
+	$default_email_body .= esc_html__('Thank you', 'give');
 
-	$email = give_get_option( 'donation_notification' );
-	$email = isset( $email ) ? stripslashes( $email ) : $default_email_body;
+	$email = give_get_option('donation_notification');
+	$email = isset($email) ? stripslashes($email) : $default_email_body;
 
-	$email_body = give_do_email_tags( $email, $payment_id );
+	$email_body = give_do_email_tags($email, $payment_id);
 
-	return apply_filters( 'give_donation_notification', wpautop( $email_body ), $payment_id, $payment_data );
+	return apply_filters('give_donation_notification', wpautop($email_body), $payment_id, $payment_data);
 }
 
 /**
@@ -266,19 +266,19 @@ 
  * @since  1.0
  */
 function give_render_receipt_in_browser() {
-	if ( ! isset( $_GET['payment_key'] ) ) {
-		wp_die( esc_html__( 'Missing donation payment key.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if ( ! isset($_GET['payment_key'])) {
+		wp_die(esc_html__('Missing donation payment key.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 	}
 
-	$key = urlencode( $_GET['payment_key'] );
+	$key = urlencode($_GET['payment_key']);
 
 	ob_start();
 	//Disallows caching of the page
-	header( "Last-Modified: " . gmdate( "D, d M Y H:i:s" ) . " GMT" );
-	header( "Cache-Control: no-store, no-cache, must-revalidate" ); // HTTP/1.1
-	header( "Cache-Control: post-check=0, pre-check=0", false );
-	header( "Pragma: no-cache" ); // HTTP/1.0
-	header( "Expires: Sat, 23 Oct 1977 05:00:00 PST" ); // Date in the past
+	header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
+	header("Cache-Control: no-store, no-cache, must-revalidate"); // HTTP/1.1
+	header("Cache-Control: post-check=0, pre-check=0", false);
+	header("Pragma: no-cache"); // HTTP/1.0
+	header("Expires: Sat, 23 Oct 1977 05:00:00 PST"); // Date in the past
 	?>
 	<!DOCTYPE html>
 	<html lang="en">
@@ -289,10 +289,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_receipt_head' );
+		do_action('give_receipt_head');
 		?>
 	</head>
-	<body class="<?php echo apply_filters( 'give_receipt_page_body_class', 'give_receipt_page' ); ?>">
+	<body class="<?php echo apply_filters('give_receipt_page_body_class', 'give_receipt_page'); ?>">
 
 	<div id="give_receipt_wrapper">
 		<?php
@@ -301,16 +301,16 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_render_receipt_in_browser_before' );
+		do_action('give_render_receipt_in_browser_before');
 
-		echo do_shortcode( '[give_receipt payment_key=' . $key . ']' );
+		echo do_shortcode('[give_receipt payment_key='.$key.']');
 
 		/**
 		 * Fires in the receipt template after the content.
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_render_receipt_in_browser_after' );
+		do_action('give_render_receipt_in_browser_after');
 		?>
 	</div>
 
@@ -320,7 +320,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_receipt_footer' );
+	do_action('give_receipt_footer');
 	?>
 	</body>
 	</html>
@@ -329,7 +329,7 @@ 
 	die();
 }
 
-add_action( 'give_view_receipt', 'give_render_receipt_in_browser' );
+add_action('give_view_receipt', 'give_render_receipt_in_browser');
 
 
 /**
@@ -344,29 +344,29 @@ 
 
 	//Payment receipt switcher
 	$payment_count = give_count_payments()->publish;
-	$payment_id    = (int) isset( $_GET['preview_id'] ) ? $_GET['preview_id'] : '';
+	$payment_id    = (int) isset($_GET['preview_id']) ? $_GET['preview_id'] : '';
 
-	if ( $payment_count <= 0 ) {
+	if ($payment_count <= 0) {
 		return false;
 	}
 
 	//Get payments.
-	$payments = new Give_Payments_Query( array(
+	$payments = new Give_Payments_Query(array(
 		'number' => 100
-	) );
+	));
 	$payments = $payments->get_payments();
 	$options  = array();
 
 	//Provide nice human readable options.
-	if ( $payments ) {
-		$options[0] = esc_html__( '- Select a donation -', 'give' );
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		$options[0] = esc_html__('- Select a donation -', 'give');
+		foreach ($payments as $payment) {
 
-			$options[ $payment->ID ] = esc_html( '#' . $payment->ID . ' - ' . $payment->email . ' - ' . $payment->form_title );
+			$options[$payment->ID] = esc_html('#'.$payment->ID.' - '.$payment->email.' - '.$payment->form_title);
 
 		}
 	} else {
-		$options[0] = esc_html__( 'No donations found.', 'give' );
+		$options[0] = esc_html__('No donations found.', 'give');
 	}
 
 	//Start constructing HTML output.
@@ -379,16 +379,16 @@ 
 			        var selected_trans = transactions.options[transactions.selectedIndex];
 				        console.log(selected_trans);
 				        if (selected_trans){
-				            var url_string = "' . get_bloginfo( 'url' ) . '?give_action=preview_email&preview_id=" + selected_trans.value;
+				            var url_string = "' . get_bloginfo('url').'?give_action=preview_email&preview_id=" + selected_trans.value;
 				                window.location = url_string;
 				        }
 				    }
 			    </script>';
 
-	$transaction_header .= '<label for="give_preview_email_payment_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">' . esc_html__( 'Preview email with a donation:', 'give' ) . '</label>';
+	$transaction_header .= '<label for="give_preview_email_payment_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">'.esc_html__('Preview email with a donation:', 'give').'</label>';
 
 	//The select field with 100 latest transactions
-	$transaction_header .= Give()->html->select( array(
+	$transaction_header .= Give()->html->select(array(
 		'name'             => 'preview_email_payment_id',
 		'selected'         => $payment_id,
 		'id'               => 'give_preview_email_payment_id',
@@ -398,12 +398,12 @@ 
 		'select_atts'      => 'onchange="change_preview()">',
 		'show_option_all'  => false,
 		'show_option_none' => false
-	) );
+	));
 
 	//Closing tag
 	$transaction_header .= '</div>';
 
-	return apply_filters( 'give_preview_email_receipt_header', $transaction_header );
+	return apply_filters('give_preview_email_receipt_header', $transaction_header);
 
 }
 
@@ -417,7 +417,7 @@ 
 function give_receipt_head_content() {
 
 	//Title.
-	$output = '<title>' . esc_html__( 'Donation Receipt', 'give' ) . '</title>';
+	$output = '<title>'.esc_html__('Donation Receipt', 'give').'</title>';
 
 	//Meta.
 	$output .= '<meta charset="utf-8"/>
@@ -431,10 +431,10 @@ 
 		<meta name="robots" content="noindex, nofollow"/>';
 
 	//CSS
-	$output .= '<link rel="stylesheet" href="' . give_get_stylesheet_uri() . '?ver=' . GIVE_VERSION . '">';
+	$output .= '<link rel="stylesheet" href="'.give_get_stylesheet_uri().'?ver='.GIVE_VERSION.'">';
 
-	echo apply_filters( 'give_receipt_head_content', $output );
+	echo apply_filters('give_receipt_head_content', $output);
 
 }
 
-add_action( 'give_receipt_head', 'give_receipt_head_content' );
\ No newline at end of file
+add_action('give_receipt_head', 'give_receipt_head_content');
\ No newline at end of file

includes/class-give-email-access.php

Spacing +62 added lines, -62 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -96,7 +96,7 @@ 
 	public function __construct() {
 
 		// get it started
-		add_action( 'init', array( $this, 'init' ) );
+		add_action('init', array($this, 'init'));
 	}
 
 	/**
@@ -111,30 +111,30 @@ 
 	 */
 	public function init() {
 
-		$is_enabled = give_get_option( 'email_access' );
+		$is_enabled = give_get_option('email_access');
 
 		//Non-logged in users only
-		if ( is_user_logged_in() || $is_enabled !== 'on' || is_admin() ) {
+		if (is_user_logged_in() || $is_enabled !== 'on' || is_admin()) {
 			return;
 		}
 
 		//Are db columns setup?
-		$is_setup = give_get_option( 'email_access_installed' );
-		if ( empty( $is_setup ) ) {
+		$is_setup = give_get_option('email_access_installed');
+		if (empty($is_setup)) {
 			$this->create_columns();
 		}
 
 		// Timeouts
-		$this->verify_throttle  = apply_filters( 'give_nl_verify_throttle', 300 );
-		$this->token_expiration = apply_filters( 'give_nl_token_expiration', 7200 );
+		$this->verify_throttle  = apply_filters('give_nl_verify_throttle', 300);
+		$this->token_expiration = apply_filters('give_nl_token_expiration', 7200);
 
 		// Setup login
 		$this->check_for_token();
 
-		if ( $this->token_exists ) {
-			add_filter( 'give_can_view_receipt', '__return_true' );
-			add_filter( 'give_user_pending_verification', '__return_false' );
-			add_filter( 'give_get_users_donations_args', array( $this, 'users_donations_args' ) );
+		if ($this->token_exists) {
+			add_filter('give_can_view_receipt', '__return_true');
+			add_filter('give_user_pending_verification', '__return_false');
+			add_filter('give_get_users_donations_args', array($this, 'users_donations_args'));
 		}
 	}
 
@@ -148,25 +148,25 @@ 
 	 *
 	 * @return bool
 	 */
-	public function can_send_email( $customer_id ) {
+	public function can_send_email($customer_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Prevent multiple emails within X minutes
-		$throttle = date( 'Y-m-d H:i:s', time() - $this->verify_throttle );
+		$throttle = date('Y-m-d H:i:s', time() - $this->verify_throttle);
 
 		// Does a user row exist?
 		$exists = (int) $wpdb->get_var(
-			$wpdb->prepare( "SELECT COUNT(*) FROM {$wpdb->prefix}give_customers WHERE id = %d", $customer_id )
+			$wpdb->prepare("SELECT COUNT(*) FROM {$wpdb->prefix}give_customers WHERE id = %d", $customer_id)
 		);
 
-		if ( 0 < $exists ) {
+		if (0 < $exists) {
 			$row_id = (int) $wpdb->get_var(
-				$wpdb->prepare( "SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d AND (verify_throttle < %s OR verify_key = '') LIMIT 1", $customer_id, $throttle )
+				$wpdb->prepare("SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d AND (verify_throttle < %s OR verify_key = '') LIMIT 1", $customer_id, $throttle)
 			);
 
-			if ( $row_id < 1 ) {
-				give_set_error( 'give_email_access_attempts_exhausted', esc_html__( 'Please wait a few minutes before requesting a new email access link.', 'give' ) );
+			if ($row_id < 1) {
+				give_set_error('give_email_access_attempts_exhausted', esc_html__('Please wait a few minutes before requesting a new email access link.', 'give'));
 
 				return false;
 			}
@@ -186,34 +186,34 @@ 
 	 *
 	 * @return void
 	 */
-	public function send_email( $customer_id, $email ) {
+	public function send_email($customer_id, $email) {
 
-		$verify_key = wp_generate_password( 20, false );
+		$verify_key = wp_generate_password(20, false);
 
 		// Generate a new verify key
-		$this->set_verify_key( $customer_id, $email, $verify_key );
+		$this->set_verify_key($customer_id, $email, $verify_key);
 
 		// Get the donation history page
-		$page_id = give_get_option( 'history_page' );
+		$page_id = give_get_option('history_page');
 
-		$access_url = add_query_arg( array(
+		$access_url = add_query_arg(array(
 			'give_nl' => $verify_key,
-		), get_permalink( $page_id ) );
+		), get_permalink($page_id));
 
 		//Nice subject and message
-		$subject = apply_filters( 'give_email_access_token_subject', sprintf( esc_html__( 'Your Access Link to %s', 'give' ), get_bloginfo( 'name' ) ) );
+		$subject = apply_filters('give_email_access_token_subject', sprintf(esc_html__('Your Access Link to %s', 'give'), get_bloginfo('name')));
 
-		$message = esc_html__( 'You or someone in your organization requested an access link be sent to this email address. This is a temporary access link for you to view your donation information. Click on the link below to view:', 'give' ) . "\n\n";
-		$message .= '<a href="' . esc_url( $access_url ) . '" target="_blank">' . esc_html__( 'Access My Donation Details &raquo;', 'give' ) . '</a>' . "\n\n";
+		$message = esc_html__('You or someone in your organization requested an access link be sent to this email address. This is a temporary access link for you to view your donation information. Click on the link below to view:', 'give')."\n\n";
+		$message .= '<a href="'.esc_url($access_url).'" target="_blank">'.esc_html__('Access My Donation Details &raquo;', 'give').'</a>'."\n\n";
 		$message .= "\n\n";
-		$message .= esc_html__( 'Sincerely,', 'give' ) . "\n";
-		$message .= get_bloginfo( 'name' ) . "\n";
+		$message .= esc_html__('Sincerely,', 'give')."\n";
+		$message .= get_bloginfo('name')."\n";
 
-		$message = apply_filters( 'give_email_access_token_message', $message );
+		$message = apply_filters('give_email_access_token_message', $message);
 
 		// Send the email
-		Give()->emails->__set( 'heading', apply_filters( 'give_email_access_token_heading', esc_html__( 'Your Access Link', 'give' ) ) );
-		Give()->emails->send( $email, $subject, $message );
+		Give()->emails->__set('heading', apply_filters('give_email_access_token_heading', esc_html__('Your Access Link', 'give')));
+		Give()->emails->send($email, $subject, $message);
 
 	}
 
@@ -227,24 +227,24 @@ 
 	 */
 	public function check_for_token() {
 
-		$token = isset( $_GET['give_nl'] ) ? $_GET['give_nl'] : '';
+		$token = isset($_GET['give_nl']) ? $_GET['give_nl'] : '';
 
 		// Check for cookie
-		if ( empty( $token ) ) {
-			$token = isset( $_COOKIE['give_nl'] ) ? $_COOKIE['give_nl'] : '';
+		if (empty($token)) {
+			$token = isset($_COOKIE['give_nl']) ? $_COOKIE['give_nl'] : '';
 		}
 
-		if ( ! empty( $token ) ) {
-			if ( ! $this->is_valid_token( $token ) ) {
-				if ( ! $this->is_valid_verify_key( $token ) ) {
+		if ( ! empty($token)) {
+			if ( ! $this->is_valid_token($token)) {
+				if ( ! $this->is_valid_verify_key($token)) {
 					return;
 				}
 			}
 
 			$this->token_exists = true;
 			// Set cookie
-			$lifetime = current_time( 'timestamp' ) + Give()->session->set_expiration_time();
-			@setcookie( 'give_nl', $token, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
+			$lifetime = current_time('timestamp') + Give()->session->set_expiration_time();
+			@setcookie('give_nl', $token, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -258,18 +258,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_valid_token( $token ) {
+	public function is_valid_token($token) {
 
 		global $wpdb;
 
 		// Make sure token isn't expired
-		$expires = date( 'Y-m-d H:i:s', time() - $this->token_expiration );
+		$expires = date('Y-m-d H:i:s', time() - $this->token_expiration);
 
 		$email = $wpdb->get_var(
-			$wpdb->prepare( "SELECT email FROM {$wpdb->prefix}give_customers WHERE token = %s AND verify_throttle >= %s LIMIT 1", $token, $expires )
+			$wpdb->prepare("SELECT email FROM {$wpdb->prefix}give_customers WHERE token = %s AND verify_throttle >= %s LIMIT 1", $token, $expires)
 		);
 
-		if ( ! empty( $email ) ) {
+		if ( ! empty($email)) {
 			$this->token_email = $email;
 			$this->token       = $token;
 
@@ -277,8 +277,8 @@ 
 		}
 
 		//Set error only if email access form isn't being submitted
-		if ( ! isset( $_POST['give_email'] ) && ! isset( $_POST['_wpnonce'] ) ) {
-			give_set_error( 'give_email_token_expired', apply_filters( 'give_email_token_expired_message', esc_html__( 'Your access token has expired. Please request a new one below:', 'give' ) ) );
+		if ( ! isset($_POST['give_email']) && ! isset($_POST['_wpnonce'])) {
+			give_set_error('give_email_token_expired', apply_filters('give_email_token_expired_message', esc_html__('Your access token has expired. Please request a new one below:', 'give')));
 		}
 
 		return false;
@@ -297,25 +297,25 @@ 
 	 *
 	 * @return void
 	 */
-	public function set_verify_key( $customer_id, $email, $verify_key ) {
+	public function set_verify_key($customer_id, $email, $verify_key) {
 		global $wpdb;
 
-		$now = date( 'Y-m-d H:i:s' );
+		$now = date('Y-m-d H:i:s');
 
 		// Insert or update?
 		$row_id = (int) $wpdb->get_var(
-			$wpdb->prepare( "SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d LIMIT 1", $customer_id )
+			$wpdb->prepare("SELECT id FROM {$wpdb->prefix}give_customers WHERE id = %d LIMIT 1", $customer_id)
 		);
 
 		// Update
-		if ( ! empty( $row_id ) ) {
+		if ( ! empty($row_id)) {
 			$wpdb->query(
-				$wpdb->prepare( "UPDATE {$wpdb->prefix}give_customers SET verify_key = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $verify_key, $now, $row_id )
+				$wpdb->prepare("UPDATE {$wpdb->prefix}give_customers SET verify_key = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $verify_key, $now, $row_id)
 			);
 		} // Insert
 		else {
 			$wpdb->query(
-				$wpdb->prepare( "INSERT INTO {$wpdb->prefix}give_customers ( verify_key, verify_throttle) VALUES (%s, %s)", $verify_key, $now )
+				$wpdb->prepare("INSERT INTO {$wpdb->prefix}give_customers ( verify_key, verify_throttle) VALUES (%s, %s)", $verify_key, $now)
 			);
 		}
 	}
@@ -330,21 +330,21 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_valid_verify_key( $token ) {
+	public function is_valid_verify_key($token) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// See if the verify_key exists
 		$row = $wpdb->get_row(
-			$wpdb->prepare( "SELECT id, email FROM {$wpdb->prefix}give_customers WHERE verify_key = %s LIMIT 1", $token )
+			$wpdb->prepare("SELECT id, email FROM {$wpdb->prefix}give_customers WHERE verify_key = %s LIMIT 1", $token)
 		);
 
-		$now = date( 'Y-m-d H:i:s' );
+		$now = date('Y-m-d H:i:s');
 
 		// Set token
-		if ( ! empty( $row ) ) {
+		if ( ! empty($row)) {
 			$wpdb->query(
-				$wpdb->prepare( "UPDATE {$wpdb->prefix}give_customers SET verify_key = '', token = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $token, $now, $row->id )
+				$wpdb->prepare("UPDATE {$wpdb->prefix}give_customers SET verify_key = '', token = %s, verify_throttle = %s WHERE id = %d LIMIT 1", $token, $now, $row->id)
 			);
 
 			$this->token_email = $row->email;
@@ -368,7 +368,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function users_donations_args( $args ) {
+	public function users_donations_args($args) {
 		$args['user'] = $this->token_email;
 
 		return $args;
@@ -389,11 +389,11 @@ 
 		global $wpdb;
 
 		//Create columns in customers table
-		$query = $wpdb->query( "ALTER TABLE {$wpdb->prefix}give_customers ADD `token` VARCHAR(255) CHARACTER SET utf8 NOT NULL, ADD `verify_key` VARCHAR(255) CHARACTER SET utf8 NOT NULL AFTER `token`, ADD `verify_throttle` DATETIME NOT NULL AFTER `verify_key`" );
+		$query = $wpdb->query("ALTER TABLE {$wpdb->prefix}give_customers ADD `token` VARCHAR(255) CHARACTER SET utf8 NOT NULL, ADD `verify_key` VARCHAR(255) CHARACTER SET utf8 NOT NULL AFTER `token`, ADD `verify_throttle` DATETIME NOT NULL AFTER `verify_key`");
 
 		//Columns added properly
-		if ( $query ) {
-			give_update_option( 'email_access_installed', 1 );
+		if ($query) {
+			give_update_option('email_access_installed', 1);
 		}
 
 	}