WordImpress / Give

includes/gateways/paypal-standard.php

Doc Comments +1 added lines, -1 removed lines

@@ -661,7 +661,7 @@
  *
  * @since 1.6.3
  *
- * @param $pending_reason
+ * @param string $pending_reason
  *
  * @return string
  */

Spacing +168 added lines, -168 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,7 +21,7 @@ 
  * @access private
  * @since  1.0
  */
-add_action( 'give_paypal_cc_form', '__return_false' );
+add_action('give_paypal_cc_form', '__return_false');
 
 /**
  * Process PayPal Payment.
@@ -32,14 +32,14 @@ 
  *
  * @return void
  */
-function give_process_paypal_payment( $payment_data ) {
+function give_process_paypal_payment($payment_data) {
 
-	if ( ! wp_verify_nonce( $payment_data['gateway_nonce'], 'give-gateway' ) ) {
-		wp_die( esc_html__( 'Nonce verification has failed.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($payment_data['gateway_nonce'], 'give-gateway')) {
+		wp_die(esc_html__('Nonce verification has failed.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	$form_id  = intval( $payment_data['post_data']['give-form-id'] );
-	$price_id = isset( $payment_data['post_data']['give-price-id'] ) ? $payment_data['post_data']['give-price-id'] : '';
+	$form_id  = intval($payment_data['post_data']['give-form-id']);
+	$price_id = isset($payment_data['post_data']['give-price-id']) ? $payment_data['post_data']['give-price-id'] : '';
 
 	// Collect payment data.
 	$payment_data = array(
@@ -57,101 +57,101 @@ 
 	);
 
 	// Record the pending payment.
-	$payment_id = give_insert_payment( $payment_data );
+	$payment_id = give_insert_payment($payment_data);
 
 	// Check payment.
-	if ( ! $payment_id ) {
+	if ( ! $payment_id) {
 		// Record the error.
 		give_record_gateway_error(
-			esc_html__( 'Payment Error', 'give' ),
+			esc_html__('Payment Error', 'give'),
 			sprintf(
 			/* translators: %s: payment data */
-				esc_html__( 'Payment creation failed before sending donor to PayPal. Payment data: %s', 'give' ),
-				json_encode( $payment_data )
+				esc_html__('Payment creation failed before sending donor to PayPal. Payment data: %s', 'give'),
+				json_encode($payment_data)
 			),
 			$payment_id
 		);
 		// Problems? Send back.
-		give_send_back_to_checkout( '?payment-mode=' . $payment_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$payment_data['post_data']['give-gateway']);
 
 	} else {
 
 		// Only send to PayPal if the pending payment is created successfully.
-		$listener_url = add_query_arg( 'give-listener', 'IPN', home_url( 'index.php' ) );
+		$listener_url = add_query_arg('give-listener', 'IPN', home_url('index.php'));
 
 		// Get the success url.
-		$return_url = add_query_arg( array(
+		$return_url = add_query_arg(array(
 			'payment-confirmation' => 'paypal',
 			'payment-id'           => $payment_id
 
-		), get_permalink( give_get_option( 'success_page' ) ) );
+		), get_permalink(give_get_option('success_page')));
 
 		// Get the PayPal redirect uri.
-		$paypal_redirect = trailingslashit( give_get_paypal_redirect() ) . '?';
+		$paypal_redirect = trailingslashit(give_get_paypal_redirect()).'?';
 
 		//Item name - pass level name if variable priced.
 		$item_name = $payment_data['post_data']['give-form-title'];
 
 		//Verify has variable prices.
-		if ( give_has_variable_prices( $form_id ) && isset( $payment_data['post_data']['give-price-id'] ) ) {
+		if (give_has_variable_prices($form_id) && isset($payment_data['post_data']['give-price-id'])) {
 
-			$item_price_level_text = give_get_price_option_name( $form_id, $payment_data['post_data']['give-price-id'] );
+			$item_price_level_text = give_get_price_option_name($form_id, $payment_data['post_data']['give-price-id']);
 
-			$price_level_amount = give_get_price_option_amount( $form_id, $payment_data['post_data']['give-price-id'] );
+			$price_level_amount = give_get_price_option_amount($form_id, $payment_data['post_data']['give-price-id']);
 
 			//Donation given doesn't match selected level (must be a custom amount).
-			if ( $price_level_amount != give_sanitize_amount( $payment_data['price'] ) ) {
-				$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
+			if ($price_level_amount != give_sanitize_amount($payment_data['price'])) {
+				$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
 				//user custom amount text if any, fallback to default if not.
-				$item_name .= ' - ' . ( ! empty( $custom_amount_text ) ? $custom_amount_text : esc_html__( 'Custom Amount', 'give' ) );
+				$item_name .= ' - '.( ! empty($custom_amount_text) ? $custom_amount_text : esc_html__('Custom Amount', 'give'));
 
 			} //Is there any donation level text?
-			elseif ( ! empty( $item_price_level_text ) ) {
-				$item_name .= ' - ' . $item_price_level_text;
+			elseif ( ! empty($item_price_level_text)) {
+				$item_name .= ' - '.$item_price_level_text;
 			}
 
 		} //Single donation: Custom Amount.
-		elseif ( give_get_form_price( $form_id ) !== give_sanitize_amount( $payment_data['price'] ) ) {
-			$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
+		elseif (give_get_form_price($form_id) !== give_sanitize_amount($payment_data['price'])) {
+			$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
 			//user custom amount text if any, fallback to default if not.
-			$item_name .= ' - ' . ( ! empty( $custom_amount_text ) ? $custom_amount_text : esc_html__( 'Custom Amount', 'give' ) );
+			$item_name .= ' - '.( ! empty($custom_amount_text) ? $custom_amount_text : esc_html__('Custom Amount', 'give'));
 		}
 
 		// Setup PayPal API params.
 		$paypal_args = array(
-			'business'      => give_get_option( 'paypal_email', false ),
+			'business'      => give_get_option('paypal_email', false),
 			'first_name'    => $payment_data['user_info']['first_name'],
 			'last_name'     => $payment_data['user_info']['last_name'],
 			'email'         => $payment_data['user_email'],
 			'invoice'       => $payment_data['purchase_key'],
 			'amount'        => $payment_data['price'],
-			'item_name'     => stripslashes( $item_name ),
+			'item_name'     => stripslashes($item_name),
 			'no_shipping'   => '1',
 			'shipping'      => '0',
 			'no_note'       => '1',
 			'currency_code' => give_get_currency(),
-			'charset'       => get_bloginfo( 'charset' ),
+			'charset'       => get_bloginfo('charset'),
 			'custom'        => $payment_id,
 			'rm'            => '2',
 			'return'        => $return_url,
-			'cancel_return' => give_get_failed_transaction_uri( '?payment-id=' . $payment_id ),
+			'cancel_return' => give_get_failed_transaction_uri('?payment-id='.$payment_id),
 			'notify_url'    => $listener_url,
 			'page_style'    => give_get_paypal_page_style(),
-			'cbt'           => get_bloginfo( 'name' ),
+			'cbt'           => get_bloginfo('name'),
 			'bn'            => 'givewp_SP'
 		);
 
 		//Add user address if present.
-		if ( ! empty( $payment_data['user_info']['address'] ) ) {
-			$paypal_args['address1'] = isset( $payment_data['user_info']['address']['line1'] ) ? $payment_data['user_info']['address']['line1'] : '';
-			$paypal_args['address2'] = isset( $payment_data['user_info']['address']['line2'] ) ? $payment_data['user_info']['address']['line2'] : '';
-			$paypal_args['city']     = isset( $payment_data['user_info']['address']['city'] ) ? $payment_data['user_info']['address']['city'] : '';
-			$paypal_args['state']    = isset( $payment_data['user_info']['address']['state'] ) ? $payment_data['user_info']['address']['state'] : '';
-			$paypal_args['country']  = isset( $payment_data['user_info']['address']['country'] ) ? $payment_data['user_info']['address']['country'] : '';
+		if ( ! empty($payment_data['user_info']['address'])) {
+			$paypal_args['address1'] = isset($payment_data['user_info']['address']['line1']) ? $payment_data['user_info']['address']['line1'] : '';
+			$paypal_args['address2'] = isset($payment_data['user_info']['address']['line2']) ? $payment_data['user_info']['address']['line2'] : '';
+			$paypal_args['city']     = isset($payment_data['user_info']['address']['city']) ? $payment_data['user_info']['address']['city'] : '';
+			$paypal_args['state']    = isset($payment_data['user_info']['address']['state']) ? $payment_data['user_info']['address']['state'] : '';
+			$paypal_args['country']  = isset($payment_data['user_info']['address']['country']) ? $payment_data['user_info']['address']['country'] : '';
 		}
 
 		//Donations or regular transactions?
-		if ( give_get_option( 'paypal_button_type' ) === 'standard' ) {
+		if (give_get_option('paypal_button_type') === 'standard') {
 			$paypal_extra_args = array(
 				'cmd' => '_xclick',
 			);
@@ -161,23 +161,23 @@ 
 			);
 		}
 
-		$paypal_args = array_merge( $paypal_extra_args, $paypal_args );
-		$paypal_args = apply_filters( 'give_paypal_redirect_args', $paypal_args, $payment_data );
+		$paypal_args = array_merge($paypal_extra_args, $paypal_args);
+		$paypal_args = apply_filters('give_paypal_redirect_args', $paypal_args, $payment_data);
 
 		// Build query.
-		$paypal_redirect .= http_build_query( $paypal_args );
+		$paypal_redirect .= http_build_query($paypal_args);
 
 		// Fix for some sites that encode the entities.
-		$paypal_redirect = str_replace( '&', '&', $paypal_redirect );
+		$paypal_redirect = str_replace('&', '&', $paypal_redirect);
 
 		// Redirect to PayPal.
-		wp_redirect( $paypal_redirect );
+		wp_redirect($paypal_redirect);
 		exit;
 	}
 
 }
 
-add_action( 'give_gateway_paypal', 'give_process_paypal_payment' );
+add_action('give_gateway_paypal', 'give_process_paypal_payment');
 
 /**
  * Listens for a PayPal IPN requests and then sends to the processing function
@@ -187,17 +187,17 @@ 
  */
 function give_listen_for_paypal_ipn() {
 	// Regular PayPal IPN
-	if ( isset( $_GET['give-listener'] ) && $_GET['give-listener'] == 'IPN' ) {
+	if (isset($_GET['give-listener']) && $_GET['give-listener'] == 'IPN') {
 		/**
 		 * Fires while verifying PayPal IPN
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_verify_paypal_ipn' );
+		do_action('give_verify_paypal_ipn');
 	}
 }
 
-add_action( 'init', 'give_listen_for_paypal_ipn' );
+add_action('init', 'give_listen_for_paypal_ipn');
 
 /**
  * Process PayPal IPN
@@ -208,7 +208,7 @@ 
 function give_process_paypal_ipn() {
 
 	// Check the request method is POST
-	if ( isset( $_SERVER['REQUEST_METHOD'] ) && $_SERVER['REQUEST_METHOD'] != 'POST' ) {
+	if (isset($_SERVER['REQUEST_METHOD']) && $_SERVER['REQUEST_METHOD'] != 'POST') {
 		return;
 	}
 
@@ -216,11 +216,11 @@ 
 	$post_data = '';
 
 	// Fallback just in case post_max_size is lower than needed
-	if ( ini_get( 'allow_url_fopen' ) ) {
-		$post_data = file_get_contents( 'php://input' );
+	if (ini_get('allow_url_fopen')) {
+		$post_data = file_get_contents('php://input');
 	} else {
 		// If allow_url_fopen is not enabled, then make sure that post_max_size is large enough
-		ini_set( 'post_max_size', '12M' );
+		ini_set('post_max_size', '12M');
 	}
 	// Start the encoded data collection with notification command
 	$encoded_data = 'cmd=_notify-validate';
@@ -229,40 +229,40 @@ 
 	$arg_separator = give_get_php_arg_separator_output();
 
 	// Verify there is a post_data
-	if ( $post_data || strlen( $post_data ) > 0 ) {
+	if ($post_data || strlen($post_data) > 0) {
 		// Append the data
-		$encoded_data .= $arg_separator . $post_data;
+		$encoded_data .= $arg_separator.$post_data;
 	} else {
 		// Check if POST is empty
-		if ( empty( $_POST ) ) {
+		if (empty($_POST)) {
 			// Nothing to do
 			return;
 		} else {
 			// Loop through each POST
-			foreach ( $_POST as $key => $value ) {
+			foreach ($_POST as $key => $value) {
 				// Encode the value and append the data.
-				$encoded_data .= $arg_separator . "$key=" . urlencode( $value );
+				$encoded_data .= $arg_separator."$key=".urlencode($value);
 			}
 		}
 	}
 
 	// Convert collected post data to an array.
-	parse_str( $encoded_data, $encoded_data_array );
+	parse_str($encoded_data, $encoded_data_array);
 
-	foreach ( $encoded_data_array as $key => $value ) {
+	foreach ($encoded_data_array as $key => $value) {
 
-		if ( false !== strpos( $key, 'amp;' ) ) {
-			$new_key = str_replace( '&', '&', $key );
-			$new_key = str_replace( 'amp;', '&', $new_key );
+		if (false !== strpos($key, 'amp;')) {
+			$new_key = str_replace('&', '&', $key);
+			$new_key = str_replace('amp;', '&', $new_key);
 
-			unset( $encoded_data_array[ $key ] );
-			$encoded_data_array[ $new_key ] = $value;
+			unset($encoded_data_array[$key]);
+			$encoded_data_array[$new_key] = $value;
 		}
 
 	}
 
 	//Validate IPN request w/ PayPal if user hasn't disabled this security measure
-	if ( give_is_setting_enabled( give_get_option( 'paypal_verification' ) ) ) {
+	if (give_is_setting_enabled(give_get_option('paypal_verification'))) {
 
 		$remote_post_vars = array(
 			'method'      => 'POST',
@@ -282,28 +282,28 @@ 
 		);
 
 		// Validate the IPN.
-		$api_response = wp_remote_post( give_get_paypal_redirect(), $remote_post_vars );
+		$api_response = wp_remote_post(give_get_paypal_redirect(), $remote_post_vars);
 
-		if ( is_wp_error( $api_response ) ) {
+		if (is_wp_error($api_response)) {
 			give_record_gateway_error(
-				esc_html__( 'IPN Error', 'give' ),
+				esc_html__('IPN Error', 'give'),
 				sprintf(
 				/* translators: %s: Paypal IPN response */
-					esc_html__( 'Invalid IPN verification response. IPN data: %s', 'give' ),
-					json_encode( $api_response )
+					esc_html__('Invalid IPN verification response. IPN data: %s', 'give'),
+					json_encode($api_response)
 				)
 			);
 
 			return; // Something went wrong
 		}
 
-		if ( $api_response['body'] !== 'VERIFIED' ) {
+		if ($api_response['body'] !== 'VERIFIED') {
 			give_record_gateway_error(
-				esc_html__( 'IPN Error', 'give' ),
+				esc_html__('IPN Error', 'give'),
 				sprintf(
 				/* translators: %s: Paypal IPN response */
-					esc_html__( 'Invalid IPN verification response. IPN data: %s', 'give' ),
-					json_encode( $api_response )
+					esc_html__('Invalid IPN verification response. IPN data: %s', 'give'),
+					json_encode($api_response)
 				)
 			);
 
@@ -313,7 +313,7 @@ 
 	}
 
 	// Check if $post_data_array has been populated
-	if ( ! is_array( $encoded_data_array ) && ! empty( $encoded_data_array ) ) {
+	if ( ! is_array($encoded_data_array) && ! empty($encoded_data_array)) {
 		return;
 	}
 
@@ -322,12 +322,12 @@ 
 		'payment_status' => ''
 	);
 
-	$encoded_data_array = wp_parse_args( $encoded_data_array, $defaults );
+	$encoded_data_array = wp_parse_args($encoded_data_array, $defaults);
 
-	$payment_id = isset( $encoded_data_array['custom'] ) ? absint( $encoded_data_array['custom'] ) : 0;
+	$payment_id = isset($encoded_data_array['custom']) ? absint($encoded_data_array['custom']) : 0;
 	$txn_type   = $encoded_data_array['txn_type'];
 
-	if ( has_action( 'give_paypal_' . $txn_type ) ) {
+	if (has_action('give_paypal_'.$txn_type)) {
 		/**
 		 * Fires while processing PayPal IPN $txn_type.
 		 *
@@ -338,7 +338,7 @@ 
 		 * @param array $encoded_data_array Encoded data.
 		 * @param int   $payment_id         Payment id.
 		 */
-		do_action( "give_paypal_{$txn_type}", $encoded_data_array, $payment_id );
+		do_action("give_paypal_{$txn_type}", $encoded_data_array, $payment_id);
 	} else {
 		/**
 		 * Fires while process PayPal IPN.
@@ -350,12 +350,12 @@ 
 		 * @param array $encoded_data_array Encoded data.
 		 * @param int   $payment_id         Payment id.
 		 */
-		do_action( 'give_paypal_web_accept', $encoded_data_array, $payment_id );
+		do_action('give_paypal_web_accept', $encoded_data_array, $payment_id);
 	}
 	exit;
 }
 
-add_action( 'give_verify_paypal_ipn', 'give_process_paypal_ipn' );
+add_action('give_verify_paypal_ipn', 'give_process_paypal_ipn');
 
 /**
  * Process web accept (one time) payment IPNs.
@@ -367,128 +367,128 @@ 
  *
  * @return void
  */
-function give_process_paypal_web_accept_and_cart( $data, $payment_id ) {
+function give_process_paypal_web_accept_and_cart($data, $payment_id) {
 
 	//Only allow through these transaction types.
-	if ( $data['txn_type'] != 'web_accept' && $data['txn_type'] != 'cart' && strtolower( $data['payment_status'] ) != 'refunded' ) {
+	if ($data['txn_type'] != 'web_accept' && $data['txn_type'] != 'cart' && strtolower($data['payment_status']) != 'refunded') {
 		return;
 	}
 
 	//Need $payment_id to continue.
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
 	// Collect donation payment details.
 	$paypal_amount  = $data['mc_gross'];
-	$payment_status = strtolower( $data['payment_status'] );
-	$currency_code  = strtolower( $data['mc_currency'] );
-	$business_email = isset( $data['business'] ) && is_email( $data['business'] ) ? trim( $data['business'] ) : trim( $data['receiver_email'] );
-	$payment_meta   = give_get_payment_meta( $payment_id );
+	$payment_status = strtolower($data['payment_status']);
+	$currency_code  = strtolower($data['mc_currency']);
+	$business_email = isset($data['business']) && is_email($data['business']) ? trim($data['business']) : trim($data['receiver_email']);
+	$payment_meta   = give_get_payment_meta($payment_id);
 
 	// Must be a PayPal standard IPN.
-	if ( give_get_payment_gateway( $payment_id ) != 'paypal' ) {
+	if (give_get_payment_gateway($payment_id) != 'paypal') {
 		return;
 	}
 
 	// Verify payment recipient
-	if ( strcasecmp( $business_email, trim( give_get_option( 'paypal_email' ) ) ) != 0 ) {
+	if (strcasecmp($business_email, trim(give_get_option('paypal_email'))) != 0) {
 
 		give_record_gateway_error(
-			esc_html__( 'IPN Error', 'give' ),
+			esc_html__('IPN Error', 'give'),
 			sprintf(
 			/* translators: %s: Paypal IPN response */
-				esc_html__( 'Invalid business email in IPN response. IPN data: %s', 'give' ),
-				json_encode( $data )
+				esc_html__('Invalid business email in IPN response. IPN data: %s', 'give'),
+				json_encode($data)
 			),
 			$payment_id
 		);
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, esc_html__( 'Payment failed due to invalid PayPal business email.', 'give' ) );
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, esc_html__('Payment failed due to invalid PayPal business email.', 'give'));
 
 		return;
 	}
 
 	// Verify payment currency.
-	if ( $currency_code != strtolower( $payment_meta['currency'] ) ) {
+	if ($currency_code != strtolower($payment_meta['currency'])) {
 
 		give_record_gateway_error(
-			esc_html__( 'IPN Error', 'give' ),
+			esc_html__('IPN Error', 'give'),
 			sprintf(
 			/* translators: %s: Paypal IPN response */
-				esc_html__( 'Invalid currency in IPN response. IPN data: %s', 'give' ),
-				json_encode( $data )
+				esc_html__('Invalid currency in IPN response. IPN data: %s', 'give'),
+				json_encode($data)
 			),
 			$payment_id
 		);
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, esc_html__( 'Payment failed due to invalid currency in PayPal IPN.', 'give' ) );
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, esc_html__('Payment failed due to invalid currency in PayPal IPN.', 'give'));
 
 		return;
 	}
 
 	//Process refunds & reversed.
-	if ( $payment_status == 'refunded' || $payment_status == 'reversed' ) {
-		give_process_paypal_refund( $data, $payment_id );
+	if ($payment_status == 'refunded' || $payment_status == 'reversed') {
+		give_process_paypal_refund($data, $payment_id);
 
 		return;
 	}
 
 	// Only complete payments once.
-	if ( get_post_status( $payment_id ) == 'publish' ) {
+	if (get_post_status($payment_id) == 'publish') {
 		return;
 	}
 
 	// Retrieve the total donation amount (before PayPal).
-	$payment_amount = give_get_payment_amount( $payment_id );
+	$payment_amount = give_get_payment_amount($payment_id);
 
 	//Check that the donation PP and local db amounts match.
-	if ( number_format( (float) $paypal_amount, 2 ) < number_format( (float) $payment_amount, 2 ) ) {
+	if (number_format((float) $paypal_amount, 2) < number_format((float) $payment_amount, 2)) {
 		// The prices don't match
 		give_record_gateway_error(
-			esc_html__( 'IPN Error', 'give' ),
+			esc_html__('IPN Error', 'give'),
 			sprintf(
 			/* translators: %s: Paypal IPN response */
-				esc_html__( 'Invalid payment amount in IPN response. IPN data: %s', 'give' ),
-				json_encode( $data )
+				esc_html__('Invalid payment amount in IPN response. IPN data: %s', 'give'),
+				json_encode($data)
 			),
 			$payment_id
 		);
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, esc_html__( 'Payment failed due to invalid amount in PayPal IPN.', 'give' ) );
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, esc_html__('Payment failed due to invalid amount in PayPal IPN.', 'give'));
 
 		return;
 	}
 
 	//Process completed donations.
-	if ( $payment_status == 'completed' || give_is_test_mode() ) {
+	if ($payment_status == 'completed' || give_is_test_mode()) {
 
 		give_insert_payment_note(
 			$payment_id,
 			sprintf(
 			/* translators: %s: Paypal transaction ID */
-				esc_html__( 'PayPal Transaction ID: %s', 'give' ),
+				esc_html__('PayPal Transaction ID: %s', 'give'),
 				$data['txn_id']
 			)
 		);
-		give_set_payment_transaction_id( $payment_id, $data['txn_id'] );
-		give_update_payment_status( $payment_id, 'publish' );
+		give_set_payment_transaction_id($payment_id, $data['txn_id']);
+		give_update_payment_status($payment_id, 'publish');
 
-	} elseif ( 'pending' == $payment_status && isset( $data['pending_reason'] ) ) {
+	} elseif ('pending' == $payment_status && isset($data['pending_reason'])) {
 
 		// Look for possible pending reasons, such as an echeck.
-		$note = give_paypal_get_pending_donation_note( strtolower( $data['pending_reason'] ) );
+		$note = give_paypal_get_pending_donation_note(strtolower($data['pending_reason']));
 
-		if ( ! empty( $note ) ) {
+		if ( ! empty($note)) {
 
-			give_insert_payment_note( $payment_id, $note );
+			give_insert_payment_note($payment_id, $note);
 
 		}
 	}
 
 }
 
-add_action( 'give_paypal_web_accept', 'give_process_paypal_web_accept_and_cart', 10, 2 );
+add_action('give_paypal_web_accept', 'give_process_paypal_web_accept_and_cart', 10, 2);
 
 /**
  * Process PayPal IPN Refunds
@@ -500,28 +500,28 @@ 
  *
  * @return void
  */
-function give_process_paypal_refund( $data, $payment_id = 0 ) {
+function give_process_paypal_refund($data, $payment_id = 0) {
 
 	// Collect payment details
 
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
-	if ( get_post_status( $payment_id ) == 'refunded' ) {
+	if (get_post_status($payment_id) == 'refunded') {
 		return; // Only refund payments once
 	}
 
-	$payment_amount = give_get_payment_amount( $payment_id );
+	$payment_amount = give_get_payment_amount($payment_id);
 	$refund_amount  = $data['payment_gross'] * - 1;
 
-	if ( number_format( (float) $refund_amount, 2 ) < number_format( (float) $payment_amount, 2 ) ) {
+	if (number_format((float) $refund_amount, 2) < number_format((float) $payment_amount, 2)) {
 
 		give_insert_payment_note(
 			$payment_id,
 			sprintf(
 			/* translators: %s: Paypal parent transaction ID */
-				esc_html__( 'Partial PayPal refund processed: %s', 'give' ),
+				esc_html__('Partial PayPal refund processed: %s', 'give'),
 				$data['parent_txn_id']
 			)
 		);
@@ -534,7 +534,7 @@ 
 		$payment_id,
 		sprintf(
 		/* translators: 1: Paypal parent transaction ID 2. Paypal reason code */
-			esc_html__( 'PayPal Payment #%1$s Refunded for reason: %2$s', 'give' ),
+			esc_html__('PayPal Payment #%1$s Refunded for reason: %2$s', 'give'),
 			$data['parent_txn_id'],
 			$data['reason_code']
 		)
@@ -543,11 +543,11 @@ 
 		$payment_id,
 		sprintf(
 		/* translators: %s: Paypal transaction ID */
-			esc_html__( 'PayPal Refund Transaction ID: %s', 'give' ),
+			esc_html__('PayPal Refund Transaction ID: %s', 'give'),
 			$data['txn_id']
 		)
 	);
-	give_update_payment_status( $payment_id, 'refunded' );
+	give_update_payment_status($payment_id, 'refunded');
 }
 
 /**
@@ -559,24 +559,24 @@ 
  *
  * @return string
  */
-function give_get_paypal_redirect( $ssl_check = false ) {
+function give_get_paypal_redirect($ssl_check = false) {
 
-	if ( is_ssl() || ! $ssl_check ) {
+	if (is_ssl() || ! $ssl_check) {
 		$protocal = 'https://';
 	} else {
 		$protocal = 'http://';
 	}
 
 	// Check the current payment mode
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		// Test mode
-		$paypal_uri = $protocal . 'www.sandbox.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocal.'www.sandbox.paypal.com/cgi-bin/webscr';
 	} else {
 		// Live mode
-		$paypal_uri = $protocal . 'www.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocal.'www.paypal.com/cgi-bin/webscr';
 	}
 
-	return apply_filters( 'give_paypal_uri', $paypal_uri );
+	return apply_filters('give_paypal_uri', $paypal_uri);
 }
 
 /**
@@ -586,9 +586,9 @@ 
  * @return string
  */
 function give_get_paypal_page_style() {
-	$page_style = trim( give_get_option( 'paypal_page_style', 'PayPal' ) );
+	$page_style = trim(give_get_option('paypal_page_style', 'PayPal'));
 
-	return apply_filters( 'give_paypal_page_style', $page_style );
+	return apply_filters('give_paypal_page_style', $page_style);
 }
 
 /**
@@ -603,26 +603,26 @@ 
  * @return string
  *
  */
-function give_paypal_success_page_content( $content ) {
+function give_paypal_success_page_content($content) {
 
-	if ( ! isset( $_GET['payment-id'] ) && ! give_get_purchase_session() ) {
+	if ( ! isset($_GET['payment-id']) && ! give_get_purchase_session()) {
 		return $content;
 	}
 
-	$payment_id = isset( $_GET['payment-id'] ) ? absint( $_GET['payment-id'] ) : false;
+	$payment_id = isset($_GET['payment-id']) ? absint($_GET['payment-id']) : false;
 
-	if ( ! $payment_id ) {
+	if ( ! $payment_id) {
 		$session    = give_get_purchase_session();
-		$payment_id = give_get_purchase_id_by_key( $session['purchase_key'] );
+		$payment_id = give_get_purchase_id_by_key($session['purchase_key']);
 	}
 
-	$payment = get_post( $payment_id );
-	if ( $payment && 'pending' == $payment->post_status ) {
+	$payment = get_post($payment_id);
+	if ($payment && 'pending' == $payment->post_status) {
 
 		// Payment is still pending so show processing indicator to fix the race condition.
 		ob_start();
 
-		give_get_template_part( 'payment', 'processing' );
+		give_get_template_part('payment', 'processing');
 
 		$content = ob_get_clean();
 
@@ -632,7 +632,7 @@ 
 
 }
 
-add_filter( 'give_payment_confirm_paypal', 'give_paypal_success_page_content' );
+add_filter('give_payment_confirm_paypal', 'give_paypal_success_page_content');
 
 /**
  * Given a Payment ID, extract the transaction ID
@@ -643,22 +643,22 @@ 
  *
  * @return string                   Transaction ID
  */
-function give_paypal_get_payment_transaction_id( $payment_id ) {
+function give_paypal_get_payment_transaction_id($payment_id) {
 
 	$transaction_id = '';
-	$notes          = give_get_payment_notes( $payment_id );
+	$notes          = give_get_payment_notes($payment_id);
 
-	foreach ( $notes as $note ) {
-		if ( preg_match( '/^PayPal Transaction ID: ([^\s]+)/', $note->comment_content, $match ) ) {
+	foreach ($notes as $note) {
+		if (preg_match('/^PayPal Transaction ID: ([^\s]+)/', $note->comment_content, $match)) {
 			$transaction_id = $match[1];
 			continue;
 		}
 	}
 
-	return apply_filters( 'give_paypal_set_payment_transaction_id', $transaction_id, $payment_id );
+	return apply_filters('give_paypal_set_payment_transaction_id', $transaction_id, $payment_id);
 }
 
-add_filter( 'give_get_payment_transaction_id-paypal', 'give_paypal_get_payment_transaction_id', 10, 1 );
+add_filter('give_get_payment_transaction_id-paypal', 'give_paypal_get_payment_transaction_id', 10, 1);
 
 /**
  * Given a transaction ID, generate a link to the PayPal transaction ID details
@@ -670,16 +670,16 @@ 
  *
  * @return string                 A link to the PayPal transaction details
  */
-function give_paypal_link_transaction_id( $transaction_id, $payment_id ) {
+function give_paypal_link_transaction_id($transaction_id, $payment_id) {
 
 	$paypal_base_url = 'https://history.paypal.com/cgi-bin/webscr?cmd=_history-details-from-hub&id=';
-	$transaction_url = '<a href="' . esc_url( $paypal_base_url . $transaction_id ) . '" target="_blank">' . $transaction_id . '</a>';
+	$transaction_url = '<a href="'.esc_url($paypal_base_url.$transaction_id).'" target="_blank">'.$transaction_id.'</a>';
 
-	return apply_filters( 'give_paypal_link_payment_details_transaction_id', $transaction_url );
+	return apply_filters('give_paypal_link_payment_details_transaction_id', $transaction_url);
 
 }
 
-add_filter( 'give_payment_details_transaction_id-paypal', 'give_paypal_link_transaction_id', 10, 2 );
+add_filter('give_payment_details_transaction_id-paypal', 'give_paypal_link_transaction_id', 10, 2);
 
 
 /**
@@ -691,64 +691,64 @@ 
  *
  * @return string
  */
-function give_paypal_get_pending_donation_note( $pending_reason ) {
+function give_paypal_get_pending_donation_note($pending_reason) {
 
 	$note = '';
 
-	switch ( $pending_reason ) {
+	switch ($pending_reason) {
 
 		case 'echeck' :
 
-			$note = esc_html__( 'Payment made via eCheck and will clear automatically in 5-8 days.', 'give' );
+			$note = esc_html__('Payment made via eCheck and will clear automatically in 5-8 days.', 'give');
 
 			break;
 
 		case 'address' :
 
-			$note = esc_html__( 'Payment requires a confirmed donor address and must be accepted manually through PayPal.', 'give' );
+			$note = esc_html__('Payment requires a confirmed donor address and must be accepted manually through PayPal.', 'give');
 
 			break;
 
 		case 'intl' :
 
-			$note = esc_html__( 'Payment must be accepted manually through PayPal due to international account regulations.', 'give' );
+			$note = esc_html__('Payment must be accepted manually through PayPal due to international account regulations.', 'give');
 
 			break;
 
 		case 'multi-currency' :
 
-			$note = esc_html__( 'Payment received in non-shop currency and must be accepted manually through PayPal.', 'give' );
+			$note = esc_html__('Payment received in non-shop currency and must be accepted manually through PayPal.', 'give');
 
 			break;
 
 		case 'paymentreview' :
 		case 'regulatory_review' :
 
-			$note = esc_html__( 'Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations.', 'give' );
+			$note = esc_html__('Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations.', 'give');
 
 			break;
 
 		case 'unilateral' :
 
-			$note = esc_html__( 'Payment was sent to non-confirmed or non-registered email address.', 'give' );
+			$note = esc_html__('Payment was sent to non-confirmed or non-registered email address.', 'give');
 
 			break;
 
 		case 'upgrade' :
 
-			$note = esc_html__( 'PayPal account must be upgraded before this payment can be accepted.', 'give' );
+			$note = esc_html__('PayPal account must be upgraded before this payment can be accepted.', 'give');
 
 			break;
 
 		case 'verify' :
 
-			$note = esc_html__( 'PayPal account is not verified. Verify account in order to accept this donation.', 'give' );
+			$note = esc_html__('PayPal account is not verified. Verify account in order to accept this donation.', 'give');
 
 			break;
 
 		case 'other' :
 
-			$note = esc_html__( 'Payment is pending for unknown reasons. Contact PayPal support for assistance.', 'give' );
+			$note = esc_html__('Payment is pending for unknown reasons. Contact PayPal support for assistance.', 'give');
 
 			break;
 

includes/class-give-cli-commands.php

Doc Comments +1 added lines, -1 removed lines

@@ -731,7 +731,7 @@
 	 * @param    bool   $colon   Check if add colon between heading and message.
 	 * @param    string $color   Heading color.
 	 *
-	 * @return    mixed
+	 * @return    string
 	 */
 	private function color_message( $heading, $message = '', $colon = true, $color = 'g' ) {
 		// Add colon.

Spacing +196 added lines, -196 removed lines

@@ -7,12 +7,12 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Add give command.
-WP_CLI::add_command( 'give', 'GIVE_CLI_COMMAND' );
+WP_CLI::add_command('give', 'GIVE_CLI_COMMAND');
 
 
 /**
@@ -68,8 +68,8 @@ 
 	 *
 	 * @subcommand    logo
 	 */
-	public function ascii( $args, $assoc_args ) {
-		WP_CLI::log( file_get_contents( GIVE_PLUGIN_DIR . 'assets/images/give-ascii-logo.txt' ) );
+	public function ascii($args, $assoc_args) {
+		WP_CLI::log(file_get_contents(GIVE_PLUGIN_DIR.'assets/images/give-ascii-logo.txt'));
 	}
 
 
@@ -94,54 +94,54 @@ 
 	 *
 	 * @subcommand    details
 	 */
-	public function details( $args, $assoc_args ) {
+	public function details($args, $assoc_args) {
 
 		/**
 		 * Plugin Information
 		 */
-		WP_CLI::log( $this->color_message( __( 'Give Version: ', 'give' ) ) . GIVE_VERSION );
+		WP_CLI::log($this->color_message(__('Give Version: ', 'give')).GIVE_VERSION);
 
 		/**
 		 * General Information.
 		 */
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'General information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('General information', 'give')).'   ####');
 
-		$success_page = give_get_option( 'success_page' );
-		$failure_page = give_get_option( 'failure_page' );
-		$history_page = give_get_option( 'history_page' );
+		$success_page = give_get_option('success_page');
+		$failure_page = give_get_option('failure_page');
+		$history_page = give_get_option('history_page');
 
-		WP_CLI::log( $this->color_message( sprintf( __( 'Success Page: ', 'give' ) ) ) . ( $success_page ? "[{$success_page}] " . get_permalink( $success_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Failed Donation Page: ', 'give' ) ) . ( $failure_page ? "[{$failure_page}] " . get_permalink( $failure_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Donation History Page: ', 'give' ) ) . ( $history_page ? "[{$history_page}] " . get_permalink( $history_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Country: ', 'give' ) ) . give_get_country() );
+		WP_CLI::log($this->color_message(sprintf(__('Success Page: ', 'give'))).($success_page ? "[{$success_page}] ".get_permalink($success_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Failed Donation Page: ', 'give')).($failure_page ? "[{$failure_page}] ".get_permalink($failure_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Donation History Page: ', 'give')).($history_page ? "[{$history_page}] ".get_permalink($history_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Country: ', 'give')).give_get_country());
 
 		/**
 		 * Currency Information.
 		 */
-		$default_gateway = give_get_option( 'default_gateway' );
+		$default_gateway = give_get_option('default_gateway');
 
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'Currency Information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('Currency Information', 'give')).'   ####');
 
-		WP_CLI::log( $this->color_message( __( 'Currency: ', 'give' ), give_get_currency() ) );
-		WP_CLI::log( $this->color_message( __( 'Currency Position: ', 'give' ), give_get_currency_position() ) );
-		WP_CLI::log( $this->color_message( __( 'Thousand Separator: ', 'give' ), give_get_price_thousand_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Decimal Separator: ', 'give' ), give_get_price_decimal_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Number of Decimals: ', 'give' ), give_get_price_decimals() ) );
-		WP_CLI::log( $this->color_message( __( 'Test Mode: ', 'give' ), ( give_get_option( 'test_mode' ) ? __( 'Yes', 'give' ) : __( 'No', 'give' ) ) ) );
-		WP_CLI::log( $this->color_message( __( 'Default Gateway: ', 'give' ), ( $default_gateway ? $default_gateway : __( 'Not Set', 'give' ) ) ) );
+		WP_CLI::log($this->color_message(__('Currency: ', 'give'), give_get_currency()));
+		WP_CLI::log($this->color_message(__('Currency Position: ', 'give'), give_get_currency_position()));
+		WP_CLI::log($this->color_message(__('Thousand Separator: ', 'give'), give_get_price_thousand_separator()));
+		WP_CLI::log($this->color_message(__('Decimal Separator: ', 'give'), give_get_price_decimal_separator()));
+		WP_CLI::log($this->color_message(__('Number of Decimals: ', 'give'), give_get_price_decimals()));
+		WP_CLI::log($this->color_message(__('Test Mode: ', 'give'), (give_get_option('test_mode') ? __('Yes', 'give') : __('No', 'give'))));
+		WP_CLI::log($this->color_message(__('Default Gateway: ', 'give'), ($default_gateway ? $default_gateway : __('Not Set', 'give'))));
 
 		// Payment gateways Information.
-		$gateways = give_get_ordered_payment_gateways( give_get_payment_gateways() );
-		WP_CLI::log( $this->color_message( __( 'Enabled Gateways: ', 'give' ) ) );
+		$gateways = give_get_ordered_payment_gateways(give_get_payment_gateways());
+		WP_CLI::log($this->color_message(__('Enabled Gateways: ', 'give')));
 
-		if ( ! empty( $gateways ) ) {
+		if ( ! empty($gateways)) {
 			self::$counter = 1;
-			foreach ( $gateways as $gateway ) {
-				WP_CLI::log( '  ' . $this->color_message( self::$counter, $gateway['admin_label'] ) );
-				self::$counter ++;
+			foreach ($gateways as $gateway) {
+				WP_CLI::log('  '.$this->color_message(self::$counter, $gateway['admin_label']));
+				self::$counter++;
 			}
 		} else {
-			WP_CLI::log( __( 'Not any payment gateways found', 'give' ) );
+			WP_CLI::log(__('Not any payment gateways found', 'give'));
 		}
 	}
 
@@ -173,15 +173,15 @@ 
 	 *
 	 * @subcommand    forms
 	 */
-	public function forms( $args, $assoc_args ) {
+	public function forms($args, $assoc_args) {
 		global $wp_query;
-		$form_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$number  = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? absint( $assoc_args['number'] ) : 10;
+		$form_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$number  = isset($assoc_args) && array_key_exists('number', $assoc_args) ? absint($assoc_args['number']) : 10;
 		$start   = time();
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -190,22 +190,22 @@ 
 		$wp_query->query_vars['number'] = $number;
 
 		// Get forms.
-		$forms = $form_id ? $this->api->get_forms( $form_id ) : $this->api->get_forms();
+		$forms = $form_id ? $this->api->get_forms($form_id) : $this->api->get_forms();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
 		// Bailout.
-		if ( array_key_exists( 'error', $forms ) ) {
+		if (array_key_exists('error', $forms)) {
 
-			WP_CLI::warning( $forms['error'] );
+			WP_CLI::warning($forms['error']);
 
 			return;
-		} elseif ( empty( $forms['forms'] ) ) {
+		} elseif (empty($forms['forms'])) {
 
-			WP_CLI::error( __( 'No forms found.', 'give' ) );
+			WP_CLI::error(__('No forms found.', 'give'));
 
 			return;
 		}
@@ -213,25 +213,25 @@ 
 		// Param to check if form typeis already showed or not.
 		$is_show_form_type = false;
 
-		if ( 1 === count( $forms ) && $form_id ) {
+		if (1 === count($forms) && $form_id) {
 			// Show single form.
-			foreach ( $forms['forms'][0] as $key => $info ) {
-				switch ( $key ) {
+			foreach ($forms['forms'][0] as $key => $info) {
+				switch ($key) {
 					case 'stats':
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
-						foreach ( $info as $heading => $data ) {
-							$this->color_sub_heading( ucfirst( $heading ) );
-							switch ( $heading ) {
+						foreach ($info as $heading => $data) {
+							$this->color_sub_heading(ucfirst($heading));
+							switch ($heading) {
 								default:
-									foreach ( $data as $subheading => $subdata ) {
+									foreach ($data as $subheading => $subdata) {
 
-										switch ( $subheading ) {
+										switch ($subheading) {
 											case 'earnings':
-												WP_CLI::log( $this->color_message( $subheading . ': ', give_currency_filter( $subdata ) ) );
+												WP_CLI::log($this->color_message($subheading.': ', give_currency_filter($subdata)));
 												break;
 											default:
-												WP_CLI::log( $this->color_message( $subheading . ': ', $subdata ) );
+												WP_CLI::log($this->color_message($subheading.': ', $subdata));
 										}
 									}
 							}
@@ -241,26 +241,26 @@ 
 					case 'pricing':
 					case 'info':
 					default:
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
 						// Show form type.
-						if ( ! $is_show_form_type ) {
-							$form              = new Give_Donate_Form( $form_id );
+						if ( ! $is_show_form_type) {
+							$form              = new Give_Donate_Form($form_id);
 							$is_show_form_type = true;
 
-							WP_CLI::log( $this->color_message( __( 'form type', 'give' ), $form->get_type() ) );
+							WP_CLI::log($this->color_message(__('form type', 'give'), $form->get_type()));
 						}
 
-						foreach ( $info as $heading => $data ) {
+						foreach ($info as $heading => $data) {
 
-							switch ( $heading ) {
+							switch ($heading) {
 								case 'id':
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									WP_CLI::log($this->color_message($heading, $data));
 									break;
 
 								default:
-									$data = empty( $data ) ? __( 'Not set', 'give' ) : $data;
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									$data = empty($data) ? __('Not set', 'give') : $data;
+									WP_CLI::log($this->color_message($heading, $data));
 							}
 						}
 				}
@@ -271,37 +271,37 @@ 
 			$is_table_first_row_set = false;
 			$table_column_count     = 0;
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%d donation forms found', 'give' ), count( $forms['forms'] ) ), '', false ) );
+			WP_CLI::line($this->color_message(sprintf(__('%d donation forms found', 'give'), count($forms['forms'])), '', false));
 
-			foreach ( $forms['forms'] as $index => $form_data ) {
+			foreach ($forms['forms'] as $index => $form_data) {
 
 				// Default table data.
 				$table_first_row = array();
 				$table_row       = array();
 
-				foreach ( $form_data['info'] as $key => $form ) {
+				foreach ($form_data['info'] as $key => $form) {
 
 					// Do not show thumbnail, content and link in table.
-					if ( in_array( $key, array( 'content', 'thumbnail', 'link' ), true ) ) {
+					if (in_array($key, array('content', 'thumbnail', 'link'), true)) {
 						continue;
 					}
 
-					if ( ! $is_table_first_row_set ) {
+					if ( ! $is_table_first_row_set) {
 						$table_first_row[] = $key;
 					}
 
 					$table_row[] = $form;
 
-					if ( 'status' === $key ) {
+					if ('status' === $key) {
 						// First array item will be an form id in our case.
-						$form = new Give_Donate_Form( absint( $table_row[0] ) );
+						$form = new Give_Donate_Form(absint($table_row[0]));
 
 						$table_row[] = $form->get_type();
 					}
 				}
 
 				// Set table first row.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 
 					// Add extra column to table.
 					$table_first_row[] = 'type';
@@ -314,7 +314,7 @@ 
 				$table_data[] = $table_row;
 			}
 
-			$this->display_table( $table_data );
+			$this->display_table($table_data);
 		}
 	}
 
@@ -366,37 +366,37 @@ 
 	 *
 	 * @subcommand    donors
 	 */
-	public function donors( $args, $assoc_args ) {
+	public function donors($args, $assoc_args) {
 		global $wp_query;
-		$donor_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$email    = isset( $assoc_args ) && array_key_exists( 'email', $assoc_args ) ? $assoc_args['email'] : false;
-		$name     = isset( $assoc_args ) && array_key_exists( 'name', $assoc_args ) ? $assoc_args['name'] : '';
-		$create   = isset( $assoc_args ) && array_key_exists( 'create', $assoc_args ) ? $assoc_args['create'] : false;
-		$number   = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
-		$form_id  = isset( $assoc_args ) && array_key_exists( 'form-id', $assoc_args ) ? $assoc_args['form-id'] : 0;
-		$format   = isset( $assoc_args ) && array_key_exists( 'format', $assoc_args ) ? $assoc_args['format'] : 'table';
+		$donor_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$email    = isset($assoc_args) && array_key_exists('email', $assoc_args) ? $assoc_args['email'] : false;
+		$name     = isset($assoc_args) && array_key_exists('name', $assoc_args) ? $assoc_args['name'] : '';
+		$create   = isset($assoc_args) && array_key_exists('create', $assoc_args) ? $assoc_args['create'] : false;
+		$number   = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
+		$form_id  = isset($assoc_args) && array_key_exists('form-id', $assoc_args) ? $assoc_args['form-id'] : 0;
+		$format   = isset($assoc_args) && array_key_exists('format', $assoc_args) ? $assoc_args['format'] : 'table';
 		$start    = time();
 
-		if ( $create ) {
+		if ($create) {
 			$number = 1;
 
-			if ( isset( $assoc_args['email'] ) && ! is_email( $email ) ) {
-				WP_CLI::warning( 'Wrong email address provided.', 'give' );
+			if (isset($assoc_args['email']) && ! is_email($email)) {
+				WP_CLI::warning('Wrong email address provided.', 'give');
 
 				return;
 			}
 
 			// Create one or more donors.
-			if ( ! $email ) {
+			if ( ! $email) {
 				// If no email is specified, look to see if we are generating arbitrary donor accounts.
-				$number = is_numeric( $create ) ? absint( $create ) : 1;
+				$number = is_numeric($create) ? absint($create) : 1;
 			}
 
-			for ( $i = 0; $i < $number; $i ++ ) {
-				if ( ! $email ) {
+			for ($i = 0; $i < $number; $i++) {
+				if ( ! $email) {
 
 					// Generate fake email.
-					$email = 'customer-' . uniqid() . '@test.com';
+					$email = 'customer-'.uniqid().'@test.com';
 				}
 
 				$args = array(
@@ -404,19 +404,19 @@ 
 					'name'  => $name,
 				);
 
-				$customer_id = Give()->customers->add( $args );
+				$customer_id = Give()->customers->add($args);
 
-				if ( $customer_id ) {
-					WP_CLI::line( $this->color_message( sprintf( __( 'Donor #%d created successfully', 'give' ), $customer_id ) ) );
+				if ($customer_id) {
+					WP_CLI::line($this->color_message(sprintf(__('Donor #%d created successfully', 'give'), $customer_id)));
 				} else {
-					WP_CLI::error( __( 'Failed to create donor', 'give' ) );
+					WP_CLI::error(__('Failed to create donor', 'give'));
 				}
 
 				// Reset email to false so it is generated on the next loop (if creating donors).
 				$email = false;
 			}
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%1$d donors created in %2$d seconds', 'give' ), $number, time() - $start ) ) );
+			WP_CLI::line($this->color_message(sprintf(__('%1$d donors created in %2$d seconds', 'give'), $number, time() - $start)));
 
 		} else {
 			// Counter.
@@ -430,7 +430,7 @@ 
 			 */
 			// Cache previous number query var.
 			$is_set_number = $cache_per_page = false;
-			if ( isset( $wp_query->query_vars['number'] ) ) {
+			if (isset($wp_query->query_vars['number'])) {
 				$cache_per_page = $wp_query->query_vars['number'];
 				$is_set_number  = true;
 			}
@@ -439,24 +439,24 @@ 
 			$wp_query->query_vars['number'] = $number;
 
 			// Get donors.
-			if ( $form_id ) {
+			if ($form_id) {
 				// @TODO: Allow user to get a list of donors by donation status.
-				$donors = $this->get_donors_by_form_id( $form_id );
+				$donors = $this->get_donors_by_form_id($form_id);
 			} else {
-				$donors = $this->api->get_customers( $search );
+				$donors = $this->api->get_customers($search);
 			}
 
 			// Reset number query var.
-			if ( $is_set_number ) {
+			if ($is_set_number) {
 				$wp_query->query_vars['number'] = $cache_per_page;
 			}
 
-			if ( isset( $donors['error'] ) ) {
-				WP_CLI::error( $donors['error'] );
+			if (isset($donors['error'])) {
+				WP_CLI::error($donors['error']);
 			}
 
-			if ( empty( $donors ) ) {
-				WP_CLI::error( __( 'No donors found.', 'give' ) );
+			if (empty($donors)) {
+				WP_CLI::error(__('No donors found.', 'give'));
 
 				return;
 			}
@@ -464,24 +464,24 @@ 
 			$table_data             = array();
 			$is_table_first_row_set = false;
 
-			foreach ( $donors['donors'] as $donor_data ) {
+			foreach ($donors['donors'] as $donor_data) {
 				// Set default table row data.
-				$table_first_row = array( __( 's_no', 'give' ) );
-				$table_row       = array( self::$counter );
+				$table_first_row = array(__('s_no', 'give'));
+				$table_row       = array(self::$counter);
 
-				foreach ( $donor_data as $key => $donor ) {
-					switch ( $key ) {
+				foreach ($donor_data as $key => $donor) {
+					switch ($key) {
 						case 'stats':
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
-								switch ( $heading ) {
+								switch ($heading) {
 									case 'total_spent':
-										$table_row[] = give_currency_filter( $data );
+										$table_row[] = give_currency_filter($data);
 										break;
 
 									default:
@@ -492,10 +492,10 @@ 
 
 						case 'info':
 						default:
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
@@ -505,7 +505,7 @@ 
 				}
 
 				// Add first row data to table data.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 					$table_data[]           = $table_first_row;
 					$is_table_first_row_set = true;
 				}
@@ -514,44 +514,44 @@ 
 				$table_data[] = $table_row;
 
 				// Increase counter.
-				self::$counter ++;
+				self::$counter++;
 			}
 
-			switch ( $format ) {
+			switch ($format) {
 				case 'json':
 					$table_column_name = $table_data[0];
-					unset( $table_data[0] );
+					unset($table_data[0]);
 
 					$new_table_data = array();
-					foreach ( $table_data as $index => $data ) {
-						foreach ( $data as $key => $value ) {
-							$new_table_data[ $index ][ $table_column_name[ $key ] ] = $value;
+					foreach ($table_data as $index => $data) {
+						foreach ($data as $key => $value) {
+							$new_table_data[$index][$table_column_name[$key]] = $value;
 						}
 					}
 
-					WP_CLI::log( json_encode( $new_table_data ) );
+					WP_CLI::log(json_encode($new_table_data));
 					break;
 
 				case 'csv':
-					$file_path = trailingslashit( WP_CONTENT_DIR ) . 'uploads/give_donors_' . date( 'Y_m_d_s', current_time( 'timestamp' ) ) . '.csv';
-					$fp        = fopen( $file_path, 'w' );
+					$file_path = trailingslashit(WP_CONTENT_DIR).'uploads/give_donors_'.date('Y_m_d_s', current_time('timestamp')).'.csv';
+					$fp        = fopen($file_path, 'w');
 
-					if ( is_writable( $file_path ) ) {
-						foreach ( $table_data as $fields ) {
-							fputcsv( $fp, $fields );
+					if (is_writable($file_path)) {
+						foreach ($table_data as $fields) {
+							fputcsv($fp, $fields);
 						}
 
-						fclose( $fp );
+						fclose($fp);
 
-						WP_CLI::success( "Donors list csv created successfully: {$file_path}" );
+						WP_CLI::success("Donors list csv created successfully: {$file_path}");
 					} else {
-						WP_CLI::warning( "Unable to create donors list csv file: {$file_path} (May folder do not have write permission)" );
+						WP_CLI::warning("Unable to create donors list csv file: {$file_path} (May folder do not have write permission)");
 					}
 
 					break;
 
 				default:
-					$this->display_table( $table_data );
+					$this->display_table($table_data);
 			}
 		}
 	}
@@ -581,13 +581,13 @@ 
 	 *
 	 * @subcommand    donations
 	 */
-	public function donations( $args, $assoc_args ) {
+	public function donations($args, $assoc_args) {
 		global $wp_query;
-		$number = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
+		$number = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -599,46 +599,46 @@ 
 		$donations = $this->api->get_recent_donations();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
-		if ( empty( $donations ) ) {
-			WP_CLI::error( __( 'No donations found.', 'give' ) );
+		if (empty($donations)) {
+			WP_CLI::error(__('No donations found.', 'give'));
 
 			return;
 		}
 
 		self::$counter = 1;
 
-		foreach ( $donations['donations'] as $key => $donation ) {
-			$this->color_main_heading( sprintf( __( '%1$s. Donation #%2$s', 'give' ), self::$counter, $donation['ID'] ), 'Y' );
-			self::$counter ++;
+		foreach ($donations['donations'] as $key => $donation) {
+			$this->color_main_heading(sprintf(__('%1$s. Donation #%2$s', 'give'), self::$counter, $donation['ID']), 'Y');
+			self::$counter++;
 
-			foreach ( $donation as $column => $data ) {
+			foreach ($donation as $column => $data) {
 
-				if ( is_array( $data ) ) {
-					$this->color_sub_heading( $column );
-					foreach ( $data as $subcolumn => $subdata ) {
+				if (is_array($data)) {
+					$this->color_sub_heading($column);
+					foreach ($data as $subcolumn => $subdata) {
 
 						// Decode html codes.
-						switch ( $subcolumn ) {
+						switch ($subcolumn) {
 							case 'name':
-								$subdata = html_entity_decode( $subdata );
+								$subdata = html_entity_decode($subdata);
 								break;
 						}
 
 						// @TODO Check if multi dimension array information is importent to show or not. For example inside donation array we have array for fees data inside payment meta.
-						if ( is_array( $subdata ) ) {
+						if (is_array($subdata)) {
 							continue;
 						}
 
-						WP_CLI::log( $this->color_message( $subcolumn, $subdata ) );
+						WP_CLI::log($this->color_message($subcolumn, $subdata));
 					}
 					continue;
 				}
 
-				WP_CLI::log( $this->color_message( $column, $data ) );
+				WP_CLI::log($this->color_message($column, $data));
 			}
 		}
 	}
@@ -677,27 +677,27 @@ 
 	 *
 	 * @return        void
 	 */
-	public function report( $args, $assoc_args ) {
+	public function report($args, $assoc_args) {
 		$stats      = new Give_Payment_Stats();
-		$date       = isset( $assoc_args ) && array_key_exists( 'date', $assoc_args ) ? $assoc_args['date'] : false;
-		$start_date = isset( $assoc_args ) && array_key_exists( 'start-date', $assoc_args ) ? $assoc_args['start-date'] : false;
-		$end_date   = isset( $assoc_args ) && array_key_exists( 'end-date', $assoc_args ) ? $assoc_args['end-date'] : false;
-		$form_id    = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? $assoc_args['id'] : 0;
+		$date       = isset($assoc_args) && array_key_exists('date', $assoc_args) ? $assoc_args['date'] : false;
+		$start_date = isset($assoc_args) && array_key_exists('start-date', $assoc_args) ? $assoc_args['start-date'] : false;
+		$end_date   = isset($assoc_args) && array_key_exists('end-date', $assoc_args) ? $assoc_args['end-date'] : false;
+		$form_id    = isset($assoc_args) && array_key_exists('id', $assoc_args) ? $assoc_args['id'] : 0;
 
-		if ( ! empty( $date ) ) {
+		if ( ! empty($date)) {
 			$start_date = $date;
 			$end_date   = false;
-		} elseif ( empty( $date ) && empty( $start_date ) ) {
+		} elseif (empty($date) && empty($start_date)) {
 			$start_date = 'this_month';
 			$end_date   = false;
 		}
 
 		// Get stats.
-		$earnings = $stats->get_earnings( $form_id, $start_date, $end_date );
-		$sales    = $stats->get_sales( $form_id, $start_date, $end_date );
+		$earnings = $stats->get_earnings($form_id, $start_date, $end_date);
+		$sales    = $stats->get_sales($form_id, $start_date, $end_date);
 
-		WP_CLI::line( $this->color_message( __( 'Earnings', 'give' ), give_currency_filter( $earnings ) ) );
-		WP_CLI::line( $this->color_message( __( 'Sales', 'give' ), $sales ) );
+		WP_CLI::line($this->color_message(__('Earnings', 'give'), give_currency_filter($earnings)));
+		WP_CLI::line($this->color_message(__('Sales', 'give'), $sales));
 	}
 
 
@@ -724,27 +724,27 @@ 
 	 *
 	 * @subcommand    cache
 	 */
-	public function cache( $args, $assoc_args ) {
-		$action = isset( $assoc_args ) && array_key_exists( 'action', $assoc_args ) ? $assoc_args['action'] : false;
+	public function cache($args, $assoc_args) {
+		$action = isset($assoc_args) && array_key_exists('action', $assoc_args) ? $assoc_args['action'] : false;
 
 		// Bailout.
-		if ( ! $action || ! in_array( $action, array( 'delete' ), true ) ) {
-			WP_CLI::warning( __( 'Type wp give cache --action=delete to delete all stat transients', 'give' ) );
+		if ( ! $action || ! in_array($action, array('delete'), true)) {
+			WP_CLI::warning(__('Type wp give cache --action=delete to delete all stat transients', 'give'));
 
 			return;
 		}
 
-		switch ( $action ) {
+		switch ($action) {
 			case 'delete' :
 				// Reset counter.
 				self::$counter = 1;
 
-				if ( $this->delete_stats_transients() ) {
+				if ($this->delete_stats_transients()) {
 					// Report .eading.
-					WP_CLI::success( 'Give cache deleted.' );
+					WP_CLI::success('Give cache deleted.');
 				} else {
 					// Report .eading.
-					WP_CLI::warning( 'We did not find any Give plugin cache to delete :)' );
+					WP_CLI::warning('We did not find any Give plugin cache to delete :)');
 				}
 				break;
 		}
@@ -773,27 +773,27 @@ 
 			ARRAY_A
 		);
 
-		if ( ! empty( $stat_option_names ) ) {
+		if ( ! empty($stat_option_names)) {
 
-			foreach ( $stat_option_names as $option_name ) {
+			foreach ($stat_option_names as $option_name) {
 				$error       = false;
 				$option_name = $option_name['option_name'];
 
-				switch ( true ) {
-					case ( false !== strpos( $option_name, 'transient' ) ):
-						$option_name = str_replace( '_transient_', '', $option_name );
-						$error       = delete_transient( $option_name );
+				switch (true) {
+					case (false !== strpos($option_name, 'transient')):
+						$option_name = str_replace('_transient_', '', $option_name);
+						$error       = delete_transient($option_name);
 						break;
 
 					default:
-						$error = delete_option( $option_name );
+						$error = delete_option($option_name);
 				}
 
-				if ( $error ) {
-					WP_CLI::log( $this->color_message( self::$counter, $option_name ) );
-					self::$counter ++;
+				if ($error) {
+					WP_CLI::log($this->color_message(self::$counter, $option_name));
+					self::$counter++;
 				} else {
-					WP_CLI::log( $this->color_message( __( 'Error while deleting this transient', 'give' ), $option_name ) );
+					WP_CLI::log($this->color_message(__('Error while deleting this transient', 'give'), $option_name));
 				}
 			}
 
@@ -814,13 +814,13 @@ 
 	 *
 	 * @return    mixed
 	 */
-	private function color_message( $heading, $message = '', $colon = true, $color = 'g' ) {
+	private function color_message($heading, $message = '', $colon = true, $color = 'g') {
 		// Add colon.
-		if ( $colon ) {
-			$heading = $heading . ': ';
+		if ($colon) {
+			$heading = $heading.': ';
 		}
 
-		return WP_CLI::colorize( "%{$color}" . $heading . '%n' ) . $message;
+		return WP_CLI::colorize("%{$color}".$heading.'%n').$message;
 	}
 
 
@@ -835,8 +835,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_main_heading( $heading, $color = 'g' ) {
-		WP_CLI::log( "\n######   " . $this->color_message( $heading, '', false, $color ) . '   ######' );
+	private function color_main_heading($heading, $color = 'g') {
+		WP_CLI::log("\n######   ".$this->color_message($heading, '', false, $color).'   ######');
 	}
 
 	/**
@@ -849,8 +849,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_sub_heading( $subheading ) {
-		WP_CLI::log( "\n--->" . $subheading . '', '', false );
+	private function color_sub_heading($subheading) {
+		WP_CLI::log("\n--->".$subheading.'', '', false);
 	}
 
 
@@ -864,17 +864,17 @@ 
 	 *
 	 * @return    void
 	 */
-	private function display_table( $data ) {
+	private function display_table($data) {
 		$table = new \cli\Table();
 
 		// Set table header.
-		$table->setHeaders( $data[0] );
+		$table->setHeaders($data[0]);
 
 		// Remove table header.
-		unset( $data[0] );
+		unset($data[0]);
 
 		// Set table data.
-		$table->setRows( $data );
+		$table->setRows($data);
 
 		// Display table.
 		$table->display();
@@ -891,14 +891,14 @@ 
 	 * @return array
 	 */
 
-	private function get_donors_by_form_id( $form_id ) {
+	private function get_donors_by_form_id($form_id) {
 		$donors = array();
 
 		$donations = new Give_Payments_Query(
 			array(
-				'give_forms' => array( $form_id ),
-				'number'     => - 1,
-				'status'     => array( 'publish' ),
+				'give_forms' => array($form_id),
+				'number'     => -1,
+				'status'     => array('publish'),
 			)
 		);
 
@@ -906,16 +906,16 @@ 
 		$skip_donors = array();
 
 		/* @var Give_Payment|object $donation Payment object. */
-		foreach ( $donations as $donation ) {
+		foreach ($donations as $donation) {
 
-			if ( in_array( $donation->customer_id, $skip_donors ) ) {
+			if (in_array($donation->customer_id, $skip_donors)) {
 				continue;
 			}
 
-			if ( ! empty( $donors ) ) {
-				$donors['donors'][] = current( current( $this->api->get_customers( (int) $donation->customer_id ) ) );
+			if ( ! empty($donors)) {
+				$donors['donors'][] = current(current($this->api->get_customers((int) $donation->customer_id)));
 			} else {
-				$donors = array_merge( $donors, $this->api->get_customers( (int) $donation->customer_id ) );
+				$donors = array_merge($donors, $this->api->get_customers((int) $donation->customer_id));
 			}
 
 			$skip_donors[] = $donation->customer_id;

includes/payments/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -326,7 +326,7 @@ 
  *
  * @since  1.0
  *
- * @param  int|bool $form_id    Form ID (default: false).
+ * @param  boolean $form_id    Form ID (default: false).
  * @param  int      $payment_id Payment ID.
  *
  * @return void
@@ -644,7 +644,7 @@ 
  * @param  int $year      Year number. Default is null.
  * @param  int $hour      Hour number. Default is null.
  *
- * @return int $earnings  Earnings
+ * @return double $earnings  Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 
@@ -959,7 +959,7 @@ 
  *
  * @param int $payment_id Payment ID.
  *
- * @return int $form_id Form ID.
+ * @return string $form_id Form ID.
  */
 function give_get_payment_form_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );

Spacing +395 added lines, -395 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
  *
  * @return object $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility.
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -66,48 +66,48 @@ 
  *
  * @return mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -116,7 +116,7 @@ 
 			return false;
 	}
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -132,23 +132,23 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise.
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = isset( $payment_data['give_price_id'] ) ? $payment_data['give_price_id'] : give_get_price_id( $payment_data['give_form_id'], $payment_data['price'] );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = isset($payment_data['give_price_id']) ? $payment_data['give_price_id'] : give_get_price_id($payment_data['give_form_id'], $payment_data['price']);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	// Set properties.
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency();
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency();
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
@@ -162,31 +162,31 @@ 
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
 	$payment->mode           = give_is_test_mode() ? 'test' : 'live';
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	// Add the donation.
 	$args = array(
 		'price'    => $payment->total,
 		'price_id' => $payment->price_id,
-		'fees'     => isset( $payment_data['fees'] ) ? $payment_data['fees'] : array(),
+		'fees'     => isset($payment_data['fees']) ? $payment_data['fees'] : array(),
 	);
 
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 	// Set date if present.
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	// Handle sequential payments.
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Clear the user's donation cache.
-	delete_transient( 'give_user_' . $payment_data['user_info']['id'] . '_purchases' );
+	delete_transient('give_user_'.$payment_data['user_info']['id'].'_purchases');
 
 	// Save payment.
 	$payment->save();
@@ -199,10 +199,10 @@ 
 	 * @param int   $payment_id   The payment ID.
 	 * @param array $payment_data Arguments passed.
 	 */
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	// Return payment ID upon success.
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -221,9 +221,9 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
-	$payment         = new Give_Payment( $payment_id );
+	$payment         = new Give_Payment($payment_id);
 	$payment->status = $new_status;
 	$updated         = $payment->save();
 
@@ -242,37 +242,37 @@ 
  *
  * @return void
  */
-function give_delete_purchase( $payment_id = 0, $update_customer = true ) {
+function give_delete_purchase($payment_id = 0, $update_customer = true) {
 	global $give_logs;
 
-	$payment     = new Give_Payment( $payment_id );
-	$amount      = give_get_payment_amount( $payment_id );
+	$payment     = new Give_Payment($payment_id);
+	$amount      = give_get_payment_amount($payment_id);
 	$status      = $payment->post_status;
-	$customer_id = give_get_payment_customer_id( $payment_id );
-	$customer    = new Give_Customer( $customer_id );
+	$customer_id = give_get_payment_customer_id($payment_id);
+	$customer    = new Give_Customer($customer_id);
 
 	// Only undo donations that aren't these statuses.
-	$dont_undo_statuses = apply_filters( 'give_undo_purchase_statuses', array(
+	$dont_undo_statuses = apply_filters('give_undo_purchase_statuses', array(
 		'pending',
 		'cancelled',
-	) );
+	));
 
-	if ( ! in_array( $status, $dont_undo_statuses ) ) {
-		give_undo_purchase( false, $payment_id );
+	if ( ! in_array($status, $dont_undo_statuses)) {
+		give_undo_purchase(false, $payment_id);
 	}
 
-	if ( $status == 'publish' ) {
+	if ($status == 'publish') {
 
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment).
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 		// Clear the This Month earnings (this_monththis_month is NOT a typo).
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 
-		if ( $customer->id && $update_customer ) {
+		if ($customer->id && $update_customer) {
 
 			// Decrement the stats for the donor.
 			$customer->decrease_purchase_count();
-			$customer->decrease_value( $amount );
+			$customer->decrease_value($amount);
 
 		}
 	}
@@ -284,17 +284,17 @@ 
 	 *
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $customer->id && $update_customer ) {
+	if ($customer->id && $update_customer) {
 
 		// Remove the payment ID from the donor.
-		$customer->remove_payment( $payment_id );
+		$customer->remove_payment($payment_id);
 
 	}
 
 	// Remove the payment.
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries.
 	$give_logs->delete_logs(
@@ -315,7 +315,7 @@ 
 	 *
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -331,25 +331,25 @@ 
  *
  * @return void
  */
-function give_undo_purchase( $form_id = false, $payment_id ) {
+function give_undo_purchase($form_id = false, $payment_id) {
 
-	if ( ! empty( $form_id ) ) {
+	if ( ! empty($form_id)) {
 		$form_id = false;
-		_give_deprected_argument( 'form_id', 'give_undo_purchase', '1.5' );
+		_give_deprected_argument('form_id', 'give_undo_purchase', '1.5');
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// Decrease earnings.
-		give_decrease_earnings( $payment->form_id, $payment->total );
+		give_decrease_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_sales = apply_filters( 'give_decrease_sales_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_sales ) {
+	$maybe_decrease_sales = apply_filters('give_decrease_sales_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_sales) {
 		// Decrease donation count.
-		give_decrease_purchase_count( $payment->form_id );
+		give_decrease_purchase_count($payment->form_id);
 	}
 
 }
@@ -366,7 +366,7 @@ 
  *
  * @return array $count Number of payments sorted by payment status.
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 
 	global $wpdb;
 
@@ -378,18 +378,18 @@ 
 		'form_id'    => null,
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$select = 'SELECT p.post_status,count( * ) AS num_posts';
 	$join   = '';
 	$where  = "WHERE p.post_type = 'give_payment'";
 
 	// Count payments for a specific user.
-	if ( ! empty( $args['user'] ) ) {
+	if ( ! empty($args['user'])) {
 
-		if ( is_email( $args['user'] ) ) {
+		if (is_email($args['user'])) {
 			$field = 'email';
-		} elseif ( is_numeric( $args['user'] ) ) {
+		} elseif (is_numeric($args['user'])) {
 			$field = 'id';
 		} else {
 			$field = '';
@@ -397,105 +397,105 @@ 
 
 		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$where .= "
 				AND m.meta_key = '_give_payment_user_{$field}'
 				AND m.meta_value = '{$args['user']}'";
 		}
 
 		// Count payments for a search.
-	} elseif ( ! empty( $args['s'] ) ) {
+	} elseif ( ! empty($args['s'])) {
 
-		if ( is_email( $args['s'] ) || strlen( $args['s'] ) == 32 ) {
+		if (is_email($args['s']) || strlen($args['s']) == 32) {
 
-			if ( is_email( $args['s'] ) ) {
+			if (is_email($args['s'])) {
 				$field = '_give_payment_user_email';
 			} else {
 				$field = '_give_payment_purchase_key';
 			}
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( '
+			$where .= $wpdb->prepare('
                 AND m.meta_key = %s
                 AND m.meta_value = %s',
 				$field,
 				$args['s']
 			);
 
-		} elseif ( '#' == substr( $args['s'], 0, 1 ) ) {
+		} elseif ('#' == substr($args['s'], 0, 1)) {
 
-			$search = str_replace( '#:', '', $args['s'] );
-			$search = str_replace( '#', '', $search );
+			$search = str_replace('#:', '', $args['s']);
+			$search = str_replace('#', '', $search);
 
 			$select = 'SELECT p2.post_status,count( * ) AS num_posts ';
 			$join   = "LEFT JOIN $wpdb->postmeta m ON m.meta_key = '_give_log_payment_id' AND m.post_id = p.ID ";
 			$join .= "INNER JOIN $wpdb->posts p2 ON m.meta_value = p2.ID ";
 			$where = "WHERE p.post_type = 'give_log' ";
-			$where .= $wpdb->prepare( 'AND p.post_parent = %d} ', $search );
+			$where .= $wpdb->prepare('AND p.post_parent = %d} ', $search);
 
-		} elseif ( is_numeric( $args['s'] ) ) {
+		} elseif (is_numeric($args['s'])) {
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$where .= $wpdb->prepare("
 				AND m.meta_key = '_give_payment_user_id'
 				AND m.meta_value = %d",
 				$args['s']
 			);
 
 		} else {
-			$search = $wpdb->esc_like( $args['s'] );
-			$search = '%' . $search . '%';
+			$search = $wpdb->esc_like($args['s']);
+			$search = '%'.$search.'%';
 
-			$where .= $wpdb->prepare( 'AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))', $search, $search );
+			$where .= $wpdb->prepare('AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))', $search, $search);
 		}
 	}
 
-	if ( ! empty( $args['form_id'] ) && is_numeric( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id']) && is_numeric($args['form_id'])) {
 
-		$where .= $wpdb->prepare( ' AND p.post_parent = %d', $args['form_id'] );
+		$where .= $wpdb->prepare(' AND p.post_parent = %d', $args['form_id']);
 
 	}
 	// Limit payments count by date.
-	if ( ! empty( $args['start-date'] ) && false !== strpos( $args['start-date'], '/' ) ) {
+	if ( ! empty($args['start-date']) && false !== strpos($args['start-date'], '/')) {
 
-		$date_parts = explode( '/', $args['start-date'] );
-		$month      = ! empty( $date_parts[0] ) && is_numeric( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day        = ! empty( $date_parts[1] ) && is_numeric( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year       = ! empty( $date_parts[2] ) && is_numeric( $date_parts[2] ) ? $date_parts[2] : 0;
+		$date_parts = explode('/', $args['start-date']);
+		$month      = ! empty($date_parts[0]) && is_numeric($date_parts[0]) ? $date_parts[0] : 0;
+		$day        = ! empty($date_parts[1]) && is_numeric($date_parts[1]) ? $date_parts[1] : 0;
+		$year       = ! empty($date_parts[2]) && is_numeric($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['start-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date >= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['start-date']);
+			$where .= $wpdb->prepare(" AND p.post_date >= '%s'", $date->format('Y-m-d'));
 
 		}
 
 		// Fixes an issue with the payments list table counts when no end date is specified (partiy with stats class).
-		if ( empty( $args['end-date'] ) ) {
+		if (empty($args['end-date'])) {
 			$args['end-date'] = $args['start-date'];
 		}
 	}
 
-	if ( ! empty( $args['end-date'] ) && false !== strpos( $args['end-date'], '/' ) ) {
+	if ( ! empty($args['end-date']) && false !== strpos($args['end-date'], '/')) {
 
-		$date_parts = explode( '/', $args['end-date'] );
+		$date_parts = explode('/', $args['end-date']);
 
-		$month = ! empty( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day   = ! empty( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year  = ! empty( $date_parts[2] ) ? $date_parts[2] : 0;
+		$month = ! empty($date_parts[0]) ? $date_parts[0] : 0;
+		$day   = ! empty($date_parts[1]) ? $date_parts[1] : 0;
+		$year  = ! empty($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['end-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date <= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['end-date']);
+			$where .= $wpdb->prepare(" AND p.post_date <= '%s'", $date->format('Y-m-d'));
 
 		}
 	}
 
-	$where = apply_filters( 'give_count_payments_where', $where );
-	$join  = apply_filters( 'give_count_payments_join', $join );
+	$where = apply_filters('give_count_payments_where', $where);
+	$join  = apply_filters('give_count_payments_join', $join);
 
 	$query = "$select
 		FROM $wpdb->posts p
@@ -504,36 +504,36 @@ 
 		GROUP BY p.post_status
 	";
 
-	$cache_key = md5( $query );
+	$cache_key = md5($query);
 
-	$count = wp_cache_get( $cache_key, 'counts' );
-	if ( false !== $count ) {
+	$count = wp_cache_get($cache_key, 'counts');
+	if (false !== $count) {
 		return $count;
 	}
 
-	$count = $wpdb->get_results( $query, ARRAY_A );
+	$count = $wpdb->get_results($query, ARRAY_A);
 
 	$stats    = array();
 	$statuses = get_post_stati();
-	if ( isset( $statuses['private'] ) && empty( $args['s'] ) ) {
-		unset( $statuses['private'] );
+	if (isset($statuses['private']) && empty($args['s'])) {
+		unset($statuses['private']);
 	}
 
-	foreach ( $statuses as $state ) {
-		$stats[ $state ] = 0;
+	foreach ($statuses as $state) {
+		$stats[$state] = 0;
 	}
 
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 
-		if ( 'private' == $row['post_status'] && empty( $args['s'] ) ) {
+		if ('private' == $row['post_status'] && empty($args['s'])) {
 			continue;
 		}
 
-		$stats[ $row['post_status'] ] = $row['num_posts'];
+		$stats[$row['post_status']] = $row['num_posts'];
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( $cache_key, $stats, 'counts' );
+	wp_cache_set($cache_key, $stats, 'counts');
 
 	return $stats;
 }
@@ -548,11 +548,11 @@ 
  *
  * @return bool $exists True if payment exists, false otherwise.
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -570,31 +570,31 @@ 
  *
  * @return bool|mixed True if payment status exists, false otherwise.
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
 	// Get payment object if no already given.
-	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment( $payment->ID );
+	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
 			// Return translated status label.
-			return $statuses[ $payment->status ];
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' == $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -610,17 +610,17 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' ),
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'revoked'     => __('Revoked', 'give'),
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -633,10 +633,10 @@ 
  * @return array $payment_status All the available payment statuses.
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -651,7 +651,7 @@ 
  *
  * @return int $earnings  Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
 	global $wpdb;
@@ -661,41 +661,41 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false,
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = get_transient( $key );
+		$earnings = get_transient($key);
 	}
 
-	if ( false === $earnings ) {
-		$sales    = get_posts( $args );
+	if (false === $earnings) {
+		$sales    = get_posts($args);
 		$earnings = 0;
-		if ( $sales ) {
-			$sales = implode( ',', $sales );
+		if ($sales) {
+			$sales = implode(',', $sales);
 
-			$earnings = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$sales})" );
+			$earnings = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$sales})");
 
 		}
 		// Cache the results for one hour.
-		set_transient( $key, $earnings, HOUR_IN_SECONDS );
+		set_transient($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -710,7 +710,7 @@ 
  *
  * @return int $count     Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead.
 	$args = array(
@@ -718,14 +718,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false,
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -736,33 +736,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$key = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = get_transient( $key );
+		$count = get_transient($key);
 	}
 
-	if ( false === $count ) {
-		$sales = new WP_Query( $args );
+	if (false === $count) {
+		$sales = new WP_Query($args);
 		$count = (int) $sales->post_count;
 		// Cache the results for one hour.
-		set_transient( $key, $count, HOUR_IN_SECONDS );
+		set_transient($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -777,19 +777,19 @@ 
  *
  * @return bool $ret True if complete, false otherwise.
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -815,57 +815,57 @@ 
  */
 function give_get_total_earnings() {
 
-	$total = get_option( 'give_earnings_total', false );
+	$total = get_option('give_earnings_total', false);
 
 	// If no total stored in DB, use old method of calculating total earnings.
-	if ( false === $total ) {
+	if (false === $total) {
 
 		global $wpdb;
 
-		$total = get_transient( 'give_earnings_total' );
+		$total = get_transient('give_earnings_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 
 			$total = (float) 0;
 
-			$args = apply_filters( 'give_get_total_earnings_args', array(
+			$args = apply_filters('give_get_total_earnings_args', array(
 				'offset' => 0,
-				'number' => - 1,
-				'status' => array( 'publish' ),
+				'number' => -1,
+				'status' => array('publish'),
 				'fields' => 'ids',
-			) );
+			));
 
-			$payments = give_get_payments( $args );
-			if ( $payments ) {
+			$payments = give_get_payments($args);
+			if ($payments) {
 
 				/**
 				 * If performing a donation, we need to skip the very last payment in the database,
 				 * since it calls give_increase_total_earnings() on completion,
 				 * which results in duplicated earnings for the very first donation.
 				 */
-				if ( did_action( 'give_update_payment_status' ) ) {
-					array_pop( $payments );
+				if (did_action('give_update_payment_status')) {
+					array_pop($payments);
 				}
 
-				if ( ! empty( $payments ) ) {
-					$payments = implode( ',', $payments );
-					$total += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})" );
+				if ( ! empty($payments)) {
+					$payments = implode(',', $payments);
+					$total += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})");
 				}
 			}
 
 			// Cache results for 1 day. This cache is cleared automatically when a payment is made.
-			set_transient( 'give_earnings_total', $total, 86400 );
+			set_transient('give_earnings_total', $total, 86400);
 
 			// Store the total for the first time.
-			update_option( 'give_earnings_total', $total );
+			update_option('give_earnings_total', $total);
 		}
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings.
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_currency_decimal_filter() ) );
+	return apply_filters('give_total_earnings', round($total, give_currency_decimal_filter()));
 }
 
 /**
@@ -878,10 +878,10 @@ 
  *
  * @return float $total  Total earnings.
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -895,13 +895,13 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -917,10 +917,10 @@ 
  *
  * @return mixed $meta Payment Meta.
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -933,10 +933,10 @@ 
  *
  * @return mixed Meta ID if successful, false if unsuccessful.
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -948,8 +948,8 @@ 
  *
  * @return string $user_info User Info Meta Values.
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -965,8 +965,8 @@ 
  *
  * @return int $form_id Form ID.
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -980,8 +980,8 @@ 
  *
  * @return string $email User email.
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -995,11 +995,11 @@ 
  *
  * @return bool $is_guest_payment If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -1011,8 +1011,8 @@ 
  *
  * @return int $user_id User ID.
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -1026,8 +1026,8 @@ 
  *
  * @return int $customer_id Customer ID.
  */
-function give_get_payment_customer_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_customer_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -1041,8 +1041,8 @@ 
  *
  * @return string $ip User IP.
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -1056,8 +1056,8 @@ 
  *
  * @return string $date The date the payment was completed.
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -1071,8 +1071,8 @@ 
  *
  * @return string $gateway Gateway.
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1086,8 +1086,8 @@ 
  *
  * @return string $currency The currency code.
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1101,10 +1101,10 @@ 
  *
  * @return string $currency The currency name.
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1116,8 +1116,8 @@ 
  *
  * @return string $key Donation key.
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1133,8 +1133,8 @@ 
  *
  * @return string $number Payment order number.
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1148,23 +1148,23 @@ 
  *
  * @return string      The formatted payment number.
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1177,17 +1177,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1196,24 +1196,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason.
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids',
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1222,13 +1222,13 @@ 
 		}
 	}
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1240,25 +1240,25 @@ 
  *
  * @return string The new Payment number without prefix and postfix.
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix.
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix.
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number.
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1274,10 +1274,10 @@ 
  *
  * @return string $amount Fully formatted payment amount.
  */
-function give_payment_amount( $payment_id = 0 ) {
-	$amount = give_get_payment_amount( $payment_id );
+function give_payment_amount($payment_id = 0) {
+	$amount = give_get_payment_amount($payment_id);
 
-	return give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1290,11 +1290,11 @@ 
  *
  * @return mixed|void
  */
-function give_get_payment_amount( $payment_id ) {
+function give_get_payment_amount($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	return apply_filters( 'give_payment_amount', floatval( $payment->total ), $payment_id );
+	return apply_filters('give_payment_amount', floatval($payment->total), $payment_id);
 }
 
 /**
@@ -1310,10 +1310,10 @@ 
  *
  * @return array Fully formatted payment subtotal.
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($subtotal), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1327,8 +1327,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted).
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new G_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new G_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1343,10 +1343,10 @@ 
  *
  * @return mixed array if payment fees found, false otherwise.
  */
-function give_get_payment_fees( $payment_id = 0, $type = 'all' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_fees($payment_id = 0, $type = 'all') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_fees( $type );
+	return $payment->get_fees($type);
 }
 
 /**
@@ -1358,8 +1358,8 @@ 
  *
  * @return string The donation ID.
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1374,15 +1374,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1395,12 +1395,12 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_key( $key ) {
+function give_get_purchase_id_by_key($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1418,12 +1418,12 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1440,19 +1440,19 @@ 
  *
  * @return array $notes Donation Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10);
 
-	$notes = get_comments( array( 'post_id' => $payment_id, 'order' => 'ASC', 'search' => $search ) );
+	$notes = get_comments(array('post_id' => $payment_id, 'order' => 'ASC', 'search' => $search));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1468,8 +1468,8 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
@@ -1481,14 +1481,14 @@ 
 	 * @param int    $payment_id Payment ID.
 	 * @param string $note       The note.
 	 */
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1497,7 +1497,7 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note',
 
-	) ) );
+	)));
 
 	/**
 	 * Fires after payment note inserted.
@@ -1508,7 +1508,7 @@ 
 	 * @param int    $payment_id Payment ID.
 	 * @param string $note       The note.
 	 */
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1523,8 +1523,8 @@ 
  *
  * @return bool True on success, false otherwise.
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
@@ -1536,9 +1536,9 @@ 
 	 * @param int $comment_id Note ID.
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
 
-	$ret = wp_delete_comment( $comment_id, true );
+	$ret = wp_delete_comment($comment_id, true);
 
 	/**
 	 * Fires after donation note deleted.
@@ -1548,7 +1548,7 @@ 
 	 * @param int $comment_id Note ID.
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1563,34 +1563,34 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = esc_html__( 'System', 'give' );
+		$user = esc_html__('System', 'give');
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id,
-	) ),
-		'give_delete_payment_note_' . $note->comment_ID
+	)),
+		'give_delete_payment_note_'.$note->comment_ID
 	);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" aria-label="' . esc_attr__( 'Delete this donation note.', 'give' ) . '">' . esc_html__( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" aria-label="'.esc_attr__('Delete this donation note.', 'give').'">'.esc_html__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1608,18 +1608,18 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+function give_hide_payment_notes($query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1631,15 +1631,15 @@ 
  *
  * @return array $clauses Updated comment clauses.
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1652,15 +1652,15 @@ 
  *
  * @return string $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( ' AND comment_type != %s', 'give_payment_note' );
+	$where .= $wpdb->prepare(' AND comment_type != %s', 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1674,32 +1674,32 @@ 
  *
  * @return array Array of comment counts.
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = wp_cache_get( "comments-{$post_id}", 'counts' );
+	$stats = wp_cache_get("comments-{$post_id}", 'counts');
 
-	if ( false !== $stats ) {
+	if (false !== $stats) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1709,30 +1709,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed',
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals.
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( "comments-{$post_id}", $stats, 'counts' );
+	wp_cache_set("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1745,9 +1745,9 @@ 
  *
  * @return string $where Modified where clause.
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week.
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1767,38 +1767,38 @@ 
  *
  * @return string $form_title Returns the full title if $only_level is false, otherwise returns the levels title.
  */
-function give_get_payment_form_title( $payment_meta, $only_level = false, $separator = '' ) {
+function give_get_payment_form_title($payment_meta, $only_level = false, $separator = '') {
 
-	$form_id    = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
-	$price_id   = isset( $payment_meta['price_id'] ) ? $payment_meta['price_id'] : null;
-	$form_title = isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '';
+	$form_id    = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
+	$price_id   = isset($payment_meta['price_id']) ? $payment_meta['price_id'] : null;
+	$form_title = isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '';
 
-	if ( $only_level == true ) {
+	if ($only_level == true) {
 		$form_title = '';
 	}
 
 	//If multi-level, append to the form title.
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
 		//Only add separator if there is a form title.
-		if ( ! empty( $form_title ) ) {
-			$form_title .= ' ' . $separator . ' ';
+		if ( ! empty($form_title)) {
+			$form_title .= ' '.$separator.' ';
 		}
 
 		$form_title .= '<span class="donation-level-text-wrap">';
 
-		if ( $price_id == 'custom' ) {
-			$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
-			$form_title .= ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
+		if ($price_id == 'custom') {
+			$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
+			$form_title .= ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
 		} else {
-			$form_title .= give_get_price_option_name( $form_id, $price_id );
+			$form_title .= give_get_price_option_name($form_id, $price_id);
 		}
 
 		$form_title .= '</span>';
 
 	}
 
-	return apply_filters( 'give_get_payment_form_title', $form_title, $payment_meta );
+	return apply_filters('give_get_payment_form_title', $form_title, $payment_meta);
 
 }
 
@@ -1812,20 +1812,20 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 
 	$price_id = 0;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = maybe_unserialize( get_post_meta( $form_id, '_give_donation_levels', true ) );
+		$levels = maybe_unserialize(get_post_meta($form_id, '_give_donation_levels', true));
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = (float) give_sanitize_amount( $level['_give_amount'] );
+			$level_amount = (float) give_sanitize_amount($level['_give_amount']);
 
 			// Check that this indeed the recurring price.
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 
@@ -1850,10 +1850,10 @@ 
  *
  * @return string|void
  */
-function give_get_form_dropdown( $args = array(), $echo = false ) {
-	$form_dropdown_html = Give()->html->forms_dropdown( $args );
+function give_get_form_dropdown($args = array(), $echo = false) {
+	$form_dropdown_html = Give()->html->forms_dropdown($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1870,17 +1870,17 @@ 
  *
  * @return string|void
  */
-function give_get_form_variable_price_dropdown( $args = array(), $echo = false ) {
+function give_get_form_variable_price_dropdown($args = array(), $echo = false) {
 
 	// Check for give form id.
-	if ( empty( $args['id'] ) ) {
+	if (empty($args['id'])) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $args['id'] );
+	$form = new Give_Donate_Form($args['id']);
 
 	// Check if form has variable prices or not.
-	if ( ! $form->ID || ! $form->has_variable_prices() ) {
+	if ( ! $form->ID || ! $form->has_variable_prices()) {
 		return false;
 	}
 
@@ -1888,22 +1888,22 @@ 
 	$variable_price_options = array();
 
 	// Check if multi donation form support custom donation or not.
-	if ( $form->is_custom_price_mode() ) {
-		$variable_price_options['custom'] = _x( 'Custom', 'custom donation dropdown item', 'give' );
+	if ($form->is_custom_price_mode()) {
+		$variable_price_options['custom'] = _x('Custom', 'custom donation dropdown item', 'give');
 	}
 
 	// Get variable price and ID from variable price array.
-	foreach ( $variable_prices as $variable_price ) {
-		$variable_price_options[ $variable_price['_give_id']['level_id'] ] = ! empty( $variable_price['_give_text'] ) ? $variable_price['_give_text'] : give_currency_filter( give_format_amount( $variable_price['_give_amount'] ) );
+	foreach ($variable_prices as $variable_price) {
+		$variable_price_options[$variable_price['_give_id']['level_id']] = ! empty($variable_price['_give_text']) ? $variable_price['_give_text'] : give_currency_filter(give_format_amount($variable_price['_give_amount']));
 	}
 
 	// Update options.
-	$args = array_merge( $args, array( 'options' => $variable_price_options ) );
+	$args = array_merge($args, array('options' => $variable_price_options));
 
 	// Generate select html.
-	$form_dropdown_html = Give()->html->select( $args );
+	$form_dropdown_html = Give()->html->select($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 

includes/actions.php

Spacing +32 added lines, -32 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_get_actions() {
 
-	$_get_action = ! empty( $_GET['give_action'] ) ? $_GET['give_action'] : null;
+	$_get_action = ! empty($_GET['give_action']) ? $_GET['give_action'] : null;
 
 	// Add backward compatibility to give-action param ( $_GET or $_POST )
-	if(  doing_action( 'admin_init' ) && empty( $_get_action ) ) {
-		$_get_action = ! empty( $_GET['give-action'] ) ? $_GET['give-action'] : null;
+	if (doing_action('admin_init') && empty($_get_action)) {
+		$_get_action = ! empty($_GET['give-action']) ? $_GET['give-action'] : null;
 	}
 
-	if ( isset( $_get_action ) ) {
+	if (isset($_get_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_GET.
 		 *
@@ -40,13 +40,13 @@ 
 		 *
 		 * @param array $_GET Array of HTTP GET variables.
 		 */
-		do_action( "give_{$_get_action}", $_GET );
+		do_action("give_{$_get_action}", $_GET);
 	}
 
 }
 
-add_action( 'init', 'give_get_actions' );
-add_action( 'admin_init', 'give_get_actions' );
+add_action('init', 'give_get_actions');
+add_action('admin_init', 'give_get_actions');
 
 /**
  * Hooks Give actions, when present in the $_POST superglobal. Every give_action
@@ -59,15 +59,15 @@ 
  */
 function give_post_actions() {
 
-	$_post_action = ! empty( $_POST['give_action'] ) ? $_POST['give_action'] : null;
+	$_post_action = ! empty($_POST['give_action']) ? $_POST['give_action'] : null;
 
 
 	// Add backward compatibility to give-action param ( $_GET or $_POST )
-	if(  doing_action( 'admin_init' ) && empty( $_post_action ) ) {
-		$_post_action = ! empty( $_POST['give-action'] ) ? $_POST['give-action'] : null;
+	if (doing_action('admin_init') && empty($_post_action)) {
+		$_post_action = ! empty($_POST['give-action']) ? $_POST['give-action'] : null;
 	}
 
-	if ( isset( $_post_action ) ) {
+	if (isset($_post_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_POST.
 		 *
@@ -75,13 +75,13 @@ 
 		 *
 		 * @param array $_POST Array of HTTP POST variables.
 		 */
-		do_action( "give_{$_post_action}", $_POST );
+		do_action("give_{$_post_action}", $_POST);
 	}
 
 }
 
-add_action( 'init', 'give_post_actions' );
-add_action( 'admin_init', 'give_post_actions' );
+add_action('init', 'give_post_actions');
+add_action('admin_init', 'give_post_actions');
 
 /**
  * Connect WordPress user with Donor.
@@ -91,27 +91,27 @@ 
  * @param  array $user_data User Data
  * @return void
  */
-function give_connect_donor_to_wpuser( $user_id, $user_data ){
-	$donor = new Give_Customer( $user_data['user_email'] );
+function give_connect_donor_to_wpuser($user_id, $user_data) {
+	$donor = new Give_Customer($user_data['user_email']);
 
 	// Validate donor id and check if do nor is already connect to wp user or not.
-	if( $donor->id && ! $donor->user_id ) {
+	if ($donor->id && ! $donor->user_id) {
 
 		// Update donor user_id.
-		if( $donor->update( array( 'user_id' => $user_id ) ) ) {
-			$donor_note = sprintf( esc_html__( 'WordPress user #%d is connected to #%d', 'give' ), $user_id, $donor->id );
-			$donor->add_note( $donor_note );
+		if ($donor->update(array('user_id' => $user_id))) {
+			$donor_note = sprintf(esc_html__('WordPress user #%d is connected to #%d', 'give'), $user_id, $donor->id);
+			$donor->add_note($donor_note);
 
 			// Update user_id meta in payments.
-			if( ! empty( $donor->payment_ids ) && ( $donations = explode( ',', $donor->payment_ids ) ) ) {
-				foreach ( $donations as $donation  ) {
-					update_post_meta( $donation, '_give_payment_user_id', $user_id );
+			if ( ! empty($donor->payment_ids) && ($donations = explode(',', $donor->payment_ids))) {
+				foreach ($donations as $donation) {
+					update_post_meta($donation, '_give_payment_user_id', $user_id);
 				}
 			}
 		}
 	}
 }
-add_action( 'give_insert_user', 'give_connect_donor_to_wpuser', 10, 2 );
+add_action('give_insert_user', 'give_connect_donor_to_wpuser', 10, 2);
 
 
 /**
@@ -125,21 +125,21 @@ 
 function give_validate_license_when_site_migrated() {
 	// Store current site address if not already stored.
 	$homeurl = home_url();
-	if( ! get_option( 'give_site_address_before_migrate' ) ) {
+	if ( ! get_option('give_site_address_before_migrate')) {
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', $homeurl );
+		update_option('give_site_address_before_migrate', $homeurl);
 
 		return;
 	}
 
-	if( $homeurl !== get_option( 'give_site_address_before_migrate' ) ) {
+	if ($homeurl !== get_option('give_site_address_before_migrate')) {
 		// Immediately run cron.
-		wp_schedule_single_event( time() , 'give_validate_license_when_site_migrated' );
+		wp_schedule_single_event(time(), 'give_validate_license_when_site_migrated');
 
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', home_url() );
+		update_option('give_site_address_before_migrate', home_url());
 	}
 
 }
-add_action( 'init', 'give_validate_license_when_site_migrated' );
-add_action( 'admin_init', 'give_validate_license_when_site_migrated' );
+add_action('init', 'give_validate_license_when_site_migrated');
+add_action('admin_init', 'give_validate_license_when_site_migrated');

includes/admin/dashboard-widgets.php

Spacing +32 added lines, -32 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_register_dashboard_widgets() {
-	if ( current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
-		wp_add_dashboard_widget( 'give_dashboard_sales', esc_html__( 'Give: Donation Statistics', 'give' ), 'give_dashboard_sales_widget' );
+	if (current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
+		wp_add_dashboard_widget('give_dashboard_sales', esc_html__('Give: Donation Statistics', 'give'), 'give_dashboard_sales_widget');
 	}
 }
 
-add_action( 'wp_dashboard_setup', 'give_register_dashboard_widgets', 10 );
+add_action('wp_dashboard_setup', 'give_register_dashboard_widgets', 10);
 
 /**
  * Sales Summary Dashboard Widget
@@ -38,7 +38,7 @@ 
  */
 function give_dashboard_sales_widget() {
 
-	if ( ! current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
+	if ( ! current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
 		return;
 	}
 	$stats = new Give_Payment_Stats; ?>
@@ -46,27 +46,27 @@ 
 	<div class="give-dashboard-widget">
 
 		<div class="give-dashboard-today give-clearfix">
-			<h3 class="give-dashboard-date-today"><?php echo date( _x( 'F j, Y', 'dashboard widget', 'give' ) ); ?></h3>
+			<h3 class="give-dashboard-date-today"><?php echo date(_x('F j, Y', 'dashboard widget', 'give')); ?></h3>
 
 			<p class="give-dashboard-happy-day"><?php
 				printf(
 				/* translators: %s: day of the week */
-					esc_html__( 'Happy %s!', 'give' ),
-					date( 'l', current_time( 'timestamp' ) )
+					esc_html__('Happy %s!', 'give'),
+					date('l', current_time('timestamp'))
 				);
 			?></p>
 
 			<p class="give-dashboard-today-earnings"><?php
-				$earnings_today = $stats->get_earnings( 0, 'today', false );
-				echo give_currency_filter( give_format_amount( $earnings_today ) );
+				$earnings_today = $stats->get_earnings(0, 'today', false);
+				echo give_currency_filter(give_format_amount($earnings_today));
 			?></p>
 
 			<p class="give-orders-today"><?php
-				$donations_today = $stats->get_sales( 0, 'today', false );
+				$donations_today = $stats->get_sales(0, 'today', false);
 				printf(
 					/* translators: %s: daily donation count */
-					esc_html__( '%s donations today', 'give' ),
-					give_format_amount( $donations_today, false )
+					esc_html__('%s donations today', 'give'),
+					give_format_amount($donations_today, false)
 				);
 			?></p>
 
@@ -76,34 +76,34 @@ 
 		<table class="give-table-stats">
 			<thead style="display: none;">
 			<tr>
-				<th><?php esc_html_e( 'This Week', 'give' ); ?></th>
-				<th><?php esc_html_e( 'This Month', 'give' ); ?></th>
-				<th><?php esc_html_e( 'Past 30 Days', 'give' ); ?></th>
+				<th><?php esc_html_e('This Week', 'give'); ?></th>
+				<th><?php esc_html_e('This Month', 'give'); ?></th>
+				<th><?php esc_html_e('Past 30 Days', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
 			<tr id="give-table-stats-tr-1">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_week' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_week'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'This Week', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('This Week', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_month' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_month'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'This Month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('This Month', 'give'); ?></p>
 				</td>
 			</tr>
 			<tr id="give-table-stats-tr-2">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'last_month' ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'last_month'))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'Last Month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('Last Month', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_year', false ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_year', false))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'This Year', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('This Year', 'give'); ?></p>
 				</td>
 			</tr>
 			</tbody>
@@ -123,23 +123,23 @@ 
  *
  * @return array
  */
-function give_dashboard_at_a_glance_widget( $items ) {
-	$num_posts = wp_count_posts( 'give_forms' );
+function give_dashboard_at_a_glance_widget($items) {
+	$num_posts = wp_count_posts('give_forms');
 
-	if ( $num_posts && $num_posts->publish ) {
+	if ($num_posts && $num_posts->publish) {
 
 		$text = sprintf(
 			/* translators: %s: number of posts published */
-			_n( '%s Give Form', '%s Give Forms', $num_posts->publish, 'give' ),
+			_n('%s Give Form', '%s Give Forms', $num_posts->publish, 'give'),
 			$num_posts->publish
 		);
 
-		$text = sprintf( $text, number_format_i18n( $num_posts->publish ) );
+		$text = sprintf($text, number_format_i18n($num_posts->publish));
 
-		if ( current_user_can( 'edit_give_forms', get_current_user_id() ) ) {
+		if (current_user_can('edit_give_forms', get_current_user_id())) {
 			$text = sprintf(
 				'<a class="give-forms-count" href="%1$s">%2$s</a>',
-				admin_url( 'edit.php?post_type=give_forms' ),
+				admin_url('edit.php?post_type=give_forms'),
 				$text
 			);
 		} else {
@@ -155,4 +155,4 @@ 
 	return $items;
 }
 
-add_filter( 'dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1 );
+add_filter('dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);

includes/admin/class-i18n-module.php

Spacing +43 added lines, -43 removed lines

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

includes/admin/customers/class-customer-table.php

Spacing +66 added lines, -66 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';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// 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
+		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?
-		) );
+		));
 
 	}
 
@@ -80,20 +80,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" 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
 	}
@@ -109,29 +109,29 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 
 			case 'num_purchases' :
-				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
+				$value = '<a href="'.
+				         admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
+				         ).'">'.esc_html($item['num_purchases']).'</a>';
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name]));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $item['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($item['date_created']));
 				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']);
 
 	}
 
@@ -142,13 +142,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $item ) {
-		$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'] );
-		$actions  = $this->get_row_actions( $item );
+	public function column_name($item) {
+		$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']);
+		$actions  = $this->get_row_actions($item);
 
-		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);
 	}
 
 	/**
@@ -160,14 +160,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => esc_html__( 'Name', 'give' ),
-			'email'         => esc_html__( 'Email', 'give' ),
-			'num_purchases' => esc_html__( 'Donations', 'give' ),
-			'amount_spent'  => esc_html__( 'Total Donated', 'give' ),
-			'date_created'  => esc_html__( 'Date Created', 'give' )
+			'name'          => esc_html__('Name', 'give'),
+			'email'         => esc_html__('Email', 'give'),
+			'num_purchases' => esc_html__('Donations', 'give'),
+			'amount_spent'  => esc_html__('Total Donated', 'give'),
+			'date_created'  => esc_html__('Date Created', 'give')
 		);
 
-		return apply_filters( 'give_report_customer_columns', $columns );
+		return apply_filters('give_report_customer_columns', $columns);
 
 	}
 
@@ -180,10 +180,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_purchases' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_purchases' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -195,34 +195,34 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $item ) {
+	public function get_row_actions($item) {
 
 		$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=' . $item['id'] ),
-				sprintf( esc_attr__( 'View "%s"', 'give' ), $item['name'] ),
-				esc_html__( 'View Donor', 'give' )
+				admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']),
+				sprintf(esc_attr__('View "%s"', 'give'), $item['name']),
+				esc_html__('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=' . $item['id'] ),
-				sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $item['name'] ),
-				esc_html__( 'Notes', 'give' )
+				admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$item['id']),
+				sprintf(esc_attr__('Notes for "%s"', 'give'), $item['name']),
+				esc_html__('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=' . $item['id'] ),
-				sprintf( esc_attr__( 'Delete "%s"', 'give' ), $item['name'] ),
-				esc_html__( 'Delete', 'give' )
+				admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$item['id']),
+				sprintf(esc_attr__('Delete "%s"', 'give'), $item['name']),
+				esc_html__('Delete', 'give')
 			)
 
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $item );
+		return apply_filters('give_donor_row_actions', $actions, $item);
 
 	}
 
@@ -233,7 +233,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.
 	}
 
@@ -245,7 +245,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;
 	}
 
 	/**
@@ -256,7 +256,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;
 	}
 
 	/**
@@ -273,10 +273,10 @@ 
 
 		$data    = array();
 		$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,
@@ -285,21 +285,21 @@ 
 			'orderby' => $orderby
 		);
 
-		if ( is_email( $search ) ) {
+		if (is_email($search)) {
 			$args['email'] = $search;
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 			$args['id'] = $search;
 		} else {
 			$args['name'] = $search;
 		}
 
-		$customers = Give()->customers->get_customers( $args );
+		$customers = Give()->customers->get_customers($args);
 
-		if ( $customers ) {
+		if ($customers) {
 
-			foreach ( $customers as $customer ) {
+			foreach ($customers as $customer) {
 
-				$user_id = ! empty( $customer->user_id ) ? intval( $customer->user_id ) : 0;
+				$user_id = ! empty($customer->user_id) ? intval($customer->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $customer->id,
@@ -333,16 +333,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 = give_count_total_customers();
 
-		$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)
+		));
 	}
 }
\ No newline at end of file

Indentation +2 added lines, -2 removed lines

@@ -173,8 +173,8 @@
 
 			case 'num_purchases' :
 				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
+						 admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
+						 ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
 				break;
 
 			case 'amount_spent' :

includes/admin/customers/customers.php

Spacing +157 added lines, -157 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
  */
 function give_customers_page() {
 	$default_views  = give_customer_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'customers';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_customer_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'customers';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_customer_view($requested_view, $default_views);
 	} else {
 		give_customers_list();
 	}
@@ -42,7 +42,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_customer_views', $views );
+	return apply_filters('give_customer_views', $views);
 
 }
 
@@ -56,7 +56,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_customer_tabs', $tabs );
+	return apply_filters('give_customer_tabs', $tabs);
 
 }
 
@@ -67,7 +67,7 @@ 
  * @return void
  */
 function give_customers_list() {
-	include( dirname( __FILE__ ) . '/class-customer-table.php' );
+	include(dirname(__FILE__).'/class-customer-table.php');
 
 	$customers_table = new Give_Customer_Reports_Table();
 	$customers_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
-			$customers_table->search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors' );
+			$customers_table->search_box(esc_html__('Search Donors', 'give'), 'give-donors');
 			$customers_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_customer_view( $view, $callbacks ) {
+function give_render_customer_view($view, $callbacks) {
 
 	$render = true;
 
-	$customer_view_role = apply_filters( 'give_view_customers_role', 'view_give_reports' );
+	$customer_view_role = apply_filters('give_view_customers_role', 'view_give_reports');
 
-	if ( ! current_user_can( $customer_view_role ) ) {
-		give_set_error( 'give-no-access', esc_html__( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($customer_view_role)) {
+		give_set_error('give-no-access', esc_html__('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
  
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_customer', esc_html__( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_customer', esc_html__('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$customer_id = (int) $_GET['id'];
-	$customer    = new Give_Customer( $customer_id );
+	$customer    = new Give_Customer($customer_id);
 
-	if ( empty( $customer->id ) ) {
-		give_set_error( 'give-invalid_customer', esc_html__( 'Invalid Donor ID.', 'give' ) );
+	if (empty($customer->id)) {
+		give_set_error('give-invalid_customer', esc_html__('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_print_errors( 0 ); ?>
+				<?php give_print_errors(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 ( $customer && $render ) : ?>
+		<?php if ($customer && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 			<?php
-			foreach ( $customer_tabs as $key => $tab ) :
+			foreach ($customer_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=' . $customer->id ) ),
-					esc_attr( $class ),
-					sanitize_html_class( $tab['dashicon'] ),
-					esc_html( $tab['title'] )
+					esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$customer->id)),
+					esc_attr($class),
+					sanitize_html_class($tab['dashicon']),
+					esc_html($tab['title'])
 				);
 			endforeach;
 			?>
 			</h2>
 
 			<div id="give-customer-card-wrapper">
-				<?php $callbacks[ $view ]( $customer ) ?>
+				<?php $callbacks[$view]($customer) ?>
 			</div>
 
 		<?php endif; ?>
@@ -189,9 +189,9 @@ 
  *
  * @return void
  */
-function give_customers_view( $customer ) {
+function give_customers_view($customer) {
 
-	$customer_edit_role = apply_filters( 'give_edit_customers_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_customers_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -200,32 +200,32 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $customer );
+	do_action('give_donor_card_top', $customer);
 	?>
 
 	<div id="donor-summary" class="info-wrapper customer-section postbox">
 
-		<form id="edit-customer-info" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $customer->id ); ?>">
+		<form id="edit-customer-info" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->id); ?>">
 
 			<div class="customer-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="customer-avatar">
-						<?php echo get_avatar( $customer->email ); ?>
+						<?php echo get_avatar($customer->email); ?>
 					</div>
 
 					<div id="customer-name-wrap" class="left">
 						<span class="customer-id">#<?php echo $customer->id; ?></span>
-						<span class="customer-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr( $customer->name ); ?>" placeholder="<?php esc_attr_e( 'Donor Name', 'give' ); ?>" /></span>
+						<span class="customer-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr($customer->name); ?>" placeholder="<?php esc_attr_e('Donor Name', 'give'); ?>" /></span>
 						<span class="customer-name info-item editable"><span data-key="name"><?php echo $customer->name; ?></span></span>
 					</div>
 					<p class="customer-since info-item">
-						<?php esc_html_e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $customer->date_created ) ) ?>
+						<?php esc_html_e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($customer->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $customer_edit_role ) ): ?>
-						<a href="#" id="edit-customer" class="button info-item editable customer-edit-link"><?php esc_html_e( 'Edit Donor', 'give' ); ?></a>
+					<?php if (current_user_can($customer_edit_role)): ?>
+						<a href="#" id="edit-customer" class="button info-item editable customer-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 ID:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php esc_html_e('User ID:', 'give'); ?></label></th>
 							<td class="row-title">
 								<span class="customer-user-id info-item edit-item">
 									<?php
@@ -251,38 +251,38 @@ 
 										'data'  => $data_atts,
 									);
 
-									if ( ! empty( $user_id ) ) {
-										$userdata           = get_userdata( $user_id );
+									if ( ! empty($user_id)) {
+										$userdata           = get_userdata($user_id);
 										$user_args['value'] = $userdata->user_login;
 									}
 
-									echo Give()->html->ajax_user_search( $user_args );
+									echo Give()->html->ajax_user_search($user_args);
 									?>
 									<input type="hidden" name="customerinfo[user_id]" data-key="user_id" value="<?php echo $customer->user_id; ?>" />
 								</span>
 			
 								<span class="customer-user-id info-item editable">
-									<?php if ( intval( $customer->user_id ) > 0 ) { ?>
+									<?php if (intval($customer->user_id) > 0) { ?>
 										<span data-key="user_id"><?php echo $customer->user_id; ?></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( $customer_edit_role ) && intval( $customer->user_id ) > 0 ) { ?>
-										<span class="disconnect-user"> - <a id="disconnect-customer" href="#disconnect" aria-label="<?php esc_attr_e( 'Disconnects the current user ID from this customer record.', 'give' ); ?>"><?php esc_html_e( 'Disconnect User', 'give' ); ?></a></span>
+									<?php if (current_user_can($customer_edit_role) && intval($customer->user_id) > 0) { ?>
+										<span class="disconnect-user"> - <a id="disconnect-customer" href="#disconnect" aria-label="<?php esc_attr_e('Disconnects the current user ID from this customer record.', 'give'); ?>"><?php esc_html_e('Disconnect User', 'give'); ?></a></span>
 									<?php } ?>
 								</span>
 							</td>
 						</tr>
-						<?php if ( isset( $customer->user_id ) && $customer->user_id > 0 ) : ?>
+						<?php if (isset($customer->user_id) && $customer->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="customer-address-wrapper">
 
 										<?php
-										$address  = get_user_meta( $customer->user_id, '_give_user_address', true );
+										$address  = get_user_meta($customer->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="customer-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="customer-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', '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', '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="customer-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $customer->id; ?>" />
-				<?php wp_nonce_field( 'edit-customer', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-customer', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-customer" />
-				<input type="submit" id="give-edit-customer-save" class="button-secondary" value="<?php esc_attr_e( 'Update Donor', 'give' ); ?>" />
-				<a id="give-edit-customer-cancel" href="" class="delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+				<input type="submit" id="give-edit-customer-save" class="button-secondary" value="<?php esc_attr_e('Update Donor', 'give'); ?>" />
+				<a id="give-edit-customer-cancel" href="" class="delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -376,24 +376,24 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $customer );
+	do_action('give_donor_before_stats', $customer);
 	?>
 
 	<div id="customer-stats-wrapper" class="customer-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $customer->email ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($customer->email)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					//Completed Donations
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $customer->purchase_count, 'give' ), $customer->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $customer );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $customer->purchase_count, 'give'), $customer->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $customer);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $customer->purchase_value ) ); ?> <?php esc_html_e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($customer->purchase_value)); ?> <?php esc_html_e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -405,7 +405,7 @@ 
 			 *
 			 * @param object $customer The customer object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $customer );
+			do_action('give_donor_stats_list', $customer);
 			?>
 		</ul>
 	</div>
@@ -418,7 +418,7 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $customer );
+	do_action('give_donor_before_tables_wrapper', $customer);
 	?>
 
 	<div id="customer-tables-wrapper" class="customer-section">
@@ -431,40 +431,40 @@ 
 		 *
 		 * @param object $customer The customer object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $customer );
+		do_action('give_donor_before_tables', $customer);
 		?>
 
-		<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( $customer->emails ) ) { ?>
+				<?php if ( ! empty($customer->emails)) { ?>
 
-					<?php foreach ( $customer->emails as $key => $email ) : ?>
+					<?php foreach ($customer->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=' . $customer->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 ), 'give_action' => 'remove_donor_email' ), $base_url ), 'give-remove-donor-email'      );
+									$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->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), 'give_action' => '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>
@@ -474,56 +474,56 @@ 
 						<td colspan="2" class="add-customer-email-td">
 							<div class="add-customer-email-wrapper">
 								<input type="hidden" name="customer-id" value="<?php echo $customer->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-customer-email" id="add-customer-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-customer-email" id="add-customer-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', 'easy-digital-downloads' ); ?></td></tr>
+					<tr><td colspan="2"><?php _e('No Emails Found', 'easy-digital-downloads'); ?></td></tr>
 				<?php } ?>
 			</tbody>
 		</table>
 
-		<h3><?php esc_html_e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $customer->payment_ids );
-		$payments    = give_get_payments( array( 'post__in' => $payment_ids ) );
-		$payments    = array_slice( $payments, 0, 10 );
+		$payment_ids = explode(',', $customer->payment_ids);
+		$payments    = give_get_payments(array('post__in' => $payment_ids));
+		$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-order-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-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')
 							);
 							?>
 
@@ -538,46 +538,46 @@ 
 							 * @param object $customer The customer object being displayed.
 							 * @param object $payment  The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $customer, $payment );
+							do_action('give_donor_recent_purchases_actions', $customer, $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 } ?>
 			</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( $customer->email );
+		$donations = give_get_users_completed_donations($customer->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>
@@ -585,7 +585,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>
@@ -599,7 +599,7 @@ 
 		 *
 		 * @param object $customer The customer object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $customer );
+		do_action('give_donor_after_tables', $customer);
 		?>
 
 	</div>
@@ -612,7 +612,7 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $customer );
+	do_action('give_donor_card_bottom', $customer);
 
 }
 
@@ -625,30 +625,30 @@ 
  *
  * @return void
  */
-function give_customer_notes_view( $customer ) {
+function give_customer_notes_view($customer) {
 
-	$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     = $customer->get_notes_count();
-	$per_page       = apply_filters( 'give_customer_notes_per_page', 20 );
-	$total_pages    = ceil( $note_count / $per_page );
-	$customer_notes = $customer->get_notes( $per_page, $paged );
+	$per_page       = apply_filters('give_customer_notes_per_page', 20);
+	$total_pages    = ceil($note_count / $per_page);
+	$customer_notes = $customer->get_notes($per_page, $paged);
 	?>
 
 	<div id="customer-notes-wrapper">
 		<div class="customer-notes-header">
-			<?php echo get_avatar( $customer->email, 30 ); ?> <span><?php echo $customer->name; ?></span>
+			<?php echo get_avatar($customer->email, 30); ?> <span><?php echo $customer->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-customer-note" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $customer->id ); ?>">
+				<form id="give-add-customer-note" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$customer->id); ?>">
 					<textarea id="customer-note" name="customer_note" class="customer-note-input" rows="10"></textarea>
 					<br />
 					<input type="hidden" id="customer-id" name="customer_id" value="<?php echo $customer->id; ?>" />
 					<input type="hidden" name="give_action" value="add-customer-note" />
-					<?php wp_nonce_field( 'add-customer-note', 'add_customer_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-customer-note', 'add_customer_note_nonce', true, true); ?>
 					<input id="add-customer-note" class="right button-primary" type="submit" value="Add Note" />
 				</form>
 			</div>
@@ -663,26 +663,26 @@ 
 			'show_all' => true
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-customer-notes" class="postbox">
-			<?php if ( count( $customer_notes ) > 0 ) { ?>
-				<?php foreach ( $customer_notes as $key => $note ) : ?>
+			<?php if (count($customer_notes) > 0) { ?>
+				<?php foreach ($customer_notes as $key => $note) : ?>
 					<div class="customer-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-customer-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>
 
@@ -698,9 +698,9 @@ 
  *
  * @return void
  */
-function give_customers_delete_view( $customer ) {
+function give_customers_delete_view($customer) {
 
-	$customer_edit_role = apply_filters( 'give_edit_customers_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_customers_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -709,15 +709,15 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_customer_delete_top', $customer );
+	do_action('give_customer_delete_top', $customer);
 	?>
 
 	<div class="info-wrapper customer-section">
 
-		<form id="delete-customer" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $customer->id ); ?>">
+		<form id="delete-customer" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$customer->id); ?>">
 
 			<div class="customer-notes-header">
-				<?php echo get_avatar( $customer->email, 30 ); ?> <span><?php echo $customer->name; ?></span>
+				<?php echo get_avatar($customer->email, 30); ?> <span><?php echo $customer->name; ?></span>
 			</div>
 
 
@@ -725,16 +725,16 @@ 
 
 				<span class="delete-customer-options">
 					<p>
-						<?php echo Give()->html->checkbox( array( 'name' => 'give-customer-delete-confirm' ) ); ?>
-						<label for="give-customer-delete-confirm"><?php esc_html_e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						<?php echo Give()->html->checkbox(array('name' => 'give-customer-delete-confirm')); ?>
+						<label for="give-customer-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-customer-delete-records',
-							'options' => array( 'disabled' => true )
-						) ); ?>
-						<label for="give-customer-delete-records"><?php esc_html_e( 'Delete all associated donations and records?', 'give' ); ?></label>
+							'options' => array('disabled' => true)
+						)); ?>
+						<label for="give-customer-delete-records"><?php esc_html_e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -747,16 +747,16 @@ 
 					 *
 					 * @param object $customer The customer object being displayed.
 					 */
-					do_action( 'give_customer_delete_inputs', $customer );
+					do_action('give_customer_delete_inputs', $customer);
 					?>
 				</span>
 
 				<span id="customer-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $customer->id; ?>" />
-					<?php wp_nonce_field( 'delete-customer', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-customer', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-customer" />
-					<input type="submit" disabled="disabled" id="give-delete-customer" class="button-primary" value="<?php esc_attr_e( 'Delete Donor', 'give' ); ?>" />
-					<a id="give-delete-customer-cancel" href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $customer->id ); ?>" class="delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+					<input type="submit" disabled="disabled" id="give-delete-customer" class="button-primary" value="<?php esc_attr_e('Delete Donor', 'give'); ?>" />
+					<a id="give-delete-customer-cancel" href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->id); ?>" class="delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -772,5 +772,5 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_customer_delete_bottom', $customer );
+	do_action('give_customer_delete_bottom', $customer);
 }

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

Indentation +2 added lines, -2 removed lines

@@ -173,8 +173,8 @@
 
 			case 'num_purchases' :
 				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
+						 admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
+						 ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
 				break;
 
 			case 'amount_spent' :

Spacing +61 added lines, -61 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
+		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;
 	}
 
@@ -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,25 +172,25 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 
 			case 'num_purchases' :
-				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
+				$value = '<a href="'.
+				         admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
+				         ).'">'.esc_html($item['num_purchases']).'</a>';
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name]));
 				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']);
 	}
 
 	/**
@@ -202,14 +202,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => esc_html__( 'Name', 'give' ),
-			'id'            => esc_html__( 'ID', 'give' ),
-			'email'         => esc_html__( 'Email', 'give' ),
-			'num_purchases' => esc_html__( 'Donations', 'give' ),
-			'amount_spent'  => esc_html__( 'Total Donated', 'give' )
+			'name'          => esc_html__('Name', 'give'),
+			'id'            => esc_html__('ID', 'give'),
+			'email'         => esc_html__('Email', 'give'),
+			'num_purchases' => esc_html__('Donations', 'give'),
+			'amount_spent'  => esc_html__('Total Donated', 'give')
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -222,10 +222,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_purchases' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_purchases' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -236,7 +236,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -248,7 +248,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;
 	}
 
 	/**
@@ -259,7 +259,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;
 	}
 
 	/**
@@ -276,10 +276,10 @@ 
 
 		$data    = array();
 		$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,
@@ -288,21 +288,21 @@ 
 			'orderby' => $orderby
 		);
 
-		if ( is_email( $search ) ) {
+		if (is_email($search)) {
 			$args['email'] = $search;
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 			$args['id'] = $search;
 		}
 
-		$donors = Give()->customers->get_customers( $args );
+		$donors = Give()->customers->get_customers($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,
@@ -335,16 +335,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 = give_count_total_customers();
 
-		$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)
+		));
 	}
 }
\ No newline at end of file