WordImpress / Give

includes/payments/class-payments-query.php

Spacing +112 added lines, -112 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,10 +54,10 @@ 
 	 *
 	 * @param $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments', // Use 'posts' to get standard post objects
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -77,7 +77,7 @@ 
 			'give_forms'      => null
 		);
 
-		$this->args = wp_parse_args( $args, $defaults );
+		$this->args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -88,11 +88,11 @@ 
 	 * @access public
 	 * @since  1.0
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -102,8 +102,8 @@ 
 	 * @access public
 	 * @since  1.0
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -115,19 +115,19 @@ 
 	 */
 	public function init() {
 
-		add_action( 'give_pre_get_payments', array( $this, 'date_filter_pre' ) );
-		add_action( 'give_post_get_payments', array( $this, 'date_filter_post' ) );
-
-		add_action( 'give_pre_get_payments', array( $this, 'orderby' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'status' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'month' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'per_page' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'page' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'user' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'search' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'mode' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'children' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'give_forms' ) );
+		add_action('give_pre_get_payments', array($this, 'date_filter_pre'));
+		add_action('give_post_get_payments', array($this, 'date_filter_post'));
+
+		add_action('give_pre_get_payments', array($this, 'orderby'));
+		add_action('give_pre_get_payments', array($this, 'status'));
+		add_action('give_pre_get_payments', array($this, 'month'));
+		add_action('give_pre_get_payments', array($this, 'per_page'));
+		add_action('give_pre_get_payments', array($this, 'page'));
+		add_action('give_pre_get_payments', array($this, 'user'));
+		add_action('give_pre_get_payments', array($this, 'search'));
+		add_action('give_pre_get_payments', array($this, 'mode'));
+		add_action('give_pre_get_payments', array($this, 'children'));
+		add_action('give_pre_get_payments', array($this, 'give_forms'));
 	}
 
 	/**
@@ -143,33 +143,33 @@ 
 	 */
 	public function get_payments() {
 
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 
-		$query = new WP_Query( $this->args );
+		$query = new WP_Query($this->args);
 
 		$custom_output = array(
 			'payments',
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
-			while ( $query->have_posts() ) {
+		if ($query->have_posts()) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 		}
 
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 
 		return $this->payments;
 	}
@@ -182,13 +182,13 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -200,11 +200,11 @@ 
 	 * @return void
 	 */
 	public function date_filter_post() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -215,12 +215,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset ( $this->args['status'] ) ) {
+		if ( ! isset ($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -231,12 +231,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset ( $this->args['page'] ) ) {
+		if ( ! isset ($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -248,17 +248,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -269,12 +269,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset ( $this->args['month'] ) ) {
+		if ( ! isset ($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -285,13 +285,13 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount' :
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 			default :
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -304,20 +304,20 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_user_id';
 		} else {
 			$user_key = '_give_payment_user_email';
 		}
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => $user_key,
 			'value' => $this->args['user']
-		) );
+		));
 	}
 
 	/**
@@ -329,33 +329,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_user_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -364,19 +364,19 @@ 
 				'compare' => 'LIKE'
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_user_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) )
+				'value' => trim(str_replace('user:', '', strtolower($search)))
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			if ( give_get_option( 'enable_sequential' ) ) {
+			if (give_get_option('enable_sequential')) {
 
 				$search_meta = array(
 					'key'     => '_give_payment_number',
@@ -384,19 +384,19 @@ 
 					'compare' => 'LIKE'
 				);
 
-				$this->__set( 'meta_query', $search_meta );
+				$this->__set('meta_query', $search_meta);
 
 				$this->args['meta_query']['relation'] = 'OR';
 
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
 		} elseif (
-			give_get_option( 'enable_sequential' ) &&
+			give_get_option('enable_sequential') &&
 			(
-				false !== strpos( $search, give_get_option( 'sequential_prefix' ) ) ||
-				false !== strpos( $search, give_get_option( 'sequential_postfix' ) )
+				false !== strpos($search, give_get_option('sequential_prefix')) ||
+				false !== strpos($search, give_get_option('sequential_postfix'))
 			)
 		) {
 
@@ -406,30 +406,30 @@ 
 				'compare' => 'LIKE'
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
 
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -443,16 +443,16 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => '_give_payment_mode',
 			'value' => $this->args['mode']
-		) );
+		));
 	}
 
 	/**
@@ -463,10 +463,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -478,7 +478,7 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
@@ -487,7 +487,7 @@ 
 		$args = array(
 			'post_parent'            => $this->args['give_forms'],
 			'log_type'               => 'sale',
-			'post_status'            => array( 'publish' ),
+			'post_status'            => array('publish'),
 			'nopaging'               => true,
 			'no_found_rows'          => true,
 			'update_post_term_cache' => false,
@@ -496,31 +496,31 @@ 
 			'fields'                 => 'ids'
 		);
 
-		if ( is_array( $this->args['give_forms'] ) ) {
-			unset( $args['post_parent'] );
+		if (is_array($this->args['give_forms'])) {
+			unset($args['post_parent']);
 			$args['post_parent__in'] = $this->args['give_forms'];
 		}
 
-		$sales = $give_logs->get_connected_logs( $args );
+		$sales = $give_logs->get_connected_logs($args);
 
-		if ( ! empty( $sales ) ) {
+		if ( ! empty($sales)) {
 
 			$payments = array();
 
-			foreach ( $sales as $sale ) {
-				$payments[] = get_post_meta( $sale, '_give_log_payment_id', true );
+			foreach ($sales as $sale) {
+				$payments[] = get_post_meta($sale, '_give_log_payment_id', true);
 			}
 
-			$this->__set( 'post__in', $payments );
+			$this->__set('post__in', $payments);
 
 		} else {
 
 			// Set post_parent to something crazy so it doesn't find anything
-			$this->__set( 'post_parent', 999999999999999 );
+			$this->__set('post_parent', 999999999999999);
 
 		}
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 }

includes/payments/actions.php

Spacing +53 added lines, -53 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,22 +28,22 @@ 
  *
  * @return void
  */
-function give_complete_purchase( $payment_id, $new_status, $old_status ) {
+function give_complete_purchase($payment_id, $new_status, $old_status) {
 
 	// Make sure that payments are only completed once
-	if ( $old_status == 'publish' || $old_status == 'complete' ) {
+	if ($old_status == 'publish' || $old_status == 'complete') {
 		return;
 	}
 
 	// Make sure the payment completion is only processed when new status is complete
-	if ( $new_status != 'publish' && $new_status != 'complete' ) {
+	if ($new_status != 'publish' && $new_status != 'complete') {
 		return;
 	}
 
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$creation_date  = get_post_field( 'post_date', $payment_id, 'raw' );
+	$creation_date  = get_post_field('post_date', $payment_id, 'raw');
 	$payment_meta   = $payment->payment_meta;
 	$completed_date = $payment->completed_date;
 	$user_info      = $payment->user_info;
@@ -52,46 +52,46 @@ 
 	$price_id       = $payment->price_id;
 	$form_id        = $payment->form_id;
 
-	do_action( 'give_pre_complete_purchase', $payment_id );
+	do_action('give_pre_complete_purchase', $payment_id);
 
 	// Ensure these actions only run once, ever
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
-		give_record_sale_in_log( $form_id, $payment_id, $price_id, $creation_date );
-		do_action( 'give_complete_form_donation', $form_id, $payment_id, $payment_meta );
+		give_record_sale_in_log($form_id, $payment_id, $price_id, $creation_date);
+		do_action('give_complete_form_donation', $form_id, $payment_id, $payment_meta);
 
 	}
 
 	// Increase the earnings for this form ID
-	give_increase_earnings( $form_id, $amount );
-	give_increase_purchase_count( $form_id );
+	give_increase_earnings($form_id, $amount);
+	give_increase_purchase_count($form_id);
 
 	// Clear the total earnings cache
-	delete_transient( 'give_earnings_total' );
+	delete_transient('give_earnings_total');
 	// Clear the This Month earnings (this_monththis_month is NOT a typo)
-	delete_transient( md5( 'give_earnings_this_monththis_month' ) );
-	delete_transient( md5( 'give_earnings_todaytoday' ) );
+	delete_transient(md5('give_earnings_this_monththis_month'));
+	delete_transient(md5('give_earnings_todaytoday'));
 
 
 	// Increase the donor's purchase stats
-	$customer = new Give_Customer( $customer_id );
+	$customer = new Give_Customer($customer_id);
 	$customer->increase_purchase_count();
-	$customer->increase_value( $amount );
+	$customer->increase_value($amount);
 
-	give_increase_total_earnings( $amount );
+	give_increase_total_earnings($amount);
 
 	// Ensure this action only runs once ever
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
 		// Save the completed date
-		$payment->completed_date = current_time( 'mysql' );
+		$payment->completed_date = current_time('mysql');
 		$payment->save();
-		do_action( 'give_complete_purchase', $payment_id );
+		do_action('give_complete_purchase', $payment_id);
 	}
 
 }
 
-add_action( 'give_update_payment_status', 'give_complete_purchase', 100, 3 );
+add_action('give_update_payment_status', 'give_complete_purchase', 100, 3);
 
 
 /**
@@ -105,19 +105,19 @@ 
  *
  * @return void
  */
-function give_record_status_change( $payment_id, $new_status, $old_status ) {
+function give_record_status_change($payment_id, $new_status, $old_status) {
 
 	// Get the list of statuses so that status in the payment note can be translated
 	$stati      = give_get_payment_statuses();
-	$old_status = isset( $stati[ $old_status ] ) ? $stati[ $old_status ] : $old_status;
-	$new_status = isset( $stati[ $new_status ] ) ? $stati[ $new_status ] : $new_status;
+	$old_status = isset($stati[$old_status]) ? $stati[$old_status] : $old_status;
+	$new_status = isset($stati[$new_status]) ? $stati[$new_status] : $new_status;
 
-	$status_change = sprintf( __( 'Status changed from %s to %s', 'give' ), $old_status, $new_status );
+	$status_change = sprintf(__('Status changed from %s to %s', 'give'), $old_status, $new_status);
 
-	give_insert_payment_note( $payment_id, $status_change );
+	give_insert_payment_note($payment_id, $status_change);
 }
 
-add_action( 'give_update_payment_status', 'give_record_status_change', 100, 3 );
+add_action('give_update_payment_status', 'give_record_status_change', 100, 3);
 
 
 /**
@@ -130,17 +130,17 @@ 
  * @param $new_status the status of the payment, probably "publish"
  * @param $old_status the status of the payment prior to being marked as "complete", probably "pending"
  */
-function give_clear_user_history_cache( $payment_id, $new_status, $old_status ) {
+function give_clear_user_history_cache($payment_id, $new_status, $old_status) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( ! empty( $payment->user_id ) ) {
-		delete_transient( 'give_user_' . $payment->user_id . '_purchases' );
+	if ( ! empty($payment->user_id)) {
+		delete_transient('give_user_'.$payment->user_id.'_purchases');
 	}
 
 }
 
-add_action( 'give_update_payment_status', 'give_clear_user_history_cache', 10, 3 );
+add_action('give_update_payment_status', 'give_clear_user_history_cache', 10, 3);
 
 /**
  * Updates all old payments, prior to 1.2, with new
@@ -154,25 +154,25 @@ 
  *
  * @return void
  */
-function give_update_old_payments_with_totals( $data ) {
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_upgrade_payments_nonce' ) ) {
+function give_update_old_payments_with_totals($data) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_upgrade_payments_nonce')) {
 		return;
 	}
 
-	if ( get_option( 'give_payment_totals_upgraded' ) ) {
+	if (get_option('give_payment_totals_upgraded')) {
 		return;
 	}
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'offset' => 0,
-		'number' => - 1,
+		'number' => -1,
 		'mode'   => 'all'
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 
-			$payment = new Give_Payment( $payment->ID );
+			$payment = new Give_Payment($payment->ID);
 			$meta    = $payment->get_meta();
 
 			$payment->total = $meta['amount'];
@@ -181,10 +181,10 @@ 
 		}
 	}
 
-	add_option( 'give_payment_totals_upgraded', 1 );
+	add_option('give_payment_totals_upgraded', 1);
 }
 
-add_action( 'give_upgrade_payments', 'give_update_old_payments_with_totals' );
+add_action('give_upgrade_payments', 'give_update_old_payments_with_totals');
 
 /**
  * Updates week-old+ 'pending' orders to 'abandoned'
@@ -195,21 +195,21 @@ 
 function give_mark_abandoned_donations() {
 	$args = array(
 		'status' => 'pending',
-		'number' => - 1,
+		'number' => -1,
 		'output' => 'give_payments',
 	);
 
-	add_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	add_filter('posts_where', 'give_filter_where_older_than_week');
 
-	$payments = give_get_payments( $args );
+	$payments = give_get_payments($args);
 
-	remove_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	remove_filter('posts_where', 'give_filter_where_older_than_week');
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
-			$gateway = give_get_payment_gateway( $payment );
+	if ($payments) {
+		foreach ($payments as $payment) {
+			$gateway = give_get_payment_gateway($payment);
 			//Skip offline gateway payments
-			if ( $gateway == 'offline' ) {
+			if ($gateway == 'offline') {
 				continue;
 			}
 			$payment->status = 'abandoned';
@@ -218,4 +218,4 @@ 
 	}
 }
 
-add_action( 'give_weekly_scheduled_events', 'give_mark_abandoned_donations' );
\ No newline at end of file
+add_action('give_weekly_scheduled_events', 'give_mark_abandoned_donations');
\ No newline at end of file

includes/payments/class-give-payment.php

Spacing +302 added lines, -302 removed lines

@@ -13,7 +13,7 @@ 
 
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -295,13 +295,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);
 	}
 
 	/**
@@ -313,11 +313,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 {
 
@@ -338,18 +338,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;
 		}
 	}
@@ -363,9 +363,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;
 		}
@@ -380,31 +380,31 @@ 
 	 *
 	 * @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;
 		}
 
 		// Allow extensions to perform actions before the payment is loaded
-		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();
@@ -419,7 +419,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();
@@ -452,7 +452,7 @@ 
 		$this->number     = $this->setup_payment_number();
 
 		// Allow extensions to add items to this object via hook
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -467,24 +467,24 @@ 
 
 		// Construct the payment title
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} else if ( ! 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;
+		} else if ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} else if ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} else if ( ! 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;
@@ -511,58 +511,58 @@ 
 			'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_purchase' ) && is_user_logged_in() ) {
-				$customer = new Give_Customer( get_current_user_id(), true );
+			if (did_action('give_pre_process_purchase') && is_user_logged_in()) {
+				$customer = new Give_Customer(get_current_user_id(), true);
 			}
 
-			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;
 		}
 
@@ -582,11 +582,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;
@@ -595,48 +595,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 || 'revoked' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status || 'revoked' === $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;
 									}
@@ -661,15 +661,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 || 'revoked' === $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 || 'revoked' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -682,17 +682,17 @@ 
 
 					case 'fees':
 
-						if ( 'publish' !== $this->status && 'complete' !== $this->status && 'revoked' !== $this->status ) {
+						if ('publish' !== $this->status && 'complete' !== $this->status && 'revoked' !== $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'];
@@ -709,43 +709,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':
@@ -761,15 +761,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':
@@ -779,11 +779,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':
@@ -792,38 +792,38 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				}
 			}
 
-			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 purchase stats
-					$customer->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$customer->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
-				} else if ( $total_change > 0 ) {
+				} else if ($total_change > 0) {
 
 					// Increase the customer's purchase 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,
@@ -835,12 +835,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;
 				}
 			}
@@ -849,8 +849,8 @@ 
 			$saved         = true;
 		}
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -867,12 +867,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;
 		}
 
@@ -883,60 +883,60 @@ 
 			'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( get_post_meta( $form_id, '_give_donation_levels', true ) );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices     = maybe_unserialize(get_post_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',
@@ -945,7 +945,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -961,7 +961,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(
@@ -969,12 +969,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;
 		}
 
@@ -987,7 +987,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1002,7 +1002,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'    => '',
@@ -1012,16 +1012,16 @@ 
 			'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;
 	}
@@ -1035,11 +1035,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;
@@ -1056,47 +1056,47 @@ 
 	 *
 	 * @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;
 
-		} else if ( 'index' !== $key ) {
+		} else if ('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;
 					}
 
@@ -1106,8 +1106,8 @@ 
 
 		}
 
-		if ( true === $removed ) {
-			$this->fees = array_values( $this->fees );
+		if (true === $removed) {
+			$this->fees = array_values($this->fees);
 		}
 
 		return $removed;
@@ -1122,14 +1122,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;
 				}
 
@@ -1139,7 +1139,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_get_payment_fees', $fees, $this->ID, $this );
+		return apply_filters('give_get_payment_fees', $fees, $this->ID, $this);
 	}
 
 	/**
@@ -1151,13 +1151,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);
 	}
 
 	/**
@@ -1169,7 +1169,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
+	private function increase_subtotal($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
@@ -1185,11 +1185,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
+	private function decrease_subtotal($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1205,7 +1205,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_fees( $amount = 0.00 ) {
+	private function increase_fees($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->fees_total += $amount;
 
@@ -1221,11 +1221,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_fees( $amount = 0.00 ) {
+	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;
 		}
 
@@ -1251,41 +1251,41 @@ 
 	 *
 	 * @return bool 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) {
 
-			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;
@@ -1297,7 +1297,7 @@ 
 					break;
 			}
 
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		}
 
@@ -1329,32 +1329,32 @@ 
 	 *
 	 * @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 = get_post_meta( $this->ID, $meta_key, $single );
+		$meta = get_post_meta($this->ID, $meta_key, $single);
 
-		if ( $meta_key === '_give_payment_meta' ) {
+		if ($meta_key === '_give_payment_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);
 	}
 
 	/**
@@ -1368,23 +1368,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;
 
-		} else if ( $meta_key == 'email' || $meta_key == '_give_payment_user_email' ) {
+		} else if ($meta_key == 'email' || $meta_key == '_give_payment_user_email') {
 
-			$meta_value = apply_filters( 'give_give_update_payment_meta_' . $meta_key, $meta_value, $this->ID );
-			update_post_meta( $this->ID, '_give_payment_user_email', $meta_value );
+			$meta_value = apply_filters('give_give_update_payment_meta_'.$meta_key, $meta_value, $this->ID);
+			update_post_meta($this->ID, '_give_payment_user_email', $meta_value);
 
 			$current_meta                       = $this->get_meta();
 			$current_meta['user_info']['email'] = $meta_value;
@@ -1394,9 +1394,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 update_post_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return update_post_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1410,30 +1410,30 @@ 
 		$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 && 'revoked' != $this->old_status ) || 'refunded' != $this->status ) {
+		if (('publish' != $this->old_status && 'revoked' != $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;
 		}
 
-		do_action( 'give_pre_refund_payment', $this );
+		do_action('give_pre_refund_payment', $this);
 
-		$decrease_store_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_store_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_store_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1457,29 +1457,29 @@ 
 		$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 && 'revoked' != $this->old_status ) || 'pending' != $this->status ) {
+		if (('publish' != $this->old_status && 'revoked' != $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_store_earnings = apply_filters( 'give_decrease_store_earnings_on_pending', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_pending', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_pending', true, $this );
+		$decrease_store_earnings = apply_filters('give_decrease_store_earnings_on_pending', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_pending', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_pending', true, $this);
 
-		$this->maybe_alter_stats( $decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 	}
 
 	/**
@@ -1493,25 +1493,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();
 			}
 
@@ -1557,13 +1557,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;
 	}
@@ -1575,7 +1575,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1585,13 +1585,13 @@ 
 	 * @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'];
 			}
 		}
@@ -1620,9 +1620,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'];
 			}
 		}
@@ -1638,7 +1638,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;
 	}
@@ -1650,7 +1650,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;
 	}
@@ -1662,7 +1662,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;
 	}
@@ -1674,12 +1674,12 @@ 
 	 * @return string The transaction ID for the payment
 	 */
 	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 ) || (int) $transaction_id === (int) $this->ID ) {
+		if (empty($transaction_id) || (int) $transaction_id === (int) $this->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);
 
 		}
 
