WordImpress / Give

includes/admin/tools/import/class-give-import-core-settings.php

Spacing +73 added lines, -73 removed lines

@@ -11,11 +11,11 @@ 
  * @since       1.8.17
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Import_Core_Settings' ) ) {
+if ( ! class_exists('Give_Import_Core_Settings')) {
 
 	/**
 	 * Give_Import_Core_Settings.
@@ -68,7 +68,7 @@ 
 		 * @return static
 		 */
 		public static function get_instance() {
-			if ( null === static::$instance ) {
+			if (null === static::$instance) {
 				self::$instance = new static();
 			}
 
@@ -95,26 +95,26 @@ 
 		 * @return void
 		 */
 		private function setup_hooks() {
-			if ( ! $this->is_donations_import_page() ) {
+			if ( ! $this->is_donations_import_page()) {
 				return;
 			}
 
 			// Do not render main import tools page.
-			remove_action( 'give_admin_field_tools_import', array( 'Give_Settings_Import', 'render_import_field', ) );
+			remove_action('give_admin_field_tools_import', array('Give_Settings_Import', 'render_import_field',));
 
 			// Render donation import page
-			add_action( 'give_admin_field_tools_import', array( $this, 'render_page' ) );
+			add_action('give_admin_field_tools_import', array($this, 'render_page'));
 
 			// Print the HTML.
-			add_action( 'give_tools_import_core_settings_form_start', array( $this, 'html' ), 10 );
+			add_action('give_tools_import_core_settings_form_start', array($this, 'html'), 10);
 
 			// Run when form submit.
-			add_action( 'give-tools_save_import', array( $this, 'save' ) );
+			add_action('give-tools_save_import', array($this, 'save'));
 
-			add_action( 'give-tools_update_notices', array( $this, 'update_notices' ), 11, 1 );
+			add_action('give-tools_update_notices', array($this, 'update_notices'), 11, 1);
 
 			// Used to add submit button.
-			add_action( 'give_tools_import_core_settings_form_end', array( $this, 'submit' ), 10 );
+			add_action('give_tools_import_core_settings_form_end', array($this, 'submit'), 10);
 		}
 
 		/**
@@ -126,9 +126,9 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function update_notices( $messages ) {
-			if ( ! empty( $_GET['tab'] ) && 'import' === give_clean( $_GET['tab'] ) ) {
-				unset( $messages['give-setting-updated'] );
+		public function update_notices($messages) {
+			if ( ! empty($_GET['tab']) && 'import' === give_clean($_GET['tab'])) {
+				unset($messages['give-setting-updated']);
 			}
 
 			return $messages;
@@ -140,7 +140,7 @@ 
 		 * @since 1.8.17
 		 */
 		public function submit() {
-			wp_nonce_field( 'give-save-settings', '_give-save-settings' );
+			wp_nonce_field('give-save-settings', '_give-save-settings');
 			?>
 			<input type="hidden" class="import-step" id="import-step" name="step" value="<?php echo $this->get_step(); ?>"/>
 			<input type="hidden" class="importer-type" value="<?php echo $this->importer_type; ?>"/>
@@ -163,7 +163,7 @@ 
 				       id="<?php echo "step-{$step}"; ?>">
 					<tbody>
 					<?php
-					switch ( $this->get_step() ) {
+					switch ($this->get_step()) {
 						case 1:
 							$this->render_upload_html();
 							break;
@@ -190,14 +190,14 @@ 
 		public function import_success() {
 			// Imported successfully
 
-			$success = (bool) ( isset( $_GET['success'] ) ? give_clean( $_GET['success'] ) : false );
-			$undo = (bool) ( isset( $_GET['undo'] ) ? give_clean( $_GET['undo'] ) : false );
+			$success = (bool) (isset($_GET['success']) ? give_clean($_GET['success']) : false);
+			$undo = (bool) (isset($_GET['undo']) ? give_clean($_GET['undo']) : false);
 			$query_arg_setting = array(
 				'post_type' => 'give_forms',
 				'page'      => 'give-settings',
 			);
 
-			if ( $undo ) {
+			if ($undo) {
 				$success = false;
 			}
 
@@ -210,30 +210,30 @@ 
 				'undo'      => 'true',
 			);
 
-			$title = __( 'Settings Importing Completed!', 'give' );
-			if ( $success ) {
+			$title = __('Settings Importing Completed!', 'give');
+			if ($success) {
 				$query_arg_success['undo'] = '1';
 				$query_arg_success['step'] = '3';
 				$query_arg_success['success'] = '1';
-				$text = __( 'Undo Importing', 'give' );
+				$text = __('Undo Importing', 'give');
 			} else {
-				if ( $undo ) {
-					$host_give_options = get_option( 'give_settings_old', array() );
-					update_option( 'give_settings', $host_give_options );
-					$title = __( 'Undo of Setting Imported Completed!', 'give' );
+				if ($undo) {
+					$host_give_options = get_option('give_settings_old', array());
+					update_option('give_settings', $host_give_options);
+					$title = __('Undo of Setting Imported Completed!', 'give');
 				} else {
-					$title = __( 'Failed to import', 'give' );
+					$title = __('Failed to import', 'give');
 				}
 
-				$text = __( 'Importing Again', 'give' );
+				$text = __('Importing Again', 'give');
 			}
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
 					<h2><?php echo $title; ?></h2>
 					<p>
-						<a class="button button-large button-secondary" href="<?php echo add_query_arg( $query_arg_success, admin_url( 'edit.php' ) ); ?>"><?php echo $text; ?></a>
-						<a class="button button-large button-secondary" href="<?php echo add_query_arg( $query_arg_setting, admin_url( 'edit.php' ) ); ?>"><?php echo __( 'View Settings', 'give' ); ?></a>
+						<a class="button button-large button-secondary" href="<?php echo add_query_arg($query_arg_success, admin_url('edit.php')); ?>"><?php echo $text; ?></a>
+						<a class="button button-large button-secondary" href="<?php echo add_query_arg($query_arg_setting, admin_url('edit.php')); ?>"><?php echo __('View Settings', 'give'); ?></a>
 					</p>
 				</th>
 			</tr>
@@ -246,14 +246,14 @@ 
 		 * @since 1.8.17
 		 */
 		public function start_import() {
-			$type = ( ! empty( $_GET['type'] ) ? give_clean( $_GET['type'] ) : 'replace' );
-			$file_name = ( ! empty( $_GET['file_name'] ) ? give_clean( $_GET['file_name'] ) : '' );
+			$type = ( ! empty($_GET['type']) ? give_clean($_GET['type']) : 'replace');
+			$file_name = ( ! empty($_GET['file_name']) ? give_clean($_GET['file_name']) : '');
 
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Importing', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'Your settings are now being imported...', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Importing', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('Your settings are now being imported...', 'give') ?></p>
 				</th>
 			</tr>
 
@@ -286,14 +286,14 @@ 
 			$step = $this->get_step();
 			?>
 			<ol class="give-progress-steps">
-				<li class="<?php echo( 1 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Upload JSON file', 'give' ); ?>
+				<li class="<?php echo(1 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Upload JSON file', 'give'); ?>
 				</li>
-				<li class="<?php echo( 2 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Import', 'give' ); ?>
+				<li class="<?php echo(2 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Import', 'give'); ?>
 				</li>
-				<li class="<?php echo( 3 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Done!', 'give' ); ?>
+				<li class="<?php echo(3 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Done!', 'give'); ?>
 				</li>
 			</ol>
 			<?php
@@ -307,14 +307,14 @@ 
 		 * @return int $step on which step doest the import is on.
 		 */
 		public function get_step() {
-			$step    = (int) ( isset( $_REQUEST['step'] ) ? give_clean( $_REQUEST['step'] ) : 0 );
+			$step    = (int) (isset($_REQUEST['step']) ? give_clean($_REQUEST['step']) : 0);
 			$on_step = 1;
 
-			if ( empty( $step ) || 1 === $step ) {
+			if (empty($step) || 1 === $step) {
 				$on_step = 1;
-			} elseif ( 2 === $step ) {
+			} elseif (2 === $step) {
 				$on_step = 2;
-			} elseif ( 3 === $step ) {
+			} elseif (3 === $step) {
 				$on_step = 3;
 			}
 
@@ -327,7 +327,7 @@ 
 		 * @since 1.8.17
 		 */
 		public function render_page() {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-import-core-settings.php';
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-import-core-settings.php';
 		}
 
 		/**
@@ -339,28 +339,28 @@ 
 		 * @return void
 		 */
 		public function render_upload_html() {
-			$json = ( isset( $_POST['json'] ) ? give_clean( $_POST['json'] ) : '' );
-			$type = ( isset( $_POST['type'] ) ? give_clean( $_POST['type'] ) : 'merge' );
+			$json = (isset($_POST['json']) ? give_clean($_POST['json']) : '');
+			$type = (isset($_POST['type']) ? give_clean($_POST['type']) : 'merge');
 			$step = $this->get_step();
 
 			?>
 			<tr valign="top">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Import Core Settings from a JSON file', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'This tool allows you to import Give settings from another Give installation. Settings imported contain data from Give core as well as any of our Premium Add-ons.', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Import Core Settings from a JSON file', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('This tool allows you to import Give settings from another Give installation. Settings imported contain data from Give core as well as any of our Premium Add-ons.', 'give') ?></p>
 				</th>
 			</tr>
 
 			<tr valign="top">
 				<th scope="row" class="titledesc">
-					<label for="json"><?php esc_html_e( 'Choose a JSON file:', 'give' ) ?></label>
+					<label for="json"><?php esc_html_e('Choose a JSON file:', 'give') ?></label>
 				</th>
 				<td class="give-forminp">
 					<div class="give-field-wrap">
 						<label for="json">
 							<input type="file" name="json" class="give-upload-json-file" value="<?php echo $json; ?>"
 							       accept=".json">
-							<p class="give-field-description"><?php esc_html_e( 'The file type must be JSON.', 'give' )?></p>
+							<p class="give-field-description"><?php esc_html_e('The file type must be JSON.', 'give')?></p>
 						</label>
 					</div>
 				</td>
@@ -369,20 +369,20 @@ 
 			$settings = array(
 				array(
 					'id'          => 'type',
-					'name'        => __( 'Merge Type:', 'give' ),
-					'description' => __( 'Select "Merge" to retain existing settings, or "Replace" to overwrite with the settings from the JSON file', 'give' ),
+					'name'        => __('Merge Type:', 'give'),
+					'description' => __('Select "Merge" to retain existing settings, or "Replace" to overwrite with the settings from the JSON file', 'give'),
 					'default'     => $type,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'merge'   => __( 'Merge', 'give' ),
-						'replace' => __( 'Replace', 'give' ),
+						'merge'   => __('Merge', 'give'),
+						'replace' => __('Replace', 'give'),
 					),
 				),
 			);
 
-			$settings = apply_filters( 'give_import_core_setting_html', $settings );
+			$settings = apply_filters('give_import_core_setting_html', $settings);
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 			?>
 			<tr valign="top">
 				<th></th>
@@ -390,7 +390,7 @@ 
 					<input type="submit"
 					       class="button button-primary button-large button-secondary <?php echo "step-{$step}"; ?>"
 					       id="recount-stats-submit"
-					       value="<?php esc_attr_e( 'Submit', 'give' ); ?>"/>
+					       value="<?php esc_attr_e('Submit', 'give'); ?>"/>
 				</th>
 			</tr>
 			<?php
@@ -407,20 +407,20 @@ 
 			$step = $this->get_step();
 
 			// Validation for first step.
-			if ( 1 === $step ) {
-				$type          = ( ! empty( $_REQUEST['type'] ) ? give_clean( $_REQUEST['type'] ) : 'replace' );
+			if (1 === $step) {
+				$type          = ( ! empty($_REQUEST['type']) ? give_clean($_REQUEST['type']) : 'replace');
 				$core_settings = self::upload_widget_settings_file();
-				if ( ! empty( $core_settings['error'] ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload a valid JSON settings file.', 'give' ) );
+				if ( ! empty($core_settings['error'])) {
+					Give_Admin_Settings::add_error('give-import-csv', __('Please upload a valid JSON settings file.', 'give'));
 				} else {
-					$file_path = explode( '/', $core_settings['file'] );
-					$count     = ( count( $file_path ) - 1 );
-					$url       = give_import_page_url( (array) apply_filters( 'give_import_core_settings_importing_url', array(
+					$file_path = explode('/', $core_settings['file']);
+					$count     = (count($file_path) - 1);
+					$url       = give_import_page_url((array) apply_filters('give_import_core_settings_importing_url', array(
 						'step'          => '2',
 						'importer-type' => $this->importer_type,
 						'type'          => $type,
-						'file_name'     => $file_path[ $count ],
-					) ) );
+						'file_name'     => $file_path[$count],
+					)));
 
 					?>
 					<script type="text/javascript">
@@ -438,7 +438,7 @@ 
 		 * @return bool
 		 */
 		private function is_donations_import_page() {
-			return 'import' === give_get_current_setting_tab() && isset( $_GET['importer-type'] ) && $this->importer_type === give_clean( $_GET['importer-type'] );
+			return 'import' === give_get_current_setting_tab() && isset($_GET['importer-type']) && $this->importer_type === give_clean($_GET['importer-type']);
 		}
 
 		/**
@@ -447,14 +447,14 @@ 
 		 */
 		public static function upload_widget_settings_file() {
 			$upload = false;
-			if ( isset( $_FILES['json'] ) ) {
-				add_filter( 'upload_mimes', array( __CLASS__, 'json_upload_mimes' ) );
+			if (isset($_FILES['json'])) {
+				add_filter('upload_mimes', array(__CLASS__, 'json_upload_mimes'));
 
-				$upload = wp_handle_upload( $_FILES['json'], array( 'test_form' => false ) );
+				$upload = wp_handle_upload($_FILES['json'], array('test_form' => false));
 
-				remove_filter( 'upload_mimes', array( __CLASS__, 'json_upload_mimes' ) );
+				remove_filter('upload_mimes', array(__CLASS__, 'json_upload_mimes'));
 			} else {
-				Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid JSON file.', 'give' ) );
+				Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid JSON file.', 'give'));
 			}
 
 			return $upload;
@@ -465,7 +465,7 @@ 
 		 *
 		 * @param array $existing_mimes
 		 */
-		public static function json_upload_mimes( $existing_mimes = array() ) {
+		public static function json_upload_mimes($existing_mimes = array()) {
 			$existing_mimes['json'] = 'application/json';
 
 			return $existing_mimes;

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

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,7 +50,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->paymentmeta = $this->table_name = $wpdb->prefix . 'give_paymentmeta';
+		$wpdb->paymentmeta = $this->table_name = $wpdb->prefix.'give_paymentmeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -84,6 +84,6 @@ 
 	 * @return bool
 	 */
 	protected function is_custom_meta_table_active() {
-		return give_has_upgrade_completed( 'v20_move_metadata_into_new_table' );
+		return give_has_upgrade_completed('v20_move_metadata_into_new_table');
 	}
 }

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

Spacing +7 added lines, -7 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,7 +50,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->logmeta     = $this->table_name = $wpdb->prefix . 'give_logmeta';
+		$wpdb->logmeta     = $this->table_name = $wpdb->prefix.'give_logmeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -86,18 +86,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function delete_row( $log_id = 0 ) {
+	public function delete_row($log_id = 0) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$log_id = absint( $log_id );
+		$log_id = absint($log_id);
 
-		if ( empty( $log_id ) ) {
+		if (empty($log_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE log_id = %d", $log_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE log_id = %d", $log_id))) {
 			return false;
 		}
 
@@ -114,7 +114,7 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
+	protected function is_valid_post_type($ID) {
 		return $ID && true;
 	}
 }

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

Spacing +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,7 +53,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->donormeta   = $this->table_name = $wpdb->prefix . 'give_donormeta';
+		$wpdb->donormeta   = $this->table_name = $wpdb->prefix.'give_donormeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -90,12 +90,12 @@ 
 	 *
 	 * @return  bool  False for failure. True for success.
 	 */
-	public function delete_all_meta( $donor_id = 0 ) {
+	public function delete_all_meta($donor_id = 0) {
 		global $wpdb;
-		$status = $wpdb->delete( $this->table_name, array( 'donor_id' => $donor_id ), array( '%d' ) );
+		$status = $wpdb->delete($this->table_name, array('donor_id' => $donor_id), array('%d'));
 
-		if( $status ) {
-			Give_Cache::delete_group( $donor_id, 'give-donors' );
+		if ($status) {
+			Give_Cache::delete_group($donor_id, 'give-donors');
 		}
 	}
 
@@ -111,8 +111,8 @@ 
 		global $wpdb;
 
 		if (
-			! give_has_upgrade_completed( 'v20_rename_donor_tables' ) &&
-			$wpdb->query( $wpdb->prepare( "SHOW TABLES LIKE %s","{$wpdb->prefix}give_customermeta" ) )
+			! give_has_upgrade_completed('v20_rename_donor_tables') &&
+			$wpdb->query($wpdb->prepare("SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customermeta"))
 		) {
 			$wpdb->donormeta = $this->table_name = "{$wpdb->prefix}give_customermeta";
 			$this->meta_type = 'customer';
@@ -131,7 +131,7 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
+	protected function is_valid_post_type($ID) {
 		return $ID && true;
 	}
 

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

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -48,8 +48,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -66,32 +66,32 @@ 
 
 		$args = array(
 			'number'  => $this->per_step,
-			'offset'  => $this->per_step * ( $this->step - 1 ),
+			'offset'  => $this->per_step * ($this->step - 1),
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$allowed_payment_status = apply_filters( 'give_recount_donors_donation_statuses', give_get_payment_status_keys() );
+			$allowed_payment_status = apply_filters('give_recount_donors_donation_statuses', give_get_payment_status_keys());
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$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 = (array) give_get_payments( $attached_args );
+				$attached_payments = (array) 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(
@@ -102,29 +102,29 @@ 
 					),
 				);
 
-				$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);
 
 				$purchase_value = 0.00;
 				$purchase_count = 0;
 				$payment_ids    = array();
 
-				if ( $payments ) {
+				if ($payments) {
 
-					foreach ( $payments as $payment ) {
+					foreach ($payments as $payment) {
 
 						$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) {
 
-							if ( apply_filters( 'give_donor_recount_should_increase_value', true, $payment ) ) {
-								$purchase_value += (float) give_donation_amount( $payment->ID, array( 'type' => 'stats' ) );
+							if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+								$purchase_value += (float) give_donation_amount($payment->ID, array('type' => 'stats'));
 							}
 
-							if ( apply_filters( 'give_donor_recount_should_increase_count', true, $payment ) ) {
-								$purchase_count ++;
+							if (apply_filters('give_donor_recount_should_increase_count', true, $payment)) {
+								$purchase_count++;
 							}
 						}
 
@@ -132,7 +132,7 @@ 
 					}
 				}
 
-				$payment_ids = implode( ',', $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
 
 				$donor_update_data = array(
 					'purchase_count' => $purchase_count,
@@ -140,8 +140,8 @@ 
 					'payment_ids'    => $payment_ids,
 				);
 
-				$donor_instance = new Give_Donor( $donor->id );
-				$donor_instance->update( $donor_update_data );
+				$donor_instance = new Give_Donor($donor->id);
+				$donor_instance->update($donor_update_data);
 
 			}// End foreach().
 
@@ -161,21 +161,21 @@ 
 	public function get_percentage_complete() {
 
 		$args = array(
-			'number'  => - 1,
+			'number'  => -1,
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$donors = Give()->donors->get_donors( $args );
-		$total     = count( $donors );
+		$donors = Give()->donors->get_donors($args);
+		$total = count($donors);
 
 		$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;
 		}
 
@@ -189,7 +189,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -200,21 +200,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		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->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;
 		}

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

Spacing +53 added lines, -53 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -69,59 +69,59 @@ 
 	 */
 	public function get_data() {
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_form_stats' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_form_stats');
 		}
 
-		$totals = $this->get_stored_data( 'give_temp_recount_form_stats' );
+		$totals = $this->get_stored_data('give_temp_recount_form_stats');
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array(
 				'earnings' => (float) 0,
 				'sales'    => 0,
 			);
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 		}
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'give_forms' => $this->form_id,
 			'number'     => $this->per_step,
 			'status'     => $accepted_statuses,
 			'paged'      => $this->step,
 			'fields'     => 'ids',
-		) );
+		));
 
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		if ( $payments ) {
-			foreach ( $payments as $payment ) {
+		if ($payments) {
+			foreach ($payments as $payment) {
 
 				// Ensure acceptable status only.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					continue;
 				}
 
 				// Ensure only payments for this form are counted.
-				if ( $payment->form_id != $this->form_id ) {
+				if ($payment->form_id != $this->form_id) {
 					continue;
 				}
 
-				$totals['sales'] ++;
+				$totals['sales']++;
 				$totals['earnings'] += $payment->total;
 
 			}
 
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 
 			return true;
 		}
 
 
-		give_update_meta( $this->form_id, '_give_form_sales', $totals['sales'] );
-		give_update_meta( $this->form_id, '_give_form_earnings', give_sanitize_amount_for_db( $totals['earnings'] ) );
+		give_update_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		give_update_meta($this->form_id, '_give_form_earnings', give_sanitize_amount_for_db($totals['earnings']));
 
 		return false;
 	}
@@ -133,35 +133,35 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
+		if ($this->step == 1) {
+			$this->delete_data('give_recount_total_'.$this->form_id);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
-		$total             = $this->get_stored_data( 'give_recount_total_' . $this->form_id );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
+		$total             = $this->get_stored_data('give_recount_total_'.$this->form_id);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = 0;
-			$args  = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args  = apply_filters('give_recount_form_stats_total_args', array(
 				'give_forms' => $this->form_id,
-				'number'     => - 1,
+				'number'     => -1,
 				'status'     => $accepted_statuses,
 				'fields'     => 'ids',
-			) );
+			));
 
-			$payments = new Give_Payments_Query( $args );
-			$total    = count( $payments->get_payments() );
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$payments = new Give_Payments_Query($args);
+			$total    = count($payments->get_payments());
+			$this->store_data('give_recount_total_'.$this->form_id, $total);
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -175,8 +175,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;
 	}
 
 	/**
@@ -187,21 +187,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_temp_recount_form_stats' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_temp_recount_form_stats');
 			$this->done    = true;
-			$this->message = sprintf( esc_html__( 'Donation counts and income amount statistics successfully recounted for "%s".', 'give' ), get_the_title( $this->form_id ) );
+			$this->message = sprintf(esc_html__('Donation counts and income amount statistics successfully recounted for "%s".', 'give'), get_the_title($this->form_id));
 
 			return false;
 		}
@@ -235,17 +235,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;
@@ -261,10 +261,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,
@@ -278,7 +278,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -290,9 +290,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }
\ No newline at end of file

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

Spacing +47 added lines, -47 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,8 +46,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -61,49 +61,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;
 
@@ -117,25 +117,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;
 		}
 
@@ -149,7 +149,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -160,21 +160,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;
 		}
@@ -211,17 +211,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;
@@ -237,10 +237,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,
@@ -254,7 +254,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -266,9 +266,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-delete-test-transactions.php

Spacing +48 added lines, -48 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,8 +46,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -62,20 +62,20 @@ 
 	 * @return array|bool $data The data for the CSV file
 	 */
 	public function get_data() {
-		$items = $this->get_stored_data( 'give_temp_delete_test_ids' );
+		$items = $this->get_stored_data('give_temp_delete_test_ids');
 
-		if ( ! is_array( $items ) ) {
+		if ( ! is_array($items)) {
 			return false;
 		}
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $items, $offset, $this->per_step );
-		$meta_table = __give_v20_bc_table_details( 'payment' );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
+		$meta_table = __give_v20_bc_table_details('payment');
 
-		if ( $step_items ) {
-			foreach ( $step_items as $item ) {
+		if ($step_items) {
+			foreach ($step_items as $item) {
 				// Delete the main payment.
-				give_delete_donation( absint( $item['id'] ) );
+				give_delete_donation(absint($item['id']));
 			}
 			return true;
 		}
@@ -91,16 +91,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$items = $this->get_stored_data( 'give_temp_delete_test_ids', false );
-		$total = count( $items );
+		$items = $this->get_stored_data('give_temp_delete_test_ids', false);
+		$total = count($items);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -114,7 +114,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -125,29 +125,29 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to delete test transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to delete test transactions.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', give_get_total_earnings( true ) );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true));
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_data( 'give_temp_delete_test_ids' );
+			$this->delete_data('give_temp_delete_test_ids');
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = __( 'Test transactions successfully deleted.', 'give' );
+			$this->message = __('Test transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -180,27 +180,27 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_delete_test_ids' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_delete_test_ids');
 		}
 
-		$items = get_option( 'give_temp_delete_test_ids', false );
+		$items = get_option('give_temp_delete_test_ids', false);
 
-		if ( false === $items ) {
+		if (false === $items) {
 			$items = array();
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_status' => 'any',
-				'number'      => - 1,
+				'number'      => -1,
 				'meta_key'    => '_give_payment_mode',
 				'meta_value'  => 'test'
-			) );
+			));
 
-			$posts    = new Give_Payments_Query( $args );
+			$posts    = new Give_Payments_Query($args);
 			$payments = $posts->get_payments();
 
 			/* @var Give_Payment $payment */
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 				$items[] = array(
 					'id'   => (int) $payment->ID,
 					'type' => 'give_payment',
@@ -209,9 +209,9 @@ 
 			
 			// Allow filtering of items to remove with an unassociative array for each item.
 			// The array contains the unique ID of the item, and a 'type' for you to use in the execution of the get_data method.
-			$items = apply_filters( 'give_delete_test_items', $items );
+			$items = apply_filters('give_delete_test_items', $items);
 
-			$this->store_data( 'give_temp_delete_test_ids', $items );
+			$this->store_data('give_temp_delete_test_ids', $items);
 		}
 
 	}
@@ -225,17 +225,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -251,10 +251,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -268,7 +268,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -280,9 +280,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

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

Spacing +97 added lines, -99 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -73,8 +73,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -89,108 +89,106 @@ 
 	 */
 	public function get_data() {
 
-		$totals             = $this->get_stored_data( 'give_temp_recount_all_stats' );
-		$payment_items      = $this->get_stored_data( 'give_temp_payment_items' );
-		$processed_payments = $this->get_stored_data( 'give_temp_processed_payments' );
-		$accepted_statuses  = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$totals             = $this->get_stored_data('give_temp_recount_all_stats');
+		$payment_items      = $this->get_stored_data('give_temp_payment_items');
+		$processed_payments = $this->get_stored_data('give_temp_processed_payments');
+		$accepted_statuses  = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array();
 		}
 
-		if ( false === $payment_items ) {
+		if (false === $payment_items) {
 			$payment_items = array();
 		}
 
-		if ( false === $processed_payments ) {
+		if (false === $processed_payments) {
 			$processed_payments = array();
 		}
 
-		$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+		$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-		$payments = $this->get_stored_data( 'give_temp_all_payments_data' );
+		$payments = $this->get_stored_data('give_temp_all_payments_data');
 
-		if ( empty( $payments ) ) {
-			$args = apply_filters( 'give_recount_form_stats_args', array(
+		if (empty($payments)) {
+			$args = apply_filters('give_recount_form_stats_args', array(
 				'give_forms' => $all_forms,
 				'number'     => $this->per_step,
 				'status'     => $accepted_statuses,
 				'paged'      => $this->step,
 				'output'     => 'give_payments',
-			) );
+			));
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 		}
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
 			//Loop through payments
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$payment_id = ( ! empty( $payment['ID'] ) ? absint( $payment['ID'] ) : ( ! empty( $payment->ID ) ? absint( $payment->ID ) : false ) );
-				$payment = new Give_Payment( $payment_id );
+				$payment_id = ( ! empty($payment['ID']) ? absint($payment['ID']) : ( ! empty($payment->ID) ? absint($payment->ID) : false));
+				$payment = new Give_Payment($payment_id);
 
 				// Prevent payments that have all ready been retrieved from a previous sales log from counting again.
-				if ( in_array( $payment->ID, $processed_payments ) ) {
+				if (in_array($payment->ID, $processed_payments)) {
 					continue;
 				}
 
 				// Verify accepted status.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					$processed_payments[] = $payment->ID;
 					continue;
 				}
 
-				$payment_item = $payment_items[ $payment->ID ];
+				$payment_item = $payment_items[$payment->ID];
 
-				$form_id = isset( $payment_item['id'] ) ? $payment_item['id'] : '';
+				$form_id = isset($payment_item['id']) ? $payment_item['id'] : '';
 
 				// Must have a form ID.
-				if ( empty( $form_id ) ) {
+				if (empty($form_id)) {
 					continue;
 				}
 
 				// Form ID must be within $all_forms array to be validated.
-				if ( ! in_array( $form_id, $all_forms ) ) {
+				if ( ! in_array($form_id, $all_forms)) {
 					continue;
 				}
 
 				// Set Sales count
-				$totals[ $form_id ]['sales'] = isset( $totals[ $form_id ]['sales'] ) ?
-					++ $totals[ $form_id ]['sales'] :
-					1;
+				$totals[$form_id]['sales'] = isset($totals[$form_id]['sales']) ?
+					++$totals[$form_id]['sales'] : 1;
 
 				// Set Total Earnings
-				$totals[ $form_id ]['earnings'] = isset( $totals[ $form_id ]['earnings'] ) ?
-					( $totals[ $form_id ]['earnings'] + $payment_item['price'] ) :
-					$payment_item['price'];
+				$totals[$form_id]['earnings'] = isset($totals[$form_id]['earnings']) ?
+					($totals[$form_id]['earnings'] + $payment_item['price']) : $payment_item['price'];
 
 				$processed_payments[] = $payment->ID;
 			}
 
 			// Get the list of form ids which does not contain any payment record.
-			$remaining_form_ids = array_diff( $all_forms, array_keys( $totals ) );
-			foreach ( $remaining_form_ids as $form_id ) {
+			$remaining_form_ids = array_diff($all_forms, array_keys($totals));
+			foreach ($remaining_form_ids as $form_id) {
 				//If array key doesn't exist, create it
-				if ( ! array_key_exists( $form_id, $totals ) ) {
-					$totals[ $form_id ] = array(
+				if ( ! array_key_exists($form_id, $totals)) {
+					$totals[$form_id] = array(
 						'sales'    => (int) 0,
 						'earnings' => (float) 0,
 					);
 				}
 			}
 
-			$this->store_data( 'give_temp_processed_payments', $processed_payments );
-			$this->store_data( 'give_temp_recount_all_stats', $totals );
+			$this->store_data('give_temp_processed_payments', $processed_payments);
+			$this->store_data('give_temp_recount_all_stats', $totals);
 
 			return true;
 		}
 
 
-		foreach ( $totals as $key => $stats ) {
-			give_update_meta( $key, '_give_form_sales', $stats['sales'] );
-			give_update_meta( $key, '_give_form_earnings', give_sanitize_amount_for_db( $stats['earnings'] ) );
+		foreach ($totals as $key => $stats) {
+			give_update_meta($key, '_give_form_sales', $stats['sales']);
+			give_update_meta($key, '_give_form_earnings', give_sanitize_amount_for_db($stats['earnings']));
 		}
 
 		return false;
@@ -205,20 +203,20 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_all_total' );
+		$total = $this->get_stored_data('give_recount_all_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$this->pre_fetch();
-			$total = $this->get_stored_data( 'give_recount_all_total' );
+			$total = $this->get_stored_data('give_recount_all_total');
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -232,8 +230,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -244,25 +242,25 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_form_ids' );
-			$this->delete_data( 'give_temp_processed_payments' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_form_ids');
+			$this->delete_data('give_temp_processed_payments');
 			$this->done    = true;
-			$this->message = esc_html__( 'Donation form income amounts and donation counts stats successfully recounted.', 'give' );
+			$this->message = esc_html__('Donation form income amounts and donation counts stats successfully recounted.', 'give');
 
 			return false;
 		}
@@ -298,70 +296,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,
@@ -369,8 +367,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);
 		}
 
 	}
@@ -384,17 +382,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;
@@ -410,10 +408,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,
@@ -427,7 +425,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -439,9 +437,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));
 	}
 
 }