WordImpress / Give

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

Doc Comments +2 added lines, -1 removed lines

@@ -50,6 +50,7 @@ 
 	 *
 	 * @param string   $tag  Email tag to be replace in email
 	 * @param callable $func Hook to run when email tag is found
+	 * @param string $description
 	 */
 	public function add( $tag, $description, $func ) {
 		if ( is_callable( $func ) ) {
@@ -477,7 +478,7 @@ 
  *
  * @param int $payment_id
  *
- * @return int payment_id
+ * @return string payment_id
  */
 function give_email_tag_payment_id( $payment_id ) {
 	return give_get_payment_number( $payment_id );

Spacing +98 added lines, -98 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;
@@ -263,10 +263,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_add_email_tags' );
+	do_action('give_add_email_tags');
 }
 
-add_action( 'init', 'give_load_email_tags', - 999 );
+add_action('init', 'give_load_email_tags', - 999);
 
 /**
  * Add default Give email template tags.
@@ -279,87 +279,87 @@ 
 	$email_tags = array(
 		array(
 			'tag'         => 'donation',
-			'description' => esc_html__( 'The name of completed donation form and the donation level chosen if applicable.', 'give' ),
+			'description' => esc_html__('The name of completed donation form and the donation level chosen if applicable.', 'give'),
 			'function'    => 'give_email_tag_donation'
 		),
 		array(
 			'tag'         => 'name',
-			'description' => esc_html__( 'The donor\'s first name.', 'give' ),
+			'description' => esc_html__('The donor\'s first name.', 'give'),
 			'function'    => 'give_email_tag_first_name'
 		),
 		array(
 			'tag'         => 'fullname',
-			'description' => esc_html__( 'The donor\'s full name, first and last.', 'give' ),
+			'description' => esc_html__('The donor\'s full name, first and last.', 'give'),
 			'function'    => 'give_email_tag_fullname'
 		),
 		array(
 			'tag'         => 'username',
-			'description' => esc_html__( 'The donor\'s user name on the site, if they registered an account.', 'give' ),
+			'description' => esc_html__('The donor\'s user name on the site, if they registered an account.', 'give'),
 			'function'    => 'give_email_tag_username'
 		),
 		array(
 			'tag'         => 'user_email',
-			'description' => esc_html__( 'The donor\'s email address.', 'give' ),
+			'description' => esc_html__('The donor\'s email address.', 'give'),
 			'function'    => 'give_email_tag_user_email'
 		),
 		array(
 			'tag'         => 'billing_address',
-			'description' => esc_html__( 'The donor\'s billing address.', 'give' ),
+			'description' => esc_html__('The donor\'s billing address.', 'give'),
 			'function'    => 'give_email_tag_billing_address'
 		),
 		array(
 			'tag'         => 'date',
-			'description' => esc_html__( 'The date of the donation.', 'give' ),
+			'description' => esc_html__('The date of the donation.', 'give'),
 			'function'    => 'give_email_tag_date'
 		),
 		array(
 			'tag'         => 'price',
-			'description' => esc_html__( 'The total price of the donation.', 'give' ),
+			'description' => esc_html__('The total price of the donation.', 'give'),
 			'function'    => 'give_email_tag_price'
 		),
 		array(
 			'tag'         => 'payment_id',
-			'description' => esc_html__( 'The unique ID number for this donation.', 'give' ),
+			'description' => esc_html__('The unique ID number for this donation.', 'give'),
 			'function'    => 'give_email_tag_payment_id'
 		),
 		array(
 			'tag'         => 'receipt_id',
-			'description' => esc_html__( 'The unique ID number for this donation receipt.', 'give' ),
+			'description' => esc_html__('The unique ID number for this donation receipt.', 'give'),
 			'function'    => 'give_email_tag_receipt_id'
 		),
 		array(
 			'tag'         => 'payment_method',
-			'description' => esc_html__( 'The method of payment used for this donation.', 'give' ),
+			'description' => esc_html__('The method of payment used for this donation.', 'give'),
 			'function'    => 'give_email_tag_payment_method'
 		),
 		array(
 			'tag'         => 'sitename',
-			'description' => esc_html__( 'The name of the site.', 'give' ),
+			'description' => esc_html__('The name of the site.', 'give'),
 			'function'    => 'give_email_tag_sitename'
 		),
 		array(
 			'tag'         => 'receipt_link',
-			'description' => esc_html__( 'The donation receipt direct link, to view the receipt on the website.', 'give' ),
+			'description' => esc_html__('The donation receipt direct link, to view the receipt on the website.', 'give'),
 			'function'    => 'give_email_tag_receipt_link'
 		),
 		array(
 			'tag'         => 'receipt_link_url',
-			'description' => esc_html__( 'The donation receipt direct URL, to view the receipt on the website.', 'give' ),
+			'description' => esc_html__('The donation receipt direct URL, to view the receipt on the website.', 'give'),
 			'function'    => 'give_email_tag_receipt_link_url'
 		),
 	);
 
 	// Apply give_email_tags filter
-	$email_tags = apply_filters( 'give_email_tags', $email_tags );
+	$email_tags = apply_filters('give_email_tags', $email_tags);
 
 	// Add email tags
-	foreach ( $email_tags as $email_tag ) {
-		give_add_email_tag( $email_tag['tag'], $email_tag['description'], $email_tag['function'] );
+	foreach ($email_tags as $email_tag) {
+		give_add_email_tag($email_tag['tag'], $email_tag['description'], $email_tag['function']);
 	}
 
 }
 
-add_action( 'give_add_email_tags', 'give_setup_email_tags' );
+add_action('give_add_email_tags', 'give_setup_email_tags');
 
 
 /**
@@ -371,15 +371,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'];
 }
@@ -393,15 +393,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'];
 }
@@ -415,15 +415,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'];
 }
@@ -437,8 +437,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;
 }
@@ -452,10 +452,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'    => '',
@@ -464,11 +464,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;
@@ -483,10 +483,10 @@ 
  *
  * @return string date
  */
-function give_email_tag_date( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_date($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return date_i18n( give_date_format(), strtotime( $payment->date ) );
+	return date_i18n(give_date_format(), strtotime($payment->date));
 }
 
 /**
@@ -498,11 +498,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');
 }
 
 /**
@@ -514,8 +514,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;
 }
@@ -529,8 +529,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;
 }
@@ -544,11 +544,11 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_donation( $payment_id ) {
-	$payment    = new Give_Payment( $payment_id );
-	$form_title = strip_tags( give_get_payment_form_title( $payment->meta, false, '-' ) );
+function give_email_tag_donation($payment_id) {
+	$payment    = new Give_Payment($payment_id);
+	$form_title = strip_tags(give_get_payment_form_title($payment->meta, false, '-'));
 
-	return ! empty( $form_title ) ? $form_title : esc_html__( 'There was an error retrieving this donation title.', 'give' );
+	return ! empty($form_title) ? $form_title : esc_html__('There was an error retrieving this donation title.', 'give');
 
 }
 
@@ -561,10 +561,10 @@ 
  *
  * @return string gateway
  */
-function give_email_tag_payment_method( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_method($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return give_get_gateway_checkout_label( $payment->gateway );
+	return give_get_gateway_checkout_label($payment->gateway);
 }
 
 /**
@@ -576,8 +576,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);
 }
 
 /**
@@ -589,19 +589,19 @@ 
  *
  * @return string receipt_link
  */
-function give_email_tag_receipt_link( $payment_id ) {
+function give_email_tag_receipt_link($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt'
-	), home_url() ) );
-	$formatted   = sprintf(
+	), home_url()));
+	$formatted = sprintf(
 		'<a href="%1$s">%2$s</a>',
 		$receipt_url,
-		esc_html__( 'View it in your browser', 'give' )
+		esc_html__('View it in your browser', 'give')
 	);
 
-	if ( give_get_option( 'email_template' ) !== 'none' ) {
+	if (give_get_option('email_template') !== 'none') {
 		return $formatted;
 	} else {
 		return $receipt_url;
@@ -620,12 +620,12 @@ 
  *
  * @return string receipt_url
  */
-function give_email_tag_receipt_link_url( $payment_id ) {
+function give_email_tag_receipt_link_url($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt'
-	), home_url() ) );
+	), home_url()));
 
 	return $receipt_url;
 

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

Indentation +2 added lines, -2 removed lines

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

Spacing +60 added lines, -60 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box donor-search" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,29 +124,29 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' == $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' == $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e( 'Donors Report', 'give' ); ?></span></h3>
+			<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e('Donors Report', 'give'); ?></span></h3>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' == $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' == $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -168,25 +168,25 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 
 			case 'num_purchases' :
-				$value = '<a href="' .
-				         admin_url( '/edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_purchases'] ) . '</a>';
+				$value = '<a href="'.
+				         admin_url('/edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
+				         ).'">'.esc_html($item['num_purchases']).'</a>';
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name]));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -198,14 +198,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => esc_html__( 'Name', 'give' ),
-			'id'            => esc_html__( 'ID', 'give' ),
-			'email'         => esc_html__( 'Email', 'give' ),
-			'num_purchases' => esc_html__( 'Purchases', 'give' ),
-			'amount_spent'  => esc_html__( 'Total Spent', 'give' )
+			'name'          => esc_html__('Name', 'give'),
+			'id'            => esc_html__('ID', 'give'),
+			'email'         => esc_html__('Email', 'give'),
+			'num_purchases' => esc_html__('Purchases', 'give'),
+			'amount_spent'  => esc_html__('Total Spent', 'give')
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -218,10 +218,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_purchases' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_purchases' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -232,7 +232,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place
 		give_report_views();
 	}
@@ -245,7 +245,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -256,7 +256,7 @@ 
 	 * @return mixed string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -273,10 +273,10 @@ 
 
 		$data    = array();
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -285,21 +285,21 @@ 
 			'orderby' => $orderby
 		);
 
-		if ( is_email( $search ) ) {
+		if (is_email($search)) {
 			$args['email'] = $search;
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 			$args['id'] = $search;
 		}
 
-		$donors = Give()->customers->get_customers( $args );
+		$donors = Give()->customers->get_customers($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -332,16 +332,16 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->reports_data();
 
 		$this->total = give_count_total_customers();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page )
-		) );
+			'total_pages' => ceil($this->total / $this->per_page)
+		));
 	}
 }
\ No newline at end of file

includes/admin/plugins.php

Spacing +16 added lines, -16 removed lines

@@ -11,7 +11,7 @@ 
 
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,16 +26,16 @@ 
  *
  * @return array $links
  */
-function give_plugin_action_links( $links, $file ) {
-	$settings_link = '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-settings' ) . '">' . esc_html__( 'Settings', 'give' ) . '</a>';
-	if ( $file == 'give/give.php' ) {
-		array_unshift( $links, $settings_link );
+function give_plugin_action_links($links, $file) {
+	$settings_link = '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-settings').'">'.esc_html__('Settings', 'give').'</a>';
+	if ($file == 'give/give.php') {
+		array_unshift($links, $settings_link);
 	}
 
 	return $links;
 }
 
-add_filter( 'plugin_action_links', 'give_plugin_action_links', 10, 2 );
+add_filter('plugin_action_links', 'give_plugin_action_links', 10, 2);
 
 
 /**
@@ -48,33 +48,33 @@ 
  *
  * @return array $input
  */
