WordImpress / Give

includes/donors/class-give-donors-query.php

Spacing +46 added lines, -47 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -85,7 +85,7 @@ 
 	 *
 	 * @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(
 			'number'     => 20,
 			'offset'     => 0,
@@ -103,7 +103,7 @@ 
 			// 'form'       => array(),
 		);
 
-		$this->args = wp_parse_args( $args, $defaults );
+		$this->args = wp_parse_args($args, $defaults);
 
 		$this->table_name      = Give()->donors->table_name;
 		$this->meta_table_name = Give()->donor_meta->table_name;
@@ -146,12 +146,12 @@ 
 		 *
 		 * @param Give_Donors_Query $this Donors query object.
 		 */
-		do_action( 'give_pre_get_donors', $this );
+		do_action('give_pre_get_donors', $this);
 
-		if ( empty( $this->args['count'] ) ) {
-			$this->donors = $wpdb->get_results( $this->get_sql() );
+		if (empty($this->args['count'])) {
+			$this->donors = $wpdb->get_results($this->get_sql());
 		} else {
-			$this->donors = $wpdb->get_var( $this->get_sql() );
+			$this->donors = $wpdb->get_var($this->get_sql());
 		}
 
 		/**
@@ -161,7 +161,7 @@ 
 		 *
 		 * @param Give_Donors_Query $this Donors query object.
 		 */
-		do_action( 'give_post_get_donors', $this );
+		do_action('give_post_get_donors', $this);
 
 		return $this->donors;
 	}
