ravinderk / Give

includes/class-give-stats.php

Spacing +106 added lines, -106 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -85,19 +85,19 @@ 
 	 */
 	public function get_predefined_dates() {
 		$predefined = array(
-			'today'        => esc_html__( 'Today', 'give' ),
-			'yesterday'    => esc_html__( 'Yesterday', 'give' ),
-			'this_week'    => esc_html__( 'This Week', 'give' ),
-			'last_week'    => esc_html__( 'Last Week', 'give' ),
-			'this_month'   => esc_html__( 'This Month', 'give' ),
-			'last_month'   => esc_html__( 'Last Month', 'give' ),
-			'this_quarter' => esc_html__( 'This Quarter', 'give' ),
-			'last_quarter' => esc_html__( 'Last Quarter', 'give' ),
-			'this_year'    => esc_html__( 'This Year', 'give' ),
-			'last_year'    => esc_html__( 'Last Year', 'give' )
+			'today'        => esc_html__('Today', 'give'),
+			'yesterday'    => esc_html__('Yesterday', 'give'),
+			'this_week'    => esc_html__('This Week', 'give'),
+			'last_week'    => esc_html__('Last Week', 'give'),
+			'this_month'   => esc_html__('This Month', 'give'),
+			'last_month'   => esc_html__('Last Month', 'give'),
+			'this_quarter' => esc_html__('This Quarter', 'give'),
+			'last_quarter' => esc_html__('Last Quarter', 'give'),
+			'this_year'    => esc_html__('This Year', 'give'),
+			'last_year'    => esc_html__('Last Year', 'give')
 		);
 
-		return apply_filters( 'give_stats_predefined_dates', $predefined );
+		return apply_filters('give_stats_predefined_dates', $predefined);
 	}
 
 	/**
@@ -113,18 +113,18 @@ 
 	 *
 	 * @return void
 	 */
