WordImpress / Give

includes/payments/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -697,7 +697,7 @@ 
  * @param  int $year Year number. Default is null.
  * @param  int $hour Hour number. Default is null.
  *
- * @return int $earnings  Earnings
+ * @return double $earnings  Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 
@@ -986,7 +986,7 @@ 
  *
  * @param int $payment_id Payment ID.
  *
- * @return array $user_info User Info Meta Values.
+ * @return string $user_info User Info Meta Values.
  */
 function give_get_payment_meta_user_info( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -1003,7 +1003,7 @@ 
  *
  * @param int $payment_id Payment ID.
  *
- * @return int $form_id Form ID.
+ * @return string $form_id Form ID.
  */
 function give_get_payment_form_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );

Spacing +416 added lines, -416 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
  *
  * @return array $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility.
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -66,48 +66,48 @@ 
  *
  * @return mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -116,7 +116,7 @@ 
 			return false;
 	}// End switch().
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -132,29 +132,29 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise.
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = give_get_payment_meta_price_id( $payment_data );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = give_get_payment_meta_price_id($payment_data);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	// Set properties.
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency();
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency();
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
 	$payment->form_id        = $form_id;
 	$payment->price_id       = $price_id;
-	$payment->donor_id       = ( ! empty( $payment_data['donor_id'] ) ? $payment_data['donor_id'] : '' );
+	$payment->donor_id       = ( ! empty($payment_data['donor_id']) ? $payment_data['donor_id'] : '');
 	$payment->user_id        = $payment_data['user_info']['id'];
 	$payment->email          = $payment_data['user_email'];
 	$payment->first_name     = $payment_data['user_info']['first_name'];
@@ -162,8 +162,8 @@ 
 	$payment->email          = $payment_data['user_info']['email'];
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
-	$payment->mode           = ( ! empty( $payment_data['mode'] ) ? (string) $payment_data['mode'] : ( give_is_test_mode() ? 'test' : 'live' ) );
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->mode           = ( ! empty($payment_data['mode']) ? (string) $payment_data['mode'] : (give_is_test_mode() ? 'test' : 'live'));
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	// Add the donation.
 	$args = array(
@@ -171,19 +171,19 @@ 
 		'price_id' => $payment->price_id,
 	);
 
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 
 	// Set date if present.
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	// Handle sequential payments.
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Save payment.
@@ -197,10 +197,10 @@ 
 	 * @param int $payment_id The payment ID.
 	 * @param array $payment_data Arguments passed.
 	 */
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	// Return payment ID upon success.
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -216,10 +216,10 @@ 
  *
  * @return bool|int
  */
