WordImpress / Give

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

Doc Comments +4 added lines, -1 removed lines

@@ -216,7 +216,7 @@ 
 	 * Return the calculated completion percentage.
 	 *
 	 * @since 1.8.12
-	 * @return int
+	 * @return double
 	 */
 	public function get_percentage_complete() {
 		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
@@ -353,6 +353,9 @@ 
 		return true;
 	}
 
+	/**
+	 * @param integer $page
+	 */
 	public function get_delete_ids( $donation_ids, $page ) {
 		$index            = $page --;
 		$count            = count( $donation_ids );

Spacing +114 added lines, -114 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -116,34 +116,34 @@ 
 		$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);
 		}
 	}
 
@@ -154,17 +154,17 @@ 
 	 * @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();
 		// Incresed 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,
@@ -172,40 +172,40 @@ 
 			'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) get_post_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) get_post_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()->donors->delete( $donor->id );
+							$count      = (int) count($donor->payment_ids);
+							if (1 === $count) {
+								Give()->donors->delete($donor->id);
 							} 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;
 				}
@@ -220,23 +220,23 @@ 
 		$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 curretn page virable 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();
 	}
@@ -248,34 +248,34 @@ 
 	 * @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);
 	}
 
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to delete Import transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to delete Import transactions.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', give_get_total_earnings( true ) );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true));
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = __( 'Imported donor and transactions successfully deleted.', 'give' );
+			$this->message = __('Imported donor and transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -293,12 +293,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;
 
@@ -309,110 +309,110 @@ 
 		$step = (int) $this->get_step();
 
 		// get teh donor ids.
-		$donor_ids = $this->get_option( $this->donor_key );
+		$donor_ids = $this->get_option($this->donor_key);
 
 		// In step to we delete all the donation in loop.
-		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[] = get_post_meta( $item, '_give_payment_form_id', true );
+			foreach ($donation_ids as $item) {
+				$form_ids[] = get_post_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);
 		}
 
 
 		// Here we delete all the donor
-		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
 					 */
-					if ( 'on' === (string) $_REQUEST['delete-import-donors'] ) {
-						wp_delete_user( $donor_ids[ $page ] );
+					if ('on' === (string) $_REQUEST['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);
 				}
 			}
 
-			$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;
@@ -424,24 +424,24 @@ 
 		return true;
 	}
 
-	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];
 	}
 
 	/**
@@ -453,8 +453,8 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	public function get_option( $key, $defalut_value = false ) {
-		return get_option( $key, $defalut_value );
+	public function get_option($key, $defalut_value = false) {
+		return get_option($key, $defalut_value);
 	}
 
 	/**
@@ -467,8 +467,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);
 	}
 
 	/**
@@ -480,8 +480,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -492,12 +492,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;
@@ -508,6 +508,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-donor.php

Doc Comments +1 added lines, -1 removed lines

@@ -918,7 +918,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 +253 added lines, -260 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -149,32 +149,32 @@ 
 	 * @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 ) )
+			|| (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);
 
 	}
 
@@ -190,19 +190,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();
@@ -216,20 +216,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' );
-		} else{
-			foreach ( $donor_vars as $donor_var => $value ) {
+			Give_Cache::set_group($donor->id, get_object_vars($this), 'give-donors');
+		} else {
+			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;
 		}
 
@@ -262,18 +262,18 @@ 
 			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];
 			}
 		}
 	}
@@ -287,16 +287,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));
 
 		}
 
@@ -312,9 +312,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;
 		}
 
@@ -322,15 +322,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'])));
 		}
 
 		/**
@@ -340,18 +340,18 @@ 
 		 *
 		 * @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;
 		}
@@ -364,7 +364,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;
 
@@ -380,13 +380,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.
@@ -396,15 +396,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;
 		}
@@ -418,7 +418,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;
 	}
@@ -436,27 +436,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)));
 
 		}
 
@@ -468,20 +468,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer 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();
@@ -497,7 +497,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;
 	}
@@ -515,33 +515,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)));
 
 		}
 
@@ -553,20 +553,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();
@@ -582,7 +582,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;
 
@@ -598,10 +598,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;
 		}
 
@@ -615,9 +615,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;
 		}
 
@@ -630,7 +630,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;
 	}
@@ -645,16 +645,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;
 		}
 
@@ -666,9 +666,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;
 		}
 
@@ -681,7 +681,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;
 	}
@@ -696,9 +696,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.
@@ -708,9 +708,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;
 		}
 
@@ -723,7 +723,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;
 	}
@@ -738,11 +738,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;
 		}
 
@@ -754,9 +754,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;
 		}
 
@@ -769,7 +769,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;
 	}
@@ -788,7 +788,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)
@@ -798,15 +798,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;
@@ -823,15 +823,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;
 
@@ -848,9 +848,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);
 
 	}
 
@@ -863,7 +863,7 @@ 
 	 *
 	 * @return string|float
 	 */