@@ -1693,7 +1693,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;
 	}
@@ -1705,7 +1705,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;
 	}
@@ -1717,7 +1717,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;
 	}
@@ -1729,10 +1729,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;
@@ -1750,15 +1750,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],
@@ -1768,29 +1768,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;
 					}
 				}
@@ -1810,7 +1810,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'    => '',
@@ -1830,7 +1830,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;
 	}
@@ -1843,7 +1843,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;
 	}
@@ -1855,7 +1855,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;
 	}
@@ -1867,7 +1867,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;
 	}
@@ -1881,11 +1881,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;
 
@@ -1902,7 +1902,7 @@ 
 	 * @return array The payment object as an array
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 	/**
@@ -1912,7 +1912,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);
 	}
 
 	/**
@@ -1922,7 +1922,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);
 	}
 
 	/**
@@ -1932,7 +1932,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);
 	}
 
 	/**
@@ -1942,7 +1942,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);
 	}
 
 	/**
@@ -1952,7 +1952,7 @@ 
 	 * @return string Transaction 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);
 	}
 
 	/**
@@ -1962,7 +1962,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);
 	}
 
 	/**
@@ -1972,7 +1972,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);
 	}
 
 	/**
@@ -1982,7 +1982,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);
 	}
 
 	/**
@@ -1992,7 +1992,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);
 	}
 
 	/**
@@ -2002,7 +2002,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);
 	}
 
 	/**
@@ -2012,7 +2012,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);
 	}
 
 	/**
@@ -2022,7 +2022,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);
 	}
 
 	/**
@@ -2032,7 +2032,7 @@ 
 	 * @return string Payment key
 	 */
 	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);
 	}
 
 	/**
@@ -2042,7 +2042,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/payments/class-payment-stats.php

Spacing +46 added lines, -46 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,35 +39,35 @@ 
 	 *
 	 * @return float|int  Total amount of donations based on the passed arguments.
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global sale stats
-			add_filter( 'give_count_payments_where', array( $this, 'count_where' ) );
+			add_filter('give_count_payments_where', array($this, 'count_where'));
 
-			if ( is_array( $status ) ) {
+			if (is_array($status)) {
 				$count = 0;
-				foreach ( $status as $payment_status ) {
+				foreach ($status as $payment_status) {
 					$count += give_count_payments()->$payment_status;
 				}
 			} else {
 				$count = give_count_payments()->$status;
 			}
 
-			remove_filter( 'give_count_payments_where', array( $this, 'count_where' ) );
+			remove_filter('give_count_payments_where', array($this, 'count_where'));
 
 		} else {
 
@@ -76,11 +76,11 @@ 
 			// Product specific stats
 			global $give_logs;
 
-			add_filter( 'posts_where', array( $this, 'payments_where' ) );
+			add_filter('posts_where', array($this, 'payments_where'));
 
-			$count = $give_logs->get_log_count( $form_id, 'sale' );
+			$count = $give_logs->get_log_count($form_id, 'sale');
 
-			remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+			remove_filter('posts_where', array($this, 'payments_where'));
 
 		}
 
@@ -102,31 +102,31 @@ 
 	 *
 	 * @return float|int Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
 		global $wpdb;
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 
 			// Global earning stats
 			$args = array(
 				'post_type'              => 'give_payment',
 				'nopaging'               => true,
-				'post_status'            => array( 'publish' ),
+				'post_status'            => array('publish'),
 				'fields'                 => 'ids',
 				'update_post_term_cache' => false,
 				'suppress_filters'       => false,
@@ -138,24 +138,24 @@ 
 			);
 
 			//Filter by Gateway ID meta_key
-			if ( $gateway_id !== false ) {
+			if ($gateway_id !== false) {
 				$args['meta_key']   = '_give_payment_gateway';
 				$args['meta_value'] = $gateway_id;
 			}
 
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
-			$earnings = get_transient( $key );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
+			$earnings = get_transient($key);
 			
-			if ( false === $earnings ) {
-				$sales    = get_posts( $args );
+			if (false === $earnings) {
+				$sales    = get_posts($args);
 				$earnings = 0;
-				if ( $sales ) {
-					$sales = implode( ',', array_map('intval', $sales ) );
-					$earnings += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})" );
+				if ($sales) {
+					$sales = implode(',', array_map('intval', $sales));
+					$earnings += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})");
 				}
 				// Cache the results for one hour
-				set_transient( $key, $earnings, HOUR_IN_SECONDS );
+				set_transient($key, $earnings, HOUR_IN_SECONDS);
 			}
 
 		} else {
@@ -175,37 +175,37 @@ 
 				'give_transient_type' => 'give_earnings',
 				// This is not a valid query arg, but is used for cache keying
 			);
-			$args = apply_filters( 'give_stats_earnings_args', $args );
-			$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+			$args = apply_filters('give_stats_earnings_args', $args);
+			$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 			//Set transient for faster stats
-			$earnings = get_transient( $key );
+			$earnings = get_transient($key);
 
-			if ( false === $earnings ) {
+			if (false === $earnings) {
 
 				$this->timestamp = false;
-				$log_ids  = $give_logs->get_connected_logs( $args, 'sale' );
+				$log_ids  = $give_logs->get_connected_logs($args, 'sale');
 				$earnings = 0;
 
-				if ( $log_ids ) {
-					$log_ids     = implode( ',', array_map('intval', $log_ids ) );
-					$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids);" );
+				if ($log_ids) {
+					$log_ids     = implode(',', array_map('intval', $log_ids));
+					$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids);");
 
-					foreach ( $payment_ids as $payment_id ) {
-						$earnings += give_get_payment_amount( $payment_id );
+					foreach ($payment_ids as $payment_id) {
+						$earnings += give_get_payment_amount($payment_id);
 					}
 					
 				}
 
 				// Cache the results for one hour
-				set_transient( $key, $earnings, 60 * 60 );
+				set_transient($key, $earnings, 60 * 60);
 			}
 		}
 
 		//remove our filter
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 
 		//return earnings
-		return round( $earnings, give_currency_decimal_filter() );
+		return round($earnings, give_currency_decimal_filter());
 
 	}
 
@@ -219,16 +219,16 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 
 		global $wpdb;
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT post_id as form_id, max(meta_value) as sales
 				FROM $wpdb->postmeta WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}

includes/payments/functions.php

Spacing +375 added lines, -375 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,15 +32,15 @@ 
  *
  * @return object $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -55,48 +55,48 @@ 
  *
  * @return      mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -105,7 +105,7 @@ 
 			return false;
 	}
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -121,23 +121,23 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = isset( $payment_data['give_price_id'] ) ? $payment_data['give_price_id'] : give_get_price_id( $payment_data['give_form_id'], $payment_data['price'] );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = isset($payment_data['give_price_id']) ? $payment_data['give_price_id'] : give_get_price_id($payment_data['give_form_id'], $payment_data['price']);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	//Set properties
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency();
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency();
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
@@ -151,40 +151,40 @@ 
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
 	$payment->mode           = give_is_test_mode() ? 'test' : 'live';
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	//Add the donation
 	$args = array(
 		'price'    => $payment->total,
 		'price_id' => $payment->price_id,
-		'fees'     => isset( $payment_data['fees'] ) ? $payment_data['fees'] : array()
+		'fees'     => isset($payment_data['fees']) ? $payment_data['fees'] : array()
 	);
 	
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 	//Set date if present
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	//Handle sequential payments
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Clear the user's purchased cache
-	delete_transient( 'give_user_' . $payment_data['user_info']['id'] . '_purchases' );
+	delete_transient('give_user_'.$payment_data['user_info']['id'].'_purchases');
 
 	//Save payment
 	$payment->save();
 
 	//Hook it
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	//Return payment ID upon success
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -203,9 +203,9 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
-	$payment         = new Give_Payment( $payment_id );
+	$payment         = new Give_Payment($payment_id);
 	$payment->status = $new_status;
 	$updated         = $payment->save();
 
@@ -225,44 +225,44 @@ 
  *
  * @return void
  */