-function give_create_payment( $payment_data ) {
+function give_create_payment($payment_data) {
 
-	$form_id  = intval( $payment_data['post_data']['give-form-id'] );
-	$price_id = isset( $payment_data['post_data']['give-price-id'] ) ? $payment_data['post_data']['give-price-id'] : '';
+	$form_id  = intval($payment_data['post_data']['give-form-id']);
+	$price_id = isset($payment_data['post_data']['give-price-id']) ? $payment_data['post_data']['give-price-id'] : '';
 
 	// Collect payment data.
 	$insert_payment_data = array(
@@ -243,10 +243,10 @@ 
 	 *
 	 * @param array $insert_payment_data
 	 */
-	$insert_payment_data = apply_filters( 'give_create_payment', $insert_payment_data );
+	$insert_payment_data = apply_filters('give_create_payment', $insert_payment_data);
 
 	// Record the pending payment.
-	return give_insert_payment( $insert_payment_data );
+	return give_insert_payment($insert_payment_data);
 }
 
 /**
@@ -259,12 +259,12 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
 	$updated = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment && $payment->ID > 0 ) {
+	if ($payment && $payment->ID > 0) {
 
 		$payment->status = $new_status;
 		$updated         = $payment->save();
@@ -286,38 +286,38 @@ 
  *
  * @return void
  */
-function give_delete_donation( $payment_id = 0, $update_donor = true ) {
+function give_delete_donation($payment_id = 0, $update_donor = true) {
 	global $give_logs;
 
-	$payment  = new Give_Payment( $payment_id );
-	$amount   = give_get_payment_amount( $payment_id );
+	$payment  = new Give_Payment($payment_id);
+	$amount   = give_get_payment_amount($payment_id);
 	$status   = $payment->post_status;
-	$donor_id = give_get_payment_donor_id( $payment_id );
-	$donor    = new Give_Donor( $donor_id );
+	$donor_id = give_get_payment_donor_id($payment_id);
+	$donor    = new Give_Donor($donor_id);
 
 	// Only undo donations that aren't these statuses.
-	$dont_undo_statuses = apply_filters( 'give_undo_donation_statuses', array(
+	$dont_undo_statuses = apply_filters('give_undo_donation_statuses', array(
 		'pending',
 		'cancelled',
-	) );
+	));
 
-	if ( ! in_array( $status, $dont_undo_statuses ) ) {
-		give_undo_donation( $payment_id );
+	if ( ! in_array($status, $dont_undo_statuses)) {
+		give_undo_donation($payment_id);
 	}
 
-	if ( $status == 'publish' ) {
+	if ($status == 'publish') {
 
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment).
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
 
-		if ( $donor->id && $update_donor ) {
+		if ($donor->id && $update_donor) {
 
 			// Decrement the stats for the donor.
 			$donor->decrease_donation_count();
-			$donor->decrease_value( $amount );
+			$donor->decrease_value($amount);
 
 		}
 	}
@@ -329,25 +329,25 @@ 
 	 *
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $donor->id && $update_donor ) {
+	if ($donor->id && $update_donor) {
 
 		// Remove the payment ID from the donor.
-		$donor->remove_payment( $payment_id );
+		$donor->remove_payment($payment_id);
 
 	}
 
 	// Remove the payment.
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries.
-	$give_logs->delete_logs( null, 'sale', array(
+	$give_logs->delete_logs(null, 'sale', array(
 		array(
 			'key'   => '_give_log_payment_id',
 			'value' => $payment_id,
 		),
-	) );
+	));
 
 	/**
 	 * Fires after payment deleted.
@@ -356,7 +356,7 @@ 
 	 *
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -371,20 +371,20 @@ 
  *
  * @return void
  */
-function give_undo_donation( $payment_id ) {
+function give_undo_donation($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// Decrease earnings.
-		give_decrease_earnings( $payment->form_id, $payment->total );
+		give_decrease_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_donations = apply_filters( 'give_decrease_donations_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_donations ) {
+	$maybe_decrease_donations = apply_filters('give_decrease_donations_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_donations) {
 		// Decrease donation count.
-		give_decrease_donation_count( $payment->form_id );
+		give_decrease_donation_count($payment->form_id);
 	}
 
 }
@@ -401,7 +401,7 @@ 
  *
  * @return object $stats Contains the number of payments per payment status.
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 
 	global $wpdb;
 
@@ -413,18 +413,18 @@ 
 		'form_id'    => null,
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$select = 'SELECT p.post_status,count( * ) AS num_posts';
 	$join   = '';
-	$where  = "WHERE p.post_type = 'give_payment' AND p.post_status IN ('" . implode( "','", give_get_payment_status_keys() ) . "')";
+	$where  = "WHERE p.post_type = 'give_payment' AND p.post_status IN ('".implode("','", give_get_payment_status_keys())."')";
 
 	// Count payments for a specific user.
-	if ( ! empty( $args['user'] ) ) {
+	if ( ! empty($args['user'])) {
 
-		if ( is_email( $args['user'] ) ) {
+		if (is_email($args['user'])) {
 			$field = 'email';
-		} elseif ( is_numeric( $args['user'] ) ) {
+		} elseif (is_numeric($args['user'])) {
 			$field = 'id';
 		} else {
 			$field = '';
@@ -432,107 +432,107 @@ 
 
 		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$where .= "
 				AND m.meta_key = '_give_payment_user_{$field}'
 				AND m.meta_value = '{$args['user']}'";
 		}
-	} elseif ( ! empty( $args['donor'] ) ) {
+	} elseif ( ! empty($args['donor'])) {
 
-		$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 		$where .= "
 			AND m.meta_key = '_give_payment_customer_id'
 			AND m.meta_value = '{$args['donor']}'";
 
 		// Count payments for a search.
-	} elseif ( ! empty( $args['s'] ) ) {
+	} elseif ( ! empty($args['s'])) {
 
-		if ( is_email( $args['s'] ) || strlen( $args['s'] ) == 32 ) {
+		if (is_email($args['s']) || strlen($args['s']) == 32) {
 
-			if ( is_email( $args['s'] ) ) {
+			if (is_email($args['s'])) {
 				$field = '_give_payment_user_email';
 			} else {
 				$field = '_give_payment_purchase_key';
 			}
 
-			$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( '
+			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+			$where .= $wpdb->prepare('
                 AND m.meta_key = %s
-                AND m.meta_value = %s', $field, $args['s'] );
+                AND m.meta_value = %s', $field, $args['s']);
 
-		} elseif ( '#' == substr( $args['s'], 0, 1 ) ) {
+		} elseif ('#' == substr($args['s'], 0, 1)) {
 
-			$search = str_replace( '#:', '', $args['s'] );
-			$search = str_replace( '#', '', $search );
+			$search = str_replace('#:', '', $args['s']);
+			$search = str_replace('#', '', $search);
 
 			$select = 'SELECT p.post_status,count( * ) AS num_posts ';
 			$join   = '';
-			$where  = $wpdb->prepare( 'WHERE p.post_type=%s  AND p.ID = %d ', 'give_payment', $search );
+			$where  = $wpdb->prepare('WHERE p.post_type=%s  AND p.ID = %d ', 'give_payment', $search);
 
-		} elseif ( is_numeric( $args['s'] ) ) {
+		} elseif (is_numeric($args['s'])) {
 
-			$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+			$where .= $wpdb->prepare("
 				AND m.meta_key = '_give_payment_user_id'
-				AND m.meta_value = %d", $args['s'] );
+				AND m.meta_value = %d", $args['s']);
 
 		} else {
-			$search = $wpdb->esc_like( $args['s'] );
-			$search = '%' . $search . '%';
+			$search = $wpdb->esc_like($args['s']);
+			$search = '%'.$search.'%';
 
-			$where .= $wpdb->prepare( 'AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))', $search, $search );
+			$where .= $wpdb->prepare('AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))', $search, $search);
 		}// End if().
 	}// End if().
 
-	if ( ! empty( $args['form_id'] ) && is_numeric( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id']) && is_numeric($args['form_id'])) {
 
-		$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-		$where .= $wpdb->prepare( '
+		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+		$where .= $wpdb->prepare('
                 AND m.meta_key = %s
-                AND m.meta_value = %s', '_give_payment_form_id', $args['form_id'] );
+                AND m.meta_value = %s', '_give_payment_form_id', $args['form_id']);
 	}
 
 	// Limit payments count by date.
-	if ( ! empty( $args['start-date'] ) && false !== strpos( $args['start-date'], '/' ) ) {
+	if ( ! empty($args['start-date']) && false !== strpos($args['start-date'], '/')) {
 
-		$date_parts = explode( '/', $args['start-date'] );
-		$month      = ! empty( $date_parts[0] ) && is_numeric( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day        = ! empty( $date_parts[1] ) && is_numeric( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year       = ! empty( $date_parts[2] ) && is_numeric( $date_parts[2] ) ? $date_parts[2] : 0;
+		$date_parts = explode('/', $args['start-date']);
+		$month      = ! empty($date_parts[0]) && is_numeric($date_parts[0]) ? $date_parts[0] : 0;
+		$day        = ! empty($date_parts[1]) && is_numeric($date_parts[1]) ? $date_parts[1] : 0;
+		$year       = ! empty($date_parts[2]) && is_numeric($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date  = new DateTime( $args['start-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date >= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['start-date']);
+			$where .= $wpdb->prepare(" AND p.post_date >= '%s'", $date->format('Y-m-d'));
 
 		}
 
 		// Fixes an issue with the payments list table counts when no end date is specified (with stats class).
-		if ( empty( $args['end-date'] ) ) {
+		if (empty($args['end-date'])) {
 			$args['end-date'] = $args['start-date'];
 		}
 	}
 
-	if ( ! empty( $args['end-date'] ) && false !== strpos( $args['end-date'], '/' ) ) {
+	if ( ! empty($args['end-date']) && false !== strpos($args['end-date'], '/')) {
 
-		$date_parts = explode( '/', $args['end-date'] );
+		$date_parts = explode('/', $args['end-date']);
 
-		$month = ! empty( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day   = ! empty( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year  = ! empty( $date_parts[2] ) ? $date_parts[2] : 0;
+		$month = ! empty($date_parts[0]) ? $date_parts[0] : 0;
+		$day   = ! empty($date_parts[1]) ? $date_parts[1] : 0;
+		$year  = ! empty($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date  = new DateTime( $args['end-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date <= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['end-date']);
+			$where .= $wpdb->prepare(" AND p.post_date <= '%s'", $date->format('Y-m-d'));
 
 		}
 	}
 
-	$where = apply_filters( 'give_count_payments_where', $where );
-	$join  = apply_filters( 'give_count_payments_join', $join );
+	$where = apply_filters('give_count_payments_where', $where);
+	$join  = apply_filters('give_count_payments_join', $join);
 
 	$query = "$select
 		FROM $wpdb->posts p
@@ -541,36 +541,36 @@ 
 		GROUP BY p.post_status
 	";
 
-	$cache_key = md5( $query );
+	$cache_key = md5($query);
 
-	$count = wp_cache_get( $cache_key, 'counts' );
-	if ( false !== $count ) {
+	$count = wp_cache_get($cache_key, 'counts');
+	if (false !== $count) {
 		return $count;
 	}
 
-	$count = $wpdb->get_results( $query, ARRAY_A );
+	$count = $wpdb->get_results($query, ARRAY_A);
 
 	$stats    = array();
 	$statuses = get_post_stati();
-	if ( isset( $statuses['private'] ) && empty( $args['s'] ) ) {
-		unset( $statuses['private'] );
+	if (isset($statuses['private']) && empty($args['s'])) {
+		unset($statuses['private']);
 	}
 
-	foreach ( $statuses as $state ) {
-		$stats[ $state ] = 0;
+	foreach ($statuses as $state) {
+		$stats[$state] = 0;
 	}
 
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 
-		if ( 'private' == $row['post_status'] && empty( $args['s'] ) ) {
+		if ('private' == $row['post_status'] && empty($args['s'])) {
 			continue;
 		}
 
-		$stats[ $row['post_status'] ] = $row['num_posts'];
+		$stats[$row['post_status']] = $row['num_posts'];
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( $cache_key, $stats, 'counts' );
+	wp_cache_set($cache_key, $stats, 'counts');
 
 	return $stats;
 }
@@ -585,11 +585,11 @@ 
  *
  * @return bool $exists True if payment exists, false otherwise.
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -607,41 +607,41 @@ 
  *
  * @return bool|mixed True if payment status exists, false otherwise.
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( is_numeric( $payment ) ) {
+	if (is_numeric($payment)) {
 
-		$payment = new Give_Payment( $payment );
+		$payment = new Give_Payment($payment);
 
-		if ( ! $payment->ID > 0 ) {
+		if ( ! $payment->ID > 0) {
 			return false;
 		}
 
 	}
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
 	// Get payment object if not already given.
-	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment( $payment->ID );
+	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
 			// Return translated status label.
-			return $statuses[ $payment->status ];
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' == $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -657,18 +657,18 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'processing'  => __( 'Processing', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' ),
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'processing'  => __('Processing', 'give'),
+		'revoked'     => __('Revoked', 'give'),
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -681,10 +681,10 @@ 
  * @return array $payment_status All the available payment statuses.
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -699,7 +699,7 @@ 
  *
  * @return int $earnings  Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
 	global $wpdb;
@@ -709,41 +709,41 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false,
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = Give_Cache::get_key( 'give_stats', $args );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 	}
 
-	if ( false === $earnings ) {
-		$donations = get_posts( $args );
+	if (false === $earnings) {
+		$donations = get_posts($args);
 		$earnings  = 0;
-		if ( $donations ) {
-			$donations = implode( ',', $donations );
+		if ($donations) {
+			$donations = implode(',', $donations);
 
-			$earnings = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})" );
+			$earnings = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})");
 
 		}
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $earnings, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -758,7 +758,7 @@ 
  *
  * @return int $count     Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead.
 	$args = array(
@@ -766,14 +766,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false,
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -784,33 +784,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = Give_Cache::get_key( 'give_stats', $args );
+	$key = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = Give_Cache::get( $key );
+		$count = Give_Cache::get($key);
 	}
 
-	if ( false === $count ) {
-		$donations = new WP_Query( $args );
+	if (false === $count) {
+		$donations = new WP_Query($args);
 		$count     = (int) $donations->post_count;
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $count, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -825,19 +825,19 @@ 
  *
  * @return bool $ret True if complete, false otherwise.
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -863,49 +863,49 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_get_total_earnings( $recalculate = false ) {
+function give_get_total_earnings($recalculate = false) {
 
-	$total = get_option( 'give_earnings_total', 0 );
+	$total = get_option('give_earnings_total', 0);
 
 	// Calculate total earnings.
-	if ( ! $total || $recalculate ) {
+	if ( ! $total || $recalculate) {
 		global $wpdb;
 
 		$total = (float) 0;
 
-		$args = apply_filters( 'give_get_total_earnings_args', array(
+		$args = apply_filters('give_get_total_earnings_args', array(
 			'offset' => 0,
-			'number' => - 1,
-			'status' => array( 'publish' ),
+			'number' => -1,
+			'status' => array('publish'),
 			'fields' => 'ids',
-		) );
+		));
 
-		$payments = give_get_payments( $args );
-		if ( $payments ) {
+		$payments = give_get_payments($args);
+		if ($payments) {
 
 			/**
 			 * If performing a donation, we need to skip the very last payment in the database,
 			 * since it calls give_increase_total_earnings() on completion,
 			 * which results in duplicated earnings for the very first donation.
 			 */
-			if ( did_action( 'give_update_payment_status' ) ) {
-				array_pop( $payments );
+			if (did_action('give_update_payment_status')) {
+				array_pop($payments);
 			}
 
-			if ( ! empty( $payments ) ) {
-				$payments = implode( ',', $payments );
-				$total    += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})" );
+			if ( ! empty($payments)) {
+				$payments = implode(',', $payments);
+				$total += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})");
 			}
 		}
 
-		update_option( 'give_earnings_total', $total, 'no' );
+		update_option('give_earnings_total', $total, 'no');
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings.
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_currency_decimal_filter() ) );
+	return apply_filters('give_total_earnings', round($total, give_currency_decimal_filter()));
 }
 
 /**
@@ -918,10 +918,10 @@ 
  *
  * @return float $total  Total earnings.
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -935,13 +935,13 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -957,10 +957,10 @@ 
  *
  * @return mixed $meta Payment Meta.
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -973,10 +973,10 @@ 
  *
  * @return mixed Meta ID if successful, false if unsuccessful.
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -988,8 +988,8 @@ 
  *
  * @return array $user_info User Info Meta Values.
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -1005,8 +1005,8 @@ 
  *
  * @return int $form_id Form ID.
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -1020,8 +1020,8 @@ 
  *
  * @return string $email User email.
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -1035,11 +1035,11 @@ 
  *
  * @return bool $is_guest_payment If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -1051,8 +1051,8 @@ 
  *
  * @return int $user_id User ID.
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -1066,8 +1066,8 @@ 
  *
  * @return int $payment->customer_id Donor ID.
  */
-function give_get_payment_donor_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_donor_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -1081,8 +1081,8 @@ 
  *
  * @return string $ip User IP.
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -1096,8 +1096,8 @@ 
  *
  * @return string $date The date the payment was completed.
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -1111,8 +1111,8 @@ 
  *
  * @return string $gateway Gateway.
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1126,8 +1126,8 @@ 
  *
  * @return string $currency The currency code.
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1141,10 +1141,10 @@ 
  *
  * @return string $currency The currency name.
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1156,8 +1156,8 @@ 
  *
  * @return string $key Donation key.
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1173,8 +1173,8 @@ 
  *
  * @return string $number Payment order number.
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1188,23 +1188,23 @@ 
  *
  * @return string      The formatted payment number.
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1217,17 +1217,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1236,24 +1236,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason.
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids',
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1262,13 +1262,13 @@ 
 		}
 	}// End if().
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1280,25 +1280,25 @@ 
  *
  * @return string The new Payment number without prefix and postfix.
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix.
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix.
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number.
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1315,10 +1315,10 @@ 
  *
  * @return string $amount Fully formatted payment amount.
  */
-function give_payment_amount( $payment_id = 0 ) {
-	$amount = give_get_payment_amount( $payment_id );
+function give_payment_amount($payment_id = 0) {
+	$amount = give_get_payment_amount($payment_id);
 
-	return give_currency_filter( give_format_amount( $amount, array( 'sanitize' => false ) ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($amount, array('sanitize' => false)), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1331,11 +1331,11 @@ 
  *
  * @return mixed
  */
-function give_get_payment_amount( $payment_id ) {
+function give_get_payment_amount($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	return apply_filters( 'give_payment_amount', floatval( $payment->total ), $payment_id );
+	return apply_filters('give_payment_amount', floatval($payment->total), $payment_id);
 }
 
 /**
@@ -1352,10 +1352,10 @@ 
  *
  * @return array Fully formatted payment subtotal.
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal, array( 'sanitize' => false ) ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($subtotal, array('sanitize' => false)), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1369,8 +1369,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted).
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1384,8 +1384,8 @@ 
  *
  * @return string The donation ID.
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1400,15 +1400,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1421,12 +1421,12 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_key( $key ) {
+function give_get_purchase_id_by_key($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1444,12 +1444,12 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1466,23 +1466,23 @@ 
  *
  * @return array $notes Donation Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10);
 
-	$notes = get_comments( array(
+	$notes = get_comments(array(
 		'post_id' => $payment_id,
 		'order'   => 'ASC',
 		'search'  => $search,
-	) );
+	));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1498,8 +1498,8 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
@@ -1511,14 +1511,14 @@ 
 	 * @param int $payment_id Payment ID.
 	 * @param string $note The note.
 	 */
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1527,7 +1527,7 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note',
 
-	) ) );
+	)));
 
 	/**
 	 * Fires after payment note inserted.
@@ -1538,7 +1538,7 @@ 
 	 * @param int $payment_id Payment ID.
 	 * @param string $note The note.
 	 */
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1553,8 +1553,8 @@ 
  *
  * @return bool True on success, false otherwise.
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
@@ -1566,9 +1566,9 @@ 
 	 * @param int $comment_id Note ID.
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
 
-	$ret = wp_delete_comment( $comment_id, true );
+	$ret = wp_delete_comment($comment_id, true);
 
 	/**
 	 * Fires after donation note deleted.
@@ -1578,7 +1578,7 @@ 
 	 * @param int $comment_id Note ID.
 	 * @param int $payment_id Payment ID.
 	 */
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1593,32 +1593,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = esc_html__( 'System', 'give' );
+		$user = esc_html__('System', 'give');
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id,
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" aria-label="' . esc_attr__( 'Delete this donation note.', 'give' ) . '">' . esc_html__( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" aria-label="'.esc_attr__('Delete this donation note.', 'give').'">'.esc_html__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1636,18 +1636,18 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+function give_hide_payment_notes($query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1659,15 +1659,15 @@ 
  *
  * @return array $clauses Updated comment clauses.
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1680,15 +1680,15 @@ 
  *
  * @return string $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( ' AND comment_type != %s', 'give_payment_note' );
+	$where .= $wpdb->prepare(' AND comment_type != %s', 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1702,32 +1702,32 @@ 
  *
  * @return array Array of comment counts.
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = wp_cache_get( "comments-{$post_id}", 'counts' );
+	$stats = wp_cache_get("comments-{$post_id}", 'counts');
 
-	if ( false !== $stats ) {
+	if (false !== $stats) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1737,30 +1737,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed',
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals.
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( "comments-{$post_id}", $stats, 'counts' );
+	wp_cache_set("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1773,9 +1773,9 @@ 
  *
  * @return string $where Modified where clause.
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week.
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1795,38 +1795,38 @@ 
  *
  * @return string $form_title Returns the full title if $only_level is false, otherwise returns the levels title.
  */
-function give_get_payment_form_title( $payment_meta, $only_level = false, $separator = '' ) {
+function give_get_payment_form_title($payment_meta, $only_level = false, $separator = '') {
 
-	$form_id    = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
-	$price_id   = isset( $payment_meta['price_id'] ) ? $payment_meta['price_id'] : null;
-	$form_title = isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '';
+	$form_id    = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
+	$price_id   = isset($payment_meta['price_id']) ? $payment_meta['price_id'] : null;
+	$form_title = isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '';
 
-	if ( $only_level == true ) {
+	if ($only_level == true) {
 		$form_title = '';
 	}
 
 	// If multi-level, append to the form title.
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
 		// Only add separator if there is a form title.
-		if ( ! empty( $form_title ) ) {
-			$form_title .= ' ' . $separator . ' ';
+		if ( ! empty($form_title)) {
+			$form_title .= ' '.$separator.' ';
 		}
 
 		$form_title .= '<span class="donation-level-text-wrap">';
 
-		if ( $price_id == 'custom' ) {
-			$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
-			$form_title         .= ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
+		if ($price_id == 'custom') {
+			$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
+			$form_title .= ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
 		} else {
-			$form_title .= give_get_price_option_name( $form_id, $price_id );
+			$form_title .= give_get_price_option_name($form_id, $price_id);
 		}
 
 		$form_title .= '</span>';
 
 	}
 
-	return apply_filters( 'give_get_payment_form_title', $form_title, $payment_meta );
+	return apply_filters('give_get_payment_form_title', $form_title, $payment_meta);
 
 }
 
@@ -1840,19 +1840,19 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 	$price_id = null;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = give_get_meta( $form_id, '_give_donation_levels', true );
+		$levels = give_get_meta($form_id, '_give_donation_levels', true);
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = give_maybe_sanitize_amount( $level['_give_amount'] );
+			$level_amount = give_maybe_sanitize_amount($level['_give_amount']);
 
 			// Check that this indeed the recurring price.
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 				break;
@@ -1860,13 +1860,13 @@ 
 			}
 		}
 
-		if( is_null( $price_id ) && give_is_custom_price_mode( $form_id ) ) {
+		if (is_null($price_id) && give_is_custom_price_mode($form_id)) {
 			$price_id = 'custom';
 		}
 	}
 
 	// Price ID must be numeric or string.
-	$price_id = ! is_numeric( $price_id ) && ! is_string( $price_id ) ? 0 : $price_id;
+	$price_id = ! is_numeric($price_id) && ! is_string($price_id) ? 0 : $price_id;
 
 	return $price_id;
 }
@@ -1884,10 +1884,10 @@ 
  *
  * @return string
  */
-function give_get_form_dropdown( $args = array(), $echo = false ) {
-	$form_dropdown_html = Give()->html->forms_dropdown( $args );
+function give_get_form_dropdown($args = array(), $echo = false) {
+	$form_dropdown_html = Give()->html->forms_dropdown($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1904,17 +1904,17 @@ 
  *
  * @return string|bool
  */
-function give_get_form_variable_price_dropdown( $args = array(), $echo = false ) {
+function give_get_form_variable_price_dropdown($args = array(), $echo = false) {
 
 	// Check for give form id.
-	if ( empty( $args['id'] ) ) {
+	if (empty($args['id'])) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $args['id'] );
+	$form = new Give_Donate_Form($args['id']);
 
 	// Check if form has variable prices or not.
-	if ( ! $form->ID || ! $form->has_variable_prices() ) {
+	if ( ! $form->ID || ! $form->has_variable_prices()) {
 		return false;
 	}
 
@@ -1922,24 +1922,24 @@ 
 	$variable_price_options = array();
 
 	// Check if multi donation form support custom donation or not.
-	if ( $form->is_custom_price_mode() ) {
-		$variable_price_options['custom'] = _x( 'Custom', 'custom donation dropdown item', 'give' );
+	if ($form->is_custom_price_mode()) {
+		$variable_price_options['custom'] = _x('Custom', 'custom donation dropdown item', 'give');
 	}
 
 	// Get variable price and ID from variable price array.
-	foreach ( $variable_prices as $variable_price ) {
-		$variable_price_options[ $variable_price['_give_id']['level_id'] ] = ! empty( $variable_price['_give_text'] ) ? $variable_price['_give_text'] : give_currency_filter( give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) ) );
+	foreach ($variable_prices as $variable_price) {
+		$variable_price_options[$variable_price['_give_id']['level_id']] = ! empty($variable_price['_give_text']) ? $variable_price['_give_text'] : give_currency_filter(give_format_amount($variable_price['_give_amount'], array('sanitize' => false)));
 	}
 
 	// Update options.
-	$args = array_merge( $args, array(
+	$args = array_merge($args, array(
 		'options' => $variable_price_options,
-	) );
+	));
 
 	// Generate select html.
-	$form_dropdown_html = Give()->html->select( $args );
+	$form_dropdown_html = Give()->html->select($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1958,16 +1958,16 @@ 
  *
  * @return string
  */
-function give_get_payment_meta_price_id( $payment_meta ) {
+function give_get_payment_meta_price_id($payment_meta) {
 
-	if ( isset( $payment_meta['give_price_id'] ) ) {
+	if (isset($payment_meta['give_price_id'])) {
 		$price_id = $payment_meta['give_price_id'];
-	} elseif ( isset( $payment_meta['price_id'] ) ) {
+	} elseif (isset($payment_meta['price_id'])) {
 		$price_id = $payment_meta['price_id'];
 	} else {
-		$price_id = give_get_price_id( $payment_meta['give_form_id'], $payment_meta['price'] );
+		$price_id = give_get_price_id($payment_meta['give_form_id'], $payment_meta['price']);
 	}
 
-	return apply_filters( 'give_get_payment_meta_price_id', $price_id );
+	return apply_filters('give_get_payment_meta_price_id', $price_id);
 
 }

includes/user-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -671,7 +671,7 @@
 /**
  * This function will check whether the donor email is primary or additional.
  *
- * @param $email Donor Email.
+ * @param string $email Donor Email.
  *
  * @since 1.8.13
  *

Spacing +144 added lines, -144 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,54 +30,54 @@ 
  *
  * @return bool|array List of all user donations.
  */
-function give_get_users_donations( $user = 0, $number = 20, $pagination = false, $status = 'complete' ) {
+function give_get_users_donations($user = 0, $number = 20, $pagination = false, $status = 'complete') {
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( 0 === $user && ! Give()->email_access->token_exists ) {
+	if (0 === $user && ! Give()->email_access->token_exists) {
 		return false;
 	}
 
 	$status = $status === 'complete' ? 'publish' : $status;
 	$paged = 1;
 
-	if ( $pagination ) {
-		if ( get_query_var( 'paged' ) ) {
-			$paged = get_query_var( 'paged' );
-		} elseif ( get_query_var( 'page' ) ) {
-			$paged = get_query_var( 'page' );
+	if ($pagination) {
+		if (get_query_var('paged')) {
+			$paged = get_query_var('paged');
+		} elseif (get_query_var('page')) {
+			$paged = get_query_var('page');
 		}
 	}
 
-	$args = apply_filters( 'give_get_users_donations_args', array(
+	$args = apply_filters('give_get_users_donations_args', array(
 		'user'    => $user,
 		'number'  => $number,
 		'status'  => $status,
 		'orderby' => 'date',
-	) );
+	));
 
-	if ( $pagination ) {
+	if ($pagination) {
 		$args['page'] = $paged;
 	} else {
 		$args['nopaging'] = true;
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
-	$donor   = new Give_Donor( $user, $by_user_id );
+	$by_user_id = is_numeric($user) ? true : false;
+	$donor = new Give_Donor($user, $by_user_id);
 
-	if ( ! empty( $donor->payment_ids ) ) {
+	if ( ! empty($donor->payment_ids)) {
 
-		unset( $args['user'] );
-		$args['post__in'] = array_map( 'absint', explode( ',', $donor->payment_ids ) );
+		unset($args['user']);
+		$args['post__in'] = array_map('absint', explode(',', $donor->payment_ids));
 
 	}
 
-	$donations = give_get_payments( apply_filters( 'give_get_users_donations_args', $args ) );
+	$donations = give_get_payments(apply_filters('give_get_users_donations_args', $args));
 
 	// No donations
-	if ( ! $donations ) {
+	if ( ! $donations) {
 		return false;
 	}
 
@@ -96,65 +96,65 @@ 
  *
  * @return bool|object List of unique forms donated by user
  */
-function give_get_users_completed_donations( $user = 0, $status = 'complete' ) {
-	if ( empty( $user ) ) {
+function give_get_users_completed_donations($user = 0, $status = 'complete') {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		return false;
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
+	$by_user_id = is_numeric($user) ? true : false;
 
-	$donor = new Give_Donor( $user, $by_user_id );
+	$donor = new Give_Donor($user, $by_user_id);
 
-	if ( empty( $donor->payment_ids ) ) {
+	if (empty($donor->payment_ids)) {
 		return false;
 	}
 
 	// Get all the items donated.
-	$payment_ids    = array_reverse( explode( ',', $donor->payment_ids ) );
-	$limit_payments = apply_filters( 'give_users_completed_donations_payments', 50 );
-	if ( ! empty( $limit_payments ) ) {
-		$payment_ids = array_slice( $payment_ids, 0, $limit_payments );
+	$payment_ids    = array_reverse(explode(',', $donor->payment_ids));
+	$limit_payments = apply_filters('give_users_completed_donations_payments', 50);
+	if ( ! empty($limit_payments)) {
+		$payment_ids = array_slice($payment_ids, 0, $limit_payments);
 	}
 	$donation_data = array();
-	foreach ( $payment_ids as $payment_id ) {
-		$donation_data[] = give_get_payment_meta( $payment_id );
+	foreach ($payment_ids as $payment_id) {
+		$donation_data[] = give_get_payment_meta($payment_id);
 	}
 
-	if ( empty( $donation_data ) ) {
+	if (empty($donation_data)) {
 		return false;
 	}
 
 	// Grab only the post ids "form_id" of the forms donated on this order
 	$completed_donations_ids = array();
-	foreach ( $donation_data as $donation_meta ) {
-		$completed_donations_ids[] = isset( $donation_meta['form_id'] ) ? $donation_meta['form_id'] : '';
+	foreach ($donation_data as $donation_meta) {
+		$completed_donations_ids[] = isset($donation_meta['form_id']) ? $donation_meta['form_id'] : '';
 	}
 
-	if ( empty( $completed_donations_ids ) ) {
+	if (empty($completed_donations_ids)) {
 		return false;
 	}
 
 	// Only include each donation once
-	$form_ids = array_unique( $completed_donations_ids );
+	$form_ids = array_unique($completed_donations_ids);
 
 	// Make sure we still have some products and a first item
-	if ( empty( $form_ids ) || ! isset( $form_ids[0] ) ) {
+	if (empty($form_ids) || ! isset($form_ids[0])) {
 		return false;
 	}
 
-	$post_type = get_post_type( $form_ids[0] );
+	$post_type = get_post_type($form_ids[0]);
 
-	$args = apply_filters( 'give_get_users_completed_donations_args', array(
+	$args = apply_filters('give_get_users_completed_donations_args', array(
 		'include'        => $form_ids,
 		'post_type'      => $post_type,
-		'posts_per_page' => - 1,
-	) );
+		'posts_per_page' => -1,
+	));
 
-	return apply_filters( 'give_users_completed_donations_list', get_posts( $args ) );
+	return apply_filters('give_users_completed_donations_list', get_posts($args));
 }
 
 
@@ -170,12 +170,12 @@ 
  *
  * @return      bool True if has donated, false other wise.
  */
-function give_has_donations( $user_id = null ) {
-	if ( empty( $user_id ) ) {
+function give_has_donations($user_id = null) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	if ( give_get_users_donations( $user_id, 1 ) ) {
+	if (give_get_users_donations($user_id, 1)) {
 		return true; // User has at least one donation.
 	}
 
@@ -196,23 +196,23 @@ 
  *
  * @return      array
  */
-function give_get_donation_stats_by_user( $user = '' ) {
+function give_get_donation_stats_by_user($user = '') {
 
 	$field = '';
 
-	if ( is_email( $user ) ) {
+	if (is_email($user)) {
 		$field = 'email';
-	} elseif ( is_numeric( $user ) ) {
+	} elseif (is_numeric($user)) {
 		$field = 'user_id';
 	}
 
-	$stats    = array();
-	$donor = Give()->donors->get_donor_by( $field, $user );
+	$stats = array();
+	$donor = Give()->donors->get_donor_by($field, $user);
 
-	if ( $donor ) {
-		$donor = new Give_Donor( $donor->id );
-		$stats['purchases']   = absint( $donor->purchase_count );
-		$stats['total_spent'] = give_maybe_sanitize_amount( $donor->purchase_value );
+	if ($donor) {
+		$donor = new Give_Donor($donor->id);
+		$stats['purchases']   = absint($donor->purchase_count);
+		$stats['total_spent'] = give_maybe_sanitize_amount($donor->purchase_value);
 	}
 
 	/**
@@ -220,7 +220,7 @@ 
 	 *
 	 * @since 1.7
 	 */
-	$stats = (array) apply_filters( 'give_donation_stats_by_user', $stats, $user );
+	$stats = (array) apply_filters('give_donation_stats_by_user', $stats, $user);
 
 	return $stats;
 }
@@ -238,21 +238,21 @@ 
  *
  * @return      int The total number of donations.
  */
-function give_count_donations_of_donor( $user = null ) {
+function give_count_donations_of_donor($user = null) {
 
 	// Logged in?
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
 	// Email access?
-	if ( empty( $user ) && Give()->email_access->token_email ) {
+	if (empty($user) && Give()->email_access->token_email) {
 		$user = Give()->email_access->token_email;
 	}
 
-	$stats = ! empty( $user ) ? give_get_donation_stats_by_user( $user ) : false;
+	$stats = ! empty($user) ? give_get_donation_stats_by_user($user) : false;
 
-	return isset( $stats['purchases'] ) ? $stats['purchases'] : 0;
+	return isset($stats['purchases']) ? $stats['purchases'] : 0;
 }
 
 /**
@@ -265,9 +265,9 @@ 
  *
  * @return      float The total amount the user has spent
  */
-function give_donation_total_of_user( $user = null ) {
+function give_donation_total_of_user($user = null) {
 
-	$stats = give_get_donation_stats_by_user( $user );
+	$stats = give_get_donation_stats_by_user($user);
 
 	return $stats['total_spent'];
 }
@@ -283,40 +283,40 @@ 
  *
  * @return bool
  */
-function give_validate_username( $username, $form_id = 0 ) {
+function give_validate_username($username, $form_id = 0) {
 	$valid = true;
 
 	// Validate username.
-	if ( ! empty( $username ) ) {
+	if ( ! empty($username)) {
 
 		// Sanitize username.
-		$sanitized_user_name = sanitize_user( $username, false );
+		$sanitized_user_name = sanitize_user($username, false);
 
 		// We have an user name, check if it already exists.
-		if ( username_exists( $username ) ) {
+		if (username_exists($username)) {
 			// Username already registered.
-			give_set_error( 'username_unavailable', __( 'Username already taken.', 'give' ) );
+			give_set_error('username_unavailable', __('Username already taken.', 'give'));
 			$valid = false;
 
 			// Check if it's valid.
-		} elseif ( $sanitized_user_name !== $username ) {
+		} elseif ($sanitized_user_name !== $username) {
 			// Invalid username.
-			if ( is_multisite() ) {
-				give_set_error( 'username_invalid', __( 'Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give' ) );
+			if (is_multisite()) {
+				give_set_error('username_invalid', __('Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give'));
 				$valid = false;
 			} else {
-				give_set_error( 'username_invalid', __( 'Invalid username.', 'give' ) );
+				give_set_error('username_invalid', __('Invalid username.', 'give'));
 				$valid = false;
 			}
 		}
 	} else {
 		// Username is empty.
-		give_set_error( 'username_empty', __( 'Enter a username.', 'give' ) );
+		give_set_error('username_empty', __('Enter a username.', 'give'));
 		$valid = false;
 
 		// Check if guest checkout is disable for form.
-		if ( $form_id && give_logged_in_only( $form_id ) ) {
-			give_set_error( 'registration_required', __( 'You must register or login to complete your donation.', 'give' ) );
+		if ($form_id && give_logged_in_only($form_id)) {
+			give_set_error('registration_required', __('You must register or login to complete your donation.', 'give'));
 			$valid = false;
 		}
 	}
@@ -330,7 +330,7 @@ 
 	 * @param string $username
 	 * @param bool   $form_id
 	 */
-	$valid = (bool) apply_filters( 'give_validate_username', $valid, $username, $form_id );
+	$valid = (bool) apply_filters('give_validate_username', $valid, $username, $form_id);
 
 	return $valid;
 }
@@ -346,25 +346,25 @@ 
  *
  * @return bool
  */
-function give_validate_user_email( $email, $registering_new_user = false ) {
+function give_validate_user_email($email, $registering_new_user = false) {
 	$valid = true;
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// No email.
-		give_set_error( 'email_empty', __( 'Enter an email.', 'give' ) );
+		give_set_error('email_empty', __('Enter an email.', 'give'));
 		$valid = false;
 
-	} elseif ( ! is_email( $email ) ) {
+	} elseif ( ! is_email($email)) {
 		// Validate email.
-		give_set_error( 'email_invalid', __( 'Invalid email.', 'give' ) );
+		give_set_error('email_invalid', __('Invalid email.', 'give'));
 		$valid = false;
 
-	} elseif ( $registering_new_user ) {
+	} elseif ($registering_new_user) {
 
 		// If donor email is not primary
-		if ( ! email_exists( $email ) && give_donor_email_exists( $email ) && give_is_additional_email( $email ) ) {
+		if ( ! email_exists($email) && give_donor_email_exists($email) && give_is_additional_email($email)) {
 			// Check if email exists.
-			give_set_error( 'email_used', __( 'The email address provided is already active for another user.', 'give' ) );
+			give_set_error('email_used', __('The email address provided is already active for another user.', 'give'));
 			$valid = false;
 		}
 	}
@@ -378,7 +378,7 @@ 
 	 * @param string $email
 	 * @param bool   $registering_new_user
 	 */
-	$valid = (bool) apply_filters( 'give_validate_user_email', $valid, $email, $registering_new_user );
+	$valid = (bool) apply_filters('give_validate_user_email', $valid, $email, $registering_new_user);
 
 	return $valid;
 }
@@ -394,34 +394,34 @@ 
  *
  * @return bool
  */
-function give_validate_user_password( $password = '', $confirm_password = '', $registering_new_user = false ) {
+function give_validate_user_password($password = '', $confirm_password = '', $registering_new_user = false) {
 	$valid = true;
 
 	// Passwords Validation For New Donors Only
-	if ( $registering_new_user ) {
+	if ($registering_new_user) {
 		// Password or confirmation missing.
-		if ( ! $password ) {
+		if ( ! $password) {
 			// The password is invalid.
-			give_set_error( 'password_empty', __( 'Enter a password.', 'give' ) );
+			give_set_error('password_empty', __('Enter a password.', 'give'));
 			$valid = false;
-		} elseif ( ! $confirm_password ) {
+		} elseif ( ! $confirm_password) {
 			// Confirmation password is invalid.
-			give_set_error( 'confirmation_empty', __( 'Enter the password confirmation.', 'give' ) );
+			give_set_error('confirmation_empty', __('Enter the password confirmation.', 'give'));
 			$valid = false;
 		}
 	}
 	// Passwords Validation For New Donors as well as Existing Donors
-	if ( $password || $confirm_password ) {
-		if ( strlen( $password ) < 6 || strlen( $confirm_password ) < 6 ) {
+	if ($password || $confirm_password) {
+		if (strlen($password) < 6 || strlen($confirm_password) < 6) {
 			// Seems Weak Password
-			give_set_error( 'password_weak', __( 'Passwords should have at least 6 characters.', 'give' ) );
+			give_set_error('password_weak', __('Passwords should have at least 6 characters.', 'give'));
 			$valid = false;
 		}
-		if ( $password && $confirm_password ) {
+		if ($password && $confirm_password) {
 			// Verify confirmation matches.
-			if ( $password != $confirm_password ) {
+			if ($password != $confirm_password) {
 				// Passwords do not match
-				give_set_error( 'password_mismatch', __( 'Passwords you entered do not match. Please try again.', 'give' ) );
+				give_set_error('password_mismatch', __('Passwords you entered do not match. Please try again.', 'give'));
 				$valid = false;
 			}
 		}
@@ -437,7 +437,7 @@ 
 	 * @param string $confirm_password
 	 * @param bool   $registering_new_user
 	 */
-	$valid = (bool) apply_filters( 'give_validate_user_email', $valid, $password, $confirm_password, $registering_new_user );
+	$valid = (bool) apply_filters('give_validate_user_email', $valid, $password, $confirm_password, $registering_new_user);
 
 	return $valid;
 }
@@ -455,20 +455,20 @@ 
  *
  * @return      void
  */
-function give_add_past_donations_to_new_user( $user_id ) {
+function give_add_past_donations_to_new_user($user_id) {
 
-	$email = get_the_author_meta( 'user_email', $user_id );
+	$email = get_the_author_meta('user_email', $user_id);
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'output' => 'payments',
 		's' => $email,
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 			/* @var $payment \Give_Payment */
 
-			if ( intval( give_get_payment_user_id( $payment->ID ) ) > 0 ) {
+			if (intval(give_get_payment_user_id($payment->ID)) > 0) {
 				continue;
 			}
 
@@ -480,7 +480,7 @@ 
 
 }
 
-add_action( 'user_register', 'give_add_past_donations_to_new_user' );
+add_action('user_register', 'give_add_past_donations_to_new_user');
 
 
 /**
@@ -506,34 +506,34 @@ 
  *
  * @return        array The donor's address, if any
  */
-function give_get_donor_address( $user_id = 0 ) {
-	if ( empty( $user_id ) ) {
+function give_get_donor_address($user_id = 0) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	$address = (array) get_user_meta( $user_id, '_give_user_address', true );
+	$address = (array) get_user_meta($user_id, '_give_user_address', true);
 
-	if ( ! isset( $address['line1'] ) ) {
+	if ( ! isset($address['line1'])) {
 		$address['line1'] = '';
 	}
 
-	if ( ! isset( $address['line2'] ) ) {
+	if ( ! isset($address['line2'])) {
 		$address['line2'] = '';
 	}
 
-	if ( ! isset( $address['city'] ) ) {
+	if ( ! isset($address['city'])) {
 		$address['city'] = '';
 	}
 
-	if ( ! isset( $address['zip'] ) ) {
+	if ( ! isset($address['zip'])) {
 		$address['zip'] = '';
 	}
 
-	if ( ! isset( $address['country'] ) ) {
+	if ( ! isset($address['country'])) {
 		$address['country'] = '';
 	}
 
-	if ( ! isset( $address['state'] ) ) {
+	if ( ! isset($address['state'])) {
 		$address['state'] = '';
 	}
 
@@ -553,53 +553,53 @@ 
  *
  * @return        void
  */
-function give_new_user_notification( $user_id = 0, $user_data = array() ) {
+function give_new_user_notification($user_id = 0, $user_data = array()) {
 
-	if ( empty( $user_id ) || empty( $user_data ) ) {
+	if (empty($user_id) || empty($user_data)) {
 		return;
 	}
-	$blogname = wp_specialchars_decode( get_option( 'blogname' ), ENT_QUOTES );
+	$blogname = wp_specialchars_decode(get_option('blogname'), ENT_QUOTES);
 
 	// New User Registration: Email sends to the site admin.
 	$emails = Give()->emails;
-	$emails->__set( 'heading', esc_html__( 'New User Registration', 'give' ) );
+	$emails->__set('heading', esc_html__('New User Registration', 'give'));
 
 	/* translators: %s: site name */
-	$message = sprintf( esc_attr__( 'A new user has registered on %s:', 'give' ), $blogname ) . "\r\n\r\n";
+	$message = sprintf(esc_attr__('A new user has registered on %s:', 'give'), $blogname)."\r\n\r\n";
 	/* translators: %s: user login */
-	$message .= '<strong>' . esc_attr__( 'Username:', 'give' ) . '</strong> ' . $user_data['user_login'] . "\r\n";
+	$message .= '<strong>'.esc_attr__('Username:', 'give').'</strong> '.$user_data['user_login']."\r\n";
 	/* translators: %s: user email */
-	$message .= '<strong>' . esc_attr__( 'E-mail:', 'give' ) . '</strong> ' . $user_data['user_email'] . "\r\n\r\n";
+	$message .= '<strong>'.esc_attr__('E-mail:', 'give').'</strong> '.$user_data['user_email']."\r\n\r\n";
 
-	$message .= '<a href="' . admin_url( 'user-edit.php?user_id=' . $user_id ) . '" target="_blank"> ' . esc_attr__( 'Click here to view &raquo;', 'give' ) . '</a>' . "\r\n";
+	$message .= '<a href="'.admin_url('user-edit.php?user_id='.$user_id).'" target="_blank"> '.esc_attr__('Click here to view &raquo;', 'give').'</a>'."\r\n";
 
 	$emails->send(
-		get_option( 'admin_email' ),
+		get_option('admin_email'),
 		sprintf(
 			/* translators: %s: site name */
-			esc_attr__( '[%s] New User Registration', 'give' ),
+			esc_attr__('[%s] New User Registration', 'give'),
 			$blogname
 		),
 		$message
 	);
 
 	// Account Information: Email sends to donor who registered.
-	$emails->__set( 'heading', esc_html__( 'Account Information', 'give' ) );
+	$emails->__set('heading', esc_html__('Account Information', 'give'));
 
-	$message = sprintf( esc_attr__( 'The following email contains your account information for %s:', 'give' ), $blogname ) . "\r\n\r\n";
+	$message = sprintf(esc_attr__('The following email contains your account information for %s:', 'give'), $blogname)."\r\n\r\n";
 
 	/* translators: %s: user login */
-	$message .= '<strong>' . esc_attr__( 'Username:', 'give' ) . '</strong> ' . $user_data['user_login'] . "\r\n";
+	$message .= '<strong>'.esc_attr__('Username:', 'give').'</strong> '.$user_data['user_login']."\r\n";
 	/* translators: %s: password */
-	$message .= '<strong>' . esc_attr__( 'Password:', 'give' ) . '</strong> ' . esc_attr__( '[Password entered during donation]', 'give' ) . "\r\n\r\n";
+	$message .= '<strong>'.esc_attr__('Password:', 'give').'</strong> '.esc_attr__('[Password entered during donation]', 'give')."\r\n\r\n";
 
-	$message .= '<a href="' . wp_login_url() . '" target="_blank"> ' . esc_attr__( 'Click here to login &raquo;', 'give' ) . '</a>' . "\r\n";
+	$message .= '<a href="'.wp_login_url().'" target="_blank"> '.esc_attr__('Click here to login &raquo;', 'give').'</a>'."\r\n";
 
 	$emails->send(
 		$user_data['user_email'],
 		sprintf(
 			/* translators: %s: site name */
-			esc_attr__( '[%s] Your username and password', 'give' ),
+			esc_attr__('[%s] Your username and password', 'give'),
 			$blogname
 		),
 		$message
@@ -607,7 +607,7 @@ 
 
 }
 
-add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
+add_action('give_insert_user', 'give_new_user_notification', 10, 2);
 
 
 /**
@@ -623,34 +623,34 @@ 
  *
  * @return      string
  */
-function give_get_donor_name_by( $id = 0, $from = 'donation' ) {
+function give_get_donor_name_by($id = 0, $from = 'donation') {
 
 	// ID shouldn't be empty
-	if ( empty( $id ) ) {
+	if (empty($id)) {
 		return;
 	}
 
 	$name = '';
 
-	switch ( $from ) {
+	switch ($from) {
 
 		case 'donation':
 
-			$user_info = give_get_payment_meta_user_info( $id );
-			$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$user_info = give_get_payment_meta_user_info($id);
+			$name = $user_info['first_name'].' '.$user_info['last_name'];
 
 		break;
 
 		case 'donor':
 
-			$donor = new Give_Donor( $id );
+			$donor = new Give_Donor($id);
 			$name = $donor->name;
 
 		break;
 
 	}
 
-	return trim( $name );
+	return trim($name);
 
 }
 
@@ -662,8 +662,8 @@ 
  * @param  string $email Donor Email.
  * @return boolean  The user's ID on success, and false on failure.
  */
-function give_donor_email_exists( $email ) {
-	if ( Give()->donors->get_donor_by( 'email', $email ) ) {
+function give_donor_email_exists($email) {
+	if (Give()->donors->get_donor_by('email', $email)) {
 		return true;
 	}
 	return false;
@@ -678,12 +678,12 @@ 
  *
  * @return bool
  */
-function give_is_additional_email( $email ) {
+function give_is_additional_email($email) {
 	global $wpdb;
-	$meta_table  = Give()->donor_meta->table_name;
-	$donor_id = $wpdb->get_var( $wpdb->prepare( "SELECT customer_id FROM {$meta_table} WHERE meta_key = 'additional_email' AND meta_value = %s LIMIT 1", $email ) );
+	$meta_table = Give()->donor_meta->table_name;
+	$donor_id = $wpdb->get_var($wpdb->prepare("SELECT customer_id FROM {$meta_table} WHERE meta_key = 'additional_email' AND meta_value = %s LIMIT 1", $email));
 
-	if ( empty( $donor_id ) ) {
+	if (empty($donor_id)) {
 		return false;
 	}
 	return true;

includes/error-tracking.php

Spacing +19 added lines, -19 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  * @return mixed array if errors are present, false if none found
  */
 function give_get_errors() {
-	return Give()->session->get( 'give_errors' );
+	return Give()->session->get('give_errors');
 }
 
 /**
@@ -42,23 +42,23 @@ 
  *
  * @return void
  */
-function give_set_error( $error_id, $error_message, $notice_args = array() ) {
+function give_set_error($error_id, $error_message, $notice_args = array()) {
 	$errors = give_get_errors();
-	if ( ! $errors ) {
+	if ( ! $errors) {
 		$errors = array();
 	}
 
-	if( is_array( $notice_args ) && ! empty( $notice_args ) ) {
-		$errors[ $error_id ] = array(
+	if (is_array($notice_args) && ! empty($notice_args)) {
+		$errors[$error_id] = array(
 			'message'     => $error_message,
 			'notice_args' => $notice_args,
 		);
 	} else {
 		// Backward compatibility v<1.8.11.
-		$errors[ $error_id ] = $error_message;
+		$errors[$error_id] = $error_message;
 	}
 
-	Give()->session->set( 'give_errors', $errors );
+	Give()->session->set('give_errors', $errors);
 }
 
 /**
@@ -69,7 +69,7 @@ 
  * @return void
  */
 function give_clear_errors() {
-	Give()->session->set( 'give_errors', null );
+	Give()->session->set('give_errors', null);
 }
 
 /**
@@ -82,19 +82,19 @@ 
  *
  * @return void
  */
-function give_unset_error( $error_id ) {
+function give_unset_error($error_id) {
 	$errors = give_get_errors();
-	if ( $errors ) {
+	if ($errors) {
 		/**
 		 * Check If $error_id exists in the array.
 		 * If exists then unset it.
 		 *
 		 * @since 1.8.13
 		 */
-		if ( isset( $errors[ $error_id ] ) ) {
-			unset( $errors[ $error_id ] );
+		if (isset($errors[$error_id])) {
+			unset($errors[$error_id]);
 		}
-		Give()->session->set( 'give_errors', $errors );
+		Give()->session->set('give_errors', $errors);
 	}
 }
 
@@ -105,7 +105,7 @@ 
  * @return string
  */
 function _give_die_handler() {
-	if ( defined( 'GIVE_UNIT_TESTS' ) ) {
+	if (defined('GIVE_UNIT_TESTS')) {
 		return '_give_die_handler';
 	} else {
 		die();
@@ -125,8 +125,8 @@ 
  *
  * @return void
  */
-function give_die( $message = '', $title = '', $status = 400 ) {
-	add_filter( 'wp_die_ajax_handler', '_give_die_handler', 10, 3 );
-	add_filter( 'wp_die_handler', '_give_die_handler', 10, 3 );
-	wp_die( $message, $title, array( 'response' => $status ) );
+function give_die($message = '', $title = '', $status = 400) {
+	add_filter('wp_die_ajax_handler', '_give_die_handler', 10, 3);
+	add_filter('wp_die_handler', '_give_die_handler', 10, 3);
+	wp_die($message, $title, array('response' => $status));
 }

includes/class-give-donor.php

Spacing +170 added lines, -170 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -139,32 +139,32 @@ 
 	 * @param bool $_id_or_email
 	 * @param bool $by_user_id
 	 */
-	public function __construct( $_id_or_email = false, $by_user_id = false ) {
+	public function __construct($_id_or_email = false, $by_user_id = false) {
 
 		$this->db = Give()->donors;
 
 		if (
 			false === $_id_or_email
-			|| ( is_numeric( $_id_or_email ) && (int) $_id_or_email !== absint( $_id_or_email ) )
+			|| (is_numeric($_id_or_email) && (int) $_id_or_email !== absint($_id_or_email))
 		) {
 			return false;
 		}
 
-		$by_user_id = is_bool( $by_user_id ) ? $by_user_id : false;
+		$by_user_id = is_bool($by_user_id) ? $by_user_id : false;
 
-		if ( is_numeric( $_id_or_email ) ) {
+		if (is_numeric($_id_or_email)) {
 			$field = $by_user_id ? 'user_id' : 'id';
 		} else {
 			$field = 'email';
 		}
 
-		$donor = $this->db->get_donor_by( $field, $_id_or_email );
+		$donor = $this->db->get_donor_by($field, $_id_or_email);
 
-		if ( empty( $donor ) || ! is_object( $donor ) ) {
+		if (empty($donor) || ! is_object($donor)) {
 			return false;
 		}
 
-		$this->setup_donor( $donor );
+		$this->setup_donor($donor);
 
 	}
 
@@ -180,15 +180,15 @@ 
 	 *
 	 * @return bool             If the setup was successful or not.
 	 */
-	private function setup_donor( $donor ) {
+	private function setup_donor($donor) {
 
-		if ( ! is_object( $donor ) ) {
+		if ( ! is_object($donor)) {
 			return false;
 		}
 
-		foreach ( $donor as $key => $value ) {
+		foreach ($donor as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				case 'notes':
 					$this->$key = $this->get_notes();
@@ -202,11 +202,11 @@ 
 		}
 
 		// Get donor's all email including primary email.
-		$this->emails = (array) $this->get_meta( 'additional_email', false );
-		$this->emails = array( 'primary' => $this->email ) + $this->emails;
+		$this->emails = (array) $this->get_meta('additional_email', false);
+		$this->emails = array('primary' => $this->email) + $this->emails;
 
 		// Donor ID and email are the only things that are necessary, make sure they exist.
-		if ( ! empty( $this->id ) && ! empty( $this->email ) ) {
+		if ( ! empty($this->id) && ! empty($this->email)) {
 			return true;
 		}
 
@@ -223,16 +223,16 @@ 
 	 *
 	 * @return mixed|\WP_Error
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-donor-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-donor-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -248,9 +248,9 @@ 
 	 *
 	 * @return bool|int    False if not a valid creation, donor ID if user is found or valid creation.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 || empty( $data ) ) {
+		if ($this->id != 0 || empty($data)) {
 			return false;
 		}
 
@@ -258,15 +258,15 @@ 
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
-		$args = $this->sanitize_columns( $args );
+		$args = wp_parse_args($data, $defaults);
+		$args = $this->sanitize_columns($args);
 
-		if ( empty( $args['email'] ) || ! is_email( $args['email'] ) ) {
+		if (empty($args['email']) || ! is_email($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
 		/**
@@ -276,18 +276,18 @@ 
 		 *
 		 * @param array $args Donor attributes.
 		 */
-		do_action( 'give_donor_pre_create', $args );
+		do_action('give_donor_pre_create', $args);
 
 		$created = false;
 
 		// The DB class 'add' implies an update if the donor being asked to be created already exists
-		if ( $this->db->add( $data ) ) {
+		if ($this->db->add($data)) {
 
 			// We've successfully added/updated the donor, reset the class vars with the new data
-			$donor = $this->db->get_donor_by( 'email', $args['email'] );
+			$donor = $this->db->get_donor_by('email', $args['email']);
 
 			// Setup the donor data with the values from DB
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$created = $this->id;
 		}
@@ -300,7 +300,7 @@ 
 		 * @param bool|int $created False if not a valid creation, donor ID if user is found or valid creation.
 		 * @param array $args Customer attributes.
 		 */
-		do_action( 'give_donor_post_create', $created, $args );
+		do_action('give_donor_post_create', $created, $args);
 
 		return $created;
 
@@ -316,13 +316,13 @@ 
 	 *
 	 * @return bool        If the update was successful or not.
 	 */
-	public function update( $data = array() ) {
+	public function update($data = array()) {
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return false;
 		}
 
-		$data = $this->sanitize_columns( $data );
+		$data = $this->sanitize_columns($data);
 
 		/**
 		 * Fires before updating donors.
@@ -332,14 +332,14 @@ 
 		 * @param int $donor_id Donor id.
 		 * @param array $data Donor attributes.
 		 */
-		do_action( 'give_donor_pre_update', $this->id, $data );
+		do_action('give_donor_pre_update', $this->id, $data);
 
 		$updated = false;
 
-		if ( $this->db->update( $this->id, $data ) ) {
+		if ($this->db->update($this->id, $data)) {
 
-			$donor = $this->db->get_donor_by( 'id', $this->id );
-			$this->setup_donor( $donor );
+			$donor = $this->db->get_donor_by('id', $this->id);
+			$this->setup_donor($donor);
 
 			$updated = true;
 		}
@@ -353,7 +353,7 @@ 
 		 * @param int $donor_id Donor id.
 		 * @param array $data Donor attributes.
 		 */
-		do_action( 'give_donor_post_update', $updated, $this->id, $data );
+		do_action('give_donor_post_update', $updated, $this->id, $data);
 
 		return $updated;
 	}
@@ -371,27 +371,27 @@ 
 	 *
 	 * @return bool            If the attachment was successfully.
 	 */
-	public function attach_payment( $payment_id = 0, $update_stats = true ) {
+	public function attach_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		if ( empty( $this->payment_ids ) ) {
+		if (empty($this->payment_ids)) {
 
 			$new_payment_ids = $payment_id;
 
 		} else {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			if ( in_array( $payment_id, $payment_ids ) ) {
+			if (in_array($payment_id, $payment_ids)) {
 				$update_stats = false;
 			}
 
 			$payment_ids[] = $payment_id;
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -403,20 +403,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_attach_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_attach_payment', $payment_id, $this->id);
 
-		$payment_added = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_added = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_added ) {
+		if ($payment_added) {
 
 			$this->payment_ids = $new_payment_ids;
 
 			// We added this payment successfully, increment the stats
-			if ( $update_stats ) {
-				$payment_amount = give_get_payment_amount( $payment_id );
+			if ($update_stats) {
+				$payment_amount = give_get_payment_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->increase_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->increase_value($payment_amount);
 				}
 
 				$this->increase_purchase_count();
@@ -432,7 +432,7 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_attach_payment', $payment_added, $payment_id, $this->id );
+		do_action('give_donor_post_attach_payment', $payment_added, $payment_id, $this->id);
 
 		return $payment_added;
 	}
@@ -450,33 +450,33 @@ 
 	 *
 	 * @return boolean               If the removal was successful.
 	 */
-	public function remove_payment( $payment_id = 0, $update_stats = true ) {
+	public function remove_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = new Give_Payment( $payment_id );
+		$payment = new Give_Payment($payment_id);
 
-		if ( 'publish' !== $payment->status && 'revoked' !== $payment->status ) {
+		if ('publish' !== $payment->status && 'revoked' !== $payment->status) {
 			$update_stats = false;
 		}
 
 		$new_payment_ids = '';
 
-		if ( ! empty( $this->payment_ids ) ) {
+		if ( ! empty($this->payment_ids)) {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			$pos = array_search( $payment_id, $payment_ids );
-			if ( false === $pos ) {
+			$pos = array_search($payment_id, $payment_ids);
+			if (false === $pos) {
 				return false;
 			}
 
-			unset( $payment_ids[ $pos ] );
-			$payment_ids = array_filter( $payment_ids );
+			unset($payment_ids[$pos]);
+			$payment_ids = array_filter($payment_ids);
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -488,20 +488,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_remove_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_remove_payment', $payment_id, $this->id);
 
-		$payment_removed = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_removed = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_removed ) {
+		if ($payment_removed) {
 
 			$this->payment_ids = $new_payment_ids;
 
-			if ( $update_stats ) {
+			if ($update_stats) {
 				// We removed this payment successfully, decrement the stats
-				$payment_amount = give_get_payment_amount( $payment_id );
+				$payment_amount = give_get_payment_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->decrease_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->decrease_value($payment_amount);
 				}
 
 				$this->decrease_donation_count();
@@ -517,7 +517,7 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id );
+		do_action('give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id);
 
 		return $payment_removed;
 
@@ -533,10 +533,10 @@ 
 	 *
 	 * @return int        The donation count.
 	 */
-	public function increase_purchase_count( $count = 1 ) {
+	public function increase_purchase_count($count = 1) {
 
 		// Make sure it's numeric and not negative.
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
@@ -550,9 +550,9 @@ 
 		 * @param int $count The number to increase by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_increase_donation_count', $count, $this->id );
+		do_action('give_donor_pre_increase_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -565,7 +565,7 @@ 
 		 * @param int $count The number increased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -580,16 +580,16 @@ 
 	 *
 	 * @return mixed      If successful, the new count, otherwise false.
 	 */
-	public function decrease_donation_count( $count = 1 ) {
+	public function decrease_donation_count($count = 1) {
 
 		// Make sure it's numeric and not negative
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
 		$new_total = (int) $this->purchase_count - (int) $count;
 
-		if ( $new_total < 0 ) {
+		if ($new_total < 0) {
 			$new_total = 0;
 		}
 
@@ -601,9 +601,9 @@ 
 		 * @param int $count The number to decrease by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_decrease_donation_count', $count, $this->id );
+		do_action('give_donor_pre_decrease_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -616,7 +616,7 @@ 
 		 * @param int $count The number decreased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -631,9 +631,9 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function increase_value( $value = 0.00 ) {
+	public function increase_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) + $value;
+		$new_value = floatval($this->purchase_value) + $value;
 
 		/**
 		 * Fires before increasing donor lifetime value.
@@ -643,9 +643,9 @@ 
 		 * @param float $value The value to increase by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_increase_value', $value, $this->id );
+		do_action('give_donor_pre_increase_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -658,7 +658,7 @@ 
 		 * @param float $value The value increased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_increase_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_increase_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -673,11 +673,11 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function decrease_value( $value = 0.00 ) {
+	public function decrease_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) - $value;
+		$new_value = floatval($this->purchase_value) - $value;
 
-		if ( $new_value < 0 ) {
+		if ($new_value < 0) {
 			$new_value = 0.00;
 		}
 
@@ -689,9 +689,9 @@ 
 		 * @param float $value The value to decrease by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_decrease_value', $value, $this->id );
+		do_action('give_donor_pre_decrease_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -704,7 +704,7 @@ 
 		 * @param float $value The value decreased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_decrease_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -723,7 +723,7 @@ 
 	 *
 	 * @return mixed              If successful, the new donation stat value, otherwise false.
 	 */
-	public function update_donation_value( $curr_amount, $new_amount ) {
+	public function update_donation_value($curr_amount, $new_amount) {
 		/**
 		 * Payment total difference value can be:
 		 *  zero   (in case amount not change)
@@ -733,15 +733,15 @@ 
 		$payment_total_diff = $new_amount - $curr_amount;
 
 		// We do not need to update donation stat if donation did not change.
-		if ( ! $payment_total_diff ) {
+		if ( ! $payment_total_diff) {
 			return false;
 		}
 
-		if ( $payment_total_diff > 0 ) {
-			$this->increase_value( $payment_total_diff );
+		if ($payment_total_diff > 0) {
+			$this->increase_value($payment_total_diff);
 		} else {
 			// Pass payment total difference as +ve value to decrease amount from user lifetime stat.
-			$this->decrease_value( - $payment_total_diff );
+			$this->decrease_value( -$payment_total_diff );
 		}
 
 		return $this->purchase_value;
@@ -758,15 +758,15 @@ 
 	 *
 	 * @return array       The notes requested.
 	 */
-	public function get_notes( $length = 20, $paged = 1 ) {
+	public function get_notes($length = 20, $paged = 1) {
 
-		$length = is_numeric( $length ) ? $length : 20;
-		$offset = is_numeric( $paged ) && $paged != 1 ? ( ( absint( $paged ) - 1 ) * $length ) : 0;
+		$length = is_numeric($length) ? $length : 20;
+		$offset = is_numeric($paged) && $paged != 1 ? ((absint($paged) - 1) * $length) : 0;
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		$desired_notes = array_slice( $notes_array, $offset, $length );
+		$desired_notes = array_slice($notes_array, $offset, $length);
 
 		return $desired_notes;
 
@@ -783,9 +783,9 @@ 
 	public function get_notes_count() {
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		return count( $notes_array );
+		return count($notes_array);
 
 	}
 
@@ -799,22 +799,22 @@ 
 	 *
 	 * @return string|boolean The new note if added successfully, false otherwise.
 	 */
-	public function add_note( $note = '' ) {
+	public function add_note($note = '') {
 
-		$note = trim( $note );
-		if ( empty( $note ) ) {
+		$note = trim($note);
+		if (empty($note)) {
 			return false;
 		}
 
 		$notes = $this->get_raw_notes();
 
-		if ( empty( $notes ) ) {
+		if (empty($notes)) {
 			$notes = '';
 		}
 
-		$note_string = date_i18n( 'F j, Y H:i:s', current_time( 'timestamp' ) ) . ' - ' . $note;
-		$new_note    = apply_filters( 'give_customer_add_note_string', $note_string );
-		$notes       .= "\n\n" . $new_note;
+		$note_string = date_i18n('F j, Y H:i:s', current_time('timestamp')).' - '.$note;
+		$new_note    = apply_filters('give_customer_add_note_string', $note_string);
+		$notes .= "\n\n".$new_note;
 
 		/**
 		 * Fires before donor note is added.
@@ -824,11 +824,11 @@ 
 		 * @param string $new_note New note to add.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_add_note', $new_note, $this->id );
+		do_action('give_donor_pre_add_note', $new_note, $this->id);
 
-		$updated = $this->update( array( 'notes' => $notes ) );
+		$updated = $this->update(array('notes' => $notes));
 
-		if ( $updated ) {
+		if ($updated) {
 			$this->notes = $this->get_notes();
 		}
 
@@ -841,7 +841,7 @@ 
 		 * @param string $new_note New note added.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_add_note', $this->notes, $new_note, $this->id );
+		do_action('give_donor_post_add_note', $this->notes, $new_note, $this->id);
 
 		// Return the formatted note, so we can test, as well as update any displays
 		return $new_note;
@@ -858,7 +858,7 @@ 
 	 */
 	private function get_raw_notes() {
 
-		$all_notes = $this->db->get_column( 'notes', $this->id );
+		$all_notes = $this->db->get_column('notes', $this->id);
 
 		return $all_notes;
 
@@ -875,8 +875,8 @@ 
 	 *
 	 * @return mixed            Will be an array if $single is false. Will be value of meta data field if $single is true.
 	 */
-	public function get_meta( $meta_key = '', $single = true ) {
-		return Give()->donor_meta->get_meta( $this->id, $meta_key, $single );
+	public function get_meta($meta_key = '', $single = true) {
+		return Give()->donor_meta->get_meta($this->id, $meta_key, $single);
 	}
 
 	/**
@@ -891,8 +891,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function add_meta( $meta_key = '', $meta_value, $unique = false ) {
-		return Give()->donor_meta->add_meta( $this->id, $meta_key, $meta_value, $unique );
+	public function add_meta($meta_key = '', $meta_value, $unique = false) {
+		return Give()->donor_meta->add_meta($this->id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -907,8 +907,8 @@ 
 	 *
 	 * @return bool               False on failure, true if success.
 	 */
-	public function update_meta( $meta_key = '', $meta_value, $prev_value = '' ) {
-		return Give()->donor_meta->update_meta( $this->id, $meta_key, $meta_value, $prev_value );
+	public function update_meta($meta_key = '', $meta_value, $prev_value = '') {
+		return Give()->donor_meta->update_meta($this->id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -922,8 +922,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function delete_meta( $meta_key = '', $meta_value = '' ) {
-		return Give()->donor_meta->delete_meta( $this->id, $meta_key, $meta_value );
+	public function delete_meta($meta_key = '', $meta_value = '') {
+		return Give()->donor_meta->delete_meta($this->id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -936,51 +936,51 @@ 
 	 *
 	 * @return array       The sanitized data, based off column defaults.
 	 */
-	private function sanitize_columns( $data ) {
+	private function sanitize_columns($data) {
 
 		$columns        = $this->db->get_columns();
 		$default_values = $this->db->get_column_defaults();
 
-		foreach ( $columns as $key => $type ) {
+		foreach ($columns as $key => $type) {
 
 			// Only sanitize data that we were provided
-			if ( ! array_key_exists( $key, $data ) ) {
+			if ( ! array_key_exists($key, $data)) {
 				continue;
 			}
 
-			switch ( $type ) {
+			switch ($type) {
 
 				case '%s':
-					if ( 'email' == $key ) {
-						$data[ $key ] = sanitize_email( $data[ $key ] );
-					} elseif ( 'notes' == $key ) {
-						$data[ $key ] = strip_tags( $data[ $key ] );
+					if ('email' == $key) {
+						$data[$key] = sanitize_email($data[$key]);
+					} elseif ('notes' == $key) {
+						$data[$key] = strip_tags($data[$key]);
 					} else {
-						$data[ $key ] = sanitize_text_field( $data[ $key ] );
+						$data[$key] = sanitize_text_field($data[$key]);
 					}
 					break;
 
 				case '%d':
-					if ( ! is_numeric( $data[ $key ] ) || (int) $data[ $key ] !== absint( $data[ $key ] ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_numeric($data[$key]) || (int) $data[$key] !== absint($data[$key])) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = absint( $data[ $key ] );
+						$data[$key] = absint($data[$key]);
 					}
 					break;
 
 				case '%f':
 					// Convert what was given to a float
-					$value = floatval( $data[ $key ] );
+					$value = floatval($data[$key]);
 
-					if ( ! is_float( $value ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_float($value)) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = $value;
+						$data[$key] = $value;
 					}
 					break;
 
 				default:
-					$data[ $key ] = sanitize_text_field( $data[ $key ] );
+					$data[$key] = sanitize_text_field($data[$key]);
 					break;
 
 			}
@@ -1000,33 +1000,33 @@ 
 	 *
 	 * @return bool            If the email was added successfully
 	 */
-	public function add_email( $email = '', $primary = false ) {
-		if ( ! is_email( $email ) ) {
+	public function add_email($email = '', $primary = false) {
+		if ( ! is_email($email)) {
 			return false;
 		}
-		$existing = new Give_Donor( $email );
+		$existing = new Give_Donor($email);
 
-		if ( $existing->id > 0 ) {
+		if ($existing->id > 0) {
 			// Email address already belongs to another donor
 			return false;
 		}
 
-		if ( email_exists( $email ) ) {
-			$user = get_user_by( 'email', $email );
-			if ( $user->ID != $this->user_id ) {
+		if (email_exists($email)) {
+			$user = get_user_by('email', $email);
+			if ($user->ID != $this->user_id) {
 				return false;
 			}
 		}
 
-		do_action( 'give_donor_pre_add_email', $email, $this->id, $this );
+		do_action('give_donor_pre_add_email', $email, $this->id, $this);
 
 		// Add is used to ensure duplicate emails are not added
-		$ret = (bool) $this->add_meta( 'additional_email', $email );
+		$ret = (bool) $this->add_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_add_email', $email, $this->id, $this );
+		do_action('give_donor_post_add_email', $email, $this->id, $this);
 
-		if ( $ret && true === $primary ) {
-			$this->set_primary_email( $email );
+		if ($ret && true === $primary) {
+			$this->set_primary_email($email);
 		}
 
 		return $ret;
@@ -1042,16 +1042,16 @@ 
 	 *
 	 * @return bool          If the email was removed successfully.
 	 */
-	public function remove_email( $email = '' ) {
-		if ( ! is_email( $email ) ) {
+	public function remove_email($email = '') {
+		if ( ! is_email($email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_remove_email', $email, $this->id, $this );
+		do_action('give_donor_pre_remove_email', $email, $this->id, $this);
 
-		$ret = (bool) $this->delete_meta( 'additional_email', $email );
+		$ret = (bool) $this->delete_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_remove_email', $email, $this->id, $this );
+		do_action('give_donor_post_remove_email', $email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1068,16 +1068,16 @@ 
 	 *
 	 * @return bool                      If the email was set as primary successfully.
 	 */
-	public function set_primary_email( $new_primary_email = '' ) {
-		if ( ! is_email( $new_primary_email ) ) {
+	public function set_primary_email($new_primary_email = '') {
+		if ( ! is_email($new_primary_email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this);
 
-		$existing = new Give_Donor( $new_primary_email );
+		$existing = new Give_Donor($new_primary_email);
 
-		if ( $existing->id > 0 && (int) $existing->id !== (int) $this->id ) {
+		if ($existing->id > 0 && (int) $existing->id !== (int) $this->id) {
 			// This email belongs to another donor.
 			return false;
 		}
@@ -1085,21 +1085,21 @@ 
 		$old_email = $this->email;
 
 		// Update donor record with new email.
-		$update = $this->update( array( 'email' => $new_primary_email ) );
+		$update = $this->update(array('email' => $new_primary_email));
 
 		// Remove new primary from list of additional emails.
-		$remove = $this->remove_email( $new_primary_email );
+		$remove = $this->remove_email($new_primary_email);
 
 		// Add old email to additional emails list.
-		$add = $this->add_email( $old_email );
+		$add = $this->add_email($old_email);
 
 		$ret = $update && $remove && $add;
 
-		if ( $ret ) {
+		if ($ret) {
 			$this->email = $new_primary_email;
 		}
 
-		do_action( 'give_donor_post_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_post_set_primary_email', $new_primary_email, $this->id, $this);
 
 		return $ret;
 	}

Doc Comments +2 added lines, -2 removed lines

@@ -902,7 +902,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $meta_key Metadata key. Default is empty.
-	 * @param  mixed $meta_value Metadata value.
+	 * @param  integer $meta_value Metadata value.
 	 * @param  mixed $prev_value Optional. Previous value to check before removing. Default is empty.
 	 *
 	 * @return bool               False on failure, true if success.
@@ -918,7 +918,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $meta_key Metadata name. Default is empty.
-	 * @param  mixed $meta_value Optional. Metadata value. Default is empty.
+	 * @param  string $meta_value Optional. Metadata value. Default is empty.
 	 *
 	 * @return bool               False for failure. True for success.
 	 */

includes/class-give-db.php

Indentation +28 added lines, -28 removed lines

@@ -165,16 +165,16 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int    $column       Column ID.
-     * @param  string $column_where Column name.
-     * @param  string $column_value Column value.
-     *
+	 *
+	 * @param  int    $column       Column ID.
+	 * @param  string $column_where Column name.
+	 * @param  string $column_value Column value.
+	 *
 	 * @return string
 	 */
 	public function get_column_by( $column, $column_where, $column_value ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Bailout.
 		if ( empty( $column ) || empty( $column_where ) || empty( $column_value ) ) {
@@ -191,15 +191,15 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  array  $data
-     * @param  string $type
-     *
+	 *
+	 * @param  array  $data
+	 * @param  string $type
+	 *
 	 * @return int
 	 */
 	public function insert( $data, $type = '' ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Set default values.
 		$data = wp_parse_args( $data, $this->get_column_defaults() );
@@ -246,16 +246,16 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int    $row_id Column ID
-     * @param  array  $data
-     * @param  string $where  Column value
-     *
+	 *
+	 * @param  int    $row_id Column ID
+	 * @param  array  $data
+	 * @param  string $where  Column value
+	 *
 	 * @return bool
 	 */
 	public function update( $row_id, $data = array(), $where = '' ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Row ID must be positive integer
 		$row_id = absint( $row_id );
@@ -293,14 +293,14 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int $row_id Column ID.
-     *
+	 *
+	 * @param  int $row_id Column ID.
+	 *
 	 * @return bool
 	 */
 	public function delete( $row_id = 0 ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Row ID must be positive integer
 		$row_id = absint( $row_id );
@@ -321,13 +321,13 @@ 
 	 *
 	 * @since  1.3.2
 	 * @access public
-     *
+	 *
 	 * @param  string $table The table name.
-     *
+	 *
 	 * @return bool          If the table name exists.
 	 */
 	public function table_exists( $table ) {
-        /* @var WPDB $wpdb */
+		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		$table = sanitize_text_field( $table );

Spacing +46 added lines, -46 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -98,16 +98,16 @@ 
 	 *
 	 * @return object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return null;
 		}
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -121,18 +121,18 @@ 
 	 *
 	 * @return object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -146,18 +146,18 @@ 
 	 *
 	 * @return string      Column value.
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -172,18 +172,18 @@ 
      *
 	 * @return string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $column_where ) || empty( $column_value ) ) {
+		if (empty($column) || empty($column_where) || empty($column_value)) {
 			return null;
 		}
 
-		$column_where = esc_sql( $column_where );
-		$column       = esc_sql( $column );
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value ) );
+		$column_where = esc_sql($column_where);
+		$column       = esc_sql($column);
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value));
 	}
 
 	/**
@@ -197,12 +197,12 @@ 
      *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Set default values.
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
 		/**
 		 * Fires before inserting data to the database.
@@ -211,22 +211,22 @@ 
 		 *
 		 * @param array $data
 		 */
-		do_action( "give_pre_insert_{$type}", $data );
+		do_action("give_pre_insert_{$type}", $data);
 
 		// Initialise column format array
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 
 		/**
 		 * Fires after inserting data to the database.
@@ -236,7 +236,7 @@ 
 		 * @param int   $insert_id
 		 * @param array $data
 		 */
-		do_action( "give_post_insert_{$type}", $wpdb->insert_id, $data );
+		do_action("give_post_insert_{$type}", $wpdb->insert_id, $data);
 
 		return $wpdb->insert_id;
 	}
@@ -253,18 +253,18 @@ 
      *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( empty( $where ) ) {
+		if (empty($where)) {
 			$where = $this->primary_key;
 		}
 
@@ -272,16 +272,16 @@ 
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		if ( false === $wpdb->update( $this->table_name, $data, array( $where => $row_id ), $column_formats ) ) {
+		if (false === $wpdb->update($this->table_name, $data, array($where => $row_id), $column_formats)) {
 			return false;
 		}
 
@@ -298,18 +298,18 @@ 
      *
 	 * @return bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id))) {
 			return false;
 		}
 
@@ -326,13 +326,13 @@ 
      *
 	 * @return bool          If the table name exists.
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
         /* @var WPDB $wpdb */
 		global $wpdb;
 
-		$table = sanitize_text_field( $table );
+		$table = sanitize_text_field($table);
 
-		return $wpdb->get_var( $wpdb->prepare( "SHOW TABLES LIKE '%s'", $table ) ) === $table;
+		return $wpdb->get_var($wpdb->prepare("SHOW TABLES LIKE '%s'", $table)) === $table;
 	}
 
 	/**
@@ -344,7 +344,7 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run.
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 	/**
@@ -354,8 +354,8 @@ 
 	 * @access public
 	 */
 	public function register_table() {
-		$current_version = get_option( $this->table_name . '_db_version' );
-		if ( ! $current_version || version_compare( $current_version, $this->version, '<' ) ) {
+		$current_version = get_option($this->table_name.'_db_version');
+		if ( ! $current_version || version_compare($current_version, $this->version, '<')) {
 			$this->create_table();
 		}
 	}
@@ -366,5 +366,5 @@ 
 	 * @since  1.8.9
 	 * @access public
 	 */
-	public function create_table(){}
+	public function create_table() {}
 }

includes/class-give-session.php

Spacing +76 added lines, -76 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -74,57 +74,57 @@ 
 	public function __construct() {
 
 		$this->use_php_sessions = $this->use_php_sessions();
-		$this->exp_option       = give_get_option( 'session_lifetime' );
+		$this->exp_option       = give_get_option('session_lifetime');
 
 		// PHP Sessions.
-		if ( $this->use_php_sessions ) {
+		if ($this->use_php_sessions) {
 
-			if ( is_multisite() ) {
+			if (is_multisite()) {
 
-				$this->prefix = '_' . get_current_blog_id();
+				$this->prefix = '_'.get_current_blog_id();
 
 			}
 
-			add_action( 'init', array( $this, 'maybe_start_session' ), - 2 );
+			add_action('init', array($this, 'maybe_start_session'), - 2);
 
 		} else {
 
-			if ( ! $this->should_start_session() ) {
+			if ( ! $this->should_start_session()) {
 				return;
 			}
 
 			// Use WP_Session.
-			if ( ! defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'give_wp_session' );
+			if ( ! defined('WP_SESSION_COOKIE')) {
+				define('WP_SESSION_COOKIE', 'give_wp_session');
 			}
 
-			if ( ! class_exists( 'Recursive_ArrayAccess' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-recursive-arrayaccess.php';
+			if ( ! class_exists('Recursive_ArrayAccess')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-recursive-arrayaccess.php';
 			}
 
 			// Include utilities class
-			if ( ! class_exists( 'WP_Session_Utils' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session-utils.php';
+			if ( ! class_exists('WP_Session_Utils')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session-utils.php';
 			}
-			if ( ! class_exists( 'WP_Session' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/wp-session.php';
+			if ( ! class_exists('WP_Session')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session.php';
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/wp-session.php';
 			}
 
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+			add_filter('wp_session_expiration_variant', array($this, 'set_expiration_variant_time'), 99999);
+			add_filter('wp_session_expiration', array($this, 'set_expiration_time'), 99999);
 
 		}
 
 		// Init Session.
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), 9999 );
+		if (empty($this->session) && ! $this->use_php_sessions) {
+			add_action('plugins_loaded', array($this, 'init'), 9999);
 		} else {
-			add_action( 'init', array( $this, 'init' ), - 1 );
+			add_action('init', array($this, 'init'), - 1);
 		}
 
 		// Set cookie on Donation Completion page.
-		add_action( 'give_pre_process_donation', array( $this, 'set_session_cookies' ) );
+		add_action('give_pre_process_donation', array($this, 'set_session_cookies'));
 
 	}
 
@@ -140,8 +140,8 @@ 
 	 */
 	public function init() {
 
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION[ 'give' . $this->prefix ] ) && is_array( $_SESSION[ 'give' . $this->prefix ] ) ? $_SESSION[ 'give' . $this->prefix ] : array();
+		if ($this->use_php_sessions) {
+			$this->session = isset($_SESSION['give'.$this->prefix]) && is_array($_SESSION['give'.$this->prefix]) ? $_SESSION['give'.$this->prefix] : array();
 		} else {
 			$this->session = WP_Session::get_instance();
 		}
@@ -176,37 +176,37 @@ 
 	 *
 	 * @return string|array      Session variable.
 	 */
-	public function get( $key ) {
-		$key    = sanitize_key( $key );
+	public function get($key) {
+		$key    = sanitize_key($key);
 		$return = false;
 
-		if ( isset( $this->session[ $key ] ) && ! empty( $this->session[ $key ] ) ) {
+		if (isset($this->session[$key]) && ! empty($this->session[$key])) {
 
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $this->session[ $key ], $matches );
-			if ( ! empty( $matches ) ) {
-				$this->set( $key, null );
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $this->session[$key], $matches);
+			if ( ! empty($matches)) {
+				$this->set($key, null);
 
 				return false;
 			}
 
-			if ( is_numeric( $this->session[ $key ] ) ) {
-				$return = $this->session[ $key ];
+			if (is_numeric($this->session[$key])) {
+				$return = $this->session[$key];
 			} else {
 
-				$maybe_json = json_decode( $this->session[ $key ] );
+				$maybe_json = json_decode($this->session[$key]);
 
 				// Since json_last_error is PHP 5.3+, we have to rely on a `null` value for failing to parse JSON.
-				if ( is_null( $maybe_json ) ) {
-					$is_serialized = is_serialized( $this->session[ $key ] );
-					if ( $is_serialized ) {
-						$value = @unserialize( $this->session[ $key ] );
-						$this->set( $key, (array) $value );
+				if (is_null($maybe_json)) {
+					$is_serialized = is_serialized($this->session[$key]);
+					if ($is_serialized) {
+						$value = @unserialize($this->session[$key]);
+						$this->set($key, (array) $value);
 						$return = $value;
 					} else {
-						$return = $this->session[ $key ];
+						$return = $this->session[$key];
 					}
 				} else {
-					$return = json_decode( $this->session[ $key ], true );
+					$return = json_decode($this->session[$key], true);
 				}
 
 			}
@@ -229,21 +229,21 @@ 
 	 *
 	 * @return string        Session variable.
 	 */
-	public function set( $key, $value ) {
+	public function set($key, $value) {
 
-		$key = sanitize_key( $key );
+		$key = sanitize_key($key);
 
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = wp_json_encode( $value );
+		if (is_array($value)) {
+			$this->session[$key] = wp_json_encode($value);
 		} else {
-			$this->session[ $key ] = esc_attr( $value );
+			$this->session[$key] = esc_attr($value);
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION[ 'give' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['give'.$this->prefix] = $this->session;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -258,10 +258,10 @@ 
 	 * @hook
 	 */
 	public function set_session_cookies() {
-		if ( ! headers_sent() ) {
-			$lifetime = current_time( 'timestamp' ) + $this->set_expiration_time();
-			@setcookie( session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
-			@setcookie( session_name() . '_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
+		if ( ! headers_sent()) {
+			$lifetime = current_time('timestamp') + $this->set_expiration_time();
+			@setcookie(session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
+			@setcookie(session_name().'_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -277,7 +277,7 @@ 
 	 */
 	public function set_expiration_variant_time() {
 
-		return ( ! empty( $this->exp_option ) ? ( intval( $this->exp_option ) - 3600 ) : 30 * 60 * 23 );
+		return ( ! empty($this->exp_option) ? (intval($this->exp_option) - 3600) : 30 * 60 * 23);
 	}
 
 	/**
@@ -292,7 +292,7 @@ 
 	 */
 	public function set_expiration_time() {
 
-		return ( ! empty( $this->exp_option ) ? intval( $this->exp_option ) : 30 * 60 * 24 );
+		return ( ! empty($this->exp_option) ? intval($this->exp_option) : 30 * 60 * 24);
 	}
 
 	/**
@@ -310,21 +310,21 @@ 
 		$ret = false;
 
 		// If the database variable is already set, no need to run auto detection.
-		$give_use_php_sessions = (bool) get_option( 'give_use_php_sessions' );
+		$give_use_php_sessions = (bool) get_option('give_use_php_sessions');
 
-		if ( ! $give_use_php_sessions ) {
+		if ( ! $give_use_php_sessions) {
 
 			// Attempt to detect if the server supports PHP sessions.
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
+			if (function_exists('session_start') && ! ini_get('safe_mode')) {
 
-				$this->set( 'give_use_php_sessions', 1 );
+				$this->set('give_use_php_sessions', 1);
 
-				if ( $this->get( 'give_use_php_sessions' ) ) {
+				if ($this->get('give_use_php_sessions')) {
 
 					$ret = true;
 
 					// Set the database option.
-					update_option( 'give_use_php_sessions', true );
+					update_option('give_use_php_sessions', true);
 
 				}
 			}
@@ -334,13 +334,13 @@ 
 		}
 
 		// Enable or disable PHP Sessions based on the GIVE_USE_PHP_SESSIONS constant.
-		if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && GIVE_USE_PHP_SESSIONS ) {
+		if (defined('GIVE_USE_PHP_SESSIONS') && GIVE_USE_PHP_SESSIONS) {
 			$ret = true;
-		} elseif ( defined( 'GIVE_USE_PHP_SESSIONS' ) && ! GIVE_USE_PHP_SESSIONS ) {
+		} elseif (defined('GIVE_USE_PHP_SESSIONS') && ! GIVE_USE_PHP_SESSIONS) {
 			$ret = false;
 		}
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -357,9 +357,9 @@ 
 
 		$start_session = true;
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {
+		if ( ! empty($_SERVER['REQUEST_URI'])) {
 
-			$blacklist = apply_filters( 'give_session_start_uri_blacklist', array(
+			$blacklist = apply_filters('give_session_start_uri_blacklist', array(
 				'feed',
 				'feed',
 				'feed/rss',
@@ -367,21 +367,21 @@ 
 				'feed/rdf',
 				'feed/atom',
 				'comments/feed/',
-			) );
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' );
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist ) ) {
+			));
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/');
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist)) {
 				$start_session = false;
 			}
-			if ( false !== strpos( $uri, 'feed=' ) ) {
+			if (false !== strpos($uri, 'feed=')) {
 				$start_session = false;
 			}
-			if ( is_admin() ) {
+			if (is_admin()) {
 				$start_session = false;
 			}
 		}
 
-		return apply_filters( 'give_start_session', $start_session );
+		return apply_filters('give_start_session', $start_session);
 	}
 
 	/**
@@ -397,11 +397,11 @@ 
 	 */
 	public function maybe_start_session() {
 
-		if ( ! $this->should_start_session() ) {
+		if ( ! $this->should_start_session()) {
 			return;
 		}
 
-		if ( ! session_id() && ! headers_sent() ) {
+		if ( ! session_id() && ! headers_sent()) {
 			session_start();
 		}
 
@@ -420,9 +420,9 @@ 
 
 		$expiration = false;
 
-		if ( session_id() && isset( $_COOKIE[ session_name() . '_expiration' ] ) ) {
+		if (session_id() && isset($_COOKIE[session_name().'_expiration'])) {
 
-			$expiration = date( 'D, d M Y h:i:s', intval( $_COOKIE[ session_name() . '_expiration' ] ) );
+			$expiration = date('D, d M Y h:i:s', intval($_COOKIE[session_name().'_expiration']));
 
 		}
 

includes/admin/upgrades/class-give-updates.php

Spacing +79 added lines, -79 removed lines

@@ -70,24 +70,24 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function register( $args ) {
+	public function register($args) {
 		$args_default = array(
 			'id'       => '',
 			'version'  => '',
 			'callback' => '',
 		);
 
-		$args = wp_parse_args( $args, $args_default );
+		$args = wp_parse_args($args, $args_default);
 
 		// You can only register database upgrade.
 		$args['type'] = 'database';
 
 		// Bailout.
-		if ( empty( $args['id'] ) || empty( $args['version'] ) || empty( $args['callback'] ) || ! is_callable( $args['callback'] ) ) {
+		if (empty($args['id']) || empty($args['version']) || empty($args['callback']) || ! is_callable($args['callback'])) {
 			return;
 		}
 
-		$this->updates[ $args['type'] ][] = $args;
+		$this->updates[$args['type']][] = $args;
 	}
 
 
@@ -102,32 +102,32 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_updates( $update_type = '', $status = 'all' ) {
+	public function get_updates($update_type = '', $status = 'all') {
 		// return all updates.
-		if ( empty( $update_type ) ) {
+		if (empty($update_type)) {
 			return $this->updates;
 		}
 
 		// Get specific update.
-		$updates = ! empty( $this->updates[ $update_type ] ) ? $this->updates[ $update_type ] : array();
+		$updates = ! empty($this->updates[$update_type]) ? $this->updates[$update_type] : array();
 
 		// Bailout.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			return $updates;
 		}
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'new':
 				// Remove already completed updates.
 				$completed_updates = give_get_completed_upgrades();
 
-				if ( ! empty( $completed_updates ) ) {
-					foreach ( $updates as $index => $update ) {
-						if ( in_array( $update['id'], $completed_updates ) ) {
-							unset( $updates[ $index ] );
+				if ( ! empty($completed_updates)) {
+					foreach ($updates as $index => $update) {
+						if (in_array($update['id'], $completed_updates)) {
+							unset($updates[$index]);
 						}
 					}
-					$updates = array_values( $updates );
+					$updates = array_values($updates);
 				}
 
 				break;
@@ -144,7 +144,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( is_null( self::$instance ) ) {
+		if (is_null(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -162,16 +162,16 @@ 
 		/**
 		 * Setup hooks.
 		 */
-		add_action( 'init', array( $this, '__register_upgrade' ), 9999 );
-		add_action( 'admin_init', array( $this, '__change_donations_label' ), 9999 );
-		add_action( 'admin_menu', array( $this, '__register_menu' ), 9999 );
-		add_action( 'give_set_upgrade_completed', array( $this, '__flush_resume_updates' ), 9999 );
-		add_action( 'wp_ajax_give_do_ajax_updates', array( $this, '__give_ajax_updates' ) );
+		add_action('init', array($this, '__register_upgrade'), 9999);
+		add_action('admin_init', array($this, '__change_donations_label'), 9999);
+		add_action('admin_menu', array($this, '__register_menu'), 9999);
+		add_action('give_set_upgrade_completed', array($this, '__flush_resume_updates'), 9999);
+		add_action('wp_ajax_give_do_ajax_updates', array($this, '__give_ajax_updates'));
 
 		/**
 		 * Load file
 		 */
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 	}
 
 	/**
@@ -184,12 +184,12 @@ 
 		$addons         = give_get_plugins();
 		$plugin_updates = get_plugin_updates();
 
-		foreach ( $addons as $key => $info ) {
-			if ( 'active' != $info['Status'] || 'add-on' != $info['Type'] || empty( $plugin_updates[ $key ] ) ) {
+		foreach ($addons as $key => $info) {
+			if ('active' != $info['Status'] || 'add-on' != $info['Type'] || empty($plugin_updates[$key])) {
 				continue;
 			}
 
-			$this->updates['plugin'][] = array_merge( $info, (array) $plugin_updates[ $key ] );
+			$this->updates['plugin'][] = array_merge($info, (array) $plugin_updates[$key]);
 		}
 	}
 
@@ -201,7 +201,7 @@ 
 	 * @access public
 	 */
 	public function __register_upgrade() {
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
@@ -210,7 +210,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_register_updates', $this );
+		do_action('give_register_updates', $this);
 	}
 
 	/**
@@ -224,17 +224,17 @@ 
 		global $submenu;
 
 		// Bailout.
-		if ( empty( $menu ) || ! $this->get_update_count() ) {
+		if (empty($menu) || ! $this->get_update_count()) {
 			return;
 		}
 
-		foreach ( $menu as $index => $menu_item ) {
-			if ( 'edit.php?post_type=give_forms' !== $menu_item[2] ) {
+		foreach ($menu as $index => $menu_item) {
+			if ('edit.php?post_type=give_forms' !== $menu_item[2]) {
 				continue;
 			}
 
-			$menu[ $index ][0] = sprintf(
-				__( 'Donations %s', 'give' ),
+			$menu[$index][0] = sprintf(
+				__('Donations %s', 'give'),
 				sprintf(
 					'<span class="update-plugins count-%1$d"><span class="plugin-count">%1$d</span></span>',
 					$this->get_update_count()
@@ -259,15 +259,15 @@ 
 		// Bailout.
 		if ( ! $this->get_update_count()) {
 			// Show complete update message if still on update setting page.
-			if ( isset($_GET['page']) && 'give-updates' === $_GET['page'] ) {
+			if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 				// Upgrades
 				add_submenu_page(
 					'edit.php?post_type=give_forms',
-					esc_html__( 'Give Updates Complete', 'give' ),
-					__( 'Updates', 'give' ),
+					esc_html__('Give Updates Complete', 'give'),
+					__('Updates', 'give'),
 					'manage_give_settings',
 					'give-updates',
-					array( $this, 'render_complete_page' )
+					array($this, 'render_complete_page')
 				);
 			}
 
@@ -277,15 +277,15 @@ 
 		// Upgrades
 		add_submenu_page(
 			'edit.php?post_type=give_forms',
-			esc_html__( 'Give Updates', 'give' ),
+			esc_html__('Give Updates', 'give'),
 			sprintf(
 				'%1$s <span class="update-plugins count-%2$d"><span class="plugin-count">%2$d</span></span>',
-				__( 'Updates', 'give' ),
+				__('Updates', 'give'),
 				$this->get_update_count()
 			),
 			'manage_give_settings',
 			'give-updates',
-			array( $this, 'render_page' )
+			array($this, 'render_page')
 		);
 	}
 
@@ -297,7 +297,7 @@ 
 	 * @return int
 	 */
 	public function get_db_update_count() {
-		return count( $this->get_updates( 'database', 'new' ) );
+		return count($this->get_updates('database', 'new'));
 	}
 
 	/**
@@ -307,7 +307,7 @@ 
 	 * @access public
 	 */
 	public function render_complete_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades-complete.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades-complete.php';
 	}
 
 	/**
@@ -317,7 +317,7 @@ 
 	 * @access public
 	 */
 	public function render_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades.php';
 	}
 
 	/**
@@ -328,7 +328,7 @@ 
 	 * @return int
 	 */
 	public function get_plugin_update_count() {
-		return count( $this->get_updates( 'plugin' ) );
+		return count($this->get_updates('plugin'));
 	}
 
 	/**
@@ -343,7 +343,7 @@ 
 		$db_update_count     = $this->get_db_update_count();
 		$plugin_update_count = $this->get_plugin_update_count();
 
-		return ( $db_update_count + $plugin_update_count );
+		return ($db_update_count + $plugin_update_count);
 	}
 
 
@@ -354,12 +354,12 @@ 
 	 * @access public
 	 */
 	public function __flush_resume_updates() {
-		delete_option( 'give_doing_upgrade' );
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+		delete_option('give_doing_upgrade');
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 
 		// Reset counter.
 		$this->step = $this->percentage = 0;
-		++ $this->update;
+		++$this->update;
 	}
 
 	/**
@@ -370,24 +370,24 @@ 
 	 */
 	public function __give_ajax_updates() {
 		// Check permission.
-		if ( ! current_user_can( 'manage_give_settings' ) ) {
+		if ( ! current_user_can('manage_give_settings')) {
 			$this->send_ajax_response(
 				array(
-					'message' => esc_html__( 'You do not have permission to do Give upgrades.', 'give' ),
+					'message' => esc_html__('You do not have permission to do Give upgrades.', 'give'),
 				),
 				'error'
 			);
 		}
 
 		// Set params.
-		$this->step   = absint( $_POST['step'] );
-		$this->update = absint( $_POST['update'] );
+		$this->step   = absint($_POST['step']);
+		$this->update = absint($_POST['update']);
 
 		// Bailout: step and update must be positive and greater then zero.
-		if ( ! $this->step ) {
+		if ( ! $this->step) {
 			$this->send_ajax_response(
 				array(
-					'message'    => __( 'Please reload this page  and try again', 'give' ),
+					'message'    => __('Please reload this page  and try again', 'give'),
 					'heading'    => '',
 					'percentage' => 0,
 				),
@@ -396,14 +396,14 @@ 
 		}
 
 		// Get updates.
-		$updates = $this->get_updates( 'database', 'new' );
+		$updates = $this->get_updates('database', 'new');
 
 		// Bailout if we do not have nay updates.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			$this->send_ajax_response(
 				array(
-					'message'    => __( 'The database is already up to date.', 'give' ),
-					'heading'    => __( 'Updates Completed.', 'give' ),
+					'message'    => __('The database is already up to date.', 'give'),
+					'heading'    => __('Updates Completed.', 'give'),
 					'percentage' => 0,
 				),
 				'success'
@@ -411,26 +411,26 @@ 
 		}
 
 		// Process update.
-		foreach ( $updates as $index => $update ) {
+		foreach ($updates as $index => $update) {
 			// Check if update depend upon any other update.
-			if ( ! empty( $update['depend'] ) && ! give_has_upgrade_completed( $update['depend'] ) ) {
+			if ( ! empty($update['depend']) && ! give_has_upgrade_completed($update['depend'])) {
 				continue;
 			}
 
 			// Run update.
-			if ( is_array( $update['callback'] ) ) {
+			if (is_array($update['callback'])) {
 				$update['callback'][0]->$update['callback'][1]();
 			} else {
 				$update['callback']();
 			}
 
 			// Check if current update completed or not.
-			if ( give_has_upgrade_completed( $update['id'] ) ) {
-				if ( 1 === count( $updates ) ) {
+			if (give_has_upgrade_completed($update['id'])) {
+				if (1 === count($updates)) {
 					$this->send_ajax_response(
 						array(
-							'message'    => __( 'Database updated successfully.', 'give' ),
-							'heading'    => __( 'Updates Completed.', 'give' ),
+							'message'    => __('Database updated successfully.', 'give'),
+							'heading'    => __('Updates Completed.', 'give'),
 							'percentage' => 0,
 						),
 						'success'
@@ -440,16 +440,16 @@ 
 
 			$doing_upgrade_args = array(
 				'update_info' => $update,
-				'step'        => ++ $this->step,
+				'step'        => ++$this->step,
 				'update'      => $this->update,
-				'heading'     => sprintf( 'Update %s of {update_count}', $this->update ),
+				'heading'     => sprintf('Update %s of {update_count}', $this->update),
 				'percentage'  => $this->percentage,
 			);
 
 			// Cache upgrade.
-			update_option( 'give_doing_upgrade', $doing_upgrade_args );
+			update_option('give_doing_upgrade', $doing_upgrade_args);
 
-			$this->send_ajax_response( $doing_upgrade_args );
+			$this->send_ajax_response($doing_upgrade_args);
 		}// End foreach().
 	}
 
@@ -462,7 +462,7 @@ 
 	 * @param        $data
 	 * @param string $type
 	 */
-	public function send_ajax_response( $data, $type = '' ) {
+	public function send_ajax_response($data, $type = '') {
 		$default = array(
 			'message'    => '',
 			'heading'    => '',
@@ -472,21 +472,21 @@ 
 		);
 
 		// Set data.
-		$data = wp_parse_args( $data, $default );
+		$data = wp_parse_args($data, $default);
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'success':
-				wp_send_json_success( $data );
+				wp_send_json_success($data);
 				break;
 
 			case 'error':
-				wp_send_json_error( $data );
+				wp_send_json_error($data);
 				break;
 
 			default:
-				wp_send_json( array(
+				wp_send_json(array(
 					'data' => $data,
-				) );
+				));
 				break;
 		}
 	}
@@ -503,8 +503,8 @@ 
 	public function resume_updates() {
 		$status = false;
 
-		if ( $update = get_option( 'give_doing_upgrade' ) ) {
-			$status = ! empty( $update['step'] ) ? $update['step'] : $status;
+		if ($update = get_option('give_doing_upgrade')) {
+			$status = ! empty($update['step']) ? $update['step'] : $status;
 		}
 
 		return $status;
@@ -520,12 +520,12 @@ 
 	 * @param $total
 	 * @param $current_total
 	 */
-	public function set_percentage( $total, $current_total ) {
+	public function set_percentage($total, $current_total) {
 		// Set percentage.
-		$this->percentage = $total ? ( ( $current_total ) / $total ) * 100 : 0;
+		$this->percentage = $total ? (($current_total) / $total) * 100 : 0;
 
 		// Verify percentage.
-		$this->percentage = ( 100 < $this->percentage ) ? 100 : $this->percentage;
+		$this->percentage = (100 < $this->percentage) ? 100 : $this->percentage;
 	}
 }
 

includes/admin/upgrades/upgrade-functions.php

Spacing +199 added lines, -199 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,56 +25,56 @@ 
  */
 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;
 
-		case version_compare( $give_version, '1.8.9', '<' ) :
+		case version_compare($give_version, '1.8.9', '<') :
 			give_v189_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.12', '<' ) :
+		case version_compare($give_version, '1.8.12', '<') :
 			give_v1812_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.13', '<' ) :
+		case version_compare($give_version, '1.8.13', '<') :
 			give_v1813_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
@@ -86,7 +86,7 @@ 
  *
  * @return void
  */
-function give_show_upgrade_notices( $give_updates ) {
+function give_show_upgrade_notices($give_updates) {
 	// v1.3.2 Upgrades
 	$give_updates->register(
 		array(
@@ -152,7 +152,7 @@ 
 	);
 }
 
-add_action( 'give_register_updates', 'give_show_upgrade_notices' );
+add_action('give_register_updates', 'give_show_upgrade_notices');
 
 /**
  * Triggers all upgrade functions
@@ -164,29 +164,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');
 
 
 /**
@@ -202,24 +202,24 @@ 
 	/* @var Give_Updates $give_updates */
 	$give_updates = Give_Updates::get_instance();
 
-	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);
 
 	$give_updates->percentage = 100;
-	give_set_upgrade_complete( 'upgrade_give_payment_customer_id' );
+	give_set_upgrade_complete('upgrade_give_payment_customer_id');
 }
 
 
@@ -243,24 +243,24 @@ 
 	$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');
 
 		}
 	}
 
 	$give_updates->percentage = 100;
-	give_set_upgrade_complete( 'upgrade_give_offline_status' );
+	give_set_upgrade_complete('upgrade_give_offline_status');
 }
 
 
@@ -273,17 +273,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.
@@ -330,9 +330,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);
 			}
 		}
 
@@ -342,15 +342,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.
@@ -393,53 +393,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(),
 		);
 
@@ -454,17 +454,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().
 }
 
@@ -494,9 +494,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);
 		}
 	}
 
@@ -530,7 +530,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',
@@ -542,48 +542,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');
 }
 
 /**
@@ -597,7 +597,7 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$forms = new WP_Query( array(
+	$forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -606,41 +606,41 @@ 
 		)
 	);
 
-	if ( $forms->have_posts() ) {
-		$give_updates->set_percentage( $forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($forms->have_posts()) {
+		$give_updates->set_percentage($forms->found_posts, ($give_updates->step * 20));
 
-		while ( $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(
@@ -660,15 +660,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().
@@ -677,7 +677,7 @@ 
 
 	} else {
 		// No more forms found, finish up.
-		give_set_upgrade_complete( 'v18_upgrades_form_metadata' );
+		give_set_upgrade_complete('v18_upgrades_form_metadata');
 	}
 }
 
@@ -726,7 +726,7 @@ 
 				'_transient_give_stats_',
 				'give_cache',
 				'_transient_give_add_ons_feed',
-				'_transient__give_ajax_works' .
+				'_transient__give_ajax_works'.
 				'_transient_give_total_api_keys',
 				'_transient_give_i18n_give_promo_hide',
 				'_transient_give_contributors',
@@ -753,24 +753,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);
 			}
 		}
 	}
@@ -788,7 +788,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(
@@ -796,9 +796,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);
 	}
 
 }
@@ -815,7 +815,7 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$donation_forms = new WP_Query( array(
+	$donation_forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -824,10 +824,10 @@ 
 		)
 	);
 
-	if ( $donation_forms->have_posts() ) {
-		$give_updates->set_percentage( $donation_forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($donation_forms->have_posts()) {
+		$give_updates->set_percentage($donation_forms->found_posts, ($give_updates->step * 20));
 
-		while ( $donation_forms->have_posts() ) {
+		while ($donation_forms->have_posts()) {
 			$donation_forms->the_post();
 			$form_id = get_the_ID();
 
@@ -835,41 +835,41 @@ 
 			update_post_meta(
 				$form_id,
 				'_give_set_price',
-				give_sanitize_amount( get_post_meta( $form_id, '_give_set_price', true ) )
+				give_sanitize_amount(get_post_meta($form_id, '_give_set_price', true))
 			);
 
 			// Remove formatting from _give_custom_amount_minimum
 			update_post_meta(
 				$form_id,
 				'_give_custom_amount_minimum',
-				give_sanitize_amount( get_post_meta( $form_id, '_give_custom_amount_minimum', true ) )
+				give_sanitize_amount(get_post_meta($form_id, '_give_custom_amount_minimum', true))
 			);
 
 			// Bailout.
-			if ( 'set' === get_post_meta( $form_id, '_give_price_option', true ) ) {
+			if ('set' === get_post_meta($form_id, '_give_price_option', true)) {
 				continue;
 			}
 
-			$donation_levels = get_post_meta( $form_id, '_give_donation_levels', true );
+			$donation_levels = get_post_meta($form_id, '_give_donation_levels', true);
 
-			if ( ! empty( $donation_levels ) ) {
+			if ( ! empty($donation_levels)) {
 
-				foreach ( $donation_levels as $index => $donation_level ) {
-					if ( isset( $donation_level['_give_amount'] ) ) {
-						$donation_levels[ $index ]['_give_amount'] = give_sanitize_amount( $donation_level['_give_amount'] );
+				foreach ($donation_levels as $index => $donation_level) {
+					if (isset($donation_level['_give_amount'])) {
+						$donation_levels[$index]['_give_amount'] = give_sanitize_amount($donation_level['_give_amount']);
 					}
 				}
 
-				update_post_meta( $form_id, '_give_donation_levels', $donation_levels );
+				update_post_meta($form_id, '_give_donation_levels', $donation_levels);
 
-				$donation_levels_amounts = wp_list_pluck( $donation_levels, '_give_amount' );
+				$donation_levels_amounts = wp_list_pluck($donation_levels, '_give_amount');
 
-				$min_amount = min( $donation_levels_amounts );
-				$max_amount = max( $donation_levels_amounts );
+				$min_amount = min($donation_levels_amounts);
+				$max_amount = max($donation_levels_amounts);
 
 				// Set Minimum and Maximum amount for Multi Level Donation Forms
-				give_update_meta( $form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount( $min_amount ) : 0 );
-				give_update_meta( $form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount( $max_amount ) : 0 );
+				give_update_meta($form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount($min_amount) : 0);
+				give_update_meta($form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount($max_amount) : 0);
 			}
 
 		}
@@ -878,7 +878,7 @@ 
 		wp_reset_postdata();
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v189_upgrades_levels_post_meta' );
+		give_set_upgrade_complete('v189_upgrades_levels_post_meta');
 	}
 
 }
@@ -934,22 +934,22 @@ 
 	$give_settings        = give_get_settings();
 	$give_setting_updated = false;
 
-	if ( $give_settings['thousands_separator'] === $give_settings['decimal_separator'] ) {
+	if ($give_settings['thousands_separator'] === $give_settings['decimal_separator']) {
 		$give_settings['number_decimals']   = 0;
 		$give_settings['decimal_separator'] = '';
 		$give_setting_updated               = true;
 
-	} elseif ( empty( $give_settings['decimal_separator'] ) ) {
+	} elseif (empty($give_settings['decimal_separator'])) {
 		$give_settings['number_decimals'] = 0;
 		$give_setting_updated             = true;
 
-	} elseif ( 6 < absint( $give_settings['number_decimals'] ) ) {
+	} elseif (6 < absint($give_settings['number_decimals'])) {
 		$give_settings['number_decimals'] = 5;
 		$give_setting_updated             = true;
 	}
 
-	if ( $give_setting_updated ) {
-		update_option( 'give_settings', $give_settings );
+	if ($give_setting_updated) {
+		update_option('give_settings', $give_settings);
 	}
 }
 
@@ -968,70 +968,70 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$donation_forms = new WP_Query( array(
+	$donation_forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
-			'post_type'      => array( 'give_forms', 'give_payment' ),
+			'post_type'      => array('give_forms', 'give_payment'),
 			'posts_per_page' => 20,
 		)
 	);
 
-	if ( $donation_forms->have_posts() ) {
-		$give_updates->set_percentage( $donation_forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($donation_forms->have_posts()) {
+		$give_updates->set_percentage($donation_forms->found_posts, ($give_updates->step * 20));
 
-		while ( $donation_forms->have_posts() ) {
+		while ($donation_forms->have_posts()) {
 			$donation_forms->the_post();
 			global $post;
 
-			$meta = get_post_meta( $post->ID );
+			$meta = get_post_meta($post->ID);
 
-			switch ( $post->post_type ) {
+			switch ($post->post_type) {
 				case 'give_forms':
 					// _give_set_price
-					if ( ! empty( $meta['_give_set_price'][0] ) ) {
-						update_post_meta( $post->ID, '_give_set_price', give_sanitize_amount_for_db( $meta['_give_set_price'][0] ) );
+					if ( ! empty($meta['_give_set_price'][0])) {
+						update_post_meta($post->ID, '_give_set_price', give_sanitize_amount_for_db($meta['_give_set_price'][0]));
 					}
 
 					// _give_custom_amount_minimum
-					if ( ! empty( $meta['_give_custom_amount_minimum'][0] ) ) {
-						update_post_meta( $post->ID, '_give_custom_amount_minimum', give_sanitize_amount_for_db( $meta['_give_custom_amount_minimum'][0] ) );
+					if ( ! empty($meta['_give_custom_amount_minimum'][0])) {
+						update_post_meta($post->ID, '_give_custom_amount_minimum', give_sanitize_amount_for_db($meta['_give_custom_amount_minimum'][0]));
 					}
 
 					// _give_levels_minimum_amount
-					if ( ! empty( $meta['_give_levels_minimum_amount'][0] ) ) {
-						update_post_meta( $post->ID, '_give_levels_minimum_amount', give_sanitize_amount_for_db( $meta['_give_levels_minimum_amount'][0] ) );
+					if ( ! empty($meta['_give_levels_minimum_amount'][0])) {
+						update_post_meta($post->ID, '_give_levels_minimum_amount', give_sanitize_amount_for_db($meta['_give_levels_minimum_amount'][0]));
 					}
 
 					// _give_levels_maximum_amount
-					if ( ! empty( $meta['_give_levels_maximum_amount'][0] ) ) {
-						update_post_meta( $post->ID, '_give_levels_maximum_amount', give_sanitize_amount_for_db( $meta['_give_levels_maximum_amount'][0] ) );
+					if ( ! empty($meta['_give_levels_maximum_amount'][0])) {
+						update_post_meta($post->ID, '_give_levels_maximum_amount', give_sanitize_amount_for_db($meta['_give_levels_maximum_amount'][0]));
 					}
 
 					// _give_set_goal
-					if ( ! empty( $meta['_give_set_goal'][0] ) ) {
-						update_post_meta( $post->ID, '_give_set_goal', give_sanitize_amount_for_db( $meta['_give_set_goal'][0] ) );
+					if ( ! empty($meta['_give_set_goal'][0])) {
+						update_post_meta($post->ID, '_give_set_goal', give_sanitize_amount_for_db($meta['_give_set_goal'][0]));
 					}
 
 					// _give_form_earnings
-					if ( ! empty( $meta['_give_form_earnings'][0] ) ) {
-						update_post_meta( $post->ID, '_give_form_earnings', give_sanitize_amount_for_db( $meta['_give_form_earnings'][0] ) );
+					if ( ! empty($meta['_give_form_earnings'][0])) {
+						update_post_meta($post->ID, '_give_form_earnings', give_sanitize_amount_for_db($meta['_give_form_earnings'][0]));
 					}
 
 					// _give_custom_amount_minimum
-					if ( ! empty( $meta['_give_donation_levels'][0] ) ) {
-						$donation_levels = unserialize( $meta['_give_donation_levels'][0] );
+					if ( ! empty($meta['_give_donation_levels'][0])) {
+						$donation_levels = unserialize($meta['_give_donation_levels'][0]);
 
-						foreach ( $donation_levels as $index => $level ) {
-							if ( empty( $level['_give_amount'] ) ) {
+						foreach ($donation_levels as $index => $level) {
+							if (empty($level['_give_amount'])) {
 								continue;
 							}
 
-							$donation_levels[ $index ]['_give_amount'] = give_sanitize_amount_for_db( $level['_give_amount'] );
+							$donation_levels[$index]['_give_amount'] = give_sanitize_amount_for_db($level['_give_amount']);
 						}
 
 						$meta['_give_donation_levels'] = $donation_levels;
-						update_post_meta( $post->ID, '_give_donation_levels', $meta['_give_donation_levels'] );
+						update_post_meta($post->ID, '_give_donation_levels', $meta['_give_donation_levels']);
 					}
 
 
@@ -1039,8 +1039,8 @@ 
 
 				case 'give_payment':
 					// _give_payment_total
-					if ( ! empty( $meta['_give_payment_total'][0] ) ) {
-						update_post_meta( $post->ID, '_give_payment_total', give_sanitize_amount_for_db( $meta['_give_payment_total'][0] ) );
+					if ( ! empty($meta['_give_payment_total'][0])) {
+						update_post_meta($post->ID, '_give_payment_total', give_sanitize_amount_for_db($meta['_give_payment_total'][0]));
 					}
 
 					break;
@@ -1051,7 +1051,7 @@ 
 		wp_reset_postdata();
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1812_update_amount_values' );
+		give_set_upgrade_complete('v1812_update_amount_values');
 	}
 }
 
@@ -1071,22 +1071,22 @@ 
 	$offset       = 1 === $give_updates->step ? 0 : $give_updates->step * 20;
 
 	// form query
-	$donors = Give()->donors->get_donors( array(
+	$donors = Give()->donors->get_donors(array(
 			'number' => 20,
 			'offset' => $offset,
 		)
 	);
 
-	if ( ! empty( $donors ) ) {
-		$give_updates->set_percentage( Give()->donors->count(), ( $give_updates->step * 20 ) );
+	if ( ! empty($donors)) {
+		$give_updates->set_percentage(Give()->donors->count(), ($give_updates->step * 20));
 
 		/* @var Object $donor */
-		foreach ( $donors as $donor ) {
-			Give()->donors->update( $donor->id, array( 'purchase_value' => give_sanitize_amount_for_db( $donor->purchase_value ) ) );
+		foreach ($donors as $donor) {
+			Give()->donors->update($donor->id, array('purchase_value' => give_sanitize_amount_for_db($donor->purchase_value)));
 		}
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1812_update_donor_purchase_values' );
+		give_set_upgrade_complete('v1812_update_donor_purchase_values');
 	}
 }
 
@@ -1101,25 +1101,25 @@ 
 	$offset       = 1 === $give_updates->step ? 0 : $give_updates->step * 20;
 
 	// Fetch all the existing donors.
-	$donors = Give()->donors->get_donors( array(
+	$donors = Give()->donors->get_donors(array(
 			'number' => 20,
 			'offset' => $offset,
 		)
 	);
 
-	if ( ! empty( $donors ) ) {
-		$give_updates->set_percentage( Give()->donors->count(), ( $give_updates->step * 20 ) );
+	if ( ! empty($donors)) {
+		$give_updates->set_percentage(Give()->donors->count(), ($give_updates->step * 20));
 
 		/* @var Object $donor */
-		foreach ( $donors as $donor ) {
+		foreach ($donors as $donor) {
 			$user_id = $donor->user_id;
 
 			// Proceed, if donor is attached with user.
-			if ( $user_id ) {
-				$user = get_userdata( $user_id );
+			if ($user_id) {
+				$user = get_userdata($user_id);
 
 				// Update user role, if user has subscriber role.
-				if ( is_array( $user->roles ) && in_array( 'subscriber', $user->roles ) ) {
+				if (is_array($user->roles) && in_array('subscriber', $user->roles)) {
 					wp_update_user(
 						array(
 							'ID'   => $user_id,
@@ -1131,7 +1131,7 @@ 
 		}
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1813_update_donor_user_roles' );
+		give_set_upgrade_complete('v1813_update_donor_user_roles');
 	}
 }
 
@@ -1143,7 +1143,7 @@ 
  */
 function give_v1813_upgrades() {
 	// Update admin setting.
-	give_update_option( 'donor_default_user_role', 'give_donor' );
+	give_update_option('donor_default_user_role', 'give_donor');
 
 	// Update Give roles.
 	$roles = new Give_Roles();

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

Spacing +303 added lines, -303 removed lines

@@ -46,24 +46,24 @@ 
 	 */
 	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);
 
 		// Maintain active tab query parameter after save.
-		add_filter( 'redirect_post_location', array( $this, 'maintain_active_tab' ), 10, 2 );
+		add_filter('redirect_post_location', array($this, 'maintain_active_tab'), 10, 2);
 	}
 
 	/**
@@ -85,24 +85,24 @@ 
 	 */
 	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_format_amount( give_get_form_goal( $post_id ), array( 'sanitize' => false ) );
-		$price_placeholder     = give_format_decimal( '1.00', false, false );
+		$price                 = give_get_form_price($post_id);
+		$custom_amount_minimum = give_get_form_minimum_price($post_id);
+		$goal                  = give_format_amount(give_get_form_goal($post_id), array('sanitize' => false));
+		$price_placeholder     = give_format_decimal('1.00', false, false);
 
 		// No empty prices - min. 1.00 for new forms
-		if ( empty( $price ) && is_null( $post_id ) ) {
+		if (empty($price) && is_null($post_id)) {
 			$price = '1.00';
 		}
 
 		// Min. $1.00 for new forms
-		if ( empty( $custom_amount_minimum ) ) {
+		if (empty($custom_amount_minimum)) {
 			$custom_amount_minimum = '1.00';
 		}
 
 		// Format amounts.
-		$price                 = give_format_amount( $price, array( 'sanitize' => false ) );
-		$custom_amount_minimum = give_format_amount( $custom_amount_minimum, array( 'sanitize' => false ) );
+		$price                 = give_format_amount($price, array('sanitize' => false));
+		$custom_amount_minimum = give_format_amount($custom_amount_minimum, array('sanitize' => false));
 
 		// Start with an underscore to hide fields from custom fields list
 		$prefix = '_give_';
@@ -111,27 +111,27 @@ 
 			/**
 			 * 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(
@@ -142,34 +142,34 @@ 
 					),
 					// 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'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					// 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(
@@ -180,37 +180,37 @@ 
 						'wrapper_class' => 'give-hidden',
 					),
 					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'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					// 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>',
 						),
 						'wrapper_class' => 'give-hidden',
 						// 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(
@@ -219,112 +219,112 @@ 
 								),
 							),
 							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'),
 							),
 							'wrapper_class' => 'give-hidden',
 						),
 						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',
 						),
@@ -332,7 +332,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
@@ -343,31 +343,31 @@ 
 			/**
 			 * 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', false, false ),
+							'placeholder' => give_format_decimal('0.00', false, false),
 							'value'       => $goal,
 							'class'       => 'give-money-field',
 						),
@@ -375,97 +375,97 @@ 
 					),
 
 					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'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					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',
 						'wrapper_class' => 'give-hidden',
 					),
 
 					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'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					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'),
 						'wrapper_class' => 'give-hidden',
 					),
 					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',
 						'wrapper_class' => 'give-hidden',
 					),
 					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',
 						'wrapper_class' => 'give-hidden',
 					),
@@ -473,51 +473,51 @@ 
 						'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'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					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' ),
-						'default'       => give_get_option( 'agreement_text' ),
+						'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',
 						'wrapper_class' => 'give-hidden',
 					),
@@ -525,18 +525,18 @@ 
 						'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;
@@ -552,8 +552,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'
 		);
@@ -569,7 +569,7 @@ 
 	function enqueue_script() {
 		global $post;
 
-		if ( is_object( $post ) && 'give_forms' === $post->post_type ) {
+		if (is_object($post) && 'give_forms' === $post->post_type) {
 
 		}
 	}
@@ -604,32 +604,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;
 				}
 			}
@@ -646,33 +646,33 @@ 
 	 */
 	public function output() {
 		// Bailout.
-		if ( $form_data_tabs = $this->get_tabs() ) :
-			$active_tab = ! empty( $_GET['give_tab'] ) ? give_clean( $_GET['give_tab'] ) : 'form_field_options';
-			wp_nonce_field( 'give_save_form_meta', 'give_form_meta_nonce' );
+		if ($form_data_tabs = $this->get_tabs()) :
+			$active_tab = ! empty($_GET['give_tab']) ? give_clean($_GET['give_tab']) : 'form_field_options';
+			wp_nonce_field('give_save_form_meta', 'give_form_meta_nonce');
 			?>
 			<input id="give_form_active_tab" type="hidden" name="give_form_active_tab">
 			<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 ) : ?>
+					<?php foreach ($form_data_tabs as $index => $form_data_tab) : ?>
 						<?php
 						// Determine if current tab is active.
 						$is_active = $active_tab === $form_data_tab['id'] ? true : false;
 						?>
-						<li class="<?php echo "{$form_data_tab['id']}_tab" . ( $is_active ? ' active' : '' ) . ( $this->has_sub_tab( $form_data_tab ) ? ' has-sub-fields' : '' ); ?>">
+						<li class="<?php echo "{$form_data_tab['id']}_tab".($is_active ? ' active' : '').($this->has_sub_tab($form_data_tab) ? ' has-sub-fields' : ''); ?>">
 							<a href="#<?php echo $form_data_tab['id']; ?>" data-tab-id="<?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']; ?>" data-tab-id="<?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>
@@ -687,30 +687,30 @@ 
 					<?php endforeach; ?>
 				</ul>
 
-				<?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"); ?>
 						<?php
 						// Determine if current panel is active.
 						$is_active = $active_tab === $setting['id'] ? true : false;
 						?>
-						<div id="<?php echo $setting['id']; ?>" class="panel give_options_panel<?php echo( $is_active ? ' active' : '' ); ?>">
-							<?php if ( ! empty( $setting['fields'] ) ) : ?>
-								<?php foreach ( $setting['fields'] as $field ) : ?>
-									<?php give_render_field( $field ); ?>
+						<div id="<?php echo $setting['id']; ?>" class="panel give_options_panel<?php echo($is_active ? ' active' : ''); ?>">
+							<?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>
@@ -734,9 +734,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;
 		}
 
@@ -750,13 +750,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]);
 			}
 		}
 
@@ -774,96 +774,96 @@ 
 	 *
 	 * @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 ] ) ) {
-					$setting_field = $this->get_setting_field( $form_meta_key );
-					if ( ! empty( $setting_field['type'] ) ) {
-						switch ( $setting_field['type'] ) {
+				if (isset($_POST[$form_meta_key])) {
+					$setting_field = $this->get_setting_field($form_meta_key);
+					if ( ! empty($setting_field['type'])) {
+						switch ($setting_field['type']) {
 							case 'textarea':
 							case 'wysiwyg':
-								$form_meta_value = wp_kses_post( $_POST[ $form_meta_key ] );
+								$form_meta_value = wp_kses_post($_POST[$form_meta_key]);
 								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);
 								break;
 
 							default:
-								$form_meta_value = give_clean( $_POST[ $form_meta_key ] );
+								$form_meta_value = give_clean($_POST[$form_meta_key]);
 						}// End switch().
 
 						/**
@@ -873,24 +873,24 @@ 
 						 */
 						$form_meta_value = apply_filters(
 							'give_pre_save_form_meta_value',
-							$this->sanitize_form_meta( $form_meta_value, $setting_field ),
+							$this->sanitize_form_meta($form_meta_value, $setting_field),
 							$form_meta_key,
 							$this,
 							$post_id
 						);
 
 						// 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);
 					}// End if().
 				}// End if().
 			}// End foreach().
 		}// End if().
 
 		// 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);
 	}
 
 
@@ -903,10 +903,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'];
 
 		}
@@ -923,12 +923,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;
 				}
 			}
@@ -946,14 +946,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;
 						}
 					}
@@ -974,14 +974,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;
@@ -998,10 +998,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']
 			: '';
 
@@ -1019,12 +1019,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;
 				}
@@ -1044,12 +1044,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;
 				}
@@ -1069,17 +1069,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;
 				}
@@ -1087,9 +1087,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;
 				}
 			}
@@ -1108,14 +1108,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;
@@ -1133,22 +1133,22 @@ 
 	 *
 	 * @return mixed
 	 */
-	function sanitize_form_meta( $meta_value, $setting_field ) {
-		switch ( $setting_field['type'] ) {
+	function sanitize_form_meta($meta_value, $setting_field) {
+		switch ($setting_field['type']) {
 			case 'group':
-				if ( ! empty( $setting_field['fields'] ) ) {
-					foreach ( $setting_field['fields'] as $field ) {
-						if ( empty( $field['data_type'] ) || 'price' !== $field['data_type'] ) {
+				if ( ! empty($setting_field['fields'])) {
+					foreach ($setting_field['fields'] as $field) {
+						if (empty($field['data_type']) || 'price' !== $field['data_type']) {
 							continue;
 						}
 
-						foreach ( $meta_value as $index => $meta_data ) {
-							if ( ! isset( $meta_value[ $index ][ $field['id'] ] ) ) {
+						foreach ($meta_value as $index => $meta_data) {
+							if ( ! isset($meta_value[$index][$field['id']])) {
 								continue;
 							}
 
-							$meta_value[ $index ][ $field['id'] ] = ! empty( $meta_value[ $index ][ $field['id'] ] )
-								? give_sanitize_amount_for_db( $meta_value[ $index ][ $field['id'] ] )
+							$meta_value[$index][$field['id']] = ! empty($meta_value[$index][$field['id']])
+								? give_sanitize_amount_for_db($meta_value[$index][$field['id']])
 								: 0;
 						}
 					}
@@ -1156,8 +1156,8 @@ 
 				break;
 
 			default:
-				if ( ! empty( $setting_field['data_type'] ) && 'price' === $setting_field['data_type'] ) {
-					$meta_value = $meta_value ? give_sanitize_amount_for_db( $meta_value ) : 0;
+				if ( ! empty($setting_field['data_type']) && 'price' === $setting_field['data_type']) {
+					$meta_value = $meta_value ? give_sanitize_amount_for_db($meta_value) : 0;
 				}
 		}
 
@@ -1175,12 +1175,12 @@ 
 	 *
 	 * @return string The URL after redirect.
 	 */
-	public function maintain_active_tab( $location, $post_id ) {
+	public function maintain_active_tab($location, $post_id) {
 		if (
-			'give_forms' === get_post_type( $post_id ) &&
-			! empty( $_POST['give_form_active_tab'] )
+			'give_forms' === get_post_type($post_id) &&
+			! empty($_POST['give_form_active_tab'])
 		) {
-			$location = add_query_arg( 'give_tab', give_clean( $_POST['give_form_active_tab'] ), $location );
+			$location = add_query_arg('give_tab', give_clean($_POST['give_form_active_tab']), $location);
 		}
 
 		return $location;

Braces +15 added lines, -6 removed lines

@@ -662,9 +662,12 @@ 
 							<a href="#<?php echo $form_data_tab['id']; ?>" data-tab-id="<?php echo $form_data_tab['id']; ?>">
 								<?php if ( ! empty( $form_data_tab['icon-html'] ) ) : ?>
 									<?php echo $form_data_tab['icon-html']; ?>
-								<?php else : ?>
+								<?php else {
+	: ?>
 									<span class="give-icon give-icon-default"></span>
-								<?php endif; ?>
+								<?php endif;
+}
+?>
 								<span class="give-label"><?php echo $form_data_tab['label']; ?></span>
 							</a>
 							<?php if ( $this->has_sub_tab( $form_data_tab ) ) : ?>
@@ -674,9 +677,12 @@ 
 											<a href="#<?php echo $sub_tab['id']; ?>" data-tab-id="<?php echo $sub_tab['id']; ?>">
 												<?php if ( ! empty( $sub_tab['icon-html'] ) ) : ?>
 													<?php echo $sub_tab['icon-html']; ?>
-												<?php else : ?>
+												<?php else {
+	: ?>
 													<span class="give-icon give-icon-default"></span>
-												<?php endif; ?>
+												<?php endif;
+}
+?>
 												<span class="give-label"><?php echo $sub_tab['label']; ?></span>
 											</a>
 										</li>
@@ -703,11 +709,14 @@ 
 						</div>
 
 						<?php do_action( "give_after_{$setting['id']}_settings" ); ?>
-					<?php else : ?>
+					<?php else {
+	: ?>
 						<?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">
+									<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 ); ?>