-	public function get_total_donation_amount( $args = array() ) {
+	public function get_total_donation_amount($args = array()) {
 
 		/**
 		 * Filter total donation amount.
@@ -874,7 +874,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);
 	}
 
 	/**
@@ -887,22 +887,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.
@@ -912,11 +912,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();
 		}
 
@@ -929,7 +929,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;
@@ -946,7 +946,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;
 
@@ -963,8 +963,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);
 	}
 
 	/**
@@ -979,8 +979,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);
 	}
 
 	/**
@@ -995,8 +995,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);
 	}
 
 	/**
@@ -1010,8 +1010,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);
 	}
 
 	/**
@@ -1024,51 +1024,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;
 
 			}
@@ -1088,33 +1088,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;
@@ -1130,16 +1130,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;
 	}
@@ -1156,16 +1156,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;
 		}
@@ -1173,21 +1173,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;
 	}
@@ -1202,17 +1202,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;
 			}
@@ -1239,30 +1239,27 @@ 
 	 *
 	 * @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;
-
-			$address_type  = $is_multi_address ?
-				array_shift( 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;
 		}
 
 		// Bailout: do not save duplicate orders
-		if( $this->is_address_exist( $address_type, $address ) ) {
+		if ($this->is_address_exist($address_type, $address)) {
 			return false;
 		}
 
@@ -1284,8 +1281,8 @@ 
 		$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(
@@ -1302,10 +1299,10 @@ 
 					)
 				);
 
-				if( ! empty( $multi_address_id ) ) {
-					$multi_address_id = absint( substr( strrchr( $multi_address_id, '_' ), 1 ) );
+				if ( ! empty($multi_address_id)) {
+					$multi_address_id = absint(substr(strrchr($multi_address_id, '_'), 1));
 					$multi_address_id++;
-				} else{
+				} else {
 					$multi_address_id = 0;
 				}
 			}
@@ -1314,9 +1311,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();
@@ -1335,23 +1332,21 @@ 
 	 *
 	 * @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}";
 		}
 
@@ -1388,23 +1383,21 @@ 
 	 *
 	 * @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}";
 		}
 
@@ -1425,12 +1418,12 @@ 
 		);
 		
 		// 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;
 		}
 
@@ -1449,39 +1442,39 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_address_exist( $current_address_type, $current_address ) {
+	public function is_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;
 					}
 				}
@@ -1502,10 +1495,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);
 	}
 
 	/**
@@ -1515,22 +1508,22 @@ 
 	 * @since   2.0
 	 * @return  object
 	 */
-	public function split_donor_name( $id ) {
-		$first_name = $last_name  = '';
-		$donor      = new Give_Donor( $id );
+	public function split_donor_name($id) {
+		$first_name = $last_name = '';
+		$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);
 	}
 
 	/**
@@ -1540,9 +1533,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;
@@ -1555,14 +1548,14 @@ 
 	 * @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 : '';
 	}
 }

includes/class-give-session.php

Spacing +76 added lines, -76 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -74,57 +74,57 @@ 
 	public function __construct() {
 
 		$this->use_php_sessions = $this->use_php_sessions();
-		$this->exp_option       = give_get_option( 'session_lifetime' );
+		$this->exp_option       = give_get_option('session_lifetime');
 
 		// PHP Sessions.
-		if ( $this->use_php_sessions ) {
+		if ($this->use_php_sessions) {
 
-			if ( is_multisite() ) {
+			if (is_multisite()) {
 
-				$this->prefix = '_' . get_current_blog_id();
+				$this->prefix = '_'.get_current_blog_id();
 
 			}
 
-			add_action( 'init', array( $this, 'maybe_start_session' ), - 2 );
+			add_action('init', array($this, 'maybe_start_session'), - 2);
 
 		} else {
 
-			if ( ! $this->should_start_session() ) {
+			if ( ! $this->should_start_session()) {
 				return;
 			}
 
 			// Use WP_Session.
-			if ( ! defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'give_wp_session' );
+			if ( ! defined('WP_SESSION_COOKIE')) {
+				define('WP_SESSION_COOKIE', 'give_wp_session');
 			}
 
-			if ( ! class_exists( 'Recursive_ArrayAccess' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-recursive-arrayaccess.php';
+			if ( ! class_exists('Recursive_ArrayAccess')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-recursive-arrayaccess.php';
 			}
 
 			// Include utilities class
-			if ( ! class_exists( 'WP_Session_Utils' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session-utils.php';
+			if ( ! class_exists('WP_Session_Utils')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session-utils.php';
 			}
-			if ( ! class_exists( 'WP_Session' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/wp-session.php';
+			if ( ! class_exists('WP_Session')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session.php';
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/wp-session.php';
 			}
 
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+			add_filter('wp_session_expiration_variant', array($this, 'set_expiration_variant_time'), 99999);
+			add_filter('wp_session_expiration', array($this, 'set_expiration_time'), 99999);
 
 		}
 
 		// Init Session.
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), 9999 );
+		if (empty($this->session) && ! $this->use_php_sessions) {
+			add_action('plugins_loaded', array($this, 'init'), 9999);
 		} else {
-			add_action( 'init', array( $this, 'init' ), - 1 );
+			add_action('init', array($this, 'init'), - 1);
 		}
 
 		// Set cookie on Donation Completion page.
-		add_action( 'give_pre_process_donation', array( $this, 'set_session_cookies' ) );
+		add_action('give_pre_process_donation', array($this, 'set_session_cookies'));
 
 	}
 
@@ -140,8 +140,8 @@ 
 	 */
 	public function init() {
 
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION[ 'give' . $this->prefix ] ) && is_array( $_SESSION[ 'give' . $this->prefix ] ) ? $_SESSION[ 'give' . $this->prefix ] : array();
+		if ($this->use_php_sessions) {
+			$this->session = isset($_SESSION['give'.$this->prefix]) && is_array($_SESSION['give'.$this->prefix]) ? $_SESSION['give'.$this->prefix] : array();
 		} else {
 			$this->session = WP_Session::get_instance();
 		}
@@ -176,37 +176,37 @@ 
 	 *
 	 * @return string|array      Session variable.
 	 */
-	public function get( $key ) {
-		$key    = sanitize_key( $key );
+	public function get($key) {
+		$key    = sanitize_key($key);
 		$return = false;
 
-		if ( isset( $this->session[ $key ] ) && ! empty( $this->session[ $key ] ) ) {
+		if (isset($this->session[$key]) && ! empty($this->session[$key])) {
 
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $this->session[ $key ], $matches );
-			if ( ! empty( $matches ) ) {
-				$this->set( $key, null );
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $this->session[$key], $matches);
+			if ( ! empty($matches)) {
+				$this->set($key, null);
 
 				return false;
 			}
 
-			if ( is_numeric( $this->session[ $key ] ) ) {
-				$return = $this->session[ $key ];
+			if (is_numeric($this->session[$key])) {
+				$return = $this->session[$key];
 			} else {
 
-				$maybe_json = json_decode( $this->session[ $key ] );
+				$maybe_json = json_decode($this->session[$key]);
 
 				// Since json_last_error is PHP 5.3+, we have to rely on a `null` value for failing to parse JSON.
-				if ( is_null( $maybe_json ) ) {
-					$is_serialized = is_serialized( $this->session[ $key ] );
-					if ( $is_serialized ) {
-						$value = @unserialize( $this->session[ $key ] );
-						$this->set( $key, (array) $value );
+				if (is_null($maybe_json)) {
+					$is_serialized = is_serialized($this->session[$key]);
+					if ($is_serialized) {
+						$value = @unserialize($this->session[$key]);
+						$this->set($key, (array) $value);
 						$return = $value;
 					} else {
-						$return = $this->session[ $key ];
+						$return = $this->session[$key];
 					}
 				} else {
-					$return = json_decode( $this->session[ $key ], true );
+					$return = json_decode($this->session[$key], true);
 				}
 
 			}
@@ -229,21 +229,21 @@ 
 	 *
 	 * @return string        Session variable.
 	 */
-	public function set( $key, $value ) {
+	public function set($key, $value) {
 
-		$key = sanitize_key( $key );
+		$key = sanitize_key($key);
 
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = wp_json_encode( $value );
+		if (is_array($value)) {
+			$this->session[$key] = wp_json_encode($value);
 		} else {
-			$this->session[ $key ] = esc_attr( $value );
+			$this->session[$key] = esc_attr($value);
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION[ 'give' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['give'.$this->prefix] = $this->session;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -258,10 +258,10 @@ 
 	 * @hook
 	 */
 	public function set_session_cookies() {
-		if ( ! headers_sent() ) {
-			$lifetime = current_time( 'timestamp' ) + $this->set_expiration_time();
-			@setcookie( session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
-			@setcookie( session_name() . '_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
+		if ( ! headers_sent()) {
+			$lifetime = current_time('timestamp') + $this->set_expiration_time();
+			@setcookie(session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
+			@setcookie(session_name().'_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -277,7 +277,7 @@ 
 	 */
 	public function set_expiration_variant_time() {
 
-		return ( ! empty( $this->exp_option ) ? ( intval( $this->exp_option ) - 3600 ) : 30 * 60 * 23 );
+		return ( ! empty($this->exp_option) ? (intval($this->exp_option) - 3600) : 30 * 60 * 23);
 	}
 
 	/**
@@ -292,7 +292,7 @@ 
 	 */
 	public function set_expiration_time() {
 
-		return ( ! empty( $this->exp_option ) ? intval( $this->exp_option ) : 30 * 60 * 24 );
+		return ( ! empty($this->exp_option) ? intval($this->exp_option) : 30 * 60 * 24);
 	}
 
 	/**
@@ -310,21 +310,21 @@ 
 		$ret = false;
 
 		// If the database variable is already set, no need to run auto detection.
-		$give_use_php_sessions = (bool) get_option( 'give_use_php_sessions' );
+		$give_use_php_sessions = (bool) get_option('give_use_php_sessions');
 
-		if ( ! $give_use_php_sessions ) {
+		if ( ! $give_use_php_sessions) {
 
 			// Attempt to detect if the server supports PHP sessions.
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
+			if (function_exists('session_start') && ! ini_get('safe_mode')) {
 
-				$this->set( 'give_use_php_sessions', 1 );
+				$this->set('give_use_php_sessions', 1);
 
-				if ( $this->get( 'give_use_php_sessions' ) ) {
+				if ($this->get('give_use_php_sessions')) {
 
 					$ret = true;
 
 					// Set the database option.
-					update_option( 'give_use_php_sessions', true );
+					update_option('give_use_php_sessions', true);
 
 				}
 			}
@@ -334,13 +334,13 @@ 
 		}
 
 		// Enable or disable PHP Sessions based on the GIVE_USE_PHP_SESSIONS constant.
-		if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && GIVE_USE_PHP_SESSIONS ) {
+		if (defined('GIVE_USE_PHP_SESSIONS') && GIVE_USE_PHP_SESSIONS) {
 			$ret = true;
-		} elseif ( defined( 'GIVE_USE_PHP_SESSIONS' ) && ! GIVE_USE_PHP_SESSIONS ) {
+		} elseif (defined('GIVE_USE_PHP_SESSIONS') && ! GIVE_USE_PHP_SESSIONS) {
 			$ret = false;
 		}
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -357,9 +357,9 @@ 
 
 		$start_session = true;
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {
+		if ( ! empty($_SERVER['REQUEST_URI'])) {
 
-			$blacklist = apply_filters( 'give_session_start_uri_blacklist', array(
+			$blacklist = apply_filters('give_session_start_uri_blacklist', array(
 				'feed',
 				'feed',
 				'feed/rss',
@@ -367,21 +367,21 @@ 
 				'feed/rdf',
 				'feed/atom',
 				'comments/feed/',
-			) );
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' );
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist ) ) {
+			));
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/');
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist)) {
 				$start_session = false;
 			}
-			if ( false !== strpos( $uri, 'feed=' ) ) {
+			if (false !== strpos($uri, 'feed=')) {
 				$start_session = false;
 			}
-			if ( is_admin() ) {
+			if (is_admin()) {
 				$start_session = false;
 			}
 		}
 
-		return apply_filters( 'give_start_session', $start_session );
+		return apply_filters('give_start_session', $start_session);
 	}
 
 	/**
@@ -397,11 +397,11 @@ 
 	 */
 	public function maybe_start_session() {
 
-		if ( ! $this->should_start_session() ) {
+		if ( ! $this->should_start_session()) {
 			return;
 		}
 
-		if ( ! session_id() && ! headers_sent() ) {
+		if ( ! session_id() && ! headers_sent()) {
 			session_start();
 		}
 
@@ -420,9 +420,9 @@ 
 
 		$expiration = false;
 
-		if ( session_id() && isset( $_COOKIE[ session_name() . '_expiration' ] ) ) {
+		if (session_id() && isset($_COOKIE[session_name().'_expiration'])) {
 
-			$expiration = date( 'D, d M Y h:i:s', intval( $_COOKIE[ session_name() . '_expiration' ] ) );
+			$expiration = date('D, d M Y h:i:s', intval($_COOKIE[session_name().'_expiration']));
 
 		}
 

includes/admin/class-blank-slate.php

Spacing +55 added lines, -55 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.8.13
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -75,63 +75,63 @@ 
 	 */
 	public function init() {
 		// Bail early if screen cannot be detected.
-		if ( empty( $this->screen ) ) {
+		if (empty($this->screen)) {
 			return null;
 		}
 
 		$content = array();
 
 		// Define content and hook into the appropriate action.
-		switch ( $this->screen ) {
+		switch ($this->screen) {
 			// Forms screen.
 			case 'edit-give_forms':
-				$this->form = $this->post_exists( 'give_forms' );
+				$this->form = $this->post_exists('give_forms');
 
-				if ( $this->form ) {
+				if ($this->form) {
 					// Form exists. Bail out.
 					return false;
 				} else {
 					// No forms exist.
-					$content = $this->get_content( 'no_forms' );
+					$content = $this->get_content('no_forms');
 				}
 
-				add_action( 'manage_posts_extra_tablenav', array( $this, 'render' ) );
+				add_action('manage_posts_extra_tablenav', array($this, 'render'));
 				break;
 			// Donations screen.
 			case 'give_forms_page_give-payment-history':
-				$this->form     = $this->post_exists( 'give_forms' );
-				$this->donation = $this->post_exists( 'give_payment' );
+				$this->form     = $this->post_exists('give_forms');
+				$this->donation = $this->post_exists('give_payment');
 
-				if ( $this->donation ) {
+				if ($this->donation) {
 					// Donation exists. Bail out.
 					return false;
-				} elseif ( ! $this->form ) {
+				} elseif ( ! $this->form) {
 					// No forms and no donations exist.
-					$content = $this->get_content( 'no_donations_or_forms' );
+					$content = $this->get_content('no_donations_or_forms');
 				} else {
 					// No donations exist but a form does exist.
-					$content = $this->get_content( 'no_donations' );
+					$content = $this->get_content('no_donations');
 				}
 
-				add_action( 'give_payments_page_bottom', array( $this, 'render' ) );
+				add_action('give_payments_page_bottom', array($this, 'render'));
 				break;
 			// Donors screen.
 			case 'give_forms_page_give-donors':
-				$this->form  = $this->post_exists( 'give_forms' );
+				$this->form  = $this->post_exists('give_forms');
 				$this->donor = $this->donor_exists();
 
-				if ( $this->donor ) {
+				if ($this->donor) {
 					// Donor exists. Bail out.
 					return false;
-				} elseif ( ! $this->form ) {
+				} elseif ( ! $this->form) {
 					// No forms and no donors exist.
-					$content = $this->get_content( 'no_donors_or_forms' );
+					$content = $this->get_content('no_donors_or_forms');
 				} else {
 					// No donors exist but a form does exist.
-					$content = $this->get_content( 'no_donors' );
+					$content = $this->get_content('no_donors');
 				}
 
-				add_action( 'give_donors_table_bottom', array( $this, 'render' ) );
+				add_action('give_donors_table_bottom', array($this, 'render'));
 				break;
 			default:
 				return null;
@@ -140,7 +140,7 @@ 
 		$this->content = $content;
 
 		// Hide non-essential UI elements.
-		add_action( 'admin_head', array( $this, 'hide_ui' ) );
+		add_action('admin_head', array($this, 'hide_ui'));
 	}
 
 	/**
@@ -150,9 +150,9 @@ 
 	 *
 	 * @param string $which The location of the list table hook: 'top' or 'bottom'.
 	 */
-	public function render( $which = 'bottom') {
+	public function render($which = 'bottom') {
 		// Bail out to prevent content from rendering twice.
-		if ( 'top' === $which ) {
+		if ('top' === $which) {
 			return null;
 		}
 
@@ -177,9 +177,9 @@ 
 		 *
 		 * @param string $screen The current screen ID.
 		 */
-		$content = apply_filters( 'give_blank_slate_content', $this->content, $screen );
+		$content = apply_filters('give_blank_slate_content', $this->content, $screen);
 
-		$template_path = GIVE_PLUGIN_DIR . 'includes/admin/views/blank-slate.php';
+		$template_path = GIVE_PLUGIN_DIR.'includes/admin/views/blank-slate.php';
 
 		include $template_path;
 	}
@@ -201,17 +201,17 @@ 
 	 * @param string $post_type Post type used in the query.
 	 * @return bool True if post exists, otherwise false.
 	 */
-	private function post_exists( $post_type ) {
+	private function post_exists($post_type) {
 		// Attempt to get a single post of the post type.
-		$query = new WP_Query( array(
+		$query = new WP_Query(array(
 			'post_type'              => $post_type,
 			'posts_per_page'         => 1,
 			'no_found_rows'          => false,
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
 			'fields'                 => 'ids',
-			'post_status'            => array( 'any', 'trash' ),
-		) );
+			'post_status'            => array('any', 'trash'),
+		));
 
 		return $query->have_posts();
 	}
@@ -224,9 +224,9 @@ 
 	 * @return bool True if donor exists, otherwise false.
 	 */
 	private function donor_exists() {
-		$donors = Give()->donors->get_donors( array( 'number' => 1 ) );
+		$donors = Give()->donors->get_donors(array('number' => 1));
 
-		return ! empty( $donors );
+		return ! empty($donors);
 	}
 
 	/**
@@ -237,17 +237,17 @@ 
 	 * @param string $context The key used to determine which content is returned.
 	 * @return array Blank slate content.
 	 */
-	private function get_content( $context ) {
+	private function get_content($context) {
 		// Define default content.
 		$defaults = array(
-			'image_url' => GIVE_PLUGIN_URL . 'assets/images/svg/give-icon-full-circle.svg',
-			'image_alt' => __( 'Give Icon', 'give' ),
-			'heading'   => __( 'No donation forms  found.', 'give' ),
-			'message'   => __( 'The first step towards accepting online donations is to create a form.', 'give' ),
-			'cta_text'  => __( 'Create Donation Form', 'give' ),
-			'cta_link'  => admin_url( 'post-new.php?post_type=give_forms' ),
+			'image_url' => GIVE_PLUGIN_URL.'assets/images/svg/give-icon-full-circle.svg',
+			'image_alt' => __('Give Icon', 'give'),
+			'heading'   => __('No donation forms  found.', 'give'),
+			'message'   => __('The first step towards accepting online donations is to create a form.', 'give'),
+			'cta_text'  => __('Create Donation Form', 'give'),
+			'cta_link'  => admin_url('post-new.php?post_type=give_forms'),
 			'help'      => sprintf(
-				__( 'Need help? Get started with %sGive 101%s.', 'wbpr' ),
+				__('Need help? Get started with %sGive 101%s.', 'wbpr'),
 				'<a href="http://docs.givewp.com/give101/" target="_blank">',
 				'</a>'
 			),
@@ -256,40 +256,40 @@ 
 		// Define contextual content.
 		$content = array(
 			'no_donations_or_forms' => array(
-				'heading' => __( 'No donations found.', 'give' ),
-				'message' => __( 'Your donation history will appear here, but first, you need a donation form!', 'give' ),
+				'heading' => __('No donations found.', 'give'),
+				'message' => __('Your donation history will appear here, but first, you need a donation form!', 'give'),
 			),
 			'no_donations'          => array(
-				'heading'  => __( 'No donations found.', 'give' ),
-				'message'  => __( 'When your first donation arrives, a record of the donation will appear here.', 'give' ),
-				'cta_text' => __( 'View All Forms', 'give' ),
-				'cta_link' => admin_url( 'edit.php?post_type=give_forms' ),
+				'heading'  => __('No donations found.', 'give'),
+				'message'  => __('When your first donation arrives, a record of the donation will appear here.', 'give'),
+				'cta_text' => __('View All Forms', 'give'),
+				'cta_link' => admin_url('edit.php?post_type=give_forms'),
 				'help'     => sprintf(
-					__( 'Need help? Learn more about %sDonations%s.', 'wbpr' ),
+					__('Need help? Learn more about %sDonations%s.', 'wbpr'),
 					'<a href="http://docs.givewp.com/core-donations/">',
 					'</a>'
 				),
 			),
 			'no_donors_or_forms'    => array(
-				'heading' => __( 'No donors  found.', 'give' ),
-				'message' => __( 'Your donor history will appear here, but first, you need a donation form!', 'give' ),
+				'heading' => __('No donors  found.', 'give'),
+				'message' => __('Your donor history will appear here, but first, you need a donation form!', 'give'),
 			),
 			'no_donors'             => array(
-				'heading'  => __( 'No donors found.', 'give' ),
-				'message'  => __( 'When your first donation arrives, the donor will appear here.', 'give' ),
-				'cta_text' => __( 'View All Forms', 'give' ),
-				'cta_link' => admin_url( 'edit.php?post_type=give_forms' ),
+				'heading'  => __('No donors found.', 'give'),
+				'message'  => __('When your first donation arrives, the donor will appear here.', 'give'),
+				'cta_text' => __('View All Forms', 'give'),
+				'cta_link' => admin_url('edit.php?post_type=give_forms'),
 				'help'     => sprintf(
-					__( 'Need help? Learn more about %sDonors%s.', 'wbpr' ),
+					__('Need help? Learn more about %sDonors%s.', 'wbpr'),
 					'<a href="http://docs.givewp.com/core-donors/">',
 					'</a>'
 				),
 			),
 		);
 
-		if ( isset( $content[ $context ] ) ) {
+		if (isset($content[$context])) {
 			// Merge contextual content with defaults.
-			return wp_parse_args( $content[ $context ], $defaults );
+			return wp_parse_args($content[$context], $defaults);
 		} else {
 			// Return defaults if context is undefined.
 			return $defaults;

includes/admin/views/html-admin-settings.php

Braces +4 added lines, -1 removed lines

@@ -95,5 +95,8 @@
 		<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+<?php else {
+	: echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+}
+?>
 <?php endif; ?>
\ No newline at end of file

Spacing +22 added lines, -22 removed lines

@@ -2,12 +2,12 @@ 
 /**
  * Admin View: Settings
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Bailout: Do not output anything if setting tab is not defined.
-if ( ! empty( $tabs ) && array_key_exists( give_get_current_setting_tab(), $tabs ) ) :
+if ( ! empty($tabs) && array_key_exists(give_get_current_setting_tab(), $tabs)) :
 	/**
 	 * Filter the form action.
 	 *
@@ -17,7 +17,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_method = apply_filters( self::$setting_filter_prefix . '_form_method_tab_' . $current_tab, 'post' );
+	$form_method = apply_filters(self::$setting_filter_prefix.'_form_method_tab_'.$current_tab, 'post');
 
 	/**
 	 * Filter the main form tab.
@@ -29,14 +29,14 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_open_tag  = apply_filters( self::$setting_filter_prefix . '_open_form', '<form method="' . $form_method . '" id="give-mainform" action="" enctype="multipart/form-data">' );
-	$form_close_tag = apply_filters( self::$setting_filter_prefix . '_close_form', '</form>' );
+	$form_open_tag  = apply_filters(self::$setting_filter_prefix.'_open_form', '<form method="'.$form_method.'" id="give-mainform" action="" enctype="multipart/form-data">');
+	$form_close_tag = apply_filters(self::$setting_filter_prefix.'_close_form', '</form>');
 
-	$wrapper_class = implode( ' ',
+	$wrapper_class = implode(' ',
 		array(
-			self::$setting_filter_prefix . '-setting-page',
-			self::$setting_filter_prefix . '-' . give_get_current_setting_section() . '-section',
-			self::$setting_filter_prefix . '-' . give_get_current_setting_tab() . '-tab',
+			self::$setting_filter_prefix.'-setting-page',
+			self::$setting_filter_prefix.'-'.give_get_current_setting_section().'-section',
+			self::$setting_filter_prefix.'-'.give_get_current_setting_tab().'-tab',
 		)
 	);
 	?>
@@ -46,8 +46,8 @@ 
 
 		/* @var Give_Settings_Page $current_setting_obj */
 		if (
-			! empty( $current_setting_obj ) &&
-			method_exists( $current_setting_obj, 'get_heading_html' )
+			! empty($current_setting_obj) &&
+			method_exists($current_setting_obj, 'get_heading_html')
 		) {
 			echo $current_setting_obj->get_heading_html();
 		} else {
@@ -55,7 +55,7 @@ 
 			// Backward compatibility.
 			echo sprintf(
 				'<h1 class="wp-heading-inline">%s</h1><hr class="wp-header-end">',
-				esc_html( $tabs[ $current_tab ] )
+				esc_html($tabs[$current_tab])
 			);
 		}
 
@@ -63,8 +63,8 @@ 
 		?>
 		<div class="nav-tab-wrapper give-nav-tab-wrapper">
 			<?php
-			foreach ( $tabs as $name => $label ) {
-				echo '<a href="' . admin_url( "edit.php?post_type=give_forms&page=" . self::$setting_filter_prefix . "&tab={$name}" ) . '" class="nav-tab ' . ( $current_tab === $name ? 'nav-tab-active' : 'give-mobile-hidden' ) . '">' . $label . '</a>';
+			foreach ($tabs as $name => $label) {
+				echo '<a href="'.admin_url("edit.php?post_type=give_forms&page=".self::$setting_filter_prefix."&tab={$name}").'" class="nav-tab '.($current_tab === $name ? 'nav-tab-active' : 'give-mobile-hidden').'">'.$label.'</a>';
 			}
 
 			/**
@@ -76,10 +76,10 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_tabs' );
+			do_action(self::$setting_filter_prefix.'_tabs');
 			?>
 			<div class="give-sub-nav-tab-wrapper">
-				<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e( 'View remaining setting tabs', 'give' ); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span>
+				<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e('View remaining setting tabs', 'give'); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span>
 				</a>
 				<nav class="give-sub-nav-tab give-hidden"></nav>
 			</div>
@@ -95,7 +95,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_sections_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_sections_{$current_tab}_page");
 
 		/**
 		 * Trigger Action.
@@ -106,15 +106,15 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_settings_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_settings_{$current_tab}_page");
 
-		if ( empty( $GLOBALS['give_hide_save_button'] ) ) : ?>
+		if (empty($GLOBALS['give_hide_save_button'])) : ?>
 			<div class="give-submit-wrap">
-				<?php wp_nonce_field( 'give-save-settings', '_give-save-settings' ); ?>
-				<input name="save" class="button-primary give-save-button" type="submit" value="<?php _e( 'Save changes', 'give' ); ?>"/>
+				<?php wp_nonce_field('give-save-settings', '_give-save-settings'); ?>
+				<input name="save" class="button-primary give-save-button" type="submit" value="<?php _e('Save changes', 'give'); ?>"/>
 			</div>
 		<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+<?php else : echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>'; ?>
 <?php endif; ?>
\ No newline at end of file

includes/admin/views/blank-slate.php

Spacing +10 added lines, -10 removed lines

@@ -17,23 +17,23 @@ 
 ?>
 
 <div class="give-blank-slate">
-	<?php if ( ! empty( $content['image_url'] ) ) : ?>
-		<img class="give-blank-slate__image" src="<?php echo esc_url( $content['image_url'] ); ?>" alt="<?php echo esc_attr( $content['image_alt'] ); ?>">
+	<?php if ( ! empty($content['image_url'])) : ?>
+		<img class="give-blank-slate__image" src="<?php echo esc_url($content['image_url']); ?>" alt="<?php echo esc_attr($content['image_alt']); ?>">
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['heading'] ) ) : ?>
-		<h2 class="give-blank-slate__heading"><?php esc_html_e( $content['heading'] ); ?></h2>
+	<?php if ( ! empty($content['heading'])) : ?>
+		<h2 class="give-blank-slate__heading"><?php esc_html_e($content['heading']); ?></h2>
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['message'] ) ) : ?>
-		<p class="give-blank-slate__message"><?php esc_html_e( $content['message'] ); ?></p>
+	<?php if ( ! empty($content['message'])) : ?>
+		<p class="give-blank-slate__message"><?php esc_html_e($content['message']); ?></p>
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['cta_text']) && ! empty( $content['cta_link'] ) ) : ?>
-		<a class="give-blank-slate__cta button button-primary" href="<?php echo esc_url( $content['cta_link'] ); ?>"><?php esc_html_e( $content['cta_text'] ); ?></a>
+	<?php if ( ! empty($content['cta_text']) && ! empty($content['cta_link'])) : ?>
+		<a class="give-blank-slate__cta button button-primary" href="<?php echo esc_url($content['cta_link']); ?>"><?php esc_html_e($content['cta_text']); ?></a>
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['help'] ) ) : ?>
+	<?php if ( ! empty($content['help'])) : ?>
 		<p class="give-blank-slate__help">
 			<?php
 			$allowed_html = array(
@@ -47,7 +47,7 @@ 
 				'code'   => array(),
 			);
 
-			echo wp_kses( $content['help'], $allowed_html );
+			echo wp_kses($content['help'], $allowed_html);
 			?>
 		</p>
 	<?php endif; ?>

includes/admin/tools/data/tools-actions.php

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,10 +20,10 @@ 
  * @since  1.5
  */
 function give_register_batch_recount_export_classes() {
-	add_action( 'give_batch_export_class_include', 'give_include_batch_export_class', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_batch_export_class', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_batch_recount_export_classes', 10 );
+add_action('give_register_batch_exporter', 'give_register_batch_recount_export_classes', 10);
 
 
 /**
@@ -35,39 +35,39 @@ 
  *
  * @return void
  */
-function give_include_batch_export_class( $class ) {
-	switch ( $class ) {
+function give_include_batch_export_class($class) {
+	switch ($class) {
 
 		case 'Give_Tools_Delete_Donors':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-test-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-test-donors.php';
 			break;
 
 		case 'Give_Tools_Import_Donors':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-import-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-import-donors.php';
 			break;
 
 		case 'Give_Tools_Delete_Test_Transactions':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-test-transactions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-test-transactions.php';
 			break;
 
 		case 'Give_Tools_Recount_Donor_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-donor-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-donor-stats.php';
 			break;
 
 		case 'Give_Tools_Reset_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-reset-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-reset-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_All_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-all-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-all-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_Form_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-form-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-form-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_Income':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-income.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-income.php';
 			break;
 	}
 }

includes/payments/class-give-payment.php

Indentation +1 added lines, -1 removed lines

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

Doc Comments +2 added lines, -2 removed lines

@@ -391,7 +391,7 @@ 
 	 *
 	 * @param  int|bool $payment_id A given payment
 	 *
-	 * @return mixed void|false
+	 * @return false|null void|false
 	 */
 	public function __construct( $payment_id = false ) {
 
@@ -1115,7 +1115,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 +292 added lines, -293 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -395,13 +395,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);
 	}
 
 	/**
@@ -414,11 +414,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 {
 
@@ -440,18 +440,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;
 		}
 	}
@@ -466,9 +466,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;
 		}
@@ -484,20 +484,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;
 		}
 
@@ -511,17 +511,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();
@@ -537,7 +537,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();
@@ -566,10 +566,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;
 			}
 		}
@@ -584,7 +584,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;
 	}
@@ -603,11 +603,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);
 	}
 
 	/**
@@ -622,24 +622,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;
@@ -668,19 +668,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;
@@ -692,42 +692,42 @@ 
 			 *
 			 * @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_first_name', $this->first_name);
+			$donor->update_meta('_give_donor_last_name', $this->last_name);
 
 			$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.
@@ -735,14 +735,14 @@ 
 			 * @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));
 			}
 
 			$this->new = true;
@@ -765,11 +765,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;
@@ -777,42 +777,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 );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price);
 
 										$total_increase += $price;
 									}
@@ -820,10 +820,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'] );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -834,27 +834,27 @@ 
 						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 'user_id':
@@ -862,58 +862,58 @@ 
 					// 	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 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
@@ -923,11 +923,11 @@ 
 							'edit_date' => true,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'completed_date':
-						$this->update_meta( '_give_completed_date', $this->completed_date );
+						$this->update_meta('_give_completed_date', $this->completed_date);
 						break;
 
 					case 'parent_payment':
@@ -936,7 +936,7 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
@@ -947,40 +947,40 @@ 
 						 *
 						 * @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);
 
-				} 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);
 
 				}
 			}
 
-			$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));
 
 			$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;
@@ -998,12 +998,12 @@ 
 	 *
 	 * @return bool           True when successful, false otherwise
 	 */
-	public function add_donation( $form_id = 0, $args = array(), $options = array() ) {
+	public function add_donation($form_id = 0, $args = array(), $options = array()) {
 
-		$donation = new Give_Donate_Form( $form_id );
+		$donation = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a give donation form.
-		if ( ! $donation || $donation->post_type !== 'give_forms' ) {
+		if ( ! $donation || $donation->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -1013,59 +1013,59 @@ 
 			'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'] ) )
+					if ((isset($args['price_id']) && isset($price['_give_id']['level_id']))
 					     && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$donation_amount = $price['_give_amount'];
 					}
 				}
 				// 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,
@@ -1073,7 +1073,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1090,7 +1090,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(
@@ -1098,12 +1098,12 @@ 
 			'price'    => false,
 			'price_id' => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$form = new Give_Donate_Form( $form_id );
+		$form = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a valid give donation form.
-		if ( ! $form || $form->post_type !== 'give_forms' ) {
+		if ( ! $form || $form->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -1116,7 +1116,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1132,13 +1132,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);
 	}
 
 	/**
@@ -1151,8 +1151,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();
@@ -1168,11 +1168,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;
 		}
 
@@ -1201,24 +1201,24 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated.
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		// standardize the 'complete(d)' status.
-		if ( $status == 'completed' || $status == 'complete' ) {
+		if ($status == 'completed' || $status == 'complete') {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes.
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
-		if ( $do_change ) {
+		if ($do_change) {
 
 			/**
 			 * Fires before changing payment status.
@@ -1229,21 +1229,21 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' ),
+				'edit_date'   => current_time('mysql'),
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions.
-			switch ( $status ) {
+			switch ($status) {
 				case 'refunded':
 					$this->process_refund();
 					break;
@@ -1270,7 +1270,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().
 
@@ -1305,24 +1305,24 @@ 
 	 *
 	 * @return mixed             The value from the post meta
 	 */
-	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' ) ) {
-			add_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4 );
+	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')) {
+			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();
 			}
 		}
@@ -1332,7 +1332,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);
 	}
 
 	/**
@@ -1347,8 +1347,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;
 		}
 
@@ -1358,9 +1358,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);
 	}
 
 	/**
@@ -1375,14 +1375,14 @@ 
 		$process_refund = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'refunded' != $this->status ) {
+		if ('publish' != $this->old_status || 'refunded' != $this->status) {
 			$process_refund = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_refund = apply_filters( 'give_should_process_refund', $process_refund, $this );
+		$process_refund = apply_filters('give_should_process_refund', $process_refund, $this);
 
-		if ( false === $process_refund ) {
+		if (false === $process_refund) {
 			return;
 		}
 
@@ -1393,13 +1393,13 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_pre_refund_payment', $this );
+		do_action('give_pre_refund_payment', $this);
 
-		$decrease_earnings       = apply_filters( 'give_decrease_store_earnings_on_refund', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_refund', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_refund', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_store_earnings_on_refund', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_refund', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_refund', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1412,7 +1412,7 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1439,26 +1439,26 @@ 
 		$process_pending = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'pending' != $this->status ) {
+		if ('publish' != $this->old_status || 'pending' != $this->status) {
 			$process_pending = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_pending = apply_filters( 'give_should_process_pending', $process_pending, $this );
+		$process_pending = apply_filters('give_should_process_pending', $process_pending, $this);
 
-		if ( false === $process_pending ) {
+		if (false === $process_pending) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_pending', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_pending', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_pending', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_pending', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_pending', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_pending', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1476,26 +1476,26 @@ 
 		$process_cancelled = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
+		if ('publish' != $this->old_status || 'cancelled' != $this->status) {
 			$process_cancelled = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
+		$process_cancelled = apply_filters('give_should_process_cancelled', $process_cancelled, $this);
 
-		if ( false === $process_cancelled ) {
+		if (false === $process_cancelled) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_cancelled', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_cancelled', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_cancelled', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_cancelled', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_cancelled', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_cancelled', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1511,26 +1511,26 @@ 
 		$process_revoked = true;
 
 		// If the payment was not in publish, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
+		if ('publish' != $this->old_status || 'revoked' != $this->status) {
 			$process_revoked = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
+		$process_revoked = apply_filters('give_should_process_revoked', $process_revoked, $this);
 
-		if ( false === $process_revoked ) {
+		if (false === $process_revoked) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_revoked', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_revoked', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_revoked', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_revoked', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_revoked', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_revoked', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1548,25 +1548,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();
 			}
 		}
@@ -1583,7 +1583,7 @@ 
 	 */
 	private function delete_sales_logs() {
 		// Remove related sale log entries.
-		Give()->logs->delete_logs( $this->ID );
+		Give()->logs->delete_logs($this->ID);
 	}
 
 	/**
@@ -1604,13 +1604,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;
 	}
@@ -1624,7 +1624,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1636,7 +1636,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');
 	}
 
 	/**
@@ -1648,9 +1648,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));
 	}
 
 	/**
@@ -1677,17 +1677,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
 			);
 
@@ -1703,7 +1702,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;
 	}
@@ -1717,11 +1716,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;
@@ -1737,7 +1736,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;
 	}
@@ -1752,7 +1751,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;
 	}
@@ -1769,8 +1768,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;
@@ -1787,10 +1786,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;
@@ -1810,22 +1809,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(),
@@ -1835,25 +1834,25 @@ 
 			}
 		} 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 '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;
 					}
 				}
@@ -1873,12 +1872,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;
 	}
@@ -1893,7 +1892,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;
 	}
@@ -1908,7 +1907,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;
 	}
@@ -1922,7 +1921,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;
 	}
@@ -1936,7 +1935,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;
 	}
@@ -1952,11 +1951,11 @@ 
 	private function setup_payment_number() {
 		$number = $this->ID;
 
-		if ( give_get_option( 'enable_sequential' ) ) {
+		if (give_get_option('enable_sequential')) {
 
-			$number = $this->get_meta( '_give_payment_number', true );
+			$number = $this->get_meta('_give_payment_number', true);
 
-			if ( ! $number ) {
+			if ( ! $number) {
 
 				$number = $this->ID;
 
@@ -1974,7 +1973,7 @@ 
 	 * @return array The payment object as an array.
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -1987,7 +1986,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -1999,7 +1998,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);
 	}
 
 	/**
@@ -2011,7 +2010,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);
 	}
 
 	/**
@@ -2023,7 +2022,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);
 	}
 
 	/**
@@ -2035,7 +2034,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);
 	}
 
 	/**
@@ -2047,7 +2046,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);
 	}
 
 	/**
@@ -2059,7 +2058,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);
 	}
 
 	/**
@@ -2071,7 +2070,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);
 	}
 
 	/**
@@ -2083,7 +2082,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);
 	}
 
 	/**
@@ -2095,7 +2094,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);
 	}
 
 	/**
@@ -2107,7 +2106,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);
 	}
 
 	/**
@@ -2119,7 +2118,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);
 	}
 
 	/**
@@ -2131,7 +2130,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);
 	}
 
 	/**
@@ -2143,7 +2142,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);
 	}
 
 	/**
@@ -2155,6 +2154,6 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 }

includes/template-functions.php

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,7 +21,7 @@ 
  * @return string
  */
 function give_get_templates_dir() {
-	return GIVE_PLUGIN_DIR . 'templates';
+	return GIVE_PLUGIN_DIR.'templates';
 }
 
 /**
@@ -31,7 +31,7 @@ 
  * @return string
  */
 function give_get_templates_url() {
-	return GIVE_PLUGIN_URL . 'templates';
+	return GIVE_PLUGIN_URL.'templates';
 }
 
 /**
@@ -44,24 +44,24 @@ 
  * @param string $template_path Template file path. Default is empty.
  * @param string $default_path  Default path. Default is empty.
  */
-function give_get_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
-	if ( ! empty( $args ) && is_array( $args ) ) {
-		extract( $args );
+function give_get_template($template_name, $args = array(), $template_path = '', $default_path = '') {
+	if ( ! empty($args) && is_array($args)) {
+		extract($args);
 	}
 
-	$template_names = array( $template_name . '.php' );
+	$template_names = array($template_name.'.php');
 
-	$located = give_locate_template( $template_names, $template_path, $default_path );
+	$located = give_locate_template($template_names, $template_path, $default_path);
 
-	if ( ! file_exists( $located ) ) {
+	if ( ! file_exists($located)) {
 		/* translators: %s: the template */
-		Give()->notices->print_frontend_notice( sprintf( __( 'The %s template was not found.', 'give' ), $located ), true );
+		Give()->notices->print_frontend_notice(sprintf(__('The %s template was not found.', 'give'), $located), true);
 
 		return;
 	}
 
 	// Allow 3rd party plugin filter template file from their plugin.
-	$located = apply_filters( 'give_get_template', $located, $template_name, $args, $template_path, $default_path );
+	$located = apply_filters('give_get_template', $located, $template_name, $args, $template_path, $default_path);
 
 	/**
 	 * Fires in give template, before the file is included.
@@ -75,9 +75,9 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_before_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_before_template_part', $template_name, $template_path, $located, $args);
 
-	include( $located );
+	include($located);
 
 	/**
 	 * Fires in give template, after the file is included.
@@ -91,7 +91,7 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_after_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_after_template_part', $template_name, $template_path, $located, $args);
 }
 
 /**
@@ -107,7 +107,7 @@ 
  *
  * @return string
  */
-function give_get_template_part( $slug, $name = null, $load = true ) {
+function give_get_template_part($slug, $name = null, $load = true) {
 
 	/**
 	 * Fires in give template part, before the template part is retrieved.
@@ -119,20 +119,20 @@ 
 	 * @param string $slug Template part file slug {slug}.php.
 	 * @param string $name Template part file name {slug}-{name}.php.
 	 */
-	do_action( "get_template_part_{$slug}", $slug, $name );
+	do_action("get_template_part_{$slug}", $slug, $name);
 
 	// Setup possible parts
 	$templates = array();
-	if ( isset( $name ) ) {
-		$templates[] = $slug . '-' . $name . '.php';
+	if (isset($name)) {
+		$templates[] = $slug.'-'.$name.'.php';
 	}
-	$templates[] = $slug . '.php';
+	$templates[] = $slug.'.php';
 
 	// Allow template parts to be filtered
-	$templates = apply_filters( 'give_get_template_part', $templates, $slug, $name );
+	$templates = apply_filters('give_get_template_part', $templates, $slug, $name);
 
 	// Return the part that is found
-	return give_locate_template( $templates, $load, false );
+	return give_locate_template($templates, $load, false);
 }
 
 /**
@@ -153,37 +153,37 @@ 
  *
  * @return string The template filename if one is located.
  */
-function give_locate_template( $template_names, $load = false, $require_once = true ) {
+function give_locate_template($template_names, $load = false, $require_once = true) {
 	// No file found yet
 	$located = false;
 
 	// Try to find a template file
-	foreach ( (array) $template_names as $template_name ) {
+	foreach ((array) $template_names as $template_name) {
 
 		// Continue if template is empty
-		if ( empty( $template_name ) ) {
+		if (empty($template_name)) {
 			continue;
 		}
 
 		// Trim off any slashes from the template name
-		$template_name = ltrim( $template_name, '/' );
+		$template_name = ltrim($template_name, '/');
 
 		// try locating this template file by looping through the template paths
-		foreach ( give_get_theme_template_paths() as $template_path ) {
+		foreach (give_get_theme_template_paths() as $template_path) {
 
-			if ( file_exists( $template_path . $template_name ) ) {
-				$located = $template_path . $template_name;
+			if (file_exists($template_path.$template_name)) {
+				$located = $template_path.$template_name;
 				break;
 			}
 		}
 
-		if ( $located ) {
+		if ($located) {
 			break;
 		}
 	}
 
-	if ( ( true == $load ) && ! empty( $located ) ) {
-		load_template( $located, $require_once );
+	if ((true == $load) && ! empty($located)) {
+		load_template($located, $require_once);
 	}
 
 	return $located;
@@ -200,17 +200,17 @@ 
 	$template_dir = give_get_theme_template_dir_name();
 
 	$file_paths = array(
-		1   => trailingslashit( get_stylesheet_directory() ) . $template_dir,
-		10  => trailingslashit( get_template_directory() ) . $template_dir,
+		1   => trailingslashit(get_stylesheet_directory()).$template_dir,
+		10  => trailingslashit(get_template_directory()).$template_dir,
 		100 => give_get_templates_dir(),
 	);
 
-	$file_paths = apply_filters( 'give_template_paths', $file_paths );
+	$file_paths = apply_filters('give_template_paths', $file_paths);
 
 	// sort the file paths based on priority
-	ksort( $file_paths, SORT_NUMERIC );
+	ksort($file_paths, SORT_NUMERIC);
 
-	return array_map( 'trailingslashit', $file_paths );
+	return array_map('trailingslashit', $file_paths);
 }
 
 /**
@@ -222,7 +222,7 @@ 
  * @return string
  */
 function give_get_theme_template_dir_name() {
-	return trailingslashit( apply_filters( 'give_templates_dir', 'give' ) );
+	return trailingslashit(apply_filters('give_templates_dir', 'give'));
 }
 
 /**
@@ -232,10 +232,10 @@ 
  * @return void
  */
 function give_version_in_header() {
-	echo '<meta name="generator" content="Give v' . GIVE_VERSION . '" />' . "\n";
+	echo '<meta name="generator" content="Give v'.GIVE_VERSION.'" />'."\n";
 }
 
-add_action( 'wp_head', 'give_version_in_header' );
+add_action('wp_head', 'give_version_in_header');
 
 /**
  * Determines if we're currently on the Donations History page.
@@ -245,9 +245,9 @@ 
  */
 function give_is_donation_history_page() {
 
-	$ret = is_page( give_get_option( 'history_page' ) );
+	$ret = is_page(give_get_option('history_page'));
 
-	return apply_filters( 'give_is_donation_history_page', $ret );
+	return apply_filters('give_is_donation_history_page', $ret);
 }
 
 /**
@@ -259,25 +259,25 @@ 
  *
  * @return array Modified array of classes
  */
-function give_add_body_classes( $class ) {
+function give_add_body_classes($class) {
 	$classes = (array) $class;
 
-	if ( give_is_success_page() ) {
+	if (give_is_success_page()) {
 		$classes[] = 'give-success';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_failed_transaction_page() ) {
+	if (give_is_failed_transaction_page()) {
 		$classes[] = 'give-failed-transaction';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_donation_history_page() ) {
+	if (give_is_donation_history_page()) {
 		$classes[] = 'give-donation-history';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		$classes[] = 'give-test-mode';
 		$classes[] = 'give-page';
 	}
@@ -286,7 +286,7 @@ 
 	/* @var WP_Theme $current_theme */
 	$current_theme = wp_get_theme();
 
-	switch ( $current_theme->get_template() ) {
+	switch ($current_theme->get_template()) {
 
 		case 'Divi':
 			$classes[] = 'give-divi';
@@ -303,10 +303,10 @@ 
 
 	}
 
-	return array_unique( $classes );
+	return array_unique($classes);
 }
 
-add_filter( 'body_class', 'give_add_body_classes' );
+add_filter('body_class', 'give_add_body_classes');
 
 
 /**
@@ -322,22 +322,22 @@ 
  *
  * @return array
  */
-function give_add_post_class( $classes, $class = '', $post_id = '' ) {
-	if ( ! $post_id || 'give_forms' !== get_post_type( $post_id ) ) {
+function give_add_post_class($classes, $class = '', $post_id = '') {
+	if ( ! $post_id || 'give_forms' !== get_post_type($post_id)) {
 		return $classes;
 	}
 
 	//@TODO: Add classes for custom taxonomy and form configurations (multi vs single donations, etc).
 
-	if ( false !== ( $key = array_search( 'hentry', $classes ) ) ) {
-		unset( $classes[ $key ] );
+	if (false !== ($key = array_search('hentry', $classes))) {
+		unset($classes[$key]);
 	}
 
 	return $classes;
 }
 
 
-add_filter( 'post_class', 'give_add_post_class', 20, 3 );
+add_filter('post_class', 'give_add_post_class', 20, 3);
 
 /**
  * Get the placeholder image URL for forms etc
@@ -347,74 +347,74 @@ 
  */
 function give_get_placeholder_img_src() {
 
-	$placeholder_url = '//placehold.it/600x600&text=' . urlencode( esc_attr__( 'Give Placeholder Image', 'give' ) );
+	$placeholder_url = '//placehold.it/600x600&text='.urlencode(esc_attr__('Give Placeholder Image', 'give'));
 
-	return apply_filters( 'give_placeholder_img_src', $placeholder_url );
+	return apply_filters('give_placeholder_img_src', $placeholder_url);
 }
 
 
 /**
  * Global
  */
-if ( ! function_exists( 'give_output_content_wrapper' ) ) {
+if ( ! function_exists('give_output_content_wrapper')) {
 
 	/**
 	 * Output the start of the page wrapper.
 	 */
 	function give_output_content_wrapper() {
-		give_get_template_part( 'global/wrapper-start' );
+		give_get_template_part('global/wrapper-start');
 	}
 }
-if ( ! function_exists( 'give_output_content_wrapper_end' ) ) {
+if ( ! function_exists('give_output_content_wrapper_end')) {
 
 	/**
 	 * Output the end of the page wrapper.
 	 */
 	function give_output_content_wrapper_end() {
-		give_get_template_part( 'global/wrapper-end' );
+		give_get_template_part('global/wrapper-end');
 	}
 }
 
 /**
  * Single Give Form
  */
-if ( ! function_exists( 'give_left_sidebar_pre_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_pre_wrap')) {
 	function give_left_sidebar_pre_wrap() {
-		echo apply_filters( 'give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">' );
+		echo apply_filters('give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">');
 	}
 }
 
-if ( ! function_exists( 'give_left_sidebar_post_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_post_wrap')) {
 	function give_left_sidebar_post_wrap() {
-		echo apply_filters( 'give_left_sidebar_post_wrap', '</div>' );
+		echo apply_filters('give_left_sidebar_post_wrap', '</div>');
 	}
 }
 
-if ( ! function_exists( 'give_get_forms_sidebar' ) ) {
+if ( ! function_exists('give_get_forms_sidebar')) {
 	function give_get_forms_sidebar() {
-		give_get_template_part( 'single-give-form/sidebar' );
+		give_get_template_part('single-give-form/sidebar');
 	}
 }
 
-if ( ! function_exists( 'give_show_form_images' ) ) {
+if ( ! function_exists('give_show_form_images')) {
 
 	/**
 	 * Output the donation form featured image.
 	 */
 	function give_show_form_images() {
-		if ( give_is_setting_enabled( give_get_option( 'form_featured_img' ) ) ) {
-			give_get_template_part( 'single-give-form/featured-image' );
+		if (give_is_setting_enabled(give_get_option('form_featured_img'))) {
+			give_get_template_part('single-give-form/featured-image');
 		}
 	}
 }
 
-if ( ! function_exists( 'give_template_single_title' ) ) {
+if ( ! function_exists('give_template_single_title')) {
 
 	/**
 	 * Output the form title.
 	 */
 	function give_template_single_title() {
-		give_get_template_part( 'single-give-form/title' );
+		give_get_template_part('single-give-form/title');
 	}
 }
 
@@ -422,7 +422,7 @@ 
  * Conditional Functions
  */
 
-if ( ! function_exists( 'is_give_form' ) ) {
+if ( ! function_exists('is_give_form')) {
 
 	/**
 	 * is_give_form
@@ -434,11 +434,11 @@ 
 	 * @return bool
 	 */
 	function is_give_form() {
-		return is_singular( array( 'give_form' ) );
+		return is_singular(array('give_form'));
 	}
 }
 
-if ( ! function_exists( 'is_give_category' ) ) {
+if ( ! function_exists('is_give_category')) {
 
 	/**
 	 * is_give_category
@@ -453,12 +453,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_category( $term = '' ) {
-		return is_tax( 'give_forms_category', $term );
+	function is_give_category($term = '') {
+		return is_tax('give_forms_category', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_tag' ) ) {
+if ( ! function_exists('is_give_tag')) {
 
 	/**
 	 * is_give_tag
@@ -473,12 +473,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_tag( $term = '' ) {
-		return is_tax( 'give_forms_tag', $term );
+	function is_give_tag($term = '') {
+		return is_tax('give_forms_tag', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_taxonomy' ) ) {
+if ( ! function_exists('is_give_taxonomy')) {
 
 	/**
 	 * is_give_taxonomy
@@ -490,6 +490,6 @@ 
 	 * @return bool
 	 */
 	function is_give_taxonomy() {
-		return is_tax( get_object_taxonomies( 'give_form' ) );
+		return is_tax(get_object_taxonomies('give_form'));
 	}
 }