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 +116 added lines, -116 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -105,8 +105,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -125,34 +125,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);
 		}
 	}
 
@@ -163,17 +163,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,
@@ -181,40 +181,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) give_get_meta( $post->ID, '_give_payment_customer_id', true );
-				if ( ! empty( $donor_id ) ) {
-					$donor = new Give_Donor( $donor_id );
-					if ( ! empty( $donor->id ) ) {
-						if ( empty( $donor->user_id ) && ! empty( $donor->payment_ids ) ) {
+				$donor_id = (int) give_get_meta($post->ID, '_give_payment_customer_id', true);
+				if ( ! empty($donor_id)) {
+					$donor = new Give_Donor($donor_id);
+					if ( ! empty($donor->id)) {
+						if (empty($donor->user_id) && ! empty($donor->payment_ids)) {
 							$add_author = false;
-							$count      = (int) count( $donor->payment_ids );
-							if ( 1 === $count ) {
-								Give()->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;
 				}
@@ -229,23 +229,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();
 	}
@@ -257,34 +257,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;
 		}
@@ -302,12 +302,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;
 
@@ -318,110 +318,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[] = give_get_meta( $item, '_give_payment_form_id', true );
+			foreach ($donation_ids as $item) {
+				$form_ids[] = give_get_meta($item, '_give_payment_form_id', true);
 
 				// Delete the main payment.
-				give_delete_donation( absint( $item ) );
+				give_delete_donation(absint($item));
 			}
 
 			// update the new form list.
-			$this->update_option( $this->form_key, $form_ids );
+			$this->update_option($this->form_key, $form_ids);
 		}
 
 
 		// 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;
@@ -433,24 +433,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];
 	}
 
 	/**
@@ -462,8 +462,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);
 	}
 
 	/**
@@ -476,8 +476,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);
 	}
 
 	/**
@@ -489,8 +489,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -501,12 +501,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;
@@ -517,6 +517,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/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 +268 added lines, -269 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.
-			$this->process_status( $status );
+			$this->process_status($status);
 
 			/**
 			 * Fires after changing payment status.
@@ -1254,7 +1254,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().
 
@@ -1289,24 +1289,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();
 			}
 		}
@@ -1316,7 +1316,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);
 	}
 
 	/**
@@ -1331,8 +1331,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;
 		}
 
@@ -1342,9 +1342,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);
 	}
 
 	/**
@@ -1357,7 +1357,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function process_status( $status ) {
+	private function process_status($status) {
 		$process = true;
 
 		// Bailout, if changed from completed to preapproval/processing.
@@ -1372,9 +1372,9 @@ 
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process = apply_filters( "give_should_process_{$status}", $process, $this );
+		$process = apply_filters("give_should_process_{$status}", $process, $this);
 
-		if ( false === $process ) {
+		if (false === $process) {
 			return;
 		}
 
@@ -1385,13 +1385,13 @@ 
 		 *
 		 * @since 1.5
 		 */
-		do_action( "give_pre_{$status}_payment", $this );
+		do_action("give_pre_{$status}_payment", $this);
 
-		$decrease_earnings       = apply_filters( "give_decrease_earnings_on_{$status}", true, $this );
-		$decrease_donor_value    = apply_filters( "give_decrease_donor_value_on_{$status}", true, $this );
-		$decrease_donation_count = apply_filters( "give_decrease_donors_donation_count_on_{$status}", true, $this );
+		$decrease_earnings       = apply_filters("give_decrease_earnings_on_{$status}", true, $this);
+		$decrease_donor_value    = apply_filters("give_decrease_donor_value_on_{$status}", true, $this);
+		$decrease_donation_count = apply_filters("give_decrease_donors_donation_count_on_{$status}", true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1404,7 +1404,7 @@ 
 		 *
 		 * @since 1.5
 		 */
-		do_action( "give_post_{$status}_payment", $this );
+		do_action("give_post_{$status}_payment", $this);
 	}
 
 	/**
@@ -1419,25 +1419,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();
 			}
 		}
@@ -1454,7 +1454,7 @@ 
 	 */
 	private function delete_sales_logs() {
 		// Remove related sale log entries.
-		Give()->logs->delete_logs( $this->ID );
+		Give()->logs->delete_logs($this->ID);
 	}
 
 	/**
@@ -1475,13 +1475,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;
 	}
@@ -1495,7 +1495,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1507,7 +1507,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');
 	}
 
 	/**
@@ -1519,9 +1519,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));
 	}
 
 	/**
@@ -1548,17 +1548,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
 			);
 
@@ -1574,7 +1573,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;
 	}
@@ -1588,11 +1587,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;
@@ -1608,7 +1607,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;
 	}
@@ -1623,7 +1622,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;
 	}
@@ -1640,8 +1639,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;
@@ -1658,10 +1657,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;
@@ -1681,22 +1680,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(),
@@ -1706,25 +1705,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;
 					}
 				}
@@ -1744,12 +1743,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;
 	}
@@ -1764,7 +1763,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;
 	}
@@ -1779,7 +1778,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;
 	}
@@ -1793,7 +1792,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;
 	}
@@ -1807,7 +1806,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;
 	}
@@ -1823,11 +1822,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;
 
@@ -1845,7 +1844,7 @@ 
 	 * @return array The payment object as an array.
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -1858,7 +1857,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -1870,7 +1869,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);
 	}
 
 	/**
@@ -1882,7 +1881,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);
 	}
 
 	/**
@@ -1894,7 +1893,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);
 	}
 
 	/**
@@ -1906,7 +1905,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);
 	}
 
 	/**
@@ -1918,7 +1917,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);
 	}
 
 	/**
@@ -1930,7 +1929,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);
 	}
 
 	/**
@@ -1942,7 +1941,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);
 	}
 
 	/**
@@ -1954,7 +1953,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);
 	}
 
 	/**
@@ -1966,7 +1965,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);
 	}
 
 	/**
@@ -1978,7 +1977,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);
 	}
 
 	/**
@@ -1990,7 +1989,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);
 	}
 
 	/**
@@ -2002,7 +2001,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);
 	}
 
 	/**
@@ -2014,7 +2013,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);
 	}
 
 	/**
@@ -2026,6 +2025,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/admin/reports/class-give-graph.php

Spacing +35 added lines, -35 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -81,15 +81,15 @@ 
 	 * @param array $_data
 	 * @param array $options
 	 */
-	public function __construct( $_data, $options = array() ) {
+	public function __construct($_data, $options = array()) {
 
-		$this->data      = $_data;
+		$this->data = $_data;
 
 		// Generate unique ID
-		$this->id = md5( rand() );
+		$this->id = md5(rand());
 
 		// Setup default options;
-		$this->options = apply_filters( 'give_graph_args', array(
+		$this->options = apply_filters('give_graph_args', array(
 			'y_mode'          => null,
 			'x_mode'          => null,
 			'y_decimals'      => 0,
@@ -107,9 +107,9 @@ 
 			'lines'           => false,
 			'points'          => true,
 			'dataType'        => array()
-		) );
+		));
 
-		$this->options = wp_parse_args( $options, $this->options );
+		$this->options = wp_parse_args($options, $this->options);
 	}
 
 	/**
@@ -120,8 +120,8 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function set( $key, $value ) {
-		$this->options[ $key ] = $value;
+	public function set($key, $value) {
+		$this->options[$key] = $value;
 	}
 
 	/**
@@ -131,8 +131,8 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function get( $key ) {
-		return isset( $this->options[ $key ] ) ? $this->options[ $key ] : false;
+	public function get($key) {
+		return isset($this->options[$key]) ? $this->options[$key] : false;
 	}
 
 	/**
@@ -141,7 +141,7 @@ 
 	 * @since 1.0
 	 */
 	public function get_data() {
-		return apply_filters( 'give_get_graph_data', $this->data, $this );
+		return apply_filters('give_get_graph_data', $this->data, $this);
 	}
 
 	/**
@@ -151,19 +151,19 @@ 
 	 */
 	public function load_scripts() {
 		// Use minified libraries if SCRIPT_DEBUG is turned off
-		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+		$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
-		wp_register_script( 'jquery-flot-orderbars', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.orderBars' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-orderbars' );
+		wp_register_script('jquery-flot-orderbars', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.orderBars'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-orderbars');
 
-		wp_register_script( 'jquery-flot-time', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.time' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-time' );
+		wp_register_script('jquery-flot-time', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.time'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-time');
 
-		wp_register_script( 'jquery-flot-resize', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.resize' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-resize' );
+		wp_register_script('jquery-flot-resize', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.resize'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-resize');
 
-		wp_register_script( 'jquery-flot', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot' . $suffix . '.js', false, GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot' );
+		wp_register_script('jquery-flot', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot'.$suffix.'.js', false, GIVE_VERSION);
+		wp_enqueue_script('jquery-flot');
 
 	}
 
@@ -190,14 +190,14 @@ 
 					[
 						<?php
 							$order = 0;
-							foreach( $this->get_data() as $label => $data ) :
+							foreach ($this->get_data() as $label => $data) :
 						?>
 						{
-							label : "<?php echo esc_attr( $label ); ?>",
-							id    : "<?php echo sanitize_key( $label ); ?>",
-							dataType  : '<?php echo ( ! empty( $this->options['dataType'][$order] ) ? $this->options['dataType'][$order] : 'count' ); ?>',
+							label : "<?php echo esc_attr($label); ?>",
+							id    : "<?php echo sanitize_key($label); ?>",
+							dataType  : '<?php echo ( ! empty($this->options['dataType'][$order]) ? $this->options['dataType'][$order] : 'count'); ?>',
 							// data format is: [ point on x, value on y ]
-							data  : [<?php foreach( $data as $point ) { echo '[' . implode( ',', $point ) . '],'; } ?>],
+							data  : [<?php foreach ($data as $point) { echo '['.implode(',', $point).'],'; } ?>],
 							points: {
 								show: <?php echo $this->options['points'] ? 'true' : 'false'; ?>,
 							},
@@ -212,7 +212,7 @@ 
 								fill     : true,
 								fillColor: {colors: [{opacity: 0.4}, {opacity: 0.1}]}
 							},
-							<?php if( $this->options[ 'multiple_y_axes' ] ) : ?>
+							<?php if ($this->options['multiple_y_axes']) : ?>
 							yaxis : <?php echo $yaxis_count; ?>
 							<?php endif; ?>
 
@@ -226,10 +226,10 @@ 
 						grid: {
 							show           : true,
 							aboveData      : false,
-							color          : "<?php echo $this->options[ 'color' ]; ?>",
-							backgroundColor: "<?php echo $this->options[ 'bgcolor' ]; ?>",
-							borderColor    : "<?php echo $this->options[ 'bordercolor' ]; ?>",
-							borderWidth    : <?php echo absint( $this->options[ 'borderwidth' ] ); ?>,
+							color          : "<?php echo $this->options['color']; ?>",
+							backgroundColor: "<?php echo $this->options['bgcolor']; ?>",
+							borderColor    : "<?php echo $this->options['bordercolor']; ?>",
+							borderWidth    : <?php echo absint($this->options['borderwidth']); ?>,
 							clickable      : false,
 							hoverable      : true
 						},
@@ -240,7 +240,7 @@ 
 							mode        : "<?php echo $this->options['x_mode']; ?>",
 							timeFormat  : "<?php echo $this->options['x_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
 							tickSize    : "<?php echo $this->options['x_mode'] == 'time' ? '' : 1; ?>",
-							<?php if( $this->options['x_mode'] != 'time' ) : ?>
+							<?php if ($this->options['x_mode'] != 'time') : ?>
 							tickDecimals: <?php echo $this->options['x_decimals']; ?>
 							<?php endif; ?>
 						},
@@ -249,7 +249,7 @@ 
 							min         : 0,
 							mode        : "<?php echo $this->options['y_mode']; ?>",
 							timeFormat  : "<?php echo $this->options['y_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
-							<?php if( $this->options['y_mode'] != 'time' ) : ?>
+							<?php if ($this->options['y_mode'] != 'time') : ?>
 							tickDecimals: <?php echo $this->options['y_decimals']; ?>,
 							<?php endif; ?>
 							tickFormatter: function(val) {
@@ -323,7 +323,7 @@ 
 		 *
 		 * @param Give_Graph $this Graph object.
 		 */
-		do_action( 'give_before_graph', $this );
+		do_action('give_before_graph', $this);
 
 		// Build the graph.
 		echo $this->build_graph();
@@ -335,7 +335,7 @@ 
 		 *
 		 * @param Give_Graph $this Graph object.
 		 */
-		do_action( 'give_after_graph', $this );
+		do_action('give_after_graph', $this);
 	}
 
 }

includes/admin/tools/import/class-give-import-donations.php

Doc Comments +2 added lines, -2 removed lines

@@ -454,7 +454,7 @@ 
 
 		/**
 		 * @param $option_value
-		 * @param $value
+		 * @param boolean $value
 		 *
 		 * @return string
 		 */
@@ -544,7 +544,7 @@ 
 		 *
 		 * @since 1.8.14
 		 *
-		 * @param $file_id
+		 * @param integer $file_id
 		 *
 		 * @return bool|int
 		 */

Indentation +2 added lines, -2 removed lines

@@ -871,8 +871,8 @@
 		 */
 		private function is_donations_import_page() {
 			return 'import' === give_get_current_setting_tab() &&
-			       isset( $_GET['importer-type'] ) &&
-			       $this->importer_type === give_clean( $_GET['importer-type'] );
+				   isset( $_GET['importer-type'] ) &&
+				   $this->importer_type === give_clean( $_GET['importer-type'] );
 		}
 	}
 

Spacing +196 added lines, -202 removed lines

@@ -11,11 +11,11 @@ 
  * @since       1.8.14
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Import_Donations' ) ) {
+if ( ! class_exists('Give_Import_Donations')) {
 
 	/**
 	 * Give_Import_Donations.
@@ -57,7 +57,7 @@ 
 		 * @access private
 		 */
 		private function __construct() {
-			self::$per_page  = ! empty( $_GET['per_page'] ) ? absint( $_GET['per_page'] ) : self::$per_page;
+			self::$per_page = ! empty($_GET['per_page']) ? absint($_GET['per_page']) : self::$per_page;
 		}
 
 		/**
@@ -69,7 +69,7 @@ 
 		 * @return static
 		 */
 		public static function get_instance() {
-			if ( null === static::$instance ) {
+			if (null === static::$instance) {
 				self::$instance = new static();
 			}
 
@@ -96,27 +96,27 @@ 
 		 * @return void
 		 */
 		private function setup_hooks() {
-			if ( ! $this->is_donations_import_page() ) {
+			if ( ! $this->is_donations_import_page()) {
 				return;
 			}
 
 			// Do not render main import tools page.
-			remove_action( 'give_admin_field_tools_import', array( 'Give_Settings_Import', 'render_import_field', ) );
+			remove_action('give_admin_field_tools_import', array('Give_Settings_Import', 'render_import_field',));
 
 
 			// Render donation import page
-			add_action( 'give_admin_field_tools_import', array( $this, 'render_page' ) );
+			add_action('give_admin_field_tools_import', array($this, 'render_page'));
 
 			// Print the HTML.
-			add_action( 'give_tools_import_donations_form_start', array( $this, 'html' ), 10 );
+			add_action('give_tools_import_donations_form_start', array($this, 'html'), 10);
 
 			// Run when form submit.
-			add_action( 'give-tools_save_import', array( $this, 'save' ) );
+			add_action('give-tools_save_import', array($this, 'save'));
 
-			add_action( 'give-tools_update_notices', array( $this, 'update_notices' ), 11, 1 );
+			add_action('give-tools_update_notices', array($this, 'update_notices'), 11, 1);
 
 			// Used to add submit button.
-			add_action( 'give_tools_import_donations_form_end', array( $this, 'submit' ), 10 );
+			add_action('give_tools_import_donations_form_end', array($this, 'submit'), 10);
 		}
 
 		/**
@@ -128,9 +128,9 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function update_notices( $messages ) {
-			if ( ! empty( $_GET['tab'] ) && 'import' === give_clean( $_GET['tab'] ) ) {
-				unset( $messages['give-setting-updated'] );
+		public function update_notices($messages) {
+			if ( ! empty($_GET['tab']) && 'import' === give_clean($_GET['tab'])) {
+				unset($messages['give-setting-updated']);
 			}
 
 			return $messages;
@@ -142,7 +142,7 @@ 
 		 * @since 1.8.14
 		 */
 		public function submit() {
-			wp_nonce_field( 'give-save-settings', '_give-save-settings' );
+			wp_nonce_field('give-save-settings', '_give-save-settings');
 			?>
 			<input type="hidden" class="import-step" id="import-step" name="step" value="<?php echo $this->get_step(); ?>"/>
 			<input type="hidden" class="importer-type" value="<?php echo $this->importer_type; ?>"/>
@@ -164,7 +164,7 @@ 
 				<table class="widefat export-options-table give-table <?php echo "step-{$step}"; ?>" id="<?php echo "step-{$step}"; ?>">
 					<tbody>
 						<?php
-						switch ( $this->get_step() ) {
+						switch ($this->get_step()) {
 							case 1:
 								$this->render_media_csv();
 								break;
@@ -181,7 +181,7 @@ 
 								$this->import_success();
 						}
 
-						if ( false === $this->check_for_dropdown_or_import() ) {
+						if (false === $this->check_for_dropdown_or_import()) {
 							?>
 							<tr valign="top">
 								<th></th>
@@ -189,8 +189,8 @@ 
 									<input type="submit"
 									       class=" button button-primary button-large button-secondary <?php echo "step-{$step}"; ?>"
 									       id="recount-stats-submit"
-									       <?php echo ( 2 === $step ) ? 'disabled' : ''; ?>
-									       value="<?php esc_attr_e( 'Submit', 'give' ); ?>"/>
+									       <?php echo (2 === $step) ? 'disabled' : ''; ?>
+									       value="<?php esc_attr_e('Submit', 'give'); ?>"/>
 								</th>
 							</tr>
 							<?php
@@ -209,57 +209,57 @@ 
 		 */
 		public function import_success() {
 
-			$delete_csv = ( ! empty( $_GET['delete_csv'] ) ? absint( $_GET['delete_csv'] ) : false );
-			$csv        = ( ! empty( $_GET['csv'] ) ? absint( $_GET['csv'] ) : false );
-			if ( ! empty( $delete_csv ) && ! empty( $csv ) ) {
-				wp_delete_attachment( $csv, true );
+			$delete_csv = ( ! empty($_GET['delete_csv']) ? absint($_GET['delete_csv']) : false);
+			$csv        = ( ! empty($_GET['csv']) ? absint($_GET['csv']) : false);
+			if ( ! empty($delete_csv) && ! empty($csv)) {
+				wp_delete_attachment($csv, true);
 			}
 
 			$report      = give_import_donation_report();
 
 			$report_html = array(
 				'duplicate_donor'    => array(
-					__( '%s duplicate %s detected', 'give' ),
-					__( 'donor', 'give' ),
-					__( 'donors', 'give' ),
+					__('%s duplicate %s detected', 'give'),
+					__('donor', 'give'),
+					__('donors', 'give'),
 				),
 				'create_donor'       => array(
-					__( '%s %s created', 'give' ),
-					__( 'donor', 'give' ),
-					__( 'donors', 'give' ),
+					__('%s %s created', 'give'),
+					__('donor', 'give'),
+					__('donors', 'give'),
 				),
 				'create_form'        => array(
-					__( '%s donation %s created', 'give' ),
-					__( 'form', 'give' ),
-					__( 'forms', 'give' ),
+					__('%s donation %s created', 'give'),
+					__('form', 'give'),
+					__('forms', 'give'),
 				),
 				'duplicate_donation' => array(
-					__( '%s duplicate %s detected', 'give' ),
-					__( 'donation', 'give' ),
-					__( 'donations', 'give' ),
+					__('%s duplicate %s detected', 'give'),
+					__('donation', 'give'),
+					__('donations', 'give'),
 				),
 				'create_donation'    => array(
-					__( '%s %s imported', 'give' ),
-					__( 'donation', 'give' ),
-					__( 'donations', 'give' ),
+					__('%s %s imported', 'give'),
+					__('donation', 'give'),
+					__('donations', 'give'),
 				),
 			);
-			$total       = (int) $_GET['total'];
-			-- $total;
+			$total = (int) $_GET['total'];
+			--$total;
 			$success = (bool) $_GET['success'];
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
 					<h2>
 						<?php
-						if ( $success ) {
+						if ($success) {
 							printf(
-								_n( 'Import complete! %s donation processed', 'Import complete! %s donations processed', $total, 'give' ),
+								_n('Import complete! %s donation processed', 'Import complete! %s donations processed', $total, 'give'),
 								"<strong>{$total}</strong>"
 							);
 						} else {
 							printf(
-								_n( 'Failed to import %s donation', 'Failed to import %s donations', $total, 'give' ),
+								_n('Failed to import %s donation', 'Failed to import %s donations', $total, 'give'),
 								"<strong>{$total}</strong>"
 							);
 						}
@@ -267,25 +267,25 @@ 
 					</h2>
 
 					<?php
-					$text      = __( 'Import Donation', 'give' );
+					$text      = __('Import Donation', 'give');
 					$query_arg = array(
 						'post_type' => 'give_forms',
 						'page'      => 'give-tools',
 						'tab'       => 'import',
 					);
-					if ( $success ) {
+					if ($success) {
 						$query_arg = array(
 							'post_type' => 'give_forms',
 							'page'      => 'give-payment-history',
 						);
-						$text      = __( 'View Donations', 'give' );
+						$text = __('View Donations', 'give');
 					}
 
-					foreach ( $report as $key => $value ) {
-						if ( array_key_exists( $key, $report_html ) && ! empty( $value ) ) {
+					foreach ($report as $key => $value) {
+						if (array_key_exists($key, $report_html) && ! empty($value)) {
 							?>
 							<p>
-								<?php echo esc_html( wp_sprintf( $report_html[ $key ][0], $value, _n( $report_html[ $key ][1], $report_html[ $key ][2], $value, 'give' ) ) ); ?>
+								<?php echo esc_html(wp_sprintf($report_html[$key][0], $value, _n($report_html[$key][1], $report_html[$key][2], $value, 'give'))); ?>
 							</p>
 							<?php
 						}
@@ -293,7 +293,7 @@ 
 					?>
 
 					<p>
-						<a class="button button-large button-secondary" href="<?php echo add_query_arg( $query_arg, admin_url( 'edit.php' ) ); ?>"><?php echo $text; ?></a>
+						<a class="button button-large button-secondary" href="<?php echo add_query_arg($query_arg, admin_url('edit.php')); ?>"><?php echo $text; ?></a>
 					</p>
 				</th>
 			</tr>
@@ -310,26 +310,26 @@ 
 			give_import_donation_report_reset();
 
 			$csv         = (int) $_REQUEST['csv'];
-			$delimiter   = ( ! empty( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv' );
+			$delimiter   = ( ! empty($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv');
 			$index_start = 1;
 			$index_end   = 1;
 			$next        = true;
-			$total       = self::get_csv_total( $csv );
-			if ( self::$per_page < $total ) {
-				$total_ajax = ceil( $total / self::$per_page );
+			$total       = self::get_csv_total($csv);
+			if (self::$per_page < $total) {
+				$total_ajax = ceil($total / self::$per_page);
 				$index_end  = self::$per_page;
 			} else {
 				$total_ajax = 1;
 				$index_end  = $total;
 				$next       = false;
 			}
-			$current_percentage = 100 / ( $total_ajax + 1 );
+			$current_percentage = 100 / ($total_ajax + 1);
 
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Importing', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'Your donations are now being imported...', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Importing', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('Your donations are now being imported...', 'give') ?></p>
 				</th>
 			</tr>
 
@@ -348,7 +348,7 @@ 
 						<div style="width: <?php echo $current_percentage; ?>%"></div>
 					</div>
 					<input type="hidden" value="3" name="step">
-					<input type="hidden" value='<?php echo maybe_serialize( $_REQUEST['mapto'] ); ?>' name="mapto"
+					<input type="hidden" value='<?php echo maybe_serialize($_REQUEST['mapto']); ?>' name="mapto"
 						   class="mapto">
 					<input type="hidden" value="<?php echo $_REQUEST['csv']; ?>" name="csv" class="csv">
 					<input type="hidden" value="<?php echo $_REQUEST['mode']; ?>" name="mode" class="mode">
@@ -357,7 +357,7 @@ 
 					<input type="hidden" value="<?php echo $_REQUEST['delete_csv']; ?>" name="delete_csv"
 						   class="delete_csv">
 					<input type="hidden" value="<?php echo $delimiter; ?>" name="delimiter">
-					<input type="hidden" value='<?php echo maybe_serialize( self::get_importer( $csv, 0, $delimiter ) ); ?>'
+					<input type="hidden" value='<?php echo maybe_serialize(self::get_importer($csv, 0, $delimiter)); ?>'
 						   name="main_key"
 						   class="main_key">
 				</th>
@@ -378,20 +378,20 @@ 
 		 */
 		public function check_for_dropdown_or_import() {
 			$return = true;
-			if ( isset( $_REQUEST['mapto'] ) ) {
+			if (isset($_REQUEST['mapto'])) {
 				$mapto = (array) $_REQUEST['mapto'];
-				if ( false === in_array( 'form_title', $mapto ) && false === in_array( 'form_id', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-form', __( 'In order to import donations, a column must be mapped to either the "Donation Form Title" or "Donation Form ID" field. Please map a column to one of those fields.', 'give' ) );
+				if (false === in_array('form_title', $mapto) && false === in_array('form_id', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-form', __('In order to import donations, a column must be mapped to either the "Donation Form Title" or "Donation Form ID" field. Please map a column to one of those fields.', 'give'));
 					$return = false;
 				}
 
-				if ( false === in_array( 'amount', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-amount', __( 'In order to import donations, a column must be mapped to the "Amount" field. Please map a column to that field.', 'give' ) );
+				if (false === in_array('amount', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-amount', __('In order to import donations, a column must be mapped to the "Amount" field. Please map a column to that field.', 'give'));
 					$return = false;
 				}
 
-				if ( false === in_array( 'email', $mapto ) && false === in_array( 'donor_id', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-donor', __( 'In order to import donations, a column must be mapped to either the "Donor Email" or "Donor ID" field. Please map a column to that field.', 'give' ) );
+				if (false === in_array('email', $mapto) && false === in_array('donor_id', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-donor', __('In order to import donations, a column must be mapped to either the "Donor Email" or "Donor ID" field. Please map a column to that field.', 'give'));
 					$return = false;
 				}
 			} else {
@@ -408,10 +408,10 @@ 
 		 */
 		public function render_dropdown() {
 			$csv       = (int) $_GET['csv'];
-			$delimiter = ( ! empty( $_GET['delimiter'] ) ? give_clean( $_GET['delimiter'] ) : 'csv' );
+			$delimiter = ( ! empty($_GET['delimiter']) ? give_clean($_GET['delimiter']) : 'csv');
 
 			// TO check if the CSV files that is being add is valid or not if not then redirect to first step again
-			if ( ! $this->is_valid_csv( $csv ) ) {
+			if ( ! $this->is_valid_csv($csv)) {
 				$url = give_import_page_url();
 				?>
 				<script type="text/javascript">
@@ -422,18 +422,18 @@ 
 				?>
 				<tr valign="top" class="give-import-dropdown">
 					<th colspan="2">
-						<h2 id="give-import-title"><?php esc_html_e( 'Map CSV fields to donations', 'give' ) ?></h2>
+						<h2 id="give-import-title"><?php esc_html_e('Map CSV fields to donations', 'give') ?></h2>
 
-						<p class="give-import-donation-required-fields-title"><?php _e( 'Required Fields' ); ?></p>
+						<p class="give-import-donation-required-fields-title"><?php _e('Required Fields'); ?></p>
 
-						<p class="give-field-description"><?php _e( 'These fields are required for the import to submitted' ); ?></p>
+						<p class="give-field-description"><?php _e('These fields are required for the import to submitted'); ?></p>
 
 						<ul class="give-import-donation-required-fields">
 							<li class="give-import-donation-required-email" title="Please configure all required fields to start the import process.">
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'Email Access', 'give' );
+									_e('Email Access', 'give');
 									?>
 								</span>
 							</li>
@@ -442,7 +442,7 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'First Name', 'give' );
+									_e('First Name', 'give');
 									?>
 								</span>
 							</li>
@@ -451,7 +451,7 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'Donation Amount', 'give' );
+									_e('Donation Amount', 'give');
 									?>
 								</span>
 							</li>
@@ -460,32 +460,32 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'Form Title or ID', 'give' );
+									_e('Form Title or ID', 'give');
 									?>
 								</span>
 							</li>
 						</ul>
 
-						<p class="give-field-description"><?php esc_html_e( 'Select fields from your CSV file to map against donations fields or to ignore during import.', 'give' ) ?></p>
+						<p class="give-field-description"><?php esc_html_e('Select fields from your CSV file to map against donations fields or to ignore during import.', 'give') ?></p>
 					</th>
 				</tr>
 
 				<tr valign="top" class="give-import-dropdown">
-					<th><b><?php esc_html_e( 'Column name', 'give' ); ?></b></th>
-					<th><b><?php esc_html_e( 'Map to field', 'give' ); ?></b></th>
+					<th><b><?php esc_html_e('Column name', 'give'); ?></b></th>
+					<th><b><?php esc_html_e('Map to field', 'give'); ?></b></th>
 				</tr>
 
 				<?php
-				$raw_key   = $this->get_importer( $csv, 0, $delimiter );
-				$mapto     = (array) ( isset( $_REQUEST['mapto'] ) ? $_REQUEST['mapto'] : array() );
+				$raw_key   = $this->get_importer($csv, 0, $delimiter);
+				$mapto     = (array) (isset($_REQUEST['mapto']) ? $_REQUEST['mapto'] : array());
 
-				foreach ( $raw_key as $index => $value ) {
+				foreach ($raw_key as $index => $value) {
 					?>
 					<tr valign="top" class="give-import-option">
 						<th><?php echo $value; ?></th>
 						<th>
 							<?php
-							$this->get_columns( $index, $value, $mapto );
+							$this->get_columns($index, $value, $mapto);
 							?>
 						</th>
 					</tr>
@@ -500,14 +500,14 @@ 
 		 *
 		 * @return string
 		 */
-		public function selected( $option_value, $value ) {
-			$option_value = strtolower( $option_value );
-			$value = strtolower( $value );
+		public function selected($option_value, $value) {
+			$option_value = strtolower($option_value);
+			$value = strtolower($value);
 
 			$selected = '';
-			if ( stristr( $value, $option_value ) ) {
+			if (stristr($value, $option_value)) {
 				$selected = 'selected';
-			} elseif ( strrpos( $value, '_' ) && stristr( $option_value, __( 'Import as Meta', 'give' ) ) ) {
+			} elseif (strrpos($value, '_') && stristr($option_value, __('Import as Meta', 'give'))) {
 				$selected = 'selected';
 			}
 
@@ -526,28 +526,28 @@ 
 		 *
 		 * @return void
 		 */
-		private function get_columns( $index, $value = false, $mapto = array() ) {
+		private function get_columns($index, $value = false, $mapto = array()) {
 			$default       = give_import_default_options();
-			$current_mapto = (string) ( ! empty( $mapto[ $index ] ) ? $mapto[ $index ] : '' );
+			$current_mapto = (string) ( ! empty($mapto[$index]) ? $mapto[$index] : '');
 			?>
 			<select name="mapto[<?php echo $index; ?>]">
-				<?php $this->get_dropdown_option_html( $default, $current_mapto, $value ); ?>
+				<?php $this->get_dropdown_option_html($default, $current_mapto, $value); ?>
 
-				<optgroup label="<?php _e( 'Donations', 'give' ); ?>">
+				<optgroup label="<?php _e('Donations', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donations_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donations_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
-				<optgroup label="<?php _e( 'Donors', 'give' ); ?>">
+				<optgroup label="<?php _e('Donors', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donor_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donor_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
-				<optgroup label="<?php _e( 'Forms', 'give' ); ?>">
+				<optgroup label="<?php _e('Forms', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donation_form_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donation_form_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
@@ -558,7 +558,7 @@ 
 				 *
 				 * @since 1.8.15
 				 */
-				do_action( 'give_import_dropdown_option', $index, $value, $mapto, $current_mapto );
+				do_action('give_import_dropdown_option', $index, $value, $mapto, $current_mapto);
 				?>
 			</select>
 			<?php
@@ -576,16 +576,16 @@ 
 		 *
 		 * @return void
 		 */
-		public function get_dropdown_option_html( $options, $current_mapto, $value = false ) {
-			foreach ( $options as $option => $option_value ) {
+		public function get_dropdown_option_html($options, $current_mapto, $value = false) {
+			foreach ($options as $option => $option_value) {
 				$option_value_texts = (array) $option_value;
 				$option_text = $option_value_texts[0];
 
-				$checked = ( ( $current_mapto === $option ) ? 'selected' : false );
-				if ( empty( $checked ) ) {
-					foreach ( $option_value_texts as $option_value_text ) {
-						$checked = $this->selected( $option_value_text, $value );
-						if ( $checked ) {
+				$checked = (($current_mapto === $option) ? 'selected' : false);
+				if (empty($checked)) {
+					foreach ($option_value_texts as $option_value_text) {
+						$checked = $this->selected($option_value_text, $value);
+						if ($checked) {
 							break;
 						}
 					}
@@ -609,13 +609,13 @@ 
 		 *
 		 * @return bool|int
 		 */
-		public function get_csv_total( $file_id ) {
+		public function get_csv_total($file_id) {
 			$total = false;
-			if ( $file_id ) {
-				$file_dir = get_attached_file( $file_id );
-				if ( $file_dir ) {
-					$file = new SplFileObject( $file_dir, 'r' );
-					$file->seek( PHP_INT_MAX );
+			if ($file_id) {
+				$file_dir = get_attached_file($file_id);
+				if ($file_dir) {
+					$file = new SplFileObject($file_dir, 'r');
+					$file->seek(PHP_INT_MAX);
 					$total = $file->key() + 1;
 				}
 			}
@@ -634,7 +634,7 @@ 
 		 *
 		 * @return array|bool $raw_data title of the CSV file fields
 		 */
-		public function get_importer( $file_id, $index = 0, $delimiter = 'csv' ) {
+		public function get_importer($file_id, $index = 0, $delimiter = 'csv') {
 			/**
 			 * Filter to modify delimiter of Import.
 			 *
@@ -642,16 +642,16 @@ 
 			 *
 			 * Return string $delimiter.
 			 */
-			$delimiter = (string) apply_filters( 'give_import_delimiter_set', $delimiter );
+			$delimiter = (string) apply_filters('give_import_delimiter_set', $delimiter);
 
 			$raw_data = false;
-			$file_dir = get_attached_file( $file_id );
-			if ( $file_dir ) {
-				if ( false !== ( $handle = fopen( $file_dir, 'r' ) ) ) {
-					$raw_data = fgetcsv( $handle, $index, $delimiter );
+			$file_dir = get_attached_file($file_id);
+			if ($file_dir) {
+				if (false !== ($handle = fopen($file_dir, 'r'))) {
+					$raw_data = fgetcsv($handle, $index, $delimiter);
 					// Remove BOM signature from the first item.
-					if ( isset( $raw_data[0] ) ) {
-						$raw_data[0] = $this->remove_utf8_bom( $raw_data[0] );
+					if (isset($raw_data[0])) {
+						$raw_data[0] = $this->remove_utf8_bom($raw_data[0]);
 					}
 				}
 			}
@@ -668,9 +668,9 @@ 
 		 *
 		 * @return string
 		 */
-		public function remove_utf8_bom( $string ) {
-			if ( 'efbbbf' === substr( bin2hex( $string ), 0, 6 ) ) {
-				$string = substr( $string, 3 );
+		public function remove_utf8_bom($string) {
+			if ('efbbbf' === substr(bin2hex($string), 0, 6)) {
+				$string = substr($string, 3);
 			}
 
 			return $string;
@@ -686,17 +686,17 @@ 
 			$step = $this->get_step();
 			?>
 			<ol class="give-progress-steps">
-				<li class="<?php echo( 1 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Upload CSV file', 'give' ); ?>
+				<li class="<?php echo(1 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Upload CSV file', 'give'); ?>
 				</li>
-				<li class="<?php echo( 2 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Column mapping', 'give' ); ?>
+				<li class="<?php echo(2 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Column mapping', 'give'); ?>
 				</li>
-				<li class="<?php echo( 3 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Import', 'give' ); ?>
+				<li class="<?php echo(3 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Import', 'give'); ?>
 				</li>
-				<li class="<?php echo( 4 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Done!', 'give' ); ?>
+				<li class="<?php echo(4 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Done!', 'give'); ?>
 				</li>
 			</ol>
 			<?php
@@ -710,16 +710,16 @@ 
 		 * @return int $step on which step doest the import is on.
 		 */
 		public function get_step() {
-			$step    = (int) ( isset( $_REQUEST['step'] ) ? give_clean( $_REQUEST['step'] ) : 0 );
+			$step    = (int) (isset($_REQUEST['step']) ? give_clean($_REQUEST['step']) : 0);
 			$on_step = 1;
 
-			if ( empty( $step ) || 1 === $step ) {
+			if (empty($step) || 1 === $step) {
 				$on_step = 1;
-			} elseif ( $this->check_for_dropdown_or_import() ) {
+			} elseif ($this->check_for_dropdown_or_import()) {
 				$on_step = 3;
-			} elseif ( 2 === $step ) {
+			} elseif (2 === $step) {
 				$on_step = 2;
-			} elseif ( 4 === $step ) {
+			} elseif (4 === $step) {
 				$on_step = 4;
 			}
 
@@ -732,7 +732,7 @@ 
 		 * @since 1.8.14
 		 */
 		public function render_page() {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-import-donations.php';
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-import-donations.php';
 		}
 
 		/**
@@ -747,37 +747,34 @@ 
 			?>
 			<tr valign="top">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Import donations from a CSV file', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'This tool allows you to import or add donation data to your give form(s) via a CSV file.', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Import donations from a CSV file', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('This tool allows you to import or add donation data to your give form(s) via a CSV file.', 'give') ?></p>
 				</th>
 			</tr>
 			<?php
-			$csv         = ( isset( $_POST['csv'] ) ? give_clean( $_POST['csv'] ) : '' );
-			$csv_id      = ( isset( $_POST['csv_id'] ) ? give_clean( $_POST['csv_id'] ) : '' );
-			$delimiter   = ( isset( $_POST['delimiter'] ) ? give_clean( $_POST['delimiter'] ) : 'csv' );
-			$mode        = empty( $_POST['mode'] ) ?
-				'disabled' :
-				( give_is_setting_enabled( give_clean( $_POST['mode'] ) ) ? 'enabled' : 'disabled' );
-			$create_user = empty( $_POST['create_user'] ) ?
-				'enabled' :
-				( give_is_setting_enabled( give_clean( $_POST['create_user'] ) ) ? 'enabled' : 'disabled' );
-			$delete_csv  = empty( $_POST['delete_csv'] ) ?
-				'enabled' :
-				( give_is_setting_enabled( give_clean( $_POST['delete_csv'] ) ) ? 'enabled' : 'disabled' );
+			$csv         = (isset($_POST['csv']) ? give_clean($_POST['csv']) : '');
+			$csv_id      = (isset($_POST['csv_id']) ? give_clean($_POST['csv_id']) : '');
+			$delimiter   = (isset($_POST['delimiter']) ? give_clean($_POST['delimiter']) : 'csv');
+			$mode        = empty($_POST['mode']) ?
+				'disabled' : (give_is_setting_enabled(give_clean($_POST['mode'])) ? 'enabled' : 'disabled');
+			$create_user = empty($_POST['create_user']) ?
+				'enabled' : (give_is_setting_enabled(give_clean($_POST['create_user'])) ? 'enabled' : 'disabled');
+			$delete_csv  = empty($_POST['delete_csv']) ?
+				'enabled' : (give_is_setting_enabled(give_clean($_POST['delete_csv'])) ? 'enabled' : 'disabled');
 
 			// Reset csv and csv_id if csv
-			if ( empty( $csv_id ) || ! $this->is_valid_csv( $csv_id, $csv ) ) {
+			if (empty($csv_id) || ! $this->is_valid_csv($csv_id, $csv)) {
 				$csv_id = $csv = '';
 			}
-			$per_page = isset( $_POST['per_page'] ) ? absint( $_POST['per_page'] ) : self::$per_page;
+			$per_page = isset($_POST['per_page']) ? absint($_POST['per_page']) : self::$per_page;
 
 			$settings = array(
 				array(
 					'id'          => 'csv',
-					'name'        => __( 'Choose a CSV file:', 'give' ),
+					'name'        => __('Choose a CSV file:', 'give'),
 					'type'        => 'file',
-					'attributes'  => array( 'editing' => 'false', 'library' => 'text' ),
-					'description' => __( 'The file must be a Comma Seperated Version (CSV) file type only.', 'give' ),
+					'attributes'  => array('editing' => 'false', 'library' => 'text'),
+					'description' => __('The file must be a Comma Seperated Version (CSV) file type only.', 'give'),
 					'fvalue'      => 'url',
 					'default'     => $csv,
 				),
@@ -788,61 +785,61 @@ 
 				),
 				array(
 					'id'          => 'delimiter',
-					'name'        => __( 'CSV Delimiter:', 'give' ),
-					'description' => __( 'In case your CSV file supports a different type of separator (or delimiter) -- like a tab or space -- you can set that here.', 'give' ),
+					'name'        => __('CSV Delimiter:', 'give'),
+					'description' => __('In case your CSV file supports a different type of separator (or delimiter) -- like a tab or space -- you can set that here.', 'give'),
 					'default'     => $delimiter,
 					'type'        => 'select',
 					'options'     => array(
-						'csv'                  => esc_html__( 'Comma', 'give' ),
-						'tab-separated-values' => esc_html__( 'Tab', 'give' ),
+						'csv'                  => esc_html__('Comma', 'give'),
+						'tab-separated-values' => esc_html__('Tab', 'give'),
 					),
 				),
 				array(
 					'id'          => 'mode',
-					'name'        => __( 'Test Mode:', 'give' ),
-					'description' => __( 'Test mode allows you to preview what this import would look like without making any actual changes to your site or your database.', 'give' ),
+					'name'        => __('Test Mode:', 'give'),
+					'description' => __('Test mode allows you to preview what this import would look like without making any actual changes to your site or your database.', 'give'),
 					'default'     => $mode,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'create_user',
-					'name'        => __( 'Create WP users for new donors:', 'give' ),
-					'description' => __( 'The importer can create WordPress user accounts based on the names and email addresses of the donations in your CSV file. Enable this option if you\'d like the importer to do that.', 'give' ),
+					'name'        => __('Create WP users for new donors:', 'give'),
+					'description' => __('The importer can create WordPress user accounts based on the names and email addresses of the donations in your CSV file. Enable this option if you\'d like the importer to do that.', 'give'),
 					'default'     => $create_user,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'delete_csv',
-					'name'        => __( 'Delete CSV after import:', 'give' ),
-					'description' => __( 'Your CSV file will be uploaded via the WordPress Media Library. It\'s a good idea to delete it after the import is finished so that your sensitive data is not accessible on the web. Disable this only if you plan to delete the file manually later.', 'give' ),
+					'name'        => __('Delete CSV after import:', 'give'),
+					'description' => __('Your CSV file will be uploaded via the WordPress Media Library. It\'s a good idea to delete it after the import is finished so that your sensitive data is not accessible on the web. Disable this only if you plan to delete the file manually later.', 'give'),
 					'default'     => $delete_csv,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'per_page',
-					'name'        => __( 'Process Rows Per Batch:', 'give' ),
+					'name'        => __('Process Rows Per Batch:', 'give'),
 					'type'        => 'number',
-					'description' => __( 'Determine how many rows you would like to import per cycle.', 'give' ),
+					'description' => __('Determine how many rows you would like to import per cycle.', 'give'),
 					'default'     => $per_page,
 					'class'       => 'give-text-small',
 				),
 			);
 
-			$settings = apply_filters( 'give_import_file_upload_html', $settings );
+			$settings = apply_filters('give_import_file_upload_html', $settings);
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -855,27 +852,24 @@ 
 			$step = $this->get_step();
 
 			// Validation for first step.
-			if ( 1 === $step ) {
-				$csv_id = absint( $_POST['csv_id'] );
+			if (1 === $step) {
+				$csv_id = absint($_POST['csv_id']);
 
-				if ( $this->is_valid_csv( $csv_id, esc_url( $_POST['csv'] ) ) ) {
+				if ($this->is_valid_csv($csv_id, esc_url($_POST['csv']))) {
 
-					$url = give_import_page_url( (array) apply_filters( 'give_import_step_two_url', array(
+					$url = give_import_page_url((array) apply_filters('give_import_step_two_url', array(
 						'step'          => '2',
 						'importer-type' => $this->importer_type,
 						'csv'           => $csv_id,
-						'delimiter'     => isset( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv',
-						'mode'          => empty( $_POST['mode'] ) ?
-							'0' :
-							( give_is_setting_enabled( give_clean( $_POST['mode'] ) ) ? '1' : '0' ),
-						'create_user'   => empty( $_POST['create_user'] ) ?
-							'0' :
-							( give_is_setting_enabled( give_clean( $_POST['create_user'] ) ) ? '1' : '0' ),
-						'delete_csv'    => empty( $_POST['delete_csv'] ) ?
-							'1' :
-							( give_is_setting_enabled( give_clean( $_POST['delete_csv'] ) ) ? '1' : '0' ),
-						'per_page'      => isset( $_POST['per_page'] ) ? absint( $_POST['per_page'] ) : self::$per_page,
-					) ) );
+						'delimiter'     => isset($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv',
+						'mode'          => empty($_POST['mode']) ?
+							'0' : (give_is_setting_enabled(give_clean($_POST['mode'])) ? '1' : '0'),
+						'create_user'   => empty($_POST['create_user']) ?
+							'0' : (give_is_setting_enabled(give_clean($_POST['create_user'])) ? '1' : '0'),
+						'delete_csv'    => empty($_POST['delete_csv']) ?
+							'1' : (give_is_setting_enabled(give_clean($_POST['delete_csv'])) ? '1' : '0'),
+						'per_page'      => isset($_POST['per_page']) ? absint($_POST['per_page']) : self::$per_page,
+					)));
 					?>
 					<script type="text/javascript">
 						window.location = "<?php echo $url; ?>"
@@ -896,25 +890,25 @@ 
 		 *
 		 * @return bool $has_error CSV is valid or not.
 		 */
-		private function is_valid_csv( $csv = false, $match_url = '' ) {
+		private function is_valid_csv($csv = false, $match_url = '') {
 			$is_valid_csv = true;
 
-			if ( $csv ) {
-				$csv_url = wp_get_attachment_url( $csv );
+			if ($csv) {
+				$csv_url = wp_get_attachment_url($csv);
 
-				$delimiter = ( ! empty( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv' );
+				$delimiter = ( ! empty($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv');
 
 				if (
 					! $csv_url ||
-					( ! empty( $match_url ) && ( $csv_url !== $match_url ) ) ||
-					( ( $mime_type = get_post_mime_type( $csv ) ) && ! strpos( $mime_type, $delimiter ) )
+					( ! empty($match_url) && ($csv_url !== $match_url)) ||
+					(($mime_type = get_post_mime_type($csv)) && ! strpos($mime_type, $delimiter))
 				) {
 					$is_valid_csv = false;
-					Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid CSV file.', 'give' ) );
+					Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid CSV file.', 'give'));
 				}
 			} else {
 				$is_valid_csv = false;
-				Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid CSV file.', 'give' ) );
+				Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid CSV file.', 'give'));
 			}
 
 			return $is_valid_csv;
@@ -930,8 +924,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_import_field( $field, $option_value ) {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-imports.php';
+		public function render_import_field($field, $option_value) {
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-imports.php';
 		}
 
 		/**
@@ -942,8 +936,8 @@ 
 		 */
 		private function is_donations_import_page() {
 			return 'import' === give_get_current_setting_tab() &&
-			       isset( $_GET['importer-type'] ) &&
-			       $this->importer_type === give_clean( $_GET['importer-type'] );
+			       isset($_GET['importer-type']) &&
+			       $this->importer_type === give_clean($_GET['importer-type']);
 		}
 	}