WordImpress / Give

includes/admin/donors/donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
 /**
  * This function is used to delete donor and related donation without redirection.
  *
- * @param int|Give_Donor $donor Donor ID or List of Donor IDs.
+ * @param Give_Donor $donor Donor ID or List of Donor IDs.
  * @param array          $args  List of arguments to handle donor and related donation deletion process.
  *
  * @type bool delete_donation Delete donor linked donations if set to true. Default is false.

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title'    => __( 'Donor Profile', 'give' ),
+			'title'    => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title'    => __( 'Donor Notes', 'give' ),
+			'title'    => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,17 +73,17 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title'    => __( 'Delete Donor', 'give' ),
+		'title'    => __('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);
 
 /**
  * Connect and Reconnect Donor with User profile.
@@ -98,7 +98,7 @@ 
  *
  * @return array
  */
-function give_connect_user_donor_profile( $donor, $donor_data, $address ) {
+function give_connect_user_donor_profile($donor, $donor_data, $address) {
 
 	$donor_id         = $donor->id;
 	$previous_user_id = $donor->user_id;
@@ -112,32 +112,32 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
 		// Create and Update Donor First Name and Last Name in Meta Fields.
-		$donor->update_meta( '_give_donor_first_name', $donor_data['first_name'] );
-		$donor->update_meta( '_give_donor_last_name', $donor_data['last_name'] );
-		$donor->update_meta( '_give_donor_title_prefix', $donor_data['title'] );
+		$donor->update_meta('_give_donor_first_name', $donor_data['first_name']);
+		$donor->update_meta('_give_donor_last_name', $donor_data['last_name']);
+		$donor->update_meta('_give_donor_title_prefix', $donor_data['title']);
 
 		// Fetch disconnected user id, if exists.
-		$disconnected_user_id = $donor->get_meta( '_give_disconnected_user_id', true );
+		$disconnected_user_id = $donor->get_meta('_give_disconnected_user_id', true);
 
 		// Flag User and Donor Disconnection.
-		delete_user_meta( $disconnected_user_id, '_give_is_donor_disconnected' );
+		delete_user_meta($disconnected_user_id, '_give_is_donor_disconnected');
 
 		// Check whether the disconnected user id and the reconnected user id are same or not.
 		// If both are same then delete user id store in donor meta.
-		if ( $donor_data['user_id'] === $disconnected_user_id ) {
-			delete_user_meta( $disconnected_user_id, '_give_disconnected_donor_id' );
-			$donor->delete_meta( '_give_disconnected_user_id' );
+		if ($donor_data['user_id'] === $disconnected_user_id) {
+			delete_user_meta($disconnected_user_id, '_give_disconnected_donor_id');
+			$donor->delete_meta('_give_disconnected_user_id');
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -154,7 +154,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
 	return $output;
 }
@@ -171,39 +171,39 @@ 
  *
  * @return int
  */
-function give_delete_donor_and_related_donation( $donor, $args = array() ) {
+function give_delete_donor_and_related_donation($donor, $args = array()) {
 
 	// Default Arguments.
 	$default_args = array(
 		'delete_donation' => false,
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	// If $donor not an instance of Give_Donor then create one.
-	if ( ! $donor instanceof Give_Donor ) {
-		$donor = new Give_Donor( $donor );
+	if ( ! $donor instanceof Give_Donor) {
+		$donor = new Give_Donor($donor);
 	}
 
-	if ( $donor->id > 0 ) {
+	if ($donor->id > 0) {
 
 		// Delete Donor.
-		$donor_deleted = Give()->donors->delete( $donor->id );
+		$donor_deleted = Give()->donors->delete($donor->id);
 
 		// Fetch linked donations of a particular donor.
-		$donation_ids  = explode( ',', $donor->payment_ids );
+		$donation_ids  = explode(',', $donor->payment_ids);
 
 		// Proceed to delete related donation, if user opted and donor is deleted successfully.
-		if ( $donor_deleted && $args['delete_donation'] ) {
-			foreach ( $donation_ids as $donation_id ) {
-				give_delete_donation( $donation_id );
+		if ($donor_deleted && $args['delete_donation']) {
+			foreach ($donation_ids as $donation_id) {
+				give_delete_donation($donation_id);
 			}
 
 			return 2; // Donor and linked Donations deleted.
 
 		} else {
-			foreach ( $donation_ids as $donation_id ) {
-				give_update_payment_meta( $donation_id, '_give_payment_donor_id', 0 );
+			foreach ($donation_ids as $donation_id) {
+				give_update_payment_meta($donation_id, '_give_payment_donor_id', 0);
 			}
 		}
 

includes/admin/tools/data/class-give-tools-delete-import-donors.php

Doc Comments +1 added lines, -1 removed lines

@@ -290,7 +290,7 @@
 	 *
 	 * @since 1.8.12
 	 *
-	 * @return int
+	 * @return double
 	 */
 	public function get_percentage_complete() {
 		return ceil( ( 100 * $this->step_completed ) / $this->total_step );

Spacing +116 added lines, -116 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -134,8 +134,8 @@ 
 	 *
 	 * @param int $_step Steps.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -154,38 +154,38 @@ 
 		$donor_ids    = array();
 
 		// Check if the ajax request if running for the first time.
-		if ( 1 === (int) $this->step ) {
+		if (1 === (int) $this->step) {
 
 			// Delete all the form ids.
-			$this->delete_option( $this->form_key );
+			$this->delete_option($this->form_key);
 
 			// Delete all the donation ids.
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			// Delete all the donor ids.
-			$this->delete_option( $this->donor_key );
+			$this->delete_option($this->donor_key);
 
 			// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
-			$this->update_option( $this->step_key, 'count' );
+			$this->update_option($this->step_key, 'count');
 
 			// Delete tha page count of the step.
-			$this->update_option( $this->step_on_key, '0' );
+			$this->update_option($this->step_on_key, '0');
 		} else {
 
 			// Get the old donors list.
-			$donor_ids = $this->get_option( $this->donor_key );
+			$donor_ids = $this->get_option($this->donor_key);
 
 			// Get the old donation list.
-			$donation_ids = $this->get_option( $this->donation_key );
+			$donation_ids = $this->get_option($this->donation_key);
 		}
 
 		// Get the step and check for it if it's on the first step( 'count' ) or not.
 		$step = (int) $this->get_step();
-		if ( 1 === $step ) {
+		if (1 === $step) {
 			/**
 			 * Will add or update the donation and donor data by running wp query.
 			 */
-			$this->count( $step, $donation_ids, $donor_ids );
+			$this->count($step, $donation_ids, $donor_ids);
 		}
 	}
 
@@ -196,18 +196,18 @@ 
 	 * @param array  $donation_ids Contain the list of all the donation id's that has being add before this.
 	 * @param array  $donor_ids    Contain the list of all the donors id's that has being add before this.
 	 */
-	private function count( $step, $donation_ids = array(), $donor_ids = array() ) {
+	private function count($step, $donation_ids = array(), $donor_ids = array()) {
 
 		// Get the Page count by default it's zero.
 		$paged = (int) $this->get_step_page();
 
 		// Increased the page count by one.
-		++ $paged;
+		++$paged;
 
 		/**
 		 * Filter add to alter the argument before the wp quest run
 		 */
-		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+		$args = apply_filters('give_tools_reset_stats_total_args', array(
 			'post_type'      => 'give_payment',
 			'post_status'    => 'any',
 			'posts_per_page' => $this->per_step,
@@ -215,41 +215,41 @@ 
 			'meta_key'       => '_give_payment_import',
 			'meta_value_num' => 1,
 			'meta_compare'   => '=',
-		) );
+		));
 
 		// Reset the post data.
 		wp_reset_postdata();
 
 		// Getting the new donation.
-		$donation_posts = new WP_Query( $args );
+		$donation_posts = new WP_Query($args);
 
 		// The Loop.
-		if ( $donation_posts->have_posts() ) {
-			while ( $donation_posts->have_posts() ) {
+		if ($donation_posts->have_posts()) {
+			while ($donation_posts->have_posts()) {
 				$add_author = true;
 				$donation_posts->the_post();
 				global $post;
 				// Add the donation id in side the array.
 				$donation_ids[] = $post->ID;
 
-				$donor_id = (int) give_get_meta( $post->ID, '_give_payment_customer_id', true );
-				if ( ! empty( $donor_id ) ) {
-					$donor = new Give_Donor( $donor_id );
-					if ( ! empty( $donor->id ) ) {
-						if ( empty( $donor->user_id ) && ! empty( $donor->payment_ids ) ) {
+				$donor_id = (int) give_get_meta($post->ID, '_give_payment_customer_id', true);
+				if ( ! empty($donor_id)) {
+					$donor = new Give_Donor($donor_id);
+					if ( ! empty($donor->id)) {
+						if (empty($donor->user_id) && ! empty($donor->payment_ids)) {
 							$add_author = false;
-							$count      = (int) count( $donor->payment_ids );
-							if ( 1 === $count ) {
-								give_delete_donor_and_related_donation( $donor );
+							$count      = (int) count($donor->payment_ids);
+							if (1 === $count) {
+								give_delete_donor_and_related_donation($donor);
 							} else {
-								$donor->remove_payment( $post->ID );
+								$donor->remove_payment($post->ID);
 								$donor->decrease_donation_count();
 							}
 						}
 					}
 				}
 
-				if ( ! empty( $add_author ) ) {
+				if ( ! empty($add_author)) {
 					// Add the donor id in side the array.
 					$donor_ids[] = (int) $post->post_author;
 				}
@@ -263,24 +263,24 @@ 
 		$max_num_pages = (int) $donation_posts->max_num_pages;
 
 		// Check current page is less then max number of page or not.
-		if ( $paged < $max_num_pages ) {
+		if ($paged < $max_num_pages) {
 
 			// Update the current page variable for the next step.
-			$this->update_option( $this->step_on_key, $paged );
+			$this->update_option($this->step_on_key, $paged);
 
 			// Calculating percentage.
 			$page_remain          = $max_num_pages - $paged;
-			$this->total_step     = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
+			$this->total_step     = (int) $max_num_pages + ($total_donation / $this->per_step) + (($page_remain * 2) * count($donor_ids));
 			$this->step_completed = $paged;
 		} else {
-			$donation_ids_count = count( $donor_ids );
-			$this->update_option( $this->step_key, 'donation' );
-			$this->update_option( $this->step_on_key, '0' );
+			$donation_ids_count = count($donor_ids);
+			$this->update_option($this->step_key, 'donation');
+			$this->update_option($this->step_on_key, '0');
 		}
 
-		$donor_ids = array_unique( $donor_ids );
-		$this->update_option( $this->donor_key, $donor_ids );
-		$this->update_option( $this->donation_key, $donation_ids );
+		$donor_ids = array_unique($donor_ids);
+		$this->update_option($this->donor_key, $donor_ids);
+		$this->update_option($this->donation_key, $donation_ids);
 
 		wp_reset_postdata();
 	}
@@ -293,7 +293,7 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
+		return ceil((100 * $this->step_completed) / $this->total_step);
 	}
 
 	/**
@@ -303,10 +303,10 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
+		if ( ! $this->can_export()) {
 			wp_die(
-				esc_html__( 'You do not have permission to delete Import transactions.', 'give' ),
-				esc_html__( 'Error', 'give' ),
+				esc_html__('You do not have permission to delete Import transactions.', 'give'),
+				esc_html__('Error', 'give'),
 				array(
 					'response' => 403,
 				)
@@ -315,18 +315,18 @@ 
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', give_get_total_earnings( true ), false );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true), false);
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			$this->done    = true;
-			$this->message = __( 'Imported donor and transactions successfully deleted.', 'give' );
+			$this->message = __('Imported donor and transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -344,12 +344,12 @@ 
 	public function get_data() {
 
 		// Get the donation id's.
-		$donation_ids = $this->get_option( $this->donation_key );
+		$donation_ids = $this->get_option($this->donation_key);
 
 		/**
 		 * Return false id not Import donation is found.
 		 */
-		if ( empty( $donation_ids ) ) {
+		if (empty($donation_ids)) {
 			$this->is_empty   = true;
 			$this->total_step = 1;
 
@@ -360,110 +360,110 @@ 
 		$step = (int) $this->get_step();
 
 		// Get the donor ids.
-		$donor_ids = $this->get_option( $this->donor_key );
+		$donor_ids = $this->get_option($this->donor_key);
 
 		// Delete all the imported donations.
-		if ( 2 === $step ) {
+		if (2 === $step) {
 			$pass_to_donor = false;
 			$page          = (int) $this->get_step_page();
-			$page ++;
-			$count = count( $donation_ids );
+			$page++;
+			$count = count($donation_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
 			$this->step_completed = $page;
 
-			if ( $count > $this->per_step ) {
+			if ($count > $this->per_step) {
 
-				$this->update_option( $this->step_on_key, $page );
-				$donation_ids = $this->get_delete_ids( $donation_ids, $page );
-				$current_page = (int) ceil( $count / $this->per_step );
+				$this->update_option($this->step_on_key, $page);
+				$donation_ids = $this->get_delete_ids($donation_ids, $page);
+				$current_page = (int) ceil($count / $this->per_step);
 
-				if ( $page === $current_page ) {
+				if ($page === $current_page) {
 					$pass_to_donor = true;
 				}
 			} else {
 				$pass_to_donor = true;
 			}
 
-			if ( true === $pass_to_donor ) {
-				$this->update_option( $this->step_key, 'donor' );
-				$this->update_option( $this->step_on_key, '0' );
+			if (true === $pass_to_donor) {
+				$this->update_option($this->step_key, 'donor');
+				$this->update_option($this->step_on_key, '0');
 			}
 
 			// Get the old form list.
-			$form_ids = (array) $this->get_option( $this->form_key );
+			$form_ids = (array) $this->get_option($this->form_key);
 
-			foreach ( $donation_ids as $item ) {
-				$form_ids[] = give_get_meta( $item, '_give_payment_form_id', true );
+			foreach ($donation_ids as $item) {
+				$form_ids[] = give_get_meta($item, '_give_payment_form_id', true);
 
 				// Delete the main payment.
-				give_delete_donation( absint( $item ) );
+				give_delete_donation(absint($item));
 			}
 
 			// Update the new form list.
-			$this->update_option( $this->form_key, $form_ids );
+			$this->update_option($this->form_key, $form_ids);
 		} // End if().
 
 		// Delete all the donors.
-		if ( 3 === $step ) {
+		if (3 === $step) {
 
 			// Get the old form list.
-			$form_ids = (array) $this->get_option( $this->form_key );
-			if ( ! empty( $form_ids ) ) {
-				$form_ids = array_unique( $form_ids );
-				foreach ( $form_ids as $form_id ) {
-					give_recount_form_income_donation( (int) $form_id );
+			$form_ids = (array) $this->get_option($this->form_key);
+			if ( ! empty($form_ids)) {
+				$form_ids = array_unique($form_ids);
+				foreach ($form_ids as $form_id) {
+					give_recount_form_income_donation((int) $form_id);
 				}
 			}
 			// update the new form list.
-			$this->update_option( $this->form_key, array() );
+			$this->update_option($this->form_key, array());
 
 			$page  = (int) $this->get_step_page();
-			$count = count( $donor_ids );
+			$count = count($donor_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
-			$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
+			$this->step_completed = $page + (count($donation_ids) / $this->per_step);
 
-			if ( ! empty( $donor_ids[ $page ] ) ) {
-				$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			if ( ! empty($donor_ids[$page])) {
+				$args = apply_filters('give_tools_reset_stats_total_args', array(
 					'post_status'    => 'any',
 					'posts_per_page' => 1,
-					'author'         => $donor_ids[ $page ],
-				) );
+					'author'         => $donor_ids[$page],
+				));
 
 				$donations = array();
-				$payments  = new Give_Payments_Query( $args );
+				$payments  = new Give_Payments_Query($args);
 				$payments  = $payments->get_payments();
-				if ( empty( $payments ) ) {
-					Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
+				if (empty($payments)) {
+					Give()->donors->delete_by_user_id($donor_ids[$page]);
 
 					/**
 					 * If Checked then delete WP user.
 					 *
 					 * @since 1.8.14
 					 */
-					$delete_import_donors = isset( $_REQUEST['delete-import-donors'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['delete-import-donors'] ) ) : '';
+					$delete_import_donors = isset($_REQUEST['delete-import-donors']) ? sanitize_text_field(wp_unslash($_REQUEST['delete-import-donors'])) : '';
 
-					if ( 'on' === (string) $delete_import_donors ) {
-						wp_delete_user( $donor_ids[ $page ] );
+					if ('on' === (string) $delete_import_donors) {
+						wp_delete_user($donor_ids[$page]);
 					}
 				} else {
-					foreach ( $payments as $payment ) {
+					foreach ($payments as $payment) {
 						$donations[] = $payment->ID;
 					}
 
-					$donor          = new Give_Donor( $donor_ids[ $page ], true );
+					$donor          = new Give_Donor($donor_ids[$page], true);
 					$data_to_update = array(
-						'purchase_count' => count( $donations ),
-						'payment_ids'    => implode( ',', $donations ),
+						'purchase_count' => count($donations),
+						'payment_ids'    => implode(',', $donations),
 					);
-					$donor->update( $data_to_update );
+					$donor->update($data_to_update);
 				}
 			} // End if().
 
-			$page ++;
-			$this->update_option( $this->step_on_key, $page );
-			if ( $count === $page ) {
+			$page++;
+			$this->update_option($this->step_on_key, $page);
+			if ($count === $page) {
 				$this->is_empty = false;
 
 				return false;
@@ -483,24 +483,24 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_delete_ids( $donation_ids, $page ) {
-		$index            = $page --;
-		$count            = count( $donation_ids );
+	public function get_delete_ids($donation_ids, $page) {
+		$index            = $page--;
+		$count            = count($donation_ids);
 		$temp             = 0;
 		$current_page     = 0;
 		$post_delete      = $this->per_step;
 		$page_donation_id = array();
 
-		foreach ( $donation_ids as $item ) {
-			$temp ++;
-			$page_donation_id[ $current_page ][] = $item;
-			if ( $temp === $post_delete ) {
-				$current_page ++;
+		foreach ($donation_ids as $item) {
+			$temp++;
+			$page_donation_id[$current_page][] = $item;
+			if ($temp === $post_delete) {
+				$current_page++;
 				$temp = 0;
 			}
 		}
 
-		return $page_donation_id[ $page ];
+		return $page_donation_id[$page];
 	}
 
 	/**
@@ -513,8 +513,8 @@ 
 	 *
 	 * @return mixed Returns the data from the database
 	 */
-	public function get_option( $key, $default_value = false ) {
-		return get_option( $key, $default_value );
+	public function get_option($key, $default_value = false) {
+		return get_option($key, $default_value);
 	}
 
 	/**
@@ -527,8 +527,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_option( $key, $value ) {
-		update_option( $key, $value, false );
+	public function update_option($key, $value) {
+		update_option($key, $value, false);
 	}
 
 	/**
@@ -540,8 +540,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -552,12 +552,12 @@ 
 	 * @return int|string
 	 */
 	private function get_step() {
-		$step_key = (string) $this->get_option( $this->step_key, false );
-		if ( 'count' === $step_key ) {
+		$step_key = (string) $this->get_option($this->step_key, false);
+		if ('count' === $step_key) {
 			return 1;
-		} elseif ( 'donation' === $step_key ) {
+		} elseif ('donation' === $step_key) {
 			return 2;
-		} elseif ( 'donor' === $step_key ) {
+		} elseif ('donor' === $step_key) {
 			return 3;
 		} else {
 			return $step_key;
@@ -568,6 +568,6 @@ 
 	 * Get the current $page value in the ajax.
 	 */
 	private function get_step_page() {
-		return $this->get_option( $this->step_on_key, false );
+		return $this->get_option($this->step_on_key, false);
 	}
 }

includes/class-give-comment.php

Doc Comments +2 added lines, -1 removed lines

@@ -445,7 +445,7 @@ 
 	 * @param int        $comment_id
 	 * @param WP_Comment $comment
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function __get_comment_author( $author, $comment_id, $comment ) {
 		if ( in_array( $comment->comment_type, $this->comment_types ) ) {
@@ -468,6 +468,7 @@ 
 	 * @access public
 	 *
 	 * @param array @comment_types
+	 * @param string[] $comment_types
 	 *
 	 * @return array
 	 */

Spacing +80 added lines, -80 removed lines

@@ -46,7 +46,7 @@ 
 	 * @return Give_Comment
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->init();
 		}
@@ -68,14 +68,14 @@ 
 		 */
 		$this->comment_types = apply_filters(
 			'give_comment_type',
-			self::get_comment_types( array( 'payment', 'donor' ) )
+			self::get_comment_types(array('payment', 'donor'))
 		);
 
-		add_action( 'pre_get_comments', array( $this, 'hide_comments' ), 10 );
-		add_filter( 'comments_clauses', array( $this, 'hide_comments_pre_wp_41' ), 10, 1 );
-		add_filter( 'comment_feed_where', array( $this, 'hide_comments_from_feeds' ), 10, 1 );
-		add_filter( 'wp_count_comments', array( $this, 'remove_comments_from_comment_counts' ), 10, 2 );
-		add_filter( 'get_comment_author', array( $this, '__get_comment_author' ), 10, 3 );
+		add_action('pre_get_comments', array($this, 'hide_comments'), 10);
+		add_filter('comments_clauses', array($this, 'hide_comments_pre_wp_41'), 10, 1);
+		add_filter('comment_feed_where', array($this, 'hide_comments_from_feeds'), 10, 1);
+		add_filter('wp_count_comments', array($this, 'remove_comments_from_comment_counts'), 10, 2);
+		add_filter('get_comment_author', array($this, '__get_comment_author'), 10, 3);
 	}
 
 	/**
@@ -91,13 +91,13 @@ 
 	 *
 	 * @return int|WP_Error
 	 */
-	public static function add( $id, $note, $comment_type, $comment_args = array() ) {
+	public static function add($id, $note, $comment_type, $comment_args = array()) {
 		// Bailout
-		if ( empty( $id ) || empty( $note ) || empty( $comment_type ) ) {
-			return new WP_Error( 'give_invalid_required_param', __( 'This comment has invalid ID or comment text or cooment type', 'give' ) );
+		if (empty($id) || empty($note) || empty($comment_type)) {
+			return new WP_Error('give_invalid_required_param', __('This comment has invalid ID or comment text or cooment type', 'give'));
 		}
 
-		$is_existing_comment = array_key_exists( 'comment_ID', $comment_args ) && ! empty( $comment_args['comment_ID'] );
+		$is_existing_comment = array_key_exists('comment_ID', $comment_args) && ! empty($comment_args['comment_ID']);
 		$action_type         = $is_existing_comment ? 'update' : 'insert';
 
 		/**
@@ -108,7 +108,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_pre_{$action_type}_{$comment_type}_note", $id, $note );
+		do_action("give_pre_{$action_type}_{$comment_type}_note", $id, $note);
 
 		$comment_args = wp_parse_args(
 			$comment_args,
@@ -116,8 +116,8 @@ 
 				'comment_post_ID'      => $id,
 				'comment_content'      => $note,
 				'user_id'              => is_admin() ? get_current_user_id() : 0,
-				'comment_date'         => current_time( 'mysql' ),
-				'comment_date_gmt'     => current_time( 'mysql', 1 ),
+				'comment_date'         => current_time('mysql'),
+				'comment_date_gmt'     => current_time('mysql', 1),
 				'comment_approved'     => 1,
 				'comment_parent'       => 0,
 				'comment_author'       => '',
@@ -131,30 +131,30 @@ 
 
 
 		// Check comment max length.
-		$error = wp_check_comment_data_max_lengths( $comment_args );
-		if( is_wp_error( $error ) ) {
+		$error = wp_check_comment_data_max_lengths($comment_args);
+		if (is_wp_error($error)) {
 			return $error;
 		}
 
 		// Remove moderation emails when comment posted.
-		remove_action( 'comment_post', 'wp_new_comment_notify_moderator' );
-		remove_action( 'comment_post', 'wp_new_comment_notify_postauthor' );
+		remove_action('comment_post', 'wp_new_comment_notify_moderator');
+		remove_action('comment_post', 'wp_new_comment_notify_postauthor');
 
 		// Remove comment flood check.
-		remove_action( 'check_comment_flood', 'check_comment_flood_db', 10 );
+		remove_action('check_comment_flood', 'check_comment_flood_db', 10);
 
 		$comment_id = $is_existing_comment
-			? wp_update_comment( $comment_args )
-			: wp_new_comment( $comment_args, true );
+			? wp_update_comment($comment_args)
+			: wp_new_comment($comment_args, true);
 
 		// Add moderation emails when comment posted.
-		add_action( 'comment_post', 'wp_new_comment_notify_moderator' );
-		add_action( 'comment_post', 'wp_new_comment_notify_postauthor' );
+		add_action('comment_post', 'wp_new_comment_notify_moderator');
+		add_action('comment_post', 'wp_new_comment_notify_postauthor');
 
 		// Add comment flood check.
-		add_action( 'check_comment_flood', 'check_comment_flood_db', 10, 4 );
+		add_action('check_comment_flood', 'check_comment_flood_db', 10, 4);
 
-		update_comment_meta( $comment_id, "_give_{$comment_type}_id", $id );
+		update_comment_meta($comment_id, "_give_{$comment_type}_id", $id);
 
 		/**
 		 * Fires after payment|donor comment inserted/updated.
@@ -165,7 +165,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_{$action_type}_{$comment_type}_note", $comment_id, $id, $note );
+		do_action("give_{$action_type}_{$comment_type}_note", $comment_id, $id, $note);
 
 		return $comment_id;
 	}
@@ -185,11 +185,11 @@ 
 	 *
 	 * @return bool True on success, false otherwise.
 	 */
-	public static function delete( $comment_id, $id, $comment_type ) {
+	public static function delete($comment_id, $id, $comment_type) {
 		$ret = false;
 
 		// Bailout
-		if ( empty( $id ) || empty( $comment_id ) || empty( $comment_type ) ) {
+		if (empty($id) || empty($comment_id) || empty($comment_type)) {
 			return $ret;
 		}
 
@@ -201,9 +201,9 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_pre_delete_{$comment_type}_note", $comment_id, $id );
+		do_action("give_pre_delete_{$comment_type}_note", $comment_id, $id);
 
-		$ret = wp_delete_comment( $comment_id, true );
+		$ret = wp_delete_comment($comment_id, true);
 
 		/**
 		 * Fires after donation note deleted.
@@ -214,7 +214,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_post_delete_{$comment_type}_note", $comment_id, $id, $ret );
+		do_action("give_post_delete_{$comment_type}_note", $comment_id, $id, $ret);
 
 		return $ret;
 	}
@@ -233,25 +233,25 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get( $id, $comment_type, $comment_args = array(), $search = '' ) {
+	public static function get($id, $comment_type, $comment_args = array(), $search = '') {
 		$comments = array();
 
 		// Set default meta_query value.
-		if ( ! isset( $comment_args['meta_query'] ) ) {
+		if ( ! isset($comment_args['meta_query'])) {
 			$comment_args['meta_query'] = array();
 		}
 
 		// Bailout
-		if ( empty( $id ) || empty( $comment_type ) ) {
+		if (empty($id) || empty($comment_type)) {
 			return $comments;
 		}
 
-		remove_action( 'pre_get_comments', array( self::$instance, 'hide_comments' ), 10 );
-		remove_filter( 'comments_clauses', array( self::$instance, 'hide_comments_pre_wp_41' ), 10 );
+		remove_action('pre_get_comments', array(self::$instance, 'hide_comments'), 10);
+		remove_filter('comments_clauses', array(self::$instance, 'hide_comments_pre_wp_41'), 10);
 
-		switch ( $comment_type ) {
+		switch ($comment_type) {
 			case 'payment':
-				$comment_args['meta_query'] = ! empty( $comment_args['meta_query'] )
+				$comment_args['meta_query'] = ! empty($comment_args['meta_query'])
 					? $comment_args['meta_query']
 					: array(
 						array(
@@ -260,7 +260,7 @@ 
 						)
 					);
 
-				$comments = get_comments( wp_parse_args(
+				$comments = get_comments(wp_parse_args(
 					$comment_args,
 					array(
 						'post_id' => $id,
@@ -268,11 +268,11 @@ 
 						'search'  => $search,
 						'type'    => 'give_payment_note'
 					)
-				) );
+				));
 				break;
 
 			case 'donor':
-				$comment_args['meta_query'] = ! empty( $comment_args['meta_query'] )
+				$comment_args['meta_query'] = ! empty($comment_args['meta_query'])
 					? $comment_args['meta_query']
 					: array(
 						array(
@@ -281,19 +281,19 @@ 
 						)
 					);
 
-				$comments = get_comments( wp_parse_args(
+				$comments = get_comments(wp_parse_args(
 					$comment_args,
 					array(
 						'order'  => 'ASC',
 						'search' => $search,
 						'type'   => 'give_donor_note'
 					)
-				) );
+				));
 				break;
 		}
 
-		add_action( 'pre_get_comments', array( self::$instance, 'hide_comments' ), 10, 1 );
-		add_filter( 'comments_clauses', array( self::$instance, 'hide_comments_pre_wp_41' ), 10, 1 );
+		add_action('pre_get_comments', array(self::$instance, 'hide_comments'), 10, 1);
+		add_filter('comments_clauses', array(self::$instance, 'hide_comments_pre_wp_41'), 10, 1);
 
 		return $comments;
 	}
@@ -309,14 +309,14 @@ 
 	 *
 	 * @return void
 	 */
-	public function hide_comments( $query ) {
-		if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-			$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-			if ( ! is_array( $types ) ) {
-				$types = array( $types );
+	public function hide_comments($query) {
+		if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+			$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+			if ( ! is_array($types)) {
+				$types = array($types);
 			}
 
-			$types = array_filter( array_merge( $types, $this->comment_types ) );
+			$types = array_filter(array_merge($types, $this->comment_types));
 
 			$query->query_vars['type__not_in'] = $types;
 		}
@@ -332,9 +332,9 @@ 
 	 *
 	 * @return array $clauses Updated comment clauses.
 	 */
-	public function hide_comments_pre_wp_41( $clauses ) {
-		if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
-			foreach ( $this->comment_types as $comment_type ) {
+	public function hide_comments_pre_wp_41($clauses) {
+		if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
+			foreach ($this->comment_types as $comment_type) {
 				$clauses['where'] .= " AND comment_type != \"{$comment_type}\"";
 			}
 		}
@@ -352,11 +352,11 @@ 
 	 *
 	 * @return string $where
 	 */
-	public function hide_comments_from_feeds( $where ) {
+	public function hide_comments_from_feeds($where) {
 		global $wpdb;
 
-		foreach ( $this->comment_types as $comment_type ) {
-			$where .= $wpdb->prepare( ' AND comment_type!=%s', $comment_type );
+		foreach ($this->comment_types as $comment_type) {
+			$where .= $wpdb->prepare(' AND comment_type!=%s', $comment_type);
 		}
 
 		return $where;
@@ -373,30 +373,30 @@ 
 	 *
 	 * @return array|object Array of comment counts.
 	 */
-	public function remove_comments_from_comment_counts( $stats, $post_id ) {
+	public function remove_comments_from_comment_counts($stats, $post_id) {
 		global $wpdb;
 
 		$post_id = (int) $post_id;
 
-		if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+		if (apply_filters('give_count_payment_notes_in_comments', false)) {
 			return $stats;
 		}
 
-		$stats = Give_Cache::get_group( "comments-{$post_id}", 'counts' );
+		$stats = Give_Cache::get_group("comments-{$post_id}", 'counts');
 
 		// Return result from cache.
-		if ( ! is_null( $stats ) ) {
+		if ( ! is_null($stats)) {
 			return $stats;
 		}
 
 		$where = 'WHERE';
 
-		foreach ( $this->comment_types as $index => $comment_type ) {
-			$where .= ( $index ? ' AND ' : ' ' ) . "comment_type != \"{$comment_type}\"";
+		foreach ($this->comment_types as $index => $comment_type) {
+			$where .= ($index ? ' AND ' : ' ')."comment_type != \"{$comment_type}\"";
 		}
 
-		if ( $post_id > 0 ) {
-			$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+		if ($post_id > 0) {
+			$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 		}
 
 		$count = $wpdb->get_results(
@@ -417,26 +417,26 @@ 
 			'post-trashed' => 'post-trashed',
 		);
 
-		foreach ( (array) $count as $row ) {
+		foreach ((array) $count as $row) {
 			// Don't count post-trashed toward totals.
-			if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+			if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 				$total += $row['num_comments'];
 			}
-			if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-				$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+			if (isset($approved[$row['comment_approved']])) {
+				$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 			}
 		}
 
 		$stats['total_comments'] = $stats['all'] = $total;
-		foreach ( $approved as $key ) {
-			if ( empty( $stats[ $key ] ) ) {
-				$stats[ $key ] = 0;
+		foreach ($approved as $key) {
+			if (empty($stats[$key])) {
+				$stats[$key] = 0;
 			}
 		}
 
 		$stats = (object) $stats;
 
-		Give_Cache::set_group( "comments-{$post_id}", $stats, 'counts' );
+		Give_Cache::set_group("comments-{$post_id}", $stats, 'counts');
 
 		return $stats;
 	}
@@ -453,12 +453,12 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __get_comment_author( $author, $comment_id, $comment ) {
-		if ( in_array( $comment->comment_type, $this->comment_types ) ) {
-			switch ( $comment->comment_type ) {
+	public function __get_comment_author($author, $comment_id, $comment) {
+		if (in_array($comment->comment_type, $this->comment_types)) {
+			switch ($comment->comment_type) {
 				case 'give_payment_note':
-					if ( get_comment_meta( $comment_id, '_give_donor_id', true ) ) {
-						$author = give_get_donor_name_by( $comment->comment_post_ID );
+					if (get_comment_meta($comment_id, '_give_donor_id', true)) {
+						$author = give_get_donor_name_by($comment->comment_post_ID);
 					}
 			}
 		}
@@ -477,9 +477,9 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_comment_types( $comment_types ) {
+	public static function get_comment_types($comment_types) {
 		$_comment_types = array();
-		foreach ( $comment_types as $comment_type ) {
+		foreach ($comment_types as $comment_type) {
 			$_comment_types[] = "give_{$comment_type}_note";
 		}
 

includes/class-give-db-meta.php

Doc Comments +2 added lines, -2 removed lines

@@ -279,7 +279,7 @@ 
 	/**
 	 * Rename query clauses for new meta table
 	 *
-	 * @param $clause
+	 * @param string $clause
 	 * @param $filter
 	 *
 	 * @return mixed
@@ -515,7 +515,7 @@ 
 	 * @since  2.0.4
 	 * @access public
 	 *
-	 * @return string
+	 * @return boolean
 	 */
 	public function get_meta_type() {
 		return $this->meta_type;

Spacing +89 added lines, -89 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -77,40 +77,40 @@ 
 		parent::__construct();
 
 		// Bailout.
-		if ( empty( $this->supports ) || ! $this->is_custom_meta_table_active() ) {
+		if (empty($this->supports) || ! $this->is_custom_meta_table_active()) {
 			return;
 		}
 
-		if ( in_array( 'add_post_metadata', $this->supports ) ) {
-			add_filter( 'add_post_metadata', array( $this, '__add_meta' ), 0, 5 );
+		if (in_array('add_post_metadata', $this->supports)) {
+			add_filter('add_post_metadata', array($this, '__add_meta'), 0, 5);
 		}
 
-		if ( in_array( 'get_post_metadata', $this->supports ) ) {
-			add_filter( 'get_post_metadata', array( $this, '__get_meta' ), 10, 4 );
+		if (in_array('get_post_metadata', $this->supports)) {
+			add_filter('get_post_metadata', array($this, '__get_meta'), 10, 4);
 		}
 
-		if ( in_array( 'update_post_metadata', $this->supports ) ) {
-			add_filter( 'update_post_metadata', array( $this, '__update_meta' ), 0, 5 );
+		if (in_array('update_post_metadata', $this->supports)) {
+			add_filter('update_post_metadata', array($this, '__update_meta'), 0, 5);
 		}
 
-		if ( in_array( 'delete_post_metadata', $this->supports ) ) {
-			add_filter( 'delete_post_metadata', array( $this, '__delete_meta' ), 0, 5 );
+		if (in_array('delete_post_metadata', $this->supports)) {
+			add_filter('delete_post_metadata', array($this, '__delete_meta'), 0, 5);
 		}
 
-		if ( in_array( 'posts_where', $this->supports ) ) {
-			add_filter( 'posts_where', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_where', $this->supports)) {
+			add_filter('posts_where', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_join', $this->supports ) ) {
-			add_filter( 'posts_join', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_join', $this->supports)) {
+			add_filter('posts_join', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_groupby', $this->supports ) ) {
-			add_filter( 'posts_groupby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_groupby', $this->supports)) {
+			add_filter('posts_groupby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_orderby', $this->supports ) ) {
-			add_filter( 'posts_orderby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_orderby', $this->supports)) {
+			add_filter('posts_orderby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 	}
 
@@ -128,16 +128,16 @@ 
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single
 	 *                                is true.
 	 */
-	public function get_meta( $id = 0, $meta_key = '', $single = false ) {
-		$id = $this->sanitize_id( $id );
+	public function get_meta($id = 0, $meta_key = '', $single = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		if ( $this->raw_result ) {
-			if ( ! ( $value = get_metadata( $this->meta_type, $id, $meta_key, false ) ) ) {
+		if ($this->raw_result) {
+			if ( ! ($value = get_metadata($this->meta_type, $id, $meta_key, false))) {
 				$value = '';
 			}
 
@@ -145,7 +145,7 @@ 
 			$this->raw_result = false;
 
 		} else {
-			$value = get_metadata( $this->meta_type, $id, $meta_key, $single );
+			$value = get_metadata($this->meta_type, $id, $meta_key, $single);
 		}
 
 		return $value;
@@ -167,18 +167,18 @@ 
 	 *
 	 * @return  int|bool                  False for failure. True for success.
 	 */
-	public function add_meta( $id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$id = $this->sanitize_id( $id );
+	public function add_meta($id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = add_metadata( $this->meta_type, $id, $meta_key, $meta_value, $unique );
+		$meta_id = add_metadata($this->meta_type, $id, $meta_key, $meta_value, $unique);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -204,18 +204,18 @@ 
 	 *
 	 * @return  int|bool                  False on failure, true if success.
 	 */
-	public function update_meta( $id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function update_meta($id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = update_metadata( $this->meta_type, $id, $meta_key, $meta_value, $prev_value );
+		$meta_id = update_metadata($this->meta_type, $id, $meta_key, $meta_value, $prev_value);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -238,18 +238,18 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $id = 0, $meta_key = '', $meta_value = '', $delete_all = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function delete_meta($id = 0, $meta_key = '', $meta_value = '', $delete_all = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$is_meta_deleted = delete_metadata( $this->meta_type, $id, $meta_key, $meta_value, $delete_all );
+		$is_meta_deleted = delete_metadata($this->meta_type, $id, $meta_key, $meta_value, $delete_all);
 
-		if ( $is_meta_deleted ) {
-			$this->delete_cache( $id );
+		if ($is_meta_deleted) {
+			$this->delete_cache($id);
 		}
 
 		return $is_meta_deleted;
@@ -266,10 +266,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function __rename_meta_table_name_in_query( $clause, $wp_query ) {
+	public function __rename_meta_table_name_in_query($clause, $wp_query) {
 		// Add new table to sql query.
-		if ( $this->is_post_type_query( $wp_query ) && ! empty( $wp_query->meta_query->queries ) ) {
-			$clause = $this->__rename_meta_table_name( $clause, current_filter() );
+		if ($this->is_post_type_query($wp_query) && ! empty($wp_query->meta_query->queries)) {
+			$clause = $this->__rename_meta_table_name($clause, current_filter());
 		}
 
 		return $clause;
@@ -284,42 +284,42 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __rename_meta_table_name( $clause, $filter ) {
+	public function __rename_meta_table_name($clause, $filter) {
 		global $wpdb;
 
-		$clause = str_replace( "{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause );
-		$clause = str_replace( $wpdb->postmeta, $this->table_name, $clause );
+		$clause = str_replace("{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause);
+		$clause = str_replace($wpdb->postmeta, $this->table_name, $clause);
 
-		switch ( $filter ) {
+		switch ($filter) {
 			case 'posts_join':
-				$joins = array( 'INNER JOIN', 'LEFT JOIN' );
+				$joins = array('INNER JOIN', 'LEFT JOIN');
 
-				foreach ( $joins as $join ) {
-					if ( false !== strpos( $clause, $join ) ) {
-						$clause = explode( $join, $clause );
+				foreach ($joins as $join) {
+					if (false !== strpos($clause, $join)) {
+						$clause = explode($join, $clause);
 
-						foreach ( $clause as $key => $clause_part ) {
-							if ( empty( $clause_part ) ) {
+						foreach ($clause as $key => $clause_part) {
+							if (empty($clause_part)) {
 								continue;
 							}
 
-							preg_match( '/' . $wpdb->prefix . 'give_' . $this->meta_type . 'meta AS (.*) ON/', $clause_part, $alias_table_name );
+							preg_match('/'.$wpdb->prefix.'give_'.$this->meta_type.'meta AS (.*) ON/', $clause_part, $alias_table_name);
 
-							if ( isset( $alias_table_name[1] ) ) {
-								$clause[ $key ] = str_replace( "{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part );
+							if (isset($alias_table_name[1])) {
+								$clause[$key] = str_replace("{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part);
 							}
 						}
 
-						$clause = implode( "{$join} ", $clause );
+						$clause = implode("{$join} ", $clause);
 					}
 				}
 				break;
 
 			case 'posts_where':
-				$clause = str_replace( array( 'mt2.post_id', 'mt1.post_id' ), array(
+				$clause = str_replace(array('mt2.post_id', 'mt1.post_id'), array(
 					"mt2.{$this->meta_type}_id",
 					"mt1.{$this->meta_type}_id"
-				), $clause );
+				), $clause);
 				break;
 		}
 
@@ -337,19 +337,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_post_type_query( $wp_query ) {
+	protected function is_post_type_query($wp_query) {
 		$status = false;
 
 		// Check if it is payment query.
-		if ( ! empty( $wp_query->query['post_type'] ) ) {
+		if ( ! empty($wp_query->query['post_type'])) {
 			if (
-				is_string( $wp_query->query['post_type'] ) &&
+				is_string($wp_query->query['post_type']) &&
 				$this->post_type === $wp_query->query['post_type']
 			) {
 				$status = true;
 			} elseif (
-				is_array( $wp_query->query['post_type'] ) &&
-				in_array( $this->post_type, $wp_query->query['post_type'] )
+				is_array($wp_query->query['post_type']) &&
+				in_array($this->post_type, $wp_query->query['post_type'])
 			) {
 				$status = true;
 			}
@@ -368,8 +368,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
-		return $ID && ( $this->post_type === get_post_type( $ID ) );
+	protected function is_valid_post_type($ID) {
+		return $ID && ($this->post_type === get_post_type($ID));
 	}
 
 	/**
@@ -395,8 +395,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_cache( $id, $meta_type = '' ) {
-		$meta_type = empty( $meta_type ) ? $this->meta_type : $meta_type;
+	private function delete_cache($id, $meta_type = '') {
+		$meta_type = empty($meta_type) ? $this->meta_type : $meta_type;
 
 		$group = array(
 			'payment'  => 'give-donations', // Backward compatibility
@@ -405,8 +405,8 @@ 
 			'customer' => 'give-donors', // Backward compatibility for pre upgrade in 2.0
 		);
 
-		if ( array_key_exists( $meta_type, $group ) ) {
-			Give_Cache::delete_group( $id, $group[ $meta_type ] );
+		if (array_key_exists($meta_type, $group)) {
+			Give_Cache::delete_group($id, $group[$meta_type]);
 		}
 	}
 
@@ -421,8 +421,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
+	public function __call($name, $arguments) {
+		switch ($name) {
 			case '__add_meta':
 				$this->check = $arguments[0];
 				$id          = $arguments[1];
@@ -431,11 +431,11 @@ 
 				$unique      = $arguments[4];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->add_meta( $id, $meta_key, $meta_value, $unique );
+				return $this->add_meta($id, $meta_key, $meta_value, $unique);
 
 			case '__get_meta':
 				$this->check = $arguments[0];
@@ -444,13 +444,13 @@ 
 				$single      = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
 				$this->raw_result = true;
 
-				return $this->get_meta( $id, $meta_key, $single );
+				return $this->get_meta($id, $meta_key, $single);
 
 			case '__update_meta':
 				$this->check = $arguments[0];
@@ -459,11 +459,11 @@ 
 				$meta_value  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->update_meta( $id, $meta_key, $meta_value );
+				return $this->update_meta($id, $meta_key, $meta_value);
 
 			case '__delete_meta':
 				$this->check = $arguments[0];
@@ -473,11 +473,11 @@ 
 				$delete_all  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->delete_meta( $id, $meta_key, $meta_value, $delete_all );
+				return $this->delete_meta($id, $meta_key, $meta_value, $delete_all);
 		}
 	}
 
@@ -502,10 +502,10 @@ 
 			KEY meta_key (meta_key({$this->min_index_length}))
 			) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version, false );
+		update_option($this->table_name.'_db_version', $this->version, false);
 	}
 
 
@@ -531,12 +531,12 @@ 
 	 *
 	 * @return  bool  False for failure. True for success.
 	 */
-	public function delete_all_meta( $id = 0 ) {
+	public function delete_all_meta($id = 0) {
 		global $wpdb;
-		$status = $wpdb->delete( $this->table_name, array( "{$this->meta_type}_id" => $id ), array( '%d' ) );
+		$status = $wpdb->delete($this->table_name, array("{$this->meta_type}_id" => $id), array('%d'));
 
-		if ( $status ) {
-			$this->delete_cache( $id, $this->meta_type );
+		if ($status) {
+			$this->delete_cache($id, $this->meta_type);
 		}
 
 		return $status;

includes/class-give-donor.php

Doc Comments +1 added lines, -1 removed lines

@@ -1052,7 +1052,7 @@
 	 * @access public
 	 *
 	 * @param  string $meta_key   Metadata name. Default is empty.
-	 * @param  mixed  $meta_value Optional. Metadata value. Default is empty.
+	 * @param  string  $meta_value Optional. Metadata value. Default is empty.
 	 *
 	 * @return bool               False for failure. True for success.
 	 */

Spacing +275 added lines, -275 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -149,29 +149,29 @@ 
 	 * @param int|bool $_id_or_email
 	 * @param bool     $by_user_id
 	 */
-	public function __construct( $_id_or_email = false, $by_user_id = false ) {
+	public function __construct($_id_or_email = false, $by_user_id = false) {
 
 		$this->db = Give()->donors;
 
-		if ( false === $_id_or_email || ( is_numeric( $_id_or_email ) && (int) $_id_or_email !== absint( $_id_or_email ) ) ) {
+		if (false === $_id_or_email || (is_numeric($_id_or_email) && (int) $_id_or_email !== absint($_id_or_email))) {
 			return false;
 		}
 
-		$by_user_id = is_bool( $by_user_id ) ? $by_user_id : false;
+		$by_user_id = is_bool($by_user_id) ? $by_user_id : false;
 
-		if ( is_numeric( $_id_or_email ) ) {
+		if (is_numeric($_id_or_email)) {
 			$field = $by_user_id ? 'user_id' : 'id';
 		} else {
 			$field = 'email';
 		}
 
-		$donor = $this->db->get_donor_by( $field, $_id_or_email );
+		$donor = $this->db->get_donor_by($field, $_id_or_email);
 
-		if ( empty( $donor ) || ! is_object( $donor ) ) {
+		if (empty($donor) || ! is_object($donor)) {
 			return false;
 		}
 
-		$this->setup_donor( $donor );
+		$this->setup_donor($donor);
 
 	}
 
@@ -187,19 +187,19 @@ 
 	 *
 	 * @return bool             If the setup was successful or not.
 	 */
-	private function setup_donor( $donor ) {
+	private function setup_donor($donor) {
 
-		if ( ! is_object( $donor ) ) {
+		if ( ! is_object($donor)) {
 			return false;
 		}
 
 		// Get cached donors.
-		$donor_vars = Give_Cache::get_group( $donor->id, 'give-donors' );
+		$donor_vars = Give_Cache::get_group($donor->id, 'give-donors');
 
-		if ( is_null( $donor_vars ) ) {
-			foreach ( $donor as $key => $value ) {
+		if (is_null($donor_vars)) {
+			foreach ($donor as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'notes':
 						$this->$key = $this->get_notes();
@@ -213,20 +213,20 @@ 
 			}
 
 			// Get donor's all email including primary email.
-			$this->emails = (array) $this->get_meta( 'additional_email', false );
-			$this->emails = array( 'primary' => $this->email ) + $this->emails;
+			$this->emails = (array) $this->get_meta('additional_email', false);
+			$this->emails = array('primary' => $this->email) + $this->emails;
 
 			$this->setup_address();
 
-			Give_Cache::set_group( $donor->id, get_object_vars( $this ), 'give-donors' );
+			Give_Cache::set_group($donor->id, get_object_vars($this), 'give-donors');
 		} else {
-			foreach ( $donor_vars as $donor_var => $value ) {
+			foreach ($donor_vars as $donor_var => $value) {
 				$this->$donor_var = $value;
 			}
 		}
 
 		// Donor ID and email are the only things that are necessary, make sure they exist.
-		if ( ! empty( $this->id ) && ! empty( $this->email ) ) {
+		if ( ! empty($this->id) && ! empty($this->email)) {
 			return true;
 		}
 
@@ -245,25 +245,25 @@ 
 		global $wpdb;
 		$meta_type = Give()->donor_meta->meta_type;
 
-		$addresses = $wpdb->get_results( $wpdb->prepare( "
+		$addresses = $wpdb->get_results($wpdb->prepare("
 				SELECT meta_key, meta_value FROM {$wpdb->donormeta}
 				WHERE meta_key
 				LIKE '%%%s%%'
 				AND {$meta_type}_id=%d
-				", 'give_donor_address', $this->id ), ARRAY_N );
+				", 'give_donor_address', $this->id), ARRAY_N);
 
-		if ( empty( $addresses ) ) {
+		if (empty($addresses)) {
 			return $this->address;
 		}
 
-		foreach ( $addresses as $address ) {
-			$address[0] = str_replace( '_give_donor_address_', '', $address[0] );
-			$address[0] = explode( '_', $address[0] );
+		foreach ($addresses as $address) {
+			$address[0] = str_replace('_give_donor_address_', '', $address[0]);
+			$address[0] = explode('_', $address[0]);
 
-			if ( 3 === count( $address[0] ) ) {
-				$this->address[ $address[0][0] ][ $address[0][2] ][ $address[0][1] ] = $address[1];
+			if (3 === count($address[0])) {
+				$this->address[$address[0][0]][$address[0][2]][$address[0][1]] = $address[1];
 			} else {
-				$this->address[ $address[0][0] ][ $address[0][1] ] = $address[1];
+				$this->address[$address[0][0]][$address[0][1]] = $address[1];
 			}
 		}
 	}
@@ -279,7 +279,7 @@ 
 	 *
 	 * @return array The donor's address, if any
 	 */
-	public function get_donor_address( $args = array() ) {
+	public function get_donor_address($args = array()) {
 		$args = wp_parse_args(
 			$args,
 			array(
@@ -297,24 +297,24 @@ 
 		);
 
 		// Backward compatibility.
-		if ( ! give_has_upgrade_completed( 'v20_upgrades_user_address' ) ) {
+		if ( ! give_has_upgrade_completed('v20_upgrades_user_address')) {
 
 			// Backward compatibility for user id param.
-			return wp_parse_args( (array) get_user_meta( $this->user_id, '_give_user_address', true ), $default_address );
+			return wp_parse_args((array) get_user_meta($this->user_id, '_give_user_address', true), $default_address);
 
 		}
 
-		if ( ! $this->id || empty( $this->address ) || ! array_key_exists( $args['address_type'], $this->address ) ) {
+		if ( ! $this->id || empty($this->address) || ! array_key_exists($args['address_type'], $this->address)) {
 			return $default_address;
 		}
 
-		switch ( true ) {
-			case is_string( end( $this->address[ $args['address_type'] ] ) ):
-				$address = wp_parse_args( $this->address[ $args['address_type'] ], $default_address );
+		switch (true) {
+			case is_string(end($this->address[$args['address_type']])):
+				$address = wp_parse_args($this->address[$args['address_type']], $default_address);
 				break;
 
-			case is_array( end( $this->address[ $args['address_type'] ] ) ):
-				$address = wp_parse_args( array_shift( $this->address[ $args['address_type'] ] ), $default_address );
+			case is_array(end($this->address[$args['address_type']])):
+				$address = wp_parse_args(array_shift($this->address[$args['address_type']]), $default_address);
 				break;
 		}
 
@@ -331,16 +331,16 @@ 
 	 *
 	 * @return mixed|\WP_Error
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-donor-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-donor-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -356,9 +356,9 @@ 
 	 *
 	 * @return bool|int    False if not a valid creation, donor ID if user is found or valid creation.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 || empty( $data ) ) {
+		if ($this->id != 0 || empty($data)) {
 			return false;
 		}
 
@@ -366,15 +366,15 @@ 
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
-		$args = $this->sanitize_columns( $args );
+		$args = wp_parse_args($data, $defaults);
+		$args = $this->sanitize_columns($args);
 
-		if ( empty( $args['email'] ) || ! is_email( $args['email'] ) ) {
+		if (empty($args['email']) || ! is_email($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'])));
 		}
 
 		/**
@@ -384,24 +384,24 @@ 
 		 *
 		 * @param array $args Donor attributes.
 		 */
-		do_action( 'give_donor_pre_create', $args );
+		do_action('give_donor_pre_create', $args);
 
 		$created = false;
 
 		// The DB class 'add' implies an update if the donor being asked to be created already exists
-		if ( $this->db->add( $data ) ) {
+		if ($this->db->add($data)) {
 
 			// We've successfully added/updated the donor, reset the class vars with the new data
-			$donor = $this->db->get_donor_by( 'email', $args['email'] );
+			$donor = $this->db->get_donor_by('email', $args['email']);
 
 			// Setup the donor data with the values from DB
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$created = $this->id;
 
 			// Set donor as non anonymous by default
-			if( $created ) {
-				Give()->donor_meta->update_meta( $this->id, '_give_anonymous_donor', 0 );
+			if ($created) {
+				Give()->donor_meta->update_meta($this->id, '_give_anonymous_donor', 0);
 			}
 		}
 
@@ -413,7 +413,7 @@ 
 		 * @param bool|int $created False if not a valid creation, donor ID if user is found or valid creation.
 		 * @param array    $args    Customer attributes.
 		 */
-		do_action( 'give_donor_post_create', $created, $args );
+		do_action('give_donor_post_create', $created, $args);
 
 		return $created;
 
@@ -429,13 +429,13 @@ 
 	 *
 	 * @return bool        If the update was successful or not.
 	 */
-	public function update( $data = array() ) {
+	public function update($data = array()) {
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return false;
 		}
 
-		$data = $this->sanitize_columns( $data );
+		$data = $this->sanitize_columns($data);
 
 		/**
 		 * Fires before updating donors.
@@ -445,15 +445,15 @@ 
 		 * @param int   $donor_id Donor id.
 		 * @param array $data     Donor attributes.
 		 */
-		do_action( 'give_donor_pre_update', $this->id, $data );
+		do_action('give_donor_pre_update', $this->id, $data);
 
 		$updated = false;
 
-		if ( $this->db->update( $this->id, $data ) ) {
+		if ($this->db->update($this->id, $data)) {
 
-			$donor = $this->db->get_donor_by( 'id', $this->id );
+			$donor = $this->db->get_donor_by('id', $this->id);
 
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$updated = true;
 		}
@@ -467,7 +467,7 @@ 
 		 * @param int   $donor_id Donor id.
 		 * @param array $data     Donor attributes.
 		 */
-		do_action( 'give_donor_post_update', $updated, $this->id, $data );
+		do_action('give_donor_post_update', $updated, $this->id, $data);
 
 		return $updated;
 	}
@@ -485,27 +485,27 @@ 
 	 *
 	 * @return bool            If the attachment was successfully.
 	 */
-	public function attach_payment( $payment_id = 0, $update_stats = true ) {
+	public function attach_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		if ( empty( $this->payment_ids ) ) {
+		if (empty($this->payment_ids)) {
 
 			$new_payment_ids = $payment_id;
 
 		} else {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			if ( in_array( $payment_id, $payment_ids ) ) {
+			if (in_array($payment_id, $payment_ids)) {
 				$update_stats = false;
 			}
 
 			$payment_ids[] = $payment_id;
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -517,20 +517,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id   Donor id.
 		 */
-		do_action( 'give_donor_pre_attach_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_attach_payment', $payment_id, $this->id);
 
-		$payment_added = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_added = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_added ) {
+		if ($payment_added) {
 
 			$this->payment_ids = $new_payment_ids;
 
 			// We added this payment successfully, increment the stats
-			if ( $update_stats ) {
-				$payment_amount = give_donation_amount( $payment_id, array( 'type' => 'stats' ) );
+			if ($update_stats) {
+				$payment_amount = give_donation_amount($payment_id, array('type' => 'stats'));
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->increase_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->increase_value($payment_amount);
 				}
 
 				$this->increase_purchase_count();
@@ -546,7 +546,7 @@ 
 		 * @param int  $payment_id    Payment id.
 		 * @param int  $donor_id      Donor id.
 		 */
-		do_action( 'give_donor_post_attach_payment', $payment_added, $payment_id, $this->id );
+		do_action('give_donor_post_attach_payment', $payment_added, $payment_id, $this->id);
 
 		return $payment_added;
 	}
@@ -564,33 +564,33 @@ 
 	 *
 	 * @return boolean               If the removal was successful.
 	 */
-	public function remove_payment( $payment_id = 0, $update_stats = true ) {
+	public function remove_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = new Give_Payment( $payment_id );
+		$payment = new Give_Payment($payment_id);
 
-		if ( 'publish' !== $payment->status && 'revoked' !== $payment->status ) {
+		if ('publish' !== $payment->status && 'revoked' !== $payment->status) {
 			$update_stats = false;
 		}
 
 		$new_payment_ids = '';
 
-		if ( ! empty( $this->payment_ids ) ) {
+		if ( ! empty($this->payment_ids)) {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			$pos = array_search( $payment_id, $payment_ids );
-			if ( false === $pos ) {
+			$pos = array_search($payment_id, $payment_ids);
+			if (false === $pos) {
 				return false;
 			}
 
-			unset( $payment_ids[ $pos ] );
-			$payment_ids = array_filter( $payment_ids );
+			unset($payment_ids[$pos]);
+			$payment_ids = array_filter($payment_ids);
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -602,20 +602,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id   Customer id.
 		 */
-		do_action( 'give_donor_pre_remove_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_remove_payment', $payment_id, $this->id);
 
-		$payment_removed = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_removed = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_removed ) {
+		if ($payment_removed) {
 
 			$this->payment_ids = $new_payment_ids;
 
-			if ( $update_stats ) {
+			if ($update_stats) {
 				// We removed this payment successfully, decrement the stats
-				$payment_amount = give_donation_amount( $payment_id );
+				$payment_amount = give_donation_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->decrease_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->decrease_value($payment_amount);
 				}
 
 				$this->decrease_donation_count();
@@ -631,7 +631,7 @@ 
 		 * @param int  $payment_id      Payment id.
 		 * @param int  $donor_id        Donor id.
 		 */
-		do_action( 'give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id );
+		do_action('give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id);
 
 		return $payment_removed;
 
@@ -647,10 +647,10 @@ 
 	 *
 	 * @return int        The donation count.
 	 */
-	public function increase_purchase_count( $count = 1 ) {
+	public function increase_purchase_count($count = 1) {
 
 		// Make sure it's numeric and not negative.
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
@@ -664,9 +664,9 @@ 
 		 * @param int $count    The number to increase by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_increase_donation_count', $count, $this->id );
+		do_action('give_donor_pre_increase_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -679,7 +679,7 @@ 
 		 * @param int $count          The number increased by.
 		 * @param int $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -694,16 +694,16 @@ 
 	 *
 	 * @return mixed      If successful, the new count, otherwise false.
 	 */
-	public function decrease_donation_count( $count = 1 ) {
+	public function decrease_donation_count($count = 1) {
 
 		// Make sure it's numeric and not negative
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
 		$new_total = (int) $this->purchase_count - (int) $count;
 
-		if ( $new_total < 0 ) {
+		if ($new_total < 0) {
 			$new_total = 0;
 		}
 
@@ -715,9 +715,9 @@ 
 		 * @param int $count    The number to decrease by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_decrease_donation_count', $count, $this->id );
+		do_action('give_donor_pre_decrease_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -730,7 +730,7 @@ 
 		 * @param int $count          The number decreased by.
 		 * @param int $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -745,9 +745,9 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function increase_value( $value = 0.00 ) {
+	public function increase_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) + $value;
+		$new_value = floatval($this->purchase_value) + $value;
 
 		/**
 		 * Fires before increasing donor lifetime value.
@@ -757,9 +757,9 @@ 
 		 * @param float $value    The value to increase by.
 		 * @param int   $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_increase_value', $value, $this->id );
+		do_action('give_donor_pre_increase_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -772,7 +772,7 @@ 
 		 * @param float $value          The value increased by.
 		 * @param int   $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_increase_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_increase_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -787,11 +787,11 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function decrease_value( $value = 0.00 ) {
+	public function decrease_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) - $value;
+		$new_value = floatval($this->purchase_value) - $value;
 
-		if ( $new_value < 0 ) {
+		if ($new_value < 0) {
 			$new_value = 0.00;
 		}
 
@@ -803,9 +803,9 @@ 
 		 * @param float $value    The value to decrease by.
 		 * @param int   $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_decrease_value', $value, $this->id );
+		do_action('give_donor_pre_decrease_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -818,7 +818,7 @@ 
 		 * @param float $value          The value decreased by.
 		 * @param int   $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_decrease_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -838,7 +838,7 @@ 
 	 *
 	 * @return mixed              If successful, the new donation stat value, otherwise false.
 	 */
-	public function update_donation_value( $curr_amount, $new_amount ) {
+	public function update_donation_value($curr_amount, $new_amount) {
 		/**
 		 * Payment total difference value can be:
 		 *  zero   (in case amount not change)
@@ -848,15 +848,15 @@ 
 		$payment_total_diff = $new_amount - $curr_amount;
 
 		// We do not need to update donation stat if donation did not change.
-		if ( ! $payment_total_diff ) {
+		if ( ! $payment_total_diff) {
 			return false;
 		}
 
-		if ( $payment_total_diff > 0 ) {
-			$this->increase_value( $payment_total_diff );
+		if ($payment_total_diff > 0) {
+			$this->increase_value($payment_total_diff);
 		} else {
 			// Pass payment total difference as +ve value to decrease amount from user lifetime stat.
-			$this->decrease_value( - $payment_total_diff );
+			$this->decrease_value( -$payment_total_diff );
 		}
 
 		return $this->purchase_value;
@@ -873,15 +873,15 @@ 
 	 *
 	 * @return array       The notes requested.
 	 */
-	public function get_notes( $length = 20, $paged = 1 ) {
+	public function get_notes($length = 20, $paged = 1) {
 
-		$length = is_numeric( $length ) ? $length : 20;
-		$offset = is_numeric( $paged ) && $paged != 1 ? ( ( absint( $paged ) - 1 ) * $length ) : 0;
+		$length = is_numeric($length) ? $length : 20;
+		$offset = is_numeric($paged) && $paged != 1 ? ((absint($paged) - 1) * $length) : 0;
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		$desired_notes = array_slice( $notes_array, $offset, $length );
+		$desired_notes = array_slice($notes_array, $offset, $length);
 
 		return $desired_notes;
 
@@ -898,9 +898,9 @@ 
 	public function get_notes_count() {
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		return count( $notes_array );
+		return count($notes_array);
 
 	}
 
@@ -913,7 +913,7 @@ 
 	 *
 	 * @return string|float
 	 */
-	public function get_total_donation_amount( $args = array() ) {
+	public function get_total_donation_amount($args = array()) {
 
 		/**
 		 * Filter total donation amount.
@@ -924,7 +924,7 @@ 
 		 * @param integer      $donor_id       Donor ID.
 		 * @param array        $args           Pass additional data.
 		 */
-		return apply_filters( 'give_get_total_donation_amount', $this->purchase_value, $this->id, $args );
+		return apply_filters('give_get_total_donation_amount', $this->purchase_value, $this->id, $args);
 	}
 
 	/**
@@ -937,22 +937,22 @@ 
 	 *
 	 * @return string|boolean The new note if added successfully, false otherwise.
 	 */
-	public function add_note( $note = '' ) {
+	public function add_note($note = '') {
 
-		$note = trim( $note );
-		if ( empty( $note ) ) {
+		$note = trim($note);
+		if (empty($note)) {
 			return false;
 		}
 
 		$notes = $this->get_raw_notes();
 
-		if ( empty( $notes ) ) {
+		if (empty($notes)) {
 			$notes = '';
 		}
 
-		$note_string = date_i18n( 'F j, Y H:i:s', current_time( 'timestamp' ) ) . ' - ' . $note;
-		$new_note    = apply_filters( 'give_customer_add_note_string', $note_string );
-		$notes       .= "\n\n" . $new_note;
+		$note_string = date_i18n('F j, Y H:i:s', current_time('timestamp')).' - '.$note;
+		$new_note    = apply_filters('give_customer_add_note_string', $note_string);
+		$notes .= "\n\n".$new_note;
 
 		/**
 		 * Fires before donor note is added.
@@ -962,11 +962,11 @@ 
 		 * @param string $new_note New note to add.
 		 * @param int    $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_add_note', $new_note, $this->id );
+		do_action('give_donor_pre_add_note', $new_note, $this->id);
 
-		$updated = $this->update( array( 'notes' => $notes ) );
+		$updated = $this->update(array('notes' => $notes));
 
-		if ( $updated ) {
+		if ($updated) {
 			$this->notes = $this->get_notes();
 		}
 
@@ -979,7 +979,7 @@ 
 		 * @param string $new_note    New note added.
 		 * @param int    $donor_id    Donor id.
 		 */
-		do_action( 'give_donor_post_add_note', $this->notes, $new_note, $this->id );
+		do_action('give_donor_post_add_note', $this->notes, $new_note, $this->id);
 
 		// Return the formatted note, so we can test, as well as update any displays
 		return $new_note;
@@ -996,7 +996,7 @@ 
 	 */
 	private function get_raw_notes() {
 
-		$all_notes = $this->db->get_column( 'notes', $this->id );
+		$all_notes = $this->db->get_column('notes', $this->id);
 
 		return $all_notes;
 
@@ -1014,8 +1014,8 @@ 
 	 * @return mixed            Will be an array if $single is false. Will be value of meta data field if $single is
 	 *                          true.
 	 */
-	public function get_meta( $meta_key = '', $single = true ) {
-		return Give()->donor_meta->get_meta( $this->id, $meta_key, $single );
+	public function get_meta($meta_key = '', $single = true) {
+		return Give()->donor_meta->get_meta($this->id, $meta_key, $single);
 	}
 
 	/**
@@ -1030,8 +1030,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function add_meta( $meta_key = '', $meta_value, $unique = false ) {
-		return Give()->donor_meta->add_meta( $this->id, $meta_key, $meta_value, $unique );
+	public function add_meta($meta_key = '', $meta_value, $unique = false) {
+		return Give()->donor_meta->add_meta($this->id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -1046,8 +1046,8 @@ 
 	 *
 	 * @return bool               False on failure, true if success.
 	 */
-	public function update_meta( $meta_key = '', $meta_value, $prev_value = '' ) {
-		return Give()->donor_meta->update_meta( $this->id, $meta_key, $meta_value, $prev_value );
+	public function update_meta($meta_key = '', $meta_value, $prev_value = '') {
+		return Give()->donor_meta->update_meta($this->id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1061,8 +1061,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function delete_meta( $meta_key = '', $meta_value = '' ) {
-		return Give()->donor_meta->delete_meta( $this->id, $meta_key, $meta_value );
+	public function delete_meta($meta_key = '', $meta_value = '') {
+		return Give()->donor_meta->delete_meta($this->id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -1075,51 +1075,51 @@ 
 	 *
 	 * @return array       The sanitized data, based off column defaults.
 	 */
-	private function sanitize_columns( $data ) {
+	private function sanitize_columns($data) {
 
 		$columns        = $this->db->get_columns();
 		$default_values = $this->db->get_column_defaults();
 
-		foreach ( $columns as $key => $type ) {
+		foreach ($columns as $key => $type) {
 
 			// Only sanitize data that we were provided
-			if ( ! array_key_exists( $key, $data ) ) {
+			if ( ! array_key_exists($key, $data)) {
 				continue;
 			}
 
-			switch ( $type ) {
+			switch ($type) {
 
 				case '%s':
-					if ( 'email' == $key ) {
-						$data[ $key ] = sanitize_email( $data[ $key ] );
-					} elseif ( 'notes' == $key ) {
-						$data[ $key ] = strip_tags( $data[ $key ] );
+					if ('email' == $key) {
+						$data[$key] = sanitize_email($data[$key]);
+					} elseif ('notes' == $key) {
+						$data[$key] = strip_tags($data[$key]);
 					} else {
-						$data[ $key ] = sanitize_text_field( $data[ $key ] );
+						$data[$key] = sanitize_text_field($data[$key]);
 					}
 					break;
 
 				case '%d':
-					if ( ! is_numeric( $data[ $key ] ) || (int) $data[ $key ] !== absint( $data[ $key ] ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_numeric($data[$key]) || (int) $data[$key] !== absint($data[$key])) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = absint( $data[ $key ] );
+						$data[$key] = absint($data[$key]);
 					}
 					break;
 
 				case '%f':
 					// Convert what was given to a float
-					$value = floatval( $data[ $key ] );
+					$value = floatval($data[$key]);
 
-					if ( ! is_float( $value ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_float($value)) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = $value;
+						$data[$key] = $value;
 					}
 					break;
 
 				default:
-					$data[ $key ] = sanitize_text_field( $data[ $key ] );
+					$data[$key] = sanitize_text_field($data[$key]);
 					break;
 
 			}
@@ -1139,33 +1139,33 @@ 
 	 *
 	 * @return bool            If the email was added successfully
 	 */
-	public function add_email( $email = '', $primary = false ) {
-		if ( ! is_email( $email ) ) {
+	public function add_email($email = '', $primary = false) {
+		if ( ! is_email($email)) {
 			return false;
 		}
-		$existing = new Give_Donor( $email );
+		$existing = new Give_Donor($email);
 
-		if ( $existing->id > 0 ) {
+		if ($existing->id > 0) {
 			// Email address already belongs to another donor
 			return false;
 		}
 
-		if ( email_exists( $email ) ) {
-			$user = get_user_by( 'email', $email );
-			if ( $user->ID != $this->user_id ) {
+		if (email_exists($email)) {
+			$user = get_user_by('email', $email);
+			if ($user->ID != $this->user_id) {
 				return false;
 			}
 		}
 
-		do_action( 'give_donor_pre_add_email', $email, $this->id, $this );
+		do_action('give_donor_pre_add_email', $email, $this->id, $this);
 
 		// Add is used to ensure duplicate emails are not added
-		$ret = (bool) $this->add_meta( 'additional_email', $email );
+		$ret = (bool) $this->add_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_add_email', $email, $this->id, $this );
+		do_action('give_donor_post_add_email', $email, $this->id, $this);
 
-		if ( $ret && true === $primary ) {
-			$this->set_primary_email( $email );
+		if ($ret && true === $primary) {
+			$this->set_primary_email($email);
 		}
 
 		return $ret;
@@ -1181,16 +1181,16 @@ 
 	 *
 	 * @return bool          If the email was removed successfully.
 	 */
-	public function remove_email( $email = '' ) {
-		if ( ! is_email( $email ) ) {
+	public function remove_email($email = '') {
+		if ( ! is_email($email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_remove_email', $email, $this->id, $this );
+		do_action('give_donor_pre_remove_email', $email, $this->id, $this);
 
-		$ret = (bool) $this->delete_meta( 'additional_email', $email );
+		$ret = (bool) $this->delete_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_remove_email', $email, $this->id, $this );
+		do_action('give_donor_post_remove_email', $email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1207,16 +1207,16 @@ 
 	 *
 	 * @return bool                      If the email was set as primary successfully.
 	 */
-	public function set_primary_email( $new_primary_email = '' ) {
-		if ( ! is_email( $new_primary_email ) ) {
+	public function set_primary_email($new_primary_email = '') {
+		if ( ! is_email($new_primary_email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this);
 
-		$existing = new Give_Donor( $new_primary_email );
+		$existing = new Give_Donor($new_primary_email);
 
-		if ( $existing->id > 0 && (int) $existing->id !== (int) $this->id ) {
+		if ($existing->id > 0 && (int) $existing->id !== (int) $this->id) {
 			// This email belongs to another donor.
 			return false;
 		}
@@ -1224,21 +1224,21 @@ 
 		$old_email = $this->email;
 
 		// Update donor record with new email.
-		$update = $this->update( array( 'email' => $new_primary_email ) );
+		$update = $this->update(array('email' => $new_primary_email));
 
 		// Remove new primary from list of additional emails.
-		$remove = $this->remove_email( $new_primary_email );
+		$remove = $this->remove_email($new_primary_email);
 
 		// Add old email to additional emails list.
-		$add = $this->add_email( $old_email );
+		$add = $this->add_email($old_email);
 
 		$ret = $update && $remove && $add;
 
-		if ( $ret ) {
+		if ($ret) {
 			$this->email = $new_primary_email;
 		}
 
-		do_action( 'give_donor_post_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_post_set_primary_email', $new_primary_email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1253,17 +1253,17 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_valid_address( $address ) {
+	private function is_valid_address($address) {
 		$is_valid_address = true;
 
 		// Address ready to process even if only one value set.
-		foreach ( $address as $address_type => $value ) {
+		foreach ($address as $address_type => $value) {
 			// @todo: Handle state field validation on basis of country.
-			if ( in_array( $address_type, array( 'line2', 'state' ) ) ) {
+			if (in_array($address_type, array('line2', 'state'))) {
 				continue;
 			}
 
-			if ( empty( $value ) ) {
+			if (empty($value)) {
 				$is_valid_address = false;
 				break;
 			}
@@ -1290,57 +1290,57 @@ 
 	 *
 	 * @return bool
 	 */
-	public function add_address( $address_type, $address ) {
+	public function add_address($address_type, $address) {
 		// Bailout.
-		if ( empty( $address_type ) || ! $this->is_valid_address( $address ) || ! $this->id ) {
+		if (empty($address_type) || ! $this->is_valid_address($address) || ! $this->id) {
 			return false;
 		}
 
 		// Check if multiple address exist or not and set params.
 		$multi_address_id = null;
-		if ( $is_multi_address = ( false !== strpos( $address_type, '[]' ) ) ) {
-			$address_type = $is_multi_address ? str_replace( '[]', '', $address_type ) : $address_type;
-		} elseif ( $is_multi_address = ( false !== strpos( $address_type, '_' ) ) ) {
-			$multi_address_id = $is_multi_address ? array_pop( explode( '_', $address_type ) ) : $address_type;
+		if ($is_multi_address = (false !== strpos($address_type, '[]'))) {
+			$address_type = $is_multi_address ? str_replace('[]', '', $address_type) : $address_type;
+		} elseif ($is_multi_address = (false !== strpos($address_type, '_'))) {
+			$multi_address_id = $is_multi_address ? array_pop(explode('_', $address_type)) : $address_type;
 
-			$address_type = $is_multi_address ? array_shift( explode( '_', $address_type ) ) : $address_type;
+			$address_type = $is_multi_address ? array_shift(explode('_', $address_type)) : $address_type;
 		}
 
 		// Bailout: do not save duplicate orders
-		if ( $this->does_address_exist( $address_type, $address ) ) {
+		if ($this->does_address_exist($address_type, $address)) {
 			return false;
 		}
 
 		// Set default address.
-		$address = wp_parse_args( $address, array(
+		$address = wp_parse_args($address, array(
 			'line1'   => '',
 			'line2'   => '',
 			'city'    => '',
 			'state'   => '',
 			'country' => '',
 			'zip'     => '',
-		) );
+		));
 
 		// Set meta key prefix.
 		global $wpdb;
 		$meta_key_prefix = "_give_donor_address_{$address_type}_{address_name}";
 		$meta_type       = Give()->donor_meta->meta_type;
 
-		if ( $is_multi_address ) {
-			if ( is_null( $multi_address_id ) ) {
+		if ($is_multi_address) {
+			if (is_null($multi_address_id)) {
 				// Get latest address key to set multi address id.
-				$multi_address_id = $wpdb->get_var( $wpdb->prepare( "
+				$multi_address_id = $wpdb->get_var($wpdb->prepare("
 						SELECT meta_key FROM {$wpdb->donormeta}
 						WHERE meta_key
 						LIKE '%%%s%%'
 						AND {$meta_type}_id=%d
 						ORDER BY meta_id DESC
 						LIMIT 1
-						", "_give_donor_address_{$address_type}_line1", $this->id ) );
+						", "_give_donor_address_{$address_type}_line1", $this->id));
 
-				if ( ! empty( $multi_address_id ) ) {
-					$multi_address_id = absint( substr( strrchr( $multi_address_id, '_' ), 1 ) );
-					$multi_address_id ++;
+				if ( ! empty($multi_address_id)) {
+					$multi_address_id = absint(substr(strrchr($multi_address_id, '_'), 1));
+					$multi_address_id++;
 				} else {
 					$multi_address_id = 0;
 				}
@@ -1350,9 +1350,9 @@ 
 		}
 
 		// Save donor address.
-		foreach ( $address as $type => $value ) {
-			$meta_key = str_replace( '{address_name}', $type, $meta_key_prefix );
-			Give()->donor_meta->update_meta( $this->id, $meta_key, $value );
+		foreach ($address as $type => $value) {
+			$meta_key = str_replace('{address_name}', $type, $meta_key_prefix);
+			Give()->donor_meta->update_meta($this->id, $meta_key, $value);
 		}
 
 		$this->setup_address();
@@ -1371,31 +1371,31 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_address( $address_id ) {
+	public function remove_address($address_id) {
 		global $wpdb;
 
 		// Get address type.
-		$is_multi_address = false !== strpos( $address_id, '_' ) ? true : false;
+		$is_multi_address = false !== strpos($address_id, '_') ? true : false;
 
-		$address_type = false !== strpos( $address_id, '_' ) ? array_shift( explode( '_', $address_id ) ) : $address_id;
+		$address_type = false !== strpos($address_id, '_') ? array_shift(explode('_', $address_id)) : $address_id;
 
-		$address_count = false !== strpos( $address_id, '_' ) ? array_pop( explode( '_', $address_id ) ) : null;
+		$address_count = false !== strpos($address_id, '_') ? array_pop(explode('_', $address_id)) : null;
 
 		// Set meta key prefix.
 		$meta_key_prefix = "_give_donor_address_{$address_type}_%";
-		if ( $is_multi_address && is_numeric( $address_count ) ) {
+		if ($is_multi_address && is_numeric($address_count)) {
 			$meta_key_prefix .= "_{$address_count}";
 		}
 
 		$meta_type = Give()->donor_meta->meta_type;
 
 		// Process query.
-		$row_affected = $wpdb->query( $wpdb->prepare( "
+		$row_affected = $wpdb->query($wpdb->prepare("
 				DELETE FROM {$wpdb->donormeta}
 				WHERE meta_key
 				LIKE '%s'
 				AND {$meta_type}_id=%d
-				", $meta_key_prefix, $this->id ) );
+				", $meta_key_prefix, $this->id));
 
 		$this->setup_address();
 
@@ -1414,39 +1414,39 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update_address( $address_id, $address ) {
+	public function update_address($address_id, $address) {
 		global $wpdb;
 
 		// Get address type.
-		$is_multi_address = false !== strpos( $address_id, '_' ) ? true : false;
+		$is_multi_address = false !== strpos($address_id, '_') ? true : false;
 
-		$address_type = false !== strpos( $address_id, '_' ) ? array_shift( explode( '_', $address_id ) ) : $address_id;
+		$address_type = false !== strpos($address_id, '_') ? array_shift(explode('_', $address_id)) : $address_id;
 
-		$address_count = false !== strpos( $address_id, '_' ) ? array_pop( explode( '_', $address_id ) ) : null;
+		$address_count = false !== strpos($address_id, '_') ? array_pop(explode('_', $address_id)) : null;
 
 		// Set meta key prefix.
 		$meta_key_prefix = "_give_donor_address_{$address_type}_%";
-		if ( $is_multi_address && is_numeric( $address_count ) ) {
+		if ($is_multi_address && is_numeric($address_count)) {
 			$meta_key_prefix .= "_{$address_count}";
 		}
 
 		$meta_type = Give()->donor_meta->meta_type;
 
 		// Process query.
-		$row_affected = $wpdb->get_results( $wpdb->prepare( "
+		$row_affected = $wpdb->get_results($wpdb->prepare("
 				SELECT meta_key FROM {$wpdb->donormeta}
 				WHERE meta_key
 				LIKE '%s'
 				AND {$meta_type}_id=%d
-				", $meta_key_prefix, $this->id ) );
+				", $meta_key_prefix, $this->id));
 
 		// Return result.
-		if ( ! count( $row_affected ) ) {
+		if ( ! count($row_affected)) {
 			return false;
 		}
 
 		// Update address.
-		if ( ! $this->add_address( $address_id, $address ) ) {
+		if ( ! $this->add_address($address_id, $address)) {
 			return false;
 		}
 
@@ -1465,39 +1465,39 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function does_address_exist( $current_address_type, $current_address ) {
+	public function does_address_exist($current_address_type, $current_address) {
 		$status = false;
 
 		// Bailout.
-		if ( empty( $current_address_type ) || empty( $current_address ) ) {
+		if (empty($current_address_type) || empty($current_address)) {
 			return null;
 		}
 
 		// Bailout.
-		if ( empty( $this->address ) || empty( $this->address[ $current_address_type ] ) ) {
+		if (empty($this->address) || empty($this->address[$current_address_type])) {
 			return $status;
 		}
 
 		// Get address.
-		$address = $this->address[ $current_address_type ];
+		$address = $this->address[$current_address_type];
 
-		switch ( true ) {
+		switch (true) {
 
 			// Single address.
-			case is_string( end( $address ) ) :
-				$status = $this->is_address_match( $current_address, $address );
+			case is_string(end($address)) :
+				$status = $this->is_address_match($current_address, $address);
 				break;
 
 			// Multi address.
-			case is_array( end( $address ) ):
+			case is_array(end($address)):
 				// Compare address.
-				foreach ( $address as $saved_address ) {
-					if ( empty( $saved_address ) ) {
+				foreach ($address as $saved_address) {
+					if (empty($saved_address)) {
 						continue;
 					}
 
 					// Exit loop immediately if address exist.
-					if ( $status = $this->is_address_match( $current_address, $saved_address ) ) {
+					if ($status = $this->is_address_match($current_address, $saved_address)) {
 						break;
 					}
 				}
@@ -1518,10 +1518,10 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_address_match( $address_1, $address_2 ) {
-		$result = array_diff( $address_1, $address_2 );
+	private function is_address_match($address_1, $address_2) {
+		$result = array_diff($address_1, $address_2);
 
-		return empty( $result );
+		return empty($result);
 	}
 
 	/**
@@ -1532,23 +1532,23 @@ 
 	 * @since   2.0
 	 * @return  object
 	 */
-	public function split_donor_name( $id ) {
+	public function split_donor_name($id) {
 		$first_name = $last_name = '';
-		$donor      = new Give_Donor( $id );
+		$donor      = new Give_Donor($id);
 
-		$split_donor_name = explode( ' ', $donor->name, 2 );
+		$split_donor_name = explode(' ', $donor->name, 2);
 
 		// Check for existence of first name after split of donor name.
-		if ( is_array( $split_donor_name ) && ! empty( $split_donor_name[0] ) ) {
+		if (is_array($split_donor_name) && ! empty($split_donor_name[0])) {
 			$first_name = $split_donor_name[0];
 		}
 
 		// Check for existence of last name after split of donor name.
-		if ( is_array( $split_donor_name ) && ! empty( $split_donor_name[1] ) ) {
+		if (is_array($split_donor_name) && ! empty($split_donor_name[1])) {
 			$last_name = $split_donor_name[1];
 		}
 
-		return (object) array( 'first_name' => $first_name, 'last_name' => $last_name );
+		return (object) array('first_name' => $first_name, 'last_name' => $last_name);
 	}
 
 	/**
@@ -1558,9 +1558,9 @@ 
 	 * @return  string
 	 */
 	public function get_first_name() {
-		$first_name = $this->get_meta( '_give_donor_first_name' );
-		if ( ! $first_name ) {
-			$first_name = $this->split_donor_name( $this->id )->first_name;
+		$first_name = $this->get_meta('_give_donor_first_name');
+		if ( ! $first_name) {
+			$first_name = $this->split_donor_name($this->id)->first_name;
 		}
 
 		return $first_name;
@@ -1573,15 +1573,15 @@ 
 	 * @return  string
 	 */
 	public function get_last_name() {
-		$first_name = $this->get_meta( '_give_donor_first_name' );
-		$last_name  = $this->get_meta( '_give_donor_last_name' );
+		$first_name = $this->get_meta('_give_donor_first_name');
+		$last_name  = $this->get_meta('_give_donor_last_name');
 
 		// This condition will prevent unnecessary splitting of donor name to fetch last name.
-		if ( ! $first_name && ! $last_name ) {
-			$last_name = $this->split_donor_name( $this->id )->last_name;
+		if ( ! $first_name && ! $last_name) {
+			$last_name = $this->split_donor_name($this->id)->last_name;
 		}
 
-		return ( $last_name ) ? $last_name : '';
+		return ($last_name) ? $last_name : '';
 	}
 
 	/**
@@ -1592,7 +1592,7 @@ 
 	 * @return  string $company_name Donor Company Name
 	 */
 	public function get_company_name() {
-		$company_name = $this->get_meta( '_give_donor_company' );
+		$company_name = $this->get_meta('_give_donor_company');
 
 		return $company_name;
 	}
@@ -1606,9 +1606,9 @@ 
 	 */
 	public function get_last_donation() {
 
-		$payments = array_unique( array_values( explode( ',', $this->payment_ids ) ) );
+		$payments = array_unique(array_values(explode(',', $this->payment_ids)));
 
-		return end( $payments );
+		return end($payments);
 
 	}
 
@@ -1621,18 +1621,18 @@ 
 	 *
 	 * @return string The date of the last donation.
 	 */
-	public function get_last_donation_date( $formatted = false ) {
+	public function get_last_donation_date($formatted = false) {
 		$completed_data = '';
 
 		// Return if donation id is invalid.
-		if( ! ( $last_donation = absint( $this->get_last_donation() ) ) ) {
+		if ( ! ($last_donation = absint($this->get_last_donation()))) {
 			return $completed_data;
 		}
 
-		$completed_data = give_get_payment_completed_date( $last_donation );
+		$completed_data = give_get_payment_completed_date($last_donation);
 
-		if ( $formatted ) {
-			return date_i18n( give_date_format(), strtotime( $completed_data ) );
+		if ($formatted) {
+			return date_i18n(give_date_format(), strtotime($completed_data));
 		}
 
 		return $completed_data;
@@ -1648,10 +1648,10 @@ 
 	 */
 	public function get_donor_initals() {
 
-		$first_name_initial = mb_substr( $this->get_first_name(), 0, 1, 'utf-8' );
-		$last_name_initial  = mb_substr( $this->get_last_name(), 0, 1, 'utf-8' );
+		$first_name_initial = mb_substr($this->get_first_name(), 0, 1, 'utf-8');
+		$last_name_initial  = mb_substr($this->get_last_name(), 0, 1, 'utf-8');
 
-		return apply_filters( 'get_donor_initals', $first_name_initial . $last_name_initial );
+		return apply_filters('get_donor_initals', $first_name_initial.$last_name_initial);
 
 	}
 

includes/donors/frontend-donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
  *
  * @since 2.2.0
  *
- * @param string|int $id_or_email
+ * @param string $id_or_email
  *
  * @return bool
  */

Spacing +42 added lines, -42 removed lines

@@ -21,15 +21,15 @@ 
  *
  * @return string HTML output.
  */
-function give_get_donor_avatar( $donor, $size = 60 ) {
+function give_get_donor_avatar($donor, $size = 60) {
 	ob_start();
 	?>
 	<div class="give-donor__image">
 		<?php
 		// Check if gravatar exists.
-		if ( give_validate_gravatar( $donor->email ) ) {
+		if (give_validate_gravatar($donor->email)) {
 			// Return avatar.
-			echo get_avatar( $donor->email, $size );
+			echo get_avatar($donor->email, $size);
 		} else {
 			// No gravatar = output initials.
 			echo $donor->get_donor_initals();
@@ -37,7 +37,7 @@ 
 	</div>
 	<?php
 
-	return apply_filters( 'give_get_donor_avatar', ob_get_clean() );
+	return apply_filters('give_get_donor_avatar', ob_get_clean());
 
 }
 
@@ -50,51 +50,51 @@ 
  *
  * @return bool
  */
-function give_validate_gravatar( $id_or_email ) {
+function give_validate_gravatar($id_or_email) {
 
 	//id or email code borrowed from wp-includes/pluggable.php
 	$email = '';
-	if ( is_numeric( $id_or_email ) ) {
+	if (is_numeric($id_or_email)) {
 		$id   = (int) $id_or_email;
-		$user = get_userdata( $id );
-		if ( $user ) {
+		$user = get_userdata($id);
+		if ($user) {
 			$email = $user->user_email;
 		}
-	} elseif ( is_object( $id_or_email ) ) {
+	} elseif (is_object($id_or_email)) {
 		// No avatar for pingbacks or trackbacks
-		$allowed_comment_types = apply_filters( 'get_avatar_comment_types', array( 'comment' ) );
-		if ( ! empty( $id_or_email->comment_type ) && ! in_array( $id_or_email->comment_type, (array) $allowed_comment_types ) ) {
+		$allowed_comment_types = apply_filters('get_avatar_comment_types', array('comment'));
+		if ( ! empty($id_or_email->comment_type) && ! in_array($id_or_email->comment_type, (array) $allowed_comment_types)) {
 			return false;
 		}
 
-		if ( ! empty( $id_or_email->user_id ) ) {
+		if ( ! empty($id_or_email->user_id)) {
 			$id   = (int) $id_or_email->user_id;
-			$user = get_userdata( $id );
-			if ( $user ) {
+			$user = get_userdata($id);
+			if ($user) {
 				$email = $user->user_email;
 			}
-		} elseif ( ! empty( $id_or_email->comment_author_email ) ) {
+		} elseif ( ! empty($id_or_email->comment_author_email)) {
 			$email = $id_or_email->comment_author_email;
 		}
 	} else {
 		$email = $id_or_email;
 	}
 
-	$hashkey = md5( strtolower( trim( $email ) ) );
-	$uri     = 'http://www.gravatar.com/avatar/' . $hashkey . '?d=404';
+	$hashkey = md5(strtolower(trim($email)));
+	$uri     = 'http://www.gravatar.com/avatar/'.$hashkey.'?d=404';
 
-	$data = wp_cache_get( $hashkey );
-	if ( false === $data ) {
-		$response = wp_remote_head( $uri );
-		if ( is_wp_error( $response ) ) {
+	$data = wp_cache_get($hashkey);
+	if (false === $data) {
+		$response = wp_remote_head($uri);
+		if (is_wp_error($response)) {
 			$data = 'not200';
 		} else {
 			$data = $response['response']['code'];
 		}
-		wp_cache_set( $hashkey, $data, $group = '', $expire = 60 * 5 );
+		wp_cache_set($hashkey, $data, $group = '', $expire = 60 * 5);
 
 	}
-	if ( $data == '200' ) {
+	if ($data == '200') {
 		return true;
 	} else {
 		return false;
@@ -114,18 +114,18 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_donor_donation_comment( $donation_id, $donor, $note, $comment_args = array() ) {
+function give_insert_donor_donation_comment($donation_id, $donor, $note, $comment_args = array()) {
 	$comment_args = wp_parse_args(
 		$comment_args,
 		array(
 			'comment_approved' => 0,
-			'comment_parent'   => give_get_payment_form_id( $donation_id )
+			'comment_parent'   => give_get_payment_form_id($donation_id)
 		)
 	);
 
-	$comment_id = Give_Comment::add( $donation_id, $note, 'payment', $comment_args );
+	$comment_id = Give_Comment::add($donation_id, $note, 'payment', $comment_args);
 
-	update_comment_meta( $comment_id, '_give_donor_id', $donor );
+	update_comment_meta($comment_id, '_give_donor_id', $donor);
 
 	return $comment_id;
 }
@@ -144,7 +144,7 @@ 
  *
  * @return WP_Comment|array
  */
-function give_get_donor_donation_comment( $donation_id, $donor_id, $search = '' ) {
+function give_get_donor_donation_comment($donation_id, $donor_id, $search = '') {
 	$comments = Give_Comment::get(
 		$donation_id,
 		'payment',
@@ -160,7 +160,7 @@ 
 		$search
 	);
 
-	return ( ! empty( $comments ) ? current( $comments ) : array() );
+	return ( ! empty($comments) ? current($comments) : array());
 }
 
 /**
@@ -176,9 +176,9 @@ 
  *
  * @return int
  */
-function give_get_donor_donation_comment_id( $donation_id, $donor_id, $search = '' ) {
+function give_get_donor_donation_comment_id($donation_id, $donor_id, $search = '') {
 	/* @var WP_Comment|array $comment */
-	$comment    = give_get_donor_donation_comment( $donation_id, $donor_id, $search );
+	$comment    = give_get_donor_donation_comment($donation_id, $donor_id, $search);
 	$comment_id = $comment instanceof WP_Comment ? $comment->comment_ID : 0;
 
 	return $comment_id;
@@ -197,7 +197,7 @@ 
  *
  * @return array
  */
-function give_get_donor_donation_comments( $donor_id, $comment_args = array(), $search = '' ) {
+function give_get_donor_donation_comments($donor_id, $comment_args = array(), $search = '') {
 	$comments = Give_Comment::get(
 		$donor_id,
 		'payment',
@@ -205,7 +205,7 @@ 
 		$search
 	);
 
-	return ( ! empty( $comments ) ? $comments : array() );
+	return ( ! empty($comments) ? $comments : array());
 }
 
 
@@ -219,19 +219,19 @@ 
  *
  * @return string
  */
-function give_get_donor_donation_comment_html( $comment, $payment_id = 0 ) {
+function give_get_donor_donation_comment_html($comment, $payment_id = 0) {
 
-	if ( is_numeric( $comment ) ) {
-		$comment = get_comment( $comment );
+	if (is_numeric($comment)) {
+		$comment = get_comment($comment);
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
 	$comment_html = sprintf(
 		'<div class="give-payment-note" id="give-payment-note-%s"><p><strong>%s</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">%s</span><br/>%s</p></div>',
 		$comment->comment_ID,
-		get_comment_author( $comment->comment_ID ),
-		date_i18n( $date_format, strtotime( $comment->comment_date ) ),
+		get_comment_author($comment->comment_ID),
+		date_i18n($date_format, strtotime($comment->comment_date)),
 		$comment->comment_content
 	);
 
@@ -250,7 +250,7 @@ 
  *
  * @return WP_Comment/array
  */
-function give_get_donor_latest_comment( $donor_id, $form_id = 0 ) {
+function give_get_donor_latest_comment($donor_id, $form_id = 0) {
 	$comment_args = array(
 		'post_id'    => 0,
 		'orderby'    => 'comment_ID',
@@ -270,11 +270,11 @@ 
 	);
 
 	// Get donor donation comment for specific form.
-	if ( $form_id ) {
+	if ($form_id) {
 		$comment_args['parent'] = $form_id;
 	}
 
-	$comment = current( give_get_donor_donation_comments( $donor_id, $comment_args ) );
+	$comment = current(give_get_donor_donation_comments($donor_id, $comment_args));
 
 	return $comment;
 }

includes/payments/class-give-payment.php

Doc Comments +2 added lines, -2 removed lines

@@ -405,7 +405,7 @@ 
 	 *
 	 * @param  int|bool $payment_id A given payment.
 	 *
-	 * @return mixed void|false
+	 * @return false|null void|false
 	 */
 	public function __construct( $payment_id = false ) {
 
@@ -1174,7 +1174,7 @@ 
 	 *
 	 * @param  string|bool $note The note to add.
 	 *
-	 * @return bool           If the note was specified or not
+	 * @return false|null           If the note was specified or not
 	 */
 	public function add_note( $note = false ) {
 		// Bail if no note specified.

Spacing +281 added lines, -282 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -407,13 +407,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);
 	}
 
 	/**
@@ -426,11 +426,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 {
 
@@ -452,18 +452,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 ( 'status' === $key ) {
+		if ('status' === $key) {
 			$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;
 		}
 	}
@@ -478,9 +478,9 @@ 
 	 *
 	 * @return boolean|null       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;
 		}
@@ -496,20 +496,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( absint( $payment_id ) );
+		$payment = get_post(absint($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;
 		}
 
@@ -523,17 +523,17 @@ 
 		 * @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);
 
 		// Get payment from cache.
-		$donation_vars = Give_Cache::get_group( $payment_id, 'give-donations' );
+		$donation_vars = Give_Cache::get_group($payment_id, 'give-donations');
 
-		if ( is_null( $donation_vars ) ) {
+		if (is_null($donation_vars)) {
 			// 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();
@@ -549,7 +549,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);
 
 			// Currency Based.
 			$this->total    = $this->setup_total();
@@ -570,7 +570,7 @@ 
 			$this->address      = $this->setup_address();
 			$this->first_name   = $this->user_info['first_name'];
 			$this->last_name    = $this->user_info['last_name'];
-			$this->title_prefix = isset( $this->user_info['title'] ) ? $this->user_info['title'] : '';
+			$this->title_prefix = isset($this->user_info['title']) ? $this->user_info['title'] : '';
 
 			// Other Identifiers.
 			$this->form_title = $this->setup_form_title();
@@ -579,10 +579,10 @@ 
 			$this->key        = $this->setup_payment_key();
 			$this->number     = $this->setup_payment_number();
 
-			Give_Cache::set_group( $this->ID, get_object_vars( $this ), 'give-donations' );
+			Give_Cache::set_group($this->ID, get_object_vars($this), 'give-donations');
 		} else {
 
-			foreach ( $donation_vars as $donation_var => $value ) {
+			foreach ($donation_vars as $donation_var => $value) {
 				$this->$donation_var = $value;
 			}
 		} // End if().
@@ -597,7 +597,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;
 	}
@@ -616,11 +616,11 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_payment_setup( $payment_id ) {
+	public function update_payment_setup($payment_id) {
 		// Delete cache.
-		Give_Cache::delete_group( $this->ID,'give-donations' );
+		Give_Cache::delete_group($this->ID, 'give-donations');
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -635,24 +635,24 @@ 
 
 		// Construct the payment title.
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} elseif ( ! empty( $this->first_name ) && empty( $this->last_name ) ) {
+		if ( ! empty($this->first_name) && ! empty($this->last_name)) {
+			$payment_title = $this->first_name.' '.$this->last_name;
+		} elseif ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} elseif ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} elseif ( ! empty($this->email) && is_email($this->email)) {
 			$payment_title = $this->email;
 		}
 
 		// Set Key.
-		if ( empty( $this->key ) ) {
+		if (empty($this->key)) {
 
-			$auth_key             = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-			$this->key            = strtolower( md5( $this->email . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );  // Unique key.
+			$auth_key             = defined('AUTH_KEY') ? AUTH_KEY : '';
+			$this->key            = strtolower(md5($this->email.date('Y-m-d H:i:s').$auth_key.uniqid('give', true))); // Unique key.
 			$this->pending['key'] = $this->key;
 		}
 
 		// Set IP.
-		if ( empty( $this->ip ) ) {
+		if (empty($this->ip)) {
 
 			$this->ip            = give_get_ip();
 			$this->pending['ip'] = $this->ip;
@@ -682,19 +682,19 @@ 
 			'status'       => $this->status,
 		);
 
-		$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;
@@ -706,43 +706,43 @@ 
 			 *
 			 * @since 1.8.13
 			 */
-			$donor = apply_filters( 'give_update_donor_information', $donor, $payment_id, $payment_data, $args );
+			$donor = apply_filters('give_update_donor_information', $donor, $payment_id, $payment_data, $args);
 
-			if ( did_action( 'give_pre_process_donation' ) && is_user_logged_in() ) {
-				$donor = new Give_Donor( get_current_user_id(), true );
+			if (did_action('give_pre_process_donation') && is_user_logged_in()) {
+				$donor = new Give_Donor(get_current_user_id(), true);
 
 				// Donor is logged in but used a different email to purchase with so assign to their donor record.
-				if ( ! empty( $donor->id ) && $this->email !== $donor->email ) {
-					$donor->add_email( $this->email );
+				if ( ! empty($donor->id) && $this->email !== $donor->email) {
+					$donor->add_email($this->email);
 				}
 			}
 
-			if ( empty( $donor->id ) ) {
-				$donor = new Give_Donor( $this->email );
+			if (empty($donor->id)) {
+				$donor = new Give_Donor($this->email);
 			}
 
-			if ( empty( $donor->id ) ) {
+			if (empty($donor->id)) {
 
 				$donor_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,
 				);
 
-				$donor->create( $donor_data );
+				$donor->create($donor_data);
 
 			}
 
 			// Update Donor Meta once donor is created.
-			$donor->update_meta( '_give_donor_first_name', $this->first_name );
-			$donor->update_meta( '_give_donor_last_name', $this->last_name );
-			$donor->update_meta( '_give_donor_title_prefix', $this->title_prefix );
+			$donor->update_meta('_give_donor_first_name', $this->first_name);
+			$donor->update_meta('_give_donor_last_name', $this->last_name);
+			$donor->update_meta('_give_donor_title_prefix', $this->title_prefix);
 
 			$this->customer_id            = $donor->id;
 			$this->pending['customer_id'] = $this->customer_id;
-			$donor->attach_payment( $this->ID, false );
+			$donor->attach_payment($this->ID, false);
 
-			$this->payment_meta = apply_filters( 'give_payment_meta', $this->payment_meta, $payment_data );
+			$this->payment_meta = apply_filters('give_payment_meta', $this->payment_meta, $payment_data);
 
 			/*
 			 * _give_payment_meta backward compatibility.
@@ -750,25 +750,25 @@ 
 			 * @since 2.0.1
 			 */
 			$custom_payment_meta = array_diff(
-				array_map( 'maybe_serialize', $this->payment_meta ),
-				array_map( 'maybe_serialize', $payment_data )
+				array_map('maybe_serialize', $this->payment_meta),
+				array_map('maybe_serialize', $payment_data)
 			);
 
-			if ( ! empty( $custom_payment_meta ) ) {
-				give_doing_it_wrong( '_give_payment_meta', __( 'This custom meta key deprecated. We are not using this meta key for storing payment meta but your custom meta data will be store because we added backward compatibility. Please change your logic because in future we can remove it.', 'give' ), '2.0.0' );
+			if ( ! empty($custom_payment_meta)) {
+				give_doing_it_wrong('_give_payment_meta', __('This custom meta key deprecated. We are not using this meta key for storing payment meta but your custom meta data will be store because we added backward compatibility. Please change your logic because in future we can remove it.', 'give'), '2.0.0');
 
-				$this->update_meta( '_give_payment_meta', array_map( 'maybe_unserialize', $custom_payment_meta ) );
+				$this->update_meta('_give_payment_meta', array_map('maybe_unserialize', $custom_payment_meta));
 			}
 
-			$give_company = ( ! empty( $_REQUEST['give_company_name'] ) ? give_clean( $_REQUEST['give_company_name'] ) : '' );
+			$give_company = ( ! empty($_REQUEST['give_company_name']) ? give_clean($_REQUEST['give_company_name']) : '');
 
 			// Check $page_url is not empty.
-			if ( $give_company ) {
-				give_update_meta( $payment_id, '_give_donation_company', $give_company );
+			if ($give_company) {
+				give_update_meta($payment_id, '_give_donation_company', $give_company);
 
-				$donor_id = absint( $donor->id );
-				if ( ! empty( $donor_id ) ) {
-					Give()->donor_meta->update_meta( $donor_id, '_give_donor_company', $give_company );
+				$donor_id = absint($donor->id);
+				if ( ! empty($donor_id)) {
+					Give()->donor_meta->update_meta($donor_id, '_give_donor_company', $give_company);
 				}
 			}
 
@@ -792,11 +792,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;
@@ -804,42 +804,42 @@ 
 		}
 
 		// 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 donation to logs.
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
-										give_record_donation_in_log( $item['id'], $this->ID, $price_id, $log_date );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
+										give_record_donation_in_log($item['id'], $this->ID, $price_id, $log_date);
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price, $this->ID );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price, $this->ID);
 
 										$total_increase += $price;
 									}
@@ -847,10 +847,10 @@ 
 
 								case 'remove':
 									$this->delete_sales_logs();
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'], $this->ID );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount'], $this->ID);
 
 										$total_decrease += $item['amount'];
 									}
@@ -861,103 +861,103 @@ 
 						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_donor_ip', $this->ip );
+						$this->update_meta('_give_payment_donor_ip', $this->ip);
 						break;
 
 					case 'customer_id':
-						$this->update_meta( '_give_payment_donor_id', $this->customer_id );
+						$this->update_meta('_give_payment_donor_id', $this->customer_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':
-						$this->update_meta( '_give_donor_billing_first_name', $this->first_name );
+						$this->update_meta('_give_donor_billing_first_name', $this->first_name);
 						break;
 
 					case 'last_name':
-						$this->update_meta( '_give_donor_billing_last_name', $this->last_name );
+						$this->update_meta('_give_donor_billing_last_name', $this->last_name);
 						break;
 
 					case 'currency':
-						$this->update_meta( '_give_payment_currency', $this->currency );
+						$this->update_meta('_give_payment_currency', $this->currency);
 						break;
 
 					case 'address':
-						if ( ! empty( $this->address ) ) {
-							foreach ( $this->address as $address_name => $address ) {
-								switch ( $address_name ) {
+						if ( ! empty($this->address)) {
+							foreach ($this->address as $address_name => $address) {
+								switch ($address_name) {
 									case 'line1':
-										$this->update_meta( '_give_donor_billing_address1', $address );
+										$this->update_meta('_give_donor_billing_address1', $address);
 										break;
 
 									case 'line2':
-										$this->update_meta( '_give_donor_billing_address2', $address );
+										$this->update_meta('_give_donor_billing_address2', $address);
 										break;
 
 									default:
-										$this->update_meta( "_give_donor_billing_{$address_name}", $address );
+										$this->update_meta("_give_donor_billing_{$address_name}", $address);
 								}
 							}
 						}
 						break;
 
 					case 'email':
-						$this->update_meta( '_give_payment_donor_email', $this->email );
+						$this->update_meta('_give_payment_donor_email', $this->email);
 						break;
 
 					case 'title_prefix':
-						$this->update_meta( '_give_payment_donor_title_prefix', $this->title_prefix );
+						$this->update_meta('_give_payment_donor_title_prefix', $this->title_prefix);
 						break;
 
 					case 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
 						// @todo: remove unused meta data.
 						// Core is using post_title to store donation serial code ( fi enabled ) instead this meta key.
 						// Do not use this meta key in your logic, can be remove in future
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
 						$args = array(
 							'ID'        => $this->ID,
-							'post_date' => date( 'Y-m-d H:i:s', strtotime( $this->date ) ),
+							'post_date' => date('Y-m-d H:i:s', strtotime($this->date)),
 							'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':
@@ -966,11 +966,11 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'total':
-						$this->update_meta( '_give_payment_total', give_sanitize_amount_for_db( $this->total ) );
+						$this->update_meta('_give_payment_total', give_sanitize_amount_for_db($this->total));
 						break;
 
 					default:
@@ -981,46 +981,46 @@ 
 						 *
 						 * @param Give_Payment $this Payment object.
 						 */
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				} // End switch().
 			} // End foreach().
 
-			if ( 'pending' !== $this->status ) {
+			if ('pending' !== $this->status) {
 
-				$donor = new Give_Donor( $this->customer_id );
+				$donor = new Give_Donor($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 donor's donation stats.
-					$donor->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$donor->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
 					$donor->decrease_donation_count();
 
-				} elseif ( $total_change > 0 ) {
+				} elseif ($total_change > 0) {
 
 					// Increase the donor's donation stats.
-					$donor->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$donor->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 					$donor->increase_purchase_count();
 
 				}
 
 				// Verify and update form meta based on the form status.
-				give_set_form_closed_status( $this->form_id );
+				give_set_form_closed_status($this->form_id);
 			}
 
 			$this->pending = array();
 			$saved         = true;
 		} // End if().
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -1038,12 +1038,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 || 'give_forms' !== $donation->post_type ) {
+		if ( ! $donation || 'give_forms' !== $donation->post_type) {
 			return false;
 		}
 
@@ -1053,25 +1053,25 @@ 
 			'price_id' => false,
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_payment_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_payment_add_donation_args', $args, $donation->ID), $defaults);
 
 		// Allow overriding the price.
-		if ( false !== $args['price'] ) {
+		if (false !== $args['price']) {
 			$donation_amount = $args['price'];
 		} else {
 
 			// Deal with variable pricing.
-			if ( give_has_variable_prices( $donation->ID ) ) {
-				$prices          = give_get_meta( $form_id, '_give_donation_levels', true );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices          = give_get_meta($form_id, '_give_donation_levels', true);
 				$donation_amount = '';
 
 				// 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'] ) &&
+						isset($args['price_id']) &&
+						isset($price['_give_id']['level_id']) &&
 						$args['price_id'] === (int) $price['_give_id']['level_id']
 					) {
 						$donation_amount = $price['_give_amount'];
@@ -1079,37 +1079,37 @@ 
 				}
 
 				// Fallback to the lowest price point.
-				if ( '' === $donation_amount ) {
-					$donation_amount  = give_get_lowest_price_option( $donation->ID );
-					$args['price_id'] = give_get_lowest_price_id( $donation->ID );
+				if ('' === $donation_amount) {
+					$donation_amount  = give_get_lowest_price_option($donation->ID);
+					$args['price_id'] = give_get_lowest_price_id($donation->ID);
 				}
 			} else {
 				// Simple form price.
-				$donation_amount = give_get_form_price( $donation->ID );
+				$donation_amount = give_get_form_price($donation->ID);
 			}
 		}
 
 		// Sanitizing the price here so we don't have a dozen calls later.
-		$donation_amount = give_maybe_sanitize_amount( $donation_amount );
-		$total           = round( $donation_amount, give_get_price_decimals( $this->ID ) );
+		$donation_amount = give_maybe_sanitize_amount($donation_amount);
+		$total           = round($donation_amount, give_get_price_decimals($this->ID));
 
 		// 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_get_price_decimals( $this->ID ) ),
-			'subtotal' => round( $total, give_get_price_decimals( $this->ID ) ),
+			'price'    => round($total, give_get_price_decimals($this->ID)),
+			'subtotal' => round($total, give_get_price_decimals($this->ID)),
 			'price_id' => $args['price_id'],
 			'action'   => 'add',
 			'options'  => $options,
@@ -1117,7 +1117,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1134,7 +1134,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(
@@ -1142,12 +1142,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 || 'give_forms' !== $form->post_type ) {
+		if ( ! $form || 'give_forms' !== $form->post_type) {
 			return false;
 		}
 
@@ -1160,7 +1160,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1176,13 +1176,13 @@ 
 	 *
 	 * @return bool           If the note was specified or not
 	 */
-	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);
 	}
 
 	/**
@@ -1195,8 +1195,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function increase_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
 		$this->recalculate_total();
@@ -1212,11 +1212,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function decrease_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1246,24 +1246,24 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated.
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		// standardize the 'complete(d)' status.
-		if ( 'completed' === $status || 'complete' === $status ) {
+		if ('completed' === $status || 'complete' === $status) {
 			$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.
@@ -1274,21 +1274,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.
-			$this->process_status( $status );
+			$this->process_status($status);
 
 			/**
 			 * Fires after changing payment status.
@@ -1299,7 +1299,7 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		} // End if().
 
@@ -1334,27 +1334,27 @@ 
 	 *
 	 * @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) {
 		if (
-			! has_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta' ) &&
-			! doing_filter( 'get_post_metadata' )
+			! has_filter('get_post_metadata', 'give_bc_v20_get_payment_meta') &&
+			! doing_filter('get_post_metadata')
 		) {
-			add_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4 );
+			add_filter('get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4);
 		}
 
-		$meta = give_get_meta( $this->ID, $meta_key, $single );
+		$meta = give_get_meta($this->ID, $meta_key, $single);
 
 		/**
 		 * Filter the specific meta key value.
 		 *
 		 * @since 1.5
 		 */
-		$meta = apply_filters( "give_get_payment_meta_{$meta_key}", $meta, $this->ID );
+		$meta = apply_filters("give_get_payment_meta_{$meta_key}", $meta, $this->ID);
 
 		// Security check.
-		if ( is_serialized( $meta ) ) {
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $meta, $matches );
-			if ( ! empty( $matches ) ) {
+		if (is_serialized($meta)) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $meta, $matches);
+			if ( ! empty($matches)) {
 				$meta = array();
 			}
 		}
@@ -1364,7 +1364,7 @@ 
 		 *
 		 * @since 1.5
 		 */
-		return apply_filters( 'give_get_payment_meta', $meta, $this->ID, $meta_key );
+		return apply_filters('give_get_payment_meta', $meta, $this->ID, $meta_key);
 	}
 
 	/**
@@ -1379,8 +1379,8 @@ 
 	 *
 	 * @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;
 		}
 
@@ -1390,9 +1390,9 @@ 
 		 *
 		 * @since 1.5
 		 */
-		$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
+		$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
 
-		return give_update_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return give_update_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1405,7 +1405,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function process_status( $status ) {
+	private function process_status($status) {
 		$process = true;
 
 		// Bailout, if changed from completed to preapproval/processing.
@@ -1420,9 +1420,9 @@ 
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process = apply_filters( "give_should_process_{$status}", $process, $this );
+		$process = apply_filters("give_should_process_{$status}", $process, $this);
 
-		if ( false === $process ) {
+		if (false === $process) {
 			return;
 		}
 
@@ -1433,13 +1433,13 @@ 
 		 *
 		 * @since 1.5
 		 */
-		do_action( "give_pre_{$status}_payment", $this );
+		do_action("give_pre_{$status}_payment", $this);
 
-		$decrease_earnings       = apply_filters( "give_decrease_earnings_on_{$status}", true, $this );
-		$decrease_donor_value    = apply_filters( "give_decrease_donor_value_on_{$status}", true, $this );
-		$decrease_donation_count = apply_filters( "give_decrease_donors_donation_count_on_{$status}", true, $this );
+		$decrease_earnings       = apply_filters("give_decrease_earnings_on_{$status}", true, $this);
+		$decrease_donor_value    = apply_filters("give_decrease_donor_value_on_{$status}", true, $this);
+		$decrease_donation_count = apply_filters("give_decrease_donors_donation_count_on_{$status}", true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1452,7 +1452,7 @@ 
 		 *
 		 * @since 1.5
 		 */
-		do_action( "give_post_{$status}_payment", $this );
+		do_action("give_post_{$status}_payment", $this);
 	}
 
 	/**
@@ -1467,25 +1467,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_donation( $this->ID );
+		give_undo_donation($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 donor.
-		if ( ! empty( $this->customer_id ) ) {
+		if ( ! empty($this->customer_id)) {
 
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( true === $alter_customer_value ) {
-				$donor->decrease_value( $this->total );
+			if (true === $alter_customer_value) {
+				$donor->decrease_value($this->total);
 			}
 
-			if ( true === $alter_customer_purchase_count ) {
+			if (true === $alter_customer_purchase_count) {
 				$donor->decrease_donation_count();
 			}
 		}
@@ -1502,7 +1502,7 @@ 
 	 */
 	private function delete_sales_logs() {
 		// Remove related sale log entries.
-		Give()->logs->delete_logs( $this->ID );
+		Give()->logs->delete_logs($this->ID);
 	}
 
 	/**
@@ -1523,13 +1523,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;
 	}
@@ -1543,7 +1543,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1555,7 +1555,7 @@ 
 	 * @return bool The payment import
 	 */
 	private function setup_import() {
-		return (bool) $this->get_meta( '_give_payment_import' );
+		return (bool) $this->get_meta('_give_payment_import');
 	}
 
 	/**
@@ -1567,9 +1567,9 @@ 
 	 * @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);
 
-		return round( floatval( $amount ), give_get_price_decimals( $this->ID ) );
+		return round(floatval($amount), give_get_price_decimals($this->ID));
 	}
 
 	/**
@@ -1596,17 +1596,16 @@ 
 	 * @return string The currency for the payment
 	 */
 	private function setup_currency() {
-		$currency = $this->get_meta( '_give_payment_currency', true );
-		$currency = ! empty( $currency ) ?
-			$currency :
-			/**
+		$currency = $this->get_meta('_give_payment_currency', true);
+		$currency = ! empty($currency) ?
+			$currency : /**
 			 * Filter the default donation currency
 			 *
 			 * @since 1.5
 			 */
 			apply_filters(
 				'give_payment_currency_default',
-				give_get_currency( $this->form_id, $this ),
+				give_get_currency($this->form_id, $this),
 				$this
 			);
 
@@ -1622,7 +1621,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;
 	}
@@ -1636,11 +1635,11 @@ 
 	 * @return string The donation ID
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) ) {
+		if (empty($transaction_id)) {
 			$gateway        = $this->gateway;
-			$transaction_id = apply_filters( "give_get_payment_transaction_id-{$gateway}", $this->ID );
+			$transaction_id = apply_filters("give_get_payment_transaction_id-{$gateway}", $this->ID);
 		}
 
 		return $transaction_id;
@@ -1656,7 +1655,7 @@ 
 	 * @return string The IP address for the payment
 	 */
 	private function setup_ip() {
-		$ip = $this->get_meta( '_give_payment_donor_ip', true );
+		$ip = $this->get_meta('_give_payment_donor_ip', true);
 
 		return $ip;
 	}
@@ -1671,7 +1670,7 @@ 
 	 * @return int The Donor ID.
 	 */
 	private function setup_donor_id() {
-		$donor_id = $this->get_meta( '_give_payment_donor_id', true );
+		$donor_id = $this->get_meta('_give_payment_donor_id', true);
 
 		return $donor_id;
 	}
@@ -1688,8 +1687,8 @@ 
 	 */
 	private function setup_user_id() {
 
-		$donor   = Give()->customers->get_customer_by( 'id', $this->customer_id );
-		$user_id = $donor ? absint( $donor->user_id ) : 0;
+		$donor   = Give()->customers->get_customer_by('id', $this->customer_id);
+		$user_id = $donor ? absint($donor->user_id) : 0;
 
 
 		return $user_id;
@@ -1706,10 +1705,10 @@ 
 	 * @return string The email address for the payment.
 	 */
 	private function setup_email() {
-		$email = $this->get_meta( '_give_payment_donor_email', true );
+		$email = $this->get_meta('_give_payment_donor_email', true);
 
-		if ( empty( $email ) && $this->customer_id ) {
-			$email = Give()->donors->get_column( 'email', $this->customer_id );
+		if (empty($email) && $this->customer_id) {
+			$email = Give()->donors->get_column('email', $this->customer_id);
 		}
 
 		return $email;
@@ -1730,22 +1729,22 @@ 
 			'last_name'  => $this->last_name,
 		);
 
-		$user_info = isset( $this->payment_meta['user_info'] ) ? $this->payment_meta['user_info'] : array();
+		$user_info = isset($this->payment_meta['user_info']) ? $this->payment_meta['user_info'] : array();
 
-		if ( is_serialized( $user_info ) ) {
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches );
-			if ( ! empty( $matches ) ) {
+		if (is_serialized($user_info)) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches);
+			if ( ! empty($matches)) {
 				$user_info = array();
 			}
 		}
 
-		$user_info = wp_parse_args( $user_info, $defaults );
+		$user_info = wp_parse_args($user_info, $defaults);
 
-		if ( empty( $user_info ) ) {
+		if (empty($user_info)) {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( $donor->id > 0 ) {
+			if ($donor->id > 0) {
 				$user_info = array(
 					'first_name' => $donor->get_first_name(),
 					'last_name'  => $donor->get_last_name(),
@@ -1755,29 +1754,29 @@ 
 			}
 		} else {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( $donor->id > 0 ) {
-				foreach ( $user_info as $key => $value ) {
-					if ( ! empty( $value ) ) {
+			if ($donor->id > 0) {
+				foreach ($user_info as $key => $value) {
+					if ( ! empty($value)) {
 						continue;
 					}
 
-					switch ( $key ) {
+					switch ($key) {
 						case 'title':
-							$user_info[ $key ] = Give()->donor_meta->get_meta( $donor->id, '_give_donor_title_prefix', true );
+							$user_info[$key] = Give()->donor_meta->get_meta($donor->id, '_give_donor_title_prefix', true);
 							break;
 
 						case 'first_name':
-							$user_info[ $key ] = $donor->get_first_name();
+							$user_info[$key] = $donor->get_first_name();
 							break;
 
 						case 'last_name':
-							$user_info[ $key ] = $donor->get_last_name();
+							$user_info[$key] = $donor->get_last_name();
 							break;
 
 						case 'email':
-							$user_info[ $key ] = $donor->email;
+							$user_info[$key] = $donor->email;
 							break;
 					}
 				}
@@ -1797,12 +1796,12 @@ 
 	 * @return array The Address information for the payment.
 	 */
 	private function setup_address() {
-		$address['line1']   = give_get_meta( $this->ID, '_give_donor_billing_address1', true, '' );
-		$address['line2']   = give_get_meta( $this->ID, '_give_donor_billing_address2', true, '' );
-		$address['city']    = give_get_meta( $this->ID, '_give_donor_billing_city', true, '' );
-		$address['state']   = give_get_meta( $this->ID, '_give_donor_billing_state', true, '' );
-		$address['zip']     = give_get_meta( $this->ID, '_give_donor_billing_zip', true, '' );
-		$address['country'] = give_get_meta( $this->ID, '_give_donor_billing_country', true, '' );
+		$address['line1']   = give_get_meta($this->ID, '_give_donor_billing_address1', true, '');
+		$address['line2']   = give_get_meta($this->ID, '_give_donor_billing_address2', true, '');
+		$address['city']    = give_get_meta($this->ID, '_give_donor_billing_city', true, '');
+		$address['state']   = give_get_meta($this->ID, '_give_donor_billing_state', true, '');
+		$address['zip']     = give_get_meta($this->ID, '_give_donor_billing_zip', true, '');
+		$address['country'] = give_get_meta($this->ID, '_give_donor_billing_country', true, '');
 
 		return $address;
 	}
@@ -1817,7 +1816,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;
 	}
@@ -1832,7 +1831,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;
 	}
@@ -1846,7 +1845,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;
 	}
@@ -1860,7 +1859,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;
 	}
@@ -1885,7 +1884,7 @@ 
 	 * @return array The payment object as an array.
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -1898,7 +1897,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -1910,7 +1909,7 @@ 
 	 * @return string Date payment was completed.
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_payment_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_payment_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -1922,7 +1921,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);
 	}
 
 	/**
@@ -1934,7 +1933,7 @@ 
 	 * @return string Payment currency code.
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_payment_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_payment_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -1946,7 +1945,7 @@ 
 	 * @return string Gateway used.
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_payment_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_payment_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -1958,7 +1957,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);
 	}
 
 	/**
@@ -1970,7 +1969,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_payment_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_payment_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -1982,7 +1981,7 @@ 
 	 * @return int Payment donor ID.
 	 */
 	private function get_donor_id() {
-		return apply_filters( 'give_payment_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_payment_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -1994,7 +1993,7 @@ 
 	 * @return int Payment user ID.
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_payment_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_payment_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -2006,7 +2005,7 @@ 
 	 * @return string Payment donor email.
 	 */
 	private function get_email() {
-		return apply_filters( 'give_payment_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_payment_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2018,7 +2017,7 @@ 
 	 * @return array Payment user info.
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_payment_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_payment_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2030,7 +2029,7 @@ 
 	 * @return array Payment billing address.
 	 */
 	private function get_address() {
-		return apply_filters( 'give_payment_address', $this->address, $this->ID, $this );
+		return apply_filters('give_payment_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2042,7 +2041,7 @@ 
 	 * @return string Payment key.
 	 */
 	private function get_key() {
-		return apply_filters( 'give_payment_key', $this->key, $this->ID, $this );
+		return apply_filters('give_payment_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2054,7 +2053,7 @@ 
 	 * @return string Payment form id
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_payment_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_payment_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2066,7 +2065,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 
 	/**
@@ -2078,7 +2077,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_serial_code( $args = array() ) {
-		return Give()->seq_donation_number->get_serial_code( $this, $args );
+	public function get_serial_code($args = array()) {
+		return Give()->seq_donation_number->get_serial_code($this, $args);
 	}
 }

includes/shortcodes.php

Doc Comments +1 added lines, -1 removed lines

@@ -789,7 +789,7 @@
  * @type string $display_style       How the form is displayed, either in new page or modal popup.
  *                                       Default 'redirect'. Accepts 'redirect', 'modal'.
  * }
- * @return string|bool The markup of the form grid or false.
+ * @return string|null The markup of the form grid or false.
  */
 function give_form_grid_shortcode( $atts ) {
 

Spacing +213 added lines, -213 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,42 +26,42 @@ 
  *
  * @return string|bool
  */
-function give_donation_history( $atts, $content = false ) {
+function give_donation_history($atts, $content = false) {
 
-	$donation_history_args = shortcode_atts( array(
+	$donation_history_args = shortcode_atts(array(
 		'id'             => true,
 		'date'           => true,
 		'donor'          => false,
 		'amount'         => true,
 		'status'         => false,
 		'payment_method' => false,
-	), $atts, 'donation_history' );
+	), $atts, 'donation_history');
 
 	// Always show receipt link.
 	$donation_history_args['details'] = true;
 
 	// Set Donation History Shortcode Arguments in session variable.
-	Give()->session->set( 'give_donation_history_args', $donation_history_args );
+	Give()->session->set('give_donation_history_args', $donation_history_args);
 
 	// If payment_key query arg exists, return receipt instead of donation history.
-	if ( isset( $_GET['payment_key'] ) ) {
+	if (isset($_GET['payment_key'])) {
 		ob_start();
 
-		echo give_receipt_shortcode( array() );
+		echo give_receipt_shortcode(array());
 
 		// Display donation history link only if Receipt Access Session is available.
-		if ( give_get_receipt_session() ) {
+		if (give_get_receipt_session()) {
 			echo sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( give_get_history_page_uri() ),
-				__( '&laquo; Return to All Donations', 'give' )
+				esc_url(give_get_history_page_uri()),
+				__('&laquo; Return to All Donations', 'give')
 			);
 		}
 
 		return ob_get_clean();
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	ob_start();
 
@@ -74,23 +74,23 @@ 
 	if (
 		is_user_logged_in()
 		|| false !== Give()->session->get_session_expiration()
-		|| ( give_is_setting_enabled( $email_access ) && Give()->email_access->token_exists )
+		|| (give_is_setting_enabled($email_access) && Give()->email_access->token_exists)
 		|| true === give_get_history_session()
 	) {
-		give_get_template_part( 'history', 'donations' );
+		give_get_template_part('history', 'donations');
 
-		if ( ! empty( $content ) ) {
-			echo do_shortcode( $content );
+		if ( ! empty($content)) {
+			echo do_shortcode($content);
 		}
 
-	} elseif ( give_is_setting_enabled( $email_access ) ) {
+	} elseif (give_is_setting_enabled($email_access)) {
 		// Is Email-based access enabled?
-		give_get_template_part( 'email', 'login-form' );
+		give_get_template_part('email', 'login-form');
 
 	} else {
 
-		echo apply_filters( 'give_donation_history_nonuser_message', Give()->notices->print_frontend_notice( __( 'You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give' ), false ) );
-		echo do_shortcode( '[give_login]' );
+		echo apply_filters('give_donation_history_nonuser_message', Give()->notices->print_frontend_notice(__('You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give'), false));
+		echo do_shortcode('[give_login]');
 	}
 
 	/**
@@ -104,10 +104,10 @@ 
 	 *
 	 * @return string HTML content
 	 */
-	return apply_filters( 'give_donation_history_shortcode_html', ob_get_clean(), $atts, $content );
+	return apply_filters('give_donation_history_shortcode_html', ob_get_clean(), $atts, $content);
 }
 
-add_shortcode( 'donation_history', 'give_donation_history' );
+add_shortcode('donation_history', 'give_donation_history');
 
 /**
  * Donation Form Shortcode
@@ -120,8 +120,8 @@ 
  *
  * @return string
  */
-function give_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'                    => '',
 		'show_title'            => true,
 		'show_goal'             => true,
@@ -129,21 +129,21 @@ 
 		'float_labels'          => '',
 		'display_style'         => '',
 		'continue_button_title' => '',
-	), $atts, 'give_form' );
+	), $atts, 'give_form');
 
 	// Convert string to bool.
-	$atts['show_title'] = filter_var( $atts['show_title'], FILTER_VALIDATE_BOOLEAN );
-	$atts['show_goal']  = filter_var( $atts['show_goal'], FILTER_VALIDATE_BOOLEAN );
+	$atts['show_title'] = filter_var($atts['show_title'], FILTER_VALIDATE_BOOLEAN);
+	$atts['show_goal']  = filter_var($atts['show_goal'], FILTER_VALIDATE_BOOLEAN);
 
 	// get the Give Form
 	ob_start();
-	give_get_donation_form( $atts );
+	give_get_donation_form($atts);
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_donate_form', $final_output, $atts );
+	return apply_filters('give_donate_form', $final_output, $atts);
 }
 
-add_shortcode( 'give_form', 'give_form_shortcode' );
+add_shortcode('give_form', 'give_form_shortcode');
 
 /**
  * Donation Form Goal Shortcode.
@@ -156,36 +156,36 @@ 
  *
  * @return string
  */
-function give_goal_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_goal_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'        => '',
 		'show_text' => true,
 		'show_bar'  => true,
-	), $atts, 'give_goal' );
+	), $atts, 'give_goal');
 
 	// get the Give Form.
 	ob_start();
 
 	// Sanity check 1: ensure there is an ID Provided.
-	if ( empty( $atts['id'] ) ) {
-		Give()->notices->print_frontend_notice( __( 'The shortcode is missing Donation Form ID attribute.', 'give' ), true );
+	if (empty($atts['id'])) {
+		Give()->notices->print_frontend_notice(__('The shortcode is missing Donation Form ID attribute.', 'give'), true);
 	}
 
 	// Sanity check 2: Check the form even has Goals enabled.
-	if ( ! give_is_setting_enabled( give_get_meta( $atts['id'], '_give_goal_option', true ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_meta($atts['id'], '_give_goal_option', true))) {
 
-		Give()->notices->print_frontend_notice( __( 'The form does not have Goals enabled.', 'give' ), true );
+		Give()->notices->print_frontend_notice(__('The form does not have Goals enabled.', 'give'), true);
 	} else {
 		// Passed all sanity checks: output Goal.
-		give_show_goal_progress( $atts['id'], $atts );
+		give_show_goal_progress($atts['id'], $atts);
 	}
 
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_goal_shortcode_output', $final_output, $atts );
+	return apply_filters('give_goal_shortcode_output', $final_output, $atts);
 }
 
-add_shortcode( 'give_goal', 'give_goal_shortcode' );
+add_shortcode('give_goal', 'give_goal_shortcode');
 
 
 /**
@@ -202,22 +202,22 @@ 
  *
  * @return string
  */
-function give_login_form_shortcode( $atts ) {
+function give_login_form_shortcode($atts) {
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		// Add backward compatibility for redirect attribute.
 		'redirect'        => '',
 		'login-redirect'  => '',
 		'logout-redirect' => '',
-	), $atts, 'give_login' );
+	), $atts, 'give_login');
 
 	// Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
-	$atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect'] ) ? $atts['redirect'] : '' );
+	$atts['login-redirect'] = ! empty($atts['login-redirect']) ? $atts['login-redirect'] : ( ! empty($atts['redirect']) ? $atts['redirect'] : '');
 
-	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
+	return give_login_form($atts['login-redirect'], $atts['logout-redirect']);
 }
 
-add_shortcode( 'give_login', 'give_login_form_shortcode' );
+add_shortcode('give_login', 'give_login_form_shortcode');
 
 /**
  * Register Shortcode.
@@ -232,15 +232,15 @@ 
  *
  * @return string
  */
-function give_register_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_register_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'redirect' => '',
-	), $atts, 'give_register' );
+	), $atts, 'give_register');
 
-	return give_register_form( $atts['redirect'] );
+	return give_register_form($atts['redirect']);
 }
 
-add_shortcode( 'give_register', 'give_register_form_shortcode' );
+add_shortcode('give_register', 'give_register_form_shortcode');
 
 /**
  * Receipt Shortcode.
@@ -253,13 +253,13 @@ 
  *
  * @return string
  */
-function give_receipt_shortcode( $atts ) {
+function give_receipt_shortcode($atts) {
 
 	global $give_receipt_args;
 	$payment_key = '';
 
-	$give_receipt_args = shortcode_atts( array(
-		'error'          => __( 'You are missing the payment key to view this donation receipt.', 'give' ),
+	$give_receipt_args = shortcode_atts(array(
+		'error'          => __('You are missing the payment key to view this donation receipt.', 'give'),
 		'price'          => true,
 		'donor'          => true,
 		'date'           => true,
@@ -269,62 +269,62 @@ 
 		'payment_status' => false,
 		'company_name'   => false,
 		'status_notice'  => true,
-	), $atts, 'give_receipt' );
+	), $atts, 'give_receipt');
 
 	// set $session var
 	$session = give_get_purchase_session();
 
 	// set payment key var
-	if ( isset( $_GET['payment_key'] ) ) {
-		$payment_key = urldecode( $_GET['payment_key'] );
-	} elseif ( $session ) {
+	if (isset($_GET['payment_key'])) {
+		$payment_key = urldecode($_GET['payment_key']);
+	} elseif ($session) {
 		$payment_key = $session['purchase_key'];
-	} elseif ( $give_receipt_args['payment_key'] ) {
+	} elseif ($give_receipt_args['payment_key']) {
 		$payment_key = $give_receipt_args['payment_key'];
 	}
 
-	if( ! wp_doing_ajax() ) {
+	if ( ! wp_doing_ajax()) {
 		ob_start();
-		give_get_template_part( 'receipt/placeholder' );
+		give_get_template_part('receipt/placeholder');
 		$placeholder = ob_get_clean();
 
 		return sprintf(
 			'<div id="give-receipt" data-shortcode="%s" data-donation-key="%s">%s</div>',
-			urlencode_deep( wp_json_encode( $atts ) ),
+			urlencode_deep(wp_json_encode($atts)),
 			$payment_key,
 			$placeholder
 		);
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	// No payment_key found & Email Access is Turned on.
-	if ( ! isset( $payment_key ) && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! isset($payment_key) && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! isset( $payment_key ) ) {
+	} elseif ( ! isset($payment_key)) {
 
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 
 	}
 
-	$user_can_view = give_can_view_receipt( $payment_key );
+	$user_can_view = give_can_view_receipt($payment_key);
 
 	// Key was provided, but user is logged out. Offer them the ability to login and view the receipt.
-	if ( ! $user_can_view && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! $user_can_view && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! $user_can_view ) {
+	} elseif ( ! $user_can_view) {
 
 		global $give_login_redirect;
 
@@ -332,9 +332,9 @@ 
 
 		ob_start();
 
-		Give()->notices->print_frontend_notice( apply_filters( 'give_must_be_logged_in_error_message', __( 'You must be logged in to view this donation receipt.', 'give' ) ) );
+		Give()->notices->print_frontend_notice(apply_filters('give_must_be_logged_in_error_message', __('You must be logged in to view this donation receipt.', 'give')));
 
-		give_get_template_part( 'shortcode', 'login' );
+		give_get_template_part('shortcode', 'login');
 
 		$login_form = ob_get_clean();
 
@@ -348,20 +348,20 @@ 
 	 * or if user is logged out and donation was made as a guest, the donation session is checked for
 	 * or if user is logged in and the user can view sensitive shop data.
 	 */
-	if ( ! apply_filters( 'give_user_can_view_receipt', $user_can_view, $give_receipt_args ) ) {
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+	if ( ! apply_filters('give_user_can_view_receipt', $user_can_view, $give_receipt_args)) {
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 	}
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'receipt' );
+	give_get_template_part('shortcode', 'receipt');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode.
@@ -380,26 +380,26 @@ 
  *
  * @return string Output generated from the profile editor
  */
-function give_profile_editor_shortcode( $atts ) {
+function give_profile_editor_shortcode($atts) {
 
 	ob_start();
 
 	// Restrict access to donor profile, if donor and user are disconnected.
-	$is_donor_disconnected = get_user_meta( get_current_user_id(), '_give_is_donor_disconnected', true );
-	if ( is_user_logged_in() && $is_donor_disconnected ) {
-		Give()->notices->print_frontend_notice( __( 'Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give' ), true, 'error' );
+	$is_donor_disconnected = get_user_meta(get_current_user_id(), '_give_is_donor_disconnected', true);
+	if (is_user_logged_in() && $is_donor_disconnected) {
+		Give()->notices->print_frontend_notice(__('Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give'), true, 'error');
 
 		return false;
 	}
 
-	give_get_template_part( 'shortcode', 'profile-editor' );
+	give_get_template_part('shortcode', 'profile-editor');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_profile_editor', 'give_profile_editor_shortcode' );
+add_shortcode('give_profile_editor', 'give_profile_editor_shortcode');
 
 /**
  * Process Profile Updater Form.
@@ -412,31 +412,31 @@ 
  *
  * @return bool
  */
-function give_process_profile_editor_updates( $data ) {
+function give_process_profile_editor_updates($data) {
 	// Profile field change request.
-	if ( empty( $_POST['give_profile_editor_submit'] ) && ! is_user_logged_in() ) {
+	if (empty($_POST['give_profile_editor_submit']) && ! is_user_logged_in()) {
 		return false;
 	}
 
 	// Nonce security.
-	if ( ! wp_verify_nonce( $data['give_profile_editor_nonce'], 'give-profile-editor-nonce' ) ) {
+	if ( ! wp_verify_nonce($data['give_profile_editor_nonce'], 'give-profile-editor-nonce')) {
 		return false;
 	}
 
 	$user_id       = get_current_user_id();
-	$old_user_data = get_userdata( $user_id );
+	$old_user_data = get_userdata($user_id);
 
 	/* @var Give_Donor $donor */
-	$donor            = new Give_Donor( $user_id, true );
+	$donor            = new Give_Donor($user_id, true);
 	$old_company_name = $donor->get_company_name();
 
-	$display_name     = isset( $data['give_display_name'] ) ? sanitize_text_field( $data['give_display_name'] ) : $old_user_data->display_name;
-	$first_name       = isset( $data['give_first_name'] ) ? sanitize_text_field( $data['give_first_name'] ) : $old_user_data->first_name;
-	$last_name        = isset( $data['give_last_name'] ) ? sanitize_text_field( $data['give_last_name'] ) : $old_user_data->last_name;
-	$company_name     = ! empty( $data['give_company_name'] ) ? sanitize_text_field( $data['give_company_name'] ) : $old_company_name;
-	$email            = isset( $data['give_email'] ) ? sanitize_email( $data['give_email'] ) : $old_user_data->user_email;
-	$password         = ! empty( $data['give_new_user_pass1'] ) ? $data['give_new_user_pass1'] : '';
-	$confirm_password = ! empty( $data['give_new_user_pass2'] ) ? $data['give_new_user_pass2'] : '';
+	$display_name     = isset($data['give_display_name']) ? sanitize_text_field($data['give_display_name']) : $old_user_data->display_name;
+	$first_name       = isset($data['give_first_name']) ? sanitize_text_field($data['give_first_name']) : $old_user_data->first_name;
+	$last_name        = isset($data['give_last_name']) ? sanitize_text_field($data['give_last_name']) : $old_user_data->last_name;
+	$company_name     = ! empty($data['give_company_name']) ? sanitize_text_field($data['give_company_name']) : $old_company_name;
+	$email            = isset($data['give_email']) ? sanitize_email($data['give_email']) : $old_user_data->user_email;
+	$password         = ! empty($data['give_new_user_pass1']) ? $data['give_new_user_pass1'] : '';
+	$confirm_password = ! empty($data['give_new_user_pass2']) ? $data['give_new_user_pass2'] : '';
 
 	$userdata = array(
 		'ID'           => $user_id,
@@ -456,60 +456,60 @@ 
 	 * @param int   $user_id  The ID of the user.
 	 * @param array $userdata User info, including ID, first name, last name, display name and email.
 	 */
-	do_action( 'give_pre_update_user_profile', $user_id, $userdata );
+	do_action('give_pre_update_user_profile', $user_id, $userdata);
 
 	// Make sure to validate first name of existing donors.
-	if ( empty( $first_name ) ) {
+	if (empty($first_name)) {
 		// Empty First Name.
-		give_set_error( 'empty_first_name', __( 'Please enter your first name.', 'give' ) );
+		give_set_error('empty_first_name', __('Please enter your first name.', 'give'));
 	}
 
 	// Make sure to validate passwords for existing Donors.
-	give_validate_user_password( $password, $confirm_password );
+	give_validate_user_password($password, $confirm_password);
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// Make sure email should not be empty.
-		give_set_error( 'email_empty', __( 'The email you entered is empty.', 'give' ) );
+		give_set_error('email_empty', __('The email you entered is empty.', 'give'));
 
-	} elseif ( ! is_email( $email ) ) {
+	} elseif ( ! is_email($email)) {
 		// Make sure email should be valid.
-		give_set_error( 'email_not_valid', __( 'The email you entered is not valid. Please use another', 'give' ) );
+		give_set_error('email_not_valid', __('The email you entered is not valid. Please use another', 'give'));
 
-	} elseif ( $email != $old_user_data->user_email ) {
+	} elseif ($email != $old_user_data->user_email) {
 		// Make sure the new email doesn't belong to another user.
-		if ( email_exists( $email ) ) {
-			give_set_error( 'user_email_exists', __( 'The email you entered belongs to another user. Please use another.', 'give' ) );
-		} elseif ( Give()->donors->get_donor_by( 'email', $email ) ) {
+		if (email_exists($email)) {
+			give_set_error('user_email_exists', __('The email you entered belongs to another user. Please use another.', 'give'));
+		} elseif (Give()->donors->get_donor_by('email', $email)) {
 			// Make sure the new email doesn't belong to another user.
-			give_set_error( 'donor_email_exists', __( 'The email you entered belongs to another donor. Please use another.', 'give' ) );
+			give_set_error('donor_email_exists', __('The email you entered belongs to another donor. Please use another.', 'give'));
 		}
 	}
 
 	// Check for errors.
 	$errors = give_get_errors();
 
-	if ( $errors ) {
+	if ($errors) {
 		// Send back to the profile editor if there are errors.
-		wp_redirect( $data['give_redirect'] );
+		wp_redirect($data['give_redirect']);
 		give_die();
 	}
 
 	// Update Donor First Name and Last Name.
-	Give()->donors->update( $donor->id, array(
-		'name' => trim( "{$first_name} {$last_name}" ),
-	) );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_first_name', $first_name );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_last_name', $last_name );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_company', $company_name );
+	Give()->donors->update($donor->id, array(
+		'name' => trim("{$first_name} {$last_name}"),
+	));
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_first_name', $first_name);
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_last_name', $last_name);
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_company', $company_name);
 
 	$current_user = wp_get_current_user();
 
 	// Compares new values with old values to detect change in values.
-	$email_update        = ( $email !== $current_user->user_email ) ? true : false;
-	$display_name_update = ( $display_name !== $current_user->display_name ) ? true : false;
-	$first_name_update   = ( $first_name !== $current_user->first_name ) ? true : false;
-	$last_name_update    = ( $last_name !== $current_user->last_name ) ? true : false;
-	$company_name_update = ( $company_name !== $old_company_name ) ? true : false;
+	$email_update        = ($email !== $current_user->user_email) ? true : false;
+	$display_name_update = ($display_name !== $current_user->display_name) ? true : false;
+	$first_name_update   = ($first_name !== $current_user->first_name) ? true : false;
+	$last_name_update    = ($last_name !== $current_user->last_name) ? true : false;
+	$company_name_update = ($company_name !== $old_company_name) ? true : false;
 	$update_code         = 0;
 
 	/**
@@ -517,35 +517,35 @@ 
 	 *
 	 * @var boolean
 	 */
-	$profile_update = ( $email_update || $display_name_update || $first_name_update || $last_name_update || $company_name_update );
+	$profile_update = ($email_update || $display_name_update || $first_name_update || $last_name_update || $company_name_update);
 
 	/**
 	 * True if password fields are filled.
 	 *
 	 * @var boolean
 	 */
-	$password_update = ( ! empty( $password ) && ! empty( $confirm_password ) );
+	$password_update = ( ! empty($password) && ! empty($confirm_password));
 
-	if ( $profile_update ) {
+	if ($profile_update) {
 
 		// If only profile fields are updated.
 		$update_code = '1';
 
-		if ( $password_update ) {
+		if ($password_update) {
 
 			// If profile fields AND password both are updated.
 			$update_code = '2';
 		}
-	} elseif ( $password_update ) {
+	} elseif ($password_update) {
 
 		// If only password is updated.
 		$update_code = '3';
 	}
 
 	// Update the user.
-	$updated = wp_update_user( $userdata );
+	$updated = wp_update_user($userdata);
 
-	if ( $updated ) {
+	if ($updated) {
 
 		/**
 		 * Fires after updating user profile.
@@ -555,7 +555,7 @@ 
 		 * @param int   $user_id  The ID of the user.
 		 * @param array $userdata User info, including ID, first name, last name, display name and email.
 		 */
-		do_action( 'give_user_profile_updated', $user_id, $userdata );
+		do_action('give_user_profile_updated', $user_id, $userdata);
 
 		$profile_edit_redirect_args = array(
 			'updated'     => 'true',
@@ -566,10 +566,10 @@ 
 		 * Update codes '2' and '3' indicate a password change.
 		 * If the password is changed, then logout and redirect to the same page.
 		 */
-		if ( '2' === $update_code || '3' === $update_code ) {
-			wp_logout( wp_redirect( add_query_arg( $profile_edit_redirect_args, $data['give_redirect'] ) ) );
+		if ('2' === $update_code || '3' === $update_code) {
+			wp_logout(wp_redirect(add_query_arg($profile_edit_redirect_args, $data['give_redirect'])));
 		} else {
-			wp_redirect( add_query_arg( $profile_edit_redirect_args, $data['give_redirect'] ) );
+			wp_redirect(add_query_arg($profile_edit_redirect_args, $data['give_redirect']));
 		}
 
 		give_die();
@@ -578,7 +578,7 @@ 
 	return false;
 }
 
-add_action( 'give_edit_user_profile', 'give_process_profile_editor_updates' );
+add_action('give_edit_user_profile', 'give_process_profile_editor_updates');
 
 /**
  * Give totals Shortcode.
@@ -591,24 +591,24 @@ 
  *
  * @return string
  */
-function give_totals_shortcode( $atts ) {
-	$total = get_option( 'give_earnings_total', false );
+function give_totals_shortcode($atts) {
+	$total = get_option('give_earnings_total', false);
 
-	$message = apply_filters( 'give_totals_message', __( 'Hey! We\'ve raised {total} of the {total_goal} we are trying to raise for this campaign!', 'give' ) );
+	$message = apply_filters('give_totals_message', __('Hey! We\'ve raised {total} of the {total_goal} we are trying to raise for this campaign!', 'give'));
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		'total_goal'   => 0, // integer
 		'ids'          => 0, // integer|array
 		'cats'         => 0, // integer|array
 		'tags'         => 0, // integer|array
 		'message'      => $message,
 		'link'         => '', // URL
-		'link_text'    => __( 'Donate Now', 'give' ), // string,
+		'link_text'    => __('Donate Now', 'give'), // string,
 		'progress_bar' => true, // boolean
-	), $atts, 'give_totals' );
+	), $atts, 'give_totals');
 
 	// Total Goal.
-	$total_goal = give_maybe_sanitize_amount( $atts['total_goal'] );
+	$total_goal = give_maybe_sanitize_amount($atts['total_goal']);
 
 	/**
 	 * Give Action fire before the shortcode is rendering is started.
@@ -617,14 +617,14 @@ 
 	 *
 	 * @param array $atts shortcode attribute.
 	 */
-	do_action( 'give_totals_goal_shortcode_before_render', $atts );
+	do_action('give_totals_goal_shortcode_before_render', $atts);
 
 	// Build query based on cat, tag and Form ids.
-	if ( ! empty( $atts['cats'] ) || ! empty( $atts['tags'] ) || ! empty( $atts['ids'] ) ) {
+	if ( ! empty($atts['cats']) || ! empty($atts['tags']) || ! empty($atts['ids'])) {
 
 		$form_ids = array();
-		if ( ! empty( $atts['ids'] ) ) {
-			$form_ids = array_filter( array_map( 'trim', explode( ',', $atts['ids'] ) ) );
+		if ( ! empty($atts['ids'])) {
+			$form_ids = array_filter(array_map('trim', explode(',', $atts['ids'])));
 		}
 
 		/**
@@ -638,23 +638,23 @@ 
 			'post_type'        => 'give_forms',
 			'post_status'      => 'publish',
 			'post__in'         => $form_ids,
-			'posts_per_page'   => - 1,
+			'posts_per_page'   => -1,
 			'fields'           => 'ids',
 			'tax_query'        => array(
 				'relation' => 'AND',
 			),
 		);
 
-		if ( ! empty( $atts['cats'] ) ) {
-			$cats                     = array_filter( array_map( 'trim', explode( ',', $atts['cats'] ) ) );
+		if ( ! empty($atts['cats'])) {
+			$cats                     = array_filter(array_map('trim', explode(',', $atts['cats'])));
 			$form_args['tax_query'][] = array(
 				'taxonomy' => 'give_forms_category',
 				'terms'    => $cats,
 			);
 		}
 
-		if ( ! empty( $atts['tags'] ) ) {
-			$tags                     = array_filter( array_map( 'trim', explode( ',', $atts['tags'] ) ) );
+		if ( ! empty($atts['tags'])) {
+			$tags                     = array_filter(array_map('trim', explode(',', $atts['tags'])));
 			$form_args['tax_query'][] = array(
 				'taxonomy' => 'give_forms_tag',
 				'terms'    => $tags,
@@ -670,15 +670,15 @@ 
 		 *
 		 * @return array $form_args WP query argument for Total Goal.
 		 */
-		$form_args = (array) apply_filters( 'give_totals_goal_shortcode_query_args', $form_args );
+		$form_args = (array) apply_filters('give_totals_goal_shortcode_query_args', $form_args);
 
-		$forms = new WP_Query( $form_args );
+		$forms = new WP_Query($form_args);
 
-		if ( isset( $forms->posts ) ) {
+		if (isset($forms->posts)) {
 			$total = 0;
-			foreach ( $forms->posts as $post ) {
-				$form_earning = give_get_meta( $post, '_give_form_earnings', true );
-				$form_earning = ! empty( $form_earning ) ? $form_earning : 0;
+			foreach ($forms->posts as $post) {
+				$form_earning = give_get_meta($post, '_give_form_earnings', true);
+				$form_earning = ! empty($form_earning) ? $form_earning : 0;
 
 				/**
 				 * Update Form earnings.
@@ -688,7 +688,7 @@ 
 				 * @param int $post Form ID.
 				 * @param string $form_earning Total earning of Form.
 				 */
-				$total += apply_filters( 'give_totals_form_earning', $form_earning, $post );
+				$total += apply_filters('give_totals_form_earning', $form_earning, $post);
 			}
 		}
 
@@ -696,23 +696,23 @@ 
 
 	// Append link with text.
 	$donate_link = '';
-	if ( ! empty( $atts['link'] ) ) {
-		$donate_link = sprintf( ' <a class="give-totals-text-link" href="%1$s">%2$s</a>', esc_url( $atts['link'] ), esc_html( $atts['link_text'] ) );
+	if ( ! empty($atts['link'])) {
+		$donate_link = sprintf(' <a class="give-totals-text-link" href="%1$s">%2$s</a>', esc_url($atts['link']), esc_html($atts['link_text']));
 	}
 
 	// Replace {total} in message.
-	$message = str_replace( '{total}', give_currency_filter(
-		give_format_amount( $total,
-			array( 'sanitize' => false )
+	$message = str_replace('{total}', give_currency_filter(
+		give_format_amount($total,
+			array('sanitize' => false)
 		)
-	), esc_html( $atts['message'] ) );
+	), esc_html($atts['message']));
 
 	// Replace {total_goal} in message.
-	$message = str_replace( '{total_goal}', give_currency_filter(
-		give_format_amount( $total_goal,
-			array( 'sanitize' => true )
+	$message = str_replace('{total_goal}', give_currency_filter(
+		give_format_amount($total_goal,
+			array('sanitize' => true)
 		)
-	), $message );
+	), $message);
 
 	/**
 	 * Update Give totals shortcode output.
@@ -722,19 +722,19 @@ 
 	 * @param string $message Shortcode Message.
 	 * @param array $atts ShortCode attributes.
 	 */
-	$message = apply_filters( 'give_totals_shortcode_message', $message, $atts );
+	$message = apply_filters('give_totals_shortcode_message', $message, $atts);
 
 	ob_start();
 	?>
 	<div class="give-totals-shortcode-wrap">
 		<?php
 		// Show Progress Bar if progress_bar set true.
-		$show_progress_bar = isset( $atts['progress_bar'] ) ? filter_var( $atts['progress_bar'], FILTER_VALIDATE_BOOLEAN ) : true;
-		if ( $show_progress_bar ) {
-			give_show_goal_totals_progress( $total, $total_goal );
+		$show_progress_bar = isset($atts['progress_bar']) ? filter_var($atts['progress_bar'], FILTER_VALIDATE_BOOLEAN) : true;
+		if ($show_progress_bar) {
+			give_show_goal_totals_progress($total, $total_goal);
 		}
 
-		echo sprintf( $message ) . $donate_link;
+		echo sprintf($message).$donate_link;
 		?>
 	</div>
 	<?php
@@ -749,7 +749,7 @@ 
 	 * @param array  $atts               shortcode attribute.
 	 * @param string $give_totals_output shortcode output.
 	 */
-	do_action( 'give_totals_goal_shortcode_after_render', $atts, $give_totals_output );
+	do_action('give_totals_goal_shortcode_after_render', $atts, $give_totals_output);
 
 	/**
 	 * Give Totals Shortcode output.
@@ -758,11 +758,11 @@ 
 	 *
 	 * @param string $give_totals_output
 	 */
-	return apply_filters( 'give_totals_shortcode_output', $give_totals_output );
+	return apply_filters('give_totals_shortcode_output', $give_totals_output);
 
 }
 
-add_shortcode( 'give_totals', 'give_totals_shortcode' );
+add_shortcode('give_totals', 'give_totals_shortcode');
 
 
 /**
@@ -793,11 +793,11 @@ 
  * }
  * @return string|bool The markup of the form grid or false.
  */
-function give_form_grid_shortcode( $atts ) {
+function give_form_grid_shortcode($atts) {
 
 	$give_settings = give_get_settings();
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		'forms_per_page'      => 12,
 		'paged'               => true,
 		'ids'                 => '',
@@ -814,11 +814,11 @@ 
 		'excerpt_length'      => 16,
 		'display_style'       => 'modal_reveal',
 		'status'              => '', // open or closed
-	), $atts );
+	), $atts);
 
 	// Validate integer attributes.
-	$atts['forms_per_page'] = intval( $atts['forms_per_page'] );
-	$atts['excerpt_length'] = intval( $atts['excerpt_length'] );
+	$atts['forms_per_page'] = intval($atts['forms_per_page']);
+	$atts['excerpt_length'] = intval($atts['excerpt_length']);
 
 	// Validate boolean attributes.
 	$boolean_attributes = array(
@@ -829,8 +829,8 @@ 
 		'show_featured_image',
 	);
 
-	foreach ( $boolean_attributes as $att ) {
-		$atts[ $att ] = filter_var( $atts[ $att ], FILTER_VALIDATE_BOOLEAN );
+	foreach ($boolean_attributes as $att) {
+		$atts[$att] = filter_var($atts[$att], FILTER_VALIDATE_BOOLEAN);
 	}
 
 	// Set default form query args.
@@ -844,9 +844,9 @@ 
 	);
 
 	// Filter results of form grid based on form status.
-	$form_closed_status = trim( $atts['status'] );
+	$form_closed_status = trim($atts['status']);
 
-	if ( ! empty( $form_closed_status ) ) {
+	if ( ! empty($form_closed_status)) {
 		$form_args['meta_query'] = array(
 			array(
 				'key'   => '_give_form_status',
@@ -856,25 +856,25 @@ 
 	}
 
 	// Maybe add pagination.
-	if ( true === $atts['paged'] ) {
-		$form_args['paged'] = get_query_var( 'paged' ) ? get_query_var( 'paged' ) : 1;
+	if (true === $atts['paged']) {
+		$form_args['paged'] = get_query_var('paged') ? get_query_var('paged') : 1;
 	}
 
 	// Maybe filter forms by IDs.
-	if ( ! empty( $atts['ids'] ) ) {
-		$form_args['post__in'] = array_filter( array_map( 'trim', explode( ',', $atts['ids'] ) ) );
+	if ( ! empty($atts['ids'])) {
+		$form_args['post__in'] = array_filter(array_map('trim', explode(',', $atts['ids'])));
 	}
 
 	// Convert comma-separated form IDs into array.
-	if ( ! empty( $atts['exclude'] ) ) {
-		$form_args['post__not_in'] = array_filter( array_map( function( $item ) {
-			return intval( trim( $item ) );
-		}, explode( ',', $atts['exclude'] ) ) );
+	if ( ! empty($atts['exclude'])) {
+		$form_args['post__not_in'] = array_filter(array_map(function($item) {
+			return intval(trim($item));
+		}, explode(',', $atts['exclude'])));
 	}
 
 	// Maybe filter by form category.
-	if ( ! empty( $atts['cats'] ) ) {
-		$cats                     = array_filter( array_map( 'trim', explode( ',', $atts['cats'] ) ) );
+	if ( ! empty($atts['cats'])) {
+		$cats                     = array_filter(array_map('trim', explode(',', $atts['cats'])));
 		$tax_query                = array(
 			'taxonomy' => 'give_forms_category',
 			'terms'    => $cats,
@@ -883,8 +883,8 @@ 
 	}
 
 	// Maybe filter by form tag.
-	if ( ! empty( $atts['tags'] ) ) {
-		$tags                     = array_filter( array_map( 'trim', explode( ',', $atts['tags'] ) ) );
+	if ( ! empty($atts['tags'])) {
+		$tags                     = array_filter(array_map('trim', explode(',', $atts['tags'])));
 		$tax_query                = array(
 			'taxonomy' => 'give_forms_tag',
 			'terms'    => $tags,
@@ -893,24 +893,24 @@ 
 	}
 
 	// Query to output donation forms.
-	$form_query = new WP_Query( $form_args );
+	$form_query = new WP_Query($form_args);
 
-	if ( $form_query->have_posts() ) {
+	if ($form_query->have_posts()) {
 		ob_start();
 
-		add_filter( 'add_give_goal_progress_class', 'add_give_goal_progress_class', 10, 1 );
-		add_filter( 'add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class', 10, 1 );
-		add_filter( 'give_form_wrap_classes', 'add_class_for_form_grid', 10, 3 );
-		add_action( 'give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10, 3 );
+		add_filter('add_give_goal_progress_class', 'add_give_goal_progress_class', 10, 1);
+		add_filter('add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class', 10, 1);
+		add_filter('give_form_wrap_classes', 'add_class_for_form_grid', 10, 3);
+		add_action('give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10, 3);
 
 		echo '<div class="give-wrap">';
-		echo '<div class="give-grid give-grid--' . esc_attr( $atts['columns'] ) . '">';
+		echo '<div class="give-grid give-grid--'.esc_attr($atts['columns']).'">';
 
-		while ( $form_query->have_posts() ) {
+		while ($form_query->have_posts()) {
 			$form_query->the_post();
 
 			// Give/templates/shortcode-form-grid.php.
-			give_get_template( 'shortcode-form-grid', array( $give_settings, $atts ) );
+			give_get_template('shortcode-form-grid', array($give_settings, $atts));
 
 		}
 
@@ -918,28 +918,28 @@ 
 
 		echo '</div><!-- .give-grid -->';
 
-		remove_filter( 'add_give_goal_progress_class', 'add_give_goal_progress_class' );
-		remove_filter( 'add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class' );
-		remove_filter( 'give_form_wrap_classes', 'add_class_for_form_grid', 10 );
-		remove_action( 'give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10 );
+		remove_filter('add_give_goal_progress_class', 'add_give_goal_progress_class');
+		remove_filter('add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class');
+		remove_filter('give_form_wrap_classes', 'add_class_for_form_grid', 10);
+		remove_action('give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10);
 
-		if ( false !== $atts['paged'] ) {
+		if (false !== $atts['paged']) {
 			$paginate_args = array(
-				'current'   => max( 1, get_query_var( 'paged' ) ),
+				'current'   => max(1, get_query_var('paged')),
 				'total'     => $form_query->max_num_pages,
 				'show_all'  => false,
 				'end_size'  => 1,
 				'mid_size'  => 2,
 				'prev_next' => true,
-				'prev_text' => __( '« Previous', 'give' ),
-				'next_text' => __( 'Next »', 'give' ),
+				'prev_text' => __('« Previous', 'give'),
+				'next_text' => __('Next »', 'give'),
 				'type'      => 'plain',
 				'add_args'  => false,
 			);
 
 			printf(
 				'<div class="give-page-numbers">%s</div>',
-				paginate_links( $paginate_args )
+				paginate_links($paginate_args)
 			);
 		}
 		echo '</div><!-- .give-wrap -->';
@@ -948,4 +948,4 @@ 
 	}
 }
 
-add_shortcode( 'give_form_grid', 'give_form_grid_shortcode' );
+add_shortcode('give_form_grid', 'give_form_grid_shortcode');

templates/shortcode-receipt.php

Spacing +57 added lines, -57 removed lines

@@ -6,29 +6,29 @@ 
 global $give_receipt_args, $payment;
 
 // Validation: Ensure $payment var is set.
-if ( empty( $payment ) ) {
-	$payment = ! empty( $give_receipt_args['id'] ) ? get_post( $give_receipt_args['id'] ) : 0;
+if (empty($payment)) {
+	$payment = ! empty($give_receipt_args['id']) ? get_post($give_receipt_args['id']) : 0;
 }
 
 // Double-Validation: Check for $payment global.
-if ( empty( $payment ) ) {
-	Give()->notices->print_frontend_notice( __( 'The specified receipt ID appears to be invalid.', 'give' ) );
+if (empty($payment)) {
+	Give()->notices->print_frontend_notice(__('The specified receipt ID appears to be invalid.', 'give'));
 
 	return;
 }
 
 $donation_id  = $payment->ID;
-$donation_number = Give()->seq_donation_number->get_serial_code( $payment->ID );
-$form_id      = give_get_payment_meta( $donation_id, '_give_payment_form_id', true );
-$donation     = give_get_donation_form_title( $donation_id );
-$user         = give_get_payment_meta_user_info( $donation_id );
-$email        = give_get_payment_user_email( $donation_id );
+$donation_number = Give()->seq_donation_number->get_serial_code($payment->ID);
+$form_id      = give_get_payment_meta($donation_id, '_give_payment_form_id', true);
+$donation     = give_get_donation_form_title($donation_id);
+$user         = give_get_payment_meta_user_info($donation_id);
+$email        = give_get_payment_user_email($donation_id);
 $status       = $payment->post_status;
-$status_label = give_get_payment_status( $payment, true );
-$company_name = give_get_payment_meta( $donation_id, '_give_donation_company', true );
+$status_label = give_get_payment_status($payment, true);
+$company_name = give_get_payment_meta($donation_id, '_give_donation_company', true);
 
 // Update donor name, if title prefix is set.
-$full_name    = give_get_donor_name_with_title_prefixes( $user['title'], "{$user['first_name']} {$user['last_name']}" );
+$full_name    = give_get_donor_name_with_title_prefixes($user['title'], "{$user['first_name']} {$user['last_name']}");
 
 /**
  * Generate Donation Receipt Arguments.
@@ -39,7 +39,7 @@ 
  * @since 1.8.8
  */
 $give_receipt_args['donation_receipt']['donor'] = array(
-	'name'    => __( 'Donor', 'give' ),
+	'name'    => __('Donor', 'give'),
 	'value'   => $full_name,
 	'display' => $give_receipt_args['donor'],
 );
@@ -54,51 +54,51 @@ 
  * @return bool show/hide company name in donation receipt page.
  */
 $give_receipt_args['donation_receipt']['company_name'] = array(
-	'name'    => __( 'Company Name', 'give' ),
-	'value'   => esc_attr( $company_name ),
+	'name'    => __('Company Name', 'give'),
+	'value'   => esc_attr($company_name),
 	// Do not show company field if empty
-	'display' => empty( $company_name ) ? false : $give_receipt_args['company_name'],
+	'display' => empty($company_name) ? false : $give_receipt_args['company_name'],
 );
 
 $give_receipt_args['donation_receipt']['date'] = array(
-	'name'    => __( 'Date', 'give' ),
-	'value'   => date_i18n( give_date_format(), strtotime( give_get_payment_completed_date( $donation_id ) ) ),
+	'name'    => __('Date', 'give'),
+	'value'   => date_i18n(give_date_format(), strtotime(give_get_payment_completed_date($donation_id))),
 	'display' => $give_receipt_args['date'],
 );
 
 $give_receipt_args['donation_receipt']['total_donation'] = array(
-	'name'    => __( 'Total Donation', 'give' ),
-	'value'   => give_donation_amount( $donation_id, array( 'currency' => true, 'amount' => true, 'type' => 'receipt' ) ),
+	'name'    => __('Total Donation', 'give'),
+	'value'   => give_donation_amount($donation_id, array('currency' => true, 'amount' => true, 'type' => 'receipt')),
 	'display' => $give_receipt_args['price'],
 );
 
 $give_receipt_args['donation_receipt']['donation'] = array(
-	'name'    => __( 'Donation', 'give' ),
+	'name'    => __('Donation', 'give'),
 	'value'   => $donation,
 	'display' => true,
 );
 
 $give_receipt_args['donation_receipt']['donation_status'] = array(
-	'name'    => __( 'Donation Status', 'give' ),
-	'value'   => esc_attr( $status_label ),
+	'name'    => __('Donation Status', 'give'),
+	'value'   => esc_attr($status_label),
 	'display' => $give_receipt_args['payment_status'],
 );
 
 $give_receipt_args['donation_receipt']['donation_id'] = array(
-	'name'    => __( 'Donation ID', 'give' ),
+	'name'    => __('Donation ID', 'give'),
 	'value'   => $donation_number,
 	'display' => $give_receipt_args['payment_id'],
 );
 
 $give_receipt_args['donation_receipt']['payment_key'] = array(
-	'name'    => __( 'Payment Key', 'give' ),
-	'value'   => get_post_meta( $donation_id, '_give_payment_purchase_key', true ),
+	'name'    => __('Payment Key', 'give'),
+	'value'   => get_post_meta($donation_id, '_give_payment_purchase_key', true),
 	'display' => $give_receipt_args['payment_key'],
 );
 
 $give_receipt_args['donation_receipt']['payment_method'] = array(
-	'name'    => __( 'Payment Method', 'give' ),
-	'value'   => give_get_gateway_checkout_label( give_get_payment_gateway( $donation_id ) ),
+	'name'    => __('Payment Method', 'give'),
+	'value'   => give_get_gateway_checkout_label(give_get_payment_gateway($donation_id)),
 	'display' => $give_receipt_args['payment_method'],
 );
 
@@ -113,10 +113,10 @@ 
  *
  * @since 1.8.8
  */
-$give_receipt_args['donation_receipt'] = apply_filters( 'give_donation_receipt_args', $give_receipt_args['donation_receipt'], $donation_id, $form_id );
+$give_receipt_args['donation_receipt'] = apply_filters('give_donation_receipt_args', $give_receipt_args['donation_receipt'], $donation_id, $form_id);
 
 // When the donation were made through offline donation, We won't show receipt and payment status though.
-if ( 'offline' === give_get_payment_gateway( $payment->ID ) && 'pending' === $status ) {
+if ('offline' === give_get_payment_gateway($payment->ID) && 'pending' === $status) {
 
 	/**
 	 * Before the offline donation receipt content starts.
@@ -126,13 +126,13 @@ 
 	 * @param Give_Payment $payment           Donation payment object.
 	 * @param array        $give_receipt_args Receipt Arguments.
 	 */
-	do_action( 'give_receipt_before_offline_payment', $payment, $give_receipt_args );
+	do_action('give_receipt_before_offline_payment', $payment, $give_receipt_args);
 	?>
-	<h2><?php echo apply_filters( 'give_receipt_offline_payment_heading', __( 'Your Donation is Almost Complete!', 'give' ) ); ?></h2>
-	<div id="give_donation_receipt" class="<?php echo esc_attr( apply_filters( 'give_receipt_offline_payment_classes', 'give_receipt_offline_payment' ) ); ?>">
+	<h2><?php echo apply_filters('give_receipt_offline_payment_heading', __('Your Donation is Almost Complete!', 'give')); ?></h2>
+	<div id="give_donation_receipt" class="<?php echo esc_attr(apply_filters('give_receipt_offline_payment_classes', 'give_receipt_offline_payment')); ?>">
 		<?php
 		// Instruction for offline donation.
-		$offline_instruction = give_get_offline_payment_instruction( $form_id, true );
+		$offline_instruction = give_get_offline_payment_instruction($form_id, true);
 
 		/**
 		 * Instruction for the offline donation.
@@ -143,7 +143,7 @@ 
 		 * @param Give_Payment $payment             Payment object.
 		 * @param integer      $form_id             Donation form id.
 		 */
-		echo apply_filters( 'give_receipt_offline_payment_instruction', $offline_instruction, $payment, $form_id );
+		echo apply_filters('give_receipt_offline_payment_instruction', $offline_instruction, $payment, $form_id);
 		?>
 	</div>
 	<?php
@@ -155,52 +155,52 @@ 
 	 * @param Give_Payment $payment           Donation payment object.
 	 * @param array        $give_receipt_args Receipt Arguments.
 	 */
-	do_action( 'give_receipt_after_offline_payment', $payment, $give_receipt_args );
+	do_action('give_receipt_after_offline_payment', $payment, $give_receipt_args);
 
 	return;
 }
 
 // Show payment status notice based on shortcode attribute.
-if ( filter_var( $give_receipt_args['status_notice'], FILTER_VALIDATE_BOOLEAN ) ) {
+if (filter_var($give_receipt_args['status_notice'], FILTER_VALIDATE_BOOLEAN)) {
 	$notice_message = '';
 	$notice_type    = 'warning';
 
-	switch ( $status ) {
+	switch ($status) {
 		case 'publish':
-			$notice_message = __( 'Payment Complete: Thank you for your donation.', 'give' );
+			$notice_message = __('Payment Complete: Thank you for your donation.', 'give');
 			$notice_type    = 'success';
 			break;
 		case 'pending':
-			$notice_message = __( 'Payment Pending: Your donation is currently processing.', 'give' );
+			$notice_message = __('Payment Pending: Your donation is currently processing.', 'give');
 			$notice_type    = 'warning';
 			break;
 		case 'refunded':
-			$notice_message = __( 'Payment Refunded: Your donation has been refunded.', 'give' );
+			$notice_message = __('Payment Refunded: Your donation has been refunded.', 'give');
 			$notice_type    = 'warning';
 			break;
 		case 'preapproval':
-			$notice_message = __( 'Payment Preapproved: Thank you for your donation.', 'give' );
+			$notice_message = __('Payment Preapproved: Thank you for your donation.', 'give');
 			$notice_type    = 'warning';
 			break;
 		case 'failed':
-			$notice_message = __( 'Payment Failed: Please contact the site owner for assistance.', 'give' );
+			$notice_message = __('Payment Failed: Please contact the site owner for assistance.', 'give');
 			$notice_type    = 'error';
 			break;
 		case 'cancelled':
-			$notice_message = __( 'Payment Cancelled: Your donation has been cancelled.', 'give' );
+			$notice_message = __('Payment Cancelled: Your donation has been cancelled.', 'give');
 			$notice_type    = 'error';
 			break;
 		case 'abandoned':
-			$notice_message = __( 'Payment Abandoned: This donation has not been completed.', 'give' );
+			$notice_message = __('Payment Abandoned: This donation has not been completed.', 'give');
 			$notice_type    = 'error';
 			break;
 		case 'revoked':
-			$notice_message = __( 'Payment Revoked: Please contact the site owner for assistance.', 'give' );
+			$notice_message = __('Payment Revoked: Please contact the site owner for assistance.', 'give');
 			$notice_type    = 'error';
 			break;
 	}
 
-	if ( ! empty( $notice_message ) ) {
+	if ( ! empty($notice_message)) {
 		/**
 		 * Filters payment status notice for receipts.
 		 *
@@ -215,7 +215,7 @@ 
 		 * @param string $status Payment status.
 		 * @param int $donation_id Donation ID.
 		 */
-		echo apply_filters( 'give_receipt_status_notice', Give()->notices->print_frontend_notice( $notice_message, false, $notice_type ), $id, $status, $donation_id );
+		echo apply_filters('give_receipt_status_notice', Give()->notices->print_frontend_notice($notice_message, false, $notice_type), $id, $status, $donation_id);
 	}
 }// End if().
 
@@ -229,7 +229,7 @@ 
  * @param object $payment           The payment object.
  * @param array  $give_receipt_args Receipt_argument.
  */
-do_action( 'give_payment_receipt_before_table', $payment, $give_receipt_args );
+do_action('give_payment_receipt_before_table', $payment, $give_receipt_args);
 ?>
 
 <table id="give_donation_receipt" class="give-table">
@@ -245,11 +245,11 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_header_before', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_header_before', $payment, $give_receipt_args);
 	?>
 	<tr>
 		<th scope="colgroup" colspan="2">
-			<span class="give-receipt-thead-text"><?php esc_html_e( 'Donation Receipt', 'give' ) ?></span>
+			<span class="give-receipt-thead-text"><?php esc_html_e('Donation Receipt', 'give') ?></span>
 		</th>
 	</tr>
 	<?php
@@ -263,7 +263,7 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_header_after', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_header_after', $payment, $give_receipt_args);
 	?>
 	</thead>
 
@@ -279,11 +279,11 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_before', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_before', $payment, $give_receipt_args);
 	?>
 
-	<?php foreach ( $give_receipt_args['donation_receipt'] as $receipt_item ) { ?>
-		<?php if ( filter_var( $receipt_item['display'], FILTER_VALIDATE_BOOLEAN ) ) : ?>
+	<?php foreach ($give_receipt_args['donation_receipt'] as $receipt_item) { ?>
+		<?php if (filter_var($receipt_item['display'], FILTER_VALIDATE_BOOLEAN)) : ?>
 			<tr>
 				<td scope="row"><strong><?php echo $receipt_item['name']; ?></strong></td>
 				<td><?php echo $receipt_item['value']; ?></td>
@@ -302,7 +302,7 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_after', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_after', $payment, $give_receipt_args);
 	?>
 	</tbody>
 </table>
@@ -318,5 +318,5 @@ 
  * @param object $payment           The payment object.
  * @param array  $give_receipt_args Receipt_argument.
  */
-do_action( 'give_payment_receipt_after_table', $payment, $give_receipt_args );
+do_action('give_payment_receipt_after_table', $payment, $give_receipt_args);
 ?>