WordImpress / Give

includes/misc-functions.php

Doc Comments +5 added lines, -5 removed lines

@@ -335,7 +335,7 @@ 
  * Returns the webhost this site is using if possible
  *
  * @since 1.0
- * @return mixed string $host if detected, false otherwise
+ * @return string string $host if detected, false otherwise
  */
 function give_get_host() {
 	$host = false;
@@ -506,7 +506,7 @@ 
  *
  * @since 1.0
  *
- * @param unknown $n
+ * @param integer $n
  *
  * @return string Short month name
  */
@@ -636,9 +636,9 @@ 
 /**
  * Create SVG library function
  *
- * @param $icon
+ * @param string $icon
  *
- * @return mixed
+ * @return string
  */
 function give_svg_icons( $icon ) {
 
@@ -696,7 +696,7 @@ 
      *
      * @param array $input A multi-dimensional array (record set) from which to pull
      *                     a column of values.
-     * @param mixed $columnKey The column of values to return. This value may be the
+     * @param string $columnKey The column of values to return. This value may be the
      *                         integer key of the column you wish to retrieve, or it
      *                         may be the string key name for an associative array.
      * @param mixed $indexKey (Optional.) The column to use as the index/keys for

Indentation +101 added lines, -101 removed lines

@@ -687,106 +687,106 @@
  */
 
 if (!function_exists('array_column')) {
-    /**
-     * Returns the values from a single column of the input array, identified by
-     * the $columnKey.
-     *
-     * Optionally, you may provide an $indexKey to index the values in the returned
-     * array by the values from the $indexKey column in the input array.
-     *
-     * @param array $input A multi-dimensional array (record set) from which to pull
-     *                     a column of values.
-     * @param mixed $columnKey The column of values to return. This value may be the
-     *                         integer key of the column you wish to retrieve, or it
-     *                         may be the string key name for an associative array.
-     * @param mixed $indexKey (Optional.) The column to use as the index/keys for
-     *                        the returned array. This value may be the integer key
-     *                        of the column, or it may be the string key name.
-     * @return array
-     */
-    function array_column($input = null, $columnKey = null, $indexKey = null)
-    {
-        // Using func_get_args() in order to check for proper number of
-        // parameters and trigger errors exactly as the built-in array_column()
-        // does in PHP 5.5.
-        $argc = func_num_args();
-        $params = func_get_args();
-
-        if ($argc < 2) {
-            trigger_error("array_column() expects at least 2 parameters, {$argc} given", E_USER_WARNING);
-            return null;
-        }
-
-        if (!is_array($params[0])) {
-            trigger_error(
-                'array_column() expects parameter 1 to be array, ' . gettype($params[0]) . ' given',
-                E_USER_WARNING
-            );
-            return null;
-        }
-
-        if (!is_int($params[1])
-            && !is_float($params[1])
-            && !is_string($params[1])
-            && $params[1] !== null
-            && !(is_object($params[1]) && method_exists($params[1], '__toString'))
-        ) {
-            trigger_error('array_column(): The column key should be either a string or an integer', E_USER_WARNING);
-            return false;
-        }
-
-        if (isset($params[2])
-            && !is_int($params[2])
-            && !is_float($params[2])
-            && !is_string($params[2])
-            && !(is_object($params[2]) && method_exists($params[2], '__toString'))
-        ) {
-            trigger_error('array_column(): The index key should be either a string or an integer', E_USER_WARNING);
-            return false;
-        }
-
-        $paramsInput = $params[0];
-        $paramsColumnKey = ($params[1] !== null) ? (string) $params[1] : null;
-
-        $paramsIndexKey = null;
-        if (isset($params[2])) {
-            if (is_float($params[2]) || is_int($params[2])) {
-                $paramsIndexKey = (int) $params[2];
-            } else {
-                $paramsIndexKey = (string) $params[2];
-            }
-        }
-
-        $resultArray = array();
-
-        foreach ($paramsInput as $row) {
-            $key = $value = null;
-            $keySet = $valueSet = false;
-
-            if ($paramsIndexKey !== null && array_key_exists($paramsIndexKey, $row)) {
-                $keySet = true;
-                $key = (string) $row[$paramsIndexKey];
-            }
-
-            if ($paramsColumnKey === null) {
-                $valueSet = true;
-                $value = $row;
-            } elseif (is_array($row) && array_key_exists($paramsColumnKey, $row)) {
-                $valueSet = true;
-                $value = $row[$paramsColumnKey];
-            }
-
-            if ($valueSet) {
-                if ($keySet) {
-                    $resultArray[$key] = $value;
-                } else {
-                    $resultArray[] = $value;
-                }
-            }
-
-        }
-
-        return $resultArray;
-    }
+	/**
+	 * Returns the values from a single column of the input array, identified by
+	 * the $columnKey.
+	 *
+	 * Optionally, you may provide an $indexKey to index the values in the returned
+	 * array by the values from the $indexKey column in the input array.
+	 *
+	 * @param array $input A multi-dimensional array (record set) from which to pull
+	 *                     a column of values.
+	 * @param mixed $columnKey The column of values to return. This value may be the
+	 *                         integer key of the column you wish to retrieve, or it
+	 *                         may be the string key name for an associative array.
+	 * @param mixed $indexKey (Optional.) The column to use as the index/keys for
+	 *                        the returned array. This value may be the integer key
+	 *                        of the column, or it may be the string key name.
+	 * @return array
+	 */
+	function array_column($input = null, $columnKey = null, $indexKey = null)
+	{
+		// Using func_get_args() in order to check for proper number of
+		// parameters and trigger errors exactly as the built-in array_column()
+		// does in PHP 5.5.
+		$argc = func_num_args();
+		$params = func_get_args();
+
+		if ($argc < 2) {
+			trigger_error("array_column() expects at least 2 parameters, {$argc} given", E_USER_WARNING);
+			return null;
+		}
+
+		if (!is_array($params[0])) {
+			trigger_error(
+				'array_column() expects parameter 1 to be array, ' . gettype($params[0]) . ' given',
+				E_USER_WARNING
+			);
+			return null;
+		}
+
+		if (!is_int($params[1])
+			&& !is_float($params[1])
+			&& !is_string($params[1])
+			&& $params[1] !== null
+			&& !(is_object($params[1]) && method_exists($params[1], '__toString'))
+		) {
+			trigger_error('array_column(): The column key should be either a string or an integer', E_USER_WARNING);
+			return false;
+		}
+
+		if (isset($params[2])
+			&& !is_int($params[2])
+			&& !is_float($params[2])
+			&& !is_string($params[2])
+			&& !(is_object($params[2]) && method_exists($params[2], '__toString'))
+		) {
+			trigger_error('array_column(): The index key should be either a string or an integer', E_USER_WARNING);
+			return false;
+		}
+
+		$paramsInput = $params[0];
+		$paramsColumnKey = ($params[1] !== null) ? (string) $params[1] : null;
+
+		$paramsIndexKey = null;
+		if (isset($params[2])) {
+			if (is_float($params[2]) || is_int($params[2])) {
+				$paramsIndexKey = (int) $params[2];
+			} else {
+				$paramsIndexKey = (string) $params[2];
+			}
+		}
+
+		$resultArray = array();
+
+		foreach ($paramsInput as $row) {
+			$key = $value = null;
+			$keySet = $valueSet = false;
+
+			if ($paramsIndexKey !== null && array_key_exists($paramsIndexKey, $row)) {
+				$keySet = true;
+				$key = (string) $row[$paramsIndexKey];
+			}
+
+			if ($paramsColumnKey === null) {
+				$valueSet = true;
+				$value = $row;
+			} elseif (is_array($row) && array_key_exists($paramsColumnKey, $row)) {
+				$valueSet = true;
+				$value = $row[$paramsColumnKey];
+			}
+
+			if ($valueSet) {
+				if ($keySet) {
+					$resultArray[$key] = $value;
+				} else {
+					$resultArray[] = $value;
+				}
+			}
+
+		}
+
+		return $resultArray;
+	}
 
 }
\ No newline at end of file

Spacing +124 added lines, -124 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
 function give_is_test_mode() {
 	global $give_options;
 
-	$ret = ! empty( $give_options['test_mode'] );
+	$ret = ! empty($give_options['test_mode']);
 
-	return (bool) apply_filters( 'give_is_test_mode', $ret );
+	return (bool) apply_filters('give_is_test_mode', $ret);
 }
 
 /**
@@ -37,9 +37,9 @@ 
  */
 function give_get_currency() {
 	global $give_options;
-	$currency = isset( $give_options['currency'] ) ? $give_options['currency'] : 'USD';
+	$currency = isset($give_options['currency']) ? $give_options['currency'] : 'USD';
 
-	return apply_filters( 'give_currency', $currency );
+	return apply_filters('give_currency', $currency);
 }
 
 
@@ -51,36 +51,36 @@ 
  */
 function give_get_currencies() {
 	$currencies = array(
-		'USD'  => __( 'US Dollars ($)', 'give' ),
-		'EUR'  => __( 'Euros (€)', 'give' ),
-		'GBP'  => __( 'Pounds Sterling (£)', 'give' ),
-		'AUD'  => __( 'Australian Dollars ($)', 'give' ),
-		'BRL'  => __( 'Brazilian Real (R$)', 'give' ),
-		'CAD'  => __( 'Canadian Dollars ($)', 'give' ),
-		'CZK'  => __( 'Czech Koruna', 'give' ),
-		'DKK'  => __( 'Danish Krone', 'give' ),
-		'HKD'  => __( 'Hong Kong Dollar ($)', 'give' ),
-		'HUF'  => __( 'Hungarian Forint', 'give' ),
-		'ILS'  => __( 'Israeli Shekel (₪)', 'give' ),
-		'JPY'  => __( 'Japanese Yen (¥)', 'give' ),
-		'MYR'  => __( 'Malaysian Ringgits', 'give' ),
-		'MXN'  => __( 'Mexican Peso ($)', 'give' ),
-		'NZD'  => __( 'New Zealand Dollar ($)', 'give' ),
-		'NOK'  => __( 'Norwegian Krone (Kr.)', 'give' ),
-		'PHP'  => __( 'Philippine Pesos', 'give' ),
-		'PLN'  => __( 'Polish Zloty', 'give' ),
-		'SGD'  => __( 'Singapore Dollar ($)', 'give' ),
-		'SEK'  => __( 'Swedish Krona', 'give' ),
-		'CHF'  => __( 'Swiss Franc', 'give' ),
-		'TWD'  => __( 'Taiwan New Dollars', 'give' ),
-		'THB'  => __( 'Thai Baht (฿)', 'give' ),
-		'INR'  => __( 'Indian Rupee (₹)', 'give' ),
-		'TRY'  => __( 'Turkish Lira (₺)', 'give' ),
-		'RIAL' => __( 'Iranian Rial (﷼)', 'give' ),
-		'RUB'  => __( 'Russian Rubles', 'give' )
+		'USD'  => __('US Dollars ($)', 'give'),
+		'EUR'  => __('Euros (€)', 'give'),
+		'GBP'  => __('Pounds Sterling (£)', 'give'),
+		'AUD'  => __('Australian Dollars ($)', 'give'),
+		'BRL'  => __('Brazilian Real (R$)', 'give'),
+		'CAD'  => __('Canadian Dollars ($)', 'give'),
+		'CZK'  => __('Czech Koruna', 'give'),
+		'DKK'  => __('Danish Krone', 'give'),
+		'HKD'  => __('Hong Kong Dollar ($)', 'give'),
+		'HUF'  => __('Hungarian Forint', 'give'),
+		'ILS'  => __('Israeli Shekel (₪)', 'give'),
+		'JPY'  => __('Japanese Yen (¥)', 'give'),
+		'MYR'  => __('Malaysian Ringgits', 'give'),
+		'MXN'  => __('Mexican Peso ($)', 'give'),
+		'NZD'  => __('New Zealand Dollar ($)', 'give'),
+		'NOK'  => __('Norwegian Krone (Kr.)', 'give'),
+		'PHP'  => __('Philippine Pesos', 'give'),
+		'PLN'  => __('Polish Zloty', 'give'),
+		'SGD'  => __('Singapore Dollar ($)', 'give'),
+		'SEK'  => __('Swedish Krona', 'give'),
+		'CHF'  => __('Swiss Franc', 'give'),
+		'TWD'  => __('Taiwan New Dollars', 'give'),
+		'THB'  => __('Thai Baht (฿)', 'give'),
+		'INR'  => __('Indian Rupee (₹)', 'give'),
+		'TRY'  => __('Turkish Lira (₺)', 'give'),
+		'RIAL' => __('Iranian Rial (﷼)', 'give'),
+		'RUB'  => __('Russian Rubles', 'give')
 	);
 
-	return apply_filters( 'give_currencies', $currencies );
+	return apply_filters('give_currencies', $currencies);
 }
 
 
@@ -94,12 +94,12 @@ 
  *
  * @return string           The symbol to use for the currency
  */
-function give_currency_symbol( $currency = '' ) {
+function give_currency_symbol($currency = '') {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
-	switch ( $currency ) :
+	switch ($currency) :
 		case "GBP" :
 			$symbol = '£';
 			break;
@@ -128,7 +128,7 @@ 
 			break;
 	endswitch;
 
-	return apply_filters( 'give_currency_symbol', $symbol, $currency );
+	return apply_filters('give_currency_symbol', $symbol, $currency);
 }
 
 
@@ -140,25 +140,25 @@ 
  */
 function give_get_current_page_url() {
 
-	if ( is_front_page() ) :
+	if (is_front_page()) :
 		$page_url = home_url();
 	else :
 		$page_url = 'http';
 
-		if ( isset( $_SERVER["HTTPS"] ) && $_SERVER["HTTPS"] == "on" ) {
+		if (isset($_SERVER["HTTPS"]) && $_SERVER["HTTPS"] == "on") {
 			$page_url .= "s";
 		}
 
 		$page_url .= "://";
 
-		if ( isset( $_SERVER["SERVER_PORT"] ) && $_SERVER["SERVER_PORT"] != "80" ) {
-			$page_url .= $_SERVER["SERVER_NAME"] . ":" . $_SERVER["SERVER_PORT"] . $_SERVER["REQUEST_URI"];
+		if (isset($_SERVER["SERVER_PORT"]) && $_SERVER["SERVER_PORT"] != "80") {
+			$page_url .= $_SERVER["SERVER_NAME"].":".$_SERVER["SERVER_PORT"].$_SERVER["REQUEST_URI"];
 		} else {
-			$page_url .= $_SERVER["SERVER_NAME"] . $_SERVER["REQUEST_URI"];
+			$page_url .= $_SERVER["SERVER_NAME"].$_SERVER["REQUEST_URI"];
 		}
 	endif;
 
-	return apply_filters( 'give_get_current_page_url', esc_url( $page_url ) );
+	return apply_filters('give_get_current_page_url', esc_url($page_url));
 }
 
 
@@ -179,15 +179,15 @@ 
 	 */
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( count( $gateways ) == 1 && ! isset( $gateways['paypal'] ) && ! isset( $gateways['manual'] ) ) {
+	if (count($gateways) == 1 && ! isset($gateways['paypal']) && ! isset($gateways['manual'])) {
 		$ret = true;
-	} else if ( count( $gateways ) == 1 ) {
+	} else if (count($gateways) == 1) {
 		$ret = false;
-	} else if ( count( $gateways ) == 2 && isset( $gateways['paypal'] ) && isset( $gateways['manual'] ) ) {
+	} else if (count($gateways) == 2 && isset($gateways['paypal']) && isset($gateways['manual'])) {
 		$ret = false;
 	}
 
-	return (bool) apply_filters( 'give_verify_credit_cards', $ret );
+	return (bool) apply_filters('give_verify_credit_cards', $ret);
 }
 
 
@@ -202,9 +202,9 @@ 
 
 	global $give_options;
 
-	$ret = ! empty( $give_options['logged_in_only'] );
+	$ret = ! empty($give_options['logged_in_only']);
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret );
+	return (bool) apply_filters('give_logged_in_only', $ret);
 
 }
 
@@ -218,26 +218,26 @@ 
 function give_get_timezone_id() {
 
 	// if site timezone string exists, return it
-	if ( $timezone = get_option( 'timezone_string' ) ) {
+	if ($timezone = get_option('timezone_string')) {
 		return $timezone;
 	}
 
 	// get UTC offset, if it isn't set return UTC
-	if ( ! ( $utc_offset = 3600 * get_option( 'gmt_offset', 0 ) ) ) {
+	if ( ! ($utc_offset = 3600 * get_option('gmt_offset', 0))) {
 		return 'UTC';
 	}
 
 	// attempt to guess the timezone string from the UTC offset
-	$timezone = timezone_name_from_abbr( '', $utc_offset );
+	$timezone = timezone_name_from_abbr('', $utc_offset);
 
 	// last try, guess timezone string manually
-	if ( $timezone === false ) {
+	if ($timezone === false) {
 
-		$is_dst = date( 'I' );
+		$is_dst = date('I');
 
-		foreach ( timezone_abbreviations_list() as $abbr ) {
-			foreach ( $abbr as $city ) {
-				if ( $city['dst'] == $is_dst && $city['offset'] == $utc_offset ) {
+		foreach (timezone_abbreviations_list() as $abbr) {
+			foreach ($abbr as $city) {
+				if ($city['dst'] == $is_dst && $city['offset'] == $utc_offset) {
 					return $city['timezone_id'];
 				}
 			}
@@ -261,17 +261,17 @@ 
 
 	$ip = '127.0.0.1';
 
-	if ( ! empty( $_SERVER['HTTP_CLIENT_IP'] ) ) {
+	if ( ! empty($_SERVER['HTTP_CLIENT_IP'])) {
 		//check ip from share internet
 		$ip = $_SERVER['HTTP_CLIENT_IP'];
-	} elseif ( ! empty( $_SERVER['HTTP_X_FORWARDED_FOR'] ) ) {
+	} elseif ( ! empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
 		//to check ip is pass from proxy
 		$ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
-	} elseif ( ! empty( $_SERVER['REMOTE_ADDR'] ) ) {
+	} elseif ( ! empty($_SERVER['REMOTE_ADDR'])) {
 		$ip = $_SERVER['REMOTE_ADDR'];
 	}
 
-	return apply_filters( 'give_get_ip', $ip );
+	return apply_filters('give_get_ip', $ip);
 }
 
 
@@ -286,8 +286,8 @@ 
  *
  * @uses  Give()->session->set()
  */
-function give_set_purchase_session( $purchase_data = array() ) {
-	Give()->session->set( 'give_purchase', $purchase_data );
+function give_set_purchase_session($purchase_data = array()) {
+	Give()->session->set('give_purchase', $purchase_data);
 }
 
 /**
@@ -301,7 +301,7 @@ 
  * @return mixed array | false
  */
 function give_get_purchase_session() {
-	return Give()->session->get( 'give_purchase' );
+	return Give()->session->get('give_purchase');
 }
 
 /**
@@ -316,14 +316,14 @@ 
  *
  * @return string
  */
-function give_get_purchase_summary( $purchase_data, $email = true ) {
+function give_get_purchase_summary($purchase_data, $email = true) {
 	$summary = '';
 
-	if ( $email ) {
-		$summary .= $purchase_data['user_email'] . ' - ';
+	if ($email) {
+		$summary .= $purchase_data['user_email'].' - ';
 	}
 
-	$summary .= get_the_title( $purchase_data['post_data']['give-form-id'] );
+	$summary .= get_the_title($purchase_data['post_data']['give-form-id']);
 
 	return $summary;
 }
@@ -340,31 +340,31 @@ 
 function give_get_host() {
 	$host = false;
 
-	if ( defined( 'WPE_APIKEY' ) ) {
+	if (defined('WPE_APIKEY')) {
 		$host = 'WP Engine';
-	} elseif ( defined( 'PAGELYBIN' ) ) {
+	} elseif (defined('PAGELYBIN')) {
 		$host = 'Pagely';
-	} elseif ( DB_HOST == 'localhost:/tmp/mysql5.sock' ) {
+	} elseif (DB_HOST == 'localhost:/tmp/mysql5.sock') {
 		$host = 'ICDSoft';
-	} elseif ( DB_HOST == 'mysqlv5' ) {
+	} elseif (DB_HOST == 'mysqlv5') {
 		$host = 'NetworkSolutions';
-	} elseif ( strpos( DB_HOST, 'ipagemysql.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, 'ipagemysql.com') !== false) {
 		$host = 'iPage';
-	} elseif ( strpos( DB_HOST, 'ipowermysql.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, 'ipowermysql.com') !== false) {
 		$host = 'IPower';
-	} elseif ( strpos( DB_HOST, '.gridserver.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.gridserver.com') !== false) {
 		$host = 'MediaTemple Grid';
-	} elseif ( strpos( DB_HOST, '.pair.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.pair.com') !== false) {
 		$host = 'pair Networks';
-	} elseif ( strpos( DB_HOST, '.stabletransit.com' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.stabletransit.com') !== false) {
 		$host = 'Rackspace Cloud';
-	} elseif ( strpos( DB_HOST, '.sysfix.eu' ) !== false ) {
+	} elseif (strpos(DB_HOST, '.sysfix.eu') !== false) {
 		$host = 'SysFix.eu Power Hosting';
-	} elseif ( strpos( $_SERVER['SERVER_NAME'], 'Flywheel' ) !== false ) {
+	} elseif (strpos($_SERVER['SERVER_NAME'], 'Flywheel') !== false) {
 		$host = 'Flywheel';
 	} else {
 		// Adding a general fallback for data gathering
-		$host = 'DBH: ' . DB_HOST . ', SRV: ' . $_SERVER['SERVER_NAME'];
+		$host = 'DBH: '.DB_HOST.', SRV: '.$_SERVER['SERVER_NAME'];
 	}
 
 	return $host;
@@ -380,67 +380,67 @@ 
  *
  * @return bool true if host matches, false if not
  */
-function give_is_host( $host = false ) {
+function give_is_host($host = false) {
 
 	$return = false;
 
-	if ( $host ) {
-		$host = str_replace( ' ', '', strtolower( $host ) );
+	if ($host) {
+		$host = str_replace(' ', '', strtolower($host));
 
-		switch ( $host ) {
+		switch ($host) {
 			case 'wpengine':
-				if ( defined( 'WPE_APIKEY' ) ) {
+				if (defined('WPE_APIKEY')) {
 					$return = true;
 				}
 				break;
 			case 'pagely':
-				if ( defined( 'PAGELYBIN' ) ) {
+				if (defined('PAGELYBIN')) {
 					$return = true;
 				}
 				break;
 			case 'icdsoft':
-				if ( DB_HOST == 'localhost:/tmp/mysql5.sock' ) {
+				if (DB_HOST == 'localhost:/tmp/mysql5.sock') {
 					$return = true;
 				}
 				break;
 			case 'networksolutions':
-				if ( DB_HOST == 'mysqlv5' ) {
+				if (DB_HOST == 'mysqlv5') {
 					$return = true;
 				}
 				break;
 			case 'ipage':
-				if ( strpos( DB_HOST, 'ipagemysql.com' ) !== false ) {
+				if (strpos(DB_HOST, 'ipagemysql.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'ipower':
-				if ( strpos( DB_HOST, 'ipowermysql.com' ) !== false ) {
+				if (strpos(DB_HOST, 'ipowermysql.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'mediatemplegrid':
-				if ( strpos( DB_HOST, '.gridserver.com' ) !== false ) {
+				if (strpos(DB_HOST, '.gridserver.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'pairnetworks':
-				if ( strpos( DB_HOST, '.pair.com' ) !== false ) {
+				if (strpos(DB_HOST, '.pair.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'rackspacecloud':
-				if ( strpos( DB_HOST, '.stabletransit.com' ) !== false ) {
+				if (strpos(DB_HOST, '.stabletransit.com') !== false) {
 					$return = true;
 				}
 				break;
 			case 'sysfix.eu':
 			case 'sysfix.eupowerhosting':
-				if ( strpos( DB_HOST, '.sysfix.eu' ) !== false ) {
+				if (strpos(DB_HOST, '.sysfix.eu') !== false) {
 					$return = true;
 				}
 				break;
 			case 'flywheel':
-				if ( strpos( $_SERVER['SERVER_NAME'], 'Flywheel' ) !== false ) {
+				if (strpos($_SERVER['SERVER_NAME'], 'Flywheel') !== false) {
 					$return = true;
 				}
 				break;
@@ -461,8 +461,8 @@ 
  * @return string $post_id
  */
 function give_get_admin_post_id() {
-	$post_id = isset( $_GET['post'] ) ? $_GET['post'] : null;
-	if ( ! $post_id && isset( $_POST['post_id'] ) ) {
+	$post_id = isset($_GET['post']) ? $_GET['post'] : null;
+	if ( ! $post_id && isset($_POST['post_id'])) {
 		$post_id = $_POST['post_id'];
 	}
 
@@ -480,11 +480,11 @@ 
  *
  * @return bool $ret True if guest checkout is enabled, false otherwise
  */
-function give_no_guest_checkout( $form_id ) {
+function give_no_guest_checkout($form_id) {
 
-	$ret = get_post_meta( $form_id, '_give_logged_in_only', true );
+	$ret = get_post_meta($form_id, '_give_logged_in_only', true);
 
-	return (bool) apply_filters( 'give_no_guest_checkout', $ret );
+	return (bool) apply_filters('give_no_guest_checkout', $ret);
 }
 
 
@@ -495,7 +495,7 @@ 
  * @return string Arg separator output
  */
 function give_get_php_arg_separator_output() {
-	return ini_get( 'arg_separator.output' );
+	return ini_get('arg_separator.output');
 }
 
 
@@ -510,10 +510,10 @@ 
  *
  * @return string Short month name
  */
-function give_month_num_to_name( $n ) {
-	$timestamp = mktime( 0, 0, 0, $n, 1, 2005 );
+function give_month_num_to_name($n) {
+	$timestamp = mktime(0, 0, 0, $n, 1, 2005);
 
-	return date_i18n( "M", $timestamp );
+	return date_i18n("M", $timestamp);
 }
 
 
@@ -526,10 +526,10 @@ 
  *
  * @return bool Whether or not function is disabled.
  */
-function give_is_func_disabled( $function ) {
-	$disabled = explode( ',', ini_get( 'disable_functions' ) );
+function give_is_func_disabled($function) {
+	$disabled = explode(',', ini_get('disable_functions'));
 
-	return in_array( $function, $disabled );
+	return in_array($function, $disabled);
 }
 
 
@@ -544,7 +544,7 @@ 
 
 		<form action="//givewp.us3.list-manage.com/subscribe/post?u=3ccb75d68bda4381e2f45794c&amp;id=12a081aa13" method="post" id="mc-embedded-subscribe-form" name="mc-embedded-subscribe-form" class="validate" target="_blank" novalidate>
 			<div class="give-newsletter-confirmation">
-				<p><?php _e( 'Thanks for Subscribing!', 'give' ); ?> :)</p>
+				<p><?php _e('Thanks for Subscribing!', 'give'); ?> :)</p>
 			</div>
 
 			<table class="form-table give-newsletter-form">
@@ -640,7 +640,7 @@ 
  *
  * @return mixed
  */
-function give_svg_icons( $icon ) {
+function give_svg_icons($icon) {
 
 	// Store your SVGs in an associative array
 	$svgs = array(
@@ -652,7 +652,7 @@ 
 	);
 
 	// Return the chosen icon's SVG string
-	return $svgs[ $icon ];
+	return $svgs[$icon];
 }
 
 /**
@@ -664,15 +664,15 @@ 
  *
  * @return mixed
  */
-function modify_nav_menu_meta_box_object( $post_type ) {
-	if ( isset( $post_type->name ) && $post_type->name == 'give_forms' ) {
+function modify_nav_menu_meta_box_object($post_type) {
+	if (isset($post_type->name) && $post_type->name == 'give_forms') {
 		$post_type->labels->name = 'Donation Forms';
 	}
 
 	return $post_type;
 }
 
-add_filter( 'nav_menu_meta_box_object', 'modify_nav_menu_meta_box_object' );
+add_filter('nav_menu_meta_box_object', 'modify_nav_menu_meta_box_object');
 
 
 /**
@@ -686,7 +686,7 @@ 
  * @license http://opensource.org/licenses/MIT MIT
  */
 
-if (!function_exists('array_column')) {
+if ( ! function_exists('array_column')) {
     /**
      * Returns the values from a single column of the input array, identified by
      * the $columnKey.
@@ -717,29 +717,29 @@ 
             return null;
         }
 
-        if (!is_array($params[0])) {
+        if ( ! is_array($params[0])) {
             trigger_error(
-                'array_column() expects parameter 1 to be array, ' . gettype($params[0]) . ' given',
+                'array_column() expects parameter 1 to be array, '.gettype($params[0]).' given',
                 E_USER_WARNING
             );
             return null;
         }
 
-        if (!is_int($params[1])
-            && !is_float($params[1])
-            && !is_string($params[1])
+        if ( ! is_int($params[1])
+            && ! is_float($params[1])
+            && ! is_string($params[1])
             && $params[1] !== null
-            && !(is_object($params[1]) && method_exists($params[1], '__toString'))
+            && ! (is_object($params[1]) && method_exists($params[1], '__toString'))
         ) {
             trigger_error('array_column(): The column key should be either a string or an integer', E_USER_WARNING);
             return false;
         }
 
         if (isset($params[2])
-            && !is_int($params[2])
-            && !is_float($params[2])
-            && !is_string($params[2])
-            && !(is_object($params[2]) && method_exists($params[2], '__toString'))
+            && ! is_int($params[2])
+            && ! is_float($params[2])
+            && ! is_string($params[2])
+            && ! (is_object($params[2]) && method_exists($params[2], '__toString'))
         ) {
             trigger_error('array_column(): The index key should be either a string or an integer', E_USER_WARNING);
             return false;

Braces +3 added lines, -1 removed lines

@@ -142,8 +142,10 @@
 
 	if ( is_front_page() ) :
 		$page_url = home_url();
-	else :
+	else {
+		:
 		$page_url = 'http';
+	}
 
 		if ( isset( $_SERVER["HTTPS"] ) && $_SERVER["HTTPS"] == "on" ) {
 			$page_url .= "s";

includes/payments/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -624,7 +624,7 @@ 
  * @param int $year      Year
  * @param int $hour      Hour
  *
- * @return int $earnings Earnings
+ * @return double $earnings Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 
@@ -946,7 +946,7 @@ 
  *
  * @param int $payment_id Payment ID
  *
- * @return array $user_info User Info Meta Values
+ * @return string $user_info User Info Meta Values
  */
 function give_get_payment_meta_user_info( $payment_id ) {
 	$payment_meta = give_get_payment_meta( $payment_id );
@@ -1639,7 +1639,7 @@ 
  * @param array $where
  * @param obj   $wp_comment_query WordPress Comment Query Object
  *
- * @return array $where
+ * @return string $where
  */
 function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
 	global $wpdb;

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * Payment Functions
- *
- * @package     Give
- * @subpackage  Payments
- * @copyright   Copyright (c) 2015, WordImpress
- * @license     http://opensource.org/licenses/gpl-1.0.php GNU Public License
- * @since       1.0
- */
+	 * Payment Functions
+	 *
+	 * @package     Give
+	 * @subpackage  Payments
+	 * @copyright   Copyright (c) 2015, WordImpress
+	 * @license     http://opensource.org/licenses/gpl-1.0.php GNU Public License
+	 * @since       1.0
+	 */
 
 // Exit if accessed directly
 if ( ! defined( 'ABSPATH' ) ) {

Spacing +426 added lines, -426 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,15 +32,15 @@ 
  *
  * @return object $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();
 }
@@ -55,44 +55,44 @@ 
  *
  * @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 = get_post( $value );
+			$payment = get_post($value);
 
-			if ( get_post_type( $payment ) != 'give_payment' ) {
+			if (get_post_type($payment) != 'give_payment') {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1
-			) );
+			));
 
-			if ( $payment ) {
+			if ($payment) {
 				$payment = $payment[0];
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1
-			) );
+			));
 
-			if ( $payment ) {
+			if ($payment) {
 				$payment = $payment[0];
 			}
 
@@ -102,7 +102,7 @@ 
 			return false;
 	}
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -118,98 +118,98 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise
  */
-function give_insert_payment( $payment_data = array() ) {
-	if ( empty( $payment_data ) ) {
+function give_insert_payment($payment_data = array()) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
 	// Make sure the payment is inserted with the correct timezone
-	date_default_timezone_set( give_get_timezone_id() );
+	date_default_timezone_set(give_get_timezone_id());
 
 	// Construct the payment title
-	if ( isset( $payment_data['user_info']['first_name'] ) || isset( $payment_data['user_info']['last_name'] ) ) {
-		$payment_title = $payment_data['user_info']['first_name'] . ' ' . $payment_data['user_info']['last_name'];
+	if (isset($payment_data['user_info']['first_name']) || isset($payment_data['user_info']['last_name'])) {
+		$payment_title = $payment_data['user_info']['first_name'].' '.$payment_data['user_info']['last_name'];
 	} else {
 		$payment_title = $payment_data['user_email'];
 	}
 
 	// Find the next payment number, if enabled
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number = give_get_next_payment_number();
 	}
 
-	$args = apply_filters( 'give_insert_payment_args', array(
+	$args = apply_filters('give_insert_payment_args', array(
 		'post_title'    => $payment_title,
-		'post_status'   => isset( $payment_data['status'] ) ? $payment_data['status'] : 'pending',
+		'post_status'   => isset($payment_data['status']) ? $payment_data['status'] : 'pending',
 		'post_type'     => 'give_payment',
-		'post_parent'   => isset( $payment_data['parent'] ) ? $payment_data['parent'] : null,
-		'post_date'     => isset( $payment_data['post_date'] ) ? $payment_data['post_date'] : null,
-		'post_date_gmt' => isset( $payment_data['post_date'] ) ? get_gmt_from_date( $payment_data['post_date'] ) : null
-	), $payment_data );
+		'post_parent'   => isset($payment_data['parent']) ? $payment_data['parent'] : null,
+		'post_date'     => isset($payment_data['post_date']) ? $payment_data['post_date'] : null,
+		'post_date_gmt' => isset($payment_data['post_date']) ? get_gmt_from_date($payment_data['post_date']) : null
+	), $payment_data);
 
 	// Create a blank payment
-	$payment = wp_insert_post( $args );
+	$payment = wp_insert_post($args);
 
-	if ( $payment ) {
+	if ($payment) {
 
 		$payment_meta = array(
 			'currency'   => $payment_data['currency'],
 			'form_title' => $payment_data['give_form_title'],
 			'form_id'    => $payment_data['give_form_id'],
-			'price_id'   => give_get_price_id( $payment_data['give_form_id'], $payment_data['price'] ),
+			'price_id'   => give_get_price_id($payment_data['give_form_id'], $payment_data['price']),
 			'user_info'  => $payment_data['user_info'],
 		);
 
 		$mode    = give_is_test_mode() ? 'test' : 'live';
-		$gateway = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-		$gateway = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
+		$gateway = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+		$gateway = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
 
-		if ( ! $payment_data['price'] ) {
+		if ( ! $payment_data['price']) {
 			// Ensures the _give_payment_total meta key is created for donations with an amount of 0
 			$payment_data['price'] = '0.00';
 		}
 
 		// Create or update a customer
-		$customer      = new Give_Customer( $payment_data['user_email'] );
+		$customer      = new Give_Customer($payment_data['user_email']);
 		$customer_data = array(
-			'name'    => $payment_data['user_info']['first_name'] . ' ' . $payment_data['user_info']['last_name'],
+			'name'    => $payment_data['user_info']['first_name'].' '.$payment_data['user_info']['last_name'],
 			'email'   => $payment_data['user_email'],
 			'user_id' => $payment_data['user_info']['id']
 		);
 
-		if ( empty( $customer->id ) ) {
-			$customer->create( $customer_data );
+		if (empty($customer->id)) {
+			$customer->create($customer_data);
 		} else {
 			// Only update the customer if their name or email has changed
-			if ( $customer_data['email'] !== $customer->email || $customer_data['name'] !== $customer->name ) {
+			if ($customer_data['email'] !== $customer->email || $customer_data['name'] !== $customer->name) {
 				// We shouldn't be updating the User ID here, that is an admin task
-				unset( $customer_data['user_id'] );
-				$customer->update( $customer_data );
+				unset($customer_data['user_id']);
+				$customer->update($customer_data);
 			}
 		}
 
-		$customer->attach_payment( $payment, false );
+		$customer->attach_payment($payment, false);
 
 		// Record the payment details
-		give_update_payment_meta( $payment, '_give_payment_meta', apply_filters( 'give_payment_meta', $payment_meta, $payment_data ) );
-		give_update_payment_meta( $payment, '_give_payment_user_id', $payment_data['user_info']['id'] );
-		give_update_payment_meta( $payment, '_give_payment_donor_id', $customer->id );
-		give_update_payment_meta( $payment, '_give_payment_user_email', $payment_data['user_email'] );
-		give_update_payment_meta( $payment, '_give_payment_user_ip', give_get_ip() );
-		give_update_payment_meta( $payment, '_give_payment_purchase_key', $payment_data['purchase_key'] );
-		give_update_payment_meta( $payment, '_give_payment_total', $payment_data['price'] );
-		give_update_payment_meta( $payment, '_give_payment_mode', $mode );
-		give_update_payment_meta( $payment, '_give_payment_gateway', $gateway );
-
-		if ( give_get_option( 'enable_sequential' ) ) {
-			give_update_payment_meta( $payment, '_give_payment_number', give_format_payment_number( $number ) );
-			update_option( 'give_last_payment_number', $number );
+		give_update_payment_meta($payment, '_give_payment_meta', apply_filters('give_payment_meta', $payment_meta, $payment_data));
+		give_update_payment_meta($payment, '_give_payment_user_id', $payment_data['user_info']['id']);
+		give_update_payment_meta($payment, '_give_payment_donor_id', $customer->id);
+		give_update_payment_meta($payment, '_give_payment_user_email', $payment_data['user_email']);
+		give_update_payment_meta($payment, '_give_payment_user_ip', give_get_ip());
+		give_update_payment_meta($payment, '_give_payment_purchase_key', $payment_data['purchase_key']);
+		give_update_payment_meta($payment, '_give_payment_total', $payment_data['price']);
+		give_update_payment_meta($payment, '_give_payment_mode', $mode);
+		give_update_payment_meta($payment, '_give_payment_gateway', $gateway);
+
+		if (give_get_option('enable_sequential')) {
+			give_update_payment_meta($payment, '_give_payment_number', give_format_payment_number($number));
+			update_option('give_last_payment_number', $number);
 		}
 
 		// Clear the user's purchased cache
-		delete_transient( 'give_user_' . $payment_data['user_info']['id'] . '_purchases' );
+		delete_transient('give_user_'.$payment_data['user_info']['id'].'_purchases');
 
-		do_action( 'give_insert_payment', $payment, $payment_data );
+		do_action('give_insert_payment', $payment, $payment_data);
 
 		return $payment; // Return the ID
 
@@ -229,43 +229,43 @@ 
  *
  * @return void
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
-	if ( $new_status == 'completed' || $new_status == 'complete' ) {
+	if ($new_status == 'completed' || $new_status == 'complete') {
 		$new_status = 'publish';
 	}
 
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
-	$payment = get_post( $payment_id );
+	$payment = get_post($payment_id);
 
-	if ( is_wp_error( $payment ) || ! is_object( $payment ) ) {
+	if (is_wp_error($payment) || ! is_object($payment)) {
 		return;
 	}
 
 	$old_status = $payment->post_status;
 
-	if ( $old_status === $new_status ) {
+	if ($old_status === $new_status) {
 		return; // Don't permit status changes that aren't changes
 	}
 
-	$do_change = apply_filters( 'give_should_update_payment_status', true, $payment_id, $new_status, $old_status );
+	$do_change = apply_filters('give_should_update_payment_status', true, $payment_id, $new_status, $old_status);
 
-	if ( $do_change ) {
+	if ($do_change) {
 
-		do_action( 'give_before_payment_status_change', $payment_id, $new_status, $old_status );
+		do_action('give_before_payment_status_change', $payment_id, $new_status, $old_status);
 
 		$update_fields = array(
 			'ID'          => $payment_id,
 			'post_status' => $new_status,
-			'edit_date'   => current_time( 'mysql' )
+			'edit_date'   => current_time('mysql')
 		);
 
-		wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+		wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
-		do_action( 'give_update_payment_status', $payment_id, $new_status, $old_status );
+		do_action('give_update_payment_status', $payment_id, $new_status, $old_status);
 
 	}
 }
@@ -281,48 +281,48 @@ 
  *
  * @return void
  */
-function give_delete_purchase( $payment_id = 0 ) {
+function give_delete_purchase($payment_id = 0) {
 	global $give_logs;
 
-	$post = get_post( $payment_id );
+	$post = get_post($payment_id);
 
-	if ( ! $post ) {
+	if ( ! $post) {
 		return;
 	}
 
-	$form_id = give_get_payment_form_id( $payment_id );
+	$form_id = give_get_payment_form_id($payment_id);
 
-	give_undo_purchase( $form_id, $payment_id );
+	give_undo_purchase($form_id, $payment_id);
 
-	$amount   = give_get_payment_amount( $payment_id );
+	$amount   = give_get_payment_amount($payment_id);
 	$status   = $post->post_status;
-	$donor_id = give_get_payment_customer_id( $payment_id );
+	$donor_id = give_get_payment_customer_id($payment_id);
 
-	if ( $status == 'revoked' || $status == 'publish' ) {
+	if ($status == 'revoked' || $status == 'publish') {
 		// 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);
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 
-		if ( $donor_id ) {
+		if ($donor_id) {
 
 			// Decrement the stats for the donor
-			Give()->customers->decrement_stats( $donor_id, $amount );
+			Give()->customers->decrement_stats($donor_id, $amount);
 
 		}
 	}
 
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $donor_id ) {
+	if ($donor_id) {
 
 		// Remove the payment ID from the donor
-		Give()->customers->remove_payment( $donor_id, $payment_id );
+		Give()->customers->remove_payment($donor_id, $payment_id);
 
 	}
 
 	// Remove the payment
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries
 	$give_logs->delete_logs(
@@ -336,7 +336,7 @@ 
 		)
 	);
 
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -349,19 +349,19 @@ 
  *
  * @return void
  */
-function give_undo_purchase( $form_id, $payment_id ) {
+function give_undo_purchase($form_id, $payment_id) {
 
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		return;
 	}
 
-	$amount = give_get_payment_amount( $payment_id );
+	$amount = give_get_payment_amount($payment_id);
 
 	// decrease earnings
-	give_decrease_earnings( $form_id, $amount );
+	give_decrease_earnings($form_id, $amount);
 
 	// decrease purchase count
-	give_decrease_purchase_count( $form_id );
+	give_decrease_purchase_count($form_id);
 
 
 }
@@ -378,7 +378,7 @@ 
  *
  * @return array $count Number of payments sorted by payment status
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 
 	global $wpdb;
 
@@ -389,17 +389,17 @@ 
 		'end-date'   => null,
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$join  = '';
 	$where = "WHERE p.post_type = 'give_payment'";
 
 	// Count payments for a specific user
-	if ( ! empty( $args['user'] ) ) {
+	if ( ! empty($args['user'])) {
 
-		if ( is_email( $args['user'] ) ) {
+		if (is_email($args['user'])) {
 			$field = 'email';
-		} elseif ( is_numeric( $args['user'] ) ) {
+		} elseif (is_numeric($args['user'])) {
 			$field = 'id';
 		} else {
 			$field = '';
@@ -407,18 +407,18 @@ 
 
 		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$where .= "
 				AND m.meta_key = '_give_payment_user_{$field}'
 				AND m.meta_value = '{$args['user']}'";
 		}
 
 		// Count payments for a search
-	} elseif ( ! empty( $args['s'] ) ) {
+	} elseif ( ! empty($args['s'])) {
 
-		if ( is_email( $args['s'] ) || strlen( $args['s'] ) == 32 ) {
+		if (is_email($args['s']) || strlen($args['s']) == 32) {
 
-			if ( is_email( $args['s'] ) ) {
+			if (is_email($args['s'])) {
 				$field = '_give_payment_user_email';
 			} else {
 				$field = '_give_payment_purchase_key';
@@ -430,7 +430,7 @@ 
 				AND m.meta_key = '{$field}'
 				AND m.meta_value = '{$args['s']}'";
 
-		} elseif ( is_numeric( $args['s'] ) ) {
+		} elseif (is_numeric($args['s'])) {
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 			$where .= "
@@ -444,48 +444,48 @@ 
 	}
 
 	// Limit payments count by date
-	if ( ! empty( $args['start-date'] ) && false !== strpos( $args['start-date'], '/' ) ) {
+	if ( ! empty($args['start-date']) && false !== strpos($args['start-date'], '/')) {
 
-		$date_parts = explode( '/', $args['start-date'] );
-		$month      = ! empty( $date_parts[0] ) && is_numeric( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day        = ! empty( $date_parts[1] ) && is_numeric( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year       = ! empty( $date_parts[2] ) && is_numeric( $date_parts[2] ) ? $date_parts[2] : 0;
+		$date_parts = explode('/', $args['start-date']);
+		$month      = ! empty($date_parts[0]) && is_numeric($date_parts[0]) ? $date_parts[0] : 0;
+		$day        = ! empty($date_parts[1]) && is_numeric($date_parts[1]) ? $date_parts[1] : 0;
+		$year       = ! empty($date_parts[2]) && is_numeric($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['start-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date >= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['start-date']);
+			$where .= $wpdb->prepare(" AND p.post_date >= '%s'", $date->format('Y-m-d'));
 
 		}
 
 		// Fixes an issue with the payments list table counts when no end date is specified (partiy with stats class)
-		if ( empty( $args['end-date'] ) ) {
+		if (empty($args['end-date'])) {
 			$args['end-date'] = $args['start-date'];
 		}
 
 	}
 
-	if ( ! empty ( $args['end-date'] ) && false !== strpos( $args['end-date'], '/' ) ) {
+	if ( ! empty ($args['end-date']) && false !== strpos($args['end-date'], '/')) {
 
-		$date_parts = explode( '/', $args['end-date'] );
+		$date_parts = explode('/', $args['end-date']);
 
-		$month = ! empty( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day   = ! empty( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year  = ! empty( $date_parts[2] ) ? $date_parts[2] : 0;
+		$month = ! empty($date_parts[0]) ? $date_parts[0] : 0;
+		$day   = ! empty($date_parts[1]) ? $date_parts[1] : 0;
+		$year  = ! empty($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['end-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date <= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['end-date']);
+			$where .= $wpdb->prepare(" AND p.post_date <= '%s'", $date->format('Y-m-d'));
 
 		}
 
 	}
 
-	$where = apply_filters( 'give_count_payments_where', $where );
-	$join  = apply_filters( 'give_count_payments_join', $join );
+	$where = apply_filters('give_count_payments_where', $where);
+	$join  = apply_filters('give_count_payments_join', $join);
 
 	$query = "SELECT p.post_status,count( * ) AS num_posts
 		FROM $wpdb->posts p
@@ -494,35 +494,35 @@ 
 		GROUP BY p.post_status
 	";
 
-	$cache_key = md5( implode( '|', $args ) . $where );
+	$cache_key = md5(implode('|', $args).$where);
 
-	$count = wp_cache_get( $cache_key, 'counts' );
-	if ( false !== $count ) {
+	$count = wp_cache_get($cache_key, 'counts');
+	if (false !== $count) {
 		return $count;
 	}
 
-	$count = $wpdb->get_results( $query, ARRAY_A );
+	$count = $wpdb->get_results($query, ARRAY_A);
 
 	$stats    = array();
 	$statuses = get_post_stati();
-	if ( isset( $statuses['private'] ) && empty( $args['s'] ) ) {
-		unset( $statuses['private'] );
+	if (isset($statuses['private']) && empty($args['s'])) {
+		unset($statuses['private']);
 	}
 
-	foreach ( $statuses as $state ) {
-		$stats[ $state ] = 0;
+	foreach ($statuses as $state) {
+		$stats[$state] = 0;
 	}
 
-	foreach ( (array) $count as $row ) {
-		if ( 'private' == $row['post_status'] && empty( $args['s'] ) ) {
+	foreach ((array) $count as $row) {
+		if ('private' == $row['post_status'] && empty($args['s'])) {
 			continue;
 		}
 
-		$stats[ $row['post_status'] ] = $row['num_posts'];
+		$stats[$row['post_status']] = $row['num_posts'];
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( $cache_key, $stats, 'counts' );
+	wp_cache_set($cache_key, $stats, 'counts');
 
 	return $stats;
 }
@@ -537,10 +537,10 @@ 
  *
  * @return bool true if payment exists, false otherwise
  */
-function give_check_for_existing_payment( $payment_id ) {
-	$payment = get_post( $payment_id );
+function give_check_for_existing_payment($payment_id) {
+	$payment = get_post($payment_id);
 
-	if ( $payment && $payment->post_status == 'publish' ) {
+	if ($payment && $payment->post_status == 'publish') {
 		return true; // Payment exists
 	}
 
@@ -557,21 +557,21 @@ 
  *
  * @return bool|mixed if payment status exists, false otherwise
  */
-function give_get_payment_status( $payment, $return_label = false ) {
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+function give_get_payment_status($payment, $return_label = false) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
-	if ( array_key_exists( $payment->post_status, $statuses ) ) {
-		if ( true === $return_label ) {
-			return $statuses[ $payment->post_status ];
+	if (array_key_exists($payment->post_status, $statuses)) {
+		if (true === $return_label) {
+			return $statuses[$payment->post_status];
 		} else {
-			return array_search( $payment->post_status, $statuses );
+			return array_search($payment->post_status, $statuses);
 		}
 	}
 
@@ -586,17 +586,17 @@ 
  */
 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' ),
-		'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'),
+		'revoked'     => __('Revoked', 'give')
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -608,10 +608,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);
 }
 
 /**
@@ -626,7 +626,7 @@ 
  *
  * @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
 
@@ -637,40 +637,40 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish', 'revoked' ),
+		'post_status'            => array('publish', 'revoked'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	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 = get_transient( $key );
+		$earnings = get_transient($key);
 	}
 
-	if ( false === $earnings ) {
-		$sales    = get_posts( $args );
+	if (false === $earnings) {
+		$sales    = get_posts($args);
 		$earnings = 0;
-		if ( $sales ) {
-			$sales = implode( ',', $sales );
-			$earnings += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})" );
+		if ($sales) {
+			$sales = implode(',', $sales);
+			$earnings += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$sales})");
 
 		}
 		// Cache the results for one hour
-		set_transient( $key, $earnings, HOUR_IN_SECONDS );
+		set_transient($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -685,7 +685,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(
@@ -693,37 +693,37 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish', 'revoked' ),
+		'post_status'            => array('publish', 'revoked'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false
 	);
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
-	$key   = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
+	$key = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	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 = get_transient( $key );
+		$count = get_transient($key);
 	}
 
-	if ( false === $count ) {
-		$sales = new WP_Query( $args );
+	if (false === $count) {
+		$sales = new WP_Query($args);
 		$count = (int) $sales->post_count;
 		// Cache the results for one hour
-		set_transient( $key, $count, HOUR_IN_SECONDS );
+		set_transient($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -738,14 +738,14 @@ 
  *
  * @return bool true if complete, false otherwise
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = get_post( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = get_post($payment_id);
 	$ret     = false;
-	if ( $payment && $payment->post_status == 'publish' ) {
+	if ($payment && $payment->post_status == 'publish') {
 		$ret = true;
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -769,29 +769,29 @@ 
  */
 function give_get_total_earnings() {
 
-	$total = get_option( 'give_earnings_total', 0 );
+	$total = get_option('give_earnings_total', 0);
 
 	// If no total stored in DB, use old method of calculating total earnings
-	if ( ! $total ) {
+	if ( ! $total) {
 
 		global $wpdb;
 
-		$total = get_transient( 'give_earnings_total' );
+		$total = get_transient('give_earnings_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 
 			$total = (float) 0;
 
-			$args = apply_filters( 'give_get_total_earnings_args', array(
+			$args = apply_filters('give_get_total_earnings_args', array(
 				'offset' => 0,
-				'number' => - 1,
-				'status' => array( 'publish', 'revoked' ),
+				'number' => -1,
+				'status' => array('publish', 'revoked'),
 				'fields' => 'ids'
-			) );
+			));
 
 
-			$payments = give_get_payments( $args );
-			if ( $payments ) {
+			$payments = give_get_payments($args);
+			if ($payments) {
 
 				/*
 				 * If performing a purchase, we need to skip the very last payment in the database, since it calls
@@ -799,30 +799,30 @@ 
 				 * first purchase
 				 */
 
-				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 $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})" );
+				if ( ! empty($payments)) {
+					$payments = implode(',', $payments);
+					$total += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})");
 				}
 
 			}
 
 			// Cache results for 1 day. This cache is cleared automatically when a payment is made
-			set_transient( 'give_earnings_total', $total, 86400 );
+			set_transient('give_earnings_total', $total, 86400);
 
 			// Store the total for the first time
-			update_option( 'give_earnings_total', $total );
+			update_option('give_earnings_total', $total);
 		}
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_currency_decimal_filter() ) );
+	return apply_filters('give_total_earnings', round($total, give_currency_decimal_filter()));
 }
 
 /**
@@ -834,10 +834,10 @@ 
  *
  * @return float $total Total earnings
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -851,13 +851,13 @@ 
  *
  * @return float $total Total earnings
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -873,28 +873,28 @@ 
  *
  * @return mixed $meta Payment Meta
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
 
-	$meta = get_post_meta( $payment_id, $meta_key, $single );
+	$meta = get_post_meta($payment_id, $meta_key, $single);
 
-	if ( $meta_key === '_give_payment_meta' ) {
+	if ($meta_key === '_give_payment_meta') {
 
-		if ( empty( $meta['key'] ) ) {
-			$meta['key'] = give_get_payment_key( $payment_id );
+		if (empty($meta['key'])) {
+			$meta['key'] = give_get_payment_key($payment_id);
 		}
 
-		if ( empty( $meta['email'] ) ) {
-			$meta['email'] = give_get_payment_user_email( $payment_id );
+		if (empty($meta['email'])) {
+			$meta['email'] = give_get_payment_user_email($payment_id);
 		}
 
-		if ( empty( $meta['date'] ) ) {
-			$meta['date'] = get_post_field( 'post_date', $payment_id );
+		if (empty($meta['date'])) {
+			$meta['date'] = get_post_field('post_date', $payment_id);
 		}
 	}
 
-	$meta = apply_filters( 'give_get_payment_meta_' . $meta_key, $meta, $payment_id );
+	$meta = apply_filters('give_get_payment_meta_'.$meta_key, $meta, $payment_id);
 
-	return apply_filters( 'give_get_payment_meta', $meta, $payment_id, $meta_key );
+	return apply_filters('give_get_payment_meta', $meta, $payment_id, $meta_key);
 }
 
 /**
@@ -907,26 +907,26 @@ 
  *
  * @return mixed               Meta ID if successful, false if unsuccessful
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
 
-	if ( empty( $payment_id ) || empty( $meta_key ) ) {
+	if (empty($payment_id) || empty($meta_key)) {
 		return;
 	}
 
-	if ( $meta_key == 'key' || $meta_key == 'date' ) {
+	if ($meta_key == 'key' || $meta_key == 'date') {
 
-		$current_meta              = give_get_payment_meta( $payment_id );
-		$current_meta[ $meta_key ] = $meta_value;
+		$current_meta              = give_get_payment_meta($payment_id);
+		$current_meta[$meta_key] = $meta_value;
 
 		$meta_key   = '_give_payment_meta';
 		$meta_value = $current_meta;
 
-	} else if ( $meta_key == 'email' || $meta_key == '_give_payment_user_email' ) {
+	} else if ($meta_key == 'email' || $meta_key == '_give_payment_user_email') {
 
-		$meta_value = apply_filters( 'give_give_update_payment_meta_' . $meta_key, $meta_value, $payment_id );
-		update_post_meta( $payment_id, '_give_payment_user_email', $meta_value );
+		$meta_value = apply_filters('give_give_update_payment_meta_'.$meta_key, $meta_value, $payment_id);
+		update_post_meta($payment_id, '_give_payment_user_email', $meta_value);
 
-		$current_meta                       = give_get_payment_meta( $payment_id );
+		$current_meta                       = give_get_payment_meta($payment_id);
 		$current_meta['user_info']['email'] = $meta_value;
 
 		$meta_key   = '_give_payment_meta';
@@ -934,9 +934,9 @@ 
 
 	}
 
-	$meta_value = apply_filters( 'give_give_update_payment_meta_' . $meta_key, $meta_value, $payment_id );
+	$meta_value = apply_filters('give_give_update_payment_meta_'.$meta_key, $meta_value, $payment_id);
 
-	return update_post_meta( $payment_id, $meta_key, $meta_value, $prev_value );
+	return update_post_meta($payment_id, $meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -948,11 +948,11 @@ 
  *
  * @return array $user_info User Info Meta Values
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment_meta = give_get_payment_meta( $payment_id );
-	$user_info    = isset( $payment_meta['user_info'] ) ? maybe_unserialize( $payment_meta['user_info'] ) : false;
+function give_get_payment_meta_user_info($payment_id) {
+	$payment_meta = give_get_payment_meta($payment_id);
+	$user_info    = isset($payment_meta['user_info']) ? maybe_unserialize($payment_meta['user_info']) : false;
 
-	return apply_filters( 'give_payment_meta_user_info', $user_info );
+	return apply_filters('give_payment_meta_user_info', $user_info);
 }
 
 /**
@@ -965,12 +965,12 @@ 
  *
  * @return int $form_id
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment_meta = give_get_payment_meta( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment_meta = give_get_payment_meta($payment_id);
 
-	$form_id = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
+	$form_id = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
 
-	return apply_filters( 'give_get_payment_form_id', $form_id );
+	return apply_filters('give_get_payment_form_id', $form_id);
 }
 
 /**
@@ -982,10 +982,10 @@ 
  *
  * @return string $email User Email
  */
-function give_get_payment_user_email( $payment_id ) {
-	$email = give_get_payment_meta( $payment_id, '_give_payment_user_email', true );
+function give_get_payment_user_email($payment_id) {
+	$email = give_get_payment_meta($payment_id, '_give_payment_user_email', true);
 
-	return apply_filters( 'give_payment_user_email', $email );
+	return apply_filters('give_payment_user_email', $email);
 }
 
 /**
@@ -997,11 +997,11 @@ 
  *
  * @return bool            If the payment is associted 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);
 }
 
 /**
@@ -1013,38 +1013,38 @@ 
  *
  * @return string $user_id User ID
  */
-function give_get_payment_user_id( $payment_id ) {
+function give_get_payment_user_id($payment_id) {
 
 	$user_id = - 1;
 
 	// check the customer record first
-	$customer_id = give_get_payment_customer_id( $payment_id );
-	$customer    = new Give_Customer( $customer_id );
+	$customer_id = give_get_payment_customer_id($payment_id);
+	$customer    = new Give_Customer($customer_id);
 
-	if ( ! empty( $customer->user_id ) && $customer->user_id > 0 ) {
+	if ( ! empty($customer->user_id) && $customer->user_id > 0) {
 		$user_id = $customer->user_id;
 	}
 
 	// check the payment meta if we're still not finding a user with the customer record
-	if ( empty( $user_id ) || $user_id < 1 ) {
-		$payment_meta_user_id = give_get_payment_meta( $payment_id, '_give_payment_user_id', true );
+	if (empty($user_id) || $user_id < 1) {
+		$payment_meta_user_id = give_get_payment_meta($payment_id, '_give_payment_user_id', true);
 
-		if ( ! empty( $payment_meta_user_id ) ) {
+		if ( ! empty($payment_meta_user_id)) {
 			$user_id = $payment_meta_user_id;
 		}
 	}
 
 	// Last ditch effort is to connect payment email with a user in the user table
-	if ( empty( $user_id ) || $user_id < 1 ) {
-		$payment_email = give_get_payment_user_email( $payment_id );
-		$user          = get_user_by( 'email', $payment_email );
+	if (empty($user_id) || $user_id < 1) {
+		$payment_email = give_get_payment_user_email($payment_id);
+		$user          = get_user_by('email', $payment_email);
 
-		if ( false !== $user ) {
+		if (false !== $user) {
 			$user_id = $user->ID;
 		}
 	}
 
-	return apply_filters( 'give_payment_user_id', (int) $user_id );
+	return apply_filters('give_payment_user_id', (int) $user_id);
 }
 
 /**
@@ -1056,10 +1056,10 @@ 
  *
  * @return string $donor_id Donor ID
  */
-function give_get_payment_customer_id( $payment_id ) {
-	$customer_id = get_post_meta( $payment_id, '_give_payment_donor_id', true );
+function give_get_payment_customer_id($payment_id) {
+	$customer_id = get_post_meta($payment_id, '_give_payment_donor_id', true);
 
-	return apply_filters( 'give_payment_donor_id', $customer_id );
+	return apply_filters('give_payment_donor_id', $customer_id);
 }
 
 /**
@@ -1071,10 +1071,10 @@ 
  *
  * @return string $ip User IP
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$ip = give_get_payment_meta( $payment_id, '_give_payment_user_ip', true );
+function give_get_payment_user_ip($payment_id) {
+	$ip = give_get_payment_meta($payment_id, '_give_payment_user_ip', true);
 
-	return apply_filters( 'give_payment_user_ip', $ip );
+	return apply_filters('give_payment_user_ip', $ip);
 }
 
 /**
@@ -1086,17 +1086,17 @@ 
  *
  * @return string $date The date the payment was completed
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
+function give_get_payment_completed_date($payment_id = 0) {
 
-	$payment = get_post( $payment_id );
+	$payment = get_post($payment_id);
 
-	if ( 'pending' == $payment->post_status || 'preapproved' == $payment->post_status ) {
+	if ('pending' == $payment->post_status || 'preapproved' == $payment->post_status) {
 		return false; // This payment was never completed
 	}
 
-	$date = ( $date = give_get_payment_meta( $payment_id, '_give_completed_date', true ) ) ? $date : $payment->modified_date;
+	$date = ($date = give_get_payment_meta($payment_id, '_give_completed_date', true)) ? $date : $payment->modified_date;
 
-	return apply_filters( 'give_payment_completed_date', $date, $payment_id );
+	return apply_filters('give_payment_completed_date', $date, $payment_id);
 }
 
 /**
@@ -1108,10 +1108,10 @@ 
  *
  * @return string $gateway Gateway
  */
-function give_get_payment_gateway( $payment_id ) {
-	$gateway = give_get_payment_meta( $payment_id, '_give_payment_gateway', true );
+function give_get_payment_gateway($payment_id) {
+	$gateway = give_get_payment_meta($payment_id, '_give_payment_gateway', true);
 
-	return apply_filters( 'give_payment_gateway', $gateway );
+	return apply_filters('give_payment_gateway', $gateway);
 }
 
 /**
@@ -1123,11 +1123,11 @@ 
  *
  * @return string $currency The currency code
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$meta     = give_get_payment_meta( $payment_id );
-	$currency = isset( $meta['currency'] ) ? $meta['currency'] : give_get_currency();
+function give_get_payment_currency_code($payment_id = 0) {
+	$meta     = give_get_payment_meta($payment_id);
+	$currency = isset($meta['currency']) ? $meta['currency'] : give_get_currency();
 
-	return apply_filters( 'give_payment_currency_code', $currency, $payment_id );
+	return apply_filters('give_payment_currency_code', $currency, $payment_id);
 }
 
 /**
@@ -1139,10 +1139,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);
 }
 
 /**
@@ -1154,10 +1154,10 @@ 
  *
  * @return string $key Purchase key
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$key = give_get_payment_meta( $payment_id, '_give_payment_purchase_key', true );
+function give_get_payment_key($payment_id = 0) {
+	$key = give_get_payment_meta($payment_id, '_give_payment_purchase_key', true);
 
-	return apply_filters( 'give_payment_key', $key, $payment_id );
+	return apply_filters('give_payment_key', $key, $payment_id);
 }
 
 /**
@@ -1171,15 +1171,15 @@ 
  *
  * @return string $number Payment order number
  */
-function give_get_payment_number( $payment_id = 0 ) {
+function give_get_payment_number($payment_id = 0) {
 
 	$number = $payment_id;
 
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 
-		$number = give_get_payment_meta( $payment_id, '_give_payment_number', true );
+		$number = give_get_payment_meta($payment_id, '_give_payment_number', true);
 
-		if ( ! $number ) {
+		if ( ! $number) {
 
 			$number = $payment_id;
 
@@ -1187,7 +1187,7 @@ 
 
 	}
 
-	return apply_filters( 'give_payment_number', $number, $payment_id );
+	return apply_filters('give_payment_number', $number, $payment_id);
 }
 
 /**
@@ -1199,23 +1199,23 @@ 
  *
  * @return string      The formatted payment number
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1228,17 +1228,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1248,24 +1248,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids'
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( $last_payment ) {
+		if ($last_payment) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== $last_payment[0] ) {
+		if ( ! empty($number) && $number !== $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1275,13 +1275,13 @@ 
 
 	}
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1293,25 +1293,25 @@ 
  *
  * @return string          The new Payment number without prefix and postfix
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1327,10 +1327,10 @@ 
  *
  * @return string $amount Fully formatted payment amount
  */
-function give_payment_amount( $payment_id = 0 ) {
-	$amount = give_get_payment_amount( $payment_id );
+function give_payment_amount($payment_id = 0) {
+	$amount = give_get_payment_amount($payment_id);
 
-	return give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1343,20 +1343,20 @@ 
  *
  * @return mixed|void
  */
-function give_get_payment_amount( $payment_id ) {
+function give_get_payment_amount($payment_id) {
 
-	$amount = give_get_payment_meta( $payment_id, '_give_payment_total', true );
+	$amount = give_get_payment_meta($payment_id, '_give_payment_total', true);
 
-	if ( empty( $amount ) && '0.00' != $amount ) {
-		$meta = give_get_payment_meta( $payment_id, '_give_payment_meta', true );
-		$meta = maybe_unserialize( $meta );
+	if (empty($amount) && '0.00' != $amount) {
+		$meta = give_get_payment_meta($payment_id, '_give_payment_meta', true);
+		$meta = maybe_unserialize($meta);
 
-		if ( isset( $meta['amount'] ) ) {
+		if (isset($meta['amount'])) {
 			$amount = $meta['amount'];
 		}
 	}
 
-	return apply_filters( 'give_payment_amount', floatval( $amount ), $payment_id );
+	return apply_filters('give_payment_amount', floatval($amount), $payment_id);
 }
 
 
@@ -1369,19 +1369,19 @@ 
  *
  * @return string The Transaction ID
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
+function give_get_payment_transaction_id($payment_id = 0) {
 
 	$transaction_id = false;
-	$transaction_id = give_get_payment_meta( $payment_id, '_give_payment_transaction_id', true );
+	$transaction_id = give_get_payment_meta($payment_id, '_give_payment_transaction_id', true);
 
-	if ( empty( $transaction_id ) ) {
+	if (empty($transaction_id)) {
 
-		$gateway        = give_get_payment_gateway( $payment_id );
-		$transaction_id = apply_filters( 'give_get_payment_transaction_id-' . $gateway, $payment_id );
+		$gateway        = give_get_payment_gateway($payment_id);
+		$transaction_id = apply_filters('give_get_payment_transaction_id-'.$gateway, $payment_id);
 
 	}
 
-	return apply_filters( 'give_get_payment_transaction_id', $transaction_id, $payment_id );
+	return apply_filters('give_get_payment_transaction_id', $transaction_id, $payment_id);
 }
 
 /**
@@ -1394,15 +1394,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);
 }
 
 /**
@@ -1416,12 +1416,12 @@ 
  *
  * @return int $purchase Purchase ID
  */
-function give_get_purchase_id_by_key( $key ) {
+function give_get_purchase_id_by_key($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1440,12 +1440,12 @@ 
  *
  * @return int $purchase Purchase ID
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1462,19 +1462,19 @@ 
  *
  * @return array $notes Payment Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
-	$notes = get_comments( array( 'post_id' => $payment_id, 'order' => 'ASC', 'search' => $search ) );
+	$notes = get_comments(array('post_id' => $payment_id, 'order' => 'ASC', 'search' => $search));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1490,19 +1490,19 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1511,9 +1511,9 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note'
 
-	) ) );
+	)));
 
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1528,14 +1528,14 @@ 
  *
  * @return bool True on success, false otherwise
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
-	$ret = wp_delete_comment( $comment_id, true );
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
+	$ret = wp_delete_comment($comment_id, true);
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1550,32 +1550,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = __( 'System', 'give' );
+		$user = __('System', 'give');
 	}
 
-	$date_format = get_option( 'date_format' ) . ', ' . get_option( 'time_format' );
+	$date_format = get_option('date_format').', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" title="' . __( 'Delete this payment 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).'" title="'.__('Delete this payment note', 'give').'">'.__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1593,20 +1593,20 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
+function give_hide_payment_notes($query) {
 	global $wp_version;
 
-	if ( version_compare( floatval( $wp_version ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+	if (version_compare(floatval($wp_version), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1618,17 +1618,17 @@ 
  *
  * @return array $clauses Updated comment clauses
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
 	global $wpdb, $wp_version;
 
-	if ( version_compare( floatval( $wp_version ), '4.1', '<' ) ) {
+	if (version_compare(floatval($wp_version), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1641,15 +1641,15 @@ 
  *
  * @return array $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( " AND comment_type != %s", 'give_payment_note' );
+	$where .= $wpdb->prepare(" AND comment_type != %s", 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1663,32 +1663,32 @@ 
  *
  * @return array Array of comment counts
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = wp_cache_get( "comments-{$post_id}", 'counts' );
+	$stats = wp_cache_get("comments-{$post_id}", 'counts');
 
-	if ( false !== $stats ) {
+	if (false !== $stats) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( " AND comment_post_ID = %d", $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(" AND comment_post_ID = %d", $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1698,30 +1698,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed'
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( "comments-{$post_id}", $stats, 'counts' );
+	wp_cache_set("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1734,9 +1734,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;
@@ -1753,20 +1753,20 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 
 	$price_id = 0;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = maybe_unserialize( get_post_meta( $form_id, '_give_donation_levels', true ) );
+		$levels = maybe_unserialize(get_post_meta($form_id, '_give_donation_levels', true));
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = (float) give_sanitize_amount( $level['_give_amount'] );
+			$level_amount = (float) give_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'];
 
@@ -1790,14 +1790,14 @@ 
  *
  * @return mixed array if payment fees found, false otherwise
  */
-function give_get_payment_fees( $payment_id = 0, $type = 'all' ) {
-	$payment_meta = give_get_payment_meta( $payment_id );
+function give_get_payment_fees($payment_id = 0, $type = 'all') {
+	$payment_meta = give_get_payment_meta($payment_id);
 	$fees         = array();
-	$payment_fees = isset( $payment_meta['fees'] ) ? $payment_meta['fees'] : false;
-	if ( ! empty( $payment_fees ) && is_array( $payment_fees ) ) {
-		foreach ( $payment_fees as $fee_id => $fee ) {
-			if ( 'all' != $type && ! empty( $fee['type'] ) && $type != $fee['type'] ) {
-				unset( $payment_fees[ $fee_id ] );
+	$payment_fees = isset($payment_meta['fees']) ? $payment_meta['fees'] : false;
+	if ( ! empty($payment_fees) && is_array($payment_fees)) {
+		foreach ($payment_fees as $fee_id => $fee) {
+			if ('all' != $type && ! empty($fee['type']) && $type != $fee['type']) {
+				unset($payment_fees[$fee_id]);
 			} else {
 				$fees[] = array(
 					'id'     => $fee_id,
@@ -1808,5 +1808,5 @@ 
 		}
 	}
 
-	return apply_filters( 'give_get_payment_fees', $fees, $payment_id );
+	return apply_filters('give_get_payment_fees', $fees, $payment_id);
 }
\ No newline at end of file

includes/process-purchase.php

Doc Comments +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
  *
  * @access      private
  * @since       1.0
- * @return      void
+ * @return      false|null
  */
 function give_process_purchase_form() {
 
@@ -326,7 +326,7 @@ 
  *
  * @since  1.0.1
  *
- * @param $payment_mode
+ * @param string $payment_mode
  *
  * @return mixed|void
  */

Spacing +226 added lines, -226 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,34 +25,34 @@ 
  */
 function give_process_purchase_form() {
 
-	do_action( 'give_pre_process_purchase' );
+	do_action('give_pre_process_purchase');
 
 	// Validate the form $_POST data
 	$valid_data = give_purchase_form_validate_fields();
 
 	// Allow themes and plugins to hook to errors
-	do_action( 'give_checkout_error_checks', $valid_data, $_POST );
+	do_action('give_checkout_error_checks', $valid_data, $_POST);
 
-	$is_ajax = isset( $_POST['give_ajax'] );
+	$is_ajax = isset($_POST['give_ajax']);
 
 	// Process the login form
-	if ( isset( $_POST['give_login_submit'] ) ) {
+	if (isset($_POST['give_login_submit'])) {
 		give_process_form_login();
 	}
 
 	// Validate the user
-	$user = give_get_purchase_form_user( $valid_data );
+	$user = give_get_purchase_form_user($valid_data);
 
-	if ( give_get_errors() || ! $user ) {
-		if ( $is_ajax ) {
-			do_action( 'give_ajax_checkout_errors' );
+	if (give_get_errors() || ! $user) {
+		if ($is_ajax) {
+			do_action('give_ajax_checkout_errors');
 			give_die();
 		} else {
 			return false;
 		}
 	}
 
-	if ( $is_ajax ) {
+	if ($is_ajax) {
 		echo 'success';
 		give_die();
 	}
@@ -66,15 +66,15 @@ 
 		'address'    => $user['address']
 	);
 
-	$auth_key = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
+	$auth_key = defined('AUTH_KEY') ? AUTH_KEY : '';
 
 	// Setup purchase information
 	$purchase_data = array(
-		'price'        => ( isset( $_POST['give-amount'] ) ? (float) apply_filters( 'give_donation_total', give_sanitize_amount( give_format_amount( $_POST['give-amount'] ) ) ) : '0.00' ),
-		'purchase_key' => strtolower( md5( $user['user_email'] . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) ),
+		'price'        => (isset($_POST['give-amount']) ? (float) apply_filters('give_donation_total', give_sanitize_amount(give_format_amount($_POST['give-amount']))) : '0.00'),
+		'purchase_key' => strtolower(md5($user['user_email'].date('Y-m-d H:i:s').$auth_key.uniqid('give', true))),
 		'user_email'   => $user['user_email'],
-		'date'         => date( 'Y-m-d H:i:s', current_time( 'timestamp' ) ),
-		'user_info'    => stripslashes_deep( $user_info ),
+		'date'         => date('Y-m-d H:i:s', current_time('timestamp')),
+		'user_info'    => stripslashes_deep($user_info),
 		'post_data'    => $_POST,
 		'gateway'      => $valid_data['gateway'],
 		'card_info'    => $valid_data['cc_info']
@@ -84,36 +84,36 @@ 
 	$valid_data['user'] = $user;
 
 	// Allow themes and plugins to hook before the gateway
-	do_action( 'give_checkout_before_gateway', $_POST, $user_info, $valid_data );
+	do_action('give_checkout_before_gateway', $_POST, $user_info, $valid_data);
 
 
-	if ( ! $purchase_data['price'] ) {
+	if ( ! $purchase_data['price']) {
 		// Revert to manual
 		$purchase_data['gateway'] = 'manual';
 		$_POST['give-gateway']    = 'manual';
 	}
 
 	// Allow the purchase data to be modified before it is sent to the gateway
-	$purchase_data = apply_filters( 'give_purchase_data_before_gateway', $purchase_data, $valid_data );
+	$purchase_data = apply_filters('give_purchase_data_before_gateway', $purchase_data, $valid_data);
 
 	// Setup the data we're storing in the purchase session
 	$session_data = $purchase_data;
 
 	// Make sure credit card numbers are never stored in sessions
-	unset( $session_data['card_info']['card_number'] );
+	unset($session_data['card_info']['card_number']);
 
 	// Used for showing data to non logged-in users after purchase, and for other plugins needing purchase data.
-	give_set_purchase_session( $session_data );
+	give_set_purchase_session($session_data);
 
 	// Send info to the gateway for payment processing
-	give_send_to_gateway( $purchase_data['gateway'], $purchase_data );
+	give_send_to_gateway($purchase_data['gateway'], $purchase_data);
 	give_die();
 
 }
 
-add_action( 'give_purchase', 'give_process_purchase_form' );
-add_action( 'wp_ajax_give_process_checkout', 'give_process_purchase_form' );
-add_action( 'wp_ajax_nopriv_give_process_checkout', 'give_process_purchase_form' );
+add_action('give_purchase', 'give_process_purchase_form');
+add_action('wp_ajax_give_process_checkout', 'give_process_purchase_form');
+add_action('wp_ajax_nopriv_give_process_checkout', 'give_process_purchase_form');
 
 /**
  * Process the checkout login form
@@ -124,32 +124,32 @@ 
  */
 function give_process_form_login() {
 
-	$is_ajax = isset( $_POST['give_ajax'] );
+	$is_ajax = isset($_POST['give_ajax']);
 
 	$user_data = give_purchase_form_validate_user_login();
 
-	if ( give_get_errors() || $user_data['user_id'] < 1 ) {
-		if ( $is_ajax ) {
-			do_action( 'give_ajax_checkout_errors' );
+	if (give_get_errors() || $user_data['user_id'] < 1) {
+		if ($is_ajax) {
+			do_action('give_ajax_checkout_errors');
 			give_die();
 		} else {
-			wp_redirect( $_SERVER['HTTP_REFERER'] );
+			wp_redirect($_SERVER['HTTP_REFERER']);
 			exit;
 		}
 	}
 
-	give_log_user_in( $user_data['user_id'], $user_data['user_login'], $user_data['user_pass'] );
+	give_log_user_in($user_data['user_id'], $user_data['user_login'], $user_data['user_pass']);
 
-	if ( $is_ajax ) {
+	if ($is_ajax) {
 		echo 'success';
 		give_die();
 	} else {
-		wp_redirect( $_SERVER['HTTP_REFERER'] );
+		wp_redirect($_SERVER['HTTP_REFERER']);
 	}
 }
 
-add_action( 'wp_ajax_give_process_checkout_login', 'give_process_form_login' );
-add_action( 'wp_ajax_nopriv_give_process_checkout_login', 'give_process_form_login' );
+add_action('wp_ajax_give_process_checkout_login', 'give_process_form_login');
+add_action('wp_ajax_nopriv_give_process_checkout_login', 'give_process_form_login');
 
 /**
  * Purchase Form Validate Fields
@@ -163,43 +163,43 @@ 
 	global $give_options;
 
 	// Check if there is $_POST
-	if ( empty( $_POST ) ) {
+	if (empty($_POST)) {
 		return false;
 	}
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start an array to collect valid data
 	$valid_data = array(
 		'gateway'          => give_purchase_form_validate_gateway(), // Gateway fallback
-		'need_new_user'    => false,     // New user flag
-		'need_user_login'  => false,     // Login user flag
-		'logged_user_data' => array(),   // Logged user collected data
-		'new_user_data'    => array(),   // New user collected data
-		'login_user_data'  => array(),   // Login user collected data
-		'guest_user_data'  => array(),   // Guest user collected data
+		'need_new_user'    => false, // New user flag
+		'need_user_login'  => false, // Login user flag
+		'logged_user_data' => array(), // Logged user collected data
+		'new_user_data'    => array(), // New user collected data
+		'login_user_data'  => array(), // Login user collected data
+		'guest_user_data'  => array(), // Guest user collected data
 		'cc_info'          => give_purchase_form_validate_cc()    // Credit card info
 	);
 
 	// Validate agree to terms
-	$terms_option = get_post_meta( $form_id, '_give_terms_option', true );
-	if ( isset( $terms_option ) && $terms_option === 'yes' ) {
+	$terms_option = get_post_meta($form_id, '_give_terms_option', true);
+	if (isset($terms_option) && $terms_option === 'yes') {
 		give_purchase_form_validate_agree_to_terms();
 	}
 
 	//Validate amount is more than 0
 
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 		// Collect logged in user data
 		$valid_data['logged_in_user'] = give_purchase_form_validate_logged_in_user();
-	} else if ( isset( $_POST['give-purchase-var'] ) && $_POST['give-purchase-var'] == 'needs-to-register' ) {
+	} else if (isset($_POST['give-purchase-var']) && $_POST['give-purchase-var'] == 'needs-to-register') {
 		// Set new user registration as required
 		$valid_data['need_new_user'] = true;
 		// Validate new user data
 		$valid_data['new_user_data'] = give_purchase_form_validate_new_user();
 		// Check if login validation is needed
-	} else if ( isset( $_POST['give-purchase-var'] ) && $_POST['give-purchase-var'] == 'needs-to-login' ) {
+	} else if (isset($_POST['give-purchase-var']) && $_POST['give-purchase-var'] == 'needs-to-login') {
 		// Set user login as required
 		$valid_data['need_user_login'] = true;
 		// Validate users login info
@@ -222,27 +222,27 @@ 
  */
 function give_purchase_form_validate_gateway() {
 
-	$gateway = give_get_default_gateway( $_REQUEST['give-form-id'] );
+	$gateway = give_get_default_gateway($_REQUEST['give-form-id']);
 
 	// Check if a gateway value is present
-	if ( ! empty( $_REQUEST['give-gateway'] ) ) {
+	if ( ! empty($_REQUEST['give-gateway'])) {
 
-		$gateway = sanitize_text_field( $_REQUEST['give-gateway'] );
+		$gateway = sanitize_text_field($_REQUEST['give-gateway']);
 
 		//Is amount being donated in LIVE mode above 0.00?
-		if ( '0.00' == $_REQUEST['give-amount'] && ! give_is_test_mode() ) {
+		if ('0.00' == $_REQUEST['give-amount'] && ! give_is_test_mode()) {
 
-			give_set_error( 'invalid_donation_amount', __( 'Please insert a valid donation amount.', 'give' ) );
+			give_set_error('invalid_donation_amount', __('Please insert a valid donation amount.', 'give'));
 
 		} //Is this test mode zero donation? Let it through but set to manual gateway
-		elseif ( '0.00' == $_REQUEST['give-amount'] && give_is_test_mode() ) {
+		elseif ('0.00' == $_REQUEST['give-amount'] && give_is_test_mode()) {
 
 			$gateway = 'manual';
 
 		} //Check if this gateway is active
-		elseif ( ! give_is_gateway_active( $gateway ) ) {
+		elseif ( ! give_is_gateway_active($gateway)) {
 
-			give_set_error( 'invalid_gateway', __( 'The selected payment gateway is not enabled', 'give' ) );
+			give_set_error('invalid_gateway', __('The selected payment gateway is not enabled', 'give'));
 
 		}
 
@@ -261,9 +261,9 @@ 
  */
 function give_purchase_form_validate_agree_to_terms() {
 	// Validate agree to terms
-	if ( ! isset( $_POST['give_agree_to_terms'] ) || $_POST['give_agree_to_terms'] != 1 ) {
+	if ( ! isset($_POST['give_agree_to_terms']) || $_POST['give_agree_to_terms'] != 1) {
 		// User did not agree
-		give_set_error( 'agree_to_terms', apply_filters( 'give_agree_to_terms_text', __( 'You must agree to the terms of use', 'give' ) ) );
+		give_set_error('agree_to_terms', apply_filters('give_agree_to_terms_text', __('You must agree to the terms of use', 'give')));
 	}
 }
 
@@ -277,47 +277,47 @@ 
  *
  * @return      array
  */
-function give_purchase_form_required_fields( $form_id ) {
+function give_purchase_form_required_fields($form_id) {
 
-	$payment_mode = give_get_chosen_gateway( $form_id );
+	$payment_mode = give_get_chosen_gateway($form_id);
 
 	$required_fields = array(
 		'give_email' => array(
 			'error_id'      => 'invalid_email',
-			'error_message' => __( 'Please enter a valid email address', 'give' )
+			'error_message' => __('Please enter a valid email address', 'give')
 		),
 		'give_first' => array(
 			'error_id'      => 'invalid_first_name',
-			'error_message' => __( 'Please enter your first name', 'give' )
+			'error_message' => __('Please enter your first name', 'give')
 		)
 	);
 
-	$require_address = give_require_billing_address( $payment_mode );
+	$require_address = give_require_billing_address($payment_mode);
 
-	if ( $require_address ) {
-		$required_fields['card_address']    = array(
+	if ($require_address) {
+		$required_fields['card_address'] = array(
 			'error_id'      => 'invalid_card_address',
-			'error_message' => __( 'Please enter your primary billing address', 'give' )
+			'error_message' => __('Please enter your primary billing address', 'give')
 		);
-		$required_fields['card_zip']        = array(
+		$required_fields['card_zip'] = array(
 			'error_id'      => 'invalid_zip_code',
-			'error_message' => __( 'Please enter your zip / postal code', 'give' )
+			'error_message' => __('Please enter your zip / postal code', 'give')
 		);
-		$required_fields['card_city']       = array(
+		$required_fields['card_city'] = array(
 			'error_id'      => 'invalid_city',
-			'error_message' => __( 'Please enter your billing city', 'give' )
+			'error_message' => __('Please enter your billing city', 'give')
 		);
 		$required_fields['billing_country'] = array(
 			'error_id'      => 'invalid_country',
-			'error_message' => __( 'Please select your billing country', 'give' )
+			'error_message' => __('Please select your billing country', 'give')
 		);
-		$required_fields['card_state']      = array(
+		$required_fields['card_state'] = array(
 			'error_id'      => 'invalid_state',
-			'error_message' => __( 'Please enter billing state / province', 'give' )
+			'error_message' => __('Please enter billing state / province', 'give')
 		);
 	}
 
-	return apply_filters( 'give_purchase_form_required_fields', $required_fields, $form_id );
+	return apply_filters('give_purchase_form_required_fields', $required_fields, $form_id);
 
 }
 
@@ -330,16 +330,16 @@ 
  *
  * @return mixed|void
  */
-function give_require_billing_address( $payment_mode ) {
+function give_require_billing_address($payment_mode) {
 
 	$return = false;
 
-	if ( isset( $_POST['billing_country'] ) || did_action( "give_{$payment_mode}_cc_form" ) || did_action( 'give_cc_form' ) ) {
+	if (isset($_POST['billing_country']) || did_action("give_{$payment_mode}_cc_form") || did_action('give_cc_form')) {
 		$return = true;
 	}
 
 	// Let payment gateways and other extensions determine if address fields should be required
-	return apply_filters( 'give_require_billing_address', $return );
+	return apply_filters('give_require_billing_address', $return);
 
 }
 
@@ -353,43 +353,43 @@ 
 function give_purchase_form_validate_logged_in_user() {
 	global $user_ID;
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start empty array to collect valid user data
 	$valid_user_data = array(
 		// Assume there will be errors
-		'user_id' => - 1
+		'user_id' => -1
 	);
 
 	// Verify there is a user_ID
-	if ( $user_ID > 0 ) {
+	if ($user_ID > 0) {
 		// Get the logged in user data
-		$user_data = get_userdata( $user_ID );
+		$user_data = get_userdata($user_ID);
 
 		// Loop through required fields and show error messages
-		foreach ( give_purchase_form_required_fields( $form_id ) as $field_name => $value ) {
-			if ( in_array( $value, give_purchase_form_required_fields( $form_id ) ) && empty( $_POST[ $field_name ] ) ) {
-				give_set_error( $value['error_id'], $value['error_message'] );
+		foreach (give_purchase_form_required_fields($form_id) as $field_name => $value) {
+			if (in_array($value, give_purchase_form_required_fields($form_id)) && empty($_POST[$field_name])) {
+				give_set_error($value['error_id'], $value['error_message']);
 			}
 		}
 
 		// Verify data
-		if ( $user_data ) {
+		if ($user_data) {
 			// Collected logged in user data
 			$valid_user_data = array(
 				'user_id'    => $user_ID,
-				'user_email' => isset( $_POST['give_email'] ) ? sanitize_email( $_POST['give_email'] ) : $user_data->user_email,
-				'user_first' => isset( $_POST['give_first'] ) && ! empty( $_POST['give_first'] ) ? sanitize_text_field( $_POST['give_first'] ) : $user_data->first_name,
-				'user_last'  => isset( $_POST['give_last'] ) && ! empty( $_POST['give_last'] ) ? sanitize_text_field( $_POST['give_last'] ) : $user_data->last_name,
+				'user_email' => isset($_POST['give_email']) ? sanitize_email($_POST['give_email']) : $user_data->user_email,
+				'user_first' => isset($_POST['give_first']) && ! empty($_POST['give_first']) ? sanitize_text_field($_POST['give_first']) : $user_data->first_name,
+				'user_last'  => isset($_POST['give_last']) && ! empty($_POST['give_last']) ? sanitize_text_field($_POST['give_last']) : $user_data->last_name,
 			);
 
-			if ( ! is_email( $valid_user_data['user_email'] ) ) {
-				give_set_error( 'email_invalid', __( 'Invalid email', 'give' ) );
+			if ( ! is_email($valid_user_data['user_email'])) {
+				give_set_error('email_invalid', __('Invalid email', 'give'));
 			}
 
 		} else {
 			// Set invalid user error
-			give_set_error( 'invalid_user', __( 'The user information is invalid', 'give' ) );
+			give_set_error('invalid_user', __('The user information is invalid', 'give'));
 		}
 	}
 
@@ -407,92 +407,92 @@ 
 function give_purchase_form_validate_new_user() {
 	$registering_new_user = false;
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start an empty array to collect valid user data
 	$valid_user_data = array(
 		// Assume there will be errors
-		'user_id'    => - 1,
+		'user_id'    => -1,
 		// Get first name
-		'user_first' => isset( $_POST['give_first'] ) ? sanitize_text_field( $_POST['give_first'] ) : '',
+		'user_first' => isset($_POST['give_first']) ? sanitize_text_field($_POST['give_first']) : '',
 		// Get last name
-		'user_last'  => isset( $_POST['give_last'] ) ? sanitize_text_field( $_POST['give_last'] ) : '',
+		'user_last'  => isset($_POST['give_last']) ? sanitize_text_field($_POST['give_last']) : '',
 	);
 
 	// Check the new user's credentials against existing ones
-	$user_login   = isset( $_POST['give_user_login'] ) ? trim( $_POST['give_user_login'] ) : false;
-	$user_email   = isset( $_POST['give_email'] ) ? trim( $_POST['give_email'] ) : false;
-	$user_pass    = isset( $_POST['give_user_pass'] ) ? trim( $_POST['give_user_pass'] ) : false;
-	$pass_confirm = isset( $_POST['give_user_pass_confirm'] ) ? trim( $_POST['give_user_pass_confirm'] ) : false;
+	$user_login   = isset($_POST['give_user_login']) ? trim($_POST['give_user_login']) : false;
+	$user_email   = isset($_POST['give_email']) ? trim($_POST['give_email']) : false;
+	$user_pass    = isset($_POST['give_user_pass']) ? trim($_POST['give_user_pass']) : false;
+	$pass_confirm = isset($_POST['give_user_pass_confirm']) ? trim($_POST['give_user_pass_confirm']) : false;
 
 	// Loop through required fields and show error messages
-	foreach ( give_purchase_form_required_fields( $form_id ) as $field_name => $value ) {
-		if ( in_array( $value, give_purchase_form_required_fields( $form_id ) ) && empty( $_POST[ $field_name ] ) ) {
-			give_set_error( $value['error_id'], $value['error_message'] );
+	foreach (give_purchase_form_required_fields($form_id) as $field_name => $value) {
+		if (in_array($value, give_purchase_form_required_fields($form_id)) && empty($_POST[$field_name])) {
+			give_set_error($value['error_id'], $value['error_message']);
 		}
 	}
 
 	// Check if we have an username to register
-	if ( $user_login && strlen( $user_login ) > 0 ) {
+	if ($user_login && strlen($user_login) > 0) {
 		$registering_new_user = true;
 
 		// We have an user name, check if it already exists
-		if ( username_exists( $user_login ) ) {
+		if (username_exists($user_login)) {
 			// Username already registered
-			give_set_error( 'username_unavailable', __( 'Username already taken', 'give' ) );
+			give_set_error('username_unavailable', __('Username already taken', 'give'));
 			// Check if it's valid
-		} else if ( ! give_validate_username( $user_login ) ) {
+		} else if ( ! give_validate_username($user_login)) {
 			// 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'));
 			} else {
-				give_set_error( 'username_invalid', __( 'Invalid username', 'give' ) );
+				give_set_error('username_invalid', __('Invalid username', 'give'));
 			}
 		} else {
 			// All the checks have run and it's good to go
 			$valid_user_data['user_login'] = $user_login;
 		}
 	} else {
-		if ( give_no_guest_checkout( $form_id ) ) {
-			give_set_error( 'registration_required', __( 'You must register or login to complete your donation', 'give' ) );
+		if (give_no_guest_checkout($form_id)) {
+			give_set_error('registration_required', __('You must register or login to complete your donation', 'give'));
 		}
 	}
 
 	// Check if we have an email to verify
-	if ( $user_email && strlen( $user_email ) > 0 ) {
+	if ($user_email && strlen($user_email) > 0) {
 		// Validate email
-		if ( ! is_email( $user_email ) ) {
-			give_set_error( 'email_invalid', __( 'Invalid email', 'give' ) );
+		if ( ! is_email($user_email)) {
+			give_set_error('email_invalid', __('Invalid email', 'give'));
 			// Check if email exists
-		} else if ( email_exists( $user_email ) && $registering_new_user ) {
-			give_set_error( 'email_used', __( 'Email already used', 'give' ) );
+		} else if (email_exists($user_email) && $registering_new_user) {
+			give_set_error('email_used', __('Email already used', 'give'));
 		} else {
 			// All the checks have run and it's good to go
 			$valid_user_data['user_email'] = $user_email;
 		}
 	} else {
 		// No email
-		give_set_error( 'email_empty', __( 'Enter an email', 'give' ) );
+		give_set_error('email_empty', __('Enter an email', 'give'));
 	}
 
 	// Check password
-	if ( $user_pass && $pass_confirm ) {
+	if ($user_pass && $pass_confirm) {
 		// Verify confirmation matches
-		if ( $user_pass != $pass_confirm ) {
+		if ($user_pass != $pass_confirm) {
 			// Passwords do not match
-			give_set_error( 'password_mismatch', __( 'Passwords don\'t match', 'give' ) );
+			give_set_error('password_mismatch', __('Passwords don\'t match', 'give'));
 		} else {
 			// All is good to go
 			$valid_user_data['user_pass'] = $user_pass;
 		}
 	} else {
 		// Password or confirmation missing
-		if ( ! $user_pass && $registering_new_user ) {
+		if ( ! $user_pass && $registering_new_user) {
 			// The password is invalid
-			give_set_error( 'password_empty', __( 'Enter a password', 'give' ) );
-		} else if ( ! $pass_confirm && $registering_new_user ) {
+			give_set_error('password_empty', __('Enter a password', 'give'));
+		} else if ( ! $pass_confirm && $registering_new_user) {
 			// Confirmation password is invalid
-			give_set_error( 'confirmation_empty', __( 'Enter the password confirmation', 'give' ) );
+			give_set_error('confirmation_empty', __('Enter the password confirmation', 'give'));
 		}
 	}
 
@@ -511,34 +511,34 @@ 
 	// Start an array to collect valid user data
 	$valid_user_data = array(
 		// Assume there will be errors
-		'user_id' => - 1
+		'user_id' => -1
 	);
 
 	// Username
-	if ( ! isset( $_POST['give_user_login'] ) || $_POST['give_user_login'] == '' ) {
-		give_set_error( 'must_log_in', __( 'You must login or register to complete your donation', 'give' ) );
+	if ( ! isset($_POST['give_user_login']) || $_POST['give_user_login'] == '') {
+		give_set_error('must_log_in', __('You must login or register to complete your donation', 'give'));
 
 		return $valid_user_data;
 	}
 
 	// Get the user by login
-	$user_data = get_user_by( 'login', strip_tags( $_POST['give_user_login'] ) );
+	$user_data = get_user_by('login', strip_tags($_POST['give_user_login']));
 
 	// Check if user exists
-	if ( $user_data ) {
+	if ($user_data) {
 		// Get password
-		$user_pass = isset( $_POST['give_user_pass'] ) ? $_POST['give_user_pass'] : false;
+		$user_pass = isset($_POST['give_user_pass']) ? $_POST['give_user_pass'] : false;
 
 		// Check user_pass
-		if ( $user_pass ) {
+		if ($user_pass) {
 			// Check if password is valid
-			if ( ! wp_check_password( $user_pass, $user_data->user_pass, $user_data->ID ) ) {
+			if ( ! wp_check_password($user_pass, $user_data->user_pass, $user_data->ID)) {
 				// Incorrect password
 				give_set_error(
 					'password_incorrect',
 					sprintf(
-						__( 'The password you entered is incorrect. %sReset Password%s', 'give' ),
-						'<a href="' . wp_lostpassword_url( "http://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]" ) . '" title="' . __( 'Lost Password' ) . '">',
+						__('The password you entered is incorrect. %sReset Password%s', 'give'),
+						'<a href="'.wp_lostpassword_url("http://$_SERVER[HTTP_HOST]$_SERVER[REQUEST_URI]").'" title="'.__('Lost Password').'">',
 						'</a>'
 					)
 				);
@@ -556,11 +556,11 @@ 
 			}
 		} else {
 			// Empty password
-			give_set_error( 'password_empty', __( 'Enter a password', 'give' ) );
+			give_set_error('password_empty', __('Enter a password', 'give'));
 		}
 	} else {
 		// no username
-		give_set_error( 'username_incorrect', __( 'The username you entered does not exist', 'give' ) );
+		give_set_error('username_incorrect', __('The username you entered does not exist', 'give'));
 	}
 
 	return $valid_user_data;
@@ -575,7 +575,7 @@ 
  */
 function give_purchase_form_validate_guest_user() {
 
-	$form_id = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : '';
+	$form_id = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : '';
 
 	// Start an array to collect valid user data
 	$valid_user_data = array(
@@ -584,32 +584,32 @@ 
 	);
 
 	// Show error message if user must be logged in
-	if ( give_logged_in_only() ) {
-		give_set_error( 'logged_in_only', __( 'You must be logged into an account to donation', 'give' ) );
+	if (give_logged_in_only()) {
+		give_set_error('logged_in_only', __('You must be logged into an account to donation', 'give'));
 	}
 
 	// Get the guest email
-	$guest_email = isset( $_POST['give_email'] ) ? $_POST['give_email'] : false;
+	$guest_email = isset($_POST['give_email']) ? $_POST['give_email'] : false;
 
 	// Check email
-	if ( $guest_email && strlen( $guest_email ) > 0 ) {
+	if ($guest_email && strlen($guest_email) > 0) {
 		// Validate email
-		if ( ! is_email( $guest_email ) ) {
+		if ( ! is_email($guest_email)) {
 			// Invalid email
-			give_set_error( 'email_invalid', __( 'Invalid email', 'give' ) );
+			give_set_error('email_invalid', __('Invalid email', 'give'));
 		} else {
 			// All is good to go
 			$valid_user_data['user_email'] = $guest_email;
 		}
 	} else {
 		// No email
-		give_set_error( 'email_empty', __( 'Enter an email', 'give' ) );
+		give_set_error('email_empty', __('Enter an email', 'give'));
 	}
 
 	// Loop through required fields and show error messages
-	foreach ( give_purchase_form_required_fields( $form_id ) as $field_name => $value ) {
-		if ( in_array( $value, give_purchase_form_required_fields( $form_id ) ) && empty( $_POST[ $field_name ] ) ) {
-			give_set_error( $value['error_id'], $value['error_message'] );
+	foreach (give_purchase_form_required_fields($form_id) as $field_name => $value) {
+		if (in_array($value, give_purchase_form_required_fields($form_id)) && empty($_POST[$field_name])) {
+			give_set_error($value['error_id'], $value['error_message']);
 		}
 	}
 
@@ -625,42 +625,42 @@ 
  * @since   1.0
  * @return  integer
  */
-function give_register_and_login_new_user( $user_data = array() ) {
+function give_register_and_login_new_user($user_data = array()) {
 	// Verify the array
-	if ( empty( $user_data ) ) {
-		return - 1;
+	if (empty($user_data)) {
+		return -1;
 	}
 
-	if ( give_get_errors() ) {
-		return - 1;
+	if (give_get_errors()) {
+		return -1;
 	}
 
-	$user_args = apply_filters( 'give_insert_user_args', array(
-		'user_login'      => isset( $user_data['user_login'] ) ? $user_data['user_login'] : '',
-		'user_pass'       => isset( $user_data['user_pass'] ) ? $user_data['user_pass'] : '',
-		'user_email'      => isset( $user_data['user_email'] ) ? $user_data['user_email'] : '',
-		'first_name'      => isset( $user_data['user_first'] ) ? $user_data['user_first'] : '',
-		'last_name'       => isset( $user_data['user_last'] ) ? $user_data['user_last'] : '',
-		'user_registered' => date( 'Y-m-d H:i:s' ),
-		'role'            => get_option( 'default_role' )
-	), $user_data );
+	$user_args = apply_filters('give_insert_user_args', array(
+		'user_login'      => isset($user_data['user_login']) ? $user_data['user_login'] : '',
+		'user_pass'       => isset($user_data['user_pass']) ? $user_data['user_pass'] : '',
+		'user_email'      => isset($user_data['user_email']) ? $user_data['user_email'] : '',
+		'first_name'      => isset($user_data['user_first']) ? $user_data['user_first'] : '',
+		'last_name'       => isset($user_data['user_last']) ? $user_data['user_last'] : '',
+		'user_registered' => date('Y-m-d H:i:s'),
+		'role'            => get_option('default_role')
+	), $user_data);
 
 	// Insert new user
-	$user_id = wp_insert_user( $user_args );
+	$user_id = wp_insert_user($user_args);
 
 	// Validate inserted user
-	if ( is_wp_error( $user_id ) ) {
-		return - 1;
+	if (is_wp_error($user_id)) {
+		return -1;
 	}
 
 	// Allow themes and plugins to filter the user data
-	$user_data = apply_filters( 'give_insert_user_data', $user_data, $user_args );
+	$user_data = apply_filters('give_insert_user_data', $user_data, $user_args);
 
 	// Allow themes and plugins to hook
-	do_action( 'give_insert_user', $user_id, $user_data );
+	do_action('give_insert_user', $user_id, $user_data);
 
 	// Login new user
-	give_log_user_in( $user_id, $user_data['user_login'], $user_data['user_pass'] );
+	give_log_user_in($user_id, $user_data['user_login'], $user_data['user_pass']);
 
 	// Return user id
 	return $user_id;
@@ -675,27 +675,27 @@ 
  * @since   1.0
  * @return  array
  */
-function give_get_purchase_form_user( $valid_data = array() ) {
+function give_get_purchase_form_user($valid_data = array()) {
 	// Initialize user
 	$user    = false;
-	$is_ajax = defined( 'DOING_AJAX' ) && DOING_AJAX;
+	$is_ajax = defined('DOING_AJAX') && DOING_AJAX;
 
-	if ( $is_ajax ) {
+	if ($is_ajax) {
 		// Do not create or login the user during the ajax submission (check for errors only)
 		return true;
-	} else if ( is_user_logged_in() ) {
+	} else if (is_user_logged_in()) {
 		// Set the valid user as the logged in collected data
 
 		$user = $valid_data['logged_in_user'];
-	} else if ( $valid_data['need_new_user'] === true || $valid_data['need_user_login'] === true ) {
+	} else if ($valid_data['need_new_user'] === true || $valid_data['need_user_login'] === true) {
 		// New user registration
-		if ( $valid_data['need_new_user'] === true ) {
+		if ($valid_data['need_new_user'] === true) {
 			// Set user
 			$user = $valid_data['new_user_data'];
 			// Register and login new user
-			$user['user_id'] = give_register_and_login_new_user( $user );
+			$user['user_id'] = give_register_and_login_new_user($user);
 			// User login
-		} else if ( $valid_data['need_user_login'] === true && ! $is_ajax ) {
+		} else if ($valid_data['need_user_login'] === true && ! $is_ajax) {
 
 			/*
 			 * The login form is now processed in the give_process_purchase_login() function.
@@ -710,48 +710,48 @@ 
 			// Set user
 			$user = $valid_data['login_user_data'];
 			// Login user
-			give_log_user_in( $user['user_id'], $user['user_login'], $user['user_pass'] );
+			give_log_user_in($user['user_id'], $user['user_login'], $user['user_pass']);
 		}
 	}
 
 	// Check guest checkout
-	if ( false === $user && false === give_no_guest_checkout( $_POST['give-form-id'] ) ) {
+	if (false === $user && false === give_no_guest_checkout($_POST['give-form-id'])) {
 		// Set user
 		$user = $valid_data['guest_user_data'];
 	}
 
 	// Verify we have an user
-	if ( false === $user || empty( $user ) ) {
+	if (false === $user || empty($user)) {
 		// Return false
 		return false;
 	}
 
 	// Get user first name
-	if ( ! isset( $user['user_first'] ) || strlen( trim( $user['user_first'] ) ) < 1 ) {
-		$user['user_first'] = isset( $_POST["give_first"] ) ? strip_tags( trim( $_POST["give_first"] ) ) : '';
+	if ( ! isset($user['user_first']) || strlen(trim($user['user_first'])) < 1) {
+		$user['user_first'] = isset($_POST["give_first"]) ? strip_tags(trim($_POST["give_first"])) : '';
 	}
 
 	// Get user last name
-	if ( ! isset( $user['user_last'] ) || strlen( trim( $user['user_last'] ) ) < 1 ) {
-		$user['user_last'] = isset( $_POST["give_last"] ) ? strip_tags( trim( $_POST["give_last"] ) ) : '';
+	if ( ! isset($user['user_last']) || strlen(trim($user['user_last'])) < 1) {
+		$user['user_last'] = isset($_POST["give_last"]) ? strip_tags(trim($_POST["give_last"])) : '';
 	}
 
 	// Get the user's billing address details
 	$user['address']            = array();
-	$user['address']['line1']   = ! empty( $_POST['card_address'] ) ? sanitize_text_field( $_POST['card_address'] ) : false;
-	$user['address']['line2']   = ! empty( $_POST['card_address_2'] ) ? sanitize_text_field( $_POST['card_address_2'] ) : false;
-	$user['address']['city']    = ! empty( $_POST['card_city'] ) ? sanitize_text_field( $_POST['card_city'] ) : false;
-	$user['address']['state']   = ! empty( $_POST['card_state'] ) ? sanitize_text_field( $_POST['card_state'] ) : false;
-	$user['address']['country'] = ! empty( $_POST['billing_country'] ) ? sanitize_text_field( $_POST['billing_country'] ) : false;
-	$user['address']['zip']     = ! empty( $_POST['card_zip'] ) ? sanitize_text_field( $_POST['card_zip'] ) : false;
-
-	if ( empty( $user['address']['country'] ) ) {
+	$user['address']['line1']   = ! empty($_POST['card_address']) ? sanitize_text_field($_POST['card_address']) : false;
+	$user['address']['line2']   = ! empty($_POST['card_address_2']) ? sanitize_text_field($_POST['card_address_2']) : false;
+	$user['address']['city']    = ! empty($_POST['card_city']) ? sanitize_text_field($_POST['card_city']) : false;
+	$user['address']['state']   = ! empty($_POST['card_state']) ? sanitize_text_field($_POST['card_state']) : false;
+	$user['address']['country'] = ! empty($_POST['billing_country']) ? sanitize_text_field($_POST['billing_country']) : false;
+	$user['address']['zip']     = ! empty($_POST['card_zip']) ? sanitize_text_field($_POST['card_zip']) : false;
+
+	if (empty($user['address']['country'])) {
 		$user['address'] = false;
 	} // Country will always be set if address fields are present
 
-	if ( ! empty( $user['user_id'] ) && $user['user_id'] > 0 && ! empty( $user['address'] ) ) {
+	if ( ! empty($user['user_id']) && $user['user_id'] > 0 && ! empty($user['address'])) {
 		// Store the address in the user's meta so the cart can be pre-populated with it on return purchases
-		update_user_meta( $user['user_id'], '_give_user_address', $user['address'] );
+		update_user_meta($user['user_id'], '_give_user_address', $user['address']);
 	}
 
 	// Return valid user
@@ -770,16 +770,16 @@ 
 	$card_data = give_get_purchase_cc_info();
 
 	// Validate the card zip
-	if ( ! empty( $card_data['card_zip'] ) ) {
-		if ( ! give_purchase_form_validate_cc_zip( $card_data['card_zip'], $card_data['card_country'] ) ) {
-			give_set_error( 'invalid_cc_zip', __( 'The zip / postal code you entered for your billing address is invalid', 'give' ) );
+	if ( ! empty($card_data['card_zip'])) {
+		if ( ! give_purchase_form_validate_cc_zip($card_data['card_zip'], $card_data['card_country'])) {
+			give_set_error('invalid_cc_zip', __('The zip / postal code you entered for your billing address is invalid', 'give'));
 		}
 	}
 
 	//Ensure no spaces
-	if ( ! empty( $card_data['card_number'] ) ) {
-		$card_data['card_number'] = str_replace( '+', '', $card_data['card_number'] ); //no "+" signs
-		$card_data['card_number'] = str_replace( ' ', '', $card_data['card_number'] ); // No spaces
+	if ( ! empty($card_data['card_number'])) {
+		$card_data['card_number'] = str_replace('+', '', $card_data['card_number']); //no "+" signs
+		$card_data['card_number'] = str_replace(' ', '', $card_data['card_number']); // No spaces
 	}
 
 	// This should validate card numbers at some point too
@@ -795,17 +795,17 @@ 
  */
 function give_get_purchase_cc_info() {
 	$cc_info                   = array();
-	$cc_info['card_name']      = isset( $_POST['card_name'] ) ? sanitize_text_field( $_POST['card_name'] ) : '';
-	$cc_info['card_number']    = isset( $_POST['card_number'] ) ? sanitize_text_field( $_POST['card_number'] ) : '';
-	$cc_info['card_cvc']       = isset( $_POST['card_cvc'] ) ? sanitize_text_field( $_POST['card_cvc'] ) : '';
-	$cc_info['card_exp_month'] = isset( $_POST['card_exp_month'] ) ? sanitize_text_field( $_POST['card_exp_month'] ) : '';
-	$cc_info['card_exp_year']  = isset( $_POST['card_exp_year'] ) ? sanitize_text_field( $_POST['card_exp_year'] ) : '';
-	$cc_info['card_address']   = isset( $_POST['card_address'] ) ? sanitize_text_field( $_POST['card_address'] ) : '';
-	$cc_info['card_address_2'] = isset( $_POST['card_address_2'] ) ? sanitize_text_field( $_POST['card_address_2'] ) : '';
-	$cc_info['card_city']      = isset( $_POST['card_city'] ) ? sanitize_text_field( $_POST['card_city'] ) : '';
-	$cc_info['card_state']     = isset( $_POST['card_state'] ) ? sanitize_text_field( $_POST['card_state'] ) : '';
-	$cc_info['card_country']   = isset( $_POST['billing_country'] ) ? sanitize_text_field( $_POST['billing_country'] ) : '';
-	$cc_info['card_zip']       = isset( $_POST['card_zip'] ) ? sanitize_text_field( $_POST['card_zip'] ) : '';
+	$cc_info['card_name']      = isset($_POST['card_name']) ? sanitize_text_field($_POST['card_name']) : '';
+	$cc_info['card_number']    = isset($_POST['card_number']) ? sanitize_text_field($_POST['card_number']) : '';
+	$cc_info['card_cvc']       = isset($_POST['card_cvc']) ? sanitize_text_field($_POST['card_cvc']) : '';
+	$cc_info['card_exp_month'] = isset($_POST['card_exp_month']) ? sanitize_text_field($_POST['card_exp_month']) : '';
+	$cc_info['card_exp_year']  = isset($_POST['card_exp_year']) ? sanitize_text_field($_POST['card_exp_year']) : '';
+	$cc_info['card_address']   = isset($_POST['card_address']) ? sanitize_text_field($_POST['card_address']) : '';
+	$cc_info['card_address_2'] = isset($_POST['card_address_2']) ? sanitize_text_field($_POST['card_address_2']) : '';
+	$cc_info['card_city']      = isset($_POST['card_city']) ? sanitize_text_field($_POST['card_city']) : '';
+	$cc_info['card_state']     = isset($_POST['card_state']) ? sanitize_text_field($_POST['card_state']) : '';
+	$cc_info['card_country']   = isset($_POST['billing_country']) ? sanitize_text_field($_POST['billing_country']) : '';
+	$cc_info['card_zip']       = isset($_POST['card_zip']) ? sanitize_text_field($_POST['card_zip']) : '';
 
 	// Return cc info
 	return $cc_info;
@@ -821,14 +821,14 @@ 
  *
  * @return bool|mixed|void
  */
-function give_purchase_form_validate_cc_zip( $zip = 0, $country_code = '' ) {
+function give_purchase_form_validate_cc_zip($zip = 0, $country_code = '') {
 	$ret = false;
 
-	if ( empty( $zip ) || empty( $country_code ) ) {
+	if (empty($zip) || empty($country_code)) {
 		return $ret;
 	}
 
-	$country_code = strtoupper( $country_code );
+	$country_code = strtoupper($country_code);
 
 	$zip_regex = array(
 		"AD" => "AD\d{3}",
@@ -988,11 +988,11 @@ 
 		"ZM" => "\d{5}"
 	);
 
-	if ( ! isset ( $zip_regex[ $country_code ] ) || preg_match( "/" . $zip_regex[ $country_code ] . "/i", $zip ) ) {
+	if ( ! isset ($zip_regex[$country_code]) || preg_match("/".$zip_regex[$country_code]."/i", $zip)) {
 		$ret = true;
 	}
 
-	return apply_filters( 'give_is_zip_valid', $ret, $zip, $country_code );
+	return apply_filters('give_is_zip_valid', $ret, $zip, $country_code);
 }
 
 
@@ -1002,48 +1002,48 @@ 
  * @since       1.0
  * @return      void
  */
-function give_check_purchase_email( $valid_data, $posted ) {
+function give_check_purchase_email($valid_data, $posted) {
 	$is_banned = false;
 	$banned    = give_get_banned_emails();
 
-	if ( empty( $banned ) ) {
+	if (empty($banned)) {
 		return;
 	}
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 
 		// The user is logged in, check that their account email is not banned
-		$user_data = get_userdata( get_current_user_id() );
-		if ( give_is_email_banned( $user_data->user_email ) ) {
+		$user_data = get_userdata(get_current_user_id());
+		if (give_is_email_banned($user_data->user_email)) {
 
 			$is_banned = true;
 		}
 
-		if ( give_is_email_banned( $posted['give_email'] ) ) {
+		if (give_is_email_banned($posted['give_email'])) {
 			$is_banned = true;
 		}
 
-	} elseif ( isset( $posted['give-purchase-var'] ) && $posted['give-purchase-var'] == 'needs-to-login' ) {
+	} elseif (isset($posted['give-purchase-var']) && $posted['give-purchase-var'] == 'needs-to-login') {
 
 		// The user is logging in, check that their email is not banned
-		$user_data = get_user_by( 'login', $posted['give_user_login'] );
-		if ( $user_data && give_is_email_banned( $user_data->user_email ) ) {
+		$user_data = get_user_by('login', $posted['give_user_login']);
+		if ($user_data && give_is_email_banned($user_data->user_email)) {
 			$is_banned = true;
 		}
 
 	} else {
 
 		// Guest purchase, check that the email is not banned
-		if ( give_is_email_banned( $posted['give_email'] ) ) {
+		if (give_is_email_banned($posted['give_email'])) {
 			$is_banned = true;
 		}
 
 	}
 
-	if ( $is_banned ) {
+	if ($is_banned) {
 		// Set an error and give the donor a general error (don't alert them that they were banned)
-		give_set_error( 'email_banned', __( 'An internal error has occurred, please try again or contact support.', 'give' ) );
+		give_set_error('email_banned', __('An internal error has occurred, please try again or contact support.', 'give'));
 	}
 }
 
-add_action( 'give_checkout_error_checks', 'give_check_purchase_email', 10, 2 );
+add_action('give_checkout_error_checks', 'give_check_purchase_email', 10, 2);

includes/user-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -270,7 +270,7 @@
  * @access      public
  * @since       1.0
  *
- * @param       $username string - the username to validate
+ * @param       string $username string - the username to validate
  *
  * @return      bool
  */

Indentation +10 added lines, -10 removed lines

@@ -1,15 +1,15 @@
 <?php
 /**
- * User Functions
- *
- * Functions related to users / donors
- *
- * @package     Give
- * @subpackage  Functions
- * @copyright   Copyright (c) 2015, WordImpress
- * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
- * @since       1.0
- */
+	 * User Functions
+	 *
+	 * Functions related to users / donors
+	 *
+	 * @package     Give
+	 * @subpackage  Functions
+	 * @copyright   Copyright (c) 2015, WordImpress
+	 * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
+	 * @since       1.0
+	 */
 
 // Exit if accessed directly
 if ( ! defined( 'ABSPATH' ) ) {

Spacing +86 added lines, -86 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,36 +30,36 @@ 
  *
  * @return bool|object List of all user purchases
  */
-function give_get_users_purchases( $user = 0, $number = 20, $pagination = false, $status = 'complete' ) {
+function give_get_users_purchases($user = 0, $number = 20, $pagination = false, $status = 'complete') {
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( 0 === $user ) {
+	if (0 === $user) {
 		return false;
 	}
 
 	$status = $status === 'complete' ? 'publish' : $status;
 
-	if ( $pagination ) {
-		if ( get_query_var( 'paged' ) ) {
-			$paged = get_query_var( 'paged' );
-		} else if ( get_query_var( 'page' ) ) {
-			$paged = get_query_var( 'page' );
+	if ($pagination) {
+		if (get_query_var('paged')) {
+			$paged = get_query_var('paged');
+		} else if (get_query_var('page')) {
+			$paged = get_query_var('page');
 		} else {
 			$paged = 1;
 		}
 	}
 
-	$args = apply_filters( 'give_get_users_purchases_args', array(
+	$args = apply_filters('give_get_users_purchases_args', array(
 		'user'    => $user,
 		'number'  => $number,
 		'status'  => $status,
 		'orderby' => 'date'
-	) );
+	));
 
-	if ( $pagination ) {
+	if ($pagination) {
 
 		$args['page'] = $paged;
 
@@ -69,20 +69,20 @@ 
 
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
-	$customer   = new Give_Customer( $user, $by_user_id );
+	$by_user_id = is_numeric($user) ? true : false;
+	$customer   = new Give_Customer($user, $by_user_id);
 
-	if ( ! empty( $customer->payment_ids ) ) {
+	if ( ! empty($customer->payment_ids)) {
 
-		unset( $args['user'] );
-		$args['post__in'] = array_map( 'absint', explode( ',', $customer->payment_ids ) );
+		unset($args['user']);
+		$args['post__in'] = array_map('absint', explode(',', $customer->payment_ids));
 
 	}
 
-	$purchases = give_get_payments( apply_filters( 'give_get_users_purchases_args', $args ) );
+	$purchases = give_get_payments(apply_filters('give_get_users_purchases_args', $args));
 
 	// No purchases
-	if ( ! $purchases ) {
+	if ( ! $purchases) {
 		return false;
 	}
 
@@ -101,64 +101,64 @@ 
  *
  * @return bool|object List of unique forms purchased 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;
 
-	$customer = new Give_Customer( $user, $by_user_id );
+	$customer = new Give_Customer($user, $by_user_id);
 
-	if ( empty( $customer->payment_ids ) ) {
+	if (empty($customer->payment_ids)) {
 		return false;
 	}
 
 	// Get all the items purchased
-	$payment_ids    = array_reverse( explode( ',', $customer->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(',', $customer->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 purchased on this order
 	$completed_donations_ids = array();
-	foreach ( $donation_data as $purchase_meta ) {
+	foreach ($donation_data as $purchase_meta) {
 		$completed_donations_ids[] = $purchase_meta['form_id'];
 	}
-	if ( empty( $completed_donations_ids ) ) {
+	if (empty($completed_donations_ids)) {
 		return false;
 	}
 
 	// Only include each product purchased 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));
 }
 
 
@@ -174,12 +174,12 @@ 
  *
  * @return      bool - true if has purchased, false other wise.
  */
-function give_has_purchases( $user_id = null ) {
-	if ( empty( $user_id ) ) {
+function give_has_purchases($user_id = null) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	if ( give_get_users_purchases( $user_id, 1 ) ) {
+	if (give_get_users_purchases($user_id, 1)) {
 		return true; // User has at least one purchase
 	}
 
@@ -200,27 +200,27 @@ 
  *
  * @return      array
  */
-function give_get_purchase_stats_by_user( $user = '' ) {
+function give_get_purchase_stats_by_user($user = '') {
 
-	if ( is_email( $user ) ) {
+	if (is_email($user)) {
 
 		$field = 'email';
 
-	} elseif ( is_numeric( $user ) ) {
+	} elseif (is_numeric($user)) {
 
 		$field = 'user_id';
 
 	}
 
-	$customer = Give()->customers->get_customer_by( $field, $user );
+	$customer = Give()->customers->get_customer_by($field, $user);
 
-	$customer = new Give_Customer( $customer->id );
+	$customer = new Give_Customer($customer->id);
 
 	$stats                = array();
-	$stats['purchases']   = absint( $customer->purchase_count );
-	$stats['total_spent'] = give_sanitize_amount( $customer->purchase_value );
+	$stats['purchases']   = absint($customer->purchase_count);
+	$stats['total_spent'] = give_sanitize_amount($customer->purchase_value);
 
-	return (array) apply_filters( 'give_donation_stats_by_user', $stats, $user );
+	return (array) apply_filters('give_donation_stats_by_user', $stats, $user);
 }
 
 
@@ -236,14 +236,14 @@ 
  *
  * @return      int - the total number of purchases
  */
-function give_count_purchases_of_customer( $user = null ) {
-	if ( empty( $user ) ) {
+function give_count_purchases_of_customer($user = null) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	$stats = give_get_purchase_stats_by_user( $user );
+	$stats = give_get_purchase_stats_by_user($user);
 
-	return isset( $stats['purchases'] ) ? $stats['purchases'] : 0;
+	return isset($stats['purchases']) ? $stats['purchases'] : 0;
 }
 
 /**
@@ -256,9 +256,9 @@ 
  *
  * @return      float - the total amount the user has spent
  */
-function give_purchase_total_of_user( $user = null ) {
+function give_purchase_total_of_user($user = null) {
 
-	$stats = give_get_purchase_stats_by_user( $user );
+	$stats = give_get_purchase_stats_by_user($user);
 
 	return $stats['total_spent'];
 }
@@ -274,11 +274,11 @@ 
  *
  * @return      bool
  */
-function give_validate_username( $username ) {
-	$sanitized = sanitize_user( $username, false );
-	$valid     = ( $sanitized == $username );
+function give_validate_username($username) {
+	$sanitized = sanitize_user($username, false);
+	$valid     = ($sanitized == $username);
 
-	return (bool) apply_filters( 'give_validate_username', $valid, $username );
+	return (bool) apply_filters('give_validate_username', $valid, $username);
 }
 
 
@@ -295,32 +295,32 @@ 
  *
  * @return      void
  */
-function give_add_past_purchases_to_new_user( $user_id ) {
+function give_add_past_purchases_to_new_user($user_id) {
 
-	$email = get_the_author_meta( 'user_email', $user_id );
+	$email = get_the_author_meta('user_email', $user_id);
 
-	$payments = give_get_payments( array( 's' => $email ) );
+	$payments = give_get_payments(array('s' => $email));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
-			if ( intval( give_get_payment_user_id( $payment->ID ) ) > 0 ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
+			if (intval(give_get_payment_user_id($payment->ID)) > 0) {
 				continue;
 			} // This payment already associated with an account
 
-			$meta                    = give_get_payment_meta( $payment->ID );
-			$meta['user_info']       = maybe_unserialize( $meta['user_info'] );
+			$meta                    = give_get_payment_meta($payment->ID);
+			$meta['user_info']       = maybe_unserialize($meta['user_info']);
 			$meta['user_info']['id'] = $user_id;
 			$meta['user_info']       = $meta['user_info'];
 
 			// Store the updated user ID in the payment meta
-			give_update_payment_meta( $payment->ID, '_give_payment_meta', $meta );
-			give_update_payment_meta( $payment->ID, '_give_payment_user_id', $user_id );
+			give_update_payment_meta($payment->ID, '_give_payment_meta', $meta);
+			give_update_payment_meta($payment->ID, '_give_payment_user_id', $user_id);
 		}
 	}
 
 }
 
-add_action( 'user_register', 'give_add_past_purchases_to_new_user' );
+add_action('user_register', 'give_add_past_purchases_to_new_user');
 
 
 /**
@@ -342,34 +342,34 @@ 
  * @since         1.0
  * @return        array - The donor's address, if any
  */
-function give_get_donor_address( $user_id = 0 ) {
-	if ( empty( $user_id ) ) {
+function give_get_donor_address($user_id = 0) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	$address = get_user_meta( $user_id, '_give_user_address', true );
+	$address = get_user_meta($user_id, '_give_user_address', true);
 
-	if ( ! isset( $address['line1'] ) ) {
+	if ( ! isset($address['line1'])) {
 		$address['line1'] = '';
 	}
 
-	if ( ! isset( $address['line2'] ) ) {
+	if ( ! isset($address['line2'])) {
 		$address['line2'] = '';
 	}
 
-	if ( ! isset( $address['city'] ) ) {
+	if ( ! isset($address['city'])) {
 		$address['city'] = '';
 	}
 
-	if ( ! isset( $address['zip'] ) ) {
+	if ( ! isset($address['zip'])) {
 		$address['zip'] = '';
 	}
 
-	if ( ! isset( $address['country'] ) ) {
+	if ( ! isset($address['country'])) {
 		$address['country'] = '';
 	}
 
-	if ( ! isset( $address['state'] ) ) {
+	if ( ! isset($address['state'])) {
 		$address['state'] = '';
 	}
 
@@ -383,13 +383,13 @@ 
  * @since         1.0
  * @return        void
  */
-function give_new_user_notification( $user_id = 0, $user_data = array() ) {
+function give_new_user_notification($user_id = 0, $user_data = array()) {
 
-	if ( empty( $user_id ) || empty( $user_data ) ) {
+	if (empty($user_id) || empty($user_data)) {
 		return;
 	}
 
-	wp_new_user_notification( $user_id, __( '[Password entered during donation]', 'give' ) );
+	wp_new_user_notification($user_id, __('[Password entered during donation]', 'give'));
 }
 
-add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
+add_action('give_insert_user', 'give_new_user_notification', 10, 2);

includes/actions.php

Spacing +7 added lines, -7 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,12 +23,12 @@ 
  * @return void
  */
 function give_get_actions() {
-	if ( isset( $_GET['give_action'] ) ) {
-		do_action( 'give_' . $_GET['give_action'], $_GET );
+	if (isset($_GET['give_action'])) {
+		do_action('give_'.$_GET['give_action'], $_GET);
 	}
 }
 
-add_action( 'init', 'give_get_actions' );
+add_action('init', 'give_get_actions');
 
 /**
  * Hooks Give actions, when present in the $_POST superglobal. Every give_action
@@ -39,9 +39,9 @@ 
  * @return void
  */
 function give_post_actions() {
-	if ( isset( $_POST['give_action'] ) ) {
-		do_action( 'give_' . $_POST['give_action'], $_POST );
+	if (isset($_POST['give_action'])) {
+		do_action('give_'.$_POST['give_action'], $_POST);
 	}
 }
 
-add_action( 'init', 'give_post_actions' );
\ No newline at end of file
+add_action('init', 'give_post_actions');
\ No newline at end of file

includes/admin/add-ons.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,12 +25,12 @@ 
 function give_add_ons_page() {
 	ob_start(); ?>
 	<div class="wrap" id="give-add-ons">
-		<h2><?php _e( 'Give Add-ons', 'give' ); ?>
-			&nbsp;&mdash;&nbsp;<a href="https://givewp.com/addons/" class="button-primary give-view-addons-all" title="<?php _e( 'Browse All Extensions', 'give' ); ?>" target="_blank"><?php _e( 'View All Add-ons', 'give' ); ?>
+		<h2><?php _e('Give Add-ons', 'give'); ?>
+			&nbsp;&mdash;&nbsp;<a href="https://givewp.com/addons/" class="button-primary give-view-addons-all" title="<?php _e('Browse All Extensions', 'give'); ?>" target="_blank"><?php _e('View All Add-ons', 'give'); ?>
 				<span class="dashicons dashicons-external"></span></a>
 		</h2>
 
-		<p><?php _e( 'The following Add-ons extend the functionality of Give.', 'give' ); ?></p>
+		<p><?php _e('The following Add-ons extend the functionality of Give.', 'give'); ?></p>
 		<?php echo give_add_ons_get_feed(); ?>
 	</div>
 	<?php
@@ -48,18 +48,18 @@ 
 function give_add_ons_get_feed() {
 
 	$addons_debug = false; //set to true to debug
-	$cache        = get_transient( 'give_add_ons_feed' );
+	$cache        = get_transient('give_add_ons_feed');
 
-	if ( $cache === false || $addons_debug === true && WP_DEBUG === true ) {
-		$feed = wp_remote_get( 'https://givewp.com/downloads/feed/', array( 'sslverify' => false ) );
+	if ($cache === false || $addons_debug === true && WP_DEBUG === true) {
+		$feed = wp_remote_get('https://givewp.com/downloads/feed/', array('sslverify' => false));
 
-		if ( ! is_wp_error( $feed ) ) {
-			if ( isset( $feed['body'] ) && strlen( $feed['body'] ) > 0 ) {
-				$cache = wp_remote_retrieve_body( $feed );
-				set_transient( 'give_add_ons_feed', $cache, 3600 );
+		if ( ! is_wp_error($feed)) {
+			if (isset($feed['body']) && strlen($feed['body']) > 0) {
+				$cache = wp_remote_retrieve_body($feed);
+				set_transient('give_add_ons_feed', $cache, 3600);
 			}
 		} else {
-			$cache = '<div class="error"><p>' . __( 'There was an error retrieving the Give Add-ons list from the server. Please try again later.', 'give' ) . '</div>';
+			$cache = '<div class="error"><p>'.__('There was an error retrieving the Give Add-ons list from the server. Please try again later.', 'give').'</div>';
 		}
 	}
 

includes/admin/admin-actions.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,13 +22,13 @@ 
  * @return void
  */
 function give_process_actions() {
-	if ( isset( $_POST['give-action'] ) ) {
-		do_action( 'give_' . $_POST['give-action'], $_POST );
+	if (isset($_POST['give-action'])) {
+		do_action('give_'.$_POST['give-action'], $_POST);
 	}
 
-	if ( isset( $_GET['give-action'] ) ) {
-		do_action( 'give_' . $_GET['give-action'], $_GET );
+	if (isset($_GET['give-action'])) {
+		do_action('give_'.$_GET['give-action'], $_GET);
 	}
 }
 
-add_action( 'admin_init', 'give_process_actions' );
\ No newline at end of file
+add_action('admin_init', 'give_process_actions');
\ No newline at end of file

includes/admin/admin-footer.php

Spacing +5 added lines, -5 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,11 +24,11 @@ 
  *
  * @return      string
  */
-function give_admin_rate_us( $footer_text ) {
+function give_admin_rate_us($footer_text) {
 	global $typenow;
 
-	if ( $typenow == 'give_forms' ) {
-		$rate_text = sprintf( __( 'If you like <strong>Give</strong> please leave us a %s&#9733;&#9733;&#9733;&#9733;&#9733;%s rating. It takes a minute and helps a lot. Thanks in advance!', 'give' ), '<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" data-rated="' . __( 'Thanks :)', 'give' ) . '">', '</a>' );
+	if ($typenow == 'give_forms') {
+		$rate_text = sprintf(__('If you like <strong>Give</strong> please leave us a %s&#9733;&#9733;&#9733;&#9733;&#9733;%s rating. It takes a minute and helps a lot. Thanks in advance!', 'give'), '<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" data-rated="'.__('Thanks :)', 'give').'">', '</a>');
 
 		return $rate_text;
 	} else {
@@ -36,4 +36,4 @@ 
 	}
 }
 
-add_filter( 'admin_footer_text', 'give_admin_rate_us' );
+add_filter('admin_footer_text', 'give_admin_rate_us');

includes/admin/admin-pages.php

Spacing +65 added lines, -65 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -34,31 +34,31 @@ 
 	//	$give_campaigns_page = add_submenu_page( 'edit.php?post_type=give_forms', $give_campaigns->labels->menu_name, $give_campaigns->labels->add_new, 'edit_' . $give_campaigns->capability_type . 's', 'post-new.php?post_type=give_campaigns', null );
 
 	//Payments
-	$give_payment       = get_post_type_object( 'give_payment' );
-	$give_payments_page = add_submenu_page( 'edit.php?post_type=give_forms', $give_payment->labels->name, $give_payment->labels->menu_name, 'edit_give_payments', 'give-payment-history', 'give_payment_history_page' );
+	$give_payment       = get_post_type_object('give_payment');
+	$give_payments_page = add_submenu_page('edit.php?post_type=give_forms', $give_payment->labels->name, $give_payment->labels->menu_name, 'edit_give_payments', 'give-payment-history', 'give_payment_history_page');
 
 	//Donors
-	$give_donors_page = add_submenu_page( 'edit.php?post_type=give_forms', __( 'Donors', 'give' ), __( 'Donors', 'give' ), 'view_give_reports', 'give-donors', 'give_customers_page' );
+	$give_donors_page = add_submenu_page('edit.php?post_type=give_forms', __('Donors', 'give'), __('Donors', 'give'), 'view_give_reports', 'give-donors', 'give_customers_page');
 
 	//Reports
-	$give_reports_page = add_submenu_page( 'edit.php?post_type=give_forms', __( 'Donation Reports', 'give' ), __( 'Reports', 'give' ), 'view_give_reports', 'give-reports', 'give_reports_page' );
+	$give_reports_page = add_submenu_page('edit.php?post_type=give_forms', __('Donation Reports', 'give'), __('Reports', 'give'), 'view_give_reports', 'give-reports', 'give_reports_page');
 
 	//Settings
-	$give_settings_page = add_submenu_page( 'edit.php?post_type=give_forms', __( 'Give Settings', 'give' ), __( 'Settings', 'give' ), 'manage_give_settings', 'give-settings', array(
+	$give_settings_page = add_submenu_page('edit.php?post_type=give_forms', __('Give Settings', 'give'), __('Settings', 'give'), 'manage_give_settings', 'give-settings', array(
 		Give()->give_settings,
 		'admin_page_display'
-	) );
+	));
 
 	//Add-ons
-	$give_add_ons_page = add_submenu_page( 'edit.php?post_type=give_forms', __( 'Give Add-ons', 'give' ), __( 'Add-ons', 'give' ), 'install_plugins', 'give-addons', 'give_add_ons_page' );
+	$give_add_ons_page = add_submenu_page('edit.php?post_type=give_forms', __('Give Add-ons', 'give'), __('Add-ons', 'give'), 'install_plugins', 'give-addons', 'give_add_ons_page');
 
 	//Upgrades
-	$give_upgrades_screen = add_submenu_page( null, __( 'Give Upgrades', 'give' ), __( 'Give Upgrades', 'give' ), 'manage_give_settings', 'give-upgrades', 'give_upgrades_screen' );
+	$give_upgrades_screen = add_submenu_page(null, __('Give Upgrades', 'give'), __('Give Upgrades', 'give'), 'manage_give_settings', 'give-upgrades', 'give_upgrades_screen');
 
 
 }
 
-add_action( 'admin_menu', 'give_add_options_links', 10 );
+add_action('admin_menu', 'give_add_options_links', 10);
 
 /**
  *  Determines whether the current admin page is a Give admin page.
@@ -73,224 +73,224 @@ 
  *
  * @return bool True if Give admin page.
  */
-function give_is_admin_page( $passed_page = '', $passed_view = '' ) {
+function give_is_admin_page($passed_page = '', $passed_view = '') {
 
 	global $pagenow, $typenow;
 
 	$found     = false;
-	$post_type = isset( $_GET['post_type'] ) ? strtolower( $_GET['post_type'] ) : false;
-	$action    = isset( $_GET['action'] ) ? strtolower( $_GET['action'] ) : false;
-	$taxonomy  = isset( $_GET['taxonomy'] ) ? strtolower( $_GET['taxonomy'] ) : false;
-	$page      = isset( $_GET['page'] ) ? strtolower( $_GET['page'] ) : false;
-	$view      = isset( $_GET['view'] ) ? strtolower( $_GET['view'] ) : false;
-	$tab       = isset( $_GET['tab'] ) ? strtolower( $_GET['tab'] ) : false;
+	$post_type = isset($_GET['post_type']) ? strtolower($_GET['post_type']) : false;
+	$action    = isset($_GET['action']) ? strtolower($_GET['action']) : false;
+	$taxonomy  = isset($_GET['taxonomy']) ? strtolower($_GET['taxonomy']) : false;
+	$page      = isset($_GET['page']) ? strtolower($_GET['page']) : false;
+	$view      = isset($_GET['view']) ? strtolower($_GET['view']) : false;
+	$tab       = isset($_GET['tab']) ? strtolower($_GET['tab']) : false;
 
-	switch ( $passed_page ) {
+	switch ($passed_page) {
 		case 'give_forms':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php') {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'post.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'post.php') {
 						$found = true;
 					}
 					break;
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'post-new.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'post-new.php') {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) || 'give_forms' === $post_type || ( 'post-new.php' == $pagenow && 'give_forms' === $post_type ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) || 'give_forms' === $post_type || ('post-new.php' == $pagenow && 'give_forms' === $post_type)) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'categories':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'tags':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'payments':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page && false === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page && false === $view) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page && 'view-order-details' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page && 'view-order-details' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'reports':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				// If you want to do something like enqueue a script on a particular report's duration, look at $_GET[ 'range' ]
 				case 'earnings':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && ( 'earnings' === $view || '-1' === $view || false === $view ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && ('earnings' === $view || '-1' === $view || false === $view)) {
 						$found = true;
 					}
 					break;
 				case 'donors':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'customers' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'customers' === $view) {
 						$found = true;
 					}
 					break;
 				case 'gateways':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'gateways' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'gateways' === $view) {
 						$found = true;
 					}
 					break;
 				case 'export':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'export' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'export' === $view) {
 						$found = true;
 					}
 					break;
 				case 'logs':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'logs' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'logs' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'settings':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'general':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && ( 'general' === $tab || false === $tab ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && ('general' === $tab || false === $tab)) {
 						$found = true;
 					}
 					break;
 				case 'gateways':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'gateways' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'gateways' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'emails':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'emails' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'emails' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'display':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'display' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'display' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'licenses':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'licenses' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'licenses' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'api':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'api' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'api' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'advanced':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'advanced' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'advanced' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'system_info':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'system_info' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'system_info' === $tab) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'addons':
-			if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-addons' === $page ) {
+			if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-addons' === $page) {
 				$found = true;
 			}
 			break;
 		case 'donors':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && false === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && false === $view) {
 						$found = true;
 					}
 					break;
 				case 'overview':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && 'overview' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && 'overview' === $view) {
 						$found = true;
 					}
 					break;
 				case 'notes':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && 'notes' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && 'notes' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'reports':
-			if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page ) {
+			if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page) {
 				$found = true;
 			}
 			break;
 		default:
 			global $give_payments_page, $give_settings_page, $give_reports_page, $give_system_info_page, $give_add_ons_page, $give_settings_export, $give_upgrades_screen, $give_customers_page;
 
-			$admin_pages = apply_filters( 'give_admin_pages', array(
+			$admin_pages = apply_filters('give_admin_pages', array(
 				$give_payments_page,
 				$give_settings_page,
 				$give_reports_page,
@@ -299,18 +299,18 @@ 
 				$give_upgrades_screen,
 				$give_settings_export,
 				$give_customers_page
-			) );
-			if ( 'give_forms' == $typenow || 'index.php' == $pagenow || 'post-new.php' == $pagenow || 'post.php' == $pagenow ) {
+			));
+			if ('give_forms' == $typenow || 'index.php' == $pagenow || 'post-new.php' == $pagenow || 'post.php' == $pagenow) {
 				$found = true;
-				if ( 'give-upgrades' === $page ) {
+				if ('give-upgrades' === $page) {
 					$found = false;
 				}
-			} elseif ( in_array( $pagenow, $admin_pages ) ) {
+			} elseif (in_array($pagenow, $admin_pages)) {
 				$found = true;
 			}
 			break;
 	}
 
-	return (bool) apply_filters( 'give_is_admin_page', $found, $page, $view, $passed_page, $passed_view );
+	return (bool) apply_filters('give_is_admin_page', $found, $page, $view, $passed_page, $passed_view);
 
 }
\ No newline at end of file