WordImpress / Give

includes/class-give-db-meta.php

Doc Comments +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
 	/**
 	 * Rename query clauses for new meta table
 	 *
-	 * @param $clause
+	 * @param string $clause
 	 * @param $filter
 	 *
 	 * @return mixed

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -75,40 +75,40 @@ 
 	 */
 	function __construct() {
 		// Bailout.
-		if ( empty( $this->supports ) || ! $this->is_custom_meta_table_active() ) {
+		if (empty($this->supports) || ! $this->is_custom_meta_table_active()) {
 			return;
 		}
 
-		if ( in_array( 'add_post_metadata', $this->supports ) ) {
-			add_filter( 'add_post_metadata', array( $this, '__add_meta' ), 0, 5 );
+		if (in_array('add_post_metadata', $this->supports)) {
+			add_filter('add_post_metadata', array($this, '__add_meta'), 0, 5);
 		}
 
-		if ( in_array( 'get_post_metadata', $this->supports ) ) {
-			add_filter( 'get_post_metadata', array( $this, '__get_meta' ), 0, 4 );
+		if (in_array('get_post_metadata', $this->supports)) {
+			add_filter('get_post_metadata', array($this, '__get_meta'), 0, 4);
 		}
 
-		if ( in_array( 'update_post_metadata', $this->supports ) ) {
-			add_filter( 'update_post_metadata', array( $this, '__update_meta' ), 0, 5 );
+		if (in_array('update_post_metadata', $this->supports)) {
+			add_filter('update_post_metadata', array($this, '__update_meta'), 0, 5);
 		}
 
-		if ( in_array( 'delete_post_metadata', $this->supports ) ) {
-			add_filter( 'delete_post_metadata', array( $this, '__delete_meta' ), 0, 5 );
+		if (in_array('delete_post_metadata', $this->supports)) {
+			add_filter('delete_post_metadata', array($this, '__delete_meta'), 0, 5);
 		}
 
-		if ( in_array( 'posts_where', $this->supports ) ) {
-			add_filter( 'posts_where', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_where', $this->supports)) {
+			add_filter('posts_where', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_join', $this->supports ) ) {
-			add_filter( 'posts_join', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_join', $this->supports)) {
+			add_filter('posts_join', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_groupby', $this->supports ) ) {
-			add_filter( 'posts_groupby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_groupby', $this->supports)) {
+			add_filter('posts_groupby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_orderby', $this->supports ) ) {
-			add_filter( 'posts_orderby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_orderby', $this->supports)) {
+			add_filter('posts_orderby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 	}
 
@@ -126,16 +126,16 @@ 
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single
 	 *                                is true.
 	 */
-	public function get_meta( $id = 0, $meta_key = '', $single = false ) {
-		$id = $this->sanitize_id( $id );
+	public function get_meta($id = 0, $meta_key = '', $single = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		if ( $this->raw_result ) {
-			if ( ! ( $value = get_metadata( $this->meta_type, $id, $meta_key, false ) ) ) {
+		if ($this->raw_result) {
+			if ( ! ($value = get_metadata($this->meta_type, $id, $meta_key, false))) {
 				$value = '';
 			}
 
@@ -143,7 +143,7 @@ 
 			$this->raw_result = false;
 
 		} else {
-			$value = get_metadata( $this->meta_type, $id, $meta_key, $single );
+			$value = get_metadata($this->meta_type, $id, $meta_key, $single);
 		}
 
 		return $value;
@@ -165,18 +165,18 @@ 
 	 *
 	 * @return  int|bool                  False for failure. True for success.
 	 */
-	public function add_meta( $id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$id = $this->sanitize_id( $id );
+	public function add_meta($id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = add_metadata( $this->meta_type, $id, $meta_key, $meta_value, $unique );
+		$meta_id = add_metadata($this->meta_type, $id, $meta_key, $meta_value, $unique);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -202,18 +202,18 @@ 
 	 *
 	 * @return  int|bool                  False on failure, true if success.
 	 */
-	public function update_meta( $id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function update_meta($id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = update_metadata( $this->meta_type, $id, $meta_key, $meta_value, $prev_value );
+		$meta_id = update_metadata($this->meta_type, $id, $meta_key, $meta_value, $prev_value);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -236,18 +236,18 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $id = 0, $meta_key = '', $meta_value = '', $delete_all = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function delete_meta($id = 0, $meta_key = '', $meta_value = '', $delete_all = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$is_meta_deleted = delete_metadata( $this->meta_type, $id, $meta_key, $meta_value, $delete_all );
+		$is_meta_deleted = delete_metadata($this->meta_type, $id, $meta_key, $meta_value, $delete_all);
 
-		if ( $is_meta_deleted ) {
-			$this->delete_cache( $id );
+		if ($is_meta_deleted) {
+			$this->delete_cache($id);
 		}
 
 		return $is_meta_deleted;
@@ -264,10 +264,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function __rename_meta_table_name_in_query( $clause, $wp_query ) {
+	public function __rename_meta_table_name_in_query($clause, $wp_query) {
 		// Add new table to sql query.
-		if ( $this->is_post_type_query( $wp_query ) && ! empty( $wp_query->meta_query->queries ) ) {
-			$clause = $this->__rename_meta_table_name( $clause, current_filter() );
+		if ($this->is_post_type_query($wp_query) && ! empty($wp_query->meta_query->queries)) {
+			$clause = $this->__rename_meta_table_name($clause, current_filter());
 		}
 
 		return $clause;
@@ -282,39 +282,39 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __rename_meta_table_name( $clause, $filter ){
+	public function __rename_meta_table_name($clause, $filter) {
 		global $wpdb;
 
-		$clause = str_replace( "{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause );
-		$clause = str_replace( $wpdb->postmeta, $this->table_name, $clause );
+		$clause = str_replace("{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause);
+		$clause = str_replace($wpdb->postmeta, $this->table_name, $clause);
 
-		switch( $filter ) {
+		switch ($filter) {
 			case 'posts_join':
-				$joins = array( 'INNER JOIN', 'LEFT JOIN' );
+				$joins = array('INNER JOIN', 'LEFT JOIN');
 
-				foreach ( $joins as $join ) {
-					if( false !== strpos( $clause, $join ) ) {
-						$clause = explode( $join, $clause );
+				foreach ($joins as $join) {
+					if (false !== strpos($clause, $join)) {
+						$clause = explode($join, $clause);
 
-						foreach ( $clause as $key => $clause_part ) {
-							if( empty( $clause_part ) ) {
+						foreach ($clause as $key => $clause_part) {
+							if (empty($clause_part)) {
 								continue;
 							}
 
-							preg_match( '/' . $wpdb->prefix . 'give_' . $this->meta_type . 'meta AS (.*) ON/', $clause_part, $alias_table_name );
+							preg_match('/'.$wpdb->prefix.'give_'.$this->meta_type.'meta AS (.*) ON/', $clause_part, $alias_table_name);
 
-							if( isset( $alias_table_name[1] ) ) {
-								$clause[$key] = str_replace( "{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part );
+							if (isset($alias_table_name[1])) {
+								$clause[$key] = str_replace("{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part);
 							}
 						}
 
-						$clause = implode( "{$join} ", $clause );
+						$clause = implode("{$join} ", $clause);
 					}
 				}
 				break;
 
 			case 'posts_where':
-				$clause = str_replace( array( 'mt2.post_id', 'mt1.post_id' ), array( "mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id" ), $clause );
+				$clause = str_replace(array('mt2.post_id', 'mt1.post_id'), array("mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id"), $clause);
 				break;
 		}
 
@@ -332,19 +332,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_post_type_query( $wp_query ) {
+	protected function is_post_type_query($wp_query) {
 		$status = false;
 
 		// Check if it is payment query.
-		if ( ! empty( $wp_query->query['post_type'] ) ) {
+		if ( ! empty($wp_query->query['post_type'])) {
 			if (
-				is_string( $wp_query->query['post_type'] ) &&
+				is_string($wp_query->query['post_type']) &&
 				$this->post_type === $wp_query->query['post_type']
 			) {
 				$status = true;
 			} elseif (
-				is_array( $wp_query->query['post_type'] ) &&
-				in_array( $this->post_type, $wp_query->query['post_type'] )
+				is_array($wp_query->query['post_type']) &&
+				in_array($this->post_type, $wp_query->query['post_type'])
 			) {
 				$status = true;
 			}
@@ -363,8 +363,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
-		return $ID && ( $this->post_type === get_post_type( $ID ) );
+	protected function is_valid_post_type($ID) {
+		return $ID && ($this->post_type === get_post_type($ID));
 	}
 
 	/**
@@ -390,8 +390,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_cache( $id, $meta_type = '' ) {
-		$meta_type = empty( $meta_type ) ? $this->meta_type : $meta_type;
+	private function delete_cache($id, $meta_type = '') {
+		$meta_type = empty($meta_type) ? $this->meta_type : $meta_type;
 
 		$group = array(
 			// 'form'    => 'give-forms',
@@ -401,8 +401,8 @@ 
 			// 'log'     => 'give-logs',
 		);
 
-		if ( array_key_exists( $meta_type, $group ) ) {
-			Give_Cache::delete_group( $id, $group[ $meta_type ] );
+		if (array_key_exists($meta_type, $group)) {
+			Give_Cache::delete_group($id, $group[$meta_type]);
 		}
 	}
 
@@ -417,8 +417,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
+	public function __call($name, $arguments) {
+		switch ($name) {
 			case '__add_meta':
 				$this->check = $arguments[0];
 				$id          = $arguments[1];
@@ -427,11 +427,11 @@ 
 				$unique      = $arguments[4];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->add_meta( $id, $meta_key, $meta_value, $unique );
+				return $this->add_meta($id, $meta_key, $meta_value, $unique);
 
 			case '__get_meta':
 				$this->check = $arguments[0];
@@ -440,13 +440,13 @@ 
 				$single      = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
 				$this->raw_result = true;
 
-				return $this->get_meta( $id, $meta_key, $single );
+				return $this->get_meta($id, $meta_key, $single);
 
 			case '__update_meta':
 				$this->check = $arguments[0];
@@ -455,11 +455,11 @@ 
 				$meta_value  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->update_meta( $id, $meta_key, $meta_value );
+				return $this->update_meta($id, $meta_key, $meta_value);
 
 			case '__delete_meta':
 				$this->check = $arguments[0];
@@ -469,11 +469,11 @@ 
 				$delete_all  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->delete_meta( $id, $meta_key, $meta_value, $delete_all );
+				return $this->delete_meta($id, $meta_key, $meta_value, $delete_all);
 		}
 	}
 }
\ No newline at end of file

includes/payments/class-payments-query.php

Spacing +149 added lines, -149 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -66,10 +66,10 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments',
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -96,7 +96,7 @@ 
 			'count'           => false,
 		);
 
-		$this->args = $this->_args = wp_parse_args( $args, $defaults );
+		$this->args = $this->_args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -110,11 +110,11 @@ 
 	 * @param $query_var
 	 * @param $value
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -126,8 +126,8 @@ 
 	 *
 	 * @param $query_var
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 		$this->give_forms();
 		$this->gateway_filter();
 
-		add_filter( 'posts_orderby', array( $this, 'custom_orderby' ), 10, 2 );
+		add_filter('posts_orderby', array($this, 'custom_orderby'), 10, 2);
 
 		/**
 		 * Fires after setup filters.
@@ -176,7 +176,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @access private
 	 */
 	private function unset_filters() {
-		remove_filter( 'posts_orderby', array( $this, 'custom_orderby' ) );
+		remove_filter('posts_orderby', array($this, 'custom_orderby'));
 
 		/**
 		 * Fires after retrieving payments.
@@ -195,7 +195,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 	}
 
 
@@ -212,11 +212,11 @@ 
 	 * @return array
 	 */
 	public function get_payments() {
-		$cache_key      = Give_Cache::get_key( 'give_payment_query', $this->args, false );
-		$this->payments = Give_Cache::get_db_query( $cache_key );
+		$cache_key      = Give_Cache::get_key('give_payment_query', $this->args, false);
+		$this->payments = Give_Cache::get_db_query($cache_key);
 
 		// Return cached result.
-		if ( ! is_null( $this->payments ) ) {
+		if ( ! is_null($this->payments)) {
 			return $this->payments;
 		}
 
@@ -224,7 +224,7 @@ 
 		// Modify the query/query arguments before we retrieve payments.
 		$this->set_filters();
 
-		$query          = new WP_Query( $this->args );
+		$query          = new WP_Query($this->args);
 		$this->payments = array();
 
 		$custom_output = array(
@@ -232,24 +232,24 @@ 
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
-			while ( $query->have_posts() ) {
+		if ($query->have_posts()) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 		}
 
-		Give_Cache::set_db_query( $cache_key, $this->payments );
+		Give_Cache::set_db_query($cache_key, $this->payments);
 
 		// Remove query filters after we retrieve payments.
 		$this->unset_filters();
@@ -268,31 +268,31 @@ 
 	public function get_payment_by_group() {
 		global $wpdb;
 
-		$allowed_groups = array( 'post_status' );
+		$allowed_groups = array('post_status');
 		$result         = array();
 
 
-		if ( in_array( $this->args['group_by'], $allowed_groups ) ) {
+		if (in_array($this->args['group_by'], $allowed_groups)) {
 			// Set only count in result.
-			if ( $this->args['count'] ) {
+			if ($this->args['count']) {
 
 				$this->set_filters();
 
-				$new_results = $wpdb->get_results( $this->get_sql(), ARRAY_N );
+				$new_results = $wpdb->get_results($this->get_sql(), ARRAY_N);
 
 				$this->unset_filters();
 
-				foreach ( $new_results as $results ) {
-					$result[ $results[0] ] = $results[1];
+				foreach ($new_results as $results) {
+					$result[$results[0]] = $results[1];
 				}
 
-				switch ( $this->args['group_by'] ) {
+				switch ($this->args['group_by']) {
 					case 'post_status':
 
 						/* @var Give_Payment $donation */
-						foreach ( give_get_payment_status_keys() as $status ) {
-							if ( ! isset( $result[ $status ] ) ) {
-								$result[ $status ] = 0;
+						foreach (give_get_payment_status_keys() as $status) {
+							if ( ! isset($result[$status])) {
+								$result[$status] = 0;
 							}
 						}
 
@@ -302,8 +302,8 @@ 
 				$donations = $this->get_payments();
 
 				/* @var $donation Give_Payment */
-				foreach ( $donations as $donation ) {
-					$result[ $donation->{$this->args['group_by']} ][] = $donation;
+				foreach ($donations as $donation) {
+					$result[$donation->{$this->args['group_by']}][] = $donation;
 				}
 			}
 		}
@@ -314,7 +314,7 @@ 
 		 *
 		 * @since 1.8.17
 		 */
-		return apply_filters( 'give_get_payment_by_group', $result, $this );
+		return apply_filters('give_get_payment_by_group', $result, $this);
 	}
 
 	/**
@@ -326,29 +326,29 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
-		$is_start_date = property_exists( __CLASS__, 'start_date' );
-		$is_end_date   = property_exists( __CLASS__, 'end_date' );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
+		$is_start_date = property_exists(__CLASS__, 'start_date');
+		$is_end_date   = property_exists(__CLASS__, 'end_date');
 
-		if ( $is_start_date || $is_end_date ) {
+		if ($is_start_date || $is_end_date) {
 			$date_query = array();
 
-			if ( $is_start_date && ! is_wp_error( $this->start_date ) ) {
-				$date_query['after'] = date( 'Y-m-d H:i:s', $this->start_date );
+			if ($is_start_date && ! is_wp_error($this->start_date)) {
+				$date_query['after'] = date('Y-m-d H:i:s', $this->start_date);
 			}
 
-			if ( $is_end_date && ! is_wp_error( $this->end_date ) ) {
-				$date_query['before'] = date( 'Y-m-d H:i:s', $this->end_date );
+			if ($is_end_date && ! is_wp_error($this->end_date)) {
+				$date_query['before'] = date('Y-m-d H:i:s', $this->end_date);
 			}
 
 			// Include Start Date and End Date while querying.
 			$date_query['inclusive'] = true;
 
-			$this->__set( 'date_query', $date_query );
+			$this->__set('date_query', $date_query);
 
 		}
 
@@ -363,12 +363,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset( $this->args['status'] ) ) {
+		if ( ! isset($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -380,12 +380,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset( $this->args['page'] ) ) {
+		if ( ! isset($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -398,17 +398,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -420,12 +420,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset( $this->args['month'] ) ) {
+		if ( ! isset($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -437,23 +437,23 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount':
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 
 			case 'status':
-				$this->__set( 'orderby', 'post_status' );
+				$this->__set('orderby', 'post_status');
 				break;
 
 			case 'donation_form':
-				$this->__set( 'orderby', 'meta_value' );
-				$this->__set( 'meta_key', '_give_payment_form_title' );
+				$this->__set('orderby', 'meta_value');
+				$this->__set('meta_key', '_give_payment_form_title');
 				break;
 
 			default:
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -470,19 +470,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function custom_orderby( $order, $query ) {
+	public function custom_orderby($order, $query) {
 
-		if ( ! empty( $query->query['post_type'] ) ) {
-			$post_types = is_array( $query->query['post_type'] ) ? $query->query['post_type'] : array( $query->query['post_type'] );
+		if ( ! empty($query->query['post_type'])) {
+			$post_types = is_array($query->query['post_type']) ? $query->query['post_type'] : array($query->query['post_type']);
 
-			if ( ! in_array( 'give_payment', $post_types ) || is_array( $query->query['orderby'] ) ) {
+			if ( ! in_array('give_payment', $post_types) || is_array($query->query['orderby'])) {
 				return $order;
 			}
 
 			global $wpdb;
-			switch ( $query->query['orderby'] ) {
+			switch ($query->query['orderby']) {
 				case 'post_status':
-					$order = $wpdb->posts . '.post_status ' . strtoupper( $query->query['order'] );
+					$order = $wpdb->posts.'.post_status '.strtoupper($query->query['order']);
 					break;
 			}
 		}
@@ -499,11 +499,11 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_donor_id';
 		} else {
 			$user_key = '_give_payment_donor_email';
@@ -525,16 +525,16 @@ 
 	 * @return  void
 	 */
 	public function donor() {
-		if ( is_null( $this->args['donor'] ) || ! is_numeric( $this->args['donor'] ) ) {
+		if (is_null($this->args['donor']) || ! is_numeric($this->args['donor'])) {
 			return;
 		}
 
 		$donor_meta_type = Give()->donor_meta->meta_type;
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => "_give_payment_{$donor_meta_type}_id",
 			'value' => (int) $this->args['donor'],
-		) );
+		));
 	}
 
 	/**
@@ -547,33 +547,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_donor_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -582,19 +582,19 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_donor_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) ),
+				'value' => trim(str_replace('user:', '', strtolower($search))),
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			if ( give_get_option( 'enable_sequential' ) ) {
+			if (give_get_option('enable_sequential')) {
 
 				$search_meta = array(
 					'key'     => '_give_payment_number',
@@ -602,19 +602,19 @@ 
 					'compare' => 'LIKE',
 				);
 
-				$this->__set( 'meta_query', $search_meta );
+				$this->__set('meta_query', $search_meta);
 
 				$this->args['meta_query']['relation'] = 'OR';
 
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
 		} elseif (
-			give_get_option( 'enable_sequential' ) &&
+			give_get_option('enable_sequential') &&
 			(
-				false !== strpos( $search, give_get_option( 'sequential_prefix' ) ) ||
-				false !== strpos( $search, give_get_option( 'sequential_postfix' ) )
+				false !== strpos($search, give_get_option('sequential_prefix')) ||
+				false !== strpos($search, give_get_option('sequential_postfix'))
 			)
 		) {
 
@@ -624,29 +624,29 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -661,8 +661,8 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
@@ -684,10 +684,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -700,13 +700,13 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['give_forms'] ) ) {
+		if (is_array($this->args['give_forms'])) {
 			$compare = 'IN';
 		}
 
@@ -719,7 +719,7 @@ 
 			)
 		);
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 
@@ -733,13 +733,13 @@ 
 	 */
 	public function gateway_filter() {
 
-		if ( empty( $this->args['gateway'] ) ) {
+		if (empty($this->args['gateway'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['gateway'] ) ) {
+		if (is_array($this->args['gateway'])) {
 			$compare = 'IN';
 		}
 
@@ -753,7 +753,7 @@ 
 			)
 		);
 
-		$this->__unset( 'gateway' );
+		$this->__unset('gateway');
 
 	}
 
@@ -773,9 +773,9 @@ 
 		global $wpdb;
 
 		$where = "WHERE {$wpdb->posts}.post_type = 'give_payment'";
-		$where .= " AND {$wpdb->posts}.post_status IN ('" . implode( "','", $this->args['post_status'] ) . "')";
+		$where .= " AND {$wpdb->posts}.post_status IN ('".implode("','", $this->args['post_status'])."')";
 
-		if( is_numeric( $this->args['post_parent'] ) ) {
+		if (is_numeric($this->args['post_parent'])) {
 			$where .= " AND {$wpdb->posts}.post_parent={$this->args['post_parent']}";
 		}
 
@@ -784,42 +784,42 @@ 
 		$group_by = '';
 
 		// Set group by.
-		if ( ! empty( $this->args['group_by'] ) ) {
+		if ( ! empty($this->args['group_by'])) {
 			$group_by = "GROUP BY {$wpdb->posts}.{$this->args['group_by']}";
 		}
 
 		// Set offset.
 		if (
-			empty( $this->args['nopaging'] ) &&
-			empty( $this->args['offset'] ) &&
-			( ! empty( $this->args['page'] ) && 0 < $this->args['page'] )
+			empty($this->args['nopaging']) &&
+			empty($this->args['offset']) &&
+			( ! empty($this->args['page']) && 0 < $this->args['page'])
 		) {
-			$this->args['offset'] = $this->args['posts_per_page'] * ( $this->args['page'] - 1 );
+			$this->args['offset'] = $this->args['posts_per_page'] * ($this->args['page'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$wpdb->posts}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$wpdb->posts}.{$this->args['fields']}";
-			} elseif ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$wpdb->posts}." . implode( " , {$wpdb->posts}.", $this->args['fields'] );
+			} elseif (is_array($this->args['fields'])) {
+				$fields = "{$wpdb->posts}.".implode(" , {$wpdb->posts}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$wpdb->posts}.ID)";
 
-			if ( ! empty( $this->args['group_by'] ) ) {
+			if ( ! empty($this->args['group_by'])) {
 				$fields = "{$wpdb->posts}.{$this->args['group_by']}, {$fields}";
 			}
 		}
 
 		// Date query.
-		if ( ! empty( $this->args['date_query'] ) ) {
-			$date_query_obj = new WP_Date_Query( $this->args['date_query'] );
-			$where          .= str_replace(
+		if ( ! empty($this->args['date_query'])) {
+			$date_query_obj = new WP_Date_Query($this->args['date_query']);
+			$where .= str_replace(
 				array(
 					"\n",
 					'(   (',
@@ -835,22 +835,22 @@ 
 		}
 
 		// Meta query.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_obj = new WP_Meta_Query( $this->args['meta_query'] );
-			$where          = implode( ' ', $meta_query_obj->get_sql( 'post', $wpdb->posts, 'ID' ) ) . " {$where}";
-			$where          = Give()->payment_meta->__rename_meta_table_name( $where, 'posts_where' );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_obj = new WP_Meta_Query($this->args['meta_query']);
+			$where          = implode(' ', $meta_query_obj->get_sql('post', $wpdb->posts, 'ID'))." {$where}";
+			$where          = Give()->payment_meta->__rename_meta_table_name($where, 'posts_where');
 		}
 
 		// Set sql query.
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$wpdb->posts} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			( empty( $this->args['nopaging'] ) ? absint( $this->args['posts_per_page'] ) : 99999999999 )
+			absint($this->args['offset']),
+			(empty($this->args['nopaging']) ? absint($this->args['posts_per_page']) : 99999999999)
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}

includes/admin/donors/donors.php

Spacing +213 added lines, -213 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  *
  * @return string
  */
-function __give_get_format_address( $address, $address_args = array() ) {
+function __give_get_format_address($address, $address_args = array()) {
 	$address_html = '';
 	$address_args = wp_parse_args(
 		$address_args,
@@ -40,7 +40,7 @@ 
 	$address_id = $address_args['type'];
 
 	// Bailout.
-	if ( empty( $address ) || ! is_array( $address ) ) {
+	if (empty($address) || ! is_array($address)) {
 		return $address_html;
 	}
 
@@ -49,19 +49,19 @@ 
 	$address_html .= sprintf(
 		'<span data-address-type="line1">%1$s</span>%2$s',
 		$address['line1'],
-		( ! empty( $address['line2'] ) ? '<br>' : '' )
+		( ! empty($address['line2']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="line2">%1$s</span>%2$s',
 		$address['line2'],
-		( ! empty( $address['city'] ) ? '<br>' : '' )
+		( ! empty($address['city']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="city">%1$s</span><span data-address-type="state">%2$s</span><span data-address-type="zip">%3$s</span>%4$s',
 		$address['city'],
-		( ! empty( $address['state'] ) ? ", {$address['state']}" : '' ),
-		( ! empty( $address['zip'] ) ? " {$address['zip']}" : '' ),
-		( ! empty( $address['country'] ) ? '<br>' : '' )
+		( ! empty($address['state']) ? ", {$address['state']}" : ''),
+		( ! empty($address['zip']) ? " {$address['zip']}" : ''),
+		( ! empty($address['country']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="country">%s</span><br>',
@@ -71,8 +71,8 @@ 
 	// Address action.
 	$address_html .= sprintf(
 		'<br><a href="#" class="js-edit">%1$s</a> | <a href="#" class="js-remove">%2$s</a>',
-		__( 'Edit', 'give' ),
-		__( 'Remove', 'give' )
+		__('Edit', 'give'),
+		__('Remove', 'give')
 	);
 
 	/**
@@ -80,14 +80,14 @@ 
 	 *
 	 * @since 2.0
 	 */
-	$address_label = apply_filters( "give_donor_{$address_args['type']}_address_label", ucfirst( $address_args['type'] ), $address_args );
+	$address_label = apply_filters("give_donor_{$address_args['type']}_address_label", ucfirst($address_args['type']), $address_args);
 
 	// Set unique id and index for multi type address.
-	if ( isset( $address_args['index'] ) ) {
+	if (isset($address_args['index'])) {
 		$address_label = "{$address_label} #{$address_args['index']}";
 	}
 
-	if ( isset( $address_args['id'] ) ) {
+	if (isset($address_args['id'])) {
 		$address_id = "{$address_id}_{$address_args['id']}";
 	}
 
@@ -112,9 +112,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -130,7 +130,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -144,7 +144,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -156,7 +156,7 @@ 
  */
 function give_donors_list() {
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/donors/class-donor-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/donors/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -169,12 +169,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
 
 		<hr class="wp-header-end">
-		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
-			<?php $donors_table->search_box( __( 'Search Donors', 'give' ), 'give-donors' ); ?>
+		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
+			<?php $donors_table->search_box(__('Search Donors', 'give'), 'give-donors'); ?>
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-donors" />
 			<input type="hidden" name="view" value="donors" />
@@ -191,7 +191,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -207,33 +207,33 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id          = (int) $_GET['id'];
-	$reconnect_user_id = ! empty( $_GET['user_id'] ) ? (int) $_GET['user_id'] : '';
-	$donor             = new Give_Donor( $donor_id );
+	$reconnect_user_id = ! empty($_GET['user_id']) ? (int) $_GET['user_id'] : '';
+	$donor             = new Give_Donor($donor_id);
 
 	// Reconnect User with Donor profile.
-	if ( $reconnect_user_id ) {
-		give_connect_user_donor_profile( $donor, array( 'user_id' => $reconnect_user_id ), array() );
+	if ($reconnect_user_id) {
+		give_connect_user_donor_profile($donor, array('user_id' => $reconnect_user_id), array());
 	}
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -242,9 +242,9 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 
@@ -252,7 +252,7 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor number */
-				esc_html__( 'Donor %s', 'give' ),
+				esc_html__('Donor %s', 'give'),
 				$donor_id
 			);
 			?>
@@ -260,26 +260,26 @@ 
 
 		<hr class="wp-header-end">
 
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 				<?php
-				foreach ( $donor_tabs as $key => $tab ) :
+				foreach ($donor_tabs as $key => $tab) :
 					$active = $key === $view ? true : false;
 					$class  = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 					printf(
-						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-						esc_attr( $class ),
-						sanitize_html_class( $tab['dashicon'] ),
-						esc_html( $tab['title'] )
+						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+						esc_attr($class),
+						sanitize_html_class($tab['dashicon']),
+						esc_html($tab['title'])
 					);
 				endforeach;
 				?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -299,9 +299,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -310,11 +310,11 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 
 	// Set Read only to the fields which needs to be locked.
 	$read_only = '';
-	if ( $donor->user_id ) {
+	if ($donor->user_id) {
 		$read_only = 'readonly="readonly"';
 	}
 	?>
@@ -322,14 +322,14 @@ 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
 		<form id="edit-donor-info" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
@@ -337,18 +337,18 @@ 
 						<span class="donor-name info-item edit-item">
 							<input <?php echo $read_only; ?> size="15" data-key="first_name"
 							                                 name="customerinfo[first_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_first_name() ); ?>"
-							                                 placeholder="<?php _e( 'First Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_first_name()); ?>"
+							                                 placeholder="<?php _e('First Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
 							<?php endif; ?>
 							<input <?php echo $read_only; ?> size="15" data-key="last_name"
 							                                 name="customerinfo[last_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_last_name() ); ?>"
-							                                 placeholder="<?php _e( 'Last Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_last_name()); ?>"
+							                                 placeholder="<?php _e('Last Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
@@ -358,12 +358,12 @@ 
 									data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php _e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php _e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
+					<?php if (current_user_can($donor_edit_role)) : ?>
 						<a href="#" id="edit-donor"
-						   class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+						   class="button info-item editable donor-edit-link"><?php _e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -373,7 +373,7 @@ 
 					<table class="widefat">
 						<tbody>
 						<tr class="alternate">
-							<th scope="col"><label for="tablecell"><?php _e( 'User:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('User:', 'give'); ?></label></th>
 							<td>
 									<span class="donor-user-id info-item edit-item">
 										<?php
@@ -390,22 +390,22 @@ 
 											'data'  => $data_atts,
 										);
 
-										if ( ! empty( $user_id ) ) {
-											$userdata              = get_userdata( $user_id );
+										if ( ! empty($user_id)) {
+											$userdata              = get_userdata($user_id);
 											$user_args['selected'] = $user_id;
 										}
 
-										echo Give()->html->ajax_user_search( $user_args );
+										echo Give()->html->ajax_user_search($user_args);
 										?>
 									</span>
 
 								<span class="donor-user-id info-item editable">
-										<?php if ( ! empty( $userdata ) ) : ?>
-											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+										<?php if ( ! empty($userdata)) : ?>
+											<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'None', 'give' ); ?></span>
+											<span data-key="user_id"><?php _e('None', 'give'); ?></span>
 										<?php endif; ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0):
 
 										echo sprintf(
 											'- <span class="disconnect-user">
@@ -414,11 +414,11 @@ 
 				                                       | <span class="view-user-profile">
  											                <a id="view-user-profile" href="%3$s" aria-label="%4$s">%5$s</a>
  										               </span>',
-											__( 'Disconnects the current user ID from this donor record.', 'give' ),
-											__( 'Disconnect User', 'give' ),
-											'user-edit.php?user_id=' . $donor->user_id,
-											__( 'View User Profile of current user ID.', 'give' ),
-											__( 'View User Profile', 'give' )
+											__('Disconnects the current user ID from this donor record.', 'give'),
+											__('Disconnect User', 'give'),
+											'user-edit.php?user_id='.$donor->user_id,
+											__('View User Profile of current user ID.', 'give'),
+											__('View User Profile', 'give')
 										);
 
 										endif; ?>
@@ -434,11 +434,11 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>"/>
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor"/>
 				<input type="submit" id="give-edit-donor-save" class="button-secondary"
-				       value="<?php _e( 'Update Donor', 'give' ); ?>"/>
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+				       value="<?php _e('Update Donor', 'give'); ?>"/>
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -453,24 +453,24 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&status=publish&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&status=publish&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations.
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->get_total_donation_amount(), array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->get_total_donation_amount(), array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -482,7 +482,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -495,11 +495,11 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_address', $donor );
+	do_action('give_donor_before_address', $donor);
 	?>
 
 	<div id="donor-address-wrapper" class="donor-section clear">
-		<h3><?php _e( 'Addresses', 'give' ); ?></h3>
+		<h3><?php _e('Addresses', 'give'); ?></h3>
 
 		<div class="postbox">
 			<div class="give-spinner-wrapper">
@@ -509,16 +509,16 @@ 
 				<div class="all-address">
 					<div class="give-grid-row">
 						<?php
-						if ( ! empty( $donor->address ) ) :
+						if ( ! empty($donor->address)) :
 							// Default address always will be at zero array index.
 							$is_set_as_default = null;
 
-							foreach ( $donor->address as $address_type => $addresses ) {
+							foreach ($donor->address as $address_type => $addresses) {
 
-								switch ( true ) {
-									case is_array( end( $addresses ) ):
+								switch (true) {
+									case is_array(end($addresses)):
 										$index = 1;
-										foreach ( $addresses as $id => $address ) {
+										foreach ($addresses as $id => $address) {
 											echo __give_get_format_address(
 												$address,
 												array(
@@ -528,11 +528,11 @@ 
 												)
 											);
 
-											$index ++;
+											$index++;
 										}
 										break;
 
-									case is_string( end( $addresses ) ):
+									case is_string(end($addresses)):
 										echo __give_get_format_address(
 											$addresses,
 											array(
@@ -545,13 +545,13 @@ 
 						endif;
 						?>
 					</div>
-					<span class="give-no-address-message<?php if ( ! empty( $donor->address ) ) {
+					<span class="give-no-address-message<?php if ( ! empty($donor->address)) {
 						echo ' give-hidden';
 					} ?>">
-						<?php _e( 'This donor does not have any addresses saved.', 'give' ); ?>
+						<?php _e('This donor does not have any addresses saved.', 'give'); ?>
 					</span>
 					<button class="button add-new-address">
-						<?php _e( 'Add Address', 'give' ); ?>
+						<?php _e('Add Address', 'give'); ?>
 					</button>
 				</div>
 
@@ -561,26 +561,26 @@ 
 							<tbody>
 							<tr>
 								<th class="col">
-									<label class="country"><?php esc_html_e( 'Country:', 'give' ); ?></label>
+									<label class="country"><?php esc_html_e('Country:', 'give'); ?></label>
 								</th>
 								<td>
 									<?php
-									echo Give()->html->select( array(
+									echo Give()->html->select(array(
 										'options'          => give_get_country_list(),
 										'name'             => 'country',
-										'selected'         => give_get_option( 'base_country' ),
+										'selected'         => give_get_option('base_country'),
 										'show_option_all'  => false,
 										'show_option_none' => false,
 										'chosen'           => true,
-										'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-										'data'             => array( 'search-type' => 'no_ajax' ),
-									) );
+										'placeholder'      => esc_attr__('Select a country', 'give'),
+										'data'             => array('search-type' => 'no_ajax'),
+									));
 									?>
 								</td>
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line1"><?php esc_html_e( 'Address 1:', 'give' ); ?></label>
+									<label for="line1"><?php esc_html_e('Address 1:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line1" name="line1" type="text" class="medium-text"/>
@@ -588,7 +588,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line2"><?php esc_html_e( 'Address 2:', 'give' ); ?></label>
+									<label for="line2"><?php esc_html_e('Address 2:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line2" type="text" name="line2" value="" class="medium-text"/>
@@ -597,7 +597,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="city"><?php esc_html_e( 'City:', 'give' ); ?></label>
+									<label for="city"><?php esc_html_e('City:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="city" type="text" name="city" value="" class="medium-text"/>
@@ -605,41 +605,41 @@ 
 							</tr>
 							<?php
 							$no_states_country = give_no_states_country_list();
-							$base_country      = give_get_option( 'base_country' );
-							if ( ! array_key_exists( $base_country, $no_states_country ) ) {
+							$base_country      = give_get_option('base_country');
+							if ( ! array_key_exists($base_country, $no_states_country)) {
 								?>
 								<tr class="give-field-wrap">
 									<th class="col">
-										<label for="state"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+										<label for="state"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 									</th>
 									<td>
 										<?php
-										$states     = give_get_states( $base_country );
+										$states     = give_get_states($base_country);
 										$state_args = array(
 											'name'  => 'state',
 											'class' => 'regular-text',
 										);
 
-										if ( empty( $states ) ) {
+										if (empty($states)) {
 
 											// Show Text field, if empty states.
-											$state_args = wp_parse_args( $state_args, array(
-												'value' => give_get_option( 'base_state' ),
-											) );
-											echo Give()->html->text( $state_args );
+											$state_args = wp_parse_args($state_args, array(
+												'value' => give_get_option('base_state'),
+											));
+											echo Give()->html->text($state_args);
 										} else {
 
 											// Show Chosen DropDown, if states are not empty.
-											$state_args = wp_parse_args( $state_args, array(
+											$state_args = wp_parse_args($state_args, array(
 												'options'          => $states,
-												'selected'         => give_get_option( 'base_state' ),
+												'selected'         => give_get_option('base_state'),
 												'show_option_all'  => false,
 												'show_option_none' => false,
 												'chosen'           => true,
-												'placeholder'      => __( 'Select a state', 'give' ),
-												'data'             => array( 'search-type' => 'no_ajax' ),
-											) );
-											echo Give()->html->select( $state_args );
+												'placeholder'      => __('Select a state', 'give'),
+												'data'             => array('search-type' => 'no_ajax'),
+											));
+											echo Give()->html->select($state_args);
 										}
 										?>
 									</td>
@@ -649,7 +649,7 @@ 
 							?>
 							<tr>
 								<th class="col">
-									<label for="zip"><?php esc_html_e( 'Zip / Postal Code:', 'give' ); ?></label>
+									<label for="zip"><?php esc_html_e('Zip / Postal Code:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="zip" type="text" name="zip" value="" class="medium-text"/>
@@ -657,12 +657,12 @@ 
 							</tr>
 							<tr>
 								<td colspan="2">
-									<?php wp_nonce_field( 'give-manage-donor-addresses', '_wpnonce', false ); ?>
+									<?php wp_nonce_field('give-manage-donor-addresses', '_wpnonce', false); ?>
 									<input type="hidden" name="address-action" value="add">
 									<input type="hidden" name="address-id" value="">
 									<input type="submit" class="button button-primary js-save"
-									       value="<?php _e( 'Save', 'give' ); ?>">&nbsp;&nbsp;<button
-											class="button js-cancel"><?php _e( 'Cancel', 'give' ); ?></button>
+									       value="<?php _e('Save', 'give'); ?>">&nbsp;&nbsp;<button
+											class="button js-cancel"><?php _e('Cancel', 'give'); ?></button>
 								</td>
 							</tr>
 							</tbody>
@@ -681,7 +681,7 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -694,46 +694,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 			<tr>
-				<th><?php _e( 'Email', 'give' ); ?></th>
-				<th><?php _e( 'Actions', 'give' ); ?></th>
+				<th><?php _e('Email', 'give'); ?></th>
+				<th><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 
 			<tbody>
-			<?php if ( ! empty( $donor->emails ) ) { ?>
+			<?php if ( ! empty($donor->emails)) { ?>
 
-				<?php foreach ( $donor->emails as $key => $email ) : ?>
+				<?php foreach ($donor->emails as $key => $email) : ?>
 					<tr data-key="<?php echo $key; ?>">
 						<td>
 							<?php echo $email; ?>
-							<?php if ( 'primary' === $key ) : ?>
+							<?php if ('primary' === $key) : ?>
 								<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 							<?php endif; ?>
 						</td>
 						<td>
-							<?php if ( 'primary' !== $key ) : ?>
+							<?php if ('primary' !== $key) : ?>
 								<?php
-								$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-								$promote_url = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+								$promote_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'set_donor_primary_email',
-								), $base_url ), 'give-set-donor-primary-email' );
-								$remove_url  = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								), $base_url), 'give-set-donor-primary-email');
+								$remove_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'remove_donor_email',
-								), $base_url ), 'give-remove-donor-email' );
+								), $base_url), 'give-remove-donor-email');
 								?>
-								<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+								<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 								&nbsp;|&nbsp;
-								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 							<?php endif; ?>
 						</td>
 					</tr>
@@ -743,14 +743,14 @@ 
 					<td colspan="2" class="add-donor-email-td">
 						<div class="add-donor-email-wrapper">
 							<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>"/>
-							<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
+							<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
 							<input type="email" name="additional-email" value=""
-							       placeholder="<?php _e( 'Email Address', 'give' ); ?>"/>&nbsp;
+							       placeholder="<?php _e('Email Address', 'give'); ?>"/>&nbsp;
 							<input type="checkbox" name="make-additional-primary" value="1"
 							       id="make-additional-primary"/>&nbsp;<label
-									for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
+									for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
 							<button class="button-secondary give-add-donor-email"
-							        id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+							        id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 							<span class="spinner"></span>
 						</div>
 						<div class="notice-wrap"></div>
@@ -758,50 +758,50 @@ 
 				</tr>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No Emails Found', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php _e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('ID', 'give'); ?></th>
+				<th scope="col"><?php _e('Amount', 'give'); ?></th>
+				<th scope="col"><?php _e('Date', 'give'); ?></th>
+				<th scope="col"><?php _e('Status', 'give'); ?></th>
+				<th scope="col"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
 						<td><?php echo $payment->ID; ?></td>
-						<td><?php echo give_donation_amount( $payment->ID, array( 'currency' => true, 'amount' => true, 'type' => 'donor' ) ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo give_donation_amount($payment->ID, array('currency' => true, 'amount' => true, 'type' => 'donor')); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 								/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								__( 'View Donation', 'give' )
+								__('View Donation', 'give')
 							);
 							?>
 
@@ -816,47 +816,47 @@ 
 							 * @param object $donor   The donor object being displayed.
 							 * @param object $payment The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php _e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php _e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php _e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 								/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								__( 'View Form', 'give' )
+								__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -864,7 +864,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -878,7 +878,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -891,7 +891,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -904,31 +904,31 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged       = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged       = absint( $paged );
+	$paged       = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged       = absint($paged);
 	$note_count  = $donor->get_notes_count();
-	$per_page    = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page    = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php _e( 'Notes', 'give' ); ?></h3>
+		<h3><?php _e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
 				<form id="give-add-donor-note" method="post"
-				      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br/>
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>"/>
 					<input type="hidden" name="give_action" value="add-donor-note"/>
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note"/>
 				</form>
 			</div>
@@ -943,26 +943,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php _e( 'No donor notes found.', 'give' ); ?>
+					<?php _e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -978,9 +978,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -989,16 +989,16 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
 		<form id="delete-donor" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -1006,20 +1006,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php _e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php _e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php _e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php _e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -1032,19 +1032,19 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>"/>
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor"/>
 					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary"
-					       value="<?php _e( 'Delete Donor', 'give' ); ?>"/>
+					       value="<?php _e('Delete Donor', 'give'); ?>"/>
 					<a id="give-delete-donor-cancel"
-					   href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>"
-					   class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+					   href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>"
+					   class="delete"><?php _e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -1060,5 +1060,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

includes/admin/donors/class-donor-table.php

Spacing +91 added lines, -91 removed lines

@@ -12,13 +12,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => __( 'Donor', 'give' ), // Singular name of the listed records.
-			'plural'   => __( 'Donors', 'give' ), // Plural name of the listed records.
+		parent::__construct(array(
+			'singular' => __('Donor', 'give'), // Singular name of the listed records.
+			'plural'   => __('Donors', 'give'), // Plural name of the listed records.
 			'ajax'     => false, // Does this table support ajax?.
-		) );
+		));
 
 	}
 
@@ -80,23 +80,23 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo sprintf( '<input type="hidden" name="orderby" value="%1$s" />', esc_attr( $_REQUEST['orderby'] ) );
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo sprintf('<input type="hidden" name="orderby" value="%1$s" />', esc_attr($_REQUEST['orderby']));
 		}
 
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo sprintf( '<input type="hidden" name="order" value="%1$s" />', esc_attr( $_REQUEST['order'] ) );
+		if ( ! empty($_REQUEST['order'])) {
+			echo sprintf('<input type="hidden" name="order" value="%1$s" />', esc_attr($_REQUEST['order']));
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?>
+			)); ?>
 		</p>
 		<?php
 	}
@@ -112,32 +112,32 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
+	public function column_default($donor, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&status=publish&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&status=publish&donor='.absint($donor['id'])),
+					esc_html($donor['num_donations'])
 				);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $donor[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($donor[$column_name], array('sanitize' => false)));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $donor['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($donor['date_created']));
 				break;
 
 			default:
-				$value = isset( $donor[ $column_name ] ) ? $donor[ $column_name ] : null;
+				$value = isset($donor[$column_name]) ? $donor[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_donors_column_{$column_name}", $value, $donor['id'] );
+		return apply_filters("give_donors_column_{$column_name}", $value, $donor['id']);
 
 	}
 
@@ -151,7 +151,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_cb( $donor ){
+	public function column_cb($donor) {
 		return sprintf(
 			'<input class="donor-selector" type="checkbox" name="%1$s[]" value="%2$d" data-name="%3$s" />',
 			$this->_args['singular'],
@@ -170,13 +170,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = '#' . $donor['id'] . ' ';
-		$name     .= ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . __( 'Unnamed Donor', 'give' ) . '</em>';
-		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] );
-		$actions  = $this->get_row_actions( $donor );
+	public function column_name($donor) {
+		$name     = '#'.$donor['id'].' ';
+		$name .= ! empty($donor['name']) ? $donor['name'] : '<em>'.__('Unnamed Donor', 'give').'</em>';
+		$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']);
+		$actions  = $this->get_row_actions($donor);
 
-		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
+		return '<a href="'.esc_url($view_url).'">'.$name.'</a>'.$this->row_actions($actions);
 	}
 
 	/**
@@ -190,14 +190,14 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
-			'date_created'  => __( 'Date Created', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
+			'date_created'  => __('Date Created', 'give'),
 		);
 
-		return apply_filters( 'give_list_donors_columns', $columns );
+		return apply_filters('give_list_donors_columns', $columns);
 
 	}
 
@@ -211,13 +211,13 @@ 
 	public function get_sortable_columns() {
 
 		$columns = array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 
-		return apply_filters( 'give_list_donors_sortable_columns', $columns );
+		return apply_filters('give_list_donors_sortable_columns', $columns);
 	}
 
 	/**
@@ -230,15 +230,15 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $donor ) {
+	public function get_row_actions($donor) {
 
 		$actions = array(
-			'view'   => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] ), sprintf( esc_attr__( 'View "%s"', 'give' ), $donor['name'] ), __( 'View Donor', 'give' ) ),
-			'notes'  => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor['id'] ), sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $donor['name'] ), __( 'Notes', 'give' ) ),
-			'delete' => sprintf( '<a class="%1$s" data-id="%2$s" href="#" aria-label="%3$s">%4$s</a>', 'give-single-donor-delete', $donor['id'],sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'view'   => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']), sprintf(esc_attr__('View "%s"', 'give'), $donor['name']), __('View Donor', 'give')),
+			'notes'  => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor['id']), sprintf(esc_attr__('Notes for "%s"', 'give'), $donor['name']), __('Notes', 'give')),
+			'delete' => sprintf('<a class="%1$s" data-id="%2$s" href="#" aria-label="%3$s">%4$s</a>', 'give-single-donor-delete', $donor['id'], sprintf(esc_attr__('Delete "%s"', 'give'), $donor['name']), __('Delete', 'give')),
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $donor );
+		return apply_filters('give_donor_row_actions', $actions, $donor);
 
 	}
 
@@ -251,7 +251,7 @@ 
 	 * @return int Current page number.
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -263,7 +263,7 @@ 
 	 * @return mixed string If search is present, false otherwise.
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -276,7 +276,7 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete' => __( 'Delete', 'give' ),
+			'delete' => __('Delete', 'give'),
 		);
 		return $actions;
 	}
@@ -289,19 +289,19 @@ 
 	 * @access protected
 	 * @since  1.8.16
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'], '_wpnonce', false );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural'], '_wpnonce', false);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
-			<?php if ( $this->has_items() ): ?>
+		<div class="tablenav <?php echo esc_attr($which); ?>">
+			<?php if ($this->has_items()): ?>
 				<div class="alignleft actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 			<?php endif;
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 			<br class="clear" />
 		</div>
@@ -322,13 +322,13 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? intval( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? intval($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -342,7 +342,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -357,9 +357,9 @@ 
 
 		$_donor_query['number'] = - 1;
 		$_donor_query['offset'] = 0;
-		$donors                 = Give()->donors->get_donors( $_donor_query );
+		$donors                 = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -372,10 +372,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -384,10 +384,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -405,9 +405,9 @@ 
 	 * @since  1.8.17
 	 * @access public
 	 */
-	public function single_row( $item ) {
-		echo sprintf( '<tr id="donor-%1$d" data-id="%2$d" data-name="%3$s">', $item['id'], $item['id'], $item['name'] );
-		$this->single_row_columns( $item );
+	public function single_row($item) {
+		echo sprintf('<tr id="donor-%1$d" data-id="%2$d" data-name="%3$s">', $item['id'], $item['id'], $item['name']);
+		$this->single_row_columns($item);
 		echo '</tr>';
 	}
 
@@ -420,11 +420,11 @@ 
 	public function display() {
 		$singular = $this->_args['singular'];
 
-		$this->display_tablenav( 'top' );
+		$this->display_tablenav('top');
 
-		$this->screen->render_screen_reader_content( 'heading_list' );
+		$this->screen->render_screen_reader_content('heading_list');
 		?>
-		<table class="wp-list-table <?php echo implode( ' ', $this->get_table_classes() ); ?>">
+		<table class="wp-list-table <?php echo implode(' ', $this->get_table_classes()); ?>">
 			<thead>
 			<tr>
 				<?php $this->print_column_headers(); ?>
@@ -432,7 +432,7 @@ 
 			</thead>
 
 			<tbody id="the-list"<?php
-			if ( $singular ) {
+			if ($singular) {
 				echo " data-wp-lists='list:$singular'";
 			} ?>>
 			<tr class="hidden"></tr>
@@ -440,7 +440,7 @@ 
 				<td colspan="6" class="colspanchange">
 
 					<fieldset class="inline-edit-col-left">
-						<legend class="inline-edit-legend"><?php _e( 'BULK DELETE', 'give' ); ?></legend>
+						<legend class="inline-edit-legend"><?php _e('BULK DELETE', 'give'); ?></legend>
 						<div class="inline-edit-col">
 							<div id="bulk-titles">
 								<div id="give-bulk-donors" class="give-bulk-donors">
@@ -453,22 +453,22 @@ 
 						<div class="inline-edit-col">
 							<label>
 								<input id="give-delete-donor-confirm" type="checkbox" name="give-delete-donor-confirm"/>
-								<?php _e( 'Are you sure you want to delete the selected donor(s)?', 'give' ); ?>
+								<?php _e('Are you sure you want to delete the selected donor(s)?', 'give'); ?>
 							</label>
 							<label>
 								<input id="give-delete-donor-records" type="checkbox" name="give-delete-donor-records"/>
-								<?php _e( 'Delete all associated donations and records?', 'give' ); ?>
+								<?php _e('Delete all associated donations and records?', 'give'); ?>
 							</label>
 						</div>
 					</fieldset>
 
 					<p class="submit inline-edit-save">
 						<input type="hidden" name="give_action" value="delete_donor"/>
-						<input type="hidden" name="s" value="<?php echo ( ! empty( $_GET['s'] ) ) ? $_GET['s'] : ''; ?>"/>
-						<input type="hidden" name="orderby" value="<?php echo ( ! empty( $_GET['orderby'] ) ) ? $_GET['orderby'] : 'id'; ?>"/>
-						<input type="hidden" name="order" value="<?php echo ( ! empty( $_GET['order'] ) ) ? $_GET['order'] : 'desc'; ?>"/>
-						<button type="button" id="give-bulk-delete-cancel" class="button cancel alignleft"><?php _e( 'Cancel', 'give' ); ?></button>
-						<input type="submit" id="give-bulk-delete-button" disabled class="button button-primary alignright" value="<?php _e( 'Delete', 'give' ); ?>">
+						<input type="hidden" name="s" value="<?php echo ( ! empty($_GET['s'])) ? $_GET['s'] : ''; ?>"/>
+						<input type="hidden" name="orderby" value="<?php echo ( ! empty($_GET['orderby'])) ? $_GET['orderby'] : 'id'; ?>"/>
+						<input type="hidden" name="order" value="<?php echo ( ! empty($_GET['order'])) ? $_GET['order'] : 'desc'; ?>"/>
+						<button type="button" id="give-bulk-delete-cancel" class="button cancel alignleft"><?php _e('Cancel', 'give'); ?></button>
+						<input type="submit" id="give-bulk-delete-button" disabled class="button button-primary alignright" value="<?php _e('Delete', 'give'); ?>">
 						<br class="clear">
 					</p>
 				</td>
@@ -478,13 +478,13 @@ 
 
 			<tfoot>
 			<tr>
-				<?php $this->print_column_headers( false ); ?>
+				<?php $this->print_column_headers(false); ?>
 			</tr>
 			</tfoot>
 
 		</table>
 		<?php
-		$this->display_tablenav( 'bottom' );
+		$this->display_tablenav('bottom');
 	}
 
 	/**
@@ -501,16 +501,16 @@ 
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->donor_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }