WordImpress / Give

includes/formatting.php

Spacing +146 added lines, -149 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return mixed
  */
 function give_get_price_decimals() {
-	return apply_filters( 'give_sanitize_amount_decimals', give_get_option( 'number_decimals', 0 ) );
+	return apply_filters('give_sanitize_amount_decimals', give_get_option('number_decimals', 0));
 }
 
 /**
@@ -35,8 +35,8 @@ 
  */
 function give_get_price_thousand_separator() {
 	$give_options       = give_get_settings();
-	$thousand_separator = isset( $give_options['thousands_separator'] ) ? $give_options['thousands_separator'] : ',';
-	$thousand_separator = empty( $thousand_separator ) ? ' ' : $thousand_separator;
+	$thousand_separator = isset($give_options['thousands_separator']) ? $give_options['thousands_separator'] : ',';
+	$thousand_separator = empty($thousand_separator) ? ' ' : $thousand_separator;
 
 	return $thousand_separator;
 }
@@ -55,11 +55,10 @@ 
 	);
 
 	$thousand_separator = give_get_price_thousand_separator();
-	$default_decimal_separator = in_array( $thousand_separator, $default_decimal_separators ) ?
-		$default_decimal_separators[$thousand_separator] :
-		'.';
+	$default_decimal_separator = in_array($thousand_separator, $default_decimal_separators) ?
+		$default_decimal_separators[$thousand_separator] : '.';
 
-	$decimal_separator = give_get_option( 'decimal_separator', $default_decimal_separator );
+	$decimal_separator = give_get_option('decimal_separator', $default_decimal_separator);
 
 	return $decimal_separator;
 }
