WordImpress / Give

includes/forms/functions.php

Spacing +213 added lines, -213 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 = give_get_meta( $args['form_id'], '_give_form_floating_labels', true );
+	if (empty($float_labels)) {
+		$float_labels = give_get_meta($args['form_id'], '_give_form_floating_labels', true);
 	}
 
-	if ( empty( $float_labels ) || ( 'global' === $float_labels ) ) {
-		$float_labels = give_get_option( 'floatlabels', 'disabled' );
+	if (empty($float_labels) || ('global' === $float_labels)) {
+		$float_labels = give_get_option('floatlabels', 'disabled');
 	}
 
-	return give_is_setting_enabled( $float_labels );
+	return give_is_setting_enabled($float_labels);
 }
 
 /**
@@ -91,7 +91,7 @@ 
 
 	$can_checkout = true;
 
-	return (bool) apply_filters( 'give_can_checkout', $can_checkout );
+	return (bool) apply_filters('give_can_checkout', $can_checkout);
 }
 
 /**
@@ -105,9 +105,9 @@ 
 function give_get_success_page_uri() {
 	$give_options = give_get_settings();
 
-	$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);
 }
 
 /**
@@ -119,9 +119,9 @@ 
  */
 function give_is_success_page() {
 	$give_options    = give_get_settings();
-	$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);
 }
 
 /**
@@ -135,17 +135,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();
 }
 
@@ -161,19 +161,19 @@ 
  * @since  1.0
  * @return Void
  */
