WordImpress / Give

includes/class-give-cache.php

Doc Comments +3 added lines, -3 removed lines

@@ -121,7 +121,7 @@ 
 	 *
 	 * @param  string $cache_key
 	 * @param  bool   $custom_key
-	 * @param  mixed  $query_args
+	 * @param  string  $query_args
 	 *
 	 * @return mixed
 	 */
@@ -237,7 +237,7 @@ 
 	 *
 	 * @param bool  $force If set to true then all cached values will be delete instead of only expired
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public static function delete_all_expired( $force = false ) {
 		global $wpdb;
@@ -669,7 +669,7 @@ 
 	 * @since  2.0
 	 * @access private
 	 *
-	 * @param $group
+	 * @param string $group
 	 *
 	 * @return mixed
 	 */

Spacing +125 added lines, -125 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,7 +53,7 @@ 
 	 * @return static
 	 */
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) && ! ( self::$instance instanceof Give_Cache ) ) {
+		if ( ! isset(self::$instance) && ! (self::$instance instanceof Give_Cache)) {
 			self::$instance = new Give_Cache();
 		}
 
@@ -68,17 +68,17 @@ 
 	 */
 	public function setup() {
 		// Currently enable cache only for backend.
-		self::$instance->is_cache = give_is_setting_enabled( give_get_option( 'cache', 'enabled' ) ) && is_admin();
+		self::$instance->is_cache = give_is_setting_enabled(give_get_option('cache', 'enabled')) && is_admin();
 
 		// weekly delete all expired cache.
-		Give_Cron::add_weekly_event( array( $this, 'delete_all_expired' ) );
+		Give_Cron::add_weekly_event(array($this, 'delete_all_expired'));
 
-		add_action( 'save_post_give_forms', array( $this, 'delete_form_related_cache' ) );
-		add_action( 'save_post_give_payment', array( $this, 'delete_payment_related_cache' ) );
-		add_action( 'give_deleted_give-donors_cache', array( $this, 'delete_donor_related_cache' ), 10, 3 );
-		add_action( 'give_deleted_give-donations_cache', array( $this, 'delete_donations_related_cache' ), 10, 3 );
+		add_action('save_post_give_forms', array($this, 'delete_form_related_cache'));
+		add_action('save_post_give_payment', array($this, 'delete_payment_related_cache'));
+		add_action('give_deleted_give-donors_cache', array($this, 'delete_donor_related_cache'), 10, 3);
+		add_action('give_deleted_give-donations_cache', array($this, 'delete_donations_related_cache'), 10, 3);
 
-		add_action( 'give_save_settings_give_settings', array( $this, 'flush_cache' ) );
+		add_action('give_save_settings_give_settings', array($this, 'flush_cache'));
 	}
 
 	/**
@@ -92,18 +92,18 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_key( $action, $query_args = null, $is_prefix = true ) {
+	public static function get_key($action, $query_args = null, $is_prefix = true) {
 		// Bailout.
-		if ( empty( $action ) ) {
-			return new WP_Error( 'give_invalid_cache_key_action', __( 'Do not pass empty action to generate cache key.', 'give' ) );
+		if (empty($action)) {
+			return new WP_Error('give_invalid_cache_key_action', __('Do not pass empty action to generate cache key.', 'give'));
 		}
 
 		// Set cache key.
 		$cache_key = $is_prefix ? "give_cache_{$action}" : $action;
 
 		// Bailout.
-		if ( ! empty( $query_args ) ) {
-			$cache_key = "{$cache_key}_" . substr( md5( serialize( $query_args ) ), 0, 15 );
+		if ( ! empty($query_args)) {
+			$cache_key = "{$cache_key}_".substr(md5(serialize($query_args)), 0, 15);
 		}
 
 		/**
@@ -111,7 +111,7 @@ 
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_get_cache_key', $cache_key, $action, $query_args );
+		return apply_filters('give_get_cache_key', $cache_key, $action, $query_args);
 	}
 
 	/**
@@ -125,26 +125,26 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get( $cache_key, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if ( ! $custom_key ) {
-				return new WP_Error( 'give_invalid_cache_key', __( 'Cache key format should be give_cache_*', 'give' ) );
+	public static function get($cache_key, $custom_key = false, $query_args = array()) {
+		if ( ! self::is_valid_cache_key($cache_key)) {
+			if ( ! $custom_key) {
+				return new WP_Error('give_invalid_cache_key', __('Cache key format should be give_cache_*', 'give'));
 			}
 
-			$cache_key = self::get_key( $cache_key, $query_args );
+			$cache_key = self::get_key($cache_key, $query_args);
 		}
 
-		$option = get_option( $cache_key );
+		$option = get_option($cache_key);
 
 		// Backward compatibility (<1.8.7).
-		if ( ! is_array( $option ) || empty( $option ) || ! array_key_exists( 'expiration', $option ) ) {
+		if ( ! is_array($option) || empty($option) || ! array_key_exists('expiration', $option)) {
 			return $option;
 		}
 
 		// Get current time.
-		$current_time = current_time( 'timestamp', 1 );
+		$current_time = current_time('timestamp', 1);
 
-		if ( empty( $option['expiration'] ) || ( $current_time < $option['expiration'] ) ) {
+		if (empty($option['expiration']) || ($current_time < $option['expiration'])) {
 			$option = $option['data'];
 		} else {
 			$option = false;
@@ -166,23 +166,23 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function set( $cache_key, $data, $expiration = null, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if ( ! $custom_key ) {
-				return new WP_Error( 'give_invalid_cache_key', __( 'Cache key format should be give_cache_*', 'give' ) );
+	public static function set($cache_key, $data, $expiration = null, $custom_key = false, $query_args = array()) {
+		if ( ! self::is_valid_cache_key($cache_key)) {
+			if ( ! $custom_key) {
+				return new WP_Error('give_invalid_cache_key', __('Cache key format should be give_cache_*', 'give'));
 			}
 
-			$cache_key = self::get_key( $cache_key, $query_args );
+			$cache_key = self::get_key($cache_key, $query_args);
 		}
 
 		$option_value = array(
 			'data'       => $data,
-			'expiration' => ! is_null( $expiration )
-				? ( $expiration + current_time( 'timestamp', 1 ) )
+			'expiration' => ! is_null($expiration)
+				? ($expiration + current_time('timestamp', 1))
 				: null,
 		);
 
-		$result = update_option( $cache_key, $option_value, 'no' );
+		$result = update_option($cache_key, $option_value, 'no');
 
 		return $result;
 	}
@@ -198,27 +198,27 @@ 
 	 *
 	 * @return bool|WP_Error
 	 */
-	public static function delete( $cache_keys ) {
+	public static function delete($cache_keys) {
 		$result       = true;
 		$invalid_keys = array();
 
-		if ( ! empty( $cache_keys ) ) {
-			$cache_keys = is_array( $cache_keys ) ? $cache_keys : array( $cache_keys );
+		if ( ! empty($cache_keys)) {
+			$cache_keys = is_array($cache_keys) ? $cache_keys : array($cache_keys);
 
-			foreach ( $cache_keys as $cache_key ) {
-				if ( ! self::is_valid_cache_key( $cache_key ) ) {
+			foreach ($cache_keys as $cache_key) {
+				if ( ! self::is_valid_cache_key($cache_key)) {
 					$invalid_keys[] = $cache_key;
 					$result         = false;
 				}
 
-				delete_option( $cache_key );
+				delete_option($cache_key);
 			}
 		}
 
-		if ( ! $result ) {
+		if ( ! $result) {
 			$result = new WP_Error(
 				'give_invalid_cache_key',
-				__( 'Cache key format should be give_cache_*', 'give' ),
+				__('Cache key format should be give_cache_*', 'give'),
 				$invalid_keys
 			);
 		}
@@ -239,7 +239,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function delete_all_expired( $force = false ) {
+	public static function delete_all_expired($force = false) {
 		global $wpdb;
 		$options = $wpdb->get_results(
 			$wpdb->prepare(
@@ -253,30 +253,30 @@ 
 		);
 
 		// Bailout.
-		if ( empty( $options ) ) {
+		if (empty($options)) {
 			return false;
 		}
 
-		$current_time = current_time( 'timestamp', 1 );
+		$current_time = current_time('timestamp', 1);
 
 		// Delete log cache.
-		foreach ( $options as $option ) {
-			$option['option_value'] = maybe_unserialize( $option['option_value'] );
+		foreach ($options as $option) {
+			$option['option_value'] = maybe_unserialize($option['option_value']);
 
 			if (
 				(
-					! self::is_valid_cache_key( $option['option_name'] )
-					|| ! is_array( $option['option_value'] ) // Backward compatibility (<1.8.7).
-					|| ! array_key_exists( 'expiration', $option['option_value'] ) // Backward compatibility (<1.8.7).
-					|| empty( $option['option_value']['expiration'] )
-					|| ( $current_time < $option['option_value']['expiration'] )
+					! self::is_valid_cache_key($option['option_name'])
+					|| ! is_array($option['option_value']) // Backward compatibility (<1.8.7).
+					|| ! array_key_exists('expiration', $option['option_value']) // Backward compatibility (<1.8.7).
+					|| empty($option['option_value']['expiration'])
+					|| ($current_time < $option['option_value']['expiration'])
 				)
 				&& ! $force
 			) {
 				continue;
 			}
 
-			self::delete( $option['option_name'] );
+			self::delete($option['option_name']);
 		}
 	}
 
@@ -294,16 +294,16 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_options_like( $option_name, $fields = false ) {
+	public static function get_options_like($option_name, $fields = false) {
 		global $wpdb;
 
-		if ( empty( $option_name ) ) {
+		if (empty($option_name)) {
 			return array();
 		}
 
 		$field_names = $fields ? 'option_name, option_value' : 'option_name';
 
-		if ( $fields ) {
+		if ($fields) {
 			$options = $wpdb->get_results(
 				$wpdb->prepare(
 					"SELECT {$field_names }
@@ -327,10 +327,10 @@ 
 			);
 		}
 
-		if ( ! empty( $options ) && $fields ) {
-			foreach ( $options as $index => $option ) {
-				$option['option_value'] = maybe_unserialize( $option['option_value'] );
-				$options[ $index ]      = $option;
+		if ( ! empty($options) && $fields) {
+			foreach ($options as $index => $option) {
+				$option['option_value'] = maybe_unserialize($option['option_value']);
+				$options[$index]      = $option;
 			}
 		}
 
@@ -347,8 +347,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_valid_cache_key( $cache_key ) {
-		$is_valid = ( false !== strpos( $cache_key, 'give_cache_' ) );
+	public static function is_valid_cache_key($cache_key) {
+		$is_valid = (false !== strpos($cache_key, 'give_cache_'));
 
 
 		/**
@@ -356,7 +356,7 @@ 
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_is_valid_cache_key', $is_valid, $cache_key );
+		return apply_filters('give_is_valid_cache_key', $is_valid, $cache_key);
 	}
 
 
@@ -371,14 +371,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_group( $id, $group = '' ) {
+	public static function get_group($id, $group = '') {
 		$cached_data = null;
 
 		// Bailout.
-		if ( self::$instance->is_cache && ! empty( $id ) ) {
-			$group = self::$instance->filter_group_name( $group );
+		if (self::$instance->is_cache && ! empty($id)) {
+			$group = self::$instance->filter_group_name($group);
 
-			$cached_data = wp_cache_get( $id, $group );
+			$cached_data = wp_cache_get($id, $group);
 			$cached_data = false !== $cached_data ? $cached_data : null;
 		}
 
@@ -398,17 +398,17 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function set_group( $id, $data, $group = '', $expire = 0 ) {
+	public static function set_group($id, $data, $group = '', $expire = 0) {
 		$status = false;
 
 		// Bailout.
-		if ( ! self::$instance->is_cache || empty( $id ) ) {
+		if ( ! self::$instance->is_cache || empty($id)) {
 			return $status;
 		}
 
-		$group = self::$instance->filter_group_name( $group );
+		$group = self::$instance->filter_group_name($group);
 
-		$status = wp_cache_set( $id, $data, $group, $expire );
+		$status = wp_cache_set($id, $data, $group, $expire);
 
 		return $status;
 	}
@@ -424,15 +424,15 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function set_db_query( $id, $data ) {
+	public static function set_db_query($id, $data) {
 		$status = false;
 
 		// Bailout.
-		if ( ! self::$instance->is_cache || empty( $id ) ) {
+		if ( ! self::$instance->is_cache || empty($id)) {
 			return $status;
 		}
 
-		return self::set_group( $id, $data, 'give-db-queries', 0 );
+		return self::set_group($id, $data, 'give-db-queries', 0);
 	}
 
 	/**
@@ -445,8 +445,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_db_query( $id ) {
-		return self::get_group( $id, 'give-db-queries' );
+	public static function get_db_query($id) {
+		return self::get_group($id, 'give-db-queries');
 	}
 
 	/**
@@ -461,20 +461,20 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function delete_group( $ids, $group = '', $expire = 0 ) {
+	public static function delete_group($ids, $group = '', $expire = 0) {
 		$status = false;
 
 		// Bailout.
-		if ( ! self::$instance->is_cache || empty( $ids ) ) {
+		if ( ! self::$instance->is_cache || empty($ids)) {
 			return $status;
 		}
 
-		$group = self::$instance->filter_group_name( $group );
+		$group = self::$instance->filter_group_name($group);
 
 		// Delete single or multiple cache items from cache.
-		if ( ! is_array( $ids ) ) {
-			$status = wp_cache_delete( $ids, $group, $expire );
-			self::$instance->get_incrementer( true );
+		if ( ! is_array($ids)) {
+			$status = wp_cache_delete($ids, $group, $expire);
+			self::$instance->get_incrementer(true);
 
 			/**
 			 * Fire action when cache deleted for specific id.
@@ -485,12 +485,12 @@ 
 			 * @param string $group
 			 * @param int    $expire
 			 */
-			do_action( "give_deleted_{$group}_cache", $ids, $group, $expire, $status );
+			do_action("give_deleted_{$group}_cache", $ids, $group, $expire, $status);
 
 		} else {
-			foreach ( $ids as $id ) {
-				$status = wp_cache_delete( $id, $group, $expire );
-				self::$instance->get_incrementer( true );
+			foreach ($ids as $id) {
+				$status = wp_cache_delete($id, $group, $expire);
+				self::$instance->get_incrementer(true);
 
 				/**
 				 * Fire action when cache deleted for specific id .
@@ -501,7 +501,7 @@ 
 				 * @param string $group
 				 * @param int    $expire
 				 */
-				do_action( "give_deleted_{$group}_cache", $id, $group, $expire, $status );
+				do_action("give_deleted_{$group}_cache", $id, $group, $expire, $status);
 			}
 		}
 
@@ -518,30 +518,30 @@ 
 	 *
 	 * @param int $form_id
 	 */
-	public function delete_form_related_cache( $form_id ) {
+	public function delete_form_related_cache($form_id) {
 		// If this is just a revision, don't send the email.
-		if ( wp_is_post_revision( $form_id ) ) {
+		if (wp_is_post_revision($form_id)) {
 			return;
 		}
 
 		$donation_query = new Give_Payments_Query(
 			array(
-				'number'     => - 1,
+				'number'     => -1,
 				'give_forms' => $form_id,
 			)
 		);
 
 		$donations = $donation_query->get_payments();
 
-		if ( ! empty( $donations ) ) {
+		if ( ! empty($donations)) {
 			/* @var Give_Payment $donation */
-			foreach ( $donations as $donation ) {
-				wp_cache_delete( $donation->ID, 'give-donations' );
-				wp_cache_delete( $donation->donor_id, 'give-donors' );
+			foreach ($donations as $donation) {
+				wp_cache_delete($donation->ID, 'give-donations');
+				wp_cache_delete($donation->donor_id, 'give-donors');
 			}
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -553,22 +553,22 @@ 
 	 *
 	 * @param int $donation_id
 	 */
-	public function delete_payment_related_cache( $donation_id ) {
+	public function delete_payment_related_cache($donation_id) {
 		// If this is just a revision, don't send the email.
-		if ( wp_is_post_revision( $donation_id ) ) {
+		if (wp_is_post_revision($donation_id)) {
 			return;
 		}
 
 		/* @var Give_Payment $donation */
-		$donation = new Give_Payment( $donation_id );
+		$donation = new Give_Payment($donation_id);
 
-		if ( $donation && $donation->donor_id ) {
-			wp_cache_delete( $donation->donor_id, 'give-donors' );
+		if ($donation && $donation->donor_id) {
+			wp_cache_delete($donation->donor_id, 'give-donors');
 		}
 
-		wp_cache_delete( $donation->ID, 'give-donations' );
+		wp_cache_delete($donation->ID, 'give-donations');
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -582,17 +582,17 @@ 
 	 * @param string $group
 	 * @param int    $expire
 	 */
-	public function delete_donor_related_cache( $id, $group, $expire ) {
-		$donor        = new Give_Donor( $id );
-		$donation_ids = array_map( 'trim', (array) explode( ',', trim( $donor->payment_ids ) ) );
+	public function delete_donor_related_cache($id, $group, $expire) {
+		$donor        = new Give_Donor($id);
+		$donation_ids = array_map('trim', (array) explode(',', trim($donor->payment_ids)));
 
-		if ( ! empty( $donation_ids ) ) {
-			foreach ( $donation_ids as $donation ) {
-				wp_cache_delete( $donation, 'give-donations' );
+		if ( ! empty($donation_ids)) {
+			foreach ($donation_ids as $donation) {
+				wp_cache_delete($donation, 'give-donations');
 			}
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -606,15 +606,15 @@ 
 	 * @param string $group
 	 * @param int    $expire
 	 */
-	public function delete_donations_related_cache( $id, $group, $expire ) {
+	public function delete_donations_related_cache($id, $group, $expire) {
 		/* @var Give_Payment $donation */
-		$donation = new Give_Payment( $id );
+		$donation = new Give_Payment($id);
 
-		if ( $donation && $donation->donor_id ) {
-			wp_cache_delete( $donation->donor_id, 'give-donors' );
+		if ($donation && $donation->donor_id) {
+			wp_cache_delete($donation->donor_id, 'give-donors');
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 
@@ -632,12 +632,12 @@ 
 	 *
 	 * @return string
 	 */
-	private function get_incrementer( $refresh = false, $incrementer_key = 'give-cahce-incrementer-db-queries' ) {
-		$incrementer_value = wp_cache_get( $incrementer_key );
+	private function get_incrementer($refresh = false, $incrementer_key = 'give-cahce-incrementer-db-queries') {
+		$incrementer_value = wp_cache_get($incrementer_key);
 
-		if ( false === $incrementer_value || true === $refresh ) {
-			$incrementer_value = microtime( true );
-			wp_cache_set( $incrementer_key, $incrementer_value );
+		if (false === $incrementer_value || true === $refresh) {
+			$incrementer_value = microtime(true);
+			wp_cache_set($incrementer_key, $incrementer_value);
 		}
 
 		return $incrementer_value;
@@ -654,11 +654,11 @@ 
 	public function flush_cache() {
 		if (
 			Give_Admin_Settings::is_saving_settings() &&
-			isset( $_POST['cache'] ) &&
-			give_is_setting_enabled( give_clean( $_POST['cache'] ) )
+			isset($_POST['cache']) &&
+			give_is_setting_enabled(give_clean($_POST['cache']))
 		) {
-			$this->get_incrementer( true );
-			$this->get_incrementer( true, 'give-cahce-incrementer' );
+			$this->get_incrementer(true);
+			$this->get_incrementer(true, 'give-cahce-incrementer');
 		}
 	}
 
@@ -673,11 +673,11 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function filter_group_name( $group ) {
-		if ( ! empty( $group ) ) {
-			$incrementer = self::$instance->get_incrementer( false, 'give-cahce-incrementer' );
+	private function filter_group_name($group) {
+		if ( ! empty($group)) {
+			$incrementer = self::$instance->get_incrementer(false, 'give-cahce-incrementer');
 
-			if ( 'give-db-queries' === $group ) {
+			if ('give-db-queries' === $group) {
 				$incrementer = self::$instance->get_incrementer();
 			}
 

includes/class-give-db-donors.php

Doc Comments +2 added lines, -2 removed lines

@@ -207,7 +207,7 @@ 
 	 * NOTE: This should not be called directly as it does not make necessary changes to
 	 * the payment meta and logs. Use give_donor_delete() instead.
 	 *
-	 * @param  bool|string|int $_id_or_email ID or Email of Donor.
+	 * @param  integer $_id_or_email ID or Email of Donor.
 	 *
 	 * @since  1.0
 	 * @access public
@@ -407,7 +407,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $field ID or email. Default is 'id'.
-	 * @param  mixed  $value The Customer ID or email to search. Default is 0.
+	 * @param  integer  $value The Customer ID or email to search. Default is 0.
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */

Spacing +107 added lines, -107 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -42,7 +42,7 @@ 
 		$this->bc_200_params();
 
 		// Set hooks and register table only if instance loading first time.
-		if ( ! ( Give()->donors instanceof Give_DB_Donors ) ) {
+		if ( ! (Give()->donors instanceof Give_DB_Donors)) {
 			// Install table.
 			$this->register_table();
 		}
@@ -91,7 +91,7 @@ 
 			'purchase_value'  => 0.00,
 			'purchase_count'  => 0,
 			'notes'           => '',
-			'date_created'    => date( 'Y-m-d H:i:s' ),
+			'date_created'    => date('Y-m-d H:i:s'),
 			'token'           => '',
 			'verify_key'      => '',
 			'verify_throttle' => '',
@@ -108,40 +108,40 @@ 
 	 *
 	 * @return int|bool
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 
 		$defaults = array(
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
-		if ( empty( $args['email'] ) ) {
+		if (empty($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'])));
 		}
 
-		$donor = $this->get_donor_by( 'email', $args['email'] );
+		$donor = $this->get_donor_by('email', $args['email']);
 
 		// update an existing donor.
-		if ( $donor ) {
+		if ($donor) {
 
 			// Update the payment IDs attached to the donor
-			if ( ! empty( $args['payment_ids'] ) ) {
+			if ( ! empty($args['payment_ids'])) {
 
-				if ( empty( $donor->payment_ids ) ) {
+				if (empty($donor->payment_ids)) {
 
 					$donor->payment_ids = $args['payment_ids'];
 
 				} else {
 
-					$existing_ids       = array_map( 'absint', explode( ',', $donor->payment_ids ) );
-					$payment_ids        = array_map( 'absint', explode( ',', $args['payment_ids'] ) );
-					$payment_ids        = array_merge( $payment_ids, $existing_ids );
-					$donor->payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+					$existing_ids       = array_map('absint', explode(',', $donor->payment_ids));
+					$payment_ids        = array_map('absint', explode(',', $args['payment_ids']));
+					$payment_ids        = array_merge($payment_ids, $existing_ids);
+					$donor->payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 				}
 
@@ -149,13 +149,13 @@ 
 
 			}
 
-			$this->update( $donor->id, $args );
+			$this->update($donor->id, $args);
 
 			return $donor->id;
 
 		} else {
 
-			return $this->insert( $args, 'donor' );
+			return $this->insert($args, 'donor');
 
 		}
 
@@ -172,12 +172,12 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 
-		$status = parent::update( $row_id, $data, $where );
+		$status = parent::update($row_id, $data, $where);
 
-		if ( $status ) {
-			Give_Cache::delete_group( $row_id, 'give-donors' );
+		if ($status) {
+			Give_Cache::delete_group($row_id, 'give-donors');
 		}
 
 		return $status;
@@ -191,11 +191,11 @@ 
 	 *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
-		$donor_id = parent::insert( $data, $type );
+	public function insert($data, $type = '') {
+		$donor_id = parent::insert($data, $type);
 
-		if ( $donor_id ) {
-			Give_Cache::delete_group( $donor_id, 'give-donors' );
+		if ($donor_id) {
+			Give_Cache::delete_group($donor_id, 'give-donors');
 		}
 
 		return $donor_id;
@@ -214,16 +214,16 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete( $_id_or_email = false ) {
+	public function delete($_id_or_email = false) {
 
-		if ( empty( $_id_or_email ) ) {
+		if (empty($_id_or_email)) {
 			return false;
 		}
 
-		$column = is_email( $_id_or_email ) ? 'email' : 'id';
-		$donor  = $this->get_donor_by( $column, $_id_or_email );
+		$column = is_email($_id_or_email) ? 'email' : 'id';
+		$donor  = $this->get_donor_by($column, $_id_or_email);
 
-		if ( $donor->id > 0 ) {
+		if ($donor->id > 0) {
 
 			global $wpdb;
 
@@ -232,11 +232,11 @@ 
 			 *
 			 * @since 1.8.14
 			 */
-			Give()->donor_meta->delete_all_meta( $donor->id );
+			Give()->donor_meta->delete_all_meta($donor->id);
 
 			// Cache already deleted in delete_all_meta fn.
 
-			return $wpdb->delete( $this->table_name, array( 'id' => $donor->id ), array( '%d' ) );
+			return $wpdb->delete($this->table_name, array('id' => $donor->id), array('%d'));
 
 		} else {
 			return false;
@@ -257,10 +257,10 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete_by_user_id( $user_id = false ) {
+	public function delete_by_user_id($user_id = false) {
 		global $wpdb;
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return false;
 		}
 
@@ -269,14 +269,14 @@ 
 		 *
 		 * @since 1.8.14
 		 */
-		$donor = new Give_Donor( $user_id, true );
-		if ( ! empty( $donor->id ) ) {
-			Give()->donor_meta->delete_all_meta( $donor->id );
+		$donor = new Give_Donor($user_id, true);
+		if ( ! empty($donor->id)) {
+			Give()->donor_meta->delete_all_meta($donor->id);
 		}
 
 		// Cache is already deleted in delete_all_meta fn.
 
-		return $wpdb->delete( $this->table_name, array( 'user_id' => $user_id ), array( '%d' ) );
+		return $wpdb->delete($this->table_name, array('user_id' => $user_id), array('%d'));
 	}
 
 	/**
@@ -290,14 +290,14 @@ 
 	 *
 	 * @return bool          True is exists, false otherwise.
 	 */
-	public function exists( $value = '', $field = 'email' ) {
+	public function exists($value = '', $field = 'email') {
 
 		$columns = $this->get_columns();
-		if ( ! array_key_exists( $field, $columns ) ) {
+		if ( ! array_key_exists($field, $columns)) {
 			return false;
 		}
 
-		return (bool) $this->get_column_by( 'id', $field, $value );
+		return (bool) $this->get_column_by('id', $field, $value);
 
 	}
 
@@ -312,16 +312,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function attach_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function attach_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		// Attach the payment, but don't increment stats, as this function previously did not
-		return $donor->attach_payment( $payment_id, false );
+		return $donor->attach_payment($payment_id, false);
 
 	}
 
@@ -336,16 +336,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function remove_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		// Remove the payment, but don't decrease stats, as this function previously did not
-		return $donor->remove_payment( $payment_id, false );
+		return $donor->remove_payment($payment_id, false);
 
 	}
 
@@ -359,18 +359,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function increment_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function increment_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		$increased_count = $donor->increase_purchase_count();
-		$increased_value = $donor->increase_value( $amount );
+		$increased_value = $donor->increase_value($amount);
 
-		return ( $increased_count && $increased_value ) ? true : false;
+		return ($increased_count && $increased_value) ? true : false;
 
 	}
 
@@ -385,18 +385,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function decrement_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function decrement_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		$decreased_count = $donor->decrease_donation_count();
-		$decreased_value = $donor->decrease_value( $amount );
+		$decreased_value = $donor->decrease_value($amount);
 
-		return ( $decreased_count && $decreased_value ) ? true : false;
+		return ($decreased_count && $decreased_value) ? true : false;
 
 	}
 
@@ -411,44 +411,44 @@ 
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */
-	public function get_donor_by( $field = 'id', $value = 0 ) {
-		$value = sanitize_text_field( $value );
+	public function get_donor_by($field = 'id', $value = 0) {
+		$value = sanitize_text_field($value);
 
 		// Bailout.
-		if ( empty( $field ) || empty( $value ) ) {
+		if (empty($field) || empty($value)) {
 			return null;
 		}
 
 		// Verify values.
-		if ( 'id' === $field || 'user_id' === $field ) {
+		if ('id' === $field || 'user_id' === $field) {
 			// Make sure the value is numeric to avoid casting objects, for example,
 			// to int 1.
-			if ( ! is_numeric( $value ) ) {
+			if ( ! is_numeric($value)) {
 				return false;
 			}
 
-			$value = absint( $value );
+			$value = absint($value);
 
-			if ( $value < 1 ) {
+			if ($value < 1) {
 				return false;
 			}
 
-		} elseif ( 'email' === $field ) {
+		} elseif ('email' === $field) {
 
-			if ( ! is_email( $value ) ) {
+			if ( ! is_email($value)) {
 				return false;
 			}
 
-			$value = trim( $value );
+			$value = trim($value);
 		}
 
 		// Bailout
-		if ( ! $value ) {
+		if ( ! $value) {
 			return false;
 		}
 
 		// Set query params.
-		switch ( $field ) {
+		switch ($field) {
 			case 'id':
 				$args['donor'] = $value;
 				break;
@@ -463,9 +463,9 @@ 
 		}
 
 		// Get donors.
-		$donor = new Give_Donors_Query( $args );
+		$donor = new Give_Donors_Query($args);
 
-		if ( ! $donor = $donor->get_donors() ) {
+		if ( ! $donor = $donor->get_donors()) {
 			// Look for donor from an additional email.
 			$args = array(
 				'meta_query' => array(
@@ -476,15 +476,15 @@ 
 				),
 			);
 
-			$donor = new Give_Donors_Query( $args );
+			$donor = new Give_Donors_Query($args);
 			$donor = $donor->get_donors();
 
-			if ( empty( $donor ) ) {
+			if (empty($donor)) {
 				return false;
 			}
 		}
 
-		return current( $donor );
+		return current($donor);
 	}
 
 	/**
@@ -497,10 +497,10 @@ 
 	 *
 	 * @return array|object|null Donors array or object. Null if not found.
 	 */
-	public function get_donors( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function get_donors($args = array()) {
+		$this->bc_1814_params($args);
 
-		$donors = new Give_Donors_Query( $args );
+		$donors = new Give_Donors_Query($args);
 
 		return $donors->get_donors();
 
@@ -517,21 +517,21 @@ 
 	 *
 	 * @return int         Total number of donors.
 	 */
-	public function count( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function count($args = array()) {
+		$this->bc_1814_params($args);
 		$args['count'] = true;
 
-		$cache_key = md5( 'give_donors_count' . serialize( $args ) );
-		$count     = Give_Cache::get_group( $cache_key, 'donors' );
+		$cache_key = md5('give_donors_count'.serialize($args));
+		$count     = Give_Cache::get_group($cache_key, 'donors');
 
-		if ( is_null( $count ) ) {
-			$donors = new Give_Donors_Query( $args );
+		if (is_null($count)) {
+			$donors = new Give_Donors_Query($args);
 			$count  = $donors->get_donors();
 
-			Give_Cache::set_group( $cache_key, $count, 'donors', 3600 );
+			Give_Cache::set_group($cache_key, $count, 'donors', 3600);
 		}
 
-		return absint( $count );
+		return absint($count);
 
 	}
 
@@ -545,9 +545,9 @@ 
 	 */
 	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 . " (
+		$sql = "CREATE TABLE ".$this->table_name." (
 		id bigint(20) NOT NULL AUTO_INCREMENT,
 		user_id bigint(20) NOT NULL,
 		email varchar(50) NOT NULL,
@@ -565,9 +565,9 @@ 
 		KEY user (user_id)
 		) 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);
 	}
 
 	/**
@@ -582,8 +582,8 @@ 
 		global $wpdb;
 
 		if (
-			! give_has_upgrade_completed( 'v20_rename_donor_tables' ) &&
-			$wpdb->query( $wpdb->prepare( "SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customers" ) )
+			! give_has_upgrade_completed('v20_rename_donor_tables') &&
+			$wpdb->query($wpdb->prepare("SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customers"))
 		) {
 			$wpdb->donors = $this->table_name = "{$wpdb->prefix}give_customers";
 		}
@@ -597,41 +597,41 @@ 
 	 *
 	 * @param $args
 	 */
-	private function bc_1814_params( &$args ) {
+	private function bc_1814_params(&$args) {
 		// Backward compatibility: user_id
-		if ( ! empty( $args['user_id'] ) ) {
+		if ( ! empty($args['user_id'])) {
 			$args['user'] = $args['user_id'];
 		}
 
 		// Backward compatibility: id
-		if ( ! empty( $args['id'] ) ) {
+		if ( ! empty($args['id'])) {
 			$args['donor'] = $args['id'];
 		}
 
 		// Backward compatibility: name
-		if ( ! empty( $args['name'] ) ) {
+		if ( ! empty($args['name'])) {
 			$args['s'] = "name:{$args['name']}";
 		}
 
 		// Backward compatibility: date
 		// Donors created for a specific date or in a date range.
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				if ( ! empty( $args['date']['start'] ) ) {
-					$args['date_query']['after'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
+				if ( ! empty($args['date']['start'])) {
+					$args['date_query']['after'] = date('Y-m-d H:i:s', strtotime($args['date']['start']));
 				}
 
-				if ( ! empty( $args['date']['end'] ) ) {
-					$args['date_query']['before'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+				if ( ! empty($args['date']['end'])) {
+					$args['date_query']['before'] = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 				}
 
 			} else {
 
-				$args['date_query']['year']  = date( 'Y', strtotime( $args['date'] ) );
-				$args['date_query']['month'] = date( 'm', strtotime( $args['date'] ) );
-				$args['date_query']['day']   = date( 'd', strtotime( $args['date'] ) );
+				$args['date_query']['year']  = date('Y', strtotime($args['date']));
+				$args['date_query']['month'] = date('m', strtotime($args['date']));
+				$args['date_query']['day']   = date('d', strtotime($args['date']));
 			}
 		}
 	}

includes/class-give-logging.php

Doc Comments +1 added lines, -1 removed lines

@@ -512,7 +512,7 @@
 	 * @since  1.7
 	 * @access public
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function delete_cache() {
 		// Add log related keys to delete.

Spacing +137 added lines, -137 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 		 * Setup properties
 		 */
 
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-logs.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-logs-meta.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-db-logs.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-db-logs-meta.php';
 		$this->log_db     = new Give_DB_Logs();
 		$this->logmeta_db = new Give_DB_Log_Meta();
 
@@ -62,22 +62,22 @@ 
 		 * Setup hooks.
 		 */
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache' ) );
-		add_action( 'update_log_metadata', array( $this, 'bc_200_set_payment_as_log_parent' ), 10, 4 );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
+		add_action('update_log_metadata', array($this, 'bc_200_set_payment_as_log_parent'), 10, 4);
 
 		// Backward compatibility.
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Create the log post type
-			add_action( 'init', array( $this, 'register_post_type' ), -2 );
+			add_action('init', array($this, 'register_post_type'), -2);
 		}
 
 		// Create types taxonomy and default types
 		// @todo: remove this taxonomy, some addon use this taxonomy with there custom log post type for example: recurring
 		// Do not use this taxonomy with your log type because we will remove it in future releases.
-		add_action( 'init', array( $this, 'register_taxonomy' ), -2 );
+		add_action('init', array($this, 'register_taxonomy'), -2);
 	}
 
 
@@ -95,7 +95,7 @@ 
 		/* Logs post type */
 		$log_args = array(
 			'labels'              => array(
-				'name' => esc_html__( 'Logs', 'give' ),
+				'name' => esc_html__('Logs', 'give'),
 			),
 			'public'              => false,
 			'exclude_from_search' => true,
@@ -104,11 +104,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true,
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -122,9 +122,9 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array(
+		register_taxonomy('give_log_type', 'give_log', array(
 			'public' => false,
-		) );
+		));
 	}
 
 	/**
@@ -144,7 +144,7 @@ 
 			'api_request',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -159,8 +159,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -179,7 +179,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = '' ) {
+	public function add($title = '', $message = '', $parent = 0, $type = '') {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -187,7 +187,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -204,12 +204,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = '', $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = '', $paged = null) {
+		return $this->get_connected_logs(array(
 			'log_parent' => $object_id,
 			'paged'      => $paged,
 			'log_type'   => $type,
-		) );
+		));
 	}
 
 	/**
@@ -223,7 +223,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$log_id = 0;
 
 		$defaults = array(
@@ -236,28 +236,28 @@ 
 			'post_status' => 'publish',
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
-		$this->bc_200_validate_params( $args, $log_meta );
+		$args = wp_parse_args($log_data, $defaults);
+		$this->bc_200_validate_params($args, $log_meta);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			global $wpdb;
 
 			// Backward Compatibility.
-			if ( ! $wpdb->get_var( "SELECT ID from {$this->log_db->table_name} ORDER BY id DESC LIMIT 1" ) ) {
-				$latest_log_id = $wpdb->get_var( "SELECT ID from $wpdb->posts ORDER BY id DESC LIMIT 1" );
-				$latest_log_id = empty( $latest_log_id ) ? 1 : ++ $latest_log_id;
+			if ( ! $wpdb->get_var("SELECT ID from {$this->log_db->table_name} ORDER BY id DESC LIMIT 1")) {
+				$latest_log_id = $wpdb->get_var("SELECT ID from $wpdb->posts ORDER BY id DESC LIMIT 1");
+				$latest_log_id = empty($latest_log_id) ? 1 : ++ $latest_log_id;
 
 				$args['ID'] = $latest_log_id;
-				$this->log_db->insert( $args );
+				$this->log_db->insert($args);
 			}
 		}
 
-		$log_id = $this->log_db->add( $args );
+		$log_id = $this->log_db->add($args);
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				$this->logmeta_db->update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				$this->logmeta_db->update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -279,7 +279,7 @@ 
 	 *
 	 * @return bool|null       True if successful, false otherwise.
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 		$log_id = 0;
 
 		/**
@@ -290,7 +290,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'log_parent'  => 0,
@@ -300,28 +300,28 @@ 
 			'post_status' => 'publish',
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
-		$this->bc_200_validate_params( $args, $log_meta );
+		$args = wp_parse_args($log_data, $defaults);
+		$this->bc_200_validate_params($args, $log_meta);
 
 		// Store the log entry
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$log_id = wp_update_post( $args );
+			$log_id = wp_update_post($args);
 
-			if ( $log_id && ! empty( $log_meta ) ) {
-				foreach ( (array) $log_meta as $key => $meta ) {
-					if ( ! empty( $meta ) ) {
-						give_update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+			if ($log_id && ! empty($log_meta)) {
+				foreach ((array) $log_meta as $key => $meta) {
+					if ( ! empty($meta)) {
+						give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 					}
 				}
 			}
 		} else {
-			$log_id = $this->log_db->add( $args );
+			$log_id = $this->log_db->add($args);
 
-			if ( $log_id && ! empty( $log_meta ) ) {
-				foreach ( (array) $log_meta as $key => $meta ) {
-					if ( ! empty( $meta ) ) {
-						$this->logmeta_db->update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+			if ($log_id && ! empty($log_meta)) {
+				foreach ((array) $log_meta as $key => $meta) {
+					if ( ! empty($meta)) {
+						$this->logmeta_db->update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 					}
 				}
 			}
@@ -336,7 +336,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -353,30 +353,30 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 		$logs = array();
 
-		$defaults   = array(
+		$defaults = array(
 			'number'      => 20,
-			'paged'       => get_query_var( 'paged' ),
+			'paged'       => get_query_var('paged'),
 			'log_type'    => false,
 
 			// Backward compatibility.
 			'post_type'   => 'give_log',
 			'post_status' => 'publish',
 		);
-		$query_args = wp_parse_args( $args, $defaults );
-		$this->bc_200_validate_params( $query_args );
+		$query_args = wp_parse_args($args, $defaults);
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs = get_posts( $query_args );
-			$this->bc_200_add_new_properties( $logs );
+			$logs = get_posts($query_args);
+			$this->bc_200_add_new_properties($logs);
 		} else {
-			$logs = $this->log_db->get_logs( $query_args );
+			$logs = $this->log_db->get_logs($query_args);
 		}
 
-		return ( ! empty( $logs ) ? $logs : false );
+		return ( ! empty($logs) ? $logs : false);
 	}
 
 	/**
@@ -394,41 +394,41 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = '', $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = '', $meta_query = null, $date_query = null) {
 		$logs_count = 0;
 
 		$query_args = array(
-			'number'      => - 1,
+			'number'      => -1,
 
 			// Backward comatibility.
 			'post_type'   => 'give_log',
 			'post_status' => 'publish',
 		);
 
-		if ( $object_id ) {
+		if ($object_id) {
 			$query_args['log_parent'] = $object_id;
 		}
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['log_type'] = $type;
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
-		$this->bc_200_validate_params( $query_args );
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs       = new WP_Query( $query_args );
+			$logs       = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 		} else {
-			$logs_count = $this->log_db->count( $query_args );
+			$logs_count = $this->log_db->count($query_args);
 		}
 
 		return $logs_count;
@@ -448,10 +448,10 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = '', $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = '', $meta_query = null) {
 		$query_args = array(
 			'log_parent'  => $object_id,
-			'number'      => - 1,
+			'number'      => -1,
 			'fields'      => 'ID',
 
 			// Backward compatibility.
@@ -459,32 +459,32 @@ 
 			'post_status' => 'publish',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['log_type'] = $type;
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$this->bc_200_validate_params( $query_args );
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs = get_posts( $query_args );
+			$logs = get_posts($query_args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					wp_delete_post( $log, true );
+			if ($logs) {
+				foreach ($logs as $log) {
+					wp_delete_post($log, true);
 				}
 			}
 		} else {
-			$logs = $this->log_db->get_logs( $query_args );
+			$logs = $this->log_db->get_logs($query_args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					if ( $this->log_db->delete( $log->ID ) ) {
-						$this->logmeta_db->delete_row( $log->ID );
+			if ($logs) {
+				foreach ($logs as $log) {
+					if ($this->log_db->delete($log->ID)) {
+						$this->logmeta_db->delete_row($log->ID);
 					}
 				}
 			}
@@ -501,9 +501,9 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
+	public function background_process_delete_cache($post_id) {
 		// Delete log cache immediately
-		wp_schedule_single_event( time() - 5, 'give_delete_log_cache' );
+		wp_schedule_single_event(time() - 5, 'give_delete_log_cache');
 	}
 
 	/**
@@ -516,17 +516,17 @@ 
 	 */
 	public function delete_cache() {
 		// Add log related keys to delete.
-		$cache_give_logs      = Give_Cache::get_options_like( 'give_logs' );
-		$cache_give_log_count = Give_Cache::get_options_like( 'log_count' );
+		$cache_give_logs      = Give_Cache::get_options_like('give_logs');
+		$cache_give_log_count = Give_Cache::get_options_like('log_count');
 
-		$cache_option_names = array_merge( $cache_give_logs, $cache_give_log_count );
+		$cache_option_names = array_merge($cache_give_logs, $cache_give_log_count);
 
 		// Bailout.
-		if ( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
-		Give_Cache::delete( $cache_option_names );
+		Give_Cache::delete($cache_option_names);
 	}
 
 	/**
@@ -538,7 +538,7 @@ 
 	 * @param array $log_query
 	 * @param array $log_meta
 	 */
-	private function bc_200_validate_params( &$log_query, &$log_meta = array() ) {
+	private function bc_200_validate_params(&$log_query, &$log_meta = array()) {
 		$query_params = array(
 			'log_title'    => 'post_title',
 			'log_parent'   => 'post_parent',
@@ -550,41 +550,41 @@ 
 			'meta_query'   => 'meta_query',
 		);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Set old params.
-			foreach ( $query_params as $new_query_param => $old_query_param ) {
+			foreach ($query_params as $new_query_param => $old_query_param) {
 
-				if ( isset( $log_query[ $old_query_param ] ) && empty( $log_query[ $new_query_param ] ) ) {
-					$log_query[ $new_query_param ] = $log_query[ $old_query_param ];
+				if (isset($log_query[$old_query_param]) && empty($log_query[$new_query_param])) {
+					$log_query[$new_query_param] = $log_query[$old_query_param];
 					continue;
-				} elseif ( ! isset( $log_query[ $new_query_param ] ) ) {
+				} elseif ( ! isset($log_query[$new_query_param])) {
 					continue;
-				} elseif( empty( $log_query[ $new_query_param ] ) ) {
+				} elseif (empty($log_query[$new_query_param])) {
 					continue;
 				}
 
-				switch ( $new_query_param ) {
+				switch ($new_query_param) {
 					case 'log_type':
 						$log_query['tax_query'] = array(
 							array(
 								'taxonomy' => 'give_log_type',
 								'field'    => 'slug',
-								'terms'    => $log_query[ $new_query_param ],
+								'terms'    => $log_query[$new_query_param],
 							),
 						);
 						break;
 
 					case 'meta_query':
-						if( ! empty( $log_query['meta_query'] ) && empty( $log_query['post_parent'] ) ) {
-							foreach ( $log_query['meta_query'] as $index => $meta_query ){
-								if( ! is_array( $meta_query ) || empty( $meta_query['key'] ) ) {
+						if ( ! empty($log_query['meta_query']) && empty($log_query['post_parent'])) {
+							foreach ($log_query['meta_query'] as $index => $meta_query) {
+								if ( ! is_array($meta_query) || empty($meta_query['key'])) {
 									continue;
 								}
 
-								switch ( $meta_query['key'] ) {
+								switch ($meta_query['key']) {
 									case '_give_log_form_id':
 										$log_query['post_parent'] = $meta_query['value'];
-										unset( $log_query['meta_query'][$index] );
+										unset($log_query['meta_query'][$index]);
 										break;
 								}
 							}
@@ -592,40 +592,40 @@ 
 						break;
 
 					default:
-						switch( $new_query_param ){
+						switch ($new_query_param) {
 							case 'log_parent':
 								$log_query['meta_query'][] = array(
 									'key' => '_give_log_payment_id',
-									'value' => $log_query[ $new_query_param ]
+									'value' => $log_query[$new_query_param]
 								);
 
 								break;
 
 							default:
-								$log_query[ $old_query_param ] = $log_query[ $new_query_param ];
+								$log_query[$old_query_param] = $log_query[$new_query_param];
 						}
 				}
 			}
 		} else {
 			// Set only old params.
-			$query_params = array_flip( $query_params );
-			foreach ( $query_params as $old_query_param => $new_query_param ) {
-				if ( isset( $log_query[ $new_query_param ] ) && empty( $log_query[ $old_query_param ] ) ) {
-					$log_query[ $old_query_param ] = $log_query[ $new_query_param ];
+			$query_params = array_flip($query_params);
+			foreach ($query_params as $old_query_param => $new_query_param) {
+				if (isset($log_query[$new_query_param]) && empty($log_query[$old_query_param])) {
+					$log_query[$old_query_param] = $log_query[$new_query_param];
 					continue;
-				} elseif ( ! isset( $log_query[ $old_query_param ] ) ) {
+				} elseif ( ! isset($log_query[$old_query_param])) {
 					continue;
 				}
 
-				switch ( $old_query_param ) {
+				switch ($old_query_param) {
 					case 'tax_query':
-						if ( isset( $log_query[ $old_query_param ][0]['terms'] ) ) {
-							$log_query[ $new_query_param ] = $log_query[ $old_query_param ][0]['terms'];
+						if (isset($log_query[$old_query_param][0]['terms'])) {
+							$log_query[$new_query_param] = $log_query[$old_query_param][0]['terms'];
 						}
 						break;
 
 					default:
-						$log_query[ $new_query_param ] = $log_query[ $old_query_param ];
+						$log_query[$new_query_param] = $log_query[$old_query_param];
 				}
 			}
 		}
@@ -639,8 +639,8 @@ 
 	 *
 	 * @param  array $logs
 	 */
-	private function bc_200_add_new_properties( &$logs ) {
-		if ( empty( $logs ) ) {
+	private function bc_200_add_new_properties(&$logs) {
+		if (empty($logs)) {
 			return;
 		}
 
@@ -653,30 +653,30 @@ 
 			'log_type'     => 'give_log_type',
 		);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
-			foreach ( $logs as $index => $log ) {
-				foreach ( $query_params as $new_query_param => $old_query_param ) {
-					if ( ! property_exists( $log, $old_query_param ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
+			foreach ($logs as $index => $log) {
+				foreach ($query_params as $new_query_param => $old_query_param) {
+					if ( ! property_exists($log, $old_query_param)) {
 						/**
 						 *  Set unmatched properties.
 						 */
 
 						// 1. log_type
-						$term = get_the_terms( $log->ID, 'give_log_type' );
-						$term = ! is_wp_error( $term ) && ! empty( $term ) ? $term[0] : array();
+						$term = get_the_terms($log->ID, 'give_log_type');
+						$term = ! is_wp_error($term) && ! empty($term) ? $term[0] : array();
 
-						$logs[ $index ]->{$new_query_param} = ! empty( $term ) ? $term->slug : '';
+						$logs[$index]->{$new_query_param} = ! empty($term) ? $term->slug : '';
 
 						continue;
 					}
 
-					switch ( $old_query_param ) {
+					switch ($old_query_param) {
 						case 'post_parent':
-							$logs[ $index ]->{$new_query_param} = give_get_meta( $log->ID, '_give_log_payment_id', true );
+							$logs[$index]->{$new_query_param} = give_get_meta($log->ID, '_give_log_payment_id', true);
 							break;
 
 						default:
-							$logs[ $index ]->{$new_query_param} = $log->{$old_query_param};
+							$logs[$index]->{$new_query_param} = $log->{$old_query_param};
 					}
 				}
 			}
@@ -696,10 +696,10 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function bc_200_set_payment_as_log_parent( $check, $log_id, $meta_key, $meta_value ) {
+	public function bc_200_set_payment_as_log_parent($check, $log_id, $meta_key, $meta_value) {
 		global $wpdb;
 		$update_status = false;
-		$post_type     = get_post_type( $log_id );
+		$post_type     = get_post_type($log_id);
 
 		// Bailout.
 		if (
@@ -719,9 +719,9 @@ 
 			)
 		);
 
-		if ( $form_id ) {
-			$this->logmeta_db->delete_meta( $log_id, '_give_log_payment_id' );
-			$this->logmeta_db->update_meta( $log_id, '_give_log_form_id', $form_id );
+		if ($form_id) {
+			$this->logmeta_db->delete_meta($log_id, '_give_log_payment_id');
+			$this->logmeta_db->update_meta($log_id, '_give_log_form_id', $form_id);
 
 			$update_status = $wpdb->update(
 				$this->log_db->table_name,

includes/install.php

Doc Comments +1 added lines, -1 removed lines

@@ -397,7 +397,7 @@
  *
  * @since 1.8.11
  *
- * @return void
+ * @return false|null
  */
 function give_create_pages() {
 

Spacing +60 added lines, -60 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,15 +28,15 @@ 
  * @global     $wpdb
  * @return void
  */
-function give_install( $network_wide = false ) {
+function give_install($network_wide = false) {
 
 	global $wpdb;
 
-	if ( is_multisite() && $network_wide ) {
+	if (is_multisite() && $network_wide) {
 
-		foreach ( $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs LIMIT 100" ) as $blog_id ) {
+		foreach ($wpdb->get_col("SELECT blog_id FROM $wpdb->blogs LIMIT 100") as $blog_id) {
 
-			switch_to_blog( $blog_id );
+			switch_to_blog($blog_id);
 			give_run_install();
 			restore_current_blog();
 
@@ -63,31 +63,31 @@ 
 	give_setup_post_types();
 
 	// Add Upgraded From Option.
-	$current_version = get_option( 'give_version' );
-	if ( $current_version ) {
-		update_option( 'give_version_upgraded_from', $current_version );
+	$current_version = get_option('give_version');
+	if ($current_version) {
+		update_option('give_version_upgraded_from', $current_version);
 	}
 
 	// Setup some default options.
 	$options = array();
 
 	//Fresh Install? Setup Test Mode, Base Country (US), Test Gateway, Currency.
-	if ( empty( $current_version ) ) {
-		$options = array_merge( $options, give_get_default_settings() );
+	if (empty($current_version)) {
+		$options = array_merge($options, give_get_default_settings());
 	}
 
 	// Populate the default values.
-	update_option( 'give_settings', array_merge( $give_options, $options ) );
+	update_option('give_settings', array_merge($give_options, $options));
 
 	/**
 	 * Run plugin upgrades.
 	 *
 	 * @since 1.8
 	 */
-	do_action( 'give_upgrades' );
+	do_action('give_upgrades');
 
-	if ( GIVE_VERSION !== get_option( 'give_version' ) ) {
-		update_option( 'give_version', GIVE_VERSION );
+	if (GIVE_VERSION !== get_option('give_version')) {
+		update_option('give_version', GIVE_VERSION);
 	}
 
 	// Create Give roles.
@@ -98,7 +98,7 @@ 
 	// Set api version, end point and refresh permalink.
 	$api = new Give_API();
 	$api->add_endpoint();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	flush_rewrite_rules();
 
@@ -113,11 +113,11 @@ 
 	$give_sessions->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created.
-	Give_Cache::set( '_give_installed', $options, 30, true );
+	Give_Cache::set('_give_installed', $options, 30, true);
 
-	if ( ! $current_version ) {
+	if ( ! $current_version) {
 
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		// When new upgrade routines are added, mark them as complete on fresh install.
 		$upgrade_routines = array(
@@ -144,18 +144,18 @@ 
 			'v20_upgrades_payment_metadata'
 		);
 
-		foreach ( $upgrade_routines as $upgrade ) {
-			give_set_upgrade_complete( $upgrade );
+		foreach ($upgrade_routines as $upgrade) {
+			give_set_upgrade_complete($upgrade);
 		}
 	}
 
 	// Bail if activating from network, or bulk.
-	if ( is_network_admin() || isset( $_GET['activate-multi'] ) ) {
+	if (is_network_admin() || isset($_GET['activate-multi'])) {
 		return;
 	}
 
 	// Add the transient to redirect.
-	Give_Cache::set( '_give_activation_redirect', true, 30, true );
+	Give_Cache::set('_give_activation_redirect', true, 30, true);
 }
 
 /**
@@ -173,11 +173,11 @@ 
  * @param  int    $site_id The Site ID.
  * @param  array  $meta    Blog Meta.
  */
-function give_on_create_blog( $blog_id, $user_id, $domain, $path, $site_id, $meta ) {
+function give_on_create_blog($blog_id, $user_id, $domain, $path, $site_id, $meta) {
 
-	if ( is_plugin_active_for_network( GIVE_PLUGIN_BASENAME ) ) {
+	if (is_plugin_active_for_network(GIVE_PLUGIN_BASENAME)) {
 
-		switch_to_blog( $blog_id );
+		switch_to_blog($blog_id);
 		give_install();
 		restore_current_blog();
 
@@ -185,7 +185,7 @@ 
 
 }
 
-add_action( 'wpmu_new_blog', 'give_on_create_blog', 10, 6 );
+add_action('wpmu_new_blog', 'give_on_create_blog', 10, 6);
 
 
 /**
@@ -198,13 +198,13 @@ 
  *
  * @return array          The tables to drop.
  */
-function give_wpmu_drop_tables( $tables, $blog_id ) {
+function give_wpmu_drop_tables($tables, $blog_id) {
 
-	switch_to_blog( $blog_id );
+	switch_to_blog($blog_id);
 	$donors_db     = new Give_DB_Donors();
 	$donor_meta_db = new Give_DB_Donor_Meta();
 
-	if ( $donors_db->installed() ) {
+	if ($donors_db->installed()) {
 		$tables[] = $donors_db->table_name;
 		$tables[] = $donor_meta_db->table_name;
 	}
@@ -214,7 +214,7 @@ 
 
 }
 
-add_filter( 'wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2 );
+add_filter('wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2);
 
 /**
  * Post-installation
@@ -226,16 +226,16 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options     = Give_Cache::get( '_give_installed', true );
-	$give_table_check = get_option( '_give_table_check', false );
+	$give_options     = Give_Cache::get('_give_installed', true);
+	$give_table_check = get_option('_give_table_check', false);
 
-	if ( false === $give_table_check || current_time( 'timestamp' ) > $give_table_check ) {
+	if (false === $give_table_check || current_time('timestamp') > $give_table_check) {
 
-		if ( ! @Give()->donor_meta->installed() ) {
+		if ( ! @Give()->donor_meta->installed()) {
 
 			// Create the donor meta database.
 			// (this ensures it creates it on multisite instances where it is network activated).
@@ -243,7 +243,7 @@ 
 
 		}
 
-		if ( ! @Give()->donors->installed() ) {
+		if ( ! @Give()->donors->installed()) {
 			// Create the donor database.
 			// (this ensures it creates it on multisite instances where it is network activated).
 			@Give()->donors->create_table();
@@ -255,22 +255,22 @@ 
 			 *
 			 * @param array $give_options Give plugin options.
 			 */
-			do_action( 'give_after_install', $give_options );
+			do_action('give_after_install', $give_options);
 		}
 
-		update_option( '_give_table_check', ( current_time( 'timestamp' ) + WEEK_IN_SECONDS ) );
+		update_option('_give_table_check', (current_time('timestamp') + WEEK_IN_SECONDS));
 
 	}
 
 	// Delete the transient
-	if ( false !== $give_options ) {
-		Give_Cache::delete( Give_Cache::get_key( '_give_installed' ) );
+	if (false !== $give_options) {
+		Give_Cache::delete(Give_Cache::get_key('_give_installed'));
 	}
 
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -285,11 +285,11 @@ 
 
 	global $wp_roles;
 
-	if ( ! is_object( $wp_roles ) ) {
+	if ( ! is_object($wp_roles)) {
 		return;
 	}
 
-	if ( ! array_key_exists( 'give_manager', $wp_roles->roles ) ) {
+	if ( ! array_key_exists('give_manager', $wp_roles->roles)) {
 
 		// Create Give plugin roles
 		$roles = new Give_Roles();
@@ -300,7 +300,7 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
+add_action('admin_init', 'give_install_roles_on_network');
 
 /**
  * Default core setting values.
@@ -339,14 +339,14 @@ 
 		'uninstall_on_delete'                         => 'disabled',
 		'the_content_filter'                          => 'enabled',
 		'scripts_footer'                              => 'disabled',
-		'agree_to_terms_label'                        => __( 'Agree to Terms?', 'give' ),
+		'agree_to_terms_label'                        => __('Agree to Terms?', 'give'),
 		'agreement_text'                              => give_get_default_agreement_text(),
 
 		// Paypal IPN verification.
 		'paypal_verification'                         => 'enabled',
 
 		// Default is manual gateway.
-		'gateways'                                    => array( 'manual' => 1, 'offline' => 1 ),
+		'gateways'                                    => array('manual' => 1, 'offline' => 1),
 		'default_gateway'                             => 'manual',
 
 		// Offline gateway setup.
@@ -374,7 +374,7 @@ 
  */
 function give_get_default_agreement_text() {
 
-	$org_name = get_bloginfo( 'name' );
+	$org_name = get_bloginfo('name');
 
 	$agreement = sprintf(
 		'<p>Acceptance of any contribution, gift or grant is at the discretion of the %1$s. The  %1$s will not accept any gift unless it can be used or expended consistently with the purpose and mission of the  %1$s.</p>
@@ -388,7 +388,7 @@ 
 		$org_name
 	);
 
-	return apply_filters( 'give_get_default_agreement_text', $agreement, $org_name );
+	return apply_filters('give_get_default_agreement_text', $agreement, $org_name);
 }
 
 
@@ -402,19 +402,19 @@ 
 function give_create_pages() {
 
 	// Bailout if pages already created.
-	if ( get_option( 'give_install_pages_created' ) ) {
+	if (get_option('give_install_pages_created')) {
 		return false;
 	}
 
 	$options = array();
 
 	// Checks if the Success Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'success_page' ) ) ) {
+	if ( ! get_post(give_get_option('success_page'))) {
 
 		// Donation Confirmation (Success) Page
 		$success = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Confirmation', 'give' ),
+				'post_title'     => esc_html__('Donation Confirmation', 'give'),
 				'post_content'   => '[give_receipt]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -428,13 +428,13 @@ 
 	}
 
 	// Checks if the Failure Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'failure_page' ) ) ) {
+	if ( ! get_post(give_get_option('failure_page'))) {
 
 		// Failed Donation Page
 		$failed = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Failed', 'give' ),
-				'post_content'   => esc_html__( 'We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give' ),
+				'post_title'     => esc_html__('Donation Failed', 'give'),
+				'post_content'   => esc_html__('We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -446,11 +446,11 @@ 
 	}
 
 	// Checks if the History Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'history_page' ) ) ) {
+	if ( ! get_post(give_get_option('history_page'))) {
 		// Donation History Page
 		$history = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation History', 'give' ),
+				'post_title'     => esc_html__('Donation History', 'give'),
 				'post_content'   => '[donation_history]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -462,11 +462,11 @@ 
 		$options['history_page'] = $history;
 	}
 
-	if ( ! empty( $options ) ) {
-		update_option( 'give_settings', array_merge( give_get_settings(), $options ) );
+	if ( ! empty($options)) {
+		update_option('give_settings', array_merge(give_get_settings(), $options));
 	}
 
-	add_option( 'give_install_pages_created', 1, '', 'no' );
+	add_option('give_install_pages_created', 1, '', 'no');
 }
 
-add_action( 'admin_init', 'give_create_pages', - 1 );
+add_action('admin_init', 'give_create_pages', - 1);

includes/payments/backward-compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
  *                              metadata entries with the specified value.
  *                              Otherwise, update all entries.
  *
- * @return mixed
+ * @return null|boolean
  */
 function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
 	// Bailout.

Spacing +166 added lines, -171 removed lines

@@ -9,74 +9,74 @@ 
  *
  * @return void
  */
-function _give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value ) {
+function _give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value) {
 	// Bailout
-	if ( empty( $meta_value ) ) {
+	if (empty($meta_value)) {
 		return;
-	} elseif ( ! is_array( $meta_value ) ) {
+	} elseif ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10 );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10);
 
 	// Date payment meta.
-	if ( ! empty( $meta_value['date'] ) ) {
-		give_update_meta( $object_id, '_give_payment_date', $meta_value['date'] );
+	if ( ! empty($meta_value['date'])) {
+		give_update_meta($object_id, '_give_payment_date', $meta_value['date']);
 	}
 
 	// Currency payment meta.
-	if ( ! empty( $meta_value['currency'] ) ) {
-		give_update_meta( $object_id, '_give_payment_currency', $meta_value['currency'] );
+	if ( ! empty($meta_value['currency'])) {
+		give_update_meta($object_id, '_give_payment_currency', $meta_value['currency']);
 	}
 
 	// User information.
-	if ( ! empty( $meta_value['user_info'] ) ) {
+	if ( ! empty($meta_value['user_info'])) {
 		// Donor first name.
-		if ( ! empty( $meta_value['user_info']['first_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name'] );
+		if ( ! empty($meta_value['user_info']['first_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name']);
 		}
 
 		// Donor last name.
-		if ( ! empty( $meta_value['user_info']['last_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name'] );
+		if ( ! empty($meta_value['user_info']['last_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name']);
 		}
 
 		// Donor address payment meta.
-		if ( ! empty( $meta_value['user_info']['address'] ) ) {
+		if ( ! empty($meta_value['user_info']['address'])) {
 
 			// Address1.
-			if ( ! empty( $meta_value['user_info']['address']['line1'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1'] );
+			if ( ! empty($meta_value['user_info']['address']['line1'])) {
+				give_update_meta($object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1']);
 			}
 
 			// Address2.
-			if ( ! empty( $meta_value['user_info']['address']['line2'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2'] );
+			if ( ! empty($meta_value['user_info']['address']['line2'])) {
+				give_update_meta($object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2']);
 			}
 
 			// City.
-			if ( ! empty( $meta_value['user_info']['address']['city'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city'] );
+			if ( ! empty($meta_value['user_info']['address']['city'])) {
+				give_update_meta($object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city']);
 			}
 
 			// Zip.
-			if ( ! empty( $meta_value['user_info']['address']['zip'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip'] );
+			if ( ! empty($meta_value['user_info']['address']['zip'])) {
+				give_update_meta($object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip']);
 			}
 
 			// State.
-			if ( ! empty( $meta_value['user_info']['address']['state'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state'] );
+			if ( ! empty($meta_value['user_info']['address']['state'])) {
+				give_update_meta($object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state']);
 			}
 
 			// Country.
-			if ( ! empty( $meta_value['user_info']['address']['country'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country'] );
+			if ( ! empty($meta_value['user_info']['address']['country'])) {
+				give_update_meta($object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country']);
 			}
 		}
 	}// End if().
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 }
 
 /**
@@ -89,103 +89,99 @@ 
  *
  * @return array
  */
-function _give_20_bc_give_payment_meta_value( $object_id, $meta_value ) {
+function _give_20_bc_give_payment_meta_value($object_id, $meta_value) {
 	$cache_key = "_give_payment_meta_{$object_id}";
-	$cache     = Give_Cache::get_db_query( $cache_key );
+	$cache     = Give_Cache::get_db_query($cache_key);
 
-	if ( ! is_null( $cache ) ) {
+	if ( ! is_null($cache)) {
 		return $cache;
 	}
 
 	// Set default value to array.
-	if ( ! is_array( $meta_value ) ) {
+	if ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
 	// Donation key.
-	$meta_value['key'] = give_get_meta( $object_id, '_give_payment_purchase_key', true );
+	$meta_value['key'] = give_get_meta($object_id, '_give_payment_purchase_key', true);
 
 	// Donation form.
-	$meta_value['form_title'] = give_get_meta( $object_id, '_give_payment_form_title', true );
+	$meta_value['form_title'] = give_get_meta($object_id, '_give_payment_form_title', true);
 
 	// Donor email.
-	$meta_value['email'] = give_get_meta( $object_id, '_give_payment_donor_email', true );
-	$meta_value['email'] = ! empty( $meta_value['email'] ) ?
-		$meta_value['email'] :
-		Give()->donors->get_column( 'email', give_get_payment_donor_id( $object_id ) );
+	$meta_value['email'] = give_get_meta($object_id, '_give_payment_donor_email', true);
+	$meta_value['email'] = ! empty($meta_value['email']) ?
+		$meta_value['email'] : Give()->donors->get_column('email', give_get_payment_donor_id($object_id));
 
 	// Form id.
-	$meta_value['form_id'] = give_get_meta( $object_id, '_give_payment_form_id', true );
+	$meta_value['form_id'] = give_get_meta($object_id, '_give_payment_form_id', true);
 
 	// Price id.
-	$meta_value['price_id'] = give_get_meta( $object_id, '_give_payment_price_id', true );
+	$meta_value['price_id'] = give_get_meta($object_id, '_give_payment_price_id', true);
 
 	// Date.
-	$meta_value['date'] = give_get_meta( $object_id, '_give_payment_date', true );
-	$meta_value['date'] = ! empty( $meta_value['date'] ) ?
-		$meta_value['date'] :
-		get_post_field( 'post_date', $object_id );
+	$meta_value['date'] = give_get_meta($object_id, '_give_payment_date', true);
+	$meta_value['date'] = ! empty($meta_value['date']) ?
+		$meta_value['date'] : get_post_field('post_date', $object_id);
 
 	// Currency.
-	$meta_value['currency'] = give_get_meta( $object_id, '_give_payment_currency', true );
+	$meta_value['currency'] = give_get_meta($object_id, '_give_payment_currency', true);
 
 	// Decode donor data.
-	$donor_names = give_get_donor_name_by( give_get_meta( $object_id, '_give_payment_donor_id', true ), 'donor' );
-	$donor_names = explode( ' ', $donor_names, 2 );
+	$donor_names = give_get_donor_name_by(give_get_meta($object_id, '_give_payment_donor_id', true), 'donor');
+	$donor_names = explode(' ', $donor_names, 2);
 
 	// Donor first name.
-	$donor_data['first_name'] = give_get_meta( $object_id, '_give_donor_billing_first_name', true );
-	$donor_data['first_name'] = ! empty( $donor_data['first_name'] ) ?
-		$donor_data['first_name'] :
-		$donor_names[0];
+	$donor_data['first_name'] = give_get_meta($object_id, '_give_donor_billing_first_name', true);
+	$donor_data['first_name'] = ! empty($donor_data['first_name']) ?
+		$donor_data['first_name'] : $donor_names[0];
 
 	// Donor last name.
-	$donor_data['last_name'] = give_get_meta( $object_id, '_give_donor_billing_last_name', true );
-	$donor_data['last_name'] = ! empty( $donor_data['last_name'] ) ?
-		$donor_data['last_name'] :
-		( isset( $donor_names[1] ) ? $donor_names[1] : '' );
+	$donor_data['last_name'] = give_get_meta($object_id, '_give_donor_billing_last_name', true);
+	$donor_data['last_name'] = ! empty($donor_data['last_name']) ?
+		$donor_data['last_name'] : (isset($donor_names[1]) ? $donor_names[1] : '');
 
 	// Donor email.
 	$donor_data['email'] = $meta_value['email'];
 
 	// User ID.
-	$donor_data['id'] = give_get_payment_user_id( $object_id );
+	$donor_data['id'] = give_get_payment_user_id($object_id);
 
 	$donor_data['address'] = false;
 
 	// Address1.
-	if ( $address1 = give_get_meta( $object_id, '_give_donor_billing_address1', true ) ) {
+	if ($address1 = give_get_meta($object_id, '_give_donor_billing_address1', true)) {
 		$donor_data['address']['line1'] = $address1;
 	}
 
 	// Address2.
-	if ( $address2 = give_get_meta( $object_id, '_give_donor_billing_address2', true ) ) {
+	if ($address2 = give_get_meta($object_id, '_give_donor_billing_address2', true)) {
 		$donor_data['address']['line2'] = $address2;
 	}
 
 	// City.
-	if ( $city = give_get_meta( $object_id, '_give_donor_billing_city', true ) ) {
+	if ($city = give_get_meta($object_id, '_give_donor_billing_city', true)) {
 		$donor_data['address']['city'] = $city;
 	}
 
 	// Zip.
-	if ( $zip = give_get_meta( $object_id, '_give_donor_billing_zip', true ) ) {
+	if ($zip = give_get_meta($object_id, '_give_donor_billing_zip', true)) {
 		$donor_data['address']['zip'] = $zip;
 	}
 
 	// State.
-	if ( $state = give_get_meta( $object_id, '_give_donor_billing_state', true ) ) {
+	if ($state = give_get_meta($object_id, '_give_donor_billing_state', true)) {
 		$donor_data['address']['state'] = $state;
 	}
 
 	// Country.
-	if ( $country = give_get_meta( $object_id, '_give_donor_billing_country', true ) ) {
+	if ($country = give_get_meta($object_id, '_give_donor_billing_country', true)) {
 		$donor_data['address']['country'] = $country;
 	}
 
-	$meta_value['user_info'] = maybe_unserialize( $donor_data );
+	$meta_value['user_info'] = maybe_unserialize($donor_data);
 
-	Give_Cache::set_db_query( $cache_key, $meta_value );
+	Give_Cache::set_db_query($cache_key, $meta_value);
 
 	return $meta_value;
 }
@@ -209,37 +205,37 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
+function _give_20_bc_saving_old_payment_meta($check, $object_id, $meta_key, $meta_value, $prev_value) {
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, array(
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, array(
 			'_give_payment_meta',
 			'_give_payment_user_email',
 			'_give_payment_customer_id',
 			'give_payment_user_ip',
-		) )
+		))
 	) {
 		return $check;
 	}
 
-	if ( '_give_payment_meta' === $meta_key ) {
-		_give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value );
-	} elseif ( '_give_payment_user_email' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_email', $meta_value );
+	if ('_give_payment_meta' === $meta_key) {
+		_give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value);
+	} elseif ('_give_payment_user_email' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_email', $meta_value);
 		$check = true;
-	} elseif ( '_give_payment_customer_id' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_id', $meta_value );
+	} elseif ('_give_payment_customer_id' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_id', $meta_value);
 		$check = true;
-	} elseif ( 'give_payment_user_ip' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_ip', $meta_value );
+	} elseif ('give_payment_user_ip' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_ip', $meta_value);
 		$check = true;
 	}
 
 	return $check;
 }
 
-add_filter( 'update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5 );
+add_filter('update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5);
 
 
 /**
@@ -254,7 +250,7 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_old_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_old_payment_meta($check, $object_id, $meta_key, $single) {
 	global $wpdb;
 
 	// Deprecated meta keys.
@@ -265,32 +261,31 @@ 
 	);
 
 	// Add _give_payment_meta to backward compatibility
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		$old_meta_keys[] = '_give_payment_meta';
 	}
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $old_meta_keys )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $old_meta_keys)
 	) {
 		return $check;
 	}
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check     = Give_Cache::get_db_query( $cache_key );
+	$check     = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle old meta keys.
 			case '_give_payment_meta':
-				remove_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta' );
+				remove_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta');
 
 				// if ( $meta_value = give_get_meta( $object_id, '_give_payment_meta' ) ) {
-				$meta_value = ! empty( $meta_value ) ?
-					current( $meta_value ) :
-					(array) maybe_unserialize(
+				$meta_value = ! empty($meta_value) ?
+					current($meta_value) : (array) maybe_unserialize(
 						$wpdb->get_var(
 							$wpdb->prepare(
 								"
@@ -304,44 +299,44 @@ 
 							)
 						)
 					);
-				$check      = _give_20_bc_give_payment_meta_value( $object_id, $meta_value );
+				$check = _give_20_bc_give_payment_meta_value($object_id, $meta_value);
 				// }
 
-				add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+				add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 				break;
 
 			case '_give_payment_customer_id':
-				if ( $donor_id = give_get_meta( $object_id, '_give_payment_donor_id', $single ) ) {
+				if ($donor_id = give_get_meta($object_id, '_give_payment_donor_id', $single)) {
 					$check = $donor_id;
 				}
 				break;
 
 			case '_give_payment_user_email':
-				if ( $donor_email = give_get_meta( $object_id, '_give_payment_donor_email', $single ) ) {
+				if ($donor_email = give_get_meta($object_id, '_give_payment_donor_email', $single)) {
 					$check = $donor_email;
 				}
 				break;
 
 			case '_give_payment_user_ip':
-				if ( $donor_ip = give_get_meta( $object_id, '_give_payment_donor_ip', $single ) ) {
+				if ($donor_ip = give_get_meta($object_id, '_give_payment_donor_ip', $single)) {
 					$check = $donor_ip;
 				}
 				break;
 		}// End switch().
 
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! is_null( $check ) ) {
-		$check = array( $check );
+	if ( ! is_null($check)) {
+		$check = array($check);
 	}
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 
 /**
@@ -356,9 +351,9 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_new_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_new_payment_meta($check, $object_id, $meta_key, $single) {
 	// Bailout: do not apply backward compatibility if upgrade done.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return $check;
 	}
 
@@ -380,26 +375,26 @@ 
 	);
 
 	// metadata_exists fx will cause of firing get_post_metadata filter again so remove it to prevent infinite loop.
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta' );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta');
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $new_meta_keys ) ||
-		metadata_exists( 'post', $object_id, $meta_key )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $new_meta_keys) ||
+		metadata_exists('post', $object_id, $meta_key)
 	) {
-		add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+		add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 		return $check;
 	}
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check    = Give_Cache::get_db_query( $cache_key );
+	$check = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle new meta keys.
 			case '_give_payment_donor_id':
@@ -442,9 +437,9 @@ 
 			case '_give_donor_billing_country':
 			case '_give_payment_date':
 			case '_give_payment_currency':
-				$donation_meta = Give_Cache::get_db_query( "_give_payment_meta_{$object_id}" );
+				$donation_meta = Give_Cache::get_db_query("_give_payment_meta_{$object_id}");
 
-				if ( is_null( $donation_meta ) ) {
+				if (is_null($donation_meta)) {
 					$donation_meta = $wpdb->get_var(
 						$wpdb->prepare(
 							"SELECT meta_value FROM {$wpdb->postmeta} WHERE post_id=%d AND meta_key=%s",
@@ -452,51 +447,51 @@ 
 							'_give_payment_meta'
 						)
 					);
-					$donation_meta = maybe_unserialize( $donation_meta );
-					$donation_meta = ! is_array( $donation_meta ) ? array() : $donation_meta;
-					Give_Cache::set_db_query( "_give_payment_meta_{$object_id}", $donation_meta );
+					$donation_meta = maybe_unserialize($donation_meta);
+					$donation_meta = ! is_array($donation_meta) ? array() : $donation_meta;
+					Give_Cache::set_db_query("_give_payment_meta_{$object_id}", $donation_meta);
 				}
 
 				// Get results.
-				if ( empty( $donation_meta ) ) {
+				if (empty($donation_meta)) {
 					$check = '';
-				} elseif ( in_array( $meta_key, array( '_give_payment_date', '_give_payment_currency' ) ) ) {
-					$payment_meta_key = str_replace( '_give_payment_', '', $meta_key );
+				} elseif (in_array($meta_key, array('_give_payment_date', '_give_payment_currency'))) {
+					$payment_meta_key = str_replace('_give_payment_', '', $meta_key);
 
-					if ( isset( $donation_meta[ $payment_meta_key ] ) ) {
-						$check = $donation_meta[ $payment_meta_key ];
+					if (isset($donation_meta[$payment_meta_key])) {
+						$check = $donation_meta[$payment_meta_key];
 					}
 				} else {
-					$payment_meta_key = str_replace( '_give_donor_billing_', '', $meta_key );
+					$payment_meta_key = str_replace('_give_donor_billing_', '', $meta_key);
 
-					switch ( $payment_meta_key ) {
+					switch ($payment_meta_key) {
 						case 'address1':
-							if ( isset( $donation_meta['user_info']['address']['line1'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line1'])) {
 								$check = $donation_meta['user_info']['address']['line1'];
 							}
 							break;
 
 						case 'address2':
-							if ( isset( $donation_meta['user_info']['address']['line2'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line2'])) {
 								$check = $donation_meta['user_info']['address']['line2'];
 							}
 							break;
 
 						case 'first_name':
-							if ( isset( $donation_meta['user_info']['first_name'] ) ) {
+							if (isset($donation_meta['user_info']['first_name'])) {
 								$check = $donation_meta['user_info']['first_name'];
 							}
 							break;
 
 						case 'last_name':
-							if ( isset( $donation_meta['user_info']['last_name'] ) ) {
+							if (isset($donation_meta['user_info']['last_name'])) {
 								$check = $donation_meta['user_info']['last_name'];
 							}
 							break;
 
 						default:
-							if ( isset( $donation_meta['user_info']['address'][ $payment_meta_key ] ) ) {
-								$check = $donation_meta['user_info']['address'][ $payment_meta_key ];
+							if (isset($donation_meta['user_info']['address'][$payment_meta_key])) {
+								$check = $donation_meta['user_info']['address'][$payment_meta_key];
 							}
 					}
 				}
@@ -505,19 +500,19 @@ 
 		}// End switch().
 
 		// Set cache.
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! $single ) {
-		$check = array( $check );
+	if ( ! $single) {
+		$check = array($check);
 	}
 
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 
 /**
@@ -529,9 +524,9 @@ 
  *
  * @return void
  */
-function _give_20_bc_support_deprecated_meta_key_query( $query ) {
+function _give_20_bc_support_deprecated_meta_key_query($query) {
 	// Bailout.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
@@ -541,21 +536,21 @@ 
 		// '_give_payment_user_ip'     => '_give_payment_donor_ip',
 	);
 
-	$deprecated_meta_keys = array_flip( $new_meta_keys );
+	$deprecated_meta_keys = array_flip($new_meta_keys);
 
 	// Set meta keys.
 	$meta_keys = array();
 
 
 	// Bailout.
-	if ( ! empty( $query->query_vars['meta_key'] ) ) {
-		if ( in_array( $query->query_vars['meta_key'], $new_meta_keys ) ) {
+	if ( ! empty($query->query_vars['meta_key'])) {
+		if (in_array($query->query_vars['meta_key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $query->query_vars['meta_key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($query->query_vars['meta_key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -566,35 +561,35 @@ 
 						'value' => $query->query_vars['meta_value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_key']],
 						'value' => $query->query_vars['meta_value'],
 					),
 				)
 			);
 
 			// Unset single meta query.
-			unset( $query->query_vars['meta_key'] );
-			unset( $query->query_vars['meta_value'] );
+			unset($query->query_vars['meta_key']);
+			unset($query->query_vars['meta_value']);
 		}
 	} elseif (
-		! empty( $query->query_vars['meta_query'] ) &&
-		( 1 === count( $query->query_vars['meta_query'] ) )
+		! empty($query->query_vars['meta_query']) &&
+		(1 === count($query->query_vars['meta_query']))
 	) {
-		$meta_query = current( $query->query_vars['meta_query'] );
+		$meta_query = current($query->query_vars['meta_query']);
 
-		if ( empty( $meta_query[0]['key'] ) ) {
+		if (empty($meta_query[0]['key'])) {
 			return;
 		}
 
-		if ( in_array( $meta_query[0]['key'], $new_meta_keys ) ) {
+		if (in_array($meta_query[0]['key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $meta_query[0]['key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($meta_query[0]['key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		} else {
 			return;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -605,7 +600,7 @@ 
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_query'][0]['key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_query'][0]['key']],
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 				)
@@ -614,7 +609,7 @@ 
 	}
 }
 
-add_action( 'pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query' );
+add_action('pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query');
 
 
 /**
@@ -627,49 +622,49 @@ 
  * @param Give_Payment $payment
  * @param string       $key
  */
-function _give_20_bc_payment_save( $payment, $key ) {
+function _give_20_bc_payment_save($payment, $key) {
 	// Bailout.
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
-	switch ( $key ) {
+	switch ($key) {
 		case 'user_info':
-			if ( empty( $payment->user_info ) ) {
+			if (empty($payment->user_info)) {
 				// Bailout.
 				break;
-			} elseif ( is_string( $payment->user_info ) ) {
+			} elseif (is_string($payment->user_info)) {
 				// Check if value serialize.
-				$payment->user_info = maybe_unserialize( $payment->user_info );
+				$payment->user_info = maybe_unserialize($payment->user_info);
 			}
 
 
 			// Save first name.
-			if ( isset( $payment->user_info['first_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_first_name', $payment->user_info['first_name'] );
+			if (isset($payment->user_info['first_name'])) {
+				$payment->update_meta('_give_donor_billing_first_name', $payment->user_info['first_name']);
 			}
 
 
 			// Save last name.
-			if ( isset( $payment->user_info['last_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_last_name', $payment->user_info['last_name'] );
+			if (isset($payment->user_info['last_name'])) {
+				$payment->update_meta('_give_donor_billing_last_name', $payment->user_info['last_name']);
 			}
 
 
 			// Save address.
-			if ( ! empty( $payment->user_info['address'] ) ) {
-				foreach ( $payment->user_info['address'] as $address_name => $address ) {
-					switch ( $address_name ) {
+			if ( ! empty($payment->user_info['address'])) {
+				foreach ($payment->user_info['address'] as $address_name => $address) {
+					switch ($address_name) {
 						case 'line1':
-							$payment->update_meta( '_give_donor_billing_address1', $address );
+							$payment->update_meta('_give_donor_billing_address1', $address);
 							break;
 
 						case 'line2':
-							$payment->update_meta( '_give_donor_billing_address2', $address );
+							$payment->update_meta('_give_donor_billing_address2', $address);
 							break;
 
 						default:
-							$payment->update_meta( "_give_donor_billing_{$address_name}", $address );
+							$payment->update_meta("_give_donor_billing_{$address_name}", $address);
 					}
 				}
 			}
@@ -678,7 +673,7 @@ 
 	}
 }
 
-add_action( 'give_payment_save', '_give_20_bc_payment_save', 10, 2 );
+add_action('give_payment_save', '_give_20_bc_payment_save', 10, 2);
 
 
 /**
@@ -691,16 +686,16 @@ 
  *
  * @return mixed
  */
-function __give_20_bc_flush_cache( $check, $object_id ) {
+function __give_20_bc_flush_cache($check, $object_id) {
 	if (
-		! give_has_upgrade_completed( 'v20_move_metadata_into_new_table' ) &&
-		'give_payment' === get_post_type( $object_id )
+		! give_has_upgrade_completed('v20_move_metadata_into_new_table') &&
+		'give_payment' === get_post_type($object_id)
 	) {
-		Give_Cache::delete_group( $object_id, 'give-donations' );
+		Give_Cache::delete_group($object_id, 'give-donations');
 	}
 
 	return $check;
 }
 
-add_action( 'update_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
-add_action( 'add_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
+add_action('update_post_metadata', '__give_20_bc_flush_cache', 9999, 2);
+add_action('add_post_metadata', '__give_20_bc_flush_cache', 9999, 2);

includes/payments/class-payments-query.php

Doc Comments +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 * @param $value
 	 */
 	public function __set( $query_var, $value ) {
@@ -124,7 +124,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 */
 	public function __unset( $query_var ) {
 		unset( $this->args[ $query_var ] );
@@ -468,7 +468,7 @@ 
 	 * @param string   $order
 	 * @param WP_Query $query
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function custom_orderby( $order, $query ) {
 

Spacing +148 added lines, -148 removed lines

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

includes/payments/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -565,7 +565,7 @@ 
  *
  * @since 1.0
  *
- * @return int $earnings Earnings
+ * @return double $earnings Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
@@ -881,7 +881,7 @@ 
  *
  * @since 1.0
  *
- * @return int $form_id Form ID.
+ * @return string $form_id Form ID.
  */
 function give_get_payment_form_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -1729,7 +1729,7 @@ 
  *
  * Retrieves the form title and appends the level name if present.
  *
- * @param int|Give_Payment $donation Donation Data Object.
+ * @param Give_Payment $donation Donation Data Object.
  * @param array            $args     a. only_level = If set to true will only return the level name if multi-level enabled.
  *                                   b. separator  = The separator between the Form Title and the Donation Level.
  *

Spacing +380 added lines, -382 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -44,15 +44,15 @@ 
  *
  * @return array $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility.
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -67,48 +67,48 @@ 
  *
  * @return mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -117,7 +117,7 @@ 
 			return false;
 	}// End switch().
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -133,9 +133,9 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise.
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
@@ -146,25 +146,25 @@ 
 	 *
 	 * @param array $payment_data Arguments passed.
 	 */
-	$payment_data = apply_filters( 'give_pre_insert_payment', $payment_data );
+	$payment_data = apply_filters('give_pre_insert_payment', $payment_data);
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = give_get_payment_meta_price_id( $payment_data );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = give_get_payment_meta_price_id($payment_data);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	// Set properties.
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency( $payment_data['give_form_id'], $payment_data );
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency($payment_data['give_form_id'], $payment_data);
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
 	$payment->form_id        = $form_id;
 	$payment->price_id       = $price_id;
-	$payment->donor_id       = ( ! empty( $payment_data['donor_id'] ) ? $payment_data['donor_id'] : '' );
+	$payment->donor_id       = ( ! empty($payment_data['donor_id']) ? $payment_data['donor_id'] : '');
 	$payment->user_id        = $payment_data['user_info']['id'];
 	$payment->email          = $payment_data['user_email'];
 	$payment->first_name     = $payment_data['user_info']['first_name'];
@@ -172,8 +172,8 @@ 
 	$payment->email          = $payment_data['user_info']['email'];
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
-	$payment->mode           = ( ! empty( $payment_data['mode'] ) ? (string) $payment_data['mode'] : ( give_is_test_mode() ? 'test' : 'live' ) );
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->mode           = ( ! empty($payment_data['mode']) ? (string) $payment_data['mode'] : (give_is_test_mode() ? 'test' : 'live'));
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	// Add the donation.
 	$args = array(
@@ -181,19 +181,19 @@ 
 		'price_id' => $payment->price_id,
 	);
 
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 
 	// Set date if present.
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	// Handle sequential payments.
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Save payment.
@@ -207,10 +207,10 @@ 
 	 * @param int   $payment_id   The payment ID.
 	 * @param array $payment_data Arguments passed.
 	 */
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	// Return payment ID upon success.
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -226,10 +226,10 @@ 
  *
  * @return bool|int
  */
-function give_create_payment( $payment_data ) {
+function give_create_payment($payment_data) {
 
-	$form_id  = intval( $payment_data['post_data']['give-form-id'] );
-	$price_id = isset( $payment_data['post_data']['give-price-id'] ) ? $payment_data['post_data']['give-price-id'] : '';
+	$form_id  = intval($payment_data['post_data']['give-form-id']);
+	$price_id = isset($payment_data['post_data']['give-price-id']) ? $payment_data['post_data']['give-price-id'] : '';
 
 	// Collect payment data.
 	$insert_payment_data = array(
@@ -240,7 +240,7 @@ 
 		'date'            => $payment_data['date'],
 		'user_email'      => $payment_data['user_email'],
 		'purchase_key'    => $payment_data['purchase_key'],
-		'currency'        => give_get_currency( $form_id, $payment_data ),
+		'currency'        => give_get_currency($form_id, $payment_data),
 		'user_info'       => $payment_data['user_info'],
 		'status'          => 'pending',
 		'gateway'         => 'paypal',
@@ -253,10 +253,10 @@ 
 	 *
 	 * @param array $insert_payment_data
 	 */
-	$insert_payment_data = apply_filters( 'give_create_payment', $insert_payment_data );
+	$insert_payment_data = apply_filters('give_create_payment', $insert_payment_data);
 
 	// Record the pending payment.
-	return give_insert_payment( $insert_payment_data );
+	return give_insert_payment($insert_payment_data);
 }
 
 /**
@@ -269,12 +269,12 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
 	$updated = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment && $payment->ID > 0 ) {
+	if ($payment && $payment->ID > 0) {
 
 		$payment->status = $new_status;
 		$updated         = $payment->save();
@@ -295,45 +295,45 @@ 
  *
  * @return void
  */
-function give_delete_donation( $payment_id = 0, $update_donor = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_delete_donation($payment_id = 0, $update_donor = true) {
+	$payment = new Give_Payment($payment_id);
 
 	// Bailout.
-	if ( ! $payment->ID ) {
+	if ( ! $payment->ID) {
 		return;
 	}
 
-	$amount   = give_donation_amount( $payment_id );
+	$amount   = give_donation_amount($payment_id);
 	$status   = $payment->post_status;
-	$donor_id = give_get_payment_donor_id( $payment_id );
-	$donor    = new Give_Donor( $donor_id );
+	$donor_id = give_get_payment_donor_id($payment_id);
+	$donor    = new Give_Donor($donor_id);
 
 	// Only undo donations that aren't these statuses.
-	$dont_undo_statuses = apply_filters( 'give_undo_donation_statuses', array(
+	$dont_undo_statuses = apply_filters('give_undo_donation_statuses', array(
 		'pending',
 		'cancelled',
-	) );
+	));
 
-	if ( ! in_array( $status, $dont_undo_statuses ) ) {
-		give_undo_donation( $payment_id );
+	if ( ! in_array($status, $dont_undo_statuses)) {
+		give_undo_donation($payment_id);
 	}
 
 	// Only undo donations that aren't these statuses.
-	$status_to_decrease_stats = apply_filters( 'give_decrease_donor_statuses', array( 'publish' ) );
+	$status_to_decrease_stats = apply_filters('give_decrease_donor_statuses', array('publish'));
 
-	if ( in_array( $status, $status_to_decrease_stats ) ) {
+	if (in_array($status, $status_to_decrease_stats)) {
 
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment).
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
 
-		if ( $donor->id && $update_donor ) {
+		if ($donor->id && $update_donor) {
 
 			// Decrement the stats for the donor.
 			$donor->decrease_donation_count();
-			$donor->decrease_value( $amount );
+			$donor->decrease_value($amount);
 
 		}
 	}
@@ -345,18 +345,18 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $donor->id && $update_donor ) {
+	if ($donor->id && $update_donor) {
 		// Remove the payment ID from the donor.
-		$donor->remove_payment( $payment_id );
+		$donor->remove_payment($payment_id);
 	}
 
 	// Remove the payment.
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries.
-	Give()->logs->delete_logs( $payment_id );
+	Give()->logs->delete_logs($payment_id);
 
 	/**
 	 * Fires after payment deleted.
@@ -365,7 +365,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -380,20 +380,20 @@ 
  *
  * @return void
  */
-function give_undo_donation( $payment_id ) {
+function give_undo_donation($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// Decrease earnings.
-		give_decrease_form_earnings( $payment->form_id, $payment->total );
+		give_decrease_form_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_donations = apply_filters( 'give_decrease_donations_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_donations ) {
+	$maybe_decrease_donations = apply_filters('give_decrease_donations_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_donations) {
 		// Decrease donation count.
-		give_decrease_donation_count( $payment->form_id );
+		give_decrease_donation_count($payment->form_id);
 	}
 
 }
@@ -410,21 +410,21 @@ 
  *
  * @return object $stats Contains the number of payments per payment status.
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 	// Backward compatibility.
-	if ( ! empty( $args['start-date'] ) ) {
+	if ( ! empty($args['start-date'])) {
 		$args['start_date'] = $args['start-date'];
-		unset( $args['start-date'] );
+		unset($args['start-date']);
 	}
 
-	if ( ! empty( $args['end-date'] ) ) {
+	if ( ! empty($args['end-date'])) {
 		$args['end_date'] = $args['end-date'];
-		unset( $args['end-date'] );
+		unset($args['end-date']);
 	}
 
-	if ( ! empty( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id'])) {
 		$args['give_forms'] = $args['form_id'];
-		unset( $args['form_id'] );
+		unset($args['form_id']);
 	}
 
 	// Extract all donations
@@ -432,7 +432,7 @@ 
 	$args['group_by']    = 'post_status';
 	$args['count']       = 'true';
 
-	$donations_obj   = new Give_Payments_Query( $args );
+	$donations_obj   = new Give_Payments_Query($args);
 	$donations_count = $donations_obj->get_payment_by_group();
 
 	/**
@@ -440,7 +440,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return (object) apply_filters( 'give_count_payments', $donations_count, $args, $donations_obj );
+	return (object) apply_filters('give_count_payments', $donations_count, $args, $donations_obj);
 }
 
 
@@ -453,11 +453,11 @@ 
  *
  * @return bool $exists True if payment exists, false otherwise.
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -475,41 +475,41 @@ 
  *
  * @return bool|mixed True if payment status exists, false otherwise.
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( is_numeric( $payment ) ) {
+	if (is_numeric($payment)) {
 
-		$payment = new Give_Payment( $payment );
+		$payment = new Give_Payment($payment);
 
-		if ( ! $payment->ID > 0 ) {
+		if ( ! $payment->ID > 0) {
 			return false;
 		}
 
 	}
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
 	// Get payment object if not already given.
-	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment( $payment->ID );
+	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
 			// Return translated status label.
-			return $statuses[ $payment->status ];
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' === $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -525,18 +525,18 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'processing'  => __( 'Processing', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' ),
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'processing'  => __('Processing', 'give'),
+		'revoked'     => __('Revoked', 'give'),
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -549,10 +549,10 @@ 
  * @return array $payment_status All the available payment statuses.
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -567,44 +567,44 @@ 
  *
  * @return int $earnings Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
 
 	global $wpdb;
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$meta_table = __give_v20_bc_table_details('payment');
 
 	$args = array(
 		'post_type'              => 'give_payment',
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false,
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( isset( $hour ) ) {
+	if (isset($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = Give_Cache::get_key( 'give_stats', $args );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 	}
 
-	if ( false === $earnings ) {
-		$donations = get_posts( $args );
+	if (false === $earnings) {
+		$donations = get_posts($args);
 		$earnings  = 0;
-		if ( $donations ) {
-			$donations      = implode( ',', $donations );
-			$earning_totals = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})" );
+		if ($donations) {
+			$donations      = implode(',', $donations);
+			$earning_totals = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})");
 
 			/**
 			 * Filter The earnings by dates.
@@ -615,13 +615,13 @@ 
 			 * @param array $donations      Donations lists.
 			 * @param array $args           Donation query args.
 			 */
-			$earnings = apply_filters( 'give_get_earnings_by_date', $earning_totals, $donations, $args );
+			$earnings = apply_filters('give_get_earnings_by_date', $earning_totals, $donations, $args);
 		}
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $earnings, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -636,7 +636,7 @@ 
  *
  * @return int $count Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead.
 	$args = array(
@@ -644,14 +644,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false,
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -662,33 +662,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( isset( $hour ) ) {
+	if (isset($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = Give_Cache::get_key( 'give_stats', $args );
+	$key = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = Give_Cache::get( $key );
+		$count = Give_Cache::get($key);
 	}
 
-	if ( false === $count ) {
-		$donations = new WP_Query( $args );
+	if (false === $count) {
+		$donations = new WP_Query($args);
 		$count     = (int) $donations->post_count;
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $count, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -703,19 +703,19 @@ 
  *
  * @return bool $ret True if complete, false otherwise.
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -741,50 +741,50 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_get_total_earnings( $recalculate = false ) {
+function give_get_total_earnings($recalculate = false) {
 
-	$total      = get_option( 'give_earnings_total', 0 );
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$total      = get_option('give_earnings_total', 0);
+	$meta_table = __give_v20_bc_table_details('payment');
 
 	// Calculate total earnings.
-	if ( ! $total || $recalculate ) {
+	if ( ! $total || $recalculate) {
 		global $wpdb;
 
 		$total = (float) 0;
 
-		$args = apply_filters( 'give_get_total_earnings_args', array(
+		$args = apply_filters('give_get_total_earnings_args', array(
 			'offset' => 0,
-			'number' => - 1,
-			'status' => array( 'publish' ),
+			'number' => -1,
+			'status' => array('publish'),
 			'fields' => 'ids',
-		) );
+		));
 
-		$payments = give_get_payments( $args );
-		if ( $payments ) {
+		$payments = give_get_payments($args);
+		if ($payments) {
 
 			/**
 			 * If performing a donation, we need to skip the very last payment in the database,
 			 * since it calls give_increase_total_earnings() on completion,
 			 * which results in duplicated earnings for the very first donation.
 			 */
-			if ( did_action( 'give_update_payment_status' ) ) {
-				array_pop( $payments );
+			if (did_action('give_update_payment_status')) {
+				array_pop($payments);
 			}
 
-			if ( ! empty( $payments ) ) {
-				$payments = implode( ',', $payments );
-				$total    += $wpdb->get_var( "SELECT SUM(meta_value) FROM {$meta_table['name']} WHERE meta_key = '_give_payment_total' AND {$meta_table['column']['id']} IN({$payments})" );
+			if ( ! empty($payments)) {
+				$payments = implode(',', $payments);
+				$total += $wpdb->get_var("SELECT SUM(meta_value) FROM {$meta_table['name']} WHERE meta_key = '_give_payment_total' AND {$meta_table['column']['id']} IN({$payments})");
 			}
 		}
 
-		update_option( 'give_earnings_total', $total, 'no' );
+		update_option('give_earnings_total', $total, 'no');
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings.
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_get_price_decimals() ), $total );
+	return apply_filters('give_total_earnings', round($total, give_get_price_decimals()), $total);
 }
 
 /**
@@ -796,10 +796,10 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -813,13 +813,13 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -835,10 +835,10 @@ 
  *
  * @return mixed $meta Payment Meta.
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -851,10 +851,10 @@ 
  *
  * @return mixed Meta ID if successful, false if unsuccessful.
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -866,8 +866,8 @@ 
  *
  * @return array $user_info User Info Meta Values.
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -883,8 +883,8 @@ 
  *
  * @return int $form_id Form ID.
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -898,8 +898,8 @@ 
  *
  * @return string $email User email.
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -913,11 +913,11 @@ 
  *
  * @return bool $is_guest_payment If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -929,8 +929,8 @@ 
  *
  * @return int $user_id User ID.
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -944,8 +944,8 @@ 
  *
  * @return int $payment->customer_id Donor ID.
  */
-function give_get_payment_donor_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_donor_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -959,8 +959,8 @@ 
  *
  * @return string $ip User IP.
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -974,8 +974,8 @@ 
  *
  * @return string $date The date the payment was completed.
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -989,8 +989,8 @@ 
  *
  * @return string $gateway Gateway.
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1004,8 +1004,8 @@ 
  *
  * @return string $currency The currency code.
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1019,10 +1019,10 @@ 
  *
  * @return string $currency The currency name.
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1034,8 +1034,8 @@ 
  *
  * @return string $key Donation key.
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1051,8 +1051,8 @@ 
  *
  * @return string $number Payment order number.
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1066,23 +1066,23 @@ 
  *
  * @return string      The formatted payment number.
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1096,17 +1096,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1115,24 +1115,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason.
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids',
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1141,13 +1141,13 @@ 
 		}
 	}// End if().
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1159,25 +1159,25 @@ 
  *
  * @return string The new Payment number without prefix and postfix.
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix.
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix.
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number.
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1195,16 +1195,16 @@ 
  *
  * @return string $amount Fully formatted donation amount.
  */
-function give_donation_amount( $donation, $format_args = array() ) {
+function give_donation_amount($donation, $format_args = array()) {
 	/* @var Give_Payment $donation */
-	if ( ! ( $donation instanceof Give_Payment ) ) {
-		$donation = new Give_Payment( absint( $donation ) );
+	if ( ! ($donation instanceof Give_Payment)) {
+		$donation = new Give_Payment(absint($donation));
 	}
 
 	$amount           = $donation->total;
 	$formatted_amount = $amount;
 
-	if ( is_bool( $format_args ) ) {
+	if (is_bool($format_args)) {
 		$format_args = array(
 			'currency' => (bool) $format_args,
 			'amount'   => (bool) $format_args,
@@ -1227,27 +1227,25 @@ 
 		)
 	);
 
-	if ( $format_args['amount'] || $format_args['currency'] ) {
+	if ($format_args['amount'] || $format_args['currency']) {
 
-		if ( $format_args['amount'] ) {
+		if ($format_args['amount']) {
 
 			$formatted_amount = give_format_amount(
 				$amount,
-				! is_array( $format_args['amount'] ) ?
+				! is_array($format_args['amount']) ?
 					array(
 						'sanitize' => false,
 						'currency' => $donation->currency,
-					) :
-					$format_args['amount']
+					) : $format_args['amount']
 			);
 		}
 
-		if ( $format_args['currency'] ) {
+		if ($format_args['currency']) {
 			$formatted_amount = give_currency_filter(
 				$formatted_amount,
-				! is_array( $format_args['currency'] ) ?
-					array( 'currency_code' => $donation->currency ) :
-					$format_args['currency']
+				! is_array($format_args['currency']) ?
+					array('currency_code' => $donation->currency) : $format_args['currency']
 			);
 		}
 	}
@@ -1262,7 +1260,7 @@ 
 	 * @param int    $donation_id      Donation ID.
 	 * @param string $type             Donation amount type.
 	 */
-	return apply_filters( 'give_donation_amount', (string) $formatted_amount, $amount, $donation, $format_args );
+	return apply_filters('give_donation_amount', (string) $formatted_amount, $amount, $donation, $format_args);
 }
 
 /**
@@ -1279,10 +1277,10 @@ 
  *
  * @return array Fully formatted payment subtotal.
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal, array( 'sanitize' => false ) ), array( 'currency_code' => give_get_payment_currency_code( $payment_id ) ) );
+	return give_currency_filter(give_format_amount($subtotal, array('sanitize' => false)), array('currency_code' => give_get_payment_currency_code($payment_id)));
 }
 
 /**
@@ -1296,8 +1294,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted).
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1311,8 +1309,8 @@ 
  *
  * @return string The donation ID.
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1327,15 +1325,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1348,10 +1346,10 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_donation_id_by_key( $key ) {
+function give_get_donation_id_by_key($key) {
 	global $wpdb;
 
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$meta_table = __give_v20_bc_table_details('payment');
 
 	$purchase = $wpdb->get_var(
 		$wpdb->prepare(
@@ -1367,7 +1365,7 @@ 
 		)
 	);
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1385,13 +1383,13 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$meta_table = __give_v20_bc_table_details('payment');
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT {$meta_table['column']['id']} FROM {$meta_table['name']} WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT {$meta_table['column']['id']} FROM {$meta_table['name']} WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1408,23 +1406,23 @@ 
  *
  * @return array $notes Donation Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10);
 
-	$notes = get_comments( array(
+	$notes = get_comments(array(
 		'post_id' => $payment_id,
 		'order'   => 'ASC',
 		'search'  => $search,
-	) );
+	));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1440,8 +1438,8 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
@@ -1453,14 +1451,14 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1469,7 +1467,7 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note',
 
-	) ) );
+	)));
 
 	/**
 	 * Fires after payment note inserted.
@@ -1480,7 +1478,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1495,8 +1493,8 @@ 
  *
  * @return bool True on success, false otherwise.
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
@@ -1508,9 +1506,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
 
-	$ret = wp_delete_comment( $comment_id, true );
+	$ret = wp_delete_comment($comment_id, true);
 
 	/**
 	 * Fires after donation note deleted.
@@ -1520,7 +1518,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1535,32 +1533,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = __( 'System', 'give' );
+		$user = __('System', 'give');
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id,
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" aria-label="' . __( 'Delete this donation note.', 'give' ) . '">' . __( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" aria-label="'.__('Delete this donation note.', 'give').'">'.__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1578,18 +1576,18 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+function give_hide_payment_notes($query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1601,15 +1599,15 @@ 
  *
  * @return array $clauses Updated comment clauses.
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1622,15 +1620,15 @@ 
  *
  * @return string $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( ' AND comment_type != %s', 'give_payment_note' );
+	$where .= $wpdb->prepare(' AND comment_type != %s', 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1644,32 +1642,32 @@ 
  *
  * @return array|object Array of comment counts.
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = Give_Cache::get_group( "comments-{$post_id}", 'counts' );
+	$stats = Give_Cache::get_group("comments-{$post_id}", 'counts');
 
-	if ( ! is_null( $stats ) ) {
+	if ( ! is_null($stats)) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1679,30 +1677,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed',
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals.
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	Give_Cache::set_group( "comments-{$post_id}", $stats, 'counts' );
+	Give_Cache::set_group("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1715,9 +1713,9 @@ 
  *
  * @return string $where Modified where clause.
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week.
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1737,13 +1735,13 @@ 
  *
  * @return string $form_title Returns the full title if $only_level is false, otherwise returns the levels title.
  */
-function give_get_donation_form_title( $donation, $args = array() ) {
+function give_get_donation_form_title($donation, $args = array()) {
 
-	if ( ! $donation instanceof Give_Payment ) {
-		$donation = new Give_Payment( $donation );
+	if ( ! $donation instanceof Give_Payment) {
+		$donation = new Give_Payment($donation);
 	}
 
-	if( ! $donation->ID ) {
+	if ( ! $donation->ID) {
 		return '';
 	}
 
@@ -1752,7 +1750,7 @@ 
 		'separator'  => '',
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$form_id     = $donation->form_id;
 	$price_id    = $donation->price_id;
@@ -1773,32 +1771,32 @@ 
 		, false
 	);
 
-	$form_title_html = Give_Cache::get_db_query( $cache_key );
+	$form_title_html = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $form_title_html ) ) {
-		if ( true === $only_level ) {
+	if (is_null($form_title_html)) {
+		if (true === $only_level) {
 			$form_title = '';
 		}
 
 		$form_title_html = $form_title;
 
-		if ( 'custom' === $price_id ) {
-			$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
-			$level_label        = ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
-		} elseif ( give_has_variable_prices( $form_id ) ) {
-			$level_label = give_get_price_option_name( $form_id, $price_id );
+		if ('custom' === $price_id) {
+			$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
+			$level_label        = ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
+		} elseif (give_has_variable_prices($form_id)) {
+			$level_label = give_get_price_option_name($form_id, $price_id);
 		}
 
 		// Only add separator if there is a form title.
 		if (
-			! empty( $form_title_html ) &&
-			! empty( $level_label )
+			! empty($form_title_html) &&
+			! empty($level_label)
 		) {
 			$form_title_html .= " {$separator} ";
 		}
 
 		$form_title_html .= "<span class=\"donation-level-text-wrap\">{$level_label}</span>";
-		Give_Cache::set_db_query( $cache_key, $form_title_html );
+		Give_Cache::set_db_query($cache_key, $form_title_html);
 	}
 
 	/**
@@ -1806,7 +1804,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_get_donation_form_title', $form_title_html, $donation->payment_meta, $donation );
+	return apply_filters('give_get_donation_form_title', $form_title_html, $donation->payment_meta, $donation);
 }
 
 /**
@@ -1819,19 +1817,19 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 	$price_id = null;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = give_get_meta( $form_id, '_give_donation_levels', true );
+		$levels = give_get_meta($form_id, '_give_donation_levels', true);
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = give_maybe_sanitize_amount( $level['_give_amount'] );
+			$level_amount = give_maybe_sanitize_amount($level['_give_amount']);
 
 			// Check that this indeed the recurring price.
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 				break;
@@ -1839,13 +1837,13 @@ 
 			}
 		}
 
-		if ( is_null( $price_id ) && give_is_custom_price_mode( $form_id ) ) {
+		if (is_null($price_id) && give_is_custom_price_mode($form_id)) {
 			$price_id = 'custom';
 		}
 	}
 
 	// Price ID must be numeric or string.
-	$price_id = ! is_numeric( $price_id ) && ! is_string( $price_id ) ? 0 : $price_id;
+	$price_id = ! is_numeric($price_id) && ! is_string($price_id) ? 0 : $price_id;
 
 	/**
 	 * Filter the price id
@@ -1855,7 +1853,7 @@ 
 	 * @param string $price_id
 	 * @param int    $form_id
 	 */
-	return apply_filters( 'give_get_price_id', $price_id, $form_id );
+	return apply_filters('give_get_price_id', $price_id, $form_id);
 }
 
 /**
@@ -1871,10 +1869,10 @@ 
  *
  * @return string
  */
-function give_get_form_dropdown( $args = array(), $echo = false ) {
-	$form_dropdown_html = Give()->html->forms_dropdown( $args );
+function give_get_form_dropdown($args = array(), $echo = false) {
+	$form_dropdown_html = Give()->html->forms_dropdown($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1891,17 +1889,17 @@ 
  *
  * @return string|bool
  */
-function give_get_form_variable_price_dropdown( $args = array(), $echo = false ) {
+function give_get_form_variable_price_dropdown($args = array(), $echo = false) {
 
 	// Check for give form id.
-	if ( empty( $args['id'] ) ) {
+	if (empty($args['id'])) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $args['id'] );
+	$form = new Give_Donate_Form($args['id']);
 
 	// Check if form has variable prices or not.
-	if ( ! $form->ID || ! $form->has_variable_prices() ) {
+	if ( ! $form->ID || ! $form->has_variable_prices()) {
 		return false;
 	}
 
@@ -1909,24 +1907,24 @@ 
 	$variable_price_options = array();
 
 	// Check if multi donation form support custom donation or not.
-	if ( $form->is_custom_price_mode() ) {
-		$variable_price_options['custom'] = _x( 'Custom', 'custom donation dropdown item', 'give' );
+	if ($form->is_custom_price_mode()) {
+		$variable_price_options['custom'] = _x('Custom', 'custom donation dropdown item', 'give');
 	}
 
 	// Get variable price and ID from variable price array.
-	foreach ( $variable_prices as $variable_price ) {
-		$variable_price_options[ $variable_price['_give_id']['level_id'] ] = ! empty( $variable_price['_give_text'] ) ? $variable_price['_give_text'] : give_currency_filter( give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) ) );
+	foreach ($variable_prices as $variable_price) {
+		$variable_price_options[$variable_price['_give_id']['level_id']] = ! empty($variable_price['_give_text']) ? $variable_price['_give_text'] : give_currency_filter(give_format_amount($variable_price['_give_amount'], array('sanitize' => false)));
 	}
 
 	// Update options.
-	$args = array_merge( $args, array(
+	$args = array_merge($args, array(
 		'options' => $variable_price_options,
-	) );
+	));
 
 	// Generate select html.
-	$form_dropdown_html = Give()->html->select( $args );
+	$form_dropdown_html = Give()->html->select($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1945,14 +1943,14 @@ 
  *
  * @return string
  */
-function give_get_payment_meta_price_id( $payment_meta ) {
+function give_get_payment_meta_price_id($payment_meta) {
 
-	if ( isset( $payment_meta['give_price_id'] ) ) {
+	if (isset($payment_meta['give_price_id'])) {
 		$price_id = $payment_meta['give_price_id'];
-	} elseif ( isset( $payment_meta['price_id'] ) ) {
+	} elseif (isset($payment_meta['price_id'])) {
 		$price_id = $payment_meta['price_id'];
 	} else {
-		$price_id = give_get_price_id( $payment_meta['give_form_id'], $payment_meta['price'] );
+		$price_id = give_get_price_id($payment_meta['give_form_id'], $payment_meta['price']);
 	}
 
 	/**
@@ -1963,6 +1961,6 @@ 
 	 * @param string $price_id
 	 * @param array  $payment_meta
 	 */
-	return apply_filters( 'give_get_payment_meta_price_id', $price_id, $payment_meta );
+	return apply_filters('give_get_payment_meta_price_id', $price_id, $payment_meta);
 
 }

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

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,7 +50,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->paymentmeta = $this->table_name = $wpdb->prefix . 'give_paymentmeta';
+		$wpdb->paymentmeta = $this->table_name = $wpdb->prefix.'give_paymentmeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -99,10 +99,10 @@ 
 			KEY meta_key (meta_key)
 			) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 	/**
@@ -113,6 +113,6 @@ 
 	 * @return bool
 	 */
 	protected function is_custom_meta_table_active() {
-		return give_has_upgrade_completed( 'v20_move_metadata_into_new_table' );
+		return give_has_upgrade_completed('v20_move_metadata_into_new_table');
 	}
 }

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

Spacing +31 added lines, -31 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.
@@ -70,25 +70,25 @@ 
 			// 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}_settings_{$this->id}_page", array( $this, 'output' ) );
+			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}_settings_{$this->id}_page", array($this, 'output'));
 
 			// Output section only if exist.
 			$sections = $this->get_sections();
-			if ( ! empty( $sections ) ) {
-				add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array(
+			if ( ! empty($sections)) {
+				add_action("{$this->current_setting_page}_sections_{$this->id}_page", array(
 					$this,
 					'output_sections',
-				) );
+				));
 			}
 
 			// Save hide button by default.
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Enable saving feature.
-			if ( $this->enable_save ) {
-				add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
+			if ($this->enable_save) {
+				add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
 			}
 		}
 
@@ -113,7 +113,7 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			return $this->default_tab;
 		}
 
@@ -126,8 +126,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;
 		}
@@ -146,7 +146,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;
@@ -159,7 +159,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());
 		}
 
 		/**
@@ -176,39 +176,39 @@ 
 			$sections = $this->get_sections();
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
 			// 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;
 			}
 
-			if ( is_null( $this->current_setting_page ) ) {
+			if (is_null($this->current_setting_page)) {
 				$this->current_setting_page = give_get_current_setting_page();
 			}
 
 			$section_list = array();
-			foreach ( $sections as $id => $label ) {
+			foreach ($sections as $id => $label) {
 				/**
 				 * Fire the filter to hide particular section on tab.
 				 *
 				 * @since 2.0
 				 */
-				if ( apply_filters( "give_hide_section_{$id}_on_{$this->id}_page", false, $sections, $this->id ) ) {
+				if (apply_filters("give_hide_section_{$id}_on_{$this->id}_page", false, $sections, $this->id)) {
 					continue;
 				}
 
-				$section_list[] = '<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>';
+				$section_list[] = '<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>';
 			}
 
 			echo sprintf(
 				'<ul class="subsubsub">%s</ul><br class="clear" /><hr>',
-				implode( ' | </li>', $section_list )
+				implode(' | </li>', $section_list)
 			);
 		}
 
@@ -221,13 +221,13 @@ 
 		 * @return void
 		 */
 		public function output() {
-			if ( $this->enable_save ) {
+			if ($this->enable_save) {
 				$GLOBALS['give_hide_save_button'] = false;
 			}
 
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -240,14 +240,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);
 		}
 
 		/**
@@ -264,11 +264,11 @@ 
 			$section         = $this->get_sections();
 			$current_section = give_get_current_setting_section();
 
-			if ( array_key_exists( $current_section, $section ) ) {
-				$heading[] = $section[ $current_section ];
+			if (array_key_exists($current_section, $section)) {
+				$heading[] = $section[$current_section];
 			}
 
-			return array_unique( $heading );
+			return array_unique($heading);
 		}
 
 		/**
@@ -282,7 +282,7 @@ 
 		public function get_heading_html() {
 			return sprintf(
 				'<h1 class="wp-heading-inline">%s</h1><hr class="wp-header-end">',
-				implode( ' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading() )
+				implode(' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading())
 			);
 		}
 	}