WordImpress / Give

includes/misc-functions.php

Indentation +8 added lines, -8 removed lines

@@ -727,10 +727,10 @@ 
 		}
 
 		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_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 );
 
@@ -738,10 +738,10 @@ 
 		}
 
 		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 );
 

Doc Comments +8 added lines, -8 removed lines

@@ -323,7 +323,7 @@ 
  *
  * @since 1.0
  * @uses  Give()->session->get()
- * @return mixed array | false
+ * @return string array | false
  */
 function give_get_purchase_session() {
 	return Give()->session->get( 'give_purchase' );
@@ -360,7 +360,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;
@@ -551,7 +551,7 @@ 
  *
  * @since 1.0
  *
- * @param unknown $n
+ * @param integer $n
  *
  * @return string Short month name
  */
@@ -681,9 +681,9 @@ 
 /**
  * Create SVG library function
  *
- * @param $icon
+ * @param string $icon
  *
- * @return mixed
+ * @return string
  */
 function give_svg_icons( $icon ) {
 
@@ -740,7 +740,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
@@ -894,11 +894,11 @@ 
 	/**
 	 * cal_days_in_month
 	 *
-	 * @param $calendar
+	 * @param integer $calendar
 	 * @param $month
 	 * @param $year
 	 *
-	 * @return bool|string
+	 * @return string
 	 */
 	function cal_days_in_month( $calendar, $month, $year ) {
 		return date( 't', mktime( 0, 0, 0, $month, 1, $year ) );

Spacing +165 added lines, -165 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,9 +51,9 @@ 
  */
 function give_get_currency_position() {
 	global $give_options;
-	$currency_pos = isset( $give_options['currency_position'] ) ? $give_options['currency_position'] : 'before';
+	$currency_pos = isset($give_options['currency_position']) ? $give_options['currency_position'] : 'before';
 
-	return apply_filters( 'give_currency_position', $currency_pos );
+	return apply_filters('give_currency_position', $currency_pos);
 }
 
 
@@ -66,39 +66,39 @@ 
 
 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 (Kč)', 'give' ),
-		'DKK'  => __( 'Danish Krone (kr)', 'give' ),
-		'HKD'  => __( 'Hong Kong Dollar ($)', 'give' ),
-		'HUF'  => __( 'Hungarian Forint (Ft)', 'give' ),
-		'ILS'  => __( 'Israeli Shekel (₪)', 'give' ),
-		'JPY'  => __( 'Japanese Yen (¥)', 'give' ),
-		'MYR'  => __( 'Malaysian Ringgits (RM)', 'give' ),
-		'MXN'  => __( 'Mexican Peso ($)', 'give' ),
-		'MAD'  => __( 'Moroccan Dirham (.د.م)', 'give' ),
-		'NZD'  => __( 'New Zealand Dollar ($)', 'give' ),
-		'NOK'  => __( 'Norwegian Krone (Kr.)', 'give' ),
-		'PHP'  => __( 'Philippine Pesos (₱)', 'give' ),
-		'PLN'  => __( 'Polish Zloty (zł)', 'give' ),
-		'SGD'  => __( 'Singapore Dollar ($)', 'give' ),
-		'KRW'  => __( 'South Korean Won (₩)', 'give' ),
-		'ZAR'  => __( 'South African Rand (R)', 'give' ),
-		'SEK'  => __( 'Swedish Krona (kr)', 'give' ),
-		'CHF'  => __( 'Swiss Franc (CHF)', 'give' ),
-		'TWD'  => __( 'Taiwan New Dollars (NT$)', '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 (Kč)', 'give'),
+		'DKK'  => __('Danish Krone (kr)', 'give'),
+		'HKD'  => __('Hong Kong Dollar ($)', 'give'),
+		'HUF'  => __('Hungarian Forint (Ft)', 'give'),
+		'ILS'  => __('Israeli Shekel (₪)', 'give'),
+		'JPY'  => __('Japanese Yen (¥)', 'give'),
+		'MYR'  => __('Malaysian Ringgits (RM)', 'give'),
+		'MXN'  => __('Mexican Peso ($)', 'give'),
+		'MAD'  => __('Moroccan Dirham (.د.م)', 'give'),
+		'NZD'  => __('New Zealand Dollar ($)', 'give'),
+		'NOK'  => __('Norwegian Krone (Kr.)', 'give'),
+		'PHP'  => __('Philippine Pesos (₱)', 'give'),
+		'PLN'  => __('Polish Zloty (zł)', 'give'),
+		'SGD'  => __('Singapore Dollar ($)', 'give'),
+		'KRW'  => __('South Korean Won (₩)', 'give'),
+		'ZAR'  => __('South African Rand (R)', 'give'),
+		'SEK'  => __('Swedish Krona (kr)', 'give'),
+		'CHF'  => __('Swiss Franc (CHF)', 'give'),
+		'TWD'  => __('Taiwan New Dollars (NT$)', '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);
 }
 
 
@@ -113,12 +113,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;
@@ -197,7 +197,7 @@ 
 	endswitch;
 
 
-	return apply_filters( 'give_currency_symbol', $symbol, $currency );
+	return apply_filters('give_currency_symbol', $symbol, $currency);
 }
 
 
@@ -209,13 +209,13 @@ 
  */
 function give_get_current_page_url() {
 
-	if ( is_front_page() ) {
-		$current_url = home_url( '/' );
+	if (is_front_page()) {
+		$current_url = home_url('/');
 	} else {
-		$current_url = set_url_scheme( 'http://' . $_SERVER['HTTP_HOST'] . untrailingslashit( $_SERVER['REQUEST_URI'] ) );
+		$current_url = set_url_scheme('http://'.$_SERVER['HTTP_HOST'].untrailingslashit($_SERVER['REQUEST_URI']));
 	}
 
-	return apply_filters( 'give_get_current_page_url', esc_url( $current_url ) );
+	return apply_filters('give_get_current_page_url', esc_url($current_url));
 }
 
 
@@ -236,15 +236,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);
 }
 
 /**
@@ -256,26 +256,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'];
 				}
 			}
@@ -299,17 +299,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);
 }
 
 
@@ -324,9 +324,9 @@ 
  *
  * @uses  Give()->session->set()
  */
-function give_set_purchase_session( $purchase_data = array() ) {
-	Give()->session->set( 'give_purchase', $purchase_data );
-	Give()->session->set( 'give_email', $purchase_data['user_email'] );
+function give_set_purchase_session($purchase_data = array()) {
+	Give()->session->set('give_purchase', $purchase_data);
+	Give()->session->set('give_email', $purchase_data['user_email']);
 }
 
 /**
@@ -340,7 +340,7 @@ 
  * @return mixed array | false
  */
 function give_get_purchase_session() {
-	return Give()->session->get( 'give_purchase' );
+	return Give()->session->get('give_purchase');
 }
 
 /**
@@ -355,14 +355,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;
 }
@@ -379,31 +379,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;
@@ -419,67 +419,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;
@@ -512,20 +512,20 @@ 
  * @param string $replacement Optional. The function that should have been called
  * @param array $backtrace Optional. Contains stack backtrace of deprecated function
  */
-function _give_deprecated_function( $function, $version, $replacement = null, $backtrace = null ) {
-	do_action( 'give_deprecated_function_run', $function, $replacement, $version );
+function _give_deprecated_function($function, $version, $replacement = null, $backtrace = null) {
+	do_action('give_deprecated_function_run', $function, $replacement, $version);
 
-	$show_errors = current_user_can( 'manage_options' );
+	$show_errors = current_user_can('manage_options');
 
 	// Allow plugin to filter the output error trigger
-	if ( WP_DEBUG && apply_filters( 'give_deprecated_function_trigger_error', $show_errors ) ) {
-		if ( ! is_null( $replacement ) ) {
-			trigger_error( sprintf( __( '%1$s is <strong>deprecated</strong> since Give version %2$s! Use %3$s instead.', 'give' ), $function, $version, $replacement ) );
-			trigger_error( print_r( $backtrace, 1 ) ); // Limited to previous 1028 characters, but since we only need to move back 1 in stack that should be fine.
+	if (WP_DEBUG && apply_filters('give_deprecated_function_trigger_error', $show_errors)) {
+		if ( ! is_null($replacement)) {
+			trigger_error(sprintf(__('%1$s is <strong>deprecated</strong> since Give version %2$s! Use %3$s instead.', 'give'), $function, $version, $replacement));
+			trigger_error(print_r($backtrace, 1)); // Limited to previous 1028 characters, but since we only need to move back 1 in stack that should be fine.
 			// Alternatively we could dump this to a file.
 		} else {
-			trigger_error( sprintf( __( '%1$s is <strong>deprecated</strong> since Give version %2$s with no alternative available.', 'give' ), $function, $version ) );
-			trigger_error( print_r( $backtrace, 1 ) );// Limited to previous 1028 characters, but since we only need to move back 1 in stack that should be fine.
+			trigger_error(sprintf(__('%1$s is <strong>deprecated</strong> since Give version %2$s with no alternative available.', 'give'), $function, $version));
+			trigger_error(print_r($backtrace, 1)); // Limited to previous 1028 characters, but since we only need to move back 1 in stack that should be fine.
 			// Alternatively we could dump this to a file.
 		}
 	}
@@ -539,8 +539,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'];
 	}
 
@@ -554,7 +554,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');
 }
 
 
@@ -569,10 +569,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);
 }
 
 
@@ -585,10 +585,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);
 }
 
 