-function give_send_back_to_checkout( $args = array() ) {
+function give_send_back_to_checkout($args = array()) {
 
-	$url     = isset( $_POST['give-current-url'] ) ? sanitize_text_field( $_POST['give-current-url'] ) : '';
+	$url     = isset($_POST['give-current-url']) ? sanitize_text_field($_POST['give-current-url']) : '';
 	$form_id = 0;
 
 	// Set the form_id.
-	if ( isset( $_POST['give-form-id'] ) ) {
-		$form_id = sanitize_text_field( $_POST['give-form-id'] );
+	if (isset($_POST['give-form-id'])) {
+		$form_id = sanitize_text_field($_POST['give-form-id']);
 	}
 
 	// Need a URL to continue. If none, redirect back to single form.
-	if ( empty( $url ) ) {
-		wp_safe_redirect( get_permalink( $form_id ) );
+	if (empty($url)) {
+		wp_safe_redirect(get_permalink($form_id));
 		give_die();
 	}
 
@@ -182,41 +182,41 @@ 
 	);
 
 	// Set the $level_id.
-	if ( isset( $_POST['give-price-id'] ) ) {
-		$defaults['level-id'] = sanitize_text_field( $_POST['give-price-id'] );
+	if (isset($_POST['give-price-id'])) {
+		$defaults['level-id'] = sanitize_text_field($_POST['give-price-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);
 
 	// Merge URL query with $args to maintain third-party URL parameters after redirect.
-	$url_data = wp_parse_url( $url );
+	$url_data = wp_parse_url($url);
 
 	// Check if an array to prevent notices before parsing.
-	if ( isset( $url_data['query'] ) && ! empty( $url_data['query'] ) ) {
-		parse_str( $url_data['query'], $query );
+	if (isset($url_data['query']) && ! empty($url_data['query'])) {
+		parse_str($url_data['query'], $query);
 
 		// Precaution: don't allow any CC info.
-		unset( $query['card_number'] );
-		unset( $query['card_cvc'] );
+		unset($query['card_number']);
+		unset($query['card_cvc']);
 
 	} else {
 		// No $url_data so pass empty array.
 		$query = array();
 	}
 
-	$new_query        = array_merge( $args, $query );
-	$new_query_string = http_build_query( $new_query );
+	$new_query        = array_merge($args, $query);
+	$new_query_string = http_build_query($new_query);
 
 	// Assemble URL parts.
-	$redirect = home_url( '/' . $url_data['path'] . '?' . $new_query_string . '#give-form-' . $form_id . '-wrap' );
+	$redirect = home_url('/'.$url_data['path'].'?'.$new_query_string.'#give-form-'.$form_id.'-wrap');
 
 	// Redirect them.
-	wp_safe_redirect( apply_filters( 'give_send_back_to_checkout', $redirect, $args ) );
+	wp_safe_redirect(apply_filters('give_send_back_to_checkout', $redirect, $args));
 	give_die();
 
 }
@@ -232,16 +232,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);
 
 }
 
@@ -254,15 +254,15 @@ 
  *
  * @return mixed|void Full URL to the Failed Donation 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) {
 	$give_options = give_get_settings();
 
-	$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);
 }
 
 /**
@@ -273,9 +273,9 @@ 
  */
 function give_is_failed_transaction_page() {
 	$give_options = give_get_settings();
-	$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);
 }
 
 /**
@@ -287,18 +287,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');
 
 /**
  * Retrieve the Donation History page URI
@@ -311,9 +311,9 @@ 
 function give_get_history_page_uri() {
 	$give_options = give_get_settings();
 
-	$history_page = isset( $give_options['history_page'] ) ? get_permalink( absint( $give_options['history_page'] ) ) : get_bloginfo( 'url' );
+	$history_page = isset($give_options['history_page']) ? get_permalink(absint($give_options['history_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_history_page_uri', $history_page );
+	return apply_filters('give_get_history_page_uri', $history_page);
 }
 
 /**
@@ -326,11 +326,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_get_required_fields( $form_id );
+	$required_fields = give_get_required_fields($form_id);
 
-	return array_key_exists( $field, $required_fields );
+	return array_key_exists($field, $required_fields);
 }
 
 /**
@@ -348,14 +348,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(
@@ -363,7 +363,7 @@ 
 		'price_id'   => (int) $price_id,
 	);
 
-	$give_logs->insert_log( $log_data, $log_meta );
+	$give_logs->insert_log($log_data, $log_meta);
 }
 
 
@@ -377,11 +377,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);
 }
 
 /**
@@ -394,11 +394,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);
 }
 
 /**
@@ -411,10 +411,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);
 }
 
 /**
@@ -427,10 +427,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);
 }
 
 
@@ -443,8 +443,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;
 }
@@ -459,8 +459,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;
 }
@@ -475,16 +475,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;
@@ -500,16 +500,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;
@@ -529,23 +529,23 @@ 
  *
  * @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);
 }
 
 
@@ -558,14 +558,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);
 }
 
 
@@ -580,35 +580,35 @@ 
  *
  * @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);
 
 	$min = $min_id = 0;
 
-	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'];
 			}
 		}
@@ -626,45 +626,45 @@ 
  *
  * @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;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
 		$min = $min_id = 0;
 
-		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);
 }
 
 /**
@@ -676,41 +676,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_id = $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);
 }
 
 /**
@@ -722,15 +722,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');
 }
 
 /**
@@ -742,15 +742,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');
 
 }
 
@@ -765,51 +765,51 @@ 
  *
  * @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);
 
 
 /**
@@ -822,19 +822,19 @@ 
  *
  * @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;
 			break;
 		};
 	}
 
-	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);
 }
 
 /**
@@ -846,13 +846,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;
 
@@ -868,27 +868,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);
 
 
 /**
@@ -900,15 +900,15 @@ 
  *
  * @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) {
 	// If _give_logged_in_only is set to enable then guest can donate from that specific form.
 	// Otherwise it is member only donation form.
-	$val = give_get_meta( $form_id, '_give_logged_in_only', true );
-	$val = ! empty( $val ) ? $val : 'enabled';
+	$val = give_get_meta($form_id, '_give_logged_in_only', true);
+	$val = ! empty($val) ? $val : 'enabled';
 
-	$ret = ! give_is_setting_enabled( $val );
+	$ret = ! give_is_setting_enabled($val);
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret, $form_id );
+	return (bool) apply_filters('give_logged_in_only', $ret, $form_id);
 }
 
 
@@ -921,11 +921,11 @@ 
  *
  * @return string
  */
-function give_show_login_register_option( $form_id ) {
+function give_show_login_register_option($form_id) {
 
-	$show_register_form = give_get_meta( $form_id, '_give_show_register_form', true );
+	$show_register_form = give_get_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);
 
 }
 
@@ -941,12 +941,12 @@ 
  *
  * @return array
  */
-function _give_get_prefill_form_field_values( $form_id ) {
+function _give_get_prefill_form_field_values($form_id) {
 	$logged_in_donor_info = array();
 
-	if ( is_user_logged_in() ) :
-		$donor_data    = get_userdata( get_current_user_id() );
-		$donor_address = get_user_meta( get_current_user_id(), '_give_user_address', true );
+	if (is_user_logged_in()) :
+		$donor_data    = get_userdata(get_current_user_id());
+		$donor_address = get_user_meta(get_current_user_id(), '_give_user_address', true);
 
 		$logged_in_donor_info = array(
 			// First name.
@@ -959,42 +959,42 @@ 
 			'give_email'      => $donor_data->user_email,
 
 			// Street address 1.
-			'card_address'    => ( ! empty( $donor_address['line1'] ) ? $donor_address['line1'] : '' ),
+			'card_address'    => ( ! empty($donor_address['line1']) ? $donor_address['line1'] : ''),
 
 			// Street address 2.
-			'card_address_2'  => ( ! empty( $donor_address['line2'] ) ? $donor_address['line2'] : '' ),
+			'card_address_2'  => ( ! empty($donor_address['line2']) ? $donor_address['line2'] : ''),
 
 			// Country.
-			'billing_country' => ( ! empty( $donor_address['country'] ) ? $donor_address['country'] : '' ),
+			'billing_country' => ( ! empty($donor_address['country']) ? $donor_address['country'] : ''),
 
 			// State.
-			'card_state'      => ( ! empty( $donor_address['state'] ) ? $donor_address['state'] : '' ),
+			'card_state'      => ( ! empty($donor_address['state']) ? $donor_address['state'] : ''),
 
 			// City.
-			'card_city'       => ( ! empty( $donor_address['city'] ) ? $donor_address['city'] : '' ),
+			'card_city'       => ( ! empty($donor_address['city']) ? $donor_address['city'] : ''),
 
 			// Zipcode
-			'card_zip'        => ( ! empty( $donor_address['zip'] ) ? $donor_address['zip'] : '' ),
+			'card_zip'        => ( ! empty($donor_address['zip']) ? $donor_address['zip'] : ''),
 		);
 	endif;
 
 	// Bailout: Auto fill form field values only form form which donor is donating.
 	if (
-		empty( $_GET['form-id'] )
+		empty($_GET['form-id'])
 		|| ! $form_id
-		|| ( $form_id !== absint( $_GET['form-id'] ) )
+		|| ($form_id !== absint($_GET['form-id']))
 	) {
 		return $logged_in_donor_info;
 	}
 
 	// Get purchase data.
-	$give_purchase_data = Give()->session->get( 'give_purchase' );
+	$give_purchase_data = Give()->session->get('give_purchase');
 
 	// Get donor info from form data.
-	$give_donor_info_in_session = empty( $give_purchase_data['post_data'] )
+	$give_donor_info_in_session = empty($give_purchase_data['post_data'])
 		? array()
 		: $give_purchase_data['post_data'];
 
 	// Output.
-	return wp_parse_args( $give_donor_info_in_session, $logged_in_donor_info );
+	return wp_parse_args($give_donor_info_in_session, $logged_in_donor_info);
 }

includes/admin/reporting/reports.php

Spacing +61 added lines, -61 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,8 +29,8 @@ 
  * @return void
  */
 function give_reports_page() {
-	$current_page = admin_url( 'edit.php?post_type=give_forms&page=give-reports' );
-	$active_tab   = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
+	$current_page = admin_url('edit.php?post_type=give_forms&page=give-reports');
+	$active_tab   = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
 	$views        = give_reports_default_views();
 	?>
 	<div class="wrap give-settings-page">
@@ -38,17 +38,17 @@ 
 		<h1 class="screen-reader-text"><?php echo get_admin_page_title(); ?></h1>
 
 		<h2 class="nav-tab-wrapper">
-			<?php foreach ( $views as $tab => $label ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php foreach ($views as $tab => $label) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => $tab,
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php echo esc_html( $label ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php echo esc_html($label); ?></a>
 			<?php } ?>
-			<?php if ( current_user_can( 'export_give_reports' ) ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php if (current_user_can('export_give_reports')) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => 'export',
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php esc_html_e( 'Export', 'give' ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php esc_html_e('Export', 'give'); ?></a>
 			<?php }
 			/**
 			 * Fires in the report tabs.
@@ -57,7 +57,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_reports_tabs' );
+			do_action('give_reports_tabs');
 			?>
 		</h2>
 
@@ -67,10 +67,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_top' );
+		do_action('give_reports_page_top');
 
 		// Set $active_tab prior to hook firing.
-		if ( in_array( $active_tab, array_keys( $views ) ) ) {
+		if (in_array($active_tab, array_keys($views))) {
 			$active_tab = 'reports';
 		}
 
@@ -79,14 +79,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_reports_tab_{$active_tab}" );
+		do_action("give_reports_tab_{$active_tab}");
 
 		/**
 		 * Fires after the report page.
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_bottom' );
+		do_action('give_reports_page_bottom');
 		?>
 	</div><!-- .wrap -->
 	<?php
@@ -100,13 +100,13 @@ 
  */
 function give_reports_default_views() {
 	$views = array(
-		'earnings' => esc_html__( 'Income', 'give' ),
-		'forms'    => esc_html__( 'Forms', 'give' ),
-		'donors'   => esc_html__( 'Donors', 'give' ),
-		'gateways' => esc_html__( 'Donation Methods', 'give' ),
+		'earnings' => esc_html__('Income', 'give'),
+		'forms'    => esc_html__('Forms', 'give'),
+		'donors'   => esc_html__('Donors', 'give'),
+		'gateways' => esc_html__('Donation Methods', 'give'),
 	);
 
-	$views = apply_filters( 'give_report_views', $views );
+	$views = apply_filters('give_report_views', $views);
 
 	return $views;
 }
@@ -121,15 +121,15 @@ 
  * @since 1.0
  * @return string $view Report View
  */
-function give_get_reporting_view( $default = 'earnings' ) {
+function give_get_reporting_view($default = 'earnings') {
 
-	if ( ! isset( $_GET['view'] ) || ! in_array( $_GET['view'], array_keys( give_reports_default_views() ) ) ) {
+	if ( ! isset($_GET['view']) || ! in_array($_GET['view'], array_keys(give_reports_default_views()))) {
 		$view = $default;
 	} else {
 		$view = $_GET['view'];
 	}
 
-	return apply_filters( 'give_get_reporting_view', $view );
+	return apply_filters('give_get_reporting_view', $view);
 }
 
 /**
@@ -142,7 +142,7 @@ 
 	$current_view = 'earnings';
 	$views        = give_reports_default_views();
 
-	if ( isset( $_GET['tab'] ) && array_key_exists( $_GET['tab'], $views ) ) {
+	if (isset($_GET['tab']) && array_key_exists($_GET['tab'], $views)) {
 		$current_view = $_GET['tab'];
 	}
 
@@ -151,10 +151,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_reports_view_{$current_view}" );
+	do_action("give_reports_view_{$current_view}");
 }
 
-add_action( 'give_reports_tab_reports', 'give_reports_tab_reports' );
+add_action('give_reports_tab_reports', 'give_reports_tab_reports');
 
 /**
  * Renders the Reports Page Views Drop Downs
@@ -164,19 +164,19 @@ 
  */
 function give_report_views() {
 	$views        = give_reports_default_views();
-	$current_view = isset( $_GET['view'] ) ? $_GET['view'] : 'earnings';
+	$current_view = isset($_GET['view']) ? $_GET['view'] : 'earnings';
 	/**
 	 * Fires before the report page actions form.
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_before' );
+	do_action('give_report_view_actions_before');
 	?>
 	<form id="give-reports-filter" method="get">
 		<select id="give-reports-view" name="view">
-			<option value="-1"><?php esc_html_e( 'Report Type', 'give' ); ?></option>
-			<?php foreach ( $views as $view_id => $label ) : ?>
-				<option value="<?php echo esc_attr( $view_id ); ?>" <?php selected( $view_id, $current_view ); ?>><?php echo $label; ?></option>
+			<option value="-1"><?php esc_html_e('Report Type', 'give'); ?></option>
+			<?php foreach ($views as $view_id => $label) : ?>
+				<option value="<?php echo esc_attr($view_id); ?>" <?php selected($view_id, $current_view); ?>><?php echo $label; ?></option>
 			<?php endforeach; ?>
 		</select>
 
@@ -188,12 +188,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_report_view_actions' );
+		do_action('give_report_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-reports"/>
-		<?php submit_button( esc_html__( 'Show', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Show', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 	/**
@@ -201,7 +201,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_after' );
+	do_action('give_report_view_actions_after');
 }
 
 /**
@@ -214,18 +214,18 @@ 
  */
 function give_reports_forms_table() {
 
-	if ( isset( $_GET['form-id'] ) ) {
+	if (isset($_GET['form-id'])) {
 		return;
 	}
 
-	include( dirname( __FILE__ ) . '/class-form-reports-table.php' );
+	include(dirname(__FILE__).'/class-form-reports-table.php');
 
 	$give_table = new Give_Form_Reports_Table();
 	$give_table->prepare_items();
 	$give_table->display();
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_forms_table' );
+add_action('give_reports_view_forms', 'give_reports_forms_table');
 
 /**
  * Renders the detailed report for a specific give form
@@ -234,20 +234,20 @@ 
  * @return void
  */
 function give_reports_form_details() {
-	if ( ! isset( $_GET['form-id'] ) ) {
+	if ( ! isset($_GET['form-id'])) {
 		return;
 	}
 	?>
 	<div class="tablenav top reports-forms-details-wrap">
 		<div class="actions bulkactions">
-			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e( 'Go Back', 'give' ); ?></button>
+			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e('Go Back', 'give'); ?></button>
 		</div>
 	</div>
 	<?php
-	give_reports_graph_of_form( absint( $_GET['form-id'] ) );
+	give_reports_graph_of_form(absint($_GET['form-id']));
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_form_details' );
+add_action('give_reports_view_forms', 'give_reports_form_details');
 
 /**
  * Renders the Reports Donors Table
@@ -258,7 +258,7 @@ 
  * @return void
  */
 function give_reports_donors_table() {
-	include( dirname( __FILE__ ) . '/class-donor-reports-table.php' );
+	include(dirname(__FILE__).'/class-donor-reports-table.php');
 
 	$give_table = new Give_Donor_Reports_Table();
 	$give_table->prepare_items();
@@ -270,11 +270,11 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_donors_table_top' );
+		do_action('give_logs_donors_table_top');
 		?>
 		<form id="give-donors-filter" method="get">
 			<?php
-			$give_table->search_box( esc_html__( 'Search', 'give' ), 'give-donors' );
+			$give_table->search_box(esc_html__('Search', 'give'), 'give-donors');
 			$give_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms"/>
@@ -287,13 +287,13 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_donors_table_bottom' );
+		do_action('give_logs_donors_table_bottom');
 		?>
 	</div>
 	<?php
 }
 
-add_action( 'give_reports_view_donors', 'give_reports_donors_table' );
+add_action('give_reports_view_donors', 'give_reports_donors_table');
 
 
 /**
@@ -305,14 +305,14 @@ 
  * @return void
  */
 function give_reports_gateways_table() {
-	include( dirname( __FILE__ ) . '/class-gateways-reports-table.php' );
+	include(dirname(__FILE__).'/class-gateways-reports-table.php');
 
 	$give_table = new Give_Gateway_Reports_Table();
 	$give_table->prepare_items();
 	$give_table->display();
 }
 
-add_action( 'give_reports_view_gateways', 'give_reports_gateways_table' );
+add_action('give_reports_view_gateways', 'give_reports_gateways_table');
 
 
 /**
@@ -324,13 +324,13 @@ 
 function give_reports_earnings() {
 	?>
 	<div class="tablenav top reports-table-nav">
-		<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e( 'Income Report', 'give' ); ?></span></h3>
+		<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e('Income Report', 'give'); ?></span></h3>
 	</div>
 	<?php
 	give_reports_graph();
 }
 
-add_action( 'give_reports_view_earnings', 'give_reports_earnings' );
+add_action('give_reports_view_earnings', 'give_reports_earnings');
 
 
 /**
@@ -341,9 +341,9 @@ 
  */
 function give_estimated_monthly_stats() {
 
-	$estimated = Give_Cache::get( 'give_estimated_monthly_stats', true );
+	$estimated = Give_Cache::get('give_estimated_monthly_stats', true);
 
-	if ( false === $estimated ) {
+	if (false === $estimated) {
 
 		$estimated = array(
 			'earnings' => 0,
@@ -352,22 +352,22 @@ 
 
 		$stats = new Give_Payment_Stats;
 
-		$to_date_earnings = $stats->get_earnings( 0, 'this_month' );
-		$to_date_sales    = $stats->get_sales( 0, 'this_month' );
+		$to_date_earnings = $stats->get_earnings(0, 'this_month');
+		$to_date_sales    = $stats->get_sales(0, 'this_month');
 
-		$current_day   = date( 'd', current_time( 'timestamp' ) );
-		$current_month = date( 'n', current_time( 'timestamp' ) );
-		$current_year  = date( 'Y', current_time( 'timestamp' ) );
-		$days_in_month = cal_days_in_month( CAL_GREGORIAN, $current_month, $current_year );
+		$current_day   = date('d', current_time('timestamp'));
+		$current_month = date('n', current_time('timestamp'));
+		$current_year  = date('Y', current_time('timestamp'));
+		$days_in_month = cal_days_in_month(CAL_GREGORIAN, $current_month, $current_year);
 
-		$estimated['earnings'] = ( $to_date_earnings / $current_day ) * $days_in_month;
-		$estimated['sales']    = ( $to_date_sales / $current_day ) * $days_in_month;
+		$estimated['earnings'] = ($to_date_earnings / $current_day) * $days_in_month;
+		$estimated['sales']    = ($to_date_sales / $current_day) * $days_in_month;
 
 		// Cache for one day
-		Give_Cache::set( 'give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true );
+		Give_Cache::set('give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true);
 	}
 
-	return maybe_unserialize( $estimated );
+	return maybe_unserialize($estimated);
 }
 
 // @TODO: After release 1.8 Donations -> Reports generates with new setting api, so we can remove some old code from this file.

includes/admin/reporting/graphing.php

Spacing +246 added lines, -246 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
 	$dates = give_get_report_dates();
 
 	// Determine graph options
-	switch ( $dates['range'] ) :
+	switch ($dates['range']) :
 		case 'today' :
 		case 'yesterday' :
 			$day_by_day = true;
@@ -37,7 +37,7 @@ 
 			$day_by_day = false;
 			break;
 		case 'other' :
-			if ( $dates['m_end'] - $dates['m_start'] >= 2 || $dates['year_end'] > $dates['year'] && ( $dates['m_start'] != '12' && $dates['m_end'] != '1' ) ) {
+			if ($dates['m_end'] - $dates['m_start'] >= 2 || $dates['year_end'] > $dates['year'] && ($dates['m_start'] != '12' && $dates['m_end'] != '1')) {
 				$day_by_day = false;
 			} else {
 				$day_by_day = true;
@@ -49,61 +49,61 @@ 
 	endswitch;
 
 	$earnings_totals = 0.00; // Total earnings for time period shown
-	$sales_totals    = 0;            // Total sales for time period shown
+	$sales_totals    = 0; // Total sales for time period shown
 
 	$earnings_data = array();
 	$sales_data    = array();
 
-	if ( $dates['range'] == 'today' || $dates['range'] == 'yesterday' ) {
+	if ($dates['range'] == 'today' || $dates['range'] == 'yesterday') {
 		// Hour by hour
 		$hour  = 1;
-		$month = date( 'n', current_time( 'timestamp' ) );
-		while ( $hour <= 23 ) :
+		$month = date('n', current_time('timestamp'));
+		while ($hour <= 23) :
 
-			$sales    = give_get_sales_by_date( $dates['day'], $month, $dates['year'], $hour );
-			$earnings = give_get_earnings_by_date( $dates['day'], $month, $dates['year'], $hour );
+			$sales    = give_get_sales_by_date($dates['day'], $month, $dates['year'], $hour);
+			$earnings = give_get_earnings_by_date($dates['day'], $month, $dates['year'], $hour);
 
 			$sales_totals += $sales;
 			$earnings_totals += $earnings;
 
-			$date            = mktime( $hour, 0, 0, $month, $dates['day'], $dates['year'] ) * 1000;
-			$sales_data[]    = array( $date, $sales );
-			$earnings_data[] = array( $date, $earnings );
+			$date            = mktime($hour, 0, 0, $month, $dates['day'], $dates['year']) * 1000;
+			$sales_data[]    = array($date, $sales);
+			$earnings_data[] = array($date, $earnings);
 
-			$hour ++;
+			$hour++;
 		endwhile;
 
-	} elseif ( $dates['range'] == 'this_week' || $dates['range'] == 'last_week' ) {
+	} elseif ($dates['range'] == 'this_week' || $dates['range'] == 'last_week') {
 
 		// Day by day
 		$day     = $dates['day'];
 		$day_end = $dates['day_end'];
 		$month   = $dates['m_start'];
-		while ( $day <= $day_end ) :
-			$sales = give_get_sales_by_date( $day, $month, $dates['year'] );
+		while ($day <= $day_end) :
+			$sales = give_get_sales_by_date($day, $month, $dates['year']);
 			$sales_totals += $sales;
 
-			$earnings = give_get_earnings_by_date( $day, $month, $dates['year'] );
+			$earnings = give_get_earnings_by_date($day, $month, $dates['year']);
 			$earnings_totals += $earnings;
 
-			$date            = mktime( 0, 0, 0, $month, $day, $dates['year'] ) * 1000;
-			$sales_data[]    = array( $date, $sales );
-			$earnings_data[] = array( $date, $earnings );
-			$day ++;
+			$date            = mktime(0, 0, 0, $month, $day, $dates['year']) * 1000;
+			$sales_data[]    = array($date, $sales);
+			$earnings_data[] = array($date, $earnings);
+			$day++;
 		endwhile;
 
 	} else {
 
 		$y = $dates['year'];
-		while ( $y <= $dates['year_end'] ) :
+		while ($y <= $dates['year_end']) :
 
-			if ( $dates['year'] == $dates['year_end'] ) {
+			if ($dates['year'] == $dates['year_end']) {
 				$month_start = $dates['m_start'];
 				$month_end   = $dates['m_end'];
-			} elseif ( $y == $dates['year'] ) {
+			} elseif ($y == $dates['year']) {
 				$month_start = $dates['m_start'];
 				$month_end   = 12;
-			} elseif ( $y == $dates['year_end'] ) {
+			} elseif ($y == $dates['year_end']) {
 				$month_start = 1;
 				$month_end   = $dates['m_end'];
 			} else {
@@ -112,48 +112,48 @@ 
 			}
 
 			$i = $month_start;
-			while ( $i <= $month_end ) :
+			while ($i <= $month_end) :
 
-				if ( $day_by_day ) {
+				if ($day_by_day) {
 
-					if ( $i == $month_end ) {
+					if ($i == $month_end) {
 
 						$num_of_days = $dates['day_end'];
 
 					} else {
 
-						$num_of_days = cal_days_in_month( CAL_GREGORIAN, $i, $y );
+						$num_of_days = cal_days_in_month(CAL_GREGORIAN, $i, $y);
 
 					}
 
 					$d = $dates['day'];
 
-					while ( $d <= $num_of_days ) :
+					while ($d <= $num_of_days) :
 
-						$sales = give_get_sales_by_date( $d, $i, $y );
+						$sales = give_get_sales_by_date($d, $i, $y);
 						$sales_totals += $sales;
 
-						$earnings = give_get_earnings_by_date( $d, $i, $y );
+						$earnings = give_get_earnings_by_date($d, $i, $y);
 						$earnings_totals += $earnings;
 
-						$date            = mktime( 0, 0, 0, $i, $d, $y ) * 1000;
-						$sales_data[]    = array( $date, $sales );
-						$earnings_data[] = array( $date, $earnings );
-						$d ++;
+						$date            = mktime(0, 0, 0, $i, $d, $y) * 1000;
+						$sales_data[]    = array($date, $sales);
+						$earnings_data[] = array($date, $earnings);
+						$d++;
 
 					endwhile;
 
 				} else {
 
-					$sales = give_get_sales_by_date( null, $i, $y );
+					$sales = give_get_sales_by_date(null, $i, $y);
 					$sales_totals += $sales;
 
-					$earnings = give_get_earnings_by_date( null, $i, $y );
+					$earnings = give_get_earnings_by_date(null, $i, $y);
 					$earnings_totals += $earnings;
 
-					if ( $i == $month_end ) {
+					if ($i == $month_end) {
 
-						$num_of_days = cal_days_in_month( CAL_GREGORIAN, $i, $y );
+						$num_of_days = cal_days_in_month(CAL_GREGORIAN, $i, $y);
 
 					} else {
 
@@ -161,24 +161,24 @@ 
 
 					}
 
-					$date            = mktime( 0, 0, 0, $i, $num_of_days, $y ) * 1000;
-					$sales_data[]    = array( $date, $sales );
-					$earnings_data[] = array( $date, $earnings );
+					$date            = mktime(0, 0, 0, $i, $num_of_days, $y) * 1000;
+					$sales_data[]    = array($date, $sales);
+					$earnings_data[] = array($date, $earnings);
 
 				}
 
-				$i ++;
+				$i++;
 
 			endwhile;
 
-			$y ++;
+			$y++;
 		endwhile;
 
 	}
 
 	$data = array(
-		esc_html__( 'Income', 'give' )    => $earnings_data,
-		esc_html__( 'Donations', 'give' ) => $sales_data
+		esc_html__('Income', 'give')    => $earnings_data,
+		esc_html__('Donations', 'give') => $sales_data
 	);
 
 	// start our own output buffer
@@ -191,12 +191,12 @@ 
 				<div class="inside">
 					<?php give_reports_graph_controls(); ?>
 					<?php
-					$graph = new Give_Graph( $data, array( 'dataType' => array( 'amount', 'count' ) ) );
-					$graph->set( 'x_mode', 'time' );
-					$graph->set( 'multiple_y_axes', true );
+					$graph = new Give_Graph($data, array('dataType' => array('amount', 'count')));
+					$graph->set('x_mode', 'time');
+					$graph->set('multiple_y_axes', true);
 					$graph->display();
 
-					if ( 'this_month' == $dates['range'] ) {
+					if ('this_month' == $dates['range']) {
 						$estimated = give_estimated_monthly_stats();
 					}
 					?>
@@ -205,21 +205,21 @@ 
 			<table class="widefat reports-table alignleft" style="max-width:450px">
 				<tbody>
 				<tr>
-					<th scope="row"><strong><?php esc_html_e( 'Total income for period:', 'give' ); ?></strong></th>
-					<td><?php echo give_currency_filter( give_format_amount( $earnings_totals ) ); ?></td>
+					<th scope="row"><strong><?php esc_html_e('Total income for period:', 'give'); ?></strong></th>
+					<td><?php echo give_currency_filter(give_format_amount($earnings_totals)); ?></td>
 				</tr>
 				<tr class="alternate">
-					<th scope="row"><strong><?php esc_html_e( 'Total donations for period:', 'give' ); ?><strong></th>
+					<th scope="row"><strong><?php esc_html_e('Total donations for period:', 'give'); ?><strong></th>
 					<td><?php echo $sales_totals; ?></td>
 				</tr>
-				<?php if ( 'this_month' == $dates['range'] ) : ?>
+				<?php if ('this_month' == $dates['range']) : ?>
 					<tr>
-						<th scope="row"><strong><?php esc_html_e( 'Estimated monthly income:', 'give' ); ?></strong></th>
-						<td><?php echo give_currency_filter( give_format_amount( $estimated['earnings'] ) ); ?></td>
+						<th scope="row"><strong><?php esc_html_e('Estimated monthly income:', 'give'); ?></strong></th>
+						<td><?php echo give_currency_filter(give_format_amount($estimated['earnings'])); ?></td>
 					</tr>
 					<tr class="alternate">
-						<th scope="row"><strong><?php esc_html_e( 'Estimated monthly donations:', 'give' ); ?></strong></th>
-						<td><?php echo floor( $estimated['sales'] ); ?></td>
+						<th scope="row"><strong><?php esc_html_e('Estimated monthly donations:', 'give'); ?></strong></th>
+						<td><?php echo floor($estimated['sales']); ?></td>
 					</tr>
 				<?php endif; ?>
 			</table>
@@ -232,7 +232,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_reports_graph_additional_stats' );
+			do_action('give_reports_graph_additional_stats');
 			?>
 
 		</div>
@@ -251,12 +251,12 @@ 
  * @since 1.0
  * @return void
  */
-function give_reports_graph_of_form( $form_id = 0 ) {
+function give_reports_graph_of_form($form_id = 0) {
 	// Retrieve the queried dates
 	$dates = give_get_report_dates();
 
 	// Determine graph options
-	switch ( $dates['range'] ) :
+	switch ($dates['range']) :
 		case 'today' :
 		case 'yesterday' :
 			$day_by_day = true;
@@ -274,7 +274,7 @@ 
 			$day_by_day = false;
 			break;
 		case 'other' :
-			if ( $dates['m_end'] - $dates['m_start'] >= 2 || $dates['year_end'] > $dates['year'] ) {
+			if ($dates['m_end'] - $dates['m_start'] >= 2 || $dates['year_end'] > $dates['year']) {
 				$day_by_day = false;
 			} else {
 				$day_by_day = true;
@@ -286,75 +286,75 @@ 
 	endswitch;
 
 	$earnings_totals = (float) 0.00; // Total earnings for time period shown
-	$sales_totals    = 0;            // Total sales for time period shown
+	$sales_totals    = 0; // Total sales for time period shown
 
 	$earnings_data = array();
 	$sales_data    = array();
 	$stats         = new Give_Payment_Stats;
 
-	if ( $dates['range'] == 'today' || $dates['range'] == 'yesterday' ) {
+	if ($dates['range'] == 'today' || $dates['range'] == 'yesterday') {
 
 		// Hour by hour
 		$month  = $dates['m_start'];
 		$hour   = 1;
 		$minute = 0;
 		$second = 0;
-		while ( $hour <= 23 ) :
+		while ($hour <= 23) :
 
-			if ( $hour == 23 ) {
+			if ($hour == 23) {
 				$minute = $second = 59;
 			}
 
-			$date     = mktime( $hour, $minute, $second, $month, $dates['day'], $dates['year'] );
-			$date_end = mktime( $hour + 1, $minute, $second, $month, $dates['day'], $dates['year'] );
+			$date     = mktime($hour, $minute, $second, $month, $dates['day'], $dates['year']);
+			$date_end = mktime($hour + 1, $minute, $second, $month, $dates['day'], $dates['year']);
 
-			$sales = $stats->get_sales( $form_id, $date, $date_end );
+			$sales = $stats->get_sales($form_id, $date, $date_end);
 			$sales_totals += $sales;
 
-			$earnings = $stats->get_earnings( $form_id, $date, $date_end );
+			$earnings = $stats->get_earnings($form_id, $date, $date_end);
 			$earnings_totals += $earnings;
 
-			$sales_data[]    = array( $date * 1000, $sales );
-			$earnings_data[] = array( $date * 1000, $earnings );
+			$sales_data[]    = array($date * 1000, $sales);
+			$earnings_data[] = array($date * 1000, $earnings);
 
-			$hour ++;
+			$hour++;
 		endwhile;
 
-	} elseif ( $dates['range'] == 'this_week' || $dates['range'] == 'last_week' ) {
+	} elseif ($dates['range'] == 'this_week' || $dates['range'] == 'last_week') {
 
 		//Day by day
 		$day     = $dates['day'];
 		$day_end = $dates['day_end'];
 		$month   = $dates['m_start'];
-		while ( $day <= $day_end ) :
+		while ($day <= $day_end) :
 
-			$date     = mktime( 0, 0, 0, $month, $day, $dates['year'] );
-			$date_end = mktime( 0, 0, 0, $month, $day + 1, $dates['year'] );
-			$sales    = $stats->get_sales( $form_id, $date, $date_end );
+			$date     = mktime(0, 0, 0, $month, $day, $dates['year']);
+			$date_end = mktime(0, 0, 0, $month, $day + 1, $dates['year']);
+			$sales    = $stats->get_sales($form_id, $date, $date_end);
 			$sales_totals += $sales;
 
-			$earnings = $stats->get_earnings( $form_id, $date, $date_end );
+			$earnings = $stats->get_earnings($form_id, $date, $date_end);
 			$earnings_totals += $earnings;
 
-			$sales_data[]    = array( $date * 1000, $sales );
-			$earnings_data[] = array( $date * 1000, $earnings );
+			$sales_data[]    = array($date * 1000, $sales);
+			$earnings_data[] = array($date * 1000, $earnings);
 
-			$day ++;
+			$day++;
 		endwhile;
 
 	} else {
 
 		$y = $dates['year'];
 
-		while ( $y <= $dates['year_end'] ) :
+		while ($y <= $dates['year_end']) :
 
 			$last_year = false;
 
-			if ( $dates['year'] == $dates['year_end'] ) {
+			if ($dates['year'] == $dates['year_end']) {
 				$month_start = $dates['m_start'];
 				$month_end   = $dates['m_end'];
 				$last_year   = true;
-			} elseif ( $y == $dates['year'] ) {
+			} elseif ($y == $dates['year']) {
 				$month_start = $dates['m_start'];
 				$month_end   = 12;
 			} else {
@@ -363,76 +363,76 @@ 
 			}
 
 			$i = $month_start;
-			while ( $i <= $month_end ) :
+			while ($i <= $month_end) :
 
-				if ( $day_by_day ) {
+				if ($day_by_day) {
 
-					if ( $i == $month_end && $last_year ) {
+					if ($i == $month_end && $last_year) {
 
 						$num_of_days = $dates['day_end'];
 
 					} else {
 
-						$num_of_days = cal_days_in_month( CAL_GREGORIAN, $i, $y );
+						$num_of_days = cal_days_in_month(CAL_GREGORIAN, $i, $y);
 
 					}
 
 					$d = $dates['day'];
-					while ( $d <= $num_of_days ) :
+					while ($d <= $num_of_days) :
 
-						$date     = mktime( 0, 0, 0, $i, $d, $y );
-						$end_date = mktime( 23, 59, 59, $i, $d, $y );
+						$date     = mktime(0, 0, 0, $i, $d, $y);
+						$end_date = mktime(23, 59, 59, $i, $d, $y);
 
-						$sales = $stats->get_sales( $form_id, $date, $end_date );
+						$sales = $stats->get_sales($form_id, $date, $end_date);
 						$sales_totals += $sales;
 
-						$earnings = $stats->get_earnings( $form_id, $date, $end_date );
+						$earnings = $stats->get_earnings($form_id, $date, $end_date);
 						$earnings_totals += $earnings;
 
-						$sales_data[]    = array( $date * 1000, $sales );
-						$earnings_data[] = array( $date * 1000, $earnings );
-						$d ++;
+						$sales_data[]    = array($date * 1000, $sales);
+						$earnings_data[] = array($date * 1000, $earnings);
+						$d++;
 
 					endwhile;
 
 				} else {
 
-					$num_of_days = cal_days_in_month( CAL_GREGORIAN, $i, $y );
+					$num_of_days = cal_days_in_month(CAL_GREGORIAN, $i, $y);
 
-					$date     = mktime( 0, 0, 0, $i, 1, $y );
-					$end_date = mktime( 23, 59, 59, $i, $num_of_days, $y );
+					$date     = mktime(0, 0, 0, $i, 1, $y);
+					$end_date = mktime(23, 59, 59, $i, $num_of_days, $y);
 
-					$sales = $stats->get_sales( $form_id, $date, $end_date );
+					$sales = $stats->get_sales($form_id, $date, $end_date);
 					$sales_totals += $sales;
 
-					$earnings = $stats->get_earnings( $form_id, $date, $end_date );
+					$earnings = $stats->get_earnings($form_id, $date, $end_date);
 					$earnings_totals += $earnings;
 
-					$sales_data[]    = array( $date * 1000, $sales );
-					$earnings_data[] = array( $date * 1000, $earnings );
+					$sales_data[]    = array($date * 1000, $sales);
+					$earnings_data[] = array($date * 1000, $earnings);
 
 				}
 
-				$i ++;
+				$i++;
 
 			endwhile;
 
-			$y ++;
+			$y++;
 		endwhile;
 
 	}
 
 	$data = array(
-		esc_html__( 'Income', 'give' )    => $earnings_data,
-		esc_html__( 'Donations', 'give' ) => $sales_data
+		esc_html__('Income', 'give')    => $earnings_data,
+		esc_html__('Donations', 'give') => $sales_data
 	);
 
 	?>
 	<h3><span><?php
 		printf(
 			/* translators: %s: form title */
-			esc_html__( 'Income Report for %s', 'give' ),
-			get_the_title( $form_id )
+			esc_html__('Income Report for %s', 'give'),
+			get_the_title($form_id)
 		);
 	?></span></h3>
 	<div id="give-dashboard-widgets-wrap">
@@ -441,9 +441,9 @@ 
 				<div class="inside">
 					<?php give_reports_graph_controls(); ?>
 					<?php
-					$graph = new Give_Graph( $data, array( 'dataType' => array( 'amount', 'count' ) ) );
-					$graph->set( 'x_mode', 'time' );
-					$graph->set( 'multiple_y_axes', true );
+					$graph = new Give_Graph($data, array('dataType' => array('amount', 'count')));
+					$graph->set('x_mode', 'time');
+					$graph->set('multiple_y_axes', true);
 					$graph->display();
 					?>
 				</div>
@@ -452,20 +452,20 @@ 
 			<table class="widefat reports-table alignleft" style="max-width:450px">
 				<tbody>
 				<tr>
-					<th scope="row"><strong><?php esc_html_e( 'Total income for period:', 'give' ); ?></strong></th>
-					<td><?php echo give_currency_filter( give_format_amount( $earnings_totals ) ); ?></td>
+					<th scope="row"><strong><?php esc_html_e('Total income for period:', 'give'); ?></strong></th>
+					<td><?php echo give_currency_filter(give_format_amount($earnings_totals)); ?></td>
 				</tr>
 				<tr class="alternate">
-					<th scope="row"><strong><?php esc_html_e( 'Total donations for period:', 'give' ); ?></strong></th>
+					<th scope="row"><strong><?php esc_html_e('Total donations for period:', 'give'); ?></strong></th>
 					<td><?php echo $sales_totals; ?></td>
 				</tr>
 				<tr>
-					<th scope="row"><strong><?php esc_html_e( 'Average monthly income:', 'give' ); ?></strong></th>
-					<td><?php echo give_currency_filter( give_format_amount( give_get_average_monthly_form_earnings( $form_id ) ) ); ?></td>
+					<th scope="row"><strong><?php esc_html_e('Average monthly income:', 'give'); ?></strong></th>
+					<td><?php echo give_currency_filter(give_format_amount(give_get_average_monthly_form_earnings($form_id))); ?></td>
 				</tr>
 				<tr class="alternate">
-					<th scope="row"><strong><?php esc_html_e( 'Average monthly donations:', 'give' ); ?></strong></th>
-					<td><?php echo number_format( give_get_average_monthly_form_sales( $form_id ), 0 ); ?></td>
+					<th scope="row"><strong><?php esc_html_e('Average monthly donations:', 'give'); ?></strong></th>
+					<td><?php echo number_format(give_get_average_monthly_form_sales($form_id), 0); ?></td>
 				</tr>
 				</tbody>
 			</table>
@@ -484,26 +484,26 @@ 
  * @return void
  */
 function give_reports_graph_controls() {
-	$date_options = apply_filters( 'give_report_date_options', array(
-		'today'        => esc_html__( 'Today', 'give' ),
-		'yesterday'    => esc_html__( 'Yesterday', 'give' ),
-		'this_week'    => esc_html__( 'This Week', 'give' ),
-		'last_week'    => esc_html__( 'Last Week', 'give' ),
-		'this_month'   => esc_html__( 'This Month', 'give' ),
-		'last_month'   => esc_html__( 'Last Month', 'give' ),
-		'this_quarter' => esc_html__( 'This Quarter', 'give' ),
-		'last_quarter' => esc_html__( 'Last Quarter', 'give' ),
-		'this_year'    => esc_html__( 'This Year', 'give' ),
-		'last_year'    => esc_html__( 'Last Year', 'give' ),
-		'other'        => esc_html__( 'Custom', 'give' )
-	) );
+	$date_options = apply_filters('give_report_date_options', array(
+		'today'        => esc_html__('Today', 'give'),
+		'yesterday'    => esc_html__('Yesterday', 'give'),
+		'this_week'    => esc_html__('This Week', 'give'),
+		'last_week'    => esc_html__('Last Week', 'give'),
+		'this_month'   => esc_html__('This Month', 'give'),
+		'last_month'   => esc_html__('Last Month', 'give'),
+		'this_quarter' => esc_html__('This Quarter', 'give'),
+		'last_quarter' => esc_html__('Last Quarter', 'give'),
+		'this_year'    => esc_html__('This Year', 'give'),
+		'last_year'    => esc_html__('Last Year', 'give'),
+		'other'        => esc_html__('Custom', 'give')
+	));
 
 	$dates   = give_get_report_dates();
 	$display = $dates['range'] == 'other' ? '' : 'display: none;';
-	$tab     = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
+	$tab     = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
 
-	if ( empty( $dates['day_end'] ) ) {
-		$dates['day_end'] = cal_days_in_month( CAL_GREGORIAN, date( 'n' ), date( 'Y' ) );
+	if (empty($dates['day_end'])) {
+		$dates['day_end'] = cal_days_in_month(CAL_GREGORIAN, date('n'), date('Y'));
 	}
 
 	/**
@@ -511,7 +511,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_graph_controls_before' );
+	do_action('give_report_graph_controls_before');
 	?>
 	<form id="give-graphs-filter" method="get">
 		<div class="tablenav top">
@@ -519,56 +519,56 @@ 
 
 				<input type="hidden" name="post_type" value="give_forms" />
 				<input type="hidden" name="page" value="give-reports" />
-				<input type="hidden" name="tab" value="<?php echo esc_attr( $tab ); ?>" />
+				<input type="hidden" name="tab" value="<?php echo esc_attr($tab); ?>" />
 
-				<?php if ( isset( $_GET['form-id'] ) ) : ?>
-					<input type="hidden" name="form-id" value="<?php echo absint( $_GET['form-id'] ); ?>" />
+				<?php if (isset($_GET['form-id'])) : ?>
+					<input type="hidden" name="form-id" value="<?php echo absint($_GET['form-id']); ?>" />
 				<?php endif; ?>
 
 				<div id="give-graphs-date-options-wrap">
 					<select id="give-graphs-date-options" name="range">
-						<?php foreach ( $date_options as $key => $option ) : ?>
-							<option value="<?php echo esc_attr( $key ); ?>"<?php selected( $key, $dates['range'] ); ?>><?php echo esc_html( $option ); ?></option>
+						<?php foreach ($date_options as $key => $option) : ?>
+							<option value="<?php echo esc_attr($key); ?>"<?php selected($key, $dates['range']); ?>><?php echo esc_html($option); ?></option>
 						<?php endforeach; ?>
 					</select>
 
-					<div id="give-date-range-options" style="<?php echo esc_attr( $display ); ?>">
-						<span class="screen-reader-text"><?php esc_html_e( 'From', 'give' ); ?>&nbsp;</span>
+					<div id="give-date-range-options" style="<?php echo esc_attr($display); ?>">
+						<span class="screen-reader-text"><?php esc_html_e('From', 'give'); ?>&nbsp;</span>
 						<select id="give-graphs-month-start" name="m_start" aria-label="Start Month">
-							<?php for ( $i = 1; $i <= 12; $i ++ ) : ?>
-								<option value="<?php echo absint( $i ); ?>" <?php echo esc_attr( selected( $i, $dates['m_start'] ) ); ?>><?php echo esc_html( give_month_num_to_name( $i ) ); ?></option>
+							<?php for ($i = 1; $i <= 12; $i++) : ?>
+								<option value="<?php echo absint($i); ?>" <?php echo esc_attr(selected($i, $dates['m_start'])); ?>><?php echo esc_html(give_month_num_to_name($i)); ?></option>
 							<?php endfor; ?>
 						</select>
 						<select id="give-graphs-day-start" name="day" aria-label="Start Day">
-							<?php for ( $i = 1; $i <= 31; $i ++ ) : ?>
-								<option value="<?php echo absint( $i ); ?>" <?php echo esc_attr( selected( $i, $dates['day'] ) ); ?>><?php echo esc_html( $i ); ?></option>
+							<?php for ($i = 1; $i <= 31; $i++) : ?>
+								<option value="<?php echo absint($i); ?>" <?php echo esc_attr(selected($i, $dates['day'])); ?>><?php echo esc_html($i); ?></option>
 							<?php endfor; ?>
 						</select>
 						<select id="give-graphs-year-start" name="year" aria-label="Start Year">
-							<?php for ( $i = 2007; $i <= date( 'Y' ); $i ++ ) : ?>
-								<option value="<?php echo absint( $i ); ?>" <?php echo esc_attr( selected( $i, $dates['year'] ) ); ?>><?php echo esc_html( $i ); ?></option>
+							<?php for ($i = 2007; $i <= date('Y'); $i++) : ?>
+								<option value="<?php echo absint($i); ?>" <?php echo esc_attr(selected($i, $dates['year'])); ?>><?php echo esc_html($i); ?></option>
 							<?php endfor; ?>
 						</select>
-						<span class="screen-reader-text"><?php esc_html_e( 'To', 'give' ); ?>&nbsp;</span>
+						<span class="screen-reader-text"><?php esc_html_e('To', 'give'); ?>&nbsp;</span>
 						<span>&ndash;</span>
 						<select id="give-graphs-month-end" name="m_end" aria-label="End Month">
-							<?php for ( $i = 1; $i <= 12; $i ++ ) : ?>
-								<option value="<?php echo absint( $i ); ?>" <?php echo esc_attr( selected( $i, $dates['m_end'] ) ); ?>><?php echo esc_html( give_month_num_to_name( $i ) ); ?></option>
+							<?php for ($i = 1; $i <= 12; $i++) : ?>
+								<option value="<?php echo absint($i); ?>" <?php echo esc_attr(selected($i, $dates['m_end'])); ?>><?php echo esc_html(give_month_num_to_name($i)); ?></option>
 							<?php endfor; ?>
 						</select>
 						<select id="give-graphs-day-end" name="day_end" aria-label="End Day">
-							<?php for ( $i = 1; $i <= 31; $i ++ ) : ?>
-								<option value="<?php echo absint( $i ); ?>" <?php echo esc_attr( selected( $i, $dates['day_end'] ) ); ?>><?php echo esc_html( $i ); ?></option>
+							<?php for ($i = 1; $i <= 31; $i++) : ?>
+								<option value="<?php echo absint($i); ?>" <?php echo esc_attr(selected($i, $dates['day_end'])); ?>><?php echo esc_html($i); ?></option>
 							<?php endfor; ?>
 						</select>
 						<select id="give-graphs-year-end" name="year_end" aria-label="End Year">
-							<?php for ( $i = 2007; $i <= date( 'Y' ); $i ++ ) : ?>
-								<option value="<?php echo absint( $i ); ?>" <?php echo esc_attr( selected( $i, $dates['year_end'] ) ); ?>><?php echo esc_html( $i ); ?></option>
+							<?php for ($i = 2007; $i <= date('Y'); $i++) : ?>
+								<option value="<?php echo absint($i); ?>" <?php echo esc_attr(selected($i, $dates['year_end'])); ?>><?php echo esc_html($i); ?></option>
 							<?php endfor; ?>
 						</select>
 					</div>
 
-					<input type="submit" class="button-secondary" value="<?php esc_attr_e( 'Filter', 'give' ); ?>" />
+					<input type="submit" class="button-secondary" value="<?php esc_attr_e('Filter', 'give'); ?>" />
 				</div>
 
 				<input type="hidden" name="give_action" value="filter_reports" />
@@ -581,7 +581,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_graph_controls_after' );
+	do_action('give_report_graph_controls_after');
 }
 
 /**
@@ -596,65 +596,65 @@ 
 function give_get_report_dates() {
 	$dates = array();
 
-	$current_time = current_time( 'timestamp' );
+	$current_time = current_time('timestamp');
 
-	$dates['range']    = isset( $_GET['range'] ) ? $_GET['range'] : 'this_month';
-	$dates['year']     = isset( $_GET['year'] ) ? $_GET['year'] : date( 'Y' );
-	$dates['year_end'] = isset( $_GET['year_end'] ) ? $_GET['year_end'] : date( 'Y' );
-	$dates['m_start']  = isset( $_GET['m_start'] ) ? $_GET['m_start'] : 1;
-	$dates['m_end']    = isset( $_GET['m_end'] ) ? $_GET['m_end'] : 12;
-	$dates['day']      = isset( $_GET['day'] ) ? $_GET['day'] : 1;
-	$dates['day_end']  = isset( $_GET['day_end'] ) ? $_GET['day_end'] : cal_days_in_month( CAL_GREGORIAN, $dates['m_end'], $dates['year'] );
+	$dates['range']    = isset($_GET['range']) ? $_GET['range'] : 'this_month';
+	$dates['year']     = isset($_GET['year']) ? $_GET['year'] : date('Y');
+	$dates['year_end'] = isset($_GET['year_end']) ? $_GET['year_end'] : date('Y');
+	$dates['m_start']  = isset($_GET['m_start']) ? $_GET['m_start'] : 1;
+	$dates['m_end']    = isset($_GET['m_end']) ? $_GET['m_end'] : 12;
+	$dates['day']      = isset($_GET['day']) ? $_GET['day'] : 1;
+	$dates['day_end']  = isset($_GET['day_end']) ? $_GET['day_end'] : cal_days_in_month(CAL_GREGORIAN, $dates['m_end'], $dates['year']);
 
 	// Modify dates based on predefined ranges
-	switch ( $dates['range'] ) :
+	switch ($dates['range']) :
 
 		case 'this_month' :
-			$dates['m_start']  = date( 'n', $current_time );
-			$dates['m_end']    = date( 'n', $current_time );
+			$dates['m_start']  = date('n', $current_time);
+			$dates['m_end']    = date('n', $current_time);
 			$dates['day']      = 1;
-			$dates['day_end']  = cal_days_in_month( CAL_GREGORIAN, $dates['m_end'], $dates['year'] );
-			$dates['year']     = date( 'Y' );
-			$dates['year_end'] = date( 'Y' );
+			$dates['day_end']  = cal_days_in_month(CAL_GREGORIAN, $dates['m_end'], $dates['year']);
+			$dates['year']     = date('Y');
+			$dates['year_end'] = date('Y');
 			break;
 
 		case 'last_month' :
-			if ( date( 'n' ) == 1 ) {
+			if (date('n') == 1) {
 				$dates['m_start']  = 12;
 				$dates['m_end']    = 12;
-				$dates['year']     = date( 'Y', $current_time ) - 1;
-				$dates['year_end'] = date( 'Y', $current_time ) - 1;
+				$dates['year']     = date('Y', $current_time) - 1;
+				$dates['year_end'] = date('Y', $current_time) - 1;
 			} else {
-				$dates['m_start']  = date( 'n' ) - 1;
-				$dates['m_end']    = date( 'n' ) - 1;
+				$dates['m_start']  = date('n') - 1;
+				$dates['m_end']    = date('n') - 1;
 				$dates['year_end'] = $dates['year'];
 			}
-			$dates['day_end'] = cal_days_in_month( CAL_GREGORIAN, $dates['m_end'], $dates['year'] );
+			$dates['day_end'] = cal_days_in_month(CAL_GREGORIAN, $dates['m_end'], $dates['year']);
 			break;
 
 		case 'today' :
-			$dates['day']     = date( 'd', $current_time );
-			$dates['m_start'] = date( 'n', $current_time );
-			$dates['m_end']   = date( 'n', $current_time );
-			$dates['year']    = date( 'Y', $current_time );
+			$dates['day']     = date('d', $current_time);
+			$dates['m_start'] = date('n', $current_time);
+			$dates['m_end']   = date('n', $current_time);
+			$dates['year']    = date('Y', $current_time);
 			break;
 
 		case 'yesterday' :
 
-			$year  = date( 'Y', $current_time );
-			$month = date( 'n', $current_time );
-			$day   = date( 'd', $current_time );
+			$year  = date('Y', $current_time);
+			$month = date('n', $current_time);
+			$day   = date('d', $current_time);
 
-			if ( $month == 1 && $day == 1 ) {
+			if ($month == 1 && $day == 1) {
 
 				$year -= 1;
 				$month = 12;
-				$day   = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+				$day   = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
-			} elseif ( $month > 1 && $day == 1 ) {
+			} elseif ($month > 1 && $day == 1) {
 
 				$month -= 1;
-				$day = cal_days_in_month( CAL_GREGORIAN, $month, $year );
+				$day = cal_days_in_month(CAL_GREGORIAN, $month, $year);
 
 			} else {
 
@@ -670,91 +670,91 @@ 
 			break;
 
 		case 'this_week' :
-			$dates['day'] = date( 'd', $current_time - ( date( 'w', $current_time ) - 1 ) * 60 * 60 * 24 ) - 1;
-			$dates['day'] += get_option( 'start_of_week' );
+			$dates['day'] = date('d', $current_time - (date('w', $current_time) - 1) * 60 * 60 * 24) - 1;
+			$dates['day'] += get_option('start_of_week');
 			$dates['day_end'] = $dates['day'] + 6;
-			$dates['m_start'] = date( 'n', $current_time );
-			$dates['m_end']   = date( 'n', $current_time );
-			$dates['year']    = date( 'Y', $current_time );
+			$dates['m_start'] = date('n', $current_time);
+			$dates['m_end']   = date('n', $current_time);
+			$dates['year']    = date('Y', $current_time);
 			break;
 
 		case 'last_week' :
-			$dates['day'] = date( 'd', $current_time - ( date( 'w' ) - 1 ) * 60 * 60 * 24 ) - 8;
-			$dates['day'] += get_option( 'start_of_week' );
+			$dates['day'] = date('d', $current_time - (date('w') - 1) * 60 * 60 * 24) - 8;
+			$dates['day'] += get_option('start_of_week');
 			$dates['day_end'] = $dates['day'] + 6;
-			$dates['year']    = date( 'Y' );
-
-			if ( date( 'j', $current_time ) <= 7 ) {
-				$dates['m_start'] = date( 'n', $current_time ) - 1;
-				$dates['m_end']   = date( 'n', $current_time ) - 1;
-				if ( $dates['m_start'] <= 1 ) {
-					$dates['year']     = date( 'Y', $current_time ) - 1;
-					$dates['year_end'] = date( 'Y', $current_time ) - 1;
+			$dates['year']    = date('Y');
+
+			if (date('j', $current_time) <= 7) {
+				$dates['m_start'] = date('n', $current_time) - 1;
+				$dates['m_end']   = date('n', $current_time) - 1;
+				if ($dates['m_start'] <= 1) {
+					$dates['year']     = date('Y', $current_time) - 1;
+					$dates['year_end'] = date('Y', $current_time) - 1;
 				}
 			} else {
-				$dates['m_start'] = date( 'n', $current_time );
-				$dates['m_end']   = date( 'n', $current_time );
+				$dates['m_start'] = date('n', $current_time);
+				$dates['m_end']   = date('n', $current_time);
 			}
 			break;
 
 		case 'this_quarter' :
-			$month_now = date( 'n', $current_time );
+			$month_now = date('n', $current_time);
 
-			if ( $month_now <= 3 ) {
+			if ($month_now <= 3) {
 
 				$dates['m_start'] = 1;
 				$dates['m_end']   = 4;
-				$dates['year']    = date( 'Y', $current_time );
+				$dates['year']    = date('Y', $current_time);
 
-			} else if ( $month_now <= 6 ) {
+			} else if ($month_now <= 6) {
 
 				$dates['m_start'] = 4;
 				$dates['m_end']   = 7;
-				$dates['year']    = date( 'Y', $current_time );
+				$dates['year']    = date('Y', $current_time);
 
-			} else if ( $month_now <= 9 ) {
+			} else if ($month_now <= 9) {
 
 				$dates['m_start'] = 7;
 				$dates['m_end']   = 10;
-				$dates['year']    = date( 'Y', $current_time );
+				$dates['year']    = date('Y', $current_time);
 
 			} else {
 
 				$dates['m_start']  = 10;
 				$dates['m_end']    = 1;
-				$dates['year']     = date( 'Y', $current_time );
-				$dates['year_end'] = date( 'Y', $current_time ) + 1;
+				$dates['year']     = date('Y', $current_time);
+				$dates['year_end'] = date('Y', $current_time) + 1;
 
 			}
 			break;
 
 		case 'last_quarter' :
-			$month_now = date( 'n' );
+			$month_now = date('n');
 
-			if ( $month_now <= 3 ) {
+			if ($month_now <= 3) {
 
 				$dates['m_start']  = 10;
 				$dates['m_end']    = 12;
-				$dates['year']     = date( 'Y', $current_time ) - 1; // Previous year
-				$dates['year_end'] = date( 'Y', $current_time ) - 1; // Previous year
+				$dates['year']     = date('Y', $current_time) - 1; // Previous year
+				$dates['year_end'] = date('Y', $current_time) - 1; // Previous year
 
-			} else if ( $month_now <= 6 ) {
+			} else if ($month_now <= 6) {
 
 				$dates['m_start'] = 1;
 				$dates['m_end']   = 3;
-				$dates['year']    = date( 'Y', $current_time );
+				$dates['year']    = date('Y', $current_time);
 
-			} else if ( $month_now <= 9 ) {
+			} else if ($month_now <= 9) {
 
 				$dates['m_start'] = 4;
 				$dates['m_end']   = 6;
-				$dates['year']    = date( 'Y', $current_time );
+				$dates['year']    = date('Y', $current_time);
 
 			} else {
 
 				$dates['m_start'] = 7;
 				$dates['m_end']   = 9;
-				$dates['year']    = date( 'Y', $current_time );
+				$dates['year']    = date('Y', $current_time);
 
 			}
 			break;
@@ -762,19 +762,19 @@ 
 		case 'this_year' :
 			$dates['m_start'] = 1;
 			$dates['m_end']   = 12;
-			$dates['year']    = date( 'Y', $current_time );
+			$dates['year']    = date('Y', $current_time);
 			break;
 
 		case 'last_year' :
 			$dates['m_start']  = 1;
 			$dates['m_end']    = 12;
-			$dates['year']     = date( 'Y', $current_time ) - 1;
-			$dates['year_end'] = date( 'Y', $current_time ) - 1;
+			$dates['year']     = date('Y', $current_time) - 1;
+			$dates['year_end'] = date('Y', $current_time) - 1;
 			break;
 
 	endswitch;
 
-	return apply_filters( 'give_report_dates', $dates );
+	return apply_filters('give_report_dates', $dates);
 }
 
 /**
@@ -785,18 +785,18 @@ 
  *
  * @param $data
  */
-function give_parse_report_dates( $data ) {
+function give_parse_report_dates($data) {
 	$dates = give_get_report_dates();
 
 	$view = give_get_reporting_view();
-	$tab  = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
-	$id   = isset( $_GET['form-id'] ) ? $_GET['form-id'] : null;
+	$tab  = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
+	$id   = isset($_GET['form-id']) ? $_GET['form-id'] : null;
 
-	wp_redirect( add_query_arg( $dates, admin_url( 'edit.php?post_type=give_forms&page=give-reports&tab=' . esc_attr( $tab ) . '&view=' . esc_attr( $view ) . '&form-id=' . absint( $id ) ) ) );
+	wp_redirect(add_query_arg($dates, admin_url('edit.php?post_type=give_forms&page=give-reports&tab='.esc_attr($tab).'&view='.esc_attr($view).'&form-id='.absint($id))));
 	give_die();
 }
 
-add_action( 'give_filter_reports', 'give_parse_report_dates' );
+add_action('give_filter_reports', 'give_parse_report_dates');
 
 
 /**
@@ -808,22 +808,22 @@ 
  */
 function give_reports_refresh_button() {
 
-	$url = wp_nonce_url( add_query_arg( array(
+	$url = wp_nonce_url(add_query_arg(array(
 		'give_action'  => 'refresh_reports_transients',
 		'give-message' => 'refreshed-reports'
-	) ), 'give-refresh-reports' );
+	)), 'give-refresh-reports');
 
 	echo '<a href="'
-	     . esc_url_raw( $url )
-	     . '" data-tooltip="'. esc_attr__( 'Clicking this will clear the reports cache.', 'give' )
+	     . esc_url_raw($url)
+	     . '" data-tooltip="'.esc_attr__('Clicking this will clear the reports cache.', 'give')
 	     . '" data-tooltip-my-position="right center"  data-tooltip-target-position="left center" class="button alignright give-admin-button give-tooltip">'
 	     . '<span class="give-admin-button-icon give-admin-button-icon-update"></span>'
-	     . esc_html__( 'Refresh Report Data', 'give' )
+	     . esc_html__('Refresh Report Data', 'give')
 	     . '</a>';
 
 }
 
-add_action( 'give_reports_graph_additional_stats', 'give_reports_refresh_button' );
+add_action('give_reports_graph_additional_stats', 'give_reports_refresh_button');
 
 /**
  * Trigger the refresh of reports transients
@@ -834,20 +834,20 @@ 
  *
  * @return void
  */
-function give_run_refresh_reports_transients( $data ) {
+function give_run_refresh_reports_transients($data) {
 
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give-refresh-reports')) {
 		return;
 	}
 
 	// Monthly stats.
-	Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+	Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
 	// Total earning.
-	delete_option( 'give_earnings_total' );
+	delete_option('give_earnings_total');
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 }
 
-add_action( 'give_refresh_reports_transients', 'give_run_refresh_reports_transients' );
\ No newline at end of file
+add_action('give_refresh_reports_transients', 'give_run_refresh_reports_transients');
\ No newline at end of file

includes/admin/reporting/class-give-graph.php

Spacing +35 added lines, -35 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -81,15 +81,15 @@ 
 	 * @param array $_data
 	 * @param array $options
 	 */
-	public function __construct( $_data, $options = array() ) {
+	public function __construct($_data, $options = array()) {
 
-		$this->data      = $_data;
+		$this->data = $_data;
 
 		// Generate unique ID
-		$this->id = md5( rand() );
+		$this->id = md5(rand());
 
 		// Setup default options;
-		$this->options = apply_filters( 'give_graph_args', array(
+		$this->options = apply_filters('give_graph_args', array(
 			'y_mode'          => null,
 			'x_mode'          => null,
 			'y_decimals'      => 0,
@@ -107,9 +107,9 @@ 
 			'lines'           => false,
 			'points'          => true,
 			'dataType'        => array()
-		) );
+		));
 
-		$this->options = wp_parse_args( $options, $this->options );
+		$this->options = wp_parse_args($options, $this->options);
 	}
 
 	/**
@@ -120,8 +120,8 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function set( $key, $value ) {
-		$this->options[ $key ] = $value;
+	public function set($key, $value) {
+		$this->options[$key] = $value;
 	}
 
 	/**
@@ -131,8 +131,8 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function get( $key ) {
-		return isset( $this->options[ $key ] ) ? $this->options[ $key ] : false;
+	public function get($key) {
+		return isset($this->options[$key]) ? $this->options[$key] : false;
 	}
 
 	/**
@@ -141,7 +141,7 @@ 
 	 * @since 1.0
 	 */
 	public function get_data() {
-		return apply_filters( 'give_get_graph_data', $this->data, $this );
+		return apply_filters('give_get_graph_data', $this->data, $this);
 	}
 
 	/**
@@ -151,19 +151,19 @@ 
 	 */
 	public function load_scripts() {
 		// Use minified libraries if SCRIPT_DEBUG is turned off
-		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+		$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
-		wp_register_script( 'jquery-flot-orderbars', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.orderBars' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-orderbars' );
+		wp_register_script('jquery-flot-orderbars', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.orderBars'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-orderbars');
 
-		wp_register_script( 'jquery-flot-time', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.time' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-time' );
+		wp_register_script('jquery-flot-time', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.time'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-time');
 
-		wp_register_script( 'jquery-flot-resize', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.resize' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-resize' );
+		wp_register_script('jquery-flot-resize', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.resize'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-resize');
 
-		wp_register_script( 'jquery-flot', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot' . $suffix . '.js', false, GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot' );
+		wp_register_script('jquery-flot', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot'.$suffix.'.js', false, GIVE_VERSION);
+		wp_enqueue_script('jquery-flot');
 
 	}
 
@@ -190,14 +190,14 @@ 
 					[
 						<?php
 							$order = 0;
-							foreach( $this->get_data() as $label => $data ) :
+							foreach ($this->get_data() as $label => $data) :
 						?>
 						{
-							label : "<?php echo esc_attr( $label ); ?>",
-							id    : "<?php echo sanitize_key( $label ); ?>",
-							dataType  : '<?php echo ( ! empty( $this->options['dataType'][$order] ) ? $this->options['dataType'][$order] : 'count' ); ?>',
+							label : "<?php echo esc_attr($label); ?>",
+							id    : "<?php echo sanitize_key($label); ?>",
+							dataType  : '<?php echo ( ! empty($this->options['dataType'][$order]) ? $this->options['dataType'][$order] : 'count'); ?>',
 							// data format is: [ point on x, value on y ]
-							data  : [<?php foreach( $data as $point ) { echo '[' . implode( ',', $point ) . '],'; } ?>],
+							data  : [<?php foreach ($data as $point) { echo '['.implode(',', $point).'],'; } ?>],
 							points: {
 								show: <?php echo $this->options['points'] ? 'true' : 'false'; ?>,
 							},
@@ -212,7 +212,7 @@ 
 								fill     : true,
 								fillColor: {colors: [{opacity: 0.4}, {opacity: 0.1}]}
 							},
-							<?php if( $this->options[ 'multiple_y_axes' ] ) : ?>
+							<?php if ($this->options['multiple_y_axes']) : ?>
 							yaxis : <?php echo $yaxis_count; ?>
 							<?php endif; ?>
 
@@ -226,10 +226,10 @@ 
 						grid: {
 							show           : true,
 							aboveData      : false,
-							color          : "<?php echo $this->options[ 'color' ]; ?>",
-							backgroundColor: "<?php echo $this->options[ 'bgcolor' ]; ?>",
-							borderColor    : "<?php echo $this->options[ 'bordercolor' ]; ?>",
-							borderWidth    : <?php echo absint( $this->options[ 'borderwidth' ] ); ?>,
+							color          : "<?php echo $this->options['color']; ?>",
+							backgroundColor: "<?php echo $this->options['bgcolor']; ?>",
+							borderColor    : "<?php echo $this->options['bordercolor']; ?>",
+							borderWidth    : <?php echo absint($this->options['borderwidth']); ?>,
 							clickable      : false,
 							hoverable      : true
 						},
@@ -240,7 +240,7 @@ 
 							mode        : "<?php echo $this->options['x_mode']; ?>",
 							timeFormat  : "<?php echo $this->options['x_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
 							tickSize    : "<?php echo $this->options['x_mode'] == 'time' ? '' : 1; ?>",
-							<?php if( $this->options['x_mode'] != 'time' ) : ?>
+							<?php if ($this->options['x_mode'] != 'time') : ?>
 							tickDecimals: <?php echo $this->options['x_decimals']; ?>
 							<?php endif; ?>
 						},
@@ -249,7 +249,7 @@ 
 							min         : 0,
 							mode        : "<?php echo $this->options['y_mode']; ?>",
 							timeFormat  : "<?php echo $this->options['y_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
-							<?php if( $this->options['y_mode'] != 'time' ) : ?>
+							<?php if ($this->options['y_mode'] != 'time') : ?>
 							tickDecimals: <?php echo $this->options['y_decimals']; ?>
 							<?php endif; ?>
 						}
@@ -320,7 +320,7 @@ 
 		 *
 		 * @param Give_Graph $this Graph object.
 		 */
-		do_action( 'give_before_graph', $this );
+		do_action('give_before_graph', $this);
 
 		// Build the graph.
 		echo $this->build_graph();
@@ -332,7 +332,7 @@ 
 		 *
 		 * @param Give_Graph $this Graph object.
 		 */
-		do_action( 'give_after_graph', $this );
+		do_action('give_after_graph', $this);
 	}
 
 }

includes/admin/class-admin-settings.php

Spacing +208 added lines, -208 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Admin_Settings' ) ) :
+if ( ! class_exists('Give_Admin_Settings')) :
 
 	/**
 	 * Give_Admin_Settings Class.
@@ -72,7 +72,7 @@ 
 			 *
 			 * @param array $settings Array of settings class object.
 			 */
-			self::$settings = apply_filters( self::$setting_filter_prefix . '_get_settings_pages', array() );
+			self::$settings = apply_filters(self::$setting_filter_prefix.'_get_settings_pages', array());
 
 			return self::$settings;
 		}
@@ -86,8 +86,8 @@ 
 		public static function save() {
 			$current_tab = give_get_current_setting_tab();
 
-			if ( empty( $_REQUEST['_give-save-settings'] ) || ! wp_verify_nonce( $_REQUEST['_give-save-settings'], 'give-save-settings' ) ) {
-				echo '<div class="notice error"><p>' . __( 'Action failed. Please refresh the page and retry.', 'give' ) . '</p></div>';
+			if (empty($_REQUEST['_give-save-settings']) || ! wp_verify_nonce($_REQUEST['_give-save-settings'], 'give-save-settings')) {
+				echo '<div class="notice error"><p>'.__('Action failed. Please refresh the page and retry.', 'give').'</p></div>';
 				die();
 			}
 
@@ -100,9 +100,9 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_save_' . $current_tab );
+			do_action(self::$setting_filter_prefix.'_save_'.$current_tab);
 
-			self::add_message( 'give-setting-updated', __( 'Your settings have been saved.', 'give' ) );
+			self::add_message('give-setting-updated', __('Your settings have been saved.', 'give'));
 
 			/**
 			 * Trigger Action.
@@ -113,7 +113,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_saved' );
+			do_action(self::$setting_filter_prefix.'_saved');
 		}
 
 		/**
@@ -126,8 +126,8 @@ 
 		 *
 		 * @return void
 		 */
-		public static function add_message( $code, $message ) {
-			self::$messages[ $code ] = $message;
+		public static function add_message($code, $message) {
+			self::$messages[$code] = $message;
 		}
 
 		/**
@@ -140,8 +140,8 @@ 
 		 *
 		 * @return void
 		 */
-		public static function add_error( $code, $message ) {
-			self::$errors[ $code ] = $message;
+		public static function add_error($code, $message) {
+			self::$errors[$code] = $message;
 		}
 
 		/**
@@ -154,18 +154,18 @@ 
 			$notice_html = '';
 			$classes     = 'give-notice settings-error notice is-dismissible';
 
-			self::$errors   = apply_filters( self::$setting_filter_prefix . '_error_notices', self::$errors );
-			self::$messages = apply_filters( self::$setting_filter_prefix . '_update_notices', self::$messages );
+			self::$errors   = apply_filters(self::$setting_filter_prefix.'_error_notices', self::$errors);
+			self::$messages = apply_filters(self::$setting_filter_prefix.'_update_notices', self::$messages);
 
-			if ( 0 < count( self::$errors ) ) {
-				foreach ( self::$errors as $code => $message ) {
-					$notice_html .= '<div id="setting-error-' . $code . '" class="' . $classes . ' error"><p><strong>' . $message . '</strong></p></div>';
+			if (0 < count(self::$errors)) {
+				foreach (self::$errors as $code => $message) {
+					$notice_html .= '<div id="setting-error-'.$code.'" class="'.$classes.' error"><p><strong>'.$message.'</strong></p></div>';
 				}
 			}
 
-			if ( 0 < count( self::$messages ) ) {
-				foreach ( self::$messages as $code => $message ) {
-					$notice_html .= '<div id="setting-error-' . $code . '" class="' . $classes . ' updated"><p><strong>' . $message . '</strong></p></div>';
+			if (0 < count(self::$messages)) {
+				foreach (self::$messages as $code => $message) {
+					$notice_html .= '<div id="setting-error-'.$code.'" class="'.$classes.' updated"><p><strong>'.$message.'</strong></p></div>';
 				}
 			}
 
@@ -185,7 +185,7 @@ 
 			self::$setting_filter_prefix = give_get_current_setting_page();
 
 			// Bailout: Exit if setting page is not defined.
-			if ( empty( self::$setting_filter_prefix ) ) {
+			if (empty(self::$setting_filter_prefix)) {
 				return false;
 			}
 
@@ -198,7 +198,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_start' );
+			do_action(self::$setting_filter_prefix.'_start');
 
 			$current_tab = give_get_current_setting_tab();
 
@@ -206,7 +206,7 @@ 
 			self::get_settings_pages();
 
 			// Save settings if data has been posted.
-			if ( ! empty( $_POST ) ) {
+			if ( ! empty($_POST)) {
 				self::save();
 			}
 
@@ -219,7 +219,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			$tabs = apply_filters( self::$setting_filter_prefix . '_tabs_array', array() );
+			$tabs = apply_filters(self::$setting_filter_prefix.'_tabs_array', array());
 
 			include 'views/html-admin-settings.php';
 
@@ -237,25 +237,25 @@ 
 		 *
 		 * @return string|bool
 		 */
-		public static function get_option( $option_name = '', $field_id = '', $default = false ) {
+		public static function get_option($option_name = '', $field_id = '', $default = false) {
 			// Bailout.
-			if ( empty( $option_name ) && empty( $field_id ) ) {
+			if (empty($option_name) && empty($field_id)) {
 				return false;
 			}
 
-			if ( ! empty( $field_id ) && ! empty( $option_name ) ) {
+			if ( ! empty($field_id) && ! empty($option_name)) {
 				// Get field value if any.
-				$option_value = get_option( $option_name );
+				$option_value = get_option($option_name);
 
-				$option_value = ( is_array( $option_value ) && array_key_exists( $field_id, $option_value ) )
-					? $option_value[ $field_id ]
+				$option_value = (is_array($option_value) && array_key_exists($field_id, $option_value))
+					? $option_value[$field_id]
 					: $default;
 			} else {
 				// If option name is empty but not field name then this means, setting is direct store to option table under there field name.
 				$option_name = ! $option_name ? $field_id : $option_name;
 
 				// Get option value if any.
-				$option_value = get_option( $option_name, $default );
+				$option_value = get_option($option_name, $default);
 			}
 
 			return $option_value;
@@ -273,7 +273,7 @@ 
 		 *
 		 * @return void
 		 */
-		public static function output_fields( $options, $option_name = '' ) {
+		public static function output_fields($options, $option_name = '') {
 			$current_tab = give_get_current_setting_tab();
 
 			// Field Default values.
@@ -286,52 +286,52 @@ 
 				'table_html' => true,
 			);
 
-			foreach ( $options as $value ) {
-				if ( ! isset( $value['type'] ) ) {
+			foreach ($options as $value) {
+				if ( ! isset($value['type'])) {
 					continue;
 				}
 
 				// Set title.
-				$defaults['title'] = isset( $value['name'] ) ? $value['name'] : '';
+				$defaults['title'] = isset($value['name']) ? $value['name'] : '';
 
 				// Set default setting.
-				$value = wp_parse_args( $value, $defaults );
+				$value = wp_parse_args($value, $defaults);
 
 				// Colorpicker field.
-				$value['class'] = ( 'colorpicker' === $value['type'] ? trim( $value['class'] ) . ' give-colorpicker' : $value['class'] );
-				$value['type']  = ( 'colorpicker' === $value['type'] ? 'text' : $value['type'] );
+				$value['class'] = ('colorpicker' === $value['type'] ? trim($value['class']).' give-colorpicker' : $value['class']);
+				$value['type']  = ('colorpicker' === $value['type'] ? 'text' : $value['type']);
 
 
 				// Custom attribute handling.
 				$custom_attributes = array();
 
-				if ( ! empty( $value['attributes'] ) && is_array( $value['attributes'] ) ) {
-					foreach ( $value['attributes'] as $attribute => $attribute_value ) {
-						$custom_attributes[] = esc_attr( $attribute ) . '="' . esc_attr( $attribute_value ) . '"';
+				if ( ! empty($value['attributes']) && is_array($value['attributes'])) {
+					foreach ($value['attributes'] as $attribute => $attribute_value) {
+						$custom_attributes[] = esc_attr($attribute).'="'.esc_attr($attribute_value).'"';
 					}
 				}
 
 				// Description handling.
-				$description          = self::get_field_description( $value );
+				$description = self::get_field_description($value);
 
 				// Switch based on type.
-				switch ( $value['type'] ) {
+				switch ($value['type']) {
 
 					// Section Titles
 					case 'title':
-						if ( ! empty( $value['title'] ) ) {
-							echo '<div class="give-setting-tab-header give-setting-tab-header-' . $current_tab . '"><h2>' . self::get_field_title( $value ) . '</h2><hr></div>';
+						if ( ! empty($value['title'])) {
+							echo '<div class="give-setting-tab-header give-setting-tab-header-'.$current_tab.'"><h2>'.self::get_field_title($value).'</h2><hr></div>';
 						}
 
-						if ( ! empty( $value['desc'] ) ) {
-							echo wpautop( wptexturize( wp_kses_post( $value['desc'] ) ) );
+						if ( ! empty($value['desc'])) {
+							echo wpautop(wptexturize(wp_kses_post($value['desc'])));
 						}
 
-						if ( $value['table_html'] ) {
-							echo '<table class="form-table give-setting-tab-body give-setting-tab-body-' . $current_tab . '">' . "\n\n";
+						if ($value['table_html']) {
+							echo '<table class="form-table give-setting-tab-body give-setting-tab-body-'.$current_tab.'">'."\n\n";
 						}
 
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -340,14 +340,14 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) );
+							do_action('give_settings_'.sanitize_title($value['id']));
 						}
 
 						break;
 
 					// Section Ends.
 					case 'sectionend':
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -356,14 +356,14 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) . '_end' );
+							do_action('give_settings_'.sanitize_title($value['id']).'_end');
 						}
 
-						if ( $value['table_html'] ) {
+						if ($value['table_html']) {
 							echo '</table>';
 						}
 
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -372,7 +372,7 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) . '_after' );
+							do_action('give_settings_'.sanitize_title($value['id']).'_after');
 						}
 
 						break;
@@ -385,22 +385,22 @@ 
 					case 'password' :
 
 						$type = $value['type'];
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
                             <input
-                                    name="<?php echo esc_attr( $value['id'] ); ?>"
-                                    id="<?php echo esc_attr( $value['id'] ); ?>"
-                                    type="<?php echo esc_attr( $type ); ?>"
-                                    style="<?php echo esc_attr( $value['css'] ); ?>"
-                                    value="<?php echo esc_attr( $option_value ); ?>"
-                                    class="give-input-field<?php echo( empty( $value['class'] ) ? '' : ' ' . esc_attr( $value['class'] ) ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
+                                    name="<?php echo esc_attr($value['id']); ?>"
+                                    id="<?php echo esc_attr($value['id']); ?>"
+                                    type="<?php echo esc_attr($type); ?>"
+                                    style="<?php echo esc_attr($value['css']); ?>"
+                                    value="<?php echo esc_attr($option_value); ?>"
+                                    class="give-input-field<?php echo(empty($value['class']) ? '' : ' '.esc_attr($value['class'])); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
                             /> <?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -409,23 +409,23 @@ 
 					// Textarea.
 					case 'textarea':
 
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 								<textarea
-                                        name="<?php echo esc_attr( $value['id'] ); ?>"
-                                        id="<?php echo esc_attr( $value['id'] ); ?>"
-                                        style="<?php echo esc_attr( $value['css'] ); ?>"
-                                        class="<?php echo esc_attr( $value['class'] ); ?>"
+                                        name="<?php echo esc_attr($value['id']); ?>"
+                                        id="<?php echo esc_attr($value['id']); ?>"
+                                        style="<?php echo esc_attr($value['css']); ?>"
+                                        class="<?php echo esc_attr($value['class']); ?>"
                                         rows="10"
                                         cols="60"
-									<?php echo implode( ' ', $custom_attributes ); ?>
-                                ><?php echo esc_textarea( $option_value ); ?></textarea>
+									<?php echo implode(' ', $custom_attributes); ?>
+                                ><?php echo esc_textarea($option_value); ?></textarea>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -435,35 +435,35 @@ 
 					case 'select' :
 					case 'multiselect' :
 
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
                             <select
-                                    name="<?php echo esc_attr( $value['id'] ); ?><?php if ( $value['type'] == 'multiselect' ) {
+                                    name="<?php echo esc_attr($value['id']); ?><?php if ($value['type'] == 'multiselect') {
 										echo '[]';
 									} ?>"
-                                    id="<?php echo esc_attr( $value['id'] ); ?>"
-                                    style="<?php echo esc_attr( $value['css'] ); ?>"
-                                    class="<?php echo esc_attr( $value['class'] ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
-								<?php echo ( 'multiselect' == $value['type'] ) ? 'multiple="multiple"' : ''; ?>
+                                    id="<?php echo esc_attr($value['id']); ?>"
+                                    style="<?php echo esc_attr($value['css']); ?>"
+                                    class="<?php echo esc_attr($value['class']); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
+								<?php echo ('multiselect' == $value['type']) ? 'multiple="multiple"' : ''; ?>
                             >
 
 								<?php
-								if ( ! empty( $value['options'] ) ) {
-									foreach ( $value['options'] as $key => $val ) {
+								if ( ! empty($value['options'])) {
+									foreach ($value['options'] as $key => $val) {
 										?>
-                                        <option value="<?php echo esc_attr( $key ); ?>" <?php
+                                        <option value="<?php echo esc_attr($key); ?>" <?php
 
-										if ( is_array( $option_value ) ) {
-											selected( in_array( $key, $option_value ), true );
+										if (is_array($option_value)) {
+											selected(in_array($key, $option_value), true);
 										} else {
-											selected( $option_value, $key );
+											selected($option_value, $key);
 										}
 
 										?>><?php echo $val ?></option>
@@ -479,28 +479,28 @@ 
 
 					// Radio inputs.
 					case 'radio_inline' :
-						$value['class'] = empty( $value['class'] ) ? 'give-radio-inline' : $value['class'] . ' give-radio-inline';
+						$value['class'] = empty($value['class']) ? 'give-radio-inline' : $value['class'].' give-radio-inline';
 					case 'radio' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?> <?php echo( ! empty( $value['class'] ) ? $value['class'] : '' ); ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?> <?php echo( ! empty($value['class']) ? $value['class'] : ''); ?>">
                             <fieldset>
                                 <ul>
 									<?php
-									foreach ( $value['options'] as $key => $val ) {
+									foreach ($value['options'] as $key => $val) {
 										?>
                                         <li>
                                             <label><input
-                                                        name="<?php echo esc_attr( $value['id'] ); ?>"
+                                                        name="<?php echo esc_attr($value['id']); ?>"
                                                         value="<?php echo $key; ?>"
                                                         type="radio"
-                                                        style="<?php echo esc_attr( $value['css'] ); ?>"
-													<?php echo implode( ' ', $custom_attributes ); ?>
-													<?php checked( $key, $option_value ); ?>
+                                                        style="<?php echo esc_attr($value['css']); ?>"
+													<?php echo implode(' ', $custom_attributes); ?>
+													<?php checked($key, $option_value); ?>
                                                 /> <?php echo $val ?></label>
                                         </li>
 										<?php
@@ -514,21 +514,21 @@ 
 
 					// Checkbox input.
 					case 'checkbox' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                        <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                        <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                             <th scope="row" class="titledesc">
-                                <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                                <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                             </th>
                             <td class="give-forminp">
                                 <input
-                                        name="<?php echo esc_attr( $value['id'] ); ?>"
-                                        id="<?php echo esc_attr( $value['id'] ); ?>"
+                                        name="<?php echo esc_attr($value['id']); ?>"
+                                        id="<?php echo esc_attr($value['id']); ?>"
                                         type="checkbox"
-                                        class="<?php echo esc_attr( isset( $value['class'] ) ? $value['class'] : '' ); ?>"
+                                        class="<?php echo esc_attr(isset($value['class']) ? $value['class'] : ''); ?>"
                                         value="1"
-									<?php checked( $option_value, 'on' ); ?>
-									<?php echo implode( ' ', $custom_attributes ); ?>
+									<?php checked($option_value, 'on'); ?>
+									<?php echo implode(' ', $custom_attributes); ?>
                                 />
 								<?php echo $description; ?>
                             </td>
@@ -538,28 +538,28 @@ 
 
 					// Multi Checkbox input.
 					case 'multicheck' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$option_value = is_array( $option_value ) ? $option_value : array();
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$option_value = is_array($option_value) ? $option_value : array();
 						?>
-                        <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                        <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                             <th scope="row" class="titledesc">
-                                <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                                <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                             </th>
-                            <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?> <?php echo( ! empty( $value['class'] ) ? $value['class'] : '' ); ?>">
+                            <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?> <?php echo( ! empty($value['class']) ? $value['class'] : ''); ?>">
                                 <fieldset>
                                     <ul>
 										<?php
-										foreach ( $value['options'] as $key => $val ) {
+										foreach ($value['options'] as $key => $val) {
 											?>
                                             <li>
                                                 <label>
                                                     <input
-                                                            name="<?php echo esc_attr( $value['id'] ); ?>[]"
+                                                            name="<?php echo esc_attr($value['id']); ?>[]"
                                                             value="<?php echo $key; ?>"
                                                             type="checkbox"
-                                                            style="<?php echo esc_attr( $value['css'] ); ?>"
-														<?php echo implode( ' ', $custom_attributes ); ?>
-														<?php if ( in_array( $key, $option_value ) ) {
+                                                            style="<?php echo esc_attr($value['css']); ?>"
+														<?php echo implode(' ', $custom_attributes); ?>
+														<?php if (in_array($key, $option_value)) {
 															echo 'checked="checked"';
 														} ?>
                                                     /> <?php echo $val ?>
@@ -577,25 +577,25 @@ 
 
 					// File input field.
 					case 'file' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
                             <div class="give-field-wrap">
                                 <label for="<?php echo $value['id'] ?>">
                                     <input
-                                            name="<?php echo esc_attr( $value['id'] ); ?>"
-                                            id="<?php echo esc_attr( $value['id'] ); ?>"
+                                            name="<?php echo esc_attr($value['id']); ?>"
+                                            id="<?php echo esc_attr($value['id']); ?>"
                                             type="text"
-                                            class="give-input-field<?php echo esc_attr( isset( $value['class'] ) ? ' ' . $value['class'] : '' ); ?>"
+                                            class="give-input-field<?php echo esc_attr(isset($value['class']) ? ' '.$value['class'] : ''); ?>"
                                             value="<?php echo $option_value; ?>"
-                                            style="<?php echo esc_attr( $value['css'] ); ?>"
-										<?php echo implode( ' ', $custom_attributes ); ?>
+                                            style="<?php echo esc_attr($value['css']); ?>"
+										<?php echo implode(' ', $custom_attributes); ?>
                                     />&nbsp;&nbsp;&nbsp;&nbsp;<input class="give-upload-button button" type="button"
-                                                                     value="<?php echo esc_html__( 'Add or Upload File', 'give' ); ?>">
+                                                                     value="<?php echo esc_html__('Add or Upload File', 'give'); ?>">
 									<?php echo $description ?>
                                     <div class="give-image-thumb<?php echo ! $option_value ? ' give-hidden' : ''; ?>">
                                         <span class="give-delete-image-thumb dashicons dashicons-no-alt"></span>
@@ -610,17 +610,17 @@ 
 					// WordPress Editor.
 					case 'wysiwyg' :
 						// Get option value.
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						// Get editor settings.
-						$editor_settings = ! empty( $value['options'] ) ? $value['options'] : array();
+						$editor_settings = ! empty($value['options']) ? $value['options'] : array();
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
-							<?php wp_editor( $option_value, $value['id'], $editor_settings ); ?>
+							<?php wp_editor($option_value, $value['id'], $editor_settings); ?>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -629,9 +629,9 @@ 
 					// Custom: System setting field.
 					case 'system_info' :
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
 							<?php give_system_info_callback(); ?>
@@ -642,14 +642,14 @@ 
 
 					// Custom: Default gateways setting field.
 					case 'default_gateway' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
-							<?php give_default_gateway_callback( $value, $option_value ); ?>
+							<?php give_default_gateway_callback($value, $option_value); ?>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -657,14 +657,14 @@ 
 
 					// Custom: Enable gateways setting field.
 					case 'enabled_gateways' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
-							<?php give_enabled_gateways_callback( $value, $option_value ); ?>
+							<?php give_enabled_gateways_callback($value, $option_value); ?>
 							<?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -673,9 +673,9 @@ 
 					// Custom: Email preview buttons field.
 					case 'email_preview_buttons' :
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
                         <td class="give-forminp">
 							<?php give_email_preview_buttons_callback(); ?>
@@ -692,22 +692,22 @@ 
 
 					// Custom: Gateway API key.
 					case 'api_key' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$type         = ! empty( $option_value ) ? 'password' : 'text';
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$type         = ! empty($option_value) ? 'password' : 'text';
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <th scope="row" class="titledesc">
-                            <label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+                            <label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
                         </th>
-                        <td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+                        <td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
                             <input
-                                    name="<?php echo esc_attr( $value['id'] ); ?>"
-                                    id="<?php echo esc_attr( $value['id'] ); ?>"
-                                    type="<?php echo esc_attr( $type ); ?>"
-                                    style="<?php echo esc_attr( $value['css'] ); ?>"
-                                    value="<?php echo esc_attr( trim( $option_value ) ); ?>"
-                                    class="give-input-field<?php echo( empty( $value['class'] ) ? '' : ' ' . esc_attr( $value['class'] ) ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
+                                    name="<?php echo esc_attr($value['id']); ?>"
+                                    id="<?php echo esc_attr($value['id']); ?>"
+                                    type="<?php echo esc_attr($type); ?>"
+                                    style="<?php echo esc_attr($value['css']); ?>"
+                                    value="<?php echo esc_attr(trim($option_value)); ?>"
+                                    class="give-input-field<?php echo(empty($value['class']) ? '' : ' '.esc_attr($value['class'])); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
                             /> <?php echo $description; ?>
                         </td>
                         </tr><?php
@@ -716,7 +716,7 @@ 
 					// Custom: Log field.
 					case 'logs' :
 
-						include GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php';
+						include GIVE_PLUGIN_DIR.'includes/admin/tools/logs/logs.php';
 
 						// Get current section.
 						$current_section = $_GET['section'] = give_get_current_setting_section();
@@ -726,7 +726,7 @@ 
 						 *
 						 * @since 1.0
 						 */
-						do_action( "give_logs_view_{$current_section}" );
+						do_action("give_logs_view_{$current_section}");
 
 						echo $description;
 						break;
@@ -734,7 +734,7 @@ 
 					// Custom: Data field.
 					case 'data' :
 
-						include  GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-data.php';
+						include  GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-data.php';
 
 						echo $description;
 						break;
@@ -742,12 +742,12 @@ 
 					// Custom: Give Docs Link field type.
 					case 'give_docs_link' :
 						?>
-                    <tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+                    <tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
                         <td class="give-docs-link" colspan="2">
 							<?php
-							echo '<p class="give-docs-link"><a href="' . esc_url( $value['url'] )
+							echo '<p class="give-docs-link"><a href="'.esc_url($value['url'])
 							     . '" target="_blank">'
-							     . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
+							     . sprintf(esc_html__('Need Help? See docs on "%s"', 'give'), $value['title'])
 							     . '<span class="dashicons dashicons-editor-help"></span></a></p>';
 							?>
                         </td>
@@ -758,8 +758,8 @@ 
 					// You can add or handle your custom field action.
 					default:
 						// Get option value.
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						do_action( 'give_admin_field_' . $value['type'], $value, $option_value );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						do_action('give_admin_field_'.$value['type'], $value, $option_value);
 						break;
 				}
 			}
@@ -775,15 +775,15 @@ 
 		 *
 		 * @return string The HTML description of the field.
 		 */
-		public static function get_field_description( $value ) {
+		public static function get_field_description($value) {
 			$description = '';
 
 			// Support for both 'description' and 'desc' args.
-			$description_key = isset( $value['description'] ) ? 'description' : 'desc';
-			$value           = ( isset( $value[ $description_key ] ) && ! empty( $value[ $description_key ] ) ) ? $value[ $description_key ] : '';
+			$description_key = isset($value['description']) ? 'description' : 'desc';
+			$value           = (isset($value[$description_key]) && ! empty($value[$description_key])) ? $value[$description_key] : '';
 
-			if ( ! empty( $value ) ) {
-				$description = '<p class="give-field-description">' . wp_kses_post( $value ) . '</p>';
+			if ( ! empty($value)) {
+				$description = '<p class="give-field-description">'.wp_kses_post($value).'</p>';
 			}
 
 			return $description;
@@ -800,11 +800,11 @@ 
 		 *
 		 * @return array The description and tip as a 2 element array
 		 */
-		public static function get_field_title( $value ) {
-			$title = esc_html( $value['title'] );
+		public static function get_field_title($value) {
+			$title = esc_html($value['title']);
 
 			// If html tag detected then allow them to print.
-			if ( strip_tags( $title ) ) {
+			if (strip_tags($title)) {
 				$title = $value['title'];
 			}
 
@@ -823,8 +823,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public static function save_fields( $options, $option_name = '' ) {
-			if ( empty( $_POST ) ) {
+		public static function save_fields($options, $option_name = '') {
+			if (empty($_POST)) {
 				return false;
 			}
 
@@ -832,37 +832,37 @@ 
 			$update_options = array();
 
 			// Loop options and get values to save.
-			foreach ( $options as $option ) {
-				if ( ! isset( $option['id'] ) || ! isset( $option['type'] ) ) {
+			foreach ($options as $option) {
+				if ( ! isset($option['id']) || ! isset($option['type'])) {
 					continue;
 				}
 
 				// Get posted value.
-				if ( strstr( $option['id'], '[' ) ) {
-					parse_str( $option['id'], $option_name_array );
-					$field_option_name = current( array_keys( $option_name_array ) );
-					$setting_name      = key( $option_name_array[ $field_option_name ] );
-					$raw_value         = isset( $_POST[ $field_option_name ][ $setting_name ] ) ? wp_unslash( $_POST[ $field_option_name ][ $setting_name ] ) : null;
+				if (strstr($option['id'], '[')) {
+					parse_str($option['id'], $option_name_array);
+					$field_option_name = current(array_keys($option_name_array));
+					$setting_name      = key($option_name_array[$field_option_name]);
+					$raw_value         = isset($_POST[$field_option_name][$setting_name]) ? wp_unslash($_POST[$field_option_name][$setting_name]) : null;
 				} else {
 					$field_option_name = $option['id'];
 					$setting_name      = '';
-					$raw_value         = isset( $_POST[ $option['id'] ] ) ? wp_unslash( $_POST[ $option['id'] ] ) : null;
+					$raw_value         = isset($_POST[$option['id']]) ? wp_unslash($_POST[$option['id']]) : null;
 				}
 
 				// Format the value based on option type.
-				switch ( $option['type'] ) {
+				switch ($option['type']) {
 					case 'checkbox' :
-						$value = is_null( $raw_value ) ? '' : 'on';
+						$value = is_null($raw_value) ? '' : 'on';
 						break;
 					case 'wysiwyg'  :
 					case 'textarea' :
-						$value = wp_kses_post( trim( $raw_value ) );
+						$value = wp_kses_post(trim($raw_value));
 						break;
 					case 'multiselect' :
-						$value = array_filter( array_map( 'give_clean', (array) $raw_value ) );
+						$value = array_filter(array_map('give_clean', (array) $raw_value));
 						break;
 					default :
-						$value = give_clean( $raw_value );
+						$value = give_clean($raw_value);
 						break;
 				}
 
@@ -871,37 +871,37 @@ 
 				 *
 				 * @since 1.8
 				 */
-				$value = apply_filters( 'give_admin_settings_sanitize_option', $value, $option, $raw_value );
+				$value = apply_filters('give_admin_settings_sanitize_option', $value, $option, $raw_value);
 
 				/**
 				 * Sanitize the value of an option by option name.
 				 *
 				 * @since 1.8
 				 */
-				$value = apply_filters( "give_admin_settings_sanitize_option_{$field_option_name}", $value, $option, $raw_value );
+				$value = apply_filters("give_admin_settings_sanitize_option_{$field_option_name}", $value, $option, $raw_value);
 
-				if ( is_null( $value ) ) {
+				if (is_null($value)) {
 					continue;
 				}
 
 				// Check if option is an array and handle that differently to single values.
-				if ( $field_option_name && $setting_name ) {
-					if ( ! isset( $update_options[ $field_option_name ] ) ) {
-						$update_options[ $field_option_name ] = get_option( $field_option_name, array() );
+				if ($field_option_name && $setting_name) {
+					if ( ! isset($update_options[$field_option_name])) {
+						$update_options[$field_option_name] = get_option($field_option_name, array());
 					}
-					if ( ! is_array( $update_options[ $field_option_name ] ) ) {
-						$update_options[ $field_option_name ] = array();
+					if ( ! is_array($update_options[$field_option_name])) {
+						$update_options[$field_option_name] = array();
 					}
-					$update_options[ $field_option_name ][ $setting_name ] = $value;
+					$update_options[$field_option_name][$setting_name] = $value;
 				} else {
-					$update_options[ $field_option_name ] = $value;
+					$update_options[$field_option_name] = $value;
 				}
 			}
 
 			// Save all options in our array or there own option name i.e. option id.
-			if ( empty( $option_name ) ) {
-				foreach ( $update_options as $name => $value ) {
-					update_option( $name, $value );
+			if (empty($option_name)) {
+				foreach ($update_options as $name => $value) {
+					update_option($name, $value);
 
 					/**
 					 * Trigger action.
@@ -910,13 +910,13 @@ 
 					 *
 					 * @since 1.8
 					 */
-					do_action( "give_save_option_{$name}", $value, $name );
+					do_action("give_save_option_{$name}", $value, $name);
 				}
 			} else {
-				$old_options    = ( $old_options = get_option( $option_name ) ) ? $old_options : array();
-				$update_options = array_merge( $old_options, $update_options );
+				$old_options    = ($old_options = get_option($option_name)) ? $old_options : array();
+				$update_options = array_merge($old_options, $update_options);
 
-				update_option( $option_name, $update_options );
+				update_option($option_name, $update_options);
 
 				/**
 				 * Trigger action.
@@ -925,7 +925,7 @@ 
 				 *
 				 * @since 1.8
 				 */
-				do_action( "give_save_settings_{$option_name}", $update_options, $option_name );
+				do_action("give_save_settings_{$option_name}", $update_options, $option_name);
 			}
 
 			return true;

includes/admin/class-addon-activation-banner.php

Spacing +45 added lines, -45 removed lines

@@ -8,7 +8,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,14 +32,14 @@ 
 	 *                               'testing'           => false, // (required) Never leave as "true" in production!!!
 	 *                               }
 	 */
-	function __construct( $_banner_details ) {
+	function __construct($_banner_details) {
 		$current_user = wp_get_current_user();
 
 		$this->plugin_activate_by   = 0;
 		$this->banner_details       = $_banner_details;
-		$this->test_mode            = ( $this->banner_details['testing'] == 'true' ) ? true : false;
-		$this->nag_meta_key         = 'give_addon_activation_ignore_' . sanitize_title( $this->banner_details['name'] );
-		$this->activate_by_meta_key = 'give_addon_' . sanitize_title( $this->banner_details['name'] ) . '_active_by_user';
+		$this->test_mode            = ($this->banner_details['testing'] == 'true') ? true : false;
+		$this->nag_meta_key         = 'give_addon_activation_ignore_'.sanitize_title($this->banner_details['name']);
+		$this->activate_by_meta_key = 'give_addon_'.sanitize_title($this->banner_details['name']).'_active_by_user';
 
 		//Get current user
 		$this->user_id = $current_user->ID;
@@ -62,19 +62,19 @@ 
 	public function init() {
 
 		//Testing?
-		if ( $this->test_mode ) {
-			delete_user_meta( $this->user_id, $this->nag_meta_key );
+		if ($this->test_mode) {
+			delete_user_meta($this->user_id, $this->nag_meta_key);
 		}
 
 		//Get the current page to add the notice to
-		add_action( 'current_screen', array( $this, 'give_addon_notice_ignore' ) );
-		add_action( 'admin_notices', array( $this, 'give_addon_activation_admin_notice' ) );
+		add_action('current_screen', array($this, 'give_addon_notice_ignore'));
+		add_action('admin_notices', array($this, 'give_addon_activation_admin_notice'));
 
 		// File path of addon must be included in banner detail other addon activate meta will not delete.
 		$file_name = $this->get_plugin_file_name();
 
-		if ( ! empty( $file_name ) ) {
-			add_action( 'deactivate_' . $file_name, array( $this, 'remove_addon_activate_meta' ) );
+		if ( ! empty($file_name)) {
+			add_action('deactivate_'.$file_name, array($this, 'remove_addon_activate_meta'));
 		}
 	}
 
@@ -89,7 +89,7 @@ 
 	private function is_plugin_page() {
 		$screen = get_current_screen();
 
-		return ( $screen->parent_file === 'plugins.php' );
+		return ($screen->parent_file === 'plugins.php');
 	}
 
 
@@ -102,12 +102,12 @@ 
 	public function give_addon_activation_admin_notice() {
 
 		// Bailout.
-		if ( ! $this->is_plugin_page() || $this->user_id !== $this->plugin_activate_by ) {
+		if ( ! $this->is_plugin_page() || $this->user_id !== $this->plugin_activate_by) {
 			return;
 		}
 
 		// If the user hasn't already dismissed the alert, output activation banner.
-		if ( ! get_user_meta( $this->user_id, $this->nag_meta_key ) ) {
+		if ( ! get_user_meta($this->user_id, $this->nag_meta_key)) {
 
 			// Output inline styles here because there's no reason
 			// to enqueued them after the alert is dismissed.
@@ -189,44 +189,44 @@ 
 					<h3><?php
 						printf(
 						/* translators: %s: Add-on name */
-							esc_html__( "Thank you for installing Give's %s Add-on!", 'give' ),
-							'<span>' . $this->banner_details['name'] . '</span>'
+							esc_html__("Thank you for installing Give's %s Add-on!", 'give'),
+							'<span>'.$this->banner_details['name'].'</span>'
 						);
 						?></h3>
 
 					<a href="<?php
 					//The Dismiss Button.
-					$nag_admin_dismiss_url = 'plugins.php?' . $this->nag_meta_key . '=0';
-					echo admin_url( $nag_admin_dismiss_url ); ?>" class="dismiss"><span
+					$nag_admin_dismiss_url = 'plugins.php?'.$this->nag_meta_key.'=0';
+					echo admin_url($nag_admin_dismiss_url); ?>" class="dismiss"><span
 							class="dashicons dashicons-dismiss"></span></a>
 
 					<div class="alert-actions">
 
 						<?php //Point them to your settings page.
-						if ( isset( $this->banner_details['settings_url'] ) ) { ?>
+						if (isset($this->banner_details['settings_url'])) { ?>
 							<a href="<?php echo $this->banner_details['settings_url']; ?>">
-								<span class="dashicons dashicons-admin-settings"></span><?php esc_html_e( 'Go to Settings', 'give' ); ?>
+								<span class="dashicons dashicons-admin-settings"></span><?php esc_html_e('Go to Settings', 'give'); ?>
 							</a>
 						<?php } ?>
 
 						<?php
 						// Show them how to configure the Addon.
-						if ( isset( $this->banner_details['documentation_url'] ) ) { ?>
+						if (isset($this->banner_details['documentation_url'])) { ?>
 							<a href="<?php echo $this->banner_details['documentation_url'] ?>" target="_blank">
 								<span class="dashicons dashicons-media-text"></span><?php
 								printf(
 								/* translators: %s: Add-on name */
-									esc_html__( 'Documentation: %s Add-on', 'give' ),
+									esc_html__('Documentation: %s Add-on', 'give'),
 									$this->banner_details['name']
 								);
 								?></a>
 						<?php } ?>
 						<?php
 						//Let them signup for plugin updates
-						if ( isset( $this->banner_details['support_url'] ) ) { ?>
+						if (isset($this->banner_details['support_url'])) { ?>
 
 							<a href="<?php echo $this->banner_details['support_url'] ?>" target="_blank">
-								<span class="dashicons dashicons-sos"></span><?php esc_html_e( 'Get Support', 'give' ); ?>
+								<span class="dashicons dashicons-sos"></span><?php esc_html_e('Get Support', 'give'); ?>
 							</a>
 
 						<?php } ?>
@@ -253,13 +253,13 @@ 
 		 * If user clicks to ignore the notice, add that to their user meta the banner then checks whether this tag exists already or not.
 		 * See here: http://codex.wordpress.org/Function_Reference/add_user_meta
 		 */
-		if ( isset( $_GET[ $this->nag_meta_key ] ) && '0' == $_GET[ $this->nag_meta_key ] ) {
+		if (isset($_GET[$this->nag_meta_key]) && '0' == $_GET[$this->nag_meta_key]) {
 
 			//Get the global user
 			$current_user = wp_get_current_user();
 			$user_id      = $current_user->ID;
 
-			add_user_meta( $user_id, $this->nag_meta_key, 'true', true );
+			add_user_meta($user_id, $this->nag_meta_key, 'true', true);
 		}
 	}
 
@@ -270,11 +270,11 @@ 
 	 * @access private
 	 */
 	private function add_addon_activate_meta() {
-		$user_id                  = get_option( $this->activate_by_meta_key );
+		$user_id                  = get_option($this->activate_by_meta_key);
 		$this->plugin_activate_by = (int) $user_id;
 
-		if ( ! $user_id ) {
-			add_option( $this->activate_by_meta_key, $this->user_id, '', 'no' );
+		if ( ! $user_id) {
+			add_option($this->activate_by_meta_key, $this->user_id, '', 'no');
 			$this->plugin_activate_by = (int) $this->user_id;
 		}
 	}
@@ -287,10 +287,10 @@ 
 	 * @access public
 	 */
 	public function remove_addon_activate_meta() {
-		$user_id = get_option( $this->activate_by_meta_key );
+		$user_id = get_option($this->activate_by_meta_key);
 
-		if ( $user_id ) {
-			delete_option( $this->activate_by_meta_key );
+		if ($user_id) {
+			delete_option($this->activate_by_meta_key);
 		}
 	}
 
@@ -303,39 +303,39 @@ 
 	 * @return mixed
 	 */
 	private function get_plugin_file_name() {
-		$active_plugins = get_option( 'active_plugins' );
+		$active_plugins = get_option('active_plugins');
 		$file_name      = '';
 
 		try {
 
 			// Check addon file path.
-			if ( ! empty( $this->banner_details['file'] ) ) {
+			if ( ! empty($this->banner_details['file'])) {
 				$file_name = '';
-				if ( $file_path = explode( '/plugins/', $this->banner_details['file'] ) ) {
-					$file_path = array_pop( $file_path );
-					$file_name = current( explode( '/', $file_path ) );
+				if ($file_path = explode('/plugins/', $this->banner_details['file'])) {
+					$file_path = array_pop($file_path);
+					$file_name = current(explode('/', $file_path));
 				}
 
-				if ( empty( $file_name ) ) {
+				if (empty($file_name)) {
 					return false;
 				}
 
-				foreach ( $active_plugins as $plugin ) {
-					if ( false !== strpos( $plugin, $file_name ) ) {
+				foreach ($active_plugins as $plugin) {
+					if (false !== strpos($plugin, $file_name)) {
 						$file_name = $plugin;
 						break;
 					}
 				}
-			} elseif ( WP_DEBUG ) {
-				throw new Exception( __( "File path must be added within the {$this->banner_details['name']} add-on in the banner details.", 'give' ) );
+			} elseif (WP_DEBUG) {
+				throw new Exception(__("File path must be added within the {$this->banner_details['name']} add-on in the banner details.", 'give'));
 			}
 
 			// Check plugin path calculated by addon file path.
-			if ( empty( $file_name ) && WP_DEBUG ) {
-				throw new Exception( __( "Empty add-on plugin path for {$this->banner_details['name']} add-on.", 'give' ) );
+			if (empty($file_name) && WP_DEBUG) {
+				throw new Exception(__("Empty add-on plugin path for {$this->banner_details['name']} add-on.", 'give'));
 			}
 
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			echo $e->getMessage();
 		}
 

includes/admin/upgrades/upgrade-functions.php

Indentation +12 added lines, -12 removed lines

@@ -50,7 +50,7 @@ 
 			give_v187_upgrades();
 			$did_upgrade = true;
 
-        case version_compare( $give_version, '1.8.8', '<' ) :
+		case version_compare( $give_version, '1.8.8', '<' ) :
 			give_v188_upgrades();
 			$did_upgrade = true;
 
@@ -909,19 +909,19 @@ 
 
 	global $wp_roles;
 
-    // Get the role object.
-    $give_worker = get_role( 'give_worker' );
+	// Get the role object.
+	$give_worker = get_role( 'give_worker' );
 
 	// A list of capabilities to add for give workers.
-    $caps_to_add = array(
-        'edit_posts',
-        'edit_pages'
-    );
-
-    foreach ( $caps_to_add as $cap ) {
-        // Add the capability.
-        $give_worker->add_cap( $cap );
-    }
+	$caps_to_add = array(
+		'edit_posts',
+		'edit_pages'
+	);
+
+	foreach ( $caps_to_add as $cap ) {
+		// Add the capability.
+		$give_worker->add_cap( $cap );
+	}
 
 }
 

Spacing +185 added lines, -185 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,44 +25,44 @@ 
  */
 function give_do_automatic_upgrades() {
 	$did_upgrade  = false;
-	$give_version = preg_replace( '/[^0-9.].*/', '', get_option( 'give_version' ) );
+	$give_version = preg_replace('/[^0-9.].*/', '', get_option('give_version'));
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	switch ( true ) {
+	switch (true) {
 
-		case version_compare( $give_version, '1.6', '<' ) :
+		case version_compare($give_version, '1.6', '<') :
 			give_v16_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.7', '<' ) :
+		case version_compare($give_version, '1.7', '<') :
 			give_v17_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8', '<' ) :
+		case version_compare($give_version, '1.8', '<') :
 			give_v18_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.7', '<' ) :
+		case version_compare($give_version, '1.8.7', '<') :
 			give_v187_upgrades();
 			$did_upgrade = true;
 
-        case version_compare( $give_version, '1.8.8', '<' ) :
+        case version_compare($give_version, '1.8.8', '<') :
 			give_v188_upgrades();
 			$did_upgrade = true;
 
 	}
 
-	if ( $did_upgrade ) {
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+	if ($did_upgrade) {
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 	}
 }
 
-add_action( 'admin_init', 'give_do_automatic_upgrades' );
-add_action( 'give_upgrades', 'give_do_automatic_upgrades' );
+add_action('admin_init', 'give_do_automatic_upgrades');
+add_action('give_upgrades', 'give_do_automatic_upgrades');
 
 /**
  * Display Upgrade Notices
@@ -72,18 +72,18 @@ 
  */
 function give_show_upgrade_notices() {
 	// Don't show notices on the upgrades page.
-	if ( isset( $_GET['page'] ) && $_GET['page'] == 'give-upgrades' ) {
+	if (isset($_GET['page']) && $_GET['page'] == 'give-upgrades') {
 		return;
 	}
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	$give_version = preg_replace( '/[^0-9.].*/', '', $give_version );
+	$give_version = preg_replace('/[^0-9.].*/', '', $give_version);
 
 	/*
 	 *  NOTICE:
@@ -94,31 +94,31 @@ 
 	 */
 
 	// v1.3.2 Upgrades
-	if ( version_compare( $give_version, '1.3.2', '<' ) || ! give_has_upgrade_completed( 'upgrade_give_payment_customer_id' ) ) {
+	if (version_compare($give_version, '1.3.2', '<') || ! give_has_upgrade_completed('upgrade_give_payment_customer_id')) {
 		printf(
 			/* translators: %s: upgrade URL */
-			'<div class="updated"><p>' . __( 'Give needs to upgrade the donor database, click <a href="%s">here</a> to start the upgrade.', 'give' ) . '</p></div>',
-			esc_url( admin_url( 'index.php?page=give-upgrades&give-upgrade=upgrade_give_payment_customer_id' ) )
+			'<div class="updated"><p>'.__('Give needs to upgrade the donor database, click <a href="%s">here</a> to start the upgrade.', 'give').'</p></div>',
+			esc_url(admin_url('index.php?page=give-upgrades&give-upgrade=upgrade_give_payment_customer_id'))
 		);
 	}
 
 	// v1.3.4 Upgrades //ensure the user has gone through 1.3.4.
-	if ( version_compare( $give_version, '1.3.4', '<' ) || ( ! give_has_upgrade_completed( 'upgrade_give_offline_status' ) && give_has_upgrade_completed( 'upgrade_give_payment_customer_id' ) ) ) {
+	if (version_compare($give_version, '1.3.4', '<') || ( ! give_has_upgrade_completed('upgrade_give_offline_status') && give_has_upgrade_completed('upgrade_give_payment_customer_id'))) {
 		printf(
 			/* translators: %s: upgrade URL */
-			'<div class="updated"><p>' . __( 'Give needs to upgrade the donations database, click <a href="%s">here</a> to start the upgrade.', 'give' ) . '</p></div>',
-			esc_url( admin_url( 'index.php?page=give-upgrades&give-upgrade=upgrade_give_offline_status' ) )
+			'<div class="updated"><p>'.__('Give needs to upgrade the donations database, click <a href="%s">here</a> to start the upgrade.', 'give').'</p></div>',
+			esc_url(admin_url('index.php?page=give-upgrades&give-upgrade=upgrade_give_offline_status'))
 		);
 	}
 
 	// Check if we have a stalled upgrade.
 	$resume_upgrade = give_maybe_resume_upgrade();
-	if ( ! empty( $resume_upgrade ) ) {
-		$resume_url = add_query_arg( $resume_upgrade, admin_url( 'index.php' ) );
+	if ( ! empty($resume_upgrade)) {
+		$resume_url = add_query_arg($resume_upgrade, admin_url('index.php'));
 		echo Give_Notices::notice_html(
 			sprintf(
-				__( 'Give needs to complete a database upgrade that was previously started, click <a href="%s">here</a> to resume the upgrade.', 'give' ),
-				esc_url( $resume_url )
+				__('Give needs to complete a database upgrade that was previously started, click <a href="%s">here</a> to resume the upgrade.', 'give'),
+				esc_url($resume_url)
 			)
 		);
 
@@ -126,11 +126,11 @@ 
 	}
 
 	// v1.8 form metadata upgrades.
-	if ( version_compare( $give_version, '1.8', '<' ) || ! give_has_upgrade_completed( 'v18_upgrades_form_metadata' ) ) {
+	if (version_compare($give_version, '1.8', '<') || ! give_has_upgrade_completed('v18_upgrades_form_metadata')) {
 		echo Give_Notices::notice_html(
 			sprintf(
-				esc_html__( 'Give needs to upgrade the form database, click %1$shere%2$s to start the upgrade.', 'give' ),
-				'<a class="give-upgrade-link" href="' . esc_url( admin_url( 'index.php?page=give-upgrades&give-upgrade=give_v18_upgrades_form_metadata' ) ) . '">',
+				esc_html__('Give needs to upgrade the form database, click %1$shere%2$s to start the upgrade.', 'give'),
+				'<a class="give-upgrade-link" href="'.esc_url(admin_url('index.php?page=give-upgrades&give-upgrade=give_v18_upgrades_form_metadata')).'">',
 				'</a>'
 			)
 		);
@@ -145,7 +145,7 @@ 
 				$upgrade_links.on( 'click', function(e){
 					e.preventDefault();
 
-					if( ! window.confirm( '<?php _e( 'Please make sure to create a database backup before initiating the upgrade.', 'give' ); ?>' ) ) {
+					if( ! window.confirm( '<?php _e('Please make sure to create a database backup before initiating the upgrade.', 'give'); ?>' ) ) {
 						return;
 					}
 
@@ -158,7 +158,7 @@ 
 	<?php
 }
 
-add_action( 'admin_notices', 'give_show_upgrade_notices' );
+add_action('admin_notices', 'give_show_upgrade_notices');
 
 /**
  * Triggers all upgrade functions
@@ -170,29 +170,29 @@ 
  */
 function give_trigger_upgrades() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
-		add_option( 'give_version', $give_version );
+		add_option('give_version', $give_version);
 	}
 
-	update_option( 'give_version', GIVE_VERSION );
-	delete_option( 'give_doing_upgrade' );
+	update_option('give_version', GIVE_VERSION);
+	delete_option('give_doing_upgrade');
 
-	if ( DOING_AJAX ) {
-		die( 'complete' );
+	if (DOING_AJAX) {
+		die('complete');
 	} // End if().
 }
 
-add_action( 'wp_ajax_give_trigger_upgrades', 'give_trigger_upgrades' );
+add_action('wp_ajax_give_trigger_upgrades', 'give_trigger_upgrades');
 
 /**
  * Check if the upgrade routine has been run for a specific action
@@ -203,15 +203,15 @@ 
  *
  * @return bool                   If the action has been added to the completed actions array
  */
-function give_has_upgrade_completed( $upgrade_action = '' ) {
+function give_has_upgrade_completed($upgrade_action = '') {
 
-	if ( empty( $upgrade_action ) ) {
+	if (empty($upgrade_action)) {
 		return false;
 	}
 
 	$completed_upgrades = give_get_completed_upgrades();
 
-	return in_array( $upgrade_action, $completed_upgrades );
+	return in_array($upgrade_action, $completed_upgrades);
 
 }
 
@@ -223,8 +223,8 @@ 
  * @return mixed   When nothing to resume returns false, otherwise starts the upgrade where it left off
  */
 function give_maybe_resume_upgrade() {
-	$doing_upgrade = get_option( 'give_doing_upgrade', false );
-	if ( empty( $doing_upgrade ) ) {
+	$doing_upgrade = get_option('give_doing_upgrade', false);
+	if (empty($doing_upgrade)) {
 		return false;
 	}
 
@@ -240,9 +240,9 @@ 
  *
  * @return bool                   If the function was successfully added
  */
-function give_set_upgrade_complete( $upgrade_action = '' ) {
+function give_set_upgrade_complete($upgrade_action = '') {
 
-	if ( empty( $upgrade_action ) ) {
+	if (empty($upgrade_action)) {
 		return false;
 	}
 
@@ -250,9 +250,9 @@ 
 	$completed_upgrades[] = $upgrade_action;
 
 	// Remove any blanks, and only show uniques.
-	$completed_upgrades = array_unique( array_values( $completed_upgrades ) );
+	$completed_upgrades = array_unique(array_values($completed_upgrades));
 
-	return update_option( 'give_completed_upgrades', $completed_upgrades );
+	return update_option('give_completed_upgrades', $completed_upgrades);
 }
 
 /**
@@ -263,9 +263,9 @@ 
  */
 function give_get_completed_upgrades() {
 
-	$completed_upgrades = get_option( 'give_completed_upgrades' );
+	$completed_upgrades = get_option('give_completed_upgrades');
 
-	if ( false === $completed_upgrades ) {
+	if (false === $completed_upgrades) {
 		$completed_upgrades = array();
 	}
 
@@ -282,31 +282,31 @@ 
  */
 function give_v132_upgrade_give_payment_customer_id() {
 	global $wpdb;
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	ignore_user_abort( true );
+	ignore_user_abort(true);
 
-	if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-		@set_time_limit( 0 );
+	if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+		@set_time_limit(0);
 	}
 
 	// UPDATE DB METAKEYS.
 	$sql   = "UPDATE $wpdb->postmeta SET meta_key = '_give_payment_customer_id' WHERE meta_key = '_give_payment_donor_id'";
-	$query = $wpdb->query( $sql );
+	$query = $wpdb->query($sql);
 
-	update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
-	give_set_upgrade_complete( 'upgrade_give_payment_customer_id' );
-	delete_option( 'give_doing_upgrade' );
-	wp_redirect( admin_url() );
+	update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
+	give_set_upgrade_complete('upgrade_give_payment_customer_id');
+	delete_option('give_doing_upgrade');
+	wp_redirect(admin_url());
 	exit;
 
 }
 
-add_action( 'give_upgrade_give_payment_customer_id', 'give_v132_upgrade_give_payment_customer_id' );
+add_action('give_upgrade_give_payment_customer_id', 'give_v132_upgrade_give_payment_customer_id');
 
 /**
  * Upgrades the Offline Status
@@ -319,16 +319,16 @@ 
 
 	global $wpdb;
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	ignore_user_abort( true );
+	ignore_user_abort(true);
 
-	if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-		@set_time_limit( 0 );
+	if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+		@set_time_limit(0);
 	}
 
 	// Get abandoned offline payments.
@@ -338,31 +338,31 @@ 
 	$where .= "AND ( p.post_status = 'abandoned' )";
 	$where .= "AND ( m.meta_key = '_give_payment_gateway' AND m.meta_value = 'offline' )";
 
-	$sql            = $select . $join . $where;
-	$found_payments = $wpdb->get_col( $sql );
+	$sql            = $select.$join.$where;
+	$found_payments = $wpdb->get_col($sql);
 
-	foreach ( $found_payments as $payment ) {
+	foreach ($found_payments as $payment) {
 
 		// Only change ones marked abandoned since our release last week because the admin may have marked some abandoned themselves.
-		$modified_time = get_post_modified_time( 'U', false, $payment );
+		$modified_time = get_post_modified_time('U', false, $payment);
 
 		// 1450124863 =  12/10/2015 20:42:25.
-		if ( $modified_time >= 1450124863 ) {
+		if ($modified_time >= 1450124863) {
 
-			give_update_payment_status( $payment, 'pending' );
+			give_update_payment_status($payment, 'pending');
 
 		}
 	}
 
-	update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
-	give_set_upgrade_complete( 'upgrade_give_offline_status' );
-	delete_option( 'give_doing_upgrade' );
-	wp_redirect( admin_url() );
+	update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
+	give_set_upgrade_complete('upgrade_give_offline_status');
+	delete_option('give_doing_upgrade');
+	wp_redirect(admin_url());
 	exit;
 
 }
 
-add_action( 'give_upgrade_give_offline_status', 'give_v134_upgrade_give_offline_status' );
+add_action('give_upgrade_give_offline_status', 'give_v134_upgrade_give_offline_status');
 
 /**
  * Cleanup User Roles
@@ -373,17 +373,17 @@ 
  */
 function give_v152_cleanup_users() {
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	$give_version = preg_replace( '/[^0-9.].*/', '', $give_version );
+	$give_version = preg_replace('/[^0-9.].*/', '', $give_version);
 
 	// v1.5.2 Upgrades
-	if ( version_compare( $give_version, '1.5.2', '<' ) || ! give_has_upgrade_completed( 'upgrade_give_user_caps_cleanup' ) ) {
+	if (version_compare($give_version, '1.5.2', '<') || ! give_has_upgrade_completed('upgrade_give_user_caps_cleanup')) {
 
 		// Delete all caps with "ss".
 		// Also delete all unused "campaign" roles.
@@ -430,9 +430,9 @@ 
 		);
 
 		global $wp_roles;
-		foreach ( $delete_caps as $cap ) {
-			foreach ( array_keys( $wp_roles->roles ) as $role ) {
-				$wp_roles->remove_cap( $role, $cap );
+		foreach ($delete_caps as $cap) {
+			foreach (array_keys($wp_roles->roles) as $role) {
+				$wp_roles->remove_cap($role, $cap);
 			}
 		}
 
@@ -442,15 +442,15 @@ 
 		$roles->add_caps();
 
 		// The Update Ran.
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
-		give_set_upgrade_complete( 'upgrade_give_user_caps_cleanup' );
-		delete_option( 'give_doing_upgrade' );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
+		give_set_upgrade_complete('upgrade_give_user_caps_cleanup');
+		delete_option('give_doing_upgrade');
 
 	}// End if().
 
 }
 
-add_action( 'admin_init', 'give_v152_cleanup_users' );
+add_action('admin_init', 'give_v152_cleanup_users');
 
 /**
  * 1.6 Upgrade routine to create the customer meta table.
@@ -490,53 +490,53 @@ 
 
 	// Get addons license key.
 	$addons = array();
-	foreach ( $give_options as $key => $value ) {
-		if ( false !== strpos( $key, '_license_key' ) ) {
-			$addons[ $key ] = $value;
+	foreach ($give_options as $key => $value) {
+		if (false !== strpos($key, '_license_key')) {
+			$addons[$key] = $value;
 		}
 	}
 
 	// Bailout: We do not have any addon license data to upgrade.
-	if ( empty( $addons ) ) {
+	if (empty($addons)) {
 		return false;
 	}
 
-	foreach ( $addons as $key => $addon_license ) {
+	foreach ($addons as $key => $addon_license) {
 
 		// Get addon shortname.
-		$shortname = str_replace( '_license_key', '', $key );
+		$shortname = str_replace('_license_key', '', $key);
 
 		// Addon license option name.
-		$addon_license_option_name = $shortname . '_license_active';
+		$addon_license_option_name = $shortname.'_license_active';
 
 		// bailout if license is empty.
-		if ( empty( $addon_license ) ) {
-			delete_option( $addon_license_option_name );
+		if (empty($addon_license)) {
+			delete_option($addon_license_option_name);
 			continue;
 		}
 
 		// Get addon name.
 		$addon_name       = array();
-		$addon_name_parts = explode( '_', str_replace( 'give_', '', $shortname ) );
-		foreach ( $addon_name_parts as $name_part ) {
+		$addon_name_parts = explode('_', str_replace('give_', '', $shortname));
+		foreach ($addon_name_parts as $name_part) {
 
 			// Fix addon name
-			switch ( $name_part ) {
+			switch ($name_part) {
 				case 'authorizenet' :
 					$name_part = 'authorize.net';
 					break;
 			}
 
-			$addon_name[] = ucfirst( $name_part );
+			$addon_name[] = ucfirst($name_part);
 		}
 
-		$addon_name = implode( ' ', $addon_name );
+		$addon_name = implode(' ', $addon_name);
 
 		// Data to send to the API
 		$api_params = array(
 			'edd_action' => 'activate_license', // never change from "edd_" to "give_"!
 			'license'    => $addon_license,
-			'item_name'  => urlencode( $addon_name ),
+			'item_name'  => urlencode($addon_name),
 			'url'        => home_url(),
 		);
 
@@ -551,17 +551,17 @@ 
 		);
 
 		// Make sure there are no errors.
-		if ( is_wp_error( $response ) ) {
-			delete_option( $addon_license_option_name );
+		if (is_wp_error($response)) {
+			delete_option($addon_license_option_name);
 			continue;
 		}
 
 		// Tell WordPress to look for updates.
-		set_site_transient( 'update_plugins', null );
+		set_site_transient('update_plugins', null);
 
 		// Decode license data.
-		$license_data = json_decode( wp_remote_retrieve_body( $response ) );
-		update_option( $addon_license_option_name, $license_data );
+		$license_data = json_decode(wp_remote_retrieve_body($response));
+		update_option($addon_license_option_name, $license_data);
 	}// End foreach().
 }
 
@@ -591,9 +591,9 @@ 
 	);
 
 	global $wp_roles;
-	foreach ( $delete_caps as $cap ) {
-		foreach ( array_keys( $wp_roles->roles ) as $role ) {
-			$wp_roles->remove_cap( $role, $cap );
+	foreach ($delete_caps as $cap) {
+		foreach (array_keys($wp_roles->roles) as $role) {
+			$wp_roles->remove_cap($role, $cap);
 		}
 	}
 
@@ -627,7 +627,7 @@ 
 function give_v18_upgrades_core_setting() {
 	// Core settings which changes from checkbox to radio.
 	$core_setting_names = array_merge(
-		array_keys( give_v18_renamed_core_settings() ),
+		array_keys(give_v18_renamed_core_settings()),
 		array(
 			'uninstall_on_delete',
 			'scripts_footer',
@@ -639,48 +639,48 @@ 
 	);
 
 	// Bailout: If not any setting define.
-	if ( $give_settings = get_option( 'give_settings' ) ) {
+	if ($give_settings = get_option('give_settings')) {
 
 		$setting_changed = false;
 
 		// Loop: check each setting field.
-		foreach ( $core_setting_names as $setting_name ) {
+		foreach ($core_setting_names as $setting_name) {
 			// New setting name.
-			$new_setting_name = preg_replace( '/^(enable_|disable_)/', '', $setting_name );
+			$new_setting_name = preg_replace('/^(enable_|disable_)/', '', $setting_name);
 
 			// Continue: If setting already set.
 			if (
-				array_key_exists( $new_setting_name, $give_settings )
-				&& in_array( $give_settings[ $new_setting_name ], array( 'enabled', 'disabled' ) )
+				array_key_exists($new_setting_name, $give_settings)
+				&& in_array($give_settings[$new_setting_name], array('enabled', 'disabled'))
 			) {
 				continue;
 			}
 
 			// Set checkbox value to radio value.
-			$give_settings[ $setting_name ] = ( ! empty( $give_settings[ $setting_name ] ) && 'on' === $give_settings[ $setting_name ] ? 'enabled' : 'disabled' );
+			$give_settings[$setting_name] = ( ! empty($give_settings[$setting_name]) && 'on' === $give_settings[$setting_name] ? 'enabled' : 'disabled');
 
 			// @see https://github.com/WordImpress/Give/issues/1063
-			if ( false !== strpos( $setting_name, 'disable_' ) ) {
+			if (false !== strpos($setting_name, 'disable_')) {
 
-				$give_settings[ $new_setting_name ] = ( give_is_setting_enabled( $give_settings[ $setting_name ] ) ? 'disabled' : 'enabled' );
-			} elseif ( false !== strpos( $setting_name, 'enable_' ) ) {
+				$give_settings[$new_setting_name] = (give_is_setting_enabled($give_settings[$setting_name]) ? 'disabled' : 'enabled');
+			} elseif (false !== strpos($setting_name, 'enable_')) {
 
-				$give_settings[ $new_setting_name ] = ( give_is_setting_enabled( $give_settings[ $setting_name ] ) ? 'enabled' : 'disabled' );
+				$give_settings[$new_setting_name] = (give_is_setting_enabled($give_settings[$setting_name]) ? 'enabled' : 'disabled');
 			}
 
 			// Tell bot to update core setting to db.
-			if ( ! $setting_changed ) {
+			if ( ! $setting_changed) {
 				$setting_changed = true;
 			}
 		}
 
 		// Update setting only if they changed.
-		if ( $setting_changed ) {
-			update_option( 'give_settings', $give_settings );
+		if ($setting_changed) {
+			update_option('give_settings', $give_settings);
 		}
 	}// End if().
 
-	give_set_upgrade_complete( 'v18_upgrades_core_setting' );
+	give_set_upgrade_complete('v18_upgrades_core_setting');
 }
 
 /**
@@ -690,22 +690,22 @@ 
  * @return void
  */
 function give_v18_upgrades_form_metadata() {
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	ignore_user_abort( true );
+	ignore_user_abort(true);
 
-	if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-		@set_time_limit( 0 );
+	if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+		@set_time_limit(0);
 	}
 
-	$step = isset( $_GET['step'] ) ? absint( $_GET['step'] ) : 1;
+	$step = isset($_GET['step']) ? absint($_GET['step']) : 1;
 
 	// form query
-	$forms = new WP_Query( array(
+	$forms = new WP_Query(array(
 			'paged'          => $step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -714,39 +714,39 @@ 
 		)
 	);
 
-	if ( $forms->have_posts() ) {
-		while ( $forms->have_posts() ) {
+	if ($forms->have_posts()) {
+		while ($forms->have_posts()) {
 			$forms->the_post();
 
 			// Form content.
 			// Note in version 1.8 display content setting split into display content and content placement setting.
 			// You can delete _give_content_option in future
-			$show_content = give_get_meta( get_the_ID(), '_give_content_option', true );
-			if ( $show_content && ! give_get_meta( get_the_ID(), '_give_display_content', true ) ) {
-				$field_value = ( 'none' !== $show_content ? 'enabled' : 'disabled' );
-				give_update_meta( get_the_ID(), '_give_display_content', $field_value );
+			$show_content = give_get_meta(get_the_ID(), '_give_content_option', true);
+			if ($show_content && ! give_get_meta(get_the_ID(), '_give_display_content', true)) {
+				$field_value = ('none' !== $show_content ? 'enabled' : 'disabled');
+				give_update_meta(get_the_ID(), '_give_display_content', $field_value);
 
-				$field_value = ( 'none' !== $show_content ? $show_content : 'give_pre_form' );
-				give_update_meta( get_the_ID(), '_give_content_placement', $field_value );
+				$field_value = ('none' !== $show_content ? $show_content : 'give_pre_form');
+				give_update_meta(get_the_ID(), '_give_content_placement', $field_value);
 			}
 
 			// "Disable" Guest Donation. Checkbox
 			// See: https://github.com/WordImpress/Give/issues/1470
-			$guest_donation = give_get_meta( get_the_ID(), '_give_logged_in_only', true );
-			$guest_donation_newval = ( in_array( $guest_donation, array( 'yes', 'on' ) ) ? 'disabled' : 'enabled' );
-			give_update_meta( get_the_ID(), '_give_logged_in_only', $guest_donation_newval );
+			$guest_donation = give_get_meta(get_the_ID(), '_give_logged_in_only', true);
+			$guest_donation_newval = (in_array($guest_donation, array('yes', 'on')) ? 'disabled' : 'enabled');
+			give_update_meta(get_the_ID(), '_give_logged_in_only', $guest_donation_newval);
 
 			// Offline Donations
 			// See: https://github.com/WordImpress/Give/issues/1579
-			$offline_donation = give_get_meta( get_the_ID(), '_give_customize_offline_donations', true );
-			if ( 'no' === $offline_donation ) {
+			$offline_donation = give_get_meta(get_the_ID(), '_give_customize_offline_donations', true);
+			if ('no' === $offline_donation) {
 				$offline_donation_newval = 'global';
-			} elseif ( 'yes' === $offline_donation ) {
+			} elseif ('yes' === $offline_donation) {
 				$offline_donation_newval = 'enabled';
 			} else {
 				$offline_donation_newval = 'disabled';
 			}
-			give_update_meta( get_the_ID(), '_give_customize_offline_donations', $offline_donation_newval );
+			give_update_meta(get_the_ID(), '_give_customize_offline_donations', $offline_donation_newval);
 
 			// Convert yes/no setting field to enabled/disabled.
 			$form_radio_settings = array(
@@ -766,15 +766,15 @@ 
 				'_give_offline_donation_enable_billing_fields_single',
 			);
 
-			foreach ( $form_radio_settings as $meta_key ) {
+			foreach ($form_radio_settings as $meta_key) {
 				// Get value.
-				$field_value = give_get_meta( get_the_ID(), $meta_key, true );
+				$field_value = give_get_meta(get_the_ID(), $meta_key, true);
 
 				// Convert meta value only if it is in yes/no/none.
-				if ( in_array( $field_value, array( 'yes', 'on', 'no', 'none' ) ) ) {
+				if (in_array($field_value, array('yes', 'on', 'no', 'none'))) {
 
-					$field_value = ( in_array( $field_value, array( 'yes', 'on' ) ) ? 'enabled' : 'disabled' );
-					give_update_meta( get_the_ID(), $meta_key, $field_value );
+					$field_value = (in_array($field_value, array('yes', 'on')) ? 'enabled' : 'disabled');
+					give_update_meta(get_the_ID(), $meta_key, $field_value);
 				}
 			}
 		}// End while().
@@ -782,27 +782,27 @@ 
 		wp_reset_postdata();
 
 		// Forms found so upgrade them
-		$step ++;
-		$redirect = add_query_arg( array(
+		$step++;
+		$redirect = add_query_arg(array(
 			'page'         => 'give-upgrades',
 			'give-upgrade' => 'give_v18_upgrades_form_metadata',
 			'step'         => $step,
-		), admin_url( 'index.php' ) );
-		wp_redirect( $redirect );
+		), admin_url('index.php'));
+		wp_redirect($redirect);
 		exit();
 
 	} else {
 		// No more forms found, finish up.
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
-		delete_option( 'give_doing_upgrade' );
-		give_set_upgrade_complete( 'v18_upgrades_form_metadata' );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
+		delete_option('give_doing_upgrade');
+		give_set_upgrade_complete('v18_upgrades_form_metadata');
 
-		wp_redirect( admin_url() );
+		wp_redirect(admin_url());
 		exit;
 	}
 }
 
-add_action( 'give_give_v18_upgrades_form_metadata', 'give_v18_upgrades_form_metadata' );
+add_action('give_give_v18_upgrades_form_metadata', 'give_v18_upgrades_form_metadata');
 
 /**
  * Get list of core setting renamed in version 1.8.
@@ -835,7 +835,7 @@ 
  * @since  1.8.7
  * @return void
  */
-function give_v187_upgrades(){
+function give_v187_upgrades() {
 	global $wpdb;
 
 	/**
@@ -875,24 +875,24 @@ 
 		ARRAY_A
 	);
 
-	if( ! empty( $user_apikey_options ) ) {
-		foreach ( $user_apikey_options as $user ) {
-			$cached_options[] = '_transient_' . md5( 'give_api_user_' . $user['meta_key'] );
-			$cached_options[] = '_transient_' . md5( 'give_api_user_public_key' . $user['user_id'] );
-			$cached_options[] = '_transient_' . md5( 'give_api_user_secret_key' . $user['user_id'] );
+	if ( ! empty($user_apikey_options)) {
+		foreach ($user_apikey_options as $user) {
+			$cached_options[] = '_transient_'.md5('give_api_user_'.$user['meta_key']);
+			$cached_options[] = '_transient_'.md5('give_api_user_public_key'.$user['user_id']);
+			$cached_options[] = '_transient_'.md5('give_api_user_secret_key'.$user['user_id']);
 		}
 	}
 
-	if ( ! empty( $cached_options ) ) {
-		foreach ( $cached_options as $option ) {
-			switch ( true ) {
-				case ( false !== strpos( $option, 'transient' ) ):
-					$option = str_replace( '_transient_', '', $option );
-					delete_transient( $option );
+	if ( ! empty($cached_options)) {
+		foreach ($cached_options as $option) {
+			switch (true) {
+				case (false !== strpos($option, 'transient')):
+					$option = str_replace('_transient_', '', $option);
+					delete_transient($option);
 					break;
 
 				default:
-					delete_option( $option );
+					delete_option($option);
 			}
 		}
 	}
@@ -910,7 +910,7 @@ 
 	global $wp_roles;
 
     // Get the role object.
-    $give_worker = get_role( 'give_worker' );
+    $give_worker = get_role('give_worker');
 
 	// A list of capabilities to add for give workers.
     $caps_to_add = array(
@@ -918,9 +918,9 @@ 
         'edit_pages'
     );
 
-    foreach ( $caps_to_add as $cap ) {
+    foreach ($caps_to_add as $cap) {
         // Add the capability.
-        $give_worker->add_cap( $cap );
+        $give_worker->add_cap($cap);
     }
 
 }

includes/admin/forms/dashboard-columns.php

Spacing +69 added lines, -69 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -27,34 +27,34 @@ 
  * @return array $form_columns Updated array of forms columns
  *  Post Type List Table
  */
-function give_form_columns( $give_form_columns ) {
+function give_form_columns($give_form_columns) {
 
 	// Standard columns
 	$give_form_columns = array(
 		'cb'            => '<input type="checkbox"/>',
-		'title'         => esc_html__( 'Name', 'give' ),
-		'form_category' => esc_html__( 'Categories', 'give' ),
-		'form_tag'      => esc_html__( 'Tags', 'give' ),
-		'price'         => esc_html__( 'Amount', 'give' ),
-		'goal'          => esc_html__( 'Goal', 'give' ),
-		'donations'     => esc_html__( 'Donations', 'give' ),
-		'earnings'      => esc_html__( 'Income', 'give' ),
-		'shortcode'     => esc_html__( 'Shortcode', 'give' ),
-		'date'          => esc_html__( 'Date', 'give' ),
+		'title'         => esc_html__('Name', 'give'),
+		'form_category' => esc_html__('Categories', 'give'),
+		'form_tag'      => esc_html__('Tags', 'give'),
+		'price'         => esc_html__('Amount', 'give'),
+		'goal'          => esc_html__('Goal', 'give'),
+		'donations'     => esc_html__('Donations', 'give'),
+		'earnings'      => esc_html__('Income', 'give'),
+		'shortcode'     => esc_html__('Shortcode', 'give'),
+		'date'          => esc_html__('Date', 'give'),
 	);
 
 	// Does the user want categories / tags?
-	if ( ! give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) ) ) {
-		unset( $give_form_columns['form_category'] );
+	if ( ! give_is_setting_enabled(give_get_option('categories', 'disabled'))) {
+		unset($give_form_columns['form_category']);
 	}
-	if ( ! give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) ) ) {
-		unset( $give_form_columns['form_tag'] );
+	if ( ! give_is_setting_enabled(give_get_option('tags', 'disabled'))) {
+		unset($give_form_columns['form_tag']);
 	}
 
-	return apply_filters( 'give_forms_columns', $give_form_columns );
+	return apply_filters('give_forms_columns', $give_form_columns);
 }
 
-add_filter( 'manage_edit-give_forms_columns', 'give_form_columns' );
+add_filter('manage_edit-give_forms_columns', 'give_form_columns');
 
 /**
  * Render Give Form Columns
@@ -66,59 +66,59 @@ 
  *
  * @return void
  */
-function give_render_form_columns( $column_name, $post_id ) {
-	if ( get_post_type( $post_id ) == 'give_forms' ) {
+function give_render_form_columns($column_name, $post_id) {
+	if (get_post_type($post_id) == 'give_forms') {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'form_category':
-				echo get_the_term_list( $post_id, 'give_forms_category', '', ', ', '' );
+				echo get_the_term_list($post_id, 'give_forms_category', '', ', ', '');
 				break;
 			case 'form_tag':
-				echo get_the_term_list( $post_id, 'give_forms_tag', '', ', ', '' );
+				echo get_the_term_list($post_id, 'give_forms_tag', '', ', ', '');
 				break;
 			case 'price':
-				if ( give_has_variable_prices( $post_id ) ) {
-					echo give_price_range( $post_id );
+				if (give_has_variable_prices($post_id)) {
+					echo give_price_range($post_id);
 				} else {
-					echo give_price( $post_id, false );
-					echo '<input type="hidden" class="formprice-' . $post_id . '" value="' . give_get_form_price( $post_id ) . '" />';
+					echo give_price($post_id, false);
+					echo '<input type="hidden" class="formprice-'.$post_id.'" value="'.give_get_form_price($post_id).'" />';
 				}
 				break;
 			case 'goal':
-				if ( give_is_setting_enabled( give_get_meta( $post_id, '_give_goal_option', true ) ) ) {
-					echo give_goal( $post_id, false );
+				if (give_is_setting_enabled(give_get_meta($post_id, '_give_goal_option', true))) {
+					echo give_goal($post_id, false);
 				} else {
-					esc_html_e( 'No Goal Set', 'give' );
+					esc_html_e('No Goal Set', 'give');
 				}
 
-				echo '<input type="hidden" class="formgoal-' . $post_id . '" value="' . give_get_form_goal( $post_id ) . '" />';
+				echo '<input type="hidden" class="formgoal-'.$post_id.'" value="'.give_get_form_goal($post_id).'" />';
 				break;
 			case 'donations':
-				if ( current_user_can( 'view_give_form_stats', $post_id ) ) {
-					echo '<a href="' . esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-tools&tab=logs&form=' . $post_id ) ) . '">';
-					echo give_get_form_sales_stats( $post_id );
+				if (current_user_can('view_give_form_stats', $post_id)) {
+					echo '<a href="'.esc_url(admin_url('edit.php?post_type=give_forms&page=give-tools&tab=logs&form='.$post_id)).'">';
+					echo give_get_form_sales_stats($post_id);
 					echo '</a>';
 				} else {
 					echo '-';
 				}
 				break;
 			case 'earnings':
-				if ( current_user_can( 'view_give_form_stats', $post_id ) ) {
-					echo '<a href="' . esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-reports&view=forms&form-id=' . $post_id ) ) . '">';
-					echo give_currency_filter( give_format_amount( give_get_form_earnings_stats( $post_id ) ) );
+				if (current_user_can('view_give_form_stats', $post_id)) {
+					echo '<a href="'.esc_url(admin_url('edit.php?post_type=give_forms&page=give-reports&view=forms&form-id='.$post_id)).'">';
+					echo give_currency_filter(give_format_amount(give_get_form_earnings_stats($post_id)));
 					echo '</a>';
 				} else {
 					echo '-';
 				}
 				break;
 			case 'shortcode':
-				echo '<input onclick="this.setSelectionRange(0, this.value.length)" type="text" class="shortcode-input" readonly="" value="[give_form id=&#34;' . absint( $post_id ) . '&#34;]">';
+				echo '<input onclick="this.setSelectionRange(0, this.value.length)" type="text" class="shortcode-input" readonly="" value="[give_form id=&#34;'.absint($post_id).'&#34;]">';
 				break;
 		}
 	}
 }
 
-add_action( 'manage_posts_custom_column', 'give_render_form_columns', 10, 2 );
+add_action('manage_posts_custom_column', 'give_render_form_columns', 10, 2);
 
 /**
  * Registers the sortable columns in the list table
@@ -129,7 +129,7 @@ 
  *
  * @return array $columns Array of sortable columns
  */
-function give_sortable_form_columns( $columns ) {
+function give_sortable_form_columns($columns) {
 	$columns['price']     = 'amount';
 	$columns['sales']     = 'sales';
 	$columns['earnings']  = 'earnings';
@@ -139,7 +139,7 @@ 
 	return $columns;
 }
 
-add_filter( 'manage_edit-give_forms_sortable_columns', 'give_sortable_form_columns' );
+add_filter('manage_edit-give_forms_sortable_columns', 'give_sortable_form_columns');
 
 /**
  * Sorts Columns in the Forms List Table
@@ -150,13 +150,13 @@ 
  *
  * @return array $vars Array of all the sort variables
  */
-function give_sort_forms( $vars ) {
+function give_sort_forms($vars) {
 	// Check if we're viewing the "give_forms" post type
-	if ( ! isset( $vars['post_type'] ) || ! isset( $vars['orderby'] ) || 'give_forms' !== $vars['post_type'] ) {
+	if ( ! isset($vars['post_type']) || ! isset($vars['orderby']) || 'give_forms' !== $vars['post_type']) {
 		return $vars;
 	}
 
-	switch ( $vars['orderby'] ) {
+	switch ($vars['orderby']) {
 		// Check if 'orderby' is set to "sales"
 		case 'sales':
 			$vars = array_merge(
@@ -225,16 +225,16 @@ 
  *
  * @return array       Array of all sort variables
  */
-function give_filter_forms( $vars ) {
-	if ( isset( $vars['post_type'] ) && 'give_forms' == $vars['post_type'] ) {
+function give_filter_forms($vars) {
+	if (isset($vars['post_type']) && 'give_forms' == $vars['post_type']) {
 
 		// If an author ID was passed, use it
-		if ( isset( $_REQUEST['author'] ) && ! current_user_can( 'view_give_reports' ) ) {
+		if (isset($_REQUEST['author']) && ! current_user_can('view_give_reports')) {
 
 			$author_id = $_REQUEST['author'];
-			if ( (int) $author_id !== get_current_user_id() ) {
+			if ((int) $author_id !== get_current_user_id()) {
 				// Tried to view the products of another person, sorry
-				wp_die( esc_html__( 'You do not have permission to view this data.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(esc_html__('You do not have permission to view this data.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 			}
 			$vars = array_merge(
 				$vars,
@@ -258,11 +258,11 @@ 
  * @return void
  */
 function give_forms_load() {
-	add_filter( 'request', 'give_sort_forms' );
-	add_filter( 'request', 'give_filter_forms' );
+	add_filter('request', 'give_sort_forms');
+	add_filter('request', 'give_filter_forms');
 }
 
-add_action( 'load-edit.php', 'give_forms_load', 9999 );
+add_action('load-edit.php', 'give_forms_load', 9999);
 
 /**
  * Remove Forms Month Filter
@@ -276,17 +276,17 @@ 
  * @global      $typenow The post type we are viewing
  * @return array Empty array disables the dropdown
  */
-function give_remove_month_filter( $dates ) {
+function give_remove_month_filter($dates) {
 	global $typenow;
 
-	if ( $typenow == 'give_forms' ) {
+	if ($typenow == 'give_forms') {
 		$dates = array();
 	}
 
 	return $dates;
 }
 
-add_filter( 'months_dropdown_results', 'give_remove_month_filter', 99 );
+add_filter('months_dropdown_results', 'give_remove_month_filter', 99);
 
 /**
  * Updates price when saving post
@@ -297,23 +297,23 @@ 
  *
  * @return int|null
  */
-function give_price_save_quick_edit( $post_id ) {
-	if ( ! isset( $_POST['post_type'] ) || 'give_forms' !== $_POST['post_type'] ) {
+function give_price_save_quick_edit($post_id) {
+	if ( ! isset($_POST['post_type']) || 'give_forms' !== $_POST['post_type']) {
 		return;
 	}
-	if ( ! current_user_can( 'edit_post', $post_id ) ) {
+	if ( ! current_user_can('edit_post', $post_id)) {
 		return $post_id;
 	}
-	if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
+	if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) {
 		return $post_id;
 	}
 
-	if ( isset( $_REQUEST['_give_regprice'] ) ) {
-		give_update_meta( $post_id, '_give_set_price', strip_tags( stripslashes( $_REQUEST['_give_regprice'] ) ) );
+	if (isset($_REQUEST['_give_regprice'])) {
+		give_update_meta($post_id, '_give_set_price', strip_tags(stripslashes($_REQUEST['_give_regprice'])));
 	}
 }
 
-add_action( 'save_post', 'give_price_save_quick_edit' );
+add_action('save_post', 'give_price_save_quick_edit');
 
 /**
  * Process bulk edit actions via AJAX
@@ -323,18 +323,18 @@ 
  */
 function give_save_bulk_edit() {
 
-	$post_ids = ( isset( $_POST['post_ids'] ) && ! empty( $_POST['post_ids'] ) ) ? $_POST['post_ids'] : array();
+	$post_ids = (isset($_POST['post_ids']) && ! empty($_POST['post_ids'])) ? $_POST['post_ids'] : array();
 
-	if ( ! empty( $post_ids ) && is_array( $post_ids ) ) {
-		$price = isset( $_POST['price'] ) ? strip_tags( stripslashes( $_POST['price'] ) ) : 0;
-		foreach ( $post_ids as $post_id ) {
+	if ( ! empty($post_ids) && is_array($post_ids)) {
+		$price = isset($_POST['price']) ? strip_tags(stripslashes($_POST['price'])) : 0;
+		foreach ($post_ids as $post_id) {
 
-			if ( ! current_user_can( 'edit_post', $post_id ) ) {
+			if ( ! current_user_can('edit_post', $post_id)) {
 				continue;
 			}
 
-			if ( ! empty( $price ) ) {
-				give_update_meta( $post_id, '_give_set_price', give_sanitize_amount( $price ) );
+			if ( ! empty($price)) {
+				give_update_meta($post_id, '_give_set_price', give_sanitize_amount($price));
 			}
 		}
 	}
@@ -342,4 +342,4 @@ 
 	die();
 }
 
-add_action( 'wp_ajax_give_save_bulk_edit', 'give_save_bulk_edit' );
+add_action('wp_ajax_give_save_bulk_edit', 'give_save_bulk_edit');

includes/admin/forms/class-metabox-form-data.php

Spacing +290 added lines, -290 removed lines

@@ -46,21 +46,21 @@ 
 	 */
 	function __construct() {
 		$this->metabox_id    = 'give-metabox-form-data';
-		$this->metabox_label = __( 'Donation Form Options', 'give' );
+		$this->metabox_label = __('Donation Form Options', 'give');
 
 		// Setup.
-		add_action( 'admin_init', array( $this, 'setup' ) );
+		add_action('admin_init', array($this, 'setup'));
 
 		// Add metabox.
-		add_action( 'add_meta_boxes', array( $this, 'add_meta_box' ), 10 );
+		add_action('add_meta_boxes', array($this, 'add_meta_box'), 10);
 
 		// Save form meta.
-		add_action( 'save_post_give_forms', array( $this, 'save' ), 10, 2 );
+		add_action('save_post_give_forms', array($this, 'save'), 10, 2);
 
 		// cmb2 old setting loaders.
 		// add_filter( 'give_metabox_form_data_settings', array( $this, 'cmb2_metabox_settings' ) );
 		// Add offline donations options.
-		add_filter( 'give_metabox_form_data_settings', array( $this, 'add_offline_donations_setting_tab' ), 0, 1 );
+		add_filter('give_metabox_form_data_settings', array($this, 'add_offline_donations_setting_tab'), 0, 1);
 	}
 
 
@@ -83,18 +83,18 @@ 
 	 */
 	function get_settings() {
 		$post_id               = give_get_admin_post_id();
-		$price                 = give_get_form_price( $post_id );
-		$custom_amount_minimum = give_get_form_minimum_price( $post_id );
-		$goal                  = give_get_form_goal( $post_id );
+		$price                 = give_get_form_price($post_id);
+		$custom_amount_minimum = give_get_form_minimum_price($post_id);
+		$goal                  = give_get_form_goal($post_id);
 
 		// No empty prices - min. 1.00 for new forms
-		if ( empty( $price ) && is_null( $post_id ) ) {
-			$price = esc_attr( give_format_decimal( '1.00' ) );
+		if (empty($price) && is_null($post_id)) {
+			$price = esc_attr(give_format_decimal('1.00'));
 		}
 
 		// Min. $1.00 for new forms
-		if ( empty( $custom_amount_minimum ) ) {
-			$custom_amount_minimum = esc_attr( give_format_decimal( '1.00' ) );
+		if (empty($custom_amount_minimum)) {
+			$custom_amount_minimum = esc_attr(give_format_decimal('1.00'));
 		}
 
 		// Start with an underscore to hide fields from custom fields list
@@ -104,215 +104,215 @@ 
 			/**
 			 * Repeatable Field Groups
 			 */
-			'form_field_options'    => apply_filters( 'give_forms_field_options', array(
+			'form_field_options'    => apply_filters('give_forms_field_options', array(
 				'id'        => 'form_field_options',
-				'title'     => __( 'Donation Options', 'give' ),
+				'title'     => __('Donation Options', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-heart"></span>',
-				'fields'    => apply_filters( 'give_forms_donation_form_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_donation_form_metabox_fields', array(
 					// Donation Option
 					array(
-						'name'        => __( 'Donation Option', 'give' ),
-						'description' => __( 'Do you want this form to have one set donation price or multiple levels (for example, $10, $20, $50)?', 'give' ),
-						'id'          => $prefix . 'price_option',
+						'name'        => __('Donation Option', 'give'),
+						'description' => __('Do you want this form to have one set donation price or multiple levels (for example, $10, $20, $50)?', 'give'),
+						'id'          => $prefix.'price_option',
 						'type'        => 'radio_inline',
 						'default'     => 'set',
-						'options'     => apply_filters( 'give_forms_price_options', array(
-							'set'   => __( 'Set Donation', 'give' ),
-							'multi' => __( 'Multi-level Donation', 'give' ),
-						) ),
+						'options'     => apply_filters('give_forms_price_options', array(
+							'set'   => __('Set Donation', 'give'),
+							'multi' => __('Multi-level Donation', 'give'),
+						)),
 					),
 					array(
-						'name'        => __( 'Set Donation', 'give' ),
-						'description' => __( 'This is the set donation amount for this form. If you have a "Custom Amount Minimum" set, make sure it is less than this amount.', 'give' ),
-						'id'          => $prefix . 'set_price',
+						'name'        => __('Set Donation', 'give'),
+						'description' => __('This is the set donation amount for this form. If you have a "Custom Amount Minimum" set, make sure it is less than this amount.', 'give'),
+						'id'          => $prefix.'set_price',
 						'type'        => 'text_small',
 						'data_type'   => 'price',
 						'attributes'  => array(
-							'placeholder' => give_format_decimal( '1.00' ),
-							'value'       => give_format_decimal( $price ),
+							'placeholder' => give_format_decimal('1.00'),
+							'value'       => give_format_decimal($price),
 							'class'       => 'give-money-field',
 						),
 					),
 					// Display Style
 					array(
-						'name'        => __( 'Display Style', 'give' ),
-						'description' => __( 'Set how the donations levels will display on the form.', 'give' ),
-						'id'          => $prefix . 'display_style',
+						'name'        => __('Display Style', 'give'),
+						'description' => __('Set how the donations levels will display on the form.', 'give'),
+						'id'          => $prefix.'display_style',
 						'type'        => 'radio_inline',
 						'default'     => 'buttons',
 						'options'     => array(
-							'buttons'  => __( 'Buttons', 'give' ),
-							'radios'   => __( 'Radios', 'give' ),
-							'dropdown' => __( 'Dropdown', 'give' ),
+							'buttons'  => __('Buttons', 'give'),
+							'radios'   => __('Radios', 'give'),
+							'dropdown' => __('Dropdown', 'give'),
 						),
 					),
 					// Custom Amount
 					array(
-						'name'        => __( 'Custom Amount', 'give' ),
-						'description' => __( 'Do you want the user to be able to input their own donation amount?', 'give' ),
-						'id'          => $prefix . 'custom_amount',
+						'name'        => __('Custom Amount', 'give'),
+						'description' => __('Do you want the user to be able to input their own donation amount?', 'give'),
+						'id'          => $prefix.'custom_amount',
 						'type'        => 'radio_inline',
 						'default'     => 'disabled',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'        => __( 'Minimum Amount', 'give' ),
-						'description' => __( 'Enter the minimum custom donation amount.', 'give' ),
-						'id'          => $prefix . 'custom_amount_minimum',
+						'name'        => __('Minimum Amount', 'give'),
+						'description' => __('Enter the minimum custom donation amount.', 'give'),
+						'id'          => $prefix.'custom_amount_minimum',
 						'type'        => 'text_small',
 						'data_type'   => 'price',
 						'attributes'  => array(
-							'placeholder' => give_format_decimal( '1.00' ),
-							'value'       => give_format_decimal( $custom_amount_minimum ),
+							'placeholder' => give_format_decimal('1.00'),
+							'value'       => give_format_decimal($custom_amount_minimum),
 							'class'       => 'give-money-field',
 						),
 					),
 					array(
-						'name'        => __( 'Custom Amount Text', 'give' ),
-						'description' => __( 'This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give' ),
-						'id'          => $prefix . 'custom_amount_text',
+						'name'        => __('Custom Amount Text', 'give'),
+						'description' => __('This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give'),
+						'id'          => $prefix.'custom_amount_text',
 						'type'        => 'text_medium',
 						'attributes'  => array(
 							'rows'        => 3,
-							'placeholder' => esc_attr__( 'Give a Custom Amount', 'give' ),
+							'placeholder' => esc_attr__('Give a Custom Amount', 'give'),
 						),
 					),
 					// Donation Levels: Repeatable CMB2 Group
 					array(
-						'id'      => $prefix . 'donation_levels',
+						'id'      => $prefix.'donation_levels',
 						'type'    => 'group',
 						'options' => array(
-							'add_button'    => __( 'Add Level', 'give' ),
-							'header_title'  => __( 'Donation Level', 'give' ),
+							'add_button'    => __('Add Level', 'give'),
+							'header_title'  => __('Donation Level', 'give'),
 							'remove_button' => '<span class="dashicons dashicons-no"></span>',
 						),
 						// Fields array works the same, except id's only need to be unique for this group.
 						// Prefix is not needed.
-						'fields'  => apply_filters( 'give_donation_levels_table_row', array(
+						'fields'  => apply_filters('give_donation_levels_table_row', array(
 							array(
-								'name' => __( 'ID', 'give' ),
-								'id'   => $prefix . 'id',
+								'name' => __('ID', 'give'),
+								'id'   => $prefix.'id',
 								'type' => 'levels_id',
 							),
 							array(
-								'name'       => __( 'Amount', 'give' ),
-								'id'         => $prefix . 'amount',
+								'name'       => __('Amount', 'give'),
+								'id'         => $prefix.'amount',
 								'type'       => 'text_small',
 								'data_type'  => 'price',
 								'attributes' => array(
-									'placeholder' => give_format_decimal( '1.00' ),
+									'placeholder' => give_format_decimal('1.00'),
 									'class'       => 'give-money-field',
 								),
 							),
 							array(
-								'name'       => __( 'Text', 'give' ),
-								'id'         => $prefix . 'text',
+								'name'       => __('Text', 'give'),
+								'id'         => $prefix.'text',
 								'type'       => 'text',
 								'attributes' => array(
-									'placeholder' => __( 'Donation Level', 'give' ),
+									'placeholder' => __('Donation Level', 'give'),
 									'class'       => 'give-multilevel-text-field',
 								),
 							),
 							array(
-								'name' => __( 'Default', 'give' ),
-								'id'   => $prefix . 'default',
+								'name' => __('Default', 'give'),
+								'id'   => $prefix.'default',
 								'type' => 'give_default_radio_inline',
 							),
-						) ),
+						)),
 					),
 					array(
 						'name'  => 'donation_options_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-donation-options',
-						'title' => __( 'Donation Options', 'give' ),
+						'title' => __('Donation Options', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Display Options
 			 */
-			'form_display_options'  => apply_filters( 'give_form_display_options', array(
+			'form_display_options'  => apply_filters('give_form_display_options', array(
 					'id'        => 'form_display_options',
-					'title'     => __( 'Form Display', 'give' ),
+					'title'     => __('Form Display', 'give'),
 					'icon-html' => '<span class="give-icon give-icon-display"></span>',
-					'fields'    => apply_filters( 'give_forms_display_options_metabox_fields', array(
+					'fields'    => apply_filters('give_forms_display_options_metabox_fields', array(
 						array(
-							'name'    => __( 'Display Options', 'give' ),
-							'desc'    => sprintf( __( 'How would you like to display donation information for this form?', 'give' ), '#' ),
-							'id'      => $prefix . 'payment_display',
+							'name'    => __('Display Options', 'give'),
+							'desc'    => sprintf(__('How would you like to display donation information for this form?', 'give'), '#'),
+							'id'      => $prefix.'payment_display',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'onpage' => __( 'All Fields', 'give' ),
-								'modal'  => __( 'Modal', 'give' ),
-								'reveal' => __( 'Reveal', 'give' ),
-								'button' => __( 'Button', 'give' ),
+								'onpage' => __('All Fields', 'give'),
+								'modal'  => __('Modal', 'give'),
+								'reveal' => __('Reveal', 'give'),
+								'button' => __('Button', 'give'),
 							),
 							'default' => 'onpage',
 						),
 						array(
-							'id'         => $prefix . 'reveal_label',
-							'name'       => __( 'Continue Button', 'give' ),
-							'desc'       => __( 'The button label for displaying the additional payment fields.', 'give' ),
+							'id'         => $prefix.'reveal_label',
+							'name'       => __('Continue Button', 'give'),
+							'desc'       => __('The button label for displaying the additional payment fields.', 'give'),
 							'type'       => 'text_small',
 							'attributes' => array(
-								'placeholder' => esc_attr__( 'Donate Now', 'give' ),
+								'placeholder' => esc_attr__('Donate Now', 'give'),
 							),
 						),
 						array(
-							'id'         => $prefix . 'checkout_label',
-							'name'       => __( 'Submit Button', 'give' ),
-							'desc'       => __( 'The button label for completing a donation.', 'give' ),
+							'id'         => $prefix.'checkout_label',
+							'name'       => __('Submit Button', 'give'),
+							'desc'       => __('The button label for completing a donation.', 'give'),
 							'type'       => 'text_small',
 							'attributes' => array(
-								'placeholder' => __( 'Donate Now', 'give' ),
+								'placeholder' => __('Donate Now', 'give'),
 							),
 						),
 						array(
-							'name' => __( 'Default Gateway', 'give' ),
-							'desc' => __( 'By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give' ),
-							'id'   => $prefix . 'default_gateway',
+							'name' => __('Default Gateway', 'give'),
+							'desc' => __('By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give'),
+							'id'   => $prefix.'default_gateway',
 							'type' => 'default_gateway',
 						),
 						array(
-							'name'    => __( 'Guest Donations', 'give' ),
-							'desc'    => __( 'Do you want to allow non-logged-in users to make donations?', 'give' ),
-							'id'      => $prefix . 'logged_in_only',
+							'name'    => __('Guest Donations', 'give'),
+							'desc'    => __('Do you want to allow non-logged-in users to make donations?', 'give'),
+							'id'      => $prefix.'logged_in_only',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Registration', 'give' ),
-							'desc'    => __( 'Display the registration and login forms in the payment section for non-logged-in users.', 'give' ),
-							'id'      => $prefix . 'show_register_form',
+							'name'    => __('Registration', 'give'),
+							'desc'    => __('Display the registration and login forms in the payment section for non-logged-in users.', 'give'),
+							'id'      => $prefix.'show_register_form',
 							'type'    => 'radio',
 							'options' => array(
-								'none'         => __( 'None', 'give' ),
-								'registration' => __( 'Registration', 'give' ),
-								'login'        => __( 'Login', 'give' ),
-								'both'         => __( 'Registration + Login', 'give' ),
+								'none'         => __('None', 'give'),
+								'registration' => __('Registration', 'give'),
+								'login'        => __('Login', 'give'),
+								'both'         => __('Registration + Login', 'give'),
 							),
 							'default' => 'none',
 						),
 						array(
-							'name'    => __( 'Floating Labels', 'give' ),
+							'name'    => __('Floating Labels', 'give'),
 							/* translators: %s: forms http://docs.givewp.com/form-floating-labels */
-							'desc'    => sprintf( __( 'Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form. Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give' ), esc_url( 'http://docs.givewp.com/form-floating-labels' ) ),
-							'id'      => $prefix . 'form_floating_labels',
+							'desc'    => sprintf(__('Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form. Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give'), esc_url('http://docs.givewp.com/form-floating-labels')),
+							'id'      => $prefix.'form_floating_labels',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'global'   => __( 'Global Option', 'give' ),
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'global'   => __('Global Option', 'give'),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 							'default' => 'global',
 						),
@@ -320,7 +320,7 @@ 
 							'name'  => 'form_display_docs',
 							'type'  => 'docs_link',
 							'url'   => 'http://docs.givewp.com/form-display-options',
-							'title' => __( 'Form Display', 'give' ),
+							'title' => __('Form Display', 'give'),
 						),
 					),
 						$post_id
@@ -331,172 +331,172 @@ 
 			/**
 			 * Donation Goals
 			 */
-			'donation_goal_options' => apply_filters( 'give_donation_goal_options', array(
+			'donation_goal_options' => apply_filters('give_donation_goal_options', array(
 				'id'        => 'donation_goal_options',
-				'title'     => __( 'Donation Goal', 'give' ),
+				'title'     => __('Donation Goal', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-target"></span>',
-				'fields'    => apply_filters( 'give_forms_donation_goal_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_donation_goal_metabox_fields', array(
 					// Goals
 					array(
-						'name'        => __( 'Donation Goal', 'give' ),
-						'description' => __( 'Do you want to set a donation goal for this form?', 'give' ),
-						'id'          => $prefix . 'goal_option',
+						'name'        => __('Donation Goal', 'give'),
+						'description' => __('Do you want to set a donation goal for this form?', 'give'),
+						'id'          => $prefix.'goal_option',
 						'type'        => 'radio_inline',
 						'default'     => 'disabled',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'        => __( 'Goal Amount', 'give' ),
-						'description' => __( 'This is the monetary goal amount you want to reach for this form.', 'give' ),
-						'id'          => $prefix . 'set_goal',
+						'name'        => __('Goal Amount', 'give'),
+						'description' => __('This is the monetary goal amount you want to reach for this form.', 'give'),
+						'id'          => $prefix.'set_goal',
 						'type'        => 'text_small',
 						'data_type'   => 'price',
 						'attributes'  => array(
-							'placeholder' => give_format_decimal( '0.00' ),
-							'value'       => give_format_decimal( $goal ),
+							'placeholder' => give_format_decimal('0.00'),
+							'value'       => give_format_decimal($goal),
 							'class'       => 'give-money-field',
 						),
 					),
 
 					array(
-						'name'        => __( 'Goal Format', 'give' ),
-						'description' => __( 'Do you want to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give' ),
-						'id'          => $prefix . 'goal_format',
+						'name'        => __('Goal Format', 'give'),
+						'description' => __('Do you want to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give'),
+						'id'          => $prefix.'goal_format',
 						'type'        => 'radio_inline',
 						'default'     => 'amount',
 						'options'     => array(
-							'amount'     => __( 'Amount', 'give' ),
-							'percentage' => __( 'Percentage', 'give' ),
+							'amount'     => __('Amount', 'give'),
+							'percentage' => __('Percentage', 'give'),
 						),
 					),
 					array(
-						'name'    => __( 'Progress Bar Color', 'give' ),
-						'desc'    => __( 'Customize the color of the goal progress bar.', 'give' ),
-						'id'      => $prefix . 'goal_color',
+						'name'    => __('Progress Bar Color', 'give'),
+						'desc'    => __('Customize the color of the goal progress bar.', 'give'),
+						'id'      => $prefix.'goal_color',
 						'type'    => 'colorpicker',
 						'default' => '#2bc253',
 					),
 
 					array(
-						'name'    => __( 'Close Form', 'give' ),
-						'desc'    => __( 'Do you want to close the donation forms and stop accepting donations once this goal has been met?', 'give' ),
-						'id'      => $prefix . 'close_form_when_goal_achieved',
+						'name'    => __('Close Form', 'give'),
+						'desc'    => __('Do you want to close the donation forms and stop accepting donations once this goal has been met?', 'give'),
+						'id'      => $prefix.'close_form_when_goal_achieved',
 						'type'    => 'radio_inline',
 						'default' => 'disabled',
 						'options' => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'       => __( 'Goal Achieved Message', 'give' ),
-						'desc'       => __( 'Do you want to display a custom message when the goal is closed?', 'give' ),
-						'id'         => $prefix . 'form_goal_achieved_message',
+						'name'       => __('Goal Achieved Message', 'give'),
+						'desc'       => __('Do you want to display a custom message when the goal is closed?', 'give'),
+						'id'         => $prefix.'form_goal_achieved_message',
 						'type'       => 'wysiwyg',
-                        'default' => __( 'Thank you to all our donors, we have met our fundraising goal.', 'give' ),
+                        'default' => __('Thank you to all our donors, we have met our fundraising goal.', 'give'),
 					),
 					array(
 						'name'  => 'donation_goal_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-donation-goal',
-						'title' => __( 'Donation Goal', 'give' ),
+						'title' => __('Donation Goal', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Content Field
 			 */
-			'form_content_options'  => apply_filters( 'give_forms_content_options', array(
+			'form_content_options'  => apply_filters('give_forms_content_options', array(
 				'id'        => 'form_content_options',
-				'title'     => __( 'Form Content', 'give' ),
+				'title'     => __('Form Content', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-edit"></span>',
-				'fields'    => apply_filters( 'give_forms_content_options_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_content_options_metabox_fields', array(
 
 					// Donation content.
 					array(
-						'name'        => __( 'Display Content', 'give' ),
-						'description' => __( 'Do you want to add custom content to this form?', 'give' ),
-						'id'          => $prefix . 'display_content',
+						'name'        => __('Display Content', 'give'),
+						'description' => __('Do you want to add custom content to this form?', 'give'),
+						'id'          => $prefix.'display_content',
 						'type'        => 'radio_inline',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 						'default'     => 'disabled',
 					),
 
 					// Content placement.
 					array(
-						'name'        => __( 'Content Placement', 'give' ),
-						'description' => __( 'This option controls where the content appears within the donation form.', 'give' ),
-						'id'          => $prefix . 'content_placement',
+						'name'        => __('Content Placement', 'give'),
+						'description' => __('This option controls where the content appears within the donation form.', 'give'),
+						'id'          => $prefix.'content_placement',
 						'type'        => 'radio_inline',
-						'options'     => apply_filters( 'give_forms_content_options_select', array(
-								'give_pre_form'  => __( 'Above fields', 'give' ),
-								'give_post_form' => __( 'Below fields', 'give' ),
+						'options'     => apply_filters('give_forms_content_options_select', array(
+								'give_pre_form'  => __('Above fields', 'give'),
+								'give_post_form' => __('Below fields', 'give'),
 							)
 						),
 						'default'     => 'give_pre_form',
 					),
 					array(
-						'name'        => __( 'Content', 'give' ),
-						'description' => __( 'This content will display on the single give form page.', 'give' ),
-						'id'          => $prefix . 'form_content',
+						'name'        => __('Content', 'give'),
+						'description' => __('This content will display on the single give form page.', 'give'),
+						'id'          => $prefix.'form_content',
 						'type'        => 'wysiwyg',
 					),
 					array(
 						'name'  => 'form_content_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-content',
-						'title' => __( 'Form Content', 'give' ),
+						'title' => __('Form Content', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Terms & Conditions
 			 */
-			'form_terms_options'    => apply_filters( 'give_forms_terms_options', array(
+			'form_terms_options'    => apply_filters('give_forms_terms_options', array(
 				'id'        => 'form_terms_options',
-				'title'     => __( 'Terms & Conditions', 'give' ),
+				'title'     => __('Terms & Conditions', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-checklist"></span>',
-				'fields'    => apply_filters( 'give_forms_terms_options_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_terms_options_metabox_fields', array(
 					// Donation Option
 					array(
-						'name'        => __( 'Terms and Conditions', 'give' ),
-						'description' => __( 'Do you want to require the donor to accept terms prior to being able to complete their donation?', 'give' ),
-						'id'          => $prefix . 'terms_option',
+						'name'        => __('Terms and Conditions', 'give'),
+						'description' => __('Do you want to require the donor to accept terms prior to being able to complete their donation?', 'give'),
+						'id'          => $prefix.'terms_option',
 						'type'        => 'radio_inline',
-						'options'     => apply_filters( 'give_forms_content_options_select', array(
-								'global'   => __( 'Global Option', 'give' ),
-								'enabled'  => __( 'Customize', 'give' ),
-								'disabled' => __( 'Disable', 'give' ),
+						'options'     => apply_filters('give_forms_content_options_select', array(
+								'global'   => __('Global Option', 'give'),
+								'enabled'  => __('Customize', 'give'),
+								'disabled' => __('Disable', 'give'),
 							)
 						),
 						'default'     => 'global',
 					),
 					array(
-						'id'         => $prefix . 'agree_label',
-						'name'       => __( 'Agreement Label', 'give' ),
-						'desc'       => __( 'The label shown next to the agree to terms check box. Add your own to customize or leave blank to use the default text placeholder.', 'give' ),
+						'id'         => $prefix.'agree_label',
+						'name'       => __('Agreement Label', 'give'),
+						'desc'       => __('The label shown next to the agree to terms check box. Add your own to customize or leave blank to use the default text placeholder.', 'give'),
 						'type'       => 'text',
 						'size'       => 'regular',
 						'attributes' => array(
-							'placeholder' => esc_attr__( 'Agree to Terms?', 'give' ),
+							'placeholder' => esc_attr__('Agree to Terms?', 'give'),
 						),
 					),
 					array(
-						'id'   => $prefix . 'agree_text',
-						'name' => __( 'Agreement Text', 'give' ),
-						'desc' => __( 'This is the actual text which the user will have to agree to in order to make a donation.', 'give' ),
+						'id'   => $prefix.'agree_text',
+						'name' => __('Agreement Text', 'give'),
+						'desc' => __('This is the actual text which the user will have to agree to in order to make a donation.', 'give'),
 						'default' => give_get_option('agreement_text'),
 						'type' => 'wysiwyg',
 					),
@@ -504,19 +504,19 @@ 
 						'name'  => 'terms_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-terms',
-						'title' => __( 'Terms and Conditions', 'give' ),
+						'title' => __('Terms and Conditions', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 		);
 
 
 		/**
 		 * Filter the metabox tabbed panel settings.
 		 */
-		$settings = apply_filters( 'give_metabox_form_data_settings', $settings, $post_id );
+		$settings = apply_filters('give_metabox_form_data_settings', $settings, $post_id);
 
 		// Output.
 		return $settings;
@@ -532,8 +532,8 @@ 
 		add_meta_box(
 			$this->get_metabox_ID(),
 			$this->get_metabox_label(),
-			array( $this, 'output' ),
-			array( 'give_forms' ),
+			array($this, 'output'),
+			array('give_forms'),
 			'normal',
 			'high'
 		);
@@ -549,7 +549,7 @@ 
 	function enqueue_script() {
 		global $post;
 
-		if ( is_object( $post ) && 'give_forms' === $post->post_type ) {
+		if (is_object($post) && 'give_forms' === $post->post_type) {
 
 		}
 	}
@@ -584,32 +584,32 @@ 
 	public function get_tabs() {
 		$tabs = array();
 
-		if ( ! empty( $this->settings ) ) {
-			foreach ( $this->settings as $setting ) {
-				if ( ! isset( $setting['id'] ) || ! isset( $setting['title'] ) ) {
+		if ( ! empty($this->settings)) {
+			foreach ($this->settings as $setting) {
+				if ( ! isset($setting['id']) || ! isset($setting['title'])) {
 					continue;
 				}
 				$tab = array(
 					'id'        => $setting['id'],
 					'label'     => $setting['title'],
-					'icon-html' => ( ! empty( $setting['icon-html'] ) ? $setting['icon-html'] : '' ),
+					'icon-html' => ( ! empty($setting['icon-html']) ? $setting['icon-html'] : ''),
 				);
 
-				if ( $this->has_sub_tab( $setting ) ) {
-					if ( empty( $setting['sub-fields'] ) ) {
+				if ($this->has_sub_tab($setting)) {
+					if (empty($setting['sub-fields'])) {
 						$tab = array();
 					} else {
-						foreach ( $setting['sub-fields'] as $sub_fields ) {
+						foreach ($setting['sub-fields'] as $sub_fields) {
 							$tab['sub-fields'][] = array(
 								'id'        => $sub_fields['id'],
 								'label'     => $sub_fields['title'],
-								'icon-html' => ( ! empty( $sub_fields['icon-html'] ) ? $sub_fields['icon-html'] : '' ),
+								'icon-html' => ( ! empty($sub_fields['icon-html']) ? $sub_fields['icon-html'] : ''),
 							);
 						}
 					}
 				}
 
-				if ( ! empty( $tab ) ) {
+				if ( ! empty($tab)) {
 					$tabs[] = $tab;
 				}
 			}
@@ -626,27 +626,27 @@ 
 	 */
 	public function output() {
 		// Bailout.
-		if ( $form_data_tabs = $this->get_tabs() ) {
-			wp_nonce_field( 'give_save_form_meta', 'give_form_meta_nonce' );
+		if ($form_data_tabs = $this->get_tabs()) {
+			wp_nonce_field('give_save_form_meta', 'give_form_meta_nonce');
 			?>
 			<div class="give-metabox-panel-wrap">
 				<ul class="give-form-data-tabs give-metabox-tabs">
-					<?php foreach ( $form_data_tabs as $index => $form_data_tab ) : ?>
-						<li class="<?php echo "{$form_data_tab['id']}_tab" . ( ! $index ? ' active' : '' ) . ( $this->has_sub_tab( $form_data_tab ) ? ' has-sub-fields' : '' ); ?>">
+					<?php foreach ($form_data_tabs as $index => $form_data_tab) : ?>
+						<li class="<?php echo "{$form_data_tab['id']}_tab".( ! $index ? ' active' : '').($this->has_sub_tab($form_data_tab) ? ' has-sub-fields' : ''); ?>">
 							<a href="#<?php echo $form_data_tab['id']; ?>">
-								<?php if ( ! empty( $form_data_tab['icon-html'] ) ) : ?>
+								<?php if ( ! empty($form_data_tab['icon-html'])) : ?>
 									<?php echo $form_data_tab['icon-html']; ?>
 								<?php else : ?>
 									<span class="give-icon give-icon-default"></span>
 								<?php endif; ?>
 								<span class="give-label"><?php echo $form_data_tab['label']; ?></span>
 							</a>
-							<?php if ( $this->has_sub_tab( $form_data_tab ) ) : ?>
+							<?php if ($this->has_sub_tab($form_data_tab)) : ?>
 								<ul class="give-metabox-sub-tabs give-hidden">
-									<?php foreach ( $form_data_tab['sub-fields'] as $sub_tab ) : ?>
+									<?php foreach ($form_data_tab['sub-fields'] as $sub_tab) : ?>
 										<li class="<?php echo "{$sub_tab['id']}_tab"; ?>">
 											<a href="#<?php echo $sub_tab['id']; ?>">
-												<?php if ( ! empty( $sub_tab['icon-html'] ) ) : ?>
+												<?php if ( ! empty($sub_tab['icon-html'])) : ?>
 													<?php echo $sub_tab['icon-html']; ?>
 												<?php else : ?>
 													<span class="give-icon give-icon-default"></span>
@@ -662,30 +662,30 @@ 
 				</ul>
 
 				<?php $show_first_tab_content = true; ?>
-				<?php foreach ( $this->settings as $setting ) : ?>
-					<?php if ( ! $this->has_sub_tab( $setting ) ) : ?>
-						<?php do_action( "give_before_{$setting['id']}_settings" ); ?>
+				<?php foreach ($this->settings as $setting) : ?>
+					<?php if ( ! $this->has_sub_tab($setting)) : ?>
+						<?php do_action("give_before_{$setting['id']}_settings"); ?>
 
 						<div id="<?php echo $setting['id']; ?>"
-							 class="panel give_options_panel<?php echo( $show_first_tab_content ? '' : ' give-hidden' );
+							 class="panel give_options_panel<?php echo($show_first_tab_content ? '' : ' give-hidden');
 						     $show_first_tab_content = false; ?>">
-							<?php if ( ! empty( $setting['fields'] ) ) : ?>
-								<?php foreach ( $setting['fields'] as $field ) : ?>
-									<?php give_render_field( $field ); ?>
+							<?php if ( ! empty($setting['fields'])) : ?>
+								<?php foreach ($setting['fields'] as $field) : ?>
+									<?php give_render_field($field); ?>
 								<?php endforeach; ?>
 							<?php endif; ?>
 						</div>
 
-						<?php do_action( "give_after_{$setting['id']}_settings" ); ?>
+						<?php do_action("give_after_{$setting['id']}_settings"); ?>
 					<?php else: ?>
-						<?php if ( $this->has_sub_tab( $setting ) ) : ?>
-							<?php if ( ! empty( $setting['sub-fields'] ) ) : ?>
-								<?php foreach ( $setting['sub-fields'] as $index => $sub_fields ) : ?>
+						<?php if ($this->has_sub_tab($setting)) : ?>
+							<?php if ( ! empty($setting['sub-fields'])) : ?>
+								<?php foreach ($setting['sub-fields'] as $index => $sub_fields) : ?>
 									<div id="<?php echo $sub_fields['id']; ?>"
 										 class="panel give_options_panel give-hidden">
-										<?php if ( ! empty( $sub_fields['fields'] ) ) : ?>
-											<?php foreach ( $sub_fields['fields'] as $sub_field ) : ?>
-												<?php give_render_field( $sub_field ); ?>
+										<?php if ( ! empty($sub_fields['fields'])) : ?>
+											<?php foreach ($sub_fields['fields'] as $sub_field) : ?>
+												<?php give_render_field($sub_field); ?>
 											<?php endforeach; ?>
 										<?php endif; ?>
 									</div>
@@ -709,9 +709,9 @@ 
 	 *
 	 * @return bool
 	 */
-	private function has_sub_tab( $field_setting ) {
+	private function has_sub_tab($field_setting) {
 		$has_sub_tab = false;
-		if ( array_key_exists( 'sub-fields', $field_setting ) ) {
+		if (array_key_exists('sub-fields', $field_setting)) {
 			$has_sub_tab = true;
 		}
 
@@ -725,13 +725,13 @@ 
 	 * @return array
 	 */
 	function cmb2_metabox_settings() {
-		$all_cmb2_settings   = apply_filters( 'cmb2_meta_boxes', array() );
+		$all_cmb2_settings   = apply_filters('cmb2_meta_boxes', array());
 		$give_forms_settings = $all_cmb2_settings;
 
 		// Filter settings: Use only give forms related settings.
-		foreach ( $all_cmb2_settings as $index => $setting ) {
-			if ( ! in_array( 'give_forms', $setting['object_types'] ) ) {
-				unset( $give_forms_settings[ $index ] );
+		foreach ($all_cmb2_settings as $index => $setting) {
+			if ( ! in_array('give_forms', $setting['object_types'])) {
+				unset($give_forms_settings[$index]);
 			}
 		}
 
@@ -749,114 +749,114 @@ 
 	 *
 	 * @return void
 	 */
-	public function save( $post_id, $post ) {
+	public function save($post_id, $post) {
 
 		// $post_id and $post are required.
-		if ( empty( $post_id ) || empty( $post ) ) {
+		if (empty($post_id) || empty($post)) {
 			return;
 		}
 
 		// Don't save meta boxes for revisions or autosaves.
-		if ( defined( 'DOING_AUTOSAVE' ) || is_int( wp_is_post_revision( $post ) ) || is_int( wp_is_post_autosave( $post ) ) ) {
+		if (defined('DOING_AUTOSAVE') || is_int(wp_is_post_revision($post)) || is_int(wp_is_post_autosave($post))) {
 			return;
 		}
 
 		// Check the nonce.
-		if ( empty( $_POST['give_form_meta_nonce'] ) || ! wp_verify_nonce( $_POST['give_form_meta_nonce'], 'give_save_form_meta' ) ) {
+		if (empty($_POST['give_form_meta_nonce']) || ! wp_verify_nonce($_POST['give_form_meta_nonce'], 'give_save_form_meta')) {
 			return;
 		}
 
 		// Check the post being saved == the $post_id to prevent triggering this call for other save_post events.
-		if ( empty( $_POST['post_ID'] ) || $_POST['post_ID'] != $post_id ) {
+		if (empty($_POST['post_ID']) || $_POST['post_ID'] != $post_id) {
 			return;
 		}
 
 		// Check user has permission to edit.
-		if ( ! current_user_can( 'edit_post', $post_id ) ) {
+		if ( ! current_user_can('edit_post', $post_id)) {
 			return;
 		}
 
 		// Fire action before saving form meta.
-		do_action( 'give_pre_process_give_forms_meta', $post_id, $post );
+		do_action('give_pre_process_give_forms_meta', $post_id, $post);
 
 		/**
 		 * Filter the meta key to save.
 		 * Third party addon developer can remove there meta keys from this array to handle saving data on there own.
 		 */
-		$form_meta_keys = apply_filters( 'give_process_form_meta_keys', $this->get_meta_keys_from_settings() );
+		$form_meta_keys = apply_filters('give_process_form_meta_keys', $this->get_meta_keys_from_settings());
 
 		// Save form meta data.
-		if ( ! empty( $form_meta_keys ) ) {
-			foreach ( $form_meta_keys as $form_meta_key ) {
+		if ( ! empty($form_meta_keys)) {
+			foreach ($form_meta_keys as $form_meta_key) {
 
 				// Set default value for checkbox fields.
 				if (
-					! isset( $_POST[ $form_meta_key ] )
-					&& ( 'checkbox' === $this->get_field_type( $form_meta_key ) )
+					! isset($_POST[$form_meta_key])
+					&& ('checkbox' === $this->get_field_type($form_meta_key))
 				) {
-					$_POST[ $form_meta_key ] = '';
+					$_POST[$form_meta_key] = '';
 				}
 
-				if ( isset( $_POST[ $form_meta_key ] ) ) {
-					if ( $field_type = $this->get_field_type( $form_meta_key ) ) {
-						switch ( $field_type ) {
+				if (isset($_POST[$form_meta_key])) {
+					if ($field_type = $this->get_field_type($form_meta_key)) {
+						switch ($field_type) {
 							case 'textarea':
 							case 'wysiwyg':
-								$form_meta_value = wp_kses_post( $_POST[ $form_meta_key ] );
-								give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+								$form_meta_value = wp_kses_post($_POST[$form_meta_key]);
+								give_update_meta($post_id, $form_meta_key, $form_meta_value);
 								break;
 
 							case 'group':
 								$form_meta_value = array();
 
-								foreach ( $_POST[ $form_meta_key ] as $index => $group ) {
+								foreach ($_POST[$form_meta_key] as $index => $group) {
 
 									// Do not save template input field values.
-									if ( '{{row-count-placeholder}}' === $index ) {
+									if ('{{row-count-placeholder}}' === $index) {
 										continue;
 									}
 
 									$group_meta_value = array();
-									foreach ( $group as $field_id => $field_value ) {
-										switch ( $this->get_field_type( $field_id, $form_meta_key ) ) {
+									foreach ($group as $field_id => $field_value) {
+										switch ($this->get_field_type($field_id, $form_meta_key)) {
 											case 'wysiwyg':
-												$group_meta_value[ $field_id ] = wp_kses_post( $field_value );
+												$group_meta_value[$field_id] = wp_kses_post($field_value);
 												break;
 
 											default:
-												$group_meta_value[ $field_id ] = give_clean( $field_value );
+												$group_meta_value[$field_id] = give_clean($field_value);
 										}
 									}
 
-									if ( ! empty( $group_meta_value ) ) {
-										$form_meta_value[ $index ] = $group_meta_value;
+									if ( ! empty($group_meta_value)) {
+										$form_meta_value[$index] = $group_meta_value;
 									}
 								}
 
 
 								// Arrange repeater field keys in order.
-								$form_meta_value = array_values( $form_meta_value );
+								$form_meta_value = array_values($form_meta_value);
 
 								// Save data.
-								give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+								give_update_meta($post_id, $form_meta_key, $form_meta_value);
 								break;
 
 							default:
-								$form_meta_value = give_clean( $_POST[ $form_meta_key ] );
+								$form_meta_value = give_clean($_POST[$form_meta_key]);
 
 								// Save data.
-								give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+								give_update_meta($post_id, $form_meta_key, $form_meta_value);
 						}
 
 						// Fire after saving form meta key.
-						do_action( "give_save_{$form_meta_key}", $form_meta_key, $form_meta_value, $post_id, $post );
+						do_action("give_save_{$form_meta_key}", $form_meta_key, $form_meta_value, $post_id, $post);
 					}
 				}
 			}
 		}
 
 		// Fire action after saving form meta.
-		do_action( 'give_post_process_give_forms_meta', $post_id, $post );
+		do_action('give_post_process_give_forms_meta', $post_id, $post);
 	}
 
 
@@ -869,10 +869,10 @@ 
 	 *
 	 * @return string
 	 */
-	private function get_field_id( $field ) {
+	private function get_field_id($field) {
 		$field_id = '';
 
-		if ( array_key_exists( 'id', $field ) ) {
+		if (array_key_exists('id', $field)) {
 			$field_id = $field['id'];
 
 		}
@@ -889,12 +889,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_fields_id( $setting ) {
+	private function get_fields_id($setting) {
 		$meta_keys = array();
 
-		if ( ! empty( $setting ) ) {
-			foreach ( $setting['fields'] as $field ) {
-				if ( $field_id = $this->get_field_id( $field ) ) {
+		if ( ! empty($setting)) {
+			foreach ($setting['fields'] as $field) {
+				if ($field_id = $this->get_field_id($field)) {
 					$meta_keys[] = $field_id;
 				}
 			}
@@ -912,14 +912,14 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_sub_fields_id( $setting ) {
+	private function get_sub_fields_id($setting) {
 		$meta_keys = array();
 
-		if ( $this->has_sub_tab( $setting ) && ! empty( $setting['sub-fields'] ) ) {
-			foreach ( $setting['sub-fields'] as $fields ) {
-				if ( ! empty( $fields['fields'] ) ) {
-					foreach ( $fields['fields'] as $field ) {
-						if ( $field_id = $this->get_field_id( $field ) ) {
+		if ($this->has_sub_tab($setting) && ! empty($setting['sub-fields'])) {
+			foreach ($setting['sub-fields'] as $fields) {
+				if ( ! empty($fields['fields'])) {
+					foreach ($fields['fields'] as $field) {
+						if ($field_id = $this->get_field_id($field)) {
 							$meta_keys[] = $field_id;
 						}
 					}
@@ -940,14 +940,14 @@ 
 	private function get_meta_keys_from_settings() {
 		$meta_keys = array();
 
-		foreach ( $this->settings as $setting ) {
-			if ( $this->has_sub_tab( $setting ) ) {
-				$meta_key = $this->get_sub_fields_id( $setting );
+		foreach ($this->settings as $setting) {
+			if ($this->has_sub_tab($setting)) {
+				$meta_key = $this->get_sub_fields_id($setting);
 			} else {
-				$meta_key = $this->get_fields_id( $setting );
+				$meta_key = $this->get_fields_id($setting);
 			}
 
-			$meta_keys = array_merge( $meta_keys, $meta_key );
+			$meta_keys = array_merge($meta_keys, $meta_key);
 		}
 
 		return $meta_keys;
@@ -964,10 +964,10 @@ 
 	 *
 	 * @return string
 	 */
-	function get_field_type( $field_id, $group_id = '' ) {
-		$field = $this->get_setting_field( $field_id, $group_id );
+	function get_field_type($field_id, $group_id = '') {
+		$field = $this->get_setting_field($field_id, $group_id);
 
-		$type = array_key_exists( 'type', $field )
+		$type = array_key_exists('type', $field)
 			? $field['type']
 			: '';
 
@@ -985,12 +985,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_field( $setting, $field_id ) {
+	private function get_field($setting, $field_id) {
 		$setting_field = array();
 
-		if ( ! empty( $setting['fields'] ) ) {
-			foreach ( $setting['fields'] as $field ) {
-				if ( array_key_exists( 'id', $field ) && $field['id'] === $field_id ) {
+		if ( ! empty($setting['fields'])) {
+			foreach ($setting['fields'] as $field) {
+				if (array_key_exists('id', $field) && $field['id'] === $field_id) {
 					$setting_field = $field;
 					break;
 				}
@@ -1010,12 +1010,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_sub_field( $setting, $field_id ) {
+	private function get_sub_field($setting, $field_id) {
 		$setting_field = array();
 
-		if ( ! empty( $setting['sub-fields'] ) ) {
-			foreach ( $setting['sub-fields'] as $fields ) {
-				if ( $field = $this->get_field( $fields, $field_id ) ) {
+		if ( ! empty($setting['sub-fields'])) {
+			foreach ($setting['sub-fields'] as $fields) {
+				if ($field = $this->get_field($fields, $field_id)) {
 					$setting_field = $field;
 					break;
 				}
@@ -1035,17 +1035,17 @@ 
 	 *
 	 * @return array
 	 */
-	function get_setting_field( $field_id, $group_id = '' ) {
+	function get_setting_field($field_id, $group_id = '') {
 		$setting_field = array();
 
 		$_field_id = $field_id;
-		$field_id  = empty( $group_id ) ? $field_id : $group_id;
+		$field_id  = empty($group_id) ? $field_id : $group_id;
 
-		if ( ! empty( $this->settings ) ) {
-			foreach ( $this->settings as $setting ) {
+		if ( ! empty($this->settings)) {
+			foreach ($this->settings as $setting) {
 				if (
-					( $this->has_sub_tab( $setting ) && ( $setting_field = $this->get_sub_field( $setting, $field_id ) ) )
-					|| ( $setting_field = $this->get_field( $setting, $field_id ) )
+					($this->has_sub_tab($setting) && ($setting_field = $this->get_sub_field($setting, $field_id)))
+					|| ($setting_field = $this->get_field($setting, $field_id))
 				) {
 					break;
 				}
@@ -1054,9 +1054,9 @@ 
 
 
 		// Get field from group.
-		if ( ! empty( $group_id ) ) {
-			foreach ( $setting_field['fields'] as $field ) {
-				if ( array_key_exists( 'id', $field ) && $field['id'] === $_field_id ) {
+		if ( ! empty($group_id)) {
+			foreach ($setting_field['fields'] as $field) {
+				if (array_key_exists('id', $field) && $field['id'] === $_field_id) {
 					$setting_field = $field;
 				}
 			}
@@ -1075,14 +1075,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	function add_offline_donations_setting_tab( $settings ) {
-		if ( give_is_gateway_active( 'offline' ) ) {
-			$settings['offline_donations_options'] = apply_filters( 'give_forms_offline_donations_options', array(
+	function add_offline_donations_setting_tab($settings) {
+		if (give_is_gateway_active('offline')) {
+			$settings['offline_donations_options'] = apply_filters('give_forms_offline_donations_options', array(
 				'id'        => 'offline_donations_options',
-				'title'     => __( 'Offline Donations', 'give' ),
+				'title'     => __('Offline Donations', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-purse"></span>',
-				'fields'    => apply_filters( 'give_forms_offline_donations_metabox_fields', array() ),
-			) );
+				'fields'    => apply_filters('give_forms_offline_donations_metabox_fields', array()),
+			));
 		}
 
 		return $settings;