WordImpress / Give

includes/admin/payments/class-payments-table.php

Spacing +191 added lines, -191 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -118,15 +118,15 @@ 
 	public function __construct() {
 
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records.
-			'plural'   => give_get_forms_label_plural(),      // Plural name of the listed records.
-			'ajax'     => false,                              // Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records.
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records.
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 		$this->get_payment_counts();
 		$this->process_bulk_action();
-		$this->base_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+		$this->base_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 	}
 
 	/**
@@ -135,29 +135,29 @@ 
 	 * @return void
 	 */
 	public function advanced_filters() {
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : '';
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : '';
 		?>
         <div id="give-payment-filters">
 			<span id="give-payment-date-filters">
 				<label for="start-date"
-                       class="give-start-date-label"><?php esc_html_e( 'Start Date:', 'give' ); ?></label>
+                       class="give-start-date-label"><?php esc_html_e('Start Date:', 'give'); ?></label>
 				<input type="text" id="start-date" name="start-date" class="give_datepicker"
                        value="<?php echo $start_date; ?>" placeholder="mm/dd/yyyy"/>
-				<label for="end-date" class="give-end-date-label"><?php esc_html_e( 'End Date:', 'give' ); ?></label>
+				<label for="end-date" class="give-end-date-label"><?php esc_html_e('End Date:', 'give'); ?></label>
 				<input type="text" id="end-date" name="end-date" class="give_datepicker"
                        value="<?php echo $end_date; ?>" placeholder="mm/dd/yyyy"/>
-				<input type="submit" class="button-secondary" value="<?php esc_attr_e( 'Apply', 'give' ); ?>"/>
+				<input type="submit" class="button-secondary" value="<?php esc_attr_e('Apply', 'give'); ?>"/>
 			</span>
-			<?php if ( ! empty( $status ) ) : ?>
-                <input type="hidden" name="status" value="<?php echo esc_attr( $status ); ?>"/>
+			<?php if ( ! empty($status)) : ?>
+                <input type="hidden" name="status" value="<?php echo esc_attr($status); ?>"/>
 			<?php endif; ?>
-			<?php if ( ! empty( $start_date ) || ! empty( $end_date ) ) : ?>
-                <a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>"
-                   class="button-secondary"><?php esc_html_e( 'Clear Filter', 'give' ); ?></a>
+			<?php if ( ! empty($start_date) || ! empty($end_date)) : ?>
+                <a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>"
+                   class="button-secondary"><?php esc_html_e('Clear Filter', 'give'); ?></a>
 			<?php endif; ?>
-			<?php $this->search_box( esc_html__( 'Search', 'give' ), 'give-payments' ); ?>
+			<?php $this->search_box(esc_html__('Search', 'give'), 'give-payments'); ?>
         </div>
 
 		<?php
@@ -174,18 +174,18 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		if ( empty( $_REQUEST['s'] ) && ! $this->has_items() ) {
+	public function search_box($text, $input_id) {
+		if (empty($_REQUEST['s']) && ! $this->has_items()) {
 			return;
 		}
 
-		$input_id = $input_id . '-search-input';
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
         <p class="search-box" role="search">
@@ -197,11 +197,11 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payment_history_search' );
+			do_action('give_payment_history_search');
 			?>
             <label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
             <input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?><br/>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?><br/>
         </p>
 		<?php
 	}
@@ -215,52 +215,52 @@ 
 	 */
 	public function get_views() {
 
-		$current         = isset( $_GET['status'] ) ? $_GET['status'] : '';
-		$total_count     = '&nbsp;<span class="count">(' . $this->total_count . ')</span>';
-		$complete_count  = '&nbsp;<span class="count">(' . $this->complete_count . ')</span>';
-		$cancelled_count = '&nbsp;<span class="count">(' . $this->cancelled_count . ')</span>';
-		$pending_count   = '&nbsp;<span class="count">(' . $this->pending_count . ')</span>';
-		$refunded_count  = '&nbsp;<span class="count">(' . $this->refunded_count . ')</span>';
-		$failed_count    = '&nbsp;<span class="count">(' . $this->failed_count . ')</span>';
-		$abandoned_count = '&nbsp;<span class="count">(' . $this->abandoned_count . ')</span>';
-		$revoked_count   = '&nbsp;<span class="count">(' . $this->revoked_count . ')</span>';
+		$current         = isset($_GET['status']) ? $_GET['status'] : '';
+		$total_count     = '&nbsp;<span class="count">('.$this->total_count.')</span>';
+		$complete_count  = '&nbsp;<span class="count">('.$this->complete_count.')</span>';
+		$cancelled_count = '&nbsp;<span class="count">('.$this->cancelled_count.')</span>';
+		$pending_count   = '&nbsp;<span class="count">('.$this->pending_count.')</span>';
+		$refunded_count  = '&nbsp;<span class="count">('.$this->refunded_count.')</span>';
+		$failed_count    = '&nbsp;<span class="count">('.$this->failed_count.')</span>';
+		$abandoned_count = '&nbsp;<span class="count">('.$this->abandoned_count.')</span>';
+		$revoked_count   = '&nbsp;<span class="count">('.$this->revoked_count.')</span>';
 
 		$views = array(
-			'all'       => sprintf( '<a href="%s"%s>%s</a>', remove_query_arg( array(
+			'all'       => sprintf('<a href="%s"%s>%s</a>', remove_query_arg(array(
 				'status',
 				'paged',
-			) ), $current === 'all' || $current == '' ? ' class="current"' : '', esc_html__( 'All', 'give' ) . $total_count ),
-			'publish'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			)), $current === 'all' || $current == '' ? ' class="current"' : '', esc_html__('All', 'give').$total_count),
+			'publish'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'publish',
 				'paged'  => false,
-			) ) ), $current === 'publish' ? ' class="current"' : '', esc_html__( 'Completed', 'give' ) . $complete_count ),
-			'pending'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'publish' ? ' class="current"' : '', esc_html__('Completed', 'give').$complete_count),
+			'pending'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'pending',
 				'paged'  => false,
-			) ) ), $current === 'pending' ? ' class="current"' : '', esc_html__( 'Pending', 'give' ) . $pending_count ),
-			'refunded'  => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'pending' ? ' class="current"' : '', esc_html__('Pending', 'give').$pending_count),
+			'refunded'  => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'refunded',
 				'paged'  => false,
-			) ) ), $current === 'refunded' ? ' class="current"' : '', esc_html__( 'Refunded', 'give' ) . $refunded_count ),
-			'revoked'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'refunded' ? ' class="current"' : '', esc_html__('Refunded', 'give').$refunded_count),
+			'revoked'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'revoked',
 				'paged'  => false,
-			) ) ), $current === 'revoked' ? ' class="current"' : '', esc_html__( 'Revoked', 'give' ) . $revoked_count ),
-			'failed'    => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'revoked' ? ' class="current"' : '', esc_html__('Revoked', 'give').$revoked_count),
+			'failed'    => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'failed',
 				'paged'  => false,
-			) ) ), $current === 'failed' ? ' class="current"' : '', esc_html__( 'Failed', 'give' ) . $failed_count ),
-			'cancelled' => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'failed' ? ' class="current"' : '', esc_html__('Failed', 'give').$failed_count),
+			'cancelled' => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'cancelled',
 				'paged'  => false,
-			) ) ), $current === 'cancelled' ? ' class="current"' : '', esc_html__( 'Cancelled', 'give' ) . $cancelled_count ),
-			'abandoned' => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'cancelled' ? ' class="current"' : '', esc_html__('Cancelled', 'give').$cancelled_count),
+			'abandoned' => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'abandoned',
 				'paged'  => false,
-			) ) ), $current === 'abandoned' ? ' class="current"' : '', esc_html__( 'Abandoned', 'give' ) . $abandoned_count ),
+			))), $current === 'abandoned' ? ' class="current"' : '', esc_html__('Abandoned', 'give').$abandoned_count),
 		);
 
-		return apply_filters( 'give_payments_table_views', $views );
+		return apply_filters('give_payments_table_views', $views);
 	}
 
 	/**
@@ -273,15 +273,15 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'donation'      => esc_html__( 'Donation', 'give' ),
-			'donation_form' => esc_html__( 'Donation Form', 'give' ),
-			'status'        => esc_html__( 'Status', 'give' ),
-			'date'          => esc_html__( 'Date', 'give' ),
-			'amount'        => esc_html__( 'Amount', 'give' ),
-			'details'       => esc_html__( 'Details', 'give' ),
+			'donation'      => esc_html__('Donation', 'give'),
+			'donation_form' => esc_html__('Donation Form', 'give'),
+			'status'        => esc_html__('Status', 'give'),
+			'date'          => esc_html__('Date', 'give'),
+			'amount'        => esc_html__('Amount', 'give'),
+			'details'       => esc_html__('Details', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_columns', $columns );
+		return apply_filters('give_payments_table_columns', $columns);
 	}
 
 	/**
@@ -293,14 +293,14 @@ 
 	 */
 	public function get_sortable_columns() {
 		$columns = array(
-			'donation'      => array( 'ID', true ),
-			'donation_form' => array( 'donation_form', false ),
-			'status'        => array( 'status', false ),
-			'amount'        => array( 'amount', false ),
-			'date'          => array( 'date', false ),
+			'donation'      => array('ID', true),
+			'donation_form' => array('donation_form', false),
+			'status'        => array('status', false),
+			'amount'        => array('amount', false),
+			'date'          => array('date', false),
 		);
 
-		return apply_filters( 'give_payments_table_sortable_columns', $columns );
+		return apply_filters('give_payments_table_sortable_columns', $columns);
 	}
 
 	/**
@@ -326,70 +326,70 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $payment, $column_name ) {
+	public function column_default($payment, $column_name) {
 
-		$single_donation_url = esc_url( add_query_arg( 'id', $payment->ID, admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details' ) ) );
-		$row_actions         = $this->get_row_actions( $payment );
+		$single_donation_url = esc_url(add_query_arg('id', $payment->ID, admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details')));
+		$row_actions         = $this->get_row_actions($payment);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'donation' :
 				$value = sprintf(
 					'<a href="%1$s" data-tooltip="%2$s">#%3$s</a>&nbsp;%4$s&nbsp;%5$s<br>',
 					$single_donation_url,
-					sprintf( esc_attr__( 'View Donation #%s', 'give' ), $payment->ID ),
+					sprintf(esc_attr__('View Donation #%s', 'give'), $payment->ID),
 					$payment->ID,
-					esc_html__( 'by', 'give' ),
-					$this->get_donor( $payment )
+					esc_html__('by', 'give'),
+					$this->get_donor($payment)
 				);
-				$value .= $this->get_donor_email( $payment );
-				$value .= $this->row_actions( $row_actions );
+				$value .= $this->get_donor_email($payment);
+				$value .= $this->row_actions($row_actions);
 				break;
 
 			case 'amount' :
-				$amount = ! empty( $payment->total ) ? $payment->total : 0;
-				$value  = give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment->ID ) );
-				$value  .= sprintf(
+				$amount = ! empty($payment->total) ? $payment->total : 0;
+				$value  = give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment->ID));
+				$value .= sprintf(
 					'<br><small>%1$s %2$s</small>',
-					__( 'via', 'give' ),
-					give_get_gateway_admin_label( $payment->gateway )
+					__('via', 'give'),
+					give_get_gateway_admin_label($payment->gateway)
 				);
 				break;
 
 			case 'donation_form' :
-				$form_title = empty( $payment->form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $payment->form_id ) : $payment->form_title;
-				$value      = '<a href="' . admin_url( 'post.php?post=' . $payment->form_id . '&action=edit' ) . '">' . $form_title . '</a>';
-				$level      = give_get_payment_form_title( $payment->meta, true );
+				$form_title = empty($payment->form_title) ? sprintf(__('Untitled (#%s)', 'give'), $payment->form_id) : $payment->form_title;
+				$value      = '<a href="'.admin_url('post.php?post='.$payment->form_id.'&action=edit').'">'.$form_title.'</a>';
+				$level      = give_get_payment_form_title($payment->meta, true);
 
-				if ( ! empty( $level ) ) {
+				if ( ! empty($level)) {
 					$value .= $level;
 				}
 
 				break;
 
 			case 'date' :
-				$date  = strtotime( $payment->date );
-				$value = date_i18n( give_date_format(), $date );
+				$date  = strtotime($payment->date);
+				$value = date_i18n(give_date_format(), $date);
 				break;
 
 			case 'status' :
-				$value = $this->get_payment_status( $payment );
+				$value = $this->get_payment_status($payment);
 				break;
 
 			case 'details' :
 				$value = sprintf(
 					'<div class="give-payment-details-link-wrap"><a href="%1$s" class="give-payment-details-link button button-small" data-tooltip="%2$s" aria-label="%2$s"><span class="dashicons dashicons-visibility"></span></a></div>',
 					$single_donation_url,
-					sprintf( esc_attr__( 'View Donation #%s', 'give' ), $payment->ID )
+					sprintf(esc_attr__('View Donation #%s', 'give'), $payment->ID)
 				);
 				break;
 
 			default:
-				$value = isset( $payment->$column_name ) ? $payment->$column_name : '';
+				$value = isset($payment->$column_name) ? $payment->$column_name : '';
 				break;
 
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, $column_name );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, $column_name);
 	}
 
 	/**
@@ -402,17 +402,17 @@ 
 	 *
 	 * @return string                Data shown in the Email column
 	 */
