WordImpress / Give

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

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,10 +20,10 @@ 
  * @since  1.5
  */
 function give_register_batch_recount_export_classes() {
-	add_action( 'give_batch_export_class_include', 'give_include_batch_export_class', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_batch_export_class', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_batch_recount_export_classes', 10 );
+add_action('give_register_batch_exporter', 'give_register_batch_recount_export_classes', 10);
 
 
 /**
@@ -35,39 +35,39 @@ 
  *
  * @return void
  */
-function give_include_batch_export_class( $class ) {
-	switch ( $class ) {
+function give_include_batch_export_class($class) {
+	switch ($class) {
 
 		case 'Give_Tools_Delete_Donors':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-test-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-test-donors.php';
 			break;
 
 		case 'Give_Tools_Import_Donors':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-import-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-import-donors.php';
 			break;
 
 		case 'Give_Tools_Delete_Test_Transactions':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-test-transactions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-test-transactions.php';
 			break;
 
 		case 'Give_Tools_Recount_Donor_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-donor-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-donor-stats.php';
 			break;
 
 		case 'Give_Tools_Reset_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-reset-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-reset-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_All_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-all-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-all-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_Form_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-form-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-form-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_Income':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-income.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-income.php';
 			break;
 	}
 }

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

Spacing +48 added lines, -48 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 += give_get_payment_amount( $payment );
+				$total += give_get_payment_amount($payment);
 
 			}
 
-			if ( $total < 0 ) {
+			if ($total < 0) {
 				$totals = 0;
 			}
 
-			$total = round( $total, give_currency_decimal_filter() );
+			$total = round($total, give_currency_decimal_filter());
 
-			$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;
 		}
@@ -176,10 +176,10 @@ 
 	 * Headers.
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -207,17 +207,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;
@@ -233,10 +233,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,
@@ -250,7 +250,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -262,9 +262,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 +59 added lines, -59 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,22 +55,22 @@ 
 	public function get_data() {
 		global $wpdb;
 
-		$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 );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
 
-		if ( $step_items ) {
+		if ($step_items) {
 
 			$step_ids = array(
 				'other' => array(),
 			);
 
-			foreach ( $step_items as $item ) {
+			foreach ($step_items as $item) {
 
 				$step_ids['other'][] = $item['id'];
 
@@ -78,31 +78,31 @@ 
 
 			$sql = array();
 
-			foreach ( $step_ids as $type => $ids ) {
+			foreach ($step_ids as $type => $ids) {
 
-				if ( empty( $ids ) ) {
+				if (empty($ids)) {
 					continue;
 				}
 
 				$parent_query = '';
 
-				switch ( $type ) {
+				switch ($type) {
 					case 'other':
 
-						$temp_ids = implode( ',', $ids );
+						$temp_ids = implode(',', $ids);
 
 						// Get all the test logs of the donations ids.
 						$parent_query = "SELECT DISTINCT post_id as id FROM $wpdb->postmeta WHERE meta_key = '_give_log_payment_id' AND meta_value IN ( $temp_ids )";
-						$parent_ids   = $wpdb->get_results( $parent_query, 'ARRAY_A' );
+						$parent_ids   = $wpdb->get_results($parent_query, 'ARRAY_A');
 
 						// List of all test logs.
-						if ( $parent_ids ) {
-							foreach ( $parent_ids as $parent_id ) {
+						if ($parent_ids) {
+							foreach ($parent_ids as $parent_id) {
 								// Adding all the test log in post ids that are going to get deleted.
 								$ids[] = $parent_id['id'];
 							}
 						}
-						$ids = implode( ',', $ids );
+						$ids = implode(',', $ids);
 
 						$sql[] = "DELETE FROM $wpdb->posts WHERE id IN ($ids)";
 						$sql[] = "DELETE FROM $wpdb->postmeta WHERE post_id IN ($ids)";
@@ -113,11 +113,11 @@ 
 
 			}
 
-			if ( ! empty( $sql ) ) {
-				foreach ( $sql as $query ) {
-					$wpdb->query( $query );
+			if ( ! empty($sql)) {
+				foreach ($sql as $query) {
+					$wpdb->query($query);
 				}
-				do_action( 'give_delete_log_cache' );
+				do_action('give_delete_log_cache');
 			}
 
 			return true;
@@ -134,16 +134,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;
 		}
 
@@ -157,7 +157,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -168,29 +168,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;
 		}
@@ -200,10 +200,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -227,27 +227,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',
@@ -256,9 +256,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);
 		}
 
 	}
@@ -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/views/html-admin-page-imports.php

Spacing +9 added lines, -9 removed lines

@@ -3,11 +3,11 @@ 
  * Admin View: Import Donations
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! current_user_can( 'manage_give_settings' ) ) {
+if ( ! current_user_can('manage_give_settings')) {
 	return;
 }
 
@@ -16,11 +16,11 @@ 
  *
  * @since 1.8.13
  */
-do_action( 'give_tools_import_main_before' );
+do_action('give_tools_import_main_before');
 ?>
     <div id="poststuff">
         <div class="postbox">
-            <h1 class="give-importer-h1" align="center"><?php esc_html_e( 'Import Donations', 'give' ); ?></h1>
+            <h1 class="give-importer-h1" align="center"><?php esc_html_e('Import Donations', 'give'); ?></h1>
             <div class="inside give-tools-setting-page-import">
 				<?php
 				/**
@@ -28,7 +28,7 @@ 
 				 *
 				 * @since 1.5
 				 */
-				do_action( 'give_tools_import_form_before_start' );
+				do_action('give_tools_import_form_before_start');
 				?>
                 <form method="post" id="give-import-donations-form" class="give-import-form tools-setting-page-import tools-setting-page-import">
 
@@ -38,7 +38,7 @@ 
 					 *
 					 * @since 1.5
 					 */
-					do_action( 'give_tools_import_form_start' );
+					do_action('give_tools_import_form_start');
 					?>
 
 					<?php
@@ -47,7 +47,7 @@ 
 					 *
 					 * @since 1.5
 					 */
-					do_action( 'give_tools_import_form_end' );
+					do_action('give_tools_import_form_end');
 					?>
                 </form>
 				<?php
@@ -56,7 +56,7 @@ 
 				 *
 				 * @since 1.5
 				 */
-				do_action( 'give_tools_import_form_after_end' );
+				do_action('give_tools_import_form_after_end');
 				?>
             </div><!-- .inside -->
         </div><!-- .postbox -->
@@ -67,4 +67,4 @@ 
  *
  * @since 1.8.13
  */
-do_action( 'give_tools_import_main_after' );
+do_action('give_tools_import_main_after');

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

Spacing +28 added lines, -28 removed lines

@@ -3,11 +3,11 @@ 
  * Admin View: Exports
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! current_user_can( 'manage_give_settings' ) ) {
+if ( ! current_user_can('manage_give_settings')) {
 	return;
 }
 
@@ -16,29 +16,29 @@ 
  *
  * @since 1.5
  */
-do_action( 'give_tools_recount_stats_before' );
+do_action('give_tools_recount_stats_before');
 ?>
 	<div id="poststuff">
 		<div class="postbox">
 
-			<h2 class="hndle ui-sortable-handle"><span><?php esc_html_e( 'Recount Stats', 'give' ); ?></span></h2>
+			<h2 class="hndle ui-sortable-handle"><span><?php esc_html_e('Recount Stats', 'give'); ?></span></h2>
 
 			<div class="inside recount-stats-controls">
-				<p><?php esc_html_e( 'Use these tools to recount stats, delete test transactions, or reset stats.', 'give' ); ?></p>
+				<p><?php esc_html_e('Use these tools to recount stats, delete test transactions, or reset stats.', 'give'); ?></p>
 				<form method="post" id="give-tools-recount-form" class="give-export-form">
 
-					<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+					<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 
 					<select name="give-export-class" id="recount-stats-type">
-						<option value="0" selected="selected" disabled="disabled"><?php esc_html_e( 'Please select an option', 'give' ); ?></option>
-						<option data-type="recount-stats" value="Give_Tools_Recount_Income"><?php esc_html_e( 'Recalculate Total Donation Income Amount', 'give' ); ?></option>
-						<option data-type="recount-form" value="Give_Tools_Recount_Form_Stats"><?php esc_html_e( 'Recalculate Income Amount and Donation Counts for a Form', 'give' ); ?></option>
-						<option data-type="recount-all" value="Give_Tools_Recount_All_Stats"><?php esc_html_e( 'Recalculate Income Amount and Donation Counts for All Forms', 'give' ); ?></option>
-						<option data-type="recount-donor-stats" value="Give_Tools_Recount_Donor_Stats"><?php esc_html_e( 'Recalculate Donor Statistics', 'give' ); ?></option>
-						<option data-type="delete-test-transactions" value="Give_Tools_Delete_Test_Transactions"><?php esc_html_e( 'Delete Test Payments', 'give' ); ?></option>
-						<option data-type="delete-test-donors"   value="Give_Tools_Delete_Donors"><?php esc_html_e( 'Delete Test Donors and Payments', 'give' ); ?></option>
-						<option data-type="delete-import-donors"   value="Give_Tools_Import_Donors"><?php esc_html_e( 'Delete Imported Donors and Payments', 'give' ); ?></option>
-						<option data-type="reset-stats" value="Give_Tools_Reset_Stats"><?php esc_html_e( 'Delete All Data', 'give' ); ?></option>
+						<option value="0" selected="selected" disabled="disabled"><?php esc_html_e('Please select an option', 'give'); ?></option>
+						<option data-type="recount-stats" value="Give_Tools_Recount_Income"><?php esc_html_e('Recalculate Total Donation Income Amount', 'give'); ?></option>
+						<option data-type="recount-form" value="Give_Tools_Recount_Form_Stats"><?php esc_html_e('Recalculate Income Amount and Donation Counts for a Form', 'give'); ?></option>
+						<option data-type="recount-all" value="Give_Tools_Recount_All_Stats"><?php esc_html_e('Recalculate Income Amount and Donation Counts for All Forms', 'give'); ?></option>
+						<option data-type="recount-donor-stats" value="Give_Tools_Recount_Donor_Stats"><?php esc_html_e('Recalculate Donor Statistics', 'give'); ?></option>
+						<option data-type="delete-test-transactions" value="Give_Tools_Delete_Test_Transactions"><?php esc_html_e('Delete Test Payments', 'give'); ?></option>
+						<option data-type="delete-test-donors"   value="Give_Tools_Delete_Donors"><?php esc_html_e('Delete Test Donors and Payments', 'give'); ?></option>
+						<option data-type="delete-import-donors"   value="Give_Tools_Import_Donors"><?php esc_html_e('Delete Imported Donors and Payments', 'give'); ?></option>
+						<option data-type="reset-stats" value="Give_Tools_Reset_Stats"><?php esc_html_e('Delete All Data', 'give'); ?></option>
 						<?php
 						/**
 						 * Fires in the recount stats selectbox.
@@ -47,7 +47,7 @@ 
 						 *
 						 * @since 1.5
 						 */
-						do_action( 'give_recount_tool_options' );
+						do_action('give_recount_tool_options');
 						?>
 					</select>
 
@@ -55,22 +55,22 @@ 
 						<?php
 						$args = array(
 							'name'   => 'form_id',
-							'number' => - 1,
+							'number' => -1,
 							'chosen' => true,
 						);
-						echo Give()->html->forms_dropdown( $args );
+						echo Give()->html->forms_dropdown($args);
 						?>
 					</span>
 
-					<input type="submit" id="recount-stats-submit" value="<?php esc_attr_e( 'Submit', 'give' ); ?>" class="button-secondary"/>
+					<input type="submit" id="recount-stats-submit" value="<?php esc_attr_e('Submit', 'give'); ?>" class="button-secondary"/>
 
 					<br/>
 
 					<span class="give-recount-stats-descriptions">
-						<span id="recount-stats"><?php esc_html_e( 'Recalculates the overall donation income amount.', 'give' ); ?></span>
-						<span id="recount-form"><?php esc_html_e( 'Recalculates the donation and income stats for a specific form.', 'give' ); ?></span>
-						<span id="recount-all"><?php esc_html_e( 'Recalculates the earnings and sales stats for all forms.', 'give' ); ?></span>
-						<span id="recount-customer-stats"><?php esc_html_e( 'Recalculates the lifetime value and donation counts for all donors.', 'give' ); ?></span>
+						<span id="recount-stats"><?php esc_html_e('Recalculates the overall donation income amount.', 'give'); ?></span>
+						<span id="recount-form"><?php esc_html_e('Recalculates the donation and income stats for a specific form.', 'give'); ?></span>
+						<span id="recount-all"><?php esc_html_e('Recalculates the earnings and sales stats for all forms.', 'give'); ?></span>
+						<span id="recount-customer-stats"><?php esc_html_e('Recalculates the lifetime value and donation counts for all donors.', 'give'); ?></span>
 						<?php
 						/**
 						 * Fires in the recount stats description area.
@@ -79,10 +79,10 @@ 
 						 *
 						 * @since 1.5
 						 */
-						do_action( 'give_recount_tool_descriptions' );
+						do_action('give_recount_tool_descriptions');
 						?>
-						<span id="delete-test-transactions"><?php _e( '<strong>Deletes</strong> all TEST donations, donors, and related log entries.', 'give' ); ?></span>
-						<span id="reset-stats"><?php _e( '<strong>Deletes</strong> ALL donations, donors, and related log entries regardless of test or live mode.', 'give' ); ?></span>
+						<span id="delete-test-transactions"><?php _e('<strong>Deletes</strong> all TEST donations, donors, and related log entries.', 'give'); ?></span>
+						<span id="reset-stats"><?php _e('<strong>Deletes</strong> ALL donations, donors, and related log entries regardless of test or live mode.', 'give'); ?></span>
 					</span>
 
 					<span class="spinner"></span>
@@ -96,7 +96,7 @@ 
 				 *
 				 * @since 1.5
 				 */
-				do_action( 'give_tools_recount_forms' );
+				do_action('give_tools_recount_forms');
 				?>
 			</div><!-- .inside -->
 		</div><!-- .postbox -->
@@ -107,4 +107,4 @@ 
  *
  * @since 1.5
  */
-do_action( 'give_tools_recount_stats_after' );
+do_action('give_tools_recount_stats_after');

includes/admin/payments/class-payments-table.php

Spacing +202 added lines, -202 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';
 }
 
 /**
@@ -134,15 +134,15 @@ 
 	public function __construct() {
 
 		// 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.
-			'ajax'     => false,                              // Does this table support ajax?
-		) );
+		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?
+		));
 
 		$this->process_bulk_action();
 		$this->get_payment_counts();
-		$this->base_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+		$this->base_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 	}
 
 	/**
@@ -151,55 +151,55 @@ 
 	 * @return void
 	 */
 	public function advanced_filters() {
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : null;
-		$status     = isset( $_GET['status'] ) ? sanitize_text_field( $_GET['status'] ) : '';
-		$donor      = isset( $_GET['donor'] ) ? sanitize_text_field( $_GET['donor'] ) : '';
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : '';
-		$form_id    = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : 0;
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : null;
+		$status     = isset($_GET['status']) ? sanitize_text_field($_GET['status']) : '';
+		$donor      = isset($_GET['donor']) ? sanitize_text_field($_GET['donor']) : '';
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : '';
+		$form_id    = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : 0;
 		?>
 		<div id="give-payment-filters" class="give-filters">
-			<?php $this->search_box( esc_html__( 'Search', 'give' ), 'give-payments' ); ?>
+			<?php $this->search_box(esc_html__('Search', 'give'), 'give-payments'); ?>
 			<div id="give-payment-date-filters">
 				<div class="give-filter give-filter-half">
 					<label for="start-date"
-					       class="give-start-date-label"><?php esc_html_e( 'Start Date', 'give' ); ?></label>
+					       class="give-start-date-label"><?php esc_html_e('Start Date', 'give'); ?></label>
 					<input type="text" id="start-date" name="start-date" class="give_datepicker"
 					       value="<?php echo $start_date; ?>" placeholder="mm/dd/yyyy" />
 				</div>
 				<div class="give-filter give-filter-half">
-					<label for="end-date" class="give-end-date-label"><?php esc_html_e( 'End Date', 'give' ); ?></label>
+					<label for="end-date" class="give-end-date-label"><?php esc_html_e('End Date', 'give'); ?></label>
 					<input type="text" id="end-date" name="end-date" class="give_datepicker"
 					       value="<?php echo $end_date; ?>" placeholder="mm/dd/yyyy" />
 				</div>
 			</div>
 			<div id="give-payment-form-filter" class="give-filter">
 				<label for="give-donation-forms-filter"
-				       class="give-donation-forms-filter-label"><?php esc_html_e( 'Form', 'give' ); ?></label>
+				       class="give-donation-forms-filter-label"><?php esc_html_e('Form', 'give'); ?></label>
 				<?php
 				// Filter Donations by Donation Forms.
-				echo Give()->html->forms_dropdown( array(
+				echo Give()->html->forms_dropdown(array(
 					'name'     => 'form_id',
 					'id'       => 'give-donation-forms-filter',
 					'class'    => 'give-donation-forms-filter',
 					'selected' => $form_id, // Make sure to have $form_id set to 0, if there is no selection.
 					'chosen'   => true,
-					'number'   => - 1,
-				) );
+					'number'   => -1,
+				));
 				?>
 			</div>
 
-			<?php if ( ! empty( $status ) ) : ?>
-				<input type="hidden" name="status" value="<?php echo esc_attr( $status ); ?>" />
+			<?php if ( ! empty($status)) : ?>
+				<input type="hidden" name="status" value="<?php echo esc_attr($status); ?>" />
 			<?php endif; ?>
 
 			<div class="give-filter">
-				<?php submit_button( __( 'Apply', 'give' ), 'secondary', '', false ); ?>
+				<?php submit_button(__('Apply', 'give'), 'secondary', '', false); ?>
 				<?php
 				// Clear active filters button.
-				if ( ! empty( $start_date ) || ! empty( $end_date ) || ! empty( $donor ) || ! empty( $search ) || ! empty( $status ) || ! empty( $form_id ) ) : ?>
-					<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>"
-					   class="button give-clear-filters-button"><?php esc_html_e( 'Clear Filters', 'give' ); ?></a>
+				if ( ! empty($start_date) || ! empty($end_date) || ! empty($donor) || ! empty($search) || ! empty($status) || ! empty($form_id)) : ?>
+					<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>"
+					   class="button give-clear-filters-button"><?php esc_html_e('Clear Filters', 'give'); ?></a>
 				<?php endif; ?>
 			</div>
 		</div>
@@ -218,18 +218,18 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		if ( empty( $_REQUEST['s'] ) && ! $this->has_items() ) {
+	public function search_box($text, $input_id) {
+		if (empty($_REQUEST['s']) && ! $this->has_items()) {
 			return;
 		}
 
-		$input_id = $input_id . '-search-input';
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<div class="give-filter give-filter-search" role="search">
@@ -241,13 +241,13 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payment_history_search' );
+			do_action('give_payment_history_search');
 			?>
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?><br />
+			)); ?><br />
 		</div>
 		<?php
 	}
@@ -261,52 +261,52 @@ 
 	 */
 	public function get_views() {
 
-		$current = isset( $_GET['status'] ) ? $_GET['status'] : '';
+		$current = isset($_GET['status']) ? $_GET['status'] : '';
 		$views   = array();
 		$tabs    = array(
 			'all'        => array(
 				'total_count',
-				esc_html__( 'All', 'give' ),
+				esc_html__('All', 'give'),
 			),
 			'publish'    => array(
 				'complete_count',
-				esc_html__( 'Completed', 'give' ),
+				esc_html__('Completed', 'give'),
 			),
 			'pending'    => array(
 				'pending_count',
-				esc_html__( 'Pending', 'give' ),
+				esc_html__('Pending', 'give'),
 			),
 			'processing' => array(
 				'processing_count',
-				esc_html__( 'Processing', 'give' ),
+				esc_html__('Processing', 'give'),
 			),
 			'refunded'   => array(
 				'refunded_count',
-				esc_html__( 'Refunded', 'give' ),
+				esc_html__('Refunded', 'give'),
 			),
 			'revoked'    => array(
 				'revoked_count',
-				esc_html__( 'Revoked', 'give' ),
+				esc_html__('Revoked', 'give'),
 			),
 			'failed'     => array(
 				'failed_count',
-				esc_html__( 'Failed', 'give' ),
+				esc_html__('Failed', 'give'),
 			),
 			'cancelled'  => array(
 				'cancelled_count',
-				esc_html__( 'Cancelled', 'give' ),
+				esc_html__('Cancelled', 'give'),
 			),
 			'abandoned'  => array(
 				'abandoned_count',
-				esc_html__( 'Abandoned', 'give' ),
+				esc_html__('Abandoned', 'give'),
 			),
 			'preapproval'  => array(
 				'preapproval_count',
-				esc_html__( 'Preapproval Pending', 'give' ),
+				esc_html__('Preapproval Pending', 'give'),
 			),
 		);
 
-		foreach ( $tabs as $key => $tab ) {
+		foreach ($tabs as $key => $tab) {
 			$count_key = $tab[0];
 			$name      = $tab[1];
 			$count     = $this->$count_key;
@@ -321,19 +321,19 @@ 
 			 * @param string $key   Current view tab value.
 			 * @param int    $count Number of donation inside the tab.
 			 */
-			if ( 'all' === $key || $key === $current || apply_filters( 'give_payments_table_show_all_status', 0 < $count, $key, $count ) ) {
+			if ('all' === $key || $key === $current || apply_filters('give_payments_table_show_all_status', 0 < $count, $key, $count)) {
 
-				$views[ $key ] = sprintf(
+				$views[$key] = sprintf(
 					'<a href="%s" %s >%s&nbsp;<span class="count">(%s)</span></a>',
-					esc_url( ( 'all' === (string) $key ) ? remove_query_arg( array( 'status', 'paged' ) ) : add_query_arg( array( 'status' => $key, 'paged' => false ) ) ),
-					( ( 'all' === $key && empty( $current ) ) ) ? 'class="current"' : ( $current == $key ) ? 'class="current"' : '',
+					esc_url(('all' === (string) $key) ? remove_query_arg(array('status', 'paged')) : add_query_arg(array('status' => $key, 'paged' => false))),
+					(('all' === $key && empty($current))) ? 'class="current"' : ($current == $key) ? 'class="current"' : '',
 					$name,
 					$count
 				);
 			}
 		}
 
-		return apply_filters( 'give_payments_table_views', $views );
+		return apply_filters('give_payments_table_views', $views);
 	}
 
 	/**
@@ -346,15 +346,15 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'donation'      => esc_html__( 'Donation', 'give' ),
-			'donation_form' => esc_html__( 'Donation Form', 'give' ),
-			'status'        => esc_html__( 'Status', 'give' ),
-			'date'          => esc_html__( 'Date', 'give' ),
-			'amount'        => esc_html__( 'Amount', 'give' ),
-			'details'       => esc_html__( 'Details', 'give' ),
+			'donation'      => esc_html__('Donation', 'give'),
+			'donation_form' => esc_html__('Donation Form', 'give'),
+			'status'        => esc_html__('Status', 'give'),
+			'date'          => esc_html__('Date', 'give'),
+			'amount'        => esc_html__('Amount', 'give'),
+			'details'       => esc_html__('Details', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_columns', $columns );
+		return apply_filters('give_payments_table_columns', $columns);
 	}
 
 	/**
@@ -366,14 +366,14 @@ 
 	 */
 	public function get_sortable_columns() {
 		$columns = array(
-			'donation'      => array( 'ID', true ),
-			'donation_form' => array( 'donation_form', false ),
-			'status'        => array( 'status', false ),
-			'amount'        => array( 'amount', false ),
-			'date'          => array( 'date', false ),
+			'donation'      => array('ID', true),
+			'donation_form' => array('donation_form', false),
+			'status'        => array('status', false),
+			'amount'        => array('amount', false),
+			'date'          => array('date', false),
 		);
 
-		return apply_filters( 'give_payments_table_sortable_columns', $columns );
+		return apply_filters('give_payments_table_sortable_columns', $columns);
 	}
 
 	/**
@@ -399,55 +399,55 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $payment, $column_name ) {
+	public function column_default($payment, $column_name) {
 
-		$single_donation_url = esc_url( add_query_arg( 'id', $payment->ID, admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details' ) ) );
-		$row_actions         = $this->get_row_actions( $payment );
+		$single_donation_url = esc_url(add_query_arg('id', $payment->ID, admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details')));
+		$row_actions         = $this->get_row_actions($payment);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'donation' :
-				$value = sprintf( '<a href="%1$s" data-tooltip="%2$s">#%3$s</a>&nbsp;%4$s&nbsp;%5$s<br>', $single_donation_url, sprintf( esc_attr__( 'View Donation #%s', 'give' ), $payment->ID ), $payment->ID, esc_html__( 'by', 'give' ), $this->get_donor( $payment ) );
-				$value .= $this->get_donor_email( $payment );
-				$value .= $this->row_actions( $row_actions );
+				$value = sprintf('<a href="%1$s" data-tooltip="%2$s">#%3$s</a>&nbsp;%4$s&nbsp;%5$s<br>', $single_donation_url, sprintf(esc_attr__('View Donation #%s', 'give'), $payment->ID), $payment->ID, esc_html__('by', 'give'), $this->get_donor($payment));
+				$value .= $this->get_donor_email($payment);
+				$value .= $this->row_actions($row_actions);
 				break;
 
 			case 'amount' :
-				$amount = ! empty( $payment->total ) ? $payment->total : 0;
-				$value  = give_currency_filter( give_format_amount( $amount, array( 'sanitize' => false ) ), give_get_payment_currency_code( $payment->ID ) );
-				$value  .= sprintf( '<br><small>%1$s %2$s</small>', __( 'via', 'give' ), give_get_gateway_admin_label( $payment->gateway ) );
+				$amount = ! empty($payment->total) ? $payment->total : 0;
+				$value  = give_currency_filter(give_format_amount($amount, array('sanitize' => false)), give_get_payment_currency_code($payment->ID));
+				$value .= sprintf('<br><small>%1$s %2$s</small>', __('via', 'give'), give_get_gateway_admin_label($payment->gateway));
 				break;
 
 			case 'donation_form' :
-				$form_title = empty( $payment->form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $payment->form_id ) : $payment->form_title;
-				$value      = '<a href="' . admin_url( 'post.php?post=' . $payment->form_id . '&action=edit' ) . '">' . $form_title . '</a>';
-				$level      = give_get_payment_form_title( $payment->meta, true );
+				$form_title = empty($payment->form_title) ? sprintf(__('Untitled (#%s)', 'give'), $payment->form_id) : $payment->form_title;
+				$value      = '<a href="'.admin_url('post.php?post='.$payment->form_id.'&action=edit').'">'.$form_title.'</a>';
+				$level      = give_get_payment_form_title($payment->meta, true);
 
-				if ( ! empty( $level ) ) {
+				if ( ! empty($level)) {
 					$value .= $level;
 				}
 
 				break;
 
 			case 'date' :
-				$date  = strtotime( $payment->date );
-				$value = date_i18n( give_date_format(), $date );
+				$date  = strtotime($payment->date);
+				$value = date_i18n(give_date_format(), $date);
 				break;
 
 			case 'status' :
-				$value = $this->get_payment_status( $payment );
+				$value = $this->get_payment_status($payment);
 				break;
 
 			case 'details' :
-				$value = sprintf( '<div class="give-payment-details-link-wrap"><a href="%1$s" class="give-payment-details-link button button-small" data-tooltip="%2$s" aria-label="%2$s"><span class="dashicons dashicons-visibility"></span></a></div>', $single_donation_url, sprintf( esc_attr__( 'View Donation #%s', 'give' ), $payment->ID ) );
+				$value = sprintf('<div class="give-payment-details-link-wrap"><a href="%1$s" class="give-payment-details-link button button-small" data-tooltip="%2$s" aria-label="%2$s"><span class="dashicons dashicons-visibility"></span></a></div>', $single_donation_url, sprintf(esc_attr__('View Donation #%s', 'give'), $payment->ID));
 				break;
 
 			default:
-				$value = isset( $payment->$column_name ) ? $payment->$column_name : '';
+				$value = isset($payment->$column_name) ? $payment->$column_name : '';
 				break;
 
 		}// End switch().
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, $column_name );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, $column_name);
 	}
 
 	/**
@@ -460,17 +460,17 @@ 
 	 *
 	 * @return string                Data shown in the Email column
 	 */
-	public function get_donor_email( $payment ) {
+	public function get_donor_email($payment) {
 
-		$email = give_get_payment_user_email( $payment->ID );
+		$email = give_get_payment_user_email($payment->ID);
 
-		if ( empty( $email ) ) {
-			$email = esc_html__( '(unknown)', 'give' );
+		if (empty($email)) {
+			$email = esc_html__('(unknown)', 'give');
 		}
 
-		$value = '<a href="mailto:' . $email . '" data-tooltip="' . esc_attr__( 'Email donor', 'give' ) . '">' . $email . '</a>';
+		$value = '<a href="mailto:'.$email.'" data-tooltip="'.esc_attr__('Email donor', 'give').'">'.$email.'</a>';
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'email' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'email');
 	}
 
 	/**
@@ -482,32 +482,32 @@ 
 	 *
 	 * @return array $actions
 	 */
-	function get_row_actions( $payment ) {
+	function get_row_actions($payment) {
 
 		$actions = array();
-		$email   = give_get_payment_user_email( $payment->ID );
+		$email   = give_get_payment_user_email($payment->ID);
 
 		// Add search term string back to base URL.
-		$search_terms = ( isset( $_GET['s'] ) ? trim( $_GET['s'] ) : '' );
-		if ( ! empty( $search_terms ) ) {
-			$this->base_url = add_query_arg( 's', $search_terms, $this->base_url );
+		$search_terms = (isset($_GET['s']) ? trim($_GET['s']) : '');
+		if ( ! empty($search_terms)) {
+			$this->base_url = add_query_arg('s', $search_terms, $this->base_url);
 		}
 
-		if ( give_is_payment_complete( $payment->ID ) && ! empty( $email ) ) {
+		if (give_is_payment_complete($payment->ID) && ! empty($email)) {
 
-			$actions['email_links'] = sprintf( '<a class="resend-single-donation-receipt" href="%1$s" aria-label="%2$s">%3$s</a>', wp_nonce_url( add_query_arg( array(
+			$actions['email_links'] = sprintf('<a class="resend-single-donation-receipt" href="%1$s" aria-label="%2$s">%3$s</a>', wp_nonce_url(add_query_arg(array(
 				'give-action' => 'email_links',
 				'purchase_id' => $payment->ID,
-			), $this->base_url ), 'give_payment_nonce' ), sprintf( esc_attr__( 'Resend Donation %s Receipt', 'give' ), $payment->ID ), esc_html__( 'Resend Receipt', 'give' ) );
+			), $this->base_url), 'give_payment_nonce'), sprintf(esc_attr__('Resend Donation %s Receipt', 'give'), $payment->ID), esc_html__('Resend Receipt', 'give'));
 
 		}
 
-		$actions['delete'] = sprintf( '<a class="delete-single-donation" href="%1$s" aria-label="%2$s">%3$s</a>', wp_nonce_url( add_query_arg( array(
+		$actions['delete'] = sprintf('<a class="delete-single-donation" href="%1$s" aria-label="%2$s">%3$s</a>', wp_nonce_url(add_query_arg(array(
 			'give-action' => 'delete_payment',
 			'purchase_id' => $payment->ID,
-		), $this->base_url ), 'give_donation_nonce' ), sprintf( esc_attr__( 'Delete Donation %s', 'give' ), $payment->ID ), esc_html__( 'Delete', 'give' ) );
+		), $this->base_url), 'give_donation_nonce'), sprintf(esc_attr__('Delete Donation %s', 'give'), $payment->ID), esc_html__('Delete', 'give'));
 
-		return apply_filters( 'give_payment_row_actions', $actions, $payment );
+		return apply_filters('give_payment_row_actions', $actions, $payment);
 	}
 
 
@@ -521,14 +521,14 @@ 
 	 *
 	 * @return string                Data shown in the Email column
 	 */
-	function get_payment_status( $payment ) {
-		$value = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
-		if ( $payment->mode == 'test' ) {
-			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This donation was made in test mode.', 'give' ) . '">' . esc_html__( 'Test', 'give' ) . '</span>';
+	function get_payment_status($payment) {
+		$value = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
+		if ($payment->mode == 'test') {
+			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This donation was made in test mode.', 'give').'">'.esc_html__('Test', 'give').'</span>';
 		}
 
-		if ( true === $payment->import && true === (bool) apply_filters( 'give_payment_show_importer_label', false ) ) {
-			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This donation was imported.', 'give' ) . '">' . esc_html__( 'Import', 'give' ) . '</span>';
+		if (true === $payment->import && true === (bool) apply_filters('give_payment_show_importer_label', false)) {
+			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This donation was imported.', 'give').'">'.esc_html__('Import', 'give').'</span>';
 		}
 
 		return $value;
@@ -544,8 +544,8 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function column_cb( $payment ) {
-		return sprintf( '<input type="checkbox" name="%1$s[]" value="%2$s" />', 'payment', $payment->ID );
+	public function column_cb($payment) {
+		return sprintf('<input type="checkbox" name="%1$s[]" value="%2$s" />', 'payment', $payment->ID);
 	}
 
 	/**
@@ -558,8 +558,8 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function get_payment_id( $payment ) {
-		return '<span class="give-payment-id">' . give_get_payment_number( $payment->ID ) . '</span>';
+	public function get_payment_id($payment) {
+		return '<span class="give-payment-id">'.give_get_payment_number($payment->ID).'</span>';
 	}
 
 	/**
@@ -572,32 +572,32 @@ 
 	 *
 	 * @return string Data shown in the User column
 	 */
-	public function get_donor( $payment ) {
+	public function get_donor($payment) {
 
-		$donor_id           = give_get_payment_donor_id( $payment->ID );
-		$donor_billing_name = give_get_donor_name_by( $payment->ID, 'donation' );
-		$donor_name         = give_get_donor_name_by( $donor_id, 'donor' );
+		$donor_id           = give_get_payment_donor_id($payment->ID);
+		$donor_billing_name = give_get_donor_name_by($payment->ID, 'donation');
+		$donor_name         = give_get_donor_name_by($donor_id, 'donor');
 
 		$value = '';
-		if ( ! empty( $donor_id ) ) {
+		if ( ! empty($donor_id)) {
 
 			// Check whether the donor name and WP_User name is same or not.
-			if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
-				$value .= $donor_billing_name . ' (';
+			if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
+				$value .= $donor_billing_name.' (';
 			}
 
-			$value .= '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id" ) ) . '">' . $donor_name . '</a>';
+			$value .= '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id")).'">'.$donor_name.'</a>';
 
 			// Check whether the donor name and WP_User name is same or not.
-			if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
+			if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
 				$value .= ')';
 			}
 		} else {
-			$email = give_get_payment_user_email( $payment->ID );
-			$value .= '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-payment-history&s=$email" ) ) . '">' . esc_html__( '(donor missing)', 'give' ) . '</a>';
+			$email = give_get_payment_user_email($payment->ID);
+			$value .= '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-payment-history&s=$email")).'">'.esc_html__('(donor missing)', 'give').'</a>';
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'donor' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'donor');
 	}
 
 	/**
@@ -609,20 +609,20 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete'                 => __( 'Delete', 'give' ),
-			'set-status-publish'     => __( 'Set To Completed', 'give' ),
-			'set-status-pending'     => __( 'Set To Pending', 'give' ),
-			'set-status-processing'  => __( 'Set To Processing', 'give' ),
-			'set-status-refunded'    => __( 'Set To Refunded', 'give' ),
-			'set-status-revoked'     => __( 'Set To Revoked', 'give' ),
-			'set-status-failed'      => __( 'Set To Failed', 'give' ),
-			'set-status-cancelled'   => __( 'Set To Cancelled', 'give' ),
-			'set-status-abandoned'   => __( 'Set To Abandoned', 'give' ),
-			'set-status-preapproval' => __( 'Set To Preapproval', 'give' ),
-			'resend-receipt'         => __( 'Resend Email Receipts', 'give' ),
+			'delete'                 => __('Delete', 'give'),
+			'set-status-publish'     => __('Set To Completed', 'give'),
+			'set-status-pending'     => __('Set To Pending', 'give'),
+			'set-status-processing'  => __('Set To Processing', 'give'),
+			'set-status-refunded'    => __('Set To Refunded', 'give'),
+			'set-status-revoked'     => __('Set To Revoked', 'give'),
+			'set-status-failed'      => __('Set To Failed', 'give'),
+			'set-status-cancelled'   => __('Set To Cancelled', 'give'),
+			'set-status-abandoned'   => __('Set To Abandoned', 'give'),
+			'set-status-preapproval' => __('Set To Preapproval', 'give'),
+			'resend-receipt'         => __('Resend Email Receipts', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_bulk_actions', $actions );
+		return apply_filters('give_payments_table_bulk_actions', $actions);
 	}
 
 	/**
@@ -633,63 +633,63 @@ 
 	 * @return void
 	 */
 	public function process_bulk_action() {
-		$ids    = isset( $_GET['payment'] ) ? $_GET['payment'] : false;
+		$ids    = isset($_GET['payment']) ? $_GET['payment'] : false;
 		$action = $this->current_action();
 
-		if ( ! is_array( $ids ) ) {
-			$ids = array( $ids );
+		if ( ! is_array($ids)) {
+			$ids = array($ids);
 		}
 
-		if ( empty( $action ) ) {
+		if (empty($action)) {
 			return;
 		}
 
-		foreach ( $ids as $id ) {
+		foreach ($ids as $id) {
 
 			// Detect when a bulk action is being triggered.
-			switch ( $this->current_action() ) {
+			switch ($this->current_action()) {
 
 				case'delete':
-					give_delete_donation( $id );
+					give_delete_donation($id);
 					break;
 
 				case 'set-status-publish':
-					give_update_payment_status( $id, 'publish' );
+					give_update_payment_status($id, 'publish');
 					break;
 
 				case 'set-status-pending':
-					give_update_payment_status( $id, 'pending' );
+					give_update_payment_status($id, 'pending');
 					break;
 
 				case 'set-status-processing':
-					give_update_payment_status( $id, 'processing' );
+					give_update_payment_status($id, 'processing');
 					break;
 
 				case 'set-status-refunded':
-					give_update_payment_status( $id, 'refunded' );
+					give_update_payment_status($id, 'refunded');
 					break;
 				case 'set-status-revoked':
-					give_update_payment_status( $id, 'revoked' );
+					give_update_payment_status($id, 'revoked');
 					break;
 
 				case 'set-status-failed':
-					give_update_payment_status( $id, 'failed' );
+					give_update_payment_status($id, 'failed');
 					break;
 
 				case 'set-status-cancelled':
-					give_update_payment_status( $id, 'cancelled' );
+					give_update_payment_status($id, 'cancelled');
 					break;
 
 				case 'set-status-abandoned':
-					give_update_payment_status( $id, 'abandoned' );
+					give_update_payment_status($id, 'abandoned');
 					break;
 
 				case 'set-status-preapproval':
-					give_update_payment_status( $id, 'preapproval' );
+					give_update_payment_status($id, 'preapproval');
 					break;
 
 				case 'resend-receipt':
-					give_email_donation_receipt( $id, false );
+					give_email_donation_receipt($id, false);
 					break;
 			}// End switch().
 
@@ -701,7 +701,7 @@ 
 			 * @param int    $id             The ID of the payment.
 			 * @param string $current_action The action that is being triggered.
 			 */
-			do_action( 'give_payments_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_payments_table_do_bulk_action', $id, $this->current_action());
 		}// End foreach().
 
 	}
@@ -717,31 +717,31 @@ 
 
 		$args = array();
 
-		if ( isset( $_GET['user'] ) ) {
-			$args['user'] = urldecode( $_GET['user'] );
-		} elseif ( isset( $_GET['donor'] ) ) {
-			$args['donor'] = absint( $_GET['donor'] );
-		} elseif ( isset( $_GET['s'] ) ) {
-			$is_user = strpos( $_GET['s'], strtolower( 'user:' ) ) !== false;
-			if ( $is_user ) {
-				$args['user'] = absint( trim( str_replace( 'user:', '', strtolower( $_GET['s'] ) ) ) );
-				unset( $args['s'] );
+		if (isset($_GET['user'])) {
+			$args['user'] = urldecode($_GET['user']);
+		} elseif (isset($_GET['donor'])) {
+			$args['donor'] = absint($_GET['donor']);
+		} elseif (isset($_GET['s'])) {
+			$is_user = strpos($_GET['s'], strtolower('user:')) !== false;
+			if ($is_user) {
+				$args['user'] = absint(trim(str_replace('user:', '', strtolower($_GET['s']))));
+				unset($args['s']);
 			} else {
-				$args['s'] = sanitize_text_field( $_GET['s'] );
+				$args['s'] = sanitize_text_field($_GET['s']);
 			}
 		}
 
-		if ( ! empty( $_GET['start-date'] ) ) {
-			$args['start-date'] = urldecode( $_GET['start-date'] );
+		if ( ! empty($_GET['start-date'])) {
+			$args['start-date'] = urldecode($_GET['start-date']);
 		}
 
-		if ( ! empty( $_GET['end-date'] ) ) {
-			$args['end-date'] = urldecode( $_GET['end-date'] );
+		if ( ! empty($_GET['end-date'])) {
+			$args['end-date'] = urldecode($_GET['end-date']);
 		}
 
-		$args['form_id'] = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
+		$args['form_id'] = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
 
-		$payment_count           = give_count_payments( $args );
+		$payment_count           = give_count_payments($args);
 		$this->complete_count    = $payment_count->publish;
 		$this->pending_count     = $payment_count->pending;
 		$this->processing_count  = $payment_count->processing;
@@ -752,7 +752,7 @@ 
 		$this->abandoned_count   = $payment_count->abandoned;
 		$this->preapproval_count = $payment_count->preapproval;
 
-		foreach ( $payment_count as $count ) {
+		foreach ($payment_count as $count) {
 			$this->total_count += $count;
 		}
 	}
@@ -767,28 +767,28 @@ 
 	public function payments_data() {
 
 		$per_page   = $this->per_page;
-		$orderby    = isset( $_GET['orderby'] ) ? urldecode( $_GET['orderby'] ) : 'ID';
-		$order      = isset( $_GET['order'] ) ? $_GET['order'] : 'DESC';
-		$user       = isset( $_GET['user'] ) ? $_GET['user'] : null;
-		$donor      = isset( $_GET['donor'] ) ? $_GET['donor'] : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : give_get_payment_status_keys();
-		$meta_key   = isset( $_GET['meta_key'] ) ? $_GET['meta_key'] : null;
-		$year       = isset( $_GET['year'] ) ? $_GET['year'] : null;
-		$month      = isset( $_GET['m'] ) ? $_GET['m'] : null;
-		$day        = isset( $_GET['day'] ) ? $_GET['day'] : null;
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : null;
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : $start_date;
-		$form_id    = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
-
-		if ( ! empty( $search ) ) {
+		$orderby    = isset($_GET['orderby']) ? urldecode($_GET['orderby']) : 'ID';
+		$order      = isset($_GET['order']) ? $_GET['order'] : 'DESC';
+		$user       = isset($_GET['user']) ? $_GET['user'] : null;
+		$donor      = isset($_GET['donor']) ? $_GET['donor'] : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : give_get_payment_status_keys();
+		$meta_key   = isset($_GET['meta_key']) ? $_GET['meta_key'] : null;
+		$year       = isset($_GET['year']) ? $_GET['year'] : null;
+		$month      = isset($_GET['m']) ? $_GET['m'] : null;
+		$day        = isset($_GET['day']) ? $_GET['day'] : null;
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : null;
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : $start_date;
+		$form_id    = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
+
+		if ( ! empty($search)) {
 			$status = 'any'; // Force all payment statuses when searching.
 		}
 
 		$args = array(
 			'output'     => 'payments',
 			'number'     => $per_page,
-			'page'       => isset( $_GET['paged'] ) ? $_GET['paged'] : null,
+			'page'       => isset($_GET['paged']) ? $_GET['paged'] : null,
 			'orderby'    => $orderby,
 			'order'      => $order,
 			'user'       => $user,
@@ -804,12 +804,12 @@ 
 			'give_forms' => $form_id,
 		);
 
-		if ( is_string( $search ) && false !== strpos( $search, 'txn:' ) ) {
+		if (is_string($search) && false !== strpos($search, 'txn:')) {
 			$args['search_in_notes'] = true;
-			$args['s']               = trim( str_replace( 'txn:', '', $args['s'] ) );
+			$args['s']               = trim(str_replace('txn:', '', $args['s']));
 		}
 
-		$p_query = new Give_Payments_Query( $args );
+		$p_query = new Give_Payments_Query($args);
 
 		return $p_query->get_payments();
 
@@ -829,17 +829,17 @@ 
 	 */
 	public function prepare_items() {
 
-		wp_reset_vars( array( 'action', 'payment', 'orderby', 'order', 's' ) );
+		wp_reset_vars(array('action', 'payment', 'orderby', 'order', 's'));
 
 		$columns  = $this->get_columns();
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 		$data     = $this->payments_data();
-		$status   = isset( $_GET['status'] ) ? $_GET['status'] : 'any';
+		$status   = isset($_GET['status']) ? $_GET['status'] : 'any';
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'publish':
 				$total_items = $this->complete_count;
 				break;
@@ -872,20 +872,20 @@ 
 				break;
 			default:
 				// Retrieve the count of the non-default-Give status.
-				$count       = wp_count_posts( 'give_payment' );
-				$total_items = isset( $count->{$status} ) ? $count->{$status} : 0;
+				$count       = wp_count_posts('give_payment');
+				$total_items = isset($count->{$status} ) ? $count->{$status} : 0;
 				break;
 		}
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $total_items,
 			// We have to calculate the total number of items.
 			'per_page'    => $this->per_page,
 			// We have to determine how many items to show on a page.
-			'total_pages' => ceil( $total_items / $this->per_page ),
+			'total_pages' => ceil($total_items / $this->per_page),
 			// We have to calculate the total number of pages.
-		) );
+		));
 	}
 }

includes/admin/class-i18n-module.php

Spacing +46 added lines, -46 removed lines

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

includes/payments/actions.php

Spacing +57 added lines, -57 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,21 +28,21 @@ 
  *
  * @return void
  */
-function give_complete_purchase( $payment_id, $new_status, $old_status ) {
+function give_complete_purchase($payment_id, $new_status, $old_status) {
 
 	// Make sure that payments are only completed once.
-	if ( $old_status == 'publish' || $old_status == 'complete' ) {
+	if ($old_status == 'publish' || $old_status == 'complete') {
 		return;
 	}
 
 	// Make sure the payment completion is only processed when new status is complete.
-	if ( $new_status != 'publish' && $new_status != 'complete' ) {
+	if ($new_status != 'publish' && $new_status != 'complete') {
 		return;
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$creation_date  = get_post_field( 'post_date', $payment_id, 'raw' );
+	$creation_date  = get_post_field('post_date', $payment_id, 'raw');
 	$payment_meta   = $payment->payment_meta;
 	$completed_date = $payment->completed_date;
 	$user_info      = $payment->user_info;
@@ -58,12 +58,12 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_pre_complete_donation', $payment_id );
+	do_action('give_pre_complete_donation', $payment_id);
 
 	// Ensure these actions only run once, ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
-		give_record_donation_in_log( $form_id, $payment_id, $price_id, $creation_date );
+		give_record_donation_in_log($form_id, $payment_id, $price_id, $creation_date);
 
 		/**
 		 * Fires after logging donation record.
@@ -74,29 +74,29 @@ 
 		 * @param int $payment_id The ID number of the payment.
 		 * @param array $payment_meta The payment meta.
 		 */
-		do_action( 'give_complete_form_donation', $form_id, $payment_id, $payment_meta );
+		do_action('give_complete_form_donation', $form_id, $payment_id, $payment_meta);
 
 	}
 
 	// Increase the earnings for this form ID.
-	give_increase_earnings( $form_id, $amount );
-	give_increase_donation_count( $form_id );
+	give_increase_earnings($form_id, $amount);
+	give_increase_donation_count($form_id);
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 
 	// Increase the donor's donation stats.
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 	$donor->increase_purchase_count();
-	$donor->increase_value( $amount );
+	$donor->increase_value($amount);
 
-	give_increase_total_earnings( $amount );
+	give_increase_total_earnings($amount);
 
 	// Ensure this action only runs once ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
 		// Save the completed date.
-		$payment->completed_date = current_time( 'mysql' );
+		$payment->completed_date = current_time('mysql');
 		$payment->save();
 
 		/**
@@ -106,12 +106,12 @@ 
 		 *
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_complete_donation', $payment_id );
+		do_action('give_complete_donation', $payment_id);
 	}
 
 }
 
-add_action( 'give_update_payment_status', 'give_complete_purchase', 100, 3 );
+add_action('give_update_payment_status', 'give_complete_purchase', 100, 3);
 
 
 /**
@@ -125,20 +125,20 @@ 
  *
  * @return void
  */
-function give_record_status_change( $payment_id, $new_status, $old_status ) {
+function give_record_status_change($payment_id, $new_status, $old_status) {
 
 	// Get the list of statuses so that status in the payment note can be translated.
 	$stati      = give_get_payment_statuses();
-	$old_status = isset( $stati[ $old_status ] ) ? $stati[ $old_status ] : $old_status;
-	$new_status = isset( $stati[ $new_status ] ) ? $stati[ $new_status ] : $new_status;
+	$old_status = isset($stati[$old_status]) ? $stati[$old_status] : $old_status;
+	$new_status = isset($stati[$new_status]) ? $stati[$new_status] : $new_status;
 
 	// translators: 1: old status 2: new status.
-	$status_change = sprintf( esc_html__( 'Status changed from %1$s to %2$s.', 'give' ), $old_status, $new_status );
+	$status_change = sprintf(esc_html__('Status changed from %1$s to %2$s.', 'give'), $old_status, $new_status);
 
-	give_insert_payment_note( $payment_id, $status_change );
+	give_insert_payment_note($payment_id, $status_change);
 }
 
-add_action( 'give_update_payment_status', 'give_record_status_change', 100, 3 );
+add_action('give_update_payment_status', 'give_record_status_change', 100, 3);
 
 
 /**
@@ -154,25 +154,25 @@ 
  *
  * @return void
  */
-function give_update_old_payments_with_totals( $data ) {
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_upgrade_payments_nonce' ) ) {
+function give_update_old_payments_with_totals($data) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_upgrade_payments_nonce')) {
 		return;
 	}
 
-	if ( get_option( 'give_payment_totals_upgraded' ) ) {
+	if (get_option('give_payment_totals_upgraded')) {
 		return;
 	}
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'offset' => 0,
-		'number' => - 1,
+		'number' => -1,
 		'mode'   => 'all',
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 
-			$payment = new Give_Payment( $payment->ID );
+			$payment = new Give_Payment($payment->ID);
 			$meta    = $payment->get_meta();
 
 			$payment->total = $meta['amount'];
@@ -181,10 +181,10 @@ 
 		}
 	}
 
-	add_option( 'give_payment_totals_upgraded', 1 );
+	add_option('give_payment_totals_upgraded', 1);
 }
 
-add_action( 'give_upgrade_payments', 'give_update_old_payments_with_totals' );
+add_action('give_upgrade_payments', 'give_update_old_payments_with_totals');
 
 /**
  * Mark Abandoned Donations
@@ -198,17 +198,17 @@ 
 function give_mark_abandoned_donations() {
 	$args = array(
 		'status' => 'pending',
-		'number' => - 1,
+		'number' => -1,
 		'output' => 'give_payments',
 	);
 
-	add_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	add_filter('posts_where', 'give_filter_where_older_than_week');
 
-	$payments = give_get_payments( $args );
+	$payments = give_get_payments($args);
 
-	remove_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	remove_filter('posts_where', 'give_filter_where_older_than_week');
 
-	if ( $payments ) {
+	if ($payments) {
 		/**
 		 * Filter payment gateways:  Used to set payment gateways which can be skip while transferring pending payment to abandon.
 		 *
@@ -216,13 +216,13 @@ 
 		 *
 		 * @param array $skip_payment_gateways Array of payment gateways
 		 */
-		$skip_payment_gateways = apply_filters( 'give_mark_abandoned_donation_gateways', array( 'offline' ) );
+		$skip_payment_gateways = apply_filters('give_mark_abandoned_donation_gateways', array('offline'));
 
-		foreach ( $payments as $payment ) {
-			$gateway = give_get_payment_gateway( $payment );
+		foreach ($payments as $payment) {
+			$gateway = give_get_payment_gateway($payment);
 
 			// Skip payment gateways.
-			if ( in_array( $gateway, $skip_payment_gateways ) ) {
+			if (in_array($gateway, $skip_payment_gateways)) {
 				continue;
 			}
 
@@ -232,7 +232,7 @@ 
 	}
 }
 
-Give_Cron::add_weekly_event( 'give_mark_abandoned_donations' );
+Give_Cron::add_weekly_event('give_mark_abandoned_donations');
 
 
 /**
@@ -244,15 +244,15 @@ 
  *
  * @return void
  */
-function give_refresh_thismonth_stat_transients( $payment_ID ) {
+function give_refresh_thismonth_stat_transients($payment_ID) {
 	// Monthly stats.
-	Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+	Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 }
 
-add_action( 'save_post_give_payment', 'give_refresh_thismonth_stat_transients' );
+add_action('save_post_give_payment', 'give_refresh_thismonth_stat_transients');
 
 /*
  * Add meta in payment that store page id and page url.
@@ -264,18 +264,18 @@ 
  *
  * @param {integer} $payment_id Payment id for which the meta value should be updated.
  */
-function give_payment_save_page_data( $payment_id ) {
-	$page_url = ( ! empty( $_REQUEST['give-current-url'] ) ? esc_url( $_REQUEST['give-current-url'] ) : false );
+function give_payment_save_page_data($payment_id) {
+	$page_url = ( ! empty($_REQUEST['give-current-url']) ? esc_url($_REQUEST['give-current-url']) : false);
 	// Check $page_url is not empty.
-	if ( $page_url ) {
-		update_post_meta( $payment_id, '_give_current_url', $page_url );
-		$page_id = url_to_postid( $page_url );
+	if ($page_url) {
+		update_post_meta($payment_id, '_give_current_url', $page_url);
+		$page_id = url_to_postid($page_url);
 		// Check $page_id is not empty.
-		if ( $page_id ) {
-			update_post_meta( $payment_id, '_give_current_page_id', $page_id );
+		if ($page_id) {
+			update_post_meta($payment_id, '_give_current_page_id', $page_id);
 		}
 	}
 }
 
 // Fire when payment is save.
-add_action( 'give_insert_payment', 'give_payment_save_page_data' );
\ No newline at end of file
+add_action('give_insert_payment', 'give_payment_save_page_data');
\ No newline at end of file

includes/class-give-cron.php

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -51,7 +51,7 @@ 
 	 * @return static
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->setup();
 		}
@@ -66,12 +66,12 @@ 
 	 * @since 1.8.13
 	 */
 	private function setup() {
-		add_filter( 'cron_schedules', array( self::$instance, '__add_schedules' ) );
-		add_action( 'wp', array( self::$instance, '__schedule_events' ) );
+		add_filter('cron_schedules', array(self::$instance, '__add_schedules'));
+		add_action('wp', array(self::$instance, '__schedule_events'));
 
 		// Load async event only when cron is running.
-		if( defined( 'DOING_CRON' ) && DOING_CRON ) {
-			add_action( 'init', array( self::$instance, '__load_async_events' ) );
+		if (defined('DOING_CRON') && DOING_CRON) {
+			add_action('init', array(self::$instance, '__load_async_events'));
 		}
 	}
 
@@ -82,23 +82,23 @@ 
 	 * @since 1.8.13
 	 */
 	public function __load_async_events() {
-		$async_events = get_option( 'give_async_events', array() );
+		$async_events = get_option('give_async_events', array());
 
 		// Bailout.
-		if ( empty( $async_events ) ) {
+		if (empty($async_events)) {
 			return;
 		}
 
-		foreach ( $async_events as $index => $event ) {
+		foreach ($async_events as $index => $event) {
 			// Set cron name.
 			$cron_name = "give_async_scheduled_events_{$index}";
 
 			// Setup cron.
-			wp_schedule_single_event( current_time( 'timestamp', 1 ), $cron_name, $event['params'] );
+			wp_schedule_single_event(current_time('timestamp', 1), $cron_name, $event['params']);
 
 			// Add cron action.
-			add_action( $cron_name, $event['callback'], 10, count( $event['params'] ) );
-			add_action( $cron_name, array( $this, '__delete_async_events' ), 9999, 0 );
+			add_action($cron_name, $event['callback'], 10, count($event['params']));
+			add_action($cron_name, array($this, '__delete_async_events'), 9999, 0);
 		}
 	}
 
@@ -108,13 +108,13 @@ 
 	 * @since 1.8.13
 	 */
 	public function __delete_async_events() {
-		$async_events    = get_option( 'give_async_events', array() );
-		$cron_name_parts = explode( '_', current_action() );
-		$cron_id         = end( $cron_name_parts );
+		$async_events    = get_option('give_async_events', array());
+		$cron_name_parts = explode('_', current_action());
+		$cron_id         = end($cron_name_parts);
 
-		if ( ! empty( $async_events[ $cron_id ] ) ) {
-			unset( $async_events[ $cron_id ] );
-			update_option( 'give_async_events', $async_events );
+		if ( ! empty($async_events[$cron_id])) {
+			unset($async_events[$cron_id]);
+			update_option('give_async_events', $async_events);
 		}
 	}
 
@@ -128,11 +128,11 @@ 
 	 *
 	 * @return array            An array of non-default cron schedules.
 	 */
-	public function __add_schedules( $schedules = array() ) {
+	public function __add_schedules($schedules = array()) {
 		// Adds once weekly to the existing schedules.
 		$schedules['weekly'] = array(
 			'interval' => 604800,
-			'display'  => __( 'Once Weekly', 'give' ),
+			'display'  => __('Once Weekly', 'give'),
 		);
 
 		return $schedules;
@@ -160,8 +160,8 @@ 
 	 * @return void
 	 */
 	private function weekly_events() {
-		if ( ! wp_next_scheduled( 'give_weekly_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'weekly', 'give_weekly_scheduled_events' );
+		if ( ! wp_next_scheduled('give_weekly_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'weekly', 'give_weekly_scheduled_events');
 		}
 	}
 
@@ -174,8 +174,8 @@ 
 	 * @return void
 	 */
 	private function daily_events() {
-		if ( ! wp_next_scheduled( 'give_daily_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'daily', 'give_daily_scheduled_events' );
+		if ( ! wp_next_scheduled('give_daily_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'daily', 'give_daily_scheduled_events');
 		}
 	}
 
@@ -189,8 +189,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_cron_action( $type = 'weekly' ) {
-		switch ( $type ) {
+	public static function get_cron_action($type = 'weekly') {
+		switch ($type) {
 			case 'daily':
 				$cron_action = 'give_daily_scheduled_events';
 				break;
@@ -212,9 +212,9 @@ 
 	 * @param        $action
 	 * @param string $type
 	 */
-	private static function add_event( $action, $type = 'weekly' ) {
-		$cron_event = self::get_cron_action( $type );
-		add_action( $cron_event, $action );
+	private static function add_event($action, $type = 'weekly') {
+		$cron_event = self::get_cron_action($type);
+		add_action($cron_event, $action);
 	}
 
 	/**
@@ -225,8 +225,8 @@ 
 	 *
 	 * @param $action
 	 */
-	public static function add_weekly_event( $action ) {
-		self::add_event( $action, 'weekly' );
+	public static function add_weekly_event($action) {
+		self::add_event($action, 'weekly');
 	}
 
 	/**
@@ -237,8 +237,8 @@ 
 	 *
 	 * @param $action
 	 */
-	public static function add_daily_event( $action ) {
-		self::add_event( $action, 'daily' );
+	public static function add_daily_event($action) {
+		self::add_event($action, 'daily');
 	}
 
 	/**
@@ -251,17 +251,17 @@ 
 	 * @param string $action
 	 * @param array  $args
 	 */
-	public static function add_async_event( $action, $args = array() ) {
+	public static function add_async_event($action, $args = array()) {
 
 		// Cache async events.
-		$async_events             = get_option( 'give_async_events', array() );
-		$async_events[ uniqid() ] = array(
+		$async_events             = get_option('give_async_events', array());
+		$async_events[uniqid()] = array(
 			'callback' => $action,
 			'params'   => $args,
 		);
 
 
-		update_option( 'give_async_events', $async_events );
+		update_option('give_async_events', $async_events);
 	}
 }