WordImpress / Give

includes/class-give-session.php

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -135,7 +135,7 @@ 
 	 * @return Give_Session
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->__setup();
 		}
@@ -150,25 +150,25 @@ 
 	 * @access public
 	 */
 	private function __setup() {  // @codingStandardsIgnoreLine
-		$this->exp_option = give_get_option( 'session_lifetime' );
-		$this->exp_option = ! empty( $this->exp_option )
+		$this->exp_option = give_get_option('session_lifetime');
+		$this->exp_option = ! empty($this->exp_option)
 			? $this->exp_option
 			: 30 * 60 * 24; // Default expiration time is 12 hours
 
 		$this->set_cookie_name();
-		$this->cookie_name = $this->get_cookie_name( 'session' );
+		$this->cookie_name = $this->get_cookie_name('session');
 		$cookie            = $this->get_session_cookie();
 
-		if ( ! empty( $cookie ) ) {
+		if ( ! empty($cookie)) {
 			$this->donor_id           = $cookie[0];
 			$this->session_expiration = $cookie[1];
 			$this->session_expiring   = $cookie[2];
 			$this->has_cookie         = true;
 
 			// Update session if its close to expiring.
-			if ( time() > $this->session_expiring ) {
+			if (time() > $this->session_expiring) {
 				$this->set_expiration_time();
-				Give()->session_db->update_session_timestamp( $this->donor_id, $this->session_expiration );
+				Give()->session_db->update_session_timestamp($this->donor_id, $this->session_expiration);
 			}
 
 			// Load session.
@@ -178,17 +178,17 @@ 
 			$this->generate_donor_id();
 		}
 
-		add_action( 'give_process_donation_after_validation', array( $this, 'maybe_start_session' ) );
+		add_action('give_process_donation_after_validation', array($this, 'maybe_start_session'));
 
-		add_action( 'shutdown', array( $this, 'save_data' ), 20 );
-		add_action( 'wp_logout', array( $this, 'destroy_session' ) );
+		add_action('shutdown', array($this, 'save_data'), 20);
+		add_action('wp_logout', array($this, 'destroy_session'));
 
-		if ( ! is_user_logged_in() ) {
-			add_filter( 'nonce_user_logged_out', array( $this, '__nonce_user_logged_out' ) );
+		if ( ! is_user_logged_in()) {
+			add_filter('nonce_user_logged_out', array($this, '__nonce_user_logged_out'));
 		}
 
 		// Remove old sessions.
-		Give_Cron::add_daily_event( array( $this, '__cleanup_sessions' ) );
+		Give_Cron::add_daily_event(array($this, '__cleanup_sessions'));
 	}
 
 	/**
@@ -200,7 +200,7 @@ 
 	 * @return array
 	 */
 	public function get_session_data() {
-		return $this->has_session() ? (array) Give()->session_db->get_session( $this->donor_id, array() ) : array();
+		return $this->has_session() ? (array) Give()->session_db->get_session($this->donor_id, array()) : array();
 	}
 
 
@@ -214,27 +214,27 @@ 
 	 */
 	public function get_session_cookie() {
 		$session      = array();
-		$cookie_value = isset( $_COOKIE[ $this->cookie_name ] ) ? give_clean( $_COOKIE[ $this->cookie_name ] ) : false; // @codingStandardsIgnoreLine.
+		$cookie_value = isset($_COOKIE[$this->cookie_name]) ? give_clean($_COOKIE[$this->cookie_name]) : false; // @codingStandardsIgnoreLine.
 
-		if ( empty( $cookie_value ) || ! is_string( $cookie_value ) ) {
+		if (empty($cookie_value) || ! is_string($cookie_value)) {
 			return $session;
 		}
 
-		list( $donor_id, $session_expiration, $session_expiring, $cookie_hash ) = explode( '||', $cookie_value );
+		list($donor_id, $session_expiration, $session_expiring, $cookie_hash) = explode('||', $cookie_value);
 
-		if ( empty( $donor_id ) ) {
+		if (empty($donor_id)) {
 			return $session;
 		}
 
 		// Validate hash.
-		$to_hash = $donor_id . '|' . $session_expiration;
-		$hash    = hash_hmac( 'md5', $to_hash, wp_hash( $to_hash ) );
+		$to_hash = $donor_id.'|'.$session_expiration;
+		$hash    = hash_hmac('md5', $to_hash, wp_hash($to_hash));
 
-		if ( empty( $cookie_hash ) || ! hash_equals( $hash, $cookie_hash ) ) {
+		if (empty($cookie_hash) || ! hash_equals($hash, $cookie_hash)) {
 			return $session;
 		}
 
-		return array( $donor_id, $session_expiration, $session_expiring, $cookie_hash );
+		return array($donor_id, $session_expiration, $session_expiring, $cookie_hash);
 	}
 
 
@@ -259,8 +259,8 @@ 
 	 * @return string Cookie name.
 	 */
 	private function set_cookie_name() {
-		$this->cookie_name       = apply_filters( 'give_session_cookie', 'wp_give_session_' . COOKIEHASH );
-		$this->nonce_cookie_name = 'wp_give_session_reset_nonce_' . COOKIEHASH;
+		$this->cookie_name       = apply_filters('give_session_cookie', 'wp_give_session_'.COOKIEHASH);
+		$this->nonce_cookie_name = 'wp_give_session_reset_nonce_'.COOKIEHASH;
 	}
 
 	/**
@@ -276,10 +276,10 @@ 
 	 *
 	 * @return string|array      Session variable.
 	 */
-	public function get( $key, $default = false ) {
-		$key = sanitize_key( $key );
+	public function get($key, $default = false) {
+		$key = sanitize_key($key);
 
-		return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : $default;
+		return isset($this->session[$key]) ? maybe_unserialize($this->session[$key]) : $default;
 	}
 
 	/**
@@ -293,13 +293,13 @@ 
 	 *
 	 * @return string        Session variable.
 	 */
-	public function set( $key, $value ) {
-		if ( $value !== $this->get( $key ) ) {
-			$this->session[ sanitize_key( $key ) ] = maybe_serialize( $value );
+	public function set($key, $value) {
+		if ($value !== $this->get($key)) {
+			$this->session[sanitize_key($key)] = maybe_serialize($value);
 			$this->session_data_changed            = true;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -313,17 +313,17 @@ 
 	 *
 	 * @param bool $set Flag to check if set cookie or not.
 	 */
-	public function set_session_cookies( $set ) {
-		if ( $set ) {
+	public function set_session_cookies($set) {
+		if ($set) {
 			$this->set_expiration_time();
 
-			$to_hash          = $this->donor_id . '|' . $this->session_expiration;
-			$cookie_hash      = hash_hmac( 'md5', $to_hash, wp_hash( $to_hash ) );
-			$cookie_value     = $this->donor_id . '||' . $this->session_expiration . '||' . $this->session_expiring . '||' . $cookie_hash;
+			$to_hash          = $this->donor_id.'|'.$this->session_expiration;
+			$cookie_hash      = hash_hmac('md5', $to_hash, wp_hash($to_hash));
+			$cookie_value     = $this->donor_id.'||'.$this->session_expiration.'||'.$this->session_expiring.'||'.$cookie_hash;
 			$this->has_cookie = true;
 
-			give_setcookie( $this->cookie_name, $cookie_value, $this->session_expiration, apply_filters( 'give_session_use_secure_cookie', false ) );
-			give_setcookie( $this->nonce_cookie_name, '1', $this->session_expiration, apply_filters( 'give_session_use_secure_cookie', false ) );
+			give_setcookie($this->cookie_name, $cookie_value, $this->session_expiration, apply_filters('give_session_use_secure_cookie', false));
+			give_setcookie($this->nonce_cookie_name, '1', $this->session_expiration, apply_filters('give_session_use_secure_cookie', false));
 		}
 	}
 
@@ -338,8 +338,8 @@ 
 	 * @return int
 	 */
 	public function set_expiration_time() {
-		$this->session_expiring   = time() + intval( apply_filters( 'give_session_expiring', ( $this->exp_option - 3600 ) ) ); // Default 11 Hours.
-		$this->session_expiration = time() + intval( apply_filters( 'give_session_expiration', $this->exp_option ) ); // Default 12 Hours.
+		$this->session_expiring   = time() + intval(apply_filters('give_session_expiring', ($this->exp_option - 3600))); // Default 11 Hours.
+		$this->session_expiration = time() + intval(apply_filters('give_session_expiration', $this->exp_option)); // Default 12 Hours.
 
 		return $this->session_expiration;
 	}
@@ -371,10 +371,10 @@ 
 	public function maybe_start_session() {
 		if (
 			! headers_sent()
-			&& empty( $this->session )
+			&& empty($this->session)
 			&& ! $this->has_cookie
 		) {
-			$this->set_session_cookies( true );
+			$this->set_session_cookies(true);
 		}
 	}
 
@@ -387,10 +387,10 @@ 
 	 * @access public
 	 */
 	public function generate_donor_id() {
-		require_once ABSPATH . 'wp-includes/class-phpass.php';
+		require_once ABSPATH.'wp-includes/class-phpass.php';
 
-		$hasher         = new PasswordHash( 8, false );
-		$this->donor_id = md5( $hasher->get_random_bytes( 32 ) );
+		$hasher         = new PasswordHash(8, false);
+		$this->donor_id = md5($hasher->get_random_bytes(32));
 	}
 
 	/**
@@ -401,14 +401,14 @@ 
 	 */
 	public function save_data() {
 		// Dirty if something changed - prevents saving nothing new.
-		if ( $this->session_data_changed && $this->has_session() ) {
+		if ($this->session_data_changed && $this->has_session()) {
 			global $wpdb;
 
 			Give()->session_db->__replace(
 				Give()->session_db->table_name,
 				array(
 					'session_key'    => $this->donor_id,
-					'session_value'  => maybe_serialize( $this->session ),
+					'session_value'  => maybe_serialize($this->session),
 					'session_expiry' => $this->session_expiration,
 				),
 				array(
@@ -429,10 +429,10 @@ 
 	 * @access public
 	 */
 	public function destroy_session() {
-		give_setcookie( $this->cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters( 'give_session_use_secure_cookie', false ) );
-		give_setcookie( $this->nonce_cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters( 'give_session_use_secure_cookie', false ) );
+		give_setcookie($this->cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters('give_session_use_secure_cookie', false));
+		give_setcookie($this->nonce_cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters('give_session_use_secure_cookie', false));
 
-		Give()->session_db->delete_session( $this->donor_id );
+		Give()->session_db->delete_session($this->donor_id);
 
 		$this->session              = array();
 		$this->session_data_changed = false;
@@ -448,10 +448,10 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_delete_nonce_cookie(){
+	public function is_delete_nonce_cookie() {
 		$value = false;
 
-		if ( Give()->session->has_session() ) {
+		if (Give()->session->has_session()) {
 			$value = true;
 		}
 
@@ -468,10 +468,10 @@ 
 	 *
 	 * @return string Cookie name
 	 */
-	public function get_cookie_name( $type = '' ) {
+	public function get_cookie_name($type = '') {
 		$name = '';
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'nonce':
 				$name = $this->nonce_cookie_name;
 				break;
@@ -495,7 +495,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function __nonce_user_logged_out( $uid ) {
+	public function __nonce_user_logged_out($uid) {
 		return $this->has_session() && $this->donor_id ? $this->donor_id : $uid;
 	}
 
@@ -541,7 +541,7 @@ 
 	 */
 	public function set_expiration_variant_time() {
 
-		return ( ! empty( $this->exp_option ) ? ( intval( $this->exp_option ) - 3600 ) : 30 * 60 * 23 );
+		return ( ! empty($this->exp_option) ? (intval($this->exp_option) - 3600) : 30 * 60 * 23);
 	}
 
 	/**
@@ -558,9 +558,9 @@ 
 	public function use_php_sessions() {
 		$ret = false;
 
-		give_doing_it_wrong( __FUNCTION__, __( 'We are using database session logic instead of PHP session', 'give' ), '2.2.0' );
+		give_doing_it_wrong(__FUNCTION__, __('We are using database session logic instead of PHP session', 'give'), '2.2.0');
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -578,10 +578,10 @@ 
 
 		$start_session = true;
 
-		give_doing_it_wrong( __FUNCTION__, __( 'We are using database session logic instead of PHP session', 'give' ), '2.2.0' );
+		give_doing_it_wrong(__FUNCTION__, __('We are using database session logic instead of PHP session', 'give'), '2.2.0');
 
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {  // @codingStandardsIgnoreLine
+		if ( ! empty($_SERVER['REQUEST_URI'])) {  // @codingStandardsIgnoreLine
 
 			$blacklist = apply_filters(
 				'give_session_start_uri_blacklist', array(
@@ -594,19 +594,19 @@ 
 					'comments/feed/',
 				)
 			);
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' ); // // @codingStandardsIgnoreLine
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist, true ) ) {
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/'); // // @codingStandardsIgnoreLine
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist, true)) {
 				$start_session = false;
 			}
-			if ( false !== strpos( $uri, 'feed=' ) ) {
+			if (false !== strpos($uri, 'feed=')) {
 				$start_session = false;
 			}
-			if ( is_admin() ) {
+			if (is_admin()) {
 				$start_session = false;
 			}
 		}
 
-		return apply_filters( 'give_start_session', $start_session );
+		return apply_filters('give_start_session', $start_session);
 	}
 }

includes/class-give-db-logs.php

Spacing +71 added lines, -73 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_logs';
+		$this->table_name  = $wpdb->prefix.'give_logs';
 		$this->primary_key = 'ID';
 		$this->version     = '1.0';
 
@@ -74,8 +74,8 @@ 
 	 * @return array  Default column values.
 	 */
 	public function get_column_defaults() {
-		$log_create_date     = current_time( 'mysql', 0 );
-		$log_create_date_gmt = get_gmt_from_date( $log_create_date );
+		$log_create_date     = current_time('mysql', 0);
+		$log_create_date_gmt = get_gmt_from_date($log_create_date);
 
 		return array(
 			'ID'           => 0,
@@ -98,39 +98,39 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 		// Valid table columns.
-		$table_columns = array_keys( $this->get_columns() );
+		$table_columns = array_keys($this->get_columns());
 
 		// Filter data.
-		foreach ( $data as $table_column => $column_data ) {
-			if ( ! in_array( $table_column, $table_columns ) ) {
-				unset( $data[ $table_column ] );
+		foreach ($data as $table_column => $column_data) {
+			if ( ! in_array($table_column, $table_columns)) {
+				unset($data[$table_column]);
 			}
 		}
 
 		// Set default values.
-		$current_log_data = wp_parse_args( $data, $this->get_column_defaults() );
+		$current_log_data = wp_parse_args($data, $this->get_column_defaults());
 
 		// Log parent should be an int.
-		$current_log_data['log_parent'] = absint( $current_log_data['log_parent'] );
+		$current_log_data['log_parent'] = absint($current_log_data['log_parent']);
 
 		// Get log.
-		$existing_log = $this->get_log_by( $current_log_data['ID'] );
+		$existing_log = $this->get_log_by($current_log_data['ID']);
 
 		// Update an existing log.
-		if ( $existing_log ) {
+		if ($existing_log) {
 
 			// Create new log data from existing and new log data.
-			$current_log_data = array_merge( $current_log_data, $existing_log );
+			$current_log_data = array_merge($current_log_data, $existing_log);
 
 			// Update log data.
-			$this->update( $current_log_data['ID'], $current_log_data );
+			$this->update($current_log_data['ID'], $current_log_data);
 
 			$log_id = $current_log_data['ID'];
 
 		} else {
-			$log_id = $this->insert( $current_log_data, 'log' );
+			$log_id = $this->insert($current_log_data, 'log');
 		}
 
 		return $log_id;
@@ -148,20 +148,20 @@ 
 	 *
 	 * @return bool|null|array
 	 */
-	public function get_log_by( $log_id = 0, $by = 'id' ) {
+	public function get_log_by($log_id = 0, $by = 'id') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 		$log = null;
 
 		// Make sure $log_id is int.
-		$log_id = absint( $log_id );
+		$log_id = absint($log_id);
 
 		// Bailout.
-		if ( empty( $log_id ) ) {
+		if (empty($log_id)) {
 			return null;
 		}
 
-		switch ( $by ) {
+		switch ($by) {
 			case 'id':
 				$log = $wpdb->get_row(
 					$wpdb->prepare(
@@ -173,7 +173,7 @@ 
 				break;
 
 			default:
-				$log = apply_filters( "give_get_log_by_{$by}", $log, $log_id );
+				$log = apply_filters("give_get_log_by_{$by}", $log, $log_id);
 		}
 
 		return $log;
@@ -189,14 +189,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_logs( $args = array() ) {
+	public function get_logs($args = array()) {
 		global $wpdb;
-		$sql_query = $this->get_sql( $args );
+		$sql_query = $this->get_sql($args);
 
 		// Get log.
-		if ( ! ( $logs = Give_Cache::get( 'give_logs', true, $sql_query ) ) ) {
-			$logs = $wpdb->get_results( $sql_query );
-			Give_Cache::set( 'give_logs', $logs, 3600, true, $sql_query );
+		if ( ! ($logs = Give_Cache::get('give_logs', true, $sql_query))) {
+			$logs = $wpdb->get_results($sql_query);
+			Give_Cache::set('give_logs', $logs, 3600, true, $sql_query);
 		}
 
 		return $logs;
@@ -213,21 +213,21 @@ 
 	 *
 	 * @return int
 	 */
-	public function count( $args = array() ) {
+	public function count($args = array()) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 		$args['number'] = - 1;
 		$args['fields'] = 'ID';
 		$args['count']  = true;
 
-		$sql_query = $this->get_sql( $args );
+		$sql_query = $this->get_sql($args);
 
-		if ( ! ( $count = Give_Cache::get( 'give_logs_count', true, $sql_query ) ) ) {
-			$count = $wpdb->get_var( $sql_query );
-			Give_Cache::set( 'give_logs_count', $count, 3600, true, $args );
+		if ( ! ($count = Give_Cache::get('give_logs_count', true, $sql_query))) {
+			$count = $wpdb->get_var($sql_query);
+			Give_Cache::set('give_logs_count', $count, 3600, true, $args);
 		}
 
-		return absint( $count );
+		return absint($count);
 	}
 
 	/**
@@ -253,10 +253,10 @@ 
         PRIMARY KEY  (ID)
         ) {$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, false );
+		update_option($this->table_name.'_db_version', $this->version, false);
 	}
 
 
@@ -270,7 +270,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_sql( $args = array() ) {
+	public function get_sql($args = array()) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
@@ -284,12 +284,12 @@ 
 			'count'   => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		// validate params.
-		$this->validate_params( $args );
+		$this->validate_params($args);
 
-		if ( $args['number'] < 1 ) {
+		if ($args['number'] < 1) {
 			$args['number'] = 99999999999;
 		}
 
@@ -297,78 +297,78 @@ 
 		$where = '';
 
 		// Get sql query for meta.
-		if ( ! empty( $args['meta_query'] ) ) {
-			$meta_query_object = new WP_Meta_Query( $args['meta_query'] );
-			$meta_query        = $meta_query_object->get_sql( 'log', $this->table_name, 'id' );
-			$where             = implode( '', $meta_query );
+		if ( ! empty($args['meta_query'])) {
+			$meta_query_object = new WP_Meta_Query($args['meta_query']);
+			$meta_query        = $meta_query_object->get_sql('log', $this->table_name, 'id');
+			$where             = implode('', $meta_query);
 		}
 
 		$where .= ' WHERE 1=1 ';
 
 		// Set offset.
-		if ( empty( $args['offset'] ) && ( 0 < $args['paged'] ) ) {
-			$args['offset'] = $args['number'] * ( $args['paged'] - 1 );
+		if (empty($args['offset']) && (0 < $args['paged'])) {
+			$args['offset'] = $args['number'] * ($args['paged'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$this->table_name}.*";
-		if ( is_string( $args['fields'] ) && ( 'all' !== $args['fields'] ) ) {
+		if (is_string($args['fields']) && ('all' !== $args['fields'])) {
 			$fields = "{$this->table_name}.{$args['fields']}";
 		}
 
 		// Set count.
-		if ( $args['count'] ) {
+		if ($args['count']) {
 			$fields = "COUNT({$fields})";
 		}
 
 		// Specific logs.
-		if ( ! empty( $args['ID'] ) ) {
+		if ( ! empty($args['ID'])) {
 
-			if ( ! is_array( $args['ID'] ) ) {
-				$args['ID'] = explode( ',', $args['ID'] );
+			if ( ! is_array($args['ID'])) {
+				$args['ID'] = explode(',', $args['ID']);
 			}
-			$log_ids = implode( ',', array_map( 'intval', $args['ID'] ) );
+			$log_ids = implode(',', array_map('intval', $args['ID']));
 
 			$where .= " AND {$this->table_name}.ID IN( {$log_ids} ) ";
 		}
 
 		// Logs created for a specific date or in a date range
-		if ( ! empty( $args['date_query'] ) ) {
-			$date_query_object = new WP_Date_Query( $args['date_query'], "{$this->table_name}.log_date" );
-			$where             .= $date_query_object->get_sql();
+		if ( ! empty($args['date_query'])) {
+			$date_query_object = new WP_Date_Query($args['date_query'], "{$this->table_name}.log_date");
+			$where .= $date_query_object->get_sql();
 		}
 
 		// Logs create for specific parent.
-		if ( ! empty( $args['log_parent'] ) ) {
-			if ( ! is_array( $args['log_parent'] ) ) {
-				$args['log_parent'] = explode( ',', $args['log_parent'] );
+		if ( ! empty($args['log_parent'])) {
+			if ( ! is_array($args['log_parent'])) {
+				$args['log_parent'] = explode(',', $args['log_parent']);
 			}
-			$parent_ids = implode( ',', array_map( 'intval', $args['log_parent'] ) );
+			$parent_ids = implode(',', array_map('intval', $args['log_parent']));
 
 			$where .= " AND {$this->table_name}.log_parent IN( {$parent_ids} ) ";
 		}
 
 		// Logs create for specific type.
 		// is_array check is for backward compatibility.
-		if ( ! empty( $args['log_type'] ) && ! is_array( $args['log_type'] ) ) {
-			if ( ! is_array( $args['log_type'] ) ) {
-				$args['log_type'] = explode( ',', $args['log_type'] );
+		if ( ! empty($args['log_type']) && ! is_array($args['log_type'])) {
+			if ( ! is_array($args['log_type'])) {
+				$args['log_type'] = explode(',', $args['log_type']);
 			}
 
-			$log_types = implode( '\',\'', array_map( 'trim', $args['log_type'] ) );
+			$log_types = implode('\',\'', array_map('trim', $args['log_type']));
 
 			$where .= " AND {$this->table_name}.log_type IN( '{$log_types}' ) ";
 		}
 
-		$args['orderby'] = ! array_key_exists( $args['orderby'], $this->get_columns() ) ? 'log_date' : $args['orderby'];
+		$args['orderby'] = ! array_key_exists($args['orderby'], $this->get_columns()) ? 'log_date' : $args['orderby'];
 
-		$args['orderby'] = esc_sql( $args['orderby'] );
-		$args['order']   = esc_sql( $args['order'] );
+		$args['orderby'] = esc_sql($args['orderby']);
+		$args['order']   = esc_sql($args['order']);
 
 		return $wpdb->prepare(
 			"SELECT {$fields} FROM {$this->table_name} {$where} ORDER BY {$this->table_name}.{$args['orderby']} {$args['order']} LIMIT %d,%d;",
-			absint( $args['offset'] ),
-			absint( $args['number'] )
+			absint($args['offset']),
+			absint($args['number'])
 		);
 	}
 
@@ -383,13 +383,11 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function validate_params( &$args ) {
+	private function validate_params(&$args) {
 		// fields params
 		$args['fields'] = 'ids' === $args['fields'] ?
-			'ID' :
-			$args['fields'];
-		$args['fields'] = array_key_exists( $args['fields'], $this->get_columns() ) ?
-			$args['fields'] :
-			'all';
+			'ID' : $args['fields'];
+		$args['fields'] = array_key_exists($args['fields'], $this->get_columns()) ?
+			$args['fields'] : 'all';
 	}
 }

includes/user-functions.php

Spacing +136 added lines, -136 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,54 +30,54 @@ 
  *
  * @return bool|array List of all user donations.
  */
-function give_get_users_donations( $user = 0, $number = 20, $pagination = false, $status = 'complete' ) {
+function give_get_users_donations($user = 0, $number = 20, $pagination = false, $status = 'complete') {
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( 0 === $user && ! Give()->email_access->token_exists ) {
+	if (0 === $user && ! Give()->email_access->token_exists) {
 		return false;
 	}
 
-	$status = ( 'complete' === $status ) ? 'publish' : $status;
+	$status = ('complete' === $status) ? 'publish' : $status;
 	$paged = 1;
 
-	if ( $pagination ) {
-		if ( get_query_var( 'paged' ) ) {
-			$paged = get_query_var( 'paged' );
-		} elseif ( get_query_var( 'page' ) ) {
-			$paged = get_query_var( 'page' );
+	if ($pagination) {
+		if (get_query_var('paged')) {
+			$paged = get_query_var('paged');
+		} elseif (get_query_var('page')) {
+			$paged = get_query_var('page');
 		}
 	}
 
-	$args = apply_filters( 'give_get_users_donations_args', array(
+	$args = apply_filters('give_get_users_donations_args', array(
 		'user'    => $user,
 		'number'  => $number,
 		'status'  => $status,
 		'orderby' => 'date',
-	) );
+	));
 
-	if ( $pagination ) {
+	if ($pagination) {
 		$args['page'] = $paged;
 	} else {
 		$args['nopaging'] = true;
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
-	$donor   = new Give_Donor( $user, $by_user_id );
+	$by_user_id = is_numeric($user) ? true : false;
+	$donor = new Give_Donor($user, $by_user_id);
 
-	if ( ! empty( $donor->payment_ids ) ) {
+	if ( ! empty($donor->payment_ids)) {
 
-		unset( $args['user'] );
-		$args['post__in'] = array_map( 'absint', explode( ',', $donor->payment_ids ) );
+		unset($args['user']);
+		$args['post__in'] = array_map('absint', explode(',', $donor->payment_ids));
 
 	}
 
-	$donations = give_get_payments( apply_filters( 'give_get_users_donations_args', $args ) );
+	$donations = give_get_payments(apply_filters('give_get_users_donations_args', $args));
 
 	// No donations.
-	if ( ! $donations ) {
+	if ( ! $donations) {
 		return false;
 	}
 
@@ -96,65 +96,65 @@ 
  *
  * @return bool|object List of unique forms donated by user
  */
-function give_get_users_completed_donations( $user = 0, $status = 'complete' ) {
-	if ( empty( $user ) ) {
+function give_get_users_completed_donations($user = 0, $status = 'complete') {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		return false;
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
+	$by_user_id = is_numeric($user) ? true : false;
 
-	$donor = new Give_Donor( $user, $by_user_id );
+	$donor = new Give_Donor($user, $by_user_id);
 
-	if ( empty( $donor->payment_ids ) ) {
+	if (empty($donor->payment_ids)) {
 		return false;
 	}
 
 	// Get all the items donated.
-	$payment_ids    = array_reverse( explode( ',', $donor->payment_ids ) );
-	$limit_payments = apply_filters( 'give_users_completed_donations_payments', 50 );
-	if ( ! empty( $limit_payments ) ) {
-		$payment_ids = array_slice( $payment_ids, 0, $limit_payments );
+	$payment_ids    = array_reverse(explode(',', $donor->payment_ids));
+	$limit_payments = apply_filters('give_users_completed_donations_payments', 50);
+	if ( ! empty($limit_payments)) {
+		$payment_ids = array_slice($payment_ids, 0, $limit_payments);
 	}
 	$donation_data = array();
-	foreach ( $payment_ids as $payment_id ) {
-		$donation_data[] = give_get_payment_meta( $payment_id );
+	foreach ($payment_ids as $payment_id) {
+		$donation_data[] = give_get_payment_meta($payment_id);
 	}
 
-	if ( empty( $donation_data ) ) {
+	if (empty($donation_data)) {
 		return false;
 	}
 
 	// Grab only the post ids "form_id" of the forms donated on this order.
 	$completed_donations_ids = array();
-	foreach ( $donation_data as $donation_meta ) {
-		$completed_donations_ids[] = isset( $donation_meta['form_id'] ) ? $donation_meta['form_id'] : '';
+	foreach ($donation_data as $donation_meta) {
+		$completed_donations_ids[] = isset($donation_meta['form_id']) ? $donation_meta['form_id'] : '';
 	}
 
-	if ( empty( $completed_donations_ids ) ) {
+	if (empty($completed_donations_ids)) {
 		return false;
 	}
 
 	// Only include each donation once.
-	$form_ids = array_unique( $completed_donations_ids );
+	$form_ids = array_unique($completed_donations_ids);
 
 	// Make sure we still have some products and a first item.
-	if ( empty( $form_ids ) || ! isset( $form_ids[0] ) ) {
+	if (empty($form_ids) || ! isset($form_ids[0])) {
 		return false;
 	}
 
-	$post_type = get_post_type( $form_ids[0] );
+	$post_type = get_post_type($form_ids[0]);
 
-	$args = apply_filters( 'give_get_users_completed_donations_args', array(
+	$args = apply_filters('give_get_users_completed_donations_args', array(
 		'include'        => $form_ids,
 		'post_type'      => $post_type,
-		'posts_per_page' => - 1,
-	) );
+		'posts_per_page' => -1,
+	));
 
-	return apply_filters( 'give_users_completed_donations_list', get_posts( $args ) );
+	return apply_filters('give_users_completed_donations_list', get_posts($args));
 }
 
 
@@ -170,12 +170,12 @@ 
  *
  * @return bool True if has donated, false other wise.
  */
-function give_has_donations( $user_id = null ) {
-	if ( empty( $user_id ) ) {
+function give_has_donations($user_id = null) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	if ( give_get_users_donations( $user_id, 1 ) ) {
+	if (give_get_users_donations($user_id, 1)) {
 		return true; // User has at least one donation.
 	}
 
@@ -196,23 +196,23 @@ 
  *
  * @return array
  */
-function give_get_donation_stats_by_user( $user = '' ) {
+function give_get_donation_stats_by_user($user = '') {
 
 	$field = '';
 
-	if ( is_email( $user ) ) {
+	if (is_email($user)) {
 		$field = 'email';
-	} elseif ( is_numeric( $user ) ) {
+	} elseif (is_numeric($user)) {
 		$field = 'user_id';
 	}
 
-	$stats    = array();
-	$donor = Give()->donors->get_donor_by( $field, $user );
+	$stats = array();
+	$donor = Give()->donors->get_donor_by($field, $user);
 
-	if ( $donor ) {
-		$donor = new Give_Donor( $donor->id );
-		$stats['purchases']   = absint( $donor->purchase_count );
-		$stats['total_spent'] = give_maybe_sanitize_amount( $donor->get_total_donation_amount() );
+	if ($donor) {
+		$donor = new Give_Donor($donor->id);
+		$stats['purchases']   = absint($donor->purchase_count);
+		$stats['total_spent'] = give_maybe_sanitize_amount($donor->get_total_donation_amount());
 	}
 
 	/**
@@ -220,7 +220,7 @@ 
 	 *
 	 * @since 1.7
 	 */
-	$stats = (array) apply_filters( 'give_donation_stats_by_user', $stats, $user );
+	$stats = (array) apply_filters('give_donation_stats_by_user', $stats, $user);
 
 	return $stats;
 }
@@ -238,21 +238,21 @@ 
  *
  * @return int The total number of donations.
  */
-function give_count_donations_of_donor( $user = null ) {
+function give_count_donations_of_donor($user = null) {
 
 	// Logged in?
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
 	// Email access?
-	if ( empty( $user ) && Give()->email_access->token_email ) {
+	if (empty($user) && Give()->email_access->token_email) {
 		$user = Give()->email_access->token_email;
 	}
 
-	$stats = ! empty( $user ) ? give_get_donation_stats_by_user( $user ) : false;
+	$stats = ! empty($user) ? give_get_donation_stats_by_user($user) : false;
 
-	return isset( $stats['purchases'] ) ? $stats['purchases'] : 0;
+	return isset($stats['purchases']) ? $stats['purchases'] : 0;
 }
 
 /**
@@ -265,9 +265,9 @@ 
  *
  * @return float The total amount the user has spent
  */
-function give_donation_total_of_user( $user = null ) {
+function give_donation_total_of_user($user = null) {
 
-	$stats = give_get_donation_stats_by_user( $user );
+	$stats = give_get_donation_stats_by_user($user);
 
 	return $stats['total_spent'];
 }
@@ -283,40 +283,40 @@ 
  *
  * @return bool
  */
-function give_validate_username( $username, $form_id = 0 ) {
+function give_validate_username($username, $form_id = 0) {
 	$valid = true;
 
 	// Validate username.
-	if ( ! empty( $username ) ) {
+	if ( ! empty($username)) {
 
 		// Sanitize username.
-		$sanitized_user_name = sanitize_user( $username, false );
+		$sanitized_user_name = sanitize_user($username, false);
 
 		// We have an user name, check if it already exists.
-		if ( username_exists( $username ) ) {
+		if (username_exists($username)) {
 			// Username already registered.
-			give_set_error( 'username_unavailable', __( 'Username already taken.', 'give' ) );
+			give_set_error('username_unavailable', __('Username already taken.', 'give'));
 			$valid = false;
 
 			// Check if it's valid.
-		} elseif ( $sanitized_user_name !== $username ) {
+		} elseif ($sanitized_user_name !== $username) {
 			// Invalid username.
-			if ( is_multisite() ) {
-				give_set_error( 'username_invalid', __( 'Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give' ) );
+			if (is_multisite()) {
+				give_set_error('username_invalid', __('Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give'));
 				$valid = false;
 			} else {
-				give_set_error( 'username_invalid', __( 'Invalid username.', 'give' ) );
+				give_set_error('username_invalid', __('Invalid username.', 'give'));
 				$valid = false;
 			}
 		}
 	} else {
 		// Username is empty.
-		give_set_error( 'username_empty', __( 'Enter a username.', 'give' ) );
+		give_set_error('username_empty', __('Enter a username.', 'give'));
 		$valid = false;
 
 		// Check if guest checkout is disable for form.
-		if ( $form_id && give_logged_in_only( $form_id ) ) {
-			give_set_error( 'registration_required', __( 'You must register or login to complete your donation.', 'give' ) );
+		if ($form_id && give_logged_in_only($form_id)) {
+			give_set_error('registration_required', __('You must register or login to complete your donation.', 'give'));
 			$valid = false;
 		}
 	}
@@ -330,7 +330,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$valid = (bool) apply_filters( 'give_validate_username', $valid, $username, $form_id );
+	$valid = (bool) apply_filters('give_validate_username', $valid, $username, $form_id);
 
 	return $valid;
 }
@@ -346,30 +346,30 @@ 
  *
  * @return bool
  */
-function give_validate_user_email( $email, $registering_new_user = false ) {
+function give_validate_user_email($email, $registering_new_user = false) {
 	$valid = true;
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// No email.
-		give_set_error( 'email_empty', __( 'Enter an email.', 'give' ) );
+		give_set_error('email_empty', __('Enter an email.', 'give'));
 		$valid = false;
 
-	} elseif ( email_exists( $email ) ) {
+	} elseif (email_exists($email)) {
 		// Email already exists.
-		give_set_error( 'email_exists', __( 'Email already exists.', 'give' ) );
+		give_set_error('email_exists', __('Email already exists.', 'give'));
 		$valid = false;
 
-	} elseif ( ! is_email( $email ) ) {
+	} elseif ( ! is_email($email)) {
 		// Validate email.
-		give_set_error( 'email_invalid', __( 'Invalid email.', 'give' ) );
+		give_set_error('email_invalid', __('Invalid email.', 'give'));
 		$valid = false;
 
-	} elseif ( $registering_new_user ) {
+	} elseif ($registering_new_user) {
 
 		// If donor email is not primary.
-		if ( ! email_exists( $email ) && give_donor_email_exists( $email ) && give_is_additional_email( $email ) ) {
+		if ( ! email_exists($email) && give_donor_email_exists($email) && give_is_additional_email($email)) {
 			// Check if email exists.
-			give_set_error( 'email_used', __( 'The email address provided is already active for another user.', 'give' ) );
+			give_set_error('email_used', __('The email address provided is already active for another user.', 'give'));
 			$valid = false;
 		}
 	}
@@ -383,7 +383,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$valid = (bool) apply_filters( 'give_validate_user_email', $valid, $email, $registering_new_user );
+	$valid = (bool) apply_filters('give_validate_user_email', $valid, $email, $registering_new_user);
 
 	return $valid;
 }
@@ -399,34 +399,34 @@ 
  *
  * @return bool
  */
-function give_validate_user_password( $password = '', $confirm_password = '', $registering_new_user = false ) {
+function give_validate_user_password($password = '', $confirm_password = '', $registering_new_user = false) {
 	$valid = true;
 
 	// Passwords Validation For New Donors Only.
-	if ( $registering_new_user ) {
+	if ($registering_new_user) {
 		// Password or confirmation missing.
-		if ( ! $password ) {
+		if ( ! $password) {
 			// The password is invalid.
-			give_set_error( 'password_empty', __( 'Enter a password.', 'give' ) );
+			give_set_error('password_empty', __('Enter a password.', 'give'));
 			$valid = false;
-		} elseif ( ! $confirm_password ) {
+		} elseif ( ! $confirm_password) {
 			// Confirmation password is invalid.
-			give_set_error( 'confirmation_empty', __( 'Enter the password confirmation.', 'give' ) );
+			give_set_error('confirmation_empty', __('Enter the password confirmation.', 'give'));
 			$valid = false;
 		}
 	}
 	// Passwords Validation For New Donors as well as Existing Donors.
-	if ( $password || $confirm_password ) {
-		if ( strlen( $password ) < 6 || strlen( $confirm_password ) < 6 ) {
+	if ($password || $confirm_password) {
+		if (strlen($password) < 6 || strlen($confirm_password) < 6) {
 			// Seems Weak Password.
-			give_set_error( 'password_weak', __( 'Passwords should have at least 6 characters.', 'give' ) );
+			give_set_error('password_weak', __('Passwords should have at least 6 characters.', 'give'));
 			$valid = false;
 		}
-		if ( $password && $confirm_password ) {
+		if ($password && $confirm_password) {
 			// Verify confirmation matches.
-			if ( $password !== $confirm_password ) {
+			if ($password !== $confirm_password) {
 				// Passwords do not match.
-				give_set_error( 'password_mismatch', __( 'Passwords you entered do not match. Please try again.', 'give' ) );
+				give_set_error('password_mismatch', __('Passwords you entered do not match. Please try again.', 'give'));
 				$valid = false;
 			}
 		}
@@ -442,7 +442,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$valid = (bool) apply_filters( 'give_validate_user_email', $valid, $password, $confirm_password, $registering_new_user );
+	$valid = (bool) apply_filters('give_validate_user_email', $valid, $password, $confirm_password, $registering_new_user);
 
 	return $valid;
 }
@@ -470,8 +470,8 @@ 
  *
  * @return array The donor's address, if any
  */
-function give_get_donor_address( $donor_id = null, $args = array() ) {
-	if ( empty( $donor_id ) ) {
+function give_get_donor_address($donor_id = null, $args = array()) {
+	if (empty($donor_id)) {
 		$donor_id = get_current_user_id();
 	}
 
@@ -493,33 +493,33 @@ 
 
 
 	// Backward compatibility for user id param.
-	$by_user_id = get_user_by( 'id', $donor_id ) ? true : false;
+	$by_user_id = get_user_by('id', $donor_id) ? true : false;
 
 	// Backward compatibility.
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_user_address' ) && $by_user_id ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_user_address') && $by_user_id) {
 		return wp_parse_args(
-			(array) get_user_meta( $donor_id, '_give_user_address', true ),
+			(array) get_user_meta($donor_id, '_give_user_address', true),
 			$default_address
 		);
 	}
 
-	$donor = new Give_Donor( $donor_id, $by_user_id );
+	$donor = new Give_Donor($donor_id, $by_user_id);
 
 	if (
 		! $donor->id ||
-		empty( $donor->address ) ||
-		! array_key_exists( $args['address_type'], $donor->address )
+		empty($donor->address) ||
+		! array_key_exists($args['address_type'], $donor->address)
 	) {
 		return $default_address;
 	}
 
-	switch ( true ) {
-		case is_string( end( $donor->address[ $args['address_type'] ] ) ):
-			$address = wp_parse_args( $donor->address[ $args['address_type'] ], $default_address );
+	switch (true) {
+		case is_string(end($donor->address[$args['address_type']])):
+			$address = wp_parse_args($donor->address[$args['address_type']], $default_address);
 			break;
 
-		case is_array( end( $donor->address[ $args['address_type'] ] ) ):
-			$address = wp_parse_args( array_shift( $donor->address[ $args['address_type'] ] ), $default_address );
+		case is_array(end($donor->address[$args['address_type']])):
+			$address = wp_parse_args(array_shift($donor->address[$args['address_type']]), $default_address);
 			break;
 	}
 
@@ -539,19 +539,19 @@ 
  *
  * @return void
  */
-function give_new_user_notification( $donation_id = 0, $donation_data = array() ) {
+function give_new_user_notification($donation_id = 0, $donation_data = array()) {
 	// Bailout.
 	if (
-		empty( $donation_id )
-		|| empty( $donation_data )
-		|| ! isset( $_POST['give_create_account'] )
-		|| 'on' !== give_clean( $_POST['give_create_account'] )
+		empty($donation_id)
+		|| empty($donation_data)
+		|| ! isset($_POST['give_create_account'])
+		|| 'on' !== give_clean($_POST['give_create_account'])
 	) {
 		return;
 	}
 
 	// For backward compatibility
-	$user = get_user_by( 'ID', $donation_data['user_info']['id'] );
+	$user = get_user_by('ID', $donation_data['user_info']['id']);
 
 	$donation_data['user_info'] = array_merge(
 		$donation_data['user_info'],
@@ -564,11 +564,11 @@ 
 		)
 	);
 
-	do_action( 'give_new-donor-register_email_notification', $donation_data['user_info']['id'], $donation_data['user_info'], $donation_id );
-	do_action( 'give_donor-register_email_notification', $donation_data['user_info']['id'], $donation_data['user_info'], $donation_id );
+	do_action('give_new-donor-register_email_notification', $donation_data['user_info']['id'], $donation_data['user_info'], $donation_id);
+	do_action('give_donor-register_email_notification', $donation_data['user_info']['id'], $donation_data['user_info'], $donation_id);
 }
 
-add_action( 'give_insert_payment', 'give_new_user_notification', 10, 2 );
+add_action('give_insert_payment', 'give_new_user_notification', 10, 2);
 
 
 /**
@@ -584,37 +584,37 @@ 
  *
  * @return string
  */
-function give_get_donor_name_by( $id = 0, $from = 'donation' ) {
+function give_get_donor_name_by($id = 0, $from = 'donation') {
 
 	// ID shouldn't be empty.
-	if ( empty( $id ) ) {
+	if (empty($id)) {
 		return '';
 	}
 
 	$name         = '';
 	$title_prefix = '';
 
-	switch ( $from ) {
+	switch ($from) {
 
 		case 'donation':
-			$title_prefix  = give_get_meta( $id, '_give_payment_donor_title_prefix', true );
-			$first_name    = give_get_meta( $id, '_give_donor_billing_first_name', true );
-			$last_name     = give_get_meta( $id, '_give_donor_billing_last_name', true );
+			$title_prefix  = give_get_meta($id, '_give_payment_donor_title_prefix', true);
+			$first_name    = give_get_meta($id, '_give_donor_billing_first_name', true);
+			$last_name     = give_get_meta($id, '_give_donor_billing_last_name', true);
 
 			$name = "{$first_name} {$last_name}";
 
 			break;
 
 		case 'donor':
-			$name         = Give()->donors->get_column( 'name', $id );
-			$title_prefix = Give()->donor_meta->get_meta( $id, '_give_donor_title_prefix', true );
+			$name         = Give()->donors->get_column('name', $id);
+			$title_prefix = Give()->donor_meta->get_meta($id, '_give_donor_title_prefix', true);
 
 			break;
 
 	}
 
 	// If title prefix is set then prepend it to name.
-	$name = give_get_donor_name_with_title_prefixes( $title_prefix, $name );
+	$name = give_get_donor_name_with_title_prefixes($title_prefix, $name);
 
 	return $name;
 
@@ -629,8 +629,8 @@ 
  *
  * @return boolean  The user's ID on success, and false on failure.
  */
-function give_donor_email_exists( $email ) {
-	if ( Give()->donors->get_donor_by( 'email', $email ) ) {
+function give_donor_email_exists($email) {
+	if (Give()->donors->get_donor_by('email', $email)) {
 		return true;
 	}
 	return false;
@@ -645,14 +645,14 @@ 
  *
  * @return bool
  */
-function give_is_additional_email( $email ) {
+function give_is_additional_email($email) {
 	global $wpdb;
 
 	$meta_table = Give()->donor_meta->table_name;
 	$meta_type  = Give()->donor_meta->meta_type;
-	$donor_id   = $wpdb->get_var( $wpdb->prepare( "SELECT {$meta_type}_id FROM {$meta_table} WHERE meta_key = 'additional_email' AND meta_value = %s LIMIT 1", $email ) );
+	$donor_id   = $wpdb->get_var($wpdb->prepare("SELECT {$meta_type}_id FROM {$meta_table} WHERE meta_key = 'additional_email' AND meta_value = %s LIMIT 1", $email));
 
-	if ( empty( $donor_id ) ) {
+	if (empty($donor_id)) {
 		return false;
 	}
 

includes/price-functions.php

Spacing +21 added lines, -21 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,13 +23,13 @@ 
  *
  * @return bool true if has variable prices, false otherwise
  */
-function give_has_variable_prices( $form_id = 0 ) {
+function give_has_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->has_variable_prices();
 }
@@ -44,13 +44,13 @@ 
  *
  * @return array|bool Variable prices
  */
-function give_get_variable_prices( $form_id = 0 ) {
+function give_get_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->prices;
 
@@ -65,13 +65,13 @@ 
  *
  * @return array Variable prices
  */
-function give_get_variable_price_ids( $form_id = 0 ) {
-	if( ! ( $prices = give_get_variable_prices( $form_id ) ) ) {
+function give_get_variable_price_ids($form_id = 0) {
+	if ( ! ($prices = give_get_variable_prices($form_id))) {
 		return array();
 	}
 
 	$price_ids = array();
-	foreach ( $prices as $price ){
+	foreach ($prices as $price) {
 		$price_ids[] = $price['_give_id']['level_id'];
 	}
 
@@ -89,12 +89,12 @@ 
  *
  * @return string $default_price
  */
-function give_get_default_multilevel_amount( $form_id ) {
+function give_get_default_multilevel_amount($form_id) {
 	$default_price = '1.00';
 
 	// Get default level price data.
-	$default_level = give_form_get_default_level( $form_id );
-	$default_price = isset( $default_level['_give_amount'] ) ? $default_level['_give_amount'] : $default_price;
+	$default_level = give_form_get_default_level($form_id);
+	$default_price = isset($default_level['_give_amount']) ? $default_level['_give_amount'] : $default_price;
 
 	return $default_price;
 }
@@ -110,19 +110,19 @@ 
  * @return string $default_price
  * @since      1.0
  */
-function give_get_default_form_amount( $form_id ) {
+function give_get_default_form_amount($form_id) {
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$default_amount = give_get_default_multilevel_amount( $form_id );
+		$default_amount = give_get_default_multilevel_amount($form_id);
 
 	} else {
 
-		$default_amount = give_get_meta( $form_id, '_give_set_price', true );
+		$default_amount = give_get_meta($form_id, '_give_set_price', true);
 
 	}
 
-	return apply_filters( 'give_default_form_amount', $default_amount, $form_id );
+	return apply_filters('give_default_form_amount', $default_amount, $form_id);
 
 }
 
@@ -140,13 +140,13 @@ 
  *
  * @return bool
  */
-function give_is_custom_price_mode( $form_id = 0 ) {
+function give_is_custom_price_mode($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->is_custom_price_mode();
 }

includes/payments/class-payment-stats.php

Spacing +47 added lines, -47 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,17 +38,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -57,19 +57,19 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 			'output'     => ''
 		);
 
-		if ( ! empty( $form_id ) ) {
+		if ( ! empty($form_id)) {
 			$args['give_forms'] = $form_id;
 		}
 
 		/* @var Give_Payments_Query $payments */
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		return count( $payments );
+		return count($payments);
 	}
 
 
@@ -86,17 +86,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 		global $wpdb;
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -106,13 +106,13 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 			'output'     => '',
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -120,41 +120,41 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		// Set transient for faster stats.
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 
-		if ( false === $earnings ) {
+		if (false === $earnings) {
 
 			$this->timestamp = false;
-			$payments        = new Give_Payments_Query( $args );
+			$payments        = new Give_Payments_Query($args);
 			$payments        = $payments->get_payments();
 			$earnings        = 0;
 
-			if ( ! empty( $payments ) ) {
-				$donation_id_col = Give()->payment_meta->get_meta_type() . '_id';
+			if ( ! empty($payments)) {
+				$donation_id_col = Give()->payment_meta->get_meta_type().'_id';
 				$query = "SELECT {$donation_id_col} as id, meta_value as total
 					FROM {$wpdb->donationmeta}
 					WHERE meta_key='_give_payment_total'
-					AND {$donation_id_col} IN ('". implode( '\',\'', $payments ) ."')";
+					AND {$donation_id_col} IN ('".implode('\',\'', $payments)."')";
 
 				$payments = $wpdb->get_results($query, ARRAY_A);
 
-				if( ! empty( $payments ) ) {
-					foreach ( $payments as $payment ) {
+				if ( ! empty($payments)) {
+					foreach ($payments as $payment) {
 						/**
 						 * Filter the donation amount
 						 * Note: this filter documented in payments/functions.php:give_donation_amount()
@@ -163,20 +163,20 @@ 
 						 */
 						$formatted_amount = apply_filters(
 							'give_donation_amount',
-							give_format_amount(  $payment['total'], array( 'donation_id' =>  $payment['id'] ) ),
+							give_format_amount($payment['total'], array('donation_id' =>  $payment['id'])),
 							$payment['total'],
 							$payment['id'],
-							array( 'type' => 'stats', 'currency'=> false, 'amount' => false )
+							array('type' => 'stats', 'currency'=> false, 'amount' => false)
 						);
 
-						$earnings += (float) give_maybe_sanitize_amount( $formatted_amount );
+						$earnings += (float) give_maybe_sanitize_amount($formatted_amount);
 					}
 				}
 
 			}
 
 			// Cache the results for one hour.
-			Give_Cache::set( $key, give_sanitize_amount_for_db( $earnings ), 60 * 60 );
+			Give_Cache::set($key, give_sanitize_amount_for_db($earnings), 60 * 60);
 		}
 
 		/**
@@ -190,10 +190,10 @@ 
 		 * @param  string|bool $end_date   Earning end date.
 		 * @param  string|bool $gateway_id Payment gateway id.
 		 */
-		$earnings = apply_filters( 'give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id );
+		$earnings = apply_filters('give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id);
 
 		//return earnings
-		return round( $earnings, give_get_price_decimals( $form_id ) );
+		return round($earnings, give_get_price_decimals($form_id));
 
 	}
 
@@ -210,17 +210,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings_cache_key($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -230,12 +230,12 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -243,19 +243,19 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		//return earnings
 		return $key;
@@ -273,17 +273,17 @@ 
 	 *
 	 * @return array       Best selling forms
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 		global $wpdb;
 
-		$meta_table = __give_v20_bc_table_details( 'form' );
+		$meta_table = __give_v20_bc_table_details('form');
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT {$meta_table['column']['id']} as form_id, max(meta_value) as sales
 				FROM {$meta_table['name']} WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}

includes/payments/functions.php

Spacing +324 added lines, -327 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,19 +67,19 @@ 
  *
  * @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;
 			}
 
@@ -95,8 +95,8 @@ 
 				)
 			);
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -111,8 +111,8 @@ 
 				)
 			);
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -121,7 +121,7 @@ 
 			return false;
 	}// End switch().
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -137,9 +137,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;
 	}
 
@@ -150,35 +150,35 @@ 
 	 *
 	 * @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'] ) ? give_clean( $_POST['give-gateway'] ) : $gateway; // WPCS: input var ok, sanitization ok, CSRF ok.
-	$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']) ? give_clean($_POST['give-gateway']) : $gateway; // WPCS: input var ok, sanitization ok, CSRF ok.
+	$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'];
 	$payment->last_name      = $payment_data['user_info']['last_name'];
-	$payment->title_prefix   = ! empty( $payment_data['user_info']['title'] ) ? $payment_data['user_info']['title'] : '';
+	$payment->title_prefix   = ! empty($payment_data['user_info']['title']) ? $payment_data['user_info']['title'] : '';
 	$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(
@@ -186,10 +186,10 @@ 
 		'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'];
 	}
 
@@ -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,17 +295,17 @@ 
  *
  * @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  = new Give_Donor( $payment->donor_id );
+	$donor  = new Give_Donor($payment->donor_id);
 
 	// Only undo donations that aren't these statuses.
 	$dont_undo_statuses = apply_filters(
@@ -315,26 +315,26 @@ 
 		)
 	);
 
-	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);
 
 		}
 	}
@@ -346,20 +346,20 @@ 
 	 *
 	 * @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);
 
-	Give()->payment_meta->delete_all_meta( $payment_id );
+	Give()->payment_meta->delete_all_meta($payment_id);
 
 	// Remove related sale log entries.
-	Give()->logs->delete_logs( $payment_id );
+	Give()->logs->delete_logs($payment_id);
 
 	/**
 	 * Fires after payment deleted.
@@ -368,7 +368,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -383,20 +383,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, $payment_id );
+		give_decrease_form_earnings($payment->form_id, $payment->total, $payment_id);
 	}
 
-	$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);
 	}
 
 }
@@ -413,21 +413,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
@@ -435,7 +435,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();
 
 	/**
@@ -443,7 +443,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);
 }
 
 
@@ -456,7 +456,7 @@ 
  *
  * @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) {
 	global $wpdb;
 
 	return (bool) $wpdb->get_var(
@@ -484,9 +484,9 @@ 
  *
  * @return bool|mixed True if payment status exists, false otherwise.
  */
-function give_get_payment_status( $payment_id, $return_label = false ) {
+function give_get_payment_status($payment_id, $return_label = false) {
 
-	if ( ! is_numeric( $payment_id ) ) {
+	if ( ! is_numeric($payment_id)) {
 		if (
 			$payment_id instanceof  Give_Payment
 			|| $payment_id instanceof WP_Post
@@ -495,28 +495,28 @@ 
 		}
 	}
 
-	if ( ! $payment_id > 0 ) {
+	if ( ! $payment_id > 0) {
 		return false;
 	}
 
-	$payment_status = get_post_status( $payment_id );
+	$payment_status = get_post_status($payment_id);
 
 	$statuses = give_get_payment_statuses();
 
-	if ( empty( $payment_status ) || ! is_array( $statuses ) || empty( $statuses ) ) {
+	if (empty($payment_status) || ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
-	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_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));
 		}
 	}
 
@@ -532,18 +532,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);
 }
 
 /**
@@ -556,10 +556,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);
 }
 
 /**
@@ -574,43 +574,43 @@ 
  *
  * @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.
@@ -621,13 +621,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);
 }
 
 /**
@@ -642,7 +642,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(
@@ -650,14 +650,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',
@@ -668,33 +668,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;
@@ -709,14 +709,14 @@ 
  *
  * @return bool $ret True if complete, false otherwise.
  */
-function give_is_payment_complete( $payment_id ) {
+function give_is_payment_complete($payment_id) {
 	$ret            = false;
 	$payment_status = '';
 
-	if ( $payment_id > 0 && 'give_payment' === get_post_type( $payment_id ) ) {
-		$payment_status = get_post_status( $payment_id );
+	if ($payment_id > 0 && 'give_payment' === get_post_type($payment_id)) {
+		$payment_status = get_post_status($payment_id);
 
-		if ( 'publish' === $payment_status ) {
+		if ('publish' === $payment_status) {
 			$ret = true;
 		}
 	}
@@ -726,7 +726,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment_status);
 }
 
 /**
@@ -752,13 +752,13 @@ 
  *
  * @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;
@@ -766,38 +766,38 @@ 
 		$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, false );
+		update_option('give_earnings_total', $total, false);
 	}
 
-	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);
 }
 
 /**
@@ -809,10 +809,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, false );
+	update_option('give_earnings_total', $total, false);
 
 	return $total;
 }
@@ -826,13 +826,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, false );
+	update_option('give_earnings_total', $total, false);
 
 	return $total;
 }
@@ -848,8 +848,8 @@ 
  *
  * @return mixed $meta Payment Meta.
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	return give_get_meta( $payment_id, $meta_key, $single );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	return give_get_meta($payment_id, $meta_key, $single);
 }
 
 /**
@@ -862,8 +862,8 @@ 
  *
  * @return mixed Meta ID if successful, false if unsuccessful.
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	return give_update_meta( $payment_id, $meta_key, $meta_value );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	return give_update_meta($payment_id, $meta_key, $meta_value);
 }
 
 /**
@@ -875,34 +875,34 @@ 
  *
  * @return array $user_info User Info Meta Values.
  */
-function give_get_payment_meta_user_info( $payment_id ) {
+function give_get_payment_meta_user_info($payment_id) {
 	$donor_id   = 0;
 	$donor_info = array(
-		'first_name' => give_get_meta( $payment_id, '_give_donor_billing_first_name', true ),
-		'last_name'  => give_get_meta( $payment_id, '_give_donor_billing_last_name', true ),
-		'email'      => give_get_meta( $payment_id, '_give_donor_billing_donor_email', true ),
+		'first_name' => give_get_meta($payment_id, '_give_donor_billing_first_name', true),
+		'last_name'  => give_get_meta($payment_id, '_give_donor_billing_last_name', true),
+		'email'      => give_get_meta($payment_id, '_give_donor_billing_donor_email', true),
 	);
 
-	if ( empty( $donor_info['first_name'] ) ) {
-		$donor_id                 = give_get_payment_donor_id( $payment_id );
-		$donor_info['first_name'] = Give()->donor_meta->get_meta( $donor_id, '_give_donor_first_name', true );
+	if (empty($donor_info['first_name'])) {
+		$donor_id                 = give_get_payment_donor_id($payment_id);
+		$donor_info['first_name'] = Give()->donor_meta->get_meta($donor_id, '_give_donor_first_name', true);
 	}
 
-	if ( empty( $donor_info['last_name'] ) ) {
-		$donor_id                = $donor_id ? $donor_id : give_get_payment_donor_id( $payment_id );
-		$donor_info['last_name'] = Give()->donor_meta->get_meta( $donor_id, '_give_donor_last_name', true );
+	if (empty($donor_info['last_name'])) {
+		$donor_id                = $donor_id ? $donor_id : give_get_payment_donor_id($payment_id);
+		$donor_info['last_name'] = Give()->donor_meta->get_meta($donor_id, '_give_donor_last_name', true);
 	}
 
-	if ( empty( $donor_info['email'] ) ) {
-		$donor_id            = $donor_id ? $donor_id : give_get_payment_donor_id( $payment_id );
-		$donor_info['email'] = Give()->donors->get_column_by( 'email', 'id', $donor_id );
+	if (empty($donor_info['email'])) {
+		$donor_id            = $donor_id ? $donor_id : give_get_payment_donor_id($payment_id);
+		$donor_info['email'] = Give()->donors->get_column_by('email', 'id', $donor_id);
 	}
 
-	$donor_info['title'] = Give()->donor_meta->get_meta( $donor_id, '_give_donor_title_prefix', true );
+	$donor_info['title'] = Give()->donor_meta->get_meta($donor_id, '_give_donor_title_prefix', true);
 
-	$donor_info['address']  = give_get_donation_address( $payment_id );
-	$donor_info['id']       = give_get_payment_user_id( $payment_id );
-	$donor_info['donor_id'] = give_get_payment_donor_id( $payment_id );
+	$donor_info['address']  = give_get_donation_address($payment_id);
+	$donor_info['id']       = give_get_payment_user_id($payment_id);
+	$donor_info['donor_id'] = give_get_payment_donor_id($payment_id);
 
 	return $donor_info;
 }
@@ -918,8 +918,8 @@ 
  *
  * @return int $form_id Form ID.
  */
-function give_get_payment_form_id( $payment_id ) {
-	return (int) give_get_meta( $payment_id, '_give_payment_form_id', true );
+function give_get_payment_form_id($payment_id) {
+	return (int) give_get_meta($payment_id, '_give_payment_form_id', true);
 }
 
 /**
@@ -931,11 +931,11 @@ 
  *
  * @return string $email User email.
  */
-function give_get_payment_user_email( $payment_id ) {
-	$email = give_get_meta( $payment_id, '_give_payment_donor_email', true );
+function give_get_payment_user_email($payment_id) {
+	$email = give_get_meta($payment_id, '_give_payment_donor_email', true);
 
-	if ( empty( $email ) && ( $donor_id = give_get_payment_donor_id( $payment_id ) ) ) {
-		$email = Give()->donors->get_column( 'email', $donor_id );
+	if (empty($email) && ($donor_id = give_get_payment_donor_id($payment_id))) {
+		$email = Give()->donors->get_column('email', $donor_id);
 	}
 
 	return $email;
@@ -950,11 +950,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);
 }
 
 /**
@@ -966,10 +966,10 @@ 
  *
  * @return int $user_id User ID.
  */
-function give_get_payment_user_id( $payment_id ) {
+function give_get_payment_user_id($payment_id) {
 	global $wpdb;
 	$paymentmeta_table = Give()->payment_meta->table_name;
-	$donationmeta_primary_key = Give()->payment_meta->get_meta_type() . '_id';
+	$donationmeta_primary_key = Give()->payment_meta->get_meta_type().'_id';
 
 	return (int) $wpdb->get_var(
 		$wpdb->prepare(
@@ -998,8 +998,8 @@ 
  *
  * @return int $payment->customer_id Donor ID.
  */
-function give_get_payment_donor_id( $payment_id ) {
-	return give_get_meta( $payment_id, '_give_payment_donor_id', true );
+function give_get_payment_donor_id($payment_id) {
+	return give_get_meta($payment_id, '_give_payment_donor_id', true);
 }
 
 /**
@@ -1011,8 +1011,8 @@ 
  *
  * @return string
  */
-function give_get_donation_donor_email( $payment_id ) {
-	return give_get_meta( $payment_id, '_give_payment_donor_email', true );
+function give_get_donation_donor_email($payment_id) {
+	return give_get_meta($payment_id, '_give_payment_donor_email', true);
 }
 
 /**
@@ -1024,8 +1024,8 @@ 
  *
  * @return string $ip User IP.
  */
-function give_get_payment_user_ip( $payment_id ) {
-	return give_get_meta( $payment_id, '_give_payment_donor_ip', true );
+function give_get_payment_user_ip($payment_id) {
+	return give_get_meta($payment_id, '_give_payment_donor_ip', true);
 }
 
 /**
@@ -1037,8 +1037,8 @@ 
  *
  * @return string $date The date the payment was completed.
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	return give_get_meta( $payment_id, '_give_completed_date', true );
+function give_get_payment_completed_date($payment_id = 0) {
+	return give_get_meta($payment_id, '_give_completed_date', true);
 }
 
 /**
@@ -1050,8 +1050,8 @@ 
  *
  * @return string $gateway Gateway.
  */
-function give_get_payment_gateway( $payment_id ) {
-	return give_get_meta( $payment_id, '_give_payment_gateway', true );
+function give_get_payment_gateway($payment_id) {
+	return give_get_meta($payment_id, '_give_payment_gateway', true);
 }
 
 /**
@@ -1064,10 +1064,9 @@ 
  *
  * @return bool
  */
-function give_has_payment_gateway( $donation_id, $gateway_id ) {
+function give_has_payment_gateway($donation_id, $gateway_id) {
 	$donation_gateway = $donation_id instanceof Give_Payment ?
-		$donation_id->gateway :
-		give_get_payment_gateway( $donation_id );
+		$donation_id->gateway : give_get_payment_gateway($donation_id);
 
 	return $gateway_id === $donation_gateway;
 }
@@ -1081,8 +1080,8 @@ 
  *
  * @return string $currency The currency code.
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	return give_get_meta( $payment_id, '_give_payment_currency', true );
+function give_get_payment_currency_code($payment_id = 0) {
+	return give_get_meta($payment_id, '_give_payment_currency', true);
 }
 
 /**
@@ -1094,10 +1093,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);
 }
 
 /**
@@ -1109,8 +1108,8 @@ 
  *
  * @return string $key Donation key.
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	return give_get_meta( $payment_id, '_give_payment_purchase_key', true );
+function give_get_payment_key($payment_id = 0) {
+	return give_get_meta($payment_id, '_give_payment_purchase_key', true);
 }
 
 /**
@@ -1124,8 +1123,8 @@ 
  *
  * @return string $number Payment order number.
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	return Give()->seq_donation_number->get_serial_code( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	return Give()->seq_donation_number->get_serial_code($payment_id);
 }
 
 
@@ -1143,17 +1142,17 @@ 
  *
  * @return string $amount Fully formatted donation amount.
  */
-function give_donation_amount( $donation_id, $format_args = array() ) {
-	if ( ! $donation_id ) {
+function give_donation_amount($donation_id, $format_args = array()) {
+	if ( ! $donation_id) {
 		return '';
-	} elseif ( ! is_numeric( $donation_id ) && ( $donation_id instanceof Give_Payment ) ) {
+	} elseif ( ! is_numeric($donation_id) && ($donation_id instanceof Give_Payment)) {
 		$donation_id = $donation_id->ID;
 	}
 
-	$amount        = $formatted_amount = give_get_payment_total( $donation_id );
-	$currency_code = give_get_payment_currency_code( $donation_id );
+	$amount        = $formatted_amount = give_get_payment_total($donation_id);
+	$currency_code = give_get_payment_currency_code($donation_id);
 
-	if ( is_bool( $format_args ) ) {
+	if (is_bool($format_args)) {
 		$format_args = array(
 			'currency' => (bool) $format_args,
 			'amount'   => (bool) $format_args,
@@ -1176,27 +1175,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' => $currency_code,
-					) :
-					$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' => $currency_code ) :
-					$format_args['currency']
+				! is_array($format_args['currency']) ?
+					array('currency_code' => $currency_code) : $format_args['currency']
 			);
 		}
 	}
@@ -1211,7 +1208,7 @@ 
 	 * @param int    $donation_id      Donation ID.
 	 * @param string $type             Donation amount type.
 	 */
-	return apply_filters( 'give_donation_amount', (string) $formatted_amount, $amount, $donation_id, $format_args );
+	return apply_filters('give_donation_amount', (string) $formatted_amount, $amount, $donation_id, $format_args);
 }
 
 /**
@@ -1228,10 +1225,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)));
 }
 
 /**
@@ -1245,8 +1242,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;
 }
@@ -1260,12 +1257,12 @@ 
  *
  * @return string The donation ID.
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$transaction_id = give_get_meta( $payment_id, '_give_payment_transaction_id', true );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$transaction_id = give_get_meta($payment_id, '_give_payment_transaction_id', true);
 
-	if ( empty( $transaction_id ) ) {
-		$gateway        = give_get_payment_gateway( $payment_id );
-		$transaction_id = apply_filters( "give_get_payment_transaction_id-{$gateway}", $payment_id );
+	if (empty($transaction_id)) {
+		$gateway        = give_get_payment_gateway($payment_id);
+		$transaction_id = apply_filters("give_get_payment_transaction_id-{$gateway}", $payment_id);
 	}
 
 	return $transaction_id;
@@ -1281,15 +1278,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);
 }
 
 /**
@@ -1302,10 +1299,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(
@@ -1321,7 +1318,7 @@ 
 		)
 	);
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1339,13 +1336,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;
 	}
 
@@ -1362,8 +1359,8 @@ 
  *
  * @return array $notes Donation Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
-	return Give_Comment::get( $payment_id,'payment', array(), $search );
+function give_get_payment_notes($payment_id = 0, $search = '') {
+	return Give_Comment::get($payment_id, 'payment', array(), $search);
 }
 
 
@@ -1377,8 +1374,8 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	return Give_Comment::add( $payment_id, $note, 'payment' );
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	return Give_Comment::add($payment_id, $note, 'payment');
 }
 
 /**
@@ -1391,8 +1388,8 @@ 
  *
  * @return bool True on success, false otherwise.
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	return Give_Comment::delete( $comment_id, $payment_id, 'payment' );
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	return Give_Comment::delete($comment_id, $payment_id, 'payment');
 }
 
 /**
@@ -1405,20 +1402,20 @@ 
  *
  * @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(
@@ -1427,14 +1424,14 @@ 
 				'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>';
 
@@ -1453,9 +1450,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;
@@ -1476,13 +1473,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_id, $args = array() ) {
+function give_get_donation_form_title($donation_id, $args = array()) {
 	// Backward compatibility.
-	if ( ! is_numeric( $donation_id ) && $donation_id instanceof Give_Payment ) {
+	if ( ! is_numeric($donation_id) && $donation_id instanceof Give_Payment) {
 		$donation_id = $donation_id->ID;
 	}
 
-	if ( ! $donation_id ) {
+	if ( ! $donation_id) {
 		return '';
 	}
 
@@ -1491,11 +1488,11 @@ 
 		'separator'  => '',
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
-	$form_id     = give_get_payment_form_id( $donation_id );
-	$price_id    = give_get_meta( $donation_id, '_give_payment_price_id', true );
-	$form_title  = give_get_meta( $donation_id, '_give_payment_form_title', true );
+	$form_id     = give_get_payment_form_id($donation_id);
+	$price_id    = give_get_meta($donation_id, '_give_payment_price_id', true);
+	$form_title  = give_get_meta($donation_id, '_give_payment_form_title', true);
 	$only_level  = $args['only_level'];
 	$separator   = $args['separator'];
 	$level_label = '';
@@ -1511,38 +1508,38 @@ 
 		), 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 ) {
+		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' );
+			$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');
 
 			// Show custom amount level only in backend otherwise hide it.
-			if ( 'set' === give_get_meta( $form_id, '_give_price_option', true ) && ! is_admin() ) {
+			if ('set' === give_get_meta($form_id, '_give_price_option', true) && ! is_admin()) {
 				$level_label = '';
 			}
-		} elseif ( give_has_variable_prices( $form_id ) ) {
-			$level_label = give_get_price_option_name( $form_id, $price_id, $donation_id, false );
+		} elseif (give_has_variable_prices($form_id)) {
+			$level_label = give_get_price_option_name($form_id, $price_id, $donation_id, false);
 		}
 
 		// 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);
 	}
 
 	/**
@@ -1551,7 +1548,7 @@ 
 	 * @since 1.0
 	 * @todo: remove third param after 2.1.0
 	 */
-	return apply_filters( 'give_get_donation_form_title', $form_title_html, $donation_id, '' );
+	return apply_filters('give_get_donation_form_title', $form_title_html, $donation_id, '');
 }
 
 /**
@@ -1564,19 +1561,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;
@@ -1584,13 +1581,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
@@ -1600,7 +1597,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);
 }
 
 /**
@@ -1616,10 +1613,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;
 	}
 
@@ -1636,17 +1633,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;
 	}
 
@@ -1654,13 +1651,13 @@ 
 	$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.
@@ -1671,9 +1668,9 @@ 
 	);
 
 	// 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;
 	}
 
@@ -1692,14 +1689,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']);
 	}
 
 	/**
@@ -1710,7 +1707,7 @@ 
 	 * @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);
 
 }
 
@@ -1724,10 +1721,10 @@ 
  *
  * @return float
  */
-function give_get_payment_total( $payment_id = 0 ) {
+function give_get_payment_total($payment_id = 0) {
 	return round(
-		floatval( give_get_meta( $payment_id, '_give_payment_total', true ) ),
-		give_get_price_decimals( $payment_id )
+		floatval(give_get_meta($payment_id, '_give_payment_total', true)),
+		give_get_price_decimals($payment_id)
 	);
 }
 
@@ -1740,13 +1737,13 @@ 
  *
  * @return array
  */
-function give_get_donation_address( $donation_id ) {
-	$address['line1']   = give_get_meta( $donation_id, '_give_donor_billing_address1', true, '' );
-	$address['line2']   = give_get_meta( $donation_id, '_give_donor_billing_address2', true, '' );
-	$address['city']    = give_get_meta( $donation_id, '_give_donor_billing_city', true, '' );
-	$address['state']   = give_get_meta( $donation_id, '_give_donor_billing_state', true, '' );
-	$address['zip']     = give_get_meta( $donation_id, '_give_donor_billing_zip', true, '' );
-	$address['country'] = give_get_meta( $donation_id, '_give_donor_billing_country', true, '' );
+function give_get_donation_address($donation_id) {
+	$address['line1']   = give_get_meta($donation_id, '_give_donor_billing_address1', true, '');
+	$address['line2']   = give_get_meta($donation_id, '_give_donor_billing_address2', true, '');
+	$address['city']    = give_get_meta($donation_id, '_give_donor_billing_city', true, '');
+	$address['state']   = give_get_meta($donation_id, '_give_donor_billing_state', true, '');
+	$address['zip']     = give_get_meta($donation_id, '_give_donor_billing_zip', true, '');
+	$address['country'] = give_get_meta($donation_id, '_give_donor_billing_country', true, '');
 
 	return $address;
 }
@@ -1761,7 +1758,7 @@ 
  *
  * @return bool
  */
-function give_is_donation_completed( $donation_id ) {
+function give_is_donation_completed($donation_id) {
 	global $wpdb;
 
 	/**

includes/class-give-cache.php

Spacing +147 added lines, -147 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,20 +68,20 @@ 
 	 */
 	public function setup() {
 		// Currently enable cache only for backend.
-		self::$instance->is_cache = ( defined( 'GIVE_CACHE' ) ? GIVE_CACHE : give_is_setting_enabled( give_get_option( 'cache', 'enabled' ) ) ) && is_admin();
+		self::$instance->is_cache = (defined('GIVE_CACHE') ? GIVE_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( __CLASS__, 'flush_cache' ) );
+		add_action('give_save_settings_give_settings', array(__CLASS__, 'flush_cache'));
 
-		add_action( 'wp', array( __CLASS__,  'prevent_caching' ) );
-		add_action( 'admin_notices', array( $this, '__notices' ) );
+		add_action('wp', array(__CLASS__, 'prevent_caching'));
+		add_action('admin_notices', array($this, '__notices'));
 	}
 
 	/**
@@ -92,19 +92,19 @@ 
 	 * @credit WooCommerce
 	 */
 	public static function prevent_caching() {
-		if ( ! is_blog_installed() ) {
+		if ( ! is_blog_installed()) {
 			return;
 		}
 
-		$page_ids = array_filter( array(
-			give_get_option( 'success_page' ),
-			give_get_option( 'failure_page' ),
-			give_get_option( 'history_page' ),
-		) );
+		$page_ids = array_filter(array(
+			give_get_option('success_page'),
+			give_get_option('failure_page'),
+			give_get_option('history_page'),
+		));
 
 		if (
-			is_page( $page_ids )
-			|| is_singular( 'give_forms' )
+			is_page($page_ids)
+			|| is_singular('give_forms')
 		) {
 			self::set_nocache_constants();
 			nocache_headers();
@@ -122,10 +122,10 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function set_nocache_constants( $return = true ) {
-		give_maybe_define_constant( 'DONOTCACHEPAGE', true );
-		give_maybe_define_constant( 'DONOTCACHEOBJECT', true );
-		give_maybe_define_constant( 'DONOTCACHEDB', true );
+	public static function set_nocache_constants($return = true) {
+		give_maybe_define_constant('DONOTCACHEPAGE', true);
+		give_maybe_define_constant('DONOTCACHEOBJECT', true);
+		give_maybe_define_constant('DONOTCACHEDB', true);
 
 		return $return;
 	}
@@ -138,18 +138,18 @@ 
 	 * @credit WooCommerce
 	 */
 	public function __notices() {
-		if ( ! function_exists( 'w3tc_pgcache_flush' ) || ! function_exists( 'w3_instance' ) ) {
+		if ( ! function_exists('w3tc_pgcache_flush') || ! function_exists('w3_instance')) {
 			return;
 		}
 
-		$config   = w3_instance( 'W3_Config' );
-		$enabled  = $config->get_integer( 'dbcache.enabled' );
-		$settings = array_map( 'trim', $config->get_array( 'dbcache.reject.sql' ) );
+		$config   = w3_instance('W3_Config');
+		$enabled  = $config->get_integer('dbcache.enabled');
+		$settings = array_map('trim', $config->get_array('dbcache.reject.sql'));
 
-		if ( $enabled && ! in_array( 'give', $settings, true ) ) {
+		if ($enabled && ! in_array('give', $settings, true)) {
 			?>
 			<div class="error">
-				<p><?php echo wp_kses_post( sprintf( __( 'In order for <strong>database caching</strong> to work with Give you must add %1$s to the "Ignored Query Strings" option in <a href="%2$s">W3 Total Cache settings</a>.', 'give' ), '<code>give</code>', esc_url( admin_url( 'admin.php?page=w3tc_dbcache' ) ) ) ); ?></p>
+				<p><?php echo wp_kses_post(sprintf(__('In order for <strong>database caching</strong> to work with Give you must add %1$s to the "Ignored Query Strings" option in <a href="%2$s">W3 Total Cache settings</a>.', 'give'), '<code>give</code>', esc_url(admin_url('admin.php?page=w3tc_dbcache')))); ?></p>
 			</div>
 			<?php
 		}
@@ -166,18 +166,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);
 		}
 
 		/**
@@ -185,7 +185,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);
 	}
 
 	/**
@@ -199,28 +199,28 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get( $cache_key, $custom_key = false, $query_args = array() ) {
-		if ( ! self::is_valid_cache_key( $cache_key ) ) {
-			if( empty( $cache_key ) ) {
-				return new WP_Error( 'give_empty_cache_key', __( 'Do not pass invalid empty cache key', 'give' ) );
-			}elseif ( ! $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 (empty($cache_key)) {
+				return new WP_Error('give_empty_cache_key', __('Do not pass invalid empty cache key', 'give'));
+			}elseif ( ! $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;
@@ -242,23 +242,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, false );
+		$result = update_option($cache_key, $option_value, false);
 
 		return $result;
 	}
@@ -274,27 +274,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
 			);
 		}
@@ -315,7 +315,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(
@@ -329,30 +329,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']);
 		}
 	}
 
@@ -370,12 +370,12 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_options_like( $option_name, $fields = false ) {
+	public static function get_options_like($option_name, $fields = false) {
 		global $wpdb;
 
 		$field_names = $fields ? 'option_name, option_value' : 'option_name';
 
-		if ( $fields ) {
+		if ($fields) {
 			$options = $wpdb->get_results(
 				$wpdb->prepare(
 					"SELECT {$field_names }
@@ -399,10 +399,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;
 			}
 		}
 
@@ -419,8 +419,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_'));
 
 
 		/**
@@ -428,7 +428,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);
 	}
 
 
@@ -443,14 +443,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;
 		}
 
@@ -470,17 +470,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;
 	}
@@ -496,15 +496,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);
 	}
 
 	/**
@@ -517,8 +517,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');
 	}
 
 	/**
@@ -533,21 +533,21 @@ 
 	 *
 	 * @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_prefix = $group;
-		$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 );
-			self::$instance->get_incrementer( true );
+		if ( ! is_array($ids)) {
+			$status = wp_cache_delete($ids, $group);
+			self::$instance->get_incrementer(true);
 
 			/**
 			 * Fire action when cache deleted for specific id.
@@ -558,12 +558,12 @@ 
 			 * @param string $group
 			 * @param int    $expire
 			 */
-			do_action( "give_deleted_{$group_prefix}_cache", $ids, $group, $expire, $status );
+			do_action("give_deleted_{$group_prefix}_cache", $ids, $group, $expire, $status);
 
 		} else {
-			foreach ( $ids as $id ) {
-				$status = wp_cache_delete( $id, $group );
-				self::$instance->get_incrementer( true );
+			foreach ($ids as $id) {
+				$status = wp_cache_delete($id, $group);
+				self::$instance->get_incrementer(true);
 
 				/**
 				 * Fire action when cache deleted for specific id .
@@ -574,7 +574,7 @@ 
 				 * @param string $group
 				 * @param int    $expire
 				 */
-				do_action( "give_deleted_{$group_prefix}_cache", $id, $group, $expire, $status );
+				do_action("give_deleted_{$group_prefix}_cache", $id, $group, $expire, $status);
 			}
 		}
 
@@ -591,15 +591,15 @@ 
 	 *
 	 * @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,
 				'output'     => '',
 				'fields'     => 'ids',
@@ -608,15 +608,15 @@ 
 
 		$donations = $donation_query->get_payments();
 
-		if ( ! empty( $donations ) ) {
+		if ( ! empty($donations)) {
 			/* @var Give_Payment $donation */
-			foreach ( $donations as $donation_id ) {
-				wp_cache_delete( $donation_id, $this->filter_group_name( 'give-donations' ) );
-				wp_cache_delete( give_get_payment_donor_id( $donation_id ), $this->filter_group_name( 'give-donors' ) );
+			foreach ($donations as $donation_id) {
+				wp_cache_delete($donation_id, $this->filter_group_name('give-donations'));
+				wp_cache_delete(give_get_payment_donor_id($donation_id), $this->filter_group_name('give-donors'));
 			}
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -628,19 +628,19 @@ 
 	 *
 	 * @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;
 		}
 
-		if ( $donation_id && ( $donor_id = give_get_payment_donor_id( $donation_id ) ) ) {
-			wp_cache_delete( $donor_id, $this->filter_group_name( 'give-donors' ) );
+		if ($donation_id && ($donor_id = give_get_payment_donor_id($donation_id))) {
+			wp_cache_delete($donor_id, $this->filter_group_name('give-donors'));
 		}
 
-		wp_cache_delete( $donation_id, $this->filter_group_name( 'give-donations' ) );
+		wp_cache_delete($donation_id, $this->filter_group_name('give-donations'));
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -654,18 +654,18 @@ 
 	 * @param string $group
 	 * @param int    $expire
 	 */
-	public function delete_donor_related_cache( $id, $group, $expire ) {
-		$donation_ids = Give()->donors->get_column( 'payment_ids', $id );
+	public function delete_donor_related_cache($id, $group, $expire) {
+		$donation_ids = Give()->donors->get_column('payment_ids', $id);
 
-		if ( ! empty( $donation_ids ) ) {
-			$donation_ids = array_map( 'trim', (array) explode( ',', trim( $donation_ids  ) ) );
+		if ( ! empty($donation_ids)) {
+			$donation_ids = array_map('trim', (array) explode(',', trim($donation_ids)));
 
-			foreach ( $donation_ids as $donation ) {
-				wp_cache_delete( $donation, $this->filter_group_name( 'give-donations' ) );
+			foreach ($donation_ids as $donation) {
+				wp_cache_delete($donation, $this->filter_group_name('give-donations'));
 			}
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 	/**
@@ -679,12 +679,12 @@ 
 	 * @param string $group
 	 * @param int    $expire
 	 */
-	public function delete_donations_related_cache( $id, $group, $expire ) {
-		if ( $id && ( $donor_id = give_get_payment_donor_id( $id ) ) ) {
-			wp_cache_delete( $donor_id, $this->filter_group_name( 'give-donors' ) );
+	public function delete_donations_related_cache($id, $group, $expire) {
+		if ($id && ($donor_id = give_get_payment_donor_id($id))) {
+			wp_cache_delete($donor_id, $this->filter_group_name('give-donors'));
 		}
 
-		self::$instance->get_incrementer( true );
+		self::$instance->get_incrementer(true);
 	}
 
 
@@ -702,12 +702,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_incrementer( $refresh = false, $incrementer_key = 'give-cache-incrementer-db-queries' ) {
-		$incrementer_value = wp_cache_get( $incrementer_key );
+	public function get_incrementer($refresh = false, $incrementer_key = 'give-cache-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;
@@ -725,21 +725,21 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function flush_cache( $force = false ) {
+	public static function flush_cache($force = false) {
 		if (
 			$force
-			|| ( Give_Admin_Settings::is_saving_settings() && isset( $_POST['cache'] ) && give_is_setting_enabled( give_clean( $_POST['cache'] ) ) )
-			|| ( wp_doing_ajax() &&  isset( $_GET['action'] ) && 'give_cache_flush' === give_clean( $_GET['action'] ) )
+			|| (Give_Admin_Settings::is_saving_settings() && isset($_POST['cache']) && give_is_setting_enabled(give_clean($_POST['cache'])))
+			|| (wp_doing_ajax() && isset($_GET['action']) && 'give_cache_flush' === give_clean($_GET['action']))
 		) {
-			self::$instance->get_incrementer( true );
-			self::$instance->get_incrementer( true, 'give-cache-incrementer' );
+			self::$instance->get_incrementer(true);
+			self::$instance->get_incrementer(true, 'give-cache-incrementer');
 
 			/**
 			 * Fire the action when all cache deleted.
 			 *
 			 * @since 2.1.0
 			 */
-			do_action( 'give_fluched_cache' );
+			do_action('give_fluched_cache');
 
 			return true;
 		}
@@ -758,23 +758,23 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function filter_group_name( $group ) {
+	private function filter_group_name($group) {
 		/**
 		 * Filter the group name
 		 *
 		 * @since 2.1.0
 		 */
-		$filtered_group = apply_filters( 'give_cache_filter_group_name', '', $group );
+		$filtered_group = apply_filters('give_cache_filter_group_name', '', $group);
 
-		if ( empty( $filtered_group ) ) {
+		if (empty($filtered_group)) {
 
-			switch ( $group ) {
+			switch ($group) {
 				case 'give-db-queries':
 					$incrementer = self::$instance->get_incrementer();
 					break;
 
 				default:
-					$incrementer = self::$instance->get_incrementer( false, 'give-cache-incrementer' );
+					$incrementer = self::$instance->get_incrementer(false, 'give-cache-incrementer');
 
 			}
 

includes/class-give-db-sequential-ordering.php

Spacing +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_sequential_ordering';
+		$this->table_name  = $wpdb->prefix.'give_sequential_ordering';
 		$this->primary_key = 'id';
 		$this->version     = '1.0';
 
@@ -108,18 +108,18 @@ 
         PRIMARY KEY  (id)
         ) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		if ( ! empty( $payment_ID ) ) {
+		if ( ! empty($payment_ID)) {
 			$auto_increment = $payment_ID + 1;
-			$wpdb->query( "ALTER TABLE {$this->table_name} AUTO_INCREMENT={$auto_increment};" );
-			give_update_option( 'sequential-ordering_number', $auto_increment );
+			$wpdb->query("ALTER TABLE {$this->table_name} AUTO_INCREMENT={$auto_increment};");
+			give_update_option('sequential-ordering_number', $auto_increment);
 		} else {
-			give_update_option( 'sequential-ordering_number', 1 );
+			give_update_option('sequential-ordering_number', 1);
 		}
 
-		update_option( $this->table_name . '_db_version', $this->version, false );
+		update_option($this->table_name.'_db_version', $this->version, false);
 	}
 
 

includes/admin/emails/class-email-access-email.php

Spacing +49 added lines, -49 removed lines

@@ -11,11 +11,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Email_Access_Email' ) ) :
+if ( ! class_exists('Give_Email_Access_Email')) :
 
 	/**
 	 * Give_Email_Access_Email
@@ -31,31 +31,31 @@ 
 		 * @since  2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                           => 'email-access',
-				'label'                        => __( 'Email access', 'give' ),
-				'description'                  => __( 'Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give' ),
-				'notification_status'          => give_get_option( 'email_access', 'disabled' ),
+				'label'                        => __('Email access', 'give'),
+				'description'                  => __('Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give'),
+				'notification_status'          => give_get_option('email_access', 'disabled'),
 				'form_metabox_setting'         => false,
 				'notification_status_editable' => false,
 				'email_tag_context'            => 'donor',
-				'recipient_group_name'         => __( 'Donor', 'give' ),
-				'default_email_subject'        => sprintf( __( 'Please confirm your email for %s', 'give' ), get_bloginfo( 'url' ) ),
+				'recipient_group_name'         => __('Donor', 'give'),
+				'default_email_subject'        => sprintf(__('Please confirm your email for %s', 'give'), get_bloginfo('url')),
 				'default_email_message'        => $this->get_default_email_message(),
-				'default_email_header'         => __( 'Confirm Email', 'give' ),
+				'default_email_header'         => __('Confirm Email', 'give'),
 				'notices' => array(
 					'non-notification-status-editable' => sprintf(
 						'%1$s <a href="%2$s">%3$s &raquo;</a>',
-						__( 'This notification is automatically toggled based on whether the email access is enabled or not.', 'give' ),
-						esc_url( admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control') ),
-						__( 'Edit Setting', 'give' )
+						__('This notification is automatically toggled based on whether the email access is enabled or not.', 'give'),
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control')),
+						__('Edit Setting', 'give')
 					)
 				),
-			) );
+			));
 
-			add_filter( "give_{$this->config['id']}_email_notification", array( $this, 'setup_email_notification' ), 10, 2 );
-			add_action( 'give_save_settings_give_settings', array( $this, 'set_notification_status' ), 10, 2 );
-			add_filter( 'give_email_preview_header', array( $this, 'email_preview_header' ), 10, 2 );
+			add_filter("give_{$this->config['id']}_email_notification", array($this, 'setup_email_notification'), 10, 2);
+			add_action('give_save_settings_give_settings', array($this, 'set_notification_status'), 10, 2);
+			add_filter('give_email_preview_header', array($this, 'email_preview_header'), 10, 2);
 		}
 
 		/**
@@ -68,11 +68,11 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_subject( $form_id = null ) {
+		public function get_email_subject($form_id = null) {
 			$subject = wp_strip_all_tags(
 				Give_Email_Notification_Util::get_value(
 					$this,
-					Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_subject',
+					Give_Email_Setting_Field::get_prefix($this, $form_id).'email_subject',
 					$form_id,
 					$this->config['default_email_subject']
 				)
@@ -84,14 +84,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$subject = apply_filters( 'give_email_access_token_subject', $subject );
+			$subject = apply_filters('give_email_access_token_subject', $subject);
 
 			/**
 			 * Filters the donation notification subject.
 			 *
 			 * @since 2.0
 			 */
-			$subject = apply_filters( "give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id );
+			$subject = apply_filters("give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id);
 
 			return $subject;
 		}
@@ -107,10 +107,10 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_message( $form_id = null ) {
+		public function get_email_message($form_id = null) {
 			$message = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_message',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_message',
 				$form_id,
 				$this->config['default_email_message']
 			);
@@ -121,14 +121,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$message = apply_filters( 'give_email_access_token_message', $message );
+			$message = apply_filters('give_email_access_token_message', $message);
 
 			/**
 			 * Filter the email message
 			 *
 			 * @since 2.0
 			 */
-			$message = apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id );
+			$message = apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id);
 
 			return $message;
 		}
@@ -143,7 +143,7 @@ 
 		 * @param int $form_id
 		 * @return array
 		 */
-		public function get_email_attachments( $form_id = null ) {
+		public function get_email_attachments($form_id = null) {
 			/**
 			 * Filters the donation notification email attachments.
 			 * By default, there is no attachment but plugins can hook in to provide one more multiple.
@@ -151,7 +151,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$attachments = apply_filters( 'give_admin_donation_notification_attachments', array() );
+			$attachments = apply_filters('give_admin_donation_notification_attachments', array());
 
 			/**
 			 * Filters the donation notification email attachments.
@@ -159,7 +159,7 @@ 
 			 *
 			 * @since 2.0
 			 */
-			$attachments = apply_filters( "give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id );
+			$attachments = apply_filters("give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id);
 
 			return $attachments;
 		}
@@ -174,11 +174,11 @@ 
 		 * @return string
 		 */
 		public function get_default_email_message() {
-			$message = __( 'Please click the link to access your donation history on {site_url}. If you did not request this email, please contact {admin_email}.', 'give' ) . "\n\n";
-			$message .= '{email_access_link}' . "\n\n";
+			$message = __('Please click the link to access your donation history on {site_url}. If you did not request this email, please contact {admin_email}.', 'give')."\n\n";
+			$message .= '{email_access_link}'."\n\n";
 			$message .= "\n\n";
-			$message .= __( 'Sincerely,', 'give' ) . "\n";
-			$message .= get_bloginfo( 'name' ) . "\n";
+			$message .= __('Sincerely,', 'give')."\n";
+			$message .= get_bloginfo('name')."\n";
 
 			/**
 			 * Filter the new donation email message
@@ -187,7 +187,7 @@ 
 			 *
 			 * @param string $message
 			 */
-			return apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this );
+			return apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this);
 		}
 
 
@@ -203,7 +203,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_name = apply_filters( 'give_donation_from_name', Give()->emails->get_from_name() );
+			$from_name = apply_filters('give_donation_from_name', Give()->emails->get_from_name());
 
 			/**
 			 * Filters the from email.
@@ -211,19 +211,19 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_email = apply_filters( 'give_donation_from_address', Give()->emails->get_from_address() );
+			$from_email = apply_filters('give_donation_from_address', Give()->emails->get_from_address());
 
-			Give()->emails->__set( 'from_name', $from_name );
-			Give()->emails->__set( 'from_email', $from_email );
-			Give()->emails->__set( 'heading', apply_filters( 'give_email_access_token_heading', $this->get_email_header() ) );
+			Give()->emails->__set('from_name', $from_name);
+			Give()->emails->__set('from_email', $from_email);
+			Give()->emails->__set('heading', apply_filters('give_email_access_token_heading', $this->get_email_header()));
 			/**
 			 * Filters the donation notification email headers.
 			 *
 			 * @since 1.0
 			 */
-			$headers = apply_filters( 'give_admin_donation_notification_headers', Give()->emails->get_headers() );
+			$headers = apply_filters('give_admin_donation_notification_headers', Give()->emails->get_headers());
 
-			Give()->emails->__set( 'headers', $headers );
+			Give()->emails->__set('headers', $headers);
 		}
 
 		/**
@@ -237,8 +237,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public function setup_email_notification( $donor_id, $email ) {
-			$donor = Give()->donors->get_donor_by( 'email', $email );
+		public function setup_email_notification($donor_id, $email) {
+			$donor = Give()->donors->get_donor_by('email', $email);
 			$this->recipient_email = $email;
 
 			// Set email data.
@@ -263,17 +263,17 @@ 
 		 * @param $update_options
 		 * @param $option_name
 		 */
-		public function set_notification_status( $update_options, $option_name ) {
+		public function set_notification_status($update_options, $option_name) {
 			// Get updated settings.
 			$update_options = give_get_settings();
 
 			if (
-				! empty( $update_options['email_access'] )
-				&& ! empty( $update_options[ "{$this->config['id']}_notification" ] )
-				&& $update_options['email_access'] !== $update_options[ "{$this->config['id']}_notification" ]
+				! empty($update_options['email_access'])
+				&& ! empty($update_options["{$this->config['id']}_notification"])
+				&& $update_options['email_access'] !== $update_options["{$this->config['id']}_notification"]
 			) {
-				$update_options[ "{$this->config['id']}_notification" ] = $update_options['email_access'];
-				update_option( $option_name, $update_options, false );
+				$update_options["{$this->config['id']}_notification"] = $update_options['email_access'];
+				update_option($option_name, $update_options, false);
 			}
 		}
 
@@ -288,8 +288,8 @@ 
 		 * @param Give_Email_Access_Email $email
 		 * @return string
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
-			if( $this->config['id'] === $email->config['id'] ) {
+		public function email_preview_header($email_preview_header, $email) {
+			if ($this->config['id'] === $email->config['id']) {
 				$email_preview_header = '';
 			}