WordImpress / Give

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

Spacing +5 added lines, -5 removed lines

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

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

Spacing +77 added lines, -77 removed lines

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

includes/filters.php

Spacing +43 added lines, -45 removed lines

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

templates/history-donations.php

Braces +8 added lines, -3 removed lines

@@ -186,7 +186,8 @@ 
 								__( 'View', 'give' ) . ' ' . give_get_payment_status( $post, true ) . ' »'
 							);
 
-						else :
+						else {
+							:
 							echo sprintf(
 								'<a href="%1$s">%2$s</a>',
 								esc_url(
@@ -198,6 +199,7 @@ 
 								),
 								__( 'View Receipt &raquo;', 'give' )
 							);
+						}
 
 						endif;
 						?>
@@ -242,6 +244,9 @@ 
 		</div>
 	</div>
 	<?php wp_reset_postdata(); ?>
-<?php else : ?>
-	<?php Give()->notices->print_frontend_notice( __( 'It looks like you haven\'t made any donations.', 'give' ), true, 'success' ); ?>
+<?php else {
+	: ?>
+	<?php Give()->notices->print_frontend_notice( __( 'It looks like you haven\'t made any donations.', 'give' ), true, 'success' );
+}
+?>
 <?php endif;
\ No newline at end of file

Spacing +62 added lines, -62 removed lines

@@ -4,56 +4,56 @@ 
  */
 
 $donations             = array();
-$donation_history_args = Give()->session->get( 'give_donation_history_args' );
+$donation_history_args = Give()->session->get('give_donation_history_args');
 
 // User's Donations.
