WordImpress / Give

includes/gateways/paypal-standard.php

Spacing +167 added lines, -167 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,10 +22,10 @@ 
  *
  * @return bool
  */
-function give_paypal_standard_billing_fields( $form_id ) {
+function give_paypal_standard_billing_fields($form_id) {
 
-	if ( give_is_setting_enabled( give_get_option( 'paypal_standard_billing_details' ) ) ) {
-		give_default_cc_address_fields( $form_id );
+	if (give_is_setting_enabled(give_get_option('paypal_standard_billing_details'))) {
+		give_default_cc_address_fields($form_id);
 
 		return true;
 	}
@@ -34,7 +34,7 @@ 
 
 }
 
-add_action( 'give_paypal_cc_form', 'give_paypal_standard_billing_fields' );
+add_action('give_paypal_cc_form', 'give_paypal_standard_billing_fields');
 
 /**
  * Process PayPal Payment.
@@ -45,27 +45,27 @@ 
  *
  * @return void
  */
-function give_process_paypal_payment( $payment_data ) {
+function give_process_paypal_payment($payment_data) {
 
 	// Validate nonce.
-	give_validate_nonce( $payment_data['gateway_nonce'], 'give-gateway' );
-	$payment_id = give_create_payment( $payment_data );
+	give_validate_nonce($payment_data['gateway_nonce'], 'give-gateway');
+	$payment_id = give_create_payment($payment_data);
 
 	// Check payment.
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		// Record the error.
-		give_record_gateway_error( __( 'Payment Error', 'give' ), sprintf( /* translators: %s: payment data */
-		__( 'Payment creation failed before sending donor to PayPal. Payment data: %s', 'give' ), json_encode( $payment_data ) ), $payment_id );
+		give_record_gateway_error(__('Payment Error', 'give'), sprintf( /* translators: %s: payment data */
+		__('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']);
 	}
 
 	// Redirect to PayPal.
-	wp_redirect( give_build_paypal_url( $payment_id, $payment_data ) );
+	wp_redirect(give_build_paypal_url($payment_id, $payment_data));
 	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.
@@ -75,17 +75,17 @@ 
  */
 function give_listen_for_paypal_ipn() {
 	// Regular PayPal IPN
-	if ( isset( $_GET['give-listener'] ) && 'IPN' === $_GET['give-listener'] ) {
+	if (isset($_GET['give-listener']) && 'IPN' === $_GET['give-listener']) {
 		/**
 		 * 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
@@ -96,7 +96,7 @@ 
 function give_process_paypal_ipn() {
 
 	// Check the request method is POST.
-	if ( isset( $_SERVER['REQUEST_METHOD'] ) && 'POST' !== $_SERVER['REQUEST_METHOD'] ) {
+	if (isset($_SERVER['REQUEST_METHOD']) && 'POST' !== $_SERVER['REQUEST_METHOD']) {
 		return;
 	}
 
@@ -104,11 +104,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';
@@ -117,39 +117,39 @@ 
 	$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',
@@ -169,25 +169,25 @@ 
 		);
 
 		// 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 ) ) {
-			give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( /* translators: %s: Paypal IPN response */
-			__( 'Invalid IPN verification response. IPN data: %s', 'give' ), json_encode( $api_response ) ) );
+		if (is_wp_error($api_response)) {
+			give_record_gateway_error(__('IPN Error', 'give'), sprintf( /* translators: %s: Paypal IPN response */
+			__('Invalid IPN verification response. IPN data: %s', 'give'), json_encode($api_response) ));
 
 			return; // Something went wrong
 		}
 
-		if ( 'VERIFIED' !== $api_response['body'] ) {
-			give_record_gateway_error( __( 'IPN Error', 'give' ), sprintf( /* translators: %s: Paypal IPN response */
-			__( 'Invalid IPN verification response. IPN data: %s', 'give' ), json_encode( $api_response ) ) );
+		if ('VERIFIED' !== $api_response['body']) {
+			give_record_gateway_error(__('IPN Error', 'give'), sprintf( /* translators: %s: Paypal IPN response */
+			__('Invalid IPN verification response. IPN data: %s', 'give'), json_encode($api_response) ));
 
 			return; // Response not okay.
 		}
 	}// End if().
 
 	// 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;
 	}
 
@@ -196,28 +196,28 @@ 
 		'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'];
 
 	// Check for PayPal IPN Notifications and update data based on it.
-	$current_timestamp = current_time( 'timestamp' );
+	$current_timestamp = current_time('timestamp');
 	$paypal_ipn_vars = array(
-		'auth_status'    => ( $api_response['body'] ) ? $api_response['body'] : 'N/A',
+		'auth_status'    => ($api_response['body']) ? $api_response['body'] : 'N/A',
 		'transaction_id' => $encoded_data_array['txn_id'],
 		'payment_id'     => $payment_id,
 	);
-	update_option( 'give_last_paypal_ipn_received', $paypal_ipn_vars );
-	give_insert_payment_note( $payment_id, sprintf(
-				__( 'Last IPN received on %s at %s', 'give' ),
-				date_i18n( 'm/d/Y', $current_timestamp ),
-				date_i18n( 'H:i', $current_timestamp )
+	update_option('give_last_paypal_ipn_received', $paypal_ipn_vars);
+	give_insert_payment_note($payment_id, sprintf(
+				__('Last IPN received on %s at %s', 'give'),
+				date_i18n('m/d/Y', $current_timestamp),
+				date_i18n('H:i', $current_timestamp)
 			)
 	);
-	give_update_meta( $payment_id, 'give_last_paypal_ipn_received', $current_timestamp );
+	give_update_meta($payment_id, 'give_last_paypal_ipn_received', $current_timestamp);
 
-	if ( has_action( 'give_paypal_' . $txn_type ) ) {
+	if (has_action('give_paypal_'.$txn_type)) {
 		/**
 		 * Fires while processing PayPal IPN $txn_type.
 		 *
@@ -228,7 +228,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.
@@ -240,12 +240,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.
@@ -257,99 +257,99 @@ 
  *
  * @return void
  */
-function give_process_paypal_web_accept( $data, $payment_id ) {
+function give_process_paypal_web_accept($data, $payment_id) {
 
 	// Only allow through these transaction types.
-	if ( 'web_accept' !== $data['txn_type'] && 'cart' !== $data['txn_type'] && 'refunded' !== strtolower( $data['payment_status'] ) ) {
+	if ('web_accept' !== $data['txn_type'] && 'cart' !== $data['txn_type'] && 'refunded' !== strtolower($data['payment_status'])) {
 		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 ( 'paypal' !== give_get_payment_gateway( $payment_id ) ) {
+	if ('paypal' !== give_get_payment_gateway($payment_id)) {
 		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( __( 'IPN Error', 'give' ), sprintf( /* translators: %s: Paypal IPN response */
-		__( '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, __( 'Payment failed due to invalid PayPal business email.', 'give' ) );
+		give_record_gateway_error(__('IPN Error', 'give'), sprintf( /* translators: %s: Paypal IPN response */
+		__('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, __('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( __( 'IPN Error', 'give' ), sprintf( /* translators: %s: Paypal IPN response */
-		__( '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, __( 'Payment failed due to invalid currency in PayPal IPN.', 'give' ) );
+		give_record_gateway_error(__('IPN Error', 'give'), sprintf( /* translators: %s: Paypal IPN response */
+		__('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, __('Payment failed due to invalid currency in PayPal IPN.', 'give'));
 
 		return;
 	}
 
 	// Process refunds & reversed.
-	if ( 'refunded' === $payment_status || 'reversed' === $payment_status ) {
-		give_process_paypal_refund( $data, $payment_id );
+	if ('refunded' === $payment_status || 'reversed' === $payment_status) {
+		give_process_paypal_refund($data, $payment_id);
 
 		return;
 	}
 
 	// Only complete payments once.
-	if ( 'publish' === get_post_status( $payment_id ) ) {
+	if ('publish' === get_post_status($payment_id)) {
 		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( __( 'IPN Error', 'give' ), sprintf( /* translators: %s: Paypal IPN response */
-		__( '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, __( 'Payment failed due to invalid amount in PayPal IPN.', 'give' ) );
+		give_record_gateway_error(__('IPN Error', 'give'), sprintf( /* translators: %s: Paypal IPN response */
+		__('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, __('Payment failed due to invalid amount in PayPal IPN.', 'give'));
 
 		return;
 	}
 
 	// Process completed donations.
-	if ( 'completed' === $payment_status || give_is_test_mode() ) {
+	if ('completed' === $payment_status || give_is_test_mode()) {
 
-		give_insert_payment_note( $payment_id, sprintf( /* translators: %s: Paypal transaction ID */
-		__( '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_insert_payment_note($payment_id, sprintf( /* translators: %s: Paypal transaction ID */
+		__('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');
 
-	} 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 ) ) {
-			give_insert_payment_note( $payment_id, $note );
+		if ( ! empty($note)) {
+			give_insert_payment_note($payment_id, $note);
 		}
 	}
 
 }
 
-add_action( 'give_paypal_web_accept', 'give_process_paypal_web_accept', 10, 2 );
+add_action('give_paypal_web_accept', 'give_process_paypal_web_accept', 10, 2);
 
 /**
  * Process PayPal IPN Refunds
@@ -361,35 +361,35 @@ 
  *
  * @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;
 	}
 
 	// Only refund payments once.
-	if ( 'refunded' === get_post_status( $payment_id ) ) {
+	if ('refunded' === get_post_status($payment_id)) {
 		return;
 	}
 
-	$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 */
-		__( 'Partial PayPal refund processed: %s', 'give' ), $data['parent_txn_id'] ) );
+		give_insert_payment_note($payment_id, sprintf( /* translators: %s: Paypal parent transaction ID */
+		__('Partial PayPal refund processed: %s', 'give'), $data['parent_txn_id'] ));
 
 		return; // This is a partial refund
 
 	}
 
-	give_insert_payment_note( $payment_id, sprintf( /* translators: 1: Paypal parent transaction ID 2. Paypal reason code */
-	__( 'PayPal Payment #%1$s Refunded for reason: %2$s', 'give' ), $data['parent_txn_id'], $data['reason_code'] ) );
-	give_insert_payment_note( $payment_id, sprintf( /* translators: %s: Paypal transaction ID */
-	__( 'PayPal Refund Transaction ID: %s', 'give' ), $data['txn_id'] ) );
-	give_update_payment_status( $payment_id, 'refunded' );
+	give_insert_payment_note($payment_id, sprintf( /* translators: 1: Paypal parent transaction ID 2. Paypal reason code */
+	__('PayPal Payment #%1$s Refunded for reason: %2$s', 'give'), $data['parent_txn_id'], $data['reason_code'] ));
+	give_insert_payment_note($payment_id, sprintf( /* translators: %s: Paypal transaction ID */
+	__('PayPal Refund Transaction ID: %s', 'give'), $data['txn_id'] ));
+	give_update_payment_status($payment_id, 'refunded');
 }
 
 /**
@@ -401,24 +401,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) {
 		$protocol = 'https://';
 	} else {
 		$protocol = 'http://';
 	}
 
 	// Check the current payment mode
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		// Test mode
-		$paypal_uri = $protocol . 'www.sandbox.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocol.'www.sandbox.paypal.com/cgi-bin/webscr';
 	} else {
 		// Live mode
-		$paypal_uri = $protocol . 'www.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocol.'www.paypal.com/cgi-bin/webscr';
 	}
 
-	return apply_filters( 'give_paypal_uri', $paypal_uri );
+	return apply_filters('give_paypal_uri', $paypal_uri);
 }
 
 /**
@@ -428,9 +428,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);
 }
 
 /**
@@ -444,26 +444,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();
 
@@ -473,7 +473,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 transaction ID, generate a link to the PayPal transaction ID details
@@ -485,16 +485,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);
 
 
 /**
@@ -506,64 +506,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 = __( 'Payment made via eCheck and will clear automatically in 5-8 days.', 'give' );
+			$note = __('Payment made via eCheck and will clear automatically in 5-8 days.', 'give');
 
 			break;
 
 		case 'address' :
 
-			$note = __( 'Payment requires a confirmed donor address and must be accepted manually through PayPal.', 'give' );
+			$note = __('Payment requires a confirmed donor address and must be accepted manually through PayPal.', 'give');
 
 			break;
 
 		case 'intl' :
 
-			$note = __( 'Payment must be accepted manually through PayPal due to international account regulations.', 'give' );
+			$note = __('Payment must be accepted manually through PayPal due to international account regulations.', 'give');
 
 			break;
 
 		case 'multi-currency' :
 
-			$note = __( 'Payment received in non-shop currency and must be accepted manually through PayPal.', 'give' );
+			$note = __('Payment received in non-shop currency and must be accepted manually through PayPal.', 'give');
 
 			break;
 
 		case 'paymentreview' :
 		case 'regulatory_review' :
 
-			$note = __( 'Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations.', 'give' );
+			$note = __('Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations.', 'give');
 
 			break;
 
 		case 'unilateral' :
 
-			$note = __( 'Payment was sent to non-confirmed or non-registered email address.', 'give' );
+			$note = __('Payment was sent to non-confirmed or non-registered email address.', 'give');
 
 			break;
 
 		case 'upgrade' :
 
-			$note = __( 'PayPal account must be upgraded before this payment can be accepted.', 'give' );
+			$note = __('PayPal account must be upgraded before this payment can be accepted.', 'give');
 
 			break;
 
 		case 'verify' :
 
-			$note = __( 'PayPal account is not verified. Verify account in order to accept this donation.', 'give' );
+			$note = __('PayPal account is not verified. Verify account in order to accept this donation.', 'give');
 
 			break;
 
 		case 'other' :
 
-			$note = __( 'Payment is pending for unknown reasons. Contact PayPal support for assistance.', 'give' );
+			$note = __('Payment is pending for unknown reasons. Contact PayPal support for assistance.', 'give');
 
 			break;
 
@@ -581,49 +581,49 @@ 
  *
  * @return mixed|string
  */
-function give_build_paypal_url( $payment_id, $payment_data ) {
+function give_build_paypal_url($payment_id, $payment_data) {
 	// 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.
-	$item_name = give_build_paypal_item_title( $payment_data );
+	$item_name = give_build_paypal_item_title($payment_data);
 
 	// 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'] ) ) {
+	if ( ! empty($payment_data['user_info']['address'])) {
 		$default_address = array(
 			'line1'   => '',
 			'line2'   => '',
@@ -633,7 +633,7 @@ 
 			'country' => '',
 		);
 
-		$address = wp_parse_args( $payment_data['user_info']['address'], $default_address );
+		$address = wp_parse_args($payment_data['user_info']['address'], $default_address);
 
 		$paypal_args['address1'] = $address['line1'];
 		$paypal_args['address2'] = $address['line2'];
@@ -654,13 +654,13 @@ 
 	 * @param array $paypal_args
 	 * @param array $payment_data
 	 */
-	$paypal_args = apply_filters( 'give_paypal_redirect_args', $paypal_args, $payment_data );
+	$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( '&amp;', '&', $paypal_redirect );
+	$paypal_redirect = str_replace('&amp;', '&', $paypal_redirect);
 
 	return $paypal_redirect;
 }
@@ -675,7 +675,7 @@ 
 function give_get_paypal_button_type() {
 	// paypal_button_type can be donation or standard.
 	$paypal_button_type = '_donations';
-	if ( 'standard' === give_get_option( 'paypal_button_type' ) ) {
+	if ('standard' === give_get_option('paypal_button_type')) {
 		$paypal_button_type = '_xclick';
 	}
 
@@ -692,30 +692,30 @@ 
  *
  * @return string
  */
-function give_build_paypal_item_title( $payment_data ) {
-	$form_id   = intval( $payment_data['post_data']['give-form-id'] );
+function give_build_paypal_item_title($payment_data) {
+	$form_id   = intval($payment_data['post_data']['give-form-id']);
 	$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'] );
-		$price_level_amount    = give_get_price_option_amount( $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']);
 
 		// Donation given doesn't match selected level (must be a custom amount).
-		if ( $price_level_amount != give_maybe_sanitize_amount( $payment_data['price'] ) ) {
-			$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
+		if ($price_level_amount != give_maybe_sanitize_amount($payment_data['price'])) {
+			$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
 			// user custom amount text if any, fallback to default if not.
-			$item_name .= ' - ' . give_check_variable( $custom_amount_text, 'empty', __( 'Custom Amount', 'give' ) );
+			$item_name .= ' - '.give_check_variable($custom_amount_text, 'empty', __('Custom Amount', 'give'));
 
-		} elseif ( ! empty( $item_price_level_text ) ) {
-			$item_name .= ' - ' . $item_price_level_text;
+		} elseif ( ! empty($item_price_level_text)) {
+			$item_name .= ' - '.$item_price_level_text;
 		}
 	} // End if().
-	elseif ( give_get_form_price( $form_id ) !== give_maybe_sanitize_amount( $payment_data['price'] ) ) {
-		$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
+	elseif (give_get_form_price($form_id) !== give_maybe_sanitize_amount($payment_data['price'])) {
+		$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
 		// user custom amount text if any, fallback to default if not.
-		$item_name .= ' - ' . give_check_variable( $custom_amount_text, 'empty', __( 'Custom Amount', 'give' ) );
+		$item_name .= ' - '.give_check_variable($custom_amount_text, 'empty', __('Custom Amount', 'give'));
 	}
 
 	return $item_name;

includes/plugin-compatibility.php

Spacing +13 added lines, -13 removed lines

@@ -19,10 +19,10 @@ 
  * @return void
  */
 function give_disable_mandrill_nl2br() {
-	add_filter( 'mandrill_nl2br', '__return_false' );
+	add_filter('mandrill_nl2br', '__return_false');
 }
 
-add_action( 'give_email_send_before', 'give_disable_mandrill_nl2br' );
+add_action('give_email_send_before', 'give_disable_mandrill_nl2br');
 
 
 /**
@@ -34,30 +34,30 @@ 
  */
 function give_clear_seo_sitemap_cache_on_settings_change() {
 	// Load required file if the fn 'is_plugin_active' doesn't exists.
-	if ( ! function_exists( 'is_plugin_active' ) ) {
-		require_once ABSPATH . 'wp-admin/includes/plugin.php';
+	if ( ! function_exists('is_plugin_active')) {
+		require_once ABSPATH.'wp-admin/includes/plugin.php';
 	}
 
-	if ( ( is_plugin_active( 'wordpress-seo/wp-seo.php' )
-	       || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
-	     && class_exists( 'WPSEO_Sitemaps_Cache' )
+	if ((is_plugin_active('wordpress-seo/wp-seo.php')
+	       || is_plugin_active('wordpress-seo-premium/wp-seo-premium.php'))
+	     && class_exists('WPSEO_Sitemaps_Cache')
 	) {
 
-		$forms_singular_option = give_get_option( 'forms_singular' );
-		$forms_archive_option  = give_get_option( 'forms_singular' );
+		$forms_singular_option = give_get_option('forms_singular');
+		$forms_archive_option  = give_get_option('forms_singular');
 
 		// If there is change detected for Single Form View and Form Archives options then proceed.
 		if (
-			( isset( $_POST['forms_singular'] ) && $_POST['forms_singular'] !== $forms_singular_option ) ||
-			( isset( $_POST['forms_archives'] ) && $_POST['forms_archives'] !== $forms_archive_option )
+			(isset($_POST['forms_singular']) && $_POST['forms_singular'] !== $forms_singular_option) ||
+			(isset($_POST['forms_archives']) && $_POST['forms_archives'] !== $forms_archive_option)
 		) {
 			// If Yoast SEO or Yoast SEO Premium plugin exists, then update seo sitemap cache.
 			$yoast_sitemaps_cache = new WPSEO_Sitemaps_Cache();
-			if ( method_exists( $yoast_sitemaps_cache, 'clear' ) ) {
+			if (method_exists($yoast_sitemaps_cache, 'clear')) {
 				WPSEO_Sitemaps_Cache::clear();
 			}
 		}
 	}
 }
 
-add_action( 'give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change' );
+add_action('give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change');

includes/post-types.php

Spacing +148 added lines, -148 removed lines

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

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

Spacing +81 added lines, -81 removed lines

@@ -9,13 +9,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -44,13 +44,13 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
-		add_action( 'give_log_view_actions', array( $this, 'give_forms_filter' ) );
+		add_action('give_log_view_actions', array($this, 'give_forms_filter'));
 	}
 
 	/**
@@ -64,41 +64,41 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		$payment = give_get_payment_by( 'id', $item['payment_id'] );
+		$payment = give_get_payment_by('id', $item['payment_id']);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'form' :
-				$form_title = get_the_title( $item[ $column_name ] );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $item[ $column_name ] ) : $form_title;
-				return '<a href="' . esc_url( add_query_arg( 'form', $item[ $column_name ] ) ) . '" >' . $form_title . '</a>';
+				$form_title = get_the_title($item[$column_name]);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $item[$column_name]) : $form_title;
+				return '<a href="'.esc_url(add_query_arg('form', $item[$column_name])).'" >'.$form_title.'</a>';
 
 			case 'donor_id' :
 				return sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['donor_id'] ) ),
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['donor_id'])),
 					$item['donor_name']
 				);
 
 			case 'amount' :
-				return give_currency_filter( give_format_amount( $item['amount'], array( 'sanitize' => false ) ) );
+				return give_currency_filter(give_format_amount($item['amount'], array('sanitize' => false)));
 
 			case 'status' :
 
-				$value = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
+				$value = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
 
-				if ( $payment->mode == 'test' ) {
-					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This donation was made in test mode.', 'give' ) . '">' . __( 'Test', 'give' ) . '</span>';
+				if ($payment->mode == 'test') {
+					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This donation was made in test mode.', 'give').'">'.__('Test', 'give').'</span>';
 				}
 
 				return $value;
 
 			case 'payment_id' :
-				return '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $item['payment_id'] ) . '">' . give_get_payment_number( $item['payment_id'] ) . '</a>';
+				return '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$item['payment_id']).'">'.give_get_payment_number($item['payment_id']).'</a>';
 
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -111,13 +111,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => __( 'Log ID', 'give' ),
-			'donor_id'   => __( 'Donor', 'give' ),
-			'form'       => __( 'Form', 'give' ),
-			'amount'     => __( 'Donation Amount', 'give' ),
-			'status'     => __( 'Status', 'give' ),
-			'payment_id' => __( 'Donation ID', 'give' ),
-			'date'       => __( 'Date', 'give' ),
+			'ID'         => __('Log ID', 'give'),
+			'donor_id'   => __('Donor', 'give'),
+			'form'       => __('Form', 'give'),
+			'amount'     => __('Donation Amount', 'give'),
+			'status'     => __('Status', 'give'),
+			'payment_id' => __('Donation ID', 'give'),
+			'date'       => __('Date', 'give'),
 		);
 
 		return $columns;
@@ -131,7 +131,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -142,7 +142,7 @@ 
 	 * @return mixed int If User ID, string If Email/Login
 	 */
 	public function get_filtered_user() {
-		return isset( $_GET['user'] ) ? absint( $_GET['user'] ) : false;
+		return isset($_GET['user']) ? absint($_GET['user']) : false;
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 	 * @return int Download ID
 	 */
 	public function get_filtered_give_form() {
-		return ! empty( $_GET['form'] ) ? absint( $_GET['form'] ) : false;
+		return ! empty($_GET['form']) ? absint($_GET['form']) : false;
 	}
 
 	/**
@@ -164,7 +164,7 @@ 
 	 * @return string|bool string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -180,20 +180,20 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -217,7 +217,7 @@ 
 
 		$meta_query = array();
 
-		if ( $user ) {
+		if ($user) {
 			// Show only logs from a specific user.
 			$meta_query[] = array(
 				'key'   => '_give_log_user_id',
@@ -226,8 +226,8 @@ 
 		}
 
 		$search = $this->get_search();
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				// This is an email search. We use this to ensure it works for guest users and logged-in users.
 				$key     = '_give_log_user_info';
 				$compare = 'LIKE';
@@ -236,32 +236,32 @@ 
 				$key     = '_give_log_user_id';
 				$compare = 'LIKE';
 
-				if ( ! is_numeric( $search ) ) {
+				if ( ! is_numeric($search)) {
 					// Searching for user by username
-					$user = get_user_by( 'login', $search );
+					$user = get_user_by('login', $search);
 
-					if ( $user ) {
+					if ($user) {
 						// Found one, set meta value to user's ID.
 						$search = $user->ID;
 					} else {
 						// No user found so let's do a real search query.
-						$users = new WP_User_Query( array(
+						$users = new WP_User_Query(array(
 							'search'         => $search,
-							'search_columns' => array( 'user_url', 'user_nicename' ),
+							'search_columns' => array('user_url', 'user_nicename'),
 							'number'         => 1,
 							'fields'         => 'ids',
-						) );
+						));
 
 						$found_user = $users->get_results();
 
-						if ( $found_user ) {
+						if ($found_user) {
 							$search = $found_user[0];
 						}
 					}
 				}
 			}
 
-			if ( ! $this->file_search ) {
+			if ( ! $this->file_search) {
 				// Meta query only works for non file name search.
 				$meta_query[] = array(
 					'key'     => $key,
@@ -282,7 +282,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -294,24 +294,24 @@ 
 	 * @return void
 	 */
 	public function give_forms_filter() {
-		$give_forms = get_posts( array(
+		$give_forms = get_posts(array(
 			'post_type'              => 'give_forms',
 			'post_status'            => 'any',
-			'posts_per_page'         => - 1,
+			'posts_per_page'         => -1,
 			'orderby'                => 'title',
 			'order'                  => 'ASC',
 			'fields'                 => 'ids',
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
-		) );
+		));
 
-		if ( $give_forms ) {
+		if ($give_forms) {
 			echo '<select name="form" id="give-log-form-filter">';
-			echo '<option value="0">' . __( 'All', 'give' ) . '</option>';
-			foreach ( $give_forms as $form ) {
-				$form_title = get_the_title( $form );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form ) : $form_title;
-				echo '<option value="' . $form . '"' . selected( $form, $this->get_filtered_give_form() ) . '>' . esc_html( $form_title ) . '</option>';
+			echo '<option value="0">'.__('All', 'give').'</option>';
+			foreach ($give_forms as $form) {
+				$form_title = get_the_title($form);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $form) : $form_title;
+				echo '<option value="'.$form.'"'.selected($form, $this->get_filtered_give_form()).'>'.esc_html($form_title).'</option>';
 			}
 			echo '</select>';
 		}
@@ -331,7 +331,7 @@ 
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
-		$give_form = empty( $_GET['s'] ) ? $this->get_filtered_give_form() : null;
+		$give_form = empty($_GET['s']) ? $this->get_filtered_give_form() : null;
 		$user      = $this->get_filtered_user();
 
 		$log_query = array(
@@ -342,37 +342,37 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$cache_key = Give_Cache::get_key( 'get_logs', $log_query );
+		$cache_key = Give_Cache::get_key('get_logs', $log_query);
 
 		// Return result from cache if exist.
-		if ( ! ( $logs_data = Give_Cache::get( $cache_key ) ) ) {
-			$logs = $give_logs->get_connected_logs( $log_query );
+		if ( ! ($logs_data = Give_Cache::get($cache_key))) {
+			$logs = $give_logs->get_connected_logs($log_query);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					$payment_id = give_get_meta( $log->ID, '_give_log_payment_id', true );
+			if ($logs) {
+				foreach ($logs as $log) {
+					$payment_id = give_get_meta($log->ID, '_give_log_payment_id', true);
 
 					// Make sure this payment hasn't been deleted.
-					if ( get_post( $payment_id ) ) :
-						$user_info      = give_get_payment_meta_user_info( $payment_id );
-						$payment_amount = give_get_payment_amount( $payment_id );
+					if (get_post($payment_id)) :
+						$user_info      = give_get_payment_meta_user_info($payment_id);
+						$payment_amount = give_get_payment_amount($payment_id);
 
 						$logs_data[] = array(
-							'ID'         => '<span class="give-item-label give-item-label-gray">' . $log->ID . '</span>',
+							'ID'         => '<span class="give-item-label give-item-label-gray">'.$log->ID.'</span>',
 							'payment_id' => $payment_id,
 							'form'       => $log->post_parent,
 							'amount'     => $payment_amount,
-							'donor_id'   => give_get_payment_donor_id( $payment_id ),
-							'donor_name' => trim( "{$user_info['first_name']} {$user_info['last_name']}" ),
-							'date'       => get_post_field( 'post_date', $payment_id ),
+							'donor_id'   => give_get_payment_donor_id($payment_id),
+							'donor_name' => trim("{$user_info['first_name']} {$user_info['last_name']}"),
+							'date'       => get_post_field('post_date', $payment_id),
 						);
 
 					endif;
 				}
 
 				// Cache results.
-				if ( ! empty( $logs_data ) ) {
-					Give_Cache::set( $cache_key, $logs_data );
+				if ( ! empty($logs_data)) {
+					Give_Cache::set($cache_key, $logs_data);
 				}
 			}
 		}
@@ -400,15 +400,15 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array();
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$current_page          = $this->get_pagenum();
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( $this->get_filtered_give_form(), 'sale', $this->get_meta_query() );
+		$total_items           = $give_logs->get_log_count($this->get_filtered_give_form(), 'sale', $this->get_meta_query());
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

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

Spacing +67 added lines, -67 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box donor-search" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h3 class="alignleft reports-earnings-title">
-					<span><?php esc_html_e( 'Donors Report', 'give' ); ?></span>
+					<span><?php esc_html_e('Donors Report', 'give'); ?></span>
 				</h3>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,35 +172,35 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['id'] ) ),
-						esc_html( $item['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['id'])),
+						esc_html($item['num_donations'])
 					);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -212,13 +212,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -231,10 +231,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -245,7 +245,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -257,7 +257,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -268,7 +268,7 @@ 
 	 * @return mixed string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -287,15 +287,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -323,9 +323,9 @@ 
 
 		$_donor_query['number'] = -1;
 		$_donor_query['offset'] = 0;
-		$donors = Give()->donors->get_donors( $_donor_query );
+		$donors = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -337,10 +337,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -349,10 +349,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -379,16 +379,16 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->reports_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/donors/donors.php

Spacing +164 added lines, -164 removed lines

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

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

Spacing +66 added lines, -66 removed lines

@@ -12,13 +12,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => __( 'Donor', 'give' ),     // Singular name of the listed records.
-			'plural'   => __( 'Donors', 'give' ),    // Plural name of the listed records.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		parent::__construct(array(
+			'singular' => __('Donor', 'give'), // Singular name of the listed records.
+			'plural'   => __('Donors', 'give'), // Plural name of the listed records.
+			'ajax'     => false, // Does this table support ajax?.
+		));
 
 	}
 
@@ -80,22 +80,22 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?>
+			)); ?>
 		</p>
 		<?php
 	}
@@ -111,31 +111,31 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($donor, $column_name) {
+		switch ($column_name) {
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor['id'])),
+					esc_html($donor['num_donations'])
 				);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $donor[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($donor[$column_name], array('sanitize' => false)));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $donor['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($donor['date_created']));
 				break;
 
 			default:
-				$value = isset( $donor[ $column_name ] ) ? $donor[ $column_name ] : null;
+				$value = isset($donor[$column_name]) ? $donor[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $donor['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $donor['id']);
 
 	}
 
@@ -146,13 +146,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = '#' . $donor['id'] . ' ';
-		$name     .= ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] );
-		$actions  = $this->get_row_actions( $donor );
+	public function column_name($donor) {
+		$name     = '#'.$donor['id'].' ';
+		$name .= ! empty($donor['name']) ? $donor['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+		$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']);
+		$actions  = $this->get_row_actions($donor);
 
-		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
+		return '<a href="'.esc_url($view_url).'">'.$name.'</a>'.$this->row_actions($actions);
 	}
 
 	/**
@@ -164,14 +164,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
-			'date_created'  => __( 'Date Created', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
+			'date_created'  => __('Date Created', 'give'),
 		);
 
-		return apply_filters( 'give_list_donors_columns', $columns );
+		return apply_filters('give_list_donors_columns', $columns);
 
 	}
 
@@ -185,13 +185,13 @@ 
 	public function get_sortable_columns() {
 
 		$columns = array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 
-		return apply_filters( 'give_list_donors_sortable_columns', $columns );
+		return apply_filters('give_list_donors_sortable_columns', $columns);
 	}
 
 	/**
@@ -204,19 +204,19 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $donor ) {
+	public function get_row_actions($donor) {
 
 		$actions = array(
 
-			'view' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] ), sprintf( esc_attr__( 'View "%s"', 'give' ), $donor['name'] ), __( 'View Donor', 'give' ) ),
+			'view' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']), sprintf(esc_attr__('View "%s"', 'give'), $donor['name']), __('View Donor', 'give')),
 
-			'notes' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor['id'] ), sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $donor['name'] ), __( 'Notes', 'give' ) ),
+			'notes' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor['id']), sprintf(esc_attr__('Notes for "%s"', 'give'), $donor['name']), __('Notes', 'give')),
 
-			'delete' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor['id'] ), sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'delete' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor['id']), sprintf(esc_attr__('Delete "%s"', 'give'), $donor['name']), __('Delete', 'give')),
 
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $donor );
+		return apply_filters('give_donor_row_actions', $actions, $donor);
 
 	}
 
@@ -230,7 +230,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place.
 	}
 
@@ -242,7 +242,7 @@ 
 	 * @return int Current page number.
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -253,7 +253,7 @@ 
 	 * @return mixed string If search is present, false otherwise.
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -270,13 +270,13 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? intval( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? intval($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -290,7 +290,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -305,9 +305,9 @@ 
 
 		$_donor_query['number'] = - 1;
 		$_donor_query['offset'] = 0;
-		$donors                 = Give()->donors->get_donors( $_donor_query );
+		$donors                 = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -319,10 +319,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -331,10 +331,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -357,16 +357,16 @@ 
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->donor_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/process-donation.php

Spacing +247 added lines, -248 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,7 +30,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_process_donation' );
+	do_action('give_pre_process_donation');
 
 	// Validate the form $_POST data
 	$valid_data = give_donation_form_validate_fields();
@@ -45,26 +45,26 @@ 
 	 * @param bool|array $valid_data Validate fields.
 	 * @param array $_POST Array of variables passed via the HTTP POST.
 	 */
-	do_action( 'give_checkout_error_checks', $valid_data, $_POST );
+	do_action('give_checkout_error_checks', $valid_data, $_POST);
 
-	$is_ajax = isset( $_POST['give_ajax'] );
+	$is_ajax = isset($_POST['give_ajax']);
 
 	// Process the login form
-	if ( isset( $_POST['give_login_submit'] ) ) {
+	if (isset($_POST['give_login_submit'])) {
 		give_process_form_login();
 	}
 
 	// Validate the user
-	$user = give_get_donation_form_user( $valid_data );
+	$user = give_get_donation_form_user($valid_data);
 
-	if ( false === $valid_data || give_get_errors() || ! $user ) {
-		if ( $is_ajax ) {
+	if (false === $valid_data || give_get_errors() || ! $user) {
+		if ($is_ajax) {
 			/**
 			 * Fires when AJAX sends back errors from the donation form.
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_ajax_donation_errors' );
+			do_action('give_ajax_donation_errors');
 			give_die();
 		} else {
 			return false;
@@ -72,17 +72,17 @@ 
 	}
 
 	// If AJAX send back success to proceed with form submission
-	if ( $is_ajax ) {
+	if ($is_ajax) {
 		echo 'success';
 		give_die();
 	}
 
 	// After AJAX: Setup session if not using php_sessions
-	if ( ! Give()->session->use_php_sessions() ) {
+	if ( ! Give()->session->use_php_sessions()) {
 		// Double-check that set_cookie is publicly accessible;
 		// we're using a slightly modified class-wp-sessions.php
-		$session_reflection = new ReflectionMethod( 'WP_Session', 'set_cookie' );
-		if ( $session_reflection->isPublic() ) {
+		$session_reflection = new ReflectionMethod('WP_Session', 'set_cookie');
+		if ($session_reflection->isPublic()) {
 			// Manually set the cookie.
 			Give()->session->init()->set_cookie();
 		}
@@ -97,20 +97,19 @@ 
 		'address'    => $user['address'],
 	);
 
-	$auth_key = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
+	$auth_key = defined('AUTH_KEY') ? AUTH_KEY : '';
 
-	$price        = isset( $_POST['give-amount'] ) ?
-		(float) apply_filters( 'give_donation_total', give_maybe_sanitize_amount( $_POST['give-amount'] ) ) :
-		'0.00';
-	$purchase_key = strtolower( md5( $user['user_email'] . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );
+	$price        = isset($_POST['give-amount']) ?
+		(float) apply_filters('give_donation_total', give_maybe_sanitize_amount($_POST['give-amount'])) : '0.00';
+	$purchase_key = strtolower(md5($user['user_email'].date('Y-m-d H:i:s').$auth_key.uniqid('give', true)));
 
 	// Setup donation information
 	$donation_data = array(
 		'price'        => $price,
 		'purchase_key' => $purchase_key,
 		'user_email'   => $user['user_email'],
-		'date'         => date( 'Y-m-d H:i:s', current_time( 'timestamp' ) ),
-		'user_info'    => stripslashes_deep( $user_info ),
+		'date'         => date('Y-m-d H:i:s', current_time('timestamp')),
+		'user_info'    => stripslashes_deep($user_info),
 		'post_data'    => $_POST,
 		'gateway'      => $valid_data['gateway'],
 		'card_info'    => $valid_data['cc_info'],
@@ -130,10 +129,10 @@ 
 	 * @param array $user_info Array containing basic user information.
 	 * @param bool|array $valid_data Validate fields.
 	 */
-	do_action( 'give_checkout_before_gateway', $_POST, $user_info, $valid_data );
+	do_action('give_checkout_before_gateway', $_POST, $user_info, $valid_data);
 
 	// Sanity check for price
-	if ( ! $donation_data['price'] ) {
+	if ( ! $donation_data['price']) {
 		// Revert to manual
 		$donation_data['gateway'] = 'manual';
 		$_POST['give-gateway']    = 'manual';
@@ -144,27 +143,27 @@ 
 	 *
 	 * @since 1.7
 	 */
-	$donation_data = apply_filters( 'give_donation_data_before_gateway', $donation_data, $valid_data );
+	$donation_data = apply_filters('give_donation_data_before_gateway', $donation_data, $valid_data);
 
 	// Setup the data we're storing in the donation session
 	$session_data = $donation_data;
 
 	// Make sure credit card numbers are never stored in sessions
-	unset( $session_data['card_info']['card_number'] );
-	unset( $session_data['post_data']['card_number'] );
+	unset($session_data['card_info']['card_number']);
+	unset($session_data['post_data']['card_number']);
 
 	// Used for showing data to non logged-in users after donation, and for other plugins needing donation data.
-	give_set_purchase_session( $session_data );
+	give_set_purchase_session($session_data);
 
 	// Send info to the gateway for payment processing
-	give_send_to_gateway( $donation_data['gateway'], $donation_data );
+	give_send_to_gateway($donation_data['gateway'], $donation_data);
 	give_die();
 
 }
 
-add_action( 'give_purchase', 'give_process_donation_form' );
-add_action( 'wp_ajax_give_process_donation', 'give_process_donation_form' );
-add_action( 'wp_ajax_nopriv_give_process_donation', 'give_process_donation_form' );
+add_action('give_purchase', 'give_process_donation_form');
+add_action('wp_ajax_give_process_donation', 'give_process_donation_form');
+add_action('wp_ajax_nopriv_give_process_donation', 'give_process_donation_form');
 
 
 /**
@@ -177,29 +176,29 @@ 
  *
  * @return void
  */
-function give_check_logged_in_user_for_existing_email( $valid_data, $post ) {
+function give_check_logged_in_user_for_existing_email($valid_data, $post) {
 
 	// Verify that the email address belongs to this customer.
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 
 		$submitted_email = $valid_data['logged_in_user']['user_email'];
-		$donor           = new Give_Donor( get_current_user_id(), true );
+		$donor           = new Give_Donor(get_current_user_id(), true);
 
 		// If this email address is not registered with this customer, see if it belongs to any other customer
 		if (
 			$submitted_email !== $donor->email
-			&& ( is_array( $donor->emails ) && ! in_array( $submitted_email, $donor->emails ) )
+			&& (is_array($donor->emails) && ! in_array($submitted_email, $donor->emails))
 		) {
-			$found_donor = new Give_Donor( $submitted_email );
+			$found_donor = new Give_Donor($submitted_email);
 
-			if ( $found_donor->id > 0 ) {
-				give_set_error( 'give-customer-email-exists', sprintf( __( 'You are logged in as %1$s, and are submitting a donation as %2$s, which is an existing donor. To ensure that the email address is tied to the correct donor, please submit this donation from a logged-out browser, or choose another email address.', 'give' ), $donor->email, $submitted_email ) );
+			if ($found_donor->id > 0) {
+				give_set_error('give-customer-email-exists', sprintf(__('You are logged in as %1$s, and are submitting a donation as %2$s, which is an existing donor. To ensure that the email address is tied to the correct donor, please submit this donation from a logged-out browser, or choose another email address.', 'give'), $donor->email, $submitted_email));
 			}
 		}
 	}
 }
 
-add_action( 'give_checkout_error_checks', 'give_check_logged_in_user_for_existing_email', 10, 2 );
+add_action('give_checkout_error_checks', 'give_check_logged_in_user_for_existing_email', 10, 2);
 
 /**
  * Process the checkout login form
@@ -209,49 +208,49 @@ 
  * @return      void
  */
 function give_process_form_login() {
-	$is_ajax = isset( $_POST['give_ajax'] );
+	$is_ajax = isset($_POST['give_ajax']);
 
 	$user_data = give_donation_form_validate_user_login();
 
-	if ( give_get_errors() || $user_data['user_id'] < 1 ) {
-		if ( $is_ajax ) {
+	if (give_get_errors() || $user_data['user_id'] < 1) {
+		if ($is_ajax) {
 			/**
 			 * Fires when AJAX sends back errors from the donation form.
 			 *
 			 * @since 1.0
 			 */
 			ob_start();
-			do_action( 'give_ajax_donation_errors' );
+			do_action('give_ajax_donation_errors');
 			$message = ob_get_contents();
 			ob_end_clean();
-			wp_send_json_error( $message );
+			wp_send_json_error($message);
 		} else {
-			wp_redirect( $_SERVER['HTTP_REFERER'] );
+			wp_redirect($_SERVER['HTTP_REFERER']);
 			exit;
 		}
 	}
 
-	give_log_user_in( $user_data['user_id'], $user_data['user_login'], $user_data['user_pass'] );
+	give_log_user_in($user_data['user_id'], $user_data['user_login'], $user_data['user_pass']);
 
-	if ( $is_ajax ) {
+	if ($is_ajax) {
 		$message = Give()->notices->print_frontend_notice(
 			sprintf(
 			/* translators: %s: user first name */
-				esc_html__( 'Welcome %s! You have successfully logged into your account.', 'give' ),
-				( ! empty( $user_data['user_first'] ) ) ? $user_data['user_first'] : $user_data['user_login']
+				esc_html__('Welcome %s! You have successfully logged into your account.', 'give'),
+				( ! empty($user_data['user_first'])) ? $user_data['user_first'] : $user_data['user_login']
 			),
 			false,
 			'success'
 		);
 
-		wp_send_json_success( $message );
+		wp_send_json_success($message);
 	} else {
-		wp_redirect( $_SERVER['HTTP_REFERER'] );
+		wp_redirect($_SERVER['HTTP_REFERER']);
 	}
 }
 
-add_action( 'wp_ajax_give_process_donation_login', 'give_process_form_login' );
-add_action( 'wp_ajax_nopriv_give_process_donation_login', 'give_process_form_login' );
+add_action('wp_ajax_give_process_donation_login', 'give_process_form_login');
+add_action('wp_ajax_nopriv_give_process_donation_login', 'give_process_form_login');
 
 /**
  * Donation Form Validate Fields.
@@ -263,55 +262,55 @@ 
 function give_donation_form_validate_fields() {
 
 	// Check if there is $_POST
-	if ( empty( $_POST ) ) {
+	if (empty($_POST)) {
 		return false;
 	}
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start an array to collect valid data
 	$valid_data = array(
 		'gateway'          => give_donation_form_validate_gateway(), // Gateway fallback (amount is validated here)
-		'need_new_user'    => false,     // New user flag
-		'need_user_login'  => false,     // Login user flag
-		'logged_user_data' => array(),   // Logged user collected data
-		'new_user_data'    => array(),   // New user collected data
-		'login_user_data'  => array(),   // Login user collected data
-		'guest_user_data'  => array(),   // Guest user collected data
-		'cc_info'          => give_donation_form_validate_cc(),// Credit card info
+		'need_new_user'    => false, // New user flag
+		'need_user_login'  => false, // Login user flag
+		'logged_user_data' => array(), // Logged user collected data
+		'new_user_data'    => array(), // New user collected data
+		'login_user_data'  => array(), // Login user collected data
+		'guest_user_data'  => array(), // Guest user collected data
+		'cc_info'          => give_donation_form_validate_cc(), // Credit card info
 	);
 
 	// Validate Honeypot First
-	if ( ! empty( $_POST['give-honeypot'] ) ) {
-		give_set_error( 'invalid_honeypot', esc_html__( 'Honeypot field detected. Go away bad bot!', 'give' ) );
+	if ( ! empty($_POST['give-honeypot'])) {
+		give_set_error('invalid_honeypot', esc_html__('Honeypot field detected. Go away bad bot!', 'give'));
 	}
 
 	// Validate agree to terms
-	if ( give_is_terms_enabled( $form_id ) ) {
+	if (give_is_terms_enabled($form_id)) {
 		give_donation_form_validate_agree_to_terms();
 	}
 
 	// Stop processing donor registration, if donor registration is optional and donor can do guest checkout.
 	// If registration form username field is empty that means donor do not want to registration instead want guest checkout.
 	if (
-		! give_logged_in_only( $form_id )
-		&& isset( $_POST['give-purchase-var'] )
+		! give_logged_in_only($form_id)
+		&& isset($_POST['give-purchase-var'])
 		&& $_POST['give-purchase-var'] == 'needs-to-register'
-		&& empty( $_POST['give_user_login'] )
+		&& empty($_POST['give_user_login'])
 	) {
-		unset( $_POST['give-purchase-var'] );
+		unset($_POST['give-purchase-var']);
 	}
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 		// Collect logged in user data.
 		$valid_data['logged_in_user'] = give_donation_form_validate_logged_in_user();
-	} elseif ( isset( $_POST['give-purchase-var'] ) && $_POST['give-purchase-var'] == 'needs-to-register' ) {
+	} elseif (isset($_POST['give-purchase-var']) && $_POST['give-purchase-var'] == 'needs-to-register') {
 		// Set new user registration as required.
 		$valid_data['need_new_user'] = true;
 		// Validate new user data.
 		$valid_data['new_user_data'] = give_donation_form_validate_new_user();
 		// Check if login validation is needed.
-	} elseif ( isset( $_POST['give-purchase-var'] ) && $_POST['give-purchase-var'] == 'needs-to-login' ) {
+	} elseif (isset($_POST['give-purchase-var']) && $_POST['give-purchase-var'] == 'needs-to-login') {
 		// Set user login as required.
 		$valid_data['need_user_login'] = true;
 		// Validate users login info.
@@ -336,41 +335,41 @@ 
  */
 function give_donation_form_validate_gateway() {
 
-	$form_id = isset( $_REQUEST['give-form-id'] ) ? $_REQUEST['give-form-id'] : 0;
-	$amount  = isset( $_REQUEST['give-amount'] ) ? give_maybe_sanitize_amount( $_REQUEST['give-amount'] ) : 0;
-	$gateway = give_get_default_gateway( $form_id );
+	$form_id = isset($_REQUEST['give-form-id']) ? $_REQUEST['give-form-id'] : 0;
+	$amount  = isset($_REQUEST['give-amount']) ? give_maybe_sanitize_amount($_REQUEST['give-amount']) : 0;
+	$gateway = give_get_default_gateway($form_id);
 
 	// Check if a gateway value is present.
-	if ( ! empty( $_REQUEST['give-gateway'] ) ) {
+	if ( ! empty($_REQUEST['give-gateway'])) {
 
-		$gateway = sanitize_text_field( $_REQUEST['give-gateway'] );
+		$gateway = sanitize_text_field($_REQUEST['give-gateway']);
 
 		// Is amount being donated in LIVE mode 0.00? If so, error:
-		if ( $amount == 0 && ! give_is_test_mode() ) {
+		if ($amount == 0 && ! give_is_test_mode()) {
 
-			give_set_error( 'invalid_donation_amount', __( 'Please insert a valid donation amount.', 'give' ) );
+			give_set_error('invalid_donation_amount', __('Please insert a valid donation amount.', 'give'));
 
 		} // End if().
-		elseif ( ! give_verify_minimum_price() ) {
+		elseif ( ! give_verify_minimum_price()) {
 			// translators: %s: minimum donation amount.
 			give_set_error(
 				'invalid_donation_minimum',
 				sprintf(
 				/* translators: %s: minimum donation amount */
-					__( 'This form has a minimum donation amount of %s.', 'give' ),
-					give_currency_filter( give_format_amount( give_get_form_minimum_price( $form_id ), array( 'sanitize' => false ) ) )
+					__('This form has a minimum donation amount of %s.', 'give'),
+					give_currency_filter(give_format_amount(give_get_form_minimum_price($form_id), array('sanitize' => false)))
 				)
 			);
 
 		} //Is this test mode zero donation? Let it through but set to manual gateway.
-		elseif ( $amount == 0 && give_is_test_mode() ) {
+		elseif ($amount == 0 && give_is_test_mode()) {
 
 			$gateway = 'manual';
 
 		} //Check if this gateway is active.
-		elseif ( ! give_is_gateway_active( $gateway ) ) {
+		elseif ( ! give_is_gateway_active($gateway)) {
 
-			give_set_error( 'invalid_gateway', __( 'The selected payment gateway is not enabled.', 'give' ) );
+			give_set_error('invalid_gateway', __('The selected payment gateway is not enabled.', 'give'));
 
 		}
 	}
@@ -388,21 +387,21 @@ 
  */
 function give_verify_minimum_price() {
 
-	$amount          = give_maybe_sanitize_amount( $_REQUEST['give-amount'] );
-	$form_id         = isset( $_REQUEST['give-form-id'] ) ? $_REQUEST['give-form-id'] : 0;
-	$price_id        = isset( $_REQUEST['give-price-id'] ) ? $_REQUEST['give-price-id'] : null;
-	$variable_prices = give_has_variable_prices( $form_id );
+	$amount          = give_maybe_sanitize_amount($_REQUEST['give-amount']);
+	$form_id         = isset($_REQUEST['give-form-id']) ? $_REQUEST['give-form-id'] : 0;
+	$price_id        = isset($_REQUEST['give-price-id']) ? $_REQUEST['give-price-id'] : null;
+	$variable_prices = give_has_variable_prices($form_id);
 
-	if ( $variable_prices && in_array( $price_id, give_get_variable_price_ids( $form_id ) ) ) {
+	if ($variable_prices && in_array($price_id, give_get_variable_price_ids($form_id))) {
 
-		$price_level_amount = give_get_price_option_amount( $form_id, $price_id );
+		$price_level_amount = give_get_price_option_amount($form_id, $price_id);
 
-		if ( $price_level_amount == $amount ) {
+		if ($price_level_amount == $amount) {
 			return true;
 		}
 	}
 
-	if ( give_get_form_minimum_price( $form_id ) > $amount ) {
+	if (give_get_form_minimum_price($form_id) > $amount) {
 		return false;
 	}
 
@@ -418,9 +417,9 @@ 
  */
 function give_donation_form_validate_agree_to_terms() {
 	// Validate agree to terms.
-	if ( ! isset( $_POST['give_agree_to_terms'] ) || $_POST['give_agree_to_terms'] != 1 ) {
+	if ( ! isset($_POST['give_agree_to_terms']) || $_POST['give_agree_to_terms'] != 1) {
 		// User did not agree.
-		give_set_error( 'agree_to_terms', apply_filters( 'give_agree_to_terms_text', __( 'You must agree to the terms and conditions.', 'give' ) ) );
+		give_set_error('agree_to_terms', apply_filters('give_agree_to_terms_text', __('You must agree to the terms and conditions.', 'give')));
 	}
 }
 
@@ -434,59 +433,59 @@ 
  *
  * @return      array
  */
-function give_get_required_fields( $form_id ) {
+function give_get_required_fields($form_id) {
 
-	$payment_mode = give_get_chosen_gateway( $form_id );
+	$payment_mode = give_get_chosen_gateway($form_id);
 
 	$required_fields = array(
 		'give_email' => array(
 			'error_id'      => 'invalid_email',
-			'error_message' => __( 'Please enter a valid email address.', 'give' ),
+			'error_message' => __('Please enter a valid email address.', 'give'),
 		),
 		'give_first' => array(
 			'error_id'      => 'invalid_first_name',
-			'error_message' => __( 'Please enter your first name.', 'give' ),
+			'error_message' => __('Please enter your first name.', 'give'),
 		),
 	);
 
-	$require_address = give_require_billing_address( $payment_mode );
+	$require_address = give_require_billing_address($payment_mode);
 
-	if ( $require_address ) {
-		$required_fields['card_address']    = array(
+	if ($require_address) {
+		$required_fields['card_address'] = array(
 			'error_id'      => 'invalid_card_address',
-			'error_message' => __( 'Please enter your primary billing address.', 'give' ),
+			'error_message' => __('Please enter your primary billing address.', 'give'),
 		);
-		$required_fields['card_zip']        = array(
+		$required_fields['card_zip'] = array(
 			'error_id'      => 'invalid_zip_code',
-			'error_message' => __( 'Please enter your zip / postal code.', 'give' ),
+			'error_message' => __('Please enter your zip / postal code.', 'give'),
 		);
-		$required_fields['card_city']       = array(
+		$required_fields['card_city'] = array(
 			'error_id'      => 'invalid_city',
-			'error_message' => __( 'Please enter your billing city.', 'give' ),
+			'error_message' => __('Please enter your billing city.', 'give'),
 		);
 		$required_fields['billing_country'] = array(
 			'error_id'      => 'invalid_country',
-			'error_message' => __( 'Please select your billing country.', 'give' ),
+			'error_message' => __('Please select your billing country.', 'give'),
 		);
 
 
 		$required_fields['card_state'] = array(
 			'error_id'      => 'invalid_state',
-			'error_message' => __( 'Please enter billing state / province / County.', 'give' ),
+			'error_message' => __('Please enter billing state / province / County.', 'give'),
 		);
 
 		// Check if billing country alredy exists.
-		if ( ! empty( $_POST['billing_country'] ) ) {
+		if ( ! empty($_POST['billing_country'])) {
 			// Get the value from $_POST.
-			$country = sanitize_text_field( $_POST['billing_country'] );
+			$country = sanitize_text_field($_POST['billing_country']);
 
 			// Get the country list that does not required any states init.
 			$states_country = give_states_not_required_country_list();
 
 			// Check if states is empty or not.
-			if ( array_key_exists( $country, $states_country ) ) {
+			if (array_key_exists($country, $states_country)) {
 				// If states is empty remove the required feilds of state in billing cart.
-				unset( $required_fields['card_state'] );
+				unset($required_fields['card_state']);
 			}
 		}
 	}
@@ -496,7 +495,7 @@ 
 	 *
 	 * @since 1.7
 	 */
-	$required_fields = apply_filters( 'give_donation_form_required_fields', $required_fields, $form_id );
+	$required_fields = apply_filters('give_donation_form_required_fields', $required_fields, $form_id);
 
 	return $required_fields;
 
@@ -511,16 +510,16 @@ 
  *
  * @return bool
  */
-function give_require_billing_address( $payment_mode ) {
+function give_require_billing_address($payment_mode) {
 
 	$return = false;
 
-	if ( isset( $_POST['billing_country'] ) || did_action( "give_{$payment_mode}_cc_form" ) || did_action( 'give_cc_form' ) ) {
+	if (isset($_POST['billing_country']) || did_action("give_{$payment_mode}_cc_form") || did_action('give_cc_form')) {
 		$return = true;
 	}
 
 	// Let payment gateways and other extensions determine if address fields should be required.
-	return apply_filters( 'give_require_billing_address', $return );
+	return apply_filters('give_require_billing_address', $return);
 
 }
 
@@ -534,42 +533,42 @@ 
 function give_donation_form_validate_logged_in_user() {
 	global $user_ID;
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start empty array to collect valid user data.
 	$valid_user_data = array(
 		// Assume there will be errors.
-		'user_id' => - 1,
+		'user_id' => -1,
 	);
 
 	// Verify there is a user_ID.
-	if ( $user_ID > 0 ) {
+	if ($user_ID > 0) {
 		// Get the logged in user data.
-		$user_data = get_userdata( $user_ID );
+		$user_data = get_userdata($user_ID);
 
 		// Loop through required fields and show error messages.
-		foreach ( give_get_required_fields( $form_id ) as $field_name => $value ) {
-			if ( in_array( $value, give_get_required_fields( $form_id ) ) && empty( $_POST[ $field_name ] ) ) {
-				give_set_error( $value['error_id'], $value['error_message'] );
+		foreach (give_get_required_fields($form_id) as $field_name => $value) {
+			if (in_array($value, give_get_required_fields($form_id)) && empty($_POST[$field_name])) {
+				give_set_error($value['error_id'], $value['error_message']);
 			}
 		}
 
 		// Verify data.
-		if ( $user_data ) {
+		if ($user_data) {
 			// Collected logged in user data.
 			$valid_user_data = array(
 				'user_id'    => $user_ID,
-				'user_email' => isset( $_POST['give_email'] ) ? sanitize_email( $_POST['give_email'] ) : $user_data->user_email,
-				'user_first' => isset( $_POST['give_first'] ) && ! empty( $_POST['give_first'] ) ? sanitize_text_field( $_POST['give_first'] ) : $user_data->first_name,
-				'user_last'  => isset( $_POST['give_last'] ) && ! empty( $_POST['give_last'] ) ? sanitize_text_field( $_POST['give_last'] ) : $user_data->last_name,
+				'user_email' => isset($_POST['give_email']) ? sanitize_email($_POST['give_email']) : $user_data->user_email,
+				'user_first' => isset($_POST['give_first']) && ! empty($_POST['give_first']) ? sanitize_text_field($_POST['give_first']) : $user_data->first_name,
+				'user_last'  => isset($_POST['give_last']) && ! empty($_POST['give_last']) ? sanitize_text_field($_POST['give_last']) : $user_data->last_name,
 			);
 
-			if ( ! is_email( $valid_user_data['user_email'] ) ) {
-				give_set_error( 'email_invalid', esc_html__( 'Invalid email.', 'give' ) );
+			if ( ! is_email($valid_user_data['user_email'])) {
+				give_set_error('email_invalid', esc_html__('Invalid email.', 'give'));
 			}
 		} else {
 			// Set invalid user error.
-			give_set_error( 'invalid_user', esc_html__( 'The user information is invalid.', 'give' ) );
+			give_set_error('invalid_user', esc_html__('The user information is invalid.', 'give'));
 		}
 	}
 
@@ -588,7 +587,7 @@ 
 	// Default user data.
 	$default_user_data = array(
 		'give-form-id'           => '',
-		'user_id'                => - 1, // Assume there will be errors.
+		'user_id'                => -1, // Assume there will be errors.
 		'user_first'             => '',
 		'user_last'              => '',
 		'give_user_login'        => false,
@@ -598,14 +597,14 @@ 
 	);
 
 	// Get user data.
-	$user_data            = wp_parse_args( array_map( 'trim', give_clean( $_POST ) ), $default_user_data );
+	$user_data            = wp_parse_args(array_map('trim', give_clean($_POST)), $default_user_data);
 	$registering_new_user = false;
-	$form_id              = absint( $user_data['give-form-id'] );
+	$form_id              = absint($user_data['give-form-id']);
 
 	// Start an empty array to collect valid user data.
 	$valid_user_data = array(
 		// Assume there will be errors.
-		'user_id'    => - 1,
+		'user_id'    => -1,
 
 		// Get first name.
 		'user_first' => $user_data['give_first'],
@@ -615,25 +614,25 @@ 
 	);
 
 	// Loop through required fields and show error messages.
-	foreach ( give_get_required_fields( $form_id ) as $field_name => $value ) {
-		if ( in_array( $value, give_get_required_fields( $form_id ) ) && empty( $_POST[ $field_name ] ) ) {
-			give_set_error( $value['error_id'], $value['error_message'] );
+	foreach (give_get_required_fields($form_id) as $field_name => $value) {
+		if (in_array($value, give_get_required_fields($form_id)) && empty($_POST[$field_name])) {
+			give_set_error($value['error_id'], $value['error_message']);
 		}
 	}
 
 	// Check if we have an username to register.
-	if ( give_validate_username( $user_data['give_user_login'] ) ) {
+	if (give_validate_username($user_data['give_user_login'])) {
 		$registering_new_user          = true;
 		$valid_user_data['user_login'] = $user_data['give_user_login'];
 	}
 
 	// Check if we have an email to verify.
-	if ( give_validate_user_email( $user_data['give_email'], $registering_new_user ) ) {
+	if (give_validate_user_email($user_data['give_email'], $registering_new_user)) {
 		$valid_user_data['user_email'] = $user_data['give_email'];
 	}
 
 	// Check password.
-	if ( give_validate_user_password( $user_data['give_user_pass'], $user_data['give_user_pass_confirm'], $registering_new_user ) ) {
+	if (give_validate_user_password($user_data['give_user_pass'], $user_data['give_user_pass_confirm'], $registering_new_user)) {
 		// All is good to go.
 		$valid_user_data['user_pass'] = $user_data['give_user_pass'];
 	}
@@ -653,36 +652,36 @@ 
 	// Start an array to collect valid user data.
 	$valid_user_data = array(
 		// Assume there will be errors.
-		'user_id' => - 1,
+		'user_id' => -1,
 	);
 
 	// Username.
-	if ( ! isset( $_POST['give_user_login'] ) || $_POST['give_user_login'] == '' ) {
-		give_set_error( 'must_log_in', __( 'You must register or login to complete your donation.', 'give' ) );
+	if ( ! isset($_POST['give_user_login']) || $_POST['give_user_login'] == '') {
+		give_set_error('must_log_in', __('You must register or login to complete your donation.', 'give'));
 
 		return $valid_user_data;
 	}
 
 	// Get the user by login.
-	$user_data = get_user_by( 'login', strip_tags( $_POST['give_user_login'] ) );
+	$user_data = get_user_by('login', strip_tags($_POST['give_user_login']));
 
 	// Check if user exists.
-	if ( $user_data ) {
+	if ($user_data) {
 		// Get password.
-		$user_pass = isset( $_POST['give_user_pass'] ) ? $_POST['give_user_pass'] : false;
+		$user_pass = isset($_POST['give_user_pass']) ? $_POST['give_user_pass'] : false;
 
 		// Check user_pass.
-		if ( $user_pass ) {
+		if ($user_pass) {
 			// Check if password is valid.
-			if ( ! wp_check_password( $user_pass, $user_data->user_pass, $user_data->ID ) ) {
+			if ( ! wp_check_password($user_pass, $user_data->user_pass, $user_data->ID)) {
 				// Incorrect password.
 				give_set_error(
 					'password_incorrect',
 					sprintf(
 						'%1$s <a href="%2$s">%3$s</a>',
-						__( 'The password you entered is incorrect.', 'give' ),
-						wp_lostpassword_url( "http://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]" ),
-						__( 'Reset Password', 'give' )
+						__('The password you entered is incorrect.', 'give'),
+						wp_lostpassword_url("http://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]"),
+						__('Reset Password', 'give')
 					)
 				);
 				// All is correct.
@@ -699,11 +698,11 @@ 
 			}
 		} else {
 			// Empty password.
-			give_set_error( 'password_empty', __( 'Enter a password.', 'give' ) );
+			give_set_error('password_empty', __('Enter a password.', 'give'));
 		}
 	} else {
 		// No username.
-		give_set_error( 'username_incorrect', __( 'The username you entered does not exist.', 'give' ) );
+		give_set_error('username_incorrect', __('The username you entered does not exist.', 'give'));
 	}// End if().
 
 	return $valid_user_data;
@@ -718,7 +717,7 @@ 
  */
 function give_donation_form_validate_guest_user() {
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start an array to collect valid user data.
 	$valid_user_data = array(
@@ -727,38 +726,38 @@ 
 	);
 
 	// Show error message if user must be logged in.
-	if ( give_logged_in_only( $form_id ) ) {
-		give_set_error( 'logged_in_only', __( 'You must be logged in to donate.', 'give' ) );
+	if (give_logged_in_only($form_id)) {
+		give_set_error('logged_in_only', __('You must be logged in to donate.', 'give'));
 	}
 
 	// Get the guest email.
-	$guest_email = isset( $_POST['give_email'] ) ? $_POST['give_email'] : false;
+	$guest_email = isset($_POST['give_email']) ? $_POST['give_email'] : false;
 
 	// Check email.
-	if ( $guest_email && strlen( $guest_email ) > 0 ) {
+	if ($guest_email && strlen($guest_email) > 0) {
 		// Validate email.
-		if ( ! is_email( $guest_email ) ) {
+		if ( ! is_email($guest_email)) {
 			// Invalid email.
-			give_set_error( 'email_invalid', __( 'Invalid email.', 'give' ) );
+			give_set_error('email_invalid', __('Invalid email.', 'give'));
 		} else {
 			// All is good to go.
 			$valid_user_data['user_email'] = $guest_email;
 
 			// Get user_id from donor if exist.
-			$donor = new Give_Donor( $guest_email );
-			if ( $donor->id && $donor->user_id ) {
+			$donor = new Give_Donor($guest_email);
+			if ($donor->id && $donor->user_id) {
 				$valid_user_data['user_id'] = $donor->user_id;
 			}
 		}
 	} else {
 		// No email.
-		give_set_error( 'email_empty', __( 'Enter an email.', 'give' ) );
+		give_set_error('email_empty', __('Enter an email.', 'give'));
 	}
 
 	// Loop through required fields and show error messages.
-	foreach ( give_get_required_fields( $form_id ) as $field_name => $value ) {
-		if ( in_array( $value, give_get_required_fields( $form_id ) ) && empty( $_POST[ $field_name ] ) ) {
-			give_set_error( $value['error_id'], $value['error_message'] );
+	foreach (give_get_required_fields($form_id) as $field_name => $value) {
+		if (in_array($value, give_get_required_fields($form_id)) && empty($_POST[$field_name])) {
+			give_set_error($value['error_id'], $value['error_message']);
 		}
 	}
 
@@ -774,36 +773,36 @@ 
  * @since   1.0
  * @return  integer
  */
-function give_register_and_login_new_user( $user_data = array() ) {
+function give_register_and_login_new_user($user_data = array()) {
 	// Verify the array.
-	if ( empty( $user_data ) ) {
-		return - 1;
+	if (empty($user_data)) {
+		return -1;
 	}
 
-	if ( give_get_errors() ) {
-		return - 1;
+	if (give_get_errors()) {
+		return -1;
 	}
 
-	$user_args = apply_filters( 'give_insert_user_args', array(
-		'user_login'      => isset( $user_data['user_login'] ) ? $user_data['user_login'] : '',
-		'user_pass'       => isset( $user_data['user_pass'] ) ? $user_data['user_pass'] : '',
-		'user_email'      => isset( $user_data['user_email'] ) ? $user_data['user_email'] : '',
-		'first_name'      => isset( $user_data['user_first'] ) ? $user_data['user_first'] : '',
-		'last_name'       => isset( $user_data['user_last'] ) ? $user_data['user_last'] : '',
-		'user_registered' => date( 'Y-m-d H:i:s' ),
-		'role'            => give_get_option( 'donor_default_user_role', 'give_donor' ),
-	), $user_data );
+	$user_args = apply_filters('give_insert_user_args', array(
+		'user_login'      => isset($user_data['user_login']) ? $user_data['user_login'] : '',
+		'user_pass'       => isset($user_data['user_pass']) ? $user_data['user_pass'] : '',
+		'user_email'      => isset($user_data['user_email']) ? $user_data['user_email'] : '',
+		'first_name'      => isset($user_data['user_first']) ? $user_data['user_first'] : '',
+		'last_name'       => isset($user_data['user_last']) ? $user_data['user_last'] : '',
+		'user_registered' => date('Y-m-d H:i:s'),
+		'role'            => give_get_option('donor_default_user_role', 'give_donor'),
+	), $user_data);
 
 	// Insert new user.
-	$user_id = wp_insert_user( $user_args );
+	$user_id = wp_insert_user($user_args);
 
 	// Validate inserted user.
-	if ( is_wp_error( $user_id ) ) {
-		return - 1;
+	if (is_wp_error($user_id)) {
+		return -1;
 	}
 
 	// Allow themes and plugins to filter the user data.
-	$user_data = apply_filters( 'give_insert_user_data', $user_data, $user_args );
+	$user_data = apply_filters('give_insert_user_data', $user_data, $user_args);
 
 	/**
 	 * Fires after inserting user.
@@ -813,7 +812,7 @@ 
 	 * @param int $user_id User id.
 	 * @param array $user_data Array containing user data.
 	 */
-	do_action( 'give_insert_user', $user_id, $user_data );
+	do_action('give_insert_user', $user_id, $user_data);
 
 	/**
 	 * Filter allow user to alter if user when to login or not when user is register for the first time.
@@ -822,9 +821,9 @@ 
 	 *
 	 * return bool True if login with registration and False if only want to register.
 	 */
-	if ( true === (bool) apply_filters( 'give_log_user_in_on_register', true ) ) {
+	if (true === (bool) apply_filters('give_log_user_in_on_register', true)) {
 		// Login new user.
-		give_log_user_in( $user_id, $user_data['user_login'], $user_data['user_pass'] );
+		give_log_user_in($user_id, $user_data['user_login'], $user_data['user_pass']);
 	}
 
 	// Return user id.
@@ -840,27 +839,27 @@ 
  * @since   1.0
  * @return  array|bool
  */
-function give_get_donation_form_user( $valid_data = array() ) {
+function give_get_donation_form_user($valid_data = array()) {
 
 	// Initialize user.
 	$user    = false;
-	$is_ajax = defined( 'DOING_AJAX' ) && DOING_AJAX;
+	$is_ajax = defined('DOING_AJAX') && DOING_AJAX;
 
-	if ( $is_ajax ) {
+	if ($is_ajax) {
 		// Do not create or login the user during the ajax submission (check for errors only).
 		return true;
-	} elseif ( is_user_logged_in() ) {
+	} elseif (is_user_logged_in()) {
 		// Set the valid user as the logged in collected data.
 		$user = $valid_data['logged_in_user'];
-	} elseif ( $valid_data['need_new_user'] === true || $valid_data['need_user_login'] === true ) {
+	} elseif ($valid_data['need_new_user'] === true || $valid_data['need_user_login'] === true) {
 		// New user registration.
-		if ( $valid_data['need_new_user'] === true ) {
+		if ($valid_data['need_new_user'] === true) {
 			// Set user.
 			$user = $valid_data['new_user_data'];
 			// Register and login new user.
-			$user['user_id'] = give_register_and_login_new_user( $user );
+			$user['user_id'] = give_register_and_login_new_user($user);
 			// User login
-		} elseif ( $valid_data['need_user_login'] === true && ! $is_ajax ) {
+		} elseif ($valid_data['need_user_login'] === true && ! $is_ajax) {
 
 			/**
 			 * The login form is now processed in the give_process_donation_login() function.
@@ -872,48 +871,48 @@ 
 			// Set user.
 			$user = $valid_data['login_user_data'];
 			// Login user.
-			give_log_user_in( $user['user_id'], $user['user_login'], $user['user_pass'] );
+			give_log_user_in($user['user_id'], $user['user_login'], $user['user_pass']);
 		}
 	}
 
 	// Check guest checkout.
-	if ( false === $user && false === give_logged_in_only( $_POST['give-form-id'] ) ) {
+	if (false === $user && false === give_logged_in_only($_POST['give-form-id'])) {
 		// Set user
 		$user = $valid_data['guest_user_data'];
 	}
 
 	// Verify we have an user.
-	if ( false === $user || empty( $user ) ) {
+	if (false === $user || empty($user)) {
 		// Return false.
 		return false;
 	}
 
 	// Get user first name.
-	if ( ! isset( $user['user_first'] ) || strlen( trim( $user['user_first'] ) ) < 1 ) {
-		$user['user_first'] = isset( $_POST['give_first'] ) ? strip_tags( trim( $_POST['give_first'] ) ) : '';
+	if ( ! isset($user['user_first']) || strlen(trim($user['user_first'])) < 1) {
+		$user['user_first'] = isset($_POST['give_first']) ? strip_tags(trim($_POST['give_first'])) : '';
 	}
 
 	// Get user last name.
-	if ( ! isset( $user['user_last'] ) || strlen( trim( $user['user_last'] ) ) < 1 ) {
-		$user['user_last'] = isset( $_POST['give_last'] ) ? strip_tags( trim( $_POST['give_last'] ) ) : '';
+	if ( ! isset($user['user_last']) || strlen(trim($user['user_last'])) < 1) {
+		$user['user_last'] = isset($_POST['give_last']) ? strip_tags(trim($_POST['give_last'])) : '';
 	}
 
 	// Get the user's billing address details.
 	$user['address']            = array();
-	$user['address']['line1']   = ! empty( $_POST['card_address'] ) ? give_clean( $_POST['card_address'] ) : false;
-	$user['address']['line2']   = ! empty( $_POST['card_address_2'] ) ? give_clean( $_POST['card_address_2'] ) : false;
-	$user['address']['city']    = ! empty( $_POST['card_city'] ) ? give_clean( $_POST['card_city'] ) : false;
-	$user['address']['state']   = ! empty( $_POST['card_state'] ) ? give_clean( $_POST['card_state'] ) : false;
-	$user['address']['zip']     = ! empty( $_POST['card_zip'] ) ? give_clean( $_POST['card_zip'] ) : false;
-	$user['address']['country'] = ! empty( $_POST['billing_country'] ) ? give_clean( $_POST['billing_country'] ) : false;
-
-	if ( empty( $user['address']['country'] ) ) {
+	$user['address']['line1']   = ! empty($_POST['card_address']) ? give_clean($_POST['card_address']) : false;
+	$user['address']['line2']   = ! empty($_POST['card_address_2']) ? give_clean($_POST['card_address_2']) : false;
+	$user['address']['city']    = ! empty($_POST['card_city']) ? give_clean($_POST['card_city']) : false;
+	$user['address']['state']   = ! empty($_POST['card_state']) ? give_clean($_POST['card_state']) : false;
+	$user['address']['zip']     = ! empty($_POST['card_zip']) ? give_clean($_POST['card_zip']) : false;
+	$user['address']['country'] = ! empty($_POST['billing_country']) ? give_clean($_POST['billing_country']) : false;
+
+	if (empty($user['address']['country'])) {
 		$user['address'] = false;
 	} // End if().
 
-	if ( ! empty( $user['user_id'] ) && $user['user_id'] > 0 && ! empty( $user['address'] ) ) {
+	if ( ! empty($user['user_id']) && $user['user_id'] > 0 && ! empty($user['address'])) {
 		// Store the address in the user's meta so the donation form can be pre-populated with it on return donation.
-		update_user_meta( $user['user_id'], '_give_user_address', $user['address'] );
+		update_user_meta($user['user_id'], '_give_user_address', $user['address']);
 	}
 
 	// Return valid user.
@@ -932,16 +931,16 @@ 
 	$card_data = give_get_donation_cc_info();
 
 	// Validate the card zip.
-	if ( ! empty( $card_data['card_zip'] ) ) {
-		if ( ! give_donation_form_validate_cc_zip( $card_data['card_zip'], $card_data['card_country'] ) ) {
-			give_set_error( 'invalid_cc_zip', __( 'The zip / postal code you entered for your billing address is invalid.', 'give' ) );
+	if ( ! empty($card_data['card_zip'])) {
+		if ( ! give_donation_form_validate_cc_zip($card_data['card_zip'], $card_data['card_country'])) {
+			give_set_error('invalid_cc_zip', __('The zip / postal code you entered for your billing address is invalid.', 'give'));
 		}
 	}
 
 	// Ensure no spaces.
-	if ( ! empty( $card_data['card_number'] ) ) {
-		$card_data['card_number'] = str_replace( '+', '', $card_data['card_number'] ); // no "+" signs
-		$card_data['card_number'] = str_replace( ' ', '', $card_data['card_number'] ); // No spaces
+	if ( ! empty($card_data['card_number'])) {
+		$card_data['card_number'] = str_replace('+', '', $card_data['card_number']); // no "+" signs
+		$card_data['card_number'] = str_replace(' ', '', $card_data['card_number']); // No spaces
 	}
 
 	// This should validate card numbers at some point too.
@@ -958,17 +957,17 @@ 
 function give_get_donation_cc_info() {
 
 	$cc_info                   = array();
-	$cc_info['card_name']      = isset( $_POST['card_name'] ) ? sanitize_text_field( $_POST['card_name'] ) : '';
-	$cc_info['card_number']    = isset( $_POST['card_number'] ) ? sanitize_text_field( $_POST['card_number'] ) : '';
-	$cc_info['card_cvc']       = isset( $_POST['card_cvc'] ) ? sanitize_text_field( $_POST['card_cvc'] ) : '';
-	$cc_info['card_exp_month'] = isset( $_POST['card_exp_month'] ) ? sanitize_text_field( $_POST['card_exp_month'] ) : '';
-	$cc_info['card_exp_year']  = isset( $_POST['card_exp_year'] ) ? sanitize_text_field( $_POST['card_exp_year'] ) : '';
-	$cc_info['card_address']   = isset( $_POST['card_address'] ) ? sanitize_text_field( $_POST['card_address'] ) : '';
-	$cc_info['card_address_2'] = isset( $_POST['card_address_2'] ) ? sanitize_text_field( $_POST['card_address_2'] ) : '';
-	$cc_info['card_city']      = isset( $_POST['card_city'] ) ? sanitize_text_field( $_POST['card_city'] ) : '';
-	$cc_info['card_state']     = isset( $_POST['card_state'] ) ? sanitize_text_field( $_POST['card_state'] ) : '';
-	$cc_info['card_country']   = isset( $_POST['billing_country'] ) ? sanitize_text_field( $_POST['billing_country'] ) : '';
-	$cc_info['card_zip']       = isset( $_POST['card_zip'] ) ? sanitize_text_field( $_POST['card_zip'] ) : '';
+	$cc_info['card_name']      = isset($_POST['card_name']) ? sanitize_text_field($_POST['card_name']) : '';
+	$cc_info['card_number']    = isset($_POST['card_number']) ? sanitize_text_field($_POST['card_number']) : '';
+	$cc_info['card_cvc']       = isset($_POST['card_cvc']) ? sanitize_text_field($_POST['card_cvc']) : '';
+	$cc_info['card_exp_month'] = isset($_POST['card_exp_month']) ? sanitize_text_field($_POST['card_exp_month']) : '';
+	$cc_info['card_exp_year']  = isset($_POST['card_exp_year']) ? sanitize_text_field($_POST['card_exp_year']) : '';
+	$cc_info['card_address']   = isset($_POST['card_address']) ? sanitize_text_field($_POST['card_address']) : '';
+	$cc_info['card_address_2'] = isset($_POST['card_address_2']) ? sanitize_text_field($_POST['card_address_2']) : '';
+	$cc_info['card_city']      = isset($_POST['card_city']) ? sanitize_text_field($_POST['card_city']) : '';
+	$cc_info['card_state']     = isset($_POST['card_state']) ? sanitize_text_field($_POST['card_state']) : '';
+	$cc_info['card_country']   = isset($_POST['billing_country']) ? sanitize_text_field($_POST['billing_country']) : '';
+	$cc_info['card_zip']       = isset($_POST['card_zip']) ? sanitize_text_field($_POST['card_zip']) : '';
 
 	// Return cc info.
 	return $cc_info;
@@ -984,14 +983,14 @@ 
  *
  * @return bool|mixed
  */
-function give_donation_form_validate_cc_zip( $zip = 0, $country_code = '' ) {
+function give_donation_form_validate_cc_zip($zip = 0, $country_code = '') {
 	$ret = false;
 
-	if ( empty( $zip ) || empty( $country_code ) ) {
+	if (empty($zip) || empty($country_code)) {
 		return $ret;
 	}
 
-	$country_code = strtoupper( $country_code );
+	$country_code = strtoupper($country_code);
 
 	$zip_regex = array(
 		'AD' => 'AD\d{3}',
@@ -1151,11 +1150,11 @@ 
 		'ZM' => '\d{5}',
 	);
 
-	if ( ! isset( $zip_regex[ $country_code ] ) || preg_match( '/' . $zip_regex[ $country_code ] . '/i', $zip ) ) {
+	if ( ! isset($zip_regex[$country_code]) || preg_match('/'.$zip_regex[$country_code].'/i', $zip)) {
 		$ret = true;
 	}
 
-	return apply_filters( 'give_is_zip_valid', $ret, $zip, $country_code );
+	return apply_filters('give_is_zip_valid', $ret, $zip, $country_code);
 }
 
 
@@ -1169,33 +1168,33 @@ 
  *
  * @return bool
  */
-function give_validate_multi_donation_form_level( $valid_data, $data ) {
+function give_validate_multi_donation_form_level($valid_data, $data) {
 	/* @var Give_Donate_Form $form */
-	$form = new Give_Donate_Form( $data['give-form-id'] );
+	$form = new Give_Donate_Form($data['give-form-id']);
 
 	$donation_level_matched = false;
 
-	if ( $form->is_multi_type_donation_form() ) {
+	if ($form->is_multi_type_donation_form()) {
 
 		// Bailout.
-		if ( ! ( $variable_prices = $form->get_prices() ) ) {
+		if ( ! ($variable_prices = $form->get_prices())) {
 			return false;
 		}
 
 		// Sanitize donation amount.
-		$data['give-amount'] = give_maybe_sanitize_amount( $data['give-amount'] );
+		$data['give-amount'] = give_maybe_sanitize_amount($data['give-amount']);
 
-		if ( $data['give-amount'] === give_maybe_sanitize_amount( give_get_price_option_amount( $data['give-form-id'], $data['give-price-id'] ) ) ) {
+		if ($data['give-amount'] === give_maybe_sanitize_amount(give_get_price_option_amount($data['give-form-id'], $data['give-price-id']))) {
 			return true;
 		}
 
 		// Find correct donation level from all donation levels.
-		foreach ( $variable_prices as $variable_price ) {
+		foreach ($variable_prices as $variable_price) {
 			// Sanitize level amount.
-			$variable_price['_give_amount'] = give_maybe_sanitize_amount( $variable_price['_give_amount'] );
+			$variable_price['_give_amount'] = give_maybe_sanitize_amount($variable_price['_give_amount']);
 
 			// Set first match donation level ID.
-			if ( $data['give-amount'] === $variable_price['_give_amount'] ) {
+			if ($data['give-amount'] === $variable_price['_give_amount']) {
 				$_POST['give-price-id'] = $variable_price['_give_id']['level_id'];
 				$donation_level_matched = true;
 				break;
@@ -1206,19 +1205,19 @@ 
 		// If yes then set price id to custom if amount is greater then custom minimum amount (if any).
 		if (
 			! $donation_level_matched
-			&& ( give_is_setting_enabled( give_get_meta( $data['give-form-id'], '_give_custom_amount', true ) ) )
+			&& (give_is_setting_enabled(give_get_meta($data['give-form-id'], '_give_custom_amount', true)))
 		) {
 			// Sanitize custom minimum amount.
-			$custom_minimum_amount = give_maybe_sanitize_amount( give_get_meta( $data['give-form-id'], '_give_custom_amount_minimum', true ) );
+			$custom_minimum_amount = give_maybe_sanitize_amount(give_get_meta($data['give-form-id'], '_give_custom_amount_minimum', true));
 
-			if ( $data['give-amount'] >= $custom_minimum_amount ) {
+			if ($data['give-amount'] >= $custom_minimum_amount) {
 				$_POST['give-price-id'] = 'custom';
 				$donation_level_matched = true;
 			}
 		}
 	}// End if().
 
-	return ( $donation_level_matched ? true : false );
+	return ($donation_level_matched ? true : false);
 }
 
-add_action( 'give_checkout_error_checks', 'give_validate_multi_donation_form_level', 10, 2 );
+add_action('give_checkout_error_checks', 'give_validate_multi_donation_form_level', 10, 2);

includes/error-tracking.php

Spacing +19 added lines, -19 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  * @return mixed array if errors are present, false if none found
  */
 function give_get_errors() {
-	return Give()->session->get( 'give_errors' );
+	return Give()->session->get('give_errors');
 }
 
 /**
@@ -42,23 +42,23 @@ 
  *
  * @return void
  */
-function give_set_error( $error_id, $error_message, $notice_args = array() ) {
+function give_set_error($error_id, $error_message, $notice_args = array()) {
 	$errors = give_get_errors();
-	if ( ! $errors ) {
+	if ( ! $errors) {
 		$errors = array();
 	}
 
-	if( is_array( $notice_args ) && ! empty( $notice_args ) ) {
-		$errors[ $error_id ] = array(
+	if (is_array($notice_args) && ! empty($notice_args)) {
+		$errors[$error_id] = array(
 			'message'     => $error_message,
 			'notice_args' => $notice_args,
 		);
 	} else {
 		// Backward compatibility v<1.8.11.
-		$errors[ $error_id ] = $error_message;
+		$errors[$error_id] = $error_message;
 	}
 
-	Give()->session->set( 'give_errors', $errors );
+	Give()->session->set('give_errors', $errors);
 }
 
 /**
@@ -69,7 +69,7 @@ 
  * @return void
  */
 function give_clear_errors() {
-	Give()->session->set( 'give_errors', null );
+	Give()->session->set('give_errors', null);
 }
 
 /**
@@ -82,19 +82,19 @@ 
  *
  * @return void
  */
-function give_unset_error( $error_id ) {
+function give_unset_error($error_id) {
 	$errors = give_get_errors();
-	if ( $errors ) {
+	if ($errors) {
 		/**
 		 * Check If $error_id exists in the array.
 		 * If exists then unset it.
 		 *
 		 * @since 1.8.13
 		 */
-		if ( isset( $errors[ $error_id ] ) ) {
-			unset( $errors[ $error_id ] );
+		if (isset($errors[$error_id])) {
+			unset($errors[$error_id]);
 		}
-		Give()->session->set( 'give_errors', $errors );
+		Give()->session->set('give_errors', $errors);
 	}
 }
 
@@ -105,7 +105,7 @@ 
  * @return string
  */
 function _give_die_handler() {
-	if ( defined( 'GIVE_UNIT_TESTS' ) ) {
+	if (defined('GIVE_UNIT_TESTS')) {
 		return '_give_die_handler';
 	} else {
 		die();
@@ -125,8 +125,8 @@ 
  *
  * @return void
  */
-function give_die( $message = '', $title = '', $status = 400 ) {
-	add_filter( 'wp_die_ajax_handler', '_give_die_handler', 10, 3 );
-	add_filter( 'wp_die_handler', '_give_die_handler', 10, 3 );
-	wp_die( $message, $title, array( 'response' => $status ) );
+function give_die($message = '', $title = '', $status = 400) {
+	add_filter('wp_die_ajax_handler', '_give_die_handler', 10, 3);
+	add_filter('wp_die_handler', '_give_die_handler', 10, 3);
+	wp_die($message, $title, array('response' => $status));
 }