-function give_delete_purchase( $payment_id = 0, $update_customer = true ) {
+function give_delete_purchase($payment_id = 0, $update_customer = true) {
 	global $give_logs;
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	give_undo_purchase( false, $payment_id );
+	give_undo_purchase(false, $payment_id);
 
-	$amount      = give_get_payment_amount( $payment_id );
+	$amount      = give_get_payment_amount($payment_id);
 	$status      = $payment->post_status;
-	$customer_id = give_get_payment_customer_id( $payment_id );
-	$customer    = new Give_Customer( $customer_id );
+	$customer_id = give_get_payment_customer_id($payment_id);
+	$customer    = new Give_Customer($customer_id);
 
-	if ( $status == 'revoked' || $status == 'publish' ) {
+	if ($status == 'revoked' || $status == 'publish') {
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment)
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 
-		if ( $customer->id && $update_customer ) {
+		if ($customer->id && $update_customer) {
 
 			// Decrement the stats for the customer
 			$customer->decrease_purchase_count();
-			$customer->decrease_value( $amount );
+			$customer->decrease_value($amount);
 
 		}
 	}
 
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $customer->id && $update_customer ) {
+	if ($customer->id && $update_customer) {
 
 		// Remove the payment ID from the customer
-		$customer->remove_payment( $payment_id );
+		$customer->remove_payment($payment_id);
 
 	}
 
 	// Remove the payment
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries
 	$give_logs->delete_logs(
@@ -276,7 +276,7 @@ 
 		)
 	);
 
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -289,26 +289,26 @@ 
  *
  * @return void
  */
-function give_undo_purchase( $form_id = false, $payment_id ) {
+function give_undo_purchase($form_id = false, $payment_id) {
 
-	if ( ! empty( $form_id ) ) {
+	if ( ! empty($form_id)) {
 		$form_id = false;
-		_give_deprected_argument( 'form_id', 'give_undo_purchase', '1.5' );
+		_give_deprected_argument('form_id', 'give_undo_purchase', '1.5');
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// decrease earnings
-		give_decrease_earnings( $payment->form_id, $payment->total );
+		give_decrease_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_sales = apply_filters( 'give_decrease_sales_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_sales ) {
+	$maybe_decrease_sales = apply_filters('give_decrease_sales_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_sales) {
 		// decrease purchase count
-		give_decrease_purchase_count( $payment->form_id );
+		give_decrease_purchase_count($payment->form_id);
 	}
 
 }
@@ -325,7 +325,7 @@ 
  *
  * @return array $count Number of payments sorted by payment status
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 
 	global $wpdb;
 
@@ -337,18 +337,18 @@ 
 		'form_id'    => null,
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$select = "SELECT p.post_status,count( * ) AS num_posts";
 	$join   = '';
 	$where  = "WHERE p.post_type = 'give_payment'";
 
 	// Count payments for a specific user
-	if ( ! empty( $args['user'] ) ) {
+	if ( ! empty($args['user'])) {
 
-		if ( is_email( $args['user'] ) ) {
+		if (is_email($args['user'])) {
 			$field = 'email';
-		} elseif ( is_numeric( $args['user'] ) ) {
+		} elseif (is_numeric($args['user'])) {
 			$field = 'id';
 		} else {
 			$field = '';
@@ -356,108 +356,108 @@ 
 
 		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$where .= "
 				AND m.meta_key = '_give_payment_user_{$field}'
 				AND m.meta_value = '{$args['user']}'";
 		}
 
 		// Count payments for a search
-	} elseif ( ! empty( $args['s'] ) ) {
+	} elseif ( ! empty($args['s'])) {
 
-		if ( is_email( $args['s'] ) || strlen( $args['s'] ) == 32 ) {
+		if (is_email($args['s']) || strlen($args['s']) == 32) {
 
-			if ( is_email( $args['s'] ) ) {
+			if (is_email($args['s'])) {
 				$field = '_give_payment_user_email';
 			} else {
 				$field = '_give_payment_purchase_key';
 			}
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$where .= $wpdb->prepare("
 				AND m.meta_key = %s
 				AND m.meta_value = %s",
 				$field,
 				$args['s']
 			);
 
-		} elseif ( '#' == substr( $args['s'], 0, 1 ) ) {
+		} elseif ('#' == substr($args['s'], 0, 1)) {
 
-			$search = str_replace( '#:', '', $args['s'] );
-			$search = str_replace( '#', '', $search );
+			$search = str_replace('#:', '', $args['s']);
+			$search = str_replace('#', '', $search);
 
 			$select = "SELECT p2.post_status,count( * ) AS num_posts ";
 			$join   = "LEFT JOIN $wpdb->postmeta m ON m.meta_key = '_give_log_payment_id' AND m.post_id = p.ID ";
 			$join .= "INNER JOIN $wpdb->posts p2 ON m.meta_value = p2.ID ";
 			$where = "WHERE p.post_type = 'give_log' ";
-			$where .= $wpdb->prepare( "AND p.post_parent = %d} ", $search );
+			$where .= $wpdb->prepare("AND p.post_parent = %d} ", $search);
 
-		} elseif ( is_numeric( $args['s'] ) ) {
+		} elseif (is_numeric($args['s'])) {
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$where .= $wpdb->prepare("
 				AND m.meta_key = '_give_payment_user_id'
 				AND m.meta_value = %d",
 				$args['s']
 			);
 
 		} else {
-			$search = $wpdb->esc_like( $args['s'] );
-			$search = '%' . $search . '%';
+			$search = $wpdb->esc_like($args['s']);
+			$search = '%'.$search.'%';
 
-			$where .= $wpdb->prepare( "AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))", $search, $search );
+			$where .= $wpdb->prepare("AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))", $search, $search);
 		}
 
 	}
 
-	if ( ! empty( $args['form_id'] ) && is_numeric( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id']) && is_numeric($args['form_id'])) {
 
-		$where .= $wpdb->prepare( " AND p.post_parent = %d", $args['form_id'] );
+		$where .= $wpdb->prepare(" AND p.post_parent = %d", $args['form_id']);
 
 	}
 	// Limit payments count by date
-	if ( ! empty( $args['start-date'] ) && false !== strpos( $args['start-date'], '/' ) ) {
+	if ( ! empty($args['start-date']) && false !== strpos($args['start-date'], '/')) {
 
-		$date_parts = explode( '/', $args['start-date'] );
-		$month      = ! empty( $date_parts[0] ) && is_numeric( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day        = ! empty( $date_parts[1] ) && is_numeric( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year       = ! empty( $date_parts[2] ) && is_numeric( $date_parts[2] ) ? $date_parts[2] : 0;
+		$date_parts = explode('/', $args['start-date']);
+		$month      = ! empty($date_parts[0]) && is_numeric($date_parts[0]) ? $date_parts[0] : 0;
+		$day        = ! empty($date_parts[1]) && is_numeric($date_parts[1]) ? $date_parts[1] : 0;
+		$year       = ! empty($date_parts[2]) && is_numeric($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['start-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date >= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['start-date']);
+			$where .= $wpdb->prepare(" AND p.post_date >= '%s'", $date->format('Y-m-d'));
 
 		}
 
 		// Fixes an issue with the payments list table counts when no end date is specified (partiy with stats class)
-		if ( empty( $args['end-date'] ) ) {
+		if (empty($args['end-date'])) {
 			$args['end-date'] = $args['start-date'];
 		}
 
 	}
 
-	if ( ! empty ( $args['end-date'] ) && false !== strpos( $args['end-date'], '/' ) ) {
+	if ( ! empty ($args['end-date']) && false !== strpos($args['end-date'], '/')) {
 
-		$date_parts = explode( '/', $args['end-date'] );
+		$date_parts = explode('/', $args['end-date']);
 
-		$month = ! empty( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day   = ! empty( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year  = ! empty( $date_parts[2] ) ? $date_parts[2] : 0;
+		$month = ! empty($date_parts[0]) ? $date_parts[0] : 0;
+		$day   = ! empty($date_parts[1]) ? $date_parts[1] : 0;
+		$year  = ! empty($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['end-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date <= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['end-date']);
+			$where .= $wpdb->prepare(" AND p.post_date <= '%s'", $date->format('Y-m-d'));
 
 		}
 
 	}
 
-	$where = apply_filters( 'give_count_payments_where', $where );
-	$join  = apply_filters( 'give_count_payments_join', $join );
+	$where = apply_filters('give_count_payments_where', $where);
+	$join  = apply_filters('give_count_payments_join', $join);
 
 	$query = "$select
 		FROM $wpdb->posts p
@@ -466,36 +466,36 @@ 
 		GROUP BY p.post_status
 	";
 
-	$cache_key = md5( $query );
+	$cache_key = md5($query);
 
-	$count = wp_cache_get( $cache_key, 'counts' );
-	if ( false !== $count ) {
+	$count = wp_cache_get($cache_key, 'counts');
+	if (false !== $count) {
 		return $count;
 	}
 
-	$count = $wpdb->get_results( $query, ARRAY_A );
+	$count = $wpdb->get_results($query, ARRAY_A);
 
 	$stats    = array();
 	$statuses = get_post_stati();
-	if ( isset( $statuses['private'] ) && empty( $args['s'] ) ) {
-		unset( $statuses['private'] );
+	if (isset($statuses['private']) && empty($args['s'])) {
+		unset($statuses['private']);
 	}
 
-	foreach ( $statuses as $state ) {
-		$stats[ $state ] = 0;
+	foreach ($statuses as $state) {
+		$stats[$state] = 0;
 	}
 
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 
-		if ( 'private' == $row['post_status'] && empty( $args['s'] ) ) {
+		if ('private' == $row['post_status'] && empty($args['s'])) {
 			continue;
 		}
 
-		$stats[ $row['post_status'] ] = $row['num_posts'];
+		$stats[$row['post_status']] = $row['num_posts'];
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( $cache_key, $stats, 'counts' );
+	wp_cache_set($cache_key, $stats, 'counts');
 
 	return $stats;
 }
@@ -510,12 +510,12 @@ 
  *
  * @return bool true if payment exists, false otherwise
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -532,29 +532,29 @@ 
  *
  * @return bool|mixed if payment status exists, false otherwise
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
-	$payment = new Give_Payment( $payment->ID );
+	$payment = new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
-			return $statuses[ $payment->status ];
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' == $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -569,17 +569,17 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' )
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'revoked'     => __('Revoked', 'give')
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -591,10 +591,10 @@ 
  * @return array $payment_status All the available payment statuses
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -609,7 +609,7 @@ 
  *
  * @return int $earnings Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead
 
@@ -620,41 +620,41 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish', 'revoked' ),
+		'post_status'            => array('publish', 'revoked'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = get_transient( $key );
+		$earnings = get_transient($key);
 	}
 
-	if ( false === $earnings ) {
-		$sales    = get_posts( $args );
+	if (false === $earnings) {
+		$sales    = get_posts($args);
 		$earnings = 0;
-		if ( $sales ) {
-			$sales = implode( ',', $sales );
+		if ($sales) {
+			$sales = implode(',', $sales);
 
-			$earnings = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$sales})" );
+			$earnings = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$sales})");
 
 		}
 		// Cache the results for one hour
-		set_transient( $key, $earnings, HOUR_IN_SECONDS );
+		set_transient($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -669,7 +669,7 @@ 
  *
  * @return int $count Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead
 	$args = array(
@@ -677,14 +677,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish', 'revoked' ),
+		'post_status'            => array('publish', 'revoked'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -695,33 +695,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$key = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = get_transient( $key );
+		$count = get_transient($key);
 	}
 
-	if ( false === $count ) {
-		$sales = new WP_Query( $args );
+	if (false === $count) {
+		$sales = new WP_Query($args);
 		$count = (int) $sales->post_count;
 		// Cache the results for one hour
-		set_transient( $key, $count, HOUR_IN_SECONDS );
+		set_transient($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -736,20 +736,20 @@ 
  *
  * @return bool true if complete, false otherwise
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -773,29 +773,29 @@ 
  */
 function give_get_total_earnings() {
 
-	$total = get_option( 'give_earnings_total', false );
+	$total = get_option('give_earnings_total', false);
 
 	// If no total stored in DB, use old method of calculating total earnings
-	if ( false === $total ) {
+	if (false === $total) {
 
 		global $wpdb;
 
-		$total = get_transient( 'give_earnings_total' );
+		$total = get_transient('give_earnings_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 
 			$total = (float) 0;
 
-			$args = apply_filters( 'give_get_total_earnings_args', array(
+			$args = apply_filters('give_get_total_earnings_args', array(
 				'offset' => 0,
-				'number' => - 1,
-				'status' => array( 'publish', 'revoked' ),
+				'number' => -1,
+				'status' => array('publish', 'revoked'),
 				'fields' => 'ids'
-			) );
+			));
 
 
-			$payments = give_get_payments( $args );
-			if ( $payments ) {
+			$payments = give_get_payments($args);
+			if ($payments) {
 
 				/*
 				 * If performing a purchase, we need to skip the very last payment in the database, since it calls
@@ -803,30 +803,30 @@ 
 				 * first purchase
 				 */
 
-				if ( did_action( 'give_update_payment_status' ) ) {
-					array_pop( $payments );
+				if (did_action('give_update_payment_status')) {
+					array_pop($payments);
 				}
 
-				if ( ! empty( $payments ) ) {
-					$payments = implode( ',', $payments );
-					$total += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})" );
+				if ( ! empty($payments)) {
+					$payments = implode(',', $payments);
+					$total += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})");
 				}
 
 			}
 
 			// Cache results for 1 day. This cache is cleared automatically when a payment is made
-			set_transient( 'give_earnings_total', $total, 86400 );
+			set_transient('give_earnings_total', $total, 86400);
 
 			// Store the total for the first time
-			update_option( 'give_earnings_total', $total );
+			update_option('give_earnings_total', $total);
 		}
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_currency_decimal_filter() ) );
+	return apply_filters('give_total_earnings', round($total, give_currency_decimal_filter()));
 }
 
 /**
@@ -838,10 +838,10 @@ 
  *
  * @return float $total Total earnings
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -855,13 +855,13 @@ 
  *
  * @return float $total Total earnings
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -877,10 +877,10 @@ 
  *
  * @return mixed $meta Payment Meta
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -893,10 +893,10 @@ 
  *
  * @return mixed               Meta ID if successful, false if unsuccessful
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -908,8 +908,8 @@ 
  *
  * @return array $user_info User Info Meta Values
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -924,8 +924,8 @@ 
  *
  * @return int $form_id
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -939,8 +939,8 @@ 
  *
  * @return string $email User Email
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -954,11 +954,11 @@ 
  *
  * @return bool            If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -970,8 +970,8 @@ 
  *
  * @return string $user_id User ID
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -985,8 +985,8 @@ 
  *
  * @return string $customer_id Customer ID
  */
-function give_get_payment_customer_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_customer_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -1000,8 +1000,8 @@ 
  *
  * @return string $ip User IP
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -1015,8 +1015,8 @@ 
  *
  * @return string $date The date the payment was completed
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -1030,8 +1030,8 @@ 
  *
  * @return string $gateway Gateway
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1045,8 +1045,8 @@ 
  *
  * @return string $currency The currency code
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1060,10 +1060,10 @@ 
  *
  * @return string $currency The currency name
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1075,8 +1075,8 @@ 
  *
  * @return string $key Purchase key
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1092,8 +1092,8 @@ 
  *
  * @return string $number Payment order number
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1107,23 +1107,23 @@ 
  *
  * @return string      The formatted payment number
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1136,17 +1136,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1156,24 +1156,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids'
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1183,13 +1183,13 @@ 
 
 	}
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1201,25 +1201,25 @@ 
  *
  * @return string          The new Payment number without prefix and postfix
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1235,10 +1235,10 @@ 
  *
  * @return string $amount Fully formatted payment amount
  */
-function give_payment_amount( $payment_id = 0 ) {
-	$amount = give_get_payment_amount( $payment_id );
+function give_payment_amount($payment_id = 0) {
+	$amount = give_get_payment_amount($payment_id);
 
-	return give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1251,11 +1251,11 @@ 
  *
  * @return mixed|void
  */
-function give_get_payment_amount( $payment_id ) {
+function give_get_payment_amount($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	return apply_filters( 'give_payment_amount', floatval( $payment->total ), $payment_id );
+	return apply_filters('give_payment_amount', floatval($payment->total), $payment_id);
 }
 
 /**
@@ -1271,10 +1271,10 @@ 
  *
  * @return array Fully formatted payment subtotal
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($subtotal), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1288,8 +1288,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted)
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new G_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new G_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1304,10 +1304,10 @@ 
  *
  * @return mixed array if payment fees found, false otherwise
  */
-function give_get_payment_fees( $payment_id = 0, $type = 'all' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_fees($payment_id = 0, $type = 'all') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_fees( $type );
+	return $payment->get_fees($type);
 }
 
 /**
@@ -1319,8 +1319,8 @@ 
  *
  * @return string The Transaction ID
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1335,15 +1335,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1357,12 +1357,12 @@ 
  *
  * @return int $purchase Purchase ID
  */
-function give_get_purchase_id_by_key( $key ) {
+function give_get_purchase_id_by_key($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1381,12 +1381,12 @@ 
  *
  * @return int $purchase Purchase ID
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1403,19 +1403,19 @@ 
  *
  * @return array $notes Payment Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
-	$notes = get_comments( array( 'post_id' => $payment_id, 'order' => 'ASC', 'search' => $search ) );
+	$notes = get_comments(array('post_id' => $payment_id, 'order' => 'ASC', 'search' => $search));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1431,19 +1431,19 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1452,9 +1452,9 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note'
 
-	) ) );
+	)));
 
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1469,14 +1469,14 @@ 
  *
  * @return bool True on success, false otherwise
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
-	$ret = wp_delete_comment( $comment_id, true );
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
+	$ret = wp_delete_comment($comment_id, true);
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1491,32 +1491,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = __( 'System', 'give' );
+		$user = __('System', 'give');
 	}
 
-	$date_format = get_option( 'date_format' ) . ', ' . get_option( 'time_format' );
+	$date_format = get_option('date_format').', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" title="' . __( 'Delete this payment note', 'give' ) . '">' . __( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" title="'.__('Delete this payment note', 'give').'">'.__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1534,20 +1534,20 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
+function give_hide_payment_notes($query) {
 	global $wp_version;
 
-	if ( version_compare( floatval( $wp_version ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+	if (version_compare(floatval($wp_version), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1559,17 +1559,17 @@ 
  *
  * @return array $clauses Updated comment clauses
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
 	global $wpdb, $wp_version;
 
-	if ( version_compare( floatval( $wp_version ), '4.1', '<' ) ) {
+	if (version_compare(floatval($wp_version), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1582,15 +1582,15 @@ 
  *
  * @return array $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( " AND comment_type != %s", 'give_payment_note' );
+	$where .= $wpdb->prepare(" AND comment_type != %s", 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1604,32 +1604,32 @@ 
  *
  * @return array Array of comment counts
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = wp_cache_get( "comments-{$post_id}", 'counts' );
+	$stats = wp_cache_get("comments-{$post_id}", 'counts');
 
-	if ( false !== $stats ) {
+	if (false !== $stats) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( " AND comment_post_ID = %d", $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(" AND comment_post_ID = %d", $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1639,30 +1639,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed'
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( "comments-{$post_id}", $stats, 'counts' );
+	wp_cache_set("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1675,9 +1675,9 @@ 
  *
  * @return string $where Modified where clause
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1696,34 +1696,34 @@ 
  *
  * @return string $form_title Returns the full title if $level_title false, otherwise returns the levels title
  */
-function give_get_payment_form_title( $payment_meta, $level_title = false ) {
+function give_get_payment_form_title($payment_meta, $level_title = false) {
 
-	$form_id    = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
-	$form_title = isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '';
-	$price_id   = isset( $payment_meta['price_id'] ) ? $payment_meta['price_id'] : give_get_price_id( $form_id, $payment_meta['price'] );
+	$form_id    = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
+	$form_title = isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '';
+	$price_id   = isset($payment_meta['price_id']) ? $payment_meta['price_id'] : give_get_price_id($form_id, $payment_meta['price']);
 
-	if($level_title == true ) {
+	if ($level_title == true) {
 		$form_title = '';
 	}
 	
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
 		$form_title .= ' <span class="donation-level-text-wrap">';
 
-		if ( $price_id == 'custom' ) {
+		if ($price_id == 'custom') {
 
-			$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
-			$form_title .= ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
+			$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
+			$form_title .= ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
 
 		} else {
-			$form_title .= give_get_price_option_name( $form_id, $price_id );
+			$form_title .= give_get_price_option_name($form_id, $price_id);
 		}
 
 		$form_title .= '</span>';
 
 	}
 
-	return apply_filters( 'give_get_payment_form_title', $form_title, $payment_meta );
+	return apply_filters('give_get_payment_form_title', $form_title, $payment_meta);
 
 }
 
@@ -1737,20 +1737,20 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 
 	$price_id = 0;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = maybe_unserialize( get_post_meta( $form_id, '_give_donation_levels', true ) );
+		$levels = maybe_unserialize(get_post_meta($form_id, '_give_donation_levels', true));
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = (float) give_sanitize_amount( $level['_give_amount'] );
+			$level_amount = (float) give_sanitize_amount($level['_give_amount']);
 
 			//check that this indeed the recurring price
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 

includes/class-give-logging.php

Spacing +56 added lines, -56 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,10 +30,10 @@ 
 	 */
 	public function __construct() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
 	}
 
@@ -47,7 +47,7 @@ 
 	public function register_post_type() {
 		/* Logs post type */
 		$log_args = array(
-			'labels'              => array( 'name' => __( 'Logs', 'give' ) ),
+			'labels'              => array('name' => __('Logs', 'give')),
 			'public'              => false,
 			'exclude_from_search' => true,
 			'publicly_queryable'  => false,
@@ -55,11 +55,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -72,7 +72,7 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array( 'public' => false ) );
+		register_taxonomy('give_log_type', 'give_log', array('public' => false));
 	}
 
 	/**
@@ -91,7 +91,7 @@ 
 			'api_request'
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -107,8 +107,8 @@ 
 	 *
 	 * @return bool Whether log type is valid
 	 */
-	function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 	 *
 	 * @return int Log ID
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -136,7 +136,7 @@ 
 			'log_type'     => $type
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -152,12 +152,12 @@ 
 	 *
 	 * @return array Array of the connected logs
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type
-		) );
+		));
 	}
 
 	/**
@@ -172,7 +172,7 @@ 
 	 *
 	 * @return int The ID of the newly created log item
 	 */
-	function insert_log( $log_data = array(), $log_meta = array() ) {
+	function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -181,26 +181,26 @@ 
 			'log_type'     => false
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -216,9 +216,9 @@ 
 	 *
 	 * @return bool True if successful, false otherwise
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -226,20 +226,20 @@ 
 			'post_parent' => 0
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
 
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -254,19 +254,19 @@ 
 	 *
 	 * @return mixed array if logs were found, false otherwise
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -276,9 +276,9 @@ 
 			);
 		}
 		
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 	
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -299,17 +299,17 @@ 
 	 *
 	 * @return int Log count
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -319,15 +319,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
-		$logs = new WP_Query( $query_args );
+		$logs = new WP_Query($query_args);
 
 		return (int) $logs->post_count;
 	}
@@ -345,16 +345,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids'
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -364,15 +364,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -400,9 +400,9 @@ 
  *
  * @return mixed ID of the new log entry
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }
\ No newline at end of file

includes/deprecated-functions.php

Spacing +8 added lines, -8 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,15 +28,15 @@ 
  *
  * @return bool $ret True if guest checkout is enabled, false otherwise
  */
-function give_no_guest_checkout( $form_id ) {
+function give_no_guest_checkout($form_id) {
 
 	$backtrace = debug_backtrace();
 
-	_give_deprecated_function( __FUNCTION__, '1.4.1', null, $backtrace );
+	_give_deprecated_function(__FUNCTION__, '1.4.1', null, $backtrace);
 
-	$ret = get_post_meta( $form_id, '_give_logged_in_only', true );
+	$ret = get_post_meta($form_id, '_give_logged_in_only', true);
 
-	return (bool) apply_filters( 'give_no_guest_checkout', $ret );
+	return (bool) apply_filters('give_no_guest_checkout', $ret);
 }
 
 /**
@@ -50,12 +50,12 @@ 
  *
  * @return void
  */
-function give_undo_donation_on_refund( $payment_id, $new_status, $old_status ) {
+function give_undo_donation_on_refund($payment_id, $new_status, $old_status) {
 
 	$backtrace = debug_backtrace();
-	_give_deprecated_function( 'give_undo_purchase_on_refund', '1.5', 'Give_Payment->refund()', $backtrace );
+	_give_deprecated_function('give_undo_purchase_on_refund', '1.5', 'Give_Payment->refund()', $backtrace);
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 	$payment->refund();
 
 }
\ No newline at end of file

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

Spacing +92 added lines, -92 removed lines

@@ -23,7 +23,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,9 +54,9 @@ 
 	 * @param string $tag Email tag to be replace in email
 	 * @param callable $func Hook to run when email tag is found
 	 */
-	public function add( $tag, $description, $func ) {
-		if ( is_callable( $func ) ) {
-			$this->tags[ $tag ] = array(
+	public function add($tag, $description, $func) {
+		if (is_callable($func)) {
+			$this->tags[$tag] = array(
 				'tag'         => $tag,
 				'description' => $description,
 				'func'        => $func
@@ -71,8 +71,8 @@ 
 	 *
 	 * @param string $tag Email tag to remove hook from
 	 */
-	public function remove( $tag ) {
-		unset( $this->tags[ $tag ] );
+	public function remove($tag) {
+		unset($this->tags[$tag]);
 	}
 
 	/**
@@ -84,8 +84,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function email_tag_exists( $tag ) {
-		return array_key_exists( $tag, $this->tags );
+	public function email_tag_exists($tag) {
+		return array_key_exists($tag, $this->tags);
 	}
 
 	/**
@@ -109,16 +109,16 @@ 
 	 *
 	 * @return string Content with email tags filtered out.
 	 */
-	public function do_tags( $content, $payment_id ) {
+	public function do_tags($content, $payment_id) {
 
 		// Check if there is atleast one tag added
-		if ( empty( $this->tags ) || ! is_array( $this->tags ) ) {
+		if (empty($this->tags) || ! is_array($this->tags)) {
 			return $content;
 		}
 
 		$this->payment_id = $payment_id;
 
-		$new_content = preg_replace_callback( "/{([A-z0-9\-\_]+)}/s", array( $this, 'do_tag' ), $content );
+		$new_content = preg_replace_callback("/{([A-z0-9\-\_]+)}/s", array($this, 'do_tag'), $content);
 
 		$this->payment_id = null;
 
@@ -134,17 +134,17 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function do_tag( $m ) {
+	public function do_tag($m) {
 
 		// Get tag
 		$tag = $m[1];
 
 		// Return tag if tag not set
-		if ( ! $this->email_tag_exists( $tag ) ) {
+		if ( ! $this->email_tag_exists($tag)) {
 			return $m[0];
 		}
 
-		return call_user_func( $this->tags[ $tag ]['func'], $this->payment_id, $tag );
+		return call_user_func($this->tags[$tag]['func'], $this->payment_id, $tag);
 	}
 
 }
@@ -158,8 +158,8 @@ 
  * @param string $description Description of the email tag added
  * @param callable $func Hook to run when email tag is found
  */
-function give_add_email_tag( $tag, $description, $func ) {
-	Give()->email_tags->add( $tag, $description, $func );
+function give_add_email_tag($tag, $description, $func) {
+	Give()->email_tags->add($tag, $description, $func);
 }
 
 /**
@@ -169,8 +169,8 @@ 
  *
  * @param string $tag Email tag to remove hook from
  */
-function give_remove_email_tag( $tag ) {
-	Give()->email_tags->remove( $tag );
+function give_remove_email_tag($tag) {
+	Give()->email_tags->remove($tag);
 }
 
 /**
@@ -182,8 +182,8 @@ 
  *
  * @return bool
  */
-function give_email_tag_exists( $tag ) {
-	return Give()->email_tags->email_tag_exists( $tag );
+function give_email_tag_exists($tag) {
+	return Give()->email_tags->email_tag_exists($tag);
 }
 
 /**
@@ -212,13 +212,13 @@ 
 	$email_tags = give_get_email_tags();
 
 	// Check
-	if ( count( $email_tags ) > 0 ) {
+	if (count($email_tags) > 0) {
 
 		// Loop
-		foreach ( $email_tags as $email_tag ) {
+		foreach ($email_tags as $email_tag) {
 
 			// Add email tag to list
-			$list .= '<code>{' . $email_tag['tag'] . '}</code> - ' . $email_tag['description'] . '<br/>';
+			$list .= '<code>{'.$email_tag['tag'].'}</code> - '.$email_tag['description'].'<br/>';
 
 		}
 
@@ -238,13 +238,13 @@ 
  *
  * @return string Content with email tags filtered out.
  */
-function give_do_email_tags( $content, $payment_id ) {
+function give_do_email_tags($content, $payment_id) {
 
 	// Replace all tags
-	$content = Give()->email_tags->do_tags( $content, $payment_id );
+	$content = Give()->email_tags->do_tags($content, $payment_id);
 
 	// Maintaining backwards compatibility
-	$content = apply_filters( 'give_email_template_tags', $content, give_get_payment_meta( $payment_id ), $payment_id );
+	$content = apply_filters('give_email_template_tags', $content, give_get_payment_meta($payment_id), $payment_id);
 
 	// Return content
 	return $content;
@@ -256,10 +256,10 @@ 
  * @since 1.0
  */
 function give_load_email_tags() {
-	do_action( 'give_add_email_tags' );
+	do_action('give_add_email_tags');
 }
 
-add_action( 'init', 'give_load_email_tags', - 999 );
+add_action('init', 'give_load_email_tags', - 999);
 
 /**
  * Add default Give email template tags
@@ -272,82 +272,82 @@ 
 	$email_tags = array(
 		array(
 			'tag'         => 'donation',
-			'description' => __( 'The name of completed donation form', 'give' ),
+			'description' => __('The name of completed donation form', 'give'),
 			'function'    => 'give_email_tag_donation'
 		),
 		array(
 			'tag'         => 'name',
-			'description' => __( 'The donor\'s first name', 'give' ),
+			'description' => __('The donor\'s first name', 'give'),
 			'function'    => 'give_email_tag_first_name'
 		),
 		array(
 			'tag'         => 'fullname',
-			'description' => __( 'The donor\'s full name, first and last', 'give' ),
+			'description' => __('The donor\'s full name, first and last', 'give'),
 			'function'    => 'give_email_tag_fullname'
 		),
 		array(
 			'tag'         => 'username',
-			'description' => __( 'The donor\'s user name on the site, if they registered an account', 'give' ),
+			'description' => __('The donor\'s user name on the site, if they registered an account', 'give'),
 			'function'    => 'give_email_tag_username'
 		),
 		array(
 			'tag'         => 'user_email',
-			'description' => __( 'The donor\'s email address', 'give' ),
+			'description' => __('The donor\'s email address', 'give'),
 			'function'    => 'give_email_tag_user_email'
 		),
 		array(
 			'tag'         => 'billing_address',
-			'description' => __( 'The donor\'s billing address', 'give' ),
+			'description' => __('The donor\'s billing address', 'give'),
 			'function'    => 'give_email_tag_billing_address'
 		),
 		array(
 			'tag'         => 'date',
-			'description' => __( 'The date of the donation', 'give' ),
+			'description' => __('The date of the donation', 'give'),
 			'function'    => 'give_email_tag_date'
 		),
 		array(
 			'tag'         => 'price',
-			'description' => __( 'The total price of the donation', 'give' ),
+			'description' => __('The total price of the donation', 'give'),
 			'function'    => 'give_email_tag_price'
 		),
 		array(
 			'tag'         => 'payment_id',
-			'description' => __( 'The unique ID number for this donation', 'give' ),
+			'description' => __('The unique ID number for this donation', 'give'),
 			'function'    => 'give_email_tag_payment_id'
 		),
 		array(
 			'tag'         => 'receipt_id',
-			'description' => __( 'The unique ID number for this donation receipt', 'give' ),
+			'description' => __('The unique ID number for this donation receipt', 'give'),
 			'function'    => 'give_email_tag_receipt_id'
 		),
 		array(
 			'tag'         => 'payment_method',
-			'description' => __( 'The method of payment used for this donation', 'give' ),
+			'description' => __('The method of payment used for this donation', 'give'),
 			'function'    => 'give_email_tag_payment_method'
 		),
 		array(
 			'tag'         => 'sitename',
-			'description' => __( 'Your site name', 'give' ),
+			'description' => __('Your site name', 'give'),
 			'function'    => 'give_email_tag_sitename'
 		),
 		array(
 			'tag'         => 'receipt_link',
-			'description' => __( 'Adds a link so users can view their receipt directly on your website if they are unable to view it in the browser correctly.', 'give' ),
+			'description' => __('Adds a link so users can view their receipt directly on your website if they are unable to view it in the browser correctly.', 'give'),
 			'function'    => 'give_email_tag_receipt_link'
 		),
 	);
 
 	// Apply give_email_tags filter
-	$email_tags = apply_filters( 'give_email_tags', $email_tags );
+	$email_tags = apply_filters('give_email_tags', $email_tags);
 
 	// Add email tags
-	foreach ( $email_tags as $email_tag ) {
-		give_add_email_tag( $email_tag['tag'], $email_tag['description'], $email_tag['function'] );
+	foreach ($email_tags as $email_tag) {
+		give_add_email_tag($email_tag['tag'], $email_tag['description'], $email_tag['function']);
 	}
 
 }
 
-add_action( 'give_add_email_tags', 'give_setup_email_tags' );
+add_action('give_add_email_tags', 'give_setup_email_tags');
 
 
 /**
@@ -358,15 +358,15 @@ 
  *
  * @return string name
  */
-function give_email_tag_first_name( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_first_name($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['name'];
 }
@@ -379,15 +379,15 @@ 
  *
  * @return string fullname
  */
-function give_email_tag_fullname( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_fullname($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['fullname'];
 }
@@ -400,15 +400,15 @@ 
  *
  * @return string username
  */
-function give_email_tag_username( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_username($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['username'];
 }
@@ -421,8 +421,8 @@ 
  *
  * @return string user_email
  */
-function give_email_tag_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -435,10 +435,10 @@ 
  *
  * @return string billing_address
  */
-function give_email_tag_billing_address( $payment_id ) {
+function give_email_tag_billing_address($payment_id) {
 
-	$user_info    = give_get_payment_meta_user_info( $payment_id );
-	$user_address = ! empty( $user_info['address'] ) ? $user_info['address'] : array(
+	$user_info    = give_get_payment_meta_user_info($payment_id);
+	$user_address = ! empty($user_info['address']) ? $user_info['address'] : array(
 		'line1'   => '',
 		'line2'   => '',
 		'city'    => '',
@@ -447,11 +447,11 @@ 
 		'zip'     => ''
 	);
 
-	$return = $user_address['line1'] . "\n";
-	if ( ! empty( $user_address['line2'] ) ) {
-		$return .= $user_address['line2'] . "\n";
+	$return = $user_address['line1']."\n";
+	if ( ! empty($user_address['line2'])) {
+		$return .= $user_address['line2']."\n";
 	}
-	$return .= $user_address['city'] . ' ' . $user_address['zip'] . ' ' . $user_address['state'] . "\n";
+	$return .= $user_address['city'].' '.$user_address['zip'].' '.$user_address['state']."\n";
 	$return .= $user_address['country'];
 
 	return $return;
@@ -465,10 +465,10 @@ 
  *
  * @return string date
  */
-function give_email_tag_date( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_date($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return date_i18n( get_option( 'date_format' ), strtotime( $payment->date ) );
+	return date_i18n(get_option('date_format'), strtotime($payment->date));
 }
 
 /**
@@ -479,11 +479,11 @@ 
  *
  * @return string price
  */
-function give_email_tag_price( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
-	$price   = give_currency_filter( give_format_amount( $payment->total ), $payment->currency );
+function give_email_tag_price($payment_id) {
+	$payment = new Give_Payment($payment_id);
+	$price   = give_currency_filter(give_format_amount($payment->total), $payment->currency);
 
-	return html_entity_decode( $price, ENT_COMPAT, 'UTF-8' );
+	return html_entity_decode($price, ENT_COMPAT, 'UTF-8');
 }
 
 /**
@@ -494,8 +494,8 @@ 
  *
  * @return int payment_id
  */
-function give_email_tag_payment_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -508,8 +508,8 @@ 
  *
  * @return string receipt_id
  */
-function give_email_tag_receipt_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_receipt_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -522,11 +522,11 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_donation( $payment_id ) {
-	$payment    = new Give_Payment( $payment_id );
-	$form_title = get_the_title( $payment->form_id );
+function give_email_tag_donation($payment_id) {
+	$payment    = new Give_Payment($payment_id);
+	$form_title = get_the_title($payment->form_id);
 
-	return ! empty( $form_title ) ? $form_title : __( 'There was an error retrieving this donation title', 'give' );
+	return ! empty($form_title) ? $form_title : __('There was an error retrieving this donation title', 'give');
 
 }
 
@@ -538,9 +538,9 @@ 
  *
  * @return string gateway
  */
-function give_email_tag_payment_method( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
-	return give_get_gateway_checkout_label( $payment->gateway );
+function give_email_tag_payment_method($payment_id) {
+	$payment = new Give_Payment($payment_id);
+	return give_get_gateway_checkout_label($payment->gateway);
 }
 
 /**
@@ -551,8 +551,8 @@ 
  *
  * @return string sitename
  */
-function give_email_tag_sitename( $payment_id ) {
-	return wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES );
+function give_email_tag_sitename($payment_id) {
+	return wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES);
 }
 
 /**
@@ -564,15 +564,15 @@ 
  *
  * @return string receipt_link
  */
-function give_email_tag_receipt_link( $payment_id ) {
+function give_email_tag_receipt_link($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt'
-	), home_url() ) );
-	$formatted   = sprintf( __( '%1$sView it in your browser %2$s', 'give' ), '<a href="' . $receipt_url . '">', '&raquo;</a>' );
+	), home_url()));
+	$formatted = sprintf(__('%1$sView it in your browser %2$s', 'give'), '<a href="'.$receipt_url.'">', '&raquo;</a>');
 
-	if ( give_get_option( 'email_template' ) !== 'none' ) {
+	if (give_get_option('email_template') !== 'none') {
 		return $formatted;
 	} else {
 		return $receipt_url;

includes/emails/functions.php

Spacing +93 added lines, -93 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,39 +26,39 @@ 
  *
  * @return void
  */
-function give_email_donation_receipt( $payment_id, $admin_notice = true ) {
+function give_email_donation_receipt($payment_id, $admin_notice = true) {
 
-	$payment_data = give_get_payment_meta( $payment_id );
+	$payment_data = give_get_payment_meta($payment_id);
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
-	$from_name = apply_filters( 'give_purchase_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
+	$from_name = apply_filters('give_purchase_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
-	$from_email = apply_filters( 'give_purchase_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
+	$from_email = apply_filters('give_purchase_from_address', $from_email, $payment_id, $payment_data);
 
-	$to_email = give_get_payment_user_email( $payment_id );
+	$to_email = give_get_payment_user_email($payment_id);
 
-	$subject = give_get_option( 'donation_subject', __( 'Donation Receipt', 'give' ) );
-	$subject = apply_filters( 'give_donation_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = give_get_option('donation_subject', __('Donation Receipt', 'give'));
+	$subject = apply_filters('give_donation_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
-	$attachments = apply_filters( 'give_receipt_attachments', array(), $payment_id, $payment_data );
-	$message     = give_do_email_tags( give_get_email_body_content( $payment_id, $payment_data ), $payment_id );
+	$attachments = apply_filters('give_receipt_attachments', array(), $payment_id, $payment_data);
+	$message     = give_do_email_tags(give_get_email_body_content($payment_id, $payment_data), $payment_id);
 
 	$emails = Give()->emails;
 
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Donation Receipt', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Donation Receipt', 'give'));
 
 
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data);
+	$emails->__set('headers', $headers);
 
-	$emails->send( $to_email, $subject, $message, $attachments );
+	$emails->send($to_email, $subject, $message, $attachments);
 
-	if ( $admin_notice && ! give_admin_notices_disabled( $payment_id ) ) {
-		do_action( 'give_admin_sale_notice', $payment_id, $payment_data );
+	if ($admin_notice && ! give_admin_notices_disabled($payment_id)) {
+		do_action('give_admin_sale_notice', $payment_id, $payment_data);
 	}
 }
 
@@ -71,29 +71,29 @@ 
  */
 function give_email_test_donation_receipt() {
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
-	$from_name = apply_filters( 'give_purchase_from_name', $from_name, 0, array() );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
+	$from_name = apply_filters('give_purchase_from_name', $from_name, 0, array());
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
-	$from_email = apply_filters( 'give_purchase_from_address', $from_email, 0, array() );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
+	$from_email = apply_filters('give_purchase_from_address', $from_email, 0, array());
 
-	$subject = give_get_option( 'donation_subject', __( 'Donation Receipt', 'give' ) );
-	$subject = apply_filters( 'give_donation_subject', wp_strip_all_tags( $subject ), 0 );
-	$subject = give_do_email_tags( $subject, 0 );
+	$subject = give_get_option('donation_subject', __('Donation Receipt', 'give'));
+	$subject = apply_filters('give_donation_subject', wp_strip_all_tags($subject), 0);
+	$subject = give_do_email_tags($subject, 0);
 
-	$attachments = apply_filters( 'give_receipt_attachments', array(), 0, array() );
+	$attachments = apply_filters('give_receipt_attachments', array(), 0, array());
 
-	$message = give_email_preview_template_tags( give_get_email_body_content( 0, array() ) );
+	$message = give_email_preview_template_tags(give_get_email_body_content(0, array()));
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Donation Receipt', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Donation Receipt', 'give'));
 
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), 0, array() );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), 0, array());
+	$emails->__set('headers', $headers);
 
-	$emails->send( give_get_admin_notice_emails(), $subject, $message, $attachments );
+	$emails->send(give_get_admin_notice_emails(), $subject, $message, $attachments);
 
 }
 
@@ -107,49 +107,49 @@ 
  *
  * @return void
  */
-function give_admin_email_notice( $payment_id = 0, $payment_data = array() ) {
+function give_admin_email_notice($payment_id = 0, $payment_data = array()) {
 
-	$payment_id = absint( $payment_id );
+	$payment_id = absint($payment_id);
 
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
-	if ( ! give_get_payment_by( 'id', $payment_id ) ) {
+	if ( ! give_get_payment_by('id', $payment_id)) {
 		return;
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
-	$from_name = apply_filters( 'give_purchase_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
+	$from_name = apply_filters('give_purchase_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
-	$from_email = apply_filters( 'give_purchase_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
+	$from_email = apply_filters('give_purchase_from_address', $from_email, $payment_id, $payment_data);
 
-	$subject = give_get_option( 'donation_notification_subject', sprintf( __( 'New Donation - Payment #%1$s', 'give' ), $payment_id ) );
-	$subject = apply_filters( 'give_admin_donation_notification_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = give_get_option('donation_notification_subject', sprintf(__('New Donation - Payment #%1$s', 'give'), $payment_id));
+	$subject = apply_filters('give_admin_donation_notification_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
-	$headers = "From: " . stripslashes_deep( html_entity_decode( $from_name, ENT_COMPAT, 'UTF-8' ) ) . " <$from_email>\r\n";
-	$headers .= "Reply-To: " . $from_email . "\r\n";
+	$headers = "From: ".stripslashes_deep(html_entity_decode($from_name, ENT_COMPAT, 'UTF-8'))." <$from_email>\r\n";
+	$headers .= "Reply-To: ".$from_email."\r\n";
 	//$headers  .= "MIME-Version: 1.0\r\n";
 	$headers .= "Content-Type: text/html; charset=utf-8\r\n";
-	$headers = apply_filters( 'give_admin_donation_notification_headers', $headers, $payment_id, $payment_data );
+	$headers = apply_filters('give_admin_donation_notification_headers', $headers, $payment_id, $payment_data);
 
-	$attachments = apply_filters( 'give_admin_donation_notification_attachments', array(), $payment_id, $payment_data );
+	$attachments = apply_filters('give_admin_donation_notification_attachments', array(), $payment_id, $payment_data);
 
-	$message = give_get_donation_notification_body_content( $payment_id, $payment_data );
+	$message = give_get_donation_notification_body_content($payment_id, $payment_data);
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'headers', $headers );
-	$emails->__set( 'heading', __( 'New Donation!', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('headers', $headers);
+	$emails->__set('heading', __('New Donation!', 'give'));
 
-	$emails->send( give_get_admin_notice_emails(), $subject, $message, $attachments );
+	$emails->send(give_get_admin_notice_emails(), $subject, $message, $attachments);
 
 }
 
-add_action( 'give_admin_sale_notice', 'give_admin_email_notice', 10, 2 );
+add_action('give_admin_sale_notice', 'give_admin_email_notice', 10, 2);
 
 /**
  * Retrieves the emails for which admin notifications are sent to (these can be
@@ -162,10 +162,10 @@ 
 function give_get_admin_notice_emails() {
 	global $give_options;
 
-	$emails = isset( $give_options['admin_notice_emails'] ) && strlen( trim( $give_options['admin_notice_emails'] ) ) > 0 ? $give_options['admin_notice_emails'] : get_bloginfo( 'admin_email' );
-	$emails = array_map( 'trim', explode( "\n", $emails ) );
+	$emails = isset($give_options['admin_notice_emails']) && strlen(trim($give_options['admin_notice_emails'])) > 0 ? $give_options['admin_notice_emails'] : get_bloginfo('admin_email');
+	$emails = array_map('trim', explode("\n", $emails));
 
-	return apply_filters( 'give_admin_notice_emails', $emails );
+	return apply_filters('give_admin_notice_emails', $emails);
 }
 
 /**
@@ -177,11 +177,11 @@ 
  *
  * @return mixed
  */
-function give_admin_notices_disabled( $payment_id = 0 ) {
+function give_admin_notices_disabled($payment_id = 0) {
 	global $give_options;
-	$retval = isset( $give_options['disable_admin_notices'] );
+	$retval = isset($give_options['disable_admin_notices']);
 
-	return apply_filters( 'give_admin_notices_disabled', $retval, $payment_id );
+	return apply_filters('give_admin_notices_disabled', $retval, $payment_id);
 }
 
 /**
@@ -195,16 +195,16 @@ 
 function give_get_default_donation_notification_email() {
 	global $give_options;
 
-	$default_email_body = __( 'Hi there,', 'give' ) . "\n\n" . __( 'This email is to inform you that a new donation has been made on your website: ', 'give' ) . '<a href="' . get_bloginfo( 'url' ) . '" target="_blank">' . get_bloginfo( 'url' ) . '</a>' . ".\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor: ', 'give' ) . '</strong> ' . ' {name}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation: ', 'give' ) . '</strong> ' . ' {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount: ', 'give' ) . '</strong> ' . ' {price}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method: ', 'give' ) . '</strong> ' . ' {payment_method}' . "\n\n";
-	$default_email_body .= __( 'Thank you,', 'give' ) . "\n\n" . '{sitename}';
+	$default_email_body = __('Hi there,', 'give')."\n\n".__('This email is to inform you that a new donation has been made on your website: ', 'give').'<a href="'.get_bloginfo('url').'" target="_blank">'.get_bloginfo('url').'</a>'.".\n\n";
+	$default_email_body .= '<strong>'.__('Donor: ', 'give').'</strong> '.' {name}'."\n";
+	$default_email_body .= '<strong>'.__('Donation: ', 'give').'</strong> '.' {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Amount: ', 'give').'</strong> '.' {price}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method: ', 'give').'</strong> '.' {payment_method}'."\n\n";
+	$default_email_body .= __('Thank you,', 'give')."\n\n".'{sitename}';
 
-	$message = ( isset( $give_options['donation_notification'] ) && ! empty( $give_options['donation_notification'] ) ) ? $give_options['donation_notification'] : $default_email_body;
+	$message = (isset($give_options['donation_notification']) && ! empty($give_options['donation_notification'])) ? $give_options['donation_notification'] : $default_email_body;
 
-	return apply_filters( 'give_default_donation_notification_email', $message );
+	return apply_filters('give_default_donation_notification_email', $message);
 }
 
 
@@ -219,24 +219,24 @@ 
 function give_get_default_donation_receipt_email() {
 	global $give_options;
 
-	$default_email_body = __( 'Dear', 'give' ) . " {name},\n\n";
-	$default_email_body .= __( 'Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give' ) . "\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor', 'give' ) . ':</strong> ' . '{fullname}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation', 'give' ) . ':</strong> ' . '{donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation Date', 'give' ) . ':</strong> ' . '{date}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount', 'give' ) . ':</strong> ' . '{price}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method', 'give' ) . ':</strong> ' . '{payment_method}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment ID', 'give' ) . ':</strong> ' . '{payment_id}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Receipt ID', 'give' ) . ':</strong> ' . '{receipt_id}' . "\n\n";
-	$default_email_body .= '{receipt_link}' . "\n\n";
+	$default_email_body = __('Dear', 'give')." {name},\n\n";
+	$default_email_body .= __('Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give')."\n\n";
+	$default_email_body .= '<strong>'.__('Donor', 'give').':</strong> '.'{fullname}'."\n";
+	$default_email_body .= '<strong>'.__('Donation', 'give').':</strong> '.'{donation}'."\n";
+	$default_email_body .= '<strong>'.__('Donation Date', 'give').':</strong> '.'{date}'."\n";
+	$default_email_body .= '<strong>'.__('Amount', 'give').':</strong> '.'{price}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method', 'give').':</strong> '.'{payment_method}'."\n";
+	$default_email_body .= '<strong>'.__('Payment ID', 'give').':</strong> '.'{payment_id}'."\n";
+	$default_email_body .= '<strong>'.__('Receipt ID', 'give').':</strong> '.'{receipt_id}'."\n\n";
+	$default_email_body .= '{receipt_link}'."\n\n";
 
 	$default_email_body .= "\n\n";
-	$default_email_body .= __( 'Sincerely,', 'give' );
-	$default_email_body .= "\n" . '{sitename}' . "\n";
+	$default_email_body .= __('Sincerely,', 'give');
+	$default_email_body .= "\n".'{sitename}'."\n";
 
-	$message = ( isset( $give_options['donation_receipt'] ) && ! empty( $give_options['donation_receipt'] ) ) ? $give_options['donation_receipt'] : $default_email_body;
+	$message = (isset($give_options['donation_receipt']) && ! empty($give_options['donation_receipt'])) ? $give_options['donation_receipt'] : $default_email_body;
 
-	return apply_filters( 'give_default_donation_receipt_email', $message );
+	return apply_filters('give_default_donation_receipt_email', $message);
 }
 
 /**
@@ -248,19 +248,19 @@ 
  *
  * @return array $email_names
  */
-function give_get_email_names( $user_info ) {
+function give_get_email_names($user_info) {
 	$email_names = array();
-	$user_info   = maybe_unserialize( $user_info );
+	$user_info   = maybe_unserialize($user_info);
 
 	$email_names['fullname'] = '';
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 && isset( $user_info['first_name'] ) ) {
-		$user_data               = get_userdata( $user_info['id'] );
+	if (isset($user_info['id']) && $user_info['id'] > 0 && isset($user_info['first_name'])) {
+		$user_data               = get_userdata($user_info['id']);
 		$email_names['name']     = $user_info['first_name'];
-		$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+		$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 		$email_names['username'] = $user_data->user_login;
-	} elseif ( isset( $user_info['first_name'] ) ) {
+	} elseif (isset($user_info['first_name'])) {
 		$email_names['name']     = $user_info['first_name'];
-		$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+		$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 		$email_names['username'] = $user_info['first_name'];
 	} else {
 		$email_names['name']     = $user_info['email'];