-if ( is_user_logged_in() ) {
-	$donations = give_get_users_donations( get_current_user_id(), 20, true, 'any' );
-} elseif ( Give()->email_access->token_exists ) {
+if (is_user_logged_in()) {
+	$donations = give_get_users_donations(get_current_user_id(), 20, true, 'any');
+} elseif (Give()->email_access->token_exists) {
 	// Email Access Token?
-	$donations = give_get_users_donations( 0, 20, true, 'any' );
+	$donations = give_get_users_donations(0, 20, true, 'any');
 } elseif (
 	false !== Give()->session->get_session_expiration() ||
 	true === give_get_history_session()
 ) {
 	// Session active?
-	$email           = Give()->session->get( 'give_email' );
-	$donor           = Give()->donors->get_donor_by( 'email', $email );
-	$donations_count = count( explode( ',', $donor->payment_ids ) );
+	$email           = Give()->session->get('give_email');
+	$donor           = Give()->donors->get_donor_by('email', $email);
+	$donations_count = count(explode(',', $donor->payment_ids));
 
-	if( $donations_count > give_get_limit_display_donations() ) {
+	if ($donations_count > give_get_limit_display_donations()) {
 
 		// Restrict Security Email Access option, if donation count of a donor is less than or equal to limit.
-		if( true !== Give_Cache::get( "give_cache_email_throttle_limit_exhausted_{$donor->id}" ) ) {
-			add_action( 'give_donation_history_table_end', 'give_donation_history_table_end' );
+		if (true !== Give_Cache::get("give_cache_email_throttle_limit_exhausted_{$donor->id}")) {
+			add_action('give_donation_history_table_end', 'give_donation_history_table_end');
 		} else {
 			$value = Give()->email_access->verify_throttle / 60;
 
-			give_set_error( 'give-limited-throttle', sprintf(
-				__( 'Too many access email requests detected. Please wait %s before requesting a new donation history access link.', 'give' ),
-				sprintf( _n( '%s minute', '%s minutes', $value, 'give' ), $value )
-			) );
+			give_set_error('give-limited-throttle', sprintf(
+				__('Too many access email requests detected. Please wait %s before requesting a new donation history access link.', 'give'),
+				sprintf(_n('%s minute', '%s minutes', $value, 'give'), $value)
+			));
 
 		}
 
-		$donations = give_get_users_donations( $email, give_get_limit_display_donations(), true, 'any' );
+		$donations = give_get_users_donations($email, give_get_limit_display_donations(), true, 'any');
 	} else {
-		$donations = give_get_users_donations( $email, 20, true, 'any' );
+		$donations = give_get_users_donations($email, 20, true, 'any');
 	}
 }
 
-Give()->notices->render_frontend_notices( 0 );
+Give()->notices->render_frontend_notices(0);
 
-if ( $donations ) : ?>
+if ($donations) : ?>
 	<?php
 	$table_headings = array(
-		'id'             => __( 'ID', 'give' ),
-		'date'           => __( 'Date', 'give' ),
-		'donor'          => __( 'Donor', 'give' ),
-		'amount'         => __( 'Amount', 'give' ),
-		'status'         => __( 'Status', 'give' ),
-		'payment_method' => __( 'Payment Method', 'give' ),
-		'details'        => __( 'Details', 'give' ),
+		'id'             => __('ID', 'give'),
+		'date'           => __('Date', 'give'),
+		'donor'          => __('Donor', 'give'),
+		'amount'         => __('Amount', 'give'),
+		'status'         => __('Status', 'give'),
+		'payment_method' => __('Payment Method', 'give'),
+		'details'        => __('Details', 'give'),
 	);
 	?>
 	<div class="give_user_history_main" >
@@ -69,14 +69,14 @@ 
 				 *
 				 * @since 1.7
 				 */
-				do_action( 'give_donation_history_header_before' );
+				do_action('give_donation_history_header_before');
 
-				foreach ( $donation_history_args as $index => $value ) {
-					if ( filter_var( $donation_history_args[ $index ], FILTER_VALIDATE_BOOLEAN ) ) :
+				foreach ($donation_history_args as $index => $value) {
+					if (filter_var($donation_history_args[$index], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<th scope="col" class="give-donation-%1$s>">%2$s</th>',
 							$index,
-							$table_headings[ $index ]
+							$table_headings[$index]
 						);
 					endif;
 				}
@@ -88,13 +88,13 @@ 
 				 *
 				 * @since 1.7
 				 */
-				do_action( 'give_donation_history_header_after' );
+				do_action('give_donation_history_header_after');
 				?>
 			</tr>
 			</thead>
-			<?php foreach ( $donations as $post ) :
-				setup_postdata( $post );
-				$donation_data = give_get_payment_meta( $post->ID ); ?>
+			<?php foreach ($donations as $post) :
+				setup_postdata($post);
+				$donation_data = give_get_payment_meta($post->ID); ?>
 				<tr class="give-donation-row">
 					<?php
 					/**
@@ -107,36 +107,36 @@ 
 					 * @param int   $post_id       The ID of the post.
 					 * @param mixed $donation_data Payment meta data.
 					 */
-					do_action( 'give_donation_history_row_start', $post->ID, $donation_data );
+					do_action('give_donation_history_row_start', $post->ID, $donation_data);
 
-					if ( filter_var( $donation_history_args['id'], FILTER_VALIDATE_BOOLEAN ) ) :
+					if (filter_var($donation_history_args['id'], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<td class="give-donation-id">#%1$s</td>',
-							give_get_payment_number( $post->ID )
+							give_get_payment_number($post->ID)
 						);
 					endif;
 
-					if ( filter_var( $donation_history_args['date'], FILTER_VALIDATE_BOOLEAN ) ) :
+					if (filter_var($donation_history_args['date'], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<td class="give-donation-date">%1$s</td>',
-							date_i18n( give_date_format(), strtotime( get_post_field( 'post_date', $post->ID ) ) )
+							date_i18n(give_date_format(), strtotime(get_post_field('post_date', $post->ID)))
 						);
 					endif;
 
-					if ( filter_var( $donation_history_args['donor'], FILTER_VALIDATE_BOOLEAN ) ) :
+					if (filter_var($donation_history_args['donor'], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<td class="give-donation-donor">%1$s</td>',
-							give_get_donor_name_by( $post->ID )
+							give_get_donor_name_by($post->ID)
 						);
 					endif;
 					?>
 
-					<?php if ( filter_var( $donation_history_args['amount'], FILTER_VALIDATE_BOOLEAN ) ) : ?>
+					<?php if (filter_var($donation_history_args['amount'], FILTER_VALIDATE_BOOLEAN)) : ?>
 						<td class="give-donation-amount">
 						<span class="give-donation-amount">
 							<?php
-							$currency_code   = give_get_payment_currency_code( $post->ID );
-							$donation_amount = give_donation_amount( $post->ID, true );
+							$currency_code   = give_get_payment_currency_code($post->ID);
+							$donation_amount = give_donation_amount($post->ID, true);
 
 							/**
 							 * Filters the donation amount on Donation History Page.
@@ -148,42 +148,42 @@ 
 							 *
 							 * @return int
 							 */
-							echo apply_filters( 'give_donation_history_row_amount', $donation_amount, $post->ID );
+							echo apply_filters('give_donation_history_row_amount', $donation_amount, $post->ID);
 							?>
 						</span>
 						</td>
 					<?php endif; ?>
 
 					<?php
-					if ( filter_var( $donation_history_args['status'], FILTER_VALIDATE_BOOLEAN ) ) :
+					if (filter_var($donation_history_args['status'], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<td class="give-donation-status">%1$s</td>',
-							give_get_payment_status( $post, true )
+							give_get_payment_status($post, true)
 						);
 					endif;
 
-					if ( filter_var( $donation_history_args['payment_method'], FILTER_VALIDATE_BOOLEAN ) ) :
+					if (filter_var($donation_history_args['payment_method'], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<td class="give-donation-payment-method">%1$s</td>',
-							give_get_gateway_checkout_label( give_get_payment_gateway( $post->ID ) )
+							give_get_gateway_checkout_label(give_get_payment_gateway($post->ID))
 						);
 					endif;
 					?>
 					<td class="give-donation-details">
 						<?php
 						// Display View Receipt or.
-						if ( 'publish' !== $post->post_status && 'subscription' !== $post->post_status ) :
+						if ('publish' !== $post->post_status && 'subscription' !== $post->post_status) :
 							echo sprintf(
 								'<a href="%1$s"><span class="give-donation-status %2$s">%3$s</span></a>',
 								esc_url(
 									add_query_arg(
 										'payment_key',
-										give_get_payment_key( $post->ID ),
+										give_get_payment_key($post->ID),
 										give_get_history_page_uri()
 									)
 								),
 								$post->post_status,
-								__( 'View', 'give' ) . ' ' . give_get_payment_status( $post, true ) . ' &raquo;'
+								__('View', 'give').' '.give_get_payment_status($post, true).' &raquo;'
 							);
 
 						else :
@@ -192,11 +192,11 @@ 
 								esc_url(
 									add_query_arg(
 										'payment_key',
-										give_get_payment_key( $post->ID ),
+										give_get_payment_key($post->ID),
 										give_get_history_page_uri()
 									)
 								),
-								__( 'View Receipt &raquo;', 'give' )
+								__('View Receipt &raquo;', 'give')
 							);
 
 						endif;
@@ -213,7 +213,7 @@ 
 					 * @param int   $post_id       The ID of the post.
 					 * @param mixed $donation_data Payment meta data.
 					 */
-					do_action( 'give_donation_history_row_end', $post->ID, $donation_data );
+					do_action('give_donation_history_row_end', $post->ID, $donation_data);
 					?>
 				</tr>
 			<?php endforeach; ?>
@@ -226,22 +226,22 @@ 
 			 *
 			 * @since 1.8.17
 			 */
-			do_action( 'give_donation_history_table_end' );
+			do_action('give_donation_history_table_end');
 			?>
 		</table>
 		<div id="give-donation-history-pagination" class="give_pagination navigation">
 			<?php
 			$big = 999999;
-			echo paginate_links( array(
-				'base'    => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
+			echo paginate_links(array(
+				'base'    => str_replace($big, '%#%', esc_url(get_pagenum_link($big))),
 				'format'  => '?paged=%#%',
-				'current' => max( 1, get_query_var( 'paged' ) ),
-				'total'   => ceil( give_count_donations_of_donor() / 20 ) // 20 items per page
-			) );
+				'current' => max(1, get_query_var('paged')),
+				'total'   => ceil(give_count_donations_of_donor() / 20) // 20 items per page
+			));
 			?>
 		</div>
 	</div>
 	<?php wp_reset_postdata(); ?>
 <?php else : ?>
-	<?php Give()->notices->print_frontend_notice( __( 'It looks like you haven\'t made any donations.', 'give' ), true, 'success' ); ?>
+	<?php Give()->notices->print_frontend_notice(__('It looks like you haven\'t made any donations.', 'give'), true, 'success'); ?>
 <?php endif;
\ No newline at end of file

includes/admin/reports/class-gateways-reports-table.php

Spacing +48 added lines, -51 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 
 	}
 
@@ -64,12 +64,12 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		$donation_list_page_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+	public function column_default($item, $column_name) {
+		$donation_list_page_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'complete_sales':
-				$value = $item[ $column_name ] ?
+				$value = $item[$column_name] ?
 					sprintf(
 						'<a href="%s">%s</a>',
 						add_query_arg(
@@ -79,13 +79,12 @@ 
 							),
 							$donation_list_page_url
 						),
-						$item[ $column_name ]
-					) :
-					$item[ $column_name ];
+						$item[$column_name]
+					) : $item[$column_name];
 				break;
 
 			case 'pending_sales':
-				$value = $item[ $column_name ] ?
+				$value = $item[$column_name] ?
 					sprintf(
 						'<a href="%s">%s</a>',
 						add_query_arg(
@@ -95,13 +94,12 @@ 
 							),
 							$donation_list_page_url
 						),
-						$item[ $column_name ]
-					) :
-					$item[ $column_name ];
+						$item[$column_name]
+					) : $item[$column_name];
 				break;
 
 			case 'total_sales':
-				$value = $item[ $column_name ] ?
+				$value = $item[$column_name] ?
 					sprintf(
 						'<a href="%s">%s</a>',
 						add_query_arg(
@@ -110,14 +108,13 @@ 
 							),
 							$donation_list_page_url
 						),
-						$item[ $column_name ]
-					) :
-					$item[ $column_name ];
+						$item[$column_name]
+					) : $item[$column_name];
 
 				break;
 
 			default:
-				$value = $item[ $column_name ];
+				$value = $item[$column_name];
 		}
 
 		return $value;
@@ -132,11 +129,11 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'label'           => esc_attr__( 'Gateway', 'give' ),
-			'complete_sales'  => esc_attr__( 'Complete Payments', 'give' ),
-			'pending_sales'   => esc_attr__( 'Pending / Failed Payments', 'give' ),
-			'total_sales'     => esc_attr__( 'Total Payments', 'give' ),
-			'total_donations' => esc_attr__( 'Total Donated', 'give' )
+			'label'           => esc_attr__('Gateway', 'give'),
+			'complete_sales'  => esc_attr__('Complete Payments', 'give'),
+			'pending_sales'   => esc_attr__('Pending / Failed Payments', 'give'),
+			'total_sales'     => esc_attr__('Total Payments', 'give'),
+			'total_donations' => esc_attr__('Total Donated', 'give')
 		);
 
 		return $columns;
@@ -151,7 +148,7 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'total_donations' => array( 'total_donations', false )
+			'total_donations' => array('total_donations', false)
 		);
 	}
 