-	public function setup_dates( $_start_date = 'this_month', $_end_date = false ) {
+	public function setup_dates($_start_date = 'this_month', $_end_date = false) {
 
-		if ( empty( $_start_date ) ) {
+		if (empty($_start_date)) {
 			$_start_date = 'this_month';
 		}
 
-		if ( empty( $_end_date ) ) {
+		if (empty($_end_date)) {
 			$_end_date = $_start_date;
 		}
 
-		$this->start_date = $this->convert_date( $_start_date );
-		$this->end_date   = $this->convert_date( $_end_date, true );
+		$this->start_date = $this->convert_date($_start_date);
+		$this->end_date   = $this->convert_date($_end_date, true);
 	}
 
 	/**
@@ -140,26 +140,26 @@ 
 	 *
 	 * @return array|WP_Error   If the date is invalid, a WP_Error object will be returned.
 	 */
-	public function convert_date( $date, $end_date = false ) {
+	public function convert_date($date, $end_date = false) {
 
 		$this->timestamp = false;
 		$second          = $end_date ? 59 : 0;
 		$minute          = $end_date ? 59 : 0;
 		$hour            = $end_date ? 23 : 0;
 		$day             = 1;
-		$month           = date( 'n', current_time( 'timestamp' ) );
-		$year            = date( 'Y', current_time( 'timestamp' ) );
+		$month           = date('n', current_time('timestamp'));
+		$year            = date('Y', current_time('timestamp'));
 
-		if ( array_key_exists( $date, $this->get_predefined_dates() ) ) {
+		if (array_key_exists($date, $this->get_predefined_dates())) {
 
 			// This is a predefined date rate, such as last_week
-			switch ( $date ) {
+			switch ($date) {
 
 				case 'this_month' :
 
-					if ( $end_date ) {
+					if ($end_date) {
 
-						$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -169,28 +169,28 @@ 
 
 				case 'last_month' :
 
-					if ( $month == 1 ) {
+					if ($month == 1) {
 
 						$month = 12;
-						$year --;
+						$year--;
 
 					} else {
 
-						$month --;
+						$month--;
 
 					}
 
-					if ( $end_date ) {
-						$day = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+					if ($end_date) {
+						$day = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 					}
 
 					break;
 
 				case 'today' :
 
-					$day = date( 'd', current_time( 'timestamp' ) );
+					$day = date('d', current_time('timestamp'));
 
-					if ( $end_date ) {
+					if ($end_date) {
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -200,23 +200,23 @@ 
 
 				case 'yesterday' :
 
-					$day = date( 'd', current_time( 'timestamp' ) ) - 1;
+					$day = date('d', current_time('timestamp')) - 1;
 
 					// Check if Today is the first day of the month (meaning subtracting one will get us 0)
-					if ( $day < 1 ) {
+					if ($day < 1) {
 
 						// If current month is 1
-						if ( 1 == $month ) {
+						if (1 == $month) {
 
 							$year -= 1; // Today is January 1, so skip back to last day of December
 							$month = 12;
-							$day   = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day   = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
 						} else {
 
 							// Go back one month and get the last day of the month
 							$month -= 1;
-							$day = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
 						}
 					}
@@ -225,12 +225,12 @@ 
 
 				case 'this_week' :
 
-					$days_to_week_start = ( date( 'w', current_time( 'timestamp' ) ) - 1 ) * 60 * 60 * 24;
-					$today              = date( 'd', current_time( 'timestamp' ) ) * 60 * 60 * 24;
+					$days_to_week_start = (date('w', current_time('timestamp')) - 1) * 60 * 60 * 24;
+					$today              = date('d', current_time('timestamp')) * 60 * 60 * 24;
 
-					if ( $today < $days_to_week_start ) {
+					if ($today < $days_to_week_start) {
 
-						if ( $month > 1 ) {
+						if ($month > 1) {
 							$month -= 1;
 						} else {
 							$month = 12;
@@ -238,19 +238,19 @@ 
 
 					}
 
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 
 						// Getting the start day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 1;
-						$day += get_option( 'start_of_week' );
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 1;
+						$day += get_option('start_of_week');
 
 					} else {
 
 						// Getting the end day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 1;
-						$day += get_option( 'start_of_week' ) + 6;
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 1;
+						$day += get_option('start_of_week') + 6;
 
 					}
 
@@ -258,12 +258,12 @@ 
 
 				case 'last_week' :
 
-					$days_to_week_start = ( date( 'w', current_time( 'timestamp' ) ) - 1 ) * 60 * 60 * 24;
-					$today              = date( 'd', current_time( 'timestamp' ) ) * 60 * 60 * 24;
+					$days_to_week_start = (date('w', current_time('timestamp')) - 1) * 60 * 60 * 24;
+					$today              = date('d', current_time('timestamp')) * 60 * 60 * 24;
 
-					if ( $today < $days_to_week_start ) {
+					if ($today < $days_to_week_start) {
 
-						if ( $month > 1 ) {
+						if ($month > 1) {
 							$month -= 1;
 						} else {
 							$month = 12;
@@ -271,19 +271,19 @@ 
 
 					}
 
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 
 						// Getting the start day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 8;
-						$day += get_option( 'start_of_week' );
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 8;
+						$day += get_option('start_of_week');
 
 					} else {
 
 						// Getting the end day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 8;
-						$day += get_option( 'start_of_week' ) + 6;
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 8;
+						$day += get_option('start_of_week') + 6;
 
 					}
 
@@ -291,39 +291,39 @@ 
 
 				case 'this_quarter' :
 
-					$month_now = date( 'n', current_time( 'timestamp' ) );
+					$month_now = date('n', current_time('timestamp'));
 
-					if ( $month_now <= 3 ) {
+					if ($month_now <= 3) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 1;
 						} else {
 							$month  = 3;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 6 ) {
+					} else if ($month_now <= 6) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 4;
 						} else {
 							$month  = 6;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 9 ) {
+					} else if ($month_now <= 9) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 7;
 						} else {
 							$month  = 9;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -331,11 +331,11 @@ 
 
 					} else {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 10;
 						} else {
 							$month  = 12;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -347,40 +347,40 @@ 
 
 				case 'last_quarter' :
 
-					$month_now = date( 'n', current_time( 'timestamp' ) );
+					$month_now = date('n', current_time('timestamp'));
 
-					if ( $month_now <= 3 ) {
+					if ($month_now <= 3) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 10;
 						} else {
 							$year -= 1;
 							$month  = 12;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 6 ) {
+					} else if ($month_now <= 6) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 1;
 						} else {
 							$month  = 3;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 9 ) {
+					} else if ($month_now <= 9) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 4;
 						} else {
 							$month  = 6;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -388,11 +388,11 @@ 
 
 					} else {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 7;
 						} else {
 							$month  = 9;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -404,11 +404,11 @@ 
 
 				case 'this_year' :
 
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 						$month = 1;
 					} else {
 						$month  = 12;
-						$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -419,11 +419,11 @@ 
 				case 'last_year' :
 
 					$year -= 1;
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 						$month = 1;
 					} else {
 						$month  = 12;
-						$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -434,30 +434,30 @@ 
 			}
 
 
-		} else if ( is_numeric( $date ) ) {
+		} else if (is_numeric($date)) {
 
 			// return $date unchanged since it is a timestamp
 			$this->timestamp = true;
 
-		} else if ( false !== strtotime( $date ) ) {
+		} else if (false !== strtotime($date)) {
 
-			$date  = strtotime( $date, current_time( 'timestamp' ) );
-			$year  = date( 'Y', $date );
-			$month = date( 'm', $date );
-			$day   = date( 'd', $date );
+			$date  = strtotime($date, current_time('timestamp'));
+			$year  = date('Y', $date);
+			$month = date('m', $date);
+			$day   = date('d', $date);
 
 		} else {
 
-			return new WP_Error( 'invalid_date', esc_html__( 'Improper date provided.', 'give' ) );
+			return new WP_Error('invalid_date', esc_html__('Improper date provided.', 'give'));
 
 		}
 
-		if ( false === $this->timestamp ) {
+		if (false === $this->timestamp) {
 			// Create an exact timestamp
-			$date = mktime( $hour, $minute, $second, $month, $day, $year );
+			$date = mktime($hour, $minute, $second, $month, $day, $year);
 		}
 
-		return apply_filters( 'give_stats_date', $date, $end_date, $this );
+		return apply_filters('give_stats_date', $date, $end_date, $this);
 
 	}
 
@@ -473,33 +473,33 @@ 
 	 * 
 	 * @return string
 	 */
-	public function count_where( $where = '' ) {
+	public function count_where($where = '') {
 		// Only get payments in our date range
 
 		$start_where = '';
 		$end_where   = '';
 
-		if ( $this->start_date ) {
+		if ($this->start_date) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 00:00:00';
 			}
 
-			$start_date  = date( $format, $this->start_date );
+			$start_date  = date($format, $this->start_date);
 			$start_where = " AND p.post_date >= '{$start_date}'";
 		}
 
-		if ( $this->end_date ) {
+		if ($this->end_date) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 23:59:59';
 			}
 
-			$end_date = date( $format, $this->end_date );
+			$end_date = date($format, $this->end_date);
 
 			$end_where = " AND p.post_date <= '{$end_date}'";
 		}
@@ -521,34 +521,34 @@ 
 	 *
 	 * @return string
 	 */
-	public function payments_where( $where = '' ) {
+	public function payments_where($where = '') {
 
 		global $wpdb;
 
 		$start_where = '';
 		$end_where   = '';
 
-		if ( ! is_wp_error( $this->start_date ) ) {
+		if ( ! is_wp_error($this->start_date)) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 00:00:00';
 			}
 
-			$start_date  = date( $format, $this->start_date );
+			$start_date  = date($format, $this->start_date);
 			$start_where = " AND $wpdb->posts.post_date >= '{$start_date}'";
 		}
 
-		if ( ! is_wp_error( $this->end_date ) ) {
+		if ( ! is_wp_error($this->end_date)) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d 00:00:00';
 			} else {
 				$format = 'Y-m-d 23:59:59';
 			}
 
-			$end_date = date( $format, $this->end_date );
+			$end_date = date($format, $this->end_date);
 
 			$end_where = " AND $wpdb->posts.post_date <= '{$end_date}'";
 		}

includes/class-give-template-loader.php

Spacing +25 added lines, -25 removed lines

@@ -10,7 +10,7 @@ 
  */

 

 // Exit if accessed directly

-if ( ! defined( 'ABSPATH' ) ) {

+if ( ! defined('ABSPATH')) {

 	exit;

 }

 

@@ -38,29 +38,29 @@ 
 		/**

 		 * Templates

 		 */

-		add_filter( 'template_include', array( __CLASS__, 'template_loader' ) );

+		add_filter('template_include', array(__CLASS__, 'template_loader'));

 

 		/**

 		 * Content Wrappers

 		 */

-		add_action( 'give_before_main_content', 'give_output_content_wrapper', 10 );

-		add_action( 'give_after_main_content', 'give_output_content_wrapper_end', 10 );

+		add_action('give_before_main_content', 'give_output_content_wrapper', 10);

+		add_action('give_after_main_content', 'give_output_content_wrapper_end', 10);

 

 		/**

 		 * Entry Summary Classes

 		 */

-		add_filter( 'give_forms_single_summary_classes', array( $this, 'give_set_single_summary_classes' ) );

+		add_filter('give_forms_single_summary_classes', array($this, 'give_set_single_summary_classes'));

 

 		/**

 		 * Sidebar

 		 */

-		add_action( 'give_before_single_form_summary', array( $this, 'give_output_sidebar_option' ), 1 );

+		add_action('give_before_single_form_summary', array($this, 'give_output_sidebar_option'), 1);

 

 		/**

 		 * Single Forms Summary Box

 		 */

-		add_action( 'give_single_form_summary', 'give_template_single_title', 5 );

-		add_action( 'give_single_form_summary', 'give_get_donation_form', 10 );

+		add_action('give_single_form_summary', 'give_template_single_title', 5);

+		add_action('give_single_form_summary', 'give_get_donation_form', 10);

 

 	}

 

@@ -75,12 +75,12 @@ 
 	 *

 	 * @return string $classes List of space separated class names.

 	 */

-	public function give_set_single_summary_classes( $classes ) {

+	public function give_set_single_summary_classes($classes) {

 

-		$sidebar_option = give_get_option( 'disable_form_sidebar' );

+		$sidebar_option = give_get_option('disable_form_sidebar');

 

 		//Add full width class when feature image is disabled AND no widgets are present

-		if ( $sidebar_option == 'on' ) {

+		if ($sidebar_option == 'on') {

 			$classes .= ' give-full-width';

 		}

 

@@ -100,14 +100,14 @@ 
 	 */

 	public function give_output_sidebar_option() {

 

-		$sidebar_option = give_get_option( 'disable_form_sidebar' );

+		$sidebar_option = give_get_option('disable_form_sidebar');

 

 		//Add full width class when feature image is disabled AND no widgets are present

-		if ( $sidebar_option !== 'on' ) {

-			add_action( 'give_before_single_form_summary', 'give_left_sidebar_pre_wrap', 5 );

-			add_action( 'give_before_single_form_summary', 'give_show_form_images', 10 );

-			add_action( 'give_before_single_form_summary', 'give_get_forms_sidebar', 20 );

-			add_action( 'give_before_single_form_summary', 'give_left_sidebar_post_wrap', 30 );

+		if ($sidebar_option !== 'on') {

+			add_action('give_before_single_form_summary', 'give_left_sidebar_pre_wrap', 5);

+			add_action('give_before_single_form_summary', 'give_show_form_images', 10);

+			add_action('give_before_single_form_summary', 'give_get_forms_sidebar', 20);

+			add_action('give_before_single_form_summary', 'give_left_sidebar_post_wrap', 30);

 		}

 

 	}

@@ -129,20 +129,20 @@ 
 	 *

 	 * @return string $template

 	 */

-	public static function template_loader( $template ) {

-		$find = array( 'give.php' );

+	public static function template_loader($template) {

+		$find = array('give.php');

 		$file = '';

 

-		if ( is_single() && get_post_type() == 'give_forms' ) {

+		if (is_single() && get_post_type() == 'give_forms') {

 			$file   = 'single-give-form.php';

 			$find[] = $file;

-			$find[] = apply_filters( 'give_template_path', 'give/' ) . $file;

+			$find[] = apply_filters('give_template_path', 'give/').$file;

 		}

 

-		if ( $file ) {

-			$template = locate_template( array_unique( $find ) );

-			if ( ! $template ) {

-				$template = GIVE_PLUGIN_DIR . '/templates/' . $file;

+		if ($file) {

+			$template = locate_template(array_unique($find));

+			if ( ! $template) {

+				$template = GIVE_PLUGIN_DIR.'/templates/'.$file;

 			}

 		}

 

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

Spacing +99 added lines, -99 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,26 +55,26 @@ 
 	public function get_data() {
 		global $give_logs, $wpdb;
 
-		$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');
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'post_parent__in' => $all_forms,
 			'post_type'       => 'give_log',
 			'posts_per_page'  => $this->per_step,
@@ -82,73 +82,73 @@ 
 			'paged'           => $this->step,
 			'log_type'        => 'sale',
 			'fields'          => 'ids',
-		) );
+		));
 
-		$log_ids = $give_logs->get_connected_logs( $args, 'sale' );
+		$log_ids = $give_logs->get_connected_logs($args, 'sale');
 
-		if ( $log_ids ) {
-			$log_ids = implode( ',', $log_ids );
+		if ($log_ids) {
+			$log_ids = implode(',', $log_ids);
 
-			$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-			unset( $log_ids );
+			$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+			unset($log_ids);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-			unset( $payment_ids );
+			$payment_ids = implode(',', $payment_ids);
+			$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+			unset($payment_ids);
 
 			//Loop through payments
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
 				// 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;
 				}
 
 				//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,
 					);
 				}
 
-				$totals[ $form_id ]['sales'] ++;
-				$totals[ $form_id ]['earnings'] += $payment_item['price'];
+				$totals[$form_id]['sales']++;
+				$totals[$form_id]['earnings'] += $payment_item['price'];
 
 				$processed_payments[] = $payment->ID;
 
 			}
 
-			$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 ) {
-			update_post_meta( $key, '_give_form_sales', $stats['sales'] );
-			update_post_meta( $key, '_give_form_earnings', $stats['earnings'] );
+		foreach ($totals as $key => $stats) {
+			update_post_meta($key, '_give_form_sales', $stats['sales']);
+			update_post_meta($key, '_give_form_earnings', $stats['earnings']);
 		}
 
 		return false;
@@ -163,20 +163,20 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_all_total', false );
+		$total = $this->get_stored_data('give_recount_all_total', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$this->pre_fetch();
-			$total = $this->get_stored_data( 'give_recount_all_total', 0 );
+			$total = $this->get_stored_data('give_recount_all_total', 0);
 		}
 
 		$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;
 		}
 
@@ -190,8 +190,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;
 	}
 
 	/**
@@ -202,34 +202,34 @@ 
 	 */
 	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_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_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;
 		}
 	}
 
 	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);
 		}
 	}
 
@@ -258,76 +258,76 @@ 
 
 		global $give_logs, $wpdb;
 
-		if ( $this->step == 1 ) {
-			$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' );
+		if ($this->step == 1) {
+			$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');
 		}
 
-		$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) {
 			$total         = 0;
-			$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);
 			}
 
-			$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+			$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-			if ( false === $all_forms ) {
+			if (false === $all_forms) {
 				$args = array(
 					'post_status'    => 'any',
 					'post_type'      => 'give_forms',
-					'posts_per_page' => - 1,
+					'posts_per_page' => -1,
 					'fields'         => 'ids',
 				);
 
-				$all_forms = get_posts( $args );
-				$this->store_data( 'give_temp_form_ids', $all_forms );
+				$all_forms = get_posts($args);
+				$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(
 				'post_parent__in' => $all_forms,
 				'post_type'       => 'give_log',
 				'post_status'     => 'publish',
 				'log_type'        => 'sale',
 				'fields'          => 'ids',
 				'nopaging'        => true,
-			) );
+			));
 
-			$all_logs = $give_logs->get_connected_logs( $args, 'sale' );
+			$all_logs = $give_logs->get_connected_logs($args, 'sale');
 
-			if ( $all_logs ) {
-				$log_ids     = implode( ',', $all_logs );
-				$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-				unset( $log_ids );
+			if ($all_logs) {
+				$log_ids     = implode(',', $all_logs);
+				$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+				unset($log_ids);
 
-				$payment_ids = implode( ',', $payment_ids );
-				$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-				unset( $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
+				$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+				unset($payment_ids);
 
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 
-					$payment = new Give_Payment( $payment->ID );
+					$payment = new Give_Payment($payment->ID);
 					$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
@@ -336,11 +336,11 @@ 
 
 				}
 
-				$total = count( $all_logs );
+				$total = count($all_logs);
 			}
 
-			$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);
 		}
 
 	}
@@ -354,11 +354,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -371,10 +371,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -388,7 +388,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -400,9 +400,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/reporting/tools/class-give-tools-recount-form-stats.php

Spacing +66 added lines, -66 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -62,23 +62,23 @@ 
 	public function get_data() {
 		global $give_logs, $wpdb;
 
-		$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(
 			'post_parent'    => $this->form_id,
 			'post_type'      => 'give_log',
 			'posts_per_page' => $this->per_step,
@@ -86,49 +86,49 @@ 
 			'paged'          => $this->step,
 			'log_type'       => 'sale',
 			'fields'         => 'ids',
-		) );
+		));
 
-		$log_ids              = $give_logs->get_connected_logs( $args, 'sale' );
+		$log_ids              = $give_logs->get_connected_logs($args, 'sale');
 		$this->_log_ids_debug = array();
 
-		if ( $log_ids ) {
-			$log_ids     = implode( ',', $log_ids );
-			$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-			unset( $log_ids );
+		if ($log_ids) {
+			$log_ids     = implode(',', $log_ids);
+			$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+			unset($log_ids);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-			unset( $payment_ids );
+			$payment_ids = implode(',', $payment_ids);
+			$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+			unset($payment_ids);
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$payment = new Give_Payment( $payment->ID );
+				$payment = new Give_Payment($payment->ID);
 
 				//Ensure acceptible 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;
 				}
 
 				$this->_log_ids_debug[] = $payment->ID;
 
-				$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;
 		}
 
 
-		update_post_meta( $this->form_id, '_give_form_sales', $totals['sales'] );
-		update_post_meta( $this->form_id, '_give_form_earnings', $totals['earnings'] );
+		update_post_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		update_post_meta($this->form_id, '_give_form_earnings', $totals['earnings']);
 
 		return false;
 
@@ -143,54 +143,54 @@ 
 	public function get_percentage_complete() {
 		global $give_logs, $wpdb;
 
-		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(
 				'post_parent' => $this->form_id,
 				'post_type'   => 'give_log',
 				'post_status' => 'publish',
 				'log_type'    => 'sale',
 				'fields'      => 'ids',
 				'nopaging'    => true,
-			) );
+			));
 
-			$log_ids = $give_logs->get_connected_logs( $args, 'sale' );
+			$log_ids = $give_logs->get_connected_logs($args, 'sale');
 
-			if ( $log_ids ) {
-				$log_ids     = implode( ',', $log_ids );
-				$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-				unset( $log_ids );
+			if ($log_ids) {
+				$log_ids     = implode(',', $log_ids);
+				$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+				unset($log_ids);
 
-				$payment_ids = implode( ',', $payment_ids );
-				$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-				unset( $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
+				$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+				unset($payment_ids);
 
-				foreach ( $payments as $payment ) {
-					if ( in_array( $payment->post_status, $accepted_statuses ) ) {
+				foreach ($payments as $payment) {
+					if (in_array($payment->post_status, $accepted_statuses)) {
 						continue;
 					}
 
-					$total ++;
+					$total++;
 				}
 			}
 
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$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;
 		}
 
@@ -204,8 +204,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;
 	}
 
 	/**
@@ -216,31 +216,31 @@ 
 	 */
 	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;
 		}
 	}
 
 	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);
 		}
 	}
 
@@ -268,11 +268,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -285,10 +285,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -302,7 +302,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -314,9 +314,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/reporting/class-api-requests-logs-list-table.php

Spacing +64 added lines, -64 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 	}
 
 	/**
@@ -63,20 +63,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$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']).'" />';
 		}
 		?>
 		<p class="search-box">
 			<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( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -90,10 +90,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'      => esc_html__( 'Log ID', 'give' ),
-			'details' => esc_html__( 'Request Details', 'give' ),
-			'ip'      => esc_html__( 'Request IP', 'give' ),
-			'date'    => esc_html__( 'Date', 'give' )
+			'ID'      => esc_html__('Log ID', 'give'),
+			'details' => esc_html__('Request Details', 'give'),
+			'ip'      => esc_html__('Request IP', 'give'),
+			'date'    => esc_html__('Date', 'give')
 		);
 
 		return $columns;
@@ -110,10 +110,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -127,26 +127,26 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_details( $item ) {
+	public function column_details($item) {
 		?>
-		<a href="#TB_inline?width=640&amp;inlineId=log-details-<?php echo $item['ID']; ?>" class="thickbox"><?php esc_html_e( 'View Request', 'give' ); ?></a>
+		<a href="#TB_inline?width=640&amp;inlineId=log-details-<?php echo $item['ID']; ?>" class="thickbox"><?php esc_html_e('View Request', 'give'); ?></a>
 		<div id="log-details-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$request = get_post_field( 'post_excerpt', $item['ID'] );
-			$error   = get_post_field( 'post_content', $item['ID'] );
-			echo '<p><strong>' . esc_html__( 'API Request:', 'give' ) . '</strong></p>';
-			echo '<div>' . $request . '</div>';
-			if ( ! empty( $error ) ) {
-				echo '<p><strong>' . esc_html__( 'Error', 'give' ) . '</strong></p>';
-				echo '<div>' . esc_html( $error ) . '</div>';
+			$request = get_post_field('post_excerpt', $item['ID']);
+			$error   = get_post_field('post_content', $item['ID']);
+			echo '<p><strong>'.esc_html__('API Request:', 'give').'</strong></p>';
+			echo '<div>'.$request.'</div>';
+			if ( ! empty($error)) {
+				echo '<p><strong>'.esc_html__('Error', 'give').'</strong></p>';
+				echo '<div>'.esc_html($error).'</div>';
 			}
-			echo '<p><strong>' . esc_html__( 'API User:', 'give' ) . '</strong></p>';
-			echo '<div>' . get_post_meta( $item['ID'], '_give_log_user', true ) . '</div>';
-			echo '<p><strong>' . esc_html__( 'API Key:', 'give' ) . '</strong></p>';
-			echo '<div>' . get_post_meta( $item['ID'], '_give_log_key', true ) . '</div>';
-			echo '<p><strong>' . esc_html__( 'Request Date:', 'give' ) . '</strong></p>';
-			echo '<div>' . get_post_field( 'post_date', $item['ID'] ) . '</div>';
+			echo '<p><strong>'.esc_html__('API User:', 'give').'</strong></p>';
+			echo '<div>'.get_post_meta($item['ID'], '_give_log_user', true).'</div>';
+			echo '<p><strong>'.esc_html__('API Key:', 'give').'</strong></p>';
+			echo '<div>'.get_post_meta($item['ID'], '_give_log_key', true).'</div>';
+			echo '<p><strong>'.esc_html__('Request Date:', 'give').'</strong></p>';
+			echo '<div>'.get_post_field('post_date', $item['ID']).'</div>';
 			?>
 		</div>
 	<?php
@@ -160,7 +160,7 @@ 
 	 * @return mixed String if search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -176,19 +176,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -210,31 +210,31 @@ 
 
 		$search = $this->get_search();
 
-		if ( $search ) {
-			if ( filter_var( $search, FILTER_VALIDATE_IP ) ) {
+		if ($search) {
+			if (filter_var($search, FILTER_VALIDATE_IP)) {
 				// This is an IP address search
 				$key = '_give_log_request_ip';
-			} else if ( is_email( $search ) ) {
+			} else if (is_email($search)) {
 				// This is an email search
-				$userdata = get_user_by( 'email', $search );
+				$userdata = get_user_by('email', $search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
 				$key = '_give_log_user';
-			} elseif ( strlen( $search ) == 32 ) {
+			} elseif (strlen($search) == 32) {
 				// Look for an API key
 				$key = '_give_log_key';
-			} elseif ( stristr( $search, 'token:' ) ) {
+			} elseif (stristr($search, 'token:')) {
 				// Look for an API token
-				$search = str_ireplace( 'token:', '', $search );
+				$search = str_ireplace('token:', '', $search);
 				$key    = '_give_log_token';
 			} else {
 				// This is (probably) a user ID search
-				$userdata = get_userdata( $search );
+				$userdata = get_userdata($search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
@@ -260,7 +260,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -270,7 +270,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -293,14 +293,14 @@ 
 			'meta_query' => $this->get_meta_query()
 		);
 
-		$logs = $give_logs->get_connected_logs( $log_query );
+		$logs = $give_logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'   => $log->ID,
-					'ip'   => get_post_meta( $log->ID, '_give_log_request_ip', true ),
+					'ip'   => get_post_meta($log->ID, '_give_log_request_ip', true),
 					'date' => $log->post_date
 				);
 			}
@@ -328,14 +328,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( 0, 'api_requests' );
+		$total_items           = $give_logs->get_log_count(0, 'api_requests');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page )
+				'total_pages' => ceil($total_items / $this->per_page)
 			)
 		);
 	}

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

Spacing +46 added lines, -46 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,50 +54,50 @@ 
 	 */
 	public function get_data() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_income' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_income');
 		}
 
