WordImpress / Give

templates/emails/body-default.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

Braces +4 added lines, -1 removed lines

@@ -6,7 +6,10 @@
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

templates/emails/footer.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 // This is the footer used if no others are available
 

Braces +4 added lines, -1 removed lines

@@ -6,7 +6,10 @@
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 // This is the footer used if no others are available
 

templates/emails/body.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

Braces +4 added lines, -1 removed lines

@@ -6,7 +6,10 @@
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

includes/plugin-compatibility.php

Indentation +2 added lines, -2 removed lines

@@ -39,8 +39,8 @@
 	}
 
 	if ( ( is_plugin_active( 'wordpress-seo/wp-seo.php' )
-	       || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
-	     && class_exists( 'WPSEO_Sitemaps_Cache' )
+		   || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
+		 && class_exists( 'WPSEO_Sitemaps_Cache' )
 	) {
 
 		$forms_singular_option = give_get_option( 'forms_singular' );

Spacing +13 added lines, -13 removed lines

@@ -19,10 +19,10 @@ 
  * @return void
  */
 function give_disable_mandrill_nl2br() {
-	add_filter( 'mandrill_nl2br', '__return_false' );
+	add_filter('mandrill_nl2br', '__return_false');
 }
 
-add_action( 'give_email_send_before', 'give_disable_mandrill_nl2br' );
+add_action('give_email_send_before', 'give_disable_mandrill_nl2br');
 
 
 /**
@@ -34,30 +34,30 @@ 
  */
 function give_clear_seo_sitemap_cache_on_settings_change() {
 	// Load required file if the fn 'is_plugin_active' doesn't exists.
-	if ( ! function_exists( 'is_plugin_active' ) ) {
-		require_once ABSPATH . 'wp-admin/includes/plugin.php';
+	if ( ! function_exists('is_plugin_active')) {
+		require_once ABSPATH.'wp-admin/includes/plugin.php';
 	}
 
-	if ( ( is_plugin_active( 'wordpress-seo/wp-seo.php' )
-	       || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
-	     && class_exists( 'WPSEO_Sitemaps_Cache' )
+	if ((is_plugin_active('wordpress-seo/wp-seo.php')
+	       || is_plugin_active('wordpress-seo-premium/wp-seo-premium.php'))
+	     && class_exists('WPSEO_Sitemaps_Cache')
 	) {
 
-		$forms_singular_option = give_get_option( 'forms_singular' );
-		$forms_archive_option  = give_get_option( 'forms_singular' );
+		$forms_singular_option = give_get_option('forms_singular');
+		$forms_archive_option  = give_get_option('forms_singular');
 
 		// If there is change detected for Single Form View and Form Archives options then proceed.
 		if (
-			( isset( $_POST['forms_singular'] ) && $_POST['forms_singular'] !== $forms_singular_option ) ||
-			( isset( $_POST['forms_archives'] ) && $_POST['forms_archives'] !== $forms_archive_option )
+			(isset($_POST['forms_singular']) && $_POST['forms_singular'] !== $forms_singular_option) ||
+			(isset($_POST['forms_archives']) && $_POST['forms_archives'] !== $forms_archive_option)
 		) {
 			// If Yoast SEO or Yoast SEO Premium plugin exists, then update seo sitemap cache.
 			$yoast_sitemaps_cache = new WPSEO_Sitemaps_Cache();
-			if ( method_exists( $yoast_sitemaps_cache, 'clear' ) ) {
+			if (method_exists($yoast_sitemaps_cache, 'clear')) {
 				WPSEO_Sitemaps_Cache::clear();
 			}
 		}
 	}
 }
 
-add_action( 'give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change' );
+add_action('give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change');

includes/class-give-stats.php

Spacing +106 added lines, -106 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -93,19 +93,19 @@ 
 	 */
 	public function get_predefined_dates() {
 		$predefined = array(
-			'today'        => esc_html__( 'Today', 'give' ),
-			'yesterday'    => esc_html__( 'Yesterday', 'give' ),
-			'this_week'    => esc_html__( 'This Week', 'give' ),
-			'last_week'    => esc_html__( 'Last Week', 'give' ),
-			'this_month'   => esc_html__( 'This Month', 'give' ),
-			'last_month'   => esc_html__( 'Last Month', 'give' ),
-			'this_quarter' => esc_html__( 'This Quarter', 'give' ),
-			'last_quarter' => esc_html__( 'Last Quarter', 'give' ),
-			'this_year'    => esc_html__( 'This Year', 'give' ),
-			'last_year'    => esc_html__( 'Last Year', 'give' ),
+			'today'        => esc_html__('Today', 'give'),
+			'yesterday'    => esc_html__('Yesterday', 'give'),
+			'this_week'    => esc_html__('This Week', 'give'),
+			'last_week'    => esc_html__('Last Week', 'give'),
+			'this_month'   => esc_html__('This Month', 'give'),
+			'last_month'   => esc_html__('Last Month', 'give'),
+			'this_quarter' => esc_html__('This Quarter', 'give'),
+			'last_quarter' => esc_html__('Last Quarter', 'give'),
+			'this_year'    => esc_html__('This Year', 'give'),
+			'last_year'    => esc_html__('Last Year', 'give'),
 		);
 
-		return apply_filters( 'give_stats_predefined_dates', $predefined );
+		return apply_filters('give_stats_predefined_dates', $predefined);
 	}
 
 	/**
@@ -121,18 +121,18 @@ 
 	 *
 	 * @return void
 	 */
-	public function setup_dates( $_start_date = 'this_month', $_end_date = false ) {
+	public function setup_dates($_start_date = 'this_month', $_end_date = false) {
 
-		if ( empty( $_start_date ) ) {
+		if (empty($_start_date)) {
 			$_start_date = 'this_month';
 		}
 
-		if ( empty( $_end_date ) ) {
+		if (empty($_end_date)) {
 			$_end_date = $_start_date;
 		}
 
-		$this->start_date = $this->convert_date( $_start_date );
-		$this->end_date   = $this->convert_date( $_end_date, true );
+		$this->start_date = $this->convert_date($_start_date);
+		$this->end_date   = $this->convert_date($_end_date, true);
 	}
 
 	/**
@@ -148,26 +148,26 @@ 
 	 *
 	 * @return array|WP_Error   If the date is invalid, a WP_Error object will be returned.
 	 */
-	public function convert_date( $date, $end_date = false ) {
+	public function convert_date($date, $end_date = false) {
 
 		$this->timestamp = false;
 		$second          = $end_date ? 59 : 0;
 		$minute          = $end_date ? 59 : 0;
 		$hour            = $end_date ? 23 : 0;
 		$day             = 1;
-		$month           = date( 'n', current_time( 'timestamp' ) );
-		$year            = date( 'Y', current_time( 'timestamp' ) );
+		$month           = date('n', current_time('timestamp'));
+		$year            = date('Y', current_time('timestamp'));
 
-		if ( array_key_exists( (string) $date, $this->get_predefined_dates() ) ) {
+		if (array_key_exists((string) $date, $this->get_predefined_dates())) {
 
 			// This is a predefined date rate, such as last_week
-			switch ( $date ) {
+			switch ($date) {
 
 				case 'this_month' :
 
-					if ( $end_date ) {
+					if ($end_date) {
 
-						$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -177,28 +177,28 @@ 
 
 				case 'last_month' :
 
-					if ( $month == 1 ) {
+					if ($month == 1) {
 
 						$month = 12;
-						$year --;
+						$year--;
 
 					} else {
 
-						$month --;
+						$month--;
 
 					}
 
-					if ( $end_date ) {
-						$day = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+					if ($end_date) {
+						$day = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 					}
 
 					break;
 
 				case 'today' :
 
-					$day = date( 'd', current_time( 'timestamp' ) );
+					$day = date('d', current_time('timestamp'));
 
-					if ( $end_date ) {
+					if ($end_date) {
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -208,23 +208,23 @@ 
 
 				case 'yesterday' :
 
-					$day = date( 'd', current_time( 'timestamp' ) ) - 1;
+					$day = date('d', current_time('timestamp')) - 1;
 
 					// Check if Today is the first day of the month (meaning subtracting one will get us 0)
-					if ( $day < 1 ) {
+					if ($day < 1) {
 
 						// If current month is 1
-						if ( 1 == $month ) {
+						if (1 == $month) {
 
 							$year -= 1; // Today is January 1, so skip back to last day of December
 							$month = 12;
-							$day   = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day   = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
 						} else {
 
 							// Go back one month and get the last day of the month
 							$month -= 1;
-							$day = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
 						}
 					}
@@ -233,12 +233,12 @@ 
 
 				case 'this_week' :
 
-					$days_to_week_start = ( date( 'w', current_time( 'timestamp' ) ) - 1 ) * 60 * 60 * 24;
-					$today              = date( 'd', current_time( 'timestamp' ) ) * 60 * 60 * 24;
+					$days_to_week_start = (date('w', current_time('timestamp')) - 1) * 60 * 60 * 24;
+					$today              = date('d', current_time('timestamp')) * 60 * 60 * 24;
 
-					if ( $today < $days_to_week_start ) {
+					if ($today < $days_to_week_start) {
 
-						if ( $month > 1 ) {
+						if ($month > 1) {
 							$month -= 1;
 						} else {
 							$month = 12;
@@ -246,19 +246,19 @@ 
 
 					}
 
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 
 						// Getting the start day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 1;
-						$day += get_option( 'start_of_week' );
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 1;
+						$day += get_option('start_of_week');
 
 					} else {
 
 						// Getting the end day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 1;
-						$day += get_option( 'start_of_week' ) + 6;
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 1;
+						$day += get_option('start_of_week') + 6;
 
 					}
 
@@ -266,12 +266,12 @@ 
 
 				case 'last_week' :
 
-					$days_to_week_start = ( date( 'w', current_time( 'timestamp' ) ) - 1 ) * 60 * 60 * 24;
-					$today              = date( 'd', current_time( 'timestamp' ) ) * 60 * 60 * 24;
+					$days_to_week_start = (date('w', current_time('timestamp')) - 1) * 60 * 60 * 24;
+					$today              = date('d', current_time('timestamp')) * 60 * 60 * 24;
 
-					if ( $today < $days_to_week_start ) {
+					if ($today < $days_to_week_start) {
 
-						if ( $month > 1 ) {
+						if ($month > 1) {
 							$month -= 1;
 						} else {
 							$month = 12;
@@ -279,19 +279,19 @@ 
 
 					}
 
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 
 						// Getting the start day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 8;
-						$day += get_option( 'start_of_week' );
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 8;
+						$day += get_option('start_of_week');
 
 					} else {
 
 						// Getting the end day
 
-						$day = date( 'd', current_time( 'timestamp' ) - $days_to_week_start ) - 8;
-						$day += get_option( 'start_of_week' ) + 6;
+						$day = date('d', current_time('timestamp') - $days_to_week_start) - 8;
+						$day += get_option('start_of_week') + 6;
 
 					}
 
@@ -299,39 +299,39 @@ 
 
 				case 'this_quarter' :
 
-					$month_now = date( 'n', current_time( 'timestamp' ) );
+					$month_now = date('n', current_time('timestamp'));
 
-					if ( $month_now <= 3 ) {
+					if ($month_now <= 3) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 1;
 						} else {
 							$month  = 3;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 6 ) {
+					} else if ($month_now <= 6) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 4;
 						} else {
 							$month  = 6;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 9 ) {
+					} else if ($month_now <= 9) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 7;
 						} else {
 							$month  = 9;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -339,11 +339,11 @@ 
 
 					} else {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 10;
 						} else {
 							$month  = 12;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -355,40 +355,40 @@ 
 
 				case 'last_quarter' :
 
-					$month_now = date( 'n', current_time( 'timestamp' ) );
+					$month_now = date('n', current_time('timestamp'));
 
-					if ( $month_now <= 3 ) {
+					if ($month_now <= 3) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 10;
 						} else {
 							$year -= 1;
 							$month  = 12;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 6 ) {
+					} else if ($month_now <= 6) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 1;
 						} else {
 							$month  = 3;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
 						}
 
-					} else if ( $month_now <= 9 ) {
+					} else if ($month_now <= 9) {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 4;
 						} else {
 							$month  = 6;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -396,11 +396,11 @@ 
 
 					} else {
 
-						if ( ! $end_date ) {
+						if ( ! $end_date) {
 							$month = 7;
 						} else {
 							$month  = 9;
-							$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+							$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 							$hour   = 23;
 							$minute = 59;
 							$second = 59;
@@ -412,11 +412,11 @@ 
 
 				case 'this_year' :
 
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 						$month = 1;
 					} else {
 						$month  = 12;
-						$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -427,11 +427,11 @@ 
 				case 'last_year' :
 
 					$year -= 1;
-					if ( ! $end_date ) {
+					if ( ! $end_date) {
 						$month = 1;
 					} else {
 						$month  = 12;
-						$day    = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+						$day    = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 						$hour   = 23;
 						$minute = 59;
 						$second = 59;
@@ -442,30 +442,30 @@ 
 			}
 
 
-		} else if ( is_numeric( $date ) ) {
+		} else if (is_numeric($date)) {
 
 			// return $date unchanged since it is a timestamp
 			$this->timestamp = true;
 
-		} else if ( false !== strtotime( $date ) ) {
+		} else if (false !== strtotime($date)) {
 
-			$date  = strtotime( $date, current_time( 'timestamp' ) );
-			$year  = date( 'Y', $date );
-			$month = date( 'm', $date );
-			$day   = date( 'd', $date );
+			$date  = strtotime($date, current_time('timestamp'));
+			$year  = date('Y', $date);
+			$month = date('m', $date);
+			$day   = date('d', $date);
 
 		} else {
 
-			return new WP_Error( 'invalid_date', esc_html__( 'Improper date provided.', 'give' ) );
+			return new WP_Error('invalid_date', esc_html__('Improper date provided.', 'give'));
 
 		}
 
-		if ( false === $this->timestamp ) {
+		if (false === $this->timestamp) {
 			// Create an exact timestamp
-			$date = mktime( $hour, $minute, $second, $month, $day, $year );
+			$date = mktime($hour, $minute, $second, $month, $day, $year);
 		}
 
-		return apply_filters( 'give_stats_date', $date, $end_date, $this );
+		return apply_filters('give_stats_date', $date, $end_date, $this);
 
 	}
 
@@ -481,33 +481,33 @@ 
 	 * 
 	 * @return string
 	 */
-	public function count_where( $where = '' ) {
+	public function count_where($where = '') {
 		// Only get payments in our date range
 
 		$start_where = '';
 		$end_where   = '';
 
-		if ( $this->start_date ) {
+		if ($this->start_date) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 00:00:00';
 			}
 
-			$start_date  = date( $format, $this->start_date );
+			$start_date  = date($format, $this->start_date);
 			$start_where = " AND p.post_date >= '{$start_date}'";
 		}
 
-		if ( $this->end_date ) {
+		if ($this->end_date) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 23:59:59';
 			}
 
-			$end_date = date( $format, $this->end_date );
+			$end_date = date($format, $this->end_date);
 
 			$end_where = " AND p.post_date <= '{$end_date}'";
 		}
@@ -529,34 +529,34 @@ 
 	 *
 	 * @return string
 	 */
-	public function payments_where( $where = '' ) {
+	public function payments_where($where = '') {
 
 		global $wpdb;
 
 		$start_where = '';
 		$end_where   = '';
 
-		if ( ! is_wp_error( $this->start_date ) ) {
+		if ( ! is_wp_error($this->start_date)) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 00:00:00';
 			}
 
-			$start_date  = date( $format, $this->start_date );
+			$start_date  = date($format, $this->start_date);
 			$start_where = " AND $wpdb->posts.post_date >= '{$start_date}'";
 		}
 
-		if ( ! is_wp_error( $this->end_date ) ) {
+		if ( ! is_wp_error($this->end_date)) {
 
-			if ( $this->timestamp ) {
+			if ($this->timestamp) {
 				$format = 'Y-m-d H:i:s';
 			} else {
 				$format = 'Y-m-d 23:59:59';
 			}
 
-			$end_date = date( $format, $this->end_date );
+			$end_date = date($format, $this->end_date);
 
 			$end_where = " AND $wpdb->posts.post_date <= '{$end_date}'";
 		}

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

Doc Comments +4 added lines, -1 removed lines

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

Spacing +117 added lines, -117 removed lines

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

includes/class-give-donor.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +254 added lines, -261 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -149,32 +149,32 @@ 
 	 * @param int|bool $_id_or_email
 	 * @param bool     $by_user_id
 	 */
-	public function __construct( $_id_or_email = false, $by_user_id = false ) {
+	public function __construct($_id_or_email = false, $by_user_id = false) {
 
 		$this->db = Give()->donors;
 
 		if (
 			false === $_id_or_email
-			|| ( is_numeric( $_id_or_email ) && (int) $_id_or_email !== absint( $_id_or_email ) )
+			|| (is_numeric($_id_or_email) && (int) $_id_or_email !== absint($_id_or_email))
 		) {
 			return false;
 		}
 
-		$by_user_id = is_bool( $by_user_id ) ? $by_user_id : false;
+		$by_user_id = is_bool($by_user_id) ? $by_user_id : false;
 
-		if ( is_numeric( $_id_or_email ) ) {
+		if (is_numeric($_id_or_email)) {
 			$field = $by_user_id ? 'user_id' : 'id';
 		} else {
 			$field = 'email';
 		}
 
-		$donor = $this->db->get_donor_by( $field, $_id_or_email );
+		$donor = $this->db->get_donor_by($field, $_id_or_email);
 
-		if ( empty( $donor ) || ! is_object( $donor ) ) {
+		if (empty($donor) || ! is_object($donor)) {
 			return false;
 		}
 
-		$this->setup_donor( $donor );
+		$this->setup_donor($donor);
 
 	}
 
@@ -190,19 +190,19 @@ 
 	 *
 	 * @return bool             If the setup was successful or not.
 	 */
-	private function setup_donor( $donor ) {
+	private function setup_donor($donor) {
 
-		if ( ! is_object( $donor ) ) {
+		if ( ! is_object($donor)) {
 			return false;
 		}
 
 		// Get cached donors.
-		$donor_vars = Give_Cache::get_group( $donor->id, 'give-donors' );
+		$donor_vars = Give_Cache::get_group($donor->id, 'give-donors');
 
-		if( is_null( $donor_vars ) ){
-			foreach ( $donor as $key => $value ) {
+		if (is_null($donor_vars)) {
+			foreach ($donor as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'notes':
 						$this->$key = $this->get_notes();
@@ -216,20 +216,20 @@ 
 			}
 
 			// Get donor's all email including primary email.
-			$this->emails = (array) $this->get_meta( 'additional_email', false );
-			$this->emails = array( 'primary' => $this->email ) + $this->emails;
+			$this->emails = (array) $this->get_meta('additional_email', false);
+			$this->emails = array('primary' => $this->email) + $this->emails;
 
 			$this->setup_address();
 
-			Give_Cache::set_group( $donor->id, get_object_vars( $this ), 'give-donors' );
-		} else{
-			foreach ( $donor_vars as $donor_var => $value ) {
+			Give_Cache::set_group($donor->id, get_object_vars($this), 'give-donors');
+		} else {
+			foreach ($donor_vars as $donor_var => $value) {
 				$this->$donor_var = $value;
 			}
 		}
 
 		// Donor ID and email are the only things that are necessary, make sure they exist.
-		if ( ! empty( $this->id ) && ! empty( $this->email ) ) {
+		if ( ! empty($this->id) && ! empty($this->email)) {
 			return true;
 		}
 
@@ -262,18 +262,18 @@ 
 			ARRAY_N
 		);
 
-		if ( empty( $addresses ) ) {
+		if (empty($addresses)) {
 			return $this->address;
 		}
 
-		foreach ( $addresses as $address ) {
-			$address[0] = str_replace( '_give_donor_address_', '', $address[0] );
-			$address[0] = explode( '_', $address[0] );
+		foreach ($addresses as $address) {
+			$address[0] = str_replace('_give_donor_address_', '', $address[0]);
+			$address[0] = explode('_', $address[0]);
 
-			if ( 3 === count( $address[0] ) ) {
-				$this->address[ $address[0][0] ][ $address[0][2] ][ $address[0][1] ] = $address[1];
+			if (3 === count($address[0])) {
+				$this->address[$address[0][0]][$address[0][2]][$address[0][1]] = $address[1];
 			} else {
-				$this->address[ $address[0][0] ][ $address[0][1] ] = $address[1];
+				$this->address[$address[0][0]][$address[0][1]] = $address[1];
 			}
 		}
 	}
@@ -287,16 +287,16 @@ 
 	 *
 	 * @return mixed|\WP_Error
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-donor-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-donor-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -312,9 +312,9 @@ 
 	 *
 	 * @return bool|int    False if not a valid creation, donor ID if user is found or valid creation.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 || empty( $data ) ) {
+		if ($this->id != 0 || empty($data)) {
 			return false;
 		}
 
@@ -322,15 +322,15 @@ 
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
-		$args = $this->sanitize_columns( $args );
+		$args = wp_parse_args($data, $defaults);
+		$args = $this->sanitize_columns($args);
 
-		if ( empty( $args['email'] ) || ! is_email( $args['email'] ) ) {
+		if (empty($args['email']) || ! is_email($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
 		/**
@@ -340,18 +340,18 @@ 
 		 *
 		 * @param array $args Donor attributes.
 		 */
-		do_action( 'give_donor_pre_create', $args );
+		do_action('give_donor_pre_create', $args);
 
 		$created = false;
 
 		// The DB class 'add' implies an update if the donor being asked to be created already exists
-		if ( $this->db->add( $data ) ) {
+		if ($this->db->add($data)) {
 
 			// We've successfully added/updated the donor, reset the class vars with the new data
-			$donor = $this->db->get_donor_by( 'email', $args['email'] );
+			$donor = $this->db->get_donor_by('email', $args['email']);
 
 			// Setup the donor data with the values from DB
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$created = $this->id;
 		}
@@ -364,7 +364,7 @@ 
 		 * @param bool|int $created False if not a valid creation, donor ID if user is found or valid creation.
 		 * @param array $args Customer attributes.
 		 */
-		do_action( 'give_donor_post_create', $created, $args );
+		do_action('give_donor_post_create', $created, $args);
 
 		return $created;
 
@@ -380,13 +380,13 @@ 
 	 *
 	 * @return bool        If the update was successful or not.
 	 */
-	public function update( $data = array() ) {
+	public function update($data = array()) {
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return false;
 		}
 
-		$data = $this->sanitize_columns( $data );
+		$data = $this->sanitize_columns($data);
 
 		/**
 		 * Fires before updating donors.
@@ -396,15 +396,15 @@ 
 		 * @param int $donor_id Donor id.
 		 * @param array $data Donor attributes.
 		 */
-		do_action( 'give_donor_pre_update', $this->id, $data );
+		do_action('give_donor_pre_update', $this->id, $data);
 
 		$updated = false;
 
-		if ( $this->db->update( $this->id, $data ) ) {
+		if ($this->db->update($this->id, $data)) {
 
-			$donor = $this->db->get_donor_by( 'id', $this->id );
+			$donor = $this->db->get_donor_by('id', $this->id);
 
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$updated = true;
 		}
@@ -418,7 +418,7 @@ 
 		 * @param int $donor_id Donor id.
 		 * @param array $data Donor attributes.
 		 */
-		do_action( 'give_donor_post_update', $updated, $this->id, $data );
+		do_action('give_donor_post_update', $updated, $this->id, $data);
 
 		return $updated;
 	}
@@ -436,27 +436,27 @@ 
 	 *
 	 * @return bool            If the attachment was successfully.
 	 */
-	public function attach_payment( $payment_id = 0, $update_stats = true ) {
+	public function attach_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		if ( empty( $this->payment_ids ) ) {
+		if (empty($this->payment_ids)) {
 
 			$new_payment_ids = $payment_id;
 
 		} else {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			if ( in_array( $payment_id, $payment_ids ) ) {
+			if (in_array($payment_id, $payment_ids)) {
 				$update_stats = false;
 			}
 
 			$payment_ids[] = $payment_id;
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -468,20 +468,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_attach_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_attach_payment', $payment_id, $this->id);
 
-		$payment_added = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_added = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_added ) {
+		if ($payment_added) {
 
 			$this->payment_ids = $new_payment_ids;
 
 			// We added this payment successfully, increment the stats
-			if ( $update_stats ) {
-				$payment_amount = give_donation_amount( $payment_id, array( 'type' => 'stats' ) );
+			if ($update_stats) {
+				$payment_amount = give_donation_amount($payment_id, array('type' => 'stats'));
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->increase_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->increase_value($payment_amount);
 				}
 
 				$this->increase_purchase_count();
@@ -497,7 +497,7 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_attach_payment', $payment_added, $payment_id, $this->id );
+		do_action('give_donor_post_attach_payment', $payment_added, $payment_id, $this->id);
 
 		return $payment_added;
 	}
@@ -515,33 +515,33 @@ 
 	 *
 	 * @return boolean               If the removal was successful.
 	 */
-	public function remove_payment( $payment_id = 0, $update_stats = true ) {
+	public function remove_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = new Give_Payment( $payment_id );
+		$payment = new Give_Payment($payment_id);
 
-		if ( 'publish' !== $payment->status && 'revoked' !== $payment->status ) {
+		if ('publish' !== $payment->status && 'revoked' !== $payment->status) {
 			$update_stats = false;
 		}
 
 		$new_payment_ids = '';
 
-		if ( ! empty( $this->payment_ids ) ) {
+		if ( ! empty($this->payment_ids)) {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			$pos = array_search( $payment_id, $payment_ids );
-			if ( false === $pos ) {
+			$pos = array_search($payment_id, $payment_ids);
+			if (false === $pos) {
 				return false;
 			}
 
-			unset( $payment_ids[ $pos ] );
-			$payment_ids = array_filter( $payment_ids );
+			unset($payment_ids[$pos]);
+			$payment_ids = array_filter($payment_ids);
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -553,20 +553,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_remove_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_remove_payment', $payment_id, $this->id);
 
-		$payment_removed = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_removed = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_removed ) {
+		if ($payment_removed) {
 
 			$this->payment_ids = $new_payment_ids;
 
-			if ( $update_stats ) {
+			if ($update_stats) {
 				// We removed this payment successfully, decrement the stats
-				$payment_amount = give_donation_amount( $payment_id );
+				$payment_amount = give_donation_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->decrease_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->decrease_value($payment_amount);
 				}
 
 				$this->decrease_donation_count();
@@ -582,7 +582,7 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id );
+		do_action('give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id);
 
 		return $payment_removed;
 
@@ -598,10 +598,10 @@ 
 	 *
 	 * @return int        The donation count.
 	 */
-	public function increase_purchase_count( $count = 1 ) {
+	public function increase_purchase_count($count = 1) {
 
 		// Make sure it's numeric and not negative.
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
@@ -615,9 +615,9 @@ 
 		 * @param int $count The number to increase by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_increase_donation_count', $count, $this->id );
+		do_action('give_donor_pre_increase_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -630,7 +630,7 @@ 
 		 * @param int $count The number increased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -645,16 +645,16 @@ 
 	 *
 	 * @return mixed      If successful, the new count, otherwise false.
 	 */
-	public function decrease_donation_count( $count = 1 ) {
+	public function decrease_donation_count($count = 1) {
 
 		// Make sure it's numeric and not negative
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
 		$new_total = (int) $this->purchase_count - (int) $count;
 
-		if ( $new_total < 0 ) {
+		if ($new_total < 0) {
 			$new_total = 0;
 		}
 
@@ -666,9 +666,9 @@ 
 		 * @param int $count The number to decrease by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_decrease_donation_count', $count, $this->id );
+		do_action('give_donor_pre_decrease_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -681,7 +681,7 @@ 
 		 * @param int $count The number decreased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -696,9 +696,9 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function increase_value( $value = 0.00 ) {
+	public function increase_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) + $value;
+		$new_value = floatval($this->purchase_value) + $value;
 
 		/**
 		 * Fires before increasing donor lifetime value.
@@ -708,9 +708,9 @@ 
 		 * @param float $value The value to increase by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_increase_value', $value, $this->id );
+		do_action('give_donor_pre_increase_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -723,7 +723,7 @@ 
 		 * @param float $value The value increased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_increase_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_increase_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -738,11 +738,11 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function decrease_value( $value = 0.00 ) {
+	public function decrease_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) - $value;
+		$new_value = floatval($this->purchase_value) - $value;
 
-		if ( $new_value < 0 ) {
+		if ($new_value < 0) {
 			$new_value = 0.00;
 		}
 
@@ -754,9 +754,9 @@ 
 		 * @param float $value The value to decrease by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_decrease_value', $value, $this->id );
+		do_action('give_donor_pre_decrease_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -769,7 +769,7 @@ 
 		 * @param float $value The value decreased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_decrease_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -788,7 +788,7 @@ 
 	 *
 	 * @return mixed              If successful, the new donation stat value, otherwise false.
 	 */
-	public function update_donation_value( $curr_amount, $new_amount ) {
+	public function update_donation_value($curr_amount, $new_amount) {
 		/**
 		 * Payment total difference value can be:
 		 *  zero   (in case amount not change)
@@ -798,15 +798,15 @@ 
 		$payment_total_diff = $new_amount - $curr_amount;
 
 		// We do not need to update donation stat if donation did not change.
-		if ( ! $payment_total_diff ) {
+		if ( ! $payment_total_diff) {
 			return false;
 		}
 
-		if ( $payment_total_diff > 0 ) {
-			$this->increase_value( $payment_total_diff );
+		if ($payment_total_diff > 0) {
+			$this->increase_value($payment_total_diff);
 		} else {
 			// Pass payment total difference as +ve value to decrease amount from user lifetime stat.
-			$this->decrease_value( - $payment_total_diff );
+			$this->decrease_value( -$payment_total_diff );
 		}
 
 		return $this->purchase_value;
@@ -823,15 +823,15 @@ 
 	 *
 	 * @return array       The notes requested.
 	 */
-	public function get_notes( $length = 20, $paged = 1 ) {
+	public function get_notes($length = 20, $paged = 1) {
 
-		$length = is_numeric( $length ) ? $length : 20;
-		$offset = is_numeric( $paged ) && $paged != 1 ? ( ( absint( $paged ) - 1 ) * $length ) : 0;
+		$length = is_numeric($length) ? $length : 20;
+		$offset = is_numeric($paged) && $paged != 1 ? ((absint($paged) - 1) * $length) : 0;
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		$desired_notes = array_slice( $notes_array, $offset, $length );
+		$desired_notes = array_slice($notes_array, $offset, $length);
 
 		return $desired_notes;
 
@@ -848,9 +848,9 @@ 
 	public function get_notes_count() {
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		return count( $notes_array );
+		return count($notes_array);
 
 	}
 
@@ -863,7 +863,7 @@ 
 	 *
 	 * @return string|float
 	 */
-	public function get_total_donation_amount( $args = array() ) {
+	public function get_total_donation_amount($args = array()) {
 
 		/**
 		 * Filter total donation amount.
@@ -874,7 +874,7 @@ 
 		 * @param integer      $donor_id       Donor ID.
 		 * @param array        $args           Pass additional data.
 		 */
-		return apply_filters( 'give_get_total_donation_amount', $this->purchase_value, $this->id, $args );
+		return apply_filters('give_get_total_donation_amount', $this->purchase_value, $this->id, $args);
 	}
 
 	/**
@@ -887,22 +887,22 @@ 
 	 *
 	 * @return string|boolean The new note if added successfully, false otherwise.
 	 */
-	public function add_note( $note = '' ) {
+	public function add_note($note = '') {
 
-		$note = trim( $note );
-		if ( empty( $note ) ) {
+		$note = trim($note);
+		if (empty($note)) {
 			return false;
 		}
 
 		$notes = $this->get_raw_notes();
 
-		if ( empty( $notes ) ) {
+		if (empty($notes)) {
 			$notes = '';
 		}
 
-		$note_string = date_i18n( 'F j, Y H:i:s', current_time( 'timestamp' ) ) . ' - ' . $note;
-		$new_note    = apply_filters( 'give_customer_add_note_string', $note_string );
-		$notes       .= "\n\n" . $new_note;
+		$note_string = date_i18n('F j, Y H:i:s', current_time('timestamp')).' - '.$note;
+		$new_note    = apply_filters('give_customer_add_note_string', $note_string);
+		$notes .= "\n\n".$new_note;
 
 		/**
 		 * Fires before donor note is added.
@@ -912,11 +912,11 @@ 
 		 * @param string $new_note New note to add.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_add_note', $new_note, $this->id );
+		do_action('give_donor_pre_add_note', $new_note, $this->id);
 
-		$updated = $this->update( array( 'notes' => $notes ) );
+		$updated = $this->update(array('notes' => $notes));
 
-		if ( $updated ) {
+		if ($updated) {
 			$this->notes = $this->get_notes();
 		}
 
@@ -929,7 +929,7 @@ 
 		 * @param string $new_note New note added.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_add_note', $this->notes, $new_note, $this->id );
+		do_action('give_donor_post_add_note', $this->notes, $new_note, $this->id);
 
 		// Return the formatted note, so we can test, as well as update any displays
 		return $new_note;
@@ -946,7 +946,7 @@ 
 	 */
 	private function get_raw_notes() {
 
-		$all_notes = $this->db->get_column( 'notes', $this->id );
+		$all_notes = $this->db->get_column('notes', $this->id);
 
 		return $all_notes;
 
@@ -963,8 +963,8 @@ 
 	 *
 	 * @return mixed            Will be an array if $single is false. Will be value of meta data field if $single is true.
 	 */
-	public function get_meta( $meta_key = '', $single = true ) {
-		return Give()->donor_meta->get_meta( $this->id, $meta_key, $single );
+	public function get_meta($meta_key = '', $single = true) {
+		return Give()->donor_meta->get_meta($this->id, $meta_key, $single);
 	}
 
 	/**
@@ -979,8 +979,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function add_meta( $meta_key = '', $meta_value, $unique = false ) {
-		return Give()->donor_meta->add_meta( $this->id, $meta_key, $meta_value, $unique );
+	public function add_meta($meta_key = '', $meta_value, $unique = false) {
+		return Give()->donor_meta->add_meta($this->id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -995,8 +995,8 @@ 
 	 *
 	 * @return bool               False on failure, true if success.
 	 */
-	public function update_meta( $meta_key = '', $meta_value, $prev_value = '' ) {
-		return Give()->donor_meta->update_meta( $this->id, $meta_key, $meta_value, $prev_value );
+	public function update_meta($meta_key = '', $meta_value, $prev_value = '') {
+		return Give()->donor_meta->update_meta($this->id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1010,8 +1010,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function delete_meta( $meta_key = '', $meta_value = '' ) {
-		return Give()->donor_meta->delete_meta( $this->id, $meta_key, $meta_value );
+	public function delete_meta($meta_key = '', $meta_value = '') {
+		return Give()->donor_meta->delete_meta($this->id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -1024,51 +1024,51 @@ 
 	 *
 	 * @return array       The sanitized data, based off column defaults.
 	 */
-	private function sanitize_columns( $data ) {
+	private function sanitize_columns($data) {
 
 		$columns        = $this->db->get_columns();
 		$default_values = $this->db->get_column_defaults();
 
-		foreach ( $columns as $key => $type ) {
+		foreach ($columns as $key => $type) {
 
 			// Only sanitize data that we were provided
-			if ( ! array_key_exists( $key, $data ) ) {
+			if ( ! array_key_exists($key, $data)) {
 				continue;
 			}
 
-			switch ( $type ) {
+			switch ($type) {
 
 				case '%s':
-					if ( 'email' == $key ) {
-						$data[ $key ] = sanitize_email( $data[ $key ] );
-					} elseif ( 'notes' == $key ) {
-						$data[ $key ] = strip_tags( $data[ $key ] );
+					if ('email' == $key) {
+						$data[$key] = sanitize_email($data[$key]);
+					} elseif ('notes' == $key) {
+						$data[$key] = strip_tags($data[$key]);
 					} else {
-						$data[ $key ] = sanitize_text_field( $data[ $key ] );
+						$data[$key] = sanitize_text_field($data[$key]);
 					}
 					break;
 
 				case '%d':
-					if ( ! is_numeric( $data[ $key ] ) || (int) $data[ $key ] !== absint( $data[ $key ] ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_numeric($data[$key]) || (int) $data[$key] !== absint($data[$key])) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = absint( $data[ $key ] );
+						$data[$key] = absint($data[$key]);
 					}
 					break;
 
 				case '%f':
 					// Convert what was given to a float
-					$value = floatval( $data[ $key ] );
+					$value = floatval($data[$key]);
 
-					if ( ! is_float( $value ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_float($value)) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = $value;
+						$data[$key] = $value;
 					}
 					break;
 
 				default:
-					$data[ $key ] = sanitize_text_field( $data[ $key ] );
+					$data[$key] = sanitize_text_field($data[$key]);
 					break;
 
 			}
@@ -1088,33 +1088,33 @@ 
 	 *
 	 * @return bool            If the email was added successfully
 	 */
-	public function add_email( $email = '', $primary = false ) {
-		if ( ! is_email( $email ) ) {
+	public function add_email($email = '', $primary = false) {
+		if ( ! is_email($email)) {
 			return false;
 		}
-		$existing = new Give_Donor( $email );
+		$existing = new Give_Donor($email);
 
-		if ( $existing->id > 0 ) {
+		if ($existing->id > 0) {
 			// Email address already belongs to another donor
 			return false;
 		}
 
-		if ( email_exists( $email ) ) {
-			$user = get_user_by( 'email', $email );
-			if ( $user->ID != $this->user_id ) {
+		if (email_exists($email)) {
+			$user = get_user_by('email', $email);
+			if ($user->ID != $this->user_id) {
 				return false;
 			}
 		}
 
-		do_action( 'give_donor_pre_add_email', $email, $this->id, $this );
+		do_action('give_donor_pre_add_email', $email, $this->id, $this);
 
 		// Add is used to ensure duplicate emails are not added
-		$ret = (bool) $this->add_meta( 'additional_email', $email );
+		$ret = (bool) $this->add_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_add_email', $email, $this->id, $this );
+		do_action('give_donor_post_add_email', $email, $this->id, $this);
 
-		if ( $ret && true === $primary ) {
-			$this->set_primary_email( $email );
+		if ($ret && true === $primary) {
+			$this->set_primary_email($email);
 		}
 
 		return $ret;
@@ -1130,16 +1130,16 @@ 
 	 *
 	 * @return bool          If the email was removed successfully.
 	 */
-	public function remove_email( $email = '' ) {
-		if ( ! is_email( $email ) ) {
+	public function remove_email($email = '') {
+		if ( ! is_email($email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_remove_email', $email, $this->id, $this );
+		do_action('give_donor_pre_remove_email', $email, $this->id, $this);
 
-		$ret = (bool) $this->delete_meta( 'additional_email', $email );
+		$ret = (bool) $this->delete_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_remove_email', $email, $this->id, $this );
+		do_action('give_donor_post_remove_email', $email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1156,16 +1156,16 @@ 
 	 *
 	 * @return bool                      If the email was set as primary successfully.
 	 */
-	public function set_primary_email( $new_primary_email = '' ) {
-		if ( ! is_email( $new_primary_email ) ) {
+	public function set_primary_email($new_primary_email = '') {
+		if ( ! is_email($new_primary_email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this);
 
-		$existing = new Give_Donor( $new_primary_email );
+		$existing = new Give_Donor($new_primary_email);
 
-		if ( $existing->id > 0 && (int) $existing->id !== (int) $this->id ) {
+		if ($existing->id > 0 && (int) $existing->id !== (int) $this->id) {
 			// This email belongs to another donor.
 			return false;
 		}
@@ -1173,21 +1173,21 @@ 
 		$old_email = $this->email;
 
 		// Update donor record with new email.
-		$update = $this->update( array( 'email' => $new_primary_email ) );
+		$update = $this->update(array('email' => $new_primary_email));
 
 		// Remove new primary from list of additional emails.
-		$remove = $this->remove_email( $new_primary_email );
+		$remove = $this->remove_email($new_primary_email);
 
 		// Add old email to additional emails list.
-		$add = $this->add_email( $old_email );
+		$add = $this->add_email($old_email);
 
 		$ret = $update && $remove && $add;
 
-		if ( $ret ) {
+		if ($ret) {
 			$this->email = $new_primary_email;
 		}
 
-		do_action( 'give_donor_post_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_post_set_primary_email', $new_primary_email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1202,17 +1202,17 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_valid_address( $address ) {
+	private function is_valid_address($address) {
 		$is_valid_address = true;
 		
 		// Address ready to process even if only one value set.
-		foreach ( $address as $address_type => $value ) {
+		foreach ($address as $address_type => $value) {
 			// @todo: Handle state field validation on basis of country.
-			if( in_array( $address_type, array( 'line2', 'state' )) ) {
+			if (in_array($address_type, array('line2', 'state'))) {
 				continue;
 			}
 
-			if ( empty( $value ) ) {
+			if (empty($value)) {
 				$is_valid_address = false;
 				break;
 			}
@@ -1239,30 +1239,27 @@ 
 	 *
 	 * @return bool
 	 */
-	public function add_address( $address_type, $address ) {
+	public function add_address($address_type, $address) {
 		// Bailout.
-		if ( empty( $address_type ) || ! $this->is_valid_address( $address ) || ! $this->id ) {
+		if (empty($address_type) || ! $this->is_valid_address($address) || ! $this->id) {
 			return false;
 		}
 
 		// Check if multiple address exist or not and set params.
 		$multi_address_id = null;
-		if( $is_multi_address = ( false !== strpos( $address_type, '[]' ) ) ) {
-			$address_type  = $is_multi_address ?
-				str_replace( '[]', '', $address_type ) :
-				$address_type;
-		} elseif ( $is_multi_address = ( false !== strpos( $address_type, '_' ) ) ){
-			$multi_address_id =  $is_multi_address ?
-				array_pop( explode( '_', $address_type ) ) :
-				$address_type;
-
-			$address_type  = $is_multi_address ?
-				array_shift( explode( '_', $address_type ) ) :
-				$address_type;
+		if ($is_multi_address = (false !== strpos($address_type, '[]'))) {
+			$address_type = $is_multi_address ?
+				str_replace('[]', '', $address_type) : $address_type;
+		} elseif ($is_multi_address = (false !== strpos($address_type, '_'))) {
+			$multi_address_id = $is_multi_address ?
+				array_pop(explode('_', $address_type)) : $address_type;
+
+			$address_type = $is_multi_address ?
+				array_shift(explode('_', $address_type)) : $address_type;
 		}
 
 		// Bailout: do not save duplicate orders
-		if( $this->is_address_exist( $address_type, $address ) ) {
+		if ($this->is_address_exist($address_type, $address)) {
 			return false;
 		}
 
@@ -1284,8 +1281,8 @@ 
 		$meta_key_prefix = "_give_donor_address_{$address_type}_{address_name}";
 		$meta_type = Give()->donor_meta->meta_type;
 
-		if ( $is_multi_address ) {
-			if ( is_null( $multi_address_id ) ) {
+		if ($is_multi_address) {
+			if (is_null($multi_address_id)) {
 				// Get latest address key to set multi address id.
 				$multi_address_id = $wpdb->get_var(
 					$wpdb->prepare(
@@ -1302,10 +1299,10 @@ 
 					)
 				);
 
-				if( ! empty( $multi_address_id ) ) {
-					$multi_address_id = absint( substr( strrchr( $multi_address_id, '_' ), 1 ) );
+				if ( ! empty($multi_address_id)) {
+					$multi_address_id = absint(substr(strrchr($multi_address_id, '_'), 1));
 					$multi_address_id++;
-				} else{
+				} else {
 					$multi_address_id = 0;
 				}
 			}
@@ -1314,9 +1311,9 @@ 
 		}
 
 		// Save donor address.
-		foreach ( $address as $type => $value ) {
-			$meta_key = str_replace( '{address_name}', $type, $meta_key_prefix );
-			Give()->donor_meta->update_meta( $this->id, $meta_key, $value );
+		foreach ($address as $type => $value) {
+			$meta_key = str_replace('{address_name}', $type, $meta_key_prefix);
+			Give()->donor_meta->update_meta($this->id, $meta_key, $value);
 		}
 
 		$this->setup_address();
@@ -1335,23 +1332,21 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_address( $address_id ) {
+	public function remove_address($address_id) {
 		global $wpdb;
 
 		// Get address type.
-		$is_multi_address = false !== strpos( $address_id, '_' ) ? true : false;
+		$is_multi_address = false !== strpos($address_id, '_') ? true : false;
 
-		$address_type = false !== strpos( $address_id, '_' ) ?
-			array_shift( explode( '_', $address_id ) ) :
-			$address_id;
+		$address_type = false !== strpos($address_id, '_') ?
+			array_shift(explode('_', $address_id)) : $address_id;
 
-		$address_count = false !== strpos( $address_id, '_' ) ?
-			array_pop( explode( '_', $address_id ) ) :
-			null;
+		$address_count = false !== strpos($address_id, '_') ?
+			array_pop(explode('_', $address_id)) : null;
 
 		// Set meta key prefix.
 		$meta_key_prefix = "_give_donor_address_{$address_type}_%";
-		if ( $is_multi_address && is_numeric( $address_count ) ) {
+		if ($is_multi_address && is_numeric($address_count)) {
 			$meta_key_prefix .= "_{$address_count}";
 		}
 
@@ -1388,23 +1383,21 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update_address( $address_id, $address ) {
+	public function update_address($address_id, $address) {
 		global $wpdb;
 
 		// Get address type.
-		$is_multi_address = false !== strpos( $address_id, '_' ) ? true : false;
+		$is_multi_address = false !== strpos($address_id, '_') ? true : false;
 
-		$address_type = false !== strpos( $address_id, '_' ) ?
-			array_shift( explode( '_', $address_id ) ) :
-			$address_id;
+		$address_type = false !== strpos($address_id, '_') ?
+			array_shift(explode('_', $address_id)) : $address_id;
 
-		$address_count = false !== strpos( $address_id, '_' ) ?
-			array_pop( explode( '_', $address_id ) ) :
-			null;
+		$address_count = false !== strpos($address_id, '_') ?
+			array_pop(explode('_', $address_id)) : null;
 
 		// Set meta key prefix.
 		$meta_key_prefix = "_give_donor_address_{$address_type}_%";
-		if ( $is_multi_address && is_numeric( $address_count ) ) {
+		if ($is_multi_address && is_numeric($address_count)) {
 			$meta_key_prefix .= "_{$address_count}";
 		}
 
@@ -1425,12 +1418,12 @@ 
 		);
 		
 		// Return result.
-		if( ! count( $row_affected ) ) {
+		if ( ! count($row_affected)) {
 			return false;
 		}
 
 		// Update address.
-		if( ! $this->add_address( $address_id, $address ) ) {
+		if ( ! $this->add_address($address_id, $address)) {
 			return false;
 		}
 
@@ -1449,39 +1442,39 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_address_exist( $current_address_type, $current_address ) {
+	public function is_address_exist($current_address_type, $current_address) {
 		$status = false;
 
 		// Bailout.
-		if( empty( $current_address_type ) || empty( $current_address ) ) {
+		if (empty($current_address_type) || empty($current_address)) {
 			return null;
 		}
 
 		// Bailout.
-		if( empty( $this->address ) || empty( $this->address[ $current_address_type ] ) ) {
+		if (empty($this->address) || empty($this->address[$current_address_type])) {
 			return $status;
 		}
 
 		// Get address.
-		$address = $this->address[ $current_address_type ];
+		$address = $this->address[$current_address_type];
 
-		switch ( true ){
+		switch (true) {
 
 			// Single address.
-			case is_string( end( $address ) ) :
-				$status = $this->is_address_match( $current_address, $address );
+			case is_string(end($address)) :
+				$status = $this->is_address_match($current_address, $address);
 				break;
 
 			// Multi address.
-			case is_array( end( $address ) ):
+			case is_array(end($address)):
 				// Compare address.
-				foreach ( $address as $saved_address ) {
-					if( empty( $saved_address ) ) {
+				foreach ($address as $saved_address) {
+					if (empty($saved_address)) {
 						continue;
 					}
 
 					// Exit loop immediately if address exist.
-					if( $status = $this->is_address_match( $current_address, $saved_address ) ) {
+					if ($status = $this->is_address_match($current_address, $saved_address)) {
 						break;
 					}
 				}
@@ -1502,10 +1495,10 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_address_match( $address_1, $address_2 ) {
-		$result = array_diff( $address_1, $address_2 );
+	private function is_address_match($address_1, $address_2) {
+		$result = array_diff($address_1, $address_2);
 
-		return empty( $result );
+		return empty($result);
 	}
 
 	/**
@@ -1515,22 +1508,22 @@ 
 	 * @since   2.0
 	 * @return  object
 	 */
-	public function split_donor_name( $id ) {
-		$first_name = $last_name  = '';
-		$donor      = new Give_Donor( $id );
+	public function split_donor_name($id) {
+		$first_name = $last_name = '';
+		$donor      = new Give_Donor($id);
 
-		$split_donor_name = explode( ' ', $donor->name, 2 );
+		$split_donor_name = explode(' ', $donor->name, 2);
 
 		// Check for existence of first name after split of donor name.
-		if( is_array( $split_donor_name ) && ! empty( $split_donor_name[0] ) ) {
+		if (is_array($split_donor_name) && ! empty($split_donor_name[0])) {
 			$first_name = $split_donor_name[0];
 		}
 
 		// Check for existence of last name after split of donor name.
-		if( is_array( $split_donor_name ) && ! empty( $split_donor_name[1] ) ) {
+		if (is_array($split_donor_name) && ! empty($split_donor_name[1])) {
 			$last_name = $split_donor_name[1];
 		}
-		return (object) array( 'first_name' => $first_name, 'last_name' => $last_name );
+		return (object) array('first_name' => $first_name, 'last_name' => $last_name);
 	}
 
 	/**
@@ -1540,9 +1533,9 @@ 
 	 * @return  string
 	 */
 	public function get_first_name() {
-		$first_name = $this->get_meta( '_give_donor_first_name');
-		if( ! $first_name ) {
-			$first_name = $this->split_donor_name( $this->id )->first_name;
+		$first_name = $this->get_meta('_give_donor_first_name');
+		if ( ! $first_name) {
+			$first_name = $this->split_donor_name($this->id)->first_name;
 		}
 
 		return $first_name;
@@ -1555,15 +1548,15 @@ 
 	 * @return  string
 	 */
 	public function get_last_name() {
-		$first_name = $this->get_meta( '_give_donor_first_name');
-		$last_name = $this->get_meta( '_give_donor_last_name');
+		$first_name = $this->get_meta('_give_donor_first_name');
+		$last_name = $this->get_meta('_give_donor_last_name');
 
 		// This condition will prevent unnecessary splitting of donor name to fetch last name.
-		if( ! $first_name && ! $last_name ) {
-			$last_name = $this->split_donor_name( $this->id )->last_name;
+		if ( ! $first_name && ! $last_name) {
+			$last_name = $this->split_donor_name($this->id)->last_name;
 		}
 
-		return ( $last_name ) ? $last_name : '';
+		return ($last_name) ? $last_name : '';
 	}
 
 	/**
@@ -1574,7 +1567,7 @@ 
 	 * @return  string $company_name Donor Company Name
 	 */
 	public function get_company_name() {
-		$company_name = $this->get_meta( '_give_donor_company' );
+		$company_name = $this->get_meta('_give_donor_company');
 
 		return $company_name;
 	}

includes/class-give-session.php

Spacing +76 added lines, -76 removed lines

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

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

Braces +4 added lines, -1 removed lines

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

Spacing +22 added lines, -22 removed lines

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