@@ -164,7 +161,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 
@@ -175,7 +172,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -187,27 +184,27 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h2 class="alignleft reports-earnings-title screen-reader-text">
-					<?php _e( 'Donation Methods Report', 'give' ); ?>
+					<?php _e('Donation Methods Report', 'give'); ?>
 				</h2>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -229,17 +226,17 @@ 
 	 *
 	 * @return int
 	 */
-	public function give_sort_total_donations( $old_value, $new_value ) {
+	public function give_sort_total_donations($old_value, $new_value) {
 		// If no sort, default to label.
-		$orderby = ( ! empty( $_REQUEST['orderby'] ) ) ? $_REQUEST['orderby'] : 'label';
+		$orderby = ( ! empty($_REQUEST['orderby'])) ? $_REQUEST['orderby'] : 'label';
 
 		//If no order, default to asc.
-		$order = ( ! empty( $_REQUEST['order'] ) ) ? $_REQUEST['order'] : 'asc';
+		$order = ( ! empty($_REQUEST['order'])) ? $_REQUEST['order'] : 'asc';
 
 		//Determine sort order.
-		$result = strcmp( $old_value[ $orderby ], $new_value[ $orderby ] );
+		$result = strcmp($old_value[$orderby], $new_value[$orderby]);
 
-		return ( $order === 'asc' ) ? $result : -$result;
+		return ($order === 'asc') ? $result : -$result;
 	}
 
 
@@ -256,10 +253,10 @@ 
 		$gateways     = give_get_payment_gateways();
 		$stats        = new Give_Payment_Stats();
 
-		foreach ( $gateways as $gateway_id => $gateway ) {
+		foreach ($gateways as $gateway_id => $gateway) {
 
-			$complete_count = give_count_sales_by_gateway( $gateway_id, 'publish' );
-			$pending_count  = give_count_sales_by_gateway( $gateway_id, array( 'pending', 'failed' ) );
+			$complete_count = give_count_sales_by_gateway($gateway_id, 'publish');
+			$pending_count  = give_count_sales_by_gateway($gateway_id, array('pending', 'failed'));
 
 			$reports_data[] = array(
 				'ID'              => $gateway_id,
@@ -267,7 +264,7 @@ 
 				'complete_sales'  => $complete_count,
 				'pending_sales'   => $pending_count,
 				'total_sales'     => $complete_count + $pending_count,
-				'total_donations' => give_currency_filter( give_format_amount( $stats->get_earnings( 0, strtotime('04/13/2015' ), current_time('timestamp' ), $gateway_id ), array( 'sanitize' => false ) ) ),
+				'total_donations' => give_currency_filter(give_format_amount($stats->get_earnings(0, strtotime('04/13/2015'), current_time('timestamp'), $gateway_id), array('sanitize' => false))),
 			);
 		}
 
@@ -288,11 +285,11 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->reports_data();
 
 		// Sort Array when we are sorting data in array.
-		usort( $this->items, array( $this, 'give_sort_total_donations' ) );
+		usort($this->items, array($this, 'give_sort_total_donations'));
 
 	}
 }