-	public function get_donor_email( $payment ) {
+	public function get_donor_email($payment) {
 
-		$email = give_get_payment_user_email( $payment->ID );
+		$email = give_get_payment_user_email($payment->ID);
 
-		if ( empty( $email ) ) {
-			$email = esc_html__( '(unknown)', 'give' );
+		if (empty($email)) {
+			$email = esc_html__('(unknown)', 'give');
 		}
 
-		$value = '<a href="mailto:' . $email . '" data-tooltip="' . esc_attr__( 'Email donor', 'give' ) . '">' . $email . '</a>';
+		$value = '<a href="mailto:'.$email.'" data-tooltip="'.esc_attr__('Email donor', 'give').'">'.$email.'</a>';
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'email' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'email');
 	}
 
 	/**
@@ -424,18 +424,18 @@ 
 	 *
 	 * @return array $actions
 	 */
-	function get_row_actions( $payment ) {
+	function get_row_actions($payment) {
 
 		$actions = array();
-		$email   = give_get_payment_user_email( $payment->ID );
+		$email   = give_get_payment_user_email($payment->ID);
 
 		// Add search term string back to base URL.
-		$search_terms = ( isset( $_GET['s'] ) ? trim( $_GET['s'] ) : '' );
-		if ( ! empty( $search_terms ) ) {
-			$this->base_url = add_query_arg( 's', $search_terms, $this->base_url );
+		$search_terms = (isset($_GET['s']) ? trim($_GET['s']) : '');
+		if ( ! empty($search_terms)) {
+			$this->base_url = add_query_arg('s', $search_terms, $this->base_url);
 		}
 
-		if ( give_is_payment_complete( $payment->ID ) && ! empty( $email ) ) {
+		if (give_is_payment_complete($payment->ID) && ! empty($email)) {
 
 			$actions['email_links'] = sprintf(
 				'<a href="%1$s" aria-label="%2$s">%3$s</a>',
@@ -449,8 +449,8 @@ 
 					),
 					'give_payment_nonce'
 				),
-				sprintf( esc_attr__( 'Resend Donation %s Receipt', 'give' ), $payment->ID ),
-				esc_html__( 'Resend Receipt', 'give' )
+				sprintf(esc_attr__('Resend Donation %s Receipt', 'give'), $payment->ID),
+				esc_html__('Resend Receipt', 'give')
 			);
 
 		}
@@ -467,11 +467,11 @@ 
 				),
 				'give_donation_nonce'
 			),
-			sprintf( esc_attr__( 'Delete Donation %s', 'give' ), $payment->ID ),
-			esc_html__( 'Delete', 'give' )
+			sprintf(esc_attr__('Delete Donation %s', 'give'), $payment->ID),
+			esc_html__('Delete', 'give')
 		);
 
-		return apply_filters( 'give_payment_row_actions', $actions, $payment );
+		return apply_filters('give_payment_row_actions', $actions, $payment);
 	}
 
 
@@ -485,10 +485,10 @@ 
 	 *
 	 * @return string                Data shown in the Email column
 	 */