-		$total = get_option( 'give_temp_recount_income', false );
+		$total = get_option('give_temp_recount_income', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = (float) 0;
-			$this->store_data( 'give_temp_recount_income', $total );
+			$this->store_data('give_temp_recount_income', $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_income_args', array(
+		$args = apply_filters('give_recount_income_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_income', $total );
+			$this->store_data('give_temp_recount_income', $total);
 
 			return true;
 
 		}
 
-		update_option( 'give_income_total', $total );
-		set_transient( 'give_income_total', $total, 86400 );
+		update_option('give_income_total', $total);
+		set_transient('give_income_total', $total, 86400);
 
 		return false;
 
@@ -111,25 +111,25 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_income_total' );
+		$total = $this->get_stored_data('give_recount_income_total');
 
-		if ( false === $total ) {
-			$args = apply_filters( 'give_recount_income_total_args', array() );
+		if (false === $total) {
+			$args = apply_filters('give_recount_income_total_args', array());
 
-			$counts = give_count_payments( $args );
-			$total  = absint( $counts->publish );
-			$total  = apply_filters( 'give_recount_store_income_total', $total );
+			$counts = give_count_payments($args);
+			$total  = absint($counts->publish);
+			$total  = apply_filters('give_recount_store_income_total', $total);
 
-			$this->store_data( 'give_recount_income_total', $total );
+			$this->store_data('give_recount_income_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;
 		}
 
@@ -143,7 +143,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -154,31 +154,31 @@ 
 	 */
 	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_income_total' );
-			$this->delete_data( 'give_temp_recount_income' );
+			$this->delete_data('give_recount_income_total');
+			$this->delete_data('give_temp_recount_income');
 			$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;
 		}
 	}
 
 	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);
 		}
 	}
 