@@ -603,7 +603,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">
@@ -699,7 +699,7 @@ 
  *
  * @return mixed
  */
-function give_svg_icons( $icon ) {
+function give_svg_icons($icon) {
 
 	// Store your SVGs in an associative array
 	$svgs = array(
@@ -711,7 +711,7 @@ 
 	);
 
 	// Return the chosen icon's SVG string
-	return $svgs[ $icon ];
+	return $svgs[$icon];
 }
 
 /**
@@ -723,15 +723,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');
 
 
 /**
@@ -744,7 +744,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.
@@ -763,56 +763,56 @@ 
 	 *
 	 * @return array
 	 */
-	function array_column( $input = null, $columnKey = null, $indexKey = null ) {
+	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 );
+		if ($argc < 2) {
+			trigger_error("array_column() expects at least 2 parameters, {$argc} given", E_USER_WARNING);
 
 			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 );
+			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' ) )
+		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 );
+			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;
+		$paramsColumnKey = ($params[1] !== null) ? (string) $params[1] : null;
 
 		$paramsIndexKey = null;
-		if ( isset( $params[2] ) ) {
-			if ( is_float( $params[2] ) || is_int( $params[2] ) ) {
+		if (isset($params[2])) {
+			if (is_float($params[2]) || is_int($params[2])) {
 				$paramsIndexKey = (int) $params[2];
 			} else {
 				$paramsIndexKey = (string) $params[2];
@@ -821,26 +821,26 @@ 
 
 		$resultArray = array();
 
-		foreach ( $paramsInput as $row ) {
+		foreach ($paramsInput as $row) {
 			$key    = $value = null;
 			$keySet = $valueSet = false;
 
-			if ( $paramsIndexKey !== null && array_key_exists( $paramsIndexKey, $row ) ) {
+			if ($paramsIndexKey !== null && array_key_exists($paramsIndexKey, $row)) {
 				$keySet = true;
-				$key    = (string) $row[ $paramsIndexKey ];
+				$key    = (string) $row[$paramsIndexKey];
 			}
 
-			if ( $paramsColumnKey === null ) {
+			if ($paramsColumnKey === null) {
 				$valueSet = true;
 				$value    = $row;
-			} elseif ( is_array( $row ) && array_key_exists( $paramsColumnKey, $row ) ) {
+			} elseif (is_array($row) && array_key_exists($paramsColumnKey, $row)) {
 				$valueSet = true;
-				$value    = $row[ $paramsColumnKey ];
+				$value    = $row[$paramsColumnKey];
 			}
 
-			if ( $valueSet ) {
-				if ( $keySet ) {
-					$resultArray[ $key ] = $value;
+			if ($valueSet) {
+				if ($keySet) {
+					$resultArray[$key] = $value;
 				} else {
 					$resultArray[] = $value;
 				}
@@ -862,40 +862,40 @@ 
  *
  * @return bool Whether the receipt is visible or not.
  */
-function give_can_view_receipt( $payment_key = '' ) {
+function give_can_view_receipt($payment_key = '') {
 
 	$return = false;
 
-	if ( empty( $payment_key ) ) {
+	if (empty($payment_key)) {
 		return $return;
 	}
 
 	global $give_receipt_args;
 
-	$give_receipt_args['id'] = give_get_purchase_id_by_key( $payment_key );
+	$give_receipt_args['id'] = give_get_purchase_id_by_key($payment_key);
 
-	$user_id = (int) give_get_payment_user_id( $give_receipt_args['id'] );
+	$user_id = (int) give_get_payment_user_id($give_receipt_args['id']);
 
-	$payment_meta = give_get_payment_meta( $give_receipt_args['id'] );
+	$payment_meta = give_get_payment_meta($give_receipt_args['id']);
 
-	if ( is_user_logged_in() ) {
-		if ( $user_id === (int) get_current_user_id() ) {
+	if (is_user_logged_in()) {
+		if ($user_id === (int) get_current_user_id()) {
 			$return = true;
-		} elseif ( wp_get_current_user()->user_email === give_get_payment_user_email( $give_receipt_args['id'] ) ) {
+		} elseif (wp_get_current_user()->user_email === give_get_payment_user_email($give_receipt_args['id'])) {
 			$return = true;
-		} elseif ( current_user_can( 'view_give_sensitive_data' ) ) {
+		} elseif (current_user_can('view_give_sensitive_data')) {
 			$return = true;
 		}
 	}
 
 	$session = give_get_purchase_session();
-	if ( ! empty( $session ) && ! is_user_logged_in() ) {
-		if ( $session['purchase_key'] === $payment_meta['key'] ) {
+	if ( ! empty($session) && ! is_user_logged_in()) {
+		if ($session['purchase_key'] === $payment_meta['key']) {
 			$return = true;
 		}
 	}
 
-	return (bool) apply_filters( 'give_can_view_receipt', $return, $payment_key );
+	return (bool) apply_filters('give_can_view_receipt', $return, $payment_key);
 
 }
 
@@ -904,7 +904,7 @@ 
  *
  * @description: Fallback in case the calendar extension is not loaded in PHP; Only supports Gregorian calendar
  */
-if ( ! function_exists( 'cal_days_in_month' ) ) {
+if ( ! function_exists('cal_days_in_month')) {
 	/**
 	 * cal_days_in_month
 	 *
@@ -914,7 +914,7 @@ 
 	 *
 	 * @return bool|string
 	 */
-	function cal_days_in_month( $calendar, $month, $year ) {
-		return date( 't', mktime( 0, 0, 0, $month, 1, $year ) );
+	function cal_days_in_month($calendar, $month, $year) {
+		return date('t', mktime(0, 0, 0, $month, 1, $year));
 	}
 }

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 Add-ons', '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 Add-ons', '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/reporting/class-gateways-reports-table.php

Spacing +33 added lines, -33 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 
 	}
 
@@ -64,10 +64,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -80,11 +80,11 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'label'           => esc_attr__( 'Gateway', 'give' ),
-			'complete_sales'  => esc_attr__( 'Complete Transactions', 'give' ),
-			'pending_sales'   => esc_attr__( 'Pending / Failed Transactions', 'give' ),
-			'total_sales'     => esc_attr__( 'Total Transactions', 'give' ),
-			'total_donations' => esc_attr__( 'Total Donations', 'give' )
+			'label'           => esc_attr__('Gateway', 'give'),
+			'complete_sales'  => esc_attr__('Complete Transactions', 'give'),
+			'pending_sales'   => esc_attr__('Pending / Failed Transactions', 'give'),
+			'total_sales'     => esc_attr__('Total Transactions', 'give'),
+			'total_donations' => esc_attr__('Total Donations', 'give')
 		);
 
 		return $columns;
@@ -99,7 +99,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 
@@ -110,7 +110,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place
 		give_report_views();
 	}
@@ -123,23 +123,23 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' == $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' == $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<h3 class="alignleft reports-earnings-title"><span><?php _e( 'Payment Methods Report', 'give' ); ?></span></h3>
+			<h3 class="alignleft reports-earnings-title"><span><?php _e('Payment Methods Report', 'give'); ?></span></h3>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -164,18 +164,18 @@ 
 		$gateways     = give_get_payment_gateways();
 		$stats        = new Give_Payment_Stats();
 
-		foreach ( $gateways as $gateway_id => $gateway ) {
+		foreach ($gateways as $gateway_id => $gateway) {
 
-			$complete_count = give_count_sales_by_gateway( $gateway_id, 'publish' );
-			$pending_count  = give_count_sales_by_gateway( $gateway_id, array( 'pending', 'failed' ) );
+			$complete_count = give_count_sales_by_gateway($gateway_id, 'publish');
+			$pending_count  = give_count_sales_by_gateway($gateway_id, array('pending', 'failed'));
 
 			$reports_data[] = array(
 				'ID'              => $gateway_id,
 				'label'           => $gateway['admin_label'],
-				'complete_sales'  => give_format_amount( $complete_count, false ),
-				'pending_sales'   => give_format_amount( $pending_count, false ),
-				'total_sales'     => give_format_amount( $complete_count + $pending_count, false ),
-				'total_donations' => give_currency_filter( give_format_amount( $stats->get_earnings( 0, 0, 0, $gateway_id ) ) )
+				'complete_sales'  => give_format_amount($complete_count, false),
+				'pending_sales'   => give_format_amount($pending_count, false),
+				'total_sales'     => give_format_amount($complete_count + $pending_count, false),
+				'total_donations' => give_currency_filter(give_format_amount($stats->get_earnings(0, 0, 0, $gateway_id)))
 			);
 		}
 
@@ -197,7 +197,7 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->reports_data();
 
 	}

includes/emails/class-give-email-tags.php

Doc Comments +2 added lines, -1 removed lines

@@ -50,6 +50,7 @@ 
 	 *
 	 * @param string   $tag  Email tag to be replace in email
 	 * @param callable $func Hook to run when email tag is found
+	 * @param string $description
 	 */
 	public function add( $tag, $description, $func ) {
 		if ( is_callable( $func ) ) {
@@ -477,7 +478,7 @@ 
  *
  * @param int $payment_id
  *
- * @return int payment_id
+ * @return string payment_id
  */
 function give_email_tag_payment_id( $payment_id ) {
 	return give_get_payment_number( $payment_id );

Spacing +93 added lines, -93 removed lines

@@ -23,7 +23,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,9 +54,9 @@ 
 	 * @param string $tag Email tag to be replace in email
 	 * @param callable $func Hook to run when email tag is found
 	 */
-	public function add( $tag, $description, $func ) {
-		if ( is_callable( $func ) ) {
-			$this->tags[ $tag ] = array(
+	public function add($tag, $description, $func) {
+		if (is_callable($func)) {
+			$this->tags[$tag] = array(
 				'tag'         => $tag,
 				'description' => $description,
 				'func'        => $func
@@ -71,8 +71,8 @@ 
 	 *
 	 * @param string $tag Email tag to remove hook from
 	 */
-	public function remove( $tag ) {
-		unset( $this->tags[ $tag ] );
+	public function remove($tag) {
+		unset($this->tags[$tag]);
 	}
 
 	/**
@@ -84,8 +84,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function email_tag_exists( $tag ) {
-		return array_key_exists( $tag, $this->tags );
+	public function email_tag_exists($tag) {
+		return array_key_exists($tag, $this->tags);
 	}
 
 	/**
@@ -109,16 +109,16 @@ 
 	 *
 	 * @return string Content with email tags filtered out.
 	 */
-	public function do_tags( $content, $payment_id ) {
+	public function do_tags($content, $payment_id) {
 
 		// Check if there is atleast one tag added
-		if ( empty( $this->tags ) || ! is_array( $this->tags ) ) {
+		if (empty($this->tags) || ! is_array($this->tags)) {
 			return $content;
 		}
 
 		$this->payment_id = $payment_id;
 
-		$new_content = preg_replace_callback( "/{([A-z0-9\-\_]+)}/s", array( $this, 'do_tag' ), $content );
+		$new_content = preg_replace_callback("/{([A-z0-9\-\_]+)}/s", array($this, 'do_tag'), $content);
 
 		$this->payment_id = null;
 
@@ -134,17 +134,17 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function do_tag( $m ) {
+	public function do_tag($m) {
 
 		// Get tag
 		$tag = $m[1];
 
 		// Return tag if tag not set
-		if ( ! $this->email_tag_exists( $tag ) ) {
+		if ( ! $this->email_tag_exists($tag)) {
 			return $m[0];
 		}
 
-		return call_user_func( $this->tags[ $tag ]['func'], $this->payment_id, $tag );
+		return call_user_func($this->tags[$tag]['func'], $this->payment_id, $tag);
 	}
 
 }
@@ -158,8 +158,8 @@ 
  * @param string $description Description of the email tag added
  * @param callable $func Hook to run when email tag is found
  */
-function give_add_email_tag( $tag, $description, $func ) {
-	Give()->email_tags->add( $tag, $description, $func );
+function give_add_email_tag($tag, $description, $func) {
+	Give()->email_tags->add($tag, $description, $func);
 }
 
 /**
@@ -169,8 +169,8 @@ 
  *
  * @param string $tag Email tag to remove hook from
  */
-function give_remove_email_tag( $tag ) {
-	Give()->email_tags->remove( $tag );
+function give_remove_email_tag($tag) {
+	Give()->email_tags->remove($tag);
 }
 
 /**
@@ -182,8 +182,8 @@ 
  *
  * @return bool
  */
-function give_email_tag_exists( $tag ) {
-	return Give()->email_tags->email_tag_exists( $tag );
+function give_email_tag_exists($tag) {
+	return Give()->email_tags->email_tag_exists($tag);
 }
 
 /**
@@ -212,13 +212,13 @@ 
 	$email_tags = give_get_email_tags();
 
 	// Check
-	if ( count( $email_tags ) > 0 ) {
+	if (count($email_tags) > 0) {
 
 		// Loop
-		foreach ( $email_tags as $email_tag ) {
+		foreach ($email_tags as $email_tag) {
 
 			// Add email tag to list
-			$list .= '<code>{' . $email_tag['tag'] . '}</code> - ' . $email_tag['description'] . '<br/>';
+			$list .= '<code>{'.$email_tag['tag'].'}</code> - '.$email_tag['description'].'<br/>';
 
 		}
 
@@ -238,13 +238,13 @@ 
  *
  * @return string Content with email tags filtered out.
  */
-function give_do_email_tags( $content, $payment_id ) {
+function give_do_email_tags($content, $payment_id) {
 
 	// Replace all tags
-	$content = Give()->email_tags->do_tags( $content, $payment_id );
+	$content = Give()->email_tags->do_tags($content, $payment_id);
 
 	// Maintaining backwards compatibility
-	$content = apply_filters( 'give_email_template_tags', $content, give_get_payment_meta( $payment_id ), $payment_id );
+	$content = apply_filters('give_email_template_tags', $content, give_get_payment_meta($payment_id), $payment_id);
 
 	// Return content
 	return $content;
@@ -256,10 +256,10 @@ 
  * @since 1.0
  */
 function give_load_email_tags() {
-	do_action( 'give_add_email_tags' );
+	do_action('give_add_email_tags');
 }
 
-add_action( 'init', 'give_load_email_tags', - 999 );
+add_action('init', 'give_load_email_tags', - 999);
 
 /**
  * Add default Give email template tags
@@ -272,82 +272,82 @@ 
 	$email_tags = array(
 		array(
 			'tag'         => 'donation',
-			'description' => __( 'The name of completed donation form and the donation level chosen if applicable.', 'give' ),
+			'description' => __('The name of completed donation form and the donation level chosen if applicable.', 'give'),
 			'function'    => 'give_email_tag_donation'
 		),
 		array(
 			'tag'         => 'name',
-			'description' => __( 'The donor\'s first name', 'give' ),
+			'description' => __('The donor\'s first name', 'give'),
 			'function'    => 'give_email_tag_first_name'
 		),
 		array(
 			'tag'         => 'fullname',
-			'description' => __( 'The donor\'s full name, first and last', 'give' ),
+			'description' => __('The donor\'s full name, first and last', 'give'),
 			'function'    => 'give_email_tag_fullname'
 		),
 		array(
 			'tag'         => 'username',
-			'description' => __( 'The donor\'s user name on the site, if they registered an account', 'give' ),
+			'description' => __('The donor\'s user name on the site, if they registered an account', 'give'),
 			'function'    => 'give_email_tag_username'
 		),
 		array(
 			'tag'         => 'user_email',
-			'description' => __( 'The donor\'s email address', 'give' ),
+			'description' => __('The donor\'s email address', 'give'),
 			'function'    => 'give_email_tag_user_email'
 		),
 		array(
 			'tag'         => 'billing_address',
-			'description' => __( 'The donor\'s billing address', 'give' ),
+			'description' => __('The donor\'s billing address', 'give'),
 			'function'    => 'give_email_tag_billing_address'
 		),
 		array(
 			'tag'         => 'date',
-			'description' => __( 'The date of the donation', 'give' ),
+			'description' => __('The date of the donation', 'give'),
 			'function'    => 'give_email_tag_date'
 		),
 		array(
 			'tag'         => 'price',
-			'description' => __( 'The total price of the donation', 'give' ),
+			'description' => __('The total price of the donation', 'give'),
 			'function'    => 'give_email_tag_price'
 		),
 		array(
 			'tag'         => 'payment_id',
-			'description' => __( 'The unique ID number for this donation', 'give' ),
+			'description' => __('The unique ID number for this donation', 'give'),
 			'function'    => 'give_email_tag_payment_id'
 		),
 		array(
 			'tag'         => 'receipt_id',
-			'description' => __( 'The unique ID number for this donation receipt', 'give' ),
+			'description' => __('The unique ID number for this donation receipt', 'give'),
 			'function'    => 'give_email_tag_receipt_id'
 		),
 		array(
 			'tag'         => 'payment_method',
-			'description' => __( 'The method of payment used for this donation', 'give' ),
+			'description' => __('The method of payment used for this donation', 'give'),
 			'function'    => 'give_email_tag_payment_method'
 		),
 		array(
 			'tag'         => 'sitename',
-			'description' => __( 'Your site name', 'give' ),
+			'description' => __('Your site name', 'give'),
 			'function'    => 'give_email_tag_sitename'
 		),
 		array(
 			'tag'         => 'receipt_link',
-			'description' => __( 'Adds a link so users can view their receipt directly on your website if they are unable to view it in the browser correctly.', 'give' ),
+			'description' => __('Adds a link so users can view their receipt directly on your website if they are unable to view it in the browser correctly.', 'give'),
 			'function'    => 'give_email_tag_receipt_link'
 		),
 	);
 
 	// Apply give_email_tags filter
-	$email_tags = apply_filters( 'give_email_tags', $email_tags );
+	$email_tags = apply_filters('give_email_tags', $email_tags);
 
 	// Add email tags
-	foreach ( $email_tags as $email_tag ) {
-		give_add_email_tag( $email_tag['tag'], $email_tag['description'], $email_tag['function'] );
+	foreach ($email_tags as $email_tag) {
+		give_add_email_tag($email_tag['tag'], $email_tag['description'], $email_tag['function']);
 	}
 
 }
 
-add_action( 'give_add_email_tags', 'give_setup_email_tags' );
+add_action('give_add_email_tags', 'give_setup_email_tags');
 
 
 /**
@@ -358,15 +358,15 @@ 
  *
  * @return string name
  */
-function give_email_tag_first_name( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_first_name($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['name'];
 }
@@ -379,15 +379,15 @@ 
  *
  * @return string fullname
  */
-function give_email_tag_fullname( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_fullname($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['fullname'];
 }
@@ -400,15 +400,15 @@ 
  *
  * @return string username
  */
-function give_email_tag_username( $payment_id ) {
-	$payment   = new Give_Payment( $payment_id );
+function give_email_tag_username($payment_id) {
+	$payment   = new Give_Payment($payment_id);
 	$user_info = $payment->user_info;
 
-	if ( empty( $user_info ) ) {
+	if (empty($user_info)) {
 		return '';
 	}
 
-	$email_name = give_get_email_names( $user_info );
+	$email_name = give_get_email_names($user_info);
 
 	return $email_name['username'];
 }
@@ -421,8 +421,8 @@ 
  *
  * @return string user_email
  */
-function give_email_tag_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -435,10 +435,10 @@ 
  *
  * @return string billing_address
  */
-function give_email_tag_billing_address( $payment_id ) {
+function give_email_tag_billing_address($payment_id) {
 
-	$user_info    = give_get_payment_meta_user_info( $payment_id );
-	$user_address = ! empty( $user_info['address'] ) ? $user_info['address'] : array(
+	$user_info    = give_get_payment_meta_user_info($payment_id);
+	$user_address = ! empty($user_info['address']) ? $user_info['address'] : array(
 		'line1'   => '',
 		'line2'   => '',
 		'city'    => '',
@@ -447,11 +447,11 @@ 
 		'zip'     => ''
 	);
 
-	$return = $user_address['line1'] . "\n";
-	if ( ! empty( $user_address['line2'] ) ) {
-		$return .= $user_address['line2'] . "\n";
+	$return = $user_address['line1']."\n";
+	if ( ! empty($user_address['line2'])) {
+		$return .= $user_address['line2']."\n";
 	}
-	$return .= $user_address['city'] . ' ' . $user_address['zip'] . ' ' . $user_address['state'] . "\n";
+	$return .= $user_address['city'].' '.$user_address['zip'].' '.$user_address['state']."\n";
 	$return .= $user_address['country'];
 
 	return $return;
@@ -465,10 +465,10 @@ 
  *
  * @return string date
  */
-function give_email_tag_date( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_date($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return date_i18n( get_option( 'date_format' ), strtotime( $payment->date ) );
+	return date_i18n(get_option('date_format'), strtotime($payment->date));
 }
 
 /**
@@ -479,11 +479,11 @@ 
  *
  * @return string price
  */
-function give_email_tag_price( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
-	$price   = give_currency_filter( give_format_amount( $payment->total ), $payment->currency );
+function give_email_tag_price($payment_id) {
+	$payment = new Give_Payment($payment_id);
+	$price   = give_currency_filter(give_format_amount($payment->total), $payment->currency);
 
-	return html_entity_decode( $price, ENT_COMPAT, 'UTF-8' );
+	return html_entity_decode($price, ENT_COMPAT, 'UTF-8');
 }
 
 /**
@@ -494,8 +494,8 @@ 
  *
  * @return int payment_id
  */
-function give_email_tag_payment_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -508,8 +508,8 @@ 
  *
  * @return string receipt_id
  */
-function give_email_tag_receipt_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_receipt_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -523,11 +523,11 @@ 
  *
  * @return string $form_title
  */
-function give_email_tag_donation( $payment_id ) {
-	$payment    = new Give_Payment( $payment_id );
-	$form_title = give_get_payment_form_title( $payment->meta, false, '-' );
+function give_email_tag_donation($payment_id) {
+	$payment    = new Give_Payment($payment_id);
+	$form_title = give_get_payment_form_title($payment->meta, false, '-');
 
-	return ! empty( $form_title ) ? $form_title : __( 'There was an error retrieving this donation title', 'give' );
+	return ! empty($form_title) ? $form_title : __('There was an error retrieving this donation title', 'give');
 
 }
 
@@ -539,10 +539,10 @@ 
  *
  * @return string gateway
  */
-function give_email_tag_payment_method( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_email_tag_payment_method($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
-	return give_get_gateway_checkout_label( $payment->gateway );
+	return give_get_gateway_checkout_label($payment->gateway);
 }
 
 /**
@@ -553,8 +553,8 @@ 
  *
  * @return string sitename
  */
-function give_email_tag_sitename( $payment_id ) {
-	return wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES );
+function give_email_tag_sitename($payment_id) {
+	return wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES);
 }
 
 /**
@@ -566,19 +566,19 @@ 
  *
  * @return string receipt_link
  */
-function give_email_tag_receipt_link( $payment_id ) {
+function give_email_tag_receipt_link($payment_id) {
 
-	$receipt_url = esc_url( add_query_arg( array(
-		'payment_key' => give_get_payment_key( $payment_id ),
+	$receipt_url = esc_url(add_query_arg(array(
+		'payment_key' => give_get_payment_key($payment_id),
 		'give_action' => 'view_receipt'
-	), home_url() ) );
-	$formatted   = sprintf(
+	), home_url()));
+	$formatted = sprintf(
 		'<a href="%1$s">%2$s</a>',
 		$receipt_url,
-		__( 'View it in your browser', 'give' )
+		__('View it in your browser', 'give')
 	);
 
-	if ( give_get_option( 'email_template' ) !== 'none' ) {
+	if (give_get_option('email_template') !== 'none') {
 		return $formatted;
 	} else {
 		return $receipt_url;

includes/admin/dashboard-widgets.php

Spacing +32 added lines, -32 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_register_dashboard_widgets() {
-	if ( current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
-		wp_add_dashboard_widget( 'give_dashboard_sales', __( 'Give: Donation Statistics', 'give' ), 'give_dashboard_sales_widget' );
+	if (current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
+		wp_add_dashboard_widget('give_dashboard_sales', __('Give: Donation Statistics', 'give'), 'give_dashboard_sales_widget');
 	}
 }
 
-add_action( 'wp_dashboard_setup', 'give_register_dashboard_widgets', 10 );
+add_action('wp_dashboard_setup', 'give_register_dashboard_widgets', 10);
 
 /**
  * Sales Summary Dashboard Widget
@@ -38,7 +38,7 @@ 
  */
 function give_dashboard_sales_widget() {
 
-	if ( ! current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
+	if ( ! current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
 		return;
 	}
 	$stats = new Give_Payment_Stats; ?>
@@ -46,19 +46,19 @@ 
 	<div class="give-dashboard-widget">
 
 		<div class="give-dashboard-today give-clearfix">
-			<h3 class="give-dashboard-date-today"><?php echo date( 'F j, Y' ); ?></h3>
+			<h3 class="give-dashboard-date-today"><?php echo date('F j, Y'); ?></h3>
 
-			<p class="give-dashboard-happy-day"><?php printf( __( 'Happy %1$s!', 'give' ), date( 'l', current_time( 'timestamp' ) ) ); ?></p>
+			<p class="give-dashboard-happy-day"><?php printf(__('Happy %1$s!', 'give'), date('l', current_time('timestamp'))); ?></p>
 
-			<?php $earnings_today = $stats->get_earnings( 0, 'today', false ); ?>
+			<?php $earnings_today = $stats->get_earnings(0, 'today', false); ?>
 
-			<p class="give-dashboard-today-earnings"><?php echo give_currency_filter( give_format_amount( $earnings_today ) ); ?></p>
+			<p class="give-dashboard-today-earnings"><?php echo give_currency_filter(give_format_amount($earnings_today)); ?></p>
 
-			<p class="give-orders-today"><?php $donations_today = $stats->get_sales( 0, 'today', false, array(
+			<p class="give-orders-today"><?php $donations_today = $stats->get_sales(0, 'today', false, array(
 					'publish',
 					'revoked'
-				) ); ?><?php echo give_format_amount( $donations_today, false ); ?>
-				<span><?php echo _x( 'donations today', 'Displays in WP admin dashboard widget after the day\'s total donations', 'give' ); ?></span>
+				)); ?><?php echo give_format_amount($donations_today, false); ?>
+				<span><?php echo _x('donations today', 'Displays in WP admin dashboard widget after the day\'s total donations', 'give'); ?></span>
 			</p>
 
 
@@ -68,34 +68,34 @@ 
 		<table class="give-table-stats">
 			<thead style="display: none;">
 			<tr>
-				<th><?php _e( 'This Week', 'give' ); ?></th>
-				<th><?php _e( 'This Month', 'give' ); ?></th>
-				<th><?php _e( 'Past 30 Days', 'give' ); ?></th>
+				<th><?php _e('This Week', 'give'); ?></th>
+				<th><?php _e('This Month', 'give'); ?></th>
+				<th><?php _e('Past 30 Days', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
 			<tr id="give-table-stats-tr-1">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_week' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_week'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'this week', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('this week', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_month' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_month'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'this month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('this month', 'give'); ?></p>
 				</td>
 			</tr>
 			<tr id="give-table-stats-tr-2">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'last_month' ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'last_month'))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'last month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('last month', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_year', false ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_year', false))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'this year', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('this year', 'give'); ?></p>
 				</td>
 			</tr>
 			</tbody>
@@ -115,19 +115,19 @@ 
  *
  * @return array
  */
-function give_dashboard_at_a_glance_widget( $items ) {
-	$num_posts = wp_count_posts( 'give_forms' );
+function give_dashboard_at_a_glance_widget($items) {
+	$num_posts = wp_count_posts('give_forms');
 
-	if ( $num_posts && $num_posts->publish ) {
+	if ($num_posts && $num_posts->publish) {
         
-		$text = sprintf( _n( '%1$s Give %2$s', '%1$s Give %3$s', $num_posts->publish, 'give' ), $num_posts->publish, give_get_forms_label_singular(), give_get_forms_label_plural() );
+		$text = sprintf(_n('%1$s Give %2$s', '%1$s Give %3$s', $num_posts->publish, 'give'), $num_posts->publish, give_get_forms_label_singular(), give_get_forms_label_plural());
 
-		$text = sprintf( $text, number_format_i18n( $num_posts->publish ) );
+		$text = sprintf($text, number_format_i18n($num_posts->publish));
 
-		if ( current_user_can( 'edit_give_forms', get_current_user_id() ) ) {
-			$text = sprintf( '<a class="give-forms-count" href="edit.php?post_type=give_forms">%1$s</a>', $text );
+		if (current_user_can('edit_give_forms', get_current_user_id())) {
+			$text = sprintf('<a class="give-forms-count" href="edit.php?post_type=give_forms">%1$s</a>', $text);
 		} else {
-			$text = sprintf( '<span class="give-forms-count">%1$s</span>', $text );
+			$text = sprintf('<span class="give-forms-count">%1$s</span>', $text);
 		}
 
 		$items[] = $text;
@@ -136,4 +136,4 @@ 
 	return $items;
 }
 
-add_filter( 'dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);
+add_filter('dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);

includes/forms/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -154,7 +154,7 @@ 
  * Used to redirect a user back to the purchase
  * page if there are errors present.
  *
- * @param array $args
+ * @param string $args
  *
  * @access public
  * @since  1.0
@@ -773,7 +773,7 @@ 
  *
  * @param int $form_id ID number of the form to retrieve the minimum price for
  *
- * @return mixed string|int Minimum price of the form
+ * @return string string|int Minimum price of the form
  */
 function give_get_form_minimum_price( $form_id = 0 ) {
 
@@ -878,7 +878,7 @@ 
  *
  * @param int $form_id ID number of the form to retrieve a goal for
  *
- * @return mixed string|int Goal of the form
+ * @return string string|int Goal of the form
  */
 function give_get_form_goal( $form_id = 0 ) {
 

Spacing +183 added lines, -183 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,14 +23,14 @@ 
 
 	global $typenow;
 
-	if ( $typenow != 'give_forms' ) {
+	if ($typenow != 'give_forms') {
 		return true;
 	}
 
 	return false;
 }
 
-add_filter( 'give_shortcode_button_condition', 'give_shortcode_button_condition' );
+add_filter('give_shortcode_button_condition', 'give_shortcode_button_condition');
 
 
 /**
@@ -40,11 +40,11 @@ 
  *
  * @return int|false
  */
-function get_form_id_from_args( $args ) {
+function get_form_id_from_args($args) {
 
-	if ( isset( $args['form_id'] ) && $args['form_id'] != 0 ) {
+	if (isset($args['form_id']) && $args['form_id'] != 0) {
 
-		return intval( $args['form_id'] );
+		return intval($args['form_id']);
 	}
 
 	return false;
@@ -59,23 +59,23 @@ 
  *
  * @return bool
  */
-function give_is_float_labels_enabled( $args ) {
+function give_is_float_labels_enabled($args) {
 
 	$float_labels = '';
 
-	if ( ! empty( $args['float_labels'] ) ) {
+	if ( ! empty($args['float_labels'])) {
 		$float_labels = $args['float_labels'];
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = get_post_meta( $args['form_id'], '_give_form_floating_labels', true );
+	if (empty($float_labels)) {
+		$float_labels = get_post_meta($args['form_id'], '_give_form_floating_labels', true);
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = give_get_option( 'enable_floatlabels' ) ? 'enabled' : 'disabled';
+	if (empty($float_labels)) {
+		$float_labels = give_get_option('enable_floatlabels') ? 'enabled' : 'disabled';
 	}
 
-	return ( $float_labels == 'enabled' ) ? true : false;
+	return ($float_labels == 'enabled') ? true : false;
 }
 
 /**
@@ -90,7 +90,7 @@ 
 
 	$can_checkout = true;
 
-	return (bool) apply_filters( 'give_can_checkout', $can_checkout );
+	return (bool) apply_filters('give_can_checkout', $can_checkout);
 }
 
 /**
@@ -103,9 +103,9 @@ 
 function give_get_success_page_uri() {
 	global $give_options;
 
-	$success_page = isset( $give_options['success_page'] ) ? get_permalink( absint( $give_options['success_page'] ) ) : get_bloginfo( 'url' );
+	$success_page = isset($give_options['success_page']) ? get_permalink(absint($give_options['success_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_success_page_uri', $success_page );
+	return apply_filters('give_get_success_page_uri', $success_page);
 }
 
 /**
@@ -116,9 +116,9 @@ 
  */
 function give_is_success_page() {
 	global $give_options;
-	$is_success_page = isset( $give_options['success_page'] ) ? is_page( $give_options['success_page'] ) : false;
+	$is_success_page = isset($give_options['success_page']) ? is_page($give_options['success_page']) : false;
 
-	return apply_filters( 'give_is_success_page', $is_success_page );
+	return apply_filters('give_is_success_page', $is_success_page);
 }
 
 /**
@@ -132,17 +132,17 @@ 
  * @since       1.0
  * @return      void
  */
-function give_send_to_success_page( $query_string = null ) {
+function give_send_to_success_page($query_string = null) {
 
 	$redirect = give_get_success_page_uri();
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$redirect .= $query_string;
 	}
 
-	$gateway = isset( $_REQUEST['give-gateway'] ) ? $_REQUEST['give-gateway'] : '';
+	$gateway = isset($_REQUEST['give-gateway']) ? $_REQUEST['give-gateway'] : '';
 
-	wp_redirect( apply_filters( 'give_success_page_redirect', $redirect, $gateway, $query_string ) );
+	wp_redirect(apply_filters('give_success_page_redirect', $redirect, $gateway, $query_string));
 	give_die();
 }
 
@@ -159,25 +159,25 @@ 
  * @since  1.0
  * @return Void
  */
-function give_send_back_to_checkout( $args = array() ) {
+function give_send_back_to_checkout($args = array()) {
 
-	$redirect = ( isset( $_POST['give-current-url'] ) ) ? $_POST['give-current-url'] : '';
-	$form_id  = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : 0;
+	$redirect = (isset($_POST['give-current-url'])) ? $_POST['give-current-url'] : '';
+	$form_id  = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : 0;
 
 	$defaults = array(
 		'form-id' => (int) $form_id
 	);
 
 	// Check for backward compatibility
-	if ( is_string( $args ) ) {
-		$args = str_replace( '?', '', $args );
+	if (is_string($args)) {
+		$args = str_replace('?', '', $args);
 	}
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
-	$redirect = add_query_arg( $args, $redirect ) . '#give-form-' . $form_id . '-wrap';
+	$redirect = add_query_arg($args, $redirect).'#give-form-'.$form_id.'-wrap';
 
-	wp_redirect( apply_filters( 'give_send_back_to_checkout', $redirect, $args ) );
+	wp_redirect(apply_filters('give_send_back_to_checkout', $redirect, $args));
 	give_die();
 }
 
@@ -192,16 +192,16 @@ 
  * @since       1.0
  * @return      string
  */
-function give_get_success_page_url( $query_string = null ) {
+function give_get_success_page_url($query_string = null) {
 
-	$success_page = give_get_option( 'success_page', 0 );
-	$success_page = get_permalink( $success_page );
+	$success_page = give_get_option('success_page', 0);
+	$success_page = get_permalink($success_page);
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$success_page .= $query_string;
 	}
 
-	return apply_filters( 'give_success_page_url', $success_page );
+	return apply_filters('give_success_page_url', $success_page);
 
 }
 
@@ -215,15 +215,15 @@ 
  *
  * @return mixed|void Full URL to the Transaction Failed page, if present, home page if it doesn't exist
  */
-function give_get_failed_transaction_uri( $extras = false ) {
+function give_get_failed_transaction_uri($extras = false) {
 	global $give_options;
 
-	$uri = ! empty( $give_options['failure_page'] ) ? trailingslashit( get_permalink( $give_options['failure_page'] ) ) : home_url();
-	if ( $extras ) {
+	$uri = ! empty($give_options['failure_page']) ? trailingslashit(get_permalink($give_options['failure_page'])) : home_url();
+	if ($extras) {
 		$uri .= $extras;
 	}
 
-	return apply_filters( 'give_get_failed_transaction_uri', $uri );
+	return apply_filters('give_get_failed_transaction_uri', $uri);
 }
 
 /**
@@ -234,9 +234,9 @@ 
  */
 function give_is_failed_transaction_page() {
 	global $give_options;
-	$ret = isset( $give_options['failure_page'] ) ? is_page( $give_options['failure_page'] ) : false;
+	$ret = isset($give_options['failure_page']) ? is_page($give_options['failure_page']) : false;
 
-	return apply_filters( 'give_is_failure_page', $ret );
+	return apply_filters('give_is_failure_page', $ret);
 }
 
 /**
@@ -248,18 +248,18 @@ 
  */
 function give_listen_for_failed_payments() {
 
-	$failed_page = give_get_option( 'failure_page', 0 );
+	$failed_page = give_get_option('failure_page', 0);
 
-	if ( ! empty( $failed_page ) && is_page( $failed_page ) && ! empty( $_GET['payment-id'] ) ) {
+	if ( ! empty($failed_page) && is_page($failed_page) && ! empty($_GET['payment-id'])) {
 
-		$payment_id = absint( $_GET['payment-id'] );
-		give_update_payment_status( $payment_id, 'failed' );
+		$payment_id = absint($_GET['payment-id']);
+		give_update_payment_status($payment_id, 'failed');
 
 	}
 
 }
 
-add_action( 'template_redirect', 'give_listen_for_failed_payments' );
+add_action('template_redirect', 'give_listen_for_failed_payments');
 
 
 /**
@@ -272,11 +272,11 @@ 
  * @since       1.0
  * @return      bool
  */
-function give_field_is_required( $field = '', $form_id ) {
+function give_field_is_required($field = '', $form_id) {
 
-	$required_fields = give_purchase_form_required_fields( $form_id );
+	$required_fields = give_purchase_form_required_fields($form_id);
 
-	return array_key_exists( $field, $required_fields );
+	return array_key_exists($field, $required_fields);
 }
 
 /**
@@ -294,14 +294,14 @@ 
  *
  * @return void
  */
-function give_record_sale_in_log( $give_form_id = 0, $payment_id, $price_id = false, $sale_date = null ) {
+function give_record_sale_in_log($give_form_id = 0, $payment_id, $price_id = false, $sale_date = null) {
 	global $give_logs;
 
 	$log_data = array(
 		'post_parent'   => $give_form_id,
 		'log_type'      => 'sale',
-		'post_date'     => isset( $sale_date ) ? $sale_date : null,
-		'post_date_gmt' => isset( $sale_date ) ? $sale_date : null
+		'post_date'     => isset($sale_date) ? $sale_date : null,
+		'post_date_gmt' => isset($sale_date) ? $sale_date : null
 	);
 
 	$log_meta = array(
@@ -309,7 +309,7 @@ 
 		'price_id'   => (int) $price_id
 	);
 
-	$give_logs->insert_log( $log_data, $log_meta );
+	$give_logs->insert_log($log_data, $log_meta);
 }
 
 
@@ -323,11 +323,11 @@ 
  *
  * @return bool|int
  */
-function give_increase_purchase_count( $form_id = 0, $quantity = 1 ) {
+function give_increase_purchase_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->increase_sales( $quantity );
+	return $form->increase_sales($quantity);
 }
 
 /**
@@ -340,11 +340,11 @@ 
  *
  * @return bool|int
  */
-function give_decrease_purchase_count( $form_id = 0, $quantity = 1 ) {
+function give_decrease_purchase_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->decrease_sales( $quantity );
+	return $form->decrease_sales($quantity);
 }
 
 /**
@@ -357,10 +357,10 @@ 
  *
  * @return bool|int
  */
-function give_increase_earnings( $give_form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $give_form_id );
+function give_increase_earnings($give_form_id = 0, $amount) {
+	$form = new Give_Donate_Form($give_form_id);
 
-	return $form->increase_earnings( $amount );
+	return $form->increase_earnings($amount);
 }
 
 /**
@@ -373,10 +373,10 @@ 
  *
  * @return bool|int
  */
-function give_decrease_earnings( $form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $form_id );
+function give_decrease_earnings($form_id = 0, $amount) {
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->decrease_earnings( $amount );
+	return $form->decrease_earnings($amount);
 }
 
 
@@ -389,8 +389,8 @@ 
  *
  * @return int $earnings Earnings for a certain form
  */
-function give_get_form_earnings_stats( $form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $form_id );
+function give_get_form_earnings_stats($form_id = 0) {
+	$give_form = new Give_Donate_Form($form_id);
 
 	return $give_form->earnings;
 }
@@ -405,8 +405,8 @@ 
  *
  * @return int $sales Amount of sales for a certain form
  */
-function give_get_form_sales_stats( $give_form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $give_form_id );
+function give_get_form_sales_stats($give_form_id = 0) {
+	$give_form = new Give_Donate_Form($give_form_id);
 
 	return $give_form->sales;
 }
@@ -421,16 +421,16 @@ 
  *
  * @return float $sales Average monthly sales
  */
-function give_get_average_monthly_form_sales( $form_id = 0 ) {
-	$sales        = give_get_form_sales_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_sales($form_id = 0) {
+	$sales        = give_get_form_sales_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$sales = ( $sales / $months );
+	if ($months > 0) {
+		$sales = ($sales / $months);
 	}
 
 	return $sales;
@@ -446,16 +446,16 @@ 
  *
  * @return float $earnings Average monthly earnings
  */
-function give_get_average_monthly_form_earnings( $form_id = 0 ) {
-	$earnings     = give_get_form_earnings_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_earnings($form_id = 0) {
+	$earnings     = give_get_form_earnings_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$earnings = ( $earnings / $months );
+	if ($months > 0) {
+		$earnings = ($earnings / $months);
 	}
 
 	return $earnings < 0 ? 0 : $earnings;
@@ -475,25 +475,25 @@ 
  *
  * @return string $price_name Name of the price option
  */
-function give_get_price_option_name( $form_id = 0, $price_id = 0, $payment_id = 0 ) {
+function give_get_price_option_name($form_id = 0, $price_id = 0, $payment_id = 0) {
 
-	$prices     = give_get_variable_prices( $form_id );
+	$prices     = give_get_variable_prices($form_id);
 	$price_name = '';
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( intval( $price['_give_id']['level_id'] ) == intval( $price_id ) ) {
+		if (intval($price['_give_id']['level_id']) == intval($price_id)) {
 
-			$price_text     = isset( $price['_give_text'] ) ? $price['_give_text'] : '';
-			$price_fallback = give_currency_filter( give_format_amount( $price['_give_amount'] ) );
-			$price_name     = ! empty( $price_text ) ? $price_text : $price_fallback;
+			$price_text     = isset($price['_give_text']) ? $price['_give_text'] : '';
+			$price_fallback = give_currency_filter(give_format_amount($price['_give_amount']));
+			$price_name     = ! empty($price_text) ? $price_text : $price_fallback;
 
 		}
 
 	}
 
 
-	return apply_filters( 'give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id );
+	return apply_filters('give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id);
 }
 
 
@@ -506,14 +506,14 @@ 
  *
  * @return string $range A fully formatted price range
  */
-function give_price_range( $form_id = 0 ) {
-	$low   = give_get_lowest_price_option( $form_id );
-	$high  = give_get_highest_price_option( $form_id );
-	$range = '<span class="give_price_range_low">' . give_currency_filter( give_format_amount( $low ) ) . '</span>';
+function give_price_range($form_id = 0) {
+	$low   = give_get_lowest_price_option($form_id);
+	$high  = give_get_highest_price_option($form_id);
+	$range = '<span class="give_price_range_low">'.give_currency_filter(give_format_amount($low)).'</span>';
 	$range .= '<span class="give_price_range_sep">&nbsp;&ndash;&nbsp;</span>';
-	$range .= '<span class="give_price_range_high">' . give_currency_filter( give_format_amount( $high ) ) . '</span>';
+	$range .= '<span class="give_price_range_high">'.give_currency_filter(give_format_amount($high)).'</span>';
 
-	return apply_filters( 'give_price_range', $range, $form_id, $low, $high );
+	return apply_filters('give_price_range', $range, $form_id, $low, $high);
 }
 
 
@@ -528,36 +528,36 @@ 
  *
  * @return int ID of the lowest price
  */
-function give_get_lowest_price_id( $form_id = 0 ) {
+function give_get_lowest_price_id($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$low    = 0.00;
 	$min_id = 1;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, $price['_give_amount'] );
+				$min = min($min, $price['_give_amount']);
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $price['_give_id']['level_id'];
 			}
 		}
@@ -575,43 +575,43 @@ 
  *
  * @return float Amount of the lowest price
  */
-function give_get_lowest_price_option( $form_id = 0 ) {
-	if ( empty( $form_id ) ) {
+function give_get_lowest_price_option($form_id = 0) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$low = 0.00;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, give_sanitize_amount( $price['_give_amount'] ) );
+				$min = min($min, give_sanitize_amount($price['_give_amount']));
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $key;
 			}
 		}
 
-		$low = $prices[ $min_id ]['_give_amount'];
+		$low = $prices[$min_id]['_give_amount'];
 
 	}
 
-	return give_sanitize_amount( $low );
+	return give_sanitize_amount($low);
 }
 
 /**
@@ -623,41 +623,41 @@ 
  *
  * @return float Amount of the highest price
  */
-function give_get_highest_price_option( $form_id = 0 ) {
+function give_get_highest_price_option($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$high = 0.00;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
 		$max = 0;
 
-		foreach ( $prices as $key => $price ) {
-			if ( empty( $price['_give_amount'] ) ) {
+		foreach ($prices as $key => $price) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
-			$give_amount = give_sanitize_amount( $price['_give_amount'] );
+			$give_amount = give_sanitize_amount($price['_give_amount']);
 
-			$max = max( $max, $give_amount );
+			$max = max($max, $give_amount);
 
-			if ( $give_amount == $max ) {
+			if ($give_amount == $max) {
 				$max_id = $key;
 			}
 		}
 
-		$high = $prices[ $max_id ]['_give_amount'];
+		$high = $prices[$max_id]['_give_amount'];
 	}
 
-	return give_sanitize_amount( $high );
+	return give_sanitize_amount($high);
 }
 
 /**
@@ -669,15 +669,15 @@ 
  *
  * @return mixed string|int Price of the form
  */
-function give_get_form_price( $form_id = 0 ) {
+function give_get_form_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'price' );
+	return $form->__get('price');
 }
 
 /**
@@ -689,15 +689,15 @@ 
  *
  * @return mixed string|int Minimum price of the form
  */
-function give_get_form_minimum_price( $form_id = 0 ) {
+function give_get_form_minimum_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'minimum_price' );
+	return $form->__get('minimum_price');
 
 }
 
@@ -712,52 +712,52 @@ 
  *
  * @return int $formatted_price
  */
-function give_price( $form_id = 0, $echo = true, $price_id = false ) {
+function give_price($form_id = 0, $echo = true, $price_id = false) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$prices = give_get_variable_prices( $form_id );
+		$prices = give_get_variable_prices($form_id);
 
-		if ( false !== $price_id ) {
+		if (false !== $price_id) {
 
 			//loop through multi-prices to see which is default
-			foreach ( $prices as $price ) {
+			foreach ($prices as $price) {
 				//this is the default price
-				if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+				if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 					$price = (float) $price['_give_amount'];
 				};
 			}
 
 		} else {
 
-			$price = give_get_lowest_price_option( $form_id );
+			$price = give_get_lowest_price_option($form_id);
 		}
 
-		$price = give_sanitize_amount( $price );
+		$price = give_sanitize_amount($price);
 
 	} else {
 
-		$price = give_get_form_price( $form_id );
+		$price = give_get_form_price($form_id);
 
 	}
 
-	$price           = apply_filters( 'give_form_price', give_sanitize_amount( $price ), $form_id );
-	$formatted_price = '<span class="give_price" id="give_price_' . $form_id . '">' . $price . '</span>';
-	$formatted_price = apply_filters( 'give_form_price_after_html', $formatted_price, $form_id, $price );
+	$price           = apply_filters('give_form_price', give_sanitize_amount($price), $form_id);
+	$formatted_price = '<span class="give_price" id="give_price_'.$form_id.'">'.$price.'</span>';
+	$formatted_price = apply_filters('give_form_price_after_html', $formatted_price, $form_id, $price);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_price;
 	} else {
 		return $formatted_price;
 	}
 }
 
-add_filter( 'give_form_price', 'give_format_amount', 10 );
-add_filter( 'give_form_price', 'give_currency_filter', 20 );
+add_filter('give_form_price', 'give_format_amount', 10);
+add_filter('give_form_price', 'give_currency_filter', 20);
 
 
 /**
@@ -771,18 +771,18 @@ 
  *
  * @return float $amount Amount of the price option
  */
-function give_get_price_option_amount( $form_id = 0, $price_id = 0 ) {
-	$prices = give_get_variable_prices( $form_id );
+function give_get_price_option_amount($form_id = 0, $price_id = 0) {
+	$prices = give_get_variable_prices($form_id);
 
 	$amount = 0.00;
 
-	foreach ( $prices as $price ) {
-		if ( isset( $price['_give_id']['level_id'] ) && $price['_give_id']['level_id'] == $price_id ) {
-			$amount = isset( $price['_give_amount'] ) ? $price['_give_amount'] : 0.00;
+	foreach ($prices as $price) {
+		if (isset($price['_give_id']['level_id']) && $price['_give_id']['level_id'] == $price_id) {
+			$amount = isset($price['_give_amount']) ? $price['_give_amount'] : 0.00;
 		};
 	}
 
-	return apply_filters( 'give_get_price_option_amount', give_sanitize_amount( $amount ), $form_id, $price_id );
+	return apply_filters('give_get_price_option_amount', give_sanitize_amount($amount), $form_id, $price_id);
 }
 
 /**
@@ -794,13 +794,13 @@ 
  *
  * @return mixed string|int Goal of the form
  */
-function give_get_form_goal( $form_id = 0 ) {
+function give_get_form_goal($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->goal;
 
@@ -816,27 +816,27 @@ 
  *
  * @return string $formatted_goal
  */
-function give_goal( $form_id = 0, $echo = true ) {
+function give_goal($form_id = 0, $echo = true) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	$goal = give_get_form_goal( $form_id );
+	$goal = give_get_form_goal($form_id);
 
-	$goal           = apply_filters( 'give_form_goal', give_sanitize_amount( $goal ), $form_id );
-	$formatted_goal = '<span class="give_price" id="give_price_' . $form_id . '">' . $goal . '</span>';
-	$formatted_goal = apply_filters( 'give_form_price_after_html', $formatted_goal, $form_id, $goal );
+	$goal           = apply_filters('give_form_goal', give_sanitize_amount($goal), $form_id);
+	$formatted_goal = '<span class="give_price" id="give_price_'.$form_id.'">'.$goal.'</span>';
+	$formatted_goal = apply_filters('give_form_price_after_html', $formatted_goal, $form_id, $goal);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_goal;
 	} else {
 		return $formatted_goal;
 	}
 }
 
-add_filter( 'give_form_goal', 'give_format_amount', 10 );
-add_filter( 'give_form_goal', 'give_currency_filter', 20 );
+add_filter('give_form_goal', 'give_format_amount', 10);
+add_filter('give_form_goal', 'give_currency_filter', 20);
 
 
 /**
@@ -846,13 +846,13 @@ 
  * @global $give_options
  * @return bool $ret Whether or not the logged_in_only setting is set
  */
-function give_logged_in_only( $form_id ) {
+function give_logged_in_only($form_id) {
 
-	$form_option = get_post_meta( $form_id, '_give_logged_in_only', true );
+	$form_option = get_post_meta($form_id, '_give_logged_in_only', true);
 
-	$ret = ! empty( $form_option ) ? $form_option : false;
+	$ret = ! empty($form_option) ? $form_option : false;
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret, $form_id );
+	return (bool) apply_filters('give_logged_in_only', $ret, $form_id);
 
 }
 
@@ -864,10 +864,10 @@ 
  *
  * @param $form_id
  */
-function give_show_login_register_option( $form_id ) {
+function give_show_login_register_option($form_id) {
 
-	$show_register_form = get_post_meta( $form_id, '_give_show_register_form', true );
+	$show_register_form = get_post_meta($form_id, '_give_show_register_form', true);
 
-	return apply_filters( 'give_show_register_form', $show_register_form, $form_id );
+	return apply_filters('give_show_register_form', $show_register_form, $form_id);
 
 }
\ No newline at end of file

includes/admin/plugins.php

Spacing +16 added lines, -16 removed lines

@@ -11,7 +11,7 @@ 
 
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,16 +26,16 @@ 
  *
  * @return array $links
  */
-function give_plugin_action_links( $links, $file ) {
-	$settings_link = '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-settings' ) . '">' . esc_html__( 'Settings', 'give' ) . '</a>';
-	if ( $file == 'give/give.php' ) {
-		array_unshift( $links, $settings_link );
+function give_plugin_action_links($links, $file) {
+	$settings_link = '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-settings').'">'.esc_html__('Settings', 'give').'</a>';
+	if ($file == 'give/give.php') {
+		array_unshift($links, $settings_link);
 	}
 
 	return $links;
 }
 
-add_filter( 'plugin_action_links', 'give_plugin_action_links', 10, 2 );
+add_filter('plugin_action_links', 'give_plugin_action_links', 10, 2);
 
 
 /**
@@ -48,33 +48,33 @@ 
  *
  * @return array $input
  */
-function give_plugin_row_meta( $input, $file ) {
-	if ( $file != 'give/give.php' ) {
+function give_plugin_row_meta($input, $file) {
+	if ($file != 'give/give.php') {
 		return $input;
 	}
 
-	$give_addons_link = esc_url( add_query_arg( array(
+	$give_addons_link = esc_url(add_query_arg(array(
 			'utm_source'   => 'plugins-page',
 			'utm_medium'   => 'plugin-row',
 			'utm_campaign' => 'admin',
-		), 'https://givewp.com/addons/' )
+		), 'https://givewp.com/addons/')
 	);
 
-	$give_docs_link = esc_url( add_query_arg( array(
+	$give_docs_link = esc_url(add_query_arg(array(
 			'utm_source'   => 'plugins-page',
 			'utm_medium'   => 'plugin-row',
 			'utm_campaign' => 'admin',
-		), 'https://givewp.com/documentation/' )
+		), 'https://givewp.com/documentation/')
 	);
 
 	$links = array(
-		'<a href="' . $give_docs_link . '" target="_blank">' . esc_html__( 'Documentation', 'give' ) . '</a>',
-		'<a href="' . $give_addons_link . '" target="_blank">' . esc_html__( 'Add-ons', 'give' ) . '</a>',
+		'<a href="'.$give_docs_link.'" target="_blank">'.esc_html__('Documentation', 'give').'</a>',
+		'<a href="'.$give_addons_link.'" target="_blank">'.esc_html__('Add-ons', 'give').'</a>',
 	);
 
-	$input = array_merge( $input, $links );
+	$input = array_merge($input, $links);
 
 	return $input;
 }
 
-add_filter( 'plugin_row_meta', 'give_plugin_row_meta', 10, 2 );
+add_filter('plugin_row_meta', 'give_plugin_row_meta', 10, 2);

includes/plugin-compatibility.php

Spacing +2 added lines, -2 removed lines

@@ -19,6 +19,6 @@
  * @return void
  */
 function give_disable_mandrill_nl2br() {
-	add_filter( 'mandrill_nl2br', '__return_false' );
+	add_filter('mandrill_nl2br', '__return_false');
 }
-add_action( 'give_email_send_before', 'give_disable_mandrill_nl2br');
+add_action('give_email_send_before', 'give_disable_mandrill_nl2br');

includes/class-give-session.php

Doc Comments +4 added lines, -5 removed lines

@@ -165,10 +165,9 @@ 
 	 *
 	 * @since 1.0
 	 *
-	 * @param $key $_SESSION key
-	 * @param $value $_SESSION variable
 	 *
-	 * @return mixed Session variable
+	 * @param string $key
+	 * @return string Session variable
 	 */
 	public function set( $key, $value ) {
 
@@ -220,12 +219,12 @@ 
 	/**
 	 * Starts a new session if one hasn't started yet.
 	 *
-	 * @return null
+	 * @return boolean
 	 * Checks to see if the server supports PHP sessions or if the GIVE_USE_PHP_SESSIONS constant is defined
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return bool $ret True if we are using PHP sessions, false otherwise
+	 * @return boolean $ret True if we are using PHP sessions, false otherwise
 	 */
 	public function use_php_sessions() {
 

Spacing +61 added lines, -61 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -70,53 +70,53 @@ 
 	public function __construct() {
 
 		$this->use_php_sessions = $this->use_php_sessions();
-		$this->exp_option       = give_get_option( 'session_lifetime' );
+		$this->exp_option       = give_get_option('session_lifetime');
 
 		//PHP Sessions
-		if ( $this->use_php_sessions ) {
+		if ($this->use_php_sessions) {
 
-			if ( is_multisite() ) {
+			if (is_multisite()) {
 
-				$this->prefix = '_' . get_current_blog_id();
+				$this->prefix = '_'.get_current_blog_id();
 
 			}
 
-			add_action( 'init', array( $this, 'maybe_start_session' ), - 2 );
+			add_action('init', array($this, 'maybe_start_session'), - 2);
 
 		} else {
 
-			if ( ! $this->should_start_session() ) {
+			if ( ! $this->should_start_session()) {
 				return;
 			}
 
 			// Use WP_Session
-			if ( ! defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'give_wp_session' );
+			if ( ! defined('WP_SESSION_COOKIE')) {
+				define('WP_SESSION_COOKIE', 'give_wp_session');
 			}
 
-			if ( ! class_exists( 'Recursive_ArrayAccess' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-recursive-arrayaccess.php';
+			if ( ! class_exists('Recursive_ArrayAccess')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-recursive-arrayaccess.php';
 			}
 
-			if ( ! class_exists( 'WP_Session' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/wp-session.php';
+			if ( ! class_exists('WP_Session')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session.php';
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/wp-session.php';
 			}
 
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+			add_filter('wp_session_expiration_variant', array($this, 'set_expiration_variant_time'), 99999);
+			add_filter('wp_session_expiration', array($this, 'set_expiration_time'), 99999);
 
 		}
 
 		//Init Session
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), - 1 );
+		if (empty($this->session) && ! $this->use_php_sessions) {
+			add_action('plugins_loaded', array($this, 'init'), - 1);
 		} else {
-			add_action( 'init', array( $this, 'init' ), - 1 );
+			add_action('init', array($this, 'init'), - 1);
 		}
 
 		//Set cookie on Donation Completion page
-		add_action( 'give_pre_process_purchase', array( $this, 'set_session_cookies' ) );
+		add_action('give_pre_process_purchase', array($this, 'set_session_cookies'));
 
 	}
 
@@ -129,8 +129,8 @@ 
 	 */
 	public function init() {
 
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION[ 'give' . $this->prefix ] ) && is_array( $_SESSION[ 'give' . $this->prefix ] ) ? $_SESSION[ 'give' . $this->prefix ] : array();
+		if ($this->use_php_sessions) {
+			$this->session = isset($_SESSION['give'.$this->prefix]) && is_array($_SESSION['give'.$this->prefix]) ? $_SESSION['give'.$this->prefix] : array();
 		} else {
 			$this->session = WP_Session::get_instance();
 		}
@@ -162,10 +162,10 @@ 
 	 *
 	 * @return string Session variable
 	 */
-	public function get( $key ) {
-		$key = sanitize_key( $key );
+	public function get($key) {
+		$key = sanitize_key($key);
 
-		return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : false;
+		return isset($this->session[$key]) ? maybe_unserialize($this->session[$key]) : false;
 
 	}
 
@@ -179,21 +179,21 @@ 
 	 *
 	 * @return mixed Session variable
 	 */
-	public function set( $key, $value ) {
+	public function set($key, $value) {
 
-		$key = sanitize_key( $key );
+		$key = sanitize_key($key);
 
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = serialize( $value );
+		if (is_array($value)) {
+			$this->session[$key] = serialize($value);
 		} else {
-			$this->session[ $key ] = $value;
+			$this->session[$key] = $value;
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION[ 'give' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['give'.$this->prefix] = $this->session;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -206,10 +206,10 @@ 
 	 * @since 1.4
 	 */
 	public function set_session_cookies() {
-		if( ! headers_sent() ) {
-			$lifetime = current_time( 'timestamp' ) + $this->set_expiration_time();
-			@setcookie( session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false  );
-			@setcookie( session_name() . '_expiration', $lifetime, $lifetime,  COOKIEPATH, COOKIE_DOMAIN, false  );
+		if ( ! headers_sent()) {
+			$lifetime = current_time('timestamp') + $this->set_expiration_time();
+			@setcookie(session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
+			@setcookie(session_name().'_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -225,7 +225,7 @@ 
 	 */
 	public function set_expiration_variant_time() {
 
-		return ( ! empty( $this->exp_option ) ? ( intval( $this->exp_option ) - 3600 ) : 30 * 60 * 23 );
+		return ( ! empty($this->exp_option) ? (intval($this->exp_option) - 3600) : 30 * 60 * 23);
 	}
 
 	/**
@@ -240,7 +240,7 @@ 
 	 */
 	public function set_expiration_time() {
 
-		return ( ! empty( $this->exp_option ) ? intval( $this->exp_option ) : 30 * 60 * 24 );
+		return ( ! empty($this->exp_option) ? intval($this->exp_option) : 30 * 60 * 24);
 	}
 
 	/**
@@ -258,21 +258,21 @@ 
 		$ret = false;
 
 		// If the database variable is already set, no need to run autodetection
-		$give_use_php_sessions = (bool) get_option( 'give_use_php_sessions' );
+		$give_use_php_sessions = (bool) get_option('give_use_php_sessions');
 
-		if ( ! $give_use_php_sessions ) {
+		if ( ! $give_use_php_sessions) {
 
 			// Attempt to detect if the server supports PHP sessions
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
+			if (function_exists('session_start') && ! ini_get('safe_mode')) {
 
-				$this->set( 'give_use_php_sessions', 1 );
+				$this->set('give_use_php_sessions', 1);
 
-				if ( $this->get( 'give_use_php_sessions' ) ) {
+				if ($this->get('give_use_php_sessions')) {
 
 					$ret = true;
 
 					// Set the database option
-					update_option( 'give_use_php_sessions', true );
+					update_option('give_use_php_sessions', true);
 
 				}
 
@@ -284,13 +284,13 @@ 
 		}
 
 		// Enable or disable PHP Sessions based on the GIVE_USE_PHP_SESSIONS constant
-		if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && GIVE_USE_PHP_SESSIONS ) {
+		if (defined('GIVE_USE_PHP_SESSIONS') && GIVE_USE_PHP_SESSIONS) {
 			$ret = true;
-		} else if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && ! GIVE_USE_PHP_SESSIONS ) {
+		} else if (defined('GIVE_USE_PHP_SESSIONS') && ! GIVE_USE_PHP_SESSIONS) {
 			$ret = false;
 		}
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -303,9 +303,9 @@ 
 
 		$start_session = true;
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {
+		if ( ! empty($_SERVER['REQUEST_URI'])) {
 
-			$blacklist = apply_filters( 'give_session_start_uri_blacklist', array(
+			$blacklist = apply_filters('give_session_start_uri_blacklist', array(
 				'feed',
 				'feed',
 				'feed/rss',
@@ -313,21 +313,21 @@ 
 				'feed/rdf',
 				'feed/atom',
 				'comments/feed/'
-			) );
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' );
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist ) ) {
+			));
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/');
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist)) {
 				$start_session = false;
 			}
-			if ( false !== strpos( $uri, 'feed=' ) ) {
+			if (false !== strpos($uri, 'feed=')) {
 				$start_session = false;
 			}
-			if ( is_admin() ) {
+			if (is_admin()) {
 				$start_session = false;
 			}
 		}
 
-		return apply_filters( 'give_start_session', $start_session );
+		return apply_filters('give_start_session', $start_session);
 	}
 
 	/**
@@ -338,11 +338,11 @@ 
 	 */
 	public function maybe_start_session() {
 
-		if ( ! $this->should_start_session() ) {
+		if ( ! $this->should_start_session()) {
 			return;
 		}
 
-		if ( ! session_id() && ! headers_sent() ) {
+		if ( ! session_id() && ! headers_sent()) {
 			session_start();
 		}
 
@@ -358,9 +358,9 @@ 
 
 		$expiration = false;
 
-		if ( session_id() && isset( $_COOKIE[ session_name() . '_expiration' ] ) ) {
+		if (session_id() && isset($_COOKIE[session_name().'_expiration'])) {
 
-			$expiration = date( 'D, d M Y h:i:s', intval( $_COOKIE[ session_name() . '_expiration' ] ) );
+			$expiration = date('D, d M Y h:i:s', intval($_COOKIE[session_name().'_expiration']));
 
 		}