WordImpress / Give

includes/forms/functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -155,7 +155,7 @@ 
  *
  * Used to redirect a user back to the donation form if there are errors present.
  *
- * @param array $args
+ * @param string $args
  *
  * @access public
  * @since  1.0
@@ -437,7 +437,7 @@ 
  *
  * @param int $form_id Give Form ID
  *
- * @return int $earnings Earnings for a certain form
+ * @return double $earnings Earnings for a certain form
  */
 function give_get_form_earnings_stats( $form_id = 0 ) {
 	$give_form = new Give_Donate_Form( $form_id );

Spacing +213 added lines, -213 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,14 +23,14 @@ 
 
 	global $typenow;
 
-	if ( $typenow != 'give_forms' ) {
+	if ($typenow != 'give_forms') {
 		return true;
 	}
 
 	return false;
 }
 
-add_filter( 'give_shortcode_button_condition', 'give_shortcode_button_condition' );
+add_filter('give_shortcode_button_condition', 'give_shortcode_button_condition');
 
 
 /**
@@ -40,11 +40,11 @@ 
  *
  * @return int|false
  */
-function get_form_id_from_args( $args ) {
+function get_form_id_from_args($args) {
 
-	if ( isset( $args['form_id'] ) && $args['form_id'] != 0 ) {
+	if (isset($args['form_id']) && $args['form_id'] != 0) {
 
-		return intval( $args['form_id'] );
+		return intval($args['form_id']);
 	}
 
 	return false;
@@ -59,23 +59,23 @@ 
  *
  * @return bool
  */
-function give_is_float_labels_enabled( $args ) {
+function give_is_float_labels_enabled($args) {
 
 	$float_labels = '';
 
-	if ( ! empty( $args['float_labels'] ) ) {
+	if ( ! empty($args['float_labels'])) {
 		$float_labels = $args['float_labels'];
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = give_get_meta( $args['form_id'], '_give_form_floating_labels', true );
+	if (empty($float_labels)) {
+		$float_labels = give_get_meta($args['form_id'], '_give_form_floating_labels', true);
 	}
 
-	if ( empty( $float_labels ) || ( 'global' === $float_labels ) ) {
-		$float_labels = give_get_option( 'floatlabels', 'disabled' );
+	if (empty($float_labels) || ('global' === $float_labels)) {
+		$float_labels = give_get_option('floatlabels', 'disabled');
 	}
 
-	return give_is_setting_enabled( $float_labels );
+	return give_is_setting_enabled($float_labels);
 }
 
 /**
@@ -91,7 +91,7 @@ 
 
 	$can_checkout = true;
 
-	return (bool) apply_filters( 'give_can_checkout', $can_checkout );
+	return (bool) apply_filters('give_can_checkout', $can_checkout);
 }
 
 /**
@@ -105,9 +105,9 @@ 
 function give_get_success_page_uri() {
 	$give_options = give_get_settings();
 
-	$success_page = isset( $give_options['success_page'] ) ? get_permalink( absint( $give_options['success_page'] ) ) : get_bloginfo( 'url' );
+	$success_page = isset($give_options['success_page']) ? get_permalink(absint($give_options['success_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_success_page_uri', $success_page );
+	return apply_filters('give_get_success_page_uri', $success_page);
 }
 
 /**
@@ -119,9 +119,9 @@ 
  */
 function give_is_success_page() {
 	$give_options    = give_get_settings();
-	$is_success_page = isset( $give_options['success_page'] ) ? is_page( $give_options['success_page'] ) : false;
+	$is_success_page = isset($give_options['success_page']) ? is_page($give_options['success_page']) : false;
 
-	return apply_filters( 'give_is_success_page', $is_success_page );
+	return apply_filters('give_is_success_page', $is_success_page);
 }
 
 /**
@@ -135,17 +135,17 @@ 
  * @since       1.0
  * @return      void
  */
-function give_send_to_success_page( $query_string = null ) {
+function give_send_to_success_page($query_string = null) {
 
 	$redirect = give_get_success_page_uri();
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$redirect .= $query_string;
 	}
 
-	$gateway = isset( $_REQUEST['give-gateway'] ) ? $_REQUEST['give-gateway'] : '';
+	$gateway = isset($_REQUEST['give-gateway']) ? $_REQUEST['give-gateway'] : '';
 
-	wp_redirect( apply_filters( 'give_success_page_redirect', $redirect, $gateway, $query_string ) );
+	wp_redirect(apply_filters('give_success_page_redirect', $redirect, $gateway, $query_string));
 	give_die();
 }
 
@@ -161,19 +161,19 @@ 
  * @since  1.0
  * @return Void
  */
-function give_send_back_to_checkout( $args = array() ) {
+function give_send_back_to_checkout($args = array()) {
 
-	$url     = isset( $_POST['give-current-url'] ) ? sanitize_text_field( $_POST['give-current-url'] ) : '';
+	$url     = isset($_POST['give-current-url']) ? sanitize_text_field($_POST['give-current-url']) : '';
 	$form_id = 0;
 
 	// Set the form_id.
-	if ( isset( $_POST['give-form-id'] ) ) {
-		$form_id = sanitize_text_field( $_POST['give-form-id'] );
+	if (isset($_POST['give-form-id'])) {
+		$form_id = sanitize_text_field($_POST['give-form-id']);
 	}
 
 	// Need a URL to continue. If none, redirect back to single form.
-	if ( empty( $url ) ) {
-		wp_safe_redirect( get_permalink( $form_id ) );
+	if (empty($url)) {
+		wp_safe_redirect(get_permalink($form_id));
 		give_die();
 	}
 
@@ -182,41 +182,41 @@ 
 	);
 
 	// Set the $level_id.
-	if ( isset( $_POST['give-price-id'] ) ) {
-		$defaults['level-id'] = sanitize_text_field( $_POST['give-price-id'] );
+	if (isset($_POST['give-price-id'])) {
+		$defaults['level-id'] = sanitize_text_field($_POST['give-price-id']);
 	}
 
 	// Check for backward compatibility.
-	if ( is_string( $args ) ) {
-		$args = str_replace( '?', '', $args );
+	if (is_string($args)) {
+		$args = str_replace('?', '', $args);
 	}
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	// Merge URL query with $args to maintain third-party URL parameters after redirect.
-	$url_data = wp_parse_url( $url );
+	$url_data = wp_parse_url($url);
 
 	// Check if an array to prevent notices before parsing.
-	if ( isset( $url_data['query'] ) && ! empty( $url_data['query'] ) ) {
-		parse_str( $url_data['query'], $query );
+	if (isset($url_data['query']) && ! empty($url_data['query'])) {
+		parse_str($url_data['query'], $query);
 
 		// Precaution: don't allow any CC info.
-		unset( $query['card_number'] );
-		unset( $query['card_cvc'] );
+		unset($query['card_number']);
+		unset($query['card_cvc']);
 
 	} else {
 		// No $url_data so pass empty array.
 		$query = array();
 	}
 
-	$new_query        = array_merge( $args, $query );
-	$new_query_string = http_build_query( $new_query );
+	$new_query        = array_merge($args, $query);
+	$new_query_string = http_build_query($new_query);
 
 	// Assemble URL parts.
-	$redirect = home_url( '/' . $url_data['path'] . '?' . $new_query_string . '#give-form-' . $form_id . '-wrap' );
+	$redirect = home_url('/'.$url_data['path'].'?'.$new_query_string.'#give-form-'.$form_id.'-wrap');
 
 	// Redirect them.
-	wp_safe_redirect( apply_filters( 'give_send_back_to_checkout', $redirect, $args ) );
+	wp_safe_redirect(apply_filters('give_send_back_to_checkout', $redirect, $args));
 	give_die();
 
 }
@@ -232,16 +232,16 @@ 
  * @since       1.0
  * @return      string
  */
-function give_get_success_page_url( $query_string = null ) {
+function give_get_success_page_url($query_string = null) {
 
-	$success_page = give_get_option( 'success_page', 0 );
-	$success_page = get_permalink( $success_page );
+	$success_page = give_get_option('success_page', 0);
+	$success_page = get_permalink($success_page);
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$success_page .= $query_string;
 	}
 
-	return apply_filters( 'give_success_page_url', $success_page );
+	return apply_filters('give_success_page_url', $success_page);
 
 }
 
@@ -254,15 +254,15 @@ 
  *
  * @return mixed|void Full URL to the Failed Donation Page, if present, home page if it doesn't exist
  */
-function give_get_failed_transaction_uri( $extras = false ) {
+function give_get_failed_transaction_uri($extras = false) {
 	$give_options = give_get_settings();
 
-	$uri = ! empty( $give_options['failure_page'] ) ? trailingslashit( get_permalink( $give_options['failure_page'] ) ) : home_url();
-	if ( $extras ) {
+	$uri = ! empty($give_options['failure_page']) ? trailingslashit(get_permalink($give_options['failure_page'])) : home_url();
+	if ($extras) {
 		$uri .= $extras;
 	}
 
-	return apply_filters( 'give_get_failed_transaction_uri', $uri );
+	return apply_filters('give_get_failed_transaction_uri', $uri);
 }
 
 /**
@@ -273,9 +273,9 @@ 
  */
 function give_is_failed_transaction_page() {
 	$give_options = give_get_settings();
-	$ret          = isset( $give_options['failure_page'] ) ? is_page( $give_options['failure_page'] ) : false;
+	$ret          = isset($give_options['failure_page']) ? is_page($give_options['failure_page']) : false;
 
-	return apply_filters( 'give_is_failure_page', $ret );
+	return apply_filters('give_is_failure_page', $ret);
 }
 
 /**
@@ -287,18 +287,18 @@ 
  */
 function give_listen_for_failed_payments() {
 
-	$failed_page = give_get_option( 'failure_page', 0 );
+	$failed_page = give_get_option('failure_page', 0);
 
-	if ( ! empty( $failed_page ) && is_page( $failed_page ) && ! empty( $_GET['payment-id'] ) ) {
+	if ( ! empty($failed_page) && is_page($failed_page) && ! empty($_GET['payment-id'])) {
 
-		$payment_id = absint( $_GET['payment-id'] );
-		give_update_payment_status( $payment_id, 'failed' );
+		$payment_id = absint($_GET['payment-id']);
+		give_update_payment_status($payment_id, 'failed');
 
 	}
 
 }
 
-add_action( 'template_redirect', 'give_listen_for_failed_payments' );
+add_action('template_redirect', 'give_listen_for_failed_payments');
 
 /**
  * Retrieve the Donation History page URI
@@ -311,9 +311,9 @@ 
 function give_get_history_page_uri() {
 	$give_options = give_get_settings();
 
-	$history_page = isset( $give_options['history_page'] ) ? get_permalink( absint( $give_options['history_page'] ) ) : get_bloginfo( 'url' );
+	$history_page = isset($give_options['history_page']) ? get_permalink(absint($give_options['history_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_history_page_uri', $history_page );
+	return apply_filters('give_get_history_page_uri', $history_page);
 }
 
 /**
@@ -326,11 +326,11 @@ 
  * @since       1.0
  * @return      bool
  */
-function give_field_is_required( $field = '', $form_id ) {
+function give_field_is_required($field = '', $form_id) {
 
-	$required_fields = give_get_required_fields( $form_id );
+	$required_fields = give_get_required_fields($form_id);
 
-	return array_key_exists( $field, $required_fields );
+	return array_key_exists($field, $required_fields);
 }
 
 /**
@@ -348,14 +348,14 @@ 
  *
  * @return void
  */
-function give_record_sale_in_log( $give_form_id = 0, $payment_id, $price_id = false, $sale_date = null ) {
+function give_record_sale_in_log($give_form_id = 0, $payment_id, $price_id = false, $sale_date = null) {
 	global $give_logs;
 
 	$log_data = array(
 		'post_parent'   => $give_form_id,
 		'log_type'      => 'sale',
-		'post_date'     => isset( $sale_date ) ? $sale_date : null,
-		'post_date_gmt' => isset( $sale_date ) ? $sale_date : null,
+		'post_date'     => isset($sale_date) ? $sale_date : null,
+		'post_date_gmt' => isset($sale_date) ? $sale_date : null,
 	);
 
 	$log_meta = array(
@@ -363,7 +363,7 @@ 
 		'price_id'   => (int) $price_id,
 	);
 
-	$give_logs->insert_log( $log_data, $log_meta );
+	$give_logs->insert_log($log_data, $log_meta);
 }
 
 
@@ -377,11 +377,11 @@ 
  *
  * @return bool|int
  */
-function give_increase_purchase_count( $form_id = 0, $quantity = 1 ) {
+function give_increase_purchase_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->increase_sales( $quantity );
+	return $form->increase_sales($quantity);
 }
 
 /**
@@ -394,11 +394,11 @@ 
  *
  * @return bool|int
  */
-function give_decrease_purchase_count( $form_id = 0, $quantity = 1 ) {
+function give_decrease_purchase_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->decrease_sales( $quantity );
+	return $form->decrease_sales($quantity);
 }
 
 /**
@@ -411,10 +411,10 @@ 
  *
  * @return bool|int
  */
-function give_increase_earnings( $give_form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $give_form_id );
+function give_increase_earnings($give_form_id = 0, $amount) {
+	$form = new Give_Donate_Form($give_form_id);
 
-	return $form->increase_earnings( $amount );
+	return $form->increase_earnings($amount);
 }
 
 /**
@@ -427,10 +427,10 @@ 
  *
  * @return bool|int
  */
-function give_decrease_earnings( $form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $form_id );
+function give_decrease_earnings($form_id = 0, $amount) {
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->decrease_earnings( $amount );
+	return $form->decrease_earnings($amount);
 }
 
 
@@ -443,8 +443,8 @@ 
  *
  * @return int $earnings Earnings for a certain form
  */
-function give_get_form_earnings_stats( $form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $form_id );
+function give_get_form_earnings_stats($form_id = 0) {
+	$give_form = new Give_Donate_Form($form_id);
 
 	return $give_form->earnings;
 }
@@ -459,8 +459,8 @@ 
  *
  * @return int $sales Amount of sales for a certain form
  */
-function give_get_form_sales_stats( $give_form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $give_form_id );
+function give_get_form_sales_stats($give_form_id = 0) {
+	$give_form = new Give_Donate_Form($give_form_id);
 
 	return $give_form->sales;
 }
@@ -475,16 +475,16 @@ 
  *
  * @return float $sales Average monthly sales
  */
-function give_get_average_monthly_form_sales( $form_id = 0 ) {
-	$sales        = give_get_form_sales_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_sales($form_id = 0) {
+	$sales        = give_get_form_sales_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$sales = ( $sales / $months );
+	if ($months > 0) {
+		$sales = ($sales / $months);
 	}
 
 	return $sales;
@@ -500,16 +500,16 @@ 
  *
  * @return float $earnings Average monthly earnings
  */
-function give_get_average_monthly_form_earnings( $form_id = 0 ) {
-	$earnings     = give_get_form_earnings_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_earnings($form_id = 0) {
+	$earnings     = give_get_form_earnings_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$earnings = ( $earnings / $months );
+	if ($months > 0) {
+		$earnings = ($earnings / $months);
 	}
 
 	return $earnings < 0 ? 0 : $earnings;
@@ -529,23 +529,23 @@ 
  *
  * @return string $price_name Name of the price option
  */
-function give_get_price_option_name( $form_id = 0, $price_id = 0, $payment_id = 0 ) {
+function give_get_price_option_name($form_id = 0, $price_id = 0, $payment_id = 0) {
 
-	$prices     = give_get_variable_prices( $form_id );
+	$prices     = give_get_variable_prices($form_id);
 	$price_name = '';
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( intval( $price['_give_id']['level_id'] ) == intval( $price_id ) ) {
+		if (intval($price['_give_id']['level_id']) == intval($price_id)) {
 
-			$price_text     = isset( $price['_give_text'] ) ? $price['_give_text'] : '';
-			$price_fallback = give_currency_filter( give_format_amount( $price['_give_amount'] ) );
-			$price_name     = ! empty( $price_text ) ? $price_text : $price_fallback;
+			$price_text     = isset($price['_give_text']) ? $price['_give_text'] : '';
+			$price_fallback = give_currency_filter(give_format_amount($price['_give_amount']));
+			$price_name     = ! empty($price_text) ? $price_text : $price_fallback;
 
 		}
 	}
 
-	return apply_filters( 'give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id );
+	return apply_filters('give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id);
 }
 
 
@@ -558,14 +558,14 @@ 
  *
  * @return string $range A fully formatted price range
  */
-function give_price_range( $form_id = 0 ) {
-	$low   = give_get_lowest_price_option( $form_id );
-	$high  = give_get_highest_price_option( $form_id );
-	$range = '<span class="give_price_range_low">' . give_currency_filter( give_format_amount( $low ) ) . '</span>';
+function give_price_range($form_id = 0) {
+	$low   = give_get_lowest_price_option($form_id);
+	$high  = give_get_highest_price_option($form_id);
+	$range = '<span class="give_price_range_low">'.give_currency_filter(give_format_amount($low)).'</span>';
 	$range .= '<span class="give_price_range_sep">&nbsp;&ndash;&nbsp;</span>';
-	$range .= '<span class="give_price_range_high">' . give_currency_filter( give_format_amount( $high ) ) . '</span>';
+	$range .= '<span class="give_price_range_high">'.give_currency_filter(give_format_amount($high)).'</span>';
 
-	return apply_filters( 'give_price_range', $range, $form_id, $low, $high );
+	return apply_filters('give_price_range', $range, $form_id, $low, $high);
 }
 
 
@@ -580,35 +580,35 @@ 
  *
  * @return int ID of the lowest price
  */
-function give_get_lowest_price_id( $form_id = 0 ) {
+function give_get_lowest_price_id($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$min = $min_id = 0;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, $price['_give_amount'] );
+				$min = min($min, $price['_give_amount']);
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $price['_give_id']['level_id'];
 			}
 		}
@@ -626,45 +626,45 @@ 
  *
  * @return float Amount of the lowest price
  */
-function give_get_lowest_price_option( $form_id = 0 ) {
-	if ( empty( $form_id ) ) {
+function give_get_lowest_price_option($form_id = 0) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$low = 0;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
 		$min = $min_id = 0;
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, give_sanitize_amount( $price['_give_amount'] ) );
+				$min = min($min, give_sanitize_amount($price['_give_amount']));
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $key;
 			}
 		}
 
-		$low = $prices[ $min_id ]['_give_amount'];
+		$low = $prices[$min_id]['_give_amount'];
 
 	}
 
-	return give_sanitize_amount( $low );
+	return give_sanitize_amount($low);
 }
 
 /**
@@ -676,41 +676,41 @@ 
  *
  * @return float Amount of the highest price
  */
-function give_get_highest_price_option( $form_id = 0 ) {
+function give_get_highest_price_option($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$high = 0.00;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
 		$max_id = $max = 0;
 
-		foreach ( $prices as $key => $price ) {
-			if ( empty( $price['_give_amount'] ) ) {
+		foreach ($prices as $key => $price) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
-			$give_amount = give_sanitize_amount( $price['_give_amount'] );
+			$give_amount = give_sanitize_amount($price['_give_amount']);
 
-			$max = max( $max, $give_amount );
+			$max = max($max, $give_amount);
 
-			if ( $give_amount == $max ) {
+			if ($give_amount == $max) {
 				$max_id = $key;
 			}
 		}
 
-		$high = $prices[ $max_id ]['_give_amount'];
+		$high = $prices[$max_id]['_give_amount'];
 	}
 
-	return give_sanitize_amount( $high );
+	return give_sanitize_amount($high);
 }
 
 /**
@@ -722,15 +722,15 @@ 
  *
  * @return mixed string|int Price of the form
  */
-function give_get_form_price( $form_id = 0 ) {
+function give_get_form_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'price' );
+	return $form->__get('price');
 }
 
 /**
@@ -742,15 +742,15 @@ 
  *
  * @return mixed string|int Minimum price of the form
  */
-function give_get_form_minimum_price( $form_id = 0 ) {
+function give_get_form_minimum_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'minimum_price' );
+	return $form->__get('minimum_price');
 
 }
 
@@ -765,51 +765,51 @@ 
  *
  * @return int $formatted_price
  */
-function give_price( $form_id = 0, $echo = true, $price_id = false ) {
+function give_price($form_id = 0, $echo = true, $price_id = false) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$prices = give_get_variable_prices( $form_id );
+		$prices = give_get_variable_prices($form_id);
 
-		if ( false !== $price_id ) {
+		if (false !== $price_id) {
 
 			// loop through multi-prices to see which is default
-			foreach ( $prices as $price ) {
+			foreach ($prices as $price) {
 				// this is the default price
-				if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+				if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 					$price = (float) $price['_give_amount'];
 				};
 			}
 		} else {
 
-			$price = give_get_lowest_price_option( $form_id );
+			$price = give_get_lowest_price_option($form_id);
 		}
 
-		$price = give_sanitize_amount( $price );
+		$price = give_sanitize_amount($price);
 
 	} else {
 
-		$price = give_get_form_price( $form_id );
+		$price = give_get_form_price($form_id);
 
 	}
 
-	$price           = apply_filters( 'give_form_price', give_sanitize_amount( $price ), $form_id );
-	$formatted_price = '<span class="give_price" id="give_price_' . $form_id . '">' . $price . '</span>';
-	$formatted_price = apply_filters( 'give_form_price_after_html', $formatted_price, $form_id, $price );
+	$price           = apply_filters('give_form_price', give_sanitize_amount($price), $form_id);
+	$formatted_price = '<span class="give_price" id="give_price_'.$form_id.'">'.$price.'</span>';
+	$formatted_price = apply_filters('give_form_price_after_html', $formatted_price, $form_id, $price);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_price;
 	} else {
 		return $formatted_price;
 	}
 }
 
-add_filter( 'give_form_price', 'give_format_amount', 10 );
-add_filter( 'give_form_price', 'give_currency_filter', 20 );
+add_filter('give_form_price', 'give_format_amount', 10);
+add_filter('give_form_price', 'give_currency_filter', 20);
 
 
 /**
@@ -822,19 +822,19 @@ 
  *
  * @return float $amount Amount of the price option
  */
-function give_get_price_option_amount( $form_id = 0, $price_id = 0 ) {
-	$prices = give_get_variable_prices( $form_id );
+function give_get_price_option_amount($form_id = 0, $price_id = 0) {
+	$prices = give_get_variable_prices($form_id);
 
 	$amount = 0.00;
 
-	foreach ( $prices as $price ) {
-		if ( isset( $price['_give_id']['level_id'] ) && $price['_give_id']['level_id'] == $price_id ) {
-			$amount = isset( $price['_give_amount'] ) ? $price['_give_amount'] : 0.00;
+	foreach ($prices as $price) {
+		if (isset($price['_give_id']['level_id']) && $price['_give_id']['level_id'] == $price_id) {
+			$amount = isset($price['_give_amount']) ? $price['_give_amount'] : 0.00;
 			break;
 		};
 	}
 
-	return apply_filters( 'give_get_price_option_amount', give_sanitize_amount( $amount ), $form_id, $price_id );
+	return apply_filters('give_get_price_option_amount', give_sanitize_amount($amount), $form_id, $price_id);
 }
 
 /**
@@ -846,13 +846,13 @@ 
  *
  * @return mixed string|int Goal of the form
  */
-function give_get_form_goal( $form_id = 0 ) {
+function give_get_form_goal($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->goal;
 
@@ -868,27 +868,27 @@ 
  *
  * @return string $formatted_goal
  */
-function give_goal( $form_id = 0, $echo = true ) {
+function give_goal($form_id = 0, $echo = true) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	$goal = give_get_form_goal( $form_id );
+	$goal = give_get_form_goal($form_id);
 
-	$goal           = apply_filters( 'give_form_goal', give_sanitize_amount( $goal ), $form_id );
-	$formatted_goal = '<span class="give_price" id="give_price_' . $form_id . '">' . $goal . '</span>';
-	$formatted_goal = apply_filters( 'give_form_price_after_html', $formatted_goal, $form_id, $goal );
+	$goal           = apply_filters('give_form_goal', give_sanitize_amount($goal), $form_id);
+	$formatted_goal = '<span class="give_price" id="give_price_'.$form_id.'">'.$goal.'</span>';
+	$formatted_goal = apply_filters('give_form_price_after_html', $formatted_goal, $form_id, $goal);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_goal;
 	} else {
 		return $formatted_goal;
 	}
 }
 
-add_filter( 'give_form_goal', 'give_format_amount', 10 );
-add_filter( 'give_form_goal', 'give_currency_filter', 20 );
+add_filter('give_form_goal', 'give_format_amount', 10);
+add_filter('give_form_goal', 'give_currency_filter', 20);
 
 
 /**
@@ -900,15 +900,15 @@ 
  *
  * @return bool  $ret Whether or not the logged_in_only setting is set
  */
-function give_logged_in_only( $form_id ) {
+function give_logged_in_only($form_id) {
 	// If _give_logged_in_only is set to enable then guest can donate from that specific form.
 	// Otherwise it is member only donation form.
-	$val = give_get_meta( $form_id, '_give_logged_in_only', true );
-	$val = ! empty( $val ) ? $val : 'enabled';
+	$val = give_get_meta($form_id, '_give_logged_in_only', true);
+	$val = ! empty($val) ? $val : 'enabled';
 
-	$ret = ! give_is_setting_enabled( $val );
+	$ret = ! give_is_setting_enabled($val);
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret, $form_id );
+	return (bool) apply_filters('give_logged_in_only', $ret, $form_id);
 }
 
 
@@ -921,11 +921,11 @@ 
  *
  * @return string
  */
-function give_show_login_register_option( $form_id ) {
+function give_show_login_register_option($form_id) {
 
-	$show_register_form = give_get_meta( $form_id, '_give_show_register_form', true );
+	$show_register_form = give_get_meta($form_id, '_give_show_register_form', true);
 
-	return apply_filters( 'give_show_register_form', $show_register_form, $form_id );
+	return apply_filters('give_show_register_form', $show_register_form, $form_id);
 
 }
 
@@ -941,12 +941,12 @@ 
  *
  * @return array
  */
-function _give_get_prefill_form_field_values( $form_id ) {
+function _give_get_prefill_form_field_values($form_id) {
 	$logged_in_donor_info = array();
 
-	if ( is_user_logged_in() ) :
-		$donor_data    = get_userdata( get_current_user_id() );
-		$donor_address = get_user_meta( get_current_user_id(), '_give_user_address', true );
+	if (is_user_logged_in()) :
+		$donor_data    = get_userdata(get_current_user_id());
+		$donor_address = get_user_meta(get_current_user_id(), '_give_user_address', true);
 
 		$logged_in_donor_info = array(
 			// First name.
@@ -959,42 +959,42 @@ 
 			'give_email'      => $donor_data->user_email,
 
 			// Street address 1.
-			'card_address'    => ( ! empty( $donor_address['line1'] ) ? $donor_address['line1'] : '' ),
+			'card_address'    => ( ! empty($donor_address['line1']) ? $donor_address['line1'] : ''),
 
 			// Street address 2.
-			'card_address_2'  => ( ! empty( $donor_address['line2'] ) ? $donor_address['line2'] : '' ),
+			'card_address_2'  => ( ! empty($donor_address['line2']) ? $donor_address['line2'] : ''),
 
 			// Country.
-			'billing_country' => ( ! empty( $donor_address['country'] ) ? $donor_address['country'] : '' ),
+			'billing_country' => ( ! empty($donor_address['country']) ? $donor_address['country'] : ''),
 
 			// State.
-			'card_state'      => ( ! empty( $donor_address['state'] ) ? $donor_address['state'] : '' ),
+			'card_state'      => ( ! empty($donor_address['state']) ? $donor_address['state'] : ''),
 
 			// City.
-			'card_city'       => ( ! empty( $donor_address['city'] ) ? $donor_address['city'] : '' ),
+			'card_city'       => ( ! empty($donor_address['city']) ? $donor_address['city'] : ''),
 
 			// Zipcode
-			'card_zip'        => ( ! empty( $donor_address['zip'] ) ? $donor_address['zip'] : '' ),
+			'card_zip'        => ( ! empty($donor_address['zip']) ? $donor_address['zip'] : ''),
 		);
 	endif;
 
 	// Bailout: Auto fill form field values only form form which donor is donating.
 	if (
-		empty( $_GET['form-id'] )
+		empty($_GET['form-id'])
 		|| ! $form_id
-		|| ( $form_id !== absint( $_GET['form-id'] ) )
+		|| ($form_id !== absint($_GET['form-id']))
 	) {
 		return $logged_in_donor_info;
 	}
 
 	// Get purchase data.
-	$give_purchase_data = Give()->session->get( 'give_purchase' );
+	$give_purchase_data = Give()->session->get('give_purchase');
 
 	// Get donor info from form data.
-	$give_donor_info_in_session = empty( $give_purchase_data['post_data'] )
+	$give_donor_info_in_session = empty($give_purchase_data['post_data'])
 		? array()
 		: $give_purchase_data['post_data'];
 
 	// Output.
-	return wp_parse_args( $give_donor_info_in_session, $logged_in_donor_info );
+	return wp_parse_args($give_donor_info_in_session, $logged_in_donor_info);
 }

includes/payments/class-give-payment.php

Doc Comments +3 added lines, -3 removed lines

@@ -392,7 +392,7 @@ 
 	 *
 	 * @param  int|bool $payment_id A given payment
 	 *
-	 * @return mixed void|false
+	 * @return false|null void|false
 	 */
 	public function __construct( $payment_id = false ) {
 
@@ -463,7 +463,7 @@ 
 	 *
 	 * @param  string $name The attribute to get
 	 *
-	 * @return boolean       If the item is set or not
+	 * @return boolean|null       If the item is set or not
 	 */
 	public function __isset( $name ) {
 		if ( property_exists( $this, $name ) ) {
@@ -1302,7 +1302,7 @@ 
 	 *
 	 * @param  string $note The note to add
 	 *
-	 * @return void
+	 * @return false|null
 	 */
 	public function add_note( $note = false ) {
 		// Bail if no note specified.

Indentation +1 added lines, -1 removed lines

@@ -1050,7 +1050,7 @@
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
 					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
-					     && $args['price_id'] == $price['_give_id']['level_id']
+						 && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}

Spacing +326 added lines, -326 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -394,13 +394,13 @@ 
 	 *
 	 * @return mixed void|false
 	 */
-	public function __construct( $payment_id = false ) {
+	public function __construct($payment_id = false) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -413,11 +413,11 @@ 
 	 *
 	 * @return mixed        The value.
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			$value = call_user_func( array( $this, 'get_' . $key ) );
+			$value = call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
@@ -439,18 +439,18 @@ 
 	 * @param  string $key The property name
 	 * @param  mixed $value The value of the property
 	 */
-	public function __set( $key, $value ) {
-		$ignore = array( '_ID' );
+	public function __set($key, $value) {
+		$ignore = array('_ID');
 
-		if ( $key === 'status' ) {
+		if ($key === 'status') {
 			$this->old_status = $this->status;
 		}
 
-		if ( ! in_array( $key, $ignore ) ) {
-			$this->pending[ $key ] = $value;
+		if ( ! in_array($key, $ignore)) {
+			$this->pending[$key] = $value;
 		}
 
-		if ( '_ID' !== $key ) {
+		if ('_ID' !== $key) {
 			$this->$key = $value;
 		}
 	}
@@ -465,9 +465,9 @@ 
 	 *
 	 * @return boolean       If the item is set or not
 	 */
-	public function __isset( $name ) {
-		if ( property_exists( $this, $name ) ) {
-			return false === empty( $this->$name );
+	public function __isset($name) {
+		if (property_exists($this, $name)) {
+			return false === empty($this->$name);
 		} else {
 			return null;
 		}
@@ -483,20 +483,20 @@ 
 	 *
 	 * @return bool            If the setup was successful or not
 	 */
-	private function setup_payment( $payment_id ) {
+	private function setup_payment($payment_id) {
 		$this->pending = array();
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = get_post( $payment_id );
+		$payment = get_post($payment_id);
 
-		if ( ! $payment || is_wp_error( $payment ) ) {
+		if ( ! $payment || is_wp_error($payment)) {
 			return false;
 		}
 
-		if ( 'give_payment' !== $payment->post_type ) {
+		if ('give_payment' !== $payment->post_type) {
 			return false;
 		}
 
@@ -510,13 +510,13 @@ 
 		 * @param Give_Payment $this Payment object.
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_pre_setup_payment', $this, $payment_id );
+		do_action('give_pre_setup_payment', $this, $payment_id);
 
 		// Primary Identifier.
-		$this->ID = absint( $payment_id );
+		$this->ID = absint($payment_id);
 
 		// Protected ID that can never be changed.
-		$this->_ID = absint( $payment_id );
+		$this->_ID = absint($payment_id);
 
 		// We have a payment, get the generic payment_meta item to reduce calls to it.
 		$this->payment_meta = $this->get_meta();
@@ -531,7 +531,7 @@ 
 		$this->parent_payment = $payment->post_parent;
 
 		$all_payment_statuses  = give_get_payment_statuses();
-		$this->status_nicename = array_key_exists( $this->status, $all_payment_statuses ) ? $all_payment_statuses[ $this->status ] : ucfirst( $this->status );
+		$this->status_nicename = array_key_exists($this->status, $all_payment_statuses) ? $all_payment_statuses[$this->status] : ucfirst($this->status);
 
 		// Items.
 		$this->fees = $this->setup_fees();
@@ -573,7 +573,7 @@ 
 		 * @param Give_Payment $this Payment object.
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -591,8 +591,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_payment_setup( $payment_id ) {
-		$this->setup_payment( $payment_id );
+	public function update_payment_setup($payment_id) {
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -607,24 +607,24 @@ 
 
 		// Construct the payment title.
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} elseif ( ! empty( $this->first_name ) && empty( $this->last_name ) ) {
+		if ( ! empty($this->first_name) && ! empty($this->last_name)) {
+			$payment_title = $this->first_name.' '.$this->last_name;
+		} elseif ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} elseif ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} elseif ( ! empty($this->email) && is_email($this->email)) {
 			$payment_title = $this->email;
 		}
 
 		// Set Key.
-		if ( empty( $this->key ) ) {
+		if (empty($this->key)) {
 
-			$auth_key             = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-			$this->key            = strtolower( md5( $this->email . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );  // Unique key
+			$auth_key             = defined('AUTH_KEY') ? AUTH_KEY : '';
+			$this->key            = strtolower(md5($this->email.date('Y-m-d H:i:s').$auth_key.uniqid('give', true))); // Unique key
 			$this->pending['key'] = $this->key;
 		}
 
 		// Set IP.
-		if ( empty( $this->ip ) ) {
+		if (empty($this->ip)) {
 
 			$this->ip            = give_get_ip();
 			$this->pending['ip'] = $this->ip;
@@ -651,63 +651,63 @@ 
 			'fees'         => $this->fees,
 		);
 
-		$args = apply_filters( 'give_insert_payment_args', array(
+		$args = apply_filters('give_insert_payment_args', array(
 			'post_title'    => $payment_title,
 			'post_status'   => $this->status,
 			'post_type'     => 'give_payment',
-			'post_date'     => ! empty( $this->date ) ? $this->date : null,
-			'post_date_gmt' => ! empty( $this->date ) ? get_gmt_from_date( $this->date ) : null,
+			'post_date'     => ! empty($this->date) ? $this->date : null,
+			'post_date_gmt' => ! empty($this->date) ? get_gmt_from_date($this->date) : null,
 			'post_parent'   => $this->parent_payment,
-		), $payment_data );
+		), $payment_data);
 
 		// Create a blank payment
-		$payment_id = wp_insert_post( $args );
+		$payment_id = wp_insert_post($args);
 
-		if ( ! empty( $payment_id ) ) {
+		if ( ! empty($payment_id)) {
 
 			$this->ID  = $payment_id;
 			$this->_ID = $payment_id;
 
 			$customer = new stdClass;
 
-			if ( did_action( 'give_pre_process_donation' ) && is_user_logged_in() ) {
-				$customer = new Give_Customer( get_current_user_id(), true );
+			if (did_action('give_pre_process_donation') && is_user_logged_in()) {
+				$customer = new Give_Customer(get_current_user_id(), true);
 
 				// Customer is logged in but used a different email to purchase with so assign to their customer record
-				if ( ! empty( $customer->id ) && $this->email != $customer->email ) {
-					$customer->add_email( $this->email );
+				if ( ! empty($customer->id) && $this->email != $customer->email) {
+					$customer->add_email($this->email);
 				}
 			}
 
-			if ( empty( $customer->id ) ) {
-				$customer = new Give_Customer( $this->email );
+			if (empty($customer->id)) {
+				$customer = new Give_Customer($this->email);
 			}
 
-			if ( empty( $customer->id ) ) {
+			if (empty($customer->id)) {
 
 				$customer_data = array(
-					'name'    => ! is_email( $payment_title ) ? $this->first_name . ' ' . $this->last_name : '',
+					'name'    => ! is_email($payment_title) ? $this->first_name.' '.$this->last_name : '',
 					'email'   => $this->email,
 					'user_id' => $this->user_id,
 				);
 
-				$customer->create( $customer_data );
+				$customer->create($customer_data);
 
 			}
 
 			$this->customer_id            = $customer->id;
 			$this->pending['customer_id'] = $this->customer_id;
-			$customer->attach_payment( $this->ID, false );
+			$customer->attach_payment($this->ID, false);
 
-			$this->payment_meta = apply_filters( 'give_payment_meta', $this->payment_meta, $payment_data );
-			if ( ! empty( $this->payment_meta['fees'] ) ) {
-				$this->fees = array_merge( $this->fees, $this->payment_meta['fees'] );
-				foreach ( $this->fees as $fee ) {
-					$this->increase_fees( $fee['amount'] );
+			$this->payment_meta = apply_filters('give_payment_meta', $this->payment_meta, $payment_data);
+			if ( ! empty($this->payment_meta['fees'])) {
+				$this->fees = array_merge($this->fees, $this->payment_meta['fees']);
+				foreach ($this->fees as $fee) {
+					$this->increase_fees($fee['amount']);
 				}
 			}
 
-			$this->update_meta( '_give_payment_meta', $this->payment_meta );
+			$this->update_meta('_give_payment_meta', $this->payment_meta);
 			$this->new = true;
 		}// End if().
 
@@ -729,11 +729,11 @@ 
 		$saved = false;
 
 		// Must have an ID.
-		if ( empty( $this->ID ) ) {
+		if (empty($this->ID)) {
 
 			$payment_id = $this->insert_payment();
 
-			if ( false === $payment_id ) {
+			if (false === $payment_id) {
 				$saved = false;
 			} else {
 				$this->ID = $payment_id;
@@ -741,48 +741,48 @@ 
 		}
 
 		// Set ID if not matching.
-		if ( $this->ID !== $this->_ID ) {
+		if ($this->ID !== $this->_ID) {
 			$this->ID = $this->_ID;
 		}
 
 		// If we have something pending, let's save it.
-		if ( ! empty( $this->pending ) ) {
+		if ( ! empty($this->pending)) {
 
 			$total_increase = 0;
 			$total_decrease = 0;
 
-			foreach ( $this->pending as $key => $value ) {
+			foreach ($this->pending as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'donations':
 						// Update totals for pending donations.
-						foreach ( $this->pending[ $key ] as $item ) {
+						foreach ($this->pending[$key] as $item) {
 
-							$quantity = isset( $item['quantity'] ) ? $item['quantity'] : 1;
-							$price_id = isset( $item['price_id'] ) ? $item['price_id'] : 0;
+							$quantity = isset($item['quantity']) ? $item['quantity'] : 1;
+							$price_id = isset($item['price_id']) ? $item['price_id'] : 0;
 
-							switch ( $item['action'] ) {
+							switch ($item['action']) {
 
 								case 'add':
 
 									$price = $item['price'];
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status) {
 
 										// Add sales logs.
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
 
 										$y = 0;
-										while ( $y < $quantity ) {
+										while ($y < $quantity) {
 
-											give_record_sale_in_log( $item['id'], $this->ID, $price_id, $log_date );
-											$y ++;
+											give_record_sale_in_log($item['id'], $this->ID, $price_id, $log_date);
+											$y++;
 										}
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price);
 
 										$total_increase += $price;
 									}
@@ -807,15 +807,15 @@ 
 										),
 									);
 
-									$found_logs = get_posts( $log_args );
-									foreach ( $found_logs as $log ) {
-										wp_delete_post( $log->ID, true );
+									$found_logs = get_posts($log_args);
+									foreach ($found_logs as $log) {
+										wp_delete_post($log->ID, true);
 									}
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'] );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -827,17 +827,17 @@ 
 
 					case 'fees':
 
-						if ( 'publish' !== $this->status && 'complete' !== $this->status ) {
+						if ('publish' !== $this->status && 'complete' !== $this->status) {
 							break;
 						}
 
-						if ( empty( $this->pending[ $key ] ) ) {
+						if (empty($this->pending[$key])) {
 							break;
 						}
 
-						foreach ( $this->pending[ $key ] as $fee ) {
+						foreach ($this->pending[$key] as $fee) {
 
-							switch ( $fee['action'] ) {
+							switch ($fee['action']) {
 
 								case 'add':
 									$total_increase += $fee['amount'];
@@ -853,43 +853,43 @@ 
 						break;
 
 					case 'status':
-						$this->update_status( $this->status );
+						$this->update_status($this->status);
 						break;
 
 					case 'gateway':
-						$this->update_meta( '_give_payment_gateway', $this->gateway );
+						$this->update_meta('_give_payment_gateway', $this->gateway);
 						break;
 
 					case 'mode':
-						$this->update_meta( '_give_payment_mode', $this->mode );
+						$this->update_meta('_give_payment_mode', $this->mode);
 						break;
 
 					case 'transaction_id':
-						$this->update_meta( '_give_payment_transaction_id', $this->transaction_id );
+						$this->update_meta('_give_payment_transaction_id', $this->transaction_id);
 						break;
 
 					case 'ip':
-						$this->update_meta( '_give_payment_user_ip', $this->ip );
+						$this->update_meta('_give_payment_user_ip', $this->ip);
 						break;
 
 					case 'customer_id':
-						$this->update_meta( '_give_payment_customer_id', $this->customer_id );
+						$this->update_meta('_give_payment_customer_id', $this->customer_id);
 						break;
 
 					case 'user_id':
-						$this->update_meta( '_give_payment_user_id', $this->user_id );
+						$this->update_meta('_give_payment_user_id', $this->user_id);
 						break;
 
 					case 'form_title':
-						$this->update_meta( '_give_payment_form_title', $this->form_title );
+						$this->update_meta('_give_payment_form_title', $this->form_title);
 						break;
 
 					case 'form_id':
-						$this->update_meta( '_give_payment_form_id', $this->form_id );
+						$this->update_meta('_give_payment_form_id', $this->form_id);
 						break;
 
 					case 'price_id':
-						$this->update_meta( '_give_payment_price_id', $this->price_id );
+						$this->update_meta('_give_payment_price_id', $this->price_id);
 						break;
 
 					case 'first_name':
@@ -905,15 +905,15 @@ 
 						break;
 
 					case 'email':
-						$this->update_meta( '_give_payment_user_email', $this->email );
+						$this->update_meta('_give_payment_user_email', $this->email);
 						break;
 
 					case 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
@@ -923,11 +923,11 @@ 
 							'edit_date' => true,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'completed_date':
-						$this->update_meta( '_give_completed_date', $this->completed_date );
+						$this->update_meta('_give_completed_date', $this->completed_date);
 						break;
 
 					case 'parent_payment':
@@ -936,7 +936,7 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
@@ -947,33 +947,33 @@ 
 						 *
 						 * @param Give_Payment $this Payment object.
 						 */
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				}// End switch().
 			}// End foreach().
 
-			if ( 'pending' !== $this->status ) {
+			if ('pending' !== $this->status) {
 
-				$customer = new Give_Customer( $this->customer_id );
+				$customer = new Give_Customer($this->customer_id);
 
 				$total_change = $total_increase - $total_decrease;
-				if ( $total_change < 0 ) {
+				if ($total_change < 0) {
 
-					$total_change = - ( $total_change );
+					$total_change = - ($total_change);
 					// Decrease the customer's donation stats.
-					$customer->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$customer->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
-				} elseif ( $total_change > 0 ) {
+				} elseif ($total_change > 0) {
 
 					// Increase the customer's donation stats.
-					$customer->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$customer->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 				}
 			}
 
-			$this->update_meta( '_give_payment_total', $this->total );
+			$this->update_meta('_give_payment_total', $this->total);
 
 			$new_meta = array(
 				'form_title' => $this->form_title,
@@ -985,12 +985,12 @@ 
 			);
 
 			$meta        = $this->get_meta();
-			$merged_meta = array_merge( $meta, $new_meta );
+			$merged_meta = array_merge($meta, $new_meta);
 
 			// Only save the payment meta if it's changed.
-			if ( md5( serialize( $meta ) ) !== md5( serialize( $merged_meta ) ) ) {
-				$updated = $this->update_meta( '_give_payment_meta', $merged_meta );
-				if ( false !== $updated ) {
+			if (md5(serialize($meta)) !== md5(serialize($merged_meta))) {
+				$updated = $this->update_meta('_give_payment_meta', $merged_meta);
+				if (false !== $updated) {
 					$saved = true;
 				}
 			}
@@ -999,8 +999,8 @@ 
 			$saved         = true;
 		}// End if().
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -1018,12 +1018,12 @@ 
 	 *
 	 * @return bool           True when successful, false otherwise
 	 */
-	public function add_donation( $form_id = 0, $args = array(), $options = array() ) {
+	public function add_donation($form_id = 0, $args = array(), $options = array()) {
 
-		$donation = new Give_Donate_Form( $form_id );
+		$donation = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a give donation form.
-		if ( ! $donation || $donation->post_type !== 'give_forms' ) {
+		if ( ! $donation || $donation->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -1034,59 +1034,59 @@ 
 			'fees'     => array(),
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_payment_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_payment_add_donation_args', $args, $donation->ID), $defaults);
 
 		// Allow overriding the price.
-		if ( false !== $args['price'] ) {
+		if (false !== $args['price']) {
 			$item_price = $args['price'];
 		} else {
 
 			// Deal with variable pricing.
-			if ( give_has_variable_prices( $donation->ID ) ) {
-				$prices     = maybe_unserialize( give_get_meta( $form_id, '_give_donation_levels', true ) );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices     = maybe_unserialize(give_get_meta($form_id, '_give_donation_levels', true));
 				$item_price = '';
 				// Loop through prices.
-				foreach ( $prices as $price ) {
+				foreach ($prices as $price) {
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
-					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
+					if ((isset($args['price_id']) && isset($price['_give_id']['level_id']))
 					     && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}
 				}
 				// Fallback to the lowest price point.
-				if ( $item_price == '' ) {
-					$item_price       = give_get_lowest_price_option( $donation->ID );
-					$args['price_id'] = give_get_lowest_price_id( $donation->ID );
+				if ($item_price == '') {
+					$item_price       = give_get_lowest_price_option($donation->ID);
+					$args['price_id'] = give_get_lowest_price_id($donation->ID);
 				}
 			} else {
 				// Simple form price.
-				$item_price = give_get_form_price( $donation->ID );
+				$item_price = give_get_form_price($donation->ID);
 			}
 		}
 
 		// Sanitizing the price here so we don't have a dozen calls later.
-		$item_price = give_sanitize_amount( $item_price );
-		$total      = round( $item_price, give_currency_decimal_filter() );
+		$item_price = give_sanitize_amount($item_price);
+		$total      = round($item_price, give_currency_decimal_filter());
 
 		// Add Options.
 		$default_options = array();
-		if ( false !== $args['price_id'] ) {
+		if (false !== $args['price_id']) {
 			$default_options['price_id'] = (int) $args['price_id'];
 		}
-		$options = wp_parse_args( $options, $default_options );
+		$options = wp_parse_args($options, $default_options);
 
 		// Do not allow totals to go negative.
-		if ( $total < 0 ) {
+		if ($total < 0) {
 			$total = 0;
 		}
 
 		$donation = array(
 			'name'     => $donation->post_title,
 			'id'       => $donation->ID,
-			'price'    => round( $total, give_currency_decimal_filter() ),
-			'subtotal' => round( $total, give_currency_decimal_filter() ),
+			'price'    => round($total, give_currency_decimal_filter()),
+			'subtotal' => round($total, give_currency_decimal_filter()),
 			'fees'     => $args['fees'],
 			'price_id' => $args['price_id'],
 			'action'   => 'add',
@@ -1095,7 +1095,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1112,7 +1112,7 @@ 
 	 *
 	 * @return bool           If the item was removed or not
 	 */
-	public function remove_donation( $form_id, $args = array() ) {
+	public function remove_donation($form_id, $args = array()) {
 
 		// Set some defaults.
 		$defaults = array(
@@ -1120,12 +1120,12 @@ 
 			'price'    => false,
 			'price_id' => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$form = new Give_Donate_Form( $form_id );
+		$form = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a valid give donation form.
-		if ( ! $form || $form->post_type !== 'give_forms' ) {
+		if ( ! $form || $form->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -1138,7 +1138,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1154,7 +1154,7 @@ 
 	 *
 	 * @return bool          If the fee was added
 	 */
-	public function add_fee( $args, $global = true ) {
+	public function add_fee($args, $global = true) {
 
 		$default_args = array(
 			'label'    => '',
@@ -1164,15 +1164,15 @@ 
 			'price_id' => 0,
 		);
 
-		$fee          = wp_parse_args( $args, $default_args );
+		$fee          = wp_parse_args($args, $default_args);
 		$this->fees[] = $fee;
 
 		$added_fee               = $fee;
 		$added_fee['action']     = 'add';
 		$this->pending['fees'][] = $added_fee;
-		reset( $this->fees );
+		reset($this->fees);
 
-		$this->increase_fees( $fee['amount'] );
+		$this->increase_fees($fee['amount']);
 
 		return true;
 	}
@@ -1187,11 +1187,11 @@ 
 	 *
 	 * @return bool     If the fee was removed successfully
 	 */
-	public function remove_fee( $key ) {
+	public function remove_fee($key) {
 		$removed = false;
 
-		if ( is_numeric( $key ) ) {
-			$removed = $this->remove_fee_by( 'index', $key );
+		if (is_numeric($key)) {
+			$removed = $this->remove_fee_by('index', $key);
 		}
 
 		return $removed;
@@ -1210,55 +1210,55 @@ 
 	 *
 	 * @return boolean            If the item is removed
 	 */
-	public function remove_fee_by( $key, $value, $global = false ) {
+	public function remove_fee_by($key, $value, $global = false) {
 
-		$allowed_fee_keys = apply_filters( 'give_payment_fee_keys', array(
+		$allowed_fee_keys = apply_filters('give_payment_fee_keys', array(
 			'index',
 			'label',
 			'amount',
 			'type',
-		) );
+		));
 
-		if ( ! in_array( $key, $allowed_fee_keys ) ) {
+		if ( ! in_array($key, $allowed_fee_keys)) {
 			return false;
 		}
 
 		$removed = false;
-		if ( 'index' === $key && array_key_exists( $value, $this->fees ) ) {
+		if ('index' === $key && array_key_exists($value, $this->fees)) {
 
-			$removed_fee             = $this->fees[ $value ];
+			$removed_fee             = $this->fees[$value];
 			$removed_fee['action']   = 'remove';
 			$this->pending['fees'][] = $removed_fee;
 
-			$this->decrease_fees( $removed_fee['amount'] );
+			$this->decrease_fees($removed_fee['amount']);
 
-			unset( $this->fees[ $value ] );
+			unset($this->fees[$value]);
 			$removed = true;
 
-		} elseif ( 'index' !== $key ) {
+		} elseif ('index' !== $key) {
 
-			foreach ( $this->fees as $index => $fee ) {
+			foreach ($this->fees as $index => $fee) {
 
-				if ( isset( $fee[ $key ] ) && $fee[ $key ] == $value ) {
+				if (isset($fee[$key]) && $fee[$key] == $value) {
 
 					$removed_fee             = $fee;
 					$removed_fee['action']   = 'remove';
 					$this->pending['fees'][] = $removed_fee;
 
-					$this->decrease_fees( $removed_fee['amount'] );
+					$this->decrease_fees($removed_fee['amount']);
 
-					unset( $this->fees[ $index ] );
+					unset($this->fees[$index]);
 					$removed = true;
 
-					if ( false === $global ) {
+					if (false === $global) {
 						break;
 					}
 				}
 			}
 		}
 
-		if ( true === $removed ) {
-			$this->fees = array_values( $this->fees );
+		if (true === $removed) {
+			$this->fees = array_values($this->fees);
 		}
 
 		return $removed;
@@ -1274,14 +1274,14 @@ 
 	 *
 	 * @return array        The Fees for the type specified
 	 */
-	public function get_fees( $type = 'all' ) {
+	public function get_fees($type = 'all') {
 		$fees = array();
 
-		if ( ! empty( $this->fees ) && is_array( $this->fees ) ) {
+		if ( ! empty($this->fees) && is_array($this->fees)) {
 
-			foreach ( $this->fees as $fee_id => $fee ) {
+			foreach ($this->fees as $fee_id => $fee) {
 
-				if ( 'all' != $type && ! empty( $fee['type'] ) && $type != $fee['type'] ) {
+				if ('all' != $type && ! empty($fee['type']) && $type != $fee['type']) {
 					continue;
 				}
 
@@ -1291,7 +1291,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_get_payment_fees', $fees, $this->ID, $this );
+		return apply_filters('give_get_payment_fees', $fees, $this->ID, $this);
 	}
 
 	/**
@@ -1304,13 +1304,13 @@ 
 	 *
 	 * @return void
 	 */
-	public function add_note( $note = false ) {
+	public function add_note($note = false) {
 		// Bail if no note specified.
-		if ( ! $note ) {
+		if ( ! $note) {
 			return false;
 		}
 
-		give_insert_payment_note( $this->ID, $note );
+		give_insert_payment_note($this->ID, $note);
 	}
 
 	/**
@@ -1323,8 +1323,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function increase_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
 		$this->recalculate_total();
@@ -1340,11 +1340,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function decrease_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1361,8 +1361,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_fees( $amount = 0.00 ) {
-		$amount           = (float) $amount;
+	private function increase_fees($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->fees_total += $amount;
 
 		$this->recalculate_total();
@@ -1378,11 +1378,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_fees( $amount = 0.00 ) {
-		$amount           = (float) $amount;
+	private function decrease_fees($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->fees_total -= $amount;
 
-		if ( $this->fees_total < 0 ) {
+		if ($this->fees_total < 0) {
 			$this->fees_total = 0;
 		}
 
@@ -1411,24 +1411,24 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated.
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		// standardize the 'complete(d)' status.
-		if ( $status == 'completed' || $status == 'complete' ) {
+		if ($status == 'completed' || $status == 'complete') {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes.
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
-		if ( $do_change ) {
+		if ($do_change) {
 
 			/**
 			 * Fires before changing payment status.
@@ -1439,21 +1439,21 @@ 
 			 * @param string $status The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' ),
+				'edit_date'   => current_time('mysql'),
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions.
-			switch ( $status ) {
+			switch ($status) {
 				case 'refunded':
 					$this->process_refund();
 					break;
@@ -1480,7 +1480,7 @@ 
 			 * @param string $status The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		}// End if().
 
@@ -1515,33 +1515,33 @@ 
 	 *
 	 * @return mixed             The value from the post meta
 	 */
-	public function get_meta( $meta_key = '_give_payment_meta', $single = true ) {
+	public function get_meta($meta_key = '_give_payment_meta', $single = true) {
 
-		$meta = give_get_meta( $this->ID, $meta_key, $single );
+		$meta = give_get_meta($this->ID, $meta_key, $single);
 
-		if ( $meta_key === '_give_payment_meta' ) {
+		if ($meta_key === '_give_payment_meta') {
 			$meta = (array) $meta;
 
-			if ( empty( $meta['key'] ) ) {
+			if (empty($meta['key'])) {
 				$meta['key'] = $this->setup_payment_key();
 			}
 
-			if ( empty( $meta['form_title'] ) ) {
+			if (empty($meta['form_title'])) {
 				$meta['form_title'] = $this->setup_form_title();
 			}
 
-			if ( empty( $meta['email'] ) ) {
+			if (empty($meta['email'])) {
 				$meta['email'] = $this->setup_email();
 			}
 
-			if ( empty( $meta['date'] ) ) {
-				$meta['date'] = get_post_field( 'post_date', $this->ID );
+			if (empty($meta['date'])) {
+				$meta['date'] = get_post_field('post_date', $this->ID);
 			}
 		}
 
-		$meta = apply_filters( "give_get_payment_meta_{$meta_key}", $meta, $this->ID );
+		$meta = apply_filters("give_get_payment_meta_{$meta_key}", $meta, $this->ID);
 
-		return apply_filters( 'give_get_payment_meta', $meta, $this->ID, $meta_key );
+		return apply_filters('give_get_payment_meta', $meta, $this->ID, $meta_key);
 	}
 
 	/**
@@ -1556,23 +1556,23 @@ 
 	 *
 	 * @return int|bool           Meta ID if the key didn't exist, true on successful update, false on failure
 	 */
-	public function update_meta( $meta_key = '', $meta_value = '', $prev_value = '' ) {
-		if ( empty( $meta_key ) ) {
+	public function update_meta($meta_key = '', $meta_value = '', $prev_value = '') {
+		if (empty($meta_key)) {
 			return false;
 		}
 
-		if ( $meta_key == 'key' || $meta_key == 'date' ) {
+		if ($meta_key == 'key' || $meta_key == 'date') {
 
 			$current_meta              = $this->get_meta();
-			$current_meta[ $meta_key ] = $meta_value;
+			$current_meta[$meta_key] = $meta_value;
 
 			$meta_key   = '_give_payment_meta';
 			$meta_value = $current_meta;
 
-		} elseif ( $meta_key == 'email' || $meta_key == '_give_payment_user_email' ) {
+		} elseif ($meta_key == 'email' || $meta_key == '_give_payment_user_email') {
 
-			$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
-			give_update_meta( $this->ID, '_give_payment_user_email', $meta_value );
+			$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
+			give_update_meta($this->ID, '_give_payment_user_email', $meta_value);
 
 			$current_meta                       = $this->get_meta();
 			$current_meta['user_info']['email'] = $meta_value;
@@ -1582,9 +1582,9 @@ 
 
 		}
 
-		$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
+		$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
 
-		return give_update_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return give_update_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1599,14 +1599,14 @@ 
 		$process_refund = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'refunded' != $this->status ) {
+		if ('publish' != $this->old_status || 'refunded' != $this->status) {
 			$process_refund = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_refund = apply_filters( 'give_should_process_refund', $process_refund, $this );
+		$process_refund = apply_filters('give_should_process_refund', $process_refund, $this);
 
-		if ( false === $process_refund ) {
+		if (false === $process_refund) {
 			return;
 		}
 
@@ -1617,13 +1617,13 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_pre_refund_payment', $this );
+		do_action('give_pre_refund_payment', $this);
 
-		$decrease_earnings       = apply_filters( 'give_decrease_store_earnings_on_refund', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_refund', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_refund', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_store_earnings_on_refund', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_refund', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_refund', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1636,7 +1636,7 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1663,26 +1663,26 @@ 
 		$process_pending = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'pending' != $this->status ) {
+		if ('publish' != $this->old_status || 'pending' != $this->status) {
 			$process_pending = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_pending = apply_filters( 'give_should_process_pending', $process_pending, $this );
+		$process_pending = apply_filters('give_should_process_pending', $process_pending, $this);
 
-		if ( false === $process_pending ) {
+		if (false === $process_pending) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_pending', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_pending', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_pending', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_pending', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_pending', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_pending', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1700,26 +1700,26 @@ 
 		$process_cancelled = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
+		if ('publish' != $this->old_status || 'cancelled' != $this->status) {
 			$process_cancelled = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
+		$process_cancelled = apply_filters('give_should_process_cancelled', $process_cancelled, $this);
 
-		if ( false === $process_cancelled ) {
+		if (false === $process_cancelled) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_cancelled', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_cancelled', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_cancelled', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_cancelled', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_cancelled', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_cancelled', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1735,26 +1735,26 @@ 
 		$process_revoked = true;
 
 		// If the payment was not in publish, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
+		if ('publish' != $this->old_status || 'revoked' != $this->status) {
 			$process_revoked = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
+		$process_revoked = apply_filters('give_should_process_revoked', $process_revoked, $this);
 
-		if ( false === $process_revoked ) {
+		if (false === $process_revoked) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_revoked', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_revoked', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_revoked', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_revoked', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_revoked', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_revoked', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1772,25 +1772,25 @@ 
 	 *
 	 * @return void
 	 */
-	private function maybe_alter_stats( $alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count ) {
+	private function maybe_alter_stats($alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count) {
 
-		give_undo_purchase( false, $this->ID );
+		give_undo_purchase(false, $this->ID);
 
 		// Decrease store earnings.
-		if ( true === $alter_store_earnings ) {
-			give_decrease_total_earnings( $this->total );
+		if (true === $alter_store_earnings) {
+			give_decrease_total_earnings($this->total);
 		}
 
 		// Decrement the stats for the customer.
-		if ( ! empty( $this->customer_id ) ) {
+		if ( ! empty($this->customer_id)) {
 
-			$customer = new Give_Customer( $this->customer_id );
+			$customer = new Give_Customer($this->customer_id);
 
-			if ( true === $alter_customer_value ) {
-				$customer->decrease_value( $this->total );
+			if (true === $alter_customer_value) {
+				$customer->decrease_value($this->total);
 			}
 
-			if ( true === $alter_customer_purchase_count ) {
+			if (true === $alter_customer_purchase_count) {
 				$customer->decrease_purchase_count();
 			}
 		}
@@ -1839,13 +1839,13 @@ 
 	 * @return string The date the payment was completed
 	 */
 	private function setup_completed_date() {
-		$payment = get_post( $this->ID );
+		$payment = get_post($this->ID);
 
-		if ( 'pending' == $payment->post_status || 'preapproved' == $payment->post_status ) {
+		if ('pending' == $payment->post_status || 'preapproved' == $payment->post_status) {
 			return false; // This payment was never completed.
 		}
 
-		$date = ( $date = $this->get_meta( '_give_completed_date', true ) ) ? $date : $payment->modified_date;
+		$date = ($date = $this->get_meta('_give_completed_date', true)) ? $date : $payment->modified_date;
 
 		return $date;
 	}
@@ -1859,7 +1859,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1871,18 +1871,18 @@ 
 	 * @return float The payment total
 	 */
 	private function setup_total() {
-		$amount = $this->get_meta( '_give_payment_total', true );
+		$amount = $this->get_meta('_give_payment_total', true);
 
-		if ( empty( $amount ) && '0.00' != $amount ) {
-			$meta = $this->get_meta( '_give_payment_meta', true );
-			$meta = maybe_unserialize( $meta );
+		if (empty($amount) && '0.00' != $amount) {
+			$meta = $this->get_meta('_give_payment_meta', true);
+			$meta = maybe_unserialize($meta);
 
-			if ( isset( $meta['amount'] ) ) {
+			if (isset($meta['amount'])) {
 				$amount = $meta['amount'];
 			}
 		}
 
-		return round( floatval( $amount ), give_currency_decimal_filter() );
+		return round(floatval($amount), give_currency_decimal_filter());
 	}
 
 	/**
@@ -1910,9 +1910,9 @@ 
 	private function setup_fees_total() {
 		$fees_total = (float) 0.00;
 
-		$payment_fees = isset( $this->payment_meta['fees'] ) ? $this->payment_meta['fees'] : array();
-		if ( ! empty( $payment_fees ) ) {
-			foreach ( $payment_fees as $fee ) {
+		$payment_fees = isset($this->payment_meta['fees']) ? $this->payment_meta['fees'] : array();
+		if ( ! empty($payment_fees)) {
+			foreach ($payment_fees as $fee) {
 				$fees_total += (float) $fee['amount'];
 			}
 		}
@@ -1930,7 +1930,7 @@ 
 	 * @return string The currency for the payment
 	 */
 	private function setup_currency() {
-		$currency = isset( $this->payment_meta['currency'] ) ? $this->payment_meta['currency'] : apply_filters( 'give_payment_currency_default', give_get_currency(), $this );
+		$currency = isset($this->payment_meta['currency']) ? $this->payment_meta['currency'] : apply_filters('give_payment_currency_default', give_get_currency(), $this);
 
 		return $currency;
 	}
@@ -1944,7 +1944,7 @@ 
 	 * @return array The Fees
 	 */
 	private function setup_fees() {
-		$payment_fees = isset( $this->payment_meta['fees'] ) ? $this->payment_meta['fees'] : array();
+		$payment_fees = isset($this->payment_meta['fees']) ? $this->payment_meta['fees'] : array();
 
 		return $payment_fees;
 	}
@@ -1958,7 +1958,7 @@ 
 	 * @return string The gateway
 	 */
 	private function setup_gateway() {
-		$gateway = $this->get_meta( '_give_payment_gateway', true );
+		$gateway = $this->get_meta('_give_payment_gateway', true);
 
 		return $gateway;
 	}
@@ -1972,11 +1972,11 @@ 
 	 * @return string The donation ID
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) ) {
+		if (empty($transaction_id)) {
 			$gateway        = $this->gateway;
-			$transaction_id = apply_filters( "give_get_payment_transaction_id-{$gateway}", $this->ID );
+			$transaction_id = apply_filters("give_get_payment_transaction_id-{$gateway}", $this->ID);
 		}
 
 		return $transaction_id;
@@ -1991,7 +1991,7 @@ 
 	 * @return string The IP address for the payment
 	 */
 	private function setup_ip() {
-		$ip = $this->get_meta( '_give_payment_user_ip', true );
+		$ip = $this->get_meta('_give_payment_user_ip', true);
 
 		return $ip;
 	}
@@ -2005,7 +2005,7 @@ 
 	 * @return int The Customer ID
 	 */
 	private function setup_customer_id() {
-		$customer_id = $this->get_meta( '_give_payment_customer_id', true );
+		$customer_id = $this->get_meta('_give_payment_customer_id', true);
 
 		return $customer_id;
 	}
@@ -2019,7 +2019,7 @@ 
 	 * @return int The User ID
 	 */
 	private function setup_user_id() {
-		$user_id = $this->get_meta( '_give_payment_user_id', true );
+		$user_id = $this->get_meta('_give_payment_user_id', true);
 
 		return $user_id;
 	}
@@ -2033,10 +2033,10 @@ 
 	 * @return string The email address for the payment
 	 */
 	private function setup_email() {
-		$email = $this->get_meta( '_give_payment_user_email', true );
+		$email = $this->get_meta('_give_payment_user_email', true);
 
-		if ( empty( $email ) ) {
-			$email = Give()->customers->get_column( 'email', $this->customer_id );
+		if (empty($email)) {
+			$email = Give()->customers->get_column('email', $this->customer_id);
 		}
 
 		return $email;
@@ -2056,15 +2056,15 @@ 
 			'last_name'  => $this->last_name,
 		);
 
-		$user_info = isset( $this->payment_meta['user_info'] ) ? maybe_unserialize( $this->payment_meta['user_info'] ) : array();
-		$user_info = wp_parse_args( $user_info, $defaults );
+		$user_info = isset($this->payment_meta['user_info']) ? maybe_unserialize($this->payment_meta['user_info']) : array();
+		$user_info = wp_parse_args($user_info, $defaults);
 
-		if ( empty( $user_info ) ) {
+		if (empty($user_info)) {
 			// Get the customer, but only if it's been created.
-			$customer = new Give_Customer( $this->customer_id );
+			$customer = new Give_Customer($this->customer_id);
 
-			if ( $customer->id > 0 ) {
-				$name      = explode( ' ', $customer->name, 2 );
+			if ($customer->id > 0) {
+				$name      = explode(' ', $customer->name, 2);
 				$user_info = array(
 					'first_name' => $name[0],
 					'last_name'  => $name[1],
@@ -2074,29 +2074,29 @@ 
 			}
 		} else {
 			// Get the customer, but only if it's been created.
-			$customer = new Give_Customer( $this->customer_id );
-			if ( $customer->id > 0 ) {
-				foreach ( $user_info as $key => $value ) {
-					if ( ! empty( $value ) ) {
+			$customer = new Give_Customer($this->customer_id);
+			if ($customer->id > 0) {
+				foreach ($user_info as $key => $value) {
+					if ( ! empty($value)) {
 						continue;
 					}
 
-					switch ( $key ) {
+					switch ($key) {
 						case 'first_name':
-							$name = explode( ' ', $customer->name, 2 );
+							$name = explode(' ', $customer->name, 2);
 
-							$user_info[ $key ] = $name[0];
+							$user_info[$key] = $name[0];
 							break;
 
 						case 'last_name':
-							$name      = explode( ' ', $customer->name, 2 );
-							$last_name = ! empty( $name[1] ) ? $name[1] : '';
+							$name      = explode(' ', $customer->name, 2);
+							$last_name = ! empty($name[1]) ? $name[1] : '';
 
-							$user_info[ $key ] = $last_name;
+							$user_info[$key] = $last_name;
 							break;
 
 						case 'email':
-							$user_info[ $key ] = $customer->email;
+							$user_info[$key] = $customer->email;
 							break;
 					}
 				}
@@ -2117,7 +2117,7 @@ 
 	 */
 	private function setup_address() {
 
-		$address = ! empty( $this->payment_meta['user_info']['address'] ) ? $this->payment_meta['user_info']['address'] : array(
+		$address = ! empty($this->payment_meta['user_info']['address']) ? $this->payment_meta['user_info']['address'] : array(
 			'line1'   => '',
 			'line2'   => '',
 			'city'    => '',
@@ -2139,7 +2139,7 @@ 
 	 */
 	private function setup_form_title() {
 
-		$form_id = $this->get_meta( '_give_payment_form_title', true );
+		$form_id = $this->get_meta('_give_payment_form_title', true);
 
 		return $form_id;
 	}
@@ -2154,7 +2154,7 @@ 
 	 */
 	private function setup_form_id() {
 
-		$form_id = $this->get_meta( '_give_payment_form_id', true );
+		$form_id = $this->get_meta('_give_payment_form_id', true);
 
 		return $form_id;
 	}
@@ -2168,7 +2168,7 @@ 
 	 * @return int The Form Price ID
 	 */
 	private function setup_price_id() {
-		$price_id = $this->get_meta( '_give_payment_price_id', true );
+		$price_id = $this->get_meta('_give_payment_price_id', true);
 
 		return $price_id;
 	}
@@ -2182,7 +2182,7 @@ 
 	 * @return string The Payment Key
 	 */
 	private function setup_payment_key() {
-		$key = $this->get_meta( '_give_payment_purchase_key', true );
+		$key = $this->get_meta('_give_payment_purchase_key', true);
 
 		return $key;
 	}
@@ -2198,11 +2198,11 @@ 
 	private function setup_payment_number() {
 		$number = $this->ID;
 
-		if ( give_get_option( 'enable_sequential' ) ) {
+		if (give_get_option('enable_sequential')) {
 
-			$number = $this->get_meta( '_give_payment_number', true );
+			$number = $this->get_meta('_give_payment_number', true);
 
-			if ( ! $number ) {
+			if ( ! $number) {
 
 				$number = $this->ID;
 
@@ -2220,7 +2220,7 @@ 
 	 * @return array The payment object as an array
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -2233,7 +2233,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -2245,7 +2245,7 @@ 
 	 * @return string Date payment was completed
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_payment_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_payment_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -2257,7 +2257,7 @@ 
 	 * @return float Payment subtotal
 	 */
 	private function get_subtotal() {
-		return apply_filters( 'give_get_payment_subtotal', $this->subtotal, $this->ID, $this );
+		return apply_filters('give_get_payment_subtotal', $this->subtotal, $this->ID, $this);
 	}
 
 	/**
@@ -2269,7 +2269,7 @@ 
 	 * @return string Payment currency code
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_payment_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_payment_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -2281,7 +2281,7 @@ 
 	 * @return string Gateway used
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_payment_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_payment_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -2293,7 +2293,7 @@ 
 	 * @return string Donation ID from merchant processor
 	 */
 	private function get_transaction_id() {
-		return apply_filters( 'give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this );
+		return apply_filters('give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this);
 	}
 
 	/**
@@ -2305,7 +2305,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_payment_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_payment_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -2317,7 +2317,7 @@ 
 	 * @return int Payment customer ID
 	 */
 	private function get_customer_id() {
-		return apply_filters( 'give_payment_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_payment_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -2329,7 +2329,7 @@ 
 	 * @return int Payment user ID
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_payment_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_payment_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -2341,7 +2341,7 @@ 
 	 * @return string Payment customer email
 	 */
 	private function get_email() {
-		return apply_filters( 'give_payment_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_payment_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2353,7 +2353,7 @@ 
 	 * @return array Payment user info
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_payment_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_payment_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2365,7 +2365,7 @@ 
 	 * @return array Payment billing address
 	 */
 	private function get_address() {
-		return apply_filters( 'give_payment_address', $this->address, $this->ID, $this );
+		return apply_filters('give_payment_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2377,7 +2377,7 @@ 
 	 * @return string Payment key
 	 */
 	private function get_key() {
-		return apply_filters( 'give_payment_key', $this->key, $this->ID, $this );
+		return apply_filters('give_payment_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2389,7 +2389,7 @@ 
 	 * @return string Payment form id
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_payment_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_payment_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2401,7 +2401,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 
 }

includes/class-give-cron.php

Spacing +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -34,8 +34,8 @@ 
 	 * @see    Give_Cron::weekly_events()
 	 */
 	public function __construct() {
-		add_filter( 'cron_schedules', array( $this, 'add_schedules' ) );
-		add_action( 'wp', array( $this, 'schedule_Events' ) );
+		add_filter('cron_schedules', array($this, 'add_schedules'));
+		add_action('wp', array($this, 'schedule_Events'));
 	}
 
 	/**
@@ -48,11 +48,11 @@ 
 	 *
 	 * @return array            An array of non-default cron schedules.
 	 */
-	public function add_schedules( $schedules = array() ) {
+	public function add_schedules($schedules = array()) {
 		// Adds once weekly to the existing schedules.
 		$schedules['weekly'] = array(
 			'interval' => 604800,
-			'display'  => esc_html__( 'Once Weekly', 'give' )
+			'display'  => esc_html__('Once Weekly', 'give')
 		);
 
 		return $schedules;
@@ -80,8 +80,8 @@ 
 	 * @return void
 	 */
 	private function weekly_events() {
-		if ( ! wp_next_scheduled( 'give_weekly_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'weekly', 'give_weekly_scheduled_events' );
+		if ( ! wp_next_scheduled('give_weekly_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'weekly', 'give_weekly_scheduled_events');
 		}
 	}
 
@@ -94,8 +94,8 @@ 
 	 * @return void
 	 */
 	private function daily_events() {
-		if ( ! wp_next_scheduled( 'give_daily_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'daily', 'give_daily_scheduled_events' );
+		if ( ! wp_next_scheduled('give_daily_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'daily', 'give_daily_scheduled_events');
 		}
 	}
 

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);
 		}
 	}
 

includes/admin/shortcodes/shortcode-give-profile-editor.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,9 +21,9 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['label'] = esc_html__( 'Profile Editor', 'give' );
+		$this->shortcode['label'] = esc_html__('Profile Editor', 'give');
 
-		parent::__construct( 'give_profile_editor' );
+		parent::__construct('give_profile_editor');
 	}
 }
 

includes/admin/shortcodes/shortcode-give-receipt.php

Spacing +23 added lines, -23 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,10 +24,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html__( 'Donation Receipt', 'give' );
-		$this->shortcode['label'] = esc_html__( 'Donation Receipt', 'give' );
+		$this->shortcode['title'] = esc_html__('Donation Receipt', 'give');
+		$this->shortcode['label'] = esc_html__('Donation Receipt', 'give');
 
-		parent::__construct( 'give_receipt' );
+		parent::__construct('give_receipt');
 	}
 
 	/**
@@ -40,60 +40,60 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong">%s</p>', esc_html__('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'price',
-				'label'   => esc_html__( 'Show Donation Amount:', 'give' ),
+				'label'   => esc_html__('Show Donation Amount:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'donor',
-				'label'   => esc_html__( 'Show Donor Name:', 'give' ),
+				'label'   => esc_html__('Show Donor Name:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'date',
-				'label'   => esc_html__( 'Show Date:', 'give' ),
+				'label'   => esc_html__('Show Date:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_key',
-				'label'   => esc_html__( 'Show Payment Key:', 'give' ),
+				'label'   => esc_html__('Show Payment Key:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_method',
-				'label'   => esc_html__( 'Show Payment Method:', 'give' ),
+				'label'   => esc_html__('Show Payment Method:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_id',
-				'label'   => esc_html__( 'Show Payment ID:', 'give' ),
+				'label'   => esc_html__('Show Payment ID:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 		);

includes/admin/shortcodes/shortcode-give-register.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,10 +21,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html__( 'Register', 'give' );
-		$this->shortcode['label'] = esc_html__( 'Register', 'give' );
+		$this->shortcode['title'] = esc_html__('Register', 'give');
+		$this->shortcode['label'] = esc_html__('Register', 'give');
 
-		parent::__construct( 'give_register' );
+		parent::__construct('give_register');
 	}
 
 	/**
@@ -37,13 +37,13 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="no-margin">%s</p>', esc_html__( 'Redirect URL (optional):', 'give' ) ),
+				'html' => sprintf('<p class="no-margin">%s</p>', esc_html__('Redirect URL (optional):', 'give')),
 			),
 			array(
 				'type'     => 'textbox',
 				'name'     => 'redirect',
 				'minWidth' => 320,
-				'tooltip'  => esc_attr__( 'Enter an URL here to redirect to after registering.', 'give' ),
+				'tooltip'  => esc_attr__('Enter an URL here to redirect to after registering.', 'give'),
 			),
 		);
 	}

includes/admin/shortcodes/shortcode-give-donation-history.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,9 +21,9 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['label'] = esc_html__( 'Donation History', 'give' );
+		$this->shortcode['label'] = esc_html__('Donation History', 'give');
 
-		parent::__construct( 'donation_history' );
+		parent::__construct('donation_history');
 	}
 }
 

includes/admin/customers/customer-functions.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views
  */
-function give_register_default_customer_views( $views ) {
+function give_register_default_customer_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_customers_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_customer_notes_view'
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_customer_views', 'give_register_default_customer_views', 1, 1 );
+add_filter('give_customer_views', 'give_register_default_customer_views', 1, 1);
 
 /**
  * Register a tab for the single customer view
@@ -46,17 +46,17 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_customer_tabs( $tabs ) {
+function give_register_default_customer_tabs($tabs) {
 
 	$default_tabs = array(
-		'overview' => array( 'dashicon' => 'dashicons-admin-users', 'title' => esc_html__( 'Donor Profile', 'give' ) ),
-		'notes'    => array( 'dashicon' => 'dashicons-admin-comments', 'title' => esc_html__( 'Donor Notes', 'give' ) )
+		'overview' => array('dashicon' => 'dashicons-admin-users', 'title' => esc_html__('Donor Profile', 'give')),
+		'notes'    => array('dashicon' => 'dashicons-admin-comments', 'title' => esc_html__('Donor Notes', 'give'))
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_customer_tabs', 'give_register_default_customer_tabs', 1, 1 );
+add_filter('give_customer_tabs', 'give_register_default_customer_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom
@@ -67,11 +67,11 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom
  */
-function give_register_delete_customer_tab( $tabs ) {
+function give_register_delete_customer_tab($tabs) {
 
-	$tabs['delete'] = array( 'dashicon' => 'dashicons-trash', 'title' => esc_html__( 'Delete Donor', 'give' ) );
+	$tabs['delete'] = array('dashicon' => 'dashicons-trash', 'title' => esc_html__('Delete Donor', 'give'));
 
 	return $tabs;
 }
 
-add_filter( 'give_customer_tabs', 'give_register_delete_customer_tab', PHP_INT_MAX, 1 );
+add_filter('give_customer_tabs', 'give_register_delete_customer_tab', PHP_INT_MAX, 1);