WordImpress / Give

includes/gateways/paypal-standard.php

Doc Comments +2 added lines, -2 removed lines

@@ -565,7 +565,7 @@ 
  *
  * @since 1.6.3
  *
- * @param $pending_reason
+ * @param string $pending_reason
  *
  * @return string
  */
@@ -642,7 +642,7 @@ 
  * @param int   $payment_id   Payment ID
  * @param array $payment_data Array of payment data.
  *
- * @return mixed|string
+ * @return string
  */
 function give_build_paypal_url( $payment_id, $payment_data ) {
 	// Only send to PayPal if the pending payment is created successfully.

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/admin/settings/class-settings-license.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_License' ) ) :
+if ( ! class_exists('Give_Settings_License')) :
 
 	/**
 	 * Give_Settings_License.
@@ -43,10 +43,10 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'licenses';
-			$this->label = esc_html__( 'Licenses', 'give' );
+			$this->label = esc_html__('Licenses', 'give');
 
-			add_filter( 'give-settings_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-settings_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter('give-settings_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-settings_settings_{$this->id}_page", array($this, 'output'));
 		}
 
 		/**
@@ -56,11 +56,11 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
+		public function add_settings_page($pages) {
 			$setting = $this->get_settings();
 			// Bailout: Do not add licenses setting tab if it does not contain any setting fields.
-			if( ! empty( $setting ) ) {
-				$pages[ $this->id ] = $this->label;
+			if ( ! empty($setting)) {
+				$pages[$this->id] = $this->label;
 			}
 
 			return $pages;
@@ -79,7 +79,7 @@ 
 			 * Filter the licenses settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_licenses', $settings );
+			$settings = apply_filters('give_settings_licenses', $settings);
 
 			/**
 			 * Filter the settings.
@@ -87,7 +87,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -102,7 +102,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 	}
 

includes/admin/settings/class-settings-addon.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Addon' ) ) :
+if ( ! class_exists('Give_Settings_Addon')) :
 
 	/**
 	 * Give_Settings_Addon.
@@ -26,7 +26,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'addons';
-			$this->label = esc_html__( 'Add-ons', 'give' );
+			$this->label = esc_html__('Add-ons', 'give');
 
 			parent::__construct();
 		}
@@ -38,12 +38,12 @@ 
 		 * @param  $setting_tab
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			$default_tab = '';
 
 			// Set default tab to first setting tab.
-			if( $sections = array_keys( $this->get_sections() ) ) {
-				$default_tab = current( $sections );
+			if ($sections = array_keys($this->get_sections())) {
+				$default_tab = current($sections);
 			}
 			return $default_tab;
 		}
@@ -55,11 +55,11 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
+		public function add_settings_page($pages) {
 			$setting = $this->get_settings();
 			// Bailout: Do not add addons setting tab if it does not contain any setting fields.
-			if( ! empty( $setting ) ) {
-				$pages[ $this->id ] = $this->label;
+			if ( ! empty($setting)) {
+				$pages[$this->id] = $this->label;
 			}
 
 			return $pages;
@@ -78,7 +78,7 @@ 
 			 * Filter the addons settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_addons', $settings );
+			$settings = apply_filters('give_settings_addons', $settings);
 
 			/**
 			 * Filter the settings.
@@ -86,7 +86,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;

includes/admin/forms/metabox.php

Spacing +6 added lines, -6 removed lines

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

 

 // Exit if accessed directly.

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

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

 	exit;

 }

 

@@ -22,19 +22,19 @@ 
  */

 function give_add_shortcode_to_publish_metabox() {

 

-	if ( 'give_forms' !== get_post_type() ) {

+	if ('give_forms' !== get_post_type()) {

 		return false;

 	}

 	global $post;

 

 	//Only enqueue scripts for CPT on post type screen

-	if ( 'give_forms' === $post->post_type ) {

+	if ('give_forms' === $post->post_type) {

 		//Shortcode column with select all input

-		$shortcode = htmlentities( '[give_form id="' . $post->ID . '"]' );

-		echo '<div class="shortcode-wrap box-sizing"><label for="shortcode-input">' . esc_html__( 'Give Form Shortcode:', 'give' ) . '</label><input onClick="this.setSelectionRange(0, this.value.length)" type="text" name="shortcode-input" id="shortcode-input" class="shortcode-input" readonly value="' . $shortcode . '"></div>';

+		$shortcode = htmlentities('[give_form id="'.$post->ID.'"]');

+		echo '<div class="shortcode-wrap box-sizing"><label for="shortcode-input">'.esc_html__('Give Form Shortcode:', 'give').'</label><input onClick="this.setSelectionRange(0, this.value.length)" type="text" name="shortcode-input" id="shortcode-input" class="shortcode-input" readonly value="'.$shortcode.'"></div>';

 

 	}

 

 }

 

-add_action( 'post_submitbox_misc_actions', 'give_add_shortcode_to_publish_metabox' );

+add_action('post_submitbox_misc_actions', 'give_add_shortcode_to_publish_metabox');

includes/admin/admin-footer.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,14 +24,14 @@ 
  *
  * @return      string
  */
-function give_admin_rate_us( $footer_text ) {
+function give_admin_rate_us($footer_text) {
 	global $typenow;
 
-	if ( $typenow == 'give_forms' ) {
+	if ($typenow == 'give_forms') {
 		$rate_text = sprintf(
 			/* translators: %s: Link to 5 star rating */
-			__( 'If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give' ),
-			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="' . esc_attr__( 'Thanks :)', 'give' ) . '">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
+			__('If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give'),
+			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="'.esc_attr__('Thanks :)', 'give').'">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
 		);
 
 		return $rate_text;
@@ -40,4 +40,4 @@ 
 	}
 }
 
-add_filter( 'admin_footer_text', 'give_admin_rate_us' );
+add_filter('admin_footer_text', 'give_admin_rate_us');

includes/admin/admin-actions.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
  *
  * @since 1.8
  *
- * @return bool
+ * @return false|null
  */
 function give_redirect_to_clean_url_admin_pages() {
 	// Give admin pages.

Spacing +150 added lines, -151 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,19 +20,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -52,13 +52,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -68,21 +68,21 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');
 
 
 /**
@@ -96,25 +96,25 @@ 
  */
 function give_hide_outdated_php_notice() {
 
-	if ( ! isset( $_POST['_give_hide_outdated_php_notices_shortly'] ) ) {
+	if ( ! isset($_POST['_give_hide_outdated_php_notices_shortly'])) {
 		give_die();
 	}
 
 	// Transient key name.
 	$transient_key = "_give_hide_outdated_php_notices_shortly";
 
-	if ( Give_Cache::get( $transient_key, true ) ) {
+	if (Give_Cache::get($transient_key, true)) {
 		return;
 	}
 
 	// Hide notice for 24 hours.
-	Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+	Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 	give_die();
 
 }
 
-add_action( 'wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice' );
+add_action('wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice');
 
 /**
  * Register admin notices.
@@ -123,23 +123,23 @@ 
  */
 function _give_register_admin_notices() {
 	// Bailout.
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
 	// Add payment bulk notice.
 	if (
-		current_user_can( 'edit_give_payments' )
-		&& isset( $_GET['action'] )
-		&& ! empty( $_GET['action'] )
-		&& isset( $_GET['payment'] )
-		&& ! empty( $_GET['payment'] )
+		current_user_can('edit_give_payments')
+		&& isset($_GET['action'])
+		&& ! empty($_GET['action'])
+		&& isset($_GET['payment'])
+		&& ! empty($_GET['payment'])
 	) {
-		$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+		$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-		switch ( $_GET['action'] ) {
+		switch ($_GET['action']) {
 			case 'delete':
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'          => 'bulk_action_delete',
 					'type'        => 'updated',
 					'description' => sprintf(
@@ -151,12 +151,12 @@ 
 						),
 						$payment_count ),
 					'show'        => true,
-				) );
+				));
 
 				break;
 
 			case 'resend-receipt':
-				Give()->notices->register_notice( array(
+				Give()->notices->register_notice(array(
 					'id'          => 'bulk_action_resend_receipt',
 					'type'        => 'updated',
 					'description' => sprintf(
@@ -169,196 +169,196 @@ 
 						$payment_count
 					),
 					'show'        => true,
-				) );
+				));
 				break;
 		}
 	}
 
 	// Add give message notices.
-	if ( ! empty( $_GET['give-message'] ) ) {
+	if ( ! empty($_GET['give-message'])) {
 		// Donation reports errors.
-		if ( current_user_can( 'view_give_reports' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('view_give_reports')) {
+			switch ($_GET['give-message']) {
 				case 'donation_deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been deleted.', 'give' ),
+						'description' => __('The donation has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'email_sent' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-sent',
 						'type'        => 'updated',
-						'description' => __( 'The donation receipt has been resent.', 'give' ),
+						'description' => __('The donation receipt has been resent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'refreshed-reports' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-refreshed-reports',
 						'type'        => 'updated',
-						'description' => __( 'The reports cache has been cleared.', 'give' ),
+						'description' => __('The reports cache has been cleared.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'donation-note-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-note-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been deleted.', 'give' ),
+						'description' => __('The donation note has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Give settings notices and errors.
-		if ( current_user_can( 'manage_give_settings' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('manage_give_settings')) {
+			switch ($_GET['give-message']) {
 				case 'settings-imported' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-settings-imported',
 						'type'        => 'updated',
-						'description' => __( 'The settings have been imported.', 'give' ),
+						'description' => __('The settings have been imported.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-generated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-generated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been generated.', 'give' ),
+						'description' => __('API keys have been generated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-exists' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-exists',
 						'type'        => 'updated',
-						'description' => __( 'The specified user already has API keys.', 'give' ),
+						'description' => __('The specified user already has API keys.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-regenerated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-regenerated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been regenerated.', 'give' ),
+						'description' => __('API keys have been regenerated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-revoked' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-revoked',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been revoked.', 'give' ),
+						'description' => __('API keys have been revoked.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'sent-test-email' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-sent-test-email',
 						'type'        => 'updated',
-						'description' => __( 'The test email has been sent.', 'give' ),
+						'description' => __('The test email has been sent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'matched-success-failure-page':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-matched-success-failure-page',
 						'type'        => 'updated',
-						'description' => __( 'You cannot set the success and failed pages to the same page', 'give' ),
+						'description' => __('You cannot set the success and failed pages to the same page', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 		// Payments errors.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'note-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-note-added',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been added.', 'give' ),
+						'description' => __('The donation note has been added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'payment-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-updated',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been updated.', 'give' ),
+						'description' => __('The donation has been updated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Donor Notices.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'donor-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donor has been deleted.', 'give' ),
+						'description' => __('The donor has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-added',
 						'type'        => 'updated',
-						'description' => __( 'Donor email added.', 'give' ),
+						'description' => __('Donor email added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-removed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-removed',
 						'type'        => 'updated',
-						'description' => __( 'Donor email removed.', 'give' ),
+						'description' => __('Donor email removed.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-remove-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-remove-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to remove donor email.', 'give' ),
+						'description' => __('Failed to remove donor email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-updated',
 						'type'        => 'updated',
-						'description' => __( 'Primary email updated for donor.', 'give' ),
+						'description' => __('Primary email updated for donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to set primary email.', 'give' ),
+						'description' => __('Failed to set primary email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 }
 
-add_action( 'admin_notices', '_give_register_admin_notices', - 1 );
+add_action('admin_notices', '_give_register_admin_notices', - 1);
 
 
 /**
@@ -368,31 +368,30 @@ 
  *
  * @return bool
  */
-function _give_show_test_mode_notice_in_admin_bar( $wp_admin_bar ) {
-	$is_test_mode = ! empty( $_POST['test_mode'] ) ?
-		give_is_setting_enabled( $_POST['test_mode'] ) :
-		give_is_test_mode();
+function _give_show_test_mode_notice_in_admin_bar($wp_admin_bar) {
+	$is_test_mode = ! empty($_POST['test_mode']) ?
+		give_is_setting_enabled($_POST['test_mode']) : give_is_test_mode();
 
 	if (
-		! current_user_can( 'view_give_reports' ) ||
+		! current_user_can('view_give_reports') ||
 		! $is_test_mode
 	) {
 		return false;
 	}
 
 	// Add the main siteadmin menu item.
-	$wp_admin_bar->add_menu( array(
+	$wp_admin_bar->add_menu(array(
 		'id'     => 'give-test-notice',
-		'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+		'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 		'parent' => 'top-secondary',
-		'title'  => esc_html__( 'Give Test Mode Active', 'give' ),
-		'meta'   => array( 'class' => 'give-test-mode-active' ),
-	) );
+		'title'  => esc_html__('Give Test Mode Active', 'give'),
+		'meta'   => array('class' => 'give-test-mode-active'),
+	));
 
 	return true;
 }
 
-add_action( 'admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1 );
+add_action('admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1);
 
 /**
  * Add Link to Import page in from donation archive and donation single page
@@ -401,13 +400,13 @@ 
  */
 function give_import_page_link_callback() {
 	?>
-	<a href="<?php echo esc_url( give_import_page_url() ); ?>"
-	   class="page-import-action page-title-action"><?php esc_html_e( 'Import Donations', 'give' ); ?></a>
+	<a href="<?php echo esc_url(give_import_page_url()); ?>"
+	   class="page-import-action page-title-action"><?php esc_html_e('Import Donations', 'give'); ?></a>
 
 	<style type="text/css">
 		<?php
 		// Check if view donation single page only.
-		if ( ! empty( $_REQUEST['view'] ) && 'view-payment-details' === (string) give_clean( $_REQUEST['view'] ) && 'give-payment-history' === give_clean( $_REQUEST['page'] ) ) {
+		if ( ! empty($_REQUEST['view']) && 'view-payment-details' === (string) give_clean($_REQUEST['view']) && 'give-payment-history' === give_clean($_REQUEST['page'])) {
 			?>
 		.wrap #transaction-details-heading {
 			display: inline-block;
@@ -429,7 +428,7 @@ 
 }
 
 
-add_action( 'give_payments_page_top', 'give_import_page_link_callback', 11 );
+add_action('give_payments_page_top', 'give_import_page_link_callback', 11);
 
 /**
  * Load donation import ajax callback
@@ -441,9 +440,9 @@ 
  */
 function give_donation_import_callback() {
 	$import_setting = array();
-	$fields         = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
+	$fields         = isset($_POST['fields']) ? $_POST['fields'] : null;
 
-	parse_str( $fields );
+	parse_str($fields);
 
 	$import_setting['create_user'] = $create_user;
 	$import_setting['mode']        = $mode;
@@ -452,54 +451,54 @@ 
 	$import_setting['delete_csv']  = $delete_csv;
 
 	// Parent key id.
-	$main_key = maybe_unserialize( $main_key );
-
-	$current    = absint( $_REQUEST['current'] );
-	$total_ajax = absint( $_REQUEST['total_ajax'] );
-	$start      = absint( $_REQUEST['start'] );
-	$end        = absint( $_REQUEST['end'] );
-	$next       = absint( $_REQUEST['next'] );
-	$total      = absint( $_REQUEST['total'] );
-	$per_page   = absint( $_REQUEST['per_page'] );
-	if ( empty( $delimiter ) ) {
+	$main_key = maybe_unserialize($main_key);
+
+	$current    = absint($_REQUEST['current']);
+	$total_ajax = absint($_REQUEST['total_ajax']);
+	$start      = absint($_REQUEST['start']);
+	$end        = absint($_REQUEST['end']);
+	$next       = absint($_REQUEST['next']);
+	$total      = absint($_REQUEST['total']);
+	$per_page   = absint($_REQUEST['per_page']);
+	if (empty($delimiter)) {
 		$delimiter = ',';
 	}
 
 	// processing done here.
-	$raw_data = give_get_donation_data_from_csv( $csv, $start, $end, $delimiter );
-	$raw_key  = maybe_unserialize( $mapto );
+	$raw_data = give_get_donation_data_from_csv($csv, $start, $end, $delimiter);
+	$raw_key  = maybe_unserialize($mapto);
 
 	//Prevent normal emails
-	remove_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
-	remove_action( 'give_insert_user', 'give_new_user_notification', 10 );
-	remove_action( 'give_insert_payment', 'give_payment_save_page_data' );
+	remove_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
+	remove_action('give_insert_user', 'give_new_user_notification', 10);
+	remove_action('give_insert_payment', 'give_payment_save_page_data');
 
-	foreach ( $raw_data as $row_data ) {
-		give_save_import_donation_to_db( $raw_key, $row_data, $main_key, $import_setting );
+	foreach ($raw_data as $row_data) {
+		give_save_import_donation_to_db($raw_key, $row_data, $main_key, $import_setting);
 	}
 
 	// Check if function exists or not.
-	if ( function_exists( 'give_payment_save_page_data' ) ) {
-		add_action( 'give_insert_payment', 'give_payment_save_page_data' );
+	if (function_exists('give_payment_save_page_data')) {
+		add_action('give_insert_payment', 'give_payment_save_page_data');
 	}
-	add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
-	add_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
+	add_action('give_insert_user', 'give_new_user_notification', 10, 2);
+	add_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
 
-	if ( $next == false ) {
+	if ($next == false) {
 		$json_data = array(
 			'success' => true,
-			'message' => __( 'All donation uploaded successfully!', 'give' ),
+			'message' => __('All donation uploaded successfully!', 'give'),
 		);
 	} else {
 		$index_start = $start;
 		$index_end   = $end;
 		$last        = false;
 		$next        = true;
-		if ( $next ) {
+		if ($next) {
 			$index_start = $index_start + $per_page;
-			$index_end   = $per_page + ( $index_start - 1 );
+			$index_end   = $per_page + ($index_start - 1);
 		}
-		if ( $index_end >= $total ) {
+		if ($index_end >= $total) {
 			$index_end = $total;
 			$last      = true;
 		}
@@ -513,26 +512,26 @@ 
 		);
 	}
 
-	$url              = give_import_page_url( array(
+	$url = give_import_page_url(array(
 		'step'       => '4',
 		'csv'        => $csv,
 		'total'      => $total,
 		'delete_csv' => $import_setting['delete_csv'],
-		'success'    => ( isset( $json_data['success'] ) ? $json_data['success'] : '' ),
-	) );
+		'success'    => (isset($json_data['success']) ? $json_data['success'] : ''),
+	));
 	$json_data['url'] = $url;
 
-	$current ++;
+	$current++;
 	$json_data['current'] = $current;
 
-	$percentage              = ( 100 / ( $total_ajax + 1 ) ) * $current;
+	$percentage              = (100 / ($total_ajax + 1)) * $current;
 	$json_data['percentage'] = $percentage;
 
-	$json_data = apply_filters( 'give_import_ajax_responces', $json_data, $fields );
-	wp_die( json_encode( $json_data ) );
+	$json_data = apply_filters('give_import_ajax_responces', $json_data, $fields);
+	wp_die(json_encode($json_data));
 }
 
-add_action( 'wp_ajax_give_donation_import', 'give_donation_import_callback' );
+add_action('wp_ajax_give_donation_import', 'give_donation_import_callback');
 
 
 /**
@@ -545,7 +544,7 @@ 
 	$blank_slate->init();
 }
 
-add_action( 'current_screen', 'give_blank_slate' );
+add_action('current_screen', 'give_blank_slate');
 
 /**
  * Get Array of WP User Roles.
@@ -558,8 +557,8 @@ 
 	$user_roles = array();
 
 	// Loop through User Roles.
-	foreach ( get_editable_roles() as $role_name => $role_info ):
-		$user_roles[ $role_name ] = $role_info['name'];
+	foreach (get_editable_roles() as $role_name => $role_info):
+		$user_roles[$role_name] = $role_info['name'];
 	endforeach;
 
 	return $user_roles;

includes/class-give-logging.php

Doc Comments +1 added lines, -1 removed lines

@@ -463,7 +463,7 @@
 	 * @since  1.7
 	 * @access public
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function delete_cache() {
 		global $wpdb;

Spacing +80 added lines, -80 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
 	 */
 	public function __setup_hooks() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache' ) );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
 	}
 
 	/**
@@ -68,7 +68,7 @@ 
 		/* Logs post type */
 		$log_args = array(
 			'labels'              => array(
-				'name' => esc_html__( 'Logs', 'give' ),
+				'name' => esc_html__('Logs', 'give'),
 			),
 			'public'              => false,
 			'exclude_from_search' => true,
@@ -77,11 +77,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true,
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -95,9 +95,9 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array(
+		register_taxonomy('give_log_type', 'give_log', array(
 			'public' => false,
-		) );
+		));
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 			'api_request',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -132,8 +132,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -152,7 +152,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -160,7 +160,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -177,12 +177,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type,
-		) );
+		));
 	}
 
 	/**
@@ -196,7 +196,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -205,7 +205,7 @@ 
 			'log_type'     => false,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		/**
 		 * Fires before inserting log entry.
@@ -215,20 +215,20 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				give_update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -241,7 +241,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -257,7 +257,7 @@ 
 	 *
 	 * @return bool|null       True if successful, false otherwise.
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
 		/**
 		 * Fires before updating log entry.
@@ -267,7 +267,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -275,15 +275,15 @@ 
 			'post_parent' => 0,
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					give_update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
@@ -297,7 +297,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -312,20 +312,20 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false,
 			'date_query'     => null
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -336,28 +336,28 @@ 
 		}
 
 		// Retrieve logs based on specific timeframe
-		if ( !empty ( $query_args['date_query'] ) && is_array( $query_args['date_query'] ) ) {
-			if ( ! empty( $query_args['date_query']['start_date'] ) ) {
-				$query_args['date_query']['after'] =  array(
-					'year'  => date( 'Y', strtotime( $query_args['date_query']['start_date'] ) ),
-					'month' => date( 'm', strtotime( $query_args['date_query']['start_date'] ) ),
-					'day'   => date( 'd', strtotime( $query_args['date_query']['start_date'] ) )
+		if ( ! empty ($query_args['date_query']) && is_array($query_args['date_query'])) {
+			if ( ! empty($query_args['date_query']['start_date'])) {
+				$query_args['date_query']['after'] = array(
+					'year'  => date('Y', strtotime($query_args['date_query']['start_date'])),
+					'month' => date('m', strtotime($query_args['date_query']['start_date'])),
+					'day'   => date('d', strtotime($query_args['date_query']['start_date']))
 				);
 			}
 
-			if ( ! empty( $query_args['date_query']['end_date'] ) ) {
-				$query_args['date_query']['before'] =  array(
-					'year'  => date( 'Y', strtotime( $query_args['date_query']['end_date'] ) ),
-					'month' => date( 'm', strtotime( $query_args['date_query']['end_date'] ) ),
-					'day'   => date( 'd', strtotime( $query_args['date_query']['end_date'] ) )
+			if ( ! empty($query_args['date_query']['end_date'])) {
+				$query_args['date_query']['before'] = array(
+					'year'  => date('Y', strtotime($query_args['date_query']['end_date'])),
+					'month' => date('m', strtotime($query_args['date_query']['end_date'])),
+					'day'   => date('d', strtotime($query_args['date_query']['end_date']))
 				);
 			}
 			$query_args['date_query']['inclusive'] = true;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -380,20 +380,20 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( $object_id ) {
+		if ($object_id) {
 			$query_args['post_parent'] = $object_id;
 		}
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -403,24 +403,24 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
 		// Get cache key for current query.
-		$cache_key = Give_Cache::get_key( 'get_log_count', $query_args );
+		$cache_key = Give_Cache::get_key('get_log_count', $query_args);
 
 		// check if cache already exist or not.
-		if ( ! ( $logs_count = Give_Cache::get( $cache_key ) ) ) {
-			$logs       = new WP_Query( $query_args );
+		if ( ! ($logs_count = Give_Cache::get($cache_key))) {
+			$logs       = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 
 			// Cache results.
-			Give_Cache::set( $cache_key, $logs_count );
+			Give_Cache::set($cache_key, $logs_count);
 		}
 
 		return $logs_count;
@@ -440,16 +440,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -459,15 +459,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -480,9 +480,9 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
+	public function background_process_delete_cache($post_id) {
 		// Delete log cache immediately
-		wp_schedule_single_event( time() - 5, 'give_delete_log_cache' );
+		wp_schedule_single_event(time() - 5, 'give_delete_log_cache');
 	}
 
 	/**
@@ -510,11 +510,11 @@ 
 		);
 
 		// Bailout.
-		if ( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
-		Give_Cache::delete( $cache_option_names );
+		Give_Cache::delete($cache_option_names);
 	}
 }
 
@@ -536,10 +536,10 @@ 
  *
  * @return int             ID of the new log entry.
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	/* @var Give_Logging $give_logs */
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }

includes/emails/class-give-email-tags.php

Doc Comments +1 added lines, -1 removed lines

@@ -530,7 +530,7 @@
  *
  * @param int $payment_id
  *
- * @return int payment_id
+ * @return string payment_id
  */
 function give_email_tag_payment_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );

Spacing +113 added lines, -113 removed lines

@@ -23,7 +23,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,9 +55,9 @@ 
 	 * @param string   $description Email tag description text
 	 * @param callable $func        Hook to run when email tag is found
 	 */
-	public function add( $tag, $description, $func ) {
-		if ( is_callable( $func ) ) {
-			$this->tags[ $tag ] = array(
+	public function add($tag, $description, $func) {
+		if (is_callable($func)) {
+			$this->tags[$tag] = array(
 				'tag'         => $tag,
 				'description' => $description,
 				'func'        => $func,
@@ -72,8 +72,8 @@ 
 	 *
 	 * @param string $tag Email tag to remove hook from
 	 */
-	public function remove( $tag ) {
-		unset( $this->tags[ $tag ] );
+	public function remove($tag) {
+		unset($this->tags[$tag]);
 	}
 
 	/**
@@ -85,8 +85,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function email_tag_exists( $tag ) {
-		return array_key_exists( $tag, $this->tags );
+	public function email_tag_exists($tag) {
+		return array_key_exists($tag, $this->tags);
 	}
 
 	/**
@@ -110,16 +110,16 @@ 
 	 *
 	 * @return string Content with email tags filtered out.
 	 */
-	public function do_tags( $content, $payment_id ) {
+	public function do_tags($content, $payment_id) {
 
 		// Check if there is at least one tag added.
-		if ( empty( $this->tags ) || ! is_array( $this->tags ) ) {
+		if (empty($this->tags) || ! is_array($this->tags)) {
 			return $content;
 		}
 
 		$this->payment_id = $payment_id;
 
-		$new_content = preg_replace_callback( "/{([A-z0-9\-\_]+)}/s", array( $this, 'do_tag' ), $content );
+		$new_content = preg_replace_callback("/{([A-z0-9\-\_]+)}/s", array($this, 'do_tag'), $content);
 
 		$this->payment_id = null;
 
@@ -135,17 +135,17 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function do_tag( $m ) {
+	public function do_tag($m) {
 
 		// Get tag
 		$tag = $m[1];
 
 		// Return tag if tag not set
-		if ( ! $this->email_tag_exists( $tag ) ) {
+		if ( ! $this->email_tag_exists($tag)) {
 			return $m[0];
 		}
 
-		return call_user_func( $this->tags[ $tag ]['func'], $this->payment_id, $tag );
+		return call_user_func($this->tags[$tag]['func'], $this->payment_id, $tag);
 	}
 
 }
@@ -159,8 +159,8 @@ 
  * @param string   $description Description of the email tag added
  * @param callable $func        Hook to run when email tag is found
  */
-function give_add_email_tag( $tag, $description, $func ) {
-	Give()->email_tags->add( $tag, $description, $func );
+function give_add_email_tag($tag, $description, $func) {
+	Give()->email_tags->add($tag, $description, $func);
 }
 
 /**
@@ -170,8 +170,8 @@ 
  *
  * @param string $tag Email tag to remove hook from
  */
-function give_remove_email_tag( $tag ) {
-	Give()->email_tags->remove( $tag );
+function give_remove_email_tag($tag) {
+	Give()->email_tags->remove($tag);
 }
 
 /**
@@ -183,8 +183,8 @@ 
  *
  * @return bool
  */
-function give_email_tag_exists( $tag ) {
-	return Give()->email_tags->email_tag_exists( $tag );
+function give_email_tag_exists($tag) {
+	return Give()->email_tags->email_tag_exists($tag);
 }
 
 /**
@@ -211,9 +211,9 @@ 
 	$email_tags = give_get_email_tags();
 
 	ob_start();
-	if ( count( $email_tags ) > 0 ) : ?>
+	if (count($email_tags) > 0) : ?>
 		<div class="give-email-tags-wrap">
-			<?php foreach ( $email_tags as $email_tag ) : ?>
+			<?php foreach ($email_tags as $email_tag) : ?>
 				<span class="give_<?php echo $email_tag['tag']; ?>_tag">
 					<code>{<?php echo $email_tag['tag']; ?>}</code> - <?php echo $email_tag['description']; ?>
 				</span>
@@ -235,12 +235,12 @@ 
  *
  * @return string Content with email tags filtered out.
  */
-function give_do_email_tags( $content, $payment_id ) {
+function give_do_email_tags($content, $payment_id) {
 
 	// Replace all tags
-	$content = Give()->email_tags->do_tags( $content, $payment_id );
+	$content = Give()->email_tags->do_tags($content, $payment_id);
 
-	$content = apply_filters( 'give_email_template_tags', $content, give_get_payment_meta( $payment_id ), $payment_id );
+	$content = apply_filters('give_email_template_tags', $content, give_get_payment_meta($payment_id), $payment_id);
 
 	// Return content
 	return $content;
@@ -259,10 +259,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_add_email_tags' );
+	do_action('give_add_email_tags');
 }
 
-add_action( 'init', 'give_load_email_tags', - 999 );
+add_action('init', 'give_load_email_tags', - 999);
 
 
 /**
@@ -276,97 +276,97 @@ 
 	$email_tags = array(
 		array(
 			'tag'         => 'donation',
-			'description' => __( 'The donation form name, and the donation level (if applicable).', 'give' ),
+			'description' => __('The donation form name, and the donation level (if applicable).', 'give'),
 			'function'    => 'give_email_tag_donation',
 		),
 		array(
 			'tag'         => 'form_title',
-			'description' => __( 'The donation form name.', 'give' ),
+			'description' => __('The donation form name.', 'give'),
 			'function'    => 'give_email_tag_form_title',
 		),
 		array(
 			'tag'         => 'amount',
-			'description' => __( 'The total donation amount with currency sign.', 'give' ),
+			'description' => __('The total donation amount with currency sign.', 'give'),
 			'function'    => 'give_email_tag_amount',
 		),
 		array(
 			'tag'         => 'price',
-			'description' => __( 'The total donation amount with currency sign.', 'give' ),
+			'description' => __('The total donation amount with currency sign.', 'give'),
 			'function'    => 'give_email_tag_price',
 		),
 		array(
 			'tag'         => 'name',
-			'description' => __( 'The donor\'s first name.', 'give' ),
+			'description' => __('The donor\'s first name.', 'give'),
 			'function'    => 'give_email_tag_first_name',
 		),
 		array(
 			'tag'         => 'fullname',
-			'description' => __( 'The donor\'s full name, first and last.', 'give' ),
+			'description' => __('The donor\'s full name, first and last.', 'give'),
 			'function'    => 'give_email_tag_fullname',
 		),
 		array(
 			'tag'         => 'username',
-			'description' => __( 'The donor\'s user name on the site, if they registered an account.', 'give' ),
+			'description' => __('The donor\'s user name on the site, if they registered an account.', 'give'),
 			'function'    => 'give_email_tag_username',
 		),
 		array(
 			'tag'         => 'user_email',
-			'description' => __( 'The donor\'s email address.', 'give' ),
+			'description' => __('The donor\'s email address.', 'give'),
 			'function'    => 'give_email_tag_user_email',
 		),
 		array(
 			'tag'         => 'billing_address',
-			'description' => __( 'The donor\'s billing address.', 'give' ),
+			'description' => __('The donor\'s billing address.', 'give'),
 			'function'    => 'give_email_tag_billing_address',
 		),
 		array(
 			'tag'         => 'date',
-			'description' => __( 'The date of the donation.', 'give' ),
+			'description' => __('The date of the donation.', 'give'),
 			'function'    => 'give_email_tag_date',
 		),
 		array(
 			'tag'         => 'payment_id',
-			'description' => __( 'The unique ID number for this donation.', 'give' ),
+			'description' => __('The unique ID number for this donation.', 'give'),
 			'function'    => 'give_email_tag_payment_id',
 		),
 		array(
 			'tag'         => 'receipt_id',
-			'description' => __( 'The unique ID number for this donation receipt.', 'give' ),
+			'description' => __('The unique ID number for this donation receipt.', 'give'),
 			'function'    => 'give_email_tag_receipt_id',
 		),
 		array(
 			'tag'         => 'payment_method',
-			'description' => __( 'The method of payment used for this donation.', 'give' ),
+			'description' => __('The method of payment used for this donation.', 'give'),
 			'function'    => 'give_email_tag_payment_method',
 		),
 		array(
 			'tag'         => 'sitename',
-			'description' => __( 'The name of your site.', 'give' ),
+			'description' => __('The name of your site.', 'give'),
 			'function'    => 'give_email_tag_sitename',
 		),
 		array(
 			'tag'         => 'receipt_link',
-			'description' => __( 'The donation receipt direct link, to view the receipt on the website.', 'give' ),
+			'description' => __('The donation receipt direct link, to view the receipt on the website.', 'give'),
 			'function'    => 'give_email_tag_receipt_link',
 		),
 		array(
 			'tag'         => 'receipt_link_url',
-			'description' => __( 'The donation receipt direct URL, to view the receipt on the website.', 'give' ),
+			'description' => __('The donation receipt direct URL, to view the receipt on the website.', 'give'),
 			'function'    => 'give_email_tag_receipt_link_url',
 		),
 	);
 
 	// Apply give_email_tags filter
-	$email_tags = apply_filters( 'give_email_tags', $email_tags );
+	$email_tags = apply_filters('give_email_tags', $email_tags);
 
 	// Add email tags
-	foreach ( $email_tags as $email_tag ) {
-		give_add_email_tag( $email_tag['tag'], $email_tag['description'], $email_tag['function'] );
+	foreach ($email_tags as $email_tag) {
+		give_add_email_tag($email_tag['tag'], $email_tag['description'], $email_tag['function']);
 	}
 
 }
 
-add_action( 'give_add_email_tags', 'give_setup_email_tags' );
+add_action('give_add_email_tags', 'give_setup_email_tags');
 
 
 /**
@@ -378,15 +378,15 @@ 
  *
  * @return string name
  */
-function give_email_tag_first_name( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_first_name($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['name'];
 }
@@ -400,15 +400,15 @@ 
  *
  * @return string fullname
  */
-function give_email_tag_fullname( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_fullname($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['fullname'];
 }
@@ -422,15 +422,15 @@ 
  *
  * @return string username.
  */
-function give_email_tag_username( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_username($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['username'];
 }
@@ -444,8 +444,8 @@ 
  *
  * @return string user_email
  */
-function give_email_tag_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -459,9 +459,9 @@ 
  *
  * @return string billing_address
  */
-function give_email_tag_billing_address( $payment_id ) {
-	$user_info    = give_get_payment_meta_user_info( $payment_id );
-	$user_address = ! empty( $user_info['address'] ) ? $user_info['address'] : array(
+function give_email_tag_billing_address($payment_id) {
+	$user_info    = give_get_payment_meta_user_info($payment_id);
+	$user_address = ! empty($user_info['address']) ? $user_info['address'] : array(
 		'line1'   => '',
 		'line2'   => '',
 		'city'    => '',
@@ -471,24 +471,24 @@ 
 	);
 
 	$return = $user_address['line1'];
-	if ( ! empty( $user_address['line2'] ) ) {
-		$return .= ', ' . $user_address['line2'];
+	if ( ! empty($user_address['line2'])) {
+		$return .= ', '.$user_address['line2'];
 	}
-	if ( ! empty( $user_address['city'] ) ) {
-		$return .= "\n" . $user_address['city'];
+	if ( ! empty($user_address['city'])) {
+		$return .= "\n".$user_address['city'];
 	}
-	if ( ! empty( $user_address['state'] ) ) {
-		$return .= ', ' . $user_address['state'];
+	if ( ! empty($user_address['state'])) {
+		$return .= ', '.$user_address['state'];
 	}
-	if ( ! empty( $user_address['zip'] ) ) {
-		$return .= ' ' . $user_address['zip'];
+	if ( ! empty($user_address['zip'])) {
+		$return .= ' '.$user_address['zip'];
 	}
-	if ( ! empty( $user_address['country'] ) ) {
-		$return .= "\n" . give_get_country_name_by_key( $user_address['country'] );
+	if ( ! empty($user_address['country'])) {
+		$return .= "\n".give_get_country_name_by_key($user_address['country']);
 	}
 
-	if ( empty( $return ) ) {
-		$return = __( 'None provided', 'give' );
+	if (empty($return)) {
+		$return = __('None provided', 'give');
 	}
 
 	return apply_filters('give_email_tag_billing_address', $return, $payment_id);
@@ -504,10 +504,10 @@ 
  *
  * @return string date
  */
-function give_email_tag_date( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_date($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return date_i18n( give_date_format(), strtotime( $payment->date ) );
+	return date_i18n(give_date_format(), strtotime($payment->date));
 }
 
 /**
@@ -519,11 +519,11 @@ 
  *
  * @return string amount
  */
-function give_email_tag_amount( $payment_id ) {
-	$payment     = new Give_Payment( $payment_id );
-	$give_amount = give_currency_filter( give_format_amount( $payment->total, array( 'sanitize' => false ) ), $payment->currency );
+function give_email_tag_amount($payment_id) {
+	$payment     = new Give_Payment($payment_id);
+	$give_amount = give_currency_filter(give_format_amount($payment->total, array('sanitize' => false)), $payment->currency);
 
-	return html_entity_decode( $give_amount, ENT_COMPAT, 'UTF-8' );
+	return html_entity_decode($give_amount, ENT_COMPAT, 'UTF-8');
 }
 
 /**
@@ -535,8 +535,8 @@ 
  *
  * @return string price
  */
-function give_email_tag_price( $payment_id ) {
-	return give_email_tag_amount( $payment_id );
+function give_email_tag_price($payment_id) {
+	return give_email_tag_amount($payment_id);
 }
 
 /**
@@ -548,8 +548,8 @@ 
  *
  * @return int payment_id
  */
-function give_email_tag_payment_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -563,8 +563,8 @@ 
  *
  * @return string receipt_id
  */
-function give_email_tag_receipt_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_receipt_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -578,14 +578,14 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_donation( $payment_id ) {
-	$payment      = new Give_Payment( $payment_id );
+function give_email_tag_donation($payment_id) {
+	$payment      = new Give_Payment($payment_id);
 	$payment_meta = $payment->payment_meta;
-	$level_title  = give_has_variable_prices( $payment->form_id );
+	$level_title  = give_has_variable_prices($payment->form_id);
 	$separator    = $level_title ? '-' : '';
-	$form_title   = strip_tags( give_get_payment_form_title( $payment_meta, false, $separator ) );
+	$form_title   = strip_tags(give_get_payment_form_title($payment_meta, false, $separator));
 
-	return ! empty( $form_title ) ? $form_title : '';
+	return ! empty($form_title) ? $form_title : '';
 
 }
 
@@ -598,11 +598,11 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_form_title( $payment_id ) {
-	$payment      = new Give_Payment( $payment_id );
+function give_email_tag_form_title($payment_id) {
+	$payment      = new Give_Payment($payment_id);
 	$payment_meta = $payment->payment_meta;
 
-	return isset( $payment_meta['form_title'] ) ? strip_tags( $payment_meta['form_title'] ) : '';
+	return isset($payment_meta['form_title']) ? strip_tags($payment_meta['form_title']) : '';
 
 }
 
@@ -615,10 +615,10 @@ 
  *
  * @return string gateway
  */
-function give_email_tag_payment_method( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_method($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return give_get_gateway_checkout_label( $payment->gateway );
+	return give_get_gateway_checkout_label($payment->gateway);
 }
 
 /**
@@ -630,8 +630,8 @@ 
  *
  * @return string sitename
  */
-function give_email_tag_sitename( $payment_id ) {
-	return wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES );
+function give_email_tag_sitename($payment_id) {
+	return wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES);
 }
 
 /**
@@ -643,19 +643,19 @@ 
  *
  * @return string receipt_link
  */
-function give_email_tag_receipt_link( $payment_id ) {
+function give_email_tag_receipt_link($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt',
-	), home_url() ) );
-	$formatted   = sprintf(
+	), home_url()));
+	$formatted = sprintf(
 		'<a href="%1$s">%2$s</a>',
 		$receipt_url,
-		esc_html__( 'View it in your browser &raquo;', 'give' )
+		esc_html__('View it in your browser &raquo;', 'give')
 	);
 
-	if ( give_get_option( 'email_template' ) !== 'none' ) {
+	if (give_get_option('email_template') !== 'none') {
 		return $formatted;
 	} else {
 		return $receipt_url;
@@ -674,12 +674,12 @@ 
  *
  * @return string receipt_url
  */
-function give_email_tag_receipt_link_url( $payment_id ) {
+function give_email_tag_receipt_link_url($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt',
-	), home_url() ) );
+	), home_url()));
 
 	return $receipt_url;
 

includes/admin/settings/class-settings-email.php

Indentation +21 added lines, -21 removed lines

@@ -79,13 +79,13 @@ 
 							'default' => get_bloginfo( 'admin_email' ),
 							'type'    => 'text'
 						),
-                        array(
-                            'name'  => esc_html__( 'Email Settings Docs Link', 'give' ),
-                            'id'    => 'email_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-emails' ),
-                            'title' => __( 'Email Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Email Settings Docs Link', 'give' ),
+							'id'    => 'email_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-emails' ),
+							'title' => __( 'Email Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_email_settings_1',
 							'type' => 'sectionend'
@@ -118,13 +118,13 @@ 
 							'type'    => 'wysiwyg',
 							'default' => give_get_default_donation_receipt_email()
 						),
-                        array(
-                            'name'  => esc_html__( 'Donation Receipt Settings Docs Link', 'give' ),
-                            'id'    => 'donation_receipt_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-receipt' ),
-                            'title' => __( 'Donation Receipt Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Donation Receipt Settings Docs Link', 'give' ),
+							'id'    => 'donation_receipt_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-donation-receipt' ),
+							'title' => __( 'Donation Receipt Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_email_settings_2',
 							'type' => 'sectionend'
@@ -175,13 +175,13 @@ 
 							'type'    => 'textarea',
 							'default' => get_bloginfo( 'admin_email' )
 						),
-                        array(
-                            'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
-                            'id'    => 'donation_notification_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
-                            'title' => __( 'Donation Notification Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
+							'id'    => 'donation_notification_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
+							'title' => __( 'Donation Notification Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_email_settings_3',
 							'type' => 'sectionend'

Spacing +46 added lines, -46 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Email' ) ) :
+if ( ! class_exists('Give_Settings_Email')) :
 
 	/**
 	 * Give_Settings_Email.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'emails';
-			$this->label = esc_html__( 'Emails', 'give' );
+			$this->label = esc_html__('Emails', 'give');
 
 			$this->default_tab = 'email-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'email-settings' :
 					$settings = array(
 						// Section 1: Email
@@ -54,36 +54,36 @@ 
 						),
 						array(
 							'id'      => 'email_template',
-							'name'    => esc_html__( 'Email Template', 'give' ),
-							'desc'    => esc_html__( 'Choose your template from the available registered template types.', 'give' ),
+							'name'    => esc_html__('Email Template', 'give'),
+							'desc'    => esc_html__('Choose your template from the available registered template types.', 'give'),
 							'type'    => 'select',
 							'options' => give_get_email_templates()
 						),
 						array(
 							'id'   => 'email_logo',
-							'name' => esc_html__( 'Logo', 'give' ),
-							'desc' => esc_html__( 'Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give' ),
+							'name' => esc_html__('Logo', 'give'),
+							'desc' => esc_html__('Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give'),
 							'type' => 'file'
 						),
 						array(
 							'id'      => 'from_name',
-							'name'    => esc_html__( 'From Name', 'give' ),
-							'desc'    => esc_html__( 'The name which appears in the "From" field in all Give donation emails.', 'give' ),
-							'default' => get_bloginfo( 'name' ),
+							'name'    => esc_html__('From Name', 'give'),
+							'desc'    => esc_html__('The name which appears in the "From" field in all Give donation emails.', 'give'),
+							'default' => get_bloginfo('name'),
 							'type'    => 'text'
 						),
 						array(
 							'id'      => 'from_email',
-							'name'    => esc_html__( 'From Email', 'give' ),
-							'desc'    => esc_html__( 'Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give' ),
-							'default' => get_bloginfo( 'admin_email' ),
+							'name'    => esc_html__('From Email', 'give'),
+							'desc'    => esc_html__('Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give'),
+							'default' => get_bloginfo('admin_email'),
 							'type'    => 'text'
 						),
                         array(
-                            'name'  => esc_html__( 'Email Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Email Settings Docs Link', 'give'),
                             'id'    => 'email_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-emails' ),
-                            'title' => __( 'Email Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-emails'),
+                            'title' => __('Email Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -102,27 +102,27 @@ 
 						),
 						array(
 							'id'      => 'donation_subject',
-							'name'    => esc_html__( 'Donation Email Subject', 'give' ),
-							'desc'    => esc_html__( 'Enter the subject line for the donation receipt email.', 'give' ),
-							'default' => esc_attr__( 'Donation Receipt', 'give' ),
+							'name'    => esc_html__('Donation Email Subject', 'give'),
+							'desc'    => esc_html__('Enter the subject line for the donation receipt email.', 'give'),
+							'default' => esc_attr__('Donation Receipt', 'give'),
 							'type'    => 'text'
 						),
 						array(
 							'id'      => 'donation_receipt',
-							'name'    => esc_html__( 'Donation Receipt', 'give' ),
+							'name'    => esc_html__('Donation Receipt', 'give'),
 							'desc'    => sprintf(
 							/* translators: %s: emails tags list */
-								__( 'Enter the email that is sent to users after completing a successful donation. HTML is accepted.<br /><strong>Available template tags:</strong> %s', 'give' ),
+								__('Enter the email that is sent to users after completing a successful donation. HTML is accepted.<br /><strong>Available template tags:</strong> %s', 'give'),
 								'<br/>'.give_get_emails_tags_list()
 							),
 							'type'    => 'wysiwyg',
 							'default' => give_get_default_donation_receipt_email()
 						),
                         array(
-                            'name'  => esc_html__( 'Donation Receipt Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Donation Receipt Settings Docs Link', 'give'),
                             'id'    => 'donation_receipt_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-receipt' ),
-                            'title' => __( 'Donation Receipt Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-donation-receipt'),
+                            'title' => __('Donation Receipt Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -141,28 +141,28 @@ 
 						),
 						array(
 							'id'      => 'admin_notices',
-							'name'    => esc_html__( 'Admin Notifications', 'give' ),
-							'desc'    => esc_html__( 'Enable/Disable all admin notifications from Give completely.', 'give' ),
+							'name'    => esc_html__('Admin Notifications', 'give'),
+							'desc'    => esc_html__('Enable/Disable all admin notifications from Give completely.', 'give'),
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
 							'id'      => 'donation_notification_subject',
-							'name'    => esc_html__( 'Donation Notification Subject', 'give' ),
-							'desc'    => esc_html__( 'Enter the subject line for the admin donation notification email.', 'give' ),
+							'name'    => esc_html__('Donation Notification Subject', 'give'),
+							'desc'    => esc_html__('Enter the subject line for the admin donation notification email.', 'give'),
 							'type'    => 'text',
-							'default' => esc_attr__( 'New Donation - #{payment_id}', 'give' )
+							'default' => esc_attr__('New Donation - #{payment_id}', 'give')
 						),
 						array(
 							'id'      => 'donation_notification',
-							'name'    => esc_html__( 'Donation Notification', 'give' ),
+							'name'    => esc_html__('Donation Notification', 'give'),
 							'desc'    => sprintf(
 							/* translators: %s: emails tags list */
-								__( 'Enter the content of the email that is sent to notify an admin of a new donation. HTML is accepted. <br /><strong>Available template tags:</strong> %s', 'give' ),
+								__('Enter the content of the email that is sent to notify an admin of a new donation. HTML is accepted. <br /><strong>Available template tags:</strong> %s', 'give'),
 								'<br/>'.give_get_emails_tags_list()
 							),
 							'type'    => 'wysiwyg',
@@ -170,16 +170,16 @@ 
 						),
 						array(
 							'id'      => 'admin_notice_emails',
-							'name'    => esc_html__( 'Donation Notification Emails', 'give' ),
-							'desc'    => __( 'Enter the email address(es) that should receive a notification anytime a donation is made, please only enter <span class="give-underline">one email address per line</span> and <strong>not separated by commas</strong>.', 'give' ),
+							'name'    => esc_html__('Donation Notification Emails', 'give'),
+							'desc'    => __('Enter the email address(es) that should receive a notification anytime a donation is made, please only enter <span class="give-underline">one email address per line</span> and <strong>not separated by commas</strong>.', 'give'),
 							'type'    => 'textarea',
-							'default' => get_bloginfo( 'admin_email' )
+							'default' => get_bloginfo('admin_email')
 						),
                         array(
-                            'name'  => esc_html__( 'Donation Notification Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Donation Notification Settings Docs Link', 'give'),
                             'id'    => 'donation_notification_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-donation-notification' ),
-                            'title' => __( 'Donation Notification Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-donation-notification'),
+                            'title' => __('Donation Notification Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -194,7 +194,7 @@ 
 			 * Filter the emails settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_emails', $settings );
+			$settings = apply_filters('give_settings_emails', $settings);
 
 			/**
 			 * Filter the settings.
@@ -202,7 +202,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -216,12 +216,12 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'email-settings'            => esc_html__( 'Email Settings', 'give' ),
-				'donation-receipt'          => esc_html__( 'Donation Receipt', 'give' ),
-				'new-donation-notification' => esc_html__( 'New Donation Notification', 'give' )
+				'email-settings'            => esc_html__('Email Settings', 'give'),
+				'donation-receipt'          => esc_html__('Donation Receipt', 'give'),
+				'new-donation-notification' => esc_html__('New Donation Notification', 'give')
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}