-function give_plugin_row_meta( $input, $file ) {
-	if ( $file != 'give/give.php' ) {
+function give_plugin_row_meta($input, $file) {
+	if ($file != 'give/give.php') {
 		return $input;
 	}
 
-	$give_addons_link = esc_url( add_query_arg( array(
+	$give_addons_link = esc_url(add_query_arg(array(
 			'utm_source'   => 'plugins-page',
 			'utm_medium'   => 'plugin-row',
 			'utm_campaign' => 'admin',
-		), 'https://givewp.com/addons/' )
+		), 'https://givewp.com/addons/')
 	);
 
-	$give_docs_link = esc_url( add_query_arg( array(
+	$give_docs_link = esc_url(add_query_arg(array(
 			'utm_source'   => 'plugins-page',
 			'utm_medium'   => 'plugin-row',
 			'utm_campaign' => 'admin',
-		), 'https://givewp.com/documentation/' )
+		), 'https://givewp.com/documentation/')
 	);
 
 	$links = array(
-		'<a href="' . $give_docs_link . '" target="_blank">' . esc_html__( 'Documentation', 'give' ) . '</a>',
-		'<a href="' . $give_addons_link . '" target="_blank">' . esc_html__( 'Add-ons', 'give' ) . '</a>',
+		'<a href="'.$give_docs_link.'" target="_blank">'.esc_html__('Documentation', 'give').'</a>',
+		'<a href="'.$give_addons_link.'" target="_blank">'.esc_html__('Add-ons', 'give').'</a>',
 	);
 
-	$input = array_merge( $input, $links );
+	$input = array_merge($input, $links);
 
 	return $input;
 }
 
-add_filter( 'plugin_row_meta', 'give_plugin_row_meta', 10, 2 );
+add_filter('plugin_row_meta', 'give_plugin_row_meta', 10, 2);

includes/plugin-compatibility.php

Spacing +2 added lines, -2 removed lines

@@ -19,6 +19,6 @@
  * @return void
  */
 function give_disable_mandrill_nl2br() {
-	add_filter( 'mandrill_nl2br', '__return_false' );
+	add_filter('mandrill_nl2br', '__return_false');
 }
-add_action( 'give_email_send_before', 'give_disable_mandrill_nl2br');
+add_action('give_email_send_before', 'give_disable_mandrill_nl2br');

includes/admin/system-info.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
  *
  * @since: 1.4
  *
- * @return bool
+ * @return boolean|null
  */
 function give_allow_sessions_for_sysinfo() {
 	if ( is_admin() && ( isset( $_GET['page'] ) && isset( $_GET['tab'] ) ) && ( $_GET['tab'] == 'system_info' && $_GET['page'] == 'give-settings' ) ) {

Spacing +146 added lines, -146 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,14 +24,14 @@ 
  */
 function give_system_info_callback() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 	?>
-	<textarea readonly="readonly" onclick="this.focus(); this.select()" id="system-info-textarea" name="give-sysinfo" aria-label="<?php esc_attr_e( 'To copy the system info, click below then press Ctrl + C (PC) or Cmd + C (Mac).', 'give' ); ?>"><?php echo give_tools_sysinfo_get(); ?></textarea>
+	<textarea readonly="readonly" onclick="this.focus(); this.select()" id="system-info-textarea" name="give-sysinfo" aria-label="<?php esc_attr_e('To copy the system info, click below then press Ctrl + C (PC) or Cmd + C (Mac).', 'give'); ?>"><?php echo give_tools_sysinfo_get(); ?></textarea>
 	<p class="submit">
 		<input type="hidden" name="give-action" value="download_sysinfo"/>
-		<?php submit_button( esc_html__( 'Download System Info File', 'give' ), 'secondary', 'give-download-sysinfo', false ); ?>
+		<?php submit_button(esc_html__('Download System Info File', 'give'), 'secondary', 'give-download-sysinfo', false); ?>
 	</p>
 	<?php
 }
@@ -47,12 +47,12 @@ 
  * @return bool
  */
 function give_allow_sessions_for_sysinfo() {
-	if ( is_admin() && ( isset( $_GET['page'] ) && isset( $_GET['tab'] ) ) && ( $_GET['tab'] == 'system_info' && $_GET['page'] == 'give-settings' ) ) {
+	if (is_admin() && (isset($_GET['page']) && isset($_GET['tab'])) && ($_GET['tab'] == 'system_info' && $_GET['page'] == 'give-settings')) {
 		return true;
 	}
 }
 
-add_filter( 'give_start_session', 'give_allow_sessions_for_sysinfo' );
+add_filter('give_start_session', 'give_allow_sessions_for_sysinfo');
 
 
 /**
@@ -67,63 +67,63 @@ 
 function give_tools_sysinfo_get() {
 	global $wpdb, $give_options;
 
-	if ( ! class_exists( 'Browser' ) ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/libraries/browser.php';
+	if ( ! class_exists('Browser')) {
+		require_once GIVE_PLUGIN_DIR.'includes/libraries/browser.php';
 	}
 
 	$browser = new Browser();
 
 	// Get theme info
-	if ( get_bloginfo( 'version' ) < '3.4' ) {
-		$theme_data = wp_get_theme( get_stylesheet_directory() . '/style.css' );
-		$theme      = $theme_data['Name'] . ' ' . $theme_data['Version'];
+	if (get_bloginfo('version') < '3.4') {
+		$theme_data = wp_get_theme(get_stylesheet_directory().'/style.css');
+		$theme      = $theme_data['Name'].' '.$theme_data['Version'];
 	} else {
 		$theme_data = wp_get_theme();
-		$theme      = $theme_data->Name . ' ' . $theme_data->Version;
+		$theme      = $theme_data->Name.' '.$theme_data->Version;
 	}
 
 	// Try to identify the hosting provider
 	$host = give_get_host();
 
-	$return = '### Begin System Info ###' . "\n\n";
+	$return = '### Begin System Info ###'."\n\n";
 
 	// Start with the basics...
-	$return .= '-- Site Info' . "\n\n";
-	$return .= 'Site URL:                 ' . site_url() . "\n";
-	$return .= 'Home URL:                 ' . home_url() . "\n";
-	$return .= 'Multisite:                ' . ( is_multisite() ? 'Yes' : 'No' ) . "\n";
+	$return .= '-- Site Info'."\n\n";
+	$return .= 'Site URL:                 '.site_url()."\n";
+	$return .= 'Home URL:                 '.home_url()."\n";
+	$return .= 'Multisite:                '.(is_multisite() ? 'Yes' : 'No')."\n";
 
-	$return = apply_filters( 'give_sysinfo_after_site_info', $return );
+	$return = apply_filters('give_sysinfo_after_site_info', $return);
 
 	// Can we determine the site's host?
-	if ( $host ) {
-		$return .= "\n" . '-- Hosting Provider' . "\n\n";
-		$return .= 'Host:                     ' . $host . "\n";
+	if ($host) {
+		$return .= "\n".'-- Hosting Provider'."\n\n";
+		$return .= 'Host:                     '.$host."\n";
 
-		$return = apply_filters( 'give_sysinfo_after_host_info', $return );
+		$return = apply_filters('give_sysinfo_after_host_info', $return);
 	}
 
 	// The local users' browser information, handled by the Browser class
-	$return .= "\n" . '-- User Browser' . "\n\n";
+	$return .= "\n".'-- User Browser'."\n\n";
 	$return .= $browser;
 
-	$return = apply_filters( 'give_sysinfo_after_user_browser', $return );
+	$return = apply_filters('give_sysinfo_after_user_browser', $return);
 
 	// WordPress configuration
-	$return .= "\n" . '-- WordPress Configuration' . "\n\n";
-	$return .= 'Version:                  ' . get_bloginfo( 'version' ) . "\n";
-	$return .= 'Language:                 ' . ( defined( 'WPLANG' ) && WPLANG ? WPLANG : 'en_US' ) . "\n";
-	$return .= 'Permalink Structure:      ' . ( get_option( 'permalink_structure' ) ? get_option( 'permalink_structure' ) : 'Default' ) . "\n";
-	$return .= 'Active Theme:             ' . $theme . "\n";
-	$return .= 'Show On Front:            ' . get_option( 'show_on_front' ) . "\n";
+	$return .= "\n".'-- WordPress Configuration'."\n\n";
+	$return .= 'Version:                  '.get_bloginfo('version')."\n";
+	$return .= 'Language:                 '.(defined('WPLANG') && WPLANG ? WPLANG : 'en_US')."\n";
+	$return .= 'Permalink Structure:      '.(get_option('permalink_structure') ? get_option('permalink_structure') : 'Default')."\n";
+	$return .= 'Active Theme:             '.$theme."\n";
+	$return .= 'Show On Front:            '.get_option('show_on_front')."\n";
 
 	// Only show page specs if frontpage is set to 'page'
-	if ( get_option( 'show_on_front' ) == 'page' ) {
-		$front_page_id = get_option( 'page_on_front' );
-		$blog_page_id  = get_option( 'page_for_posts' );
+	if (get_option('show_on_front') == 'page') {
+		$front_page_id = get_option('page_on_front');
+		$blog_page_id  = get_option('page_for_posts');
 
-		$return .= 'Page On Front:            ' . ( $front_page_id != 0 ? get_the_title( $front_page_id ) . ' (#' . $front_page_id . ')' : 'Unset' ) . "\n";
-		$return .= 'Page For Posts:           ' . ( $blog_page_id != 0 ? get_the_title( $blog_page_id ) . ' (#' . $blog_page_id . ')' : 'Unset' ) . "\n";
+		$return .= 'Page On Front:            '.($front_page_id != 0 ? get_the_title($front_page_id).' (#'.$front_page_id.')' : 'Unset')."\n";
+		$return .= 'Page For Posts:           '.($blog_page_id != 0 ? get_the_title($blog_page_id).' (#'.$blog_page_id.')' : 'Unset')."\n";
 	}
 
 	// Make sure wp_remote_post() is working
@@ -132,205 +132,205 @@ 
 	$params = array(
 		'sslverify'  => false,
 		'timeout'    => 60,
-		'user-agent' => 'Give/' . GIVE_VERSION,
+		'user-agent' => 'Give/'.GIVE_VERSION,
 		'body'       => $request
 	);
 
-	$response = wp_remote_post( 'https://www.paypal.com/cgi-bin/webscr', $params );
+	$response = wp_remote_post('https://www.paypal.com/cgi-bin/webscr', $params);
 
-	if ( ! is_wp_error( $response ) && $response['response']['code'] >= 200 && $response['response']['code'] < 300 ) {
+	if ( ! is_wp_error($response) && $response['response']['code'] >= 200 && $response['response']['code'] < 300) {
 		$WP_REMOTE_POST = 'wp_remote_post() works';
 	} else {
 		$WP_REMOTE_POST = 'wp_remote_post() does not work';
 	}
 
-	$return .= 'Remote Post:              ' . $WP_REMOTE_POST . "\n";
-	$return .= 'Table Prefix:             ' . 'Length: ' . strlen( $wpdb->prefix ) . '   Status: ' . ( strlen( $wpdb->prefix ) > 16 ? 'ERROR: Too long' : 'Acceptable' ) . "\n";
-	$return .= 'Admin AJAX:               ' . ( give_test_ajax_works() ? 'Accessible' : 'Inaccessible' ) . "\n";
-	$return .= 'WP_DEBUG:                 ' . ( defined( 'WP_DEBUG' ) ? WP_DEBUG ? 'Enabled' : 'Disabled' : 'Not set' ) . "\n";
-	$return .= 'Memory Limit:             ' . WP_MEMORY_LIMIT . "\n";
-	$return .= 'Registered Post Stati:    ' . implode( ', ', get_post_stati() ) . "\n";
+	$return .= 'Remote Post:              '.$WP_REMOTE_POST."\n";
+	$return .= 'Table Prefix:             '.'Length: '.strlen($wpdb->prefix).'   Status: '.(strlen($wpdb->prefix) > 16 ? 'ERROR: Too long' : 'Acceptable')."\n";
+	$return .= 'Admin AJAX:               '.(give_test_ajax_works() ? 'Accessible' : 'Inaccessible')."\n";
+	$return .= 'WP_DEBUG:                 '.(defined('WP_DEBUG') ? WP_DEBUG ? 'Enabled' : 'Disabled' : 'Not set')."\n";
+	$return .= 'Memory Limit:             '.WP_MEMORY_LIMIT."\n";
+	$return .= 'Registered Post Stati:    '.implode(', ', get_post_stati())."\n";
 
-	$return = apply_filters( 'give_sysinfo_after_wordpress_config', $return );
+	$return = apply_filters('give_sysinfo_after_wordpress_config', $return);
 
 	// GIVE configuration
-	$return .= "\n" . '-- Give Configuration' . "\n\n";
-	$return .= 'Version:                  ' . GIVE_VERSION . "\n";
-	$return .= 'Upgraded From:            ' . get_option( 'give_version_upgraded_from', 'None' ) . "\n";
-	$return .= 'Test Mode:                ' . ( give_is_test_mode() ? "Enabled\n" : "Disabled\n" );
-	$return .= 'Currency Code:            ' . give_get_currency() . "\n";
-	$return .= 'Currency Position:        ' . give_get_option( 'currency_position', 'before' ) . "\n";
-	$return .= 'Decimal Separator:        ' . give_get_option( 'decimal_separator', '.' ) . "\n";
-	$return .= 'Thousands Separator:      ' . give_get_option( 'thousands_separator', ',' ) . "\n";
+	$return .= "\n".'-- Give Configuration'."\n\n";
+	$return .= 'Version:                  '.GIVE_VERSION."\n";
+	$return .= 'Upgraded From:            '.get_option('give_version_upgraded_from', 'None')."\n";
+	$return .= 'Test Mode:                '.(give_is_test_mode() ? "Enabled\n" : "Disabled\n");
+	$return .= 'Currency Code:            '.give_get_currency()."\n";
+	$return .= 'Currency Position:        '.give_get_option('currency_position', 'before')."\n";
+	$return .= 'Decimal Separator:        '.give_get_option('decimal_separator', '.')."\n";
+	$return .= 'Thousands Separator:      '.give_get_option('thousands_separator', ',')."\n";
 
-	$return = apply_filters( 'give_sysinfo_after_give_config', $return );
+	$return = apply_filters('give_sysinfo_after_give_config', $return);
 
 	// GIVE pages
-	$return .= "\n" . '-- Give Page Configuration' . "\n\n";
-	$return .= 'Success Page:             ' . ( ! empty( $give_options['success_page'] ) ? get_permalink( $give_options['success_page'] ) . "\n" : "Unset\n" );
-	$return .= 'Failure Page:             ' . ( ! empty( $give_options['failure_page'] ) ? get_permalink( $give_options['failure_page'] ) . "\n" : "Unset\n" );
-	$return .= 'Give Forms Slug:           ' . ( defined( 'GIVE_SLUG' ) ? '/' . GIVE_SLUG . "\n" : "/donations\n" );
+	$return .= "\n".'-- Give Page Configuration'."\n\n";
+	$return .= 'Success Page:             '.( ! empty($give_options['success_page']) ? get_permalink($give_options['success_page'])."\n" : "Unset\n");
+	$return .= 'Failure Page:             '.( ! empty($give_options['failure_page']) ? get_permalink($give_options['failure_page'])."\n" : "Unset\n");
+	$return .= 'Give Forms Slug:           '.(defined('GIVE_SLUG') ? '/'.GIVE_SLUG."\n" : "/donations\n");
 
-	$return = apply_filters( 'give_sysinfo_after_give_pages', $return );
+	$return = apply_filters('give_sysinfo_after_give_pages', $return);
 
 	// GIVE gateways
-	$return .= "\n" . '-- Give Gateway Configuration' . "\n\n";
+	$return .= "\n".'-- Give Gateway Configuration'."\n\n";
 
 	$active_gateways = give_get_enabled_payment_gateways();
-	if ( $active_gateways ) {
-		$default_gateway_is_active = give_is_gateway_active( give_get_default_gateway( null ) );
-		if ( $default_gateway_is_active ) {
-			$default_gateway = give_get_default_gateway( null );
-			$default_gateway = $active_gateways[ $default_gateway ]['admin_label'];
+	if ($active_gateways) {
+		$default_gateway_is_active = give_is_gateway_active(give_get_default_gateway(null));
+		if ($default_gateway_is_active) {
+			$default_gateway = give_get_default_gateway(null);
+			$default_gateway = $active_gateways[$default_gateway]['admin_label'];
 		} else {
 			$default_gateway = 'Test Donation';
 		}
 
 		$gateways = array();
-		foreach ( $active_gateways as $gateway ) {
+		foreach ($active_gateways as $gateway) {
 			$gateways[] = $gateway['admin_label'];
 		}
 
-		$return .= 'Enabled Gateways:         ' . implode( ', ', $gateways ) . "\n";
-		$return .= 'Default Gateway:          ' . $default_gateway . "\n";
+		$return .= 'Enabled Gateways:         '.implode(', ', $gateways)."\n";
+		$return .= 'Default Gateway:          '.$default_gateway."\n";
 	} else {
-		$return .= 'Enabled Gateways:         None' . "\n";
+		$return .= 'Enabled Gateways:         None'."\n";
 	}
 
-	$return = apply_filters( 'give_sysinfo_after_give_gateways', $return );
+	$return = apply_filters('give_sysinfo_after_give_gateways', $return);
 
 	// GIVE Templates
-	$dir = get_stylesheet_directory() . '/give_templates/*';
-	if ( is_dir( $dir ) && ( count( glob( "$dir/*" ) ) !== 0 ) ) {
-		$return .= "\n" . '-- Give Template Overrides' . "\n\n";
+	$dir = get_stylesheet_directory().'/give_templates/*';
+	if (is_dir($dir) && (count(glob("$dir/*")) !== 0)) {
+		$return .= "\n".'-- Give Template Overrides'."\n\n";
 
-		foreach ( glob( $dir ) as $file ) {
-			$return .= 'Filename:                 ' . basename( $file ) . "\n";
+		foreach (glob($dir) as $file) {
+			$return .= 'Filename:                 '.basename($file)."\n";
 		}
 
-		$return = apply_filters( 'give_sysinfo_after_give_templates', $return );
+		$return = apply_filters('give_sysinfo_after_give_templates', $return);
 	}
 
 	// Must-use plugins
 	$muplugins = get_mu_plugins();
-	if ( count( $muplugins > 0 ) ) {
-		$return .= "\n" . '-- Must-Use Plugins' . "\n\n";
+	if (count($muplugins > 0)) {
+		$return .= "\n".'-- Must-Use Plugins'."\n\n";
 
-		foreach ( $muplugins as $plugin => $plugin_data ) {
-			$return .= $plugin_data['Name'] . ': ' . $plugin_data['Version'] . "\n";
+		foreach ($muplugins as $plugin => $plugin_data) {
+			$return .= $plugin_data['Name'].': '.$plugin_data['Version']."\n";
 		}
 
-		$return = apply_filters( 'give_sysinfo_after_wordpress_mu_plugins', $return );
+		$return = apply_filters('give_sysinfo_after_wordpress_mu_plugins', $return);
 	}
 
 	// WordPress active plugins
-	$return .= "\n" . '-- WordPress Active Plugins' . "\n\n";
+	$return .= "\n".'-- WordPress Active Plugins'."\n\n";
 
 	$plugins        = get_plugins();
-	$active_plugins = get_option( 'active_plugins', array() );
+	$active_plugins = get_option('active_plugins', array());
 
-	foreach ( $plugins as $plugin_path => $plugin ) {
-		if ( ! in_array( $plugin_path, $active_plugins ) ) {
+	foreach ($plugins as $plugin_path => $plugin) {
+		if ( ! in_array($plugin_path, $active_plugins)) {
 			continue;
 		}
 
-		$return .= $plugin['Name'] . ': ' . $plugin['Version'] . "\n";
+		$return .= $plugin['Name'].': '.$plugin['Version']."\n";
 	}
 
-	$return = apply_filters( 'give_sysinfo_after_wordpress_plugins', $return );
+	$return = apply_filters('give_sysinfo_after_wordpress_plugins', $return);
 
 	// WordPress inactive plugins
-	$return .= "\n" . '-- WordPress Inactive Plugins' . "\n\n";
+	$return .= "\n".'-- WordPress Inactive Plugins'."\n\n";
 
-	foreach ( $plugins as $plugin_path => $plugin ) {
-		if ( in_array( $plugin_path, $active_plugins ) ) {
+	foreach ($plugins as $plugin_path => $plugin) {
+		if (in_array($plugin_path, $active_plugins)) {
 			continue;
 		}
 
-		$return .= $plugin['Name'] . ': ' . $plugin['Version'] . "\n";
+		$return .= $plugin['Name'].': '.$plugin['Version']."\n";
 	}
 
-	$return = apply_filters( 'give_sysinfo_after_wordpress_plugins_inactive', $return );
+	$return = apply_filters('give_sysinfo_after_wordpress_plugins_inactive', $return);
 
-	if ( is_multisite() ) {
+	if (is_multisite()) {
 		// WordPress Multisite active plugins
-		$return .= "\n" . '-- Network Active Plugins' . "\n\n";
+		$return .= "\n".'-- Network Active Plugins'."\n\n";
 
 		$plugins        = wp_get_active_network_plugins();
-		$active_plugins = get_site_option( 'active_sitewide_plugins', array() );
+		$active_plugins = get_site_option('active_sitewide_plugins', array());
 
-		foreach ( $plugins as $plugin_path ) {
-			$plugin_base = plugin_basename( $plugin_path );
+		foreach ($plugins as $plugin_path) {
+			$plugin_base = plugin_basename($plugin_path);
 
-			if ( ! array_key_exists( $plugin_base, $active_plugins ) ) {
+			if ( ! array_key_exists($plugin_base, $active_plugins)) {
 				continue;
 			}
 
-			$plugin = get_plugin_data( $plugin_path );
-			$return .= $plugin['Name'] . ': ' . $plugin['Version'] . "\n";
+			$plugin = get_plugin_data($plugin_path);
+			$return .= $plugin['Name'].': '.$plugin['Version']."\n";
 		}
 
-		$return = apply_filters( 'give_sysinfo_after_wordpress_ms_plugins', $return );
+		$return = apply_filters('give_sysinfo_after_wordpress_ms_plugins', $return);
 	}
 
 	// Server configuration (really just versioning)
-	$return .= "\n" . '-- Webserver Configuration' . "\n\n";
-	$return .= 'PHP Version:              ' . PHP_VERSION . "\n";
-	$return .= 'MySQL Version:            ' . $wpdb->db_version() . "\n";
-	$return .= 'Webserver Info:           ' . $_SERVER['SERVER_SOFTWARE'] . "\n";
+	$return .= "\n".'-- Webserver Configuration'."\n\n";
+	$return .= 'PHP Version:              '.PHP_VERSION."\n";
+	$return .= 'MySQL Version:            '.$wpdb->db_version()."\n";
+	$return .= 'Webserver Info:           '.$_SERVER['SERVER_SOFTWARE']."\n";
 
-	$return = apply_filters( 'give_sysinfo_after_webserver_config', $return );
+	$return = apply_filters('give_sysinfo_after_webserver_config', $return);
 
 	// PHP configs... now we're getting to the important stuff
-	$return .= "\n" . '-- PHP Configuration' . "\n\n";
-	$return .= 'Safe Mode:                ' . ( ini_get( 'safe_mode' ) ? 'Enabled' : 'Disabled' . "\n" );
-	$return .= 'Memory Limit:             ' . ini_get( 'memory_limit' ) . "\n";
-	$return .= 'Upload Max Size:          ' . ini_get( 'upload_max_filesize' ) . "\n";
-	$return .= 'Post Max Size:            ' . ini_get( 'post_max_size' ) . "\n";
-	$return .= 'Upload Max Filesize:      ' . ini_get( 'upload_max_filesize' ) . "\n";
-	$return .= 'Time Limit:               ' . ini_get( 'max_execution_time' ) . "\n";
-	$return .= 'Max Input Vars:           ' . ini_get( 'max_input_vars' ) . "\n";
-	$return .= 'URL-aware fopen:          ' . ( ini_get( 'allow_url_fopen' ) ? 'On (' . ini_get( 'allow_url_fopen' ) . ')' : 'N/A' ) . "\n";
-	$return .= 'Display Errors:           ' . ( ini_get( 'display_errors' ) ? 'On (' . ini_get( 'display_errors' ) . ')' : 'N/A' ) . "\n";
-
-	$return = apply_filters( 'give_sysinfo_after_php_config', $return );
+	$return .= "\n".'-- PHP Configuration'."\n\n";
+	$return .= 'Safe Mode:                '.(ini_get('safe_mode') ? 'Enabled' : 'Disabled'."\n");
+	$return .= 'Memory Limit:             '.ini_get('memory_limit')."\n";
+	$return .= 'Upload Max Size:          '.ini_get('upload_max_filesize')."\n";
+	$return .= 'Post Max Size:            '.ini_get('post_max_size')."\n";
+	$return .= 'Upload Max Filesize:      '.ini_get('upload_max_filesize')."\n";
+	$return .= 'Time Limit:               '.ini_get('max_execution_time')."\n";
+	$return .= 'Max Input Vars:           '.ini_get('max_input_vars')."\n";
+	$return .= 'URL-aware fopen:          '.(ini_get('allow_url_fopen') ? 'On ('.ini_get('allow_url_fopen').')' : 'N/A')."\n";
+	$return .= 'Display Errors:           '.(ini_get('display_errors') ? 'On ('.ini_get('display_errors').')' : 'N/A')."\n";
+
+	$return = apply_filters('give_sysinfo_after_php_config', $return);
 
 	// PHP extensions and such
-	$return .= "\n" . '-- PHP Extensions' . "\n\n";
-	$return .= 'cURL:                     ' . ( function_exists( 'curl_init' ) ? 'Supported' : 'Not Supported' ) . "\n";
+	$return .= "\n".'-- PHP Extensions'."\n\n";
+	$return .= 'cURL:                     '.(function_exists('curl_init') ? 'Supported' : 'Not Supported')."\n";
 
 	//cURL version
-	if ( function_exists( 'curl_init' ) && function_exists( 'curl_version' ) ) {
+	if (function_exists('curl_init') && function_exists('curl_version')) {
 		$curl_values = curl_version();
-		$return .= 'cURL Version:             ' . $curl_values["version"] . "\n";
+		$return .= 'cURL Version:             '.$curl_values["version"]."\n";
 	}
-	$return .= 'zlib:                     ' . ( function_exists( 'gzcompress' ) ? 'Supported' : 'Not Supported' ) . "\n";
-	$return .= 'GD:                       ' . ( ( extension_loaded( 'gd' ) && function_exists( 'gd_info' ) ) ? 'Supported' : 'Not Supported' ) . "\n";
-	$return .= 'fsockopen:                ' . ( function_exists( 'fsockopen' ) ? 'Supported' : 'Not Supported' ) . "\n";
-	$return .= 'SOAP Client:              ' . ( class_exists( 'SoapClient' ) ? 'Installed' : 'Not Installed' ) . "\n";
-	$return .= 'Suhosin:                  ' . ( extension_loaded( 'suhosin' ) ? 'Installed' : 'Not Installed' ) . "\n";
-	$return .= 'DOM:                      ' . ( extension_loaded( 'dom' ) ? 'Installed' : 'Not Installed' ) . "\n";
-	$return .= 'MBString:                 ' . ( extension_loaded( 'mbstring' ) ? 'Installed' : 'Not Installed' ) . "\n";
+	$return .= 'zlib:                     '.(function_exists('gzcompress') ? 'Supported' : 'Not Supported')."\n";
+	$return .= 'GD:                       '.((extension_loaded('gd') && function_exists('gd_info')) ? 'Supported' : 'Not Supported')."\n";
+	$return .= 'fsockopen:                '.(function_exists('fsockopen') ? 'Supported' : 'Not Supported')."\n";
+	$return .= 'SOAP Client:              '.(class_exists('SoapClient') ? 'Installed' : 'Not Installed')."\n";
+	$return .= 'Suhosin:                  '.(extension_loaded('suhosin') ? 'Installed' : 'Not Installed')."\n";
+	$return .= 'DOM:                      '.(extension_loaded('dom') ? 'Installed' : 'Not Installed')."\n";
+	$return .= 'MBString:                 '.(extension_loaded('mbstring') ? 'Installed' : 'Not Installed')."\n";
 
-	$return = apply_filters( 'give_sysinfo_after_php_ext', $return );
+	$return = apply_filters('give_sysinfo_after_php_ext', $return);
 
 	// Session stuff
-	$return .= "\n" . '-- Session Configuration' . "\n\n";
-	$return .= 'Give Use Sessions:        ' . ( defined( 'GIVE_USE_PHP_SESSIONS' ) && GIVE_USE_PHP_SESSIONS ? 'Enforced' : ( Give()->session->use_php_sessions() ? 'Enabled' : 'Disabled' ) ) . "\n";
-	$return .= 'Session:                  ' . ( isset( $_SESSION ) ? 'Enabled' : 'Disabled' ) . "\n";
+	$return .= "\n".'-- Session Configuration'."\n\n";
+	$return .= 'Give Use Sessions:        '.(defined('GIVE_USE_PHP_SESSIONS') && GIVE_USE_PHP_SESSIONS ? 'Enforced' : (Give()->session->use_php_sessions() ? 'Enabled' : 'Disabled'))."\n";
+	$return .= 'Session:                  '.(isset($_SESSION) ? 'Enabled' : 'Disabled')."\n";
 
 	// The rest of this is only relevant is session is enabled
-	if ( isset( $_SESSION ) ) {
-		$return .= 'Session Name:             ' . esc_html( ini_get( 'session.name' ) ) . "\n";
-		$return .= 'Cookie Path:              ' . esc_html( ini_get( 'session.cookie_path' ) ) . "\n";
-		$return .= 'Save Path:                ' . esc_html( ini_get( 'session.save_path' ) ) . "\n";
-		$return .= 'Use Cookies:              ' . ( ini_get( 'session.use_cookies' ) ? 'On' : 'Off' ) . "\n";
-		$return .= 'Use Only Cookies:         ' . ( ini_get( 'session.use_only_cookies' ) ? 'On' : 'Off' ) . "\n";
+	if (isset($_SESSION)) {
+		$return .= 'Session Name:             '.esc_html(ini_get('session.name'))."\n";
+		$return .= 'Cookie Path:              '.esc_html(ini_get('session.cookie_path'))."\n";
+		$return .= 'Save Path:                '.esc_html(ini_get('session.save_path'))."\n";
+		$return .= 'Use Cookies:              '.(ini_get('session.use_cookies') ? 'On' : 'Off')."\n";
+		$return .= 'Use Only Cookies:         '.(ini_get('session.use_only_cookies') ? 'On' : 'Off')."\n";
 	}
 
-	$return = apply_filters( 'give_sysinfo_after_session_config', $return );
+	$return = apply_filters('give_sysinfo_after_session_config', $return);
 
-	$return .= "\n" . '### End System Info ###';
+	$return .= "\n".'### End System Info ###';
 
 	return $return;
 }
@@ -344,17 +344,17 @@ 
  */
 function give_tools_sysinfo_download() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 
 	nocache_headers();
 
-	header( 'Content-Type: text/plain' );
-	header( 'Content-Disposition: attachment; filename="give-system-info.txt"' );
+	header('Content-Type: text/plain');
+	header('Content-Disposition: attachment; filename="give-system-info.txt"');
 
-	echo wp_strip_all_tags( $_POST['give-sysinfo'] );
+	echo wp_strip_all_tags($_POST['give-sysinfo']);
 	give_die();
 }
 
-add_action( 'give_download_sysinfo', 'give_tools_sysinfo_download' );
\ No newline at end of file
+add_action('give_download_sysinfo', 'give_tools_sysinfo_download');
\ No newline at end of file

includes/class-give-session.php

Spacing +61 added lines, -61 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -70,53 +70,53 @@ 
 	public function __construct() {
 
 		$this->use_php_sessions = $this->use_php_sessions();
-		$this->exp_option       = give_get_option( 'session_lifetime' );
+		$this->exp_option       = give_get_option('session_lifetime');
 
 		//PHP Sessions
-		if ( $this->use_php_sessions ) {
+		if ($this->use_php_sessions) {
 
-			if ( is_multisite() ) {
+			if (is_multisite()) {
 
-				$this->prefix = '_' . get_current_blog_id();
+				$this->prefix = '_'.get_current_blog_id();
 
 			}
 
-			add_action( 'init', array( $this, 'maybe_start_session' ), - 2 );
+			add_action('init', array($this, 'maybe_start_session'), - 2);
 
 		} else {
 
-			if ( ! $this->should_start_session() ) {
+			if ( ! $this->should_start_session()) {
 				return;
 			}
 
 			// Use WP_Session
-			if ( ! defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'give_wp_session' );
+			if ( ! defined('WP_SESSION_COOKIE')) {
+				define('WP_SESSION_COOKIE', 'give_wp_session');
 			}
 
-			if ( ! class_exists( 'Recursive_ArrayAccess' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-recursive-arrayaccess.php';
+			if ( ! class_exists('Recursive_ArrayAccess')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-recursive-arrayaccess.php';
 			}
 
-			if ( ! class_exists( 'WP_Session' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/wp-session.php';
+			if ( ! class_exists('WP_Session')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session.php';
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/wp-session.php';
 			}
 
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+			add_filter('wp_session_expiration_variant', array($this, 'set_expiration_variant_time'), 99999);
+			add_filter('wp_session_expiration', array($this, 'set_expiration_time'), 99999);
 
 		}
 
 		//Init Session
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), - 1 );
+		if (empty($this->session) && ! $this->use_php_sessions) {
+			add_action('plugins_loaded', array($this, 'init'), - 1);
 		} else {
-			add_action( 'init', array( $this, 'init' ), - 1 );
+			add_action('init', array($this, 'init'), - 1);
 		}
 
 		//Set cookie on Donation Completion page
-		add_action( 'give_pre_process_purchase', array( $this, 'set_session_cookies' ) );
+		add_action('give_pre_process_purchase', array($this, 'set_session_cookies'));
 
 	}
 
@@ -129,8 +129,8 @@ 
 	 */
 	public function init() {
 
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION[ 'give' . $this->prefix ] ) && is_array( $_SESSION[ 'give' . $this->prefix ] ) ? $_SESSION[ 'give' . $this->prefix ] : array();
+		if ($this->use_php_sessions) {
+			$this->session = isset($_SESSION['give'.$this->prefix]) && is_array($_SESSION['give'.$this->prefix]) ? $_SESSION['give'.$this->prefix] : array();
 		} else {
 			$this->session = WP_Session::get_instance();
 		}
@@ -162,10 +162,10 @@ 
 	 *
 	 * @return string Session variable
 	 */
-	public function get( $key ) {
-		$key = sanitize_key( $key );
+	public function get($key) {
+		$key = sanitize_key($key);
 
-		return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : false;
+		return isset($this->session[$key]) ? maybe_unserialize($this->session[$key]) : false;
 
 	}
 
@@ -179,21 +179,21 @@ 
 	 *
 	 * @return mixed Session variable
 	 */
-	public function set( $key, $value ) {
+	public function set($key, $value) {
 
-		$key = sanitize_key( $key );
+		$key = sanitize_key($key);
 
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = serialize( $value );
+		if (is_array($value)) {
+			$this->session[$key] = serialize($value);
 		} else {
-			$this->session[ $key ] = $value;
+			$this->session[$key] = $value;
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION[ 'give' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['give'.$this->prefix] = $this->session;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -206,10 +206,10 @@ 
 	 * @since 1.4
 	 */
 	public function set_session_cookies() {
-		if( ! headers_sent() ) {
-			$lifetime = current_time( 'timestamp' ) + $this->set_expiration_time();
-			@setcookie( session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false  );
-			@setcookie( session_name() . '_expiration', $lifetime, $lifetime,  COOKIEPATH, COOKIE_DOMAIN, false  );
+		if ( ! headers_sent()) {
+			$lifetime = current_time('timestamp') + $this->set_expiration_time();
+			@setcookie(session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
+			@setcookie(session_name().'_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -225,7 +225,7 @@ 
 	 */
 	public function set_expiration_variant_time() {
 
-		return ( ! empty( $this->exp_option ) ? ( intval( $this->exp_option ) - 3600 ) : 30 * 60 * 23 );
+		return ( ! empty($this->exp_option) ? (intval($this->exp_option) - 3600) : 30 * 60 * 23);
 	}
 
 	/**
@@ -240,7 +240,7 @@ 
 	 */
 	public function set_expiration_time() {
 
-		return ( ! empty( $this->exp_option ) ? intval( $this->exp_option ) : 30 * 60 * 24 );
+		return ( ! empty($this->exp_option) ? intval($this->exp_option) : 30 * 60 * 24);
 	}
 
 	/**
@@ -258,21 +258,21 @@ 
 		$ret = false;
 
 		// If the database variable is already set, no need to run autodetection
-		$give_use_php_sessions = (bool) get_option( 'give_use_php_sessions' );
+		$give_use_php_sessions = (bool) get_option('give_use_php_sessions');
 
-		if ( ! $give_use_php_sessions ) {
+		if ( ! $give_use_php_sessions) {
 
 			// Attempt to detect if the server supports PHP sessions
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
+			if (function_exists('session_start') && ! ini_get('safe_mode')) {
 
-				$this->set( 'give_use_php_sessions', 1 );
+				$this->set('give_use_php_sessions', 1);
 
-				if ( $this->get( 'give_use_php_sessions' ) ) {
+				if ($this->get('give_use_php_sessions')) {
 
 					$ret = true;
 
 					// Set the database option
-					update_option( 'give_use_php_sessions', true );
+					update_option('give_use_php_sessions', true);
 
 				}
 
@@ -284,13 +284,13 @@ 
 		}
 
 		// Enable or disable PHP Sessions based on the GIVE_USE_PHP_SESSIONS constant
-		if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && GIVE_USE_PHP_SESSIONS ) {
+		if (defined('GIVE_USE_PHP_SESSIONS') && GIVE_USE_PHP_SESSIONS) {
 			$ret = true;
-		} else if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && ! GIVE_USE_PHP_SESSIONS ) {
+		} else if (defined('GIVE_USE_PHP_SESSIONS') && ! GIVE_USE_PHP_SESSIONS) {
 			$ret = false;
 		}
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -303,9 +303,9 @@ 
 
 		$start_session = true;
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {
+		if ( ! empty($_SERVER['REQUEST_URI'])) {
 
-			$blacklist = apply_filters( 'give_session_start_uri_blacklist', array(
+			$blacklist = apply_filters('give_session_start_uri_blacklist', array(
 				'feed',
 				'feed',
 				'feed/rss',
@@ -313,21 +313,21 @@ 
 				'feed/rdf',
 				'feed/atom',
 				'comments/feed/'
-			) );
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' );
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist ) ) {
+			));
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/');
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist)) {
 				$start_session = false;
 			}
-			if ( false !== strpos( $uri, 'feed=' ) ) {
+			if (false !== strpos($uri, 'feed=')) {
 				$start_session = false;
 			}
-			if ( is_admin() ) {
+			if (is_admin()) {
 				$start_session = false;
 			}
 		}
 
-		return apply_filters( 'give_start_session', $start_session );
+		return apply_filters('give_start_session', $start_session);
 	}
 
 	/**
@@ -338,11 +338,11 @@ 
 	 */
 	public function maybe_start_session() {
 
-		if ( ! $this->should_start_session() ) {
+		if ( ! $this->should_start_session()) {
 			return;
 		}
 
-		if ( ! session_id() && ! headers_sent() ) {
+		if ( ! session_id() && ! headers_sent()) {
 			session_start();
 		}
 
@@ -358,9 +358,9 @@ 
 
 		$expiration = false;
 
-		if ( session_id() && isset( $_COOKIE[ session_name() . '_expiration' ] ) ) {
+		if (session_id() && isset($_COOKIE[session_name().'_expiration'])) {
 
-			$expiration = date( 'D, d M Y h:i:s', intval( $_COOKIE[ session_name() . '_expiration' ] ) );
+			$expiration = date('D, d M Y h:i:s', intval($_COOKIE[session_name().'_expiration']));
 
 		}
 

Doc Comments +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
 	 * @param  string $key   Session key.
 	 * @param  string $value Session variable.
 	 *
-	 * @return mixed         Session variable.
+	 * @return string         Session variable.
 	 */
 	public function set( $key, $value ) {
 

Indentation +13 added lines, -13 removed lines

@@ -182,10 +182,10 @@ 
 	}
 
 	/**
-     * Set Session
-     *
+	 * Set Session
+	 *
 	 * Create a new session.
-     *
+	 *
 	 * @since  1.0
 	 * @access public
 	 *
@@ -315,7 +315,7 @@ 
 	 * Determines if we should start sessions.
 	 *
 	 * @since  1.4
-     * @access public
+	 * @access public
 	 *
 	 * @return bool
 	 */
@@ -352,11 +352,11 @@ 
 
 	/**
 	 * Maybe Start Session
-     *
+	 *
 	 * Starts a new session if one hasn't started yet.
-     *
-     * @access public
-     *
+	 *
+	 * @access public
+	 *
 	 * @see    http://php.net/manual/en/function.session-set-cookie-params.php
 	 *
 	 * @return void
@@ -375,12 +375,12 @@ 
 
 	/**
 	 * Get Session Expiration
-     *
+	 *
 	 * Looks at the session cookies and returns the expiration date for this session if applicable
-     *
-     * @access public
-     *
-     * @return string Formatted expiration date string.
+	 *
+	 * @access public
+	 *
+	 * @return string Formatted expiration date string.
 	 */
 	public function get_session_expiration() {
 

includes/class-give-db-customers.php

Spacing +129 added lines, -129 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,11 +33,11 @@ 
 
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_customers';
+		$this->table_name  = $wpdb->prefix.'give_customers';
 		$this->primary_key = 'id';
 		$this->version     = '1.0';
 		
-		add_action( 'profile_update', array( $this, 'update_customer_email_on_user_update' ), 10, 2 );
+		add_action('profile_update', array($this, 'update_customer_email_on_user_update'), 10, 2);
 
 	}
 
@@ -76,7 +76,7 @@ 
 			'purchase_value' => 0.00,
 			'purchase_count' => 0,
 			'notes'          => '',
-			'date_created'   => date( 'Y-m-d H:i:s' ),
+			'date_created'   => date('Y-m-d H:i:s'),
 		);
 	}
 
@@ -86,40 +86,40 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 
 		$defaults = array(
 			'payment_ids' => ''
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
-		if ( empty( $args['email'] ) ) {
+		if (empty($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
-		$customer = $this->get_customer_by( 'email', $args['email'] );
+		$customer = $this->get_customer_by('email', $args['email']);
 
-		if ( $customer ) {
+		if ($customer) {
 			// update an existing customer
 
 			// Update the payment IDs attached to the customer
-			if ( ! empty( $args['payment_ids'] ) ) {
+			if ( ! empty($args['payment_ids'])) {
 
-				if ( empty( $customer->payment_ids ) ) {
+				if (empty($customer->payment_ids)) {
 
 					$customer->payment_ids = $args['payment_ids'];
 
 				} else {
 
-					$existing_ids          = array_map( 'absint', explode( ',', $customer->payment_ids ) );
-					$payment_ids           = array_map( 'absint', explode( ',', $args['payment_ids'] ) );
-					$payment_ids           = array_merge( $payment_ids, $existing_ids );
-					$customer->payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+					$existing_ids          = array_map('absint', explode(',', $customer->payment_ids));
+					$payment_ids           = array_map('absint', explode(',', $args['payment_ids']));
+					$payment_ids           = array_merge($payment_ids, $existing_ids);
+					$customer->payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 				}
 
@@ -127,13 +127,13 @@ 
 
 			}
 
-			$this->update( $customer->id, $args );
+			$this->update($customer->id, $args);
 
 			return $customer->id;
 
 		} else {
 
-			return $this->insert( $args, 'customer' );
+			return $this->insert($args, 'customer');
 
 		}
 
@@ -152,20 +152,20 @@ 
 	 *
 	 * @return bool|false|int
 	 */
-	public function delete( $_id_or_email = false ) {
+	public function delete($_id_or_email = false) {
 
-		if ( empty( $_id_or_email ) ) {
+		if (empty($_id_or_email)) {
 			return false;
 		}
 
-		$column   = is_email( $_id_or_email ) ? 'email' : 'id';
-		$customer = $this->get_customer_by( $column, $_id_or_email );
+		$column   = is_email($_id_or_email) ? 'email' : 'id';
+		$customer = $this->get_customer_by($column, $_id_or_email);
 
-		if ( $customer->id > 0 ) {
+		if ($customer->id > 0) {
 
 			global $wpdb;
 
-			return $wpdb->delete( $this->table_name, array( 'id' => $customer->id ), array( '%d' ) );
+			return $wpdb->delete($this->table_name, array('id' => $customer->id), array('%d'));
 
 		} else {
 			return false;
@@ -179,14 +179,14 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function exists( $value = '', $field = 'email' ) {
+	public function exists($value = '', $field = 'email') {
 		
 		$columns = $this->get_columns();
-		if ( ! array_key_exists( $field, $columns ) ) {
+		if ( ! array_key_exists($field, $columns)) {
 			return false;
 		}
 
-		return (bool) $this->get_column_by( 'id', $field, $value );
+		return (bool) $this->get_column_by('id', $field, $value);
 
 	}
 
@@ -196,16 +196,16 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function attach_payment( $customer_id = 0, $payment_id = 0 ) {
+	public function attach_payment($customer_id = 0, $payment_id = 0) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( empty( $customer->id ) ) {
+		if (empty($customer->id)) {
 			return false;
 		}
 
 		// Attach the payment, but don't increment stats, as this function previously did not
-		return $customer->attach_payment( $payment_id, false );
+		return $customer->attach_payment($payment_id, false);
 
 	}
 
@@ -215,16 +215,16 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function remove_payment( $customer_id = 0, $payment_id = 0 ) {
+	public function remove_payment($customer_id = 0, $payment_id = 0) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( ! $customer ) {
+		if ( ! $customer) {
 			return false;
 		}
 
 		// Remove the payment, but don't decrease stats, as this function previously did not
-		return $customer->remove_payment( $payment_id, false );
+		return $customer->remove_payment($payment_id, false);
 
 	}
 
@@ -236,18 +236,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function increment_stats( $customer_id = 0, $amount = 0.00 ) {
+	public function increment_stats($customer_id = 0, $amount = 0.00) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( empty( $customer->id ) ) {
+		if (empty($customer->id)) {
 			return false;
 		}
 
 		$increased_count = $customer->increase_purchase_count();
-		$increased_value = $customer->increase_value( $amount );
+		$increased_value = $customer->increase_value($amount);
 
-		return ( $increased_count && $increased_value ) ? true : false;
+		return ($increased_count && $increased_value) ? true : false;
 
 	}
 
@@ -257,18 +257,18 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function decrement_stats( $customer_id = 0, $amount = 0.00 ) {
+	public function decrement_stats($customer_id = 0, $amount = 0.00) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( ! $customer ) {
+		if ( ! $customer) {
 			return false;
 		}
 
 		$decreased_count = $customer->decrease_purchase_count();
-		$decreased_value = $customer->decrease_value( $amount );
+		$decreased_value = $customer->decrease_value($amount);
 
-		return ( $decreased_count && $decreased_value ) ? true : false;
+		return ($decreased_count && $decreased_value) ? true : false;
 
 	}
 
@@ -284,37 +284,37 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update_customer_email_on_user_update( $user_id = 0, $old_user_data ) {
+	public function update_customer_email_on_user_update($user_id = 0, $old_user_data) {
 
-		$customer = new Give_Customer( $user_id, true );
+		$customer = new Give_Customer($user_id, true);
 
-		if( ! $customer ) {
+		if ( ! $customer) {
 			return false;
 		}
 
-		$user = get_userdata( $user_id );
+		$user = get_userdata($user_id);
 
-		if( ! empty( $user ) && $user->user_email !== $customer->email ) {
+		if ( ! empty($user) && $user->user_email !== $customer->email) {
 
-			if( ! $this->get_customer_by( 'email', $user->user_email ) ) {
+			if ( ! $this->get_customer_by('email', $user->user_email)) {
 
-				$success = $this->update( $customer->id, array( 'email' => $user->user_email ) );
+				$success = $this->update($customer->id, array('email' => $user->user_email));
 
-				if( $success ) {
+				if ($success) {
 					// Update some payment meta if we need to
-					$payments_array = explode( ',', $customer->payment_ids );
+					$payments_array = explode(',', $customer->payment_ids);
 
-					if( ! empty( $payments_array ) ) {
+					if ( ! empty($payments_array)) {
 
-						foreach ( $payments_array as $payment_id ) {
+						foreach ($payments_array as $payment_id) {
 
-							give_update_payment_meta( $payment_id, 'email', $user->user_email );
+							give_update_payment_meta($payment_id, 'email', $user->user_email);
 
 						}
 
 					}
 
-					do_action( 'give_update_customer_email_on_user_update', $user, $customer );
+					do_action('give_update_customer_email_on_user_update', $user, $customer);
 
 				}
 
@@ -335,45 +335,45 @@ 
 	 *
 	 * @return mixed          Upon success, an object of the customer. Upon failure, NULL
 	 */
-	public function get_customer_by( $field = 'id', $value = 0 ) {
+	public function get_customer_by($field = 'id', $value = 0) {
 		global $wpdb;
 
-		if ( empty( $field ) || empty( $value ) ) {
+		if (empty($field) || empty($value)) {
 			return null;
 		}
 
-		if ( 'id' == $field || 'user_id' == $field ) {
+		if ('id' == $field || 'user_id' == $field) {
 			// Make sure the value is numeric to avoid casting objects, for example,
 			// to int 1.
-			if ( ! is_numeric( $value ) ) {
+			if ( ! is_numeric($value)) {
 				return false;
 			}
 
-			$value = intval( $value );
+			$value = intval($value);
 
-			if ( $value < 1 ) {
+			if ($value < 1) {
 				return false;
 			}
 
-		} elseif ( 'email' === $field ) {
+		} elseif ('email' === $field) {
 
-			if ( ! is_email( $value ) ) {
+			if ( ! is_email($value)) {
 				return false;
 			}
 
-			$value = trim( $value );
+			$value = trim($value);
 		}
 
-		if ( ! $value ) {
+		if ( ! $value) {
 			return false;
 		}
 
-		switch ( $field ) {
+		switch ($field) {
 			case 'id':
 				$db_field = 'id';
 				break;
 			case 'email':
-				$value    = sanitize_text_field( $value );
+				$value    = sanitize_text_field($value);
 				$db_field = 'email';
 				break;
 			case 'user_id':
@@ -383,7 +383,7 @@ 
 				return false;
 		}
 
-		if ( ! $customer = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $db_field = %s LIMIT 1", $value ) ) ) {
+		if ( ! $customer = $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $db_field = %s LIMIT 1", $value))) {
 			return false;
 		}
 
@@ -396,7 +396,7 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function get_customers( $args = array() ) {
+	public function get_customers($args = array()) {
 
 		global $wpdb;
 
@@ -408,21 +408,21 @@ 
 			'order'   => 'DESC'
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		if ( $args['number'] < 1 ) {
+		if ($args['number'] < 1) {
 			$args['number'] = 999999999999;
 		}
 
 		$where = ' WHERE 1=1 ';
 
 		// specific customers
-		if ( ! empty( $args['id'] ) ) {
+		if ( ! empty($args['id'])) {
 
-			if ( is_array( $args['id'] ) ) {
-				$ids = implode( ',', array_map( 'intval', $args['id'] ) );
+			if (is_array($args['id'])) {
+				$ids = implode(',', array_map('intval', $args['id']));
 			} else {
-				$ids = intval( $args['id'] );
+				$ids = intval($args['id']);
 			}
 
 			$where .= " AND `id` IN( {$ids} ) ";
@@ -430,12 +430,12 @@ 
 		}
 
 		// customers for specific user accounts
-		if ( ! empty( $args['user_id'] ) ) {
+		if ( ! empty($args['user_id'])) {
 
-			if ( is_array( $args['user_id'] ) ) {
-				$user_ids = implode( ',', array_map( 'intval', $args['user_id'] ) );
+			if (is_array($args['user_id'])) {
+				$user_ids = implode(',', array_map('intval', $args['user_id']));
 			} else {
-				$user_ids = intval( $args['user_id'] );
+				$user_ids = intval($args['user_id']);
 			}
 
 			$where .= " AND `user_id` IN( {$user_ids} ) ";
@@ -443,41 +443,41 @@ 
 		}
 
 		//specific customers by email
-		if( ! empty( $args['email'] ) ) {
+		if ( ! empty($args['email'])) {
 
-			if( is_array( $args['email'] ) ) {
+			if (is_array($args['email'])) {
 
-				$emails_count       = count( $args['email'] );
-				$emails_placeholder = array_fill( 0, $emails_count, '%s' );
-				$emails             = implode( ', ', $emails_placeholder );
+				$emails_count       = count($args['email']);
+				$emails_placeholder = array_fill(0, $emails_count, '%s');
+				$emails             = implode(', ', $emails_placeholder);
 
-				$where .= $wpdb->prepare( " AND `email` IN( $emails ) ", $args['email'] );
+				$where .= $wpdb->prepare(" AND `email` IN( $emails ) ", $args['email']);
 			} else {
-				$where .= $wpdb->prepare( " AND `email` = %s ", $args['email'] );
+				$where .= $wpdb->prepare(" AND `email` = %s ", $args['email']);
 			}
 		}
 
 		// specific customers by name
-		if( ! empty( $args['name'] ) ) {
-			$where .= $wpdb->prepare( " AND `name` LIKE '%%%%" . '%s' . "%%%%' ", $args['name'] );
+		if ( ! empty($args['name'])) {
+			$where .= $wpdb->prepare(" AND `name` LIKE '%%%%".'%s'."%%%%' ", $args['name']);
 		}
 
 		// Customers created for a specific date or in a date range
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				if ( ! empty( $args['date']['start'] ) ) {
+				if ( ! empty($args['date']['start'])) {
 
-					$start = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
+					$start = date('Y-m-d H:i:s', strtotime($args['date']['start']));
 
 					$where .= " AND `date_created` >= '{$start}'";
 
 				}
 
-				if ( ! empty( $args['date']['end'] ) ) {
+				if ( ! empty($args['date']['end'])) {
 
-					$end = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+					$end = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 
 					$where .= " AND `date_created` <= '{$end}'";
 
@@ -485,31 +485,31 @@ 
 
 			} else {
 
-				$year  = date( 'Y', strtotime( $args['date'] ) );
-				$month = date( 'm', strtotime( $args['date'] ) );
-				$day   = date( 'd', strtotime( $args['date'] ) );
+				$year  = date('Y', strtotime($args['date']));
+				$month = date('m', strtotime($args['date']));
+				$day   = date('d', strtotime($args['date']));
 
 				$where .= " AND $year = YEAR ( date_created ) AND $month = MONTH ( date_created ) AND $day = DAY ( date_created )";
 			}
 
 		}
 
-		$args['orderby'] = ! array_key_exists( $args['orderby'], $this->get_columns() ) ? 'id' : $args['orderby'];
+		$args['orderby'] = ! array_key_exists($args['orderby'], $this->get_columns()) ? 'id' : $args['orderby'];
 
-		if ( 'purchase_value' == $args['orderby'] ) {
+		if ('purchase_value' == $args['orderby']) {
 			$args['orderby'] = 'purchase_value+0';
 		}
 
-		$cache_key = md5( 'give_customers_' . serialize( $args ) );
+		$cache_key = md5('give_customers_'.serialize($args));
 
-		$customers = wp_cache_get( $cache_key, 'customers' );
+		$customers = wp_cache_get($cache_key, 'customers');
 
-		$args['orderby'] = esc_sql( $args['orderby'] );
-		$args['order']   = esc_sql( $args['order'] );
+		$args['orderby'] = esc_sql($args['orderby']);
+		$args['order']   = esc_sql($args['order']);
 
-		if ( $customers === false ) {
-			$customers = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM  $this->table_name $where ORDER BY {$args['orderby']} {$args['order']} LIMIT %d,%d;", absint( $args['offset'] ), absint( $args['number'] ) ) );
-			wp_cache_set( $cache_key, $customers, 'customers', 3600 );
+		if ($customers === false) {
+			$customers = $wpdb->get_results($wpdb->prepare("SELECT * FROM  $this->table_name $where ORDER BY {$args['orderby']} {$args['order']} LIMIT %d,%d;", absint($args['offset']), absint($args['number'])));
+			wp_cache_set($cache_key, $customers, 'customers', 3600);
 		}
 
 		return $customers;
@@ -523,26 +523,26 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function count( $args = array() ) {
+	public function count($args = array()) {
 
 		global $wpdb;
 
 		$where = ' WHERE 1=1 ';
 
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				$start = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
-				$end   = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+				$start = date('Y-m-d H:i:s', strtotime($args['date']['start']));
+				$end   = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 
 				$where .= " AND `date_created` >= '{$start}' AND `date_created` <= '{$end}'";
 
 			} else {
 
-				$year  = date( 'Y', strtotime( $args['date'] ) );
-				$month = date( 'm', strtotime( $args['date'] ) );
-				$day   = date( 'd', strtotime( $args['date'] ) );
+				$year  = date('Y', strtotime($args['date']));
+				$month = date('m', strtotime($args['date']));
+				$day   = date('d', strtotime($args['date']));
 
 				$where .= " AND $year = YEAR ( date_created ) AND $month = MONTH ( date_created ) AND $day = DAY ( date_created )";
 			}
@@ -550,16 +550,16 @@ 
 		}
 
 
-		$cache_key = md5( 'give_customers_count' . serialize( $args ) );
+		$cache_key = md5('give_customers_count'.serialize($args));
 
-		$count = wp_cache_get( $cache_key, 'customers' );
+		$count = wp_cache_get($cache_key, 'customers');
 
-		if ( $count === false ) {
-			$count = $wpdb->get_var( "SELECT COUNT($this->primary_key) FROM " . $this->table_name . "{$where};" );
-			wp_cache_set( $cache_key, $count, 'customers', 3600 );
+		if ($count === false) {
+			$count = $wpdb->get_var("SELECT COUNT($this->primary_key) FROM ".$this->table_name."{$where};");
+			wp_cache_set($cache_key, $count, 'customers', 3600);
 		}
 
-		return absint( $count );
+		return absint($count);
 
 	}
 
@@ -571,9 +571,9 @@ 
 	 */
 	public function create_table() {
 		
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
-		$sql = "CREATE TABLE " . $this->table_name . " (
+		$sql = "CREATE TABLE ".$this->table_name." (
 		id bigint(20) NOT NULL AUTO_INCREMENT,
 		user_id bigint(20) NOT NULL,
 		email varchar(50) NOT NULL,
@@ -588,9 +588,9 @@ 
 		KEY user (user_id)
 		) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 	
 	/**
@@ -600,6 +600,6 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 }

Doc Comments +3 added lines, -3 removed lines

@@ -158,7 +158,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param  bool|string|int $_id_or_email
+	 * @param  integer $_id_or_email
 	 *
 	 * @return bool|int
 	 */
@@ -313,7 +313,7 @@ 
 	 * @param  int     $user_id       User ID.
 	 * @param  WP_User $old_user_data User data.
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function update_customer_email_on_user_update( $user_id = 0, $old_user_data ) {
 
@@ -362,7 +362,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $field ID or email. Default is 'id'.
-	 * @param  mixed  $value The Customer ID or email to search. Default is 0.
+	 * @param  integer  $value The Customer ID or email to search. Default is 0.
 	 *
 	 * @return mixed         Upon success, an object of the customer. Upon failure, NULL
 	 */

Indentation +10 added lines, -10 removed lines

@@ -427,13 +427,13 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  array $args
-     *
-     * @return array|object|null Customers array or object. Null if not found.
+	 *
+	 * @param  array $args
+	 *
+	 * @return array|object|null Customers array or object. Null if not found.
 	 */
 	public function get_customers( $args = array() ) {
-        /* @var WPDB $wpdb */
+		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		$defaults = array(
@@ -558,13 +558,13 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  array $args
-     *
-     * @return int         Total number of customers.
+	 *
+	 * @param  array $args
+	 *
+	 * @return int         Total number of customers.
 	 */
 	public function count( $args = array() ) {
-        /* @var WPDB $wpdb */
+		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		$where = ' WHERE 1=1 ';

includes/admin/reporting/tools/class-give-tools-reset-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $wpdb;

Spacing +64 added lines, -64 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,16 +55,16 @@ 
 	public function get_data() {
 		global $wpdb;
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids' );
+		$items = $this->get_stored_data('give_temp_reset_ids');
 
-		if ( ! is_array( $items ) ) {
+		if ( ! is_array($items)) {
 			return false;
 		}
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $items, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
 
-		if ( $step_items ) {
+		if ($step_items) {
 
 			$step_ids = array(
 				'customers'  => array(),
@@ -72,9 +72,9 @@ 
 				'other'      => array(),
 			);
 
-			foreach ( $step_items as $item ) {
+			foreach ($step_items as $item) {
 
-				switch ( $item['type'] ) {
+				switch ($item['type']) {
 					case 'customer':
 						$step_ids['customers'][] = $item['id'];
 						break;
@@ -82,8 +82,8 @@ 
 						$step_ids['give_forms'][] = $item['id'];
 						break;
 					default:
-						$item_type                = apply_filters( 'give_reset_item_type', 'other', $item );
-						$step_ids[ $item_type ][] = $item['id'];
+						$item_type                = apply_filters('give_reset_item_type', 'other', $item);
+						$step_ids[$item_type][] = $item['id'];
 						break;
 				}
 
@@ -91,17 +91,17 @@ 
 
 			$sql = array();
 
-			foreach ( $step_ids as $type => $ids ) {
+			foreach ($step_ids as $type => $ids) {
 
-				if ( empty( $ids ) ) {
+				if (empty($ids)) {
 					continue;
 				}
 
-				$ids = implode( ',', $ids );
+				$ids = implode(',', $ids);
 
-				switch ( $type ) {
+				switch ($type) {
 					case 'customers':
-						$table_name = $wpdb->prefix . 'give_customers';
+						$table_name = $wpdb->prefix.'give_customers';
 						$sql[]      = "DELETE FROM $table_name WHERE id IN ($ids)";
 						break;
 					case 'forms':
@@ -116,18 +116,18 @@ 
 						break;
 				}
 
-				if ( ! in_array( $type, array( 'customers', 'forms', 'other' ) ) ) {
+				if ( ! in_array($type, array('customers', 'forms', 'other'))) {
 					// Allows other types of custom post types to filter on their own post_type
 					// and add items to the query list, for the IDs found in their post type.
-					$sql = apply_filters( "give_reset_add_queries_{$type}", $sql, $ids );
+					$sql = apply_filters("give_reset_add_queries_{$type}", $sql, $ids);
 				}
 
 		
 			}
 
-			if ( ! empty( $sql ) ) {
-				foreach ( $sql as $query ) {
-					$wpdb->query( $query );
+			if ( ! empty($sql)) {
+				foreach ($sql as $query) {
+					$wpdb->query($query);
 				}
 			}
 
@@ -147,16 +147,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids', false );
-		$total = count( $items );
+		$items = $this->get_stored_data('give_temp_reset_ids', false);
+		$total = count($items);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -170,7 +170,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -181,30 +181,30 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to reset data.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to reset data.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', 0 );
-			delete_transient( 'give_earnings_total' );
-			delete_transient( 'give_estimated_monthly_stats' . true );
-			delete_transient( 'give_estimated_monthly_stats' . false );
-			$this->delete_data( 'give_temp_reset_ids' );
+			update_option('give_earnings_total', 0);
+			delete_transient('give_earnings_total');
+			delete_transient('give_estimated_monthly_stats'.true);
+			delete_transient('give_estimated_monthly_stats'.false);
+			$this->delete_data('give_temp_reset_ids');
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = esc_html__( 'Donation forms, income, donations counts, and logs successfully reset.', 'give' );
+			$this->message = esc_html__('Donation forms, income, donations counts, and logs successfully reset.', 'give');
 
 			return false;
 		}
@@ -214,10 +214,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -241,35 +241,35 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_reset_ids' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_reset_ids');
 		}
 
-		$items = get_option( 'give_temp_reset_ids', false );
+		$items = get_option('give_temp_reset_ids', false);
 
-		if ( false === $items ) {
+		if (false === $items) {
 			$items = array();
 
-			$give_types_for_reset = array( 'give_forms', 'give_log', 'give_payment' );
-			$give_types_for_reset = apply_filters( 'give_reset_store_post_types', $give_types_for_reset );
+			$give_types_for_reset = array('give_forms', 'give_log', 'give_payment');
+			$give_types_for_reset = apply_filters('give_reset_store_post_types', $give_types_for_reset);
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_type'      => $give_types_for_reset,
 				'post_status'    => 'any',
-				'posts_per_page' => - 1,
-			) );
+				'posts_per_page' => -1,
+			));
 
-			$posts = get_posts( $args );
-			foreach ( $posts as $post ) {
+			$posts = get_posts($args);
+			foreach ($posts as $post) {
 				$items[] = array(
 					'id'   => (int) $post->ID,
 					'type' => $post->post_type,
 				);
 			}
 
-			$customer_args = array( 'number' => - 1 );
-			$customers     = Give()->customers->get_customers( $customer_args );
-			foreach ( $customers as $customer ) {
+			$customer_args = array('number' => -1);
+			$customers     = Give()->customers->get_customers($customer_args);
+			foreach ($customers as $customer) {
 				$items[] = array(
 					'id'   => (int) $customer->id,
 					'type' => 'customer',
@@ -278,9 +278,9 @@ 
 
 			// Allow filtering of items to remove with an unassociative array for each item
 			// The array contains the unique ID of the item, and a 'type' for you to use in the execution of the get_data method
-			$items = apply_filters( 'give_reset_store_items', $items );
+			$items = apply_filters('give_reset_store_items', $items);
 
-			$this->store_data( 'give_temp_reset_ids', $items );
+			$this->store_data('give_temp_reset_ids', $items);
 		}
 
 	}
@@ -294,11 +294,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -311,10 +311,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -328,7 +328,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -340,9 +340,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

includes/payments/class-give-payment.php

Doc Comments +3 added lines, -3 removed lines

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

Indentation +94 added lines, -94 removed lines

@@ -513,22 +513,22 @@ 
 		return true;
 	}
 
-    /**
-     * Payment class object is storing various meta value in object parameter.
-     * So if user is updating payment meta but not updating payment object, then payment meta values will not reflect/changes on payment meta automatically
-     * and you can still access payment meta old value in any old payment object ( previously created ) which can cause to show or save wrong payment data.
-     * To prevent that user can use this function after updating any payment meta value ( in bulk or single update ).
-     *  
-     * @since  1.6
-     * @access public
-     * 
-     * @param  int $payment_id Payment ID.
-     * 
-     * @return void
-     */
-    public function update_payment_setup( $payment_id ){
-        $this->setup_payment( $payment_id );
-    }
+	/**
+	 * Payment class object is storing various meta value in object parameter.
+	 * So if user is updating payment meta but not updating payment object, then payment meta values will not reflect/changes on payment meta automatically
+	 * and you can still access payment meta old value in any old payment object ( previously created ) which can cause to show or save wrong payment data.
+	 * To prevent that user can use this function after updating any payment meta value ( in bulk or single update ).
+	 *  
+	 * @since  1.6
+	 * @access public
+	 * 
+	 * @param  int $payment_id Payment ID.
+	 * 
+	 * @return void
+	 */
+	public function update_payment_setup( $payment_id ){
+		$this->setup_payment( $payment_id );
+	}
 
 	/**
 	 * Create the base of a payment.
@@ -650,7 +650,7 @@ 
 	 *
 	 * Once items have been set, an update is needed to save them to the database.
 	 *
-     * @access public
+	 * @access public
 	 *
 	 * @return bool  True of the save occurred, false if it failed or wasn't needed
 	 */
@@ -937,7 +937,7 @@ 
 	 * Add a donation to a given payment
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  int   $form_id The donation form to add
 	 * @param  array $args    Other arguments to pass to the function
@@ -977,7 +977,7 @@ 
 					//Find a match between price_id and level_id
 					//First verify array keys exists THEN make the match
 					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
-					     && $args['price_id'] == $price['_give_id']['level_id']
+						 && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}
@@ -1033,7 +1033,7 @@ 
 	 * Remove a donation from the payment
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  int   $form_id The form ID to remove
 	 * @param  array $args    Arguments to pass to identify (quantity, amount, price_id)
@@ -1075,7 +1075,7 @@ 
 	 * Add a fee to a given payment
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  array $args   Array of arguments for the fee to add
 	 * @param  bool  $global
@@ -1110,7 +1110,7 @@ 
 	 * Remove a fee from the payment
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  int $key The array key index to remove
 	 *
@@ -1130,7 +1130,7 @@ 
 	 * Remove a fee by the defined attributed
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  string     $key    The key to remove by
 	 * @param  int|string $value  The value to search for
@@ -1200,7 +1200,7 @@ 
 	 * Get the fees, filterable by type
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  string $type All, item, fee
 	 *
@@ -1230,7 +1230,7 @@ 
 	 * Add a note to a payment
 	 *
 	 * @since  1.0
-     * @access public
+	 * @access public
 	 *
 	 * @param  string $note The note to add
 	 *
@@ -1249,7 +1249,7 @@ 
 	 * Increase the payment's subtotal
 	 *
 	 * @since  1.5
-     * @access private
+	 * @access private
 	 *
 	 * @param  float $amount The amount to increase the payment subtotal by
 	 *
@@ -1266,7 +1266,7 @@ 
 	 * Decrease the payment's subtotal
 	 *
 	 * @since  1.5
-     * @access private
+	 * @access private
 	 *
 	 * @param  float $amount The amount to decrease the payment subtotal by
 	 *
@@ -1287,7 +1287,7 @@ 
 	 * Increase the payment's subtotal
 	 *
 	 * @since  1.5
-     * @access private
+	 * @access private
 	 *
 	 * @param  float $amount The amount to increase the payment subtotal by
 	 *
@@ -1304,7 +1304,7 @@ 
 	 * Decrease the payment's subtotal
 	 *
 	 * @since  1.5
-     * @access private
+	 * @access private
 	 *
 	 * @param  float $amount The amount to decrease the payment subtotal by
 	 *
@@ -1325,7 +1325,7 @@ 
 	 * Set or update the total for a payment
 	 *
 	 * @since  1.0
-     * @access private
+	 * @access private
 	 *
 	 * @return void
 	 */
@@ -1337,7 +1337,7 @@ 
 	 * Set the payment status and run any status specific changes necessary
 	 *
 	 * @since  1.0
-     * @access public
+	 * @access public
 	 *
 	 * @param  string $status  The status to set the payment to
 	 *
@@ -1387,12 +1387,12 @@ 
 				case 'pending':
 					$this->process_pending();
 					break;
-                case 'cancelled':
-                    $this->process_cancelled();
-                    break;
-                case 'revoked':
-                    $this->process_revoked();
-                    break;
+				case 'cancelled':
+					$this->process_cancelled();
+					break;
+				case 'revoked':
+					$this->process_revoked();
+					break;
 			}
 
 			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
@@ -1407,7 +1407,7 @@ 
 	 * Change the status of the payment to refunded, and run the necessary changes
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @return void
 	 */
@@ -1423,7 +1423,7 @@ 
 	 * Get a post meta item for the payment
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  string  $meta_key The Meta Key
 	 * @param  boolean $single   Return single item or array
@@ -1462,7 +1462,7 @@ 
 	 * Update the post meta
 	 *
 	 * @since  1.5
-     * @access public
+	 * @access public
 	 *
 	 * @param  string $meta_key   The meta key to update
 	 * @param  string $meta_value The meta value
@@ -1588,77 +1588,77 @@ 
 		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
 	}
 
-    /**
-     * Process when a payment moves to cancelled
-     *
-     * @since  1.5
+	/**
+	 * Process when a payment moves to cancelled
+	 *
+	 * @since  1.5
 	 * @access private
 	 *
-     * @return void
-     */
-    private function process_cancelled() {
-        $process_cancelled = true;
+	 * @return void
+	 */
+	private function process_cancelled() {
+		$process_cancelled = true;
 
-        // If the payment was not in publish or revoked status, don't decrement stats as they were never incremented
-        if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
-            $process_cancelled = false;
-        }
+		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented
+		if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
+			$process_cancelled = false;
+		}
 
-        // Allow extensions to filter for their own payment types, Example: Recurring Payments
-        $process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
+		// Allow extensions to filter for their own payment types, Example: Recurring Payments
+		$process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
 
-        if ( false === $process_cancelled ) {
-            return;
-        }
+		if ( false === $process_cancelled ) {
+			return;
+		}
 
-        $decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_cancelled', true, $this );
-        $decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_cancelled', true, $this );
-        $decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_cancelled', true, $this );
+		$decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_cancelled', true, $this );
+		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_cancelled', true, $this );
+		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_cancelled', true, $this );
 
-        $this->maybe_alter_stats( $decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count );
-        $this->delete_sales_logs();
+		$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->completed_date = false;
+		$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' ) );
-    }
+		// Clear the This Month earnings (this_monththis_month is NOT a typo)
+		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+	}
 
-    /**
-     * Process when a payment moves to revoked
-     *
-     * @since  1.5
-     * @return void
-     */
-    private function process_revoked() {
-        $process_revoked = true;
+	/**
+	 * Process when a payment moves to revoked
+	 *
+	 * @since  1.5
+	 * @return void
+	 */
+	private function process_revoked() {
+		$process_revoked = true;
 
-        // If the payment was not in publish, don't decrement stats as they were never incremented
-        if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
-            $process_revoked = false;
-        }
+		// If the payment was not in publish, don't decrement stats as they were never incremented
+		if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
+			$process_revoked = false;
+		}
 
-        // Allow extensions to filter for their own payment types, Example: Recurring Payments
-        $process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
+		// Allow extensions to filter for their own payment types, Example: Recurring Payments
+		$process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
 
-        if ( false === $process_revoked ) {
-            return;
-        }
+		if ( false === $process_revoked ) {
+			return;
+		}
 
-        $decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_revoked', true, $this );
-        $decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_revoked', true, $this );
-        $decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_revoked', true, $this );
+		$decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_revoked', true, $this );
+		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_revoked', true, $this );
+		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_revoked', true, $this );
 
-        $this->maybe_alter_stats( $decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count );
-        $this->delete_sales_logs();
+		$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->completed_date = false;
+		$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' ) );
-    }
+		// Clear the This Month earnings (this_monththis_month is NOT a typo)
+		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+	}
 
 	/**
 	 * Used during the process of moving to refunded or pending, to decrement stats

Spacing +322 added lines, -322 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -347,13 +347,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);
 	}
 
 	/**
@@ -366,11 +366,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 {
 
@@ -392,18 +392,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;
 		}
 	}
@@ -418,9 +418,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;
 		}
@@ -436,20 +436,20 @@ 
 	 *
 	 * @return bool            If the setup was successful or not
 	 */
-	private function setup_payment( $payment_id ) {
+	private function setup_payment($payment_id) {
 		$this->pending = array();
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = get_post( $payment_id );
+		$payment = get_post($payment_id);
 
-		if ( ! $payment || is_wp_error( $payment ) ) {
+		if ( ! $payment || is_wp_error($payment)) {
 			return false;
 		}
 
-		if ( 'give_payment' !== $payment->post_type ) {
+		if ('give_payment' !== $payment->post_type) {
 			return false;
 		}
 
@@ -463,13 +463,13 @@ 
 		 * @param Give_Payment $this       Payment object.
 		 * @param int          $payment_id The ID of the payment.
 		 */
-		do_action( 'give_pre_setup_payment', $this, $payment_id );
+		do_action('give_pre_setup_payment', $this, $payment_id);
 
 		// Primary Identifier
-		$this->ID = absint( $payment_id );
+		$this->ID = absint($payment_id);
 
 		// Protected ID that can never be changed
-		$this->_ID = absint( $payment_id );
+		$this->_ID = absint($payment_id);
 
 		// We have a payment, get the generic payment_meta item to reduce calls to it
 		$this->payment_meta = $this->get_meta();
@@ -484,7 +484,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();
@@ -526,7 +526,7 @@ 
 		 * @param Give_Payment $this       Payment object.
 		 * @param int          $payment_id The ID of the payment.
 		 */
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -544,8 +544,8 @@ 
      * 
      * @return void
      */
-    public function update_payment_setup( $payment_id ){
-        $this->setup_payment( $payment_id );
+    public function update_payment_setup($payment_id) {
+        $this->setup_payment($payment_id);
     }
 
 	/**
@@ -560,24 +560,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;
@@ -604,58 +604,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_donation_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_donation_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;
 		}
 
@@ -677,11 +677,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;
@@ -690,48 +690,48 @@ 
 		}
 
 		//Set ID if not matching
-		if ( $this->ID !== $this->_ID ) {
+		if ($this->ID !== $this->_ID) {
 			$this->ID = $this->_ID;
 		}
 
 		// If we have something pending, let's save it
-		if ( ! empty( $this->pending ) ) {
+		if ( ! empty($this->pending)) {
 
 			$total_increase = 0;
 			$total_decrease = 0;
 
-			foreach ( $this->pending as $key => $value ) {
+			foreach ($this->pending as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'donations':
 						// Update totals for pending donations
-						foreach ( $this->pending[ $key ] as $item ) {
+						foreach ($this->pending[$key] as $item) {
 
-							$quantity = isset( $item['quantity'] ) ? $item['quantity'] : 1;
-							$price_id = isset( $item['price_id'] ) ? $item['price_id'] : 0;
+							$quantity = isset($item['quantity']) ? $item['quantity'] : 1;
+							$price_id = isset($item['price_id']) ? $item['price_id'] : 0;
 
-							switch ( $item['action'] ) {
+							switch ($item['action']) {
 
 								case 'add':
 
 									$price = $item['price'];
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status) {
 
 										// Add sales logs
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
 
 										$y = 0;
-										while ( $y < $quantity ) {
+										while ($y < $quantity) {
 
-											give_record_sale_in_log( $item['id'], $this->ID, $price_id, $log_date );
-											$y ++;
+											give_record_sale_in_log($item['id'], $this->ID, $price_id, $log_date);
+											$y++;
 										}
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price);
 
 										$total_increase += $price;
 									}
@@ -756,15 +756,15 @@ 
 										)
 									);
 
-									$found_logs = get_posts( $log_args );
-									foreach ( $found_logs as $log ) {
-										wp_delete_post( $log->ID, true );
+									$found_logs = get_posts($log_args);
+									foreach ($found_logs as $log) {
+										wp_delete_post($log->ID, true);
 									}
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'] );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -777,17 +777,17 @@ 
 
 					case 'fees':
 
-						if ( 'publish' !== $this->status && 'complete' !== $this->status ) {
+						if ('publish' !== $this->status && 'complete' !== $this->status) {
 							break;
 						}
 
-						if ( empty( $this->pending[ $key ] ) ) {
+						if (empty($this->pending[$key])) {
 							break;
 						}
 
-						foreach ( $this->pending[ $key ] as $fee ) {
+						foreach ($this->pending[$key] as $fee) {
 
-							switch ( $fee['action'] ) {
+							switch ($fee['action']) {
 
 								case 'add':
 									$total_increase += $fee['amount'];
@@ -804,43 +804,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':
@@ -856,15 +856,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':
@@ -874,11 +874,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':
@@ -887,7 +887,7 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
@@ -898,34 +898,34 @@ 
 						 *
 						 * @param Give_Payment $this Payment object.
 						 */
-						do_action( 'give_donation_save', $this, $key );
+						do_action('give_donation_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 donation 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 donation stats
-					$customer->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$customer->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 				}
 
 			}
 
-			$this->update_meta( '_give_payment_total', $this->total );
+			$this->update_meta('_give_payment_total', $this->total);
 
 			$new_meta = array(
 				'form_title' => $this->form_title,
@@ -937,12 +937,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;
 				}
 			}
@@ -951,8 +951,8 @@ 
 			$saved         = true;
 		}
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -970,12 +970,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;
 		}
 
@@ -986,60 +986,60 @@ 
 			'fees'     => array(),
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_donation_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_donation_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',
@@ -1048,7 +1048,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1065,7 +1065,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(
@@ -1073,12 +1073,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;
 		}
 
@@ -1091,7 +1091,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1107,7 +1107,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'    => '',
@@ -1117,16 +1117,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;
 	}
@@ -1141,11 +1141,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;
@@ -1164,47 +1164,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_donation_fee_keys', array(
+		$allowed_fee_keys = apply_filters('give_donation_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;
 					}
 
@@ -1214,8 +1214,8 @@ 
 
 		}
 
-		if ( true === $removed ) {
-			$this->fees = array_values( $this->fees );
+		if (true === $removed) {
+			$this->fees = array_values($this->fees);
 		}
 
 		return $removed;
@@ -1231,14 +1231,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;
 				}
 
@@ -1248,7 +1248,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_get_payment_fees', $fees, $this->ID, $this );
+		return apply_filters('give_get_payment_fees', $fees, $this->ID, $this);
 	}
 
 	/**
@@ -1261,13 +1261,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);
 	}
 
 	/**
@@ -1280,7 +1280,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
+	private function increase_subtotal($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
@@ -1297,11 +1297,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;
 		}
 
@@ -1318,7 +1318,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_fees( $amount = 0.00 ) {
+	private function increase_fees($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->fees_total += $amount;
 
@@ -1335,11 +1335,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;
 		}
 
@@ -1368,25 +1368,25 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		//standardize the 'complete(d)' status
-		if ( $status == 'completed' || $status == 'complete' ) {
+		if ($status == 'completed' || $status == 'complete') {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
 
-		if ( $do_change ) {
+		if ($do_change) {
 
 			/**
 			 * Fires before changing payment status.
@@ -1397,21 +1397,21 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' )
+				'edit_date'   => current_time('mysql')
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions
-			switch ( $status ) {
+			switch ($status) {
 				case 'refunded':
 					$this->process_refund();
 					break;
@@ -1438,7 +1438,7 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		}
 
@@ -1473,32 +1473,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);
 	}
 
 	/**
@@ -1513,23 +1513,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;
@@ -1539,9 +1539,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);
 	}
 
 	/**
@@ -1556,14 +1556,14 @@ 
 		$process_refund = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented
-		if ( 'publish' != $this->old_status || 'refunded' != $this->status ) {
+		if ('publish' != $this->old_status || 'refunded' != $this->status) {
 			$process_refund = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments
-		$process_refund = apply_filters( 'give_should_process_refund', $process_refund, $this );
+		$process_refund = apply_filters('give_should_process_refund', $process_refund, $this);
 
-		if ( false === $process_refund ) {
+		if (false === $process_refund) {
 			return;
 		}
 
@@ -1574,17 +1574,17 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		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'));
 
 		/**
 		 * Fires after refunding payment.
@@ -1593,7 +1593,7 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1620,29 +1620,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 || 'pending' != $this->status ) {
+		if ('publish' != $this->old_status || 'pending' != $this->status) {
 			$process_pending = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments
-		$process_pending = apply_filters( 'give_should_process_pending', $process_pending, $this );
+		$process_pending = apply_filters('give_should_process_pending', $process_pending, $this);
 
-		if ( false === $process_pending ) {
+		if (false === $process_pending) {
 			return;
 		}
 
-		$decrease_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'));
 	}
 
     /**
@@ -1657,29 +1657,29 @@ 
         $process_cancelled = true;
 
         // If the payment was not in publish or revoked status, don't decrement stats as they were never incremented
-        if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
+        if ('publish' != $this->old_status || 'cancelled' != $this->status) {
             $process_cancelled = false;
         }
 
         // Allow extensions to filter for their own payment types, Example: Recurring Payments
-        $process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
+        $process_cancelled = apply_filters('give_should_process_cancelled', $process_cancelled, $this);
 
-        if ( false === $process_cancelled ) {
+        if (false === $process_cancelled) {
             return;
         }
 
-        $decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_cancelled', true, $this );
-        $decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_cancelled', true, $this );
-        $decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_cancelled', true, $this );
+        $decrease_store_earnings = apply_filters('give_decrease_store_earnings_on_cancelled', true, $this);
+        $decrease_customer_value = apply_filters('give_decrease_customer_value_on_cancelled', true, $this);
+        $decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_cancelled', 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'));
     }
 
     /**
@@ -1692,29 +1692,29 @@ 
         $process_revoked = true;
 
         // If the payment was not in publish, don't decrement stats as they were never incremented
-        if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
+        if ('publish' != $this->old_status || 'revoked' != $this->status) {
             $process_revoked = false;
         }
 
         // Allow extensions to filter for their own payment types, Example: Recurring Payments
-        $process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
+        $process_revoked = apply_filters('give_should_process_revoked', $process_revoked, $this);
 
-        if ( false === $process_revoked ) {
+        if (false === $process_revoked) {
             return;
         }
 
-        $decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_revoked', true, $this );
-        $decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_revoked', true, $this );
-        $decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_revoked', true, $this );
+        $decrease_store_earnings = apply_filters('give_decrease_store_earnings_on_revoked', true, $this);
+        $decrease_customer_value = apply_filters('give_decrease_customer_value_on_revoked', true, $this);
+        $decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_revoked', 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'));
     }
 
 	/**
@@ -1729,25 +1729,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();
 			}
 
@@ -1797,13 +1797,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;
 	}
@@ -1817,7 +1817,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1829,13 +1829,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'];
 			}
 		}
@@ -1868,9 +1868,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'];
 			}
 		}
@@ -1888,7 +1888,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_donation_currency_default', give_get_currency(), $this );
+		$currency = isset($this->payment_meta['currency']) ? $this->payment_meta['currency'] : apply_filters('give_donation_currency_default', give_get_currency(), $this);
 
 		return $currency;
 	}
@@ -1902,7 +1902,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;
 	}
@@ -1916,7 +1916,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;
 	}
@@ -1930,12 +1930,12 @@ 
 	 * @return string The donation ID
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) || (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);
 
 		}
 
@@ -1951,7 +1951,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;
 	}
@@ -1965,7 +1965,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;
 	}
@@ -1979,7 +1979,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;
 	}
@@ -1993,10 +1993,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;
@@ -2016,15 +2016,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],
@@ -2034,29 +2034,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;
 					}
 				}
@@ -2078,7 +2078,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'    => '',
@@ -2100,7 +2100,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;
 	}
@@ -2115,7 +2115,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;
 	}
@@ -2129,7 +2129,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;
 	}
@@ -2143,7 +2143,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;
 	}
@@ -2159,11 +2159,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;
 
@@ -2182,7 +2182,7 @@ 
 	 * @return array The payment object as an array
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 	/**
@@ -2194,7 +2194,7 @@ 
 	 * @return string Date payment was completed
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_donation_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_donation_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -2206,7 +2206,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);
 	}
 
 	/**
@@ -2218,7 +2218,7 @@ 
 	 * @return string Payment currency code
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_donation_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_donation_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -2230,7 +2230,7 @@ 
 	 * @return string Gateway used
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_donation_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_donation_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -2242,7 +2242,7 @@ 
 	 * @return string Donation ID from merchant processor
 	 */
 	private function get_transaction_id() {
-		return apply_filters( 'give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this );
+		return apply_filters('give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this);
 	}
 
 	/**
@@ -2254,7 +2254,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_donation_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_donation_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -2266,7 +2266,7 @@ 
 	 * @return int Payment customer ID
 	 */
 	private function get_customer_id() {
-		return apply_filters( 'give_donation_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_donation_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -2278,7 +2278,7 @@ 
 	 * @return int Payment user ID
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_donation_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_donation_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -2290,7 +2290,7 @@ 
 	 * @return string Payment customer email
 	 */
 	private function get_email() {
-		return apply_filters( 'give_donation_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_donation_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2302,7 +2302,7 @@ 
 	 * @return array Payment user info
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_donation_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_donation_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2314,7 +2314,7 @@ 
 	 * @return array Payment billing address
 	 */
 	private function get_address() {
-		return apply_filters( 'give_donation_address', $this->address, $this->ID, $this );
+		return apply_filters('give_donation_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2326,7 +2326,7 @@ 
 	 * @return string Payment key
 	 */
 	private function get_key() {
-		return apply_filters( 'give_donation_key', $this->key, $this->ID, $this );
+		return apply_filters('give_donation_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2338,7 +2338,7 @@ 
 	 * @return string Payment form id
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_donation_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_donation_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2350,7 +2350,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_donation_number', $this->number, $this->ID, $this );
+		return apply_filters('give_donation_number', $this->number, $this->ID, $this);
 	}
 
 }