@@ -178,7 +178,7 @@ 
 	public function get_sql() {
 		global $wpdb;
 
-		if ( $this->args['number'] < 1 ) {
+		if ($this->args['number'] < 1) {
 			$this->args['number'] = 999999999999;
 		}
 
@@ -186,22 +186,22 @@ 
 
 
 		// Set offset.
-		if ( empty( $this->args['offset'] ) && ( 0 < $this->args['paged'] ) ) {
-			$this->args['offset'] = $this->args['number'] * ( $this->args['paged'] - 1 );
+		if (empty($this->args['offset']) && (0 < $this->args['paged'])) {
+			$this->args['offset'] = $this->args['number'] * ($this->args['paged'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$this->table_name}.*";
-		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 = "{$this->table_name}.{$this->args['fields']}";
-			} else if ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$this->table_name}." . implode( " , {$this->table_name}.", $this->args['fields'] );
+			} else if (is_array($this->args['fields'])) {
+				$fields = "{$this->table_name}.".implode(" , {$this->table_name}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$this->table_name}.id)";
 		}
 
@@ -209,8 +209,8 @@ 
 
 		return $wpdb->prepare(
 			"SELECT {$fields} FROM {$this->table_name} {$where} {$orderby} {$this->args['order']} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			absint( $this->args['number'] )
+			absint($this->args['offset']),
+			absint($this->args['number'])
 		);
 	}
 
@@ -227,14 +227,14 @@ 
 		$where = '';
 
 		// Get sql query for meta.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_object = new WP_Meta_Query( $this->args['meta_query'] );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_object = new WP_Meta_Query($this->args['meta_query']);
 			$meta_query        = $meta_query_object->get_sql(
 				$this->meta_type,
 				$this->table_name,
 				'id'
 			);
-			$where             = implode( '', $meta_query );
+			$where = implode('', $meta_query);
 		}
 
 		$where .= 'WHERE 1=1';
@@ -261,17 +261,17 @@ 
 
 		$where = '';
 
-		if ( ! empty( $this->args['email'] ) ) {
+		if ( ! empty($this->args['email'])) {
 
-			if ( is_array( $this->args['email'] ) ) {
+			if (is_array($this->args['email'])) {
 
-				$emails_count       = count( $this->args['email'] );
-				$emails_placeholder = array_fill( 0, $emails_count, '%s' );
-				$emails             = implode( ', ', $emails_placeholder );
+				$emails_count       = count($this->args['email']);
+				$emails_placeholder = array_fill(0, $emails_count, '%s');
+				$emails             = implode(', ', $emails_placeholder);
 
-				$where .= $wpdb->prepare( " AND {$this->table_name}.email IN( $emails ) ", $this->args['email'] );
+				$where .= $wpdb->prepare(" AND {$this->table_name}.email IN( $emails ) ", $this->args['email']);
 			} else {
-				$where .= $wpdb->prepare( " AND {$this->table_name}.email = %s ", $this->args['email'] );
+				$where .= $wpdb->prepare(" AND {$this->table_name}.email = %s ", $this->args['email']);
 			}
 		}
 
@@ -291,11 +291,11 @@ 
 		$where = '';
 
 		// Specific donors.
-		if ( ! empty( $this->args['donor'] ) ) {
-			if ( ! is_array( $this->args['donor'] ) ) {
-				$this->args['donor'] = explode( ',', $this->args['donor'] );
+		if ( ! empty($this->args['donor'])) {
+			if ( ! is_array($this->args['donor'])) {
+				$this->args['donor'] = explode(',', $this->args['donor']);
 			}
-			$log_ids = implode( ',', array_map( 'intval', $this->args['donor'] ) );
+			$log_ids = implode(',', array_map('intval', $this->args['donor']));
 
 			$where .= " AND {$this->table_name}.id IN( {$log_ids} ) ";
 		}
@@ -316,7 +316,7 @@ 
 		$where = '';
 
 		// Donors created for a specific date or in a date range
-		if ( ! empty( $this->args['date_query'] ) ) {
+		if ( ! empty($this->args['date_query'])) {
 			$date_query_object = new WP_Date_Query(
 				$this->args['date_query'],
 				"{$this->table_name}.date_created"
@@ -341,11 +341,11 @@ 
 		$where = '';
 
 		// Donors created for a specific date or in a date range
-		if ( ! empty( $this->args['s'] ) && false !== strpos( $this->args['s'], ':' ) ) {
-			$search_parts = explode( ':', $this->args['s'] );
+		if ( ! empty($this->args['s']) && false !== strpos($this->args['s'], ':')) {
+			$search_parts = explode(':', $this->args['s']);
 
-			if ( ! empty( $search_parts[0] ) ) {
-				switch ( $search_parts[0] ) {
+			if ( ! empty($search_parts[0])) {
+				switch ($search_parts[0]) {
 					case 'name':
 						$where = " AND {$this->table_name}.name LIKE '%{$search_parts[1]}%' ";
 						break;
@@ -373,11 +373,11 @@ 
 		$where = '';
 
 		// Donors create for specific wp user.
-		if ( ! empty( $this->args['user'] ) ) {
-			if ( ! is_array( $this->args['user'] ) ) {
-				$this->args['user'] = explode( ',', $this->args['user'] );
+		if ( ! empty($this->args['user'])) {
+			if ( ! is_array($this->args['user'])) {
+				$this->args['user'] = explode(',', $this->args['user']);
 			}
-			$user_ids = implode( ',', array_map( 'intval', $this->args['user'] ) );
+			$user_ids = implode(',', array_map('intval', $this->args['user']));
 
 			$where .= " AND {$this->table_name}.user_id IN( {$user_ids} ) ";
 		}
@@ -396,14 +396,13 @@ 
 	private function get_order_query() {
 		$table_columns = Give()->donors->get_columns();
 
-		$this->args['orderby'] = ! array_key_exists( $this->args['orderby'], $table_columns ) ?
-			'id' :
-			$this->args['orderby'];
+		$this->args['orderby'] = ! array_key_exists($this->args['orderby'], $table_columns) ?
+			'id' : $this->args['orderby'];
 
-		$this->args['orderby'] = esc_sql( $this->args['orderby'] );
-		$this->args['order']   = esc_sql( $this->args['order'] );
+		$this->args['orderby'] = esc_sql($this->args['orderby']);
+		$this->args['order']   = esc_sql($this->args['order']);
 
-		switch ( $table_columns[ $this->args['orderby'] ] ) {
+		switch ($table_columns[$this->args['orderby']]) {
 			case '%d':
 			case '%f':
 				$query = "ORDER BY {$this->table_name}.{$this->args['orderby']}+0";

includes/formatting.php

Indentation +3 added lines, -3 removed lines

@@ -134,9 +134,9 @@
 	// Handle thousand separator as '.'
 	// Handle sanitize database values.
 	$is_db_sanitize_val = ( 2 === count( $number_parts ) &&
-	                        is_numeric( $number_parts[0] ) &&
-	                        is_numeric( $number_parts[1] ) &&
-	                        ( 6 === strlen( $number_parts[1] ) ) );
+							is_numeric( $number_parts[0] ) &&
+							is_numeric( $number_parts[1] ) &&
+							( 6 === strlen( $number_parts[1] ) ) );
 
 	if ( $is_db_sanitize_val ) {
 		// Sanitize database value.

Spacing +153 added lines, -156 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return mixed
  */
 function give_get_price_decimals() {
-	return apply_filters( 'give_sanitize_amount_decimals', give_get_option( 'number_decimals', 0 ) );
+	return apply_filters('give_sanitize_amount_decimals', give_get_option('number_decimals', 0));
 }
 
 /**
@@ -35,8 +35,8 @@ 
  */
 function give_get_price_thousand_separator() {
 	$give_options       = give_get_settings();
-	$thousand_separator = isset( $give_options['thousands_separator'] ) ? $give_options['thousands_separator'] : ',';
-	$thousand_separator = empty( $thousand_separator ) ? ' ' : $thousand_separator;
+	$thousand_separator = isset($give_options['thousands_separator']) ? $give_options['thousands_separator'] : ',';
+	$thousand_separator = empty($thousand_separator) ? ' ' : $thousand_separator;
 
 	return $thousand_separator;
 }
@@ -55,11 +55,10 @@ 
 	);
 
 	$thousand_separator = give_get_price_thousand_separator();
-	$default_decimal_separator = in_array( $thousand_separator, $default_decimal_separators ) ?
-		$default_decimal_separators[$thousand_separator] :
-		'.';
+	$default_decimal_separator = in_array($thousand_separator, $default_decimal_separators) ?
+		$default_decimal_separators[$thousand_separator] : '.';
 
-	$decimal_separator = give_get_option( 'decimal_separator', $default_decimal_separator );
+	$decimal_separator = give_get_option('decimal_separator', $default_decimal_separator);
 
 	return $decimal_separator;
 }
@@ -74,8 +73,8 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount_for_db( $number ) {
-	return give_maybe_sanitize_amount( $number, 6 );
+function give_sanitize_amount_for_db($number) {
+	return give_maybe_sanitize_amount($number, 6);
 }
 
 /**
@@ -89,29 +88,29 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_maybe_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_maybe_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 	$thousand_separator = give_get_price_thousand_separator();
 	$decimal_separator  = give_get_price_decimal_separator();
-	$number_decimals    = is_bool( $dp ) ? give_get_price_decimals() : $dp;
+	$number_decimals    = is_bool($dp) ? give_get_price_decimals() : $dp;
 
 	// Explode number by . decimal separator.
-	$number_parts = explode( '.', $number );
+	$number_parts = explode('.', $number);
 
 	/*
 	 * Bailout: Quick format number
 	 */
-	if ( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}
 
 	// Remove currency symbols from number if any.
-	$number = trim( str_replace( give_currency_symbols( true ), '', $number ) );
+	$number = trim(str_replace(give_currency_symbols(true), '', $number));
 
 	if (
 		// Non formatted number.
 		(
-			( false === strpos( $number, $thousand_separator ) ) &&
-			( false === strpos( $number, $decimal_separator ) )
+			(false === strpos($number, $thousand_separator)) &&
+			(false === strpos($number, $decimal_separator))
 		) ||
 
 		// Decimal formatted number.
@@ -122,35 +121,35 @@ 
 		(
 			$number_decimals &&
 			'.' === $thousand_separator &&
-			false !== strpos( $number, $thousand_separator ) &&
-			false === strpos( $number, $decimal_separator ) &&
-			2 === count( $number_parts ) &&
-			( $number_decimals >= strlen( $number_parts[1] ) )
+			false !== strpos($number, $thousand_separator) &&
+			false === strpos($number, $decimal_separator) &&
+			2 === count($number_parts) &&
+			($number_decimals >= strlen($number_parts[1]))
 		)
 	) {
-		return number_format( $number, $number_decimals, '.', '' );
+		return number_format($number, $number_decimals, '.', '');
 	}
 
 	// Handle thousand separator as '.'
 	// Handle sanitize database values.
-	$is_db_sanitize_val = ( 2 === count( $number_parts ) &&
-	                        is_numeric( $number_parts[0] ) &&
-	                        is_numeric( $number_parts[1] ) &&
-	                        ( 6 === strlen( $number_parts[1] ) ) );
+	$is_db_sanitize_val = (2 === count($number_parts) &&
+	                        is_numeric($number_parts[0]) &&
+	                        is_numeric($number_parts[1]) &&
+	                        (6 === strlen($number_parts[1])));
 
-	if ( $is_db_sanitize_val ) {
+	if ($is_db_sanitize_val) {
 		// Sanitize database value.
-		return number_format( $number, $number_decimals, '.', '' );
+		return number_format($number, $number_decimals, '.', '');
 
 	} elseif (
 		'.' === $thousand_separator &&
-		false !== strpos( $number, $thousand_separator )
+		false !== strpos($number, $thousand_separator)
 	) {
 		// Fix point thousand separator value.
-		$number = str_replace( '.', '', $number );
+		$number = str_replace('.', '', $number);
 	}
 
-	return give_sanitize_amount( $number, $number_decimals, $trim_zeros );
+	return give_sanitize_amount($number, $number_decimals, $trim_zeros);
 }
 
 /**
@@ -168,67 +167,67 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 
 	// Bailout.
-	if ( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}
 
 	// Remove slash from amount.
 	// If thousand or decimal separator is set to ' then in $_POST or $_GET param we will get an escaped number.
 	// To prevent notices and warning remove slash from amount/number.
-	$number = wp_unslash( $number );
+	$number = wp_unslash($number);
 
 	$thousand_separator = give_get_price_thousand_separator();
 
 	$locale   = localeconv();
-	$decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
+	$decimals = array(give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point']);
 
 	// Remove locale from string
-	if ( ! is_float( $number ) ) {
-		$number = str_replace( $decimals, '.', $number );
+	if ( ! is_float($number)) {
+		$number = str_replace($decimals, '.', $number);
 	}
 
 	// Remove thousand amount formatting if amount has.
 	// This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
 	// Do not replace thousand separator from price if it is same as decimal separator, because it will be already replace by above code.
-	if ( ! in_array( $thousand_separator, $decimals ) && ( false !== strpos( $number, $thousand_separator ) ) ) {
-		$number = str_replace( $thousand_separator, '', $number );
-	} elseif ( in_array( $thousand_separator, $decimals ) ) {
-		$number = preg_replace( '/\.(?=.*\.)/', '', $number );
+	if ( ! in_array($thousand_separator, $decimals) && (false !== strpos($number, $thousand_separator))) {
+		$number = str_replace($thousand_separator, '', $number);
+	} elseif (in_array($thousand_separator, $decimals)) {
+		$number = preg_replace('/\.(?=.*\.)/', '', $number);
 	}
 
 	// Remove non numeric entity before decimal separator.
-	$number     = preg_replace( '/[^0-9\.]/', '', $number );
+	$number     = preg_replace('/[^0-9\.]/', '', $number);
 	$default_dp = give_get_price_decimals();
 
 	// Reset negative amount to zero.
-	if ( 0 > $number ) {
-		$number = number_format( 0, $default_dp, '.' );
+	if (0 > $number) {
+		$number = number_format(0, $default_dp, '.');
 	}
 
 	// If number does not have decimal then add number of decimals to it.
 	if (
-		false === strpos( $number, '.' )
-		|| ( $default_dp > strlen( substr( $number, strpos( $number, '.' ) + 1 ) ) )
+		false === strpos($number, '.')
+		|| ($default_dp > strlen(substr($number, strpos($number, '.') + 1)))
 	) {
-		$number = number_format( $number, $default_dp, '.', '' );
+		$number = number_format($number, $default_dp, '.', '');
 	}
 
 	// Format number by custom number of decimals.
-	if ( false !== $dp ) {
-		$dp     = intval( is_bool( $dp ) ? $default_dp : $dp );
-		$dp     = apply_filters( 'give_sanitize_amount_decimals', $dp, $number );
-		$number = number_format( floatval( $number ), $dp, '.', '' );
+	if (false !== $dp) {
+		$dp     = intval(is_bool($dp) ? $default_dp : $dp);
+		$dp     = apply_filters('give_sanitize_amount_decimals', $dp, $number);
+		$number = number_format(floatval($number), $dp, '.', '');
 	}
 
 	// Trim zeros.
-	if ( $trim_zeros && strstr( $number, '.' ) ) {
-		$number = rtrim( rtrim( $number, '0' ), '.' );
+	if ($trim_zeros && strstr($number, '.')) {
+		$number = rtrim(rtrim($number, '0'), '.');
 	}
 
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -241,10 +240,10 @@ 
  *
  * @return string $amount   Newly formatted amount or Price Not Available
  */
-function give_format_amount( $amount, $args = array() ) {
+function give_format_amount($amount, $args = array()) {
 	// Backward compatibility.
-	if( is_bool( $args ) ) {
-		$args = array( 'decimal' => $args );
+	if (is_bool($args)) {
+		$args = array('decimal' => $args);
 	}
 
 	$default_args = array(
@@ -253,60 +252,59 @@ 
 		'currency' => give_get_currency(),
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	$formatted     = 0;
 	$thousands_sep = give_get_price_thousand_separator();
 	$decimal_sep   = give_get_price_decimal_separator();
-	$decimals      = ! empty( $args['decimal'] ) ? give_get_price_decimals() : 0;
+	$decimals      = ! empty($args['decimal']) ? give_get_price_decimals() : 0;
 	$currency      = $args['currency'];
 
-	if ( ! empty( $amount ) ) {
+	if ( ! empty($amount)) {
 		// Sanitize amount before formatting.
-		$amount = ! empty( $args['sanitize'] ) ?
-			give_maybe_sanitize_amount( $amount, $decimals ) :
-			number_format( $amount, $decimals, '.', '' );
+		$amount = ! empty($args['sanitize']) ?
+			give_maybe_sanitize_amount($amount, $decimals) : number_format($amount, $decimals, '.', '');
 
-		switch ( $currency ) {
+		switch ($currency) {
 			case 'INR':
 				$decimal_amount = '';
 
 				// Extract decimals from amount
-				if ( ( $pos = strpos( $amount, '.' ) ) !== false ) {
-					if ( ! empty( $decimals ) ) {
-						$decimal_amount = substr( round( substr( $amount, $pos ), $decimals ), 1 );
-						$amount         = substr( $amount, 0, $pos );
-
-						if ( ! $decimal_amount ) {
-							$decimal_amount = substr( '.0000000000', 0, ( $decimals + 1 ) );
-						} elseif ( ( $decimals + 1 ) > strlen( $decimal_amount ) ) {
-							$decimal_amount = substr( "{$decimal_amount}000000000", 0, ( $decimals + 1 ) );
+				if (($pos = strpos($amount, '.')) !== false) {
+					if ( ! empty($decimals)) {
+						$decimal_amount = substr(round(substr($amount, $pos), $decimals), 1);
+						$amount         = substr($amount, 0, $pos);
+
+						if ( ! $decimal_amount) {
+							$decimal_amount = substr('.0000000000', 0, ($decimals + 1));
+						} elseif (($decimals + 1) > strlen($decimal_amount)) {
+							$decimal_amount = substr("{$decimal_amount}000000000", 0, ($decimals + 1));
 						}
 
 					} else {
-						$amount = number_format( $amount, $decimals, $decimal_sep, '' );
+						$amount = number_format($amount, $decimals, $decimal_sep, '');
 					}
 				}
 
 				// Extract last 3 from amount
-				$result = substr( $amount, - 3 );
-				$amount = substr( $amount, 0, - 3 );
+				$result = substr($amount, - 3);
+				$amount = substr($amount, 0, - 3);
 
 				// Apply digits 2 by 2
-				while ( strlen( $amount ) > 0 ) {
-					$result = substr( $amount, - 2 ) . $thousands_sep . $result;
-					$amount = substr( $amount, 0, - 2 );
+				while (strlen($amount) > 0) {
+					$result = substr($amount, - 2).$thousands_sep.$result;
+					$amount = substr($amount, 0, - 2);
 				}
 
-				$formatted = $result . $decimal_amount;
+				$formatted = $result.$decimal_amount;
 				break;
 
 			default:
-				$formatted = number_format( $amount, $decimals, $decimal_sep, $thousands_sep );
+				$formatted = number_format($amount, $decimals, $decimal_sep, $thousands_sep);
 		}
 	}
 
-	return apply_filters( 'give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args );
+	return apply_filters('give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args);
 }
 
 
@@ -324,51 +322,51 @@ 
  *
  * @return float|string  formatted amount number with large number names.
  */
-function give_human_format_large_amount( $amount, $args = array() ) {
+function give_human_format_large_amount($amount, $args = array()) {
 	$default_args = array(
 		'currency' => give_get_currency(),
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	// Get thousand separator.
 	$thousands_sep = give_get_price_thousand_separator();
 
 	// Sanitize amount.
-	$sanitize_amount = give_maybe_sanitize_amount( $amount );
+	$sanitize_amount = give_maybe_sanitize_amount($amount);
 
 	// Explode amount to calculate name of large numbers.
-	$amount_array = explode( $thousands_sep, $amount );
+	$amount_array = explode($thousands_sep, $amount);
 
 	// Calculate amount parts count.
-	$amount_count_parts = count( $amount_array );
+	$amount_count_parts = count($amount_array);
 
 	// Human format amount (default).
 	$human_format_amount = $amount;
 
-	switch ( $args['currency'] ) {
+	switch ($args['currency']) {
 		case 'INR':
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s arab', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s crore', 'give' ), round( ( $sanitize_amount / 10000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s lakh', 'give' ), round( ( $sanitize_amount / 100000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s arab', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s crore', 'give'), round(($sanitize_amount / 10000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s lakh', 'give'), round(($sanitize_amount / 100000), 2));
 			}
 			break;
 		default:
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s million', 'give' ), round( ( $sanitize_amount / 1000000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s trillion', 'give'), round(($sanitize_amount / 1000000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s billion', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s million', 'give'), round(($sanitize_amount / 1000000), 2));
 			}
 	}
 
-	return apply_filters( 'give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount );
+	return apply_filters('give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount);
 }
 
 /**
@@ -382,17 +380,16 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_decimal( $amount, $dp = false, $sanitize = true ) {
+function give_format_decimal($amount, $dp = false, $sanitize = true) {
 	$decimal_separator = give_get_price_decimal_separator();
 	$formatted_amount  = $sanitize ?
-		give_maybe_sanitize_amount( $amount, $dp ) :
-		number_format( $amount, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		give_maybe_sanitize_amount($amount, $dp) : number_format($amount, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 
-	if ( false !== strpos( $formatted_amount, '.' ) ) {
-		$formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+	if (false !== strpos($formatted_amount, '.')) {
+		$formatted_amount = str_replace('.', $decimal_separator, $formatted_amount);
 	}
 
-	return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+	return apply_filters('give_format_decimal', $formatted_amount, $amount, $decimal_separator);
 }
 
 /**
@@ -406,24 +403,24 @@ 
  *
  * @return mixed|string
  */
-function give_currency_filter( $price = '', $currency = '', $decode_currency = false ) {
+function give_currency_filter($price = '', $currency = '', $decode_currency = false) {
 
-	if ( empty( $currency ) || ! array_key_exists( (string) $currency, give_get_currencies() ) ) {
+	if (empty($currency) || ! array_key_exists((string) $currency, give_get_currencies())) {
 		$currency = give_get_currency();
 	}
 
-	$position = give_get_option( 'currency_position', 'before' );
+	$position = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$symbol = give_currency_symbol( $currency, $decode_currency );
+	$symbol = give_currency_symbol($currency, $decode_currency);
 
-	switch ( $currency ) :
+	switch ($currency) :
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -452,13 +449,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $position ? $symbol . $price : $price . $symbol );
+			$formatted = ('before' === $position ? $symbol.$price : $price.$symbol);
 			break;
 		case 'NOK' :
-			$formatted = ( 'before' === $position ? $symbol . ' ' . $price : $price . ' ' . $symbol );
+			$formatted = ('before' === $position ? $symbol.' '.$price : $price.' '.$symbol);
 			break;
 		default :
-			$formatted = ( 'before' === $position ? $currency . ' ' . $price : $price . ' ' . $currency );
+			$formatted = ('before' === $position ? $currency.' '.$price : $price.' '.$currency);
 			break;
 	endswitch;
 
@@ -473,11 +470,11 @@ 
 	 *           and if currency is USD and currency position is after then
 	 *           filter name will be give_usd_currency_filter_after
 	 */
-	$formatted = apply_filters( 'give_' . strtolower( $currency ) . "_currency_filter_{$position}", $formatted, $currency, $price );
+	$formatted = apply_filters('give_'.strtolower($currency)."_currency_filter_{$position}", $formatted, $currency, $price);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -493,21 +490,21 @@ 
  */
 function give_currency_decimal_filter() {
 
-	remove_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	remove_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Set default number of decimals.
 	$decimals = give_get_price_decimals();
 
-	add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Get number of decimals with backward compatibility ( version < 1.6 )
-	if ( 1 <= func_num_args() ) {
-		$decimals = ( false === func_get_arg( 0 ) ? $decimals : absint( func_get_arg( 0 ) ) );
+	if (1 <= func_num_args()) {
+		$decimals = (false === func_get_arg(0) ? $decimals : absint(func_get_arg(0)));
 	}
 
 	$currency = give_get_currency();
 
-	switch ( $currency ) {
+	switch ($currency) {
 		case 'RIAL' :
 		case 'JPY' :
 		case 'TWD' :
@@ -517,11 +514,11 @@ 
 			break;
 	}
 
-	return apply_filters( 'give_currency_decimal_count', $decimals, $currency );
+	return apply_filters('give_currency_decimal_count', $decimals, $currency);
 }
 
-add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
-add_filter( 'give_format_amount_decimals', 'give_currency_decimal_filter' );
+add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
+add_filter('give_format_amount_decimals', 'give_currency_decimal_filter');
 
 
 /**
@@ -533,7 +530,7 @@ 
  *
  * @return string                  Date format string
  */
-function give_date_format( $date_context = '' ) {
+function give_date_format($date_context = '') {
 	/**
 	 * Filter the date context
 	 *
@@ -554,19 +551,19 @@ 
 	 *
 	 *    }
 	 */
-	$date_format_contexts = apply_filters( 'give_date_format_contexts', array() );
+	$date_format_contexts = apply_filters('give_date_format_contexts', array());
 
 	// Set date format to default date format.
-	$date_format = get_option( 'date_format' );
+	$date_format = get_option('date_format');
 
 	// Update date format if we have non empty date format context array and non empty date format string for that context.
-	if ( $date_context && ! empty( $date_format_contexts ) && array_key_exists( $date_context, $date_format_contexts ) ) {
-		$date_format = ! empty( $date_format_contexts[ $date_context ] )
-			? $date_format_contexts[ $date_context ]
+	if ($date_context && ! empty($date_format_contexts) && array_key_exists($date_context, $date_format_contexts)) {
+		$date_format = ! empty($date_format_contexts[$date_context])
+			? $date_format_contexts[$date_context]
 			: $date_format;
 	}
 
-	return apply_filters( 'give_date_format', $date_format );
+	return apply_filters('give_date_format', $date_format);
 }
 
 /**
@@ -580,8 +577,8 @@ 
  *
  * @return string
  */
-function give_get_cache_key( $action, $query_args ) {
-	return Give_Cache::get_key( $action, $query_args );
+function give_get_cache_key($action, $query_args) {
+	return Give_Cache::get_key($action, $query_args);
 }
 
 /**
@@ -594,11 +591,11 @@ 
  *
  * @return string|array
  */
-function give_clean( $var ) {
-	if ( is_array( $var ) ) {
-		return array_map( 'give_clean', $var );
+function give_clean($var) {
+	if (is_array($var)) {
+		return array_map('give_clean', $var);
 	} else {
-		return is_scalar( $var ) ? sanitize_text_field( $var ) : $var;
+		return is_scalar($var) ? sanitize_text_field($var) : $var;
 	}
 }
 
@@ -611,10 +608,10 @@ 
  *
  * @return int
  */
-function give_let_to_num( $size ) {
-	$l   = substr( $size, - 1 );
-	$ret = substr( $size, 0, - 1 );
-	switch ( strtoupper( $l ) ) {
+function give_let_to_num($size) {
+	$l   = substr($size, - 1);
+	$ret = substr($size, 0, - 1);
+	switch (strtoupper($l)) {
 		case 'P':
 			$ret *= 1024;
 		case 'T':
@@ -639,19 +636,19 @@ 
  * @param int   $action
  * @param array $wp_die_args
  */
-function give_validate_nonce( $nonce, $action = - 1, $wp_die_args = array() ) {
+function give_validate_nonce($nonce, $action = - 1, $wp_die_args = array()) {
 
 	$default_wp_die_args = array(
-		'message' => esc_html__( 'Nonce verification has failed.', 'give' ),
-		'title'   => esc_html__( 'Error', 'give' ),
+		'message' => esc_html__('Nonce verification has failed.', 'give'),
+		'title'   => esc_html__('Error', 'give'),
 		'args'    => array(
 			'response' => 403,
 		),
 	);
 
-	$wp_die_args = wp_parse_args( $wp_die_args, $default_wp_die_args );
+	$wp_die_args = wp_parse_args($wp_die_args, $default_wp_die_args);
 
-	if ( ! wp_verify_nonce( $nonce, $action ) ) {
+	if ( ! wp_verify_nonce($nonce, $action)) {
 		wp_die(
 			$wp_die_args['message'],
 			$wp_die_args['title'],
@@ -673,23 +670,23 @@ 
  *
  * @return mixed
  */
-function give_check_variable( $variable, $conditional = '', $default = false ) {
+function give_check_variable($variable, $conditional = '', $default = false) {
 
-	switch ( $conditional ) {
+	switch ($conditional) {
 		case 'isset_empty':
-			$variable = ( isset( $variable ) && ! empty( $variable ) ) ? $variable : $default;
+			$variable = (isset($variable) && ! empty($variable)) ? $variable : $default;
 			break;
 
 		case 'empty':
-			$variable = ! empty( $variable ) ? $variable : $default;
+			$variable = ! empty($variable) ? $variable : $default;
 			break;
 
 		case 'null':
-			$variable = ! is_null( $variable ) ? $variable : $default;
+			$variable = ! is_null($variable) ? $variable : $default;
 			break;
 
 		default:
-			$variable = isset( $variable ) ? $variable : $default;
+			$variable = isset($variable) ? $variable : $default;
 
 	}
 

includes/emails/functions.php

Spacing +130 added lines, -130 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,11 +26,11 @@ 
  *
  * @return void
  */
-function give_email_donation_receipt( $payment_id, $admin_notice = true ) {
+function give_email_donation_receipt($payment_id, $admin_notice = true) {
 
-	$payment_data = give_get_payment_meta( $payment_id );
+	$payment_data = give_get_payment_meta($payment_id);
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
 	/**
 	 * Filters the from name.
@@ -40,9 +40,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	$from_name = apply_filters( 'give_donation_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = apply_filters('give_donation_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/**
 	 * Filters the from email.
@@ -52,19 +52,19 @@ 
 	 *
 	 * @since 1.0
 	 */
-	$from_email = apply_filters( 'give_donation_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = apply_filters('give_donation_from_address', $from_email, $payment_id, $payment_data);
 
-	$to_email = give_get_payment_user_email( $payment_id );
+	$to_email = give_get_payment_user_email($payment_id);
 
-	$subject = give_get_option( 'donation_subject', __( 'Donation Receipt', 'give' ) );
+	$subject = give_get_option('donation_subject', __('Donation Receipt', 'give'));
 
 	/**
 	 * Filters the donation email receipt subject.
 	 *
 	 * @since 1.0
 	 */
-	$subject = apply_filters( 'give_donation_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = apply_filters('give_donation_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
 	/**
 	 * Filters the donation email receipt attachments. By default, there is no attachment but plugins can hook in to provide one more multiple for the donor. Examples would be a printable ticket or PDF receipt.
@@ -74,14 +74,14 @@ 
 	 *
 	 * @since 1.0
 	 */
-	$attachments = apply_filters( 'give_receipt_attachments', array(), $payment_id, $payment_data );
-	$message     = give_do_email_tags( give_get_email_body_content( $payment_id, $payment_data ), $payment_id );
+	$attachments = apply_filters('give_receipt_attachments', array(), $payment_id, $payment_data);
+	$message     = give_do_email_tags(give_get_email_body_content($payment_id, $payment_data), $payment_id);
 
 	$emails = Give()->emails;
 
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Donation Receipt', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Donation Receipt', 'give'));
 
 	/**
 	 * Filters the donation receipt's email headers.
@@ -91,14 +91,14 @@ 
 	 *
 	 * @since 1.0
 	 */
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), $payment_id, $payment_data);
+	$emails->__set('headers', $headers);
 
 	//Send the donation receipt.
-	$emails->send( $to_email, $subject, $message, $attachments );
+	$emails->send($to_email, $subject, $message, $attachments);
 
 	//If admin notifications are on, send the admin notice.
-	if ( $admin_notice && ! give_admin_notices_disabled( $payment_id ) ) {
+	if ($admin_notice && ! give_admin_notices_disabled($payment_id)) {
 		/**
 		 * Fires in the donation email receipt.
 		 *
@@ -109,7 +109,7 @@ 
 		 * @param int   $payment_id   Payment id.
 		 * @param mixed $payment_data Payment meta data.
 		 */
-		do_action( 'give_admin_donation_email', $payment_id, $payment_data );
+		do_action('give_admin_donation_email', $payment_id, $payment_data);
 	}
 }
 
@@ -122,41 +122,41 @@ 
  */
 function give_email_test_donation_receipt() {
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
 	/**
 	 * Filters the from name.
 	 *
 	 * @since 1.7
 	 */
-	$from_name = apply_filters( 'give_donation_from_name', $from_name, 0, array() );
+	$from_name = apply_filters('give_donation_from_name', $from_name, 0, array());
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/**
 	 * Filters the from email.
 	 *
 	 * @since 1.7
 	 */
-	$from_email = apply_filters( 'give_donation_from_address', $from_email, 0, array() );
+	$from_email = apply_filters('give_donation_from_address', $from_email, 0, array());
 
-	$subject = give_get_option( 'donation_subject', __( 'Donation Receipt', 'give' ) );
-	$subject = apply_filters( 'give_donation_subject', wp_strip_all_tags( $subject ), 0 );
-	$subject = give_do_email_tags( $subject, 0 );
+	$subject = give_get_option('donation_subject', __('Donation Receipt', 'give'));
+	$subject = apply_filters('give_donation_subject', wp_strip_all_tags($subject), 0);
+	$subject = give_do_email_tags($subject, 0);
 
-	$attachments = apply_filters( 'give_receipt_attachments', array(), 0, array() );
+	$attachments = apply_filters('give_receipt_attachments', array(), 0, array());
 
-	$message = give_email_preview_template_tags( give_get_email_body_content( 0, array() ) );
+	$message = give_email_preview_template_tags(give_get_email_body_content(0, array()));
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'heading', __( 'Donation Receipt', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('heading', __('Donation Receipt', 'give'));
 
-	$headers = apply_filters( 'give_receipt_headers', $emails->get_headers(), 0, array() );
-	$emails->__set( 'headers', $headers );
+	$headers = apply_filters('give_receipt_headers', $emails->get_headers(), 0, array());
+	$emails->__set('headers', $headers);
 
-	$emails->send( give_get_admin_notice_emails(), $subject, $message, $attachments );
+	$emails->send(give_get_admin_notice_emails(), $subject, $message, $attachments);
 
 }
 
@@ -170,49 +170,49 @@ 
  *
  * @return void
  */
-function give_admin_email_notice( $payment_id = 0, $payment_data = array() ) {
+function give_admin_email_notice($payment_id = 0, $payment_data = array()) {
 
-	$payment_id = absint( $payment_id );
+	$payment_id = absint($payment_id);
 
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
-	if ( ! give_get_payment_by( 'id', $payment_id ) ) {
+	if ( ! give_get_payment_by('id', $payment_id)) {
 		return;
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
 	/**
 	 * Filters the from name.
 	 *
 	 * @since 1.0
 	 */
-	$from_name = apply_filters( 'give_donation_from_name', $from_name, $payment_id, $payment_data );
+	$from_name = apply_filters('give_donation_from_name', $from_name, $payment_id, $payment_data);
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/**
 	 * Filters the from email.
 	 *
 	 * @since 1.0
 	 */
-	$from_email = apply_filters( 'give_donation_from_address', $from_email, $payment_id, $payment_data );
+	$from_email = apply_filters('give_donation_from_address', $from_email, $payment_id, $payment_data);
 
 	/* translators: %s: payment id */
-	$subject = give_get_option( 'donation_notification_subject', sprintf( esc_html__( 'New Donation - Payment #%s', 'give' ), $payment_id ) );
+	$subject = give_get_option('donation_notification_subject', sprintf(esc_html__('New Donation - Payment #%s', 'give'), $payment_id));
 
 	/**
 	 * Filters the donation notification subject.
 	 *
 	 * @since 1.0
 	 */
-	$subject = apply_filters( 'give_admin_donation_notification_subject', wp_strip_all_tags( $subject ), $payment_id );
-	$subject = give_do_email_tags( $subject, $payment_id );
+	$subject = apply_filters('give_admin_donation_notification_subject', wp_strip_all_tags($subject), $payment_id);
+	$subject = give_do_email_tags($subject, $payment_id);
 
-	$headers = "From: " . stripslashes_deep( html_entity_decode( $from_name, ENT_COMPAT, 'UTF-8' ) ) . " <$from_email>\r\n";
-	$headers .= "Reply-To: " . $from_email . "\r\n";
+	$headers = "From: ".stripslashes_deep(html_entity_decode($from_name, ENT_COMPAT, 'UTF-8'))." <$from_email>\r\n";
+	$headers .= "Reply-To: ".$from_email."\r\n";
 	//$headers  .= "MIME-Version: 1.0\r\n";
 	$headers .= "Content-Type: text/html; charset=utf-8\r\n";
 
@@ -221,28 +221,28 @@ 
 	 *
 	 * @since 1.0
 	 */
-	$headers = apply_filters( 'give_admin_donation_notification_headers', $headers, $payment_id, $payment_data );
+	$headers = apply_filters('give_admin_donation_notification_headers', $headers, $payment_id, $payment_data);
 
 	/**
 	 * Filters the donation notification email attachments. By default, there is no attachment but plugins can hook in to provide one more multiple.
 	 *
 	 * @since 1.0
 	 */
-	$attachments = apply_filters( 'give_admin_donation_notification_attachments', array(), $payment_id, $payment_data );
+	$attachments = apply_filters('give_admin_donation_notification_attachments', array(), $payment_id, $payment_data);
 
-	$message = give_get_donation_notification_body_content( $payment_id, $payment_data );
+	$message = give_get_donation_notification_body_content($payment_id, $payment_data);
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'headers', $headers );
-	$emails->__set( 'heading', __( 'New Donation!', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('headers', $headers);
+	$emails->__set('heading', __('New Donation!', 'give'));
 
-	$emails->send( give_get_admin_notice_emails(), $subject, $message, $attachments );
+	$emails->send(give_get_admin_notice_emails(), $subject, $message, $attachments);
 
 }
 
-add_action( 'give_admin_donation_email', 'give_admin_email_notice', 10, 2 );
+add_action('give_admin_donation_email', 'give_admin_email_notice', 10, 2);
 
 /**
  * Retrieves the emails for which admin notifications are sent to (these can be changed in the Give Settings).
@@ -252,12 +252,12 @@ 
  */
 function give_get_admin_notice_emails() {
 
-	$email_option = give_get_option( 'admin_notice_emails' );
+	$email_option = give_get_option('admin_notice_emails');
 
-	$emails = ! empty( $email_option ) && strlen( trim( $email_option ) ) > 0 ? $email_option : get_bloginfo( 'admin_email' );
-	$emails = array_map( 'trim', explode( "\n", $emails ) );
+	$emails = ! empty($email_option) && strlen(trim($email_option)) > 0 ? $email_option : get_bloginfo('admin_email');
+	$emails = array_map('trim', explode("\n", $emails));
 
-	return apply_filters( 'give_admin_notice_emails', $emails );
+	return apply_filters('give_admin_notice_emails', $emails);
 }
 
 /**
@@ -269,11 +269,11 @@ 
  *
  * @return mixed
  */
-function give_admin_notices_disabled( $payment_id = 0 ) {
+function give_admin_notices_disabled($payment_id = 0) {
 
 	return apply_filters(
 		'give_admin_notices_disabled',
-		! give_is_setting_enabled( give_get_option( 'admin_notices' ) ),
+		! give_is_setting_enabled(give_get_option('admin_notices')),
 		$payment_id
 	);
 }
@@ -288,19 +288,19 @@ 
  */
 function give_get_default_donation_notification_email() {
 
-	$default_email_body = __( 'Hi there,', 'give' ) . "\n\n";
-	$default_email_body .= __( 'This email is to inform you that a new donation has been made on your website:', 'give' ) . ' <a href="' . get_bloginfo( 'url' ) . '" target="_blank">' . get_bloginfo( 'url' ) . '</a>' . ".\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {name}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation:', 'give' ) . '</strong> {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method:', 'give' ) . '</strong> {payment_method}' . "\n\n";
-	$default_email_body .= __( 'Thank you,', 'give' ) . "\n\n";
-	$default_email_body .= '{sitename}' . "\n";
+	$default_email_body = __('Hi there,', 'give')."\n\n";
+	$default_email_body .= __('This email is to inform you that a new donation has been made on your website:', 'give').' <a href="'.get_bloginfo('url').'" target="_blank">'.get_bloginfo('url').'</a>'.".\n\n";
+	$default_email_body .= '<strong>'.__('Donor:', 'give').'</strong> {name}'."\n";
+	$default_email_body .= '<strong>'.__('Donation:', 'give').'</strong> {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method:', 'give').'</strong> {payment_method}'."\n\n";
+	$default_email_body .= __('Thank you,', 'give')."\n\n";
+	$default_email_body .= '{sitename}'."\n";
 
-	$custom_message = give_get_option( 'donation_notification' );
-	$message        = ! empty( $custom_message ) ? $custom_message : $default_email_body;
+	$custom_message = give_get_option('donation_notification');
+	$message        = ! empty($custom_message) ? $custom_message : $default_email_body;
 
-	return apply_filters( 'give_default_donation_notification_email', $message );
+	return apply_filters('give_default_donation_notification_email', $message);
 }
 
 
@@ -314,25 +314,25 @@ 
  */
 function give_get_default_donation_receipt_email() {
 
-	$default_email_body = __( 'Dear', 'give' ) . " {name},\n\n";
-	$default_email_body .= __( 'Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give' ) . "\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation:', 'give' ) . '</strong> {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation Date:', 'give' ) . '</strong> {date}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method:', 'give' ) . '</strong> {payment_method}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment ID:', 'give' ) . '</strong> {payment_id}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Receipt ID:', 'give' ) . '</strong> {receipt_id}' . "\n\n";
-	$default_email_body .= '{receipt_link}' . "\n\n";
+	$default_email_body = __('Dear', 'give')." {name},\n\n";
+	$default_email_body .= __('Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give')."\n\n";
+	$default_email_body .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
+	$default_email_body .= '<strong>'.__('Donation:', 'give').'</strong> {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Donation Date:', 'give').'</strong> {date}'."\n";
+	$default_email_body .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method:', 'give').'</strong> {payment_method}'."\n";
+	$default_email_body .= '<strong>'.__('Payment ID:', 'give').'</strong> {payment_id}'."\n";
+	$default_email_body .= '<strong>'.__('Receipt ID:', 'give').'</strong> {receipt_id}'."\n\n";
+	$default_email_body .= '{receipt_link}'."\n\n";
 	$default_email_body .= "\n\n";
-	$default_email_body .= __( 'Sincerely,', 'give' ) . "\n";
-	$default_email_body .= '{sitename}' . "\n";
+	$default_email_body .= __('Sincerely,', 'give')."\n";
+	$default_email_body .= '{sitename}'."\n";
 
-	$custom_message = give_get_option( 'donation_receipt' );
+	$custom_message = give_get_option('donation_receipt');
 
-	$message = ! empty( $custom_message ) ? $custom_message : $default_email_body;
+	$message = ! empty($custom_message) ? $custom_message : $default_email_body;
 
-	return apply_filters( 'give_default_donation_receipt_email', $message );
+	return apply_filters('give_default_donation_receipt_email', $message);
 }
 
 /**
@@ -345,22 +345,22 @@ 
  *
  * @return array $email_names
  */
-function give_get_email_names( $user_info, $payment = false ) {
+function give_get_email_names($user_info, $payment = false) {
 	$email_names = array();
 
-	if ( is_a( $payment, 'Give_Payment' ) ) {
+	if (is_a($payment, 'Give_Payment')) {
 
-		if ( $payment->user_id > 0 ) {
+		if ($payment->user_id > 0) {
 
-			$user_data               = get_userdata( $payment->user_id );
+			$user_data               = get_userdata($payment->user_id);
 			$email_names['name']     = $payment->first_name;
-			$email_names['fullname'] = trim( $payment->first_name . ' ' . $payment->last_name );
+			$email_names['fullname'] = trim($payment->first_name.' '.$payment->last_name);
 			$email_names['username'] = $user_data->user_login;
 
-		} elseif ( ! empty( $payment->first_name ) ) {
+		} elseif ( ! empty($payment->first_name)) {
 
 			$email_names['name']     = $payment->first_name;
-			$email_names['fullname'] = trim( $payment->first_name . ' ' . $payment->last_name );
+			$email_names['fullname'] = trim($payment->first_name.' '.$payment->last_name);
 			$email_names['username'] = $payment->first_name;
 
 		} else {
@@ -373,30 +373,30 @@ 
 	} else {
 
 		// Support for old serialized data.
-		if ( is_serialized( $user_info ) ) {
+		if (is_serialized($user_info)) {
 
 			// Security check.
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches );
-			if ( ! empty( $matches ) ) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches);
+			if ( ! empty($matches)) {
 				return array(
 					'name'     => '',
 					'fullname' => '',
 					'username' => '',
 				);
 			} else {
-				$user_info = maybe_unserialize( $user_info );
+				$user_info = maybe_unserialize($user_info);
 			}
 
 		}
 
-		if ( isset( $user_info['id'] ) && $user_info['id'] > 0 && isset( $user_info['first_name'] ) ) {
-			$user_data               = get_userdata( $user_info['id'] );
+		if (isset($user_info['id']) && $user_info['id'] > 0 && isset($user_info['first_name'])) {
+			$user_data               = get_userdata($user_info['id']);
 			$email_names['name']     = $user_info['first_name'];
-			$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 			$email_names['username'] = $user_data->user_login;
-		} elseif ( isset( $user_info['first_name'] ) ) {
+		} elseif (isset($user_info['first_name'])) {
 			$email_names['name']     = $user_info['first_name'];
-			$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 			$email_names['username'] = $user_info['first_name'];
 		} else {
 			$email_names['name']     = $user_info['email'];
@@ -416,37 +416,37 @@ 
  *
  * @since 1.8.14
  */
-function give_admin_email_user_donor_disconnection( $user_id, $donor_id ) {
+function give_admin_email_user_donor_disconnection($user_id, $donor_id) {
 
-	$user_id  = absint( $user_id );
-	$donor_id = absint( $donor_id );
+	$user_id  = absint($user_id);
+	$donor_id = absint($donor_id);
 
 	// Bail Out, if user id doesn't exists.
-	if ( empty( $user_id ) ) {
+	if (empty($user_id)) {
 		return;
 	}
 
 	// Bail Out, if donor id doesn't exists.
-	if ( empty( $donor_id ) ) {
+	if (empty($donor_id)) {
 		return;
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/* translators: %s: payment id */
-	$subject = __( 'Attention: User tries to login whose Donor profile is disconnected!', 'give' );
+	$subject = __('Attention: User tries to login whose Donor profile is disconnected!', 'give');
 
 	/**
 	 * Filters the Donor-User Disconnection notification subject.
 	 *
 	 * @since 1.8.14
 	 */
-	$subject = apply_filters( 'give_admin_donor_user_disconnection_notification_subject', wp_strip_all_tags( $subject ) );
+	$subject = apply_filters('give_admin_donor_user_disconnection_notification_subject', wp_strip_all_tags($subject));
 
-	$headers = "From: " . stripslashes_deep( html_entity_decode( $from_name, ENT_COMPAT, 'UTF-8' ) ) . " <$from_email>\r\n";
-	$headers .= "Reply-To: " . $from_email . "\r\n";
+	$headers = "From: ".stripslashes_deep(html_entity_decode($from_name, ENT_COMPAT, 'UTF-8'))." <$from_email>\r\n";
+	$headers .= "Reply-To: ".$from_email."\r\n";
 	$headers .= "Content-Type: text/html; charset=utf-8\r\n";
 
 	/**
@@ -454,25 +454,25 @@ 
 	 *
 	 * @since 1.8.14
 	 */
-	$headers = apply_filters( 'give_admin_donor_user_disconnection_notification_headers', $headers );
+	$headers = apply_filters('give_admin_donor_user_disconnection_notification_headers', $headers);
 
-	$message = __( 'Hi Admin,', 'give' ) . "\n\n";
-	$message .= __( 'This email is to inform you that a user has tried logging in. But, User was unable to login due to User-Donor profile disconnection.', 'give' ) . "\n\n";
-	$message .= __( 'Do you want to reconnect User and Donor profile again?', 'give' ) . "\n\n";
+	$message = __('Hi Admin,', 'give')."\n\n";
+	$message .= __('This email is to inform you that a user has tried logging in. But, User was unable to login due to User-Donor profile disconnection.', 'give')."\n\n";
+	$message .= __('Do you want to reconnect User and Donor profile again?', 'give')."\n\n";
 	$message .= sprintf(
 		'<a href="%1$s">%2$s</a>',
-		esc_url( admin_url() . 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&user_id=' . $user_id . '&give-message=reconnect-user' ),
-		__( 'Reconnect User', 'give' ) . "\n\n"
+		esc_url(admin_url().'edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&user_id='.$user_id.'&give-message=reconnect-user'),
+		__('Reconnect User', 'give')."\n\n"
 	);
-	$message .= __( 'Thank you,', 'give' ) . "\n\n";
-	$message .= '{sitename}' . "\n";
+	$message .= __('Thank you,', 'give')."\n\n";
+	$message .= '{sitename}'."\n";
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'headers', $headers );
-	$emails->__set( 'heading', __( 'User - Donor Profile Disconnection', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('headers', $headers);
+	$emails->__set('heading', __('User - Donor Profile Disconnection', 'give'));
 
-	$emails->send( give_get_admin_notice_emails(), $subject, give_do_email_tags( $message ) );
+	$emails->send(give_get_admin_notice_emails(), $subject, give_do_email_tags($message));
 
 }

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

Spacing +17 added lines, -17 removed lines

@@ -2,13 +2,13 @@ 
 /**
  * Admin View: Settings
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 
 // Bailout: Do not output anything if setting tab is not defined.
-if( ! empty( $tabs ) && array_key_exists( give_get_current_setting_tab(), $tabs ) ) :
+if ( ! empty($tabs) && array_key_exists(give_get_current_setting_tab(), $tabs)) :
 	/**
 	 * Filter the form action.
 	 *
@@ -18,7 +18,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_method    = apply_filters( self::$setting_filter_prefix . '_form_method_tab_' . $current_tab, 'post' );
+	$form_method = apply_filters(self::$setting_filter_prefix.'_form_method_tab_'.$current_tab, 'post');
 
 	/**
 	 * Filter the main form tab.
@@ -30,12 +30,12 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_open_tag  = apply_filters( self::$setting_filter_prefix . '_open_form', '<form method="' . $form_method  . '" id="give-mainform" action="" enctype="multipart/form-data">' );
-	$form_close_tag = apply_filters( self::$setting_filter_prefix . '_close_form', '</form>' );
+	$form_open_tag  = apply_filters(self::$setting_filter_prefix.'_open_form', '<form method="'.$form_method.'" id="give-mainform" action="" enctype="multipart/form-data">');
+	$form_close_tag = apply_filters(self::$setting_filter_prefix.'_close_form', '</form>');
 	?>
-	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix . '-setting-page'; ?>">
+	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix.'-setting-page'; ?>">
 		<?php echo $form_open_tag; ?>
-		<h1 class="wp-heading-inline"><?php echo esc_html( $tabs[ $current_tab ] ); ?></h1>
+		<h1 class="wp-heading-inline"><?php echo esc_html($tabs[$current_tab]); ?></h1>
 		<?php
 		// Show messages.
 		self::show_messages();
@@ -45,8 +45,8 @@ 
 
 		<div class="nav-tab-wrapper give-nav-tab-wrapper">
 			<?php
-			foreach ( $tabs as $name => $label ) {
-				echo '<a href="' . admin_url( "edit.php?post_type=give_forms&page=" . self::$setting_filter_prefix . "&tab={$name}" ) . '" class="nav-tab ' . ( $current_tab == $name ? 'nav-tab-active' : '' ) . '">' . $label . '</a>';
+			foreach ($tabs as $name => $label) {
+				echo '<a href="'.admin_url("edit.php?post_type=give_forms&page=".self::$setting_filter_prefix."&tab={$name}").'" class="nav-tab '.($current_tab == $name ? 'nav-tab-active' : '').'">'.$label.'</a>';
 			}
 
 			/**
@@ -58,11 +58,11 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_tabs' );
+			do_action(self::$setting_filter_prefix.'_tabs');
 			?>
 		</div>
 		<div class="give-sub-nav-tab-wrapper">
-			<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e( 'View remaining setting tabs', 'give' ); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span></a>
+			<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e('View remaining setting tabs', 'give'); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span></a>
 			<nav class="give-sub-nav-tab give-hidden"></nav>
 		</div>
 		<?php
@@ -76,7 +76,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_sections_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_sections_{$current_tab}_page");
 
 		/**
 		 * Trigger Action.
@@ -87,15 +87,15 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_settings_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_settings_{$current_tab}_page");
 
-		if ( empty( $GLOBALS['give_hide_save_button'] ) ) : ?>
+		if (empty($GLOBALS['give_hide_save_button'])) : ?>
 			<div class="give-submit-wrap">
-				<?php wp_nonce_field( 'give-save-settings', '_give-save-settings' ); ?>
-				<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e( 'Save changes', 'give' ); ?>" />
+				<?php wp_nonce_field('give-save-settings', '_give-save-settings'); ?>
+				<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e('Save changes', 'give'); ?>" />
 			</div>
 		<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+<?php else : echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>'; ?>
 <?php endif; ?>
\ No newline at end of file

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

Spacing +66 added lines, -66 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.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		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,22 +80,22 @@ 
 	 *
 	 * @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 '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" 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
 	}
@@ -111,31 +111,31 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($donor, $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&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&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']);
 
 	}
 
@@ -146,13 +146,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = '#' . $donor['id'] . ' ';
-		$name     .= ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . esc_html__( '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>'.esc_html__('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);
 	}
 
 	/**
@@ -164,14 +164,14 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'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);
 
 	}
 
@@ -185,13 +185,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);
 	}
 
 	/**
@@ -204,19 +204,19 @@ 
 	 *
 	 * @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' ) ),
+			'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' ) ),
+			'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 href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor['id'] ), sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'delete' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$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);
 
 	}
 
@@ -230,7 +230,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place.
 	}
 
@@ -242,7 +242,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;
 	}
 
 	/**
@@ -253,7 +253,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;
 	}
 
 	/**
@@ -270,13 +270,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,
@@ -290,7 +290,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -305,9 +305,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);
 	}
 
 	/**
@@ -319,10 +319,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,
@@ -331,10 +331,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;
@@ -357,16 +357,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),
+		));
 	}
 }

includes/admin/donors/donor-actions.php

Spacing +190 added lines, -190 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,17 +23,17 @@ 
  *
  * @return array|bool $output Response messages
  */
-function give_edit_donor( $args ) {
+function give_edit_donor($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -41,15 +41,15 @@ 
 	$donor_id   = (int) $args['customerinfo']['id'];
 	$nonce      = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin’ uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin’ uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
-	if ( empty( $donor->id ) ) {
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -64,67 +64,67 @@ 
 		'country' => '',
 	);
 
-	$donor_info = wp_parse_args( $donor_info, $defaults );
+	$donor_info = wp_parse_args($donor_info, $defaults);
 
-	if ( (int) $donor_info['user_id'] !== (int) $donor->user_id ) {
+	if ((int) $donor_info['user_id'] !== (int) $donor->user_id) {
 
 		// Make sure we don't already have this user attached to a donor.
-		if ( ! empty( $donor_info['user_id'] ) && false !== Give()->donors->get_donor_by( 'user_id', $donor_info['user_id'] ) ) {
-			give_set_error( 'give-invalid-donor-user_id', sprintf( __( 'The User ID #%d is already associated with a different donor.', 'give' ), $donor_info['user_id'] ) );
+		if ( ! empty($donor_info['user_id']) && false !== Give()->donors->get_donor_by('user_id', $donor_info['user_id'])) {
+			give_set_error('give-invalid-donor-user_id', sprintf(__('The User ID #%d is already associated with a different donor.', 'give'), $donor_info['user_id']));
 		}
 
 		// Make sure it's actually a user.
-		$user = get_user_by( 'id', $donor_info['user_id'] );
-		if ( ! empty( $donor_info['user_id'] ) && false === $user ) {
-			give_set_error( 'give-invalid-user_id', sprintf( __( 'The User ID #%d does not exist. Please assign an existing user.', 'give' ), $donor_info['user_id'] ) );
+		$user = get_user_by('id', $donor_info['user_id']);
+		if ( ! empty($donor_info['user_id']) && false === $user) {
+			give_set_error('give-invalid-user_id', sprintf(__('The User ID #%d does not exist. Please assign an existing user.', 'give'), $donor_info['user_id']));
 		}
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
 	// Setup the donor address, if present.
 	$address = array();
-	if ( intval( $donor_info['user_id'] ) > 0 ) {
-
-		$current_address = (array) get_user_meta( $donor_info['user_id'], '_give_user_address', true );
-
-		if ( is_array( $current_address ) && 0 < count( $current_address ) ) {
-			$current_address    = wp_parse_args( $current_address, $defaults );
-			$address['line1']   = ! empty( $donor_info['line1'] ) ? $donor_info['line1'] : $current_address['line1'];
-			$address['line2']   = ! empty( $donor_info['line2'] ) ? $donor_info['line2'] : $current_address['line2'];
-			$address['city']    = ! empty( $donor_info['city'] ) ? $donor_info['city'] : $current_address['city'];
-			$address['country'] = ! empty( $donor_info['country'] ) ? $donor_info['country'] : $current_address['country'];
-			$address['zip']     = ! empty( $donor_info['zip'] ) ? $donor_info['zip'] : $current_address['zip'];
-			$address['state']   = ! empty( $donor_info['state'] ) ? $donor_info['state'] : $current_address['state'];
+	if (intval($donor_info['user_id']) > 0) {
+
+		$current_address = (array) get_user_meta($donor_info['user_id'], '_give_user_address', true);
+
+		if (is_array($current_address) && 0 < count($current_address)) {
+			$current_address    = wp_parse_args($current_address, $defaults);
+			$address['line1']   = ! empty($donor_info['line1']) ? $donor_info['line1'] : $current_address['line1'];
+			$address['line2']   = ! empty($donor_info['line2']) ? $donor_info['line2'] : $current_address['line2'];
+			$address['city']    = ! empty($donor_info['city']) ? $donor_info['city'] : $current_address['city'];
+			$address['country'] = ! empty($donor_info['country']) ? $donor_info['country'] : $current_address['country'];
+			$address['zip']     = ! empty($donor_info['zip']) ? $donor_info['zip'] : $current_address['zip'];
+			$address['state']   = ! empty($donor_info['state']) ? $donor_info['state'] : $current_address['state'];
 		} else {
-			$address['line1']   = ! empty( $donor_info['line1'] ) ? $donor_info['line1'] : '';
-			$address['line2']   = ! empty( $donor_info['line2'] ) ? $donor_info['line2'] : '';
-			$address['city']    = ! empty( $donor_info['city'] ) ? $donor_info['city'] : '';
-			$address['country'] = ! empty( $donor_info['country'] ) ? $donor_info['country'] : '';
-			$address['zip']     = ! empty( $donor_info['zip'] ) ? $donor_info['zip'] : '';
-			$address['state']   = ! empty( $donor_info['state'] ) ? $donor_info['state'] : '';
+			$address['line1']   = ! empty($donor_info['line1']) ? $donor_info['line1'] : '';
+			$address['line2']   = ! empty($donor_info['line2']) ? $donor_info['line2'] : '';
+			$address['city']    = ! empty($donor_info['city']) ? $donor_info['city'] : '';
+			$address['country'] = ! empty($donor_info['country']) ? $donor_info['country'] : '';
+			$address['zip']     = ! empty($donor_info['zip']) ? $donor_info['zip'] : '';
+			$address['state']   = ! empty($donor_info['state']) ? $donor_info['state'] : '';
 		}
 
 	}
 
 	// Sanitize the inputs.
 	$donor_data            = array();
-	$donor_data['name']    = strip_tags( stripslashes( $donor_info['name'] ) );
+	$donor_data['name']    = strip_tags(stripslashes($donor_info['name']));
 	$donor_data['user_id'] = $donor_info['user_id'];
 
-	$donor_data = apply_filters( 'give_edit_donor_info', $donor_data, $donor_id );
-	$address    = apply_filters( 'give_edit_donor_address', $address, $donor_id );
+	$donor_data = apply_filters('give_edit_donor_info', $donor_data, $donor_id);
+	$address    = apply_filters('give_edit_donor_address', $address, $donor_id);
 
-	$donor_data = array_map( 'sanitize_text_field', $donor_data );
-	$address    = array_map( 'sanitize_text_field', $address );
+	$donor_data = array_map('sanitize_text_field', $donor_data);
+	$address    = array_map('sanitize_text_field', $address);
 
-	$output = give_connect_user_donor_profile( $donor, $donor_data, $address );
+	$output = give_connect_user_donor_profile($donor, $donor_data, $address);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
@@ -132,7 +132,7 @@ 
 
 }
 
-add_action( 'give_edit-donor', 'give_edit_donor', 10, 1 );
+add_action('give_edit-donor', 'give_edit_donor', 10, 1);
 
 /**
  * Save a donor note.
@@ -143,40 +143,40 @@ 
  *
  * @return int         The Note ID that was saved, or 0 if nothing was saved.
  */
-function give_donor_save_note( $args ) {
+function give_donor_save_note($args) {
 
-	$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 ( ! is_admin() || ! current_user_can( $donor_view_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_view_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
-	$donor_note = trim( sanitize_text_field( $args['donor_note'] ) );
+	$donor_note = trim(sanitize_text_field($args['donor_note']));
 	$donor_id   = (int) $args['customer_id'];
 	$nonce      = $args['add_donor_note_nonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'add-donor-note' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'add-donor-note')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( empty( $donor_note ) ) {
-		give_set_error( 'empty-donor-note', __( 'A note is required.', 'give' ) );
+	if (empty($donor_note)) {
+		give_set_error('empty-donor-note', __('A note is required.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
-	$donor    = new Give_Donor( $donor_id );
-	$new_note = $donor->add_note( $donor_note );
+	$donor    = new Give_Donor($donor_id);
+	$new_note = $donor->add_note($donor_note);
 
 	/**
 	 * Fires before inserting donor note.
@@ -186,22 +186,22 @@ 
 	 * @param int    $donor_id The ID of the donor.
 	 * @param string $new_note Note content.
 	 */
-	do_action( 'give_pre_insert_donor_note', $donor_id, $new_note );
+	do_action('give_pre_insert_donor_note', $donor_id, $new_note);
 
-	if ( ! empty( $new_note ) && ! empty( $donor->id ) ) {
+	if ( ! empty($new_note) && ! empty($donor->id)) {
 
 		ob_start();
 		?>
 		<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 			<span class="note-content-wrap">
-				<?php echo stripslashes( $new_note ); ?>
+				<?php echo stripslashes($new_note); ?>
 			</span>
 		</div>
 		<?php
 		$output = ob_get_contents();
 		ob_end_clean();
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+		if (defined('DOING_AJAX') && DOING_AJAX) {
 			echo $output;
 			exit;
 		}
@@ -214,7 +214,7 @@ 
 
 }
 
-add_action( 'give_add-donor-note', 'give_donor_save_note', 10, 1 );
+add_action('give_add-donor-note', 'give_donor_save_note', 10, 1);
 
 /**
  * Delete a donor.
@@ -225,41 +225,41 @@ 
  *
  * @return int Whether it was a successful deletion.
  */
-function give_donor_delete( $args ) {
+function give_donor_delete($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to delete donors.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to delete donors.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
 	$donor_id    = (int) $args['customer_id'];
-	$confirm     = ! empty( $args['give-donor-delete-confirm'] ) ? true : false;
-	$remove_data = ! empty( $args['give-donor-delete-records'] ) ? true : false;
+	$confirm     = ! empty($args['give-donor-delete-confirm']) ? true : false;
+	$remove_data = ! empty($args['give-donor-delete-records']) ? true : false;
 	$nonce       = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'delete-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'delete-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( ! $confirm ) {
-		give_set_error( 'donor-delete-no-confirm', __( 'Please confirm you want to delete this donor.', 'give' ) );
+	if ( ! $confirm) {
+		give_set_error('donor-delete-no-confirm', __('Please confirm you want to delete this donor.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
-		wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id ) );
+	if (give_get_errors()) {
+		wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id));
 		exit;
 	}
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
 	/**
 	 * Fires before deleting donor.
@@ -270,50 +270,50 @@ 
 	 * @param bool $confirm     Delete confirmation.
 	 * @param bool $remove_data Records delete confirmation.
 	 */
-	do_action( 'give_pre_delete_donor', $donor_id, $confirm, $remove_data );
+	do_action('give_pre_delete_donor', $donor_id, $confirm, $remove_data);
 
-	if ( $donor->id > 0 ) {
+	if ($donor->id > 0) {
 
-		$payments_array = explode( ',', $donor->payment_ids );
-		$success        = Give()->donors->delete( $donor->id );
+		$payments_array = explode(',', $donor->payment_ids);
+		$success        = Give()->donors->delete($donor->id);
 
-		if ( $success ) {
+		if ($success) {
 
-			if ( $remove_data ) {
+			if ($remove_data) {
 
 				// Remove all donations, logs, etc.
-				foreach ( $payments_array as $payment_id ) {
-					give_delete_donation( $payment_id );
+				foreach ($payments_array as $payment_id) {
+					give_delete_donation($payment_id);
 				}
 			} else {
 
 				// Just set the donations to customer_id of 0.
-				foreach ( $payments_array as $payment_id ) {
-					give_update_payment_meta( $payment_id, '_give_payment_customer_id', 0 );
+				foreach ($payments_array as $payment_id) {
+					give_update_payment_meta($payment_id, '_give_payment_customer_id', 0);
 				}
 			}
 
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&give-message=donor-deleted' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&give-message=donor-deleted');
 
 		} else {
 
-			give_set_error( 'give-donor-delete-failed', __( 'Error deleting donor.', 'give' ) );
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor_id );
+			give_set_error('give-donor-delete-failed', __('Error deleting donor.', 'give'));
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor_id);
 
 		}
 	} else {
 
-		give_set_error( 'give-donor-delete-invalid-id', __( 'Invalid Donor ID.', 'give' ) );
-		$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors' );
+		give_set_error('give-donor-delete-invalid-id', __('Invalid Donor ID.', 'give'));
+		$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors');
 
 	}
 
-	wp_redirect( $redirect );
+	wp_redirect($redirect);
 	exit;
 
 }
 
-add_action( 'give_delete-donor', 'give_donor_delete', 10, 1 );
+add_action('give_delete-donor', 'give_donor_delete', 10, 1);
 
 /**
  * Disconnect a user ID from a donor
@@ -324,17 +324,17 @@ 
  *
  * @return bool|array        If the disconnect was successful.
  */
-function give_disconnect_donor_user_id( $args ) {
+function give_disconnect_donor_user_id($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -342,14 +342,14 @@ 
 
 	$nonce = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
-	if ( empty( $donor->id ) ) {
+	$donor = new Give_Donor($donor_id);
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -363,34 +363,34 @@ 
 	 * @param int $donor_id The ID of the donor.
 	 * @param int $user_id  The ID of the user.
 	 */
-	do_action( 'give_pre_donor_disconnect_user_id', $donor_id, $user_id );
+	do_action('give_pre_donor_disconnect_user_id', $donor_id, $user_id);
 
 	$output     = array();
 	$donor_args = array(
 		'user_id' => 0,
 	);
 
-	if ( $donor->update( $donor_args ) ) {
+	if ($donor->update($donor_args)) {
 		global $wpdb;
 
-		if ( ! empty( $donor->payment_ids ) ) {
-			$wpdb->query( "UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $donor->payment_ids )" );
+		if ( ! empty($donor->payment_ids)) {
+			$wpdb->query("UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $donor->payment_ids )");
 		}
 
 		// Set Donor Disconnection status true, if user and donor are disconnected with each other.
-		update_user_meta( $user_id, '_give_is_donor_disconnected', true );
-		update_user_meta( $user_id, '_give_disconnected_donor_id', $donor->id );
-		$donor->update_meta( '_give_disconnected_user_id', $user_id );
+		update_user_meta($user_id, '_give_is_donor_disconnected', true);
+		update_user_meta($user_id, '_give_disconnected_donor_id', $donor->id);
+		$donor->update_meta('_give_disconnected_user_id', $user_id);
 
 		$output['success'] = true;
 
 	} else {
 
 		$output['success'] = false;
-		give_set_error( 'give-disconnect-user-fail', __( 'Failed to disconnect user from donor.', 'give' ) );
+		give_set_error('give-disconnect-user-fail', __('Failed to disconnect user from donor.', 'give'));
 	}
 
-	$output['redirect'] = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' ) . $donor_id;
+	$output['redirect'] = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id=').$donor_id;
 
 	/**
 	 * Fires after disconnecting user ID from a donor.
@@ -399,11 +399,11 @@ 
 	 *
 	 * @param int $donor_id The ID of the donor.
 	 */
-	do_action( 'give_post_donor_disconnect_user_id', $donor_id );
+	do_action('give_post_donor_disconnect_user_id', $donor_id);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
@@ -411,7 +411,7 @@ 
 
 }
 
-add_action( 'give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1 );
+add_action('give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1);
 
 /**
  * Add an email address to the donor from within the admin and log a donor note.
@@ -422,90 +422,90 @@ 
  *
  * @return mixed        If DOING_AJAX echos out JSON, otherwise returns array of success (bool) and message (string).
  */
-function give_add_donor_email( $args ) {
+function give_add_donor_email($args) {
 	/**
 	 * Define variable
 	 *
 	 * @since 1.8.14
 	 */
 	$donor_id = '';
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
 	$output = array();
-	if ( empty( $args ) || empty( $args['email'] ) || empty( $args['customer_id'] ) ) {
+	if (empty($args) || empty($args['email']) || empty($args['customer_id'])) {
 		$output['success'] = false;
-		if ( empty( $args['email'] ) ) {
-			$output['message'] = __( 'Email address is required.', 'give' );
-		} elseif ( empty( $args['customer_id'] ) ) {
-			$output['message'] = __( 'Donor ID is required.', 'give' );
+		if (empty($args['email'])) {
+			$output['message'] = __('Email address is required.', 'give');
+		} elseif (empty($args['customer_id'])) {
+			$output['message'] = __('Donor ID is required.', 'give');
 		} else {
-			$output['message'] = __( 'An error has occurred. Please try again.', 'give' );
+			$output['message'] = __('An error has occurred. Please try again.', 'give');
 		}
-	} elseif ( ! wp_verify_nonce( $args['_wpnonce'], 'give_add_donor_email' ) ) {
+	} elseif ( ! wp_verify_nonce($args['_wpnonce'], 'give_add_donor_email')) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Nonce verification failed.', 'give' ),
+			'message' => __('Nonce verification failed.', 'give'),
 		);
-	} elseif ( ! is_email( $args['email'] ) ) {
+	} elseif ( ! is_email($args['email'])) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Invalid email.', 'give' ),
+			'message' => __('Invalid email.', 'give'),
 		);
 	} else {
-		$email    = sanitize_email( $args['email'] );
+		$email    = sanitize_email($args['email']);
 		$donor_id = (int) $args['customer_id'];
 		$primary  = 'true' === $args['primary'] ? true : false;
-		$donor    = new Give_Donor( $donor_id );
-		if ( false === $donor->add_email( $email, $primary ) ) {
-			if ( in_array( $email, $donor->emails ) ) {
+		$donor    = new Give_Donor($donor_id);
+		if (false === $donor->add_email($email, $primary)) {
+			if (in_array($email, $donor->emails)) {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email already associated with this donor.', 'give' ),
+					'message' => __('Email already associated with this donor.', 'give'),
 				);
 			} else {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email address is already associated with another donor.', 'give' ),
+					'message' => __('Email address is already associated with another donor.', 'give'),
 				);
 			}
 		} else {
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&give-message=email-added' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&give-message=email-added');
 			$output   = array(
 				'success'  => true,
-				'message'  => __( 'Email successfully added to donor.', 'give' ),
+				'message'  => __('Email successfully added to donor.', 'give'),
 				'redirect' => $redirect,
 			);
 
 			$user       = wp_get_current_user();
-			$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-			$donor_note = sprintf( __( 'Email address %1$s added by %2$s', 'give' ), $email, $user_login );
-			$donor->add_note( $donor_note );
+			$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+			$donor_note = sprintf(__('Email address %1$s added by %2$s', 'give'), $email, $user_login);
+			$donor->add_note($donor_note);
 
-			if ( $primary ) {
-				$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $email, $user_login );
-				$donor->add_note( $donor_note );
+			if ($primary) {
+				$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $email, $user_login);
+				$donor->add_note($donor_note);
 			}
 		}
 	}// End if().
 
-	do_action( 'give_post_add_donor_email', $donor_id, $args );
+	do_action('give_post_add_donor_email', $donor_id, $args);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
 	return $output;
 }
 
-add_action( 'give_add_donor_email', 'give_add_donor_email', 10, 1 );
+add_action('give_add_donor_email', 'give_add_donor_email', 10, 1);
 
 
 /**
@@ -515,39 +515,39 @@ 
  * @return bool|null
  */
 function give_remove_donor_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
-	if ( ! wp_verify_nonce( $nonce, 'give-remove-donor-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-remove-donor-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
-	if ( $donor->remove_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-message', 'email-removed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	$donor = new Give_Donor($_GET['id']);
+	if ($donor->remove_email($_GET['email'])) {
+		$url        = add_query_arg('give-message', 'email-removed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s removed by %2$s', 'give' ), $_GET['email'], $user_login );
-		$donor->add_note( $donor_note );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s removed by %2$s', 'give'), $_GET['email'], $user_login);
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-message', 'email-remove-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-message', 'email-remove-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_remove_donor_email', 'give_remove_donor_email', 10 );
+add_action('give_remove_donor_email', 'give_remove_donor_email', 10);
 
 
 /**
@@ -558,41 +558,41 @@ 
  * @return bool|null
  */
 function give_set_donor_primary_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'give-set-donor-primary-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-set-donor-primary-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
+	$donor = new Give_Donor($_GET['id']);
 
-	if ( $donor->set_primary_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-message', 'primary-email-updated', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	if ($donor->set_primary_email($_GET['email'])) {
+		$url        = add_query_arg('give-message', 'primary-email-updated', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $_GET['email'], $user_login );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $_GET['email'], $user_login);
 
-		$donor->add_note( $donor_note );
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-message', 'primary-email-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-message', 'primary-email-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_set_donor_primary_email', 'give_set_donor_primary_email', 10 );
+add_action('give_set_donor_primary_email', 'give_set_donor_primary_email', 10);

includes/admin/donors/donors.php

Spacing +170 added lines, -170 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,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();
 	}
@@ -42,7 +42,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -56,7 +56,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -67,7 +67,7 @@ 
  * @return void
  */
 function give_donors_list() {
-	include dirname( __FILE__ ) . '/class-donor-table.php';
+	include dirname(__FILE__).'/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -80,14 +80,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
 
 		<hr class="wp-header-end">
 
-		<form id="give-donors-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
+		<form id="give-donors-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' );
+			$donors_table->search_box(__('Search Donors', 'give'), 'give-donors');
 			$donors_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms" />
@@ -100,7 +100,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -116,33 +116,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;
 	}
 
@@ -151,9 +151,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; ?>
 
@@ -161,7 +161,7 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor number */
-				esc_html__( 'Donor %s', 'give' ),
+				esc_html__('Donor %s', 'give'),
 				$donor_id
 			);
 			?>
@@ -169,26 +169,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; ?>
@@ -208,9 +208,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.
@@ -219,32 +219,32 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 	?>
 
 	<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 ); ?>">
+		<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); ?>">
 
 			<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">
 						<span class="donor-id">#<?php echo $donor->id; ?></span>
-						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr( $donor->name ); ?>" placeholder="<?php _e( 'Donor Name', 'give' ); ?>" /></span>
+						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr($donor->name); ?>" placeholder="<?php _e('Donor Name', 'give'); ?>" /></span>
 						<span class="donor-name info-item editable"><span 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 ) ) : ?>
-						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+					<?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>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -254,7 +254,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
@@ -271,48 +271,48 @@ 
 										'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 } ?>
-									<?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) { ?>
 										<span class="disconnect-user">
  											-
- 											<a id="disconnect-donor" href="#disconnect" aria-label="<?php _e( 'Disconnects the current user ID from this donor record.', 'give' ); ?>">
- 												<?php _e( 'Disconnect User', 'give' ); ?>
+ 											<a id="disconnect-donor" href="#disconnect" aria-label="<?php _e('Disconnects the current user ID from this donor record.', 'give'); ?>">
+ 												<?php _e('Disconnect User', 'give'); ?>
 											</a>
  										</span>
 										<span class="view-user-profile">
  											|
- 											<a id="view-user-profile" href="<?php echo 'user-edit.php?user_id=' . $donor->user_id; ?>" aria-label="<?php _e( 'View User Profile of current user ID.', 'give' ); ?>">
- 												<?php _e( 'View User Profile', 'give' ); ?>
+ 											<a id="view-user-profile" href="<?php echo 'user-edit.php?user_id='.$donor->user_id; ?>" aria-label="<?php _e('View User Profile of current user ID.', 'give'); ?>">
+ 												<?php _e('View User Profile', 'give'); ?>
 											</a>
  										</span>
 									<?php } ?>
 								</span>
 							</td>
 						</tr>
-						<?php if ( isset( $donor->user_id ) && $donor->user_id > 0 ) : ?>
+						<?php if (isset($donor->user_id) && $donor->user_id > 0) : ?>
 
 							<tr>
-								<th scope="col"><?php _e( 'Address:', 'give' ); ?></th>
+								<th scope="col"><?php _e('Address:', 'give'); ?></th>
 								<td class="row-title">
 
 									<div class="donor-address-wrapper">
 
 										<?php
-										$address  = get_user_meta( $donor->user_id, '_give_user_address', true );
+										$address  = get_user_meta($donor->user_id, '_give_user_address', true);
 										$defaults = array(
 											'line1'   => '',
 											'line2'   => '',
@@ -322,10 +322,10 @@ 
 											'zip'     => '',
 										);
 
-										$address = wp_parse_args( $address, $defaults );
+										$address = wp_parse_args($address, $defaults);
 										?>
 
-										<?php if ( ! empty( $address ) ) { ?>
+										<?php if ( ! empty($address)) { ?>
 											<span class="donor-address info-item editable">
 												<span class="info-item" data-key="line1"><?php echo $address['line1']; ?></span>
 												<span class="info-item" data-key="line2"><?php echo $address['line2']; ?></span>
@@ -342,8 +342,8 @@ 
 
 										// For State.
 										$selected_state = give_get_state();
-										$states         = give_get_states( $selected_country );
-										$selected_state = ( isset( $address['state'] ) ? $address['state'] : $selected_state );
+										$states         = give_get_states($selected_country);
+										$selected_state = (isset($address['state']) ? $address['state'] : $selected_state);
 
 										// Get the country list that does not have any states init.
 										$no_states_country = give_no_states_country_list();
@@ -351,32 +351,32 @@ 
 										<span class="donor-address info-item edit-item">
 											<select data-key="country" name="customerinfo[country]" id="billing_country" class="billing_country give-select edit-item">
 												<?php
-												foreach ( $countries as $country_code => $country ) {
-													echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+												foreach ($countries as $country_code => $country) {
+													echo '<option value="'.esc_attr($country_code).'"'.selected($country_code, $selected_country, false).'>'.$country.'</option>';
 												}
 												?>
 											</select>
-											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php _e( 'Address 1', 'give' ); ?>" value="<?php echo $address['line1']; ?>" />
-											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php _e( 'Address 2', 'give' ); ?>" value="<?php echo $address['line2']; ?>" />
+											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php _e('Address 1', 'give'); ?>" value="<?php echo $address['line1']; ?>" />
+											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php _e('Address 2', 'give'); ?>" value="<?php echo $address['line2']; ?>" />
 											<?php
-											if ( ! empty( $states ) ) {
+											if ( ! empty($states)) {
 												?>
 												<select data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-select info-item">
 													<?php
-													foreach ( $states as $state_code => $state ) {
-														echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+													foreach ($states as $state_code => $state) {
+														echo '<option value="'.$state_code.'"'.selected($state_code, $selected_state, false).'>'.$state.'</option>';
 													}
 													?>
 												</select>
 												<?php
 											} else {
 												?>
-												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item <?php echo ( ! empty( $selected_country ) && array_key_exists( $selected_country, $no_states_country ) ? 'give-hidden' : '' ); ?>" placeholder="<?php _e( 'State / Province / County', 'give' ); ?>" />
+												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item <?php echo ( ! empty($selected_country) && array_key_exists($selected_country, $no_states_country) ? 'give-hidden' : ''); ?>" placeholder="<?php _e('State / Province / County', 'give'); ?>" />
 												<?php
 											}
 											?>
-											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php _e( 'City', 'give' ); ?>" value="<?php echo $address['city']; ?>" />
-											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php _e( 'Zip / Postal Code', 'give' ); ?>" value="<?php echo $address['zip']; ?>" />
+											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php _e('City', 'give'); ?>" value="<?php echo $address['city']; ?>" />
+											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php _e('Zip / Postal Code', 'give'); ?>" value="<?php echo $address['zip']; ?>" />
 										</span>
 
 									</div>
@@ -392,10 +392,10 @@ 
 
 			<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>
+				<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>
 			</span>
 
 		</form>
@@ -410,24 +410,24 @@ 
 	 *
 	 * @param object $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&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&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->purchase_value, array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->purchase_value, array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -439,7 +439,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>
@@ -452,7 +452,7 @@ 
 	 *
 	 * @param object $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">
@@ -465,46 +465,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>
@@ -514,59 +514,59 @@ 
 					<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 ); ?>
-							<input type="email" name="additional-email" value="" 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>
-							<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+							<?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;
+							<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>
+							<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 							<span class="spinner"></span>
 						</div>
 						<div class="notice-wrap"></div>
 					</td>
 				</tr>
 			<?php } else { ?>
-				<tr><td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td></tr>
+				<tr><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_payment_amount( $payment->ID ); ?></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_payment_amount($payment->ID); ?></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')
 							);
 							?>
 
@@ -581,47 +581,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>
@@ -629,7 +629,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>
@@ -643,7 +643,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -656,7 +656,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -669,30 +669,30 @@ 
  *
  * @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 ); ?>">
+				<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); ?>">
 					<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>
@@ -707,26 +707,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>
 
@@ -742,9 +742,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.
@@ -753,15 +753,15 @@ 
 	 *
 	 * @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 ); ?>">
+		<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); ?>">
 
 			<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>
 
 
@@ -769,20 +769,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
@@ -795,16 +795,16 @@ 
 					 *
 					 * @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' ); ?>" />
-					<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>
+					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" 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>
 				</span>
 
 			</div>
@@ -820,5 +820,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/donor-functions.php

Spacing +28 added lines, -28 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title' => __( 'Donor Profile', 'give' ),
+			'title' => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title' => __( 'Donor Notes', 'give' ),
+			'title' => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,17 +73,17 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title'    => __( 'Delete Donor', 'give' ),
+		'title'    => __('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);
 
 /**
  * Connect and Reconnect Donor with User profile.
@@ -96,7 +96,7 @@ 
  *
  * @return array
  */
-function give_connect_user_donor_profile( $donor, $donor_data, $address ) {
+function give_connect_user_donor_profile($donor, $donor_data, $address) {
 
 	$donor_id         = $donor->id;
 	$previous_user_id = $donor->user_id;
@@ -110,40 +110,40 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
-		if ( ! empty( $donor->user_id ) && $donor->user_id > 0 ) {
-			update_user_meta( $donor->user_id, '_give_user_address', $address );
+		if ( ! empty($donor->user_id) && $donor->user_id > 0) {
+			update_user_meta($donor->user_id, '_give_user_address', $address);
 		}
 
 		// Update some donation meta if we need to.
-		$payments_array = explode( ',', $donor->payment_ids );
+		$payments_array = explode(',', $donor->payment_ids);
 
-		if ( $donor->user_id !== $previous_user_id ) {
-			foreach ( $payments_array as $payment_id ) {
-				give_update_payment_meta( $payment_id, '_give_payment_user_id', $donor->user_id );
+		if ($donor->user_id !== $previous_user_id) {
+			foreach ($payments_array as $payment_id) {
+				give_update_payment_meta($payment_id, '_give_payment_user_id', $donor->user_id);
 			}
 		}
 
 		// Fetch disconnected user id, if exists.
-		$disconnected_user_id = $donor->get_meta( '_give_disconnected_user_id', true );
+		$disconnected_user_id = $donor->get_meta('_give_disconnected_user_id', true);
 
 		// Flag User and Donor Disconnection.
-		delete_user_meta( $disconnected_user_id, '_give_is_donor_disconnected' );
+		delete_user_meta($disconnected_user_id, '_give_is_donor_disconnected');
 
 		// Check whether the disconnected user id and the reconnected user id are same or not.
 		// If both are same then delete user id store in donor meta.
-		if( $donor_data['user_id'] === $disconnected_user_id ) {
-			delete_user_meta( $disconnected_user_id, '_give_disconnected_donor_id' );
-			$donor->delete_meta( '_give_disconnected_user_id' );
+		if ($donor_data['user_id'] === $disconnected_user_id) {
+			delete_user_meta($disconnected_user_id, '_give_disconnected_donor_id');
+			$donor->delete_meta('_give_disconnected_user_id');
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -160,7 +160,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
 
 	return $output;

includes/admin/class-admin-settings.php

Indentation +5 added lines, -5 removed lines

@@ -93,8 +93,8 @@ 
 
 			// Show error message if Akismet not configured and Admin try to save 'enabled' option.
 			if ( isset( $_POST['akismet_spam_protection'] )
-			     && give_is_setting_enabled( $_POST['akismet_spam_protection'] )
-			     && ! give_check_akismet_key()
+				 && give_is_setting_enabled( $_POST['akismet_spam_protection'] )
+				 && ! give_check_akismet_key()
 			) {
 				self::add_error( 'give-akismet-protection', __( 'Please properly configure Akismet to enable SPAM protection.', 'give' ) );
 
@@ -763,9 +763,9 @@ 
                         <td class="give-docs-link" colspan="2">
 							<?php
 							echo '<p class="give-docs-link"><a href="' . esc_url( $value['url'] )
-							     . '" target="_blank">'
-							     . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
-							     . '<span class="dashicons dashicons-editor-help"></span></a></p>';
+								 . '" target="_blank">'
+								 . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
+								 . '<span class="dashicons dashicons-editor-help"></span></a></p>';
 							?>
                         </td>
                         </tr><?php

Spacing +218 added lines, -218 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Admin_Settings' ) ) :
+if ( ! class_exists('Give_Admin_Settings')) :
 
 	/**
 	 * Give_Admin_Settings Class.
@@ -72,7 +72,7 @@ 
 			 *
 			 * @param array $settings Array of settings class object.
 			 */
-			self::$settings = apply_filters( self::$setting_filter_prefix . '_get_settings_pages', array() );
+			self::$settings = apply_filters(self::$setting_filter_prefix.'_get_settings_pages', array());
 
 			return self::$settings;
 		}
@@ -86,17 +86,17 @@ 
 		public static function save() {
 			$current_tab = give_get_current_setting_tab();
 
-			if ( empty( $_REQUEST['_give-save-settings'] ) || ! wp_verify_nonce( $_REQUEST['_give-save-settings'], 'give-save-settings' ) ) {
-				echo '<div class="notice error"><p>' . __( 'Action failed. Please refresh the page and retry.', 'give' ) . '</p></div>';
+			if (empty($_REQUEST['_give-save-settings']) || ! wp_verify_nonce($_REQUEST['_give-save-settings'], 'give-save-settings')) {
+				echo '<div class="notice error"><p>'.__('Action failed. Please refresh the page and retry.', 'give').'</p></div>';
 				die();
 			}
 
 			// Show error message if Akismet not configured and Admin try to save 'enabled' option.
-			if ( isset( $_POST['akismet_spam_protection'] )
-			     && give_is_setting_enabled( $_POST['akismet_spam_protection'] )
+			if (isset($_POST['akismet_spam_protection'])
+			     && give_is_setting_enabled($_POST['akismet_spam_protection'])
 			     && ! give_check_akismet_key()
 			) {
-				self::add_error( 'give-akismet-protection', __( 'Please properly configure Akismet to enable SPAM protection.', 'give' ) );
+				self::add_error('give-akismet-protection', __('Please properly configure Akismet to enable SPAM protection.', 'give'));
 
 				return;
 			}
@@ -110,9 +110,9 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_save_' . $current_tab );
+			do_action(self::$setting_filter_prefix.'_save_'.$current_tab);
 
-			self::add_message( 'give-setting-updated', __( 'Your settings have been saved.', 'give' ) );
+			self::add_message('give-setting-updated', __('Your settings have been saved.', 'give'));
 
 			/**
 			 * Trigger Action.
@@ -123,7 +123,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_saved' );
+			do_action(self::$setting_filter_prefix.'_saved');
 		}
 
 		/**
@@ -136,8 +136,8 @@ 
 		 *
 		 * @return void
 		 */
-		public static function add_message( $code, $message ) {
-			self::$messages[ $code ] = $message;
+		public static function add_message($code, $message) {
+			self::$messages[$code] = $message;
 		}
 
 		/**
@@ -150,8 +150,8 @@ 
 		 *
 		 * @return void
 		 */
-		public static function add_error( $code, $message ) {
-			self::$errors[ $code ] = $message;
+		public static function add_error($code, $message) {
+			self::$errors[$code] = $message;
 		}
 
 		/**
@@ -164,18 +164,18 @@ 
 			$notice_html = '';
 			$classes     = 'give-notice settings-error notice is-dismissible';
 
-			self::$errors   = apply_filters( self::$setting_filter_prefix . '_error_notices', self::$errors );
-			self::$messages = apply_filters( self::$setting_filter_prefix . '_update_notices', self::$messages );
+			self::$errors   = apply_filters(self::$setting_filter_prefix.'_error_notices', self::$errors);
+			self::$messages = apply_filters(self::$setting_filter_prefix.'_update_notices', self::$messages);
 
-			if ( 0 < count( self::$errors ) ) {
-				foreach ( self::$errors as $code => $message ) {
-					$notice_html .= '<div id="setting-error-' . $code . '" class="' . $classes . ' error"><p><strong>' . $message . '</strong></p></div>';
+			if (0 < count(self::$errors)) {
+				foreach (self::$errors as $code => $message) {
+					$notice_html .= '<div id="setting-error-'.$code.'" class="'.$classes.' error"><p><strong>'.$message.'</strong></p></div>';
 				}
 			}
 
-			if ( 0 < count( self::$messages ) ) {
-				foreach ( self::$messages as $code => $message ) {
-					$notice_html .= '<div id="setting-error-' . $code . '" class="' . $classes . ' updated"><p><strong>' . $message . '</strong></p></div>';
+			if (0 < count(self::$messages)) {
+				foreach (self::$messages as $code => $message) {
+					$notice_html .= '<div id="setting-error-'.$code.'" class="'.$classes.' updated"><p><strong>'.$message.'</strong></p></div>';
 				}
 			}
 
@@ -195,7 +195,7 @@ 
 			self::$setting_filter_prefix = give_get_current_setting_page();
 
 			// Bailout: Exit if setting page is not defined.
-			if ( empty( self::$setting_filter_prefix ) ) {
+			if (empty(self::$setting_filter_prefix)) {
 				return false;
 			}
 
@@ -208,7 +208,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_start' );
+			do_action(self::$setting_filter_prefix.'_start');
 
 			$current_tab = give_get_current_setting_tab();
 
@@ -216,7 +216,7 @@ 
 			self::get_settings_pages();
 
 			// Save settings if data has been posted.
-			if ( ! empty( $_POST ) ) {
+			if ( ! empty($_POST)) {
 				self::save();
 			}
 
@@ -229,7 +229,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			$tabs = apply_filters( self::$setting_filter_prefix . '_tabs_array', array() );
+			$tabs = apply_filters(self::$setting_filter_prefix.'_tabs_array', array());
 
 			include 'views/html-admin-settings.php';
 
@@ -247,25 +247,25 @@ 
 		 *
 		 * @return string|bool
 		 */
-		public static function get_option( $option_name = '', $field_id = '', $default = false ) {
+		public static function get_option($option_name = '', $field_id = '', $default = false) {
 			// Bailout.
-			if ( empty( $option_name ) && empty( $field_id ) ) {
+			if (empty($option_name) && empty($field_id)) {
 				return false;
 			}
 
-			if ( ! empty( $field_id ) && ! empty( $option_name ) ) {
+			if ( ! empty($field_id) && ! empty($option_name)) {
 				// Get field value if any.
-				$option_value = get_option( $option_name );
+				$option_value = get_option($option_name);
 
-				$option_value = ( is_array( $option_value ) && array_key_exists( $field_id, $option_value ) )
-					? $option_value[ $field_id ]
+				$option_value = (is_array($option_value) && array_key_exists($field_id, $option_value))
+					? $option_value[$field_id]
 					: $default;
 			} else {
 				// If option name is empty but not field name then this means, setting is direct store to option table under there field name.
 				$option_name = ! $option_name ? $field_id : $option_name;
 
 				// Get option value if any.
-				$option_value = get_option( $option_name, $default );
+				$option_value = get_option($option_name, $default);
 			}
 
 			return $option_value;
@@ -283,7 +283,7 @@ 
 		 *
 		 * @return void
 		 */
-		public static function output_fields( $options, $option_name = '' ) {
+		public static function output_fields($options, $option_name = '') {
 			$current_tab = give_get_current_setting_tab();
 
 			// Field Default values.
@@ -296,52 +296,52 @@ 
 				'table_html' => true,
 			);
 
-			foreach ( $options as $value ) {
-				if ( ! isset( $value['type'] ) ) {
+			foreach ($options as $value) {
+				if ( ! isset($value['type'])) {
 					continue;
 				}
 
 				// Set title.
-				$defaults['title'] = isset( $value['name'] ) ? $value['name'] : '';
+				$defaults['title'] = isset($value['name']) ? $value['name'] : '';
 
 				// Set default setting.
-				$value = wp_parse_args( $value, $defaults );
+				$value = wp_parse_args($value, $defaults);
 
 				// Colorpicker field.
-				$value['class'] = ( 'colorpicker' === $value['type'] ? trim( $value['class'] ) . ' give-colorpicker' : $value['class'] );
-				$value['type']  = ( 'colorpicker' === $value['type'] ? 'text' : $value['type'] );
+				$value['class'] = ('colorpicker' === $value['type'] ? trim($value['class']).' give-colorpicker' : $value['class']);
+				$value['type']  = ('colorpicker' === $value['type'] ? 'text' : $value['type']);
 
 
 				// Custom attribute handling.
 				$custom_attributes = array();
 
-				if ( ! empty( $value['attributes'] ) && is_array( $value['attributes'] ) ) {
-					foreach ( $value['attributes'] as $attribute => $attribute_value ) {
-						$custom_attributes[] = esc_attr( $attribute ) . '="' . esc_attr( $attribute_value ) . '"';
+				if ( ! empty($value['attributes']) && is_array($value['attributes'])) {
+					foreach ($value['attributes'] as $attribute => $attribute_value) {
+						$custom_attributes[] = esc_attr($attribute).'="'.esc_attr($attribute_value).'"';
 					}
 				}
 
 				// Description handling.
-				$description          = self::get_field_description( $value );
+				$description = self::get_field_description($value);
 
 				// Switch based on type.
-				switch ( $value['type'] ) {
+				switch ($value['type']) {
 
 					// Section Titles
 					case 'title':
-						if ( ! empty( $value['title'] ) ) {
-							echo '<div class="give-setting-tab-header give-setting-tab-header-' . $current_tab . '"><h2>' . self::get_field_title( $value ) . '</h2><hr></div>';
+						if ( ! empty($value['title'])) {
+							echo '<div class="give-setting-tab-header give-setting-tab-header-'.$current_tab.'"><h2>'.self::get_field_title($value).'</h2><hr></div>';
 						}
 
-						if ( ! empty( $value['desc'] ) ) {
-							echo wpautop( wptexturize( wp_kses_post( $value['desc'] ) ) );
+						if ( ! empty($value['desc'])) {
+							echo wpautop(wptexturize(wp_kses_post($value['desc'])));
 						}
 
-						if ( $value['table_html'] ) {
-							echo '<table class="form-table give-setting-tab-body give-setting-tab-body-' . $current_tab . '">' . "\n\n";
+						if ($value['table_html']) {
+							echo '<table class="form-table give-setting-tab-body give-setting-tab-body-'.$current_tab.'">'."\n\n";
 						}
 
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -350,14 +350,14 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) );
+							do_action('give_settings_'.sanitize_title($value['id']));
 						}
 
 						break;
 
 					// Section Ends.
 					case 'sectionend':
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -366,14 +366,14 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) . '_end' );
+							do_action('give_settings_'.sanitize_title($value['id']).'_end');
 						}
 
-						if ( $value['table_html'] ) {
+						if ($value['table_html']) {
 							echo '</table>';
 						}
 
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -382,7 +382,7 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) . '_after' );
+							do_action('give_settings_'.sanitize_title($value['id']).'_after');
 						}
 
 						break;
@@ -390,27 +390,27 @@ 
 					// Standard text inputs and subtypes like 'number'.
 					case 'colorpicker':
 					case 'hidden' :
-						$value['wrapper_class'] = empty( $value['wrapper_class'] ) ? 'give-hidden' : trim( $value['wrapper_class'] ) . ' give-hidden';
+						$value['wrapper_class'] = empty($value['wrapper_class']) ? 'give-hidden' : trim($value['wrapper_class']).' give-hidden';
 					case 'text':
 					case 'email':
 					case 'number':
 					case 'password' :
 						$type = $value['type'];
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-						<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+						<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 							<th scope="row" class="titledesc">
-								<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+								<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 							</th>
-							<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+							<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 								<input
-										name="<?php echo esc_attr( $value['id'] ); ?>"
-										id="<?php echo esc_attr( $value['id'] ); ?>"
-										type="<?php echo esc_attr( $type ); ?>"
-										style="<?php echo esc_attr( $value['css'] ); ?>"
-										value="<?php echo esc_attr( $option_value ); ?>"
-										class="give-input-field<?php echo( empty( $value['class'] ) ? '' : ' ' . esc_attr( $value['class'] ) ); ?>"
-									<?php echo implode( ' ', $custom_attributes ); ?>
+										name="<?php echo esc_attr($value['id']); ?>"
+										id="<?php echo esc_attr($value['id']); ?>"
+										type="<?php echo esc_attr($type); ?>"
+										style="<?php echo esc_attr($value['css']); ?>"
+										value="<?php echo esc_attr($option_value); ?>"
+										class="give-input-field<?php echo(empty($value['class']) ? '' : ' '.esc_attr($value['class'])); ?>"
+									<?php echo implode(' ', $custom_attributes); ?>
 								/> <?php echo $description; ?>
 							</td>
 						</tr>
@@ -420,23 +420,23 @@ 
 					// Textarea.
 					case 'textarea':
 
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 								<textarea
-                                        name="<?php echo esc_attr( $value['id'] ); ?>"
-                                        id="<?php echo esc_attr( $value['id'] ); ?>"
-                                        style="<?php echo esc_attr( $value['css'] ); ?>"
-                                        class="<?php echo esc_attr( $value['class'] ); ?>"
+                                        name="<?php echo esc_attr($value['id']); ?>"
+                                        id="<?php echo esc_attr($value['id']); ?>"
+                                        style="<?php echo esc_attr($value['css']); ?>"
+                                        class="<?php echo esc_attr($value['class']); ?>"
                                         rows="10"
                                         cols="60"
-									<?php echo implode( ' ', $custom_attributes ); ?>
-                                ><?php echo esc_textarea( $option_value ); ?></textarea>
+									<?php echo implode(' ', $custom_attributes); ?>
+                                ><?php echo esc_textarea($option_value); ?></textarea>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -446,35 +446,35 @@ 
 					case 'select' :
 					case 'multiselect' :
 
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
                             <select
-                                    name="<?php echo esc_attr( $value['id'] ); ?><?php if ( $value['type'] == 'multiselect' ) {
+                                    name="<?php echo esc_attr($value['id']); ?><?php if ($value['type'] == 'multiselect') {
 										echo '[]';
 									} ?>"
-                                    id="<?php echo esc_attr( $value['id'] ); ?>"
-                                    style="<?php echo esc_attr( $value['css'] ); ?>"
-                                    class="<?php echo esc_attr( $value['class'] ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
-								<?php echo ( 'multiselect' == $value['type'] ) ? 'multiple="multiple"' : ''; ?>
+                                    id="<?php echo esc_attr($value['id']); ?>"
+                                    style="<?php echo esc_attr($value['css']); ?>"
+                                    class="<?php echo esc_attr($value['class']); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
+								<?php echo ('multiselect' == $value['type']) ? 'multiple="multiple"' : ''; ?>
                             >
 
 								<?php
-								if ( ! empty( $value['options'] ) ) {
-									foreach ( $value['options'] as $key => $val ) {
+								if ( ! empty($value['options'])) {
+									foreach ($value['options'] as $key => $val) {
 										?>
-                                        <option value="<?php echo esc_attr( $key ); ?>" <?php
+                                        <option value="<?php echo esc_attr($key); ?>" <?php
 
-										if ( is_array( $option_value ) ) {
-											selected( in_array( $key, $option_value ), true );
+										if (is_array($option_value)) {
+											selected(in_array($key, $option_value), true);
 										} else {
-											selected( $option_value, $key );
+											selected($option_value, $key);
 										}
 
 										?>><?php echo $val ?></option>
@@ -490,28 +490,28 @@ 
 
 					// Radio inputs.
 					case 'radio_inline' :
-						$value['class'] = empty( $value['class'] ) ? 'give-radio-inline' : $value['class'] . ' give-radio-inline';
+						$value['class'] = empty($value['class']) ? 'give-radio-inline' : $value['class'].' give-radio-inline';
 					case 'radio' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?> <?php echo( ! empty( $value['class'] ) ? $value['class'] : '' ); ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?> <?php echo( ! empty($value['class']) ? $value['class'] : ''); ?>">
                             <fieldset>
                                 <ul>
 									<?php
-									foreach ( $value['options'] as $key => $val ) {
+									foreach ($value['options'] as $key => $val) {
 										?>
                                         <li>
                                             <label><input
-                                                        name="<?php echo esc_attr( $value['id'] ); ?>"
+                                                        name="<?php echo esc_attr($value['id']); ?>"
                                                         value="<?php echo $key; ?>"
                                                         type="radio"
-                                                        style="<?php echo esc_attr( $value['css'] ); ?>"
-													<?php echo implode( ' ', $custom_attributes ); ?>
-													<?php checked( $key, $option_value ); ?>
+                                                        style="<?php echo esc_attr($value['css']); ?>"
+													<?php echo implode(' ', $custom_attributes); ?>
+													<?php checked($key, $option_value); ?>
                                                 /> <?php echo $val ?></label>
                                         </li>
 										<?php
@@ -525,21 +525,21 @@ 
 
 					// Checkbox input.
 					case 'checkbox' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                        <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                        <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                             <th scope="row" class="titledesc">
-                                <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                                <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                             </th>
                             <td class="give-forminp">
                                 <input
-                                        name="<?php echo esc_attr( $value['id'] ); ?>"
-                                        id="<?php echo esc_attr( $value['id'] ); ?>"
+                                        name="<?php echo esc_attr($value['id']); ?>"
+                                        id="<?php echo esc_attr($value['id']); ?>"
                                         type="checkbox"
-                                        class="<?php echo esc_attr( isset( $value['class'] ) ? $value['class'] : '' ); ?>"
+                                        class="<?php echo esc_attr(isset($value['class']) ? $value['class'] : ''); ?>"
                                         value="1"
-									<?php checked( $option_value, 'on' ); ?>
-									<?php echo implode( ' ', $custom_attributes ); ?>
+									<?php checked($option_value, 'on'); ?>
+									<?php echo implode(' ', $custom_attributes); ?>
                                 />
 								<?php echo $description; ?>
                             </td>
@@ -549,28 +549,28 @@ 
 
 					// Multi Checkbox input.
 					case 'multicheck' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$option_value = is_array( $option_value ) ? $option_value : array();
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$option_value = is_array($option_value) ? $option_value : array();
 						?>
-                        <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                        <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                             <th scope="row" class="titledesc">
-                                <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                                <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                             </th>
-                            <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?> <?php echo( ! empty( $value['class'] ) ? $value['class'] : '' ); ?>">
+                            <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?> <?php echo( ! empty($value['class']) ? $value['class'] : ''); ?>">
                                 <fieldset>
                                     <ul>
 										<?php
-										foreach ( $value['options'] as $key => $val ) {
+										foreach ($value['options'] as $key => $val) {
 											?>
                                             <li>
                                                 <label>
                                                     <input
-                                                            name="<?php echo esc_attr( $value['id'] ); ?>[]"
+                                                            name="<?php echo esc_attr($value['id']); ?>[]"
                                                             value="<?php echo $key; ?>"
                                                             type="checkbox"
-                                                            style="<?php echo esc_attr( $value['css'] ); ?>"
-														<?php echo implode( ' ', $custom_attributes ); ?>
-														<?php if ( in_array( $key, $option_value ) ) {
+                                                            style="<?php echo esc_attr($value['css']); ?>"
+														<?php echo implode(' ', $custom_attributes); ?>
+														<?php if (in_array($key, $option_value)) {
 															echo 'checked="checked"';
 														} ?>
                                                     /> <?php echo $val ?>
@@ -589,35 +589,35 @@ 
 					// File input field.
 					case 'file' :
 					case 'media' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$button_label = esc_html__( sprintf( 'Add or Upload %s', ( 'file' === $value['type'] ? 'File' : 'Image' ) ), 'give' );
-						$fvalue       = empty( $value['fvalue'] ) ? 'url' : $value['fvalue'];
-
-						$allow_media_preview_tags = array( 'jpg', 'jpeg', 'png', 'gif', 'ico' );
-						$preview_image_src        = $option_value ? ( 'id' === $fvalue ? wp_get_attachment_url( $option_value ) : $option_value ) : '#';
-						$preview_image_extension  = $preview_image_src ? pathinfo( $preview_image_src, PATHINFO_EXTENSION ) : '';
-						$is_show_preview = in_array( $preview_image_extension, $allow_media_preview_tags );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$button_label = esc_html__(sprintf('Add or Upload %s', ('file' === $value['type'] ? 'File' : 'Image')), 'give');
+						$fvalue       = empty($value['fvalue']) ? 'url' : $value['fvalue'];
+
+						$allow_media_preview_tags = array('jpg', 'jpeg', 'png', 'gif', 'ico');
+						$preview_image_src        = $option_value ? ('id' === $fvalue ? wp_get_attachment_url($option_value) : $option_value) : '#';
+						$preview_image_extension  = $preview_image_src ? pathinfo($preview_image_src, PATHINFO_EXTENSION) : '';
+						$is_show_preview = in_array($preview_image_extension, $allow_media_preview_tags);
 						?>
-						<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+						<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 							<th scope="row" class="titledesc">
-								<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+								<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 							</th>
 							<td class="give-forminp">
 								<div class="give-field-wrap">
 									<label for="<?php echo $value['id'] ?>">
 										<input
-												name="<?php echo esc_attr( $value['id'] ); ?>"
-												id="<?php echo esc_attr( $value['id'] ); ?>"
+												name="<?php echo esc_attr($value['id']); ?>"
+												id="<?php echo esc_attr($value['id']); ?>"
 												type="text"
-												class="give-input-field<?php echo esc_attr( isset( $value['class'] ) ? ' ' . $value['class'] : '' ); ?>"
+												class="give-input-field<?php echo esc_attr(isset($value['class']) ? ' '.$value['class'] : ''); ?>"
 												value="<?php echo $option_value; ?>"
-												style="<?php echo esc_attr( $value['css'] ); ?>"
-											<?php echo implode( ' ', $custom_attributes ); ?>
+												style="<?php echo esc_attr($value['css']); ?>"
+											<?php echo implode(' ', $custom_attributes); ?>
 										/>&nbsp;&nbsp;&nbsp;&nbsp;<input class="give-upload-button button" type="button" data-fvalue="<?php echo $fvalue; ?>" data-field-type="<?php echo $value['type']; ?>" value="<?php echo $button_label; ?>">
 										<?php echo $description ?>
 										<div class="give-image-thumb<?php echo ! $option_value || ! $is_show_preview ? ' give-hidden' : ''; ?>">
 											<span class="give-delete-image-thumb dashicons dashicons-no-alt"></span>
-											<img src="<?php echo $preview_image_src ; ?>" alt="">
+											<img src="<?php echo $preview_image_src; ?>" alt="">
 										</div>
 									</label>
 								</div>
@@ -629,17 +629,17 @@ 
 					// WordPress Editor.
 					case 'wysiwyg' :
 						// Get option value.
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						// Get editor settings.
-						$editor_settings = ! empty( $value['options'] ) ? $value['options'] : array();
+						$editor_settings = ! empty($value['options']) ? $value['options'] : array();
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
-							<?php wp_editor( $option_value, $value['id'], $editor_settings ); ?>
+							<?php wp_editor($option_value, $value['id'], $editor_settings); ?>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -648,9 +648,9 @@ 
 					// Custom: System setting field.
 					case 'system_info' :
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
 							<?php give_system_info_callback(); ?>
@@ -661,14 +661,14 @@ 
 
 					// Custom: Default gateways setting field.
 					case 'default_gateway' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
-							<?php give_default_gateway_callback( $value, $option_value ); ?>
+							<?php give_default_gateway_callback($value, $option_value); ?>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -676,14 +676,14 @@ 
 
 					// Custom: Enable gateways setting field.
 					case 'enabled_gateways' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
-							<?php give_enabled_gateways_callback( $value, $option_value ); ?>
+							<?php give_enabled_gateways_callback($value, $option_value); ?>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -692,9 +692,9 @@ 
 					// Custom: Email preview buttons field.
 					case 'email_preview_buttons' :
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
 							<?php give_email_preview_buttons_callback(); ?>
@@ -711,22 +711,22 @@ 
 
 					// Custom: Gateway API key.
 					case 'api_key' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$type         = ! empty( $option_value ) ? 'password' : 'text';
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$type         = ! empty($option_value) ? 'password' : 'text';
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
                             <input
-                                    name="<?php echo esc_attr( $value['id'] ); ?>"
-                                    id="<?php echo esc_attr( $value['id'] ); ?>"
-                                    type="<?php echo esc_attr( $type ); ?>"
-                                    style="<?php echo esc_attr( $value['css'] ); ?>"
-                                    value="<?php echo esc_attr( trim( $option_value ) ); ?>"
-                                    class="give-input-field<?php echo( empty( $value['class'] ) ? '' : ' ' . esc_attr( $value['class'] ) ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
+                                    name="<?php echo esc_attr($value['id']); ?>"
+                                    id="<?php echo esc_attr($value['id']); ?>"
+                                    type="<?php echo esc_attr($type); ?>"
+                                    style="<?php echo esc_attr($value['css']); ?>"
+                                    value="<?php echo esc_attr(trim($option_value)); ?>"
+                                    class="give-input-field<?php echo(empty($value['class']) ? '' : ' '.esc_attr($value['class'])); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
                             /> <?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -743,7 +743,7 @@ 
 						 *
 						 * @since 1.0
 						 */
-						do_action( "give_logs_view_{$current_section}" );
+						do_action("give_logs_view_{$current_section}");
 
 						echo $description;
 						break;
@@ -751,7 +751,7 @@ 
 					// Custom: Data field.
 					case 'data' :
 
-						include  GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-data.php';
+						include  GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-data.php';
 
 						echo $description;
 						break;
@@ -759,12 +759,12 @@ 
 					// Custom: Give Docs Link field type.
 					case 'give_docs_link' :
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <td class="give-docs-link" colspan="2">
 							<?php
-							echo '<p class="give-docs-link"><a href="' . esc_url( $value['url'] )
+							echo '<p class="give-docs-link"><a href="'.esc_url($value['url'])
 							     . '" target="_blank">'
-							     . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
+							     . sprintf(esc_html__('Need Help? See docs on "%s"', 'give'), $value['title'])
 							     . '<span class="dashicons dashicons-editor-help"></span></a></p>';
 							?>
                         </td>
@@ -775,8 +775,8 @@ 
 					// You can add or handle your custom field action.
 					default:
 						// Get option value.
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						do_action( 'give_admin_field_' . $value['type'], $value, $option_value );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						do_action('give_admin_field_'.$value['type'], $value, $option_value);
 						break;
 				}
 			}
@@ -792,15 +792,15 @@ 
 		 *
 		 * @return string The HTML description of the field.
 		 */
-		public static function get_field_description( $value ) {
+		public static function get_field_description($value) {
 			$description = '';
 
 			// Support for both 'description' and 'desc' args.
-			$description_key = isset( $value['description'] ) ? 'description' : 'desc';
-			$value           = ( isset( $value[ $description_key ] ) && ! empty( $value[ $description_key ] ) ) ? $value[ $description_key ] : '';
+			$description_key = isset($value['description']) ? 'description' : 'desc';
+			$value           = (isset($value[$description_key]) && ! empty($value[$description_key])) ? $value[$description_key] : '';
 
-			if ( ! empty( $value ) ) {
-				$description = '<p class="give-field-description">' . wp_kses_post( $value ) . '</p>';
+			if ( ! empty($value)) {
+				$description = '<p class="give-field-description">'.wp_kses_post($value).'</p>';
 			}
 
 			return $description;
@@ -817,11 +817,11 @@ 
 		 *
 		 * @return array The description and tip as a 2 element array
 		 */
-		public static function get_field_title( $value ) {
-			$title = esc_html( $value['title'] );
+		public static function get_field_title($value) {
+			$title = esc_html($value['title']);
 
 			// If html tag detected then allow them to print.
-			if ( strip_tags( $title ) ) {
+			if (strip_tags($title)) {
 				$title = $value['title'];
 			}
 
@@ -840,8 +840,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public static function save_fields( $options, $option_name = '' ) {
-			if ( empty( $_POST ) ) {
+		public static function save_fields($options, $option_name = '') {
+			if (empty($_POST)) {
 				return false;
 			}
 
@@ -849,37 +849,37 @@ 
 			$update_options = array();
 
 			// Loop options and get values to save.
-			foreach ( $options as $option ) {
-				if ( ! isset( $option['id'] ) || ! isset( $option['type'] ) ) {
+			foreach ($options as $option) {
+				if ( ! isset($option['id']) || ! isset($option['type'])) {
 					continue;
 				}
 
 				// Get posted value.
-				if ( strstr( $option['id'], '[' ) ) {
-					parse_str( $option['id'], $option_name_array );
-					$field_option_name = current( array_keys( $option_name_array ) );
-					$setting_name      = key( $option_name_array[ $field_option_name ] );
-					$raw_value         = isset( $_POST[ $field_option_name ][ $setting_name ] ) ? wp_unslash( $_POST[ $field_option_name ][ $setting_name ] ) : null;
+				if (strstr($option['id'], '[')) {
+					parse_str($option['id'], $option_name_array);
+					$field_option_name = current(array_keys($option_name_array));
+					$setting_name      = key($option_name_array[$field_option_name]);
+					$raw_value         = isset($_POST[$field_option_name][$setting_name]) ? wp_unslash($_POST[$field_option_name][$setting_name]) : null;
 				} else {
 					$field_option_name = $option['id'];
 					$setting_name      = '';
-					$raw_value         = isset( $_POST[ $option['id'] ] ) ? wp_unslash( $_POST[ $option['id'] ] ) : null;
+					$raw_value         = isset($_POST[$option['id']]) ? wp_unslash($_POST[$option['id']]) : null;
 				}
 
 				// Format the value based on option type.
-				switch ( $option['type'] ) {
+				switch ($option['type']) {
 					case 'checkbox' :
-						$value = is_null( $raw_value ) ? '' : 'on';
+						$value = is_null($raw_value) ? '' : 'on';
 						break;
 					case 'wysiwyg'  :
 					case 'textarea' :
-						$value = wp_kses_post( trim( $raw_value ) );
+						$value = wp_kses_post(trim($raw_value));
 						break;
 					case 'multiselect' :
-						$value = array_filter( array_map( 'give_clean', (array) $raw_value ) );
+						$value = array_filter(array_map('give_clean', (array) $raw_value));
 						break;
 					default :
-						$value = give_clean( $raw_value );
+						$value = give_clean($raw_value);
 						break;
 				}
 
@@ -888,37 +888,37 @@ 
 				 *
 				 * @since 1.8
 				 */
-				$value = apply_filters( 'give_admin_settings_sanitize_option', $value, $option, $raw_value );
+				$value = apply_filters('give_admin_settings_sanitize_option', $value, $option, $raw_value);
 
 				/**
 				 * Sanitize the value of an option by option name.
 				 *
 				 * @since 1.8
 				 */
-				$value = apply_filters( "give_admin_settings_sanitize_option_{$field_option_name}", $value, $option, $raw_value );
+				$value = apply_filters("give_admin_settings_sanitize_option_{$field_option_name}", $value, $option, $raw_value);
 
-				if ( is_null( $value ) ) {
+				if (is_null($value)) {
 					continue;
 				}
 
 				// Check if option is an array and handle that differently to single values.
-				if ( $field_option_name && $setting_name ) {
-					if ( ! isset( $update_options[ $field_option_name ] ) ) {
-						$update_options[ $field_option_name ] = get_option( $field_option_name, array() );
+				if ($field_option_name && $setting_name) {
+					if ( ! isset($update_options[$field_option_name])) {
+						$update_options[$field_option_name] = get_option($field_option_name, array());
 					}
-					if ( ! is_array( $update_options[ $field_option_name ] ) ) {
-						$update_options[ $field_option_name ] = array();
+					if ( ! is_array($update_options[$field_option_name])) {
+						$update_options[$field_option_name] = array();
 					}
-					$update_options[ $field_option_name ][ $setting_name ] = $value;
+					$update_options[$field_option_name][$setting_name] = $value;
 				} else {
-					$update_options[ $field_option_name ] = $value;
+					$update_options[$field_option_name] = $value;
 				}
 			}
 
 			// Save all options in our array or there own option name i.e. option id.
-			if ( empty( $option_name ) ) {
-				foreach ( $update_options as $name => $value ) {
-					update_option( $name, $value );
+			if (empty($option_name)) {
+				foreach ($update_options as $name => $value) {
+					update_option($name, $value);
 
 					/**
 					 * Trigger action.
@@ -927,13 +927,13 @@ 
 					 *
 					 * @since 1.8
 					 */
-					do_action( "give_save_option_{$name}", $value, $name );
+					do_action("give_save_option_{$name}", $value, $name);
 				}
 			} else {
-				$old_options    = ( $old_options = get_option( $option_name ) ) ? $old_options : array();
-				$update_options = array_merge( $old_options, $update_options );
+				$old_options    = ($old_options = get_option($option_name)) ? $old_options : array();
+				$update_options = array_merge($old_options, $update_options);
 
-				update_option( $option_name, $update_options );
+				update_option($option_name, $update_options);
 
 				/**
 				 * Trigger action.
@@ -942,7 +942,7 @@ 
 				 *
 				 * @since 1.8
 				 */
-				do_action( "give_save_settings_{$option_name}", $update_options, $option_name );
+				do_action("give_save_settings_{$option_name}", $update_options, $option_name);
 			}
 
 			return true;