@@ -74,8 +73,8 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount_for_db( $number ) {
-	return give_maybe_sanitize_amount( $number, 6 );
+function give_sanitize_amount_for_db($number) {
+	return give_maybe_sanitize_amount($number, 6);
 }
 
 /**
@@ -89,40 +88,40 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_maybe_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_maybe_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 	$thousand_separator = give_get_price_thousand_separator();
 	$decimal_separator  = give_get_price_decimal_separator();
 
 	// Bailout.
-	if( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}elseif (
-		( false == strpos( $number, $thousand_separator ) ) &&
-		( false === strpos( $number, $decimal_separator ) )
+		(false == strpos($number, $thousand_separator)) &&
+		(false === strpos($number, $decimal_separator))
 	) {
-		return number_format( $number, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		return number_format($number, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 	}
 
 	// Handle thousand separator as '.'
 	// Handle sanitize database values.
-	$number_parts = explode( '.', $number );
-	$is_db_sanitize_val = ( 2 === count( $number_parts ) && ( 6 === strlen( $number_parts[1] ) ) );
+	$number_parts = explode('.', $number);
+	$is_db_sanitize_val = (2 === count($number_parts) && (6 === strlen($number_parts[1])));
 
-	if( $is_db_sanitize_val ) {
+	if ($is_db_sanitize_val) {
 		// Sanitize database value.
-		return number_format( $number, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		return number_format($number, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 
 	} elseif (
 		'.' === $thousand_separator &&
-		false !== strpos( $number, $thousand_separator )
-	){
+		false !== strpos($number, $thousand_separator)
+	) {
 		// Fix point thousand separator value.
-		if( ! $is_db_sanitize_val ) {
-			$number = str_replace( '.', '', $number );
+		if ( ! $is_db_sanitize_val) {
+			$number = str_replace('.', '', $number);
 		}
 	}
 
-	return give_sanitize_amount( $number, $dp, $trim_zeros );
+	return give_sanitize_amount($number, $dp, $trim_zeros);
 }
 
 /**
@@ -138,67 +137,67 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 
 	// Bailout.
-	if ( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}
 
 	// Remove slash from amount.
 	// If thousand or decimal separator is set to ' then in $_POST or $_GET param we will get an escaped number.
 	// To prevent notices and warning remove slash from amount/number.
-	$number = wp_unslash( $number );
+	$number = wp_unslash($number);
 
 	$thousand_separator = give_get_price_thousand_separator();
 
 	$locale   = localeconv();
-	$decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
+	$decimals = array(give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point']);
 
 	// Remove locale from string
-	if ( ! is_float( $number ) ) {
-		$number = str_replace( $decimals, '.', $number );
+	if ( ! is_float($number)) {
+		$number = str_replace($decimals, '.', $number);
 	}
 
 	// Remove thousand amount formatting if amount has.
 	// This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
 	// Do not replace thousand separator from price if it is same as decimal separator, because it will be already replace by above code.
-	if ( ! in_array( $thousand_separator, $decimals ) && ( false !== strpos( $number, $thousand_separator ) ) ) {
-		$number = str_replace( $thousand_separator, '', $number );
-	} elseif ( in_array( $thousand_separator, $decimals ) ) {
-		$number = preg_replace( '/\.(?=.*\.)/', '', $number );
+	if ( ! in_array($thousand_separator, $decimals) && (false !== strpos($number, $thousand_separator))) {
+		$number = str_replace($thousand_separator, '', $number);
+	} elseif (in_array($thousand_separator, $decimals)) {
+		$number = preg_replace('/\.(?=.*\.)/', '', $number);
 	}
 
 	// Remove non numeric entity before decimal separator.
-	$number     = preg_replace( '/[^0-9\.]/', '', $number );
+	$number     = preg_replace('/[^0-9\.]/', '', $number);
 	$default_dp = give_get_price_decimals();
 
 	// Reset negative amount to zero.
-	if ( 0 > $number ) {
-		$number = number_format( 0, $default_dp, '.' );
+	if (0 > $number) {
+		$number = number_format(0, $default_dp, '.');
 	}
 
 	// If number does not have decimal then add number of decimals to it.
 	if (
-		false === strpos( $number, '.' )
-		|| ( $default_dp > strlen( substr( $number, strpos( $number, '.' ) + 1 ) ) )
+		false === strpos($number, '.')
+		|| ($default_dp > strlen(substr($number, strpos($number, '.') + 1)))
 	) {
-		$number = number_format( $number, $default_dp, '.', '' );
+		$number = number_format($number, $default_dp, '.', '');
 	}
 
 	// Format number by custom number of decimals.
-	if ( false !== $dp ) {
-		$dp     = intval( is_bool( $dp ) ? $default_dp : $dp );
-		$dp     = apply_filters( 'give_sanitize_amount_decimals', $dp, $number );
-		$number = number_format( floatval( $number ), $dp, '.', '' );
+	if (false !== $dp) {
+		$dp     = intval(is_bool($dp) ? $default_dp : $dp);
+		$dp     = apply_filters('give_sanitize_amount_decimals', $dp, $number);
+		$number = number_format(floatval($number), $dp, '.', '');
 	}
 
 	// Trim zeros.
-	if ( $trim_zeros && strstr( $number, '.' ) ) {
-		$number = rtrim( rtrim( $number, '0' ), '.' );
+	if ($trim_zeros && strstr($number, '.')) {
+		$number = rtrim(rtrim($number, '0'), '.');
 	}
 
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -211,10 +210,10 @@ 
  *
  * @return string $amount   Newly formatted amount or Price Not Available
  */
-function give_format_amount( $amount, $args = array() ) {
+function give_format_amount($amount, $args = array()) {
 	// Backward compatibility.
-	if( is_bool( $args ) ) {
-		$args = array( 'decimal' => $args );
+	if (is_bool($args)) {
+		$args = array('decimal' => $args);
 	}
 
 	$default_args = array(
@@ -223,60 +222,59 @@ 
 		'currency' => give_get_currency(),
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	$formatted     = 0;
 	$thousands_sep = give_get_price_thousand_separator();
 	$decimal_sep   = give_get_price_decimal_separator();
-	$decimals      = ! empty( $args['decimal'] ) ? give_get_price_decimals() : 0;
+	$decimals      = ! empty($args['decimal']) ? give_get_price_decimals() : 0;
 	$currency      = $args['currency'];
 
-	if ( ! empty( $amount ) ) {
+	if ( ! empty($amount)) {
 		// Sanitize amount before formatting.
-		$amount = ! empty( $args['sanitize'] ) ?
-			give_maybe_sanitize_amount( $amount, $decimals ) :
-			number_format( $amount, $decimals, '.', '' );
+		$amount = ! empty($args['sanitize']) ?
+			give_maybe_sanitize_amount($amount, $decimals) : number_format($amount, $decimals, '.', '');
 
-		switch ( $currency ) {
+		switch ($currency) {
 			case 'INR':
 				$decimal_amount = '';
 
 				// Extract decimals from amount
-				if ( ( $pos = strpos( $amount, '.' ) ) !== false ) {
-					if ( ! empty( $decimals ) ) {
-						$decimal_amount = substr( round( substr( $amount, $pos ), $decimals ), 1 );
-						$amount         = substr( $amount, 0, $pos );
-
-						if ( ! $decimal_amount ) {
-							$decimal_amount = substr( '.0000000000', 0, ( $decimals + 1 ) );
-						} elseif ( ( $decimals + 1 ) > strlen( $decimal_amount ) ) {
-							$decimal_amount = substr( "{$decimal_amount}000000000", 0, ( $decimals + 1 ) );
+				if (($pos = strpos($amount, '.')) !== false) {
+					if ( ! empty($decimals)) {
+						$decimal_amount = substr(round(substr($amount, $pos), $decimals), 1);
+						$amount         = substr($amount, 0, $pos);
+
+						if ( ! $decimal_amount) {
+							$decimal_amount = substr('.0000000000', 0, ($decimals + 1));
+						} elseif (($decimals + 1) > strlen($decimal_amount)) {
+							$decimal_amount = substr("{$decimal_amount}000000000", 0, ($decimals + 1));
 						}
 
 					} else {
-						$amount = number_format( $amount, $decimals, $decimal_sep, '' );
+						$amount = number_format($amount, $decimals, $decimal_sep, '');
 					}
 				}
 
 				// Extract last 3 from amount
-				$result = substr( $amount, - 3 );
-				$amount = substr( $amount, 0, - 3 );
+				$result = substr($amount, - 3);
+				$amount = substr($amount, 0, - 3);
 
 				// Apply digits 2 by 2
-				while ( strlen( $amount ) > 0 ) {
-					$result = substr( $amount, - 2 ) . $thousands_sep . $result;
-					$amount = substr( $amount, 0, - 2 );
+				while (strlen($amount) > 0) {
+					$result = substr($amount, - 2).$thousands_sep.$result;
+					$amount = substr($amount, 0, - 2);
 				}
 
-				$formatted = $result . $decimal_amount;
+				$formatted = $result.$decimal_amount;
 				break;
 
 			default:
-				$formatted = number_format( $amount, $decimals, $decimal_sep, $thousands_sep );
+				$formatted = number_format($amount, $decimals, $decimal_sep, $thousands_sep);
 		}
 	}
 
-	return apply_filters( 'give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args );
+	return apply_filters('give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args);
 }
 
 
@@ -294,51 +292,51 @@ 
  *
  * @return float|string  formatted amount number with large number names.
  */
-function give_human_format_large_amount( $amount, $args = array() ) {
+function give_human_format_large_amount($amount, $args = array()) {
 	$default_args = array(
 		'currency' => give_get_currency(),
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	// Get thousand separator.
 	$thousands_sep = give_get_price_thousand_separator();
 
 	// Sanitize amount.
-	$sanitize_amount = give_maybe_sanitize_amount( $amount );
+	$sanitize_amount = give_maybe_sanitize_amount($amount);
 
 	// Explode amount to calculate name of large numbers.
-	$amount_array = explode( $thousands_sep, $amount );
+	$amount_array = explode($thousands_sep, $amount);
 
 	// Calculate amount parts count.
-	$amount_count_parts = count( $amount_array );
+	$amount_count_parts = count($amount_array);
 
 	// Human format amount (default).
 	$human_format_amount = $amount;
 
-	switch ( $args['currency'] ) {
+	switch ($args['currency']) {
 		case 'INR':
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s arab', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s crore', 'give' ), round( ( $sanitize_amount / 10000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s lakh', 'give' ), round( ( $sanitize_amount / 100000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s arab', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s crore', 'give'), round(($sanitize_amount / 10000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s lakh', 'give'), round(($sanitize_amount / 100000), 2));
 			}
 			break;
 		default:
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s million', 'give' ), round( ( $sanitize_amount / 1000000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s trillion', 'give'), round(($sanitize_amount / 1000000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s billion', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s million', 'give'), round(($sanitize_amount / 1000000), 2));
 			}
 	}
 
-	return apply_filters( 'give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount );
+	return apply_filters('give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount);
 }
 
 /**
@@ -352,17 +350,16 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_decimal( $amount, $dp = false, $sanitize = true ) {
+function give_format_decimal($amount, $dp = false, $sanitize = true) {
 	$decimal_separator = give_get_price_decimal_separator();
 	$formatted_amount  = $sanitize ?
-		give_maybe_sanitize_amount( $amount, $dp ) :
-		number_format( $amount, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		give_maybe_sanitize_amount($amount, $dp) : number_format($amount, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 
-	if ( false !== strpos( $formatted_amount, '.' ) ) {
-		$formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+	if (false !== strpos($formatted_amount, '.')) {
+		$formatted_amount = str_replace('.', $decimal_separator, $formatted_amount);
 	}
 
-	return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+	return apply_filters('give_format_decimal', $formatted_amount, $amount, $decimal_separator);
 }
 
 /**
@@ -376,24 +373,24 @@ 
  *
  * @return mixed|string
  */
-function give_currency_filter( $price = '', $currency = '', $decode_currency = false ) {
+function give_currency_filter($price = '', $currency = '', $decode_currency = false) {
 
-	if ( empty( $currency ) || ! array_key_exists( (string) $currency, give_get_currencies() ) ) {
+	if (empty($currency) || ! array_key_exists((string) $currency, give_get_currencies())) {
 		$currency = give_get_currency();
 	}
 
-	$position = give_get_option( 'currency_position', 'before' );
+	$position = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$symbol = give_currency_symbol( $currency, $decode_currency );
+	$symbol = give_currency_symbol($currency, $decode_currency);
 
-	switch ( $currency ) :
+	switch ($currency) :
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -422,13 +419,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $position ? $symbol . $price : $price . $symbol );
+			$formatted = ('before' === $position ? $symbol.$price : $price.$symbol);
 			break;
 		case 'NOK' :
-			$formatted = ( 'before' === $position ? $symbol . ' ' . $price : $price . ' ' . $symbol );
+			$formatted = ('before' === $position ? $symbol.' '.$price : $price.' '.$symbol);
 			break;
 		default :
-			$formatted = ( 'before' === $position ? $currency . ' ' . $price : $price . ' ' . $currency );
+			$formatted = ('before' === $position ? $currency.' '.$price : $price.' '.$currency);
 			break;
 	endswitch;
 
@@ -443,11 +440,11 @@ 
 	 *           and if currency is USD and currency position is after then
 	 *           filter name will be give_usd_currency_filter_after
 	 */
-	$formatted = apply_filters( 'give_' . strtolower( $currency ) . "_currency_filter_{$position}", $formatted, $currency, $price );
+	$formatted = apply_filters('give_'.strtolower($currency)."_currency_filter_{$position}", $formatted, $currency, $price);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -463,21 +460,21 @@ 
  */
 function give_currency_decimal_filter() {
 
-	remove_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	remove_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Set default number of decimals.
 	$decimals = give_get_price_decimals();
 
-	add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Get number of decimals with backward compatibility ( version < 1.6 )
-	if ( 1 <= func_num_args() ) {
-		$decimals = ( false === func_get_arg( 0 ) ? $decimals : absint( func_get_arg( 0 ) ) );
+	if (1 <= func_num_args()) {
+		$decimals = (false === func_get_arg(0) ? $decimals : absint(func_get_arg(0)));
 	}
 
 	$currency = give_get_currency();
 
-	switch ( $currency ) {
+	switch ($currency) {
 		case 'RIAL' :
 		case 'JPY' :
 		case 'TWD' :
@@ -487,11 +484,11 @@ 
 			break;
 	}
 
-	return apply_filters( 'give_currency_decimal_count', $decimals, $currency );
+	return apply_filters('give_currency_decimal_count', $decimals, $currency);
 }
 
-add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
-add_filter( 'give_format_amount_decimals', 'give_currency_decimal_filter' );
+add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
+add_filter('give_format_amount_decimals', 'give_currency_decimal_filter');
 
 
 /**
@@ -503,7 +500,7 @@ 
  *
  * @return string                  Date format string
  */
-function give_date_format( $date_context = '' ) {
+function give_date_format($date_context = '') {
 	/**
 	 * Filter the date context
 	 *
@@ -524,19 +521,19 @@ 
 	 *
 	 *    }
 	 */
-	$date_format_contexts = apply_filters( 'give_date_format_contexts', array() );
+	$date_format_contexts = apply_filters('give_date_format_contexts', array());
 
 	// Set date format to default date format.
-	$date_format = get_option( 'date_format' );
+	$date_format = get_option('date_format');
 
 	// Update date format if we have non empty date format context array and non empty date format string for that context.
-	if ( $date_context && ! empty( $date_format_contexts ) && array_key_exists( $date_context, $date_format_contexts ) ) {
-		$date_format = ! empty( $date_format_contexts[ $date_context ] )
-			? $date_format_contexts[ $date_context ]
+	if ($date_context && ! empty($date_format_contexts) && array_key_exists($date_context, $date_format_contexts)) {
+		$date_format = ! empty($date_format_contexts[$date_context])
+			? $date_format_contexts[$date_context]
 			: $date_format;
 	}
 
-	return apply_filters( 'give_date_format', $date_format );
+	return apply_filters('give_date_format', $date_format);
 }
 
 /**
@@ -550,8 +547,8 @@ 
  *
  * @return string
  */
-function give_get_cache_key( $action, $query_args ) {
-	return Give_Cache::get_key( $action, $query_args );
+function give_get_cache_key($action, $query_args) {
+	return Give_Cache::get_key($action, $query_args);
 }
 
 /**
@@ -564,11 +561,11 @@ 
  *
  * @return string|array
  */
-function give_clean( $var ) {
-	if ( is_array( $var ) ) {
-		return array_map( 'give_clean', $var );
+function give_clean($var) {
+	if (is_array($var)) {
+		return array_map('give_clean', $var);
 	} else {
-		return is_scalar( $var ) ? sanitize_text_field( $var ) : $var;
+		return is_scalar($var) ? sanitize_text_field($var) : $var;
 	}
 }
 
@@ -581,10 +578,10 @@ 
  *
  * @return int
  */
-function give_let_to_num( $size ) {
-	$l   = substr( $size, - 1 );
-	$ret = substr( $size, 0, - 1 );
-	switch ( strtoupper( $l ) ) {
+function give_let_to_num($size) {
+	$l   = substr($size, - 1);
+	$ret = substr($size, 0, - 1);
+	switch (strtoupper($l)) {
 		case 'P':
 			$ret *= 1024;
 		case 'T':
@@ -609,19 +606,19 @@ 
  * @param int   $action
  * @param array $wp_die_args
  */
-function give_validate_nonce( $nonce, $action = - 1, $wp_die_args = array() ) {
+function give_validate_nonce($nonce, $action = - 1, $wp_die_args = array()) {
 
 	$default_wp_die_args = array(
-		'message' => esc_html__( 'Nonce verification has failed.', 'give' ),
-		'title'   => esc_html__( 'Error', 'give' ),
+		'message' => esc_html__('Nonce verification has failed.', 'give'),
+		'title'   => esc_html__('Error', 'give'),
 		'args'    => array(
 			'response' => 403,
 		),
 	);
 
-	$wp_die_args = wp_parse_args( $wp_die_args, $default_wp_die_args );
+	$wp_die_args = wp_parse_args($wp_die_args, $default_wp_die_args);
 
-	if ( ! wp_verify_nonce( $nonce, $action ) ) {
+	if ( ! wp_verify_nonce($nonce, $action)) {
 		wp_die(
 			$wp_die_args['message'],
 			$wp_die_args['title'],
@@ -643,23 +640,23 @@ 
  *
  * @return mixed
  */
-function give_check_variable( $variable, $conditional = '', $default = false ) {
+function give_check_variable($variable, $conditional = '', $default = false) {
 
-	switch ( $conditional ) {
+	switch ($conditional) {
 		case 'isset_empty':
-			$variable = ( isset( $variable ) && ! empty( $variable ) ) ? $variable : $default;
+			$variable = (isset($variable) && ! empty($variable)) ? $variable : $default;
 			break;
 
 		case 'empty':
-			$variable = ! empty( $variable ) ? $variable : $default;
+			$variable = ! empty($variable) ? $variable : $default;
 			break;
 
 		case 'null':
-			$variable = ! is_null( $variable ) ? $variable : $default;
+			$variable = ! is_null($variable) ? $variable : $default;
 			break;
 
 		default:
-			$variable = isset( $variable ) ? $variable : $default;
+			$variable = isset($variable) ? $variable : $default;
 
 	}
 

includes/post-types.php

Spacing +148 added lines, -148 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,36 +23,36 @@ 
 function give_setup_post_types() {
 
 	// Give Forms single post and archive options.
-	$give_forms_singular = give_is_setting_enabled( give_get_option( 'forms_singular' ) );
-	$give_forms_archives = give_is_setting_enabled( give_get_option( 'forms_archives' ) );
+	$give_forms_singular = give_is_setting_enabled(give_get_option('forms_singular'));
+	$give_forms_archives = give_is_setting_enabled(give_get_option('forms_archives'));
 
-	$give_forms_slug = defined( 'GIVE_SLUG' ) ? GIVE_SLUG : 'donations';
+	$give_forms_slug = defined('GIVE_SLUG') ? GIVE_SLUG : 'donations';
 	// Support for old 'GIVE_FORMS_SLUG' constant
-	if ( defined( 'GIVE_FORMS_SLUG' ) ) {
+	if (defined('GIVE_FORMS_SLUG')) {
 		$give_forms_slug = GIVE_FORMS_SLUG;
 	}
 
-	$give_forms_rewrite = defined( 'GIVE_DISABLE_FORMS_REWRITE' ) && GIVE_DISABLE_FORMS_REWRITE ? false : array(
+	$give_forms_rewrite = defined('GIVE_DISABLE_FORMS_REWRITE') && GIVE_DISABLE_FORMS_REWRITE ? false : array(
 		'slug'       => $give_forms_slug,
 		'with_front' => false,
 	);
 
-	$give_forms_labels = apply_filters( 'give_forms_labels', array(
-		'name'               => __( 'Donation Forms', 'give' ),
-		'singular_name'      => __( 'Form', 'give' ),
-		'add_new'            => __( 'Add Form', 'give' ),
-		'add_new_item'       => __( 'Add New Donation Form', 'give' ),
-		'edit_item'          => __( 'Edit Donation Form', 'give' ),
-		'new_item'           => __( 'New Form', 'give' ),
-		'all_items'          => __( 'All Forms', 'give' ),
-		'view_item'          => __( 'View Form', 'give' ),
-		'search_items'       => __( 'Search Forms', 'give' ),
-		'not_found'          => __( 'No forms found.', 'give' ),
-		'not_found_in_trash' => __( 'No forms found in Trash.', 'give' ),
+	$give_forms_labels = apply_filters('give_forms_labels', array(
+		'name'               => __('Donation Forms', 'give'),
+		'singular_name'      => __('Form', 'give'),
+		'add_new'            => __('Add Form', 'give'),
+		'add_new_item'       => __('Add New Donation Form', 'give'),
+		'edit_item'          => __('Edit Donation Form', 'give'),
+		'new_item'           => __('New Form', 'give'),
+		'all_items'          => __('All Forms', 'give'),
+		'view_item'          => __('View Form', 'give'),
+		'search_items'       => __('Search Forms', 'give'),
+		'not_found'          => __('No forms found.', 'give'),
+		'not_found_in_trash' => __('No forms found in Trash.', 'give'),
 		'parent_item_colon'  => '',
-		'menu_name'          => apply_filters( 'give_menu_name', __( 'Donations', 'give' ) ),
-		'name_admin_bar'     => apply_filters( 'give_name_admin_bar_name', __( 'Donation Form', 'give' ) ),
-	) );
+		'menu_name'          => apply_filters('give_menu_name', __('Donations', 'give')),
+		'name_admin_bar'     => apply_filters('give_name_admin_bar_name', __('Donation Form', 'give')),
+	));
 
 	// Default give_forms supports.
 	$give_form_supports = array(
@@ -64,14 +64,14 @@ 
 	);
 
 	// Has the user disabled the excerpt?
-	if ( ! give_is_setting_enabled( give_get_option( 'forms_excerpt' ) ) ) {
-		unset( $give_form_supports[2] );
+	if ( ! give_is_setting_enabled(give_get_option('forms_excerpt'))) {
+		unset($give_form_supports[2]);
 	}
 
 	// Has user disabled the featured image?
-	if ( ! give_is_setting_enabled( give_get_option( 'form_featured_img' ) ) ) {
-		unset( $give_form_supports[1] );
-		remove_action( 'give_before_single_form_summary', 'give_show_form_images' );
+	if ( ! give_is_setting_enabled(give_get_option('form_featured_img'))) {
+		unset($give_form_supports[1]);
+		remove_action('give_before_single_form_summary', 'give_show_form_images');
 	}
 
 	$give_forms_args = array(
@@ -86,42 +86,42 @@ 
 		'has_archive'        => $give_forms_archives,
 		'menu_icon'          => 'dashicons-give',
 		'hierarchical'       => false,
-		'supports'           => apply_filters( 'give_forms_supports', $give_form_supports ),
+		'supports'           => apply_filters('give_forms_supports', $give_form_supports),
 	);
-	register_post_type( 'give_forms', apply_filters( 'give_forms_post_type_args', $give_forms_args ) );
+	register_post_type('give_forms', apply_filters('give_forms_post_type_args', $give_forms_args));
 
 	/** Donation Post Type */
 	$payment_labels = array(
-		'name'               => _x( 'Donations', 'post type general name', 'give' ),
-		'singular_name'      => _x( 'Donation', 'post type singular name', 'give' ),
-		'add_new'            => __( 'Add New', 'give' ),
-		'add_new_item'       => __( 'Add New Donation', 'give' ),
-		'edit_item'          => __( 'Edit Donation', 'give' ),
-		'new_item'           => __( 'New Donation', 'give' ),
-		'all_items'          => __( 'All Donations', 'give' ),
-		'view_item'          => __( 'View Donation', 'give' ),
-		'search_items'       => __( 'Search Donations', 'give' ),
-		'not_found'          => __( 'No donations found.', 'give' ),
-		'not_found_in_trash' => __( 'No donations found in Trash.', 'give' ),
+		'name'               => _x('Donations', 'post type general name', 'give'),
+		'singular_name'      => _x('Donation', 'post type singular name', 'give'),
+		'add_new'            => __('Add New', 'give'),
+		'add_new_item'       => __('Add New Donation', 'give'),
+		'edit_item'          => __('Edit Donation', 'give'),
+		'new_item'           => __('New Donation', 'give'),
+		'all_items'          => __('All Donations', 'give'),
+		'view_item'          => __('View Donation', 'give'),
+		'search_items'       => __('Search Donations', 'give'),
+		'not_found'          => __('No donations found.', 'give'),
+		'not_found_in_trash' => __('No donations found in Trash.', 'give'),
 		'parent_item_colon'  => '',
-		'menu_name'          => __( 'Donations', 'give' ),
+		'menu_name'          => __('Donations', 'give'),
 	);
 
 	$payment_args = array(
-		'labels'          => apply_filters( 'give_payment_labels', $payment_labels ),
+		'labels'          => apply_filters('give_payment_labels', $payment_labels),
 		'public'          => false,
 		'query_var'       => false,
 		'rewrite'         => false,
 		'map_meta_cap'    => true,
 		'capability_type' => 'give_payment',
-		'supports'        => array( 'title' ),
+		'supports'        => array('title'),
 		'can_export'      => true,
 	);
-	register_post_type( 'give_payment', $payment_args );
+	register_post_type('give_payment', $payment_args);
 
 }
 
-add_action( 'init', 'give_setup_post_types', 1 );
+add_action('init', 'give_setup_post_types', 1);
 
 
 /**
@@ -134,30 +134,30 @@ 
  */
 function give_setup_taxonomies() {
 
-	$slug = defined( 'GIVE_FORMS_SLUG' ) ? GIVE_FORMS_SLUG : 'donations';
+	$slug = defined('GIVE_FORMS_SLUG') ? GIVE_FORMS_SLUG : 'donations';
 
 	/** Categories */
 	$category_labels = array(
-		'name'              => _x( 'Form Categories', 'taxonomy general name', 'give' ),
-		'singular_name'     => _x( 'Category', 'taxonomy singular name', 'give' ),
-		'search_items'      => __( 'Search Categories', 'give' ),
-		'all_items'         => __( 'All Categories', 'give' ),
-		'parent_item'       => __( 'Parent Category', 'give' ),
-		'parent_item_colon' => __( 'Parent Category:', 'give' ),
-		'edit_item'         => __( 'Edit Category', 'give' ),
-		'update_item'       => __( 'Update Category', 'give' ),
-		'add_new_item'      => __( 'Add New Category', 'give' ),
-		'new_item_name'     => __( 'New Category Name', 'give' ),
-		'menu_name'         => __( 'Categories', 'give' ),
+		'name'              => _x('Form Categories', 'taxonomy general name', 'give'),
+		'singular_name'     => _x('Category', 'taxonomy singular name', 'give'),
+		'search_items'      => __('Search Categories', 'give'),
+		'all_items'         => __('All Categories', 'give'),
+		'parent_item'       => __('Parent Category', 'give'),
+		'parent_item_colon' => __('Parent Category:', 'give'),
+		'edit_item'         => __('Edit Category', 'give'),
+		'update_item'       => __('Update Category', 'give'),
+		'add_new_item'      => __('Add New Category', 'give'),
+		'new_item_name'     => __('New Category Name', 'give'),
+		'menu_name'         => __('Categories', 'give'),
 	);
 
-	$category_args = apply_filters( 'give_forms_category_args', array(
+	$category_args = apply_filters('give_forms_category_args', array(
 			'hierarchical' => true,
-			'labels'       => apply_filters( 'give_forms_category_labels', $category_labels ),
+			'labels'       => apply_filters('give_forms_category_labels', $category_labels),
 			'show_ui'      => true,
 			'query_var'    => 'give_forms_category',
 			'rewrite'      => array(
-				'slug'         => $slug . '/category',
+				'slug'         => $slug.'/category',
 				'with_front'   => false,
 				'hierarchical' => true,
 			),
@@ -171,33 +171,33 @@ 
 	);
 
 	// Does the user want categories?
-	if ( give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) ) ) {
-		register_taxonomy( 'give_forms_category', array( 'give_forms' ), $category_args );
-		register_taxonomy_for_object_type( 'give_forms_category', 'give_forms' );
+	if (give_is_setting_enabled(give_get_option('categories', 'disabled'))) {
+		register_taxonomy('give_forms_category', array('give_forms'), $category_args);
+		register_taxonomy_for_object_type('give_forms_category', 'give_forms');
 	}
 
 	/** Tags */
 	$tag_labels = array(
-		'name'                  => _x( 'Form Tags', 'taxonomy general name', 'give' ),
-		'singular_name'         => _x( 'Tag', 'taxonomy singular name', 'give' ),
-		'search_items'          => __( 'Search Tags', 'give' ),
-		'all_items'             => __( 'All Tags', 'give' ),
-		'parent_item'           => __( 'Parent Tag', 'give' ),
-		'parent_item_colon'     => __( 'Parent Tag:', 'give' ),
-		'edit_item'             => __( 'Edit Tag', 'give' ),
-		'update_item'           => __( 'Update Tag', 'give' ),
-		'add_new_item'          => __( 'Add New Tag', 'give' ),
-		'new_item_name'         => __( 'New Tag Name', 'give' ),
-		'menu_name'             => __( 'Tags', 'give' ),
-		'choose_from_most_used' => __( 'Choose from most used tags.', 'give' ),
+		'name'                  => _x('Form Tags', 'taxonomy general name', 'give'),
+		'singular_name'         => _x('Tag', 'taxonomy singular name', 'give'),
+		'search_items'          => __('Search Tags', 'give'),
+		'all_items'             => __('All Tags', 'give'),
+		'parent_item'           => __('Parent Tag', 'give'),
+		'parent_item_colon'     => __('Parent Tag:', 'give'),
+		'edit_item'             => __('Edit Tag', 'give'),
+		'update_item'           => __('Update Tag', 'give'),
+		'add_new_item'          => __('Add New Tag', 'give'),
+		'new_item_name'         => __('New Tag Name', 'give'),
+		'menu_name'             => __('Tags', 'give'),
+		'choose_from_most_used' => __('Choose from most used tags.', 'give'),
 	);
 
-	$tag_args = apply_filters( 'give_forms_tag_args', array(
+	$tag_args = apply_filters('give_forms_tag_args', array(
 			'hierarchical' => false,
-			'labels'       => apply_filters( 'give_forms_tag_labels', $tag_labels ),
+			'labels'       => apply_filters('give_forms_tag_labels', $tag_labels),
 			'show_ui'      => true,
 			'query_var'    => 'give_forms_tag',
-			'rewrite'      => array( 'slug' => $slug . '/tag', 'with_front' => false, 'hierarchical' => true ),
+			'rewrite'      => array('slug' => $slug.'/tag', 'with_front' => false, 'hierarchical' => true),
 			'capabilities' => array(
 				'manage_terms' => 'manage_give_form_terms',
 				'edit_terms'   => 'edit_give_form_terms',
@@ -207,14 +207,14 @@ 
 		)
 	);
 
-	if ( give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) ) ) {
-		register_taxonomy( 'give_forms_tag', array( 'give_forms' ), $tag_args );
-		register_taxonomy_for_object_type( 'give_forms_tag', 'give_forms' );
+	if (give_is_setting_enabled(give_get_option('tags', 'disabled'))) {
+		register_taxonomy('give_forms_tag', array('give_forms'), $tag_args);
+		register_taxonomy_for_object_type('give_forms_tag', 'give_forms');
 	}
 
 }
 
-add_action( 'init', 'give_setup_taxonomies', 0 );
+add_action('init', 'give_setup_taxonomies', 0);
 
 
 /**
@@ -225,11 +225,11 @@ 
  */
 function give_get_default_form_labels() {
 	$defaults = array(
-		'singular' => __( 'Form', 'give' ),
-		'plural'   => __( 'Forms', 'give' ),
+		'singular' => __('Form', 'give'),
+		'plural'   => __('Forms', 'give'),
 	);
 
-	return apply_filters( 'give_default_form_name', $defaults );
+	return apply_filters('give_default_form_name', $defaults);
 }
 
 /**
@@ -241,10 +241,10 @@ 
  *
  * @return string $defaults['singular'] Singular label
  */
-function give_get_forms_label_singular( $lowercase = false ) {
+function give_get_forms_label_singular($lowercase = false) {
 	$defaults = give_get_default_form_labels();
 
-	return ( $lowercase ) ? strtolower( $defaults['singular'] ) : $defaults['singular'];
+	return ($lowercase) ? strtolower($defaults['singular']) : $defaults['singular'];
 }
 
 /**
@@ -253,10 +253,10 @@ 
  * @since 1.0
  * @return string $defaults['plural'] Plural label
  */
-function give_get_forms_label_plural( $lowercase = false ) {
+function give_get_forms_label_plural($lowercase = false) {
 	$defaults = give_get_default_form_labels();
 
-	return ( $lowercase ) ? strtolower( $defaults['plural'] ) : $defaults['plural'];
+	return ($lowercase) ? strtolower($defaults['plural']) : $defaults['plural'];
 }
 
 /**
@@ -268,24 +268,24 @@ 
  *
  * @return string $title New placeholder text
  */
-function give_change_default_title( $title ) {
+function give_change_default_title($title) {
 	// If a frontend plugin uses this filter (check extensions before changing this function)
-	if ( ! is_admin() ) {
-		$title = __( 'Enter form title here', 'give' );
+	if ( ! is_admin()) {
+		$title = __('Enter form title here', 'give');
 
 		return $title;
 	}
 
 	$screen = get_current_screen();
 
-	if ( 'give_forms' == $screen->post_type ) {
-		$title = __( 'Enter form title here', 'give' );
+	if ('give_forms' == $screen->post_type) {
+		$title = __('Enter form title here', 'give');
 	}
 
 	return $title;
 }
 
-add_filter( 'enter_title_here', 'give_change_default_title' );
+add_filter('enter_title_here', 'give_change_default_title');
 
 /**
  * Registers Custom Post Statuses which are used by the Payments
@@ -295,67 +295,67 @@ 
  */
 function give_register_post_type_statuses() {
 	// Payment Statuses
-	register_post_status( 'refunded', array(
-		'label'                     => __( 'Refunded', 'give' ),
+	register_post_status('refunded', array(
+		'label'                     => __('Refunded', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Refunded <span class="count">(%s)</span>', 'Refunded <span class="count">(%s)</span>', 'give' ),
-	) );
-	register_post_status( 'failed', array(
-		'label'                     => __( 'Failed', 'give' ),
+		'label_count'               => _n_noop('Refunded <span class="count">(%s)</span>', 'Refunded <span class="count">(%s)</span>', 'give'),
+	));
+	register_post_status('failed', array(
+		'label'                     => __('Failed', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'give' ),
-	) );
-	register_post_status( 'revoked', array(
-		'label'                     => __( 'Revoked', 'give' ),
+		'label_count'               => _n_noop('Failed <span class="count">(%s)</span>', 'Failed <span class="count">(%s)</span>', 'give'),
+	));
+	register_post_status('revoked', array(
+		'label'                     => __('Revoked', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Revoked <span class="count">(%s)</span>', 'Revoked <span class="count">(%s)</span>', 'give' ),
-	) );
-	register_post_status( 'cancelled', array(
-		'label'                     => __( 'Cancelled', 'give' ),
+		'label_count'               => _n_noop('Revoked <span class="count">(%s)</span>', 'Revoked <span class="count">(%s)</span>', 'give'),
+	));
+	register_post_status('cancelled', array(
+		'label'                     => __('Cancelled', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Cancelled <span class="count">(%s)</span>', 'Cancelled <span class="count">(%s)</span>', 'give' ),
-	) );
-	register_post_status( 'abandoned', array(
-		'label'                     => __( 'Abandoned', 'give' ),
+		'label_count'               => _n_noop('Cancelled <span class="count">(%s)</span>', 'Cancelled <span class="count">(%s)</span>', 'give'),
+	));
+	register_post_status('abandoned', array(
+		'label'                     => __('Abandoned', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Abandoned <span class="count">(%s)</span>', 'Abandoned <span class="count">(%s)</span>', 'give' ),
-	) );
-	register_post_status( 'processing', array(
-		'label'                     => _x( 'Processing', 'Processing payment status', 'give' ),
+		'label_count'               => _n_noop('Abandoned <span class="count">(%s)</span>', 'Abandoned <span class="count">(%s)</span>', 'give'),
+	));
+	register_post_status('processing', array(
+		'label'                     => _x('Processing', 'Processing payment status', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Processing <span class="count">(%s)</span>', 'Processing <span class="count">(%s)</span>', 'give' )
-	)  );
+		'label_count'               => _n_noop('Processing <span class="count">(%s)</span>', 'Processing <span class="count">(%s)</span>', 'give')
+	));
 
-	register_post_status( 'preapproval', array(
-		'label'                     => _x( 'Preapproval', 'Preapproval payment status', 'give' ),
+	register_post_status('preapproval', array(
+		'label'                     => _x('Preapproval', 'Preapproval payment status', 'give'),
 		'public'                    => true,
 		'exclude_from_search'       => false,
 		'show_in_admin_all_list'    => true,
 		'show_in_admin_status_list' => true,
-		'label_count'               => _n_noop( 'Preapproval <span class="count">(%s)</span>', 'Preapproval <span class="count">(%s)</span>', 'give' ),
-	) );
+		'label_count'               => _n_noop('Preapproval <span class="count">(%s)</span>', 'Preapproval <span class="count">(%s)</span>', 'give'),
+	));
 
 }
 
-add_action( 'init', 'give_register_post_type_statuses' );
+add_action('init', 'give_register_post_type_statuses');
 
 /**
  * Updated Messages
@@ -368,27 +368,27 @@ 
  *
  * @return array $messages New post updated messages
  */
-function give_updated_messages( $messages ) {
+function give_updated_messages($messages) {
 	global $post, $post_ID;
 
-	if ( ! give_is_setting_enabled( give_get_option( 'forms_singular' ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_option('forms_singular'))) {
 
 		$messages['give_forms'] = array(
-			1 => __( 'Form updated.', 'give' ),
-			4 => __( 'Form updated.', 'give' ),
-			6 => __( 'Form published.', 'give' ),
-			7 => __( 'Form saved.', 'give' ),
-			8 => __( 'Form submitted.', 'give' ),
+			1 => __('Form updated.', 'give'),
+			4 => __('Form updated.', 'give'),
+			6 => __('Form published.', 'give'),
+			7 => __('Form saved.', 'give'),
+			8 => __('Form submitted.', 'give'),
 		);
 
 	} else {
 
 		$messages['give_forms'] = array(
-			1 => sprintf( '%1$s <a href="%2$s">%3$s</a>', __( 'Form updated.', 'give' ), get_permalink( $post_ID ), __( 'View Form', 'give' ) ),
-			4 => sprintf( '%1$s <a href="%2$s">%3$s</a>', __( 'Form updated.', 'give' ), get_permalink( $post_ID ), __( 'View Form', 'give' ) ),
-			6 => sprintf( '%1$s <a href="%2$s">%3$s</a>', __( 'Form published.', 'give' ), get_permalink( $post_ID ), __( 'View Form', 'give' ) ),
-			7 => sprintf( '%1$s <a href="%2$s">%3$s</a>', __( 'Form saved.', 'give' ), get_permalink( $post_ID ), __( 'View Form', 'give' ) ),
-			8 => sprintf( '%1$s <a href="%2$s">%3$s</a>', __( 'Form submitted.', 'give' ), get_permalink( $post_ID ), __( 'View Form', 'give' ) ),
+			1 => sprintf('%1$s <a href="%2$s">%3$s</a>', __('Form updated.', 'give'), get_permalink($post_ID), __('View Form', 'give')),
+			4 => sprintf('%1$s <a href="%2$s">%3$s</a>', __('Form updated.', 'give'), get_permalink($post_ID), __('View Form', 'give')),
+			6 => sprintf('%1$s <a href="%2$s">%3$s</a>', __('Form published.', 'give'), get_permalink($post_ID), __('View Form', 'give')),
+			7 => sprintf('%1$s <a href="%2$s">%3$s</a>', __('Form saved.', 'give'), get_permalink($post_ID), __('View Form', 'give')),
+			8 => sprintf('%1$s <a href="%2$s">%3$s</a>', __('Form submitted.', 'give'), get_permalink($post_ID), __('View Form', 'give')),
 		);
 
 	}
@@ -396,24 +396,24 @@ 
 	return $messages;
 }
 
-add_filter( 'post_updated_messages', 'give_updated_messages' );
+add_filter('post_updated_messages', 'give_updated_messages');
 
 /**
  * Ensure post thumbnail support is turned on
  */
 function give_add_thumbnail_support() {
-	if ( ! give_is_setting_enabled( give_get_option( 'form_featured_img' ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_option('form_featured_img'))) {
 		return;
 	}
 
-	if ( ! current_theme_supports( 'post-thumbnails' ) ) {
-		add_theme_support( 'post-thumbnails' );
+	if ( ! current_theme_supports('post-thumbnails')) {
+		add_theme_support('post-thumbnails');
 	}
 
-	add_post_type_support( 'give_forms', 'thumbnail' );
+	add_post_type_support('give_forms', 'thumbnail');
 }
 
-add_action( 'after_setup_theme', 'give_add_thumbnail_support', 10 );
+add_action('after_setup_theme', 'give_add_thumbnail_support', 10);
 
 /**
  * Give Sidebars
@@ -424,21 +424,21 @@ 
 
 	// Single Give Forms (disabled if single turned off in settings)
 	if (
-		give_is_setting_enabled( give_get_option( 'forms_singular' ) )
-		&& give_is_setting_enabled( give_get_option( 'form_sidebar' ) )
+		give_is_setting_enabled(give_get_option('forms_singular'))
+		&& give_is_setting_enabled(give_get_option('form_sidebar'))
 	) {
 
-		register_sidebar( apply_filters( 'give_forms_single_sidebar', array(
-			'name'          => __( 'Give Single Form Sidebar', 'give' ),
+		register_sidebar(apply_filters('give_forms_single_sidebar', array(
+			'name'          => __('Give Single Form Sidebar', 'give'),
 			'id'            => 'give-forms-sidebar',
-			'description'   => __( 'Widgets in this area will be shown on the single Give forms aside area. This sidebar will not display for embedded forms.', 'give' ),
+			'description'   => __('Widgets in this area will be shown on the single Give forms aside area. This sidebar will not display for embedded forms.', 'give'),
 			'before_widget' => '<div id="%1$s" class="widget %2$s">',
 			'after_widget'  => '</div>',
 			'before_title'  => '<h3 class="widgettitle widget-title">',
 			'after_title'   => '</h3>',
-		) ) );
+		)));
 
 	}
 }
 
-add_action( 'widgets_init', 'give_widgets_init', 999 );
+add_action('widgets_init', 'give_widgets_init', 999);

includes/admin/tools/data/class-give-tools-delete-test-transactions.php

Spacing +56 added lines, -56 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,11 +272,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);
 	}
 
 	/**
@@ -289,10 +289,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,
@@ -306,7 +306,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -318,9 +318,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-donors.php

Spacing +91 added lines, -91 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -109,32 +109,32 @@ 
 		$donor_ids    = array();
 
 		// Check if the ajax request if running for the first time.
-		if ( 1 === (int) $this->step ) {
+		if (1 === (int) $this->step) {
 			// Delete all the donation ids.
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 			// Delete all the donor ids.
-			$this->delete_option( $this->donor_key );
+			$this->delete_option($this->donor_key);
 
 			// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
-			$this->update_option( $this->step_key, 'count' );
+			$this->update_option($this->step_key, 'count');
 
 			// Delete tha page count of the step.
-			$this->update_option( $this->step_on_key, '0' );
+			$this->update_option($this->step_on_key, '0');
 		} else {
 			// Get the old donors list.
-			$donor_ids = $this->get_option( $this->donor_key );
+			$donor_ids = $this->get_option($this->donor_key);
 
 			// Get the old donation list.
-			$donation_ids = $this->get_option( $this->donation_key );
+			$donation_ids = $this->get_option($this->donation_key);
 		}
 
 		// Get the step and check for it if it's on the first step( 'count' ) or not.
 		$step = (int) $this->get_step();
-		if ( 1 === $step ) {
+		if (1 === $step) {
 			/**
 			 * Will add or update the donation and donor data by running wp query.
 			 */
-			$this->count( $step, $donation_ids, $donor_ids );
+			$this->count($step, $donation_ids, $donor_ids);
 		}
 	}
 
@@ -145,17 +145,17 @@ 
 	 * @param array  $donation_ids Contain the list of all the donation id's that has being add before this
 	 * @param array  $donor_ids    Contain the list of all the donors id's that has being add before this
 	 */
-	private function count( $step, $donation_ids = array(), $donor_ids = array() ) {
+	private function count($step, $donation_ids = array(), $donor_ids = array()) {
 
 		// Get the Page count by default it's zero.
 		$paged = (int) $this->get_step_page();
 		// Incresed the page count by one.
-		++ $paged;
+		++$paged;
 
 		/**
 		 * Filter add to alter the argument before the wp quest run
 		 */
-		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+		$args = apply_filters('give_tools_reset_stats_total_args', array(
 			'post_type'      => 'give_payment',
 			'post_status'    => 'any',
 			'posts_per_page' => $this->per_step,
@@ -163,16 +163,16 @@ 
 			// ONLY TEST MODE TRANSACTIONS!!!
 			'meta_key'       => '_give_payment_mode',
 			'meta_value'     => 'test',
-		) );
+		));
 
 		// Reset the post data.
 		wp_reset_postdata();
 		// Getting the new donation.
-		$donation_posts = new WP_Query( $args );
+		$donation_posts = new WP_Query($args);
 
 		// The Loop.
-		if ( $donation_posts->have_posts() ) {
-			while ( $donation_posts->have_posts() ) {
+		if ($donation_posts->have_posts()) {
+			while ($donation_posts->have_posts()) {
 				$donation_posts->the_post();
 				global $post;
 				// Add the donation id in side the array.
@@ -191,23 +191,23 @@ 
 		$max_num_pages = (int) $donation_posts->max_num_pages;
 
 		// Check current page is less then max number of page or not
-		if ( $paged < $max_num_pages ) {
+		if ($paged < $max_num_pages) {
 			// Update the curretn page virable for the next step
-			$this->update_option( $this->step_on_key, $paged );
+			$this->update_option($this->step_on_key, $paged);
 
 			// Calculating percentage.
 			$page_remain          = $max_num_pages - $paged;
-			$this->total_step     = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
+			$this->total_step     = (int) $max_num_pages + ($total_donation / $this->per_step) + (($page_remain * 2) * count($donor_ids));
 			$this->step_completed = $paged;
 		} else {
-			$donation_ids_count = count( $donor_ids );
-			$this->update_option( $this->step_key, 'donation' );
-			$this->update_option( $this->step_on_key, '0' );
+			$donation_ids_count = count($donor_ids);
+			$this->update_option($this->step_key, 'donation');
+			$this->update_option($this->step_on_key, '0');
 		}
 
-		$donor_ids = array_unique( $donor_ids );
-		$this->update_option( $this->donor_key, $donor_ids );
-		$this->update_option( $this->donation_key, $donation_ids );
+		$donor_ids = array_unique($donor_ids);
+		$this->update_option($this->donor_key, $donor_ids);
+		$this->update_option($this->donation_key, $donation_ids);
 
 		wp_reset_postdata();
 	}
@@ -219,34 +219,34 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
+		return ceil((100 * $this->step_completed) / $this->total_step);
 	}
 
 	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_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			// 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 donor and transactions successfully deleted.', 'give' );
+			$this->message = __('Test donor and transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -264,12 +264,12 @@ 
 	public function get_data() {
 
 		// Get the donation id's.
-		$donation_ids = $this->get_option( $this->donation_key );
+		$donation_ids = $this->get_option($this->donation_key);
 
 		/**
 		 * Return false id not test donation is found.
 		 */
-		if ( empty( $donation_ids ) ) {
+		if (empty($donation_ids)) {
 			$this->is_empty   = true;
 			$this->total_step = 1;
 
@@ -280,81 +280,81 @@ 
 		$step = (int) $this->get_step();
 
 		// get teh donor ids.
-		$donor_ids = $this->get_option( $this->donor_key );
+		$donor_ids = $this->get_option($this->donor_key);
 
 		// In step to we delete all the donation in loop.
-		if ( 2 === $step ) {
+		if (2 === $step) {
 			$pass_to_donor = false;
 			$page          = (int) $this->get_step_page();
-			$page ++;
-			$count = count( $donation_ids );
+			$page++;
+			$count = count($donation_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
 			$this->step_completed = $page;
 
 
-			if ( $count > $this->per_step ) {
+			if ($count > $this->per_step) {
 
-				$this->update_option( $this->step_on_key, $page );
-				$donation_ids = $this->get_delete_ids( $donation_ids, $page );
-				$current_page = (int) ceil( $count / $this->per_step );
+				$this->update_option($this->step_on_key, $page);
+				$donation_ids = $this->get_delete_ids($donation_ids, $page);
+				$current_page = (int) ceil($count / $this->per_step);
 
-				if ( $page === $current_page ) {
+				if ($page === $current_page) {
 					$pass_to_donor = true;
 				}
 			} else {
 				$pass_to_donor = true;
 			}
 
-			if ( true === $pass_to_donor ) {
-				$this->update_option( $this->step_key, 'donor' );
-				$this->update_option( $this->step_on_key, '0' );
+			if (true === $pass_to_donor) {
+				$this->update_option($this->step_key, 'donor');
+				$this->update_option($this->step_on_key, '0');
 			}
 
 			global $wpdb;
-			foreach ( $donation_ids as $item ) {
+			foreach ($donation_ids as $item) {
 
 				// will delete the payment log first.
-				$parent_query = $wpdb->prepare( "SELECT post_id as id FROM $wpdb->postmeta WHERE meta_key = %s AND meta_value = %d", '_give_log_payment_id', (int) $item );
-				$log_id       = $wpdb->get_row( $parent_query, ARRAY_A );
+				$parent_query = $wpdb->prepare("SELECT post_id as id FROM $wpdb->postmeta WHERE meta_key = %s AND meta_value = %d", '_give_log_payment_id', (int) $item);
+				$log_id       = $wpdb->get_row($parent_query, ARRAY_A);
 				// Check if payment has it log or not if yes then delete it.
-				if ( ! empty( $log_id['id'] ) ) {
+				if ( ! empty($log_id['id'])) {
 					// Deleting the payment log.
-					wp_delete_post( $log_id['id'], true );
+					wp_delete_post($log_id['id'], true);
 				}
 
 				// Delete the main payment.
-				wp_delete_post( $item, true );
+				wp_delete_post($item, true);
 			}
-			do_action( 'give_delete_log_cache' );
+			do_action('give_delete_log_cache');
 		}
 
 
 		// Here we delete all the donor
-		if ( 3 === $step ) {
+		if (3 === $step) {
 			$page  = (int) $this->get_step_page();
-			$count = count( $donor_ids );
+			$count = count($donor_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
-			$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
+			$this->step_completed = $page + (count($donation_ids) / $this->per_step);
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_type'      => 'give_payment',
 				'post_status'    => 'any',
 				'posts_per_page' => 1,
 				'meta_key'       => '_give_payment_mode',
 				'meta_value'     => 'live',
-				'author'         => $donor_ids[ $page ],
-			) );
+				'author'         => $donor_ids[$page],
+			));
 
-			$donation_posts = get_posts( $args );
-			if ( empty( $donation_posts ) ) {
-				Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
+			$donation_posts = get_posts($args);
+			if (empty($donation_posts)) {
+				Give()->donors->delete_by_user_id($donor_ids[$page]);
 			}
 
-			$page ++;
-			$this->update_option( $this->step_on_key, $page );
-			if ( $count === $page ) {
+			$page++;
+			$this->update_option($this->step_on_key, $page);
+			if ($count === $page) {
 				$this->is_empty = false;
 
 				return false;
@@ -366,24 +366,24 @@ 
 		return true;
 	}
 
-	public function get_delete_ids( $donation_ids, $page ) {
-		$index            = $page --;
-		$count            = count( $donation_ids );
+	public function get_delete_ids($donation_ids, $page) {
+		$index            = $page--;
+		$count            = count($donation_ids);
 		$temp             = 0;
 		$current_page     = 0;
 		$post_delete      = $this->per_step;
 		$page_donation_id = array();
 
-		foreach ( $donation_ids as $item ) {
-			$temp ++;
-			$page_donation_id[ $current_page ][] = $item;
-			if ( $temp === $post_delete ) {
-				$current_page ++;
+		foreach ($donation_ids as $item) {
+			$temp++;
+			$page_donation_id[$current_page][] = $item;
+			if ($temp === $post_delete) {
+				$current_page++;
 				$temp = 0;
 			}
 		}
 
-		return $page_donation_id[ $page ];
+		return $page_donation_id[$page];
 	}
 
 	/**
@@ -395,8 +395,8 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	public function get_option( $key, $defalut_value = false ) {
-		return get_option( $key, $defalut_value );
+	public function get_option($key, $defalut_value = false) {
+		return get_option($key, $defalut_value);
 	}
 
 	/**
@@ -409,8 +409,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_option( $key, $value ) {
-		update_option( $key, $value, false );
+	public function update_option($key, $value) {
+		update_option($key, $value, false);
 	}
 
 	/**
@@ -422,8 +422,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -434,12 +434,12 @@ 
 	 * @return int|string
 	 */
 	private function get_step() {
-		$step_key = (string) $this->get_option( $this->step_key, false );
-		if ( 'count' === $step_key ) {
+		$step_key = (string) $this->get_option($this->step_key, false);
+		if ('count' === $step_key) {
 			return 1;
-		} elseif ( 'donation' === $step_key ) {
+		} elseif ('donation' === $step_key) {
 			return 2;
-		} elseif ( 'donor' === $step_key ) {
+		} elseif ('donor' === $step_key) {
 			return 3;
 		} else {
 			return $step_key;
@@ -450,6 +450,6 @@ 
 	 * Get the current $page value in the ajax.
 	 */
 	private function get_step_page() {
-		return $this->get_option( $this->step_on_key, false );
+		return $this->get_option($this->step_on_key, false);
 	}
 }

includes/admin/tools/logs/class-sales-logs-list-table.php

Spacing +81 added lines, -81 removed lines

@@ -9,13 +9,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';
 }
 
 /**
@@ -44,13 +44,13 @@ 
 		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
-			'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?
+		));
 
-		add_action( 'give_log_view_actions', array( $this, 'give_forms_filter' ) );
+		add_action('give_log_view_actions', array($this, 'give_forms_filter'));
 	}
 
 	/**
@@ -64,41 +64,41 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		$payment = give_get_payment_by( 'id', $item['payment_id'] );
+		$payment = give_get_payment_by('id', $item['payment_id']);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'form' :
-				$form_title = get_the_title( $item[ $column_name ] );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $item[ $column_name ] ) : $form_title;
-				return '<a href="' . esc_url( add_query_arg( 'form', $item[ $column_name ] ) ) . '" >' . $form_title . '</a>';
+				$form_title = get_the_title($item[$column_name]);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $item[$column_name]) : $form_title;
+				return '<a href="'.esc_url(add_query_arg('form', $item[$column_name])).'" >'.$form_title.'</a>';
 
 			case 'donor_id' :
 				return sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['donor_id'] ) ),
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['donor_id'])),
 					$item['donor_name']
 				);
 
 			case 'amount' :
-				return give_currency_filter( give_format_amount( $item['amount'], array( 'sanitize' => false ) ) );
+				return give_currency_filter(give_format_amount($item['amount'], array('sanitize' => false)));
 
 			case 'status' :
 
-				$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>';
+				$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' ) . '">' . __( 'Test', 'give' ) . '</span>';
+				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').'">'.__('Test', 'give').'</span>';
 				}
 
 				return $value;
 
 			case 'payment_id' :
-				return '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $item['payment_id'] ) . '">' . give_get_payment_number( $item['payment_id'] ) . '</a>';
+				return '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$item['payment_id']).'">'.give_get_payment_number($item['payment_id']).'</a>';
 
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -111,13 +111,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => __( 'Log ID', 'give' ),
-			'donor_id'   => __( 'Donor', 'give' ),
-			'form'       => __( 'Form', 'give' ),
-			'amount'     => __( 'Donation Amount', 'give' ),
-			'status'     => __( 'Status', 'give' ),
-			'payment_id' => __( 'Donation ID', 'give' ),
-			'date'       => __( 'Date', 'give' ),
+			'ID'         => __('Log ID', 'give'),
+			'donor_id'   => __('Donor', 'give'),
+			'form'       => __('Form', 'give'),
+			'amount'     => __('Donation Amount', 'give'),
+			'status'     => __('Status', 'give'),
+			'payment_id' => __('Donation ID', 'give'),
+			'date'       => __('Date', 'give'),
 		);
 
 		return $columns;
@@ -131,7 +131,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;
 	}
 
 	/**
@@ -142,7 +142,7 @@ 
 	 * @return mixed int If User ID, string If Email/Login
 	 */
 	public function get_filtered_user() {
-		return isset( $_GET['user'] ) ? absint( $_GET['user'] ) : false;
+		return isset($_GET['user']) ? absint($_GET['user']) : false;
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 	 * @return int Download ID
 	 */
 	public function get_filtered_give_form() {
-		return ! empty( $_GET['form'] ) ? absint( $_GET['form'] ) : false;
+		return ! empty($_GET['form']) ? absint($_GET['form']) : false;
 	}
 
 	/**
@@ -164,7 +164,7 @@ 
 	 * @return string|bool 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;
 	}
 
 
@@ -180,20 +180,20 @@ 
 	 *
 	 * @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 <?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"/>
@@ -217,7 +217,7 @@ 
 
 		$meta_query = array();
 
-		if ( $user ) {
+		if ($user) {
 			// Show only logs from a specific user.
 			$meta_query[] = array(
 				'key'   => '_give_log_user_id',
@@ -226,8 +226,8 @@ 
 		}
 
 		$search = $this->get_search();
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				// This is an email search. We use this to ensure it works for guest users and logged-in users.
 				$key     = '_give_log_user_info';
 				$compare = 'LIKE';
@@ -236,32 +236,32 @@ 
 				$key     = '_give_log_user_id';
 				$compare = 'LIKE';
 
-				if ( ! is_numeric( $search ) ) {
+				if ( ! is_numeric($search)) {
 					// Searching for user by username
-					$user = get_user_by( 'login', $search );
+					$user = get_user_by('login', $search);
 
-					if ( $user ) {
+					if ($user) {
 						// Found one, set meta value to user's ID.
 						$search = $user->ID;
 					} else {
 						// No user found so let's do a real search query.
-						$users = new WP_User_Query( array(
+						$users = new WP_User_Query(array(
 							'search'         => $search,
-							'search_columns' => array( 'user_url', 'user_nicename' ),
+							'search_columns' => array('user_url', 'user_nicename'),
 							'number'         => 1,
 							'fields'         => 'ids',
-						) );
+						));
 
 						$found_user = $users->get_results();
 
-						if ( $found_user ) {
+						if ($found_user) {
 							$search = $found_user[0];
 						}
 					}
 				}
 			}
 
-			if ( ! $this->file_search ) {
+			if ( ! $this->file_search) {
 				// Meta query only works for non file name search.
 				$meta_query[] = array(
 					'key'     => $key,
@@ -282,7 +282,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -294,24 +294,24 @@ 
 	 * @return void
 	 */
 	public function give_forms_filter() {
-		$give_forms = get_posts( array(
+		$give_forms = get_posts(array(
 			'post_type'              => 'give_forms',
 			'post_status'            => 'any',
-			'posts_per_page'         => - 1,
+			'posts_per_page'         => -1,
 			'orderby'                => 'title',
 			'order'                  => 'ASC',
 			'fields'                 => 'ids',
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
-		) );
+		));
 
-		if ( $give_forms ) {
+		if ($give_forms) {
 			echo '<select name="form" id="give-log-form-filter">';
-			echo '<option value="0">' . __( 'All', 'give' ) . '</option>';
-			foreach ( $give_forms as $form ) {
-				$form_title = get_the_title( $form );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form ) : $form_title;
-				echo '<option value="' . $form . '"' . selected( $form, $this->get_filtered_give_form() ) . '>' . esc_html( $form_title ) . '</option>';
+			echo '<option value="0">'.__('All', 'give').'</option>';
+			foreach ($give_forms as $form) {
+				$form_title = get_the_title($form);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $form) : $form_title;
+				echo '<option value="'.$form.'"'.selected($form, $this->get_filtered_give_form()).'>'.esc_html($form_title).'</option>';
 			}
 			echo '</select>';
 		}
@@ -331,7 +331,7 @@ 
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
-		$give_form = empty( $_GET['s'] ) ? $this->get_filtered_give_form() : null;
+		$give_form = empty($_GET['s']) ? $this->get_filtered_give_form() : null;
 		$user      = $this->get_filtered_user();
 
 		$log_query = array(
@@ -342,37 +342,37 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$cache_key = Give_Cache::get_key( 'get_logs', $log_query );
+		$cache_key = Give_Cache::get_key('get_logs', $log_query);
 
 		// Return result from cache if exist.
-		if ( ! ( $logs_data = Give_Cache::get( $cache_key ) ) ) {
-			$logs = $give_logs->get_connected_logs( $log_query );
+		if ( ! ($logs_data = Give_Cache::get($cache_key))) {
+			$logs = $give_logs->get_connected_logs($log_query);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					$payment_id = give_get_meta( $log->ID, '_give_log_payment_id', true );
+			if ($logs) {
+				foreach ($logs as $log) {
+					$payment_id = give_get_meta($log->ID, '_give_log_payment_id', true);
 
 					// Make sure this payment hasn't been deleted.
-					if ( get_post( $payment_id ) ) :
-						$user_info      = give_get_payment_meta_user_info( $payment_id );
-						$payment_amount = give_get_payment_amount( $payment_id );
+					if (get_post($payment_id)) :
+						$user_info      = give_get_payment_meta_user_info($payment_id);
+						$payment_amount = give_get_payment_amount($payment_id);
 
 						$logs_data[] = array(
-							'ID'         => '<span class="give-item-label give-item-label-gray">' . $log->ID . '</span>',
+							'ID'         => '<span class="give-item-label give-item-label-gray">'.$log->ID.'</span>',
 							'payment_id' => $payment_id,
 							'form'       => $log->post_parent,
 							'amount'     => $payment_amount,
-							'donor_id'   => give_get_payment_donor_id( $payment_id ),
-							'donor_name' => trim( "{$user_info['first_name']} {$user_info['last_name']}" ),
-							'date'       => get_post_field( 'post_date', $payment_id ),
+							'donor_id'   => give_get_payment_donor_id($payment_id),
+							'donor_name' => trim("{$user_info['first_name']} {$user_info['last_name']}"),
+							'date'       => get_post_field('post_date', $payment_id),
 						);
 
 					endif;
 				}
 
 				// Cache results.
-				if ( ! empty( $logs_data ) ) {
-					Give_Cache::set( $cache_key, $logs_data );
+				if ( ! empty($logs_data)) {
+					Give_Cache::set($cache_key, $logs_data);
 				}
 			}
 		}
@@ -400,15 +400,15 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array();
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$current_page          = $this->get_pagenum();
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( $this->get_filtered_give_form(), 'sale', $this->get_meta_query() );
+		$total_items           = $give_logs->get_log_count($this->get_filtered_give_form(), 'sale', $this->get_meta_query());
 
-		$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/reports/class-donor-reports-table.php

Spacing +67 added lines, -67 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';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_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 donor-search" role="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( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @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 give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h3 class="alignleft reports-earnings-title">
-					<span><?php esc_html_e( 'Donors Report', 'give' ); ?></span>
+					<span><?php esc_html_e('Donors Report', 'give'); ?></span>
 				</h3>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,35 +172,35 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['id'] ) ),
-						esc_html( $item['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['id'])),
+						esc_html($item['num_donations'])
 					);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -212,13 +212,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -231,10 +231,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -245,7 +245,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -257,7 +257,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;
 	}
 
 	/**
@@ -268,7 +268,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;
 	}
 
 	/**
@@ -287,15 +287,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -323,9 +323,9 @@ 
 
 		$_donor_query['number'] = -1;
 		$_donor_query['offset'] = 0;
-		$donors = Give()->donors->get_donors( $_donor_query );
+		$donors = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -337,10 +337,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -349,10 +349,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -379,16 +379,16 @@ 
 		$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();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/upgrades/class-give-updates.php

Spacing +79 added lines, -79 removed lines

@@ -70,24 +70,24 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function register( $args ) {
+	public function register($args) {
 		$args_default = array(
 			'id'       => '',
 			'version'  => '',
 			'callback' => '',
 		);
 
-		$args = wp_parse_args( $args, $args_default );
+		$args = wp_parse_args($args, $args_default);
 
 		// You can only register database upgrade.
 		$args['type'] = 'database';
 
 		// Bailout.
-		if ( empty( $args['id'] ) || empty( $args['version'] ) || empty( $args['callback'] ) || ! is_callable( $args['callback'] ) ) {
+		if (empty($args['id']) || empty($args['version']) || empty($args['callback']) || ! is_callable($args['callback'])) {
 			return;
 		}
 
-		$this->updates[ $args['type'] ][] = $args;
+		$this->updates[$args['type']][] = $args;
 	}
 
 
@@ -102,32 +102,32 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_updates( $update_type = '', $status = 'all' ) {
+	public function get_updates($update_type = '', $status = 'all') {
 		// return all updates.
-		if ( empty( $update_type ) ) {
+		if (empty($update_type)) {
 			return $this->updates;
 		}
 
 		// Get specific update.
-		$updates = ! empty( $this->updates[ $update_type ] ) ? $this->updates[ $update_type ] : array();
+		$updates = ! empty($this->updates[$update_type]) ? $this->updates[$update_type] : array();
 
 		// Bailout.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			return $updates;
 		}
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'new':
 				// Remove already completed updates.
 				$completed_updates = give_get_completed_upgrades();
 
-				if ( ! empty( $completed_updates ) ) {
-					foreach ( $updates as $index => $update ) {
-						if ( in_array( $update['id'], $completed_updates ) ) {
-							unset( $updates[ $index ] );
+				if ( ! empty($completed_updates)) {
+					foreach ($updates as $index => $update) {
+						if (in_array($update['id'], $completed_updates)) {
+							unset($updates[$index]);
 						}
 					}
-					$updates = array_values( $updates );
+					$updates = array_values($updates);
 				}
 
 				break;
@@ -144,7 +144,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( is_null( self::$instance ) ) {
+		if (is_null(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -162,16 +162,16 @@ 
 		/**
 		 * Setup hooks.
 		 */
-		add_action( 'init', array( $this, '__register_upgrade' ), 9999 );
-		add_action( 'admin_init', array( $this, '__change_donations_label' ), 9999 );
-		add_action( 'admin_menu', array( $this, '__register_menu' ), 9999 );
-		add_action( 'give_set_upgrade_completed', array( $this, '__flush_resume_updates' ), 9999 );
-		add_action( 'wp_ajax_give_do_ajax_updates', array( $this, '__give_ajax_updates' ) );
+		add_action('init', array($this, '__register_upgrade'), 9999);
+		add_action('admin_init', array($this, '__change_donations_label'), 9999);
+		add_action('admin_menu', array($this, '__register_menu'), 9999);
+		add_action('give_set_upgrade_completed', array($this, '__flush_resume_updates'), 9999);
+		add_action('wp_ajax_give_do_ajax_updates', array($this, '__give_ajax_updates'));
 
 		/**
 		 * Load file
 		 */
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 	}
 
 	/**
@@ -184,12 +184,12 @@ 
 		$addons         = give_get_plugins();
 		$plugin_updates = get_plugin_updates();
 
-		foreach ( $addons as $key => $info ) {
-			if ( 'active' != $info['Status'] || 'add-on' != $info['Type'] || empty( $plugin_updates[ $key ] ) ) {
+		foreach ($addons as $key => $info) {
+			if ('active' != $info['Status'] || 'add-on' != $info['Type'] || empty($plugin_updates[$key])) {
 				continue;
 			}
 
-			$this->updates['plugin'][] = array_merge( $info, (array) $plugin_updates[ $key ] );
+			$this->updates['plugin'][] = array_merge($info, (array) $plugin_updates[$key]);
 		}
 	}
 
@@ -201,7 +201,7 @@ 
 	 * @access public
 	 */
 	public function __register_upgrade() {
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
@@ -210,7 +210,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_register_updates', $this );
+		do_action('give_register_updates', $this);
 	}
 
 	/**
@@ -224,17 +224,17 @@ 
 		global $submenu;
 
 		// Bailout.
-		if ( empty( $menu ) || ! $this->get_update_count() ) {
+		if (empty($menu) || ! $this->get_update_count()) {
 			return;
 		}
 
-		foreach ( $menu as $index => $menu_item ) {
-			if ( 'edit.php?post_type=give_forms' !== $menu_item[2] ) {
+		foreach ($menu as $index => $menu_item) {
+			if ('edit.php?post_type=give_forms' !== $menu_item[2]) {
 				continue;
 			}
 
-			$menu[ $index ][0] = sprintf(
-				__( 'Donations <span class="update-plugins count-%1$d"><span class="plugin-count">%1$d</span></span>', 'give' ),
+			$menu[$index][0] = sprintf(
+				__('Donations <span class="update-plugins count-%1$d"><span class="plugin-count">%1$d</span></span>', 'give'),
 				$this->get_update_count()
 			);
 
@@ -256,15 +256,15 @@ 
 		// Bailout.
 		if ( ! $this->get_update_count()) {
 			// Show complete update message if still on update setting page.
-			if ( isset($_GET['page']) && 'give-updates' === $_GET['page'] ) {
+			if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 				// Upgrades
 				add_submenu_page(
 					'edit.php?post_type=give_forms',
-					esc_html__( 'Give Updates Complete', 'give' ),
-					__( 'Updates', 'give' ),
+					esc_html__('Give Updates Complete', 'give'),
+					__('Updates', 'give'),
 					'manage_give_settings',
 					'give-updates',
-					array( $this, 'render_complete_page' )
+					array($this, 'render_complete_page')
 				);
 			}
 
@@ -274,15 +274,15 @@ 
 		// Upgrades
 		add_submenu_page(
 			'edit.php?post_type=give_forms',
-			esc_html__( 'Give Updates', 'give' ),
+			esc_html__('Give Updates', 'give'),
 			sprintf(
 				'%1$s <span class="update-plugins count-%2$d"><span class="plugin-count">%2$d</span></span>',
-				__( 'Updates', 'give' ),
+				__('Updates', 'give'),
 				$this->get_update_count()
 			),
 			'manage_give_settings',
 			'give-updates',
-			array( $this, 'render_page' )
+			array($this, 'render_page')
 		);
 	}
 
@@ -294,7 +294,7 @@ 
 	 * @return int
 	 */
 	public function get_db_update_count() {
-		return count( $this->get_updates( 'database', 'new' ) );
+		return count($this->get_updates('database', 'new'));
 	}
 
 	/**
@@ -304,7 +304,7 @@ 
 	 * @access public
 	 */
 	public function render_complete_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades-complete.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades-complete.php';
 	}
 
 	/**
@@ -314,7 +314,7 @@ 
 	 * @access public
 	 */
 	public function render_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades.php';
 	}
 
 	/**
@@ -325,7 +325,7 @@ 
 	 * @return int
 	 */
 	public function get_plugin_update_count() {
-		return count( $this->get_updates( 'plugin' ) );
+		return count($this->get_updates('plugin'));
 	}
 
 	/**
@@ -340,7 +340,7 @@ 
 		$db_update_count     = $this->get_db_update_count();
 		$plugin_update_count = $this->get_plugin_update_count();
 
-		return ( $db_update_count + $plugin_update_count );
+		return ($db_update_count + $plugin_update_count);
 	}
 
 
@@ -351,12 +351,12 @@ 
 	 * @access public
 	 */
 	public function __flush_resume_updates() {
-		delete_option( 'give_doing_upgrade' );
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+		delete_option('give_doing_upgrade');
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 
 		// Reset counter.
 		$this->step = $this->percentage = 0;
-		++ $this->update;
+		++$this->update;
 	}
 
 	/**
@@ -367,24 +367,24 @@ 
 	 */
 	public function __give_ajax_updates() {
 		// Check permission.
-		if ( ! current_user_can( 'manage_give_settings' ) ) {
+		if ( ! current_user_can('manage_give_settings')) {
 			$this->send_ajax_response(
 				array(
-					'message' => esc_html__( 'You do not have permission to do Give upgrades.', 'give' ),
+					'message' => esc_html__('You do not have permission to do Give upgrades.', 'give'),
 				),
 				'error'
 			);
 		}
 
 		// Set params.
-		$this->step   = absint( $_POST['step'] );
-		$this->update = absint( $_POST['update'] );
+		$this->step   = absint($_POST['step']);
+		$this->update = absint($_POST['update']);
 
 		// Bailout: step and update must be positive and greater then zero.
-		if ( ! $this->step ) {
+		if ( ! $this->step) {
 			$this->send_ajax_response(
 				array(
-					'message'    => __( 'Please reload this page  and try again', 'give' ),
+					'message'    => __('Please reload this page  and try again', 'give'),
 					'heading'    => '',
 					'percentage' => 0,
 				),
@@ -393,14 +393,14 @@ 
 		}
 
 		// Get updates.
-		$updates = $this->get_updates( 'database', 'new' );
+		$updates = $this->get_updates('database', 'new');
 
 		// Bailout if we do not have nay updates.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			$this->send_ajax_response(
 				array(
-					'message'    => __( 'The database is already up to date.', 'give' ),
-					'heading'    => __( 'Updates Completed.', 'give' ),
+					'message'    => __('The database is already up to date.', 'give'),
+					'heading'    => __('Updates Completed.', 'give'),
 					'percentage' => 0,
 				),
 				'success'
@@ -408,26 +408,26 @@ 
 		}
 
 		// Process update.
-		foreach ( $updates as $index => $update ) {
+		foreach ($updates as $index => $update) {
 			// Check if update depend upon any other update.
-			if ( ! empty( $update['depend'] ) && ! give_has_upgrade_completed( $update['depend'] ) ) {
+			if ( ! empty($update['depend']) && ! give_has_upgrade_completed($update['depend'])) {
 				continue;
 			}
 
 			// Run update.
-			if ( is_array( $update['callback'] ) ) {
+			if (is_array($update['callback'])) {
 				$update['callback'][0]->$update['callback'][1]();
 			} else {
 				$update['callback']();
 			}
 
 			// Check if current update completed or not.
-			if ( give_has_upgrade_completed( $update['id'] ) ) {
-				if ( 1 === count( $updates ) ) {
+			if (give_has_upgrade_completed($update['id'])) {
+				if (1 === count($updates)) {
 					$this->send_ajax_response(
 						array(
-							'message'    => __( 'Database updated successfully.', 'give' ),
-							'heading'    => __( 'Updates Completed.', 'give' ),
+							'message'    => __('Database updated successfully.', 'give'),
+							'heading'    => __('Updates Completed.', 'give'),
 							'percentage' => 0,
 						),
 						'success'
@@ -437,16 +437,16 @@ 
 
 			$doing_upgrade_args = array(
 				'update_info' => $update,
-				'step'        => ++ $this->step,
+				'step'        => ++$this->step,
 				'update'      => $this->update,
-				'heading'     => sprintf( 'Update %s of {update_count}', $this->update ),
+				'heading'     => sprintf('Update %s of {update_count}', $this->update),
 				'percentage'  => $this->percentage,
 			);
 
 			// Cache upgrade.
-			update_option( 'give_doing_upgrade', $doing_upgrade_args );
+			update_option('give_doing_upgrade', $doing_upgrade_args);
 
-			$this->send_ajax_response( $doing_upgrade_args );
+			$this->send_ajax_response($doing_upgrade_args);
 		}// End foreach().
 	}
 
@@ -459,7 +459,7 @@ 
 	 * @param        $data
 	 * @param string $type
 	 */
-	public function send_ajax_response( $data, $type = '' ) {
+	public function send_ajax_response($data, $type = '') {
 		$default = array(
 			'message'    => '',
 			'heading'    => '',
@@ -469,21 +469,21 @@ 
 		);
 
 		// Set data.
-		$data = wp_parse_args( $data, $default );
+		$data = wp_parse_args($data, $default);
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'success':
-				wp_send_json_success( $data );
+				wp_send_json_success($data);
 				break;
 
 			case 'error':
-				wp_send_json_error( $data );
+				wp_send_json_error($data);
 				break;
 
 			default:
-				wp_send_json( array(
+				wp_send_json(array(
 					'data' => $data,
-				) );
+				));
 				break;
 		}
 	}
@@ -500,8 +500,8 @@ 
 	public function resume_updates() {
 		$status = false;
 
-		if ( $update = get_option( 'give_doing_upgrade' ) ) {
-			$status = ! empty( $update['step'] ) ? $update['step'] : $status;
+		if ($update = get_option('give_doing_upgrade')) {
+			$status = ! empty($update['step']) ? $update['step'] : $status;
 		}
 
 		return $status;
@@ -517,12 +517,12 @@ 
 	 * @param $total
 	 * @param $current_total
 	 */
-	public function set_percentage( $total, $current_total ) {
+	public function set_percentage($total, $current_total) {
 		// Set percentage.
-		$this->percentage = $total ? ( ( $current_total ) / $total ) * 100 : 0;
+		$this->percentage = $total ? (($current_total) / $total) * 100 : 0;
 
 		// Verify percentage.
-		$this->percentage = ( 100 < $this->percentage ) ? 100 : $this->percentage;
+		$this->percentage = (100 < $this->percentage) ? 100 : $this->percentage;
 	}
 }
 

includes/admin/donors/donors.php

Spacing +164 added lines, -164 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -42,7 +42,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -56,7 +56,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -67,7 +67,7 @@ 
  * @return void
  */
 function give_donors_list() {
-	include dirname( __FILE__ ) . '/class-donor-table.php';
+	include dirname(__FILE__).'/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -80,11 +80,11 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
-		<form id="give-donors-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
+		<form id="give-donors-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
 			<?php
-			$donors_table->search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors' );
+			$donors_table->search_box(esc_html__('Search Donors', 'give'), 'give-donors');
 			$donors_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms" />
@@ -97,7 +97,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -113,27 +113,27 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id = (int) $_GET['id'];
-	$donor    = new Give_Donor( $donor_id );
+	$donor    = new Give_Donor($donor_id);
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -142,34 +142,34 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 
-		<h1 class="screen-reader-text"><?php esc_html_e( 'Donor', 'give' ); ?></h1>
+		<h1 class="screen-reader-text"><?php esc_html_e('Donor', 'give'); ?></h1>
 
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 			<?php
-			foreach ( $donor_tabs as $key => $tab ) :
+			foreach ($donor_tabs as $key => $tab) :
 				$active = $key === $view ? true : false;
 				$class = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 				printf(
-					'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-					esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-					esc_attr( $class ),
-					sanitize_html_class( $tab['dashicon'] ),
-					esc_html( $tab['title'] )
+					'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+					esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+					esc_attr($class),
+					sanitize_html_class($tab['dashicon']),
+					esc_html($tab['title'])
 				);
 			endforeach;
 			?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -189,9 +189,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -200,32 +200,32 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 	?>
 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
-		<form id="edit-donor-info" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		<form id="edit-donor-info" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
 						<span class="donor-id">#<?php echo $donor->id; ?></span>
-						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr( $donor->name ); ?>" placeholder="<?php esc_attr_e( 'Donor Name', 'give' ); ?>" /></span>
+						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr($donor->name); ?>" placeholder="<?php esc_attr_e('Donor Name', 'give'); ?>" /></span>
 						<span class="donor-name info-item editable"><span data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php esc_html_e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php esc_html_e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
-						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php esc_html_e( 'Edit Donor', 'give' ); ?></a>
+					<?php if (current_user_can($donor_edit_role)) : ?>
+						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php esc_html_e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -235,7 +235,7 @@ 
 					<table class="widefat">
 						<tbody>
 						<tr class="alternate">
-							<th scope="col"><label for="tablecell"><?php esc_html_e( 'User:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php esc_html_e('User:', 'give'); ?></label></th>
 							<td>
 								<span class="donor-user-id info-item edit-item">
 									<?php
@@ -252,37 +252,37 @@ 
 										'data'  => $data_atts,
 									);
 
-									if ( ! empty( $user_id ) ) {
-										$userdata           = get_userdata( $user_id );
+									if ( ! empty($user_id)) {
+										$userdata = get_userdata($user_id);
 										$user_args['selected'] = $user_id;
 									}
 
-									echo Give()->html->ajax_user_search( $user_args );
+									echo Give()->html->ajax_user_search($user_args);
 									?>
 								</span>
 
 								<span class="donor-user-id info-item editable">
-									<?php if ( ! empty( $userdata ) ) { ?>
-										<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+									<?php if ( ! empty($userdata)) { ?>
+										<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 									<?php } else { ?>
-										<span data-key="user_id"><?php esc_html_e( 'None', 'give' ); ?></span>
+										<span data-key="user_id"><?php esc_html_e('None', 'give'); ?></span>
 									<?php } ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ) { ?>
-										<span class="disconnect-user"> - <a id="disconnect-donor" href="#disconnect" aria-label="<?php esc_attr_e( 'Disconnects the current user ID from this donor record.', 'give' ); ?>"><?php esc_html_e( 'Disconnect User', 'give' ); ?></a></span>
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0) { ?>
+										<span class="disconnect-user"> - <a id="disconnect-donor" href="#disconnect" aria-label="<?php esc_attr_e('Disconnects the current user ID from this donor record.', 'give'); ?>"><?php esc_html_e('Disconnect User', 'give'); ?></a></span>
 									<?php } ?>
 								</span>
 							</td>
 						</tr>
-						<?php if ( isset( $donor->user_id ) && $donor->user_id > 0 ) : ?>
+						<?php if (isset($donor->user_id) && $donor->user_id > 0) : ?>
 
 							<tr>
-								<th scope="col"><?php esc_html_e( 'Address:', 'give' ); ?></th>
+								<th scope="col"><?php esc_html_e('Address:', 'give'); ?></th>
 								<td class="row-title">
 
 									<div class="donor-address-wrapper">
 
 										<?php
-										$address  = get_user_meta( $donor->user_id, '_give_user_address', true );
+										$address  = get_user_meta($donor->user_id, '_give_user_address', true);
 										$defaults = array(
 											'line1'   => '',
 											'line2'   => '',
@@ -292,10 +292,10 @@ 
 											'zip'     => '',
 										);
 
-										$address = wp_parse_args( $address, $defaults );
+										$address = wp_parse_args($address, $defaults);
 										?>
 
-										<?php if ( ! empty( $address ) ) { ?>
+										<?php if ( ! empty($address)) { ?>
 											<span class="donor-address info-item editable">
 												<span class="info-item" data-key="line1"><?php echo $address['line1']; ?></span>
 												<span class="info-item" data-key="line2"><?php echo $address['line2']; ?></span>
@@ -306,43 +306,43 @@ 
 											</span>
 										<?php } ?>
 										<span class="donor-address info-item edit-item">
-											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php esc_attr_e( 'Address 1', 'give' ); ?>" value="<?php echo $address['line1']; ?>" />
-											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php esc_attr_e( 'Address 2', 'give' ); ?>" value="<?php echo $address['line2']; ?>" />
-											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php esc_attr_e( 'City', 'give' ); ?>" value="<?php echo $address['city']; ?>" />
+											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php esc_attr_e('Address 1', 'give'); ?>" value="<?php echo $address['line1']; ?>" />
+											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php esc_attr_e('Address 2', 'give'); ?>" value="<?php echo $address['line2']; ?>" />
+											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php esc_attr_e('City', 'give'); ?>" value="<?php echo $address['city']; ?>" />
 											<select data-key="country" name="customerinfo[country]" id="billing_country" class="billing_country give-select edit-item">
 												<?php
 
 												$selected_country = $address['country'];
 
 												$countries = give_get_country_list();
-												foreach ( $countries as $country_code => $country ) {
-													echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+												foreach ($countries as $country_code => $country) {
+													echo '<option value="'.esc_attr($country_code).'"'.selected($country_code, $selected_country, false).'>'.$country.'</option>';
 												}
 												?>
 											</select>
 											<?php
 											$selected_state = give_get_state();
-											$states         = give_get_states( $selected_country );
+											$states         = give_get_states($selected_country);
 
-											$selected_state = isset( $address['state'] ) ? $address['state'] : $selected_state;
+											$selected_state = isset($address['state']) ? $address['state'] : $selected_state;
 
-											if ( ! empty( $states ) ) {
+											if ( ! empty($states)) {
 												?>
 												<select data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-select info-item">
 													<?php
-													foreach ( $states as $state_code => $state ) {
-														echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+													foreach ($states as $state_code => $state) {
+														echo '<option value="'.$state_code.'"'.selected($state_code, $selected_state, false).'>'.$state.'</option>';
 													}
 													?>
 												</select>
 												<?php
 											} else {
 												?>
-												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item" placeholder="<?php esc_attr_e( 'State / Province / County', 'give' ); ?>" />
+												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item" placeholder="<?php esc_attr_e('State / Province / County', 'give'); ?>" />
 												<?php
 											}
 											?>
-											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php esc_attr_e( 'Zip / Postal Code', 'give' ); ?>" value="<?php echo $address['zip']; ?>" />
+											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php esc_attr_e('Zip / Postal Code', 'give'); ?>" value="<?php echo $address['zip']; ?>" />
 										</span>
 
 									</div>
@@ -358,10 +358,10 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>" />
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor" />
-				<input type="submit" id="give-edit-donor-save" class="button-secondary" value="<?php esc_attr_e( 'Update Donor', 'give' ); ?>" />
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+				<input type="submit" id="give-edit-donor-save" class="button-secondary" value="<?php esc_attr_e('Update Donor', 'give'); ?>" />
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -376,24 +376,24 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->purchase_value, array( 'sanitize' => false ) ) ); ?> <?php esc_html_e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->purchase_value, array('sanitize' => false))); ?> <?php esc_html_e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -405,7 +405,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -418,7 +418,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -431,46 +431,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 				<tr>
-					<th><?php _e( 'Email', 'give' ); ?></th>
-					<th><?php _e( 'Actions', 'give' ); ?></th>
+					<th><?php _e('Email', 'give'); ?></th>
+					<th><?php _e('Actions', 'give'); ?></th>
 				</tr>
 			</thead>
 
 			<tbody>
-				<?php if ( ! empty( $donor->emails ) ) { ?>
+				<?php if ( ! empty($donor->emails)) { ?>
 
-					<?php foreach ( $donor->emails as $key => $email ) : ?>
+					<?php foreach ($donor->emails as $key => $email) : ?>
 						<tr data-key="<?php echo $key; ?>">
 							<td>
 								<?php echo $email; ?>
-								<?php if ( 'primary' === $key ) : ?>
+								<?php if ('primary' === $key) : ?>
 									<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 								<?php endif; ?>
 							</td>
 							<td>
-								<?php if ( 'primary' !== $key ) : ?>
+								<?php if ('primary' !== $key) : ?>
 									<?php
-									$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-									$promote_url = wp_nonce_url( add_query_arg( array(
-										'email' => rawurlencode( $email ),
+									$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+									$promote_url = wp_nonce_url(add_query_arg(array(
+										'email' => rawurlencode($email),
 										'give_action' => 'set_donor_primary_email',
-									), $base_url ), 'give-set-donor-primary-email' );
-									$remove_url  = wp_nonce_url( add_query_arg( array(
-										'email' => rawurlencode( $email ),
+									), $base_url), 'give-set-donor-primary-email');
+									$remove_url = wp_nonce_url(add_query_arg(array(
+										'email' => rawurlencode($email),
 										'give_action' => 'remove_donor_email',
-									), $base_url ), 'give-remove-donor-email' );
+									), $base_url), 'give-remove-donor-email');
 									?>
-									<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+									<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 									&nbsp;|&nbsp;
-									<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+									<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 								<?php endif; ?>
 							</td>
 						</tr>
@@ -480,59 +480,59 @@ 
 						<td colspan="2" class="add-donor-email-td">
 							<div class="add-donor-email-wrapper">
 								<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>" />
-								<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
-								<input type="email" name="additional-email" value="" placeholder="<?php _e( 'Email Address', 'give' ); ?>" />&nbsp;
-								<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
-								<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+								<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
+								<input type="email" name="additional-email" value="" placeholder="<?php _e('Email Address', 'give'); ?>" />&nbsp;
+								<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
+								<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 								<span class="spinner"></span>
 							</div>
 							<div class="notice-wrap"></div>
 						</td>
 					</tr>
 				<?php } else { ?>
-					<tr><td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td></tr>
+					<tr><td colspan="2"><?php _e('No Emails Found', 'give'); ?></td></tr>
 				<?php }// End if().
 	?>
 			</tbody>
 		</table>
 
-		<h3><?php esc_html_e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php esc_html_e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php esc_html_e('ID', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Amount', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Date', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Status', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
 						<td><?php echo $payment->ID; ?></td>
-						<td><?php echo give_payment_amount( $payment->ID ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo give_payment_amount($payment->ID); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 									/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								esc_html__( 'View Donation', 'give' )
+								esc_html__('View Donation', 'give')
 							);
 							?>
 
@@ -547,47 +547,47 @@ 
 							 * @param object $donor The donor object being displayed.
 							 * @param object $payment  The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php esc_html_e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php esc_html_e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 	?>
 			</tbody>
 		</table>
 
-		<h3><?php esc_html_e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php esc_html_e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php esc_html_e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php esc_html_e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php esc_html_e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 									/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								esc_html__( 'View Form', 'give' )
+								esc_html__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -595,7 +595,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php esc_html_e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php esc_html_e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -609,7 +609,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -622,7 +622,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -635,30 +635,30 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged          = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged          = absint( $paged );
+	$paged          = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged          = absint($paged);
 	$note_count     = $donor->get_notes_count();
-	$per_page       = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages    = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page       = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages    = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php esc_html_e( 'Notes', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
-				<form id="give-add-donor-note" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				<form id="give-add-donor-note" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br />
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>" />
 					<input type="hidden" name="give_action" value="add-donor-note" />
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note" />
 				</form>
 			</div>
@@ -673,26 +673,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php esc_html_e( 'No donor notes found.', 'give' ); ?>
+					<?php esc_html_e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -708,9 +708,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -719,15 +719,15 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
-		<form id="delete-donor" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		<form id="delete-donor" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -735,20 +735,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php esc_html_e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php esc_html_e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php esc_html_e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php esc_html_e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -761,16 +761,16 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>" />
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor" />
-					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" value="<?php esc_attr_e( 'Delete Donor', 'give' ); ?>" />
-					<a id="give-delete-donor-cancel" href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>" class="delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" value="<?php esc_attr_e('Delete Donor', 'give'); ?>" />
+					<a id="give-delete-donor-cancel" href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>" class="delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -786,5 +786,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

includes/admin/donors/class-donor-table.php

Spacing +66 added lines, -66 removed lines

@@ -12,13 +12,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';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => __( 'Donor', 'give' ),     // Singular name of the listed records.
-			'plural'   => __( 'Donors', 'give' ),    // Plural name of the listed records.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		parent::__construct(array(
+			'singular' => __('Donor', 'give'), // Singular name of the listed records.
+			'plural'   => __('Donors', 'give'), // Plural name of the listed records.
+			'ajax'     => false, // Does this table support ajax?.
+		));
 
 	}
 
@@ -80,22 +80,22 @@ 
 	 *
 	 * @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" role="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',
-			) ); ?>
+			)); ?>
 		</p>
 		<?php
 	}
@@ -111,31 +111,31 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($donor, $column_name) {
+		switch ($column_name) {
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor['id'])),
+					esc_html($donor['num_donations'])
 				);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $donor[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($donor[$column_name], array('sanitize' => false)));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $donor['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($donor['date_created']));
 				break;
 
 			default:
-				$value = isset( $donor[ $column_name ] ) ? $donor[ $column_name ] : null;
+				$value = isset($donor[$column_name]) ? $donor[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $donor['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $donor['id']);
 
 	}
 
@@ -146,13 +146,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = '#' . $donor['id'] . ' ';
-		$name     .= ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] );
-		$actions  = $this->get_row_actions( $donor );
+	public function column_name($donor) {
+		$name     = '#'.$donor['id'].' ';
+		$name .= ! empty($donor['name']) ? $donor['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+		$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']);
+		$actions  = $this->get_row_actions($donor);
 
-		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
+		return '<a href="'.esc_url($view_url).'">'.$name.'</a>'.$this->row_actions($actions);
 	}
 
 	/**
@@ -164,14 +164,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
-			'date_created'  => __( 'Date Created', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
+			'date_created'  => __('Date Created', 'give'),
 		);
 
-		return apply_filters( 'give_list_donors_columns', $columns );
+		return apply_filters('give_list_donors_columns', $columns);
 
 	}
 
@@ -185,13 +185,13 @@ 
 	public function get_sortable_columns() {
 
 		$columns = array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 
-		return apply_filters( 'give_list_donors_sortable_columns', $columns );
+		return apply_filters('give_list_donors_sortable_columns', $columns);
 	}
 
 	/**
@@ -204,19 +204,19 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $donor ) {
+	public function get_row_actions($donor) {
 
 		$actions = array(
 
-			'view' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] ), sprintf( esc_attr__( 'View "%s"', 'give' ), $donor['name'] ), __( 'View Donor', 'give' ) ),
+			'view' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']), sprintf(esc_attr__('View "%s"', 'give'), $donor['name']), __('View Donor', 'give')),
 
-			'notes' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor['id'] ), sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $donor['name'] ), __( 'Notes', 'give' ) ),
+			'notes' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor['id']), sprintf(esc_attr__('Notes for "%s"', 'give'), $donor['name']), __('Notes', 'give')),
 
-			'delete' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor['id'] ), sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'delete' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor['id']), sprintf(esc_attr__('Delete "%s"', 'give'), $donor['name']), __('Delete', 'give')),
 
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $donor );
+		return apply_filters('give_donor_row_actions', $actions, $donor);
 
 	}
 
@@ -230,7 +230,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.
 	}
 
@@ -242,7 +242,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;
 	}
 
 	/**
@@ -253,7 +253,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;
 	}
 
 	/**
@@ -270,13 +270,13 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? intval( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? intval($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -290,7 +290,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -305,9 +305,9 @@ 
 
 		$_donor_query['number'] = - 1;
 		$_donor_query['offset'] = 0;
-		$donors                 = Give()->donors->get_donors( $_donor_query );
+		$donors                 = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -319,10 +319,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -331,10 +331,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -357,16 +357,16 @@ 
 		$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->donor_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }