WordImpress / Give

includes/api/class-give-api-v1.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 

includes/class-give-db-customer-meta.php

Spacing +25 added lines, -25 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,11 +33,11 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_customermeta';
+		$this->table_name  = $wpdb->prefix.'give_customermeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
-		add_action( 'plugins_loaded', array( $this, 'register_table' ), 11 );
+		add_action('plugins_loaded', array($this, 'register_table'), 11);
 
 	}
 
@@ -85,13 +85,13 @@ 
 	 *
 	 * @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( $customer_id = 0, $meta_key = '', $single = false ) {
-		$customer_id = $this->sanitize_customer_id( $customer_id );
-		if ( false === $customer_id ) {
+	public function get_meta($customer_id = 0, $meta_key = '', $single = false) {
+		$customer_id = $this->sanitize_customer_id($customer_id);
+		if (false === $customer_id) {
 			return false;
 		}
 
-		return get_metadata( 'customer', $customer_id, $meta_key, $single );
+		return get_metadata('customer', $customer_id, $meta_key, $single);
 	}
 
 	/**
@@ -109,13 +109,13 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function add_meta( $customer_id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$customer_id = $this->sanitize_customer_id( $customer_id );
-		if ( false === $customer_id ) {
+	public function add_meta($customer_id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$customer_id = $this->sanitize_customer_id($customer_id);
+		if (false === $customer_id) {
 			return false;
 		}
 
-		return add_metadata( 'customer', $customer_id, $meta_key, $meta_value, $unique );
+		return add_metadata('customer', $customer_id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -138,13 +138,13 @@ 
 	 *
 	 * @return  bool                  False on failure, true if success.
 	 */
-	public function update_meta( $customer_id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$customer_id = $this->sanitize_customer_id( $customer_id );
-		if ( false === $customer_id ) {
+	public function update_meta($customer_id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$customer_id = $this->sanitize_customer_id($customer_id);
+		if (false === $customer_id) {
 			return false;
 		}
 
-		return update_metadata( 'customer', $customer_id, $meta_key, $meta_value, $prev_value );
+		return update_metadata('customer', $customer_id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -165,8 +165,8 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $customer_id = 0, $meta_key = '', $meta_value = '' ) {
-		return delete_metadata( 'customer', $customer_id, $meta_key, $meta_value );
+	public function delete_meta($customer_id = 0, $meta_key = '', $meta_value = '') {
+		return delete_metadata('customer', $customer_id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -179,7 +179,7 @@ 
 	 */
 	public function create_table() {
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
 		$sql = "CREATE TABLE {$this->table_name} (
 			meta_id bigint(20) NOT NULL AUTO_INCREMENT,
@@ -191,9 +191,9 @@ 
 			KEY meta_key (meta_key)
 			) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 	/**
@@ -206,23 +206,23 @@ 
 	 *
 	 * @return int|bool                The normalized customer ID or false if it's found to not be valid.
 	 */
-	private function sanitize_customer_id( $customer_id ) {
-		if ( ! is_numeric( $customer_id ) ) {
+	private function sanitize_customer_id($customer_id) {
+		if ( ! is_numeric($customer_id)) {
 			return false;
 		}
 
 		$customer_id = (int) $customer_id;
 
 		// We were given a non positive number
-		if ( absint( $customer_id ) !== $customer_id ) {
+		if (absint($customer_id) !== $customer_id) {
 			return false;
 		}
 
-		if ( empty( $customer_id ) ) {
+		if (empty($customer_id)) {
 			return false;
 		}
 
-		return absint( $customer_id );
+		return absint($customer_id);
 
 	}
 

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( $date, $this->get_predefined_dates() ) ) {
+		if (array_key_exists($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 00:00:00';
 			} 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/class-give-cli-commands.php

Spacing +196 added lines, -196 removed lines

@@ -7,12 +7,12 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Add give command.
-WP_CLI::add_command( 'give', 'GIVE_CLI_COMMAND' );
+WP_CLI::add_command('give', 'GIVE_CLI_COMMAND');
 
 
 /**
@@ -68,8 +68,8 @@ 
 	 *
 	 * @subcommand    logo
 	 */
-	public function ascii( $args, $assoc_args ) {
-		WP_CLI::log( file_get_contents( GIVE_PLUGIN_DIR . 'assets/images/give-ascii-logo.txt' ) );
+	public function ascii($args, $assoc_args) {
+		WP_CLI::log(file_get_contents(GIVE_PLUGIN_DIR.'assets/images/give-ascii-logo.txt'));
 	}
 
 
@@ -94,54 +94,54 @@ 
 	 *
 	 * @subcommand    details
 	 */
-	public function details( $args, $assoc_args ) {
+	public function details($args, $assoc_args) {
 
 		/**
 		 * Plugin Information
 		 */
-		WP_CLI::log( $this->color_message( __( 'Give Version: ', 'give' ) ) . GIVE_VERSION );
+		WP_CLI::log($this->color_message(__('Give Version: ', 'give')).GIVE_VERSION);
 
 		/**
 		 * General Information.
 		 */
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'General information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('General information', 'give')).'   ####');
 
-		$success_page = give_get_option( 'success_page' );
-		$failure_page = give_get_option( 'failure_page' );
-		$history_page = give_get_option( 'history_page' );
+		$success_page = give_get_option('success_page');
+		$failure_page = give_get_option('failure_page');
+		$history_page = give_get_option('history_page');
 
-		WP_CLI::log( $this->color_message( sprintf( __( 'Success Page: ', 'give' ) ) ) . ( $success_page ? "[{$success_page}] " . get_permalink( $success_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Failed Donation Page: ', 'give' ) ) . ( $failure_page ? "[{$failure_page}] " . get_permalink( $failure_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Donation History Page: ', 'give' ) ) . ( $history_page ? "[{$history_page}] " . get_permalink( $history_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Country: ', 'give' ) ) . give_get_country() );
+		WP_CLI::log($this->color_message(sprintf(__('Success Page: ', 'give'))).($success_page ? "[{$success_page}] ".get_permalink($success_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Failed Donation Page: ', 'give')).($failure_page ? "[{$failure_page}] ".get_permalink($failure_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Donation History Page: ', 'give')).($history_page ? "[{$history_page}] ".get_permalink($history_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Country: ', 'give')).give_get_country());
 
 		/**
 		 * Currency Information.
 		 */
-		$default_gateway = give_get_option( 'default_gateway' );
+		$default_gateway = give_get_option('default_gateway');
 
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'Currency Information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('Currency Information', 'give')).'   ####');
 
-		WP_CLI::log( $this->color_message( __( 'Currency: ', 'give' ), give_get_currency() ) );
-		WP_CLI::log( $this->color_message( __( 'Currency Position: ', 'give' ), give_get_currency_position() ) );
-		WP_CLI::log( $this->color_message( __( 'Thousand Separator: ', 'give' ), give_get_price_thousand_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Decimal Separator: ', 'give' ), give_get_price_decimal_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Number of Decimals: ', 'give' ), give_get_price_decimals() ) );
-		WP_CLI::log( $this->color_message( __( 'Test Mode: ', 'give' ), ( give_get_option( 'test_mode' ) ? __( 'Yes', 'give' ) : __( 'No', 'give' ) ) ) );
-		WP_CLI::log( $this->color_message( __( 'Default Gateway: ', 'give' ), ( $default_gateway ? $default_gateway : __( 'Not Set', 'give' ) ) ) );
+		WP_CLI::log($this->color_message(__('Currency: ', 'give'), give_get_currency()));
+		WP_CLI::log($this->color_message(__('Currency Position: ', 'give'), give_get_currency_position()));
+		WP_CLI::log($this->color_message(__('Thousand Separator: ', 'give'), give_get_price_thousand_separator()));
+		WP_CLI::log($this->color_message(__('Decimal Separator: ', 'give'), give_get_price_decimal_separator()));
+		WP_CLI::log($this->color_message(__('Number of Decimals: ', 'give'), give_get_price_decimals()));
+		WP_CLI::log($this->color_message(__('Test Mode: ', 'give'), (give_get_option('test_mode') ? __('Yes', 'give') : __('No', 'give'))));
+		WP_CLI::log($this->color_message(__('Default Gateway: ', 'give'), ($default_gateway ? $default_gateway : __('Not Set', 'give'))));
 
 		// Payment gateways Information.
-		$gateways = give_get_ordered_payment_gateways( give_get_payment_gateways() );
-		WP_CLI::log( $this->color_message( __( 'Enabled Gateways: ', 'give' ) ) );
+		$gateways = give_get_ordered_payment_gateways(give_get_payment_gateways());
+		WP_CLI::log($this->color_message(__('Enabled Gateways: ', 'give')));
 
-		if ( ! empty( $gateways ) ) {
+		if ( ! empty($gateways)) {
 			self::$counter = 1;
-			foreach ( $gateways as $gateway ) {
-				WP_CLI::log( '  ' . $this->color_message( self::$counter, $gateway['admin_label'] ) );
-				self::$counter ++;
+			foreach ($gateways as $gateway) {
+				WP_CLI::log('  '.$this->color_message(self::$counter, $gateway['admin_label']));
+				self::$counter++;
 			}
 		} else {
-			WP_CLI::log( __( 'Not any payment gateways found', 'give' ) );
+			WP_CLI::log(__('Not any payment gateways found', 'give'));
 		}
 	}
 
@@ -173,15 +173,15 @@ 
 	 *
 	 * @subcommand    forms
 	 */
-	public function forms( $args, $assoc_args ) {
+	public function forms($args, $assoc_args) {
 		global $wp_query;
-		$form_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$number  = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? absint( $assoc_args['number'] ) : 10;
+		$form_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$number  = isset($assoc_args) && array_key_exists('number', $assoc_args) ? absint($assoc_args['number']) : 10;
 		$start   = time();
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -190,22 +190,22 @@ 
 		$wp_query->query_vars['number'] = $number;
 
 		// Get forms.
-		$forms = $form_id ? $this->api->get_forms( $form_id ) : $this->api->get_forms();
+		$forms = $form_id ? $this->api->get_forms($form_id) : $this->api->get_forms();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
 		// Bailout.
-		if ( array_key_exists( 'error', $forms ) ) {
+		if (array_key_exists('error', $forms)) {
 
-			WP_CLI::warning( $forms['error'] );
+			WP_CLI::warning($forms['error']);
 
 			return;
-		} elseif ( empty( $forms['forms'] ) ) {
+		} elseif (empty($forms['forms'])) {
 
-			WP_CLI::error( __( 'No forms found.', 'give' ) );
+			WP_CLI::error(__('No forms found.', 'give'));
 
 			return;
 		}
@@ -213,25 +213,25 @@ 
 		// Param to check if form typeis already showed or not.
 		$is_show_form_type = false;
 
-		if ( 1 === count( $forms ) && $form_id ) {
+		if (1 === count($forms) && $form_id) {
 			// Show single form.
-			foreach ( $forms['forms'][0] as $key => $info ) {
-				switch ( $key ) {
+			foreach ($forms['forms'][0] as $key => $info) {
+				switch ($key) {
 					case 'stats':
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
-						foreach ( $info as $heading => $data ) {
-							$this->color_sub_heading( ucfirst( $heading ) );
-							switch ( $heading ) {
+						foreach ($info as $heading => $data) {
+							$this->color_sub_heading(ucfirst($heading));
+							switch ($heading) {
 								default:
-									foreach ( $data as $subheading => $subdata ) {
+									foreach ($data as $subheading => $subdata) {
 
-										switch ( $subheading ) {
+										switch ($subheading) {
 											case 'earnings':
-												WP_CLI::log( $this->color_message( $subheading . ': ', give_currency_filter( $subdata ) ) );
+												WP_CLI::log($this->color_message($subheading.': ', give_currency_filter($subdata)));
 												break;
 											default:
-												WP_CLI::log( $this->color_message( $subheading . ': ', $subdata ) );
+												WP_CLI::log($this->color_message($subheading.': ', $subdata));
 										}
 									}
 							}
@@ -241,26 +241,26 @@ 
 					case 'pricing':
 					case 'info':
 					default:
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
 						// Show form type.
-						if ( ! $is_show_form_type ) {
-							$form              = new Give_Donate_Form( $form_id );
+						if ( ! $is_show_form_type) {
+							$form              = new Give_Donate_Form($form_id);
 							$is_show_form_type = true;
 
-							WP_CLI::log( $this->color_message( __( 'form type', 'give' ), $form->get_type() ) );
+							WP_CLI::log($this->color_message(__('form type', 'give'), $form->get_type()));
 						}
 
-						foreach ( $info as $heading => $data ) {
+						foreach ($info as $heading => $data) {
 
-							switch ( $heading ) {
+							switch ($heading) {
 								case 'id':
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									WP_CLI::log($this->color_message($heading, $data));
 									break;
 
 								default:
-									$data = empty( $data ) ? __( 'Not set', 'give' ) : $data;
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									$data = empty($data) ? __('Not set', 'give') : $data;
+									WP_CLI::log($this->color_message($heading, $data));
 							}
 						}
 				}
@@ -271,37 +271,37 @@ 
 			$is_table_first_row_set = false;
 			$table_column_count     = 0;
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%d donation forms found', 'give' ), count( $forms['forms'] ) ), '', false ) );
+			WP_CLI::line($this->color_message(sprintf(__('%d donation forms found', 'give'), count($forms['forms'])), '', false));
 
-			foreach ( $forms['forms'] as $index => $form_data ) {
+			foreach ($forms['forms'] as $index => $form_data) {
 
 				// Default table data.
 				$table_first_row = array();
 				$table_row       = array();
 
-				foreach ( $form_data['info'] as $key => $form ) {
+				foreach ($form_data['info'] as $key => $form) {
 
 					// Do not show thumbnail, content and link in table.
-					if ( in_array( $key, array( 'content', 'thumbnail', 'link' ), true ) ) {
+					if (in_array($key, array('content', 'thumbnail', 'link'), true)) {
 						continue;
 					}
 
-					if ( ! $is_table_first_row_set ) {
+					if ( ! $is_table_first_row_set) {
 						$table_first_row[] = $key;
 					}
 
 					$table_row[] = $form;
 
-					if ( 'status' === $key ) {
+					if ('status' === $key) {
 						// First array item will be an form id in our case.
-						$form = new Give_Donate_Form( absint( $table_row[0] ) );
+						$form = new Give_Donate_Form(absint($table_row[0]));
 
 						$table_row[] = $form->get_type();
 					}
 				}
 
 				// Set table first row.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 
 					// Add extra column to table.
 					$table_first_row[] = 'type';
@@ -314,7 +314,7 @@ 
 				$table_data[] = $table_row;
 			}
 
-			$this->display_table( $table_data );
+			$this->display_table($table_data);
 		}
 	}
 
@@ -366,37 +366,37 @@ 
 	 *
 	 * @subcommand    donors
 	 */
-	public function donors( $args, $assoc_args ) {
+	public function donors($args, $assoc_args) {
 		global $wp_query;
-		$donor_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$email    = isset( $assoc_args ) && array_key_exists( 'email', $assoc_args ) ? $assoc_args['email'] : false;
-		$name     = isset( $assoc_args ) && array_key_exists( 'name', $assoc_args ) ? $assoc_args['name'] : '';
-		$create   = isset( $assoc_args ) && array_key_exists( 'create', $assoc_args ) ? $assoc_args['create'] : false;
-		$number   = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
-		$form_id  = isset( $assoc_args ) && array_key_exists( 'form-id', $assoc_args ) ? $assoc_args['form-id'] : 0;
-		$format   = isset( $assoc_args ) && array_key_exists( 'format', $assoc_args ) ? $assoc_args['format'] : 'table';
+		$donor_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$email    = isset($assoc_args) && array_key_exists('email', $assoc_args) ? $assoc_args['email'] : false;
+		$name     = isset($assoc_args) && array_key_exists('name', $assoc_args) ? $assoc_args['name'] : '';
+		$create   = isset($assoc_args) && array_key_exists('create', $assoc_args) ? $assoc_args['create'] : false;
+		$number   = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
+		$form_id  = isset($assoc_args) && array_key_exists('form-id', $assoc_args) ? $assoc_args['form-id'] : 0;
+		$format   = isset($assoc_args) && array_key_exists('format', $assoc_args) ? $assoc_args['format'] : 'table';
 		$start    = time();
 
-		if ( $create ) {
+		if ($create) {
 			$number = 1;
 
-			if ( isset( $assoc_args['email'] ) && ! is_email( $email ) ) {
-				WP_CLI::warning( 'Wrong email address provided.', 'give' );
+			if (isset($assoc_args['email']) && ! is_email($email)) {
+				WP_CLI::warning('Wrong email address provided.', 'give');
 
 				return;
 			}
 
 			// Create one or more donors.
-			if ( ! $email ) {
+			if ( ! $email) {
 				// If no email is specified, look to see if we are generating arbitrary donor accounts.
-				$number = is_numeric( $create ) ? absint( $create ) : 1;
+				$number = is_numeric($create) ? absint($create) : 1;
 			}
 
-			for ( $i = 0; $i < $number; $i ++ ) {
-				if ( ! $email ) {
+			for ($i = 0; $i < $number; $i++) {
+				if ( ! $email) {
 
 					// Generate fake email.
-					$email = 'customer-' . uniqid() . '@test.com';
+					$email = 'customer-'.uniqid().'@test.com';
 				}
 
 				$args = array(
@@ -404,19 +404,19 @@ 
 					'name'  => $name,
 				);
 
-				$customer_id = Give()->customers->add( $args );
+				$customer_id = Give()->customers->add($args);
 
-				if ( $customer_id ) {
-					WP_CLI::line( $this->color_message( sprintf( __( 'Donor #%d created successfully', 'give' ), $customer_id ) ) );
+				if ($customer_id) {
+					WP_CLI::line($this->color_message(sprintf(__('Donor #%d created successfully', 'give'), $customer_id)));
 				} else {
-					WP_CLI::error( __( 'Failed to create donor', 'give' ) );
+					WP_CLI::error(__('Failed to create donor', 'give'));
 				}
 
 				// Reset email to false so it is generated on the next loop (if creating donors).
 				$email = false;
 			}
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%1$d donors created in %2$d seconds', 'give' ), $number, time() - $start ) ) );
+			WP_CLI::line($this->color_message(sprintf(__('%1$d donors created in %2$d seconds', 'give'), $number, time() - $start)));
 
 		} else {
 			// Counter.
@@ -430,7 +430,7 @@ 
 			 */
 			// Cache previous number query var.
 			$is_set_number = $cache_per_page = false;
-			if ( isset( $wp_query->query_vars['number'] ) ) {
+			if (isset($wp_query->query_vars['number'])) {
 				$cache_per_page = $wp_query->query_vars['number'];
 				$is_set_number  = true;
 			}
@@ -439,24 +439,24 @@ 
 			$wp_query->query_vars['number'] = $number;
 
 			// Get donors.
-			if ( $form_id ) {
+			if ($form_id) {
 				// @TODO: Allow user to get a list of donors by donation status.
-				$donors = $this->get_donors_by_form_id( $form_id );
+				$donors = $this->get_donors_by_form_id($form_id);
 			} else {
-				$donors = $this->api->get_customers( $search );
+				$donors = $this->api->get_customers($search);
 			}
 
 			// Reset number query var.
-			if ( $is_set_number ) {
+			if ($is_set_number) {
 				$wp_query->query_vars['number'] = $cache_per_page;
 			}
 
-			if ( isset( $donors['error'] ) ) {
-				WP_CLI::error( $donors['error'] );
+			if (isset($donors['error'])) {
+				WP_CLI::error($donors['error']);
 			}
 
-			if ( empty( $donors ) ) {
-				WP_CLI::error( __( 'No donors found.', 'give' ) );
+			if (empty($donors)) {
+				WP_CLI::error(__('No donors found.', 'give'));
 
 				return;
 			}
@@ -464,24 +464,24 @@ 
 			$table_data             = array();
 			$is_table_first_row_set = false;
 
-			foreach ( $donors['donors'] as $donor_data ) {
+			foreach ($donors['donors'] as $donor_data) {
 				// Set default table row data.
-				$table_first_row = array( __( 's_no', 'give' ) );
-				$table_row       = array( self::$counter );
+				$table_first_row = array(__('s_no', 'give'));
+				$table_row       = array(self::$counter);
 
-				foreach ( $donor_data as $key => $donor ) {
-					switch ( $key ) {
+				foreach ($donor_data as $key => $donor) {
+					switch ($key) {
 						case 'stats':
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
-								switch ( $heading ) {
+								switch ($heading) {
 									case 'total_spent':
-										$table_row[] = give_currency_filter( $data );
+										$table_row[] = give_currency_filter($data);
 										break;
 
 									default:
@@ -492,10 +492,10 @@ 
 
 						case 'info':
 						default:
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
@@ -505,7 +505,7 @@ 
 				}
 
 				// Add first row data to table data.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 					$table_data[]           = $table_first_row;
 					$is_table_first_row_set = true;
 				}
@@ -514,44 +514,44 @@ 
 				$table_data[] = $table_row;
 
 				// Increase counter.
-				self::$counter ++;
+				self::$counter++;
 			}
 
-			switch ( $format ) {
+			switch ($format) {
 				case 'json':
 					$table_column_name = $table_data[0];
-					unset( $table_data[0] );
+					unset($table_data[0]);
 
 					$new_table_data = array();
-					foreach ( $table_data as $index => $data ) {
-						foreach ( $data as $key => $value ) {
-							$new_table_data[ $index ][ $table_column_name[ $key ] ] = $value;
+					foreach ($table_data as $index => $data) {
+						foreach ($data as $key => $value) {
+							$new_table_data[$index][$table_column_name[$key]] = $value;
 						}
 					}
 
-					WP_CLI::log( json_encode( $new_table_data ) );
+					WP_CLI::log(json_encode($new_table_data));
 					break;
 
 				case 'csv':
-					$file_path = trailingslashit( WP_CONTENT_DIR ) . 'uploads/give_donors_' . date( 'Y_m_d_s', current_time( 'timestamp' ) ) . '.csv';
-					$fp        = fopen( $file_path, 'w' );
+					$file_path = trailingslashit(WP_CONTENT_DIR).'uploads/give_donors_'.date('Y_m_d_s', current_time('timestamp')).'.csv';
+					$fp        = fopen($file_path, 'w');
 
-					if ( is_writable( $file_path ) ) {
-						foreach ( $table_data as $fields ) {
-							fputcsv( $fp, $fields );
+					if (is_writable($file_path)) {
+						foreach ($table_data as $fields) {
+							fputcsv($fp, $fields);
 						}
 
-						fclose( $fp );
+						fclose($fp);
 
-						WP_CLI::success( "Donors list csv created successfully: {$file_path}" );
+						WP_CLI::success("Donors list csv created successfully: {$file_path}");
 					} else {
-						WP_CLI::warning( "Unable to create donors list csv file: {$file_path} (May folder do not have write permission)" );
+						WP_CLI::warning("Unable to create donors list csv file: {$file_path} (May folder do not have write permission)");
 					}
 
 					break;
 
 				default:
-					$this->display_table( $table_data );
+					$this->display_table($table_data);
 			}
 		}
 	}
@@ -581,13 +581,13 @@ 
 	 *
 	 * @subcommand    donations
 	 */
-	public function donations( $args, $assoc_args ) {
+	public function donations($args, $assoc_args) {
 		global $wp_query;
-		$number = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
+		$number = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -599,46 +599,46 @@ 
 		$donations = $this->api->get_recent_donations();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
-		if ( empty( $donations ) ) {
-			WP_CLI::error( __( 'No donations found.', 'give' ) );
+		if (empty($donations)) {
+			WP_CLI::error(__('No donations found.', 'give'));
 
 			return;
 		}
 
 		self::$counter = 1;
 
-		foreach ( $donations['donations'] as $key => $donation ) {
-			$this->color_main_heading( sprintf( __( '%1$s. Donation #%2$s', 'give' ), self::$counter, $donation['ID'] ), 'Y' );
-			self::$counter ++;
+		foreach ($donations['donations'] as $key => $donation) {
+			$this->color_main_heading(sprintf(__('%1$s. Donation #%2$s', 'give'), self::$counter, $donation['ID']), 'Y');
+			self::$counter++;
 
-			foreach ( $donation as $column => $data ) {
+			foreach ($donation as $column => $data) {
 
-				if ( is_array( $data ) ) {
-					$this->color_sub_heading( $column );
-					foreach ( $data as $subcolumn => $subdata ) {
+				if (is_array($data)) {
+					$this->color_sub_heading($column);
+					foreach ($data as $subcolumn => $subdata) {
 
 						// Decode html codes.
-						switch ( $subcolumn ) {
+						switch ($subcolumn) {
 							case 'name':
-								$subdata = html_entity_decode( $subdata );
+								$subdata = html_entity_decode($subdata);
 								break;
 						}
 
 						// @TODO Check if multi dimension array information is importent to show or not. For example inside donation array we have array for fees data inside payment meta.
-						if ( is_array( $subdata ) ) {
+						if (is_array($subdata)) {
 							continue;
 						}
 
-						WP_CLI::log( $this->color_message( $subcolumn, $subdata ) );
+						WP_CLI::log($this->color_message($subcolumn, $subdata));
 					}
 					continue;
 				}
 
-				WP_CLI::log( $this->color_message( $column, $data ) );
+				WP_CLI::log($this->color_message($column, $data));
 			}
 		}
 	}
@@ -677,27 +677,27 @@ 
 	 *
 	 * @return        void
 	 */
-	public function report( $args, $assoc_args ) {
+	public function report($args, $assoc_args) {
 		$stats      = new Give_Payment_Stats();
-		$date       = isset( $assoc_args ) && array_key_exists( 'date', $assoc_args ) ? $assoc_args['date'] : false;
-		$start_date = isset( $assoc_args ) && array_key_exists( 'start-date', $assoc_args ) ? $assoc_args['start-date'] : false;
-		$end_date   = isset( $assoc_args ) && array_key_exists( 'end-date', $assoc_args ) ? $assoc_args['end-date'] : false;
-		$form_id    = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? $assoc_args['id'] : 0;
+		$date       = isset($assoc_args) && array_key_exists('date', $assoc_args) ? $assoc_args['date'] : false;
+		$start_date = isset($assoc_args) && array_key_exists('start-date', $assoc_args) ? $assoc_args['start-date'] : false;
+		$end_date   = isset($assoc_args) && array_key_exists('end-date', $assoc_args) ? $assoc_args['end-date'] : false;
+		$form_id    = isset($assoc_args) && array_key_exists('id', $assoc_args) ? $assoc_args['id'] : 0;
 
-		if ( ! empty( $date ) ) {
+		if ( ! empty($date)) {
 			$start_date = $date;
 			$end_date   = false;
-		} elseif ( empty( $date ) && empty( $start_date ) ) {
+		} elseif (empty($date) && empty($start_date)) {
 			$start_date = 'this_month';
 			$end_date   = false;
 		}
 
 		// Get stats.
-		$earnings = $stats->get_earnings( $form_id, $start_date, $end_date );
-		$sales    = $stats->get_sales( $form_id, $start_date, $end_date );
+		$earnings = $stats->get_earnings($form_id, $start_date, $end_date);
+		$sales    = $stats->get_sales($form_id, $start_date, $end_date);
 
-		WP_CLI::line( $this->color_message( __( 'Earnings', 'give' ), give_currency_filter( $earnings ) ) );
-		WP_CLI::line( $this->color_message( __( 'Sales', 'give' ), $sales ) );
+		WP_CLI::line($this->color_message(__('Earnings', 'give'), give_currency_filter($earnings)));
+		WP_CLI::line($this->color_message(__('Sales', 'give'), $sales));
 	}
 
 
@@ -724,27 +724,27 @@ 
 	 *
 	 * @subcommand    cache
 	 */
-	public function cache( $args, $assoc_args ) {
-		$action = isset( $assoc_args ) && array_key_exists( 'action', $assoc_args ) ? $assoc_args['action'] : false;
+	public function cache($args, $assoc_args) {
+		$action = isset($assoc_args) && array_key_exists('action', $assoc_args) ? $assoc_args['action'] : false;
 
 		// Bailout.
-		if ( ! $action || ! in_array( $action, array( 'delete' ), true ) ) {
-			WP_CLI::warning( __( 'Type wp give cache --action=delete to delete all stat transients', 'give' ) );
+		if ( ! $action || ! in_array($action, array('delete'), true)) {
+			WP_CLI::warning(__('Type wp give cache --action=delete to delete all stat transients', 'give'));
 
 			return;
 		}
 
-		switch ( $action ) {
+		switch ($action) {
 			case 'delete' :
 				// Reset counter.
 				self::$counter = 1;
 
-				if ( $this->delete_stats_transients() ) {
+				if ($this->delete_stats_transients()) {
 					// Report .eading.
-					WP_CLI::success( 'Give cache deleted.' );
+					WP_CLI::success('Give cache deleted.');
 				} else {
 					// Report .eading.
-					WP_CLI::warning( 'We did not find any Give plugin cache to delete :)' );
+					WP_CLI::warning('We did not find any Give plugin cache to delete :)');
 				}
 				break;
 		}
@@ -773,27 +773,27 @@ 
 			ARRAY_A
 		);
 
-		if ( ! empty( $stat_option_names ) ) {
+		if ( ! empty($stat_option_names)) {
 
-			foreach ( $stat_option_names as $option_name ) {
+			foreach ($stat_option_names as $option_name) {
 				$error       = false;
 				$option_name = $option_name['option_name'];
 
-				switch ( true ) {
-					case ( false !== strpos( $option_name, 'transient' ) ):
-						$option_name = str_replace( '_transient_', '', $option_name );
-						$error       = delete_transient( $option_name );
+				switch (true) {
+					case (false !== strpos($option_name, 'transient')):
+						$option_name = str_replace('_transient_', '', $option_name);
+						$error       = delete_transient($option_name);
 						break;
 
 					default:
-						$error = delete_option( $option_name );
+						$error = delete_option($option_name);
 				}
 
-				if ( $error ) {
-					WP_CLI::log( $this->color_message( self::$counter, $option_name ) );
-					self::$counter ++;
+				if ($error) {
+					WP_CLI::log($this->color_message(self::$counter, $option_name));
+					self::$counter++;
 				} else {
-					WP_CLI::log( $this->color_message( __( 'Error while deleting this transient', 'give' ), $option_name ) );
+					WP_CLI::log($this->color_message(__('Error while deleting this transient', 'give'), $option_name));
 				}
 			}
 
@@ -814,13 +814,13 @@ 
 	 *
 	 * @return   string
 	 */
-	private function color_message( $heading, $message = '', $colon = true, $color = 'g' ) {
+	private function color_message($heading, $message = '', $colon = true, $color = 'g') {
 		// Add colon.
-		if ( $colon ) {
-			$heading = $heading . ': ';
+		if ($colon) {
+			$heading = $heading.': ';
 		}
 
-		return WP_CLI::colorize( "%{$color}" . $heading . '%n' ) . $message;
+		return WP_CLI::colorize("%{$color}".$heading.'%n').$message;
 	}
 
 
@@ -835,8 +835,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_main_heading( $heading, $color = 'g' ) {
-		WP_CLI::log( "\n######   " . $this->color_message( $heading, '', false, $color ) . '   ######' );
+	private function color_main_heading($heading, $color = 'g') {
+		WP_CLI::log("\n######   ".$this->color_message($heading, '', false, $color).'   ######');
 	}
 
 	/**
@@ -849,8 +849,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_sub_heading( $subheading ) {
-		WP_CLI::log( "\n--->" . $subheading . '', '', false );
+	private function color_sub_heading($subheading) {
+		WP_CLI::log("\n--->".$subheading.'', '', false);
 	}
 
 
@@ -864,17 +864,17 @@ 
 	 *
 	 * @return    void
 	 */
-	private function display_table( $data ) {
+	private function display_table($data) {
 		$table = new \cli\Table();
 
 		// Set table header.
-		$table->setHeaders( $data[0] );
+		$table->setHeaders($data[0]);
 
 		// Remove table header.
-		unset( $data[0] );
+		unset($data[0]);
 
 		// Set table data.
-		$table->setRows( $data );
+		$table->setRows($data);
 
 		// Display table.
 		$table->display();
@@ -891,14 +891,14 @@ 
 	 * @return array
 	 */
 
-	private function get_donors_by_form_id( $form_id ) {
+	private function get_donors_by_form_id($form_id) {
 		$donors = array();
 
 		$donations = new Give_Payments_Query(
 			array(
-				'give_forms' => array( $form_id ),
-				'number'     => - 1,
-				'status'     => array( 'publish' ),
+				'give_forms' => array($form_id),
+				'number'     => -1,
+				'status'     => array('publish'),
 			)
 		);
 
@@ -906,16 +906,16 @@ 
 		$skip_donors = array();
 
 		/* @var Give_Payment|object $donation Payment object. */
-		foreach ( $donations as $donation ) {
+		foreach ($donations as $donation) {
 
-			if ( in_array( $donation->customer_id, $skip_donors ) ) {
+			if (in_array($donation->customer_id, $skip_donors)) {
 				continue;
 			}
 
-			if ( ! empty( $donors ) ) {
-				$donors['donors'][] = current( current( $this->api->get_customers( (int) $donation->customer_id ) ) );
+			if ( ! empty($donors)) {
+				$donors['donors'][] = current(current($this->api->get_customers((int) $donation->customer_id)));
 			} else {
-				$donors = array_merge( $donors, $this->api->get_customers( (int) $donation->customer_id ) );
+				$donors = array_merge($donors, $this->api->get_customers((int) $donation->customer_id));
 			}
 
 			$skip_donors[] = $donation->customer_id;

includes/filters.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,29 +26,29 @@ 
  *
  * @return void
  */
-function give_set_settings_with_disable_prefix( $old_settings, $settings ) {
+function give_set_settings_with_disable_prefix($old_settings, $settings) {
 	// Get old setting names.
-	$old_settings   = array_flip( give_v18_renamed_core_settings() );
+	$old_settings   = array_flip(give_v18_renamed_core_settings());
 	$update_setting = false;
 
-	foreach ( $settings as $key => $value ) {
+	foreach ($settings as $key => $value) {
 
 		// Check 1. Check if new option is really updated or not.
 		// Check 2. Continue if key is not renamed.
-		if ( ! isset( $old_settings[ $key ] ) ) {
+		if ( ! isset($old_settings[$key])) {
 			continue;
 		}
 
 		// Set old setting.
-		$settings[ $old_settings[ $key ] ] = 'on';
+		$settings[$old_settings[$key]] = 'on';
 
 		// Do not need to set old setting if new setting is not set.
 		if (
-			( give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'disable_' ) ) )
-			|| ( ! give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'enable_' ) ) )
+			(give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'disable_')))
+			|| ( ! give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'enable_')))
 
 		) {
-			unset( $settings[ $old_settings[ $key ] ] );
+			unset($settings[$old_settings[$key]]);
 		}
 
 		// Tell bot to update setting.
@@ -56,8 +56,8 @@ 
 	}
 
 	// Update setting if any old setting set.
-	if ( $update_setting ) {
-		update_option( 'give_settings', $settings );
+	if ($update_setting) {
+		update_option('give_settings', $settings);
 	}
 }
-add_action( 'update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2 );
+add_action('update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2);

includes/gateways/actions.php

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,14 +21,14 @@ 
  *
  * @param $data
  */
-function give_process_gateway_select( $data ) {
-	if ( isset( $_POST['gateway_submit'] ) ) {
-		wp_redirect( esc_url( add_query_arg( 'payment-mode', $_POST['payment-mode'] ) ) );
+function give_process_gateway_select($data) {
+	if (isset($_POST['gateway_submit'])) {
+		wp_redirect(esc_url(add_query_arg('payment-mode', $_POST['payment-mode'])));
 		exit;
 	}
 }
 
-add_action( 'give_gateway_select', 'give_process_gateway_select' );
+add_action('give_gateway_select', 'give_process_gateway_select');
 
 /**
  * Loads a payment gateway via AJAX.
@@ -38,20 +38,20 @@ 
  * @return void
  */
 function give_load_ajax_gateway() {
-	if ( isset( $_POST['give_payment_mode'] ) ) {
+	if (isset($_POST['give_payment_mode'])) {
 		/**
 		 * Fire to render donation form.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form', $_POST['give_form_id'] );
+		do_action('give_donation_form', $_POST['give_form_id']);
 
 		exit();
 	}
 }
 
-add_action( 'wp_ajax_give_load_gateway', 'give_load_ajax_gateway' );
-add_action( 'wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway' );
+add_action('wp_ajax_give_load_gateway', 'give_load_ajax_gateway');
+add_action('wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway');
 
 /**
  * Sets an error within the donation form if no gateways are enabled.
@@ -63,11 +63,11 @@ 
 function give_no_gateway_error() {
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( empty( $gateways ) ) {
-		give_set_error( 'no_gateways', esc_html__( 'You must enable a payment gateway to use Give.', 'give' ) );
+	if (empty($gateways)) {
+		give_set_error('no_gateways', esc_html__('You must enable a payment gateway to use Give.', 'give'));
 	} else {
-		give_unset_error( 'no_gateways' );
+		give_unset_error('no_gateways');
 	}
 }
 
-add_action( 'init', 'give_no_gateway_error' );
+add_action('init', 'give_no_gateway_error');

includes/admin/reporting/class-donor-reports-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return mixed string If search is present, false otherwise
+	 * @return string|false string If search is present, false otherwise
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Indentation +2 added lines, -2 removed lines

@@ -185,8 +185,8 @@
 				
 			case 'num_donations' :
 				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
+						 admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
+						 ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
 				break;
 
 			case 'amount_spent' :

Spacing +68 added lines, -68 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box donor-search" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h3 class="alignleft reports-earnings-title">
-					<span><?php esc_html_e( 'Donors Report', 'give' ); ?></span>
+					<span><?php esc_html_e('Donors Report', 'give'); ?></span>
 				</h3>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,33 +172,33 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
-				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
+				$value = '<a href="'.
+				         admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
+				         ).'">'.esc_html($item['num_donations']).'</a>';
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name]));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -210,13 +210,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => esc_html__( 'Name', 'give' ),
-			'email'         => esc_html__( 'Email', 'give' ),
-			'num_donations' => esc_html__( 'Donations', 'give' ),
-			'amount_spent'  => esc_html__( 'Total Donated', 'give' ),
+			'name'          => esc_html__('Name', 'give'),
+			'email'         => esc_html__('Email', 'give'),
+			'num_donations' => esc_html__('Donations', 'give'),
+			'amount_spent'  => esc_html__('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -229,10 +229,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -243,7 +243,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -255,7 +255,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -266,7 +266,7 @@ 
 	 * @return mixed string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -285,15 +285,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->customers->get_customers( $args );
+		$donors = Give()->customers->get_customers($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -320,9 +320,9 @@ 
 		$_donor_query = $this->get_donor_query();
 
 		$_donor_query['number'] = -1;
-		$donors = Give()->customers->get_customers( $_donor_query );
+		$donors = Give()->customers->get_customers($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -334,10 +334,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -346,10 +346,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -376,16 +376,16 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->reports_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/abstract-admin-settings-page.php

Spacing +22 added lines, -22 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Page' ) ) :
+if ( ! class_exists('Give_Settings_Page')) :
 
 	/**
 	 * Give_Settings_Page.
@@ -62,11 +62,11 @@ 
 			// Get current setting page.
 			$this->current_setting_page = give_get_current_setting_page();
 
-			add_filter( "give_default_setting_tab_section_{$this->id}", array( $this, 'set_default_setting_tab' ), 10 );
-			add_filter( "{$this->current_setting_page}_tabs_array", array( $this, 'add_settings_page' ), 20 );
-			add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array( $this, 'output_sections' ) );
-			add_action( "{$this->current_setting_page}_settings_{$this->id}_page", array( $this, 'output' ) );
-			add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
+			add_filter("give_default_setting_tab_section_{$this->id}", array($this, 'set_default_setting_tab'), 10);
+			add_filter("{$this->current_setting_page}_tabs_array", array($this, 'add_settings_page'), 20);
+			add_action("{$this->current_setting_page}_sections_{$this->id}_page", array($this, 'output_sections'));
+			add_action("{$this->current_setting_page}_settings_{$this->id}_page", array($this, 'output'));
+			add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
 		}
 
 		/**
@@ -78,7 +78,7 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			return $this->default_tab;
 		}
 
@@ -91,8 +91,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -111,7 +111,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, array() );
+			$settings = apply_filters('give_get_settings_'.$this->id, array());
 
 			// Output.
 			return $settings;
@@ -124,7 +124,7 @@ 
 		 * @return array
 		 */
 		public function get_sections() {
-			return apply_filters( 'give_get_sections_' . $this->id, array() );
+			return apply_filters('give_get_sections_'.$this->id, array());
 		}
 
 		/**
@@ -141,29 +141,29 @@ 
 			$sections = $this->get_sections();
 
 			// Show section settings only if setting section exist.
-			if ( $current_section && ! in_array( $current_section, array_keys( $sections ) ) ) {
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+			if ($current_section && ! in_array($current_section, array_keys($sections))) {
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
-			if ( is_null( $this->current_setting_page ) ) {
+			if (is_null($this->current_setting_page)) {
 				$this->current_setting_page = give_get_current_setting_page();
 			}
 
 			echo '<ul class="subsubsub">';
 
 			// Get section keys.
-			$array_keys = array_keys( $sections );
+			$array_keys = array_keys($sections);
 
-			foreach ( $sections as $id => $label ) {
-				echo '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=' . $this->current_setting_page . '&tab=' . $this->id . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $current_section == $id ? 'current' : '' ) . '">' . $label . '</a> ' . ( end( $array_keys ) == $id ? '' : '|' ) . ' </li>';
+			foreach ($sections as $id => $label) {
+				echo '<li><a href="'.admin_url('edit.php?post_type=give_forms&page='.$this->current_setting_page.'&tab='.$this->id.'&section='.sanitize_title($id)).'" class="'.($current_section == $id ? 'current' : '').'">'.$label.'</a> '.(end($array_keys) == $id ? '' : '|').' </li>';
 			}
 
 			echo '</ul><br class="clear" /><hr>';
@@ -178,7 +178,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -191,14 +191,14 @@ 
 			$settings        = $this->get_settings();
 			$current_section = give_get_current_setting_section();
 
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
 
 			/**
 			 * Trigger Action
 			 *
 			 * @since 1.8
 			 */
-			do_action( 'give_update_options_' . $this->id . '_' . $current_section );
+			do_action('give_update_options_'.$this->id.'_'.$current_section);
 		}
 	}
 

includes/admin/reporting/class-settings-gateways.php

Spacing +16 added lines, -16 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Gateways' ) ) :
+if ( ! class_exists('Give_Settings_Gateways')) :
 
 	/**
 	 * Give_Settings_Gateways.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'gateways';
-			$this->label = esc_html__( 'Donation Methods', 'give' );
+			$this->label = esc_html__('Donation Methods', 'give');
 
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
-			add_action( 'give_admin_field_report_gateways', array( $this, 'render_report_gateways_field' ), 10, 2 );
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
+			add_action('give_admin_field_report_gateways', array($this, 'render_report_gateways_field'), 10, 2);
 
 			// Do not use main form for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( 'give-reports_open_form', '__return_empty_string' );
-				add_action( 'give-reports_close_form', '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action('give-reports_open_form', '__return_empty_string');
+				add_action('give-reports_close_form', '__return_empty_string');
 			}
 		}
 
@@ -63,8 +63,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -86,7 +86,7 @@ 
 			 * @param  array $settings
 			 */
 			$settings = apply_filters(
-				'give_get_settings_' . $this->id,
+				'give_get_settings_'.$this->id,
 				array(
 					array(
 						'id'   => 'give_reports_gateways',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'gateways',
-						'name' => esc_html__( 'Gateways', 'give' ),
+						'name' => esc_html__('Gateways', 'give'),
 						'type' => 'report_gateways',
 					),
 					array(
@@ -119,7 +119,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -131,8 +131,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_report_gateways_field( $field, $option_value ) {
-			do_action( 'give_reports_view_gateways');
+		public function render_report_gateways_field($field, $option_value) {
+			do_action('give_reports_view_gateways');
 		}
 	}