WordImpress / Give

includes/emails/functions.php

Spacing +70 added lines, -70 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,14 +26,14 @@ 
  *
  * @return void
  */
-function give_email_donation_receipt( $payment_id, $admin_notice = true ) {
+function give_email_donation_receipt($payment_id, $admin_notice = true) {
 	/**
 	 * Fire the action
 	 */
-	do_action( 'give_donation-receipt_email_notification', $payment_id );
+	do_action('give_donation-receipt_email_notification', $payment_id);
 
 	// If admin notifications are on, send the admin notice.
-	if ( $admin_notice && give_is_setting_enabled( Give_Email_Notification::get_instance('new-donation' )->get_notification_status() ) ) {
+	if ($admin_notice && give_is_setting_enabled(Give_Email_Notification::get_instance('new-donation')->get_notification_status())) {
 		/**
 		 * Fires in the donation email receipt.
 		 *
@@ -44,7 +44,7 @@ 
 		 * @param int   $payment_id   Payment id.
 		 * @param mixed $payment_data Payment meta data.
 		 */
-		do_action( 'give_new-donation_email_notification', $payment_id, give_get_payment_meta( $payment_id ) );
+		do_action('give_new-donation_email_notification', $payment_id, give_get_payment_meta($payment_id));
 	}
 }
 
@@ -57,7 +57,7 @@ 
  *
  * @return void
  */
-function give_admin_email_notice( $payment_id ) {
+function give_admin_email_notice($payment_id) {
 	/**
 	 * Fires in the donation email receipt.
 	 *
@@ -68,10 +68,10 @@ 
 	 * @param int   $payment_id   Payment id.
 	 * @param mixed $payment_data Payment meta data.
 	 */
-	do_action( 'give_new-donation_email_notification', $payment_id );
+	do_action('give_new-donation_email_notification', $payment_id);
 }
 
-add_action( 'give_admin_donation_email', 'give_admin_email_notice' );
+add_action('give_admin_donation_email', 'give_admin_email_notice');
 
 
 /**
@@ -84,16 +84,16 @@ 
  */
 function give_get_default_donation_notification_email() {
 
-	$default_email_body = __( 'Hi there,', 'give' ) . "\n\n";
-	$default_email_body .= __( 'This email is to inform you that a new donation has been made on your website:', 'give' ) . ' <a href="' . get_bloginfo( 'url' ) . '" target="_blank">' . get_bloginfo( 'url' ) . '</a>' . ".\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {name}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation:', 'give' ) . '</strong> {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method:', 'give' ) . '</strong> {payment_method}' . "\n\n";
-	$default_email_body .= __( 'Thank you,', 'give' ) . "\n\n";
-	$default_email_body .= '{sitename}' . "\n";
+	$default_email_body = __('Hi there,', 'give')."\n\n";
+	$default_email_body .= __('This email is to inform you that a new donation has been made on your website:', 'give').' <a href="'.get_bloginfo('url').'" target="_blank">'.get_bloginfo('url').'</a>'.".\n\n";
+	$default_email_body .= '<strong>'.__('Donor:', 'give').'</strong> {name}'."\n";
+	$default_email_body .= '<strong>'.__('Donation:', 'give').'</strong> {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method:', 'give').'</strong> {payment_method}'."\n\n";
+	$default_email_body .= __('Thank you,', 'give')."\n\n";
+	$default_email_body .= '{sitename}'."\n";
 
-	return apply_filters( 'give_default_donation_notification_email', $default_email_body );
+	return apply_filters('give_default_donation_notification_email', $default_email_body);
 }
 
 
@@ -107,21 +107,21 @@ 
  */
 function give_get_default_donation_receipt_email() {
 
-	$default_email_body = __( 'Dear', 'give' ) . " {name},\n\n";
-	$default_email_body .= __( 'Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give' ) . "\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation:', 'give' ) . '</strong> {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation Date:', 'give' ) . '</strong> {date}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method:', 'give' ) . '</strong> {payment_method}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment ID:', 'give' ) . '</strong> {payment_id}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Receipt ID:', 'give' ) . '</strong> {receipt_id}' . "\n\n";
-	$default_email_body .= '{receipt_link}' . "\n\n";
+	$default_email_body = __('Dear', 'give')." {name},\n\n";
+	$default_email_body .= __('Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give')."\n\n";
+	$default_email_body .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
+	$default_email_body .= '<strong>'.__('Donation:', 'give').'</strong> {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Donation Date:', 'give').'</strong> {date}'."\n";
+	$default_email_body .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method:', 'give').'</strong> {payment_method}'."\n";
+	$default_email_body .= '<strong>'.__('Payment ID:', 'give').'</strong> {payment_id}'."\n";
+	$default_email_body .= '<strong>'.__('Receipt ID:', 'give').'</strong> {receipt_id}'."\n\n";
+	$default_email_body .= '{receipt_link}'."\n\n";
 	$default_email_body .= "\n\n";
-	$default_email_body .= __( 'Sincerely,', 'give' ) . "\n";
-	$default_email_body .= '{sitename}' . "\n";
+	$default_email_body .= __('Sincerely,', 'give')."\n";
+	$default_email_body .= '{sitename}'."\n";
 
-	return apply_filters( 'give_default_donation_receipt_email', $default_email_body );
+	return apply_filters('give_default_donation_receipt_email', $default_email_body);
 }
 
 /**
@@ -134,22 +134,22 @@ 
  *
  * @return array $email_names
  */
-function give_get_email_names( $user_info, $payment = false ) {
+function give_get_email_names($user_info, $payment = false) {
 	$email_names = array();
 
-	if ( is_a( $payment, 'Give_Payment' ) ) {
+	if (is_a($payment, 'Give_Payment')) {
 
-		if ( $payment->user_id > 0 ) {
+		if ($payment->user_id > 0) {
 
-			$user_data               = get_userdata( $payment->user_id );
+			$user_data               = get_userdata($payment->user_id);
 			$email_names['name']     = $payment->first_name;
-			$email_names['fullname'] = trim( $payment->first_name . ' ' . $payment->last_name );
+			$email_names['fullname'] = trim($payment->first_name.' '.$payment->last_name);
 			$email_names['username'] = $user_data->user_login;
 
-		} elseif ( ! empty( $payment->first_name ) ) {
+		} elseif ( ! empty($payment->first_name)) {
 
 			$email_names['name']     = $payment->first_name;
-			$email_names['fullname'] = trim( $payment->first_name . ' ' . $payment->last_name );
+			$email_names['fullname'] = trim($payment->first_name.' '.$payment->last_name);
 			$email_names['username'] = $payment->first_name;
 
 		} else {
@@ -162,30 +162,30 @@ 
 	} else {
 
 		// Support for old serialized data.
-		if ( is_serialized( $user_info ) ) {
+		if (is_serialized($user_info)) {
 
 			// Security check.
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches );
-			if ( ! empty( $matches ) ) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches);
+			if ( ! empty($matches)) {
 				return array(
 					'name'     => '',
 					'fullname' => '',
 					'username' => '',
 				);
 			} else {
-				$user_info = maybe_unserialize( $user_info );
+				$user_info = maybe_unserialize($user_info);
 			}
 
 		}
 
-		if ( isset( $user_info['id'] ) && $user_info['id'] > 0 && isset( $user_info['first_name'] ) ) {
-			$user_data               = get_userdata( $user_info['id'] );
+		if (isset($user_info['id']) && $user_info['id'] > 0 && isset($user_info['first_name'])) {
+			$user_data               = get_userdata($user_info['id']);
 			$email_names['name']     = $user_info['first_name'];
-			$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 			$email_names['username'] = $user_data->user_login;
-		} elseif ( isset( $user_info['first_name'] ) ) {
+		} elseif (isset($user_info['first_name'])) {
 			$email_names['name']     = $user_info['first_name'];
-			$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 			$email_names['username'] = $user_info['first_name'];
 		} else {
 			$email_names['name']     = $user_info['email'];
@@ -205,37 +205,37 @@ 
  *
  * @since 1.8.14
  */
-function give_admin_email_user_donor_disconnection( $user_id, $donor_id ) {
+function give_admin_email_user_donor_disconnection($user_id, $donor_id) {
 
-	$user_id  = absint( $user_id );
-	$donor_id = absint( $donor_id );
+	$user_id  = absint($user_id);
+	$donor_id = absint($donor_id);
 
 	// Bail Out, if user id doesn't exists.
-	if ( empty( $user_id ) ) {
+	if (empty($user_id)) {
 		return;
 	}
 
 	// Bail Out, if donor id doesn't exists.
-	if ( empty( $donor_id ) ) {
+	if (empty($donor_id)) {
 		return;
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/* translators: %s: payment id */
-	$subject = __( 'Attention: User tries to login whose Donor profile is disconnected!', 'give' );
+	$subject = __('Attention: User tries to login whose Donor profile is disconnected!', 'give');
 
 	/**
 	 * Filters the Donor-User Disconnection notification subject.
 	 *
 	 * @since 1.8.14
 	 */
-	$subject = apply_filters( 'give_admin_donor_user_disconnection_notification_subject', wp_strip_all_tags( $subject ) );
+	$subject = apply_filters('give_admin_donor_user_disconnection_notification_subject', wp_strip_all_tags($subject));
 
-	$headers = "From: " . stripslashes_deep( html_entity_decode( $from_name, ENT_COMPAT, 'UTF-8' ) ) . " <$from_email>\r\n";
-	$headers .= "Reply-To: " . $from_email . "\r\n";
+	$headers = "From: ".stripslashes_deep(html_entity_decode($from_name, ENT_COMPAT, 'UTF-8'))." <$from_email>\r\n";
+	$headers .= "Reply-To: ".$from_email."\r\n";
 	$headers .= "Content-Type: text/html; charset=utf-8\r\n";
 
 	/**
@@ -243,25 +243,25 @@ 
 	 *
 	 * @since 1.8.14
 	 */
-	$headers = apply_filters( 'give_admin_donor_user_disconnection_notification_headers', $headers );
+	$headers = apply_filters('give_admin_donor_user_disconnection_notification_headers', $headers);
 
-	$message = __( 'Hi Admin,', 'give' ) . "\n\n";
-	$message .= __( 'This email is to inform you that a user has tried logging in. But, User was unable to login due to User-Donor profile disconnection.', 'give' ) . "\n\n";
-	$message .= __( 'Do you want to reconnect User and Donor profile again?', 'give' ) . "\n\n";
+	$message = __('Hi Admin,', 'give')."\n\n";
+	$message .= __('This email is to inform you that a user has tried logging in. But, User was unable to login due to User-Donor profile disconnection.', 'give')."\n\n";
+	$message .= __('Do you want to reconnect User and Donor profile again?', 'give')."\n\n";
 	$message .= sprintf(
 		'<a href="%1$s">%2$s</a>',
-		esc_url( admin_url() . 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&user_id=' . $user_id . '&give-messages[]=reconnect-user' ),
-		__( 'Reconnect User', 'give' ) . "\n\n"
+		esc_url(admin_url().'edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&user_id='.$user_id.'&give-messages[]=reconnect-user'),
+		__('Reconnect User', 'give')."\n\n"
 	);
-	$message .= __( 'Thank you,', 'give' ) . "\n\n";
-	$message .= '{sitename}' . "\n";
+	$message .= __('Thank you,', 'give')."\n\n";
+	$message .= '{sitename}'."\n";
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'headers', $headers );
-	$emails->__set( 'heading', __( 'User - Donor Profile Disconnection', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('headers', $headers);
+	$emails->__set('heading', __('User - Donor Profile Disconnection', 'give'));
 
-	$emails->send( give_get_admin_notice_emails(), $subject, give_do_email_tags( $message ) );
+	$emails->send(give_get_admin_notice_emails(), $subject, give_do_email_tags($message));
 
 }

includes/admin/reports/graphing.php

Spacing +246 added lines, -246 removed lines

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

includes/admin/payments/actions.php

Spacing +124 added lines, -124 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,48 +25,48 @@ 
  *
  * @return      void
  */
-function give_update_payment_details( $data ) {
+function give_update_payment_details($data) {
 
-	if ( ! current_user_can( 'edit_give_payments', $data['give_payment_id'] ) ) {
-		wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $data['give_payment_id'])) {
+		wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	check_admin_referer( 'give_update_payment_details_nonce' );
+	check_admin_referer('give_update_payment_details_nonce');
 
 	// Retrieve the payment ID.
-	$payment_id = absint( $data['give_payment_id'] );
+	$payment_id = absint($data['give_payment_id']);
 
 	/* @var Give_Payment $payment */
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
 	$status = $data['give-payment-status'];
-	$date   = sanitize_text_field( $data['give-payment-date'] );
-	$hour   = sanitize_text_field( $data['give-payment-time-hour'] );
+	$date   = sanitize_text_field($data['give-payment-date']);
+	$hour   = sanitize_text_field($data['give-payment-time-hour']);
 
 	// Restrict to our high and low.
-	if ( $hour > 23 ) {
+	if ($hour > 23) {
 		$hour = 23;
-	} elseif ( $hour < 0 ) {
+	} elseif ($hour < 0) {
 		$hour = 00;
 	}
 
-	$minute = sanitize_text_field( $data['give-payment-time-min'] );
+	$minute = sanitize_text_field($data['give-payment-time-min']);
 
 	// Restrict to our high and low.
-	if ( $minute > 59 ) {
+	if ($minute > 59) {
 		$minute = 59;
-	} elseif ( $minute < 0 ) {
+	} elseif ($minute < 0) {
 		$minute = 00;
 	}
 
-	$address = give_clean( $data['give-payment-address'][0] );
+	$address = give_clean($data['give-payment-address'][0]);
 
 	$curr_total = $payment->total;
-	$new_total  = give_maybe_sanitize_amount( ( ! empty( $data['give-payment-total'] ) ? $data['give-payment-total'] : 0 ) );
-	$date       = date( 'Y-m-d', strtotime( $date ) ) . ' ' . $hour . ':' . $minute . ':00';
+	$new_total  = give_maybe_sanitize_amount(( ! empty($data['give-payment-total']) ? $data['give-payment-total'] : 0));
+	$date       = date('Y-m-d', strtotime($date)).' '.$hour.':'.$minute.':00';
 
-	$curr_donor_id = sanitize_text_field( $data['give-current-donor'] );
-	$new_donor_id  = sanitize_text_field( $data['donor-id'] );
+	$curr_donor_id = sanitize_text_field($data['give-current-donor']);
+	$new_donor_id  = sanitize_text_field($data['donor-id']);
 
 	/**
 	 * Fires before updating edited donation.
@@ -76,71 +76,71 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_update_edited_donation', $payment_id );
+	do_action('give_update_edited_donation', $payment_id);
 
 	$payment->date = $date;
 	$updated       = $payment->save();
 
-	if ( 0 === $updated ) {
-		wp_die( __( 'Error Updating Donation.', 'give' ), __( 'Error', 'give' ), array( 'response' => 400 ) );
+	if (0 === $updated) {
+		wp_die(__('Error Updating Donation.', 'give'), __('Error', 'give'), array('response' => 400));
 	}
 
 	$donor_changed = false;
 
-	if ( isset( $data['give-new-donor'] ) && $data['give-new-donor'] == '1' ) {
+	if (isset($data['give-new-donor']) && $data['give-new-donor'] == '1') {
 
-		$email      = ! empty( $data['give-new-donor-email'] ) ? sanitize_text_field( $data['give-new-donor-email'] ) : '';
-		$first_name = ! empty( $data['give-new-donor-first-name'] ) ? sanitize_text_field( $data['give-new-donor-first-name'] ) : '';
-		$last_name  = ! empty( $data['give-new-donor-last-name'] ) ? sanitize_text_field( $data['give-new-donor-last-name'] ) : '';
-		$names      = strip_tags( wp_unslash( trim( "{$first_name} {$last_name}" ) ) );
+		$email      = ! empty($data['give-new-donor-email']) ? sanitize_text_field($data['give-new-donor-email']) : '';
+		$first_name = ! empty($data['give-new-donor-first-name']) ? sanitize_text_field($data['give-new-donor-first-name']) : '';
+		$last_name  = ! empty($data['give-new-donor-last-name']) ? sanitize_text_field($data['give-new-donor-last-name']) : '';
+		$names      = strip_tags(wp_unslash(trim("{$first_name} {$last_name}")));
 
-		if ( empty( $email ) || empty( $first_name ) ) {
-			wp_die( __( 'New Donor requires first name and email address.', 'give' ), __( 'Error', 'give' ), array( 'response' => 400 ) );
+		if (empty($email) || empty($first_name)) {
+			wp_die(__('New Donor requires first name and email address.', 'give'), __('Error', 'give'), array('response' => 400));
 		}
 
-		$donor = new Give_Donor( $email );
-		if ( empty( $donor->id ) ) {
-			$donor_data = array( 'name' => $names, 'email' => $email );
-			$user_id       = email_exists( $email );
-			if ( false !== $user_id ) {
+		$donor = new Give_Donor($email);
+		if (empty($donor->id)) {
+			$donor_data = array('name' => $names, 'email' => $email);
+			$user_id = email_exists($email);
+			if (false !== $user_id) {
 				$donor_data['user_id'] = $user_id;
 			}
 
-			if ( ! $donor->create( $donor_data ) ) {
+			if ( ! $donor->create($donor_data)) {
 				// Failed to create the new donor, assume the previous donor.
 				$donor_changed = false;
-				$donor         = new Give_Donor( $curr_donor_id );
-				give_set_error( 'give-payment-new-donor-fail', __( 'Error creating new donor.', 'give' ) );
+				$donor         = new Give_Donor($curr_donor_id);
+				give_set_error('give-payment-new-donor-fail', __('Error creating new donor.', 'give'));
 			}
 		}
 
 		// Create and Update Donor First Name and Last Name in Meta Fields.
-		$donor->update_meta( '_give_donor_first_name', $first_name );
-		$donor->update_meta( '_give_donor_last_name', $last_name );
+		$donor->update_meta('_give_donor_first_name', $first_name);
+		$donor->update_meta('_give_donor_last_name', $last_name);
 
 		$new_donor_id = $donor->id;
 
-		$previous_donor = new Give_Donor( $curr_donor_id );
+		$previous_donor = new Give_Donor($curr_donor_id);
 
 		$donor_changed = true;
 
-	} elseif ( $curr_donor_id !== $new_donor_id ) {
+	} elseif ($curr_donor_id !== $new_donor_id) {
 
-		$donor = new Give_Donor( $new_donor_id );
+		$donor = new Give_Donor($new_donor_id);
 		$email    = $donor->email;
 		$names    = $donor->name;
 
-		$previous_donor = new Give_Donor( $curr_donor_id );
+		$previous_donor = new Give_Donor($curr_donor_id);
 
 		$donor_changed = true;
 
 	} else {
-		$donor = new Give_Donor( $curr_donor_id );
+		$donor = new Give_Donor($curr_donor_id);
 		$email    = $donor->email;
 		$names    = $donor->name;
 	}
 
-	if ( $donor_changed ) {
+	if ($donor_changed) {
 
 		// Setup first and last name from input values.
 		$first_name = $donor->get_first_name();
@@ -150,26 +150,26 @@ 
 		$payment->last_name  = $last_name;
 
 		// Remove the stats and payment from the previous donor and attach it to the new donor.
-		$previous_donor->remove_payment( $payment_id, false );
-		$donor->attach_payment( $payment_id, false );
+		$previous_donor->remove_payment($payment_id, false);
+		$donor->attach_payment($payment_id, false);
 
-		if ( 'publish' == $status ) {
+		if ('publish' == $status) {
 
 			// Reduce previous user donation count and amount.
 			$previous_donor->decrease_donation_count();
-			$previous_donor->decrease_value( $curr_total );
+			$previous_donor->decrease_value($curr_total);
 
 			// If donation was completed adjust stats of new donors.
 			$donor->increase_purchase_count();
-			$donor->increase_value( $new_total );
+			$donor->increase_value($new_total);
 		}
 
 		$payment->customer_id = $donor->id;
 	} else {
 
-		if ( 'publish' === $status ) {
+		if ('publish' === $status) {
 			// Update user donation stat.
-			$donor->update_donation_value( $curr_total, $new_total );
+			$donor->update_donation_value($curr_total, $new_total);
 		}
 	}
 
@@ -180,10 +180,10 @@ 
 	$payment->total      = $new_total;
 
 	// Check for payment notes.
-	if ( ! empty( $data['give-payment-note'] ) ) {
+	if ( ! empty($data['give-payment-note'])) {
 
-		$note = wp_kses( $data['give-payment-note'], array() );
-		give_insert_payment_note( $payment_id, $note );
+		$note = wp_kses($data['give-payment-note'], array());
+		give_insert_payment_note($payment_id, $note);
 
 	}
 
@@ -191,17 +191,17 @@ 
 	$payment->status = $status;
 
 	// Adjust total store earnings if the payment total has been changed.
-	if ( $new_total !== $curr_total && 'publish' == $status ) {
+	if ($new_total !== $curr_total && 'publish' == $status) {
 
-		if ( $new_total > $curr_total ) {
+		if ($new_total > $curr_total) {
 			// Increase if our new total is higher.
 			$difference = $new_total - $curr_total;
-			give_increase_total_earnings( $difference );
+			give_increase_total_earnings($difference);
 
-		} elseif ( $curr_total > $new_total ) {
+		} elseif ($curr_total > $new_total) {
 			// Decrease if our new total is lower.
 			$difference = $curr_total - $new_total;
-			give_decrease_total_earnings( $difference );
+			give_decrease_total_earnings($difference);
 
 		}
 	}
@@ -209,55 +209,55 @@ 
 	$payment->save();
 
 	// Get new give form ID.
-	$new_form_id     = absint( $data['give-payment-form-select'] );
-	$current_form_id = absint( $payment->get_meta( '_give_payment_form_id' ) );
+	$new_form_id     = absint($data['give-payment-form-select']);
+	$current_form_id = absint($payment->get_meta('_give_payment_form_id'));
 
 	// We are adding payment transfer code in last to remove any conflict with above functionality.
 	// For example: above code will automatically handle form stat (increase/decrease) when payment status changes.
 	// Check if user want to transfer current payment to new give form id.
-	if ( $new_form_id && $new_form_id != $current_form_id ) {
+	if ($new_form_id && $new_form_id != $current_form_id) {
 
 		// Get new give form title.
-		$new_form_title = get_the_title( $new_form_id );
+		$new_form_title = get_the_title($new_form_id);
 
 		// Update payment give form meta data.
-		$payment->update_meta( '_give_payment_form_id', $new_form_id );
-		$payment->update_meta( '_give_payment_form_title', $new_form_title );
+		$payment->update_meta('_give_payment_form_id', $new_form_id);
+		$payment->update_meta('_give_payment_form_title', $new_form_title);
 
 		// Update price id payment metadata.
-		if ( ! give_has_variable_prices( $new_form_id ) ) {
-			$payment->update_meta( '_give_payment_price_id', '' );
+		if ( ! give_has_variable_prices($new_form_id)) {
+			$payment->update_meta('_give_payment_price_id', '');
 		}
 
 		// If donation was completed, adjust stats of forms.
-		if ( 'publish' == $status ) {
+		if ('publish' == $status) {
 
 			// Decrease sale of old give form. For other payment status.
-			$current_form = new Give_Donate_Form( $current_form_id );
+			$current_form = new Give_Donate_Form($current_form_id);
 			$current_form->decrease_sales();
-			$current_form->decrease_earnings( $curr_total, $payment->ID );
+			$current_form->decrease_earnings($curr_total, $payment->ID);
 
 			// Increase sale of new give form.
-			$new_form = new Give_Donate_Form( $new_form_id );
+			$new_form = new Give_Donate_Form($new_form_id);
 			$new_form->increase_sales();
-			$new_form->increase_earnings( $new_total, $payment->ID );
+			$new_form->increase_earnings($new_total, $payment->ID);
 		}
 
 		// Re setup payment to update new meta value in object.
-		$payment->update_payment_setup( $payment->ID );
+		$payment->update_payment_setup($payment->ID);
 
 		// Update form id in payment logs.
-		Give()->async_process->data( array(
-			'data' => array( $new_form_id, $payment_id ),
+		Give()->async_process->data(array(
+			'data' => array($new_form_id, $payment_id),
 			'hook' => 'give_update_log_form_id',
-		) )->dispatch();
+		))->dispatch();
 	}
 
 	// Update price id if current form is variable form.
 	/* @var Give_Donate_Form $form */
-	$form = new Give_Donate_Form( $payment->form_id );
+	$form = new Give_Donate_Form($payment->form_id);
 
-	if ( isset( $data['give-variable-price'] ) && $form->has_variable_prices() ) {
+	if (isset($data['give-variable-price']) && $form->has_variable_prices()) {
 
 		// Get payment meta data.
 		$payment_meta = $payment->get_meta();
@@ -266,9 +266,9 @@ 
 		$price_id = '';
 
 		// Get price info
-		if( 0 <= $data['give-variable-price'] ) {
-			foreach ( $form->prices as $variable_price ) {
-				if( $new_total === give_maybe_sanitize_amount( $variable_price['_give_amount'] ) ) {
+		if (0 <= $data['give-variable-price']) {
+			foreach ($form->prices as $variable_price) {
+				if ($new_total === give_maybe_sanitize_amount($variable_price['_give_amount'])) {
 					$price_info = $variable_price;
 					break;
 				}
@@ -276,15 +276,15 @@ 
 		}
 
 		// Set price id.
-		if( ! empty( $price_info ) ) {
+		if ( ! empty($price_info)) {
 			$price_id = $data['give-variable-price'];
 
-			if( $data['give-variable-price'] !== $price_info['_give_id']['level_id'] ) {
+			if ($data['give-variable-price'] !== $price_info['_give_id']['level_id']) {
 				// Set price id to amount match.
 				$price_id = $price_info['_give_id']['level_id'];
 			}
 
-		} elseif( $form->is_custom_price_mode() ){
+		} elseif ($form->is_custom_price_mode()) {
 			$price_id = 'custom';
 		}
 
@@ -292,11 +292,11 @@ 
 		$payment_meta['price_id'] = $price_id;
 
 		// Update payment give form meta data.
-		$payment->update_meta( '_give_payment_price_id', $price_id );
-		$payment->update_meta( '_give_payment_meta', $payment_meta );
+		$payment->update_meta('_give_payment_price_id', $price_id);
+		$payment->update_meta('_give_payment_meta', $payment_meta);
 
 		// Re setup payment to update new meta value in object.
-		$payment->update_payment_setup( $payment->ID );
+		$payment->update_payment_setup($payment->ID);
 	}
 
 	/**
@@ -307,13 +307,13 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_updated_edited_donation', $payment_id );
+	do_action('give_updated_edited_donation', $payment_id);
 
-	wp_safe_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-messages[]=payment-updated&id=' . $payment_id ) );
+	wp_safe_redirect(admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-messages[]=payment-updated&id='.$payment_id));
 	exit;
 }
 
-add_action( 'give_update_payment_details', 'give_update_payment_details' );
+add_action('give_update_payment_details', 'give_update_payment_details');
 
 /**
  * Trigger a Donation Deletion.
@@ -324,48 +324,48 @@ 
  *
  * @return void
  */
-function give_trigger_donation_delete( $data ) {
-	if ( wp_verify_nonce( $data['_wpnonce'], 'give_donation_nonce' ) ) {
+function give_trigger_donation_delete($data) {
+	if (wp_verify_nonce($data['_wpnonce'], 'give_donation_nonce')) {
 
-		$payment_id = absint( $data['purchase_id'] );
+		$payment_id = absint($data['purchase_id']);
 
-		if ( ! current_user_can( 'edit_give_payments', $payment_id ) ) {
-			wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! current_user_can('edit_give_payments', $payment_id)) {
+			wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
-		give_delete_donation( $payment_id );
-		wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&give-messages[]=donation-deleted' ) );
+		give_delete_donation($payment_id);
+		wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-payment-history&give-messages[]=donation-deleted'));
 		give_die();
 	}
 }
 
-add_action( 'give_delete_payment', 'give_trigger_donation_delete' );
+add_action('give_delete_payment', 'give_trigger_donation_delete');
 
 /**
  * AJAX Store Donation Note
  */
 function give_ajax_store_payment_note() {
 
-	$payment_id = absint( $_POST['payment_id'] );
-	$note       = wp_kses( $_POST['note'], array() );
+	$payment_id = absint($_POST['payment_id']);
+	$note       = wp_kses($_POST['note'], array());
 
-	if ( ! current_user_can( 'edit_give_payments', $payment_id ) ) {
-		wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $payment_id)) {
+		wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	if ( empty( $payment_id ) ) {
-		die( '-1' );
+	if (empty($payment_id)) {
+		die('-1');
 	}
 
-	if ( empty( $note ) ) {
-		die( '-1' );
+	if (empty($note)) {
+		die('-1');
 	}
 
-	$note_id = give_insert_payment_note( $payment_id, $note );
-	die( give_get_payment_note_html( $note_id ) );
+	$note_id = give_insert_payment_note($payment_id, $note);
+	die(give_get_payment_note_html($note_id));
 }
 
-add_action( 'wp_ajax_give_insert_payment_note', 'give_ajax_store_payment_note' );
+add_action('wp_ajax_give_insert_payment_note', 'give_ajax_store_payment_note');
 
 /**
  * Triggers a donation note deletion without ajax
@@ -376,24 +376,24 @@ 
  *
  * @return void
  */
-function give_trigger_payment_note_deletion( $data ) {
+function give_trigger_payment_note_deletion($data) {
 
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_delete_payment_note_' . $data['note_id'] ) ) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_delete_payment_note_'.$data['note_id'])) {
 		return;
 	}
 
-	if ( ! current_user_can( 'edit_give_payments', $data['payment_id'] ) ) {
-		wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $data['payment_id'])) {
+		wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	$edit_order_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-messages[]=donation-note-deleted&id=' . absint( $data['payment_id'] ) );
+	$edit_order_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-messages[]=donation-note-deleted&id='.absint($data['payment_id']));
 
-	give_delete_payment_note( $data['note_id'], $data['payment_id'] );
+	give_delete_payment_note($data['note_id'], $data['payment_id']);
 
-	wp_redirect( $edit_order_url );
+	wp_redirect($edit_order_url);
 }
 
-add_action( 'give_delete_payment_note', 'give_trigger_payment_note_deletion' );
+add_action('give_delete_payment_note', 'give_trigger_payment_note_deletion');
 
 /**
  * Delete a payment note deletion with ajax
@@ -404,16 +404,16 @@ 
  */
 function give_ajax_delete_payment_note() {
 
-	if ( ! current_user_can( 'edit_give_payments', $_POST['payment_id'] ) ) {
-		wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $_POST['payment_id'])) {
+		wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	if ( give_delete_payment_note( $_POST['note_id'], $_POST['payment_id'] ) ) {
-		die( '1' );
+	if (give_delete_payment_note($_POST['note_id'], $_POST['payment_id'])) {
+		die('1');
 	} else {
-		die( '-1' );
+		die('-1');
 	}
 
 }
 
-add_action( 'wp_ajax_give_delete_payment_note', 'give_ajax_delete_payment_note' );
+add_action('wp_ajax_give_delete_payment_note', 'give_ajax_delete_payment_note');

includes/admin/payments/view-payment-details.php

Spacing +151 added lines, -151 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! current_user_can( 'view_give_payments' ) ) {
+if ( ! current_user_can('view_give_payments')) {
 	wp_die(
-		__( 'Sorry, you are not allowed to access this page.', 'give' ), __( 'Error', 'give' ), array(
+		__('Sorry, you are not allowed to access this page.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
 		)
 	);
@@ -28,35 +28,35 @@ 
  * @since 1.0
  * @return void
  */
-if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-	wp_die( __( 'Donation ID not supplied. Please try again.', 'give' ), __( 'Error', 'give' ), array( 'response' => 400 ) );
+if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+	wp_die(__('Donation ID not supplied. Please try again.', 'give'), __('Error', 'give'), array('response' => 400));
 }
 
 // Setup the variables
-$payment_id = absint( $_GET['id'] );
-$payment    = new Give_Payment( $payment_id );
+$payment_id = absint($_GET['id']);
+$payment    = new Give_Payment($payment_id);
 
 // Sanity check... fail if donation ID is invalid
 $payment_exists = $payment->ID;
-if ( empty( $payment_exists ) ) {
-	wp_die( __( 'The specified ID does not belong to a donation. Please try again.', 'give' ), __( 'Error', 'give' ), array( 'response' => 400 ) );
+if (empty($payment_exists)) {
+	wp_die(__('The specified ID does not belong to a donation. Please try again.', 'give'), __('Error', 'give'), array('response' => 400));
 }
 
 $number       = $payment->number;
 $payment_meta = $payment->get_meta();
 
-$company_name   = ! empty( $payment_meta['_give_donation_company'] ) ? esc_attr( $payment_meta['_give_donation_company'] ) : '';
-$transaction_id = esc_attr( $payment->transaction_id );
+$company_name   = ! empty($payment_meta['_give_donation_company']) ? esc_attr($payment_meta['_give_donation_company']) : '';
+$transaction_id = esc_attr($payment->transaction_id);
 $user_id        = $payment->user_id;
 $donor_id       = $payment->customer_id;
-$payment_date   = strtotime( $payment->date );
-$user_info      = give_get_payment_meta_user_info( $payment_id );
+$payment_date   = strtotime($payment->date);
+$user_info      = give_get_payment_meta_user_info($payment_id);
 $address        = $payment->address;
 $currency_code  = $payment->currency;
 $gateway        = $payment->gateway;
 $currency_code  = $payment->currency;
 $payment_mode   = $payment->mode;
-$base_url       = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+$base_url       = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 
 ?>
 <div class="wrap give-wrap">
@@ -65,13 +65,13 @@ 
 	<?php
 		printf(
 			/* translators: %s: donation number */
-			esc_html__( 'Donation %s', 'give' ),
+			esc_html__('Donation %s', 'give'),
 			$number
 		);
-		if ( $payment_mode == 'test' ) {
+		if ($payment_mode == 'test') {
 			echo Give()->tooltips->render_span(array(
-				'label' => __( 'This donation was made in test mode.', 'give' ),
-				'tag_content' => __( 'Test Donation', 'give' ),
+				'label' => __('This donation was made in test mode.', 'give'),
+				'tag_content' => __('Test Donation', 'give'),
 				'position'=> 'right',
 				'attributes' => array(
 					'id' => 'test-payment-label',
@@ -90,7 +90,7 @@ 
 	 *
 	 * @param int $payment_id Payment id.
 	 */
-	do_action( 'give_view_donation_details_before', $payment_id );
+	do_action('give_view_donation_details_before', $payment_id);
 	?>
 
 	<hr class="wp-header-end">
@@ -104,7 +104,7 @@ 
 		 *
 		 * @param int $payment_id Payment id.
 		 */
-		do_action( 'give_view_donation_details_form_top', $payment_id );
+		do_action('give_view_donation_details_form_top', $payment_id);
 		?>
 		<div id="poststuff">
 			<div id="give-dashboard-widgets-wrap">
@@ -120,16 +120,16 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_sidebar_before', $payment_id );
+							do_action('give_view_donation_details_sidebar_before', $payment_id);
 							?>
 
 							<div id="give-order-update" class="postbox give-order-data">
 
 								<div class="give-order-top">
-									<h3 class="hndle"><?php _e( 'Update Donation', 'give' ); ?></h3>
+									<h3 class="hndle"><?php _e('Update Donation', 'give'); ?></h3>
 
 									<?php
-									if ( current_user_can( 'view_give_payments' ) ) {
+									if (current_user_can('view_give_payments')) {
 										echo sprintf(
 											'<span class="delete-donation" id="delete-donation-%d"><a class="delete-single-donation delete-donation-button dashicons dashicons-trash" href="%s" aria-label="%s"></a></span>',
 											$payment_id,
@@ -141,7 +141,7 @@ 
 													), $base_url
 												), 'give_donation_nonce'
 											),
-											sprintf( __( 'Delete Donation %s', 'give' ), $payment_id )
+											sprintf(__('Delete Donation %s', 'give'), $payment_id)
 										);
 									}
 									?>
@@ -158,33 +158,33 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_totals_before', $payment_id );
+										do_action('give_view_donation_details_totals_before', $payment_id);
 										?>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-status" class="strong"><?php _e( 'Status:', 'give' ); ?></label>&nbsp;
+												<label for="give-payment-status" class="strong"><?php _e('Status:', 'give'); ?></label>&nbsp;
 												<select id="give-payment-status" name="give-payment-status" class="medium-text">
-													<?php foreach ( give_get_payment_statuses() as $key => $status ) : ?>
-														<option value="<?php echo esc_attr( $key ); ?>"<?php selected( $payment->status, $key, true ); ?>><?php echo esc_html( $status ); ?></option>
+													<?php foreach (give_get_payment_statuses() as $key => $status) : ?>
+														<option value="<?php echo esc_attr($key); ?>"<?php selected($payment->status, $key, true); ?>><?php echo esc_html($status); ?></option>
 													<?php endforeach; ?>
 												</select>
-												<span class="give-donation-status status-<?php echo sanitize_title( $payment->status ); ?>"><span class="give-donation-status-icon"></span></span>
+												<span class="give-donation-status status-<?php echo sanitize_title($payment->status); ?>"><span class="give-donation-status-icon"></span></span>
 											</p>
 										</div>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-date" class="strong"><?php _e( 'Date:', 'give' ); ?></label>&nbsp;
-												<input type="text" id="give-payment-date" name="give-payment-date" value="<?php echo esc_attr( date( 'm/d/Y', $payment_date ) ); ?>" class="medium-text give_datepicker"/>
+												<label for="give-payment-date" class="strong"><?php _e('Date:', 'give'); ?></label>&nbsp;
+												<input type="text" id="give-payment-date" name="give-payment-date" value="<?php echo esc_attr(date('m/d/Y', $payment_date)); ?>" class="medium-text give_datepicker"/>
 											</p>
 										</div>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-time-hour" class="strong"><?php _e( 'Time:', 'give' ); ?></label>&nbsp;
-												<input type="number" step="1" max="24" id="give-payment-time-hour" name="give-payment-time-hour" value="<?php echo esc_attr( date_i18n( 'H', $payment_date ) ); ?>" class="small-text give-payment-time-hour"/>&nbsp;:&nbsp;
-												<input type="number" step="1" max="59" id="give-payment-time-min" name="give-payment-time-min" value="<?php echo esc_attr( date( 'i', $payment_date ) ); ?>" class="small-text give-payment-time-min"/>
+												<label for="give-payment-time-hour" class="strong"><?php _e('Time:', 'give'); ?></label>&nbsp;
+												<input type="number" step="1" max="24" id="give-payment-time-hour" name="give-payment-time-hour" value="<?php echo esc_attr(date_i18n('H', $payment_date)); ?>" class="small-text give-payment-time-hour"/>&nbsp;:&nbsp;
+												<input type="number" step="1" max="59" id="give-payment-time-min" name="give-payment-time-min" value="<?php echo esc_attr(date('i', $payment_date)); ?>" class="small-text give-payment-time-min"/>
 											</p>
 										</div>
 
@@ -198,14 +198,14 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_update_inner', $payment_id );
+										do_action('give_view_donation_details_update_inner', $payment_id);
 										?>
 
 										<div class="give-order-payment give-admin-box-inside">
 											<p>
-												<label for="give-payment-total" class="strong"><?php _e( 'Total Donation:', 'give' ); ?></label>&nbsp;
-												<?php echo give_currency_symbol( $payment->currency ); ?>
-												&nbsp;<input id="give-payment-total" name="give-payment-total" type="text" class="small-text give-price-field" value="<?php echo esc_attr( give_format_decimal( give_donation_amount( $payment_id ), false, false ) ); ?>"/>
+												<label for="give-payment-total" class="strong"><?php _e('Total Donation:', 'give'); ?></label>&nbsp;
+												<?php echo give_currency_symbol($payment->currency); ?>
+												&nbsp;<input id="give-payment-total" name="give-payment-total" type="text" class="small-text give-price-field" value="<?php echo esc_attr(give_format_decimal(give_donation_amount($payment_id), false, false)); ?>"/>
 											</p>
 										</div>
 
@@ -217,7 +217,7 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_totals_after', $payment_id );
+										do_action('give_view_donation_details_totals_after', $payment_id);
 										?>
 
 									</div>
@@ -235,27 +235,27 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_view_donation_details_update_before', $payment_id );
+									do_action('give_view_donation_details_update_before', $payment_id);
 									?>
 
 									<div id="major-publishing-actions">
 										<div id="publishing-action">
 
-											<input type="submit" class="button button-primary right" value="<?php esc_attr_e( 'Save Donation', 'give' ); ?>"/>
+											<input type="submit" class="button button-primary right" value="<?php esc_attr_e('Save Donation', 'give'); ?>"/>
 											<?php
-											if ( give_is_payment_complete( $payment_id ) ) {
+											if (give_is_payment_complete($payment_id)) {
 												$url = add_query_arg(
 													array(
 														'give-action' => 'email_links',
 														'purchase_id' => $payment_id,
 													),
-													admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment_id )
+													admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment_id)
 												);
 
 												echo sprintf(
 													'<a href="%1$s" id="give-resend-receipt" class="button-secondary right">%2$s</a>',
-													esc_url( $url ),
-													esc_html__( 'Resend Receipt', 'give' )
+													esc_url($url),
+													esc_html__('Resend Receipt', 'give')
 												);
 											}
 											?>
@@ -270,7 +270,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_view_donation_details_update_after', $payment_id );
+									do_action('give_view_donation_details_update_after', $payment_id);
 									?>
 
 								</div>
@@ -281,7 +281,7 @@ 
 
 							<div id="give-order-details" class="postbox give-order-data">
 
-								<h3 class="hndle"><?php _e( 'Donation Meta', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Donation Meta', 'give'); ?></h3>
 
 								<div class="inside">
 									<div class="give-admin-box">
@@ -294,30 +294,30 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_payment_meta_before', $payment_id );
+										do_action('give_view_donation_details_payment_meta_before', $payment_id);
 
-										$gateway = give_get_payment_gateway( $payment_id );
-										if ( $gateway ) :
+										$gateway = give_get_payment_gateway($payment_id);
+										if ($gateway) :
 										?>
 											<div class="give-order-gateway give-admin-box-inside">
 												<p>
-													<strong><?php _e( 'Gateway:', 'give' ); ?></strong>&nbsp;
-													<?php echo give_get_gateway_admin_label( $gateway ); ?>
+													<strong><?php _e('Gateway:', 'give'); ?></strong>&nbsp;
+													<?php echo give_get_gateway_admin_label($gateway); ?>
 												</p>
 											</div>
 										<?php endif; ?>
 
 										<div class="give-order-payment-key give-admin-box-inside">
 											<p>
-												<strong><?php _e( 'Key:', 'give' ); ?></strong>&nbsp;
-												<?php echo give_get_payment_key( $payment_id ); ?>
+												<strong><?php _e('Key:', 'give'); ?></strong>&nbsp;
+												<?php echo give_get_payment_key($payment_id); ?>
 											</p>
 										</div>
 
 										<div class="give-order-ip give-admin-box-inside">
 											<p>
-												<strong><?php _e( 'IP:', 'give' ); ?></strong>&nbsp;
-												<?php echo esc_html( give_get_payment_user_ip( $payment_id ) ); ?>
+												<strong><?php _e('IP:', 'give'); ?></strong>&nbsp;
+												<?php echo esc_html(give_get_payment_user_ip($payment_id)); ?>
 											</p>
 										</div>
 
@@ -325,18 +325,18 @@ 
 										// Display the transaction ID present.
 										// The transaction ID is the charge ID from the gateway.
 										// For instance, stripe "ch_BzvwYCchqOy5Nt".
-										if ( $transaction_id != $payment_id ) : ?>
+										if ($transaction_id != $payment_id) : ?>
 											<div class="give-order-tx-id give-admin-box-inside">
 												<p>
-													<strong><?php _e( 'Transaction ID:', 'give' ); ?> <span class="give-tooltip give-icon give-icon-question"  data-tooltip="<?php echo sprintf( esc_attr__( 'The transaction ID within %s.', 'give' ), $gateway); ?>"></span></strong>&nbsp;
-													<?php echo apply_filters( "give_payment_details_transaction_id-{$gateway}", $transaction_id, $payment_id ); ?>
+													<strong><?php _e('Transaction ID:', 'give'); ?> <span class="give-tooltip give-icon give-icon-question"  data-tooltip="<?php echo sprintf(esc_attr__('The transaction ID within %s.', 'give'), $gateway); ?>"></span></strong>&nbsp;
+													<?php echo apply_filters("give_payment_details_transaction_id-{$gateway}", $transaction_id, $payment_id); ?>
 												</p>
 											</div>
 										<?php endif; ?>
 
 										<div class="give-admin-box-inside">
-											<p><?php $purchase_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( give_get_payment_donor_id( $payment_id ) ) ); ?>
-												<a href="<?php echo $purchase_url; ?>"><?php _e( 'View all donations for this donor &raquo;', 'give' ); ?></a>
+											<p><?php $purchase_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint(give_get_payment_donor_id($payment_id))); ?>
+												<a href="<?php echo $purchase_url; ?>"><?php _e('View all donations for this donor &raquo;', 'give'); ?></a>
 											</p>
 										</div>
 
@@ -348,7 +348,7 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_payment_meta_after', $payment_id );
+										do_action('give_view_donation_details_payment_meta_after', $payment_id);
 										?>
 
 									</div>
@@ -368,7 +368,7 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_sidebar_after', $payment_id );
+							do_action('give_view_donation_details_sidebar_after', $payment_id);
 							?>
 
 						</div>
@@ -388,31 +388,31 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_main_before', $payment_id );
+							do_action('give_view_donation_details_main_before', $payment_id);
 							?>
 
 							<?php $column_count = 'columns-3'; ?>
 							<div id="give-donation-overview" class="postbox <?php echo $column_count; ?>">
-								<h3 class="hndle"><?php _e( 'Donation Information', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Donation Information', 'give'); ?></h3>
 
 								<div class="inside">
 
 									<div class="column-container">
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Donation Form ID:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donation Form ID:', 'give'); ?></strong><br>
 												<?php
-												if ( $payment_meta['form_id'] ) :
+												if ($payment_meta['form_id']) :
 													printf(
 														'<a href="%1$s">%2$s</a>',
-														admin_url( 'post.php?action=edit&post=' . $payment_meta['form_id'] ),
+														admin_url('post.php?action=edit&post='.$payment_meta['form_id']),
 														$payment_meta['form_id']
 													);
 												endif;
 												?>
 											</p>
 											<p>
-												<strong><?php esc_html_e( 'Donation Form Title:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donation Form Title:', 'give'); ?></strong><br>
 												<?php
 												echo Give()->html->forms_dropdown(
 													array(
@@ -428,21 +428,21 @@ 
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Donation Date:', 'give' ); ?></strong><br>
-												<?php echo date_i18n( give_date_format(), $payment_date ); ?>
+												<strong><?php _e('Donation Date:', 'give'); ?></strong><br>
+												<?php echo date_i18n(give_date_format(), $payment_date); ?>
 											</p>
 											<p>
-												<strong><?php _e( 'Donation Level:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donation Level:', 'give'); ?></strong><br>
 												<span class="give-donation-level">
 													<?php
-													$var_prices = give_has_variable_prices( $payment_meta['form_id'] );
-													if ( empty( $var_prices ) ) {
-														_e( 'n/a', 'give' );
+													$var_prices = give_has_variable_prices($payment_meta['form_id']);
+													if (empty($var_prices)) {
+														_e('n/a', 'give');
 													} else {
 														$prices_atts = array();
-														if ( $variable_prices = give_get_variable_prices( $payment_meta['form_id'] ) ) {
-															foreach ( $variable_prices as $variable_price ) {
-																$prices_atts[ $variable_price['_give_id']['level_id'] ] = give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) );
+														if ($variable_prices = give_get_variable_prices($payment_meta['form_id'])) {
+															foreach ($variable_prices as $variable_price) {
+																$prices_atts[$variable_price['_give_id']['level_id']] = give_format_amount($variable_price['_give_amount'], array('sanitize' => false));
 															}
 														}
 														// Variable price dropdown options.
@@ -451,12 +451,12 @@ 
 															'name'             => 'give-variable-price',
 															'chosen'           => true,
 															'show_option_all'  => '',
-															'show_option_none' => ( '' === get_post_meta( $payment_id, '_give_payment_price_id', true ) ? __( 'None', 'give' ) : '' ),
-															'select_atts'      => 'data-prices=' . esc_attr( wp_json_encode( $prices_atts ) ),
+															'show_option_none' => ('' === get_post_meta($payment_id, '_give_payment_price_id', true) ? __('None', 'give') : ''),
+															'select_atts'      => 'data-prices='.esc_attr(wp_json_encode($prices_atts)),
 															'selected'         => $payment_meta['price_id'],
 														);
 														// Render variable prices select tag html.
-														give_get_form_variable_price_dropdown( $variable_price_dropdown_option, true );
+														give_get_form_variable_price_dropdown($variable_price_dropdown_option, true);
 													}
 													?>
 												</span>
@@ -464,8 +464,8 @@ 
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Total Donation:', 'give' ); ?></strong><br>
-												<?php echo give_donation_amount( $payment, true ); ?>
+												<strong><?php esc_html_e('Total Donation:', 'give'); ?></strong><br>
+												<?php echo give_donation_amount($payment, true); ?>
 											</p>
 
 											<p>
@@ -479,7 +479,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_thead_before', $payment_id );
+												do_action('give_donation_details_thead_before', $payment_id);
 
 
 												/**
@@ -491,7 +491,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_thead_after', $payment_id );
+												do_action('give_donation_details_thead_after', $payment_id);
 
 												/**
 												 * Fires in donation details page, in the donation-information metabox, before the body elements.
@@ -502,7 +502,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_tbody_before', $payment_id );
+												do_action('give_donation_details_tbody_before', $payment_id);
 
 												/**
 												 * Fires in donation details page, in the donation-information metabox, after the body elements.
@@ -513,7 +513,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_tbody_after', $payment_id );
+												do_action('give_donation_details_tbody_after', $payment_id);
 												?>
 											</p>
 										</div>
@@ -533,59 +533,59 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_donor_detail_before', $payment_id );
+							do_action('give_view_donation_details_donor_detail_before', $payment_id);
 							?>
 
 							<div id="give-donor-details" class="postbox">
-								<h3 class="hndle"><?php _e( 'Donor Details', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Donor Details', 'give'); ?></h3>
 
 								<div class="inside">
 
-									<?php $donor = new Give_Donor( $donor_id ); ?>
+									<?php $donor = new Give_Donor($donor_id); ?>
 
 									<div class="column-container donor-info">
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Donor ID:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donor ID:', 'give'); ?></strong><br>
 												<?php
-												if ( ! empty( $donor->id ) ) {
+												if ( ! empty($donor->id)) {
 													printf(
 														'<a href="%1$s">%2$s</a>',
-														admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ),
+														admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id),
 														$donor->id
 													);
 												}
 												?>
-												<span>(<a href="#new" class="give-payment-new-donor"><?php _e( 'Create New Donor', 'give' ); ?></a>)</span>
+												<span>(<a href="#new" class="give-payment-new-donor"><?php _e('Create New Donor', 'give'); ?></a>)</span>
 											</p>
 											<p>
-												<strong><?php _e( 'Donor Since:', 'give' ); ?></strong><br>
-												<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+												<strong><?php _e('Donor Since:', 'give'); ?></strong><br>
+												<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Donor Name:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donor Name:', 'give'); ?></strong><br>
 												<?php
-												$donor_billing_name = give_get_donor_name_by( $payment_id, 'donation' );
-												$donor_name         = give_get_donor_name_by( $donor_id, 'donor' );
+												$donor_billing_name = give_get_donor_name_by($payment_id, 'donation');
+												$donor_name         = give_get_donor_name_by($donor_id, 'donor');
 
 												// Check whether the donor name and WP_User name is same or not.
-												if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
-													echo $donor_billing_name . ' (<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id" ) ) . '">' . $donor_name . '</a>)';
+												if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
+													echo $donor_billing_name.' (<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id")).'">'.$donor_name.'</a>)';
 												} else {
 													echo $donor_name;
 												}
 												?>
 											</p>
 											<p>
-												<strong><?php _e( 'Donor Email:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donor Email:', 'give'); ?></strong><br>
 												<?php echo $donor->email; ?>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Change Donor:', 'give' ); ?></strong><br>
+												<strong><?php _e('Change Donor:', 'give'); ?></strong><br>
 												<?php
 												echo Give()->html->donor_dropdown(
 													array(
@@ -596,9 +596,9 @@ 
 												?>
 											</p>
 											<p>
-												<?php if ( ! empty( $company_name ) ) {
+												<?php if ( ! empty($company_name)) {
 													?>
-													<strong><?php esc_html_e( 'Company Name:', 'give' ); ?></strong><br>
+													<strong><?php esc_html_e('Company Name:', 'give'); ?></strong><br>
 													<?php
 													echo $company_name;
 												} ?>
@@ -609,19 +609,19 @@ 
 									<div class="column-container new-donor" style="display: none">
 										<div class="column">
 											<p>
-												<label for="give-new-donor-first-name"><?php _e( 'New Donor First Name:', 'give' ); ?></label>
+												<label for="give-new-donor-first-name"><?php _e('New Donor First Name:', 'give'); ?></label>
 												<input id="give-new-donor-first-name" type="text" name="give-new-donor-first-name" value="" class="medium-text"/>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<label for="give-new-donor-last-name"><?php _e( 'New Donor Last Name:', 'give' ); ?></label>
+												<label for="give-new-donor-last-name"><?php _e('New Donor Last Name:', 'give'); ?></label>
 												<input id="give-new-donor-last-name" type="text" name="give-new-donor-last-name" value="" class="medium-text"/>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<label for="give-new-donor-email"><?php _e( 'New Donor Email:', 'give' ); ?></label>
+												<label for="give-new-donor-email"><?php _e('New Donor Email:', 'give'); ?></label>
 												<input id="give-new-donor-email" type="email" name="give-new-donor-email" value="" class="medium-text"/>
 											</p>
 										</div>
@@ -629,9 +629,9 @@ 
 											<p>
 												<input type="hidden" name="give-current-donor" value="<?php echo $donor->id; ?>"/>
 												<input type="hidden" id="give-new-donor" name="give-new-donor" value="0"/>
-												<a href="#cancel" class="give-payment-new-donor-cancel give-delete"><?php _e( 'Cancel', 'give' ); ?></a>
+												<a href="#cancel" class="give-payment-new-donor-cancel give-delete"><?php _e('Cancel', 'give'); ?></a>
 												<br>
-												<em><?php _e( 'Click "Save Donation" to create new donor.', 'give' ); ?></em>
+												<em><?php _e('Click "Save Donation" to create new donor.', 'give'); ?></em>
 											</p>
 										</div>
 									</div>
@@ -646,7 +646,7 @@ 
 									 * @param array $payment_meta Payment meta.
 									 * @param array $user_info    User information.
 									 */
-									do_action( 'give_payment_personal_details_list', $payment_meta, $user_info );
+									do_action('give_payment_personal_details_list', $payment_meta, $user_info);
 
 									/**
 									 * Fires on the donation details page, in the donor-details metabox.
@@ -655,7 +655,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_view_details', $payment_id );
+									do_action('give_payment_view_details', $payment_id);
 									?>
 
 								</div>
@@ -671,11 +671,11 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_billing_before', $payment_id );
+							do_action('give_view_donation_details_billing_before', $payment_id);
 							?>
 
 							<div id="give-billing-details" class="postbox">
-								<h3 class="hndle"><?php _e( 'Billing Address', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Billing Address', 'give'); ?></h3>
 
 								<div class="inside">
 
@@ -685,9 +685,9 @@ 
 											<div class="data column-container">
 
 												<?php
-												$address['country'] = ( ! empty( $address['country'] ) ? $address['country'] : give_get_country() );
+												$address['country'] = ( ! empty($address['country']) ? $address['country'] : give_get_country());
 
-												$address['state'] = ( ! empty( $address['state'] ) ? $address['state'] : '' );
+												$address['state'] = ( ! empty($address['state']) ? $address['state'] : '');
 
 												// Get the country list that does not have any states init.
 												$no_states_country = give_no_states_country_list();
@@ -695,7 +695,7 @@ 
 
 												<div class="row">
 													<div id="give-order-address-country-wrap">
-														<label class="order-data-address-line"><?php _e( 'Country:', 'give' ); ?></label>
+														<label class="order-data-address-line"><?php _e('Country:', 'give'); ?></label>
 														<?php
 														echo Give()->html->select(
 															array(
@@ -705,8 +705,8 @@ 
 																'show_option_all'  => false,
 																'show_option_none' => false,
 																'chosen'           => true,
-																'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-																'data'             => array( 'search-type' => 'no_ajax' ),
+																'placeholder'      => esc_attr__('Select a country', 'give'),
+																'data'             => array('search-type' => 'no_ajax'),
 															)
 														);
 														?>
@@ -715,35 +715,35 @@ 
 
 												<div class="row">
 													<div class="give-wrap-address-line1">
-														<label for="give-payment-address-line1" class="order-data-address"><?php _e( 'Address 1:', 'give' ); ?></label>
-														<input id="give-payment-address-line1" type="text" name="give-payment-address[0][line1]" value="<?php echo esc_attr( $address['line1'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-line1" class="order-data-address"><?php _e('Address 1:', 'give'); ?></label>
+														<input id="give-payment-address-line1" type="text" name="give-payment-address[0][line1]" value="<?php echo esc_attr($address['line1']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<div class="row">
 													<div class="give-wrap-address-line2">
-														<label for="give-payment-address-line2" class="order-data-address-line"><?php _e( 'Address 2:', 'give' ); ?></label>
-														<input id="give-payment-address-line2" type="text" name="give-payment-address[0][line2]" value="<?php echo esc_attr( $address['line2'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-line2" class="order-data-address-line"><?php _e('Address 2:', 'give'); ?></label>
+														<input id="give-payment-address-line2" type="text" name="give-payment-address[0][line2]" value="<?php echo esc_attr($address['line2']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<div class="row">
 													<div class="give-wrap-address-city">
-														<label for="give-payment-address-city" class="order-data-address-line"><?php esc_html_e( 'City:', 'give' ); ?></label>
-														<input id="give-payment-address-city" type="text" name="give-payment-address[0][city]" value="<?php echo esc_attr( $address['city'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-city" class="order-data-address-line"><?php esc_html_e('City:', 'give'); ?></label>
+														<input id="give-payment-address-city" type="text" name="give-payment-address[0][city]" value="<?php echo esc_attr($address['city']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<?php
-												$state_exists = ( ! empty( $address['country'] ) && array_key_exists( $address['country'], $no_states_country ) ? true : false );
+												$state_exists = ( ! empty($address['country']) && array_key_exists($address['country'], $no_states_country) ? true : false);
 												?>
 												<div class="row">
-													<div class="<?php echo( ! empty( $state_exists ) ? 'column-full' : 'column' ); ?> give-column give-column-state">
-														<div id="give-order-address-state-wrap" class="<?php echo( ! empty( $state_exists ) ? 'give-hidden' : '' ); ?>">
-															<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+													<div class="<?php echo( ! empty($state_exists) ? 'column-full' : 'column'); ?> give-column give-column-state">
+														<div id="give-order-address-state-wrap" class="<?php echo( ! empty($state_exists) ? 'give-hidden' : ''); ?>">
+															<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 															<?php
-															$states = give_get_states( $address['country'] );
-															if ( ! empty( $states ) ) {
+															$states = give_get_states($address['country']);
+															if ( ! empty($states)) {
 																echo Give()->html->select(
 																	array(
 																		'options'          => $states,
@@ -752,23 +752,23 @@ 
 																		'show_option_all'  => false,
 																		'show_option_none' => false,
 																		'chosen'           => true,
-																		'placeholder'      => esc_attr__( 'Select a state', 'give' ),
-																		'data'             => array( 'search-type' => 'no_ajax' ),
+																		'placeholder'      => esc_attr__('Select a state', 'give'),
+																		'data'             => array('search-type' => 'no_ajax'),
 																	)
 																);
 															} else {
 																?>
-																<input id="give-payment-address-state" type="text" name="give-payment-address[0][state]" value="<?php echo esc_attr( $address['state'] ); ?>" class="medium-text"/>
+																<input id="give-payment-address-state" type="text" name="give-payment-address[0][state]" value="<?php echo esc_attr($address['state']); ?>" class="medium-text"/>
 																<?php
 															}
 															?>
 														</div>
 													</div>
 
-													<div class="<?php echo( ! empty( $state_exists ) ? 'column-full' : 'column' ); ?> give-column give-column-zip">
+													<div class="<?php echo( ! empty($state_exists) ? 'column-full' : 'column'); ?> give-column give-column-zip">
 														<div class="give-wrap-address-zip">
-															<label for="give-payment-address-zip" class="order-data-address-line"><?php _e( 'Zip / Postal Code:', 'give' ); ?></label>
-															<input id="give-payment-address-zip" type="text" name="give-payment-address[0][zip]" value="<?php echo esc_attr( $address['zip'] ); ?>" class="medium-text"/>
+															<label for="give-payment-address-zip" class="order-data-address-line"><?php _e('Zip / Postal Code:', 'give'); ?></label>
+															<input id="give-payment-address-zip" type="text" name="give-payment-address[0][zip]" value="<?php echo esc_attr($address['zip']); ?>" class="medium-text"/>
 														</div>
 													</div>
 												</div>
@@ -787,7 +787,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_billing_details', $payment_id );
+									do_action('give_payment_billing_details', $payment_id);
 									?>
 
 								</div>
@@ -803,34 +803,34 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_billing_after', $payment_id );
+							do_action('give_view_donation_details_billing_after', $payment_id);
 							?>
 
 							<div id="give-payment-notes" class="postbox">
-								<h3 class="hndle"><?php _e( 'Donation Notes', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Donation Notes', 'give'); ?></h3>
 
 								<div class="inside">
 									<div id="give-payment-notes-inner">
 										<?php
-										$notes = give_get_payment_notes( $payment_id );
-										if ( ! empty( $notes ) ) {
+										$notes = give_get_payment_notes($payment_id);
+										if ( ! empty($notes)) {
 											$no_notes_display = ' style="display:none;"';
-											foreach ( $notes as $note ) :
+											foreach ($notes as $note) :
 
-												echo give_get_payment_note_html( $note, $payment_id );
+												echo give_get_payment_note_html($note, $payment_id);
 
 											endforeach;
 										} else {
 											$no_notes_display = '';
 										}
 
-										echo '<p class="give-no-payment-notes"' . $no_notes_display . '>' . esc_html__( 'No donation notes.', 'give' ) . '</p>';
+										echo '<p class="give-no-payment-notes"'.$no_notes_display.'>'.esc_html__('No donation notes.', 'give').'</p>';
 										?>
 									</div>
 									<textarea name="give-payment-note" id="give-payment-note" class="large-text"></textarea>
 
 									<div class="give-clearfix">
-										<button id="give-add-payment-note" class="button button-secondary button-small" data-payment-id="<?php echo absint( $payment_id ); ?>"><?php _e( 'Add Note', 'give' ); ?></button>
+										<button id="give-add-payment-note" class="button button-secondary button-small" data-payment-id="<?php echo absint($payment_id); ?>"><?php _e('Add Note', 'give'); ?></button>
 									</div>
 
 								</div>
@@ -846,7 +846,7 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_main_after', $payment_id );
+							do_action('give_view_donation_details_main_after', $payment_id);
 							?>
 
 						</div>
@@ -868,11 +868,11 @@ 
 		 *
 		 * @param int $payment_id Payment id.
 		 */
-		do_action( 'give_view_donation_details_form_bottom', $payment_id );
+		do_action('give_view_donation_details_form_bottom', $payment_id);
 
-		wp_nonce_field( 'give_update_payment_details_nonce' );
+		wp_nonce_field('give_update_payment_details_nonce');
 		?>
-		<input type="hidden" name="give_payment_id" value="<?php echo esc_attr( $payment_id ); ?>"/>
+		<input type="hidden" name="give_payment_id" value="<?php echo esc_attr($payment_id); ?>"/>
 		<input type="hidden" name="give_action" value="update_payment_details"/>
 	</form>
 	<?php
@@ -883,6 +883,6 @@ 
 	 *
 	 * @param int $payment_id Payment id.
 	 */
-	do_action( 'give_view_donation_details_after', $payment_id );
+	do_action('give_view_donation_details_after', $payment_id);
 	?>
 </div><!-- /.wrap -->

includes/admin/shortcodes/shortcode-give-totals.php

Spacing +37 added lines, -37 removed lines

@@ -9,7 +9,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,10 +23,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = __( 'Give Totals', 'give' );
-		$this->shortcode['label'] = __( 'Give Totals', 'give' );
+		$this->shortcode['title'] = __('Give Totals', 'give');
+		$this->shortcode['label'] = __('Give Totals', 'give');
 
-		parent::__construct( 'give_totals' );
+		parent::__construct('give_totals');
 	}
 
 	/**
@@ -39,69 +39,69 @@ 
 
 		$category_options = array();
 		$category_lists   = array();
-		$categories       = get_terms( 'give_forms_category', apply_filters( 'give_forms_category_dropdown', array() ) );
-		if ( give_is_setting_enabled( give_get_option( 'categories' ) ) && ! is_wp_error( $categories ) ) {
-			foreach ( $categories as $category ) {
-				$category_options[ absint( $category->term_id ) ] = esc_html( $category->name );
+		$categories       = get_terms('give_forms_category', apply_filters('give_forms_category_dropdown', array()));
+		if (give_is_setting_enabled(give_get_option('categories')) && ! is_wp_error($categories)) {
+			foreach ($categories as $category) {
+				$category_options[absint($category->term_id)] = esc_html($category->name);
 			}
 
 			$category_lists['type']    = 'listbox';
 			$category_lists['name']    = 'cats';
-			$category_lists['label']   = __( 'Select a Donation Form Category:', 'give' );
-			$category_lists['tooltip'] = __( 'Select a Donation Form Category', 'give' );
+			$category_lists['label']   = __('Select a Donation Form Category:', 'give');
+			$category_lists['tooltip'] = __('Select a Donation Form Category', 'give');
 			$category_lists['options'] = $category_options;
 		}
 
 		$tag_options = array();
 		$tag_lists   = array();
-		$tags        = get_terms( 'give_forms_tag', apply_filters( 'give_forms_tag_dropdown', array() ) );
-		if ( give_is_setting_enabled( give_get_option( 'tags' ) ) && ! is_wp_error( $tags ) ) {
-			$tags = get_terms( 'give_forms_tag', apply_filters( 'give_forms_tag_dropdown', array() ) );
-			foreach ( $tags as $tag ) {
-				$tag_options[ absint( $tag->term_id ) ] = esc_html( $tag->name );
+		$tags        = get_terms('give_forms_tag', apply_filters('give_forms_tag_dropdown', array()));
+		if (give_is_setting_enabled(give_get_option('tags')) && ! is_wp_error($tags)) {
+			$tags = get_terms('give_forms_tag', apply_filters('give_forms_tag_dropdown', array()));
+			foreach ($tags as $tag) {
+				$tag_options[absint($tag->term_id)] = esc_html($tag->name);
 			}
 
 			$tag_lists['type']    = 'listbox';
 			$tag_lists['name']    = 'tags';
-			$tag_lists['label']   = __( 'Select a Donation Form Tag:', 'give' );
-			$tag_lists['tooltip'] = __( 'Select a Donation Form Tag', 'give' );
+			$tag_lists['label']   = __('Select a Donation Form Tag:', 'give');
+			$tag_lists['tooltip'] = __('Select a Donation Form Tag', 'give');
 			$tag_lists['options'] = $tag_options;
 		}
 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="give-totals-shortcode-container-message">%s</p>',
-					__( 'This shortcode shows the total amount raised towards a custom goal for one or several forms regardless of whether they have goals enabled or not.', 'give' )
+				'html' => sprintf('<p class="give-totals-shortcode-container-message">%s</p>',
+					__('This shortcode shows the total amount raised towards a custom goal for one or several forms regardless of whether they have goals enabled or not.', 'give')
 				),
 			),
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong margin-top">%s</p>', __( 'Shortcode Configuration', 'give' ) ),
+				'html' => sprintf('<p class="strong margin-top">%s</p>', __('Shortcode Configuration', 'give')),
 			),
 			array(
 				'type'    => 'textbox',
 				'name'    => 'ids',
-				'label'   => __( 'Donation Form IDs:', 'give' ),
-				'tooltip' => __( 'Enter the IDs separated by commas for the donation forms you would like to combine within the totals.', 'give' ),
+				'label'   => __('Donation Form IDs:', 'give'),
+				'tooltip' => __('Enter the IDs separated by commas for the donation forms you would like to combine within the totals.', 'give'),
 			),
 			$category_lists,
 			$tag_lists,
 			array(
 				'type'     => 'textbox',
 				'name'     => 'total_goal',
-				'label'    => __( 'Total Goal:', 'give' ),
-				'tooltip'  => __( 'Enter the total goal amount that you would like to display.', 'give' ),
+				'label'    => __('Total Goal:', 'give'),
+				'tooltip'  => __('Enter the total goal amount that you would like to display.', 'give'),
 				'required' => array(
-					'alert' => esc_html__( 'Please enter a valid total goal amount.', 'give' ),
+					'alert' => esc_html__('Please enter a valid total goal amount.', 'give'),
 				),
 			),
 			array(
 				'type'      => 'textbox',
 				'name'      => 'message',
-				'label'     => __( 'Message:', 'give' ),
-				'tooltip'   => __( 'Enter a message to display encouraging donors to support the goal.', 'give' ),
-				'value'     => apply_filters( 'give_totals_message', __( 'Hey! We\'ve raised {total} of the {total_goal} we are trying to raise for this campaign!', 'give' ) ),
+				'label'     => __('Message:', 'give'),
+				'tooltip'   => __('Enter a message to display encouraging donors to support the goal.', 'give'),
+				'value'     => apply_filters('give_totals_message', __('Hey! We\'ve raised {total} of the {total_goal} we are trying to raise for this campaign!', 'give')),
 				'multiline' => true,
 				'minWidth'  => 300,
 				'minHeight' => 60,
@@ -109,24 +109,24 @@ 
 			array(
 				'type'    => 'textbox',
 				'name'    => 'link',
-				'label'   => __( 'Link:', 'give' ),
-				'tooltip' => __( 'Enter a link to the main campaign donation form.', 'give' ),
+				'label'   => __('Link:', 'give'),
+				'tooltip' => __('Enter a link to the main campaign donation form.', 'give'),
 			),
 			array(
 				'type'    => 'textbox',
 				'name'    => 'link_text',
-				'label'   => __( 'Link Text:', 'give' ),
-				'tooltip' => __( 'Enter hyperlink text for the link to the main campaign donation form.', 'give' ),
-				'value'   => __( 'Donate!', 'give' ),
+				'label'   => __('Link Text:', 'give'),
+				'tooltip' => __('Enter hyperlink text for the link to the main campaign donation form.', 'give'),
+				'value'   => __('Donate!', 'give'),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'progress_bar',
-				'label'   => __( 'Show Progress Bar:', 'give' ),
-				'tooltip' => __( 'Select whether you would like to show a goal progress bar.', 'give' ),
+				'label'   => __('Show Progress Bar:', 'give'),
+				'tooltip' => __('Select whether you would like to show a goal progress bar.', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 				'value'   => 'true',
 			),

includes/admin/shortcodes/class-shortcode-button.php

Spacing +52 added lines, -52 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,7 +32,7 @@ 
 	 * Class constructor
 	 */
 	public function __construct() {
-		add_action( 'admin_init', array( $this, 'init'), 999 );
+		add_action('admin_init', array($this, 'init'), 999);
 	}
 
 	/**
@@ -41,17 +41,17 @@ 
 	 * @since 2.1.0
 	 * @access public
 	 */
-	public function init(){
-		if ( $this->is_add_button() ) {
-			add_filter( 'mce_external_plugins', array( $this, 'mce_external_plugins' ), 15 );
+	public function init() {
+		if ($this->is_add_button()) {
+			add_filter('mce_external_plugins', array($this, 'mce_external_plugins'), 15);
 
-			add_action( 'admin_enqueue_scripts', array( $this, 'admin_enqueue_assets' ) );
-			add_action( 'admin_enqueue_scripts', array( $this, 'admin_localize_scripts' ), 13 );
-			add_action( 'media_buttons', array( $this, 'shortcode_button' ) );
+			add_action('admin_enqueue_scripts', array($this, 'admin_enqueue_assets'));
+			add_action('admin_enqueue_scripts', array($this, 'admin_localize_scripts'), 13);
+			add_action('media_buttons', array($this, 'shortcode_button'));
 		}
 
-		add_action( "wp_ajax_give_shortcode", array( $this, 'shortcode_ajax' ) );
-		add_action( "wp_ajax_nopriv_give_shortcode", array( $this, 'shortcode_ajax' ) );
+		add_action("wp_ajax_give_shortcode", array($this, 'shortcode_ajax'));
+		add_action("wp_ajax_nopriv_give_shortcode", array($this, 'shortcode_ajax'));
 	}
 
 	/**
@@ -63,13 +63,13 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function mce_external_plugins( $plugin_array ) {
+	public function mce_external_plugins($plugin_array) {
 
-		if ( ! current_user_can( 'edit_posts' ) && ! current_user_can( 'edit_pages' ) ) {
+		if ( ! current_user_can('edit_posts') && ! current_user_can('edit_pages')) {
 			return false;
 		}
 
-		$plugin_array['give_shortcode'] = GIVE_PLUGIN_URL . 'includes/admin/shortcodes/mce-plugin.js';
+		$plugin_array['give_shortcode'] = GIVE_PLUGIN_URL.'includes/admin/shortcodes/mce-plugin.js';
 
 		return $plugin_array;
 	}
@@ -82,19 +82,19 @@ 
 	 * @since 1.0
 	 */
 	public function admin_enqueue_assets() {
-		$direction = ( is_rtl() || isset( $_GET['d'] ) && 'rtl' === $_GET['d'] ) ? '.rtl' : '';
+		$direction = (is_rtl() || isset($_GET['d']) && 'rtl' === $_GET['d']) ? '.rtl' : '';
 
 		wp_enqueue_script(
 			'give_shortcode',
-			GIVE_PLUGIN_URL . 'assets/dist/js/admin-shortcodes.js',
-			array( 'jquery' ),
+			GIVE_PLUGIN_URL.'assets/dist/js/admin-shortcodes.js',
+			array('jquery'),
 			GIVE_VERSION,
 			true
 		);
 
 		wp_enqueue_style(
 			'give-admin-shortcode-button-style',
-			GIVE_PLUGIN_URL . 'assets/dist/css/admin-shortcode-button' . $direction . '.css',
+			GIVE_PLUGIN_URL.'assets/dist/css/admin-shortcode-button'.$direction.'.css',
 			array(),
 			GIVE_VERSION
 		);
@@ -109,17 +109,17 @@ 
 	 */
 	public function admin_localize_scripts() {
 
-		if ( ! empty( self::$shortcodes ) ) {
+		if ( ! empty(self::$shortcodes)) {
 
 			$variables = array();
 
-			foreach ( self::$shortcodes as $shortcode => $values ) {
-				if ( ! empty( $values['required'] ) ) {
-					$variables[ $shortcode ] = $values['required'];
+			foreach (self::$shortcodes as $shortcode => $values) {
+				if ( ! empty($values['required'])) {
+					$variables[$shortcode] = $values['required'];
 				}
 			}
 
-			wp_localize_script( 'give_shortcode', 'scShortcodes', $variables );
+			wp_localize_script('give_shortcode', 'scShortcodes', $variables);
 		}
 	}
 
@@ -134,16 +134,16 @@ 
 
 		$shortcodes = array();
 
-		foreach ( self::$shortcodes as $shortcode => $values ) {
+		foreach (self::$shortcodes as $shortcode => $values) {
 
 			/**
 			 * Filters the condition for including the current shortcode
 			 *
 			 * @since 1.0
 			 */
-			if ( apply_filters( sanitize_title( $shortcode ) . '_condition', true ) ) {
+			if (apply_filters(sanitize_title($shortcode).'_condition', true)) {
 
-				$shortcodes[ $shortcode ] = sprintf(
+				$shortcodes[$shortcode] = sprintf(
 					'<div class="sc-shortcode mce-menu-item give-shortcode-item-%1$s" data-shortcode="%2$s">%3$s</div>',
 					$shortcode,
 					$shortcode,
@@ -152,37 +152,37 @@ 
 			}
 		}
 
-		if ( ! empty( $shortcodes ) ) {
+		if ( ! empty($shortcodes)) {
 
 			// check current WP version
-			$img = ( version_compare( get_bloginfo( 'version' ), '3.5', '<' ) )
-				? '<img src="' . GIVE_PLUGIN_URL . 'assets/dist/images/give-media.png" />'
-				: '<span class="wp-media-buttons-icon" id="give-media-button" style="background-image: url(' . give_svg_icons( 'give_grey' ) . ');"></span>';
+			$img = (version_compare(get_bloginfo('version'), '3.5', '<'))
+				? '<img src="'.GIVE_PLUGIN_URL.'assets/dist/images/give-media.png" />'
+				: '<span class="wp-media-buttons-icon" id="give-media-button" style="background-image: url('.give_svg_icons('give_grey').');"></span>';
 
-			reset( $shortcodes );
+			reset($shortcodes);
 
-			if ( 1 === count( $shortcodes ) ) {
+			if (1 === count($shortcodes)) {
 
-				$shortcode = key( $shortcodes );
+				$shortcode = key($shortcodes);
 
 				printf(
 					'<button type="button" class="button sc-shortcode" data-shortcode="%s">%s</button>',
 					$shortcode,
-					sprintf( '%s %s %s',
+					sprintf('%s %s %s',
 						$img,
-						__( 'Insert', 'give' ),
-						self::$shortcodes[ $shortcode ]['label']
+						__('Insert', 'give'),
+						self::$shortcodes[$shortcode]['label']
 					)
 				);
 			} else {
 				printf(
-					'<div class="sc-wrap">' .
-					'<button class="button sc-button" type="button">%s %s</button>' .
-					'<div class="sc-menu mce-menu">%s</div>' .
+					'<div class="sc-wrap">'.
+					'<button class="button sc-button" type="button">%s %s</button>'.
+					'<div class="sc-menu mce-menu">%s</div>'.
 					'</div>',
 					$img,
-					__( 'Give Shortcodes', 'give' ),
-					implode( '', array_values( $shortcodes ) )
+					__('Give Shortcodes', 'give'),
+					implode('', array_values($shortcodes))
 				);
 			}
 		}
@@ -197,15 +197,15 @@ 
 	 */
 	public function shortcode_ajax() {
 
-		$shortcode = isset( $_POST['shortcode'] ) ? $_POST['shortcode'] : false;
+		$shortcode = isset($_POST['shortcode']) ? $_POST['shortcode'] : false;
 		$response  = false;
 
-		if ( $shortcode && array_key_exists( $shortcode, self::$shortcodes ) ) {
+		if ($shortcode && array_key_exists($shortcode, self::$shortcodes)) {
 
-			$data = self::$shortcodes[ $shortcode ];
+			$data = self::$shortcodes[$shortcode];
 
-			if ( ! empty( $data['errors'] ) ) {
-				$data['btn_okay'] = array( esc_html__( 'Okay', 'give' ) );
+			if ( ! empty($data['errors'])) {
+				$data['btn_okay'] = array(esc_html__('Okay', 'give'));
 			}
 
 			$response = array(
@@ -217,10 +217,10 @@ 
 			);
 		} else {
 			// todo: handle error
-			error_log( print_r( 'AJAX error!', 1 ) );
+			error_log(print_r('AJAX error!', 1));
 		}
 
-		wp_send_json( $response );
+		wp_send_json($response);
 	}
 
 
@@ -234,21 +234,21 @@ 
 	private function is_add_button() {
 		global $pagenow;
 
-		$shortcode_button_pages = apply_filters( 'give_shortcode_button_pages', array(
+		$shortcode_button_pages = apply_filters('give_shortcode_button_pages', array(
 			'post.php',
 			'page.php',
 			'post-new.php',
 			'post-edit.php',
 			'edit.php',
 			'edit.php?post_type=page',
-		) );
+		));
 
 		// Only run in admin post/page creation and edit screens
 		if (
 			! is_admin()
-			|| ! in_array( $pagenow, $shortcode_button_pages )
-			|| ! apply_filters( 'give_shortcode_button_condition', true )
-			|| empty( self::$shortcodes )
+			|| ! in_array($pagenow, $shortcode_button_pages)
+			|| ! apply_filters('give_shortcode_button_condition', true)
+			|| empty(self::$shortcodes)
 		) {
 			return false;
 		}

includes/admin/forms/dashboard-columns.php

Spacing +76 added lines, -76 removed lines

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

includes/admin/abstract-admin-settings-page.php

Spacing +34 added lines, -34 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Page' ) ) :
+if ( ! class_exists('Give_Settings_Page')) :
 
 	/**
 	 * Give_Settings_Page.
@@ -70,25 +70,25 @@ 
 			// Get current setting page.
 			$this->current_setting_page = give_get_current_setting_page();
 
-			add_filter( "give_default_setting_tab_section_{$this->id}", array( $this, 'set_default_setting_tab' ), 10 );
-			add_filter( "{$this->current_setting_page}_tabs_array", array( $this, 'add_settings_page' ), 20 );
-			add_action( "{$this->current_setting_page}_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter("give_default_setting_tab_section_{$this->id}", array($this, 'set_default_setting_tab'), 10);
+			add_filter("{$this->current_setting_page}_tabs_array", array($this, 'add_settings_page'), 20);
+			add_action("{$this->current_setting_page}_settings_{$this->id}_page", array($this, 'output'));
 
 			// Output section only if exist.
 			$sections = $this->get_sections();
-			if ( ! empty( $sections ) ) {
-				add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array(
+			if ( ! empty($sections)) {
+				add_action("{$this->current_setting_page}_sections_{$this->id}_page", array(
 					$this,
 					'output_sections',
-				) );
+				));
 			}
 
 			// Save hide button by default.
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Enable saving feature.
-			if ( $this->enable_save ) {
-				add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
+			if ($this->enable_save) {
+				add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
 			}
 		}
 
@@ -113,7 +113,7 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			return $this->default_tab;
 		}
 
@@ -126,8 +126,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -146,7 +146,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, array() );
+			$settings = apply_filters('give_get_settings_'.$this->id, array());
 
 			// Output.
 			return $settings;
@@ -159,7 +159,7 @@ 
 		 * @return array
 		 */
 		public function get_sections() {
-			return apply_filters( 'give_get_sections_' . $this->id, array() );
+			return apply_filters('give_get_sections_'.$this->id, array());
 		}
 
 		/**
@@ -176,40 +176,40 @@ 
 			$sections = $this->get_sections();
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
 			// Show section settings only if setting section exist.
-			if ( $current_section && ! in_array( $current_section, array_keys( $sections ), true ) ) {
-				echo wp_kses_post( '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>' );
+			if ($current_section && ! in_array($current_section, array_keys($sections), true)) {
+				echo wp_kses_post('<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>');
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
-			if ( is_null( $this->current_setting_page ) ) {
+			if (is_null($this->current_setting_page)) {
 				$this->current_setting_page = give_get_current_setting_page();
 			}
 
 			$section_list = array();
-			foreach ( $sections as $id => $label ) {
+			foreach ($sections as $id => $label) {
 				/**
 				 * Fire the filter to hide particular section on tab.
 				 *
 				 * @since 2.0
 				 */
-				if ( apply_filters( "give_hide_section_{$id}_on_{$this->id}_page", false, $sections, $this->id ) ) {
+				if (apply_filters("give_hide_section_{$id}_on_{$this->id}_page", false, $sections, $this->id)) {
 					continue;
 				}
 
-				$section_list[] = '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=' . $this->current_setting_page . '&tab=' . $this->id . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $current_section === $id ? 'current' : '' ) . '">' . $label . '</a>';
+				$section_list[] = '<li><a href="'.admin_url('edit.php?post_type=give_forms&page='.$this->current_setting_page.'&tab='.$this->id.'&section='.sanitize_title($id)).'" class="'.($current_section === $id ? 'current' : '').'">'.$label.'</a>';
 			}
 
-			echo wp_kses_post( sprintf(
+			echo wp_kses_post(sprintf(
 				'<ul class="give-subsubsub">%s</ul><br class="clear" /><hr>',
-				implode( ' | </li>', $section_list )
-			) );
+				implode(' | </li>', $section_list)
+			));
 		}
 
 		/**
@@ -221,13 +221,13 @@ 
 		 * @return void
 		 */
 		public function output() {
-			if ( $this->enable_save ) {
+			if ($this->enable_save) {
 				$GLOBALS['give_hide_save_button'] = false;
 			}
 
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -245,8 +245,8 @@ 
 			 *
 			 * @since 2.1
 			 */
-			if ( apply_filters( "give_save_options_{$this->id}_{$current_section}", true ) ) {
-				Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			if (apply_filters("give_save_options_{$this->id}_{$current_section}", true)) {
+				Give_Admin_Settings::save_fields($settings, 'give_settings');
 			}
 
 			/**
@@ -254,7 +254,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( 'give_update_options_' . $this->id . '_' . $current_section );
+			do_action('give_update_options_'.$this->id.'_'.$current_section);
 		}
 
 		/**
@@ -271,11 +271,11 @@ 
 			$section         = $this->get_sections();
 			$current_section = give_get_current_setting_section();
 
-			if ( array_key_exists( $current_section, $section ) ) {
-				$heading[] = $section[ $current_section ];
+			if (array_key_exists($current_section, $section)) {
+				$heading[] = $section[$current_section];
 			}
 
-			return array_unique( $heading );
+			return array_unique($heading);
 		}
 
 		/**
@@ -289,7 +289,7 @@ 
 		public function get_heading_html() {
 			return sprintf(
 				'<h1 class="wp-heading-inline">%s</h1><hr class="wp-header-end">',
-				implode( ' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading() )
+				implode(' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading())
 			);
 		}
 	}

includes/admin/settings/class-settings-license.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
 
-if ( ! class_exists( 'Give_Settings_License' ) ) :
+if ( ! class_exists('Give_Settings_License')) :
 
 	/**
 	 * Give_Settings_License.
@@ -27,12 +27,12 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'licenses';
-			$this->label = esc_html__( 'Licenses', 'give' );
+			$this->label = esc_html__('Licenses', 'give');
 
 			parent::__construct();
 
 			// Filter to remove the license tab.
-			add_filter( 'give-settings_tabs_array', array( $this, 'remove_license_tab' ), 9999999, 1 );
+			add_filter('give-settings_tabs_array', array($this, 'remove_license_tab'), 9999999, 1);
 
 		}
 
@@ -49,7 +49,7 @@ 
 			 * Filter the licenses settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_licenses', $settings );
+			$settings = apply_filters('give_settings_licenses', $settings);
 
 			/**
 			 * Filter the settings.
@@ -58,7 +58,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -74,13 +74,13 @@ 
 		 *
 		 * @return array
 		 */
-		public function remove_license_tab( $tabs ) {
+		public function remove_license_tab($tabs) {
 			/**
 			 * Remove the license tab if no Give licensed addon
 			 * is activated.
 			 */
-			if ( ! $this->is_show_setting_page() ) {
-				unset( $tabs['licenses'] );
+			if ( ! $this->is_show_setting_page()) {
+				unset($tabs['licenses']);
 			}
 
 			return $tabs;
@@ -97,9 +97,9 @@ 
 		 */
 		private function is_show_setting_page() {
 			$licensed_addons   = Give_License::get_licensed_addons();
-			$activated_plugins = get_option( 'active_plugins', array() );
+			$activated_plugins = get_option('active_plugins', array());
 
-			return (bool) count( array_intersect( $activated_plugins, $licensed_addons ) );
+			return (bool) count(array_intersect($activated_plugins, $licensed_addons));
 		}
 	}