-	function get_payment_status( $payment ) {
-		$value = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
-		if ( $payment->mode == 'test' ) {
-			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This donation was made in test mode.', 'give' ) . '">' . esc_html__( 'Test', 'give' ) . '</span>';
+	function get_payment_status($payment) {
+		$value = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
+		if ($payment->mode == 'test') {
+			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This donation was made in test mode.', 'give').'">'.esc_html__('Test', 'give').'</span>';
 		}
 
 		return $value;
@@ -504,7 +504,7 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function column_cb( $payment ) {
+	public function column_cb($payment) {
 		return sprintf(
 			'<input type="checkbox" name="%1$s[]" value="%2$s" />',
 			'payment',
@@ -522,8 +522,8 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function get_payment_id( $payment ) {
-		return '<span class="give-payment-id">' . give_get_payment_number( $payment->ID ) . '</span>';
+	public function get_payment_id($payment) {
+		return '<span class="give-payment-id">'.give_get_payment_number($payment->ID).'</span>';
 	}
 
 	/**
@@ -536,19 +536,19 @@ 
 	 *
 	 * @return string Data shown in the User column
 	 */
-	public function get_donor( $payment ) {
+	public function get_donor($payment) {
 
-		$customer_id = give_get_payment_customer_id( $payment->ID );
+		$customer_id = give_get_payment_customer_id($payment->ID);
 
-		if ( ! empty( $customer_id ) ) {
-			$customer = new Give_Customer( $customer_id );
-			$value    = '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$customer_id" ) ) . '">' . $customer->name . '</a>';
+		if ( ! empty($customer_id)) {
+			$customer = new Give_Customer($customer_id);
+			$value    = '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$customer_id")).'">'.$customer->name.'</a>';
 		} else {
-			$email = give_get_payment_user_email( $payment->ID );
-			$value = '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-payment-history&s=$email" ) ) . '">' . esc_html__( '(donor missing)', 'give' ) . '</a>';
+			$email = give_get_payment_user_email($payment->ID);
+			$value = '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-payment-history&s=$email")).'">'.esc_html__('(donor missing)', 'give').'</a>';
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'donor' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'donor');
 	}
 
 	/**
@@ -560,18 +560,18 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete'               => esc_html__( 'Delete', 'give' ),
-			'set-status-publish'   => esc_html__( 'Set To Completed', 'give' ),
-			'set-status-pending'   => esc_html__( 'Set To Pending', 'give' ),
-			'set-status-refunded'  => esc_html__( 'Set To Refunded', 'give' ),
-			'set-status-revoked'   => esc_html__( 'Set To Revoked', 'give' ),
-			'set-status-failed'    => esc_html__( 'Set To Failed', 'give' ),
-			'set-status-cancelled' => esc_html__( 'Set To Cancelled', 'give' ),
-			'set-status-abandoned' => esc_html__( 'Set To Abandoned', 'give' ),
-			'resend-receipt'       => esc_html__( 'Resend Email Receipts', 'give' ),
+			'delete'               => esc_html__('Delete', 'give'),
+			'set-status-publish'   => esc_html__('Set To Completed', 'give'),
+			'set-status-pending'   => esc_html__('Set To Pending', 'give'),
+			'set-status-refunded'  => esc_html__('Set To Refunded', 'give'),
+			'set-status-revoked'   => esc_html__('Set To Revoked', 'give'),
+			'set-status-failed'    => esc_html__('Set To Failed', 'give'),
+			'set-status-cancelled' => esc_html__('Set To Cancelled', 'give'),
+			'set-status-abandoned' => esc_html__('Set To Abandoned', 'give'),
+			'resend-receipt'       => esc_html__('Resend Email Receipts', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_bulk_actions', $actions );
+		return apply_filters('give_payments_table_bulk_actions', $actions);
 	}
 
 	/**
@@ -582,59 +582,59 @@ 
 	 * @return void
 	 */
 	public function process_bulk_action() {
-		$ids    = isset( $_GET['payment'] ) ? $_GET['payment'] : false;
+		$ids    = isset($_GET['payment']) ? $_GET['payment'] : false;
 		$action = $this->current_action();
 
-		if ( ! is_array( $ids ) ) {
-			$ids = array( $ids );
+		if ( ! is_array($ids)) {
+			$ids = array($ids);
 		}
 
-		if ( empty( $action ) ) {
+		if (empty($action)) {
 			return;
 		}
 
-		foreach ( $ids as $id ) {
+		foreach ($ids as $id) {
 
 			// Detect when a bulk action is being triggered.
-			switch ( $this->current_action() ) {
+			switch ($this->current_action()) {
 
 				case'delete':
-					give_delete_purchase( $id );
+					give_delete_purchase($id);
 					break;
 
 				case 'set-status-publish':
-					give_update_payment_status( $id, 'publish' );
+					give_update_payment_status($id, 'publish');
 					break;
 
 				case 'set-status-pending':
-					give_update_payment_status( $id, 'pending' );
+					give_update_payment_status($id, 'pending');
 					break;
 
 				case 'set-status-refunded':
-					give_update_payment_status( $id, 'refunded' );
+					give_update_payment_status($id, 'refunded');
 					break;
 				case 'set-status-revoked':
-					give_update_payment_status( $id, 'revoked' );
+					give_update_payment_status($id, 'revoked');
 					break;
 
 				case 'set-status-failed':
-					give_update_payment_status( $id, 'failed' );
+					give_update_payment_status($id, 'failed');
 					break;
 
 				case 'set-status-cancelled':
-					give_update_payment_status( $id, 'cancelled' );
+					give_update_payment_status($id, 'cancelled');
 					break;
 
 				case 'set-status-abandoned':
-					give_update_payment_status( $id, 'abandoned' );
+					give_update_payment_status($id, 'abandoned');
 					break;
 
 				case 'set-status-preapproval':
-					give_update_payment_status( $id, 'preapproval' );
+					give_update_payment_status($id, 'preapproval');
 					break;
 
 				case 'resend-receipt':
-					give_email_donation_receipt( $id, false );
+					give_email_donation_receipt($id, false);
 					break;
 			}
 
@@ -646,7 +646,7 @@ 
 			 * @param int    $id             The ID of the payment.
 			 * @param string $current_action The action that is being triggered.
 			 */
-			do_action( 'give_payments_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_payments_table_do_bulk_action', $id, $this->current_action());
 		}
 
 	}
@@ -662,27 +662,27 @@ 
 
 		$args = array();
 
-		if ( isset( $_GET['user'] ) ) {
-			$args['user'] = urldecode( $_GET['user'] );
-		} elseif ( isset( $_GET['s'] ) ) {
-			$is_user = strpos( $_GET['s'], strtolower( 'user:' ) ) !== false;
-			if ( $is_user ) {
-				$args['user'] = absint( trim( str_replace( 'user:', '', strtolower( $_GET['s'] ) ) ) );
-				unset( $args['s'] );
+		if (isset($_GET['user'])) {
+			$args['user'] = urldecode($_GET['user']);
+		} elseif (isset($_GET['s'])) {
+			$is_user = strpos($_GET['s'], strtolower('user:')) !== false;
+			if ($is_user) {
+				$args['user'] = absint(trim(str_replace('user:', '', strtolower($_GET['s']))));
+				unset($args['s']);
 			} else {
-				$args['s'] = sanitize_text_field( $_GET['s'] );
+				$args['s'] = sanitize_text_field($_GET['s']);
 			}
 		}
 
-		if ( ! empty( $_GET['start-date'] ) ) {
-			$args['start-date'] = urldecode( $_GET['start-date'] );
+		if ( ! empty($_GET['start-date'])) {
+			$args['start-date'] = urldecode($_GET['start-date']);
 		}
 
-		if ( ! empty( $_GET['end-date'] ) ) {
-			$args['end-date'] = urldecode( $_GET['end-date'] );
+		if ( ! empty($_GET['end-date'])) {
+			$args['end-date'] = urldecode($_GET['end-date']);
 		}
 
-		$payment_count         = give_count_payments( $args );
+		$payment_count         = give_count_payments($args);
 		$this->complete_count  = $payment_count->publish;
 		$this->pending_count   = $payment_count->pending;
 		$this->refunded_count  = $payment_count->refunded;
@@ -691,7 +691,7 @@ 
 		$this->cancelled_count = $payment_count->cancelled;
 		$this->abandoned_count = $payment_count->abandoned;
 
-		foreach ( $payment_count as $count ) {
+		foreach ($payment_count as $count) {
 			$this->total_count += $count;
 		}
 	}
@@ -706,26 +706,26 @@ 
 	public function payments_data() {
 
 		$per_page   = $this->per_page;
-		$orderby    = isset( $_GET['orderby'] ) ? urldecode( $_GET['orderby'] ) : 'ID';
-		$order      = isset( $_GET['order'] ) ? $_GET['order'] : 'DESC';
-		$user       = isset( $_GET['user'] ) ? $_GET['user'] : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : give_get_payment_status_keys();
-		$meta_key   = isset( $_GET['meta_key'] ) ? $_GET['meta_key'] : null;
-		$year       = isset( $_GET['year'] ) ? $_GET['year'] : null;
-		$month      = isset( $_GET['m'] ) ? $_GET['m'] : null;
-		$day        = isset( $_GET['day'] ) ? $_GET['day'] : null;
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : null;
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : $start_date;
-
-		if ( ! empty( $search ) ) {
+		$orderby    = isset($_GET['orderby']) ? urldecode($_GET['orderby']) : 'ID';
+		$order      = isset($_GET['order']) ? $_GET['order'] : 'DESC';
+		$user       = isset($_GET['user']) ? $_GET['user'] : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : give_get_payment_status_keys();
+		$meta_key   = isset($_GET['meta_key']) ? $_GET['meta_key'] : null;
+		$year       = isset($_GET['year']) ? $_GET['year'] : null;
+		$month      = isset($_GET['m']) ? $_GET['m'] : null;
+		$day        = isset($_GET['day']) ? $_GET['day'] : null;
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : null;
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : $start_date;
+
+		if ( ! empty($search)) {
 			$status = 'any'; // Force all payment statuses when searching.
 		}
 
 		$args = array(
 			'output'     => 'payments',
 			'number'     => $per_page,
-			'page'       => isset( $_GET['paged'] ) ? $_GET['paged'] : null,
+			'page'       => isset($_GET['paged']) ? $_GET['paged'] : null,
 			'orderby'    => $orderby,
 			'order'      => $order,
 			'user'       => $user,
@@ -739,12 +739,12 @@ 
 			'end_date'   => $end_date,
 		);
 
-		if ( is_string( $search ) && false !== strpos( $search, 'txn:' ) ) {
+		if (is_string($search) && false !== strpos($search, 'txn:')) {
 			$args['search_in_notes'] = true;
-			$args['s']               = trim( str_replace( 'txn:', '', $args['s'] ) );
+			$args['s']               = trim(str_replace('txn:', '', $args['s']));
 		}
 
-		$p_query = new Give_Payments_Query( $args );
+		$p_query = new Give_Payments_Query($args);
 
 		return $p_query->get_payments();
 
@@ -764,17 +764,17 @@ 
 	 */
 	public function prepare_items() {
 
-		wp_reset_vars( array( 'action', 'payment', 'orderby', 'order', 's' ) );
+		wp_reset_vars(array('action', 'payment', 'orderby', 'order', 's'));
 
 		$columns  = $this->get_columns();
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 		$data     = $this->payments_data();
-		$status   = isset( $_GET['status'] ) ? $_GET['status'] : 'any';
+		$status   = isset($_GET['status']) ? $_GET['status'] : 'any';
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'publish':
 				$total_items = $this->complete_count;
 				break;
@@ -801,19 +801,19 @@ 
 				break;
 			default:
 				// Retrieve the count of the non-default-Give status.
-				$count       = wp_count_posts( 'give_payment' );
-				$total_items = isset( $count->{$status} ) ? $count->{$status} : 0;
+				$count       = wp_count_posts('give_payment');
+				$total_items = isset($count->{$status} ) ? $count->{$status} : 0;
 				break;
 		}
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				// We have to calculate the total number of items.
 				'per_page'    => $this->per_page,
 				// We have to determine how many items to show on a page.
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 				// We have to calculate the total number of pages.
 			)
 		);

includes/admin/tools/export/class-export-earnings.php

Spacing +28 added lines, -28 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,16 +38,16 @@ 
 	 * @return void
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 
 		nocache_headers();
-		header( 'Content-Type: text/csv; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=' . apply_filters( 'give_earnings_export_filename', 'give-export-' . $this->export_type . '-' . date( 'n' ) . '-' . date( 'Y' ) ) . '.csv' );
-		header( "Expires: 0" );
+		header('Content-Type: text/csv; charset=utf-8');
+		header('Content-Disposition: attachment; filename='.apply_filters('give_earnings_export_filename', 'give-export-'.$this->export_type.'-'.date('n').'-'.date('Y')).'.csv');
+		header("Expires: 0");
 
 	}
 
@@ -61,10 +61,10 @@ 
 	public function csv_cols() {
 
 		$cols = array(
-			'date'      => __( 'Date', 'give' ),
-			'donations' => __( 'Donations', 'give' ),
+			'date'      => __('Date', 'give'),
+			'donations' => __('Donations', 'give'),
 			/* translators: %s: currency */
-			'earnings'  => sprintf( __( 'Income (%s)', 'give' ), give_currency_symbol('', true) )
+			'earnings'  => sprintf(__('Income (%s)', 'give'), give_currency_symbol('', true))
 		);
 
 		return $cols;
@@ -79,28 +79,28 @@ 
 	 */
 	public function get_data() {
 
-		$start_year  = isset( $_POST['start_year'] ) ? absint( $_POST['start_year'] ) : date( 'Y' );
-		$end_year    = isset( $_POST['end_year'] ) ? absint( $_POST['end_year'] ) : date( 'Y' );
-		$start_month = isset( $_POST['start_month'] ) ? absint( $_POST['start_month'] ) : date( 'n' );
-		$end_month   = isset( $_POST['end_month'] ) ? absint( $_POST['end_month'] ) : date( 'n' );
+		$start_year  = isset($_POST['start_year']) ? absint($_POST['start_year']) : date('Y');
+		$end_year    = isset($_POST['end_year']) ? absint($_POST['end_year']) : date('Y');
+		$start_month = isset($_POST['start_month']) ? absint($_POST['start_month']) : date('n');
+		$end_month   = isset($_POST['end_month']) ? absint($_POST['end_month']) : date('n');
 
 		$data  = array();
 		$year  = $start_year;
 		$stats = new Give_Payment_Stats;
 
-		while ( $year <= $end_year ) {
+		while ($year <= $end_year) {
 
-			if ( $year == $start_year && $year == $end_year ) {
+			if ($year == $start_year && $year == $end_year) {
 
 				$m1 = $start_month;
 				$m2 = $end_month;
 
-			} elseif ( $year == $start_year ) {
+			} elseif ($year == $start_year) {
 
 				$m1 = $start_month;
 				$m2 = 12;
 
-			} elseif ( $year == $end_year ) {
+			} elseif ($year == $end_year) {
 
 				$m1 = 1;
 				$m2 = $end_month;
@@ -112,28 +112,28 @@ 
 
 			}
 
-			while ( $m1 <= $m2 ) {
+			while ($m1 <= $m2) {
 
-				$date1 = mktime( 0, 0, 0, $m1, 1, $year );
-				$date2 = mktime( 0, 0, 0, $m1, cal_days_in_month( CAL_GREGORIAN, $m1, $year ), $year );
+				$date1 = mktime(0, 0, 0, $m1, 1, $year);
+				$date2 = mktime(0, 0, 0, $m1, cal_days_in_month(CAL_GREGORIAN, $m1, $year), $year);
 
 				$data[] = array(
-					'date'      => date_i18n( 'F Y', $date1 ),
-					'donations' => $stats->get_sales( 0, $date1, $date2 ),
-					'earnings'  => give_format_amount( $stats->get_earnings( 0, $date1, $date2 ) ),
+					'date'      => date_i18n('F Y', $date1),
+					'donations' => $stats->get_sales(0, $date1, $date2),
+					'earnings'  => give_format_amount($stats->get_earnings(0, $date1, $date2)),
 				);
 
-				$m1 ++;
+				$m1++;
 
 			}
 
 
-			$year ++;
+			$year++;
 
 		}
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 		return $data;
 	}

includes/admin/tools/export/pdf-reports.php

Indentation +41 added lines, -41 removed lines

@@ -47,8 +47,8 @@ 
 		)
 	);
 
-    $categories_enabled = give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) );
-    $tags_enabled = give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) );
+	$categories_enabled = give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) );
+	$tags_enabled = give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) );
 
 	$pdf = new give_pdf();
 	$pdf->AddPage( 'L', 'A4' );
@@ -80,15 +80,15 @@ 
 	$pdf->Cell( 70, 6, utf8_decode( __( 'Form Name', 'give' ) ), 1, 0, 'L', true );
 	$pdf->Cell( 30, 6, utf8_decode( __( 'Price', 'give' ) ), 1, 0, 'L', true );
 
-    // Display Categories Heading only, if user has opted for it.
-    if ( $categories_enabled ) {
+	// Display Categories Heading only, if user has opted for it.
+	if ( $categories_enabled ) {
 	   $pdf->Cell( 45, 6, utf8_decode( __( 'Categories', 'give' ) ), 1, 0, 'L', true );
-    }
+	}
 
-    // Display Tags Heading only, if user has opted for it.
-    if ( $tags_enabled ) {
+	// Display Tags Heading only, if user has opted for it.
+	if ( $tags_enabled ) {
 	   $pdf->Cell( 45, 6, utf8_decode( __( 'Tags', 'give' ) ), 1, 0, 'L', true );
-    }
+	}
 
 	$pdf->Cell( 45, 6, utf8_decode( __( 'Number of Donations', 'give' ) ), 1, 0, 'L', true );
 	$pdf->Cell( 45, 6, utf8_decode( __( 'Income to Date', 'give' ) ), 1, 1, 'L', true );
@@ -125,17 +125,17 @@ 
 				$price = give_currency_filter( give_get_form_price( $form->ID ), '', true );
 			}
 
-            // Display Categories Data only, if user has opted for it.
-            if ( $categories_enabled ) {
-                $categories = get_the_term_list( $form->ID, 'give_forms_category', '', ', ', '' );
-                $categories = ! is_wp_error( $categories ) ? strip_tags( $categories ) : '';
-            }
+			// Display Categories Data only, if user has opted for it.
+			if ( $categories_enabled ) {
+				$categories = get_the_term_list( $form->ID, 'give_forms_category', '', ', ', '' );
+				$categories = ! is_wp_error( $categories ) ? strip_tags( $categories ) : '';
+			}
 
-            // Display Tags Data only, if user has opted for it.
-            if ( $tags_enabled ) {
-                $tags = get_the_term_list( $form->ID, 'give_forms_tag', '', ', ', '' );
-                $tags = ! is_wp_error( $tags ) ? strip_tags( $tags ) : '';
-            }
+			// Display Tags Data only, if user has opted for it.
+			if ( $tags_enabled ) {
+				$tags = get_the_term_list( $form->ID, 'give_forms_tag', '', ', ', '' );
+				$tags = ! is_wp_error( $tags ) ? strip_tags( $tags ) : '';
+			}
 
 			$sales    = give_get_form_sales_stats( $form->ID );
 			$link     = get_permalink( $form->ID );
@@ -147,38 +147,38 @@ 
 				$earnings = iconv( 'UTF-8', 'windows-1252', utf8_encode( $earnings ) );
 			}
 
-            // This will help filter data before appending it to PDF Receipt.
-            $prepare_pdf_data = array();
-            $prepare_pdf_data[] = $title;
-            $prepare_pdf_data[] = $price;
+			// This will help filter data before appending it to PDF Receipt.
+			$prepare_pdf_data = array();
+			$prepare_pdf_data[] = $title;
+			$prepare_pdf_data[] = $price;
 
-            // Append Categories Data only, if user has opted for it.
-            if ( $categories_enabled ) {
-                $prepare_pdf_data[] = $categories;
-            }
+			// Append Categories Data only, if user has opted for it.
+			if ( $categories_enabled ) {
+				$prepare_pdf_data[] = $categories;
+			}
 
-            // Append Tags Data only, if user has opted for it.
-            if ( $tags_enabled ) {
-                $prepare_pdf_data[] = $tags;
-            }
+			// Append Tags Data only, if user has opted for it.
+			if ( $tags_enabled ) {
+				$prepare_pdf_data[] = $tags;
+			}
 
-            $prepare_pdf_data[] = $sales;
-            $prepare_pdf_data[] = $earnings;
+			$prepare_pdf_data[] = $sales;
+			$prepare_pdf_data[] = $earnings;
 
 			$pdf->Row( $prepare_pdf_data );
 		endforeach;
 	} else {
 
-        // Fix: Minor Styling Alignment Issue for PDF
-        if( $categories_enabled && $tags_enabled ) {
-            $pdf->SetWidths( array( 280 ) );
-        } elseif( $categories_enabled || $tags_enabled ) {
-            $pdf->SetWidths( array( 235 ) );
-        } else {
-            $pdf->SetWidths( array( 190 ) );
-        }
+		// Fix: Minor Styling Alignment Issue for PDF
+		if( $categories_enabled && $tags_enabled ) {
+			$pdf->SetWidths( array( 280 ) );
+		} elseif( $categories_enabled || $tags_enabled ) {
+			$pdf->SetWidths( array( 235 ) );
+		} else {
+			$pdf->SetWidths( array( 190 ) );
+		}
 
-        $title = utf8_decode( __( 'No forms found.', 'give' ) );
+		$title = utf8_decode( __( 'No forms found.', 'give' ) );
 		$pdf->Row( array( $title ) );
 	}
 

Spacing +158 added lines, -158 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly..
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,94 +25,94 @@ 
  *
  * @uses   give_pdf
  */
-function give_generate_pdf( $data ) {
+function give_generate_pdf($data) {
 
-	if ( ! current_user_can( 'view_give_reports' ) ) {
-		wp_die( __( 'You do not have permission to generate PDF sales reports.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('view_give_reports')) {
+		wp_die(__('You do not have permission to generate PDF sales reports.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	if ( ! wp_verify_nonce( $_GET['_wpnonce'], 'give_generate_pdf' ) ) {
-		wp_die( __( 'Nonce verification failed.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($_GET['_wpnonce'], 'give_generate_pdf')) {
+		wp_die(__('Nonce verification failed.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/fpdf/fpdf.php';
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/fpdf/give_pdf.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/fpdf/fpdf.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/fpdf/give_pdf.php';
 
 	$daterange = utf8_decode(
 		sprintf(
 		/* translators: 1: start date 2: end date */
-			__( '%1$s to %2$s', 'give' ),
-			date_i18n( give_date_format(), mktime( 0, 0, 0, 1, 1, date( 'Y' ) ) ),
-			date_i18n( give_date_format() )
+			__('%1$s to %2$s', 'give'),
+			date_i18n(give_date_format(), mktime(0, 0, 0, 1, 1, date('Y'))),
+			date_i18n(give_date_format())
 		)
 	);
 
-    $categories_enabled = give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) );
-    $tags_enabled = give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) );
+    $categories_enabled = give_is_setting_enabled(give_get_option('categories', 'disabled'));
+    $tags_enabled = give_is_setting_enabled(give_get_option('tags', 'disabled'));
 
 	$pdf = new give_pdf();
-	$pdf->AddPage( 'L', 'A4' );
+	$pdf->AddPage('L', 'A4');
 
-	$pdf->SetTitle( utf8_decode( __( 'Donation report for the current year for all forms', 'give' ) ) );
-	$pdf->SetAuthor( utf8_decode( __( 'Give - Democratizing Generosity', 'give' ) ) );
-	$pdf->SetCreator( utf8_decode( __( 'Give - Democratizing Generosity', 'give' ) ) );
+	$pdf->SetTitle(utf8_decode(__('Donation report for the current year for all forms', 'give')));
+	$pdf->SetAuthor(utf8_decode(__('Give - Democratizing Generosity', 'give')));
+	$pdf->SetCreator(utf8_decode(__('Give - Democratizing Generosity', 'give')));
 
-	$pdf->Image( apply_filters( 'give_pdf_export_logo', GIVE_PLUGIN_URL . 'assets/images/give-logo-small.png' ), 247, 8 );
+	$pdf->Image(apply_filters('give_pdf_export_logo', GIVE_PLUGIN_URL.'assets/images/give-logo-small.png'), 247, 8);
 
-	$pdf->SetMargins( 8, 8, 8 );
-	$pdf->SetX( 8 );
+	$pdf->SetMargins(8, 8, 8);
+	$pdf->SetX(8);
 
-	$pdf->SetFont( 'Helvetica', '', 16 );
-	$pdf->SetTextColor( 50, 50, 50 );
-	$pdf->Cell( 0, 3, utf8_decode( __( 'Donation report for the current year for all forms', 'give' ) ), 0, 2, 'L', false );
+	$pdf->SetFont('Helvetica', '', 16);
+	$pdf->SetTextColor(50, 50, 50);
+	$pdf->Cell(0, 3, utf8_decode(__('Donation report for the current year for all forms', 'give')), 0, 2, 'L', false);
 
-	$pdf->SetFont( 'Helvetica', '', 13 );
+	$pdf->SetFont('Helvetica', '', 13);
 	$pdf->Ln();
-	$pdf->SetTextColor( 150, 150, 150 );
-	$pdf->Cell( 0, 6, utf8_decode( __( 'Date Range: ', 'give' ) ) . $daterange, 0, 2, 'L', false );
+	$pdf->SetTextColor(150, 150, 150);
+	$pdf->Cell(0, 6, utf8_decode(__('Date Range: ', 'give')).$daterange, 0, 2, 'L', false);
 	$pdf->Ln();
-	$pdf->SetTextColor( 50, 50, 50 );
-	$pdf->SetFont( 'Helvetica', '', 14 );
-	$pdf->Cell( 0, 10, utf8_decode( __( 'Table View', 'give' ) ), 0, 2, 'L', false );
-	$pdf->SetFont( 'Helvetica', '', 12 );
+	$pdf->SetTextColor(50, 50, 50);
+	$pdf->SetFont('Helvetica', '', 14);
+	$pdf->Cell(0, 10, utf8_decode(__('Table View', 'give')), 0, 2, 'L', false);
+	$pdf->SetFont('Helvetica', '', 12);
 
-	$pdf->SetFillColor( 238, 238, 238 );
-	$pdf->Cell( 70, 6, utf8_decode( __( 'Form Name', 'give' ) ), 1, 0, 'L', true );
-	$pdf->Cell( 30, 6, utf8_decode( __( 'Price', 'give' ) ), 1, 0, 'L', true );
+	$pdf->SetFillColor(238, 238, 238);
+	$pdf->Cell(70, 6, utf8_decode(__('Form Name', 'give')), 1, 0, 'L', true);
+	$pdf->Cell(30, 6, utf8_decode(__('Price', 'give')), 1, 0, 'L', true);
 
     // Display Categories Heading only, if user has opted for it.
-    if ( $categories_enabled ) {
-	   $pdf->Cell( 45, 6, utf8_decode( __( 'Categories', 'give' ) ), 1, 0, 'L', true );
+    if ($categories_enabled) {
+	   $pdf->Cell(45, 6, utf8_decode(__('Categories', 'give')), 1, 0, 'L', true);
     }
 
     // Display Tags Heading only, if user has opted for it.
-    if ( $tags_enabled ) {
-	   $pdf->Cell( 45, 6, utf8_decode( __( 'Tags', 'give' ) ), 1, 0, 'L', true );
+    if ($tags_enabled) {
+	   $pdf->Cell(45, 6, utf8_decode(__('Tags', 'give')), 1, 0, 'L', true);
     }
 
-	$pdf->Cell( 45, 6, utf8_decode( __( 'Number of Donations', 'give' ) ), 1, 0, 'L', true );
-	$pdf->Cell( 45, 6, utf8_decode( __( 'Income to Date', 'give' ) ), 1, 1, 'L', true );
+	$pdf->Cell(45, 6, utf8_decode(__('Number of Donations', 'give')), 1, 0, 'L', true);
+	$pdf->Cell(45, 6, utf8_decode(__('Income to Date', 'give')), 1, 1, 'L', true);
 
-	$year       = date( 'Y' );
-	$give_forms = get_posts( array( 'post_type' => 'give_forms', 'year' => $year, 'posts_per_page' => - 1 ) );
+	$year       = date('Y');
+	$give_forms = get_posts(array('post_type' => 'give_forms', 'year' => $year, 'posts_per_page' => -1));
 
-	if ( $give_forms ) {
-		$pdf->SetWidths( array( 70, 30, 45, 45, 45, 45 ) );
+	if ($give_forms) {
+		$pdf->SetWidths(array(70, 30, 45, 45, 45, 45));
 
-		foreach ( $give_forms as $form ):
-			$pdf->SetFillColor( 255, 255, 255 );
+		foreach ($give_forms as $form):
+			$pdf->SetFillColor(255, 255, 255);
 
 			$title = $form->post_title;
 
-			if ( give_has_variable_prices( $form->ID ) ) {
+			if (give_has_variable_prices($form->ID)) {
 
-				$prices = give_get_variable_prices( $form->ID );
+				$prices = give_get_variable_prices($form->ID);
 
 				$first = $prices[0]['_give_amount'];
-				$last  = array_pop( $prices );
+				$last  = array_pop($prices);
 				$last  = $last['_give_amount'];
 
-				if ( $first < $last ) {
+				if ($first < $last) {
 					$min = $first;
 					$max = $last;
 				} else {
@@ -120,31 +120,31 @@ 
 					$max = $first;
 				}
 
-				$price = give_currency_filter( give_format_amount( $min ), '', true ) . ' - ' . give_currency_filter( give_format_amount( $max ), '', true );
+				$price = give_currency_filter(give_format_amount($min), '', true).' - '.give_currency_filter(give_format_amount($max), '', true);
 			} else {
-				$price = give_currency_filter( give_get_form_price( $form->ID ), '', true );
+				$price = give_currency_filter(give_get_form_price($form->ID), '', true);
 			}
 
             // Display Categories Data only, if user has opted for it.
-            if ( $categories_enabled ) {
-                $categories = get_the_term_list( $form->ID, 'give_forms_category', '', ', ', '' );
-                $categories = ! is_wp_error( $categories ) ? strip_tags( $categories ) : '';
+            if ($categories_enabled) {
+                $categories = get_the_term_list($form->ID, 'give_forms_category', '', ', ', '');
+                $categories = ! is_wp_error($categories) ? strip_tags($categories) : '';
             }
 
             // Display Tags Data only, if user has opted for it.
-            if ( $tags_enabled ) {
-                $tags = get_the_term_list( $form->ID, 'give_forms_tag', '', ', ', '' );
-                $tags = ! is_wp_error( $tags ) ? strip_tags( $tags ) : '';
+            if ($tags_enabled) {
+                $tags = get_the_term_list($form->ID, 'give_forms_tag', '', ', ', '');
+                $tags = ! is_wp_error($tags) ? strip_tags($tags) : '';
             }
 
-			$sales    = give_get_form_sales_stats( $form->ID );
-			$link     = get_permalink( $form->ID );
-			$earnings = give_currency_filter( give_get_form_earnings_stats( $form->ID ), '', true );
+			$sales    = give_get_form_sales_stats($form->ID);
+			$link     = get_permalink($form->ID);
+			$earnings = give_currency_filter(give_get_form_earnings_stats($form->ID), '', true);
 
-			if ( function_exists( 'iconv' ) ) {
+			if (function_exists('iconv')) {
 				// Ensure characters like euro; are properly converted.
-				$price    = iconv( 'UTF-8', 'windows-1252', utf8_encode( $price ) );
-				$earnings = iconv( 'UTF-8', 'windows-1252', utf8_encode( $earnings ) );
+				$price    = iconv('UTF-8', 'windows-1252', utf8_encode($price));
+				$earnings = iconv('UTF-8', 'windows-1252', utf8_encode($earnings));
 			}
 
             // This will help filter data before appending it to PDF Receipt.
@@ -153,51 +153,51 @@ 
             $prepare_pdf_data[] = $price;
 
             // Append Categories Data only, if user has opted for it.
-            if ( $categories_enabled ) {
+            if ($categories_enabled) {
                 $prepare_pdf_data[] = $categories;
             }
 
             // Append Tags Data only, if user has opted for it.
-            if ( $tags_enabled ) {
+            if ($tags_enabled) {
                 $prepare_pdf_data[] = $tags;
             }
 
             $prepare_pdf_data[] = $sales;
             $prepare_pdf_data[] = $earnings;
 
-			$pdf->Row( $prepare_pdf_data );
+			$pdf->Row($prepare_pdf_data);
 		endforeach;
 	} else {
 
         // Fix: Minor Styling Alignment Issue for PDF
-        if( $categories_enabled && $tags_enabled ) {
-            $pdf->SetWidths( array( 280 ) );
-        } elseif( $categories_enabled || $tags_enabled ) {
-            $pdf->SetWidths( array( 235 ) );
+        if ($categories_enabled && $tags_enabled) {
+            $pdf->SetWidths(array(280));
+        } elseif ($categories_enabled || $tags_enabled) {
+            $pdf->SetWidths(array(235));
         } else {
-            $pdf->SetWidths( array( 190 ) );
+            $pdf->SetWidths(array(190));
         }
 
-        $title = utf8_decode( __( 'No forms found.', 'give' ) );
-		$pdf->Row( array( $title ) );
+        $title = utf8_decode(__('No forms found.', 'give'));
+		$pdf->Row(array($title));
 	}
 
 	$pdf->Ln();
-	$pdf->SetTextColor( 50, 50, 50 );
-	$pdf->SetFont( 'Helvetica', '', 14 );
-	$pdf->Cell( 0, 10, utf8_decode( __( 'Graph View', 'give' ) ), 0, 2, 'L', false );
-	$pdf->SetFont( 'Helvetica', '', 12 );
-
-	$image = html_entity_decode( urldecode( give_draw_chart_image() ) );
-	$image = str_replace( ' ', '%20', $image );
-
-	$pdf->SetX( 25 );
-	$pdf->Image( $image . '&file=.png' );
-	$pdf->Ln( 7 );
-	$pdf->Output( apply_filters( 'give_sales_earnings_pdf_export_filename', 'give-report-' . date_i18n( 'Y-m-d' ) ) . '.pdf', 'D' );
+	$pdf->SetTextColor(50, 50, 50);
+	$pdf->SetFont('Helvetica', '', 14);
+	$pdf->Cell(0, 10, utf8_decode(__('Graph View', 'give')), 0, 2, 'L', false);
+	$pdf->SetFont('Helvetica', '', 12);
+
+	$image = html_entity_decode(urldecode(give_draw_chart_image()));
+	$image = str_replace(' ', '%20', $image);
+
+	$pdf->SetX(25);
+	$pdf->Image($image.'&file=.png');
+	$pdf->Ln(7);
+	$pdf->Output(apply_filters('give_sales_earnings_pdf_export_filename', 'give-report-'.date_i18n('Y-m-d')).'.pdf', 'D');
 }
 
-add_action( 'give_generate_pdf', 'give_generate_pdf' );
+add_action('give_generate_pdf', 'give_generate_pdf');
 
 /**
  * Draws Chart for PDF Report.
@@ -214,38 +214,38 @@ 
  * @return string $chart->getUrl() URL for the Google Chart
  */
 function give_draw_chart_image() {
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/googlechartlib/GoogleChart.php';
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/googlechartlib/markers/GoogleChartShapeMarker.php';
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/googlechartlib/markers/GoogleChartTextMarker.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/googlechartlib/GoogleChart.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/googlechartlib/markers/GoogleChartShapeMarker.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/googlechartlib/markers/GoogleChartTextMarker.php';
 
-	$chart = new GoogleChart( 'lc', 900, 330 );
+	$chart = new GoogleChart('lc', 900, 330);
 
 	$i        = 1;
 	$earnings = "";
 	$sales    = "";
 
-	while ( $i <= 12 ) :
-		$earnings .= give_get_earnings_by_date( null, $i, date( 'Y' ) ) . ",";
-		$sales .= give_get_sales_by_date( null, $i, date( 'Y' ) ) . ",";
-		$i ++;
+	while ($i <= 12) :
+		$earnings .= give_get_earnings_by_date(null, $i, date('Y')).",";
+		$sales .= give_get_sales_by_date(null, $i, date('Y')).",";
+		$i++;
 	endwhile;
 
-	$earnings_array = explode( ",", $earnings );
-	$sales_array    = explode( ",", $sales );
+	$earnings_array = explode(",", $earnings);
+	$sales_array    = explode(",", $sales);
 
 	$i = 0;
-	while ( $i <= 11 ) {
-		if ( empty( $sales_array[ $i ] ) ) {
-			$sales_array[ $i ] = 0;
+	while ($i <= 11) {
+		if (empty($sales_array[$i])) {
+			$sales_array[$i] = 0;
 		}
-		$i ++;
+		$i++;
 	}
 
 	$min_earnings   = 0;
-	$max_earnings   = max( $earnings_array );
-	$earnings_scale = round( $max_earnings, - 1 );
+	$max_earnings   = max($earnings_array);
+	$earnings_scale = round($max_earnings, - 1);
 
-	$data = new GoogleChartData( array(
+	$data = new GoogleChartData(array(
 		$earnings_array[0],
 		$earnings_array[1],
 		$earnings_array[2],
@@ -258,25 +258,25 @@ 
 		$earnings_array[9],
 		$earnings_array[10],
 		$earnings_array[11]
-	) );
+	));
 
-	$data->setLegend( __( 'Income', 'give' ) );
-	$data->setColor( '1b58a3' );
-	$chart->addData( $data );
+	$data->setLegend(__('Income', 'give'));
+	$data->setColor('1b58a3');
+	$chart->addData($data);
 
-	$shape_marker = new GoogleChartShapeMarker( GoogleChartShapeMarker::CIRCLE );
-	$shape_marker->setColor( '000000' );
-	$shape_marker->setSize( 7 );
-	$shape_marker->setBorder( 2 );
-	$shape_marker->setData( $data );
-	$chart->addMarker( $shape_marker );
+	$shape_marker = new GoogleChartShapeMarker(GoogleChartShapeMarker::CIRCLE);
+	$shape_marker->setColor('000000');
+	$shape_marker->setSize(7);
+	$shape_marker->setBorder(2);
+	$shape_marker->setData($data);
+	$chart->addMarker($shape_marker);
 
-	$value_marker = new GoogleChartTextMarker( GoogleChartTextMarker::VALUE );
-	$value_marker->setColor( '000000' );
-	$value_marker->setData( $data );
-	$chart->addMarker( $value_marker );
+	$value_marker = new GoogleChartTextMarker(GoogleChartTextMarker::VALUE);
+	$value_marker->setColor('000000');
+	$value_marker->setData($data);
+	$chart->addMarker($value_marker);
 
-	$data = new GoogleChartData( array(
+	$data = new GoogleChartData(array(
 		$sales_array[0],
 		$sales_array[1],
 		$sales_array[2],
@@ -289,46 +289,46 @@ 
 		$sales_array[9],
 		$sales_array[10],
 		$sales_array[11]
-	) );
-	$data->setLegend( __( 'Donations', 'give' ) );
-	$data->setColor( 'ff6c1c' );
-	$chart->addData( $data );
-
-	$chart->setTitle( __( 'Donations by Month for all Give Forms', 'give' ), '336699', 18 );
-
-	$chart->setScale( 0, $max_earnings );
-
-	$y_axis = new GoogleChartAxis( 'y' );
-	$y_axis->setDrawTickMarks( true )->setLabels( array( 0, $max_earnings ) );
-	$chart->addAxis( $y_axis );
-
-	$x_axis = new GoogleChartAxis( 'x' );
-	$x_axis->setTickMarks( 5 );
-	$x_axis->setLabels( array(
-		__( 'Jan', 'give' ),
-		__( 'Feb', 'give' ),
-		__( 'Mar', 'give' ),
-		__( 'Apr', 'give' ),
-		__( 'May', 'give' ),
-		__( 'June', 'give' ),
-		__( 'July', 'give' ),
-		__( 'Aug', 'give' ),
-		__( 'Sept', 'give' ),
-		__( 'Oct', 'give' ),
-		__( 'Nov', 'give' ),
-		__( 'Dec', 'give' )
-	) );
-	$chart->addAxis( $x_axis );
-
-	$shape_marker = new GoogleChartShapeMarker( GoogleChartShapeMarker::CIRCLE );
-	$shape_marker->setSize( 6 );
-	$shape_marker->setBorder( 2 );
-	$shape_marker->setData( $data );
-	$chart->addMarker( $shape_marker );
-
-	$value_marker = new GoogleChartTextMarker( GoogleChartTextMarker::VALUE );
-	$value_marker->setData( $data );
-	$chart->addMarker( $value_marker );
+	));
+	$data->setLegend(__('Donations', 'give'));
+	$data->setColor('ff6c1c');
+	$chart->addData($data);
+
+	$chart->setTitle(__('Donations by Month for all Give Forms', 'give'), '336699', 18);
+
+	$chart->setScale(0, $max_earnings);
+
+	$y_axis = new GoogleChartAxis('y');
+	$y_axis->setDrawTickMarks(true)->setLabels(array(0, $max_earnings));
+	$chart->addAxis($y_axis);
+
+	$x_axis = new GoogleChartAxis('x');
+	$x_axis->setTickMarks(5);
+	$x_axis->setLabels(array(
+		__('Jan', 'give'),
+		__('Feb', 'give'),
+		__('Mar', 'give'),
+		__('Apr', 'give'),
+		__('May', 'give'),
+		__('June', 'give'),
+		__('July', 'give'),
+		__('Aug', 'give'),
+		__('Sept', 'give'),
+		__('Oct', 'give'),
+		__('Nov', 'give'),
+		__('Dec', 'give')
+	));
+	$chart->addAxis($x_axis);
+
+	$shape_marker = new GoogleChartShapeMarker(GoogleChartShapeMarker::CIRCLE);
+	$shape_marker->setSize(6);
+	$shape_marker->setBorder(2);
+	$shape_marker->setData($data);
+	$chart->addMarker($shape_marker);
+
+	$value_marker = new GoogleChartTextMarker(GoogleChartTextMarker::VALUE);
+	$value_marker->setData($data);
+	$chart->addMarker($value_marker);
 
 	return $chart->getUrl();
 }

includes/admin/tools/export/class-batch-export-payments.php

Spacing +71 added lines, -71 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,30 +39,30 @@ 
 	 */
 	public function csv_cols() {
 		$cols = array(
-			'id'        => __( 'ID', 'give' ), // unaltered payment ID (use for querying).
-			'seq_id'    => __( 'Payment Number', 'give' ), // sequential payment ID.
-			'email'     => __( 'Email', 'give' ),
-			'first'     => __( 'First Name', 'give' ),
-			'last'      => __( 'Last Name', 'give' ),
-			'address1'  => __( 'Address 1', 'give' ),
-			'address2'  => __( 'Address 2', 'give' ),
-			'city'      => __( 'City', 'give' ),
-			'state'     => __( 'State', 'give' ),
-			'country'   => __( 'Country', 'give' ),
-			'zip'       => __( 'Zip / Postal Code', 'give' ),
-			'form_id'   => __( 'Form ID', 'give' ),
-			'form_name' => __( 'Form Name', 'give' ),
-			'amount'    => __( 'Amount', 'give' ) . ' (' . give_currency_symbol( '', true ) . ')',
-			'gateway'   => __( 'Payment Method', 'give' ),
-			'trans_id'  => __( 'Transaction ID', 'give' ),
-			'key'       => __( 'Key', 'give' ),
-			'date'      => __( 'Date', 'give' ),
-			'user'      => __( 'User', 'give' ),
-			'status'    => __( 'Status', 'give' )
+			'id'        => __('ID', 'give'), // unaltered payment ID (use for querying).
+			'seq_id'    => __('Payment Number', 'give'), // sequential payment ID.
+			'email'     => __('Email', 'give'),
+			'first'     => __('First Name', 'give'),
+			'last'      => __('Last Name', 'give'),
+			'address1'  => __('Address 1', 'give'),
+			'address2'  => __('Address 2', 'give'),
+			'city'      => __('City', 'give'),
+			'state'     => __('State', 'give'),
+			'country'   => __('Country', 'give'),
+			'zip'       => __('Zip / Postal Code', 'give'),
+			'form_id'   => __('Form ID', 'give'),
+			'form_name' => __('Form Name', 'give'),
+			'amount'    => __('Amount', 'give').' ('.give_currency_symbol('', true).')',
+			'gateway'   => __('Payment Method', 'give'),
+			'trans_id'  => __('Transaction ID', 'give'),
+			'key'       => __('Key', 'give'),
+			'date'      => __('Date', 'give'),
+			'user'      => __('User', 'give'),
+			'status'    => __('Status', 'give')
 		);
 
-		if ( ! give_get_option( 'enable_sequential' ) ) {
-			unset( $cols['seq_id'] );
+		if ( ! give_get_option('enable_sequential')) {
+			unset($cols['seq_id']);
 		}
 
 		return $cols;
@@ -86,12 +86,12 @@ 
 			'status' => $this->status
 		);
 
-		if ( ! empty( $this->start ) || ! empty( $this->end ) ) {
+		if ( ! empty($this->start) || ! empty($this->end)) {
 
 			$args['date_query'] = array(
 				array(
-					'after'     => date( 'Y-n-d 00:00:00', strtotime( $this->start ) ),
-					'before'    => date( 'Y-n-d 23:59:59', strtotime( $this->end ) ),
+					'after'     => date('Y-n-d 00:00:00', strtotime($this->start)),
+					'before'    => date('Y-n-d 23:59:59', strtotime($this->end)),
 					'inclusive' => true
 				)
 			);
@@ -99,11 +99,11 @@ 
 		}
 
 		// Add category or tag to payment query if any.
-		if ( ! empty( $this->categories ) || ! empty( $this->tags ) ) {
+		if ( ! empty($this->categories) || ! empty($this->tags)) {
 			$form_args = array(
 				'post_type'      => 'give_forms',
 				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'fields'         => 'ids',
 				'tax_query'      => array(
 					'relation' => 'AND',
@@ -111,72 +111,72 @@ 
 			);
 
 
-			if ( ! empty( $this->categories ) ) {
+			if ( ! empty($this->categories)) {
 				$form_args['tax_query'][] = array(
 					'taxonomy' => 'give_forms_category',
 					'terms'    => $this->categories,
 				);
 			}
 
-			if ( ! empty( $this->tags ) ) {
+			if ( ! empty($this->tags)) {
 				$form_args['tax_query'][] = array(
 					'taxonomy' => 'give_forms_tag',
 					'terms'    => $this->tags,
 				);
 			}
 
-			$forms = new WP_Query( $form_args );
+			$forms = new WP_Query($form_args);
 
-			if ( empty( $forms->posts ) ) {
+			if (empty($forms->posts)) {
 				return array();
 			}
 
 			$args['give_forms'] = $forms->posts;
 		}
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( $payments ) {
+		if ($payments) {
 
-			foreach ( $payments as $payment ) {
-				$payment_meta = give_get_payment_meta( $payment->ID );
-				$user_info    = give_get_payment_meta_user_info( $payment->ID );
-				$total        = give_get_payment_amount( $payment->ID );
-				$user_id      = isset( $user_info['id'] ) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
+			foreach ($payments as $payment) {
+				$payment_meta = give_get_payment_meta($payment->ID);
+				$user_info    = give_get_payment_meta_user_info($payment->ID);
+				$total        = give_get_payment_amount($payment->ID);
+				$user_id      = isset($user_info['id']) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
 
-				if ( is_numeric( $user_id ) ) {
-					$user = get_userdata( $user_id );
+				if (is_numeric($user_id)) {
+					$user = get_userdata($user_id);
 				} else {
 					$user = false;
 				}
 
 				$data[] = array(
 					'id'        => $payment->ID,
-					'seq_id'    => give_get_payment_number( $payment->ID ),
+					'seq_id'    => give_get_payment_number($payment->ID),
 					'email'     => $payment_meta['email'],
 					'first'     => $user_info['first_name'],
 					'last'      => $user_info['last_name'],
-					'address1'  => isset( $user_info['address']['line1'] ) ? $user_info['address']['line1'] : '',
-					'address2'  => isset( $user_info['address']['line2'] ) ? $user_info['address']['line2'] : '',
-					'city'      => isset( $user_info['address']['city'] ) ? $user_info['address']['city'] : '',
-					'state'     => isset( $user_info['address']['state'] ) ? $user_info['address']['state'] : '',
-					'country'   => isset( $user_info['address']['country'] ) ? $user_info['address']['country'] : '',
-					'zip'       => isset( $user_info['address']['zip'] ) ? $user_info['address']['zip'] : '',
-					'form_id'   => isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : '',
-					'form_name' => isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '',
-					'amount'    => html_entity_decode( give_format_amount( $total ) ),
-					'gateway'   => give_get_gateway_admin_label( give_get_meta( $payment->ID, '_give_payment_gateway', true ) ),
-					'trans_id'  => give_get_payment_transaction_id( $payment->ID ),
+					'address1'  => isset($user_info['address']['line1']) ? $user_info['address']['line1'] : '',
+					'address2'  => isset($user_info['address']['line2']) ? $user_info['address']['line2'] : '',
+					'city'      => isset($user_info['address']['city']) ? $user_info['address']['city'] : '',
+					'state'     => isset($user_info['address']['state']) ? $user_info['address']['state'] : '',
+					'country'   => isset($user_info['address']['country']) ? $user_info['address']['country'] : '',
+					'zip'       => isset($user_info['address']['zip']) ? $user_info['address']['zip'] : '',
+					'form_id'   => isset($payment_meta['form_id']) ? $payment_meta['form_id'] : '',
+					'form_name' => isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '',
+					'amount'    => html_entity_decode(give_format_amount($total)),
+					'gateway'   => give_get_gateway_admin_label(give_get_meta($payment->ID, '_give_payment_gateway', true)),
+					'trans_id'  => give_get_payment_transaction_id($payment->ID),
 					'key'       => $payment_meta['key'],
 					'date'      => $payment->post_date,
-					'user'      => $user ? $user->display_name : __( 'guest', 'give' ),
-					'status'    => give_get_payment_status( $payment, true )
+					'user'      => $user ? $user->display_name : __('guest', 'give'),
+					'status'    => give_get_payment_status($payment, true)
 				);
 
 			}
 
-			$data = apply_filters( 'give_export_get_data', $data );
-			$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+			$data = apply_filters('give_export_get_data', $data);
+			$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 			return $data;
 
@@ -196,27 +196,27 @@ 
 
 		$status = $this->status;
 		$args   = array(
-			'start-date' => date( 'n/d/Y', strtotime( $this->start ) ),
-			'end-date'   => date( 'n/d/Y', strtotime( $this->end ) ),
+			'start-date' => date('n/d/Y', strtotime($this->start)),
+			'end-date'   => date('n/d/Y', strtotime($this->end)),
 		);
 
-		if ( 'any' == $status ) {
+		if ('any' == $status) {
 
-			$total = array_sum( (array) give_count_payments( $args ) );
+			$total = array_sum((array) give_count_payments($args));
 
 		} else {
 
-			$total = give_count_payments( $args )->$status;
+			$total = give_count_payments($args)->$status;
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -230,11 +230,11 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
-		$this->start      = isset( $request['start'] ) ? sanitize_text_field( $request['start'] ) : '';
-		$this->end        = isset( $request['end'] ) ? sanitize_text_field( $request['end'] ) : '';
-		$this->status     = isset( $request['status'] ) ? sanitize_text_field( $request['status'] ) : 'complete';
-		$this->categories = isset( $request['give_forms_categories'] ) ? give_clean( $request['give_forms_categories'] ) : array();
-		$this->tags       = isset( $request['give_forms_tags'] ) ? give_clean( $request['give_forms_tags'] ) : array();
+	public function set_properties($request) {
+		$this->start      = isset($request['start']) ? sanitize_text_field($request['start']) : '';
+		$this->end        = isset($request['end']) ? sanitize_text_field($request['end']) : '';
+		$this->status     = isset($request['status']) ? sanitize_text_field($request['status']) : 'complete';
+		$this->categories = isset($request['give_forms_categories']) ? give_clean($request['give_forms_categories']) : array();
+		$this->tags       = isset($request['give_forms_tags']) ? give_clean($request['give_forms_tags']) : array();
 	}
 }

includes/admin/tools/export/class-batch-export-forms.php

Spacing +51 added lines, -51 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -41,20 +41,20 @@ 
 	public function csv_cols() {
 
 		$cols = array(
-			'ID'                      => __( 'ID', 'give' ),
-			'post_name'               => __( 'Slug', 'give' ),
-			'post_title'              => __( 'Name', 'give' ),
-			'post_date'               => __( 'Date Created', 'give' ),
-			'post_author'             => __( 'Author', 'give' ),
-			'post_content'            => __( 'Description', 'give' ),
-			'post_excerpt'            => __( 'Excerpt', 'give' ),
-			'post_status'             => __( 'Status', 'give' ),
-			'categories'              => __( 'Categories', 'give' ),
-			'tags'                    => __( 'Tags', 'give' ),
-			'give_price'              => __( 'Price', 'give' ),
-			'_thumbnail_id'           => __( 'Featured Image', 'give' ),
-			'_give_form_sales'        => __( 'Donations', 'give' ),
-			'_give_download_earnings' => __( 'Income', 'give' ),
+			'ID'                      => __('ID', 'give'),
+			'post_name'               => __('Slug', 'give'),
+			'post_title'              => __('Name', 'give'),
+			'post_date'               => __('Date Created', 'give'),
+			'post_author'             => __('Author', 'give'),
+			'post_content'            => __('Description', 'give'),
+			'post_excerpt'            => __('Excerpt', 'give'),
+			'post_status'             => __('Status', 'give'),
+			'categories'              => __('Categories', 'give'),
+			'tags'                    => __('Tags', 'give'),
+			'give_price'              => __('Price', 'give'),
+			'_thumbnail_id'           => __('Featured Image', 'give'),
+			'_give_form_sales'        => __('Donations', 'give'),
+			'_give_download_earnings' => __('Income', 'give'),
 		);
 
 		return $cols;
@@ -84,43 +84,43 @@ 
 			'paged'          => $this->step
 		);
 
-		$forms = new WP_Query( $args );
+		$forms = new WP_Query($args);
 
-		if ( $forms->posts ) {
-			foreach ( $forms->posts as $form ) {
+		if ($forms->posts) {
+			foreach ($forms->posts as $form) {
 
 				$row = [];
 
-				foreach ( $this->csv_cols() as $key => $value ) {
+				foreach ($this->csv_cols() as $key => $value) {
 
 					// Setup default value/
-					$row[ $key ] = '';
+					$row[$key] = '';
 
-					if ( in_array( $key, $meta ) ) {
+					if (in_array($key, $meta)) {
 
-						switch ( $key ) {
+						switch ($key) {
 
 							case '_thumbnail_id' :
 
-								$image_id    = get_post_thumbnail_id( $form->ID );
-								$row[ $key ] = wp_get_attachment_url( $image_id );
+								$image_id    = get_post_thumbnail_id($form->ID);
+								$row[$key] = wp_get_attachment_url($image_id);
 
 								break;
 
 							case 'give_price' :
 
-								if ( give_has_variable_prices( $form->ID ) ) {
+								if (give_has_variable_prices($form->ID)) {
 
 									$prices = [];
-									foreach ( give_get_variable_prices( $form->ID ) as $price ) {
-										$prices[] = $price['name'] . ': ' . $price['amount'];
+									foreach (give_get_variable_prices($form->ID) as $price) {
+										$prices[] = $price['name'].': '.$price['amount'];
 									}
 
-									$row[ $key ] = implode( ' | ', $prices );
+									$row[$key] = implode(' | ', $prices);
 
 								} else {
 
-									$row[ $key ] = give_get_form_price( $form->ID );
+									$row[$key] = give_get_form_price($form->ID);
 
 								}
 
@@ -128,44 +128,44 @@ 
 
 							default :
 
-								$row[ $key ] = give_get_meta( $form->ID, $key, TRUE );
+								$row[$key] = give_get_meta($form->ID, $key, TRUE);
 
 								break;
 
 						}
 
-					} elseif ( isset( $form->$key ) ) {
+					} elseif (isset($form->$key)) {
 
-						switch ( $key ) {
+						switch ($key) {
 
 							case 'post_author' :
 
-								$row[ $key ] = get_the_author_meta( 'user_login', $form->post_author );
+								$row[$key] = get_the_author_meta('user_login', $form->post_author);
 
 								break;
 
 							default :
 
-								$row[ $key ] = $form->$key;
+								$row[$key] = $form->$key;
 
 								break;
 						}
 
-					} elseif ( 'tags' == $key ) {
+					} elseif ('tags' == $key) {
 
-						$terms = get_the_terms( $form->ID, 'form_tag' );
-						if ( $terms ) {
-							$terms       = wp_list_pluck( $terms, 'name' );
-							$row[ $key ] = implode( ' | ', $terms );
+						$terms = get_the_terms($form->ID, 'form_tag');
+						if ($terms) {
+							$terms       = wp_list_pluck($terms, 'name');
+							$row[$key] = implode(' | ', $terms);
 						}
 
 
-					} elseif ( 'categories' == $key ) {
+					} elseif ('categories' == $key) {
 
-						$terms = get_the_terms( $form->ID, 'form_category' );
-						if ( $terms ) {
-							$terms       = wp_list_pluck( $terms, 'name' );
-							$row[ $key ] = implode( ' | ', $terms );
+						$terms = get_the_terms($form->ID, 'form_category');
+						if ($terms) {
+							$terms       = wp_list_pluck($terms, 'name');
+							$row[$key] = implode(' | ', $terms);
 						}
 
 					}
@@ -176,8 +176,8 @@ 
 
 			}
 
-			$data = apply_filters( 'give_export_get_data', $data );
-			$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+			$data = apply_filters('give_export_get_data', $data);
+			$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 			return $data;
 
@@ -197,20 +197,20 @@ 
 
 		$args = array(
 			'post_type'      => 'give_forms',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'any',
 			'fields'         => 'ids',
 		);
 
-		$forms  = new WP_Query( $args );
+		$forms = new WP_Query($args);
 		$total      = (int) $forms->post_count;
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 

includes/admin/tools/views/html-admin-page-exports.php

Spacing +46 added lines, -46 removed lines

@@ -3,7 +3,7 @@ 
  * Admin View: Exports
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 } ?>
 
@@ -18,14 +18,14 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_top' );
+				do_action('give_tools_tab_export_content_top');
 				?>
 
 				<table class="widefat export-options-table give-table">
 					<thead>
 					<tr>
-						<th scope="col"><?php esc_html_e( 'Export Type', 'give' ); ?></th>
-						<th scope="col"><?php esc_html_e( 'Export Options', 'give' ); ?></th>
+						<th scope="col"><?php esc_html_e('Export Type', 'give'); ?></th>
+						<th scope="col"><?php esc_html_e('Export Options', 'give'); ?></th>
 					</tr>
 					</thead>
 					<tbody>
@@ -38,43 +38,43 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_top' );
+					do_action('give_tools_tab_export_table_top');
 					?>
 					<tr class="give-export-pdf-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export PDF of Donations and Income', 'give' ); ?></span>
+								<span><?php esc_html_e('Export PDF of Donations and Income', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a PDF of Donations and Income reports for all forms for the current year.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a PDF of Donations and Income reports for all forms for the current year.', 'give'); ?></p>
 						</td>
 						<td>
 							<a class="button"
-							   href="<?php echo wp_nonce_url( add_query_arg( [
+							   href="<?php echo wp_nonce_url(add_query_arg([
 								   'give-action' => 'generate_pdf',
-							   ] ), 'give_generate_pdf' ); ?>"><?php esc_html_e( 'Generate PDF', 'give' ); ?></a>
+							   ]), 'give_generate_pdf'); ?>"><?php esc_html_e('Generate PDF', 'give'); ?></a>
 						</td>
 					</tr>
 					<tr class="alternate give-export-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Income and Donation Stats', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Income and Donation Stats', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a CSV of income and donations over time.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a CSV of income and donations over time.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post">
 								<?php
 								printf(
 									/* translators: 1: start date dropdown 2: end date dropdown */
-									esc_html__( '%1$s to %2$s', 'give' ),
-									Give()->html->year_dropdown( 'start_year' ) . ' ' . Give()->html->month_dropdown( 'start_month' ),
-									Give()->html->year_dropdown( 'end_year' ) . ' ' . Give()->html->month_dropdown( 'end_month' )
+									esc_html__('%1$s to %2$s', 'give'),
+									Give()->html->year_dropdown('start_year').' '.Give()->html->month_dropdown('start_month'),
+									Give()->html->year_dropdown('end_year').' '.Give()->html->month_dropdown('end_month')
 								);
 								?>
 								<input type="hidden" name="give-action"
 								       value="earnings_export"/>
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 							</form>
 						</td>
@@ -82,9 +82,9 @@ 
 					<tr class="give-export-payment-history">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Donation History', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Donation History', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a CSV of all donations recorded.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a CSV of all donations recorded.', 'give'); ?></p>
 						</td>
 						<td>
 							<form id="give-export-payments"
@@ -93,27 +93,27 @@ 
 								$args = [
 									'id'          => 'give-payment-export-start',
 									'name'        => 'start',
-									'placeholder' => esc_attr__( 'Start date', 'give' ),
+									'placeholder' => esc_attr__('Start date', 'give'),
 								];
-								echo Give()->html->date_field( $args ); ?>
+								echo Give()->html->date_field($args); ?>
 								<?php
 								$args = [
 									'id'          => 'give-payment-export-end',
 									'name'        => 'end',
-									'placeholder' => esc_attr__( 'End date', 'give' ),
+									'placeholder' => esc_attr__('End date', 'give'),
 								];
-								echo Give()->html->date_field( $args ); ?>
+								echo Give()->html->date_field($args); ?>
 								<select name="status">
-									<option value="any"><?php esc_html_e( 'All Statuses', 'give' ); ?></option>
+									<option value="any"><?php esc_html_e('All Statuses', 'give'); ?></option>
 									<?php
 									$statuses = give_get_payment_statuses();
-									foreach ( $statuses as $status => $label ) {
-										echo '<option value="' . $status . '">' . $label . '</option>';
+									foreach ($statuses as $status => $label) {
+										echo '<option value="'.$status.'">'.$label.'</option>';
 									}
 									?>
 								</select>
 								<?php
-								if ( give_is_setting_enabled( give_get_option( 'categories' ) ) ) {
+								if (give_is_setting_enabled(give_get_option('categories'))) {
 									echo Give()->html->category_dropdown(
 										'give_forms_categories[]',
 										0,
@@ -123,12 +123,12 @@ 
 											'multiple'        => true,
 											'selected'        => [],
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from categories', 'give' ),
+											'placeholder'     => __('Choose one or more from categories', 'give'),
 										]
 									);
 								}
 
-								if ( give_is_setting_enabled( give_get_option( 'tags' ) ) ) {
+								if (give_is_setting_enabled(give_get_option('tags'))) {
 									echo Give()->html->tags_dropdown(
 										'give_forms_tags[]',
 										0,
@@ -138,18 +138,18 @@ 
 											'multiple'        => true,
 											'selected'        => [],
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from tags', 'give' ),
+											'placeholder'     => __('Choose one or more from tags', 'give'),
 										]
 									);
 								}
 
-								wp_nonce_field( 'give_ajax_export', 'give_ajax_export' );
+								wp_nonce_field('give_ajax_export', 'give_ajax_export');
 								?>
 								<input type="hidden" name="give-export-class"
 								       value="Give_Batch_Payments_Export"/>
 								<span>
 									<input type="submit"
-									       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+									       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 									       class="button-secondary"/>
 									<span class="spinner"></span>
 								</span>
@@ -159,9 +159,9 @@ 
 					<tr class="alternate give-export-donors">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Donors in CSV', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Donors in CSV', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download an export of donors for all donation forms or only those who have given to a particular form.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download an export of donors for all donation forms or only those who have given to a particular form.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post" id="give_donor_export"
@@ -173,73 +173,73 @@ 
 									'id'     => 'give_donor_export_form',
 									'chosen' => true,
 								];
-								echo Give()->html->forms_dropdown( $args ); ?>
+								echo Give()->html->forms_dropdown($args); ?>
 
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 
 								<div id="export-donor-options-wrap"
 								     class="give-clearfix">
-									<p><?php esc_html_e( 'Export Columns:', 'give' ); ?></p>
+									<p><?php esc_html_e('Export Columns:', 'give'); ?></p>
 									<ul id="give-export-option-ul">
 										<li>
 											<label for="give-export-fullname">
 												<input type="checkbox" checked
 												       name="give_export_option[full_name]"
-												       id="give-export-fullname"><?php esc_html_e( 'Name', 'give' ); ?>
+												       id="give-export-fullname"><?php esc_html_e('Name', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-email">
 												<input type="checkbox" checked
 												       name="give_export_option[email]"
-												       id="give-export-email"><?php esc_html_e( 'Email', 'give' ); ?>
+												       id="give-export-email"><?php esc_html_e('Email', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-address">
 												<input type="checkbox" checked
 												       name="give_export_option[address]"
-												       id="give-export-address"><?php esc_html_e( 'Address', 'give' ); ?>
+												       id="give-export-address"><?php esc_html_e('Address', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-userid">
 												<input type="checkbox" checked
 												       name="give_export_option[userid]"
-												       id="give-export-userid"><?php esc_html_e( 'User ID', 'give' ); ?>
+												       id="give-export-userid"><?php esc_html_e('User ID', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-first-donation-date">
 												<input type="checkbox" checked
 												       name="give_export_option[date_first_donated]"
-												       id="give-export-first-donation-date"><?php esc_html_e( 'First Donation Date', 'give' ); ?>
+												       id="give-export-first-donation-date"><?php esc_html_e('First Donation Date', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-number">
 												<input type="checkbox" checked
 												       name="give_export_option[donations]"
-												       id="give-export-donation-number"><?php esc_html_e( 'Number of Donations', 'give' ); ?>
+												       id="give-export-donation-number"><?php esc_html_e('Number of Donations', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-sum">
 												<input type="checkbox" checked
 												       name="give_export_option[donation_sum]"
-												       id="give-export-donation-sum"><?php esc_html_e( 'Total Donated', 'give' ); ?>
+												       id="give-export-donation-sum"><?php esc_html_e('Total Donated', 'give'); ?>
 											</label>
 										</li>
 									</ul>
 								</div>
-								<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+								<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 								<input type="hidden" name="give-export-class"
 								       value="Give_Batch_Customers_Export"/>
 								<input type="hidden"
 								       name="give_export_option[query_id]"
-								       value="<?php echo uniqid( 'give_' ); ?>"/>
+								       value="<?php echo uniqid('give_'); ?>"/>
 							</form>
 						</td>
 					</tr>
@@ -252,7 +252,7 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_bottom' );
+					do_action('give_tools_tab_export_table_bottom');
 					?>
 					</tbody>
 				</table>
@@ -263,7 +263,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_bottom' );
+				do_action('give_tools_tab_export_content_bottom');
 				?>
 
 			</div>

includes/admin/customers/customers.php

Spacing +164 added lines, -164 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
  */
 function give_customers_page() {
 	$default_views  = give_customer_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'customers';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_customer_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'customers';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_customer_view($requested_view, $default_views);
 	} else {
 		give_customers_list();
 	}
@@ -42,7 +42,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_customer_views', $views );
+	return apply_filters('give_customer_views', $views);
 
 }
 
@@ -56,7 +56,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_customer_tabs', $tabs );
+	return apply_filters('give_customer_tabs', $tabs);
 
 }
 
@@ -67,7 +67,7 @@ 
  * @return void
  */
 function give_customers_list() {
-	include( dirname( __FILE__ ) . '/class-customer-table.php' );
+	include(dirname(__FILE__).'/class-customer-table.php');
 
 	$customers_table = new Give_Customer_Reports_Table();
 	$customers_table->prepare_items();
@@ -80,11 +80,11 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
-		<form id="give-donors-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
+		<form id="give-donors-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
 			<?php
-			$customers_table->search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors' );
+			$customers_table->search_box(esc_html__('Search Donors', 'give'), 'give-donors');
 			$customers_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms" />
@@ -97,7 +97,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -113,27 +113,27 @@ 
  *
  * @return void
  */
-function give_render_customer_view( $view, $callbacks ) {
+function give_render_customer_view($view, $callbacks) {
 
 	$render = true;
 
-	$customer_view_role = apply_filters( 'give_view_customers_role', 'view_give_reports' );
+	$customer_view_role = apply_filters('give_view_customers_role', 'view_give_reports');
 
-	if ( ! current_user_can( $customer_view_role ) ) {
-		give_set_error( 'give-no-access', esc_html__( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($customer_view_role)) {
+		give_set_error('give-no-access', esc_html__('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_customer', esc_html__( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_customer', esc_html__('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$customer_id = (int) $_GET['id'];
-	$customer    = new Give_Customer( $customer_id );
+	$customer    = new Give_Customer($customer_id);
 
-	if ( empty( $customer->id ) ) {
-		give_set_error( 'give-invalid_customer', esc_html__( 'Invalid Donor ID.', 'give' ) );
+	if (empty($customer->id)) {
+		give_set_error('give-invalid_customer', esc_html__('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -142,34 +142,34 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php give_print_errors( 0 ); ?>
+				<?php give_print_errors(0); ?>
 			</div>
 		<?php endif; ?>
 
-		<h1 class="screen-reader-text"><?php esc_html_e( 'Donor', 'give' ); ?></h1>
+		<h1 class="screen-reader-text"><?php esc_html_e('Donor', 'give'); ?></h1>
 
-		<?php if ( $customer && $render ) : ?>
+		<?php if ($customer && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 			<?php
-			foreach ( $customer_tabs as $key => $tab ) :
+			foreach ($customer_tabs as $key => $tab) :
 				$active = $key === $view ? true : false;
 				$class = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 				printf(
-					'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-					esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $customer->id ) ),
-					esc_attr( $class ),
-					sanitize_html_class( $tab['dashicon'] ),
-					esc_html( $tab['title'] )
+					'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+					esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$customer->id)),
+					esc_attr($class),
+					sanitize_html_class($tab['dashicon']),
+					esc_html($tab['title'])
 				);
 			endforeach;
 			?>
 			</h2>
 
 			<div id="give-customer-card-wrapper">
-				<?php $callbacks[ $view ]( $customer ) ?>
+				<?php $callbacks[$view]($customer) ?>
 			</div>
 
 		<?php endif; ?>
@@ -189,9 +189,9 @@ 
  *
  * @return void
  */
-function give_customers_view( $customer ) {
+function give_customers_view($customer) {
 
-	$customer_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -200,32 +200,32 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $customer );
+	do_action('give_donor_card_top', $customer);
 	?>
 
 	<div id="donor-summary" class="info-wrapper customer-section postbox">
 
-		<form id="edit-customer-info" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $customer->id ); ?>">
+		<form id="edit-customer-info" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->id); ?>">
 
 			<div class="customer-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="customer-avatar">
-						<?php echo get_avatar( $customer->email ); ?>
+						<?php echo get_avatar($customer->email); ?>
 					</div>
 
 					<div id="customer-name-wrap" class="left">
 						<span class="customer-id">#<?php echo $customer->id; ?></span>
-						<span class="customer-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr( $customer->name ); ?>" placeholder="<?php esc_attr_e( 'Donor Name', 'give' ); ?>" /></span>
+						<span class="customer-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr($customer->name); ?>" placeholder="<?php esc_attr_e('Donor Name', 'give'); ?>" /></span>
 						<span class="customer-name info-item editable"><span data-key="name"><?php echo $customer->name; ?></span></span>
 					</div>
 					<p class="customer-since info-item">
-						<?php esc_html_e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $customer->date_created ) ) ?>
+						<?php esc_html_e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($customer->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $customer_edit_role ) ) : ?>
-						<a href="#" id="edit-customer" class="button info-item editable customer-edit-link"><?php esc_html_e( 'Edit Donor', 'give' ); ?></a>
+					<?php if (current_user_can($customer_edit_role)) : ?>
+						<a href="#" id="edit-customer" class="button info-item editable customer-edit-link"><?php esc_html_e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -235,7 +235,7 @@ 
 					<table class="widefat">
 						<tbody>
 						<tr class="alternate">
-							<th scope="col"><label for="tablecell"><?php esc_html_e( 'User:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php esc_html_e('User:', 'give'); ?></label></th>
 							<td>
 								<span class="customer-user-id info-item edit-item">
 									<?php
@@ -252,37 +252,37 @@ 
 										'data'  => $data_atts,
 									);
 
-									if ( ! empty( $user_id ) ) {
-										$userdata           = get_userdata( $user_id );
+									if ( ! empty($user_id)) {
+										$userdata           = get_userdata($user_id);
 										$user_args['value'] = $userdata->user_login;
 									}
 
-									echo Give()->html->ajax_user_search( $user_args );
+									echo Give()->html->ajax_user_search($user_args);
 									?>
 								</span>
 
 								<span class="customer-user-id info-item editable">
-									<?php if ( !empty( $userdata ) ) { ?>
-										<span data-key="user_id">#<?php echo $customer->user_id . ' - ' . $userdata->display_name; ?></span>
+									<?php if ( ! empty($userdata)) { ?>
+										<span data-key="user_id">#<?php echo $customer->user_id.' - '.$userdata->display_name; ?></span>
 									<?php } else { ?>
-										<span data-key="user_id"><?php esc_html_e( 'None', 'give' ); ?></span>
+										<span data-key="user_id"><?php esc_html_e('None', 'give'); ?></span>
 									<?php } ?>
-									<?php if ( current_user_can( $customer_edit_role ) && intval( $customer->user_id ) > 0 ) { ?>
-										<span class="disconnect-user"> - <a id="disconnect-customer" href="#disconnect" aria-label="<?php esc_attr_e( 'Disconnects the current user ID from this customer record.', 'give' ); ?>"><?php esc_html_e( 'Disconnect User', 'give' ); ?></a></span>
+									<?php if (current_user_can($customer_edit_role) && intval($customer->user_id) > 0) { ?>
+										<span class="disconnect-user"> - <a id="disconnect-customer" href="#disconnect" aria-label="<?php esc_attr_e('Disconnects the current user ID from this customer record.', 'give'); ?>"><?php esc_html_e('Disconnect User', 'give'); ?></a></span>
 									<?php } ?>
 								</span>
 							</td>
 						</tr>
-						<?php if ( isset( $customer->user_id ) && $customer->user_id > 0 ) : ?>
+						<?php if (isset($customer->user_id) && $customer->user_id > 0) : ?>
 
 							<tr>
-								<th scope="col"><?php esc_html_e( 'Address:', 'give' ); ?></th>
+								<th scope="col"><?php esc_html_e('Address:', 'give'); ?></th>
 								<td class="row-title">
 
 									<div class="customer-address-wrapper">
 
 										<?php
-										$address  = get_user_meta( $customer->user_id, '_give_user_address', true );
+										$address  = get_user_meta($customer->user_id, '_give_user_address', true);
 										$defaults = array(
 											'line1'   => '',
 											'line2'   => '',
@@ -292,10 +292,10 @@ 
 											'zip'     => '',
 										);
 
-										$address = wp_parse_args( $address, $defaults );
+										$address = wp_parse_args($address, $defaults);
 										?>
 
-										<?php if ( ! empty( $address ) ) { ?>
+										<?php if ( ! empty($address)) { ?>
 											<span class="customer-address info-item editable">
 												<span class="info-item" data-key="line1"><?php echo $address['line1']; ?></span>
 												<span class="info-item" data-key="line2"><?php echo $address['line2']; ?></span>
@@ -306,43 +306,43 @@ 
 											</span>
 										<?php } ?>
 										<span class="customer-address info-item edit-item">
-											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php esc_attr_e( 'Address 1', 'give' ); ?>" value="<?php echo $address['line1']; ?>" />
-											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php esc_attr_e( 'Address 2', 'give' ); ?>" value="<?php echo $address['line2']; ?>" />
-											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php esc_attr_e( 'City', 'give' ); ?>" value="<?php echo $address['city']; ?>" />
+											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php esc_attr_e('Address 1', 'give'); ?>" value="<?php echo $address['line1']; ?>" />
+											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php esc_attr_e('Address 2', 'give'); ?>" value="<?php echo $address['line2']; ?>" />
+											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php esc_attr_e('City', 'give'); ?>" value="<?php echo $address['city']; ?>" />
 											<select data-key="country" name="customerinfo[country]" id="billing_country" class="billing_country give-select edit-item">
 												<?php
 
 												$selected_country = $address['country'];
 
 												$countries = give_get_country_list();
-												foreach ( $countries as $country_code => $country ) {
-													echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+												foreach ($countries as $country_code => $country) {
+													echo '<option value="'.esc_attr($country_code).'"'.selected($country_code, $selected_country, false).'>'.$country.'</option>';
 												}
 												?>
 											</select>
 											<?php
 											$selected_state = give_get_state();
-											$states         = give_get_states( $selected_country );
+											$states         = give_get_states($selected_country);
 
-											$selected_state = isset( $address['state'] ) ? $address['state'] : $selected_state;
+											$selected_state = isset($address['state']) ? $address['state'] : $selected_state;
 
-											if ( ! empty( $states ) ) {
+											if ( ! empty($states)) {
 												?>
 												<select data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-select info-item">
 													<?php
-													foreach ( $states as $state_code => $state ) {
-														echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+													foreach ($states as $state_code => $state) {
+														echo '<option value="'.$state_code.'"'.selected($state_code, $selected_state, false).'>'.$state.'</option>';
 													}
 													?>
 												</select>
 												<?php
 											} else {
 												?>
-												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item" placeholder="<?php esc_attr_e( 'State / Province', 'give' ); ?>" />
+												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item" placeholder="<?php esc_attr_e('State / Province', 'give'); ?>" />
 												<?php
 											}
 											?>
-											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php esc_attr_e( 'Zip / Postal Code', 'give' ); ?>" value="<?php echo $address['zip']; ?>" />
+											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php esc_attr_e('Zip / Postal Code', 'give'); ?>" value="<?php echo $address['zip']; ?>" />
 										</span>
 
 									</div>
@@ -358,10 +358,10 @@ 
 
 			<span id="customer-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $customer->id; ?>" />
-				<?php wp_nonce_field( 'edit-customer', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-customer', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-customer" />
-				<input type="submit" id="give-edit-customer-save" class="button-secondary" value="<?php esc_attr_e( 'Update Donor', 'give' ); ?>" />
-				<a id="give-edit-customer-cancel" href="" class="delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+				<input type="submit" id="give-edit-customer-save" class="button-secondary" value="<?php esc_attr_e('Update Donor', 'give'); ?>" />
+				<a id="give-edit-customer-cancel" href="" class="delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -376,24 +376,24 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $customer );
+	do_action('give_donor_before_stats', $customer);
 	?>
 
 	<div id="customer-stats-wrapper" class="customer-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $customer->email ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($customer->email)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $customer->purchase_count, 'give' ), $customer->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $customer );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $customer->purchase_count, 'give'), $customer->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $customer);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $customer->purchase_value ) ); ?> <?php esc_html_e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($customer->purchase_value)); ?> <?php esc_html_e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -405,7 +405,7 @@ 
 			 *
 			 * @param object $customer The customer object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $customer );
+			do_action('give_donor_stats_list', $customer);
 			?>
 		</ul>
 	</div>
@@ -418,7 +418,7 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $customer );
+	do_action('give_donor_before_tables_wrapper', $customer);
 	?>
 
 	<div id="customer-tables-wrapper" class="customer-section">
@@ -431,46 +431,46 @@ 
 		 *
 		 * @param object $customer The customer object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $customer );
+		do_action('give_donor_before_tables', $customer);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 				<tr>
-					<th><?php _e( 'Email', 'give' ); ?></th>
-					<th><?php _e( 'Actions', 'give' ); ?></th>
+					<th><?php _e('Email', 'give'); ?></th>
+					<th><?php _e('Actions', 'give'); ?></th>
 				</tr>
 			</thead>
 
 			<tbody>
-				<?php if ( ! empty( $customer->emails ) ) { ?>
+				<?php if ( ! empty($customer->emails)) { ?>
 
-					<?php foreach ( $customer->emails as $key => $email ) : ?>
+					<?php foreach ($customer->emails as $key => $email) : ?>
 						<tr data-key="<?php echo $key; ?>">
 							<td>
 								<?php echo $email; ?>
-								<?php if ( 'primary' === $key ) : ?>
+								<?php if ('primary' === $key) : ?>
 									<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 								<?php endif; ?>
 							</td>
 							<td>
-								<?php if ( 'primary' !== $key ) : ?>
+								<?php if ('primary' !== $key) : ?>
 									<?php
-									$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $customer->id );
-									$promote_url = wp_nonce_url( add_query_arg( array(
-										'email' => rawurlencode( $email ),
+									$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->id);
+									$promote_url = wp_nonce_url(add_query_arg(array(
+										'email' => rawurlencode($email),
 										'give_action' => 'set_donor_primary_email',
-									), $base_url ), 'give-set-donor-primary-email' );
-									$remove_url  = wp_nonce_url( add_query_arg( array(
-										'email' => rawurlencode( $email ),
+									), $base_url), 'give-set-donor-primary-email');
+									$remove_url = wp_nonce_url(add_query_arg(array(
+										'email' => rawurlencode($email),
 										'give_action' => 'remove_donor_email',
-									), $base_url ), 'give-remove-donor-email' );
+									), $base_url), 'give-remove-donor-email');
 									?>
-									<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+									<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 									&nbsp;|&nbsp;
-									<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+									<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 								<?php endif; ?>
 							</td>
 						</tr>
@@ -480,59 +480,59 @@ 
 						<td colspan="2" class="add-customer-email-td">
 							<div class="add-customer-email-wrapper">
 								<input type="hidden" name="customer-id" value="<?php echo $customer->id; ?>" />
-								<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
-								<input type="email" name="additional-email" value="" placeholder="<?php _e( 'Email Address', 'give' ); ?>" />&nbsp;
-								<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
-								<button class="button-secondary give-add-customer-email" id="add-customer-email"><?php _e( 'Add Email', 'give' ); ?></button>
+								<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
+								<input type="email" name="additional-email" value="" placeholder="<?php _e('Email Address', 'give'); ?>" />&nbsp;
+								<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
+								<button class="button-secondary give-add-customer-email" id="add-customer-email"><?php _e('Add Email', 'give'); ?></button>
 								<span class="spinner"></span>
 							</div>
 							<div class="notice-wrap"></div>
 						</td>
 					</tr>
 				<?php } else { ?>
-					<tr><td colspan="2"><?php _e( 'No Emails Found', 'easy-digital-downloads' ); ?></td></tr>
+					<tr><td colspan="2"><?php _e('No Emails Found', 'easy-digital-downloads'); ?></td></tr>
 				<?php }// End if().
 	?>
 			</tbody>
 		</table>
 
-		<h3><?php esc_html_e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $customer->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $customer->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php esc_html_e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php esc_html_e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php esc_html_e('ID', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Amount', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Date', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Status', 'give'); ?></th>
+				<th scope="col"><?php esc_html_e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
 						<td><?php echo $payment->ID; ?></td>
-						<td><?php echo give_payment_amount( $payment->ID ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo give_payment_amount($payment->ID); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id='.$payment->ID),
 								sprintf(
 									/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								esc_html__( 'View Donation', 'give' )
+								esc_html__('View Donation', 'give')
 							);
 							?>
 
@@ -547,47 +547,47 @@ 
 							 * @param object $customer The customer object being displayed.
 							 * @param object $payment  The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $customer, $payment );
+							do_action('give_donor_recent_purchases_actions', $customer, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php esc_html_e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php esc_html_e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 	?>
 			</tbody>
 		</table>
 
-		<h3><?php esc_html_e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $customer->email );
+		$donations = give_get_users_completed_donations($customer->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php esc_html_e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php esc_html_e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php esc_html_e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php esc_html_e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 									/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								esc_html__( 'View Form', 'give' )
+								esc_html__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -595,7 +595,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php esc_html_e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php esc_html_e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -609,7 +609,7 @@ 
 		 *
 		 * @param object $customer The customer object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $customer );
+		do_action('give_donor_after_tables', $customer);
 		?>
 
 	</div>
@@ -622,7 +622,7 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $customer );
+	do_action('give_donor_card_bottom', $customer);
 
 }
 
@@ -635,30 +635,30 @@ 
  *
  * @return void
  */
-function give_customer_notes_view( $customer ) {
+function give_customer_notes_view($customer) {
 
-	$paged          = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged          = absint( $paged );
+	$paged          = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged          = absint($paged);
 	$note_count     = $customer->get_notes_count();
-	$per_page       = apply_filters( 'give_customer_notes_per_page', 20 );
-	$total_pages    = ceil( $note_count / $per_page );
-	$customer_notes = $customer->get_notes( $per_page, $paged );
+	$per_page       = apply_filters('give_customer_notes_per_page', 20);
+	$total_pages    = ceil($note_count / $per_page);
+	$customer_notes = $customer->get_notes($per_page, $paged);
 	?>
 
 	<div id="customer-notes-wrapper">
 		<div class="customer-notes-header">
-			<?php echo get_avatar( $customer->email, 30 ); ?> <span><?php echo $customer->name; ?></span>
+			<?php echo get_avatar($customer->email, 30); ?> <span><?php echo $customer->name; ?></span>
 		</div>
-		<h3><?php esc_html_e( 'Notes', 'give' ); ?></h3>
+		<h3><?php esc_html_e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
-				<form id="give-add-customer-note" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $customer->id ); ?>">
+				<form id="give-add-customer-note" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$customer->id); ?>">
 					<textarea id="customer-note" name="customer_note" class="customer-note-input" rows="10"></textarea>
 					<br />
 					<input type="hidden" id="customer-id" name="customer_id" value="<?php echo $customer->id; ?>" />
 					<input type="hidden" name="give_action" value="add-customer-note" />
-					<?php wp_nonce_field( 'add-customer-note', 'add_customer_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-customer-note', 'add_customer_note_nonce', true, true); ?>
 					<input id="add-customer-note" class="right button-primary" type="submit" value="Add Note" />
 				</form>
 			</div>
@@ -673,26 +673,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-customer-notes" class="postbox">
-			<?php if ( count( $customer_notes ) > 0 ) { ?>
-				<?php foreach ( $customer_notes as $key => $note ) : ?>
+			<?php if (count($customer_notes) > 0) { ?>
+				<?php foreach ($customer_notes as $key => $note) : ?>
 					<div class="customer-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-customer-notes">
-					<?php esc_html_e( 'No donor notes found.', 'give' ); ?>
+					<?php esc_html_e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -708,9 +708,9 @@ 
  *
  * @return void
  */
-function give_customers_delete_view( $customer ) {
+function give_customers_delete_view($customer) {
 
-	$customer_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -719,15 +719,15 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_customer_delete_top', $customer );
+	do_action('give_customer_delete_top', $customer);
 	?>
 
 	<div class="info-wrapper customer-section">
 
-		<form id="delete-customer" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $customer->id ); ?>">
+		<form id="delete-customer" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$customer->id); ?>">
 
 			<div class="customer-notes-header">
-				<?php echo get_avatar( $customer->email, 30 ); ?> <span><?php echo $customer->name; ?></span>
+				<?php echo get_avatar($customer->email, 30); ?> <span><?php echo $customer->name; ?></span>
 			</div>
 
 
@@ -735,20 +735,20 @@ 
 
 				<span class="delete-customer-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-customer-delete-confirm',
-						) ); ?>
-						<label for="give-customer-delete-confirm"><?php esc_html_e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-customer-delete-confirm"><?php esc_html_e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-customer-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-customer-delete-records"><?php esc_html_e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-customer-delete-records"><?php esc_html_e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -761,16 +761,16 @@ 
 					 *
 					 * @param object $customer The customer object being displayed.
 					 */
-					do_action( 'give_customer_delete_inputs', $customer );
+					do_action('give_customer_delete_inputs', $customer);
 					?>
 				</span>
 
 				<span id="customer-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $customer->id; ?>" />
-					<?php wp_nonce_field( 'delete-customer', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-customer', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-customer" />
-					<input type="submit" disabled="disabled" id="give-delete-customer" class="button-primary" value="<?php esc_attr_e( 'Delete Donor', 'give' ); ?>" />
-					<a id="give-delete-customer-cancel" href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $customer->id ); ?>" class="delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+					<input type="submit" disabled="disabled" id="give-delete-customer" class="button-primary" value="<?php esc_attr_e('Delete Donor', 'give'); ?>" />
+					<a id="give-delete-customer-cancel" href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->id); ?>" class="delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -786,5 +786,5 @@ 
 	 *
 	 * @param object $customer The customer object being displayed.
 	 */
-	do_action( 'give_customer_delete_bottom', $customer );
+	do_action('give_customer_delete_bottom', $customer);
 }