@@ -206,11 +206,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -223,10 +223,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -240,7 +240,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -252,9 +252,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/gateways/manual.php

Spacing +14 added lines, -14 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,7 +20,7 @@ 
  * @since 1.0
  * @return void
  */
-add_action( 'give_manual_cc_form', '__return_false' );
+add_action('give_manual_cc_form', '__return_false');
 
 /**
  * Processes the purchase data and uses the Manual Payment gateway to record
@@ -32,17 +32,17 @@ 
  *
  * @return void
  */
-function give_manual_payment( $purchase_data ) {
+function give_manual_payment($purchase_data) {
 
-	if ( ! wp_verify_nonce( $purchase_data['gateway_nonce'], 'give-gateway' ) ) {
-		wp_die( esc_html__( 'Nonce verification has failed.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($purchase_data['gateway_nonce'], 'give-gateway')) {
+		wp_die(esc_html__('Nonce verification has failed.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 	
 	//Create payment_data array
 	$payment_data = array(
 		'price'           => $purchase_data['price'],
 		'give_form_title' => $purchase_data['post_data']['give-form-title'],
-		'give_form_id'    => intval( $purchase_data['post_data']['give-form-id'] ),
+		'give_form_id'    => intval($purchase_data['post_data']['give-form-id']),
 		'give_price_id'   => isset($purchase_data['post_data']['give-price-id']) ? $purchase_data['post_data']['give-price-id'] : '',
 		'date'            => $purchase_data['date'],
 		'user_email'      => $purchase_data['user_email'],
@@ -52,24 +52,24 @@ 
 		'status'          => 'pending'
 	);
 	// Record the pending payment
-	$payment = give_insert_payment( $payment_data );
+	$payment = give_insert_payment($payment_data);
 
-	if ( $payment ) {
-		give_update_payment_status( $payment, 'publish' );
+	if ($payment) {
+		give_update_payment_status($payment, 'publish');
 		give_send_to_success_page();
 	} else {
 		give_record_gateway_error(
-			esc_html__( 'Payment Error', 'give' ),
+			esc_html__('Payment Error', 'give'),
 			sprintf(
 				/* translators: %s: payment data */
-				esc_html__( 'The payment creation failed while processing a manual (free or test) donation. Payment data: %s', 'give' ),
-				json_encode( $payment_data )
+				esc_html__('The payment creation failed while processing a manual (free or test) donation. Payment data: %s', 'give'),
+				json_encode($payment_data)
 			),
 			$payment
 		);
 		// If errors are present, send the user back to the purchase page so they can be corrected
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
 	}
 }
 
-add_action( 'give_gateway_manual', 'give_manual_payment' );
+add_action('give_gateway_manual', 'give_manual_payment');

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

Spacing +33 added lines, -33 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 
 	}
 
@@ -64,10 +64,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -80,11 +80,11 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'label'           => esc_attr__( 'Gateway', 'give' ),
-			'complete_sales'  => esc_attr__( 'Complete Transactions', 'give' ),
-			'pending_sales'   => esc_attr__( 'Pending / Failed Transactions', 'give' ),
-			'total_sales'     => esc_attr__( 'Total Transactions', 'give' ),
-			'total_donations' => esc_attr__( 'Total Donations', 'give' )
+			'label'           => esc_attr__('Gateway', 'give'),
+			'complete_sales'  => esc_attr__('Complete Transactions', 'give'),
+			'pending_sales'   => esc_attr__('Pending / Failed Transactions', 'give'),
+			'total_sales'     => esc_attr__('Total Transactions', 'give'),
+			'total_donations' => esc_attr__('Total Donations', 'give')
 		);
 
 		return $columns;
@@ -99,7 +99,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 
@@ -110,7 +110,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place
 		give_report_views();
 	}
@@ -123,23 +123,23 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' == $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' == $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e( 'Donation Methods Report', 'give' ); ?></span></h3>
+			<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e('Donation Methods Report', 'give'); ?></span></h3>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -164,18 +164,18 @@ 
 		$gateways     = give_get_payment_gateways();
 		$stats        = new Give_Payment_Stats();
 
-		foreach ( $gateways as $gateway_id => $gateway ) {
+		foreach ($gateways as $gateway_id => $gateway) {
 
-			$complete_count = give_count_sales_by_gateway( $gateway_id, 'publish' );
-			$pending_count  = give_count_sales_by_gateway( $gateway_id, array( 'pending', 'failed' ) );
+			$complete_count = give_count_sales_by_gateway($gateway_id, 'publish');
+			$pending_count  = give_count_sales_by_gateway($gateway_id, array('pending', 'failed'));
 
 			$reports_data[] = array(
 				'ID'              => $gateway_id,
 				'label'           => $gateway['admin_label'],
-				'complete_sales'  => give_format_amount( $complete_count, false ),
-				'pending_sales'   => give_format_amount( $pending_count, false ),
-				'total_sales'     => give_format_amount( $complete_count + $pending_count, false ),
-				'total_donations' => give_currency_filter( give_format_amount( $stats->get_earnings( 0, 0, 0, $gateway_id ) ) )
+				'complete_sales'  => give_format_amount($complete_count, false),
+				'pending_sales'   => give_format_amount($pending_count, false),
+				'total_sales'     => give_format_amount($complete_count + $pending_count, false),
+				'total_donations' => give_currency_filter(give_format_amount($stats->get_earnings(0, 0, 0, $gateway_id)))
 			);
 		}
 
@@ -197,7 +197,7 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->reports_data();
 
 	}

includes/admin/reporting/logs.php

Spacing +20 added lines, -20 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return void
  */
 function give_logs_view_sales() {
-	include( dirname( __FILE__ ) . '/class-sales-logs-list-table.php' );
+	include(dirname(__FILE__).'/class-sales-logs-list-table.php');
 
 	$logs_table = new Give_Sales_Log_Table();
 	$logs_table->prepare_items();
@@ -31,7 +31,7 @@ 
 
 }
 
-add_action( 'give_logs_view_sales', 'give_logs_view_sales' );
+add_action('give_logs_view_sales', 'give_logs_view_sales');
 
 
 /**
@@ -43,14 +43,14 @@ 
  * @return void
  */
 function give_logs_view_gateway_errors() {
-	include( dirname( __FILE__ ) . '/class-gateway-error-logs-list-table.php' );
+	include(dirname(__FILE__).'/class-gateway-error-logs-list-table.php');
 
 	$logs_table = new Give_Gateway_Error_Log_Table();
 	$logs_table->prepare_items();
 	$logs_table->display();
 }
 
-add_action( 'give_logs_view_gateway_errors', 'give_logs_view_gateway_errors' );
+add_action('give_logs_view_gateway_errors', 'give_logs_view_gateway_errors');
 
 /**
  * API Request Logs
@@ -62,7 +62,7 @@ 
  * @return void
  */
 function give_logs_view_api_requests() {
-	include( dirname( __FILE__ ) . '/class-api-requests-logs-list-table.php' );
+	include(dirname(__FILE__).'/class-api-requests-logs-list-table.php');
 
 	$logs_table = new Give_API_Request_Log_Table();
 	$logs_table->prepare_items();
@@ -75,12 +75,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_api_requests_top' );
+		do_action('give_logs_api_requests_top');
 		?>
 
-		<form id="give-logs-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-reports&tab=logs' ); ?>">
+		<form id="give-logs-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-reports&tab=logs'); ?>">
 			<?php
-			$logs_table->search_box( esc_html__( 'Search', 'give' ), 'give-api-requests' );
+			$logs_table->search_box(esc_html__('Search', 'give'), 'give-api-requests');
 			$logs_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms"/>
@@ -93,14 +93,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_api_requests_bottom' );
+		do_action('give_logs_api_requests_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_api_requests', 'give_logs_view_api_requests' );
+add_action('give_logs_view_api_requests', 'give_logs_view_api_requests');
 
 
 /**
@@ -111,12 +111,12 @@ 
  */
 function give_log_default_views() {
 	$views = array(
-		'sales'          => esc_html__( 'Donations', 'give' ),
-		'gateway_errors' => esc_html__( 'Payment Errors', 'give' ),
-		'api_requests'   => esc_html__( 'API Requests', 'give' )
+		'sales'          => esc_html__('Donations', 'give'),
+		'gateway_errors' => esc_html__('Payment Errors', 'give'),
+		'api_requests'   => esc_html__('API Requests', 'give')
 	);
 
-	$views = apply_filters( 'give_log_views', $views );
+	$views = apply_filters('give_log_views', $views);
 
 	return $views;
 }
@@ -129,13 +129,13 @@ 
  */
 function give_log_views() {
 	$views        = give_log_default_views();
-	$current_view = isset( $_GET['view'] ) && array_key_exists( $_GET['view'], give_log_default_views() ) ? sanitize_text_field( $_GET['view'] ) : 'sales';
+	$current_view = isset($_GET['view']) && array_key_exists($_GET['view'], give_log_default_views()) ? sanitize_text_field($_GET['view']) : 'sales';
 	?>
 	<form id="give-logs-filter" method="get" action="edit.php">
 		<select id="give-logs-view" name="view">
 			<optgroup label="Log Type:">
-				<?php foreach ( $views as $view_id => $label ): ?>
-					<option value="<?php echo esc_attr( $view_id ); ?>" <?php selected( $view_id, $current_view ); ?>><?php echo $label; ?></option>
+				<?php foreach ($views as $view_id => $label): ?>
+					<option value="<?php echo esc_attr($view_id); ?>" <?php selected($view_id, $current_view); ?>><?php echo $label; ?></option>
 				<?php endforeach; ?>
 			</optgroup>
 		</select>
@@ -148,14 +148,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_log_view_actions' );
+		do_action('give_log_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-reports"/>
 		<input type="hidden" name="tab" value="logs"/>
 
-		<?php submit_button( esc_html__( 'Apply', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Apply', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 }
\ No newline at end of file