includes/admin/upgrades/views/upgrades-complete.php

Spacing +3 added lines, -3 removed lines

@@ -10,17 +10,17 @@
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 ?>
 <div class="wrap" id="poststuff">
 	<div id="give-updates">
-		<h1 id="give-updates-h1"><?php esc_html_e( 'Give - Updates Complete', 'give' ); ?></h1>
+		<h1 id="give-updates-h1"><?php esc_html_e('Give - Updates Complete', 'give'); ?></h1>
 		<hr class="wp-header-end">
 
 		<div class="give-update-panel-content">
-			<p><?php esc_html_e( 'Congratulations! You are running the latest versions of Give and its add-ons.', 'give' ); ?></p>
+			<p><?php esc_html_e('Congratulations! You are running the latest versions of Give and its add-ons.', 'give'); ?></p>
 		</div>
 
 	</div>

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

Spacing +95 added lines, -97 removed lines

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

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

Spacing +71 added lines, -71 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,40 +53,40 @@ 
 	 */
 	public function get_data() {
 
-		$donor    = new Give_Donor( $this->customer_id );
-		$payments = $this->get_stored_data( 'give_recount_donor_payments_' . $donor->id );
+		$donor    = new Give_Donor($this->customer_id);
+		$payments = $this->get_stored_data('give_recount_donor_payments_'.$donor->id);
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $payments, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($payments, $offset, $this->per_step);
 
-		if ( count( $step_items ) > 0 ) {
-			$pending_total = (float) $this->get_stored_data( 'give_stats_donor_pending_total' . $donor->id );
+		if (count($step_items) > 0) {
+			$pending_total = (float) $this->get_stored_data('give_stats_donor_pending_total'.$donor->id);
 			$step_total    = 0;
 
-			$found_payment_ids = $this->get_stored_data( 'give_stats_found_payments_' . $donor->id );
+			$found_payment_ids = $this->get_stored_data('give_stats_found_payments_'.$donor->id);
 
-			foreach ( $step_items as $payment ) {
-				$payment = get_post( $payment->ID );
+			foreach ($step_items as $payment) {
+				$payment = get_post($payment->ID);
 
-				if ( is_null( $payment ) || is_wp_error( $payment ) || 'give_payment' !== $payment->post_type ) {
+				if (is_null($payment) || is_wp_error($payment) || 'give_payment' !== $payment->post_type) {
 
-					$missing_payments   = $this->get_stored_data( 'give_stats_missing_payments' . $donor->id );
+					$missing_payments   = $this->get_stored_data('give_stats_missing_payments'.$donor->id);
 					$missing_payments[] = $payment->ID;
-					$this->store_data( 'give_stats_missing_payments' . $donor->id, $missing_payments );
+					$this->store_data('give_stats_missing_payments'.$donor->id, $missing_payments);
 
 					continue;
 				}
 
 				$should_process_payment = 'publish' == $payment->post_status ? true : false;
-				$should_process_payment = apply_filters( 'give_donor_recount_should_process_donation', $should_process_payment, $payment );
+				$should_process_payment = apply_filters('give_donor_recount_should_process_donation', $should_process_payment, $payment);
 
-				if ( true === $should_process_payment ) {
+				if (true === $should_process_payment) {
 
 					$found_payment_ids[] = $payment->ID;
 
-					if ( apply_filters( 'give_donor_recount_should_increase_value', true, $payment ) ) {
-						$payment_amount = (float) give_donation_amount( $payment->ID, array( 'type' => 'stats' ) );
-						$step_total     += $payment_amount;
+					if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+						$payment_amount = (float) give_donation_amount($payment->ID, array('type' => 'stats'));
+						$step_total += $payment_amount;
 					}
 
 				}
@@ -94,8 +94,8 @@ 
 			}
 
 			$updated_total = $pending_total + $step_total;
-			$this->store_data( 'give_stats_donor_pending_total' . $donor->id, $updated_total );
-			$this->store_data( 'give_stats_found_payments_' . $donor->id, $found_payment_ids );
+			$this->store_data('give_stats_donor_pending_total'.$donor->id, $updated_total);
+			$this->store_data('give_stats_found_payments_'.$donor->id, $found_payment_ids);
 
 			return true;
 		}
@@ -112,16 +112,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$payments = $this->get_stored_data( 'give_recount_donor_payments_' . $this->customer_id );
-		$total    = count( $payments );
+		$payments = $this->get_stored_data('give_recount_donor_payments_'.$this->customer_id);
+		$total    = count($payments);
 
 		$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;
 		}
 
@@ -135,8 +135,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->customer_id = isset( $request['customer_id'] ) ? sanitize_text_field( $request['customer_id'] ) : false;
+	public function set_properties($request) {
+		$this->customer_id = isset($request['customer_id']) ? sanitize_text_field($request['customer_id']) : false;
 	}
 
 	/**
@@ -147,53 +147,53 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$donor       = new Give_Donor( $this->customer_id );
-			$payment_ids = get_option( 'give_stats_found_payments_' . $donor->id, array() );
-			$this->delete_data( 'give_stats_found_payments_' . $donor->id );
+			$donor       = new Give_Donor($this->customer_id);
+			$payment_ids = get_option('give_stats_found_payments_'.$donor->id, array());
+			$this->delete_data('give_stats_found_payments_'.$donor->id);
 
-			$removed_payments = array_unique( get_option( 'give_stats_missing_payments' . $donor->id, array() ) );
+			$removed_payments = array_unique(get_option('give_stats_missing_payments'.$donor->id, array()));
 
 			// Find non-existing payments (deleted) and total up the donation count
 			$purchase_count = 0;
-			foreach ( $payment_ids as $key => $payment_id ) {
-				if ( in_array( $payment_id, $removed_payments ) ) {
-					unset( $payment_ids[ $key ] );
+			foreach ($payment_ids as $key => $payment_id) {
+				if (in_array($payment_id, $removed_payments)) {
+					unset($payment_ids[$key]);
 					continue;
 				}
 
-				$payment = get_post( $payment_id );
-				if ( apply_filters( 'give_donor_recount_should_increase_count', true, $payment ) ) {
-					$purchase_count ++;
+				$payment = get_post($payment_id);
+				if (apply_filters('give_donor_recount_should_increase_count', true, $payment)) {
+					$purchase_count++;
 				}
 			}
 
-			$this->delete_data( 'give_stats_missing_payments' . $donor->id );
+			$this->delete_data('give_stats_missing_payments'.$donor->id);
 
-			$pending_total = $this->get_stored_data( 'give_stats_donor_pending_total' . $donor->id );
-			$this->delete_data( 'give_stats_donor_pending_total' . $donor->id );
-			$this->delete_data( 'give_recount_donor_stats_' . $donor->id );
-			$this->delete_data( 'give_recount_donor_payments_' . $this->customer_id );
+			$pending_total = $this->get_stored_data('give_stats_donor_pending_total'.$donor->id);
+			$this->delete_data('give_stats_donor_pending_total'.$donor->id);
+			$this->delete_data('give_recount_donor_stats_'.$donor->id);
+			$this->delete_data('give_recount_donor_payments_'.$this->customer_id);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$donor->update( array(
+			$payment_ids = implode(',', $payment_ids);
+			$donor->update(array(
 				'payment_ids'    => $payment_ids,
 				'purchase_count' => $purchase_count,
 				'purchase_value' => $pending_total,
-			) );
+			));
 
 			$this->done    = true;
-			$this->message = esc_html__( 'Donor stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Donor stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -226,26 +226,26 @@ 
 	 * @return void
 	 */
 	public function pre_fetch() {
-		if ( $this->step === 1 ) {
-			$allowed_payment_status = apply_filters( 'give_recount_donors_donation_statuses', give_get_payment_status_keys() );
+		if ($this->step === 1) {
+			$allowed_payment_status = apply_filters('give_recount_donors_donation_statuses', give_get_payment_status_keys());
 
 			// Before we start, let's zero out the customer's data
-			$donor = new Give_Donor( $this->customer_id );
-			$donor->update( array( 'purchase_value' => give_format_amount( 0, array( 'sanitize' => false ) ), 'purchase_count' => 0 ) );
+			$donor = new Give_Donor($this->customer_id);
+			$donor->update(array('purchase_value' => give_format_amount(0, array('sanitize' => false)), 'purchase_count' => 0));
 
-			$attached_payment_ids = explode( ',', $donor->payment_ids );
+			$attached_payment_ids = explode(',', $donor->payment_ids);
 
 			$attached_args = array(
 				'post__in' => $attached_payment_ids,
-				'number'   => - 1,
+				'number'   => -1,
 				'status'   => $allowed_payment_status,
 			);
 
-			$attached_payments = give_get_payments( $attached_args );
+			$attached_payments = give_get_payments($attached_args);
 
 			$unattached_args = array(
 				'post__not_in' => $attached_payment_ids,
-				'number'       => - 1,
+				'number'       => -1,
 				'status'       => $allowed_payment_status,
 				'meta_query'   => array(
 					array(
@@ -255,11 +255,11 @@ 
 				),
 			);
 
-			$unattached_payments = give_get_payments( $unattached_args );
+			$unattached_payments = give_get_payments($unattached_args);
 
-			$payments = array_merge( $attached_payments, $unattached_payments );
+			$payments = array_merge($attached_payments, $unattached_payments);
 
-			$this->store_data( 'give_recount_donor_payments_' . $donor->id, $payments );
+			$this->store_data('give_recount_donor_payments_'.$donor->id, $payments);
 		}
 	}
 
@@ -272,17 +272,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -298,10 +298,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -315,7 +315,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -327,9 +327,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

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

Spacing +45 added lines, -45 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,49 +53,49 @@ 
 	 */
 	public function get_data() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_earnings' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_earnings');
 		}
 
-		$total = get_option( 'give_temp_recount_earnings', false );
+		$total = get_option('give_temp_recount_earnings', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = (float) 0;
-			$this->store_data( 'give_temp_recount_earnings', $total );
+			$this->store_data('give_temp_recount_earnings', $total);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		$args = apply_filters( 'give_recount_earnings_args', array(
+		$args = apply_filters('give_recount_earnings_args', array(
 			'number' => $this->per_step,
 			'page'   => $this->step,
 			'status' => $accepted_statuses,
 			'fields' => 'ids'
-		) );
+		));
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$total += (float) give_donation_amount( $payment, array( 'type' => 'stats' ) );
+				$total += (float) give_donation_amount($payment, array('type' => 'stats'));
 
 			}
 
-			if ( $total < 0 ) {
+			if ($total < 0) {
 				$totals = 0;
 			}
 
-			$total = round( $total, give_get_price_decimals() );
+			$total = round($total, give_get_price_decimals());
 
-			$this->store_data( 'give_temp_recount_earnings', $total );
+			$this->store_data('give_temp_recount_earnings', $total);
 
 			return true;
 
 		}
 
-		update_option( 'give_earnings_total', $total );
+		update_option('give_earnings_total', $total);
 
 		return false;
 
@@ -109,25 +109,25 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_earnings_total' );
+		$total = $this->get_stored_data('give_recount_earnings_total');
 
-		if ( false === $total ) {
-			$args = apply_filters( 'give_recount_earnings_total_args', array() );
+		if (false === $total) {
+			$args = apply_filters('give_recount_earnings_total_args', array());
 
-			$counts = give_count_payments( $args );
-			$total  = absint( $counts->publish );
-			$total  = apply_filters( 'give_recount_store_earnings_total', $total );
+			$counts = give_count_payments($args);
+			$total  = absint($counts->publish);
+			$total  = apply_filters('give_recount_store_earnings_total', $total);
 
-			$this->store_data( 'give_recount_earnings_total', $total );
+			$this->store_data('give_recount_earnings_total', $total);
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -141,7 +141,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -152,21 +152,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_earnings_total' );
-			$this->delete_data( 'give_temp_recount_earnings' );
+			$this->delete_data('give_recount_earnings_total');
+			$this->delete_data('give_temp_recount_earnings');
 			$this->done    = true;
-			$this->message = esc_html__( 'Income stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Income stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -203,17 +203,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -229,10 +229,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -246,7 +246,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